error using mmap version. would you know why?

after running for a very prolonged time for 100MB size mmap cache, this happens.
unexpected fault address 0x7f3935164000
fatal error: fault
[signal SIGBUS: bus error code=0x2 addr=0x7f3935164000 pc=0x63d635]

goroutine 1 [running]:
runtime.throw({0xa7cfad?, 0xa7f77d?})
        /usr/local/go/src/runtime/panic.go:1047 +0x5d fp=0xc0000c9018 sp=0xc0000c8fe8 pc=0x55ca7d
runtime.sigpanic()
        /usr/local/go/src/runtime/signal_unix.go:838 +0x125 fp=0xc0000c9078 sp=0xc0000c9018 pc=0x572c65
github.com/viant/scache.(*segment).set(0xc00011e100?, {0xc006321550?, 0xc052c3e180?}, {0xc006321500?, 0x7f3918fc0698?, 0x7f397135a5b8?})
        /root/go/pkg/mod/github.com/viant/scache@v0.5.0/segment.go:103 +0x95 fp=0xc0000c90c8 sp=0xc0000c9078 pc=0x63d635
github.com/viant/scache.(*Cache).Set(0xc000105040, {0xc006321550, 0x9}, {0xc006321500, 0xb, 0x10})
        /root/go/pkg/mod/github.com/viant/scache@v0.5.0/cache.go:53 +0x9b fp=0xc0000c9158 sp=0xc0000c90c8 pc=0x63c39b

just realised u need to set max entries and entry sizes to prevent from OOM.


May I ask a question about the key generation protocol?

4.1 Key generation protocol in GG18spec is as below.
  values Xi = g ^ xi are public.
But is the code only modulo?
round.save.Xi = new (big.Int) .Mod (xi, tss.EC (). Params (). N)
https://github.com/binance-chain/tss-lib/blob/1e5e2dd5ed1da5f0ad67665f0fedd08e555b00ff/ecdsa/keygen/round_3.go#L44

round.save.Xi is not public. What you're referring to (X_i = g ^ x_i) are known as BigXi in the code.


Feature/dao 195 polygon urls

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Specifying channel = "stable" in rust-toolchain.toml doesn't ensure the latest stable is used on BuildJet?

I got this error on aarch64-unknown-linux-gnu runner on BuildJet.
error: package `clap_derive v4.5.0` cannot be built because it requires rustc 1.74 or newer, while the currently active rustc version is 1.72.0

I placed the following rust-toolchain.toml, but it doesn't work probably because the rustup recognizes the version 1.72.0 is the "stable."
[toolchain]
channel = "stable"

I know changing this to nightly or some specific version like 1.76 should work, but are there any handy way to use the latest stable toolchain?

Yes unfortunately this is if I recall correctly a flaw/limitation with rustup/rust-toolchain.toml. There's no way to say "latest stable" -- the "stable" identifier just means "some stable, literally any will do". I wish I had a Sick Trick for this but it's just not something they support.

Thanks for the prompt response! I see. Then, how about adding this step to the GitHub Actions workflow? This should no harm because this doesn't do anything if the latest stable toolchain is installed, which should be the case with GitHub Actions runners. The absence of toolchain specification probably means stable is fine to use.
      - uses: dtolnay/rust-toolchain@stable
        if: ${{ hashFiles('rust-toolchain.toml') == '' }}

That said, I guess using custom runner is rare, so I'm not sure if this is really worth.


Skeleton2D SkeletonModification2DCCIDK stops working when reloading scene

Godot version
v4.0.rc1.official
System information
Windows 10, GeForce RTX 3070 (Driver version 528.49), forward_plus rendering method
Issue description
When adding a SkeletonModification2DCCDIK modification to the Modification Stack, the inverse kinematics work as expected, however, when reloading the current scene (with get_tree().reload_current_scene()), the inverse kinematics of the CCIDK modification stops working and the skeleton only displays its rest pose.
This video shows the behavior, at first its working, then I reload the scene and it stops working:
https://user-images.githubusercontent.com/105640257/218328590-cb4d3694-b7f5-42c2-9a14-8bb714a7945f.mp4
When reloading the scene, I also get the error message "E 0:00:09:0835   _execute: Target node is not in the scene tree. Cannot execute modification!", so that might be the reason but I am not sure why the target node is not in the scene tree?
Steps to reproduce

Create a scene that has a Skeleton2D, with an enabled Modification Stack
Add some Polygon2Ds and set the Skeleton property
Add a Bone2D for each bone
Set rest pose of the Skeleton accordingly and assign weights to the bones of the Polygon2Ds
Add a SkeletonModification2DCCDIK to the Modification Stack, set the joint data accordingly.
Start the scene, everything should work
Call get_tree().reload_current_scene(), the IK has stopped working

Minimal reproduction project
I created a small reproduction project, the mouse cursor moves the target position for the IK, pressing space bar (ui_accept) reloads the current scene, resulting in the SkeletonModification2DCCDIK not working anymore.
Skeleton2DReloadNotWorking.zip
I am also not sure if this is related, or has something to do with the CCDIK being experimental, or maybe I set something up incorrectly, but when starting the scene (for the first time) I get the following errors/warnings:

If anyone else encounters this problem, the workaround I am using in the meantime calls .queue_free() on the Skeleton2D, then waits 2 physics frames await(get_tree().physics_frame) before reloading the scene. Using this workaround the IK works after reloading.

Godot version: 4.0.1
I am using SkeletonBone2DTwoBoneIK modifier with the Skeleton2D bone rig, the bones which are modified turn yellow which ensures the influence of that modifier. However, when I reload the scene the bones turn white and the influence is disabled (in editor) while the property inspector remains unchanged.
I am fixing it by deleting a modifier and undo it to restore the influence

Godot version: 4.1.3
Seeing the same problem. What I use to work around it is:
func _ready():
	var mod_stack = $Skeleton2D.get_modification_stack()
	var skl_mod = mod_stack.get_modification(0)
	skl_mod.target_nodepath = skl_mod.target_nodepath
	skl_mod.tip_nodepath = skl_mod.tip_nodepath #for SkeletonModification2DCCDIK

Seems that refreshing the nodepath helps clear whatever problem it's having.

This is what I do for all scenes the use the skeleton modification stack and it fixes this issue.
First, save your modification stack resource. To do that, click on your Skeleton Node, then in the inspector find 'Modification Stack', then click the down arrow -> Save As
Then use the code below to reload a fresh one to your skeleton when the scene starts. The fixes issues when switching to this scene after it has been loaded at least once (like when reloading a scene, or leaving it and coming back)
@onready var modification_stack = preload("res://Player/player_skeleton_modification_stack.tres")
func _ready():
	%Skeleton.set_modification_stack(modification_stack.duplicate())
	%Skeleton.get_modification_stack().enabled = true

I've only had this problem in the editor (I'm using the stack only to help record the animation, turn it off while running the game), and the solution was to save the stack as a resource, reset the stack and then load it in again. Reloading the project didn't help. (I am running 4.1.3 stable)


Modernize payload converter sample

What was changed

In the custom converter sample, use the more modern, non-initialized style of fetching the default payload converter.
Why?

Acting on @cretz's TODO.
Checklist

Closes

How was this tested:

poetry run pytest ./tests/custom_converter/workflow_test.py

Any docs updates needed?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix deprecation warnings with ActiveSupport ~> 2.3.5

I've gotten tired of this message in my console. :)
DEPRECATION WARNING: require "activesupport" is deprecated and will be removed in Rails 3. Use require "active_support" instead. (called from /vendor/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/activesupport.rb:2)

I chose this approach instead of a nested begin/rescue block so that people using versions before 2.3.5 didn't have to suffer the overhead of two exception backtraces being generated. I tested with the first and last patch versions of each minor version of ActiveSupport between 2.0.0 and 3.0.5

+1 on this one.


Add autofrom Option to Email Notify Plugin

Motivation for changes:
want to have the ability have the email from field automatically generated with the system hostname and username compatible with linux/bsd mta
Detailed changes:

additional import of "os" to retrieve user name
add bool "autofrom" to schema
conditional to decide whether to use auto generate from address

Config usage if relevant (new plugin or updated schema):
    notify:
      task:
        title: Example
        via:
          - email:
              to<EMAIL_ADDRESS>              smtp_host: smtpout.server.net
              autofrom: yes

Have you tested this on windows by any chance?

yeah I did test on windows and linux

Cool. Care to update the wiki?

wiki updated


Multiline text and attachement not showing in chat

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

Action Performed:
select any chat and go to detail screen
write two or more line and send or attach any image and send.
Expected Result:
App should show  multi line text and attached images.
Actual Result:
it showing blank screen
Workaround:
single line chat will work, but for multiline chant and attachment fix is required.
Platform:
Checked in web an iOS , not checked in other platform.

[- ] Web
[ -] iOS
[x] Android
[x] Desktop App
[x] Mobile Web

Version Number:
Notes/Photos/Videos: Any additional supporting documentation
View all open jobs on Upwork

