Commands drag and drop reordering doesn't work?

Before submitting your issue, please make sure that you've checked all of the checkboxes below.

[x] You're running the latest release version of Deployer.
[x] Ensure that you're running at least PHP 7.0.8, you can check this by running php -v
[x] You've ran composer install --no-dev from the root of your installation.
[x] You've ran npm install --production from the root of your installation.
[x] You've ran rm -rf bootstrap/cache/* from the root of your installation.
[x] You have restarted the queue listener and node socket server.

Describe the bug
List of commands don't move when you try to drag and drop them.
Expected behaviour
It should be possible to change order of commands using drag and drop
Actual behaviour
When you hover your mouse over a command it changes to a finger cursor but you can't move anything.
Steps to reproduce
Go to you project >_Commands, select Step to Configure, add commands and try to reorder them
Screenshots

Environment info
REBELinBLUE/deployer Latest version 0.0.47
Windows 10 64bit
I've tried in Chrome 70.0.3538.77, Firefox 63.0.1, in IE 11 the list of commands don't show up at all (but who cares)

Laravel Environment

Laravel Version: 5.4.36
Timezone: Europe/London
Debug Mode: ✔
Storage Dir Writable: ✔
Cache Dir Writable: ✘
Decomposer Version: ^1.0
App Size: 203 MB

Server Environment

PHP Version: 7.1.20-1+ubuntu16.04.1+deb.sury.org+1
Server Software: nginx/1.10.3
Server OS: Linux deployer 4.4.0-1070-aws #80-Ubuntu SMP Thu Oct 4 13:56:07 UTC 2018 x86_64
Database: mysql
SSL Installed: ✔
Cache Driver: file
Session Driver: file
Openssl Ext: ✔
PDO Ext: ✔
Mbstring Ext: ✔
Tokenizer Ext: ✔
XML Ext: ✔
Curl Ext: ✔
GD Ext: ✔
JSON Ext: ✔

Installed Packages & their version numbers

andywer/js-localization : dev-laravel-5
backup-manager/laravel : ^1.1
creativeorange/gravatar : ~1.0
cweagans/composer-patches : ^1.6
doctrine/dbal : ~2.5
fideloper/proxy : ^3.1
graham-campbell/binput : ~3.3
htmlmin/htmlmin : ~4.5
guzzlehttp/guzzle : ^6.2
intervention/image : ^2.3
laracademy/interactive-make : ^1.1
laravel-notification-channels/hipchat : ~0.2
laravel-notification-channels/twilio : ~1.0.2
laravel-notification-channels/webhook : ~1.0.2
laravel/framework : ^5.4
laravel/tinker : 1.0.*
lubusin/laravel-decomposer : ^1.0
mccool/laravel-auto-presenter : ^5.0
melihovv/laravel-log-viewer : ^1.2
michele-angioni/multi-language : 0.3
pda/pheanstalk : ~3.1
pragmarx/google2fa : ^1.0.0
predis/predis : ~1.0
tymon/jwt-auth : ~0.5
rebelinblue/laravel5-zxcvbn : dev-master
version/version : ^2.2

Extra Information

proc_open enabled : ✔

Logs (see storage/logs/) or other output that would be helpful
No console errors in any browser

This is working for me on 0.0.46 so not sure about 0.0.47


Add tooltips describing a Safe App's functionality

Description
Although there is a small description per app on the main app screen, a tooltip with more detailed info about what each app does could come a long way in terms of accessibility for end users.
This could be something specially useful for third party apps as their description can be a bit cryptic, providing 0 functional feedback.

This can be considered close with the new drawer implementation
https://github.com/safe-global/web-core/issues/1414


[SPARK-47196][CORE][BUILD][3.4] Fix core module to succeed SBT tests

What changes were proposed in this pull request?
This PR aims to fix core module to succeed SBT tests by preserving mockito-core's byte-buddy test dependency.
Currently, Maven respects mockito-core's byte-buddy dependency while SBT doesn't.
MAVEN
$ build/mvn dependency:tree -pl core | grep byte-buddy
...
[INFO] |  +- net.bytebuddy:byte-buddy:jar:1.12.10:test
[INFO] |  +- net.bytebuddy:byte-buddy-agent:jar:1.12.10:test

SBT
$ build/sbt "core/test:dependencyTree" | grep byte-buddy
...
[info]   | | | | +-net.bytebuddy:byte-buddy:1.12.10 (evicted by: 1.12.18)
[info]   | | | | +-net.bytebuddy:byte-buddy:1.12.18
...

Note that this happens at branch-3.4 from Apache Spark 3.4.0~3.4.2 only. branch-3.3/branch-3.5/master are okay.
Why are the changes needed?
BEFORE
$ build/sbt "core/testOnly *.DAGSchedulerSuite"
[info] DAGSchedulerSuite:
[info] - [SPARK-3353] parent stage should have lower stage id *** FAILED *** (439 milliseconds)
[info]   java.lang.IllegalStateException: Could not initialize plugin: interface org.mockito.plugins.MockMaker (alternate: null)
...
[info] *** 1 SUITE ABORTED ***
[info] *** 118 TESTS FAILED ***
[error] Error during tests:
[error] 	org.apache.spark.scheduler.DAGSchedulerSuite
[error] (core / Test / testOnly) sbt.TestsFailedException: Tests unsuccessful
[error] Total time: 48 s, completed Feb 27, 2024, 1:26:27 PM

AFTER
$ build/sbt "core/testOnly *.DAGSchedulerSuite"
...
[info] All tests passed.
[success] Total time: 22 s, completed Feb 27, 2024, 1:24:34 PM

Does this PR introduce any user-facing change?
No, this is a test-only fix.
How was this patch tested?
Pass the CIs and manual tests.
Was this patch authored or co-authored using generative AI tooling?
No.

Could you review this PR, @huaxingao ?

LGTM. Thanks for the fix.

Thank you, @huaxingao !


[runtime] add float for podvalue

Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Given that we already have operator double, let us simply ignore the float and use the double as positional argument in the function signature(say TypedPackedFunc, then convert later?)

@tqchen Sounds good.


Tests for serialization and deserialization reference the abstract WiseAgent class

We should update the YAML files used in these tests and the objects referenced in the tests to reference the actual WiseAgent implementations instead.

Just poking around to see if we can enforce not being able to instantiate it I tried:
import abc
class WiseAgent(yaml.YAMLObject, abc.ABC):

However that gives some metadata conflict, probably because both YAMLObject and ABC use those.
But I found https://stackoverflow.com/questions/7989042/preventing-a-class-from-direct-instantiation-in-python, and tried
class WiseAgent(yaml.YAMLObject):
    ''' A WiseAgent is an abstract class that represents an agent that can send and receive messages to and from other agents.
    '''
    yaml_tag = u'!wiseagents.WiseAgent'
    yaml_loader = WiseAgentsLoader

    def __new__(cls, *args, **kwargs):
        '''Create a new instance of the class, setting default values for the instance variables.'''
        if cls is WiseAgent:
            raise TypeError(f"only children of '{cls.__name__}' may be instantiated")
        obj = super().__new__(cls)

This seems to work better. A lot of agent tests pass, and it seems the ones where WiseAgent is instantiated directly fail as desired.
I've not looked into the exact usages

We'll likely need this in our other abstract classes as well.


Fix StackBlitz config by renaming .stackblitz to .stackblitzrc

Hey Edmund- just ran across this and noticed your stackblitz branch is using the incorrect file name for config (.stackblitz instead of .stackblitzrc). With that change the dev command should automatically run on boot.
Let me know if there's anything else I (or our team) can do to help as y'all prepare for launch! 🚀

Hi Eric. Thanks for reaching out and offer helps on this! I fix the file name now but we are still blocked at the moment because most of the remix packages are hosted on a private registry. But this should be resolved once it is published to NPM next week. 🔥  I will let you know if I find other issues at that time :)

I got the packages installed on stackblitz now but it is stuck on building the files. Do you have any suggestions on how to debug this?
https://stackblitz.com/github/edmundhung/remix-sandbox/tree/stackblitz


Sentry Integration stop working after v1.48.4

Describe the bug
I've just updated my Tanstack Router from 1.48.4 to 1.49.1.
I use Sentry to collect errors and, as soon as I updated the router version, the app stop working.
Doesn't render any pages or components.
Just throw an error: TypeError: Cannot read properties of undefined (reading 'replace').
Example available on StackBlitz.
Your Example Website or App
https://stackblitz.com/edit/github-pp1nvw?file=src%2Fmain.tsx
Steps to Reproduce the Bug or Issue

Copied the Quick File Based Route example on StackBlitz
Install @sentry/react, @sentry/tracing @sentry/vite-plugin
Configure Sentry.init

Router doesn't work
Expected behavior
Sentry integration with Tanstack Router should work as it was in the previous versions.
Screenshots or Videos
No response
Platform

OS: Windows 10
Browser: Chrome
Version: 127.0.6533.120

Additional context
No response

From further testing, weirdly enough, the problem relies specifically in the dsn key of the Sentry.init object.
If you remove the dsn, every works.
Wonder what could have caused that.

    /**
     * The Dsn used to connect to Sentry and identify the project. If omitted, the
     * SDK will not send any data to Sentry.
     */

so this probably does not execute the problematic code then if dsn is not present

I get a different error in your example:

Although different message, the trace is apparently in the same trimPathRight function

    /**
     * The Dsn used to connect to Sentry and identify the project. If omitted, the
     * SDK will not send any data to Sentry.
     */

so this probably does not execute the problematic code then if dsn is not present

Also works if you remove the Sentry.Integration line and keep the dsn. So is the combination of both that makes the error happen.

I found the issue, working on a fix

#2165 fixes this issue
here is your reproducer with the PR-built package: https://stackblitz.com/edit/github-pp1nvw-aay6ib?file=src%2Fmain.tsx,package.json


Rive artboard not updating when state changes.

I have a React component where I use a Rive animation with useRive hook.
In the beggining i set the artboard parameter conditionally based on an external prop like so:
  const { RiveComponent, rive } = useRive({     src: "/animations/hamburger_menu.riv",     stateMachines: "stateMachine",     autoplay: true,     artboard: theme === NavbarComponentTheme.Orange ? "White" : "Grey",   });
The problem is that when the prop changes the artboard doesn't change.
How should I manage this problem?

Hi @mihael998 , we currently aren't listening to these parameter changes, but we definitely should. There's a bit of work under the hood to coordinate this. We'll work on adding support for this. In the mean time, you might try two different components for this bit. We'll update on this issue when we get support for this!


Beta: Distance calculations incorrect for finding a study group

@ReveurGAM commented on Wed Jan 24 2018
****Area & Link
Join a study group
****Problem Description
(Sorry, I aborted a report before because there was no cancel button, but then this app wouldn't reload the template, so I'm writing without the template).
The website is not calculating distances correctly.  I highly doubt that CA is closer to Milwaukee, WI than WV, nor is it just past Sheboygan. ;)
****Browser & Environment
Chrome Version 63.0.3239.132 (Official Build) (64-bit)
Desktop with W7 Ultimate
****Screenshots

@raisedadead the issue here isn't miscalculation of distance, its wrong co-ordinates on the entry for Lake Forest, CA.  It has co-ords for Lake Forest, Illinois. I'll fix this.

@johnkennedy9147
All of the distances are off somewhat, but within a reasonable number of miles (usually about 5).  However, Wheeling, WV, is also seriously wrong as it should show about 578 mi.
Thanks for your help!

@ReveurGAM thanks for pointing that out, I've put in another PR to fix, again the co-ords were for a same name location in Il.  Regarding the others being a few miles out, it might just be you browser reporting your location as that of your internet provider, rather than where you are physically located at the time.

@johnkennedy9147 I'm not worried about those minor ones; I figured they are because the exact location may be outside of the stated city/town.   Your supposition also makes sense. :)

@raisedadead Hi, this can be closed, fixed by two of my PRs yesterday, thanks.


[CompliantRecordingBot] Timestamp of delivered video and audio

Both video buffers and audio buffers have Timestamp attribute. I have two questions about that.
Q1. Am I correct that time stamps on different streams (but in the same call) are based on the same base clock? In other words, can I rely on that timestamp to synchronize between video and audio -or- among different videos?
Q2. When I recorded a simple single participant video, I got quite fluctuated durations among samples. I am afraid that this timestamp does not necessarily represent media's timestamp. I might use presentation timestamp of H.264 packets, but as far as I examined delivered H.264 data itself does not have any timestamp. How may I reconstruct the correct media timestamps?
Below is an example of 10 samples in the middle of recording. I expect timestamps have near-constant interval like 33333 (if it's 30fps), but the data I got from the events are so random.

Index
Timestamp
Delta
Length
MediaSourceId
Stride
Height
Width
Color
BitRate
FrameRate
OriginalHeight
OriginalWidth
OriginalColor
OrigitnalBitRate
OriginalFrameRate

100
37924514451528200
499008
7102
13
0
720
1280
H264
0
0
720
1280
H264
0
0

101
37924514451729800
201536
2652
13
0
720
1280
H264
0
0
720
1280
H264
0
0

102
37924514452189000
459248
6708
13
0
720
1280
H264
0
0
720
1280
H264
0
0

103
37924514452289200
100168
1673
13
0
720
1280
H264
0
0
720
1280
H264
0
0

104
37924514452680200
391024
5454
13
0
720
1280
H264
0
0
720
1280
H264
0
0

105
37924514452780000
99728
1846
13
0
720
1280
H264
0
0
720
1280
H264
0
0

106
37924514453230500
450528
6691
13
0
720
1280
H264
0
0
720
1280
H264
0
0

107
37924514453341100
110624
2396
13
0
720
1280
H264
0
0
720
1280
H264
0
0

108
37924514453938500
597400
9611
13
0
720
1280
H264
0
0
720
1280
H264
0
0

109
37924514454087700
149168
3765
13
0
720
1280
H264
0
0
720
1280
H264
0
0

110
37924514454527600
439912
6365
13
0
720
1280
H264
0
0
720
1280
H264
0
0

Never mind. I tried various examples and understand that these timestamps are real value. They are so varied. As far as I rely on these, audio and video are really synchronized correctly.


Bug - Prefab children not blue

Current Unity version used
2021.3.7f1
Describe the bug
When inspecting a prefab's children, they are not blue and so it looks like they are not 'applied' to the prefab.
To Reproduce (If applicable)
Create a prefab with children, and drag it into a scene. Unfold the prefab to view the children
Screenshots
https://sxcu.net/5F9DcZ2XI
As you can see I make a prefab and the children are not blue (blue indicating that the child is applied to said prefab)
Also notice that the child momentarily turns blue when you delete it from the inspector view.

Hey Pippin,
thanks for bringing this to my attention. I know the hierarchy currently has some colour issues when the two-tone background is on.
I want to enhance that feature with custom settings etc, but I'm going to go ahead and fix the selection colours for v0.9 and get the two-tone colours improved in v0.10.

Hi Pippin...
took way longer than it should have but prefabs are displaying much more accurately to the normal hierarchy with some slight color adjustments.

Displaying selection colour
Instance label colour varied depending on selection or active state
Prefab children/root displaying correct icons.

Will push with next release but is currently on develop.


adsf-test-harness wouldn't compile...

Getting following trying to build emotiq-quickdist:
; While executing: CHECK-LISP-COMPILE-WARNINGS, in process listener(1).
> Error: Error while trying to load definition for system NIL from pathname /private/var/tmp/emotiq/systems/asdf-test-harness/eg/fiveam-asdf-example.asd: Component FIVEAM-ASDF not found, required by NIL
> While executing: (:INTERNAL LOAD-ASD), in process listener(1).
> Type :GO to continue, :POP to abort, :R for a list of available restarts.
> If continued: Skip loading "build-dist.lisp"
> Type :? for other options.
1 >

@echupriyanov  Please try https://github.com/emotiq/emotiq-quickdist/pull/4 which neuters the example ASDF definition if you have a greedy ASDF configuration strategy locally.


[Content UI] Math text too long on frqi page

Steps to reproduce the problem

Go to https://learn.qiskit.org/course/ch-applications/flexible-representation-of-quantum-images-frqi
Scroll down to https://learn.qiskit.org/course/ch-applications/flexible-representation-of-quantum-images-frqi#flexible-29-0
See math equation that is too long

What is the current behavior?
Math goes beyond container
What is the expected behavior?
Math equation should fit the container
Screenshots (optional but very helpful)

Additional context (optional)

Duplicate of https://github.com/Qiskit/platypus/issues/466


fcitx/input manager popup location at wrong position on linux

When I have an edit line and use some input manager like fcitx to input some non-english characters, the popup for auto completion/word suggestions is at a wrong position. This is probably because the input manager does not know if there is any input box or where it is. I am not sure if this also happens with the input managers on windows or osx. I think on Linux the problem could be solved by using XIM, but I need to do more research on that.
Right now this is how it looks:

The popup at the bottom left should be at the EditLine.
Also the input manager thinks it's in an input box the whole time so it will also block any key presses to other widgets listening to them. So supporting the events for entering or leaving an input box would also be useful, though I need to do more research here too.

well I would look into how to fix this bug I just broke my system and after repairing it my fcitx doesn't have a popup anymore and I can't get it to the state how it was before. So I can't really fix this atm

probably some API should be used to tell input methods where input position is.
Are you working under windows?

Might be possible with https://wiki.libsdl.org/SDL_SetTextInputRect
It takes the rectangle where the temporary input text should be displayed. You might also need to call SDL_StartTextInput()
See https://wiki.libsdl.org/Tutorials/TextInput


Wrapping this library up as an npm package

@SergiySW : It would be great to have a wonderful library like this wrapped up as an npm package to enable easy use by third-party developers, along with documentation for the API functions. I tried skimming through some of the functions in the .rpc.js but the function signatures are not completely clear by looking at them, (also the RaiBlocks RPC wiki does not have a definitions section so that makes the functions in this library even harder to understand).
Documentation for this library would be really nice as it would enable API use by third party developers. Also, cc-ing: @clemahieu if this library could be put under a Github Raiblocks org account or something ? An "official" looking JS API library for RaiBlocks RPC calls would be great.

Functions to create blocks?
I think it's good idea add to npm with npm blake2 implementation if I have time...

@SergiySW I've forked the project and looking to set it up as an npm package. I'm running into an issue when trying to test it out though. Is there documentation for how to setup (what I'm assuming is a node) the localhost server? I have the official wallet, but I think that RPC isn't setup by default. I also tried running the docker project (https://github.com/clemahieu/raiblocks/wiki/Docker-node) but still had trouble getting it working. Any advice? Really looking forward to contributing :)


Space between image and info in latest iTerm2 versions

Description
In the latest iterm2 versions (v3.4.0+) there is a large amount of space between the image and the info.  See screenshots for example.  This is not an issue in iterm2 v3.3.12, which displays a smaller gap between the image and the info.  The same neofetch configuration was used for both examples below.

[x] Does this issue still occur in the master branch? (Required if issue)
Yes

Neofetch version
Neofetch 7.1.0

Running on OS X Catalina 10.15.7
Screenshot
With iterm2 v3.4.0

With iterm2 v3.3.12

Config file
https://gist.github.com/njenan/6ea9a2d93b210a627badc49b1224bfd2
Verbose log

Run neofetch -vv 2> neofetchlog
Upload the contents of neofetchlog to pastebin, gist or equivalent.

iterm2 v3.3.12 neofetch logs:
https://gist.github.com/njenan/216534f7e4451229c0f435672afc61d7
iterm2 v3.4.0 neofetch logs:
https://gist.github.com/njenan/81f6adf222ba70aef757643aa686c132

Same problem
iTerm2 Version: Build 3.4.3
Neofetch 7.1.0

Same problem
iTerm2 Version: Build 3.4.3
Neofetch 7.1.0

same problem

same problem

in lieu of any traction on this bug; there is a workaround that can be achieved by toggling some flags in the iTerm settings.

Navigate to Preferenced > Advanced
Search for: Disable potentially insecure escape sequences and set to false
Search for: Show inline images at retina resolution and set to false

Screenshots are really pretty but verbose logs (like in the first post) would be great.

Neofetch is functioning identically (finding same terminal size) in both iterm2 versions.

https://gist.github.com/njenan/216534f7e4451229c0f435672afc61d7#file-gistfile1-txt-L140-L141
https://gist.github.com/njenan/81f6adf222ba70aef757643aa686c132#file-gistfile1-txt-L140-L141

I believe neofetch needs to return the retina resolution for screen size on macOS (we need to detect when retina is used too(?)).
But, this may break prior iterm2 versions (where this same code has worked fine for years).
Maybe look at iterm2 and what changed between versions to cause the breakage over here. Maybe also look at their image documentation to see if we're doing anything wrong (possibly also how to handle retina on the application side).

Got the same problem

Maybe my logs can be useful
https://gist.github.com/Kagameow/206383a556cb6528c2b034fa9a5a1066

in lieu of any traction on this bug; there is a workaround that can be achieved by toggling some flags in the iTerm settings.

Navigate to Preferences > Advanced
Search for: Disable potentially insecure escape sequences and set to false
Search for: Show inline images at retina resolution and set to false

This Worked for me!

Neofetch is functioning identically (finding same terminal size) in both iterm2 versions.

https://gist.github.com/njenan/216534f7e4451229c0f435672afc61d7#file-gistfile1-txt-L140-L141
https://gist.github.com/njenan/81f6adf222ba70aef757643aa686c132#file-gistfile1-txt-L140-L141

I believe neofetch needs to return the retina resolution for screen size on macOS (we need to detect when retina is used too(?)). But, this may break prior iterm2 versions (where this same code has worked fine for years).
Maybe look at iterm2 and what changed between versions to cause the breakage over here. Maybe also look at their image documentation to see if we're doing anything wrong (possibly also how to handle retina on the application side).

There are still problems after setting up async：

in lieu of any traction on this bug; there is a workaround that can be achieved by toggling some flags in the iTerm settings.

Navigate to Preferences > Advanced
Search for: Disable potentially insecure escape sequences and set to false
Search for: Show inline images at retina resolution and set to false

This problem still exists after setting up async:

in lieu of any traction on this bug; there is a workaround that can be achieved by toggling some flags in the iTerm settings.

Navigate to Preferences > Advanced
Search for: Disable potentially insecure escape sequences and set to false
Search for: Show inline images at retina resolution and set to false

So I have encountered the same issue with the following versions of software below. Setting the 'show inline images at retina resolutions to NO seems to resolve the problem. However, maybe this is a workaround at best and the issue needs further addressing? I'm including my verbose logs to hopefully help with this.
MacOS: 12.1 | iTerm2: 3.4.14 | Neofetch: 7.1.0
https://github.com/Gex2501/neofetch-logs

The display issue still shows up when the commands are used in an async fashion
Here's my config without async stuff:
# See this wiki page for more info:
# https://github.com/dylanaraps/neofetch/wiki/Customizing-Info
print_info() {
    info title
    info underline

    info "OS" distro
    info "Host" model
    info "Kernel" kernel
    info "Uptime" uptime
    info "Packages" packages
    info "Shell" shell
    info "Resolution" resolution
    info "DE" de
    info "WM" wm
    info "WM Theme" wm_theme
    info "Theme" theme
    info "Icons" icons
    info "Terminal" term
    info "Terminal Font" term_font
    info "CPU" cpu
    info "GPU" gpu
    info "Memory" memory
    # info "CPU Usage" cpu_usage &
    # info "Disk" disk &
    # info "Battery" battery &
    # info "GPU Driver" gpu_driver & # Linux/macOS only
    # info "Font" font
    # info "Song" song
    # [[ "$player" ]] && prin "Music Player" "$player"
    # info "Local IP" local_ip &
    # info "Public IP" public_ip &
    # info "Users" users
    # info "Locale" locale  # This only works on glibc systems.

    # Wait for the functions to complete
    # wait

    info cols
}
...

Here's the log without async stuff:
out.log
Here's the config with the async stuff:
Here's my `config` without `async` stuff:

See this wiki page for more info:
https://github.com/dylanaraps/neofetch/wiki/Customizing-Info
print_info() {
info title
info underline
info "OS" distro &
info "Host" model &
info "Kernel" kernel &
info "Uptime" uptime &
info "Packages" packages &
info "Shell" shell &
info "Resolution" resolution &
info "DE" de &
info "WM" wm &
info "WM Theme" wm_theme &
info "Theme" theme &
info "Icons" icons &
info "Terminal" term &
info "Terminal Font" term_font &
info "CPU" cpu &
info "GPU" gpu &
info "Memory" memory &
# info "CPU Usage" cpu_usage &
# info "Disk" disk &
# info "Battery" battery &
# info "GPU Driver" gpu_driver & # Linux/macOS only
# info "Font" font
# info "Song" song
# [[ "$player" ]] && prin "Music Player" "$player"
# info "Local IP" local_ip &
# info "Public IP" public_ip &
# info "Users" users
# info "Locale" locale  # This only works on glibc systems.

# Wait for the functions to complete
wait

info cols

}
...
Here's the log with all the async stuff:
out.log
Please let me know if I'm missing something, thank you.

Same problem
iTerm2 Version: Build 3.4.3 Neofetch 7.1.0

I  replaced neofetch to fastfetch and problem solved.

@sephymartin I don't think this issue has been addressed, I switched to fastfetch (I see you switched to that too). Although I'm interested in the AppleSilicon logo, is this something you picked up from the Apple website or do you have the ascii art? Thanks.

@sephymartin I don't think this issue has been addressed, I switched to fastfetch (I see you switched to that too). Although I'm interested in the AppleSilicon logo, is this something you picked up from the Apple website or do you have the ascii art? Thanks.

I got this pic from Apple website. link.
And this is my fastfetch config
--logo-type "iterm"
--logo "path/to/the/pic/on/your/mac"
--logo-width "55"
--logo-height "25"

I was fiddling with the advanced settings in iTerm2 and I found that there is one particular setting, Show inline images at Retina resolution, which seems to affect how images scales. See the following comparisons:
Show inline images at Retina resolution: Yes (Default)

Show inline images at Retina resolution: No


Update Documentation to Align with Product UI for MFA based on User Sign-in

Issue
Currently, there is no option to select "MFA based on user sign-in" via the product UI. However, the documentation suggests that this option is available, which is inaccurate. To ensure alignment between the product UI and documentation
Describe the improvement
Update the documentation to reflect the actual state of the product UI.

Related document : https://is.docs.wso2.com/en/next/guides/authentication/conditional-auth/sign-in-option-based-template/

Product UI :
Classic Editor

Visual Editor

Product version: wso2is-7.0.0-beta6-SNAPSHOT

This doc is removed from IS 7.0 docs


Simulation inconsistent results

Hi
I've been playing around with Zenbot on windows for a week now, although I know you don't support it the only 2 things that doesn't work are the server, because of path problems, and the launch option, which can be averted by launching zenbot in separate processes.
I decided to go this way because the reducer is way faster than using it inside docker, so I decided to ditch the server for now since I can live without it.
On my first try I restored the database that you provide and backfilled the rest, runned the reducer and the simulations where consistent with your results.
The second time I decided to start from scratch and backfill all the data, for BTC in EUR and USD, it took +-1 day to backfill and reduce.
I ran the simulations but this time they do not yield the same results, I've got BTC/USD = 1.297 ROI and BTC/EUR = 0.797774 ROI
The main thing I'm, confused is why the results where identical when I restored the database and why are they so off now. I also noticed that the bot only traded 4 times in BTC/EUR but traded 44 times in BTC/USD so something strange is happening here.
I'm a newbie in trading so there must be something I'm doing wrong, but cant figure out what.
Here are my configs

BTC/USD

var c = module.exports = require('./config')

c.assets = [
  "BTC"
]
c.currencies = [
  "USD",
  "EUR",
  "BTC"
]

// default selector for indicators, etc
c.default_selector = "gdax.BTC-USD"
//c.logic = require('./logic_2');

BTC/EUR

var c = module.exports = require('./config')

c.assets = [
  "BTC"
]
c.currencies = [
  "EUR",
  "USD",
  "BTC"
]

// default selector for indicators, etc
c.default_selector = "gdax.BTC-EUR"
//c.logic = require('./logic_2')

Here is my latest sim data.
BTCUSDsim.txt
BTCEURsim.txt

Hummm
In the sim log for BTC/EUR I am getting
gdax.BTC-EUR missing RSI tick: 06/15/2016 11:40:00 PM WEST
and then a lot of
gdax.BTC-EUR aborting sell due to low perf. = -0.048
How can I fix this without backfilling all again?

Zenbot 4 is out, so I'm cleaning up 3.x issues. Please use and test 4, thanks!


@conda-forge-admin, please add bot automerge

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/mne-qt-browser-feedstock#4.


Create Catalog Entity

Create a catalog entity that includes the fields details in our discussions. Should have a model in a similar way as Artifacts.

Closed in #478


New version: LibSerialPort v0.2.0

Registering package: LibSerialPort
Repository: https://github.com/JuliaIO/LibSerialPort.jl
Created by: @andrewadare
Version: v0.2.0
Commit: 9d4ec4b695011e455f0e8028af8605f54df4b1e1
Reviewed by: @andrewadare
Reference: https://github.com/JuliaIO/LibSerialPort.jl/issues/39#issuecomment-547926720

Your new version pull request does not meet the following guidelines for auto-merging:

The following dependencies do not have a compat entry that has an upper bound: BinDeps, julia

Note that the guidelines are only required for the pull request to be merged automatically. However, it is strongly recommended to follow them, since otherwise the pull request needs to be manually reviewed and merged by a human.

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.


Add some documentation for index and registry stuff.

This adds some internal docs for index and registry things.
Split out of #9111.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 838e538420e38b93228feec37eb93fdbaa96f843 has been approved by alexcrichton

:hourglass: Testing commit 838e538420e38b93228feec37eb93fdbaa96f843 with merge 398514abd61ddf07dc13e44c9c7c76b8b2586ae5...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 398514abd61ddf07dc13e44c9c7c76b8b2586ae5 to master...


feat(Tactic/ComputeDegree): add helper lemmas for compute_degree_le

This PR is a prequel to #5882: it simply adds the helper lemmas about polynomials that the tactic uses.
Zulip discussion

Thanks! I pushed a minor golf. Feel free to either accept it or revert it.
bors d+

bors r+


fsync/fsyncdir API documentation typo?

I believe in README.md
ops.fsync(path, fd, datasync, cb)
should be
ops.fsync(path, datasync, fd, cb)

no the docs are correct. the fd is the 2nd argument. are you not experiencing this?

I will double check.

Yes, you are right. The current document is correct.
Btw, have you ever experienced error like this:
fuse: bad error value: 53220
I'm getting the exactly same error every time I run npm i fuse-bindings in the mounted folder. Same for other compiling/node-gyp operations:
npm i utf-8-validate gets fuse: bad error value: 19020
npm i bufferutil is fuse: bad error value: 19092
I'm using Mac OS X 10.10.3 with OSXFUSE 2.8

which file system are you mounting?

You meant the backend of the handlers? It's mongodb. It doesn't seem to be caused by the implementation of handlers.

I'm sure what you mean. Could you share a full code example?

Sure, just did. I am closing this one since it's off topic.

Did you have a chance to try the code? Any problems?


the code of lidarSim

In your experiments, you have demonstrated that your method perfected better than lidarSim.Great job! But could your share the code of the lidarSim?Thank you very much!

Hi, you can refer to the implementation in LiDAR-NeRF (lidarnvs) for the baselines.


undefined symbol

Hi, guys.
After installing Dnscrypt-Proxy 1.8.1 and running it, an error happens in Linux Mint 18.1. This error says: dnscrypt-proxy: symbol lookup error: dnscrypt-proxy: undefined symbol: crypto_box_curve25519xchacha20poly1305_open_easy.
Any idea? or, Any solution?
Why does this happen?
Thanks.
Regards.

Can you describe how you compiled/installed libsodium and dnscrypt-proxy?
Looks like the binary was compiled on a different system, with a different libsodium version.

Hi.
libsodium was compiled in this way:
./configure --prefix=/usr
make
sudo make install
libsodium version: 1.0.11
I was trying with libsodium git version.
And, dnscrypt-proxy was compiled in this way:
./configure --prefix=/usr --with-systemd
make
sudo make install
dnscrypt-proxy version: 1.8.1
I was trying with dnscrypt-proxy git version.
I don't know if I do it wrong.
I need something more?

To quote the message that was printed when you compiled this:
*** This is unstable, untested, development code.
It might not compile. It might not work as expected.
It might be totally insecure.
Do not use this in production.
Use releases available at https://download.libsodium.org/libsodium/releases/ instead.

This is especially true during the holiday season, where I have no Linux machine, and I get limited sleep, but drink too much.
More seriously, that symbol was introduced in non-released code, but as we're not even close to a release, the library version number is the same as when this function was not present.
My guess is that you compiled dnscrypt-proxy with one version of libsodium, but then it uses a different version when it runs. Check that you don't have multiple installations, such as one in /usr and another one in /usr/local.

Hi friend.
Effectively, it had 2 versions of libsodium in my system. Error fatal, I hadn't noticed this, I had the libsodium version with the number 18 installed from the repositories of Ubuntu and the libsodium version 1.0.11 installed by me. Problem solved!!
Thank you very much for help me.

Glad that it's solved!
If you are using the git version of the proxy, feel free to test the recent changes and to improve the new documentation so that version 1.9.0 can be released soon!
Cheers.

Hi Frank.
I would like to collaborate with the documentation but my English is not very good, I am from Latin America and I speak Spanish. I am currently using Manjaro Linux and the git version of dnscrypt.
Greeting!


fix deprecated variable access in templates

fix deprecated variable access in templates in puppet 3x

Anything missing here?
Without this merge this module does not work with Puppet 4.

that should be already the case in master. but we merge it nevertheless in case I forgot something to fix.


S3C-287 FIX: Always set bucket location

We should always explicitly set a location constraint on a bucket if data backend equals 'multiple'.

@ironman-machine go

Hello @LaurenSpiegel
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "fix/alwayslocation"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11983 for CI status.


crash with TheBetweenland

https://paste.dimdev.org/ohucinexix.txt
fix it!!!!!!!!!!! >:(
crashes when entering the betweenland dimension

Same problem here, but with Stellar API (and some other mods)
crash on entering the overworld (crash uploaded by VannilaFix)
https://paste.dimdev.org/hobaviyoqe.mccrash
sorry for my bad english, this isn't my main laguage


UnitConverter: Automatically flip units when users set one to the same type than the other

Fixes #266
Description of the changes:

replace OBSERVABLE_PROPERTY_RW for Unit1 and Unit2 by custom code.
verify if the new value of Unit1 is equals to the current value of Unit2, if it's the case, set Unit2 to the former value of Unit1.
add unit tests (and adapt existing ones)
[NIT optimization] replace Unit1 and Unit2 by m_Unit1 and m_Unit2 when possible

How changes were validated:

Manually tested + unit tests

Locking the PR since the associated Issue isn't approved yet.  That way conversation on the problem being addressed is discussed in the Issue, so that future PR discussion can be limited to the implementation of the fix.

rebase done

We're converting Calculator to C# with a PR for the conversion coming in the next few days, so we're going to close this PR. If you want to revisit the issue, please open a new PR after the conversion has completed.


Unicode Combining Characters Not Accessible

Unicode combining characters (e.g., U+0304 Combining Macron), do not appear in the results when codepoints searched fro with Unicode Search.

You are correct, not all characters are present in the plugin.
Your character belongs in the Nonspacing Mark category, and that is why you cannot find it.
Included in the plugin are only characters based on criteria:

Only characters from the Basic Multilingual Plane (Hex value is less than 0xFFFF) are included.
General categories starting with C (control) and M (marks) are excluded.
Codepoints with names beginning and ending with brackets <> are excluded.

This was a compromise, for usability of the plugin, mainly from a performance perspective of the search (it would freeze periodically during search).
The former plugin version (especially on mobile devices) wasn't great at filtering (quickly or well), so I removed as I could at the time.
The overall expansion of UCD characters is one of the next steps in the project, although it requires some thought.

Thank you for this fabulous and slick plugin! This fills such an important and critical need within Obsidian.
I'd also really like to be able to use it to add unicode characters from the Control and Mark categories.
Would it be possible please to add an option to support the full unicode set, or, if that is too heavy, then to add an option for users to specify additional ranges of unicode characters to support?

Thank you for your kind words!
I'm glad that you find the plugin useful.
I was considering implementing the ability for users to choose code planes/blocks they would like to have accessible to them.

Implemented in Release 0.6.0.
I've added a plugin settings tab, where you can select which blocks and categories do you want to include in the search results.


unit test JS

Codecov Report

Merging #64 into master will increase coverage by 2.23%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   71.03%   73.26%   +2.23%
==========================================
  Files          49       49
  Lines        2213     2162      -51
  Branches      464      438      -26
==========================================
+ Hits         1572     1584      +12
+ Misses        565      502      -63
  Partials       76       76

Impacted Files
Coverage Δ

packages/aws-amplify/src/Common/JS.ts
100% <100%> (+73.25%)
:arrow_up:

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 e1ea070...2528fe3. Read the comment docs.

+1


[CX_CLEANUP] - Fix build with the dependency-tasks feature for the QuorumProposalRecv task PR

Related issue: #2951. To be merged into #2970 since this PR fixes the build for it.
This PR:

Fixes build without the dependency-tasks feature.

Adds documentation for Temporary* types.

Fixes build with the dependency-tasks feature.

Updates fields of QuorumProposalRecvTaskState.
Removes unused publish_proposal and vote_if_able functions for the dependency-tasks feature.
Adds not(feature = "dependency-tasks") gate to publish_proposal and vote_if_able calls in the consensus task.
Fixes other lint errors.

Adds a command to justfile to build with dependency-tasks.

This PR does not:
Change main logic.
Key places to review:

Question added as TODO in crates/hotshot/src/lib.rs.
cancel_tasks call added to quorum_proposal_recv.rs.
Whether the feature gates are added correctly.

These are integrated and we can close it. Sorry, I didn't see this and was halfway through doing most of this. Thanks for doing this!

Closing this PR since the major changes are now in https://github.com/EspressoSystems/HotShot/pull/2970.


CPM_INDENT is not inserted everywhere and no CPM_DESINDENT

Hello,
Thx for you nice tool it's wonderful.
I noticed that CPM_INDENT is not used everywhere in the code? This is expected ?
I would like to add an CPM_DESINDENT or other name to be able to use CPM_INDENT to use colorization. For now I can do it but I can't desactivate the color.
It wouyld be then possible to do :
set(CPM_INDENT MyColor)
set(CPM_DESINDENT White)

Would you accept a PR to add this variable CPM_DESINDENT or a better name ?

Hey thanks for the feedback, happy you like it!

I noticed that CPM_INDENT is not used everywhere in the code? This is expected ?

Hm I think there are some places where the indent wouldn't make sense as they are not scoped, e.g. fatal errors. For other purposes this may be an oversight.

I would like to add an CPM_DESINDENT or other name to be able to use CPM_INDENT to use colorization. For now I can do it but I can't desactivate the color.

Hm not sure if that would be too far off from its original meaning. Maybe it would be better to introduce two new variables like CPM_LEFT_LOG, CPM_RIGHT_LOG? Probably it would also make sense to add a function for logging, for more consistency and easier customisation.

Yes I agree the name is not very nice. And the ones proposed are much better..
Concerning the indent on fatal errors it's true indentation are not useful but I used CPM_INDENT as [CPM].
Addind the values you propose looks great. For a logger I wrote something like this : https://github.com/flagarde/CMakeCM/blob/main/modules/Missives.cmake but it's maybe too much code just for logging some message in cpm

Would you allow a PR with CPM_LEFT_LOG CPM_RIGHT_LOG ?

Sure, though if its a lot of extra code, I think I'd prefer a small helper function for logging. But I'd be happy to take a look at the PR!

Well it depends a lot of what you expect from a logger.

Colorization : Possible but need to hack a bit as CMake doesn't provide any hint to know if the terminal or IDE accept ANSI color escape sequences. I raised an issue https://gitlab.kitware.com/cmake/cmake/-/issues/22954 for this.
Add level : be able to register new levels based on the default ones.

This is what can be done with around ~200 lines of code (would need more to check TERM env for colorization)
missive_mode(NAME "MY_ERROR" PARENT_MODE FATAL_ERROR STYLE "Blue" APPEND_BEGIN "#### " APPEND_END " !!!!!!!!" APPEND_STYLE_BEGIN "Green" APPEND_STYLE_END "BoldYellow")
missive_mode(NAME "PYTHON" STYLE "Green" APPEND_BEGIN "[Python]:" APPEND_END "!!" APPEND_STYLE_BEGIN "BackGreen" APPEND_STYLE_END "Yellow" )
missive_mode(NAME "SEND_ERROR" STYLE "BoldRed")

missive(SEND_ERROR "ERROR!!!")
message(STATUS "")
missive(PYTHON "I'm a python message")
message(STATUS "")
missive(MY_ERROR "OUPS")

After CMAke 3.21 :

Before:


DESIGN: dia_matrix handles positive and negative diagonals differently (or, the same, when it shouldn't?)

When you create a dia_matrix, and you specify diagonal values and offsets, you would expect those diagonals to start with the values given.  This is true for negative offset diagonals, but not positive offset diagonals.  To wit:
import scipy.sparse as sp
data = np.array([[1, 2, 3]]).repeat(3, axis=0)
sp.dia_matrix((data, [0,-1,-2]), shape=(4, 4)).toarray()

Results in:
array([[1, 0, 0, 0],
       [1, 2, 0, 0],
       [1, 2, 3, 0],
       [0, 2, 3, 0]])

But, for positive diagonals:
import scipy.sparse as sp
data = np.array([[1, 2, 3]]).repeat(3, axis=0)
sp.dia_matrix((data, [0,1,2]), shape=(4, 4)).toarray()

The output is:
array([[1, 2, 3, 0],
       [0, 2, 3, 0],
       [0, 0, 3, 0],
       [0, 0, 0, 0]])

I.e., column 0 is always used as the reference column, no matter whether the diagonal offset is negative or positive.
This would make it quite cumbersome to specify an array with diagonals [1, 2, 3, 4, 5] on the three main diagonals:
data = np.array([[1, 2, 3, 4], [1, 2, 3, 4], [0, 1, 2, 3]])
sp.dia_matrix((data, [-1, 0, 1]), shape=(4, 4)).toarray()

Yielding:
array([[1, 1, 0, 0],
       [1, 2, 2, 0],
       [0, 2, 3, 3],
       [0, 0, 3, 4]])

While the current behaviour may appear a bit strange, it's consistent with the BLAS/LAPACK general band format. The fact that sp.dia_matrix follows the same format may be helpful, for example, to ease and speed up the prototyping of code using the general sparse format provided by scipy.sparse before re-implementing for "production" using BLAS/LAPACK band-matrix optimised code (dgbmv, dtbmv, dtbsv, ...).
One way to mentally "understand" the format is that columns of the data input are (subsets of) columns of the generated sparse matrix.


Get treats json.RawMessage like db.RawBytes

I have a struct where one of the fields is a json.RawMessage. When I use DB.Get() to populate it, the field is volatile and changes with more calls to DB.Get().
Here is a simple program to reproduce it.
package main

import (
    "encoding/json"
    "fmt"

    _ "github.com/go-sql-driver/mysql"
    "github.com/jmoiron/sqlx"
)

type Var struct {
    Raw json.RawMessage
}

func main() {
    db, err := sqlx.Open("mysql", "root@/")
    if err != nil {
        panic(err)
    }
    defer db.Close()

    var v Var
    if err = db.Get(&v, `SELECT '{"a":"b"}' AS raw`); err != nil {
        panic(err)
    }
    // first time works great
    fmt.Printf("v: %s\n", v.Raw)

    var q Var
    if err = db.Get(&q, `SELECT 'null' AS raw`); err != nil {
        panic(err)
    }
    // "v" has changed???
    fmt.Printf("v: %s\n", v.Raw)
}

The output I get is:
v: {"a":"b"}
v: null�

I believe this is a Go issue and I've submitted an issue for it.  That ticket describes the problem in more detail;  rest assured that this problem exists with db.QueryRow as well.  Unfortunately, for now you'll have to use []byte to get a copy of that data, though it should not involve a copy to type-convert it into json.RawMessage.
Here's a test for sqlx_test.go that shows that this behavior is consistent across any named []byte type:
func TestIssue197(t *testing.T) {
	type mybyte []byte
	type Var struct{ Raw json.RawMessage }
	type Var2 struct{ Raw []byte }
	type Var3 struct{ Raw mybyte }
	RunWithSchema(defaultSchema, t, func(db *DB, t *testing.T) {
		var err error
		var v, q Var
		if err = db.Get(&v, `SELECT '{"a": "b"}' AS raw`); err != nil {
			t.Fatal(err)
		}
		fmt.Printf("%s: v %s\n", db.DriverName(), v.Raw)
		if err = db.Get(&q, `SELECT 'null' AS raw`); err != nil {
			t.Fatal(err)
		}
		fmt.Printf("%s: v %s\n", db.DriverName(), v.Raw)

		var v2, q2 Var2
		if err = db.Get(&v2, `SELECT '{"a": "b"}' AS raw`); err != nil {
			t.Fatal(err)
		}
		fmt.Printf("%s: v2 %s\n", db.DriverName(), v2.Raw)
		if err = db.Get(&q2, `SELECT 'null' AS raw`); err != nil {
			t.Fatal(err)
		}
		fmt.Printf("%s: v2 %s\n", db.DriverName(), v2.Raw)

		var v3, q3 Var3
		if err = db.QueryRow(`SELECT '{"a": "b"}' AS raw`).Scan(&v3.Raw); err != nil {
			t.Fatal(err)
		}
		fmt.Printf("v3 %s\n", v3.Raw)
		if err = db.QueryRow(`SELECT '{"c": "d"}' AS raw`).Scan(&q3.Raw); err != nil {
			t.Fatal(err)
		}
		fmt.Printf("v3 %s\n", v3.Raw)
		t.Fail()
	})
}

Output:
postgres: v  1": "b"}
postgres: v  1l
postgres: v2 {"a": "b"}
postgres: v2 {"a": "b"}
v3  1": "b"}
v3  1": "d"}
sqlite3: v {"a": "b"}
sqlite3: v {"a": "b"}
sqlite3: v2 {"a": "b"}
sqlite3: v2 {"a": "b"}
v3 {"a": "b"}
v3 {"a": "b"}
mysql: v {"a": "b"}
mysql: v null�
mysql: v2 {"a": "b"}
mysql: v2 {"a": "b"}
v3 {"a": "b"}
v3 {"c": "d"}

I'm going to close this issue because it's not an sqlx issue, however it does make some things in sqlx/types potentially dangerous to use, so I think I will document that this bug is there for certain Go versions.

Okay thanks for shedding light on this. In the meanwhile I am explicitly using []byte and converting to a json.RawMessage afterwards, as you suggested.

@jmoiron Are you sure the warnings are appropriate for types that implement Scanner? It would seem they are always passed driver memory and are expected to copy the []byte.

You're right, the types versions are not vulnerable because they implement Scanner, and other types that implement scanner are also not vulnerable.  Any type foo []byte that is not implementing Scanner is vulnerable.

Cool. If you haven't, see my comment on golang/go#13905 for a workaround on scanning types that are convertible to []byte. Could sqlx detect non-scannable structs that are so convertible, e.g. under fieldsByTraversal(), then convert the pointer value to *byte[] and pass that Row.Scan(...)?

FYI - I submitted @jmoiron's patch, which has been included in Go 1.7, so you should be safe against this problem if you are using it!


Bound Wisps are very broken

Based on some testing done on the Hexxycraft server:

Bound wisps only cast their stored spell once
If there's still media left after this cast, the wisp just sits there doing nothing
If it runs out of media partway through the cast, it crashes the server

Attempting to consume a bound wisp also acts strangely:

When I load the Consume Wisp spell from a scroll and attempt to hermes it, hermes turns grey and hangs
Exiting and reopening the casting grid makes the grey hermes vanish
Casting hermes a second time successfully consumes the wisp, but it triggers the increased cost for consuming a wisp owned by another player (and yes, I checked, I do still own the wisp)

fixed for the next version, didn't experience the consume wisp throwing exceptions occasionally issue so it may be fixed by the fixes to the wisps, or it may pop up again, make a new issue if it does


SCEPTAR GetArrayNumber =/= GetDetector

In the most up-to-date version (Vinzenz has checked) the GetArrayNumber function for SCEPTAR isn't returning what we would like, a number from 1-20 (or 0-19). It is using a similar function as for HPGe detectors, returning 4*DetectorNumber + 1.
Vinzenz and I spoke about what the default GetArrayNumber function should be (since many ancillary detectors don't have crystals or segments) and that it should give a sensible number regardless of detector type.
He will update in the most recent version he is working on (once he fixes the reading of TChannel).

We should either default GetArrayNumber to return detector number in TGRSIDetectorHit, or we should make it a pure virtual function to force it to be defined for every detector.

In the split branch I've decided to have the default GetArrayNumber return the detector number and have it overwritten by any detector that needs something different (the clover detectors and BGOs so far). This is something that needs to be checked for TIGRESS ancillary detectors like S3, or Sharc. I don't know how the array number should be calculated for these if that is something people want.


[NO-QA]: Company cards toggle

Details
Company cards toggle in workspace More features
Fixed Issues
$ https://github.com/Expensify/App/issues/47376
PROPOSAL:
Tests
Test with collect Plan:

Open any workspace in which you are an admin
Go to More Features
Verify that user sees a new item "Company Cards"
Switch on - verify that upgrade plan right panel will be shown
Upgrade to a new plan
Switch on -> verify that a new row appears in left menu item
Verify that the new menu item is highlighted
Click in Company Cards - Verify that a new screen is opened

Test disabled toggle:

Switch on a new feature "Company Cards"
Add true flag to line 135 src/pages/workspace/WorkspaceMoreFeaturesPage.tsx
Verify that toggle will have lock icon
Click on toggle
Verify that toggle is still in active state
Verify that modal is shown
Verify that Cancel button is closing modal
Verify that go to concierge button will open concierge chat

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

Offline tests
Same as above
QA Steps
Same as above

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

PR Author Checklist

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

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

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

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

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

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

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

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

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

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

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

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

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

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

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/2138dbfe-c51e-46cb-92f6-c962cfe5f5f2

Android: mWeb Chrome
https://github.com/user-attachments/assets/1fbc3a6f-b652-46ea-b314-7bb4785a1574

iOS: Native
https://github.com/user-attachments/assets/24023994-a27a-4392-b7fb-cb4606f9777d

iOS: mWeb Safari
https://github.com/user-attachments/assets/6d3f5b8b-b4fa-4a03-88d1-dc73c60f74b4

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/8a84d7e1-3459-4677-84d1-7d6b1820f821

MacOS: Desktop
https://github.com/user-attachments/assets/cdcd13a3-5eca-43bb-867a-a566ecec78f8
https://github.com/user-attachments/assets/e4ae69c5-bead-4b80-b9cc-5e273f4857aa

Reviewer Checklist

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@narefyev91 are you getting a success response from EnablePolicyCompanyCards API? I am receiving 404 for it.

@iwiznia friendly bump on this one - we need this PR to merge to unblock others
cc @mountiny @alexpensify

@iwiznia approved

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

🚀 Deployed to staging by https://github.com/iwiznia in version: 9.0.25-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/iwiznia in version: 9.0.25-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Changes to name, etc.

Made changes to the bot including changing the command prefix to Hex instead of x10bot. Also added the nickserv querying back in before making someone an admin.

Take a look at the commits and you'll see why I can't accept this pull request. Please properly check stuff before you send a pull request, and don't commit to the master branch.


Slurm train scripts

No, it is generated by your setup scripts, not by mine. I forget to add it to .gitignore.

On 24 Feb 2023, at 2:11 PM, Ren Tianhe @.***> wrote:
@rentainhe commented on this pull request.
In detrex/config/configs https://github.com/IDEA-Research/detrex/pull/215#discussion_r1116542440:

@@ -0,0 +1 @@
+/mnt/lustre/zhulei1/GitRepo/detrex/configs
Hi~ this seems like a specific submodule linked to your local projects, if this is needed or not~

—
Reply to this email directly, view it on GitHub https://github.com/IDEA-Research/detrex/pull/215#pullrequestreview-1312561237, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEYCTO5X5QRGYYDWHA3AJILWZBGHXANCNFSM6AAAAAAVGOFRVQ.
You are receiving this because you authored the thread.

Maybe you can try to add it to gitignore or just remove this file (not add it by git add)~ the other parts are OK~ @rayleizhu

Done.

Merged~ Thanks a lot for your contribution~

BTW, would you like to provide some tutorial about how to use the slurm script with a new pull request~ @rayleizhu , maybe create a new markdown file under tools~

BTW, would you like to provide some tutorial about how to use the slurm script with a new pull request~ @rayleizhu , maybe create a new markdown file under tools~

Fine. Maybe by next Tuesday.


Add BM25 retriever

Description
Add implementation of bm25 for document search.
It works well as keyword search, or using vector store and this together works better.
Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[ ] Added new unit/integration tests
[x] Added new notebook (that tests end-to-end)
[ ] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

@logan-markewich  Should we add hybrid retriever as a class not only in notebook?

@Ja-sonYun lets leave a dedicated component for another PR. In my mind, it's valuable to showcase how people can customize/build their own retrievers, rather than relying on pre-packaged solutions 💪🏻


First iteration/draft for the quantum sensing modules

Added the first two modules
added the first iteration

Will need to clean code and create step-by-step guide. Separate Ramsey experiment into own module.

Fix CompleteSensing notebook and maybe change file name to be correct if need be.


fix concurent write error w/signoff

Notes for Reviewers
This PR fixes #
Fix concurent write condition.

Signed commits

[ ] Yes, I signed my commits.

//@dragon-slayer875 @Yashsharma1911


File is null is sent to backend

Hi,
maybe someone has experience in source code of libraryand can help with next issue.
Seems like user can add file and it is sending. However, on backend side I do not see file. It is null.
When I deep dive in the source code of Upload.upload - I see a lot of transformations with data before it will be sent.
So, the question - is it possible that file will be selected and putted in data without error, but later will be missed (because of some transformations within upload.upload function)???
Thanks!

:) more than six months have passed until someone putted a comment.
Cause it is not often issue - we closed eyes on it. But generally I did not find the answer.


Deploy to readthedocs

Successful documentation build from main
https://mercedestrenz.readthedocs.io/en/latest/index.html


Gorilla Mux is no longer maintained

Is your feature request related to a problem? Please describe.
I want to use the http service package in the SDK to subscribe to a pub-sub topic, but I don't want to use the NewService() function as I need to provide my own HTTP server/router. Previously I used NewServiceWithMux() however that only supports Gorilla.
As of end of 2022 Gorilla Mux is no longer maintained, so I have moved to using Chi
The package github.com/dapr/go-sdk/service/http is tightly bound to this now no longer supported library
Describe the solution you'd like
Support other popular Go HTTP libraries like Chi, Gin or Echo or find a more general purpose solution
Describe alternatives you've considered
Not using the Dapr SDK at all for subscripting to topics as it's needlessly complex, and providing very little benefit
Additional context
See post about Gorilla here https://github.com/gorilla#gorilla-toolkit

+1 on the use of Chi. Looks the nearest neighbour in terms of syntax.

+1 We use Chi as well. The cleanest approach might be to use the http.handler interface as the mux arg:
func NewServiceWithMux(address string, mux http.Handler) common.Service {
...

For now I've developed by own Chi based helper, which supports subscribing to topics and adding handlers to receive messages on those topics
https://github.com/benc-uk/go-rest-api/blob/main/pkg/dapr/pubsub/helper.go

/assign

/close

Can this be closed please?


Updated kops instructions

Codecov Report

Merging #58 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   46.57%   46.57%
=======================================
  Files          14       14
  Lines         277      277
=======================================
  Hits          129      129
  Misses        148      148

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 f77f6ba...d48a5f0. Read the comment docs.

could we do a better integration such as using terraform to create bucket,
and maybe preparing command line, and maybe generating a config ?


Add method to fetch requests

Add method to fetch requests

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (21fce51) to head (3fa5474).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #25   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          175       228   +53
  Branches         3         6    +3
=========================================
+ Hits           175       228   +53

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


Adding more dots per day

Maybe an idea to add support for multiple dots per day.
Thanks,
Tom

That would be a great addon :)
Also a way to update the dotmarker, when you return from lets say a popover. So you don't have to call commitCalendarViewUpdate() to update the dots. Or is there a more gentle way calling dotMarker(shouldShowOnDayView dayView: CVCalendarDayView) ?

Hello @tomvlk,
Thank you for the suggested feature. I was thinking about it while working on dots. What stopped me from going this way was that there should be a limit of dots. According to that point it becomes useless since dots are presented for indicating an event on a particular day view. Personally I don't find it useful enough but if you need this feature I will definitely work it out.
By the way what do you think about personal background color for every day view? With this feature you'll be able to set default background colors not only for all day views but for specific ones. Microsoft's Outlook app contains a calendar with this feature for current day. Let me know what you think about this.
Regards,
Eugene

For me, it would be very nice to see because I'm going to make a
calendar with multiple type of events. With multiple dots you can see if
there is an event of a specified type on that day.
So it would be great to have
Thanks,
Tom

Hello @martjemeyer,
As for committing updates of frames you don't have to call it every time you view is about to appear. Only once. Anyway, I'll fix it in a new release. And relatively your second issue it depends. Do you mean a feature of dynamically changing dots e.g. if a user completes a task and there is no more event on a particular day view?
Regards,
Eugene

@tomvlk,
Then there should be a limit of dots since having extra ones won't make your app looks pretty. In this case I'll add this feature with an upcoming release. Check out repo for updates.
Regards,
Eugene

Thank you :smiley:
:+1:
Tom

Atm for simplicity I press a button and it pops a new view where I can choose a time, duration etc. If I press save, it will set a local notification and dismiss the popup. At that moment I want to add a dot if there isn't any there. But that requires me to update the whole calendar view or swipe a few times over :)

@martjemeyer,
I got what you mean. That's really important. I've added it to the new release features. Thank you for suggesting such a useful enhancement.
Regards,
Eugene

Thank you for all your hard work on this great calendar!

Thank you guys for using this software! It makes me keep working on it. Together we build high quality products. :smiley:

Thats the plan :D

:+1: +1! If you worry about to many dots, imho - you can limit it with 3 dots it's enough in most cases.

Did this feature ever get added? Or have you not been able to get around to it yet? If not, I can work on it and then make a pull request

@thomasbaldwinj it would be nice! :+1:

@thomasbaldwinj :+1:

Alright gentlemen, check out my pull request https://github.com/Mozharovsky/CVCalendar/pull/84. Let me know what you think and if there's a cleaner way you think it can be done, let me know and I'll fix it
@skywinder @tomvlk

@thomasbaldwinj Your PR assumes that the dots should remain the same shape but a different color. I'm not sure if I'm in the majority, but I feel that it would be more beneficial to make the shape changeable as well.
For example, let's say you have a way of displaying three different kinds of events on the calendar (birthday shape, circle, and square) and you wanted to display the different shape if an event happened on the date.

@thomasbaldwinj Seems to be awesome! Didn't test yet, but from seeing screens it lookes great!
Different colors are just my must have!

when we add event how can we add line on that particular day.... if u tell that it is very helpful ....

@sreeji44 I don't understand, please rephrase your question.

i am using this cvcalendar is for add events, display events this all i am using.... when we select the date and add one event i want to display line on that event added day... instead of dot how can i add this...

@elsesy  i am using this cvcalendar is for add events, display events this all i am using.... when we select the date and add one event i want to display line on that event added day... instead of dot how can i add this...

@sreeji44 Please open a seperate issue for your request.


skeleton-navigation es2016-asp.net5 fails to run

Here is what happens after a successful build:
1>------ Build started: Project: skeleton-navigation-es2016-vs, Configuration: Debug Any CPU ------
1>
1>  Build succeeded.
1>           0 Warning(s).
1>           0 Error(s).
1>
1>  Time elapsed 00:00:00.9790539
1>
========== Build: 1 succeeded or up-to-date, 0 failed, 0 skipped ==========

Using either "Start debugging" or "Start without debugging" I get to see the same event:

The problem as it seems is that the jspm install task was never executed, so as the result a complete set of modules that jspm resolves is missing - see below the VS 2015 view of what it has available to run this skeleton:

No aurelia modules, no systemjs ...

Step 5 of the install process on the readme is to jspm install -y
I'm open to improving the process so that it works for all environments, else SideWaffle is releasing some Visual Studio-specific templates we can promote that are targeted to that IDE only.

Step 5 of the install process on the readme is to jspm install -y

I may have missed this, since the README file seemed just like a copy of the README file for es2016 version (no Visual Studio)
I am proposing to remove any and all console (cmd.exe) based configuration and do everything via Visual Studio extensions (.vsx) - as this image demonstrates:

This is the standard method for configuring any Visual Studio solution - and the whole purpose of Aurelia Navigation Skeleton apps is to allow "friction-less entry" into the world of Aurelia

My final point in favor of a complete modernization of VS based skeleton is also the strongest one: despite the popular opinion, Aurelia based application is more often than not a part of a larger, distributed system (front end, back end identity management PaaS services, ...) and asp.net5 is likely the best platform for such distributed. This, Aurelia should fit into VS as good (and natural) as possible

Over this weekend I created a few articles explaining my proposed approach to refreshing Visual Studio versions of the Skeleton navigation - see
Aurelia Application Skeletons
for more detals

@adriatic I'm working on updating the ASP.NET Typescript template so that it can be included in SideWaffle. What I ended up doing today to get it working on my end with Visual Studio was I created a gulp task that executes the command jspm install anytime I build the application. Here's what I added to the build.js file.
var exec = require('child_process').exec;

gulp.task('build-jspm', function () {
    exec('jspm install', function (err, stout, stderr) {
    });
});

I'll start looking over your articles tomorrow, but I do believe that this can be fixed without having to rewrite the template.

I'll start looking over your articles tomorrow, but I do believe that this can be fixed without having to rewrite the template.

As I explained in my Gitter based message to @PWKad, I do not want to "disrupt" the currently going process of maintaining the VS based skeletons - but trying to use these skeletons for Aurelia-KendoUI bridge tutorials, I discovered that they simply do not work.
Regardless of the amount of the work needed to make them work, I think that my proposed approach is more modern and easier to maintain because my final version will require only Visual Studio, without having any excursions into console based invocations of npm and jspm`.

Here's what I added to the build.js file.

Unless I do not understand your proposal, I believe that it is not the best we can take. All integration with npm, jspm, gulp, browserSync etc have to be done using Visual Studio Extensions

As I explained in my Gitter based message to @PWKad, I do not want to "disrupt" the currently going process of maintaining the VS based skeletons - but trying to use these skeletons for Aurelia-KendoUI bridge tutorials, I discovered that they simply do not work.

Ah, I have trouble keeping up with the Gitter chat so I must have missed that message.

Unless I do not understand your proposal, I believe that it is not the best we can take. All integration with npm, jspm, gulp, browserSync etc have to be done using Visual Studio Extensions

If you'll look at the Gulp file it pulls its tasks from the build/tasks folder. In that folder are a series of .js files containing all the Gulp tasks. I added my jspm task in the build.js file. From what I remember of Scott Allens article on using JSPM and ASP.NET he just ran the commands manually. By using the Gulp task that I have created we can run the jspm install command directly from the Task Runner Explorer, which used to be a 3rd Party Visual Studio Extension but is now part of Visual Studio 2015.
I hope that clears any confusion.

We've got some updates coming and there seems to be a plan for this, so I'm closing as resolved.


Output plots or raster files (or both?)

The current version of the pipeline saves outputs as .png files of maps produced with ggplot2.  Is that the desired output, or should the output be some kind of raster format (e.g. .tiff) that can be used as inputs to other geospatial workflows?

Main output is the data of the mean DOY over the normals period for different thresholds.  @jeffswitzer—any particular filetype or GDAL driver you'd like me to use when exporting these rasters?


Add opensciencemap vectiles

the big one. this may or may not become part of the original migurski/TileStache

This is no longer relevant, closing.