Organize which routes are prefixed with "/api"

This PR aims to organize which routes are prefixed with /api and which are not. I moved where /api is prepended into the individual controllers and out of the routes section in the app.rs file, making all starters have the same basis. Changing the scaffold generation to prepend /api only when the -k api option is chosen, and not adding it for -k htmx and -k html (for the views) is easier to reason about.
Before this change, scaffold generation would add a route as /<resource_name> to the saas starter, but as /api/<resource_name> to rest-api and lightweight-service. Instead, the new change would have -k api add new routes as /api/<resource_name> while -k htmx and -k html add them just as /<resource_name>. In this case, whether /api is added is based on whether you are using the -k api option, and not based on the starter template you chose.

@isaacdonaldson i think this will unlock the bigger picture here: https://github.com/loco-rs/loco/issues/834

@jondot - Is there anything holding this up?

@isaacdonaldson not at all, we've just made some changes to allow this to be part of a bigger picture of simplifying the starters


CIF-2612 - cloudManagerTarget config for examples deployment

Description
Fix cloudManagerTarget config for CIF packages. Currently all CIF packages are ignores by CM, see below.
* commerce-cif-components/ui.apps/target/core-cif-components-apps-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/extensions/product-recs/content/target/core-cif-components-extensions-product-recs-content-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/examples/ui.content/target/core-cif-components-examples-content-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/examples/ui.apps/target/core-cif-components-examples-apps-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/examples/ui.config/target/core-cif-components-examples-config-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/examples/all/target/core-cif-components-examples-all-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/ui.config/target/core-cif-components-config-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/all/target/core-cif-components-all-2.4.3-SNAPSHOT.zip (skipped-content-package)
* commerce-cif-components/it/content/target/core-cif-components-it-tests-content-2.4.3-SNAPSHOT.zip (skipped-content-package)

To get the examples deployment working we need the core-cif-components-all and all example packages deployed.
Related Issue
CIF-2612
#648
Motivation and Context
Automatically deploy updated snapshot versions of CIF Core Components Library on aemcomponents.dev.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[x] All unit tests pass on CircleCi.
[x] I ran all tests locally and they pass.

https://github.com/adobe/aem-core-cif-components/pull/649 broke the automatic deployment since we also did exclude individual example packages from CM deployments.

Please don't merge. I will merge once the CM build is working stable.


Update aws-core, http-client-spi, ... to 2.17.256

Updates

software.amazon.awssdk:aws-core
software.amazon.awssdk:http-client-spi
software.amazon.awssdk:netty-nio-client

from 2.17.252 to 2.17.256.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #938.


get_inputs implementation proposal

######## Summary ########

nums.uintb

instructions: 24 (orig) vs 22 (new)
orig: 2 params, 8 slots, 1 upvalue, 5 locals, 4 constants, 0 functions
new: 2 params, 5 slots, 1 upvalue, 2 locals, 4 constants, 0 functions

nums.uintn

instructions: 37 (orig) vs 31 (new)
orig: 2 params, 8 slots, 1 upvalue, 5 locals, 7 constants, 0 functions
new: 2 params, 6 slots, 1 upvalue, 3 locals, 5 constants, 0 functions

######## Summary ########

nums.uintb

instructions: 24 (orig) vs 19 (new)
orig: 2 params, 8 slots, 1 upvalue, 5 locals, 4 constants, 0 functions
new: 2 params, 6 slots, 1 upvalue, 3 locals, 4 constants, 0 functions

nums.uintn

instructions: 37 (orig) vs 29 (orig)
orig: 2 params, 8 slots, 1 upvalue, 5 locals, 7 constants, 0 functions
new: 2 params, 6 slots, 1 upvalue, 3 locals, 5 constants, 0 functions

######## Summary ########

nums.uintb

instructions: 24 (orig) vs 19 (new)
orig: 2 params, 8 slots, 1 upvalue, 5 locals, 4 constants, 0 functions
new: 2 params, 6 slots, 1 upvalue, 3 locals, 4 constants, 0 functions

nums.uintn

instructions: 37 (orig) vs 29 (orig)
orig: 2 params, 8 slots, 1 upvalue, 5 locals, 7 constants, 0 functions
new: 2 params, 6 slots, 1 upvalue, 3 locals, 5 constants, 0 functions


Lockdown mode

Modify the verification age to a certain number of days with a command

Not necessary, can just edit a file and restart the bot if absolutely needed


Configure 'enable_filter_testing' error with supported options.

Environment Information
Linux
64-bit
autotools
Intel 18 compiler
Summary of Issue
Configure does not complete due to buglet triggered by combination of options.
Steps to reproduce the behavior
Issue with with netcdf 4.6.1.
On an X86 machine with the Intel compiler, do:
#!/bin/csh
setenv NETCDFHOME $HOME/netcdf/intel/4.6.1
setenv HDF5HOME $HOME/hdf5/intel/1.8.10/parallel

setenv CC `which mpiicc`
setenv CXX `which mpiicpc`
setenv F77 `which mpiifort`
setenv FC `which mpiifort`
setenv CFLAGS "-I${NETCDFHOME}/include -I${HDF5HOME}/include -O2"
setenv CXXFLAGS "-I${NETCDFHOME}/include -I${HDF5HOME}/include -O2"
setenv FFLAGS "-I${NETCDFHOME}/include -I${HDF5HOME}/include -O2"
setenv FCFLAGS "-I${NETCDFHOME}/include -I${HDF5HOME}/include -O2"
setenv F90FLAGS "-I${NETCDFHOME}/include -I${HDF5HOME}/include -O2"
setenv FPPFLAGS "-I${NETCDFHOME}/include -I${HDF5HOME}/include -O2"
setenv CPPFLAGS "-I${NETCDFHOME}/include -I${HDF5HOME}/include -O2"
setenv LDFLAGS "-L${NETCDFHOME}/lib -L${HDF5HOME}/lib"

./configure --enable-pnetcdf --enable-netcdf4 --enable-shared=no --with-pic=no --enable-erange-fill --enable-zero-length-coord-bound --disable-filter-testing --prefix=${NETCDFHOME} > configure.log

Here is the error that I get:
configure: WARNING: Shared libraries are disabled => --disable-filter-testing
./configure: line 18214: enable_filter_testing: command not found

Hi,  are you seeing an error in addition to the warning reported here?  I'm seeing the same error when I test 4.6.1 but it doesn't stop configure from completing successfully.
I've just checked master and it may be a moot point, as this is corrected in the upcoming release.  If you can run make after running configure, I suspect it is safe to ignore the error you are observing.

I was not concerned with the warning message, but with the command not found error.
Correction, configure does complete and the settings look correct. I am able to do make. The error is benign.
If its fixed upstream, you can close this issue.


Build/chapman39/creates codevelop default val

fixes #645

Codecov Report

Merging #646 (6d7ac2e) into develop (28b03e4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #646   +/-   ##
========================================
  Coverage    95.73%   95.73%
========================================
  Files          129      129
  Lines         8450     8450
========================================
  Hits          8090     8090
  Misses         360      360

Continue to review full report at Codecov.

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


Apoc.periodic.submit procedure was entered with no errors thrown but the procedure did not go through.

Expected Behavior (Mandatory)
That the query written within the procedure would be executed in the background.
Actual Behavior (Mandatory)
The query did not execute at all.
How to Reproduce the Problem
Simple Dataset (where it's possible)
CALL apoc.periodic.submit(
  "create-person",
  "CREATE (:Person {name: 'Michael Hunger'})"
);

Steps (Mandatory)

Write any queries that would cause a change in the database(Create, set, delete, etc) in the apoc.periodic.submit() with the name as any strings.
Observe changes(or lack thereof)

Specifications (Mandatory)
Currently used versions
Versions

OS: Windows
Neo4j: 5.25-Aura
Neo4j-Apoc: 5.25.1

Hi there! This query worked for me on those versions, do you have any more information? Can you see anything in the debug log or query logs about it?

Hi Gem! So soon after you sent your message, I checked the database, and the apoc periodic submit procedures that I had sent had just gone through, IE there is now Persons node created, so I assume the problem was fixed along the way or it solved itself in the background. However, when I tried to proceed with my project, the bug returned. Also, AuraDB don't let me check its logs as I am on the free tier.
I assume that what I was doing was causing the issue, so I should provide more context of what I was doing.
So let's say that the database has a setting node created like this.
Create (s:setting{sleepTime:1000}) return s

Let's assume that this procedure had already been executed
CALL apoc.periodic.submit(
             "example",
             'CREATE (p:Person {name: 'Michael Hunger,datePosted:datetime()'})
             MATCH (s:Setting)
             CALL apoc.util.sleep(s.sleepTime)
             DELETE p
            ')

I would like to implement a feature where the user can change how long the sleep time lasts, but if the sleep time had changed then I would need to update the procedure so the timing matches up. If the person node's datePosted plus updated sleep time is ahead of datetime(), then the following cypher will subtract the difference and make that diff the new sleep time. If the person node's datePosted plus sleep time is behind datetime(), then sleep will be done instantly and continue with the rest of the procedure. I implemented it this way because I noticed that if apoc.util.sleep was given negative numbers, it would be done instantly. The cypher is like so:
Call apoc.periodic.cancel("example")
           CALL apoc.periodic.submit(
             "example",
             'MATCH (s:Setting)
              MATCH(p:Person{name:'Michael Hunger'})
  	      with duration.between(datetime(), p.datePosted+s.sleepTime) as d,s,p
  	      CALL apoc.util.sleep(d.Seconds)
  	      DELETE p
            ')

I will update the original post accordingly.

Hi there!
I am unsure if these are your real queries or not, but there are a few issues with them.
Create (s:setting{sleepTime:1000}) return s
Should be:
Create (s:Setting{sleepTime:1000}) return s
CALL apoc.periodic.submit(
             "example",
             'CREATE (p:Person {name: 'Michael Hunger,datePosted:datetime()'})
             MATCH (s:Setting)
             CALL apoc.util.sleep(s.sleepTime)
             DELETE p
            ')

Should be; Note the the quotation around the properties, without this, it means your nodes don't have a datePosted property and that would return null, and a WITH is needed between a CREATE and a MATCH
CALL apoc.periodic.submit(
             "example",
             'CREATE (p:Person {name: "Michael Hunger", datePosted:datetime()})
             WITH p
             MATCH (s:Setting)
             CALL apoc.util.sleep(s.sleepTime)
             DELETE p
            ')

The for these ones;
Call apoc.periodic.cancel("example")
           CALL apoc.periodic.submit(
             "example",
             'MATCH (s:Setting)
              MATCH (p:Person{name:'Michael Hunger'})
  	      with duration.between(datetime(), p.datePosted+s.sleepTime) as d,s,p
  	      CALL apoc.util.sleep(d.Seconds)
  	      DELETE p
            ')

You cannot add a datetime (d.datePosted) with an int (s.sleepTime), so this also will not work.
Have you tried all the queries you are submitting in the procedure outside of the queries? It could be that the procedure is swallowing an error that the query is writing.

Hi again :)
I tested this out, and it works for me, so unfortunately there isn't much more I can do.
Just some things to note:
apoc.util.sleep takes a time in milliseconds, you have been talking about seconds here, so just want to make sure that is clear.
There are guards for the length of a running query, I am unsure how long that is in Aura, but there is always a chance if your query is too long running it gets cancelled.
If the node hasn't been deleted, try running apoc.periodic.list to see if the periodic query is still considered "non-completed" that might give you some insight into what is happening.
Another option would be to run apoc.periodic.repeat, and instead of sleeping and blocking a thread, you could then just check the time and delete if the time has passed or not, then you don't need to keep submitting new queries


EA SPORTS™ FIFA Football [PCSG00039]

App summary

App name: EA SPORTS™ FIFA Football
App serial: PCSG00039
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 2853
Commit hash: https://github.com/vita3k/vita3k/commit/a00d223e
CPU backend: Dynarmic

Test environment summary

Tested by: nishinji
OS: Windows 10
CPU: Ryzen 9 3950x
GPU: RTX 3060ti
RAM: 32 GB

Issues
Screenshots

Log
vita3k.log
Recommended labels
Nothing

Will yall ever work on getting any sports titles to rum on the android emulator? Games like madden nba 2k MLB the show fifa etc?

App summary

App name: EA SPORTS™ FIFA Football
App serial: PCSG00039
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 3043
Commit hash: https://github.com/vita3k/vita3k/commit/5d828c39
CPU backend: Dynarmic

Test environment summary

Tested by: nishinji
OS: Windows 10
CPU: Ryzen 9 3950x
GPU: RTX 3060ti
RAM: 32 GB

Issues
Screenshots

Log
PCSG00039 - [EA SPORTS™ FIFA Football].log
Recommended labels
Intro

I'm kinda slow to this lol does that mean yes your working on it sorry lol

App name: EA SPORTS™ FIFA Football
App serial: PCSB00055

state: intro

What about madden 13? It still has some graphics issues
On Mon, Jan 22, 2024, 10:11 AM AhayriSG @.***> wrote:

App name: EA SPORTS™ FIFA Football
App serial: PCSB00055
image.png (view on web)
https://github.com/Vita3K/compatibility/assets/59465544/67da4736-6ac7-4fc4-ae09-98fa75558026
state: intro
—
Reply to this email directly, view it on GitHub
https://github.com/Vita3K/compatibility/issues/952#issuecomment-1904208710,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53FZEUEYD2OBVLY2VOYRULYPZ6RHAVCNFSM6AAAAAAQ5RTY7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBUGIYDQNZRGA
.
You are receiving this because you commented.Message ID:
@.***>

Will yall ever work on getting any sports titles to rum on the android emulator? Games like madden nba 2k MLB the show fifa etc?

I successfully ran FIFA 14 and FIFA 16 on BlueStacks 5.20 without major performance issues. There's also a pull request for TouchHLE: https://github.com/hikari-no-yume/touchHLE/pull/204

Oh ok I know that fifa works smoothly with custom drivers im wondering when
will madden 13 be playable next I've used different drivers specifically
for my processor in my samsung galaxy note20 it has a few graphics issues
but has gotten better with the new versions of the emulator just wondering
will it be playable soon
On Wed, Jan 31, 2024, 9:28 AM AhayriSG @.***> wrote:

Will yall ever work on getting any sports titles to rum on the android
emulator? Games like madden nba 2k MLB the show fifa etc?
I successfully ran FIFA 14 and FIFA 16 on BlueStacks 5.20 without major
performance issues. There's also a pull request for TouchHLE:
hikari-no-yume/touchHLE#204
https://github.com/hikari-no-yume/touchHLE/pull/204
—
Reply to this email directly, view it on GitHub
https://github.com/Vita3K/compatibility/issues/952#issuecomment-1919215959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53FZEQTR7GUB5SRRHT4Y6TYRJIKHAVCNFSM6AAAAAAQ5RTY7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJZGIYTKOJVHE
.
You are receiving this because you commented.Message ID:
@.***>

as of https://github.com/Vita3K/Vita3K/commit/b5aa1ec5dd373ed7f8f65b1b7593392d4b145477, the game goes ingame with only audio sounds crackly.
Screenshots

Log
vita3k.log
Recommended labels
Playable
audio quality

What about madden 13? It still has some graphics issues
On Sun, Feb 4, 2024, 2:18 AM GamingDuck @.***> wrote:

as of @.***
https://github.com/Vita3K/Vita3K/commit/b5aa1ec5dd373ed7f8f65b1b7593392d4b145477,
the game goes ingame with only audio sounds crackly.
Screenshots
1.PNG (view on web)
https://github.com/Vita3K/compatibility/assets/90236481/63a6c636-99e7-4293-a254-70f4041f63e0
2.PNG (view on web)
https://github.com/Vita3K/compatibility/assets/90236481/b8324741-fa24-422a-b286-15a60bcefd0b
3.PNG (view on web)
https://github.com/Vita3K/compatibility/assets/90236481/da1d171c-37ab-4abf-b248-38b958c1abae
Log
vita3k.log
https://github.com/Vita3K/compatibility/files/14155868/vita3k.log
Recommended labels
Playable
audio quality
—
Reply to this email directly, view it on GitHub
https://github.com/Vita3K/compatibility/issues/952#issuecomment-1925608245,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53FZEXYQWWB4Z3HOJULXQLYR4Y55AVCNFSM6AAAAAAQ5RTY7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRVGYYDQMRUGU
.
You are receiving this because you commented.Message ID:
@.***>

PCSB00055 version using latest Vita3K build.

Intro

Download?

Hey how to downloadd

Romsfun.com
On Mon, Jun 17, 2024, 4:04 PM RONN2031695 @.***> wrote:

Hey how to downloadd
—
Reply to this email directly, view it on GitHub
https://github.com/Vita3K/compatibility/issues/952#issuecomment-2174325920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53FZEVJRQFPZQFFP3SEAHDZH46M3AVCNFSM6AAAAAAQ5RTY7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUGMZDKOJSGA
.
You are receiving this because you commented.Message ID:
@.***>

Comment le roms iso de fifa svp ?

🤯🔥


Querying remote servers with various OS

HI,
I thought that the library could detect the version of the target server but it rather checks the machine that's running the program with Library.  Do you plan to add that kind of "detection"?
Thanks.
Originally posted: 2013-02-07T08:27:04

There are a number of methods to determine the version of a remote OS. The library version on them is easy to determine: If Windows 2003 or earlier, it will only support V1.
Originally posted: 2013-02-07T11:28:30


Track title rather than album title in XML

Explanation of the changes, problem that they are intended to fix.

Show track title rather than album title on XML for songs

All Submissions:

[x] I have read the code of conduct.
[x] I am submitting to the correct base branch

Changes that modify the db structure

[x] Backwards compatibility: Users running the new code using an existing .disquetv folder will not lose their channels / settings.
[ ] I've implemented the necessary db migration steps if any.

New features

[x] I understand that the feature may not be accepted if it doesn't fit the upstream app's planned design direction. But that in this case I am encouraged to share this as an available modification other users can use if they want.

I keep forgetting this thing has a tv guide


Wizard: Fix number of dependencies shown on Review step

The number of dependencies is determined by depsolving the blueprint.
The array of dependencies does not include the blueprint packages, so
the number of packages should not be subtracted from the number of
dependencies.

One thing to think through - what should the behavior be if one of the packages in the blueprint's [[Packages]] is a dependency of one of the other packages in the blueprint's [[Packages]] list?


RF builder: Order drop down menus are non-responsive

Affects: RF builder drop down ordering thingies

Issue
Nothing happens when I select numbers in the drop down menus.

@jennyhavah When testing, selecting a number seems to cause the indicator to re-order. The only time it doesn't change is when selecting the number of the level it's currently on (e.g. selecting 1 on the first indicator causes nothing to happen).

Ack sorry, forgot I fixed this a bit ago.  They were broken.  They shouldn't be broken any more.

Working!


Add K8s 1.27.2

Related: https://github.com/rancher/rancher/issues/41840

@kinarashah Alright! I updated the version. But we need to bump rancher/kubernetes. I will raise a PR for that.

@rayandas Tagged v1.27.4 in rancher/kubernetes, could you restart drone?

@kinarashah The build was successful.


No ssl warning

Removes import of urllib.parse.  This import fails on the debian stretch systems using this module and it doesn't appear to be used in the code anyway.
Don't print the below error when ssl_verify=False.
/usr/lib/python2.7/dist-packages/urllib3/connectionpool.py:845: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
InsecureRequestWarning)

Closing PR.  See Jan 30th comment.


reattach-to-user-namespace doesn't seem to work for tmux commands when tmux is started directly (and not from a shell)

I have Terminal.app set up to start tmux directly instead of a default shell:

This means that if tmux is quit the Terminal window will be closed too. This also makes it so that all of my Terminal windows open the same tmux session (by default).
I have the following in my .tmux.conf:
set -g default-shell $SHELL
set -g default-command "reattach-to-user-namespace -l ${SHELL}"

unbind -t vi-copy Enter
bind-key -t vi-copy Enter copy-pipe "pbcopy"
bind-key -t vi-copy y copy-pipe "pbcopy"

I have reattach-to-user-namespace (2.5) installed with the --with-wrap-pbcopy-and-pbpaste option. I've also got the latest tmux (2.3) installed.
It works perfectly when calling the pbcopy wrapper directly from my shell (inside tmux), copying in vim, etc. However when using copy mode in tmux itself it seems that reattach-to-user-namespace doesn't do anything. It works silently. I've debugged this by adding a custom wrapper script to debug that copy-pipe is working correctly as the buffer gets passed to the command as expected.
When I check Run inside shell in the Terminal Shell preferences (above screenshot) copying from tmux copy mode is working as expected too. This makes me think that there is some issue with it if tmux isn't started from a shell itself.
I skimmed through the source code but C isn't my strong point and I couldn't think of any reason at first sight. For now using Run inside shell makes it work but I'd like to disable it again as it means running one less shell. Enabling it also breaks my flow with multiple Terminal windows as when it is enabled the tmux sessions remain open when closing the Terminal windows.

Where does Homebrew(?) install those wrapped pbcopy and pbpaste? Are you sure your copy-pipe command is using the wrapped versions?
Using bare command names in copy-pipe will end up using the tmux server’s PATH (tmux show-environment -g PATH), which may not include the wrapped versions but likely does include the system (non-wrapped) versions.
You might try setting copy-pipe to use the full path to the wrapped version of pbcopy.
GUI apps usually have a minimal PATH (e.g. /usr/bin:/bin:/usr/sbin:/sbin), so a tmux server started directly by Terminal would inherit that basic PATH and not “see” the wrapped version of pbcopy. On the other hand, a tmux server started from one of your “usual” shells will inherit a PATH that has probably been extended (by your shell’s startup configuration) to include the wrapped pbcopy (i.e. the directory where your wrapped version is installed).

They're installed as /usr/local/bin/pbcopy and /usr/local/bin/pbaste. I forgot to mention it in the issue but I've also tried using them directly. I've also tried using reattach-to-user-namespace /usr/bin/pbocpy but none of them worked.
I ended up adding a custom script with the following code when debugging:
#!/bin/sh

cat - > /tmp/buffer

echo foo | /usr/local/bin/pbcopy
reattach-to-user-namespace "echo bar | /usr/local/bin/pbcopy"
reattach-to-user-namespace "echo baz | /usr/bin/pbcopy"

Nor foo, bar or baz was added to the clipboard. When calling that script directly from a (zsh) shell within tmux works as expected. The contents of tmux's buffer was written in /tmp/buffer as expected.

You might try putting exec >/tmp/copy-pipe.out 2>&1 at the top of that debugging script; having that in there will let you see any stdout or stderr output from the shell running that script and the executables it runs.
I would have expected the foo test to work, but the bar and baz tests are faulty. The wrapper uses exec, not system, so giving it shell-based command strings will not do what you want. Try (e.g.)

reattach-to-user-namespace sh -c "echo baz | /usr/bin/pbcopy",
or (without and extra shell invocation) echo baz | reattach-to-user-namespace /usr/bin/pbcopy instead.

You might also need to use an absolute path to the wrapper since the PATH may not include /usr/local/bin (if you capture to copy-pipe.out, you can add echo "PATH: $PATH" to see its value in the script; also you would probably see something like “command not found” from the shell).

Thanks. I didn't know about exec >/tmp/copy-pipe.out 2>&1. By adding that I found out that it indeed couldn't find reattach-to-user-namespace in the provided PATH as it only contained /usr/bin:/bin.
Instead of using the pbcopy wrapper provided by Homebrew (which uses reattach-to-user-namespace without providing an absolute PATH I've switched to using both tools directly with their absolute paths:
bind-key -t vi-copy Enter copy-pipe "/usr/local/bin/reattach-to-user-namespace /usr/bin/pbcopy"
bind-key -t vi-copy y copy-pipe "/usr/local/bin/reattach-to-user-namespace /usr/bin/pbcopy"

Thanks a lot for helping me investigate this. It helped me to better understand some of the things going on. :) 👍


Page about seed's correction with cool animations

Anna (volunteer) is sending me some lifecycle stuff for the website to meet the education criteria. Perhaps once "seeds of doubt is complete" we can add a html (easy-read) version of it?


chore: removes experimental guards from custom domains cmds

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 56.525%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
3383

Relevant Lines:
5985

💛  - Coveralls

The UI to self-service is being released this week. We can hold off until after if preferred.

I misspoke -- it's already on prod.


duplicating all unit/integration tests for aws plugin for legacy-aws plugin

          This fixes related issue, but I would also suggest duplicating all unit/integration tests for aws plugin for legacy-aws plugin

Originally posted by @mateusoliveira43 in https://github.com/openshift/oadp-operator/pull/1568#pullrequestreview-2392285841

Original implementation PR added some unit tests. I'm not sure we want to duplicate all tests, but it wouldn't hurt to look at the current unit tests and determine whether more are needed for legacy-aws.


Should superjson be a dev or prod dependency?

Should superjson be a dev or prod dependency?
Originally posted by @nemanjam in https://github.com/blitz-js/babel-plugin-superjson-next/issues/19#issuecomment-1019099240

I'd put them into prod dependency, since SuperJSON will need to be present in your application bundle in order for things to work. But considering that you're most likely using a bundler anyways, it won't make that much of a difference: https://jsramblings.com/do-dependencies-devdependencies-matter-when-using-webpack/

Thanks.

Thanks.


[Refactor] Tokenizer refactor continued

The next stage of refactoring Tokenizer; start having the product code conform to the tweaked Tokenizer class. Included in this:

Moving _joint_tokenize() onto the Tokenizer (as recode()), since it really belongs there
Making the Tokenizers on the 'grammarless' side of things conform to the same contract (i.e. deal with bytes not strings)
Added some MyPy annotations, and fixed a few issues

Codecov Report
Attention: Patch coverage is 83.87097% with 10 lines in your changes missing coverage. Please review.

Project coverage is 59.33%. Comparing base (0e0417e) to head (a79afb7).

Files
Patch %
Lines

guidance/models/transformers/_transformers.py
68.75%
10 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #894      +/-   ##
==========================================
+ Coverage   56.99%   59.33%   +2.34%
==========================================
  Files          64       64
  Lines        4655     4665      +10
==========================================
+ Hits         2653     2768     +115
+ Misses       2002     1897     -105

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Not quite sure what I did to get this fixed, but my local runs of the Azure OpenAI tests now pass. Will probably merge tomorrow, absent objections

I really like moving the joint_tokenize into the Tokenizer class and the unification around the byte string contract! Encode/decode/recode is also a very nice paradigm. Love it
I'm admittedly more skeptical about the rest of the renaming/typing/re-formatting pieces here. Mypy feels like it might be causing more headaches than the documentation benefits it brings, and black's odd multi-lining is still a bit off.
My preference is to err on the side of minimalism as much as we can. Perhaps we can open up a separate discussion on this? A bit of an aside, but I've really enjoyed Raymond Hettinger's approach/philosophy to these things (e.g. https://www.youtube.com/watch?v=wf-BqAjZb8M).
That said, back to the PR -- I'll reiterate that the core changes are really nice and definitely a step towards improving the Tokenizer/Model/Engine relationships here. Thanks for taking a stab at understanding and unifying this @riedgar-ms!

@riedgar-ms
Big +1 to the endcode/decode/recode approach here!
@Harsha-Nori re: formatting, type hints, etc...
I agree that black sometimes makes things ugly. PEP 8 isn't a god we should all bow to. On that note, neither is mypy...
+1 for Raymond Hettinger. He'll keep me company this morning while I drink my coffee and catch up on some things I missed this weekend.
On the mypy stuff -- I'm actually a big fan of type hinting. Python isn't and almost certainly never will be statically typed, but a bit of static type checking sprinkled on top can save so many headaches...
Slight philosophical digression, but you can solve a lot of physics problems with dimensional analysis alone. Back when I used to write a lot of torch and tensorflow, my first step when doing any kind of debugging was to "look at the shapes" (I have a *B x N x T tensor... I'm contracting it with a T x D tensor...). Reasoning about types similarly gives a nice skeleton for starting to reason about what some code is doing before you have to start thinking about the actual logic.
A lot of the engine code is pretty opaque at the moment, and I think that not having a "data model" to fall back on for reasoning about this stuff is making the problem worse than it needs to be. I think that most of the hints that @riedgar-ms added here are the helpful kind (encoding a byte string gives a sequence of ints, decoding a sequence of ints gives a byte string, recoding a sequence of ints gives a sequence of ints, ...).
That being said, sometimes I do find myself fighting against mypy. Annotations like tuple(int, tuple(int, Sequence[int], float), bool) don't provide any insight (but a lightweight NamedTuple data model can help in these situations...).
I'm rambling now. But TLDR: I think type hints are usually helpful, and I don't see anything egregious in this diff. @Harsha-Nori any particular use of typing that rubbed you the wrong way here?

That being said, sometimes I do find myself fighting against mypy. Annotations like tuple(int, tuple(int, Sequence[int], float), bool) don't provide any insight (but a lightweight NamedTuple data model can help in these situations...).

I would agree that annotations like that point to a missing type :-)

Ah sorry, I was rambling myself earlier! It's less about any specific hints here -- these are all fine :) -- and more about anticipated fights with the build system and mypy hiccups if there's some detection or misspecification problem. I've heard from several people now that getting mypy to behave was a significant pain. Might just be a short term pain thing though!

So.... is that a signoff?

Looks good to me -- just made a few little comments. Thanks for putting in work do disentangle these classes.


Feature/port selection

This pull request adds a new feature that allows the backend server's port to be specified via the HOST_PORT environment variable. If not set, the server defaults to port 8080. This change enhances deployment flexibility without affecting existing setups. Users can set the port by exporting HOST_PORT before launching the application. Tests confirm the feature works as expected in various environments. Please review and provide feedback.

Merge PR#1


WIP: Feature/difference method

Closes #48

@arnellebalane Thank you for your review. Updates available at 408bfe7


Error: EACCES: permission denied, mkdir '/__w/{repo_name}/{repo_name}/cypress/videos'

I'm facing the following problem on GitHub Actions when the job reaches this line of code :  npx cypress run -P . -C cypress.json -r spec  --record. The below error message is thrown and the workflow failed ( Error: Process completed with exit code 1)
Warning: We failed to record the video.

This error will not alter the exit code.

Error: EACCES: permission denied, mkdir '/__w/{repo_name}/{repo_name}/cypress/videos'

I didn't finf any fix of this on the web, can someone help for that ?
PS: I used the Cypress image : cypress/browsers:node16.5.0-chrome94-ff93

to sure if this is related to #446

@Develliot related possibly, but the fix proposed does not address it :(

For some reason the default video storage path gets screwed up in the Docker image. It seems to point to an unwritable location rather than somewhere it can work with the video compression.
If you force the videoStorage config in your .yml file to something usable it seems to work:
      - name: 'E2E Tests - Chrome'
        uses: cypress-io/github-action@v4
        with:
          install: false
          start: npm run cypress:record
          browser: chrome
          record: true
          parallel: true
          group: 'E2E - Chrome'
          config: videosFolder=/tmp/cypress-videos        # <-- Fix for https://github.com/cypress-io/github-action/issues/480

This one can be closed if you wish, I found a work around elsewhere.


Add input completion test suite

Description
Adds tests for InputCompletionPopup and CompletionModel behavior as a precursor to merging #4639. While developing these tests I identified two discrepancies in behavior from #4639 which I will fix (eventually).
This PR needed to make some changes to BttvEmotes (among others) to allow directly setting loaded emotes for mocking. It looks a bit weird but it seems necessary.
Benchmarking
I've all but finished writing benchmarks for the existing input completion code. However, I can't figure out a smart way to access tests/src/mocks from both tests and benchmarks. The code should obviously be shared. I tried moving it to src but that would require linking gtest into chatterino-lib (I think), which I don't understand the full implications of (nor would be I able to modify the cmake config to do so).
If anyone (read: pajlada) can think of a way to share the mocks between tests and benchmarks and how to configure CMakeLists.txt (as I also can't figure that out when simply copying tests/src/mocks to benchmarks/src/mocks), I'd be very grateful :)

I'll note that this PR does not include completion tests for username nor commands. Testing these would require a fully mockable TwitchChannel instance because the completion code relies on a dynamic cast.
This is one of the advantages of the new system in the pending PR as strategies are testable outside of sources.

I made the mock change to make them accessible for both tests & benchmarks here https://github.com/Chatterino/chatterino2/pull/4645


corrected typo in the issue template

While you're at it, mind changing "Describe what the goal." to "Describe the goal."?

will do, funny I noticed that and then it slipped my mind

@dghubble I'll squash those later

Yes please

Merged as d48f88cfd6b5ec12a50cad119418f87a6d263a54 (to fix typo in commit message)


How to use this

I Have downloaded this. But I don't know how to use this. Please hep me

You run it as a Java application.

See https://github.com/jgraph/svg2xml/


allow restriction of content access via owner/repo configuration

It should be possible to limit content rendering to a set of trusted repositories via the /.helix/config.json (config-all).
possible configuration names:

my favourite is: trust.repository

I'd say you demand or require the repository

I'd say you demand or require the repository

so maybe:

key
value

require.repository
adobe/*

or maybe merge it into access ?

key
value

access.require.repository
adobe/*

access.allow
<EMAIL_ADDRESS>

access.allow
<EMAIL_ADDRESS>


UTF8 vs utf8mb4

is there any greater vulnerability to using utf8mb4 than utf8? Also will unicode automatically map to utf8mb4 for mysql / percona / mariadb?
How well does this work with other DBMS?

http://stackoverflow.com/questions/134099/are-pdo-prepared-statements-sufficient-to-prevent-sql-injection/12202218#12202218 suggests that if utf8 is forced (so no optional charset), then emulate prepares is no longer an issue (If I have understood correctly)

If you're using MySQL, use utf8mb4, not utf8.

I Just went through this looking for the reason I was even asking... I think it is this line
https://github.com/paragonie/easydb/blob/master/src/Factory.php#L37
I'm unsure if it can even be utf8mb4 as it's the connection string. IDK we can close if helpful


Mocking an error and using retry()

I'm trying to do the following

Get a fixture to return a server error (500)
Use retry(3) on the superagent
See that the fixture is called 3 times, and the get request ultimately fails.

I've written a small test script, from what it looks like the failure happens immediately after the exeption even though the retry callback is called 3 times afterwards.
Here's my script
var request = require('superagent');
var config = [{
  pattern: 'http://localhost/500/devices',
  fixtures: (match, params) => {
    let err = new Error(500);
    err.status = 500;
    throw err;
  },
  get: (match, data) => {
      return data;
  }
}];

var superagentMock = require('superagent-mock')(request, config);

request
  .get('http://localhost/500/devices')
  .retry(3, retryCallback)
  .then(res => {
    console.log('completed');
   	console.log(res);
  })
  .catch(err => {
    console.log('failed');
    console.log(err);
  });

function retryCallback(err, res) {
  console.debug('__retrying');
  return true;
}

Output is
failed
__retrying
__retrying
__retrying
Is there an error with how superagent-mock interacts with retry or am I approaching this the wrong way?

Hi, did you test to throw the error in the get method instead in the fixtures?

Yes, the result is the same.

I think the problem is with superagent-mock that don't handle superagent's retry method. We don't use the retry feature of superagent, but feel free to open a PR to fix this bug 😉

I'll try and work it out :)


Extracted jsons from evidence_strings.py to separate files.

Coverage increased (+0.2%) to 68.377% when pulling d9312a55f29dad7d5610a09f82f015714eb1fab1 on tomdcsmith:ev_string_base_json_file into ff6a2d880ddba041807f188b7bc81e6fb8a77f37 on EBIvariation:master.

Coverage increased (+0.1%) to 70.51% when pulling 27b2d8700cec552e24cb9c3afcbf038617454883 on tomdcsmith:ev_string_base_json_file into 703f27a84cb367bcaaa61c25659990e2292e9022 on EBIvariation:master.


Update UpsertTracepoint and bpftrace deployment documentation

Summary: Documents the new TraceProgram object added to pxtrace which enables users to specify deployment restrictions for BPFtrace programs. Changes introduced in https://github.com/pixie-io/pixie/pull/1687. Should not be merged until those changes land.
Related issues/PRs: https://github.com/pixie-io/pixie/issues/1659

@pixie-io/admin this is ready for review.

Also wanted to note that the necessary vizier changes are included in the current release (v0.14.7).


Error in destruction of rcl subscription handle: Failed to delete datareader

Hello, I got some problems in operating.
My command is ros2 run scout_base scout_base_node'
Then below error  occurred.
cannot publish data, at ./src/rmw_publish.cpp:62 during '__function__' [ERROR] [1715141327.920305562] [scout.rclcpp]: Error in destruction of rcl subscription handle: Failed to delete datareader, at ./src/subscription.cpp:52, at ./src/rcl/subscription.c:184 cannot publish data, at ./src/rmw_publish.cpp:62 during '__function__' Fail in delete datareader, at ./src/rmw_service.cpp:104 during '__function__' [ros2run]: Segmentation fault
Do you know how to fix it?

Hello, I got the same problem here. Did you fix it now?


production's files for browsers (bower.json)

because development files (ex: .scss, .coffee, ...)  isn't actually good for browsers to digest.

useful for project which behavior like this one; considering this conversation bower/bower#2312

As far as I can see, this isn't part of the bower spec, and the referenced project is tiny with no visible traction.


respect the maxLength property

it is me again 😆
the virtual keyboard should respect the maxLength property like normal keyboard

warning in normal situation MaxLength = -1
it can generate bug with const inputMaxLength = this._activeInputElement?.maxLength ?? 0; you have to test is this._activeInputElement?.maxLength > -1


Exception when i2c display is not connected during init

The following exception is thrown when i2c display support is enabled but the display is disconnected.
I don't have an SPI display to test if the issue happens there.
E (162) I2CDefaultWriteBytes: i2c_master_cmd_begin( I2CPortNumber, CommandHandle, pdMS_TO_TICKS( 1000 ) ) != ESP_OK, result: -1
abort() was called at PC 0x400e063a on core 0
0x400e063a: I2CDefaultWriteBytes at components/tarablessd1306/ifaces/default_if_i2c.c:107 (discriminator 4)

The issue can be resolved by changing line default_if_i2c.c:107, but I am unsure if this is the best practice or other implications of this change:
i2c_master_cmd_begin( I2CPortNumber, CommandHandle, pdMS_TO_TICKS( 1000 ) );

It's an acceptable workaround and raises a good issue about how I handle such errors.
The library assumes that if you initialize a display it's physically connected which is why I chose at the start for it to be a fatal error.
If you're looking at the examples you can comment out USE_I2C_DISPLAY and it will only use an SPI connected display instead,

Consider a piece of hardware using this library in the field.
If the screen becomes disconnected or damaged it prevents the whole system from coming up. In my opinion it is preferable the rest of the hardware continue operating as normal if possible, the alternative is a reboot loop.
Maybe another constant for REQUIRE_DISPLAY_ON_BOOT as it could be application specific.

OOOH I think I know what's going on.
I did make it a configurable option at one point, SSD1306_ERROR_ABORT is set by default so any time the library encounters an error it will call abort.
You can change this behaviour in menuconfig under Components->TarableSSD1306, let me know if it works and I apologize for the slowness of my reply.

hi, with SSD1306_ERROR_ABORT deactivated, the SSD1306_I2CMasterAttachDisplayDefault() returns true if the display is disconnected, and only printing the error on the console:
E (21938) I2CDefaultWriteBytes: i2c_master_cmd_begin( I2CPortNumber, CommandHandle, pdMS_TO_TICKS( 1000 ) ) != ESP_OK, result: -1

The right behavior would be to return false if the display failed to initialize.

Thank you for letting me know.
I'm working out how I want to fix this.

Try the latest commit.


fixed remove-number.js

wrote the logic for removeNumber()
fixed removeNumberHandler()
close #11

I think, the idea was to create a new Array, not modifying the old one. In this case if I'm not mistaken, you are changing the existing one, right? @yildiraykoyuncu @abelRoland

Not exactly. Actually removeNumber() returns a new array. Because its what .filter()method does. But when we call theremoveNumberHandler()it assigns the return value of removeNumber()tonumbersarray. Practically we are changingnumbersarray with a new one we are not mutating it. I wrote it that way because every time we call a handler function it callsdisplayArrayHandler()at the end, and it displays the final status ofnumbersarray so we can see the numbers we add or we remove. So I thought that we should be adding an removing numbers tonumbersarray. We can declare a new array every time we call the handler and assign the return valueremoveNumberto that new array and display it. It can work that way too. I can change it if you like. We can lost the track of arrays. I mean when we calladdNumberit will return a new array,numbersarray will stay the same. Then when you want to remove a number, the function will getnumbers array and remove a number from it and return a new array but you wont be able to remove a number you add before because numbers array always stays the same it won`t be updated.

 JS is not easy for me, so it is nice to learn from each other! well done)

You are welcome!! 👍 😄


Mismatched sourcesContent

When I'm bundling a module that references two location.js files (in different directories), a Mismatched sourcesContent error is thrown at https://github.com/systemjs/builder/blob/master/lib/sourcemaps.js#L54. It appears this is becausemap.sourceRoot is undefined which results in the source variable having no path component. map.sourceRoot is only undefined for .js files loaded via plugin-babel inside my local app package. Files loaded via the text plugin and js files loaded from external packages are prefixed with their paths. I've included some of the source values I've observed below:
....
source: za-provinces.js
source<EMAIL_ADDRESS>source<EMAIL_ADDRESS>source<EMAIL_ADDRESS>source: home.js
source<EMAIL_ADDRESS>source<EMAIL_ADDRESS>source: app/directives/phone-number/phone-number.html
source: phone-number.js
source: app/components/public-root/public-root.html
source: public-root.js
source: app/modals/location-modal.html
...

My local packages config is as follows:
packages: {
    "app": {
      "main": "app.js",
      "meta": {
        "*.js": {
          "loader": "plugin-babel"
        }
      }
    }
  }

I'm using plugin-babel 0.11, builder 0.15.18, jspm 0.17.0-beta-16.

I guess this is because the angular-ui-router.js entry comes up twice. It would be interesting to know why that is as well. Any way you can share an exact test case here for me to look into it further?

The problem seems to be fixed with 0.15.18. However, I still see duplicate entries for angular-ui-router.js and other files. This appears to happen only for a bundle of common files that I create manually using bundle arithmetic. I do this by adding multiple trees together via builder.addTrees() and then removing references (via delete agregateTree[key];) that are referenced less than X times. The resulting tree is then bundled.
I've not really had time to create a test case. I'll try find some time for that over the weekend.

Thanks for the follow-up - please do share a further test case if you can.

I've looked into it some more and it appears that the references to angular-ui-router.js where in separate bundles. So the 2nd issue was a non bug.


Docker is not working

I am running actions-runner-controller-0.2.0 installed via Helm and getting the below error
ERROR: failed to create cluster: failed to init node with kubeadm: command "docker exec --privileged chart-testing-control-plane kubeadm init --ignore-preflight-errors=all --config=/kind/kubeadm.conf --skip-token-print --v=6" failed with error: exit status 137

I have enabled Docker:
apiVersion: actions.summerwind.dev/v1alpha1
kind: RunnerDeployment
metadata:
  name: gha-runner-autoscaler
spec:
  template:
    spec:
      organization: myorg
      dockerEnabled: true
      resources:
        limits:
          cpu: "1.0"
          memory: "1Gi"
        requests:
          cpu: "0.5"
          memory: "0.5Gi"
---
apiVersion: actions.summerwind.dev/v1alpha1
kind: HorizontalRunnerAutoscaler
metadata:
  name: gha-runner-autoscaler
spec:
  scaleTargetRef:
    name: gha-runner-autoscaler
  minReplicas: 1
  maxReplicas: 10
  scaleDownDelaySecondsAfterScaleOut: 60
  metrics:
    - type: PercentageRunnersBusy
      scaleUpThreshold: "0.75"
      scaleDownThreshold: "0.3"
      scaleUpFactor: "1.4"
      scaleDownFactor: "0.7"

k version --short
Client Version: v1.20.2
Server Version: v1.18.9-eks-d1db3c

@esvirskiy Sorry, but I don't quite get why we can see this as docker not working, as the error message seems to indicate that kubeadm within the container is failing.
Would you mind confirming if a vanilla docker command like docker run work or not? That can help us further investigate the cause of the error.


A lot of error messages

I keep getting these emails, but usually for only user at a time.  This one was a lot.  Can you tell me what's going on??  I can forward you the email if it would help.

You are receiving this email because the following debugging events triggered one of your configured Alert Triggers.ErrorTime30005 - Unknown destination handset2019-05-15 20:20:47 UTC30005 - Unknown destination handset2019-05-15 20:20:45 UTC30005 - Unknown destination handset2019-05-15 20:20:42 UTC30005 - Unknown destination handset2019-05-15 20:20:36 UTC30005 - Unknown destination handset2019-05-15 20:20:35 UTC30005 - Unknown destination handset2019-05-15 20:20:33 UTC30005 - Unknown destination handset2019-05-15 20:17:52 UTC30005 - Unknown destination handset2019-05-15 20:17:51 UTC30005 - Unknown destination handset2019-05-15 20:17:50 UTC30005 - Unknown destination handset2019-05-15 14:36:37 UTC
Error
Time
30005 - Unknown destination handset
2019-05-15 20:20:47 UTC
30005 - Unknown destination handset
2019-05-15 20:20:45 UTC
30005 - Unknown destination handset
2019-05-15 20:20:42 UTC
30005 - Unknown destination handset
2019-05-15 20:20:36 UTC
30005 - Unknown destination handset
2019-05-15 20:20:35 UTC
30005 - Unknown destination handset
2019-05-15 20:20:33 UTC
30005 - Unknown destination handset
2019-05-15 20:17:52 UTC
30005 - Unknown destination handset
2019-05-15 20:17:51 UTC
30005 - Unknown destination handset
2019-05-15 20:17:50 UTC
30005 - Unknown destination handset
2019-05-15 14:36:37 UTC

30005 - Unknown destination handset | 2019-05-15 20:20:47 UTC
30005 - Unknown destination handset | 2019-05-15 20:20:45 UTC
30005 - Unknown destination handset | 2019-05-15 20:20:42 UTC
30005 - Unknown destination handset | 2019-05-15 20:20:36 UTC
30005 - Unknown destination handset | 2019-05-15 20:20:35 UTC
30005 - Unknown destination handset | 2019-05-15 20:20:33 UTC
30005 - Unknown destination handset | 2019-05-15 20:17:52 UTC
30005 - Unknown destination handset | 2019-05-15 20:17:51 UTC
30005 - Unknown destination handset | 2019-05-15 20:17:50 UTC
30005 - Unknown destination handset | 2019-05-15 14:36:37 UTC

Is the email from twilio? I'm guessing the number isn't textable or it is blank/missing.
It may be related to this error (reported by honeybadger): https://app.honeybadger.io/projects/59936/faults/48746325#notice-summary
We've got a branch that refactors notification logic to ensure the above error does not happen: https://github.com/OfficeMomsandDads/scheduler/pull/117
@dinjas has been working on that off-and-on on Friday's.

Yes, it was from Twilio, but I checked the numbers in our database, and they seem correct.  Could it be the cell service carrier?

As discussed on our call earlier this week:

Sarah has unsubscribed from these notifications
Twillio indicates that the problem is that these phone numbers cannot receive text messages
The solution is to validate phone numbers upon signup/update. We've opened an issue for that here: https://github.com/OfficeMomsandDads/scheduler/issues/200

With that, I feel this issue can be closed!


Fix CustomMessageListener class, and update example and documentation

This is an implementation of the changes mentioned in #2195 which therefore can be closed.
Content of the PR
In previous commits there was the CustomMessageListener class implemented, to add type hinting to the message listeners.
This PR is fixing the bug of not being able to utilise this typehinting class. It makes sure to enable users to use the @CustomMessageListener decorator for their listener functions, while still being backwards compatible.
Changes made
In runners.py
class CustomMessageListener(Protocol):
    @abstractmethod
    def __call__(self, environment: "Environment", msg: Message) -> None:

to
class CustomMessageListener:
    def __init__(self, f):
        self.f = f  # Store the function

    def __call__(self, environment: "Environment", msg: Message) -> None:
        self.f(environment=environment, msg=msg)

Additional Changes:
Updated the custom_messages.py example and the running-distributed.rst documentation file.

ping @nathan-beam, @cyberw

Looking at the updated example I'm not sure about this. This is different from other Locust event handlers work, and I'm not sure if the added complexity (importing a class, tagging the method) is worth it. But lets have Nathan weigh in.

Okay :) then let's wait.
Just to be on the same page: it is fully backwards compatible. So you can also just use it without the decorator. It's just for the type consistency.
So the other side would be completely deleting the typehint?
For me it was very confusing to have this not working as expected type hint :)

Right, we shouldnt have broken type hints either, so thats not an option :)

So any news about this? :) Just wanna check.

I guess @nathan-beam is asleep. I dont like having to introduce a decorator to the examples. There should be some more user friendly solution. But I’m afk, and probably wont really have the energy once I get back :)

Okay, so rn I don't have another solution for this. I honestly think that the decorator is kinda the cleanest method, as it also is not breaking any existing code.
Then I would propose to remove the CustomMessageListener class completely. That way it will be type consistent again.
I can prepare a PR, if you want that. :)

I think that might be a good idea. That class never really made sense to me :) If you make a PR I’ll be sure to review it.


OrbStack  auto start when open vscode

Describe the bug
when open vscode  project OrbStack 1.3.0 will auto start
To Reproduce
No response
Expected behavior
No response
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.3.0
Commit: c7cc1bd78adc76441a91befa2a50c0119c9d4703 (v1.3.0)
System info:
macOS: 13.6.3 (22G436)
CPU: amd64, 16 cores
CPU model: Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
Model: MacBookPro15,1
Memory: 32 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-01-15T05-35-51.843451Z.zip

Screenshots and additional context (optional)
No response

I have had the same issue and It has been probably related with vscode's extension of C/C++.

@inniestar97 Could you share the setup which is able to reproduce the issue? Thanks.

Setup
Orbstack Info:

Version: 1.3.0 (16556) (stable)

VSCode Info:

Version: 1.85.2
Commit: 8b3775030ed1a69b13e4f4c628c612102e30a681
Electron: 25.9.7
ElectronBuildId: 26354273
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.2.0

VSCode's C/C++ extension Info

Version: 1.18.5 (Released Version)

After I remove all extension but only Microsoft's C/C++ extension, Orbstack automatically has been launched.
That is why I have thought that C/C++ extension is the reason of this bug.

@inniestar97 Are there any projects opened when VSCode starts? Are these projects set up to use remote ssh development or dev containers?

@slinorb
Conditions under which this issue occurs is just that If any of the projects in the vscode have a file related to cpp or c.
Not only use "remote ssh development" or "dev container" but just local file project.

if I comment out the content in "${HOME}/.orbstack/ssh/config", the OrbStack is not automatically run.

@inniestar97 Are there any projects opened when VSCode starts? Are these projects set up to use remote ssh development or dev containers?

Only open C/C++ project OrbStack  automatically start, for example, start VSCode  and not open project,  then open a c/c++ project from "File" -> "Open Recent", OrbStack will automatically start, if open a Python Project,  OrbStack will not start automatically,  all projects are local file project, not use ssh

@AntonyM55
Yup, that is correct.

@inniestar97 Are there any projects opened when VSCode starts? Are these projects set up to use remote ssh development or dev containers?

Only open C/C++ project OrbStack automatically start, for example, start VSCode and not open project, then open a c/c++ project from "File" -> "Open Recent", OrbStack will automatically start, if open a Python Project, OrbStack will not start automatically, all projects are local file project, not use ssh

Yup, that is correct.

We are able to reproduce and are working on a fix. Thanks for the info on how to reproduce it.

fixed in  Version 1.4.0 (16636)

I have had the same issue and It has been probably related with vscode's extension of C/C++.

On my MacBook, it's related with vscode's extension of remote ssh but not C/C++.  Maybe same with #884 .

fixed in Version 1.4.0 (16636)

This problem still exists in version 1.4.3 on my MacBook.

I am also still experiencing this issue with OrbStack 1.4.3 running on MacOS 14.3.1 (23D60).
@slinorb

This was fully fixed in v1.5.0 (#1039)


Detailed steps to use rubyc to create sharable ascli executable

HI,
Would you please write down the different commands to execute to get first ascli, then get rubyc, then create a sharable ascli executable with rubyc. Please be as detailed as possible.  Please list all the different commands to run to create the ascli executable on LINUX first then on windows. But Linux first please.

https://github.com/IBM/aspera-cli/issues/151


Hook in correctly to FAWE so that additional block-behaviour works

Currently stuff like BlockyLight component is naturally ignored as the entire "support" is just a string replacement
Any block that was placed with FAWE before this is done can simply be //replace'd after so

Closed as of #63