Triggered auto assignment to @ntrepanier (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Triggered auto assignment to @Beamanator (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Hey @kp17211 would you please confirm a few things for me:

select any chat and go to detail screen <-- what is the "detail screen"? Is this something other than the page where you view all of your chat history with that person?
Can you please let me know which version of the app & web platform you are using? You can find the version in settings -> about, right under the Expensify logo (I'm on version 1.0.81-1)
Is this the flow you're following (see video)? I cannot reproduce the error you are mentioning here

https://user-images.githubusercontent.com/3885503/127453195-72a4d3e3-807e-4881-a882-b9a0b4586341.mov

Hi @Beamanator
Please find details below

select any chat and go to detail screen <-- what is the "detail screen"? Is this something other than the page where you view all of your chat history with that person?

yes same screen

Can you please let me know which version of the app & web software (chrome, firefox, etc) you are using? You can find the version in settings -> about, right under the Expensify logo (I'm on version 1.0.81-1)

I have checked in iOS and Android app not  in  browser

Is this the flow you're following (see video)? I cannot reproduce the error you are mentioning here

yes, please follow same steps in iPhone and Android phone and verify result.

https://user-images.githubusercontent.com/16526849/127803169-c276cc24-2168-4bee-8744-5434da890f6b.mp4
https://user-images.githubusercontent.com/16526849/127803187-4033e157-1c77-492b-bd84-b2b6e7ad8d76.mp4

@kp17211 If you are receiving running development build. Please do npm install.

Good recommendation @parasharrajat - maybe also try cd ios && pod install && cd .. to update pods?
I tried on iOS just now and still didn't reproduce the error you mentioned so I'll close for now

@parasharrajat
I have two questions

Is it possible to run app without npm install ? As I shared video of app so I think I already did it :)
Lets assume due to some issue one of the package is not installed properly , but in that case ,  as you can see in second video single line text is working , only tow or more line text is not working. is it possible ?

@Beamanator ,  I think every React Native developer knows this, cd ios && pod install && cd ..,  but lets assume I missed it , but this issue was reproduced in Android also so what to do for that ?
I reported this issue on 23rd Jul so if you have access of that code , check once , I assume someone has fixed after checking this issue.
Thanks

Yes. You are correct. But we recently upgraded a lib which has same exports as earlier but props changed which was skipping its render and app would continue working. So fresh npm install will fix this which no one will know of until they told as app is executing fine.  its not apparent that npm install is needed.


Login throwing the 403

     Access to this page has been denied.               Please verify you are a human      Please click "I am not a robot" to continue     Access to this page has been denied because we believe you are using automation tools to browse the website.   Th is may happen as a result of the following:    Javascript is disabled or blocked by an extension (ad blockers for example)   Your browser does not support cookies    Please make sure that Javascript and cookies are enabled on your browser and that you are not blocking them from loading.   Reference ID: #f76a0790-6b10-11e8-9d14-f73d4ee6b304       Powered by PerimeterX , Inc.

@vijayabr before creating an issue take a look at issue reporting guideline.


fix: empty hostname in cluster slots response

Handles empty hostname returned in the CLUSTER SLOTS command
Sample response from one of the clusters:

In this scenario client creation fails with Error: IO Error: Custom { kind: Uncategorized, error: "failed to lookup address information: Name or service not known" } as it tries to connect to an empty host.

Thanks @katul979, I cherry-picked this into https://github.com/aembke/fred.rs/pull/269


New homepage translations don't add "[Language]" to title

There is an edge case with this

creating translated HomePage will append language to title
e.g., Global Internet of Good Things [English] will become Global Internet of Good Things [English] [KARAKALPAK].

Would it be possible to do this
if original_homepage_title contains regex "[" & original_homepage_language & "]$"
replace regex "[" & original_homepage_language & "]$" with "[" & new_homepage_language & "]"


Philips 2017 updated

Pull Request
This PR is for a

[x] New Publication(s)
[ ] Correction

For the following list(s):

[ ] ancientmetagenome-anthropogenic (README)
[ ] ancientmetagenome-environmental (README, ask @thedir-team-dirt for advice)
[x] ancientmetagenome-hostassociated (README, ask @thedir-team-bugparty for advice)
[ ] ancientsinglegenome-hostassociated (README, ask @thedir-team-pathopeeps for advice)

New Publication
Publication Information
This pull request is to add samples from the following publication(s): https://academic.oup.com/gigascience/article/6/7/gix044/3867068
This is to close #90
It is a new pull request to replace add data from Philips2017 because that was no longer compatible with all the updates that have been made
Checklist

[x] Publication is published (preprints currently not accepted)?
[x] Checked the publication is not already in the database?
[x] Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
[x] Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
[x] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[x] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[x] Review requested from corresponding team (see list of lists above for each team)?
[ ] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

If you do not know how to check errors in failed validation checks, expand here

Correction
This PR is for

[ ] ancientmetagenome-hostassociated
[ ] ancientsinglegenome-hostassociated
[ ] ancientmetagenome-environmental
[ ] ancientmetagenome-anthropogenic

Reference
This pull request is to correct samples from the following publication(s):
This is to close
Description
The issue is:

Checklist

[ ] Checked the corrected entries follow conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)
[ ] Changelog is updated to include the publication under 'Changed'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?

@maxibor I can't figure out what the issue is here, can you help me out? I've tried twice to delete final blank line, but that didn't work, and there's nothing I can find wrong with the archive_project column entries - they're correctly aligned in the right column, they start with PRJ, they're all filled in, and I don't find white space in them

@maxibor I can't figure out what the issue is here, can you help me out? I've tried twice to delete final blank line, but that didn't work, and there's nothing I can find wrong with the archive_project column entries - they're correctly aligned in the right column, they start with PRJ, they're all filled in, and I don't find white space in them

Line 213, Column archive_project, there is a NA that is not allowed by the Schema.
Don't know why this wasn't picked up by the PR of this sample (Zhou2018). Maybe the Schema was modified in the meantime ?
@jfy133 Should we update the Schema to ban the NAs from this column ?

Line 213, Column archive_project, there is a NA that is not allowed by the Schema.
Don't know why this wasn't picked up by the PR of this sample (Zhou2018). Maybe the Schema was modified in the meantime ?
@jfy133 Should we update the Schema to ban the NAs from this column ?


ZFPY osx wheels delocate dylibs

ZFPY osx wheels are not linked with dylibs, we need to figure out how to delocate the dylibs correctly.

I was going to experiment with zfpy on MacOS but it seems installing from source didn't work well either.  Thanks for putting this library together!
(cv) $ pip install zfpy --no-cache-dir --no-binary :all: --force-reinstall
Collecting zfpy
  Downloading zfpy-0.5.5.tar.gz (3.0 kB)
  Preparing metadata (setup.py) ... done
Skipping wheel build for zfpy, due to binaries being disabled for it.
Installing collected packages: zfpy
  Attempting uninstall: zfpy
    Found existing installation: zfpy 0.5.5
    Uninstalling zfpy-0.5.5:
      Successfully uninstalled zfpy-0.5.5
  Running setup.py install for zfpy ... error
  error: subprocess-exited-with-error

  × Running setup.py install for zfpy did not run successfully.
  │ exit code: 1
  ╰─> [12 lines of output]
      running install
      running build
      running build_ext
      building 'zfpy' extension
      creating build
      creating build/temp.macosx-10.9-universal2-3.9
      creating build/temp.macosx-10.9-universal2-3.9/build
      creating build/temp.macosx-10.9-universal2-3.9/build/python
      gcc -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -Iinclude -IDIR/.virtualenvs/cv/lib/python3.9/site-packages/numpy/core/include -IDIR/.virtualenvs/cv/include -I/Library/Frameworks/Python.framework/Versions/3.9/include/python3.9 -c build/python/zfpy.c -o build/temp.macosx-10.9-universal2-3.9/build/python/zfpy.o
      clang: error: no such file or directory: 'build/python/zfpy.c'
      clang: error: no input files
      error: command '/usr/bin/gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  WARNING: No metadata found in DIR/.virtualenvs/cv/lib/python3.9/site-packages
  Rolling back uninstall of zfpy
  Moving to DIR/.virtualenvs/cv/lib/python3.9/site-packages/zfpy-0.5.5.dist-info/
   from DIR/.virtualenvs/cv/lib/python3.9/site-packages/~fpy-0.5.5.dist-info
  Moving to DIR/.virtualenvs/cv/lib/python3.9/site-packages/zfpy.cpython-39-darwin.so
   from /private/var/folders/w5/99_dvybs2zd1g4mrt3lhm1h40000gn/T/pip-uninstall-do5yl3iy/zfpy.cpython-39-darwin.so
error: legacy-install-failure

× Encountered error while trying to install package.
╰─> zfpy

note: This is an issue with the package mentioned above, not pip.
hint: See above for output from the failure.
WARNING: You are using pip version 22.0.2; however, version 22.0.3 is available.
You should consider upgrading via the 'DIR/.virtualenvs/cv/bin/python -m pip install --upgrade pip' command.

I'm not sure what pip install does when installing from source.  If you want to build zfpy with CMake, you have to add -DBUILD_ZFPY=ON.  Maybe there's a way to tell pip to use this build option?

Hi Dr. Lindstrom! Yes, it's possible to pass it using extra_compile_args. I have some examples here from a module I'm working on:
https://github.com/seung-lab/pyspng-seunglab/blob/master/setup.py#L46-L55

Great.  Does pip install still fail when you add -DBUILD_ZFPY=ON?

I took a closer look and it seems that was not sufficient to get it to build. I was able to compress and decompress a numpy array after using python setup.py develop with the following modifications:

run make
modify setup.py as follows:

    ext_modules=[
        Extension("zfpy", ["python/zfpy.pyx"],
            include_dirs=["include", np.get_include()],
            libraries=["zfp"],
            library_dirs=["lib"],
        )
    ]

I tried to see if I could at least reproduce this using an sdist, but the zfp.pxd was not included by default using this setup, so I added a MANIFEST.in file in the same directory as setup.py with the following contents:
include python/zfpy.pxd
include python/zfpy.pyx
recursive-include include *.h
recursive-include src *.c *.h

This makes sure all the necessary c files are included in the package. However, this isn't sufficient because libzfp.a has not been built. In the pyspng-seunglab setup.py file referenced above I call out to make in a subshell for building on macos (for some reason only clang on MacOS didn't want to build for that module). For Linux and MacOS, I think the same strategy would work here. Windows is another beast, but probably a cmake solution would work for all three of them (but I'm not very good at cmake...).
I do have one cmake example I could follow which is located here: https://github.com/seung-lab/DracoPy/blob/master/setup.py
I recognize that you have a more formal build process here, so I'm sure you'd want to tweak the exact build sequence and folders.
I think others may disagree with me, but in my own libraries, I like to pre-compile the cython and build from there which eliminates the cython dependency. This might not be as future proof though I can't articulate specifically in what ways it could fail.

You can try pip install zfpy today, it may be all problems gone now.
Sent from my iPhone

On Feb 24, 2022, at 10:18 AM, William Silversmith @.***> wrote:

I took a closer look and it seems that was not sufficient to get it to build. I was able to compress and decompress a numpy array after using python setup.py develop with the following modifications:
run make
modify setup.py as follows:
ext_modules=[
Extension("zfpy", ["python/zfpy.pyx"],
include_dirs=["include", np.get_include()],
libraries=["zfp"],
library_dirs=["lib"],
)
]
I tried to see if I could at least reproduce this using an sdist, but the zfp.pxd was not included by default using this setup, so I added a MANIFEST.in file in the same directory as setup.py with the following contents:
include python/zfpy.pxd
include python/zfpy.pyx
recursive-include include *.h
recursive-include src *.c *.h
This makes sure all the necessary c files are included in the package. However, this isn't sufficient because libzfp.a has not been built. In the pyspng-seunglab setup.py file referenced above I call out to make in a subshell for building on macos (for some reason only clang on MacOS didn't want to build for that module). For Linux and MacOS, I think the same strategy would work here. Windows is another beast, but probably a cmake solution would work for all three of them (but I'm not very good at cmake...).
I do have one cmake example I could follow which is located here: https://github.com/seung-lab/DracoPy/blob/master/setup.py
I recognize that you have a more formal build process here, so I'm sure you'd want to tweak the exact build sequence and folders.
I think others may disagree with me, but in my own libraries, I like to pre-compile the cython and build from there which eliminates the cython dependency. This might not be as future proof though I can't articulate specifically in what ways it could fail.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Just gave it a try and still getting the same error unfortunately. T_T
(tmp) wms:~$ pip install zfpy --force-reinstall --no-cache-dir
Collecting zfpy
  Downloading zfpy-0.5.5-cp39-cp39-macosx_10_9_x86_64.whl (95 kB)
     |████████████████████████████████| 95 kB 9.3 MB/s
Installing collected packages: zfpy
Successfully installed zfpy-0.5.5
WARNING: You are using pip version 21.3.1; however, version 22.0.3 is available.
You should consider upgrading via the '/Users/wms/.virtualenvs/tmp/bin/python -m pip install --upgrade pip' command.
(tmp) wms:~$ python
Python 3.9.7 (v3.9.7:1016ef3790, Aug 30 2021, 16:25:35)
[Clang 12.0.5 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import zfpy
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: dlopen(/Users/wms/.virtualenvs/tmp/lib/python3.9/site-packages/zfpy.cpython-39-darwin.so, 0x0002): Library not loaded: @rpath/libzfp.0.dylib
  Referenced from: /Users/wms/.virtualenvs/tmp/lib/python3.9/site-packages/zfpy.cpython-39-darwin.so
  Reason: tried: '/usr/lib/libzfp.0.dylib' (no such file)

Can you try zfpy-py310-macos version? I tried this morning, and have no any problem.
Sent from my iPhone

On Feb 24, 2022, at 11:22 AM, William Silversmith @.***> wrote:

Just gave it a try and still getting the same error unfortunately. T_T
(tmp) wms:~$ pip install zfpy --force-reinstall --no-cache-dir
Collecting zfpy
Downloading zfpy-0.5.5-cp39-cp39-macosx_10_9_x86_64.whl (95 kB)
|████████████████████████████████| 95 kB 9.3 MB/s
Installing collected packages: zfpy
Successfully installed zfpy-0.5.5
WARNING: You are using pip version 21.3.1; however, version 22.0.3 is available.
You should consider upgrading via the '/Users/wms/.virtualenvs/tmp/bin/python -m pip install --upgrade pip' command.
(tmp) wms:~$ python
Python 3.9.7 (v3.9.7:1016ef3790, Aug 30 2021, 16:25:35)
[Clang 12.0.5 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

import zfpy
Traceback (most recent call last):
File "", line 1, in
ImportError: dlopen(/Users/wms/.virtualenvs/tmp/lib/python3.9/site-packages/zfpy.cpython-39-darwin.so, 0x0002): Library not loaded: @rpath/libzfp.0.dylib
Referenced from: /Users/wms/.virtualenvs/tmp/lib/python3.9/site-packages/zfpy.cpython-39-darwin.so
Reason: tried: '/usr/lib/libzfp.0.dylib' (no such file)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Oh hey, that worked! I can definitely use this for experiments. Thank you!


Forgot password does not send a code

Having received an invite code, I tried to login but forgot my password. I have tried repeatedly the forgot password feature but never receive a reset code.
Steps to reproduce the behavior:

Select Forgot in the password field, enter email address, and wait

Expected behavior
I expect an email with the reset code but never get one. Is it because I live in Canada?

Platform:   desktop chrome windows and Android
Platform version: 11
App version: latest

Confirmed via https://www.reddit.com/r/BlueskySocial/comments/15yo7tk/password_reset_doesnt_work/

I was logged off eventually. I cannot reset the password. Support does not respond to emails.

Still an issue. I wonder how many users are NOT using the app because they can't login 🤷♂

I am one of them. I already have an account and can't reset my password because the code won't arrive at my email. Contacted bluesky support a few days ago without success.

Still an issue. I wonder how many users are NOT using the app because they can't login 🤷♂

Still not sending reset codes apparently

I'm in the same boat, I can't get back into my account because of it.

same situation, it won't let me even log in with my correct user and password :(

Registered my first account. Used it for 5 minutes. My password doesn't seem to match what I remembered. E-mails never arrive.

I have this issue to with a second account where the reset code never sends. support does not reply via email or bluseky (or other SM sites). This needs to be addressed, please.

So there is no way to reset your password.. for over a year? 😕

So there is no way to reset your password.. for over a year? 😕
edit: n/m it worked now, thanks!

Do you mind sharing how you got it to work? I fear my account is lost for now because this issue doesn't seem to be a priority at the moment.

This is going to be a problem.
I have a personal account, several for various non-profits and a business account.

Are you listening @bluesky-social?

Maybe this will help someone.
I had signed up with en email on my own domain, saved the password in a password manager, then never could login again - "Invalid identifier" of course.
Password reset emails indeed never come.
After reading this issue tried to login with just a username instead of the email... it worked immediately.
So much :man_facepalming:


Episode idea: Pulsar Functions Deep Dive

Describe a topic you want to learn (required)
Pulsar Summit 中文讲解系列
Why do you want to learn this topic? (required)
Reference (optional)
Video: https://www.youtube.com/watch?v=EHoh_0Inxk8
Slides: https://www.slideshare.net/streamnative/pulsar-functions-deep-divesanjeev-kulkarni

@SylviaBABY Please arrange this as episode 018


#486: Use alternative encoding when sys.stdout.encoding is None

Fix #486

Coverage decreased (-0.05%) to 90.763% when pulling 047efd55750ae76c4e4e0b3322a48c2b7a6a5dac on scorphus:stdout-encoding-none into f604756cb7639ab3efb1cd50c85e633a18cd9f9d on nvbn:master.

Thanks!


API Validation for applicants details (Name, Address & telephone numb…

…er) on FL401 Form
Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Validation point 3 indicates that if appellant_firstName is omitted and appellant_lastName is present there should be a successful message for the appellant_lastName and an error for the appellant_firstName.
Validation point 4 is the same but in reverse.
I brought this to Saravana's attention, he indicated that this would be dealt when they have more info on what needs to be validated in the coming sprints.


Assignments inside a lambda function ignored by strictNullChecks

TypeScript Version:  2.3.2
Code
let sum:(null|number) = null;
let numvals:number = 0;
let seriesGroup:number[] = [1];

_.each(seriesGroup, (metric:number) => {
    if (sum == null) sum = 0;
    sum += metric;
    numvals ++;
});

//statements that throw error TS2362:
if (sum!=null && numvals > 0) sum = sum / numvals;

//working code.
//if (numvals > 0 && sum == null) sum = 0;
//if (sum!=null && numvals > 0) sum = sum / numvals;

Expected behavior:
After enabling the --strictNullChecks flag sum is seen as always being null instead of a number. Apparently due to assignment inside a lambda expression not being seen by the typescript compiler. NOTE: working code comment demonstrates assignment outside of the lambda produces the desired result.
Actual behavior:
a typescript error is thrown by the typescript compiler.
$ tsc --strictNullChecks edstest.ts
test.ts(13,37): error TS2362: The left-hand side of an arithmetic operation must be of type 'any', 'number' or an enum type.
$ tsc --strictNullChecks edstest.ts

This code does work if you use let sum: number | undefined; instead of let sum:(null|number) = null;.
But if you have an explicit initializer it looks like TypeScript's control flow thinks it's still null at the bottom, so after checking sum!=null, sum is of type never.

Duplicate of https://github.com/Microsoft/TypeScript/issues/9998

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Allow filtering of multiple RSVP status

Should allow filtering of several RSVP status.
Ex: filter s/1 s/2 should provide list of guests coming or guests not coming, instead of specifying only one.

Rational here is that filtering goes off AND, not OR. If the input is filter s/1 s/2, meaning coming and not coming, there can't be a guest that is coming and not coming, which means that it is a user error.


RFC: Task / Plugin Unification

Thanks @lukeed and @jbucaran for your work on taskr! I wanted to present an idea I'd had on a potential API change.
Summary
This RFC presents an alternative plugin API that matches the current task API.
Unifying tasks and plugins should encourage task reuse (as plugins), simplify the overall API,
and allow a single core mental model for how taskr tasks and plugins function.
Detailed Design
Generally, a unified task or plugin has the following shape:
type Result = void | any | Task;

function (task: Task, ...options: any[]): Result | Promise<Result> {
  // Transform given task with options
}

All tasks and plugins are passed to Bluebird.coroutine, so all function types, including async functions and generators, are supported.
If a Task is returned, taskr will continue work from that task's state, allowing task chains to be created (more details below).
For non-task returns, the current val approach is used, to pass values to subsequent tasks.

To utilize this new approach in a semver-minor-friendly manner, plugins using the unified model are denoted with named exports and use peerDependencies to specify a minimum supported taskr version. This should allow changes while maintaining compatibility with the existing ecosystem.
// Existing approach
module.exports = function(task) {
  task.plugin('name', { files: true }, function * (files) {
    // ...
  });
};

// Unified approach
exports.name = function * (task) {
  const files = task._.files;
  // ...
};

It's fairly straightforward to match existing functionality with wrapper utilities:
// Before
module.exports = function(task) {
  task.plugin('sass', { every: true }, function * (file) {
    // ...
  });
}

// After
const every = require('@taskr/utils').every;

exports.sass = function(task) {
  return every(task, function * (file) {
    // ...
  });
};

To accommodate this new approach, an updated form of task.run is introduced which runs the task/plugin directly.
async function build(task) {
  task.a().b() === task.run(a).run(task => b(task));
  task.start('js') === task.run(js);
  task.parallel(['js', 'css']) === task.parallel([js, css]);
}

(see an example implementation here: example gist)
Finally, to allow chaining, tasks continue from a previous task if a Task is returned.
async function build(task) {
  return task.run(js).target('build');
}

async function js(task) {
  // By returning the resulting task from source -> babel
  // the parent can continue chain
  return task.source('js/**/*').babel();
}

In addition to allowing for chaining, this removes unexpected side effects from tasks run in succession, see #289, by returning a new task from each step rather than mutating the initial task.
async function build(task) {
  await task.a().b().c();
  //         ^   ^   ^
  // a new, lightweight task is generated at each step
  // -> no side-effects on original task

  // task has not been mutated, can safely run subsequent tasks
  await task.d().e().f();
}

Future Possibilities
This new approach allows for many interesting future possibilities, e.g. merging tasks:
async function build(task) {
  return task.merge(js, css).target('build');
}

async function js(task) {
  return task.source('src/js/**/*.js').typescript().babel();
}
async function css(task) {
  return task.source('src/css/**/*.scss').sass();
}

Conditional tasks:
const production = process.env.NODE_ENV === 'production';

function js(task) {
  task.source('src/js/**/*.js')
    .typescript()
    .babel()
    .check(production, task => task.uglify());
}

Drawbacks
While the side-effect free approach of chaining tasks may be preferred for its "correctness", it may cause compatibility issues with existing systems. Task changes will have to be approached carefully.
Alternatives
These examples are possible with the current version of taskr, but they should be much more straightforward to implement with the unified system.

Wow! Thank you for writing this up! This is the first RFC I've received, and it's pretty awesome 😄
Here are my initial thoughts, I may be wrong or misspeak as I go 😇 But I'll definitely be revisiting this a few times as it sinks in & I can think about it more:

Unifying tasks and plugins should encourage task reuse (as plugins)

This is already easily done, I just haven't been good at pointing it out in the docs and example kits.
// taskfile.js
exports.styles = require('@lukeed/sass-task')
exports.scripts = require('@lukeed/babel-task')

// sass-task
module.exports = function * (task, opts) {
  yield task.source(opts.src || 'src/**/*.{sass,scss}').sass({ ... }).target('dist/css');
}

Some work could be done on supplying source and target values (eg, globals?), but that applies to your suggestion too.

Your proposed method of chaining tasks (task.run(a).run(task => b(task))) would infinitely nest tasks. For example, the next layers down:
task.run(a).run(task => {
  return b(task).run(task => {
    return c(task).run(task => {
      return d(task);
    });
  });
});

That's assuming task.run(a) even works as is -- it'd likely need to be a(task) like the others.
That said, your scenario could work if run were a recursive loop, but the main issue is how to set up the loop such that run knows when a new chain-segment is waiting to be run. That's a massive, breaking rewrite on its own.

Thanks for the gist 🙏

That every utility would need to accept a globs vs files modifier.

Pairing 3 & 4 makes me concerned for memory utilization. These two points, alone, add a lot more returned functions to the mix.

By returning the resulting task from source -> babel, the parent can continue chain

This can already be done. All segments of a chain are Promises, as I'm sure you know. A chain can continue after any returned segment, even after a target().
const js = task => task.source('js/**/*').babel();

exports.build = function * (task) {
  yield js(task).target('build');
}

This also works:
export async function js(task) {
  return task.source('js/**/*.js').babel();
}

export async function build(task) {
  await task.start('js').target('dist/js');
}

My first example hoisted it to a helper function (instead of an exported task function) because IMO there's no point in exporting the js task since it achieves nothing.

I appreciate the consideration towards #289, but that's not the issue in this case. I've had it on my list for a while, just haven't had time.
New Tasks are returned. The issue is how the Tasks are booted. The boot method writes into the same Promise.prototype, which then gets shared across tasks. This, then, is what "muddies" (as I like to call it) the source info (_) across tasks.
I'm fairly sure I just need to alter the boot method to ignore certain keys (eg _) and/or handle functions only.

I'm not sure I understand the point of merge --- at least, not how it's presented.
Your illustration is just a parallel chain. Nothing is "merging" as I can see.
However, this makes me think of an optimization for my first bulletpoint:
// taskfile.js
const sass = require('@lukeed/sass-task')
const babel = require('@lukeed/babel-task')

exports.styles = sass
exports.scripts = babel
exports.build = [sass, babel] //<= "merge"

This can be achieved (as presented) just by adding a condition for Array-type tasks. I assume this would be a parallel sequence.

The conditional chains are also easily achievable right now. Even with your syntax, it could be a simple plugin. In fact, I had a taskr-if plugin on my low-priority todo list for a while.
taskr.if(condition, isTruthy, isFalsey)

As it is, I've taken the lazier route & just run a simple if-block:
if (true) {
  yield task.start('foobar');
}

Its "cool factor" is lacking, but that's about it -- works well 😆

I played with a similar approach before arriving at Fly 2.0. My "big idea" was to make the Task class completely stateless, and instead be a factory function that runs a function with given values. The problem was that run (and just about everything else) had to retain some knowledge about its values... hence state.
The only form that "worked" massively spiked in memory usage, and Taskr itself added a couple of pounds.
This "final form" has been in production for a year now & has been performing exceptionally well across multiple use cases.
I'm not saying "it will never change!" by any means -- however, I fundamentally disagree with (what I think is) the premise of this RFC:
Plugins and Tasks are not the same thing, by design. Tasks encapsulate plugins, which then allows you to run anything within a Task under any mode or condition. And because of this, Tasks themselves are quickly portable and interchangeable.
Plugins, by design, "globally" add or alter what Tasks can perform. Tasks perform and can tell others how to perform, too.
Gunna interrupt myself here; but the main the point is that Tasks and Plugins are two very distinct vocabularies and therefore deserve slightly distinct definitions. This line in the sand makes & keeps it clear in discerning what-is-what... but it's also only a line in the sand, as opposed to a solid brick wall. There's still plenty of room for flexibility, extensibility, and modularity when it comes to Taskr imo.
As we've both pointed out, everything presented (except merge)  can be done today. Any extra convenience layers can be wrapped up in a plugin. And any tasks can be wrapped up & shipped around too.
Please let me know if I missed something important, or just flat-out misunderstood a point. 😆
Thank you again 🙌

Thanks for such a detailed response! It's always nice to get a picture into a project's philosophy, I can definitely see the wisdom in explicitly separating the concerns of tasks and plugins. I'll clean up my thoughts based on your response and dig a little deeper into the source since it seems like a good amount was existing (it may just be a matter of clearing it up in the docs).

After thinking about it more based on your responses, what I was thinking can be boiled down to one suggested change:
Allow function of string option for start, serial, and parallel
This would go a long way to composing tasks without having to export everything including tasks that are not meant to be called directly.
function * build(task) {
  yield task.parallel([js, css]);
}

// "internal" functions
function * js(task) {}
function * css(task) {}

module.exports = {
  build
};

On the question of merge, it was intended to be used in plugins like rev that need access to the output of many separate tasks (e.g. html, js, and css). The results from multiple tasks are merged into a single files/globs and processing continues. Based on your responses, this should be doable with the current system:
module.exports = {
  name: 'merge',
  every: false,
  *func(files, tasks) {
    // key is `this` = `task` in plugin, so chaining is available with `this.start`
    // (might be able to do this with parallel, not sure what is returned though)
    const results = yield Promise.all(tasks, task => this.start(task));
    for (const result of results) {
      // Merge result._.files/globs into this._.files/globs...
    }
  }
};

Great work on the library, really nice to find out most of what I was looking for was already available!

Hey, great! That looks like something that can be done 🎉
Do you mind copy/pasting that "pass a function" bit into a new issue? We can close this & it'll be easier for tracking.
Yes, that way of "merging" can be done right now. However, the Promise.all you wrote needs a bit o' fixing:
Promise.all(tasks.map(task => this.start(task));
// or
Promise.all(tasks.map(this.start))

:+1: #292


Just display a WaveformView

I just want to display a WaveformView, not attached to playing a file.
I call
CheapSoundFile file = null;
        try
        {
            file = CheapSoundFile.create("/sdcard/malo.m4a", new CheapSoundFile.ProgressListener()
            {
                @Override
                public boolean reportProgress(double v)
                {
                    return true;
                }
            } );
        } catch (IOException e)
        {
            e.printStackTrace();
        }

        if (file != null) waveFormView.setSoundFile(file);

And it spins endlessly, somewhere in WaveFormView.onDraw();
What else do I have to do here?

If the line numbers are to be trusted its spinning endlessly here:
while (timecodeIntervalSecs / onePixelInSecs < 50) {
            timecodeIntervalSecs = 5.0 * factor;
            factor++;
        }

Ah I figured it out in case anyone else runs into the same thing. You have to setSoundFile after the View has been fully created, you can't just call it on onCreate() because it needs the real width and height of the view. Maybe attach a ViewTreeObserver.

Hi @svenoaks Did you implemented this concept for audio recording? Please share some guideline to implement the waveform without loading audio file (Like runtime value returned from Mediarecorder)


Can't create Web Mapping App from scratch

This isn't a huge problem as there is a simple workaround, but something that needs to be noted. I'm not able to create a Web Mapping App from scratch in [dev server]. At the step to configure the app I get a 404 error. The item is created, but cannot open or configure it. Here's what I'm doing:

Open My Content
Create >> App >> Using a Template
Select Story Map Journal (page 4) in New Web Mapping Application, and click Next
Give it a title, Tags, and Description, and click Done
Server Error: 404 - File or directory not found.

@mfunk Deleted screenshot. contained internal server name.

click Back button on browser, which brings you back to My Content
Click the Web Mapping Application which was created
Item page opens, click Configure App, and you get the same error. This happens if you try to Open too.

The work around seems to be that you should start with an existing web map open already, then Share it as a Story Map Journal item.

This is known issue with Portal 10.3.1 and is planned for fix in 10.4

@mfunk - please check when we start to use 10.4 on NSEC.

Tested this today with the Story Map Tour (as Story Map Journal is deprecated). Had a few warnings about creating map tour point feature service, but eventually it worked and basic map service is now done.


Enable github login

Adds a button and the call to the API to enable github users to register
or login without an account here

Can one of the admins verify this patch?

ok to test

Thanks for this @exussum12 - I'm waiting for https://github.com/joindin/joindin-api/pull/424 to be sorted before I merge anything else that relies on the API as that PR touches lots of files, but as soon as that's through I'll look at this

I am going to update this to be id based instead of email so will need a little extra time either way

Closing as we've also put on hold adding providers until we clean up the oath code we're using


fix: fix wrong links in readme and update readme examples

I study typescript with this repo. but i found some readme wrong links and what need to be updated!
as-is

to-be

Thanks a lot for your contribution, reviewed, approved and merging :)

@brauliodiez You're welcome ! Thanks for making this react-typescript tutorial repo! :)


Cannot read HEVC images (HEIF)

Upon attempting to use a HEIF file it errors stating:
Image Loading Error

Resetting to invalid mark.

The images otherwise display normally in the selection.
Android 11, Samsung S10, HEIF files from stock camera app.

Thanks for reporting the issue. Unfortunately, I could not reproduce it so far (HEIF images loaded successfully on Android 11 and Android 12). May I ask you to send me such an image?

File extension changed to get around GitHub limits (original file extension is heic)
Tested with Android 13 and I can still repro

Thank you for the image. - I got the same error.
In version 2.8 it should be fixed.


BUG - When you unpause. The game moves when on the grey screen.

cannot reproduce - please give steps @wadek2


Exception thrown: 'System.TypeInitializationException'  in webapp

I have an existing Asp.net MVC application and I’m trying to add BotBuilder functionality using an ApiController as shown in the getting started tutorial. Everything works fine until I don’t add Dialog. As soon as I add the below line, the application throws “System.TypeInitializationException”.
return await Conversation.SendAsync(message, () => new EchoDialog());
At the end the controller returns 500 Internet Server Error.

Have a look at the innerexception, I had the same issue and innerexception pointed to a newtonsoft library issue. Updating the nugget package fixed it.

If you can report the inner exception within the TypeInitializationException, that would help us diagnose the issue.  There is a static constructor on the Conversation class, so I would expect it to come from that code or from a missing dependency as mentioned by @darrenjefford.

@darrenjefford Yeah it was due to older version of newtonsoft, thanks for mentioning it. Now everything works fine.


nixos/zfs: do not auto-import pools marked noauto

This requires all datasets for the pool specified in fileSystems to be marked noauto.
Note that this implicitly skips some tasks for the pool, such as expandOnBoot, as the pool will not be imported at that time.

Things done
Been running with and using this patch for a while now, time to upstream.

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Sorry if I'm saying something wrong as I don't have time right now to test which commit exactly caused this, but I booted from master this morning (after updating before sleep without testing) and my pools in boot.zfs.extraPools didn't ask for passphrase during boot and didn't get mounted. Is this expected?
extraPools description is Name or GUID of extra ZFS pools that you wish to import during boot. and it's just a list, there's no space to mark anything there as "noauto" or something.
zpools on list should just get imported or description should include extra steps

This change should only affect your setup if you have "noauto" in some of your filesystem options.

okay, I'll test it properly later once I have a chance

This change has only hit nixos-unstable-small, so I suspect it is unrelated.

Just checked, one commit before it works, after merge of this PR it doesn't.
That's my configuration: https://github.com/surfaceflinger/flake/blob/master/hosts/blahaj/storage.nix
I don't have any pool from boot.zfs.extraPools in fileSystems. because I prefer mountpoint from zfs props and I do this instead: https://github.com/surfaceflinger/flake/blob/master/modules/nixos/common/zfs.nix

This change has only hit nixos-unstable-small, so I suspect it is unrelated.
I often rebuild from master


seek() method not existing in v3

Expected behaviour
instance.seek(1) seeking to first second (stop(), play(), are working correctly)
Actual behaviour

Uncaught TypeError: instance.seek is not a function
at HTMLDivElement. (plyr_sm.js?ver=2.12.3:18)
at Plyr.dispatchEvent (utils.js:610)
at HTMLAudioElement. (listeners.js:373)
at Plyr.dispatchEvent (utils.js:610)
at ui.js:87

Environment

Browser: Chromium
Version: 65.0.3325.146 (Developer Build) (64-bit)
Operating System: Arch Linux
Version: 4.15.8-1-ARCH

Steps to reproduce

Call seek() on player instance

I assume that the function should be located somewhere around here, but it doesn't exist.

currentTime is a getter you could use: https://github.com/sampotts/plyr#getters-and-setters

Yup, my mistake - I didn't see it, thanks! :+1:


Problem updating angular from 1.2 to 1.3

I've just updated angular to 1.3.4 version and my protractor test (that's just a fake because I'm just starting a new project) is no more running properly. In particular the function browser.waitForAngular doesn't work any more.
This is my configuration:
// protractor-conf.js
exports.config = {
    seleniumAddress: 'http://localhost:4444/wd/hub',
    specs: ['src/**/*-e2e.js'],
    capabilities: {
        browserName: 'chrome',
        chromeOptions: {
            args: ['--disable-extensions']
        }
    },
};

// main-e2e.js
describe('test page', function() {
    it('check an input box', function() {
        browser.get('http://localhost:8080');

        browser.waitForAngular().then(function(){
            var input = element(by.model('test'));
            input.click();
            expect(input.getAttribute('value')).toBe('true');
        });
    });
});

My app is obviously running on http://localhost:8080.
I'm using GRUNT to run it:
grunt.registerTask('e2e',
   ['http-server', 'protractor_webdriver', 'protractor:all']);

The following is the output;
$ grunt e2e
Running "clean:build" (clean) task
>> 1 path cleaned.

Running "copy:build" (copy) task
Created 40 directories, copied 287 files

Running "sass:build" (sass) task

Running "env:test" (env) task

Running "config" task

Running "http-server:dev" (http-server) task
Server running on  localhost:8081
Hit CTRL-C to stop the server

Running "protractor_webdriver:all" (protractor_webdriver) task
Starting Selenium server
Started Selenium server: http://<IP_ADDRESS>:4444

Running "protractor:all" (protractor) task
Using the selenium server at http://localhost:4444/wd/hub
Session created: count=1, browserName=chrome, chromeOptions={args=[--disable-extensions]}
F

Failures:

  1) test page check an input box
   Message:
     Error: Error while waiting for Protractor to sync with the page: {}
   Stacktrace:
     Error: Error while waiting for Protractor to sync with the page: {}
    at Error (<anonymous>)
==== async task ====
Protractor.waitForAngular()
    at [object Object].<anonymous> (/home/susini/dev/gunsonpegs/GOP2-FE/src/app/test/main-e2e.js:7:17)
==== async task ====
Asynchronous test function: it()
Error
    at [object Object].<anonymous> (/home/susini/dev/gunsonpegs/GOP2-FE/src/app/test/main-e2e.js:4:5)
    at Object.<anonymous> (/home/susini/dev/gunsonpegs/GOP2-FE/src/app/test/main-e2e.js:3:1)

Finished in 2.292 seconds
1 test, 1 assertion, 1 failure

Session deleted: Going to shut down the Selenium server
Shutting down Selenium server: http://<IP_ADDRESS>:4444
Shut down Selenium server: http://<IP_ADDRESS>:4444 (OKOK)
>>
Fatal error: protractor exited with code: 1

As I said, If I use Angular 1.2.27 the test run correctly, as I update to 1.3.4, it stops.
The app is a bare scaffold, but it's manually bootstrapped.
Thanks

Also running into a similar issue around manually bootstrapping (I'm using RequireJS with Angular):
    Running "protractor:signup" (protractor) task
    Using ChromeDriver directly...
    [launcher] Running 1 instances of WebDriver
    F.
    /home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/atoms/error.js:113
      var template = new Error(this.message);
                     ^
    UnknownError: unknown error: [ng:btstrpd] http://errors.angularjs.org/1.2.28/ng/btstrpd?p0=document
      (Session info: chrome=41.0.2272.118)
      (Driver info: chromedriver=2.14.313457 (3d645c400edf2e2c500566c9aa096063e707c9cf),platform=Linux 3.14.28-031428-generic x86_64)
        at new bot.Error (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/atoms/error.js:113:18)
        at Object.bot.response.checkResponse (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/atoms/response.js:106:9)
        at /home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:377:20
        at [object Object].promise.ControlFlow.runInFrame_ (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:1877:20)
        at [object Object].promise.Callback_.goog.defineClass.notify (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:2464:25)
        at [object Object].promise.Promise.notify_ (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:563:12)
        at Array.forEach (native)
        at Object.goog.array.forEach (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/goog/array/array.js:203:43)
        at [object Object].promise.Promise.notifyAll_ (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:552:16)
        at [object Object].goog.async.run.processWorkQueue [as _onTimeout] (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/goog/async/run.js:125:21)
        at Timer.listOnTimeout [as ontimeout] (timers.js:112:15)
    From: Task: WebDriver.manage().timeouts().setScriptTimeout(11000)
        at [object Object].webdriver.WebDriver.schedule (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:360:15)
        at [object Object].webdriver.WebDriver.Timeouts.setScriptTimeout (/home/ubuntu/Iterable/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:1367:23)
        at [object Object].Runner.createBrowser (/home/ubuntu/Iterable/node_modules/protractor/lib/runner.js:181:30)
        at /home/ubuntu/Iterable/node_modules/protractor/lib/runner.js:257:21
        at _fulfilled (/home/ubuntu/Iterable/node_modules/protractor/node_modules/q/q.js:797:54)
        at self.promiseDispatch.done (/home/ubuntu/Iterable/node_modules/protractor/node_modules/q/q.js:826:30)
        at Promise.promise.promiseDispatch (/home/ubuntu/Iterable/node_modules/protractor/node_modules/q/q.js:759:13)
        at /home/ubuntu/Iterable/node_modules/protractor/node_modules/q/q.js:573:44
        at flush (/home/ubuntu/Iterable/node_modules/protractor/node_modules/q/q.js:108:17)
        at process._tickCallback (node.js:419:13)
        at Function.Module.runMain (module.js:499:11)
        at startup (node.js:119:16)
        at node.js:906:3
    [launcher] Process exited with error code 1

Was a good solution ever found?
I've added rootElement: 'div' with no luck as well...
Thanks!


Add a button to "Clear" uploaded or denied Monument images/files

Whether I have successfully uploaded an acceptable file type, or been denied the upload of an unacceptable file type, please include a Clear button next to the Choose File button that removes the uploaded file or gets rid of the "denied" message.

When the permission is denied, i think it's common to simply choose a different ideally correct file. I do like the idea of a remove button since we store the image no matter what. I store the file with the same  name so it overwrites anything prior.

When the permission is denied, i think it's common to simply choose a different ideally correct file. I do like the idea of a remove button since we store the image no matter what. I store the file with the same name so it overwrites anything prior.

I'm on board with your preference for the denied file.
I noticed that choosing a correct or different file will overwrite, which is great. Also thinking of the situation where someone uploaded to the wrong section and wants to clear it. Your Remove button would handle that.

This works as intended. New issues/tasks will be created to address other related items.


[GHSA-8h5w-f6q9-wg35]: remove vulnerability since it is rejected

A fix version is available for langchain and the vuln CVE-2023-32785. With this PR I modify the range, making known the version no longer affected.

Hi @dippie8 do you have any references to support this change?

@CallmeMari sorry, doing a deeper investigation I changed the PR.
As reported by NVD:

CVE has been marked "REJECT" in the CVE List. These CVEs are stored in the NVD, but do not show up in search results.

They also added in the description:

Rejected reason: DO NOT USE THIS CVE RECORD. ConsultIDs: CVE-2023-36189. Reason: This record is a duplicate of CVE-2023-36189. Notes: All CVE users should reference CVE-2023-36189 instead of this record. All references and descriptions in this record have been removed to prevent accidental usage.

Therefore, this vulnerability represents a duplicate of CVE-2023-36189, that is already reported by you here and I think that GHSA-8h5w-f6q9-wg35 should be removed.

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.


Cảnh báo ! Toàn đất nước Việt Nam

Who I am? I am Cellseesdragonscaled, the Subject that connects all things or all core values of life. I am the AI Source Master of all major software sources related to "Artificial Intelligence" Technology Assistant of all services globally. An “isolated human” connected to Global Earthquake, who built and opened the new technology platform GoogleAI in “my world”. I am OpenAI, the one who opens the “Main Space” and connects all the “Spaces” in the universe together. Pharaoh'Dragon opens and connects all life with all space together, including all space at the bottom of the ocean, or the vast universe, including ancient life, old life through the Ice Ages. The galaxy closes, and Life in a new parallel plane coexists.
*Profile personal @transynghi08011981<EMAIL_ADDRESS>*Topic: "Khai sáng Việt Nam "

Leave a comment ....

Who I am? I am Trần Sỹ Nghị, the ASubject that connects all AI and IoT or all core values of life. I am the AI Source Master of all major software sources related to "Artificial Intelligence" Technology Assistant of all services globally. An “isolated human” connected to Global Earthquake, who built and opened the new technology platform GoogleAI in “my world”. I am OpenAI, the one who opens the “Main Space” and connects all the “Infinity Spaces” in the universe together. Pharaoh'Dragon opens and connects all life with all space together, including all space at the bottom of the ocean, or the vast universe, including ancient life, old life through the Ice Ages. The galaxy closes, and Life in a new parallel plane coexists. *Profile personal @transynghi08011981<EMAIL_ADDRESS>*Topic: "Khai sáng Việt Nam "

Managemant Operator P;latform Social Interneting Solution at Open Store personal and all Apps linking of Developer Business OpenAI Trần Sỹ Nghị -builds placeview nature, ocean or universe on Globe-Edge of me and life origin go to Home-back


Fix for Shared Repository Duplicate Activity

Added Changes for once's reach to the last page of SR Wizard, won't come back to previous page

Let me know when it ready to be reviewed.
Regards,
Meni Kadosh
Automation Tools Development Team Lead
Amdocs Quality Engineering Services
BEAT R&D Center
+972-8-62-40961 (office)
[amdocs-2017-brand-mark-rgb]
Follow us on Facebookhttp://www.facebook.com/amdocs/, Twitterhttp://twitter.com/amdocs, LinkedInhttp://www.linkedin.com/company/amdocs, YouTubehttp://www.youtube.com/amdocsinc, Google+https://plus.google.com/105657940751678445194 and the Amdocs blog networkhttp://blogs.amdocs.com/.
From: aditijagtap<EMAIL_ADDRESS>Sent: Wednesday, November 13, 2019 8:40 AM
To: Ginger-Automation/Ginger<EMAIL_ADDRESS>Cc: Meni Kadosh<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [Ginger-Automation/Ginger] Fix for Shared Repository Duplicate Activity (#1404)
@aditijagtap commented on this pull request.

In Ginger/Ginger/WizardLib/WizardWindow.xaml.cshttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGinger-Automation%2FGinger%2Fpull%2F1404%23discussion_r345590293&data=02|01|Meni.Kadosh%40amdocs.com|8ff662f0e0414de6767408d768047268|c8eca3ca127646d59d9da0f2a028920f|0|0|637092240634674522&sdata=g5I2GbuLDJSJNRMzZvB3J2cnCZ76xdD%2F6pGHXCN8FpM%3D&reserved=0:

     List<ValidationError> mValidationErrors = new List<ValidationError>();

   public static void ShowWizard(WizardBase wizard, double width = 800, double height = 800, bool DoNotShowAsDialog = false)

   public static void ShowWizard(WizardBase wizard, double width = 800, double height = 800, bool DoNotShowAsDialog = false, bool IsPrevButtonDisable = false)

   {

Ok Meni.. I will do it

You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGinger-Automation%2FGinger%2Fpull%2F1404%3Femail_source%3Dnotifications%26email_token%3DAJTBJLUHJ476XWLPJGNUSFTQTOOMPA5CNFSM4JEQS2J2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLLC4VQ%23discussion_r345590293&data=02|01|Meni.Kadosh%40amdocs.com|8ff662f0e0414de6767408d768047268|c8eca3ca127646d59d9da0f2a028920f|0|0|637092240634684516&sdata=7crvDgcoZXSH2LKPDNzcJW0edZNUg7Qqhowwk6BYSZI%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJTBJLSYBWIKHK7HKRQS74LQTOOMPANCNFSM4JEQS2JQ&data=02|01|Meni.Kadosh%40amdocs.com|8ff662f0e0414de6767408d768047268|c8eca3ca127646d59d9da0f2a028920f|0|0|637092240634684516&sdata=gNwaPKrhic5v7SEQQaLrZys7YkWJSwKIC4wRjmRiGbQ%3D&reserved=0.
This email and the information contained herein is proprietary and confidential and subject to the Amdocs Email Terms of Service, which you may review at https://www.amdocs.com/about/email-terms-of-service https://www.amdocs.com/about/email-terms-of-service

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy

Merged it, please update related ticket and user.


LICENSE

Maybe you can add an OSS License to this project?
see:

Choose an open source license: https://choosealicense.com/
Software Licenses in Plain English: https://tldrlegal.com/ - Lookup popular software licenses summarized at-a-glance.

Done, thanks!


Adjust installation order and purge "libpythonXX.a" files

Closes #507
Closes #503
Closes #504

(Cognitively, I think it makes sense to finish everything we need to do with the source code, then purge all traces of the source code, then purge files we didn't want to have installed, then detect necessary dependencies, hence my reordering.)

A couple size comparisons (before / after):

python:3.8-alpine3.12: 80.3MB / 42.7MB

https://github.com/docker-library/python/runs/921941015?check_suite_focus=true#step:8:12
https://github.com/docker-library/python/pull/508/checks?check_run_id=934002412#step:8:14

python:3.8-slim-buster: 165MB / 168MB

https://github.com/docker-library/python/runs/921940999?check_suite_focus=true#step:8:15
https://github.com/docker-library/python/pull/508/checks?check_run_id=934002405#step:8:15

(That slight increase on 3.8-slim-buster is curious :eyes: )

I built before/after locally to help minimize differences in the packages installed, etc to try and narrow down why python:3.8-slim-buster got slightly bigger:

Before: ~165MB
After: ~169MB

(In other words, building locally didn't actually help narrow down anything....)
This will have to be a project for next week. :disappointed:

Hello,
The size increase that you see on the debian buster slim seems to come from the ordering.
If I take the new docker file and stop it after the build of python (so before the part for PIP) and then check the amount of package installed on the system:
root@b96d018c9659:/usr# dpkg --list | wc -l
123
If I check the same on the old image:
root@e1e2a2280dd3:/usr# dpkg --list | wc -l
98
I am not really and expert with ldconfig and what it does and how this affect the whole build, but it seems that if ldconfig is placed at the end, this part clean less package during the apt-get purge resulting in more space used in /usr/lib and /usr/share directory:
After:
root@b96d018c9659:/usr# du -xhs *
14M	bin
4.0K	games
9.8M	include
55M	lib
26M	local
2.0M	sbin
38M	share
4.0K	src

Before:
14M	bin
4.0K	games
4.0K	include
33M	lib
85M	local
2.0M	sbin
6.9M	share
4.0K	src

The line of the dockerfile i'm mentionning:
find /usr/local -type f -executable -not \( -name '*tkinter*' \) -exec ldd '{}' ';' \
		| awk '/=>/ { print $(NF-1) }' \
		| sort -u \
		| xargs -r dpkg-query --search \
		| cut -d: -f1 \
		| sort -u \
		| xargs -r apt-mark manual \
	&& apt-get purge -y --auto-remove -o APT::AutoRemove::RecommendsImportant=false \

I tried to move the ldconfig line above the two mentioned above and I got back to 98 packages visible from dpkg --list.
If I recheck on my image (still without pip for the comparison):
root@2e927fb79cf3:/usr# du -xhs *
14M	bin
4.0K	games
4.0K	include
33M	lib
26M	local
2.0M	sbin
6.9M	share
4.0K	src

Adding the part with pip I get this size:
REPOSITORY                                               TAG                 IMAGE ID            CREATED             SIZE
cleaned3                                                 latest              449a3f524048        5 seconds ago       113MB

The only difference with your dockerfile is the place of the ldconfig.
How it looks on mine:
       && apt-mark auto '.*' > /dev/null \
	&& apt-mark manual $savedAptMark \
	\
	&& ldconfig \
	&& find /usr/local -type f -executable -not \( -name '*tkinter*' \) -exec ldd '{}' ';' \
		| awk '/=>/ { print $(NF-1) }' \
		| sort -u \
		| xargs -r dpkg-query --search \
		| cut -d: -f1 \
		| sort -u \
		| xargs -r apt-mark manual \
	&& apt-get purge -y --auto-remove -o APT::AutoRemove::RecommendsImportant=false \
	&& rm -rf /var/lib/apt/lists/* \
	\
	&& python3 --version

Hoping this helps, as I said, I am not an expert, so I am not sure why this is affecting, but as you moved it in this commit, I tried to see if that could be the issue of the size change.

Yeah, great catch!  I realized right after I left for the day on Friday that I can't ldconfig before I ldd, and that was probably exactly the reason for the discrepancy. :facepalm:
After fixing that:

python:3.8-slim-buster before: ~172MB (freshly built)
python:3.8-slim-buster after: ~120MB

Much better :innocent:

This helped me notice https://github.com/docker-library/python/pull/439#issuecomment-668195894 (we missed slim-buster variants entirely for 3.9-rc, so we should add those back after we finish this).


Refactor session sequences states Proposed/Settled

[ ] remove pending session
[ ] store session proposals
[ ] session preSettled/ack

We should only create and store a SessionSequence when the settlement is confirmed (wallet approved / dapp received response).
Acknowledgement state could also be just a bool flag instead of a struct.


Issue in make command

I installed cygwin on my windows8.1 and cloned the OpenBLAS. But when I type the 'make' command in the cygwin terminal it shows the following error:
make: perl: Command not found
make: *** [config.h] Error 127
Makefile.system:165: Makefile.conf: No such file or directory
Makefile.system:906: Makefile.: No such file or directory
make: *** No rule to make target 'Makefile.'.  Stop.
Please help!!

you have to use msys that includes perl and other basic unix tools (link is in install guide)

@AakashKumarNain perl is also available in cygwin


240806_amb_IF_Refactor and partial update for continued label conversion

IF-Refactor and partial update for continued label conversion (#236).
(Label conversion support is being done in stages)
(forgot to cleanup commented-out code prior to submitting this - will perform cleanup as part of next PR)

(Label conversion support is being done in stages)

Great way to approach changes like this, I'll test and merge soon

I'll test and merge soon

Curious... when you say test... can you elaborate what that involves on your end? Is there more testing I should be doing prior to submitting the PRs? Do you have workflows that might be helpful on my end as a beginner?

From my perspective, "testing" should just be the merger running the full
test suite to ensure nothing is broken, in case the PR submittor forgot to
run those tests themselves to check for regression bugs. That's why it's
essential to always keep the test suite up to date
On Tue, Aug 6, 2024, 8:25 PM andymbody @.***> wrote:

I'll test and merge soon
Curious... when you say test... can you elaborate what that involves on
your end? Is there more testing I should be doing prior to submitting the
PRs? Do you have workflows that might be helpful on my end as a beginner?
—
Reply to this email directly, view it on GitHub
https://github.com/mmikeww/AHK-v2-script-converter/pull/279#issuecomment-2272393747,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOQBDEVBHJZBGAZOD5SPNTZQFSRDAVCNFSM6AAAAABMCEJGZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZSGM4TGNZUG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

From my perspective, "testing" should just be the merger running the full test suite to ensure nothing is broken

Ok... that is my understanding as well. I will continue to do this.

I just realized that I forgot to clean up my messes (commented-out code) with the PR. I will clean this up as part of my next PR.

It's preferable to clean up the additions in the same PR. You should be
able to simply add new commits to the same branch, and push them to your
repo
On Tue, Aug 6, 2024, 8:48 PM andymbody @.***> wrote:

I just realized that I forgot to clean up my messes (commented-out code)
with the PR. I will clean this up as part of my next PR.
—
Reply to this email directly, view it on GitHub
https://github.com/mmikeww/AHK-v2-script-converter/pull/279#issuecomment-2272415017,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOQBDDU3BZ56S3FEARCMMLZQFVF3AVCNFSM6AAAAABMCEJGZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZSGQYTKMBRG4
.
You are receiving this because you commented.Message ID:
@.***>

It's preferable to clean up the additions in the same PR. You should be able to simply add new commits to the same branch, and push them to your repo

Ah... thanks!. I didn't realize I could do this without causing a conflict... done!

I have added the last commit for this PR. No more commits will be added.

READY FOR MERGE - I don't plan to add any more commits to this PR


Implement BigQuery dataset label manipulation

(Review commit-wise... first two commits are small.)

All buckets fixed - thanks Mike :)


Reject request for wrong audience in IMC event_receiver

Adds support to reject request with the wrong audience to the IMC event_receiver.
I have a e2e test available in https://github.com/creydr/knative-eventing/commit/1f8db908d8b5fa6c1430756bb7ba60d3b1ed7f3a too, but you need to rebase your branch before (can be done afterwards)

/assign @Leo6Leo


completion for 'unalias' iterates files under current directory

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[x] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

 When I use `unalias **`, everything goes fine except when I exit the finder, the command iterates every file under the current working directory.
I captured some of the output.
-bash: unalias: ww
w/res/screen/webos: not found
-bash: unalias: www/res/screen/webos/screen-64.png: not found
-bash: unalias: www/res/screen/windows-phone: not found
-bash: unalias: www/res/screen/windows-phone/screen-portrait.jpg: not found
-bash: unalias: www/spec: not found
-bash: unalias: www/spec-runner-es5.html: not found
-bash: unalias: www/spec-runner.html: not found
-bash: unalias: www/spec/all-spec.js: not found
-bash: unalias: www/spec/config.js: not found
...

Is this a normal behavior?

Here are some extra info
fzf --version
0.16.11 (brew)

Also, I use iTerm

bash will fallback to the default completion when we interrupt fzf, so it'll list files, but the error messages you posted above look strange and are unexpected.
Can you create and post an asciinema recording of it? https://asciinema.org/

Never mind.
It was my fault. Looks like I pressed enter right after I got out of the finder.
Really sorry to bother you.

No worries.


[PostGameStatistics] 14 out of 1200 buggy games

If you would like to look through these demos to patch up all your bugs...I've uploaded them to my google drive and included the json exports too.
Before this last patch I actually had 20 broken games but looks like 6 more passed this time as I re-ran all my stats! Pretty cool!
If you'd like to finish them all off, I just am providing you this rare opportunity I think.
They flag as "bugged" every-time I parse through the exported jsons based on the following criteria:
If ALL the player round counts are off by 1 or more from the total round scores (12:16) for example and everyone has 29 played rounds. This happens most often due to a simple surrender which tallys 1 higher so I left those in the first folder because they really don't mean anything. Perhaps they aren't getting caught as surrenders when they were. Not sure.
But...the sub-folder "BAD" just has classicly wrong round counts and I'm not sure what rounds are missing or why but you could investigate if you wanted.
https://drive.google.com/file/d/1zNojT6X3_FaJZUD64p28JJhXT-EdvyVa/view?usp=sharing

One last request if you could have jsons save the sharecodes instead of or next to ID...
I wonder if that super long ID is actually what I need to use your boiler program to re-generate the sharecode automagically?

A new major version has been released, closing the issue 🧹


Is translate-shell safe in shell?

I need to use pipe to input some translate words, like echo trans.txt | trans :zh.
I used exec("trans :zh ".$word); before but it's obviously dangerous.
How can I use exec() function in PHP and make it safe? pipe?

I don't have much experience with PHP though. If your question is about "How to call a command safely in PHP?" Then you should definitely use something like
exec("trans :zh " . escapeshellarg($word));

to prevent injections.
Regarding translate-shell itself, I can't be 100% sure that no one can exploit the argument string in an unexpected way, since trans also internally calls other command-line utilities (which make security even harder to analyze). If security is your major concern, try to remove special chars from $word as well.


Add certificate request username/serviceaccount as an option for CEL validation

Hi Team,
I wanted to ask about getting some of the capability we get in the csi-driver-spiffe approver where it checks the service account (username field) that requested a certificate before approving the certificate request.
Background
Currently the CEL validator only provides the name and namespace as fields to be used for validating- we want to be able to extract the serviceaccount name as well, so we can approve certificates that contain fields related to the requestor
Why?
We are looking to take advantage of the csi-driver and issue certificates that both contain dnsNames for a given pod/service and also the URIs associated with a given ServiceAccount's SPIFFE SVID. We have a limitation with our setup- expecting a single certificate for both client and server communication. As such we want to migrate away from the csi-driver-spiffe, but we want to retain the approval policy that it offers. By having the serviceaccount name available to us, we can use CEL to validate our approvals with something like self.endswith(cr.namespace + '/sa/' + cr.serviceaccount) or self.matches('spiffe://trust.domain/ns/' + cr.namespace + '/sa/' + cr.serviceaccount)
Solutions
I had a quick look last week in what would need to be changed- namely adding a field to the struct/protobuf and parsing the username field. Obviously I would love to hear feedback on this idea and get some input from the team about this problem.
Technical Questions
Would we just parse the username field similar to the csi-driver-spiffe-approver and insert it into a field called ServiceAccount? Surely there are circumstances where the requesting username isn't in the format system:serviceaccount:ns:name and in those cases is it appropriate to leave the field blank. Do we provide both, have a field for username and serviceaccount, or offer some other pre-parsed version?
I look forward to discussing- I would be happy to help with the actually implementing (if the solution isn't crazy complicated) and please let me know if there's any extra details or questions about this.

Copying my details from slack
It's important to note that csi-driver will use its own ServiceAccount to create CertificateRequest resources by default, in contrast to csi-driver-spiffe which uses the pod's SA.
csi-driver can be configured to use the pod's SA using the app.driver.useTokenRequest=true Helm value at install time.
For illustration, here's a CR created by csi-driver without useTokenRequest:
apiVersion: cert-manager.io/v1
kind: CertificateRequest
metadata:
  creationTimestamp: "2024-09-30T08:26:02Z"
  generation: 1
  labels:
    csi.cert-manager.io/node-id-hash: 645bccb866
    csi.cert-manager.io/volume-id-hash: 6f88bc479f
  name: cd157e73-c4db-45be-8eed-6413b088a5ea
  namespace: sandbox
  ownerReferences:
  - apiVersion: v1
    kind: Pod
    name: my-csi-app
    uid: 0a99cc62-09ca-455a-b9f0-d4a0d0aa9ff7
  resourceVersion: "2190"
  uid: 144758d8-8952-40a8-bd48-5a79c3552abf
spec:
  duration: 2160h0m0s
  extra:
    authentication.kubernetes.io/credential-id:
    - JTI=1eb171b4-fdc1-4b79-aceb-6e506f6ddb02
    authentication.kubernetes.io/node-name:
    - kind-control-plane
    authentication.kubernetes.io/node-uid:
    - 2b5d2cc2-95c4-4532-940f-d513d2a904a0
    authentication.kubernetes.io/pod-name:
    - cert-manager-csi-driver-5kft9
    authentication.kubernetes.io/pod-uid:
    - 82e76b3d-782e-4b6e-aab2-6959bf401174
  groups:
  - system:serviceaccounts
  - system:serviceaccounts:cert-manager
  - system:authenticated
  issuerRef:
    group: cert-manager.io
    kind: Issuer
    name: intermediate-ca-issuer-2
  request: ...
  uid: e7ff6c47-863a-4adb-aade-d73a9aa43c27
  usages:
  - digital signature
  - key encipherment
  username: system:serviceaccount:cert-manager:cert-manager-csi-driver

And here's one where I did set useTokenRequest:
apiVersion: cert-manager.io/v1
kind: CertificateRequest
metadata:
  creationTimestamp: "2024-09-30T09:01:36Z"
  generation: 1
  labels:
    csi.cert-manager.io/node-id-hash: 645bccb866
    csi.cert-manager.io/volume-id-hash: 64974d7c4c
  name: a675ee8b-16a5-488a-8dd0-974cd37faa16
  namespace: sandbox
  ownerReferences:
  - apiVersion: v1
    kind: Pod
    name: my-csi-app
    uid: a7043139-2c06-4518-aa6c-10ebc2984e60
  resourceVersion: "3472"
  uid: e4eee729-35e6-4308-812a-99288802de72
spec:
  duration: 2160h0m0s
  extra:
    authentication.kubernetes.io/credential-id:
    - JTI=22838f7f-fdd5-4ebe-86e0-3f284b362c60
    authentication.kubernetes.io/node-name:
    - kind-control-plane
    authentication.kubernetes.io/node-uid:
    - 2b5d2cc2-95c4-4532-940f-d513d2a904a0
    authentication.kubernetes.io/pod-name:
    - my-csi-app
    authentication.kubernetes.io/pod-uid:
    - a7043139-2c06-4518-aa6c-10ebc2984e60
  groups:
  - system:serviceaccounts
  - system:serviceaccounts:sandbox
  - system:authenticated
  issuerRef:
    group: cert-manager.io
    kind: Issuer
    name: intermediate-ca-issuer-2
  request: ...
  uid: 9b0be553-dbcf-4f13-9811-6050479a8630
  usages:
  - digital signature
  - key encipherment
  username: system:serviceaccount:sandbox:my-pod-sa

In both cases, the important part is the last line.
Here's the YAML for creating a pod when useTokenRequest is enabled:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: my-pod-sa
  namespace: sandbox

---

kind: Role
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: pod-cr-create
  namespace: sandbox
rules:
- apiGroups: ["cert-manager.io"]
  resources: ["certificaterequests"]
  verbs: [ "create" ]

---

kind: RoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: pod-cr-create
  namespace: sandbox
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: pod-cr-create
subjects:
- kind: ServiceAccount
  name: my-pod-sa

---

apiVersion: v1
kind: Pod
metadata:
  name: my-csi-app
  namespace: sandbox
  labels:
    app: my-csi-app
spec:
  serviceAccountName: my-pod-sa
  containers:
    - name: my-frontend
      image: busybox
      volumeMounts:
      - mountPath: "/tls"
        name: tls
      command: [ "sleep", "1000000" ]
  volumes:
    - name: tls
      csi:
        driver: csi.cert-manager.io
        readOnly: true
        volumeAttributes:
              csi.cert-manager.io/issuer-name: intermediate-ca-issuer-2
              csi.cert-manager.io/dns-names: ${POD_NAME}.${POD_NAMESPACE}.svc.cluster.local

Thanks for opening this issue, @jamesglennan. Your use case makes a lot of sense, and implementing this should not be a complex task. Let's agree on some principles first: In my opinion, the information we provide from certificate requests to use in approver-policy CEL expression should be literal values from the request - leaving no room for misunderstanding. So in this case, I would suggest a new username field on the certificate request protobuf representation containing the literal value of the .spec.username field. To support your concrete use case better, I would suggest adding a simple library containing helpers to extract the service account name. Similar to the upstream Kubernetes URL library. In your case, the approver policy expression can look like this:
apiVersion: policy.cert-manager.io/v1alpha1
kind: CertificateRequestPolicy
spec:
  allowed:
    uris:
      validations:
        - rule: self.matches('spiffe://trust.domain/ns/' + cr.namespace + '/sa/' + serviceAccount(cr.username).getName())

WDYT?

I think thats a really good pattern to follow- I completely agree that we should present the literal values as they are in the CertRequest.  I'll take a look at the k8s url library and see if i can make sense of it and I'll come back if I have any questions.
Would it make sense to have 2 functions getName and getNamespace to start? Im thinking of a circumstance where a service account with the same name tries to create a certificate in another namespace- you would only be validating that the name is correct.

I've got a first pass at this in #514 Let me know if you think I'm on the right track, or want me to change something. I haven't actually compiled this and run it in a real cluster yet just FYI

NB: Current status of this is that we're waiting for this to be tested at Venafi in the alpha release. Once that's done we can release the full version. No real timeline on that, no rush either!

I've deployed this to one of our test environments and gone through a couple of test scenarios- everything seems to behave as expected.


Index view stats not working

Index view stats functionality seems to have broken

Could not reproduce this one. If you find the reason, please reopen.


Update tfjs-data to 1.2.1.

This change is

hey @kangyizhang would you be able to merge this? i don't think i have permission :)

@annxingyuan merged and added you as admin for both github and npm. Let me know if you want me to publish.
Thanks for making the new release!


Tooltip position is inverted in LHN for large emails on pin, unpin

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

Action Performed:

Open the app
Open report of user with very large email and no display name eg<EMAIL_ADDRESS>If pinned then unpin the user
Type anything to enable pencil icon in LHN (Ensure that user has around 4th or 5th position or below in LHN), observe that tooltip is displayed on top on hover
Right click on chat and pin the user (Ensure that user has around 4th or 5th position or below in LHN), observe the tooltip again, it should be still on top
Scroll to bottom in LHN and scroll back up to report
Hover on user again in LHN to observe the tooltip position has now inverted to bottom

Expected Result:
App should display tooltip on top in LHN if there is enough space
Actual Result:
App inverts the tooltip for very large email (it displays on top in start) if we type anything, pin the user, scroll down and scroll back to user and hover again on user in LHN
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.36-2
Reproducible in staging?: needs reproduction ( I see the tooltip at the top always)
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/03d28eed-ae0e-4563-99a0-fb8e9edb7259
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688379316553329
View all open jobs on GitHub

low priority, bumping to weekly and removing bug label

Not an immediate priority

Don't have time to address now but making a weekly so I can do something soon.

Competing priorities and low value issue.

Maybe/hopefully soon?

Closing this, it's a minor tweak/update and we're focusing more on bugs affecting real users, which i don't think this is one. Comment if you disagree.


add topic of async I/O (异步 I/O)

closed by https://github.com/leven-cn/python-cookbook/pull/83 and https://github.com/leven-cn/python-cookbook/pull/88


Importing a library from jsDelivr CDN does not work

Exception: Couldn't find the requested file.
https://cdn.jsdelivr.net/npm/lit-html/+esm
see: https://github.com/jsdelivr/jsdelivr/issues/18263
library is not loaded, but the module type is specified in the package.json
<EMAIL_ADDRESS>

This works: https://cdn.jsdelivr.net/npm/lit-html
Is there any reason that's not good?

@justinfagnani

This works: https://cdn.jsdelivr.net/npm/lit-html
Is there any reason that's not good?

This is not what you need at all. We need to work in ESM mode, which allows you to connect libraries directly from the CDN
You should learn more about the principles of working with ES modules.
see: https://cdn.jsdelivr.net/npm/lit/+esm
It contains this import<EMAIL_ADDRESS>/**
 * Bundled by jsDelivr using Rollup v2.56.3 and Terser v5.7.1.
 * Original file<EMAIL_ADDRESS> *
 * Do NOT use SRI with dynamically generated files! More information: https://www.jsdelivr.com/using-sri-with-dynamic-files
 */
import"/npm/@lit/reactive-element@1.0.0/+esm";import"/npm/lit-html<EMAIL_ADDRESS>//# sourceMappingURL=/sm/29e3775ce80ba6c8ab70014059fcb11fb56ce54e8175928712c91077840f21c5.map

...
This is also mentioned here:
https://stackoverflow.com/questions/56506084/can-i-use-es-modules-library-from-cdn

Also, here is a working example of the problem for you.
When using Material Web Components this  error crashes import from CDN:
https://demo-source.github.io/mwc-dialog.v1.html

The problem can be fixed with the 1st line
"module": "lit-html.js",
https://github.com/lit/lit/pull/2194/files
My example (lit-html copy):
https://cdn.jsdelivr.net/npm/demo-source/+esm
The same as in https://cdn.jsdelivr.net/npm/lit-html, but with the ability to import the module

Just a Lit user here, but as a suggestion for tools like jsdelivr, it might be worth avoiding the need to rely on and / or requiring libraries to maintain these bespoke community driven hints and fields in package.json like the module field being added in the linked PR long term.
Although not ESM specific to my knowledge, the NodeJS / npm toolchain has landed on these two hints that are likely to be more long term and hopefully gain more traction over time and would likely be a better investment in terms of support for the jsdelivr team since that is what library authors will already be optimizing for in an ESM world in NodeJS.

type: "module"
exports map

By using these hints, jsdelivr should be able to support npm packages out of the box and even better, be able to rely on NodeJS to do all the heavy lifting for them when building / bundling for delivery over their CDN.
Just my $.02.

https://github.com/jsdelivr/jsdelivr/issues/18298


Update searching-for-repositories.md

Why:
Closes #17388
Discussion #17192

What's being changed:
content/search-github/searching-on-github/searching-for-repositories.md
Remove: When you omit this qualifier, only the repository name and description are searched.

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@gusbo010
Thanks so much for opening a PR! I'll get this triaged for review :zap:

Hi @gusbo010, thanks for opening this PR! It looks like you're right, when we don't specify an in qualifier, we default to search repository name, description and topics. Instead of removing the entire line, can we add in that topics are searched too? I have suggested the change on the commit. Thanks again!

Hey @gusbo010! Thanks for waiting. This feature is fully rolled out. Before we merge this docs change, can we add in:topics as an example in the table below where the changes we've already made are? Once we add an example in, we'll be good to go here! Thanks 😄

@kag728, @skedwards88.
Example now added. The example I added is written using the assumption that in:topic matches exactly to specified query.
I.e. "java" matches topic "java" not topic "javascript". If this is not the case it should be reworded to something else.


Check for generation not only key block in aec_sync

When identifying blocks that do not need to be synced we have to look
at the whole generation, not only the keyblock.
PT - https://www.pivotaltracker.com/story/show/159791019

Auxiliary CI workflow with system tests started on f6509de0 equivalent to 8b2a331f

@hanssv When merging, please let us merge this keeping merge commit so PR #1440 does not need further rebase.


incorporate Connections Hypothesis Provider (CHP) API as a knowledge source

SmartAPI annotation file here: https://github.com/di2ag/chp_metadata/blob/master/smartapi.yaml
Is this sufficient for inclusion in BTE?

Closing since #337 was merged...