[Bug]: not an editor command FlutterOutlineOpen and FlutterOutlineToggle

I am on a fresh install with blank config. I get these 2 errors when I try either way of opening the outline.

@samuelnihoul without any information about your setup at all I have no idea how to help. How have you installed he plugins? how have you verified that the plugin is actually loaded etc? Based on the limited information here I think it's safe to say that you haven't loaded the plugin since by default these commands are added granted you are actually in a dart file there is no point calling these functions outside of a dart file and this plugin won't even load if your not at least in a dart file

Here is my plugins/init.lua:
local plugins = {
['nvim-lua/plenary.nvim']={},
['akinsho/flutter-tools.nvim']={}
}

-- Load all plugins
local present, packer = pcall(require, "packer")

if present then
  vim.cmd "packadd packer.nvim"

  -- Override with default plugins with user ones
  plugins = require("core.utils").merge_plugins(plugins)

  packer.init()
  packer.startup { plugins }
end

I am in a Dart file when I try the commands.

My full minimum reproducible example can be found on the github samuelnihoul/nvchad branch test.

It is basically a clean NVChad install where I deleted all the plugins and put only yours and nvim-lua/plenary.nvim.

I also deleted the packer options part.

@samuelnihoul this isn't a normal neovim minimal configuration, using nvchad to reproduce an issues isn't really valid since that is not a clean neovim environment. I highly recommend you try neovim without all of that and just put the plugin somewhere you can packadd it and jsut see if that works between packer and nvchad any number of things could be going wrong. Digging through those separate tools and how their setup and affecting this plugin isn't really my remit. I can verify locally that this plugin works as expected when loaded correctly so in all likelihood this issue is with your config.

OK sorry

Closing, but if you get a working minimal reproduction (without things like nvchad) please post here.


IPv6 NDP PAckets get lost at OpenVPN

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue     -> Did my best to check but i am just a small sysadmin with little coding skills 😅

I am trying to setup a VPN gateway with the Opnsense and OpenVPN.
It runs on TCP on IPv6 and IPv4.
Connecting by either IPv4 or IPv6 is working fine.
IPv4 Connections to the internet work fine.
IPv6 is not working.
I did a Packet trace an it seems it is a Problem with the VPN clients IPv6s not answering to NDP Packets.
A clear and concise description of what the bug is, including last known working version (if any).
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:

Setup OpenVPN on OPNsense like Setup SSL VPN Road Warrior
Add a IPv6 Network
Connect a client an try to reach IPv6 Internet
No Connection possible

Expected behavior
Geting IPv6 internet Acces via VPN.
Relevant log files
Found  nothing regarding the lost Packets.
Additional context
It should be a Multi-WAN VPN Gateway witch acts be the dafault gateway at the Client.
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 22.1.6 (amd64, LiberSSL).
Intel Core Processor (Broadwell, no TSX, IBRS) (1 cores, 1 threads)
Hosted at Vultr

Sorry but here are the IPv6 Infos:
Main IPv6:  2a05:XXXX:XXXX:e27:5400:03ff:fef1:86bf
Block from Hoster:  2a05:XXXX:XXXX:e27::/64
Client IPv6: 2a05:XXXX:XXXX:e27::1000


[GH-38] fix: updated agentkeepalive version to 4.0.2

agentkeepalive version, updated to prevent error: node-modules/agentkeepalive#35

Coverage remained the same at 80.0% when pulling d40a0b4844e654a623b1211da787071153952ee6 on GH-38-agentkeepalive-update into 560373073e13e944a850b31b49018564a2bcd18a on master.


Python 3.7 Compatibility

It seems Python 3.7 broke compatibility in the internals of PyThreadState, so building simplerandom in 3.7 fails with the error:
cython/_iterators_cython.c:29454:19: error: ‘PyThreadState {aka struct _ts}’ has no member named ‘exc_type’; did you mean ‘curexc_type’?

See for example https://github.com/mcfletch/pyopengl/issues/11 which encountered the same problem.

Please try release 0.13.1. I have created it using Cython 0.28.5. Hopefully this is compatible with Python 3.7, however I haven't tried it myself since I don't currently have Python 3.7 on a Linux system.

Have you had a chance to try 0.13.1?

Sorry for the late response - was on vacation.
Yes, I just tried it w/ 0.13.1 and it works on both 3.6 and 3.7.
Thanks!
Oren.
On Thu, Sep 13, 2018 at 4:29 AM, Craig McQueen<EMAIL_ADDRESS>wrote:

Have you had a chance to try 0.13.1?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cmcqueen/simplerandom/issues/4#issuecomment-420852419,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEKXpPZ7lWw5c9MmkXp67y907aOipCIks5uabT6gaJpZM4WWojp
.

I've now updated this for Python 3.8 too.


Legion Spy does not log kind of logging without -lg:spy

$ ./regent.py tests/regent/run_pass/spy1.rg -lg:spy -logfile spy_%.log
$ grep Detailed spy_0.log
[0 - 1111c5600]    0.000727 {3}{legion_spy}: Legion Spy Detailed Logging
$ ./regent.py tests/regent/run_pass/spy1.rg -logfile spy_%.log
$ grep Detailed spy_0.log

(The latter still has Legion Spy logs, just not Legion Spy Detailed Logging.)
This makes legion_prof_rs unhappy because it looks into the logger to figure out what kind of logging it was built with.

This is by design: it's just a bit. If you don't see the statement then you know it's not detailed logging.

What kind of logging do you get when you compile with -DLEGION_SPY but don't pass -lg:spy on the command line?

-DLEGION_SPY automatically sets the -lg:spy flag whether you pass it on the command line or not.

-DLEGION_SPY is always detailed logging. So there are only three possibilities:

No logging.
Lightweight logging when you built without -DLEGION_SPY, and run with -lg:spy
Full logging when you build with -DLEGION_SPY and it doesn't matter what you pass on the command line.

That is not the behavior I'm seeing.
CXXFLAGS='-DLEGION_SPY' ./install.py --debug
./regent.py tests/regent/run_pass/spy1.rg -logfile spy_no_flag_%.log
./regent.py tests/regent/run_pass/spy1.rg -lg:spy -logfile spy_flag_%.log

First 10 lines of spy_no_flag_0.log:
[0 - 11bdd9600]    0.004121 {4}{threads}: reservation ('utility proc 1d00000000000000') cannot be satisfied
[0 - 11bdd9600]    0.061657 {3}{legion_spy}: Rt User Event<PHONE_NUMBER>100009 0
[0 - 11bdd9600]    0.061886 {3}{legion_spy}: Rt User Event Trigger<PHONE_NUMBER>100009
[0 - 700000be2000]    0.063805 {3}{legion_spy}: Task ID Name 10000 hello
[0 - 700000be2000]    0.065407 {3}{legion_spy}: Task Variant 10000 1 0 1 1 primary
[0 - 700000be2000]    0.065447 {3}{legion_spy}: Task ID Name 10000 hello
[0 - 700000be2000]    0.066045 {3}{legion_spy}: Task Variant 10000 2 0 1 1 primary
[0 - 700000be2000]    0.066067 {3}{legion_spy}: Task ID Name 10001 world
[0 - 700000be2000]    0.066114 {3}{legion_spy}: Task Variant 10001 3 0 1 1 primary
[0 - 700000be2000]    0.066128 {3}{legion_spy}: Task ID Name 10001 world

First 10 lines of spy_flag_0.log:
[0 - 113bd0600]    0.000165 {4}{threads}: reservation ('dedicated worker (generic) #1') cannot be satisfied
[0 - 113bd0600]    0.000758 {3}{legion_spy}: Legion Spy Detailed Logging
[0 - 113bd0600]    0.037381 {3}{legion_spy}: Rt User Event<PHONE_NUMBER>100009 0
[0 - 113bd0600]    0.037614 {3}{legion_spy}: Rt User Event Trigger<PHONE_NUMBER>100009
[0 - 700007cd9000]    0.037746 {3}{legion_spy}: Task ID Name 10000 hello
[0 - 700007cd9000]    0.037869 {3}{legion_spy}: Task Variant 10000 1 0 1 1 primary
[0 - 700007cd9000]    0.037886 {3}{legion_spy}: Task ID Name 10000 hello
[0 - 700007cd9000]    0.037923 {3}{legion_spy}: Task Variant 10000 2 0 1 1 primary
[0 - 700007cd9000]    0.037935 {3}{legion_spy}: Task ID Name 10001 world
[0 - 700007cd9000]    0.037974 {3}{legion_spy}: Task Variant 10001 3 0 1 1 primary

Grep confirms the only logging statement (of any kind) is in spy_flag_0.log:
$ grep 'Legion Spy.*Logging' spy_*flag_0.log
spy_flag_0.log:[0 - 113bd0600]    0.000758 {3}{legion_spy}: Legion Spy Detailed Logging

Full log files for posterity:
spy_no_flag_0.log
spy_flag_0.log
This is legion_prof_rs_spy, but I've updated the branch to be based on master commit 8152027d9cedd369fb5d48e3612790387452f20f (so it's pretty recent).

Just a small refactoring bug from a long time ago, try again:
https://gitlab.com/StanfordLegion/legion/-/commit/f87fe3042cc4aaa1ea44f714e093446652ac397b

Thanks!


Homebridge stopped working with iOS Beta

Since Installation of iOS 10.2.1 Beta 3 homebridge devices stopped working. Beta 4 brings no relief.
All Homebridge devices are still visible and working fine on iOS Devices running iOS release version 10.2 an 10.2.1 prior to Beta 3

Interesting, I'll take a look later today.

in the same boat. tried repairing devices and accounts, but it seems to be code related. ill dig around during the weekend.

I just tested homebridge 0.4.16 with iOS 10.2.1 Beta 4 and it's working fine. Please check your plugins are not malfunctioning. And it might be helpful if you two can list the plugins you used and see if there is a common one there.

im having trouble with harmonyhub plugin. tried to update & upgrade, but to no luck.
ive cleared the config files, deleted homebridge from Home.app and re-added it, and it now works fine.

I'm using the HomeMatic and ZWayServer - both (and home bridge itself) updated to the latest version.
Respectively disabling the two plugins makes no difference: the two devices (iPhone and iPad) with iOS 10.2.1 Beta are still not working, 2 iPhones and another iPad running the release version of iOS 10 are working flawlessly.
My AppleTV ist Rundung iOS 10.1 - maybe I'll have time later to install iOS 10.2 on the AppleTV. @joyciticonti - are you using an AppleTV as HomeKit hub? And if yes - what iOS version have you running?

atv - hub - yes - 10.1.1b2
iphones / ipads - 10.2.1b4
running homebridge on a macbook shoved in my server closet.

After a complete reset (clearing all caches, cleaning Home App on iOS) Homebeige now works fine. However it could be interesting to find out what was the cause since reconfiguring/reinstalling Homebridge and the plugins every time a new version is released is a bit of a hassle.
Nonetheless - thank you all for your good work


Range Date Selection

Is it possible to select a range of days?

@RedondaAnalia Not yet. Btw You have some alternatives like ngx-datepicker that can do it.


Replace PyX dependency by matplotlib

We should render our circuit diagrams with matplotlib as this dependency is much easier to install and does not require LaTeX to be installed.

@goerz Are you already working on this?

I haven't started implementing this, but feel free to assign me to the issue

I have started implementing this, using and intermediate representation. Circuits are now translated into a matrix of "tiles". These tiles are then rendered into ascii, a matplotlib diagram, or tikz code. I actually especially like the ascii rendering, which makes it easy to look at diagrams inside of a debugger:

Looks great!
On Oct 22, 2016, at 10:10 AM, Michael Goerz <notifications<EMAIL_ADDRESS>wrote:
I have started implementing this, using and intermediate representation. Circuits are now translated into a matrix of "tiles". These tiles are then rendered into ascii, a matplotlib diagram, or tikz code. I actually especially like the ascii rendering, which makes it easy to look at diagrams inside of a debugger:
[ascii_circuit]https://cloud.githubusercontent.com/assets/112306/19621112/d2e75b66-9858-11e6-8349-e6c14a55028c.png

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/mabuchilab/QNET/issues/8#issuecomment-255540883, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABgESWPis9_rIwSyk5ryIL1KUwtuukr0ks5q2kOTgaJpZM4GbE1V.

PyX has become considerably more user-friendly (and conda-installable) in the last two years. Thus, we can keep it as a dependency for now. Also, implementing all the existing features in matplotlib turned out to be more difficult than expected.
The ascii-drawing of circuits will be added to v2.0 as a separate feature. We could still move to matplotlib at some point in the future, but I would recommend starting with a matplotlib-based visualization in parallel to the PyX based one. Once that matures and has all the features of PyX, we could drop PyX, but this would be in a v3.0 at the earliest.


Does anyone tried integrating mockito with redisson?

I just tried write an unit test for a function using Redisson, but it seems impossible to return a faked RSet data set only by a mocked RedissonClient. Or, usually which library you all used to mock a Redisson client?

Try http://jmockit.github.io/ it's more powerful alternative to mockito

Jmockit looks good. My other question is if this possible to create Faked RSet<>, RMap<> data set in the unit testing without starting the real/ embedded server?

I have the same problem of @kevinchwong , i have to make unit test in isolation, @kevinchwong
you managed to run the server in embedded mode ?

The built-in redis runner will be able to start an isolated instance and terminates it after it has finished. All you need is to point it to the location of redis-server

@jackygurui can you show me a code example? I'm blocked in this moment


Calico network policies

Demonstrate using Calico with Kubernetes to introduce security group like functionality.

PR ready for review:  https://github.com/arun-gupta/kubernetes-aws-workshop/pull/54


Create clickable search bar icon changer

When people look at the search bar in the LBRY app they see a magnifying glass that symbolizes search.
However, when a word is typed in the search bar that word is not searched for, but instead, a file is attempted to be accessed at the lbry:// URL of that word.

An easy way to fix this for users is to create a clickable icon where the magnifying glass is right now in the search bar. This feature would enable users to select what they are using the search bar for: default would be general search (magnifying glass), but click on the icon and you can make the search bar search for channels only (@ symbol) or specific URLs only (file symbol).
In addition, rather than having the 'Exact URL' pop up with a small label beside the content of lbry://content we could have it say 'lbry://content' where it says 'Exact URL' right now. This would make it more clear to users what an exact URL is in the first place, and advertises the niftiness of the LBRY protocol's URL claiming technology.

@jamesbiller - agreed this can still be confusing, see the previous discussion here https://github.com/lbryio/lbry-desktop/issues/671 for why why default to lbry URLs instead of search. Think we can still improve upon the behavior.
One of the shorter term fixes would be https://github.com/lbryio/lbry-desktop/issues/1699


Can't start Mycroft Assistant

This is my first issue on Github so sorry for any misunderstanding. I successfully compile and install mycroft-core, but now i can't activate assistant. It takes almost 10 minutes and fails anyway, moreover it also crashes gnome-shell, forcing it to reload. Is there any way to start this assistant from terminal to see what's going on?

Can you provide more details about your system ?
run
journalctl /usr/bin/gnome-shell -f -o cat > ~/Desktop/gnome-shell.log
Restart your shell by Alt+f2  then type 'r' and 'enter'
Try starting the service. If it hangs or fails, please provide me the log file of your gnome-shell activity on your desktop. Delete it later.
The following procedure is not required and could be done through the extension but
to start the service manually (No extension).
Open terminal. Go to your mycroft-core installation folder.  Run -
./mycroft.sh start in a new window.
After service, skill, voice and cli have started
screen -r mycroft-cli
Type something into CLI.
You will then get an access code (6 Characters). Hear/read the statement in the CLI carefully. After getting the access code. Go to home.mycroft.ai. Sign Up. Then go into devices. Add a device and  fill in the details. You could then .
./mycroft.sh stop
Setup the Mycroft core location folder in the extension settings page and then start the service from the extension. If your shell hangs log please provide me the log file.

Did you make sure and browse into the mycroft-core folder, and select-folder before starting the extension? You may have to re-start your shell.


chore(main): release 0.5.0

:robot: I have created a release beep boop
0.5.0 (2023-10-28)
Features

新增组件 AMapPolygon (ef8dee9)
新增组件 AMapPolyline (dd292de)
新增组件 AMapPolylineEditor (19b442b)

Bug Fixes

AMapPolygonEditor: ts 中无法设置 computeAdsorbPolygons 为 null (3285e54)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/xyy94813/amap-react-components/releases/tag/v0.5.0 :sunflower:


assigned IP changed after creation copy of container

Required information

Distribution: Ubuntu
Distribution version: 20.04
The output of "lxc info" or if that fails:
driver: qemu | lxc
driver_version: 5.2.0 | 4.0.9
firewall: xtables
kernel: Linux
kernel_architecture: x86_64
kernel_features:
netnsid_getifaddrs: "true"
seccomp_listener: "true"
seccomp_listener_continue: "true"
shiftfs: "false"
uevent_injection: "true"
unpriv_fscaps: "true"
kernel_version: 5.4.0-73-generic
lxc_features:
cgroup2: "true"
devpts_fd: "true"
mount_injection_file: "true"
network_gateway_device_route: "true"
network_ipvlan: "true"
network_l2proxy: "true"
network_phys_macvlan_mtu: "true"
network_veth_router: "true"
pidfd: "true"
seccomp_allow_deny_syntax: "true"
seccomp_notify: "true"
seccomp_proxy_send_notify_fd: "true"
os_name: Ubuntu
os_version: "20.04"
project: default
server: lxd
server_clustered: false
server_name: server1
server_pid: 1398
server_version: 4.0.6
storage: btrfs
storage_version: 4.15.1

Issue description
After creating new container2 as copy from running container1 snapshot the running container IP is changed, not immediately but after 1hour.
New created container2 is created as STOPPED , both have configured same IP but it shouldnt change IP of running container.
I try fix it by edit config IP new and original container but it is not work well as original contaner is not able get/set old IP.
Maybe restart of dnsmaq or LXD will help i dont try it.
Steps to reproduce

lxc config device add contaner1 eth0 nic name=etho nictype=bridged parent=lxdbr0 ipv4.address=<IP_ADDRESS>
lxc snapshot contaner1 forcopy
lxc copy contaner1/forcopy container2

Information to attach

[ ] Any relevant kernel output (dmesg)
[ ] Container log (lxc info NAME --show-log)
[ ] Container configuration (lxc config show NAME --expanded)
[ ] Main daemon log (at /var/log/lxd/lxd.log or /var/snap/lxd/common/lxd/logs/lxd.log)
[ ] Output of the client with --debug
[ ] Output of the daemon with --debug (alternatively output of lxc monitor while reproducing the issue)

@tomponline thoughts on this one?
We obviously must make the copy succeed as a snapshot is immutable but we also don't want to confuse dnsmasq too much.
dnsmasq doesn't seem to tolerate two machines with the same static lease and then goes on to give a dynamic one which the container can then renew later on, causing the described issue.
I can think of a couple of ways out of this:

When using a managed network, don't allow two instances to have the same static lease on the same node. This would force the use of lxc copy -d to override the ipv4.address field when copying from a snapshot.
Move static lease handling to instance startup, updating dnsmasq then and ensuring that we never have two running instances with the same address.

In theory the second sounds better as it still allows for easy copy of instances and instance snapshots, but it may also end up being a bit racy and odd to implement.

@stgraber this functionality is currently provided by the bridged NIC driver's Add() function:
https://github.com/lxc/lxd/blob/master/lxd/device/nic_bridged.go#L232-L240
So we would need to ensure that we don't generate the static lease file at add time, and instead at start time (and important remove it at stop time).
This is unfortunately pretty inefficient as it would require reloading dnsmasq when adding/removing lease files, which would impact on the speed we can launch containers, as well as potentially impacting on the service provided by dnsmasq.
For those reasons I would prefer the first option of validating and preventing duplicate IPs being used in LXD's config.

@tomponline yeah, I guess we should setup validation then.

This issue is pretty similar to #8799

Indeed

Probably ought to also validate that we don't have duplicate hwaddr while we're at it :)

Probably ought to also validate that we don't have duplicate hwaddr while we're at it :)

Good idea. I'm also working on enforcing the same checks we have when using network=<managed bridged> to parent=<bridge> where the bridge name matches a managed bridge.


[BUG][build failed] attribute android:requiredSplitTypes not found

Information

2.9.3/2.10.0 -
Mac -
APKPure -
java version "1.8.0_271" -

I'm using xapk-to-apk to transfer xapk to apk of Photovoltaic Calculations_2.1.2_APKPure.xapk, but also shows that repacked failed, and i dig into it,
found that in step apktool b failed. xapk-to-apk tool will produce middle product in .xpaktoapk directory in current directory,
and then this tool will execute apktool b command, and failed.
While I going into .xapktoapk directory, and execute apktool b command by my self, it shows errors below.
Steps to Reproduce
➜  xapk-to-apk git:(development) ✗ python xapktoapk.py Photovoltaic\ Calculations_2.1.2_APKPure.xapk
➜  xapk-to-apk git:(development) ✗ cd .xapktoapk
➜  .xapktoapk git:(development) ✗ apktool b it.ettoregallina.calcolifotovoltaici
I: Using Apktool 2.9.3
I: Checking whether resources has changed...
I: Building resources...
W: /Users/Tools/xapk-to-apk/.xapktoapk/it.ettoregallina.calcolifotovoltaici/AndroidManifest.xml:1: error: attribute android:requiredSplitTypes not found.
W: /Users/Tools/xapk-to-apk/.xapktoapk/it.ettoregallina.calcolifotovoltaici/AndroidManifest.xml:1: error: attribute android:splitTypes not found.
W: error: failed processing manifest.

APK
Photovoltaic Calculations_2.1.2_APKPure.xapk
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything?:Yes
If you are trying to install a modified apk, did you resign it?:Not modified not resign
Are you using the latest apktool version?:Yes, tried already

I'm not exactly sure why those attributes cannot be found. They look like a relic of building splits. I'm not confident yet, but I believe I'll have to update apktool to remove those attributes as it finds them.
Since Apktool cannot remake splits - it seems like right call.

Yeah I guess so. cause xapktoapk tool will make xapk splits into apk, then it goes to make apktool b, this build command deals with no splits but as a whole package to pack apk. Looking forward to your updates. hope this will finally solve this problem!

I'm not exactly sure why those attributes cannot be found. They look like a relic of building splits. I'm not confident yet, but I believe I'll have to update apktool to remove those attributes as it finds them.
Since Apktool cannot remake splits - it seems like right call.

I highly advise against touching attributes that have significant meaning.
requiredSplitTypes is not a relic, the types in there indicate the split APKs that are non-optional for the app to work.
Try Gboard split variant using the base.apk only and you'll have a broken keyboard.
If you sneakily remove the attribute - you mask the fact that the app won't work without the required split APKs.
Apktool does not and cannot deal with split APKs at this time, and should attempt to do so.
If someone wants to use some split APK merging tricks, they should remove the split APK-specific attributes themselves, plenty of ways to do that.

I see, Thank you for your practical advice.


Empty line

There is an extra empty line in the beginning.

Thanks for flagging -- fixed in commit https://github.com/mayakyler/link-shorteners/commit/4f4eda3e6724ba804f96ebb246fea5d58f2bfc79


fix: accept precison during decode json point

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


No need to wait for data after Wire.requestFrom()

In Adafruit_FXAS21002C.cpp, in the function read8() is a while-loop after the Wire.requestFrom() to wait for data. That line can be removed, since there is no such thing to wait for incoming data. The Wire.requestFrom() waits until the I2C transaction has completely finished. When the Wire.requestFrom() returns, the received data is in a buffer in the Wire library.

Fixed


RFC: start of audio system

Starts to address #32.
I have this working prototype for an audio interface and I was hoping for feedback before I go further.  I'm not experienced with Rust/C interfaces, so there could be mistakes there.  Does this fit crankstart?  Would it expand to the rest of the sound API OK?
I mainly patterned this after the graphics module, with some tweaks.  I attempted to follow crankstart and Playdate standards, veering away only where it seemed clear.  The most important change is probably the sound modules/subsystems - the Playdate audio APIs are the largest set of APIs, so I wanted a design that could split out the main subsystems.  sound is the top-level, and sampleplayer is the specific subsystem I implemented here, for sound effects.
A few specific decisions worth consideration:

Storing the subsystem pointer (e.g. *const crankstart_sys::playdate_sound_sampleplayer) in Sound as well as the specific structs SamplePlayer and AudioSample for convenience.
Handling errors that I can't really test and aren't documented, e.g. when play returns non-1.
The use of &mut on the AudioSample argument to set_sample - I think I originally did this because of the use of *mut for arguments all over the place in the bindings, but in the user-facing API, I don't think it matters..?  I should probably make it &.

If this design is basically OK, I'd want to implement fileplayer as well before merging, to make sure the modular approach works cleanly, and because fileplayer is for music, probably the next most important subsystem after sound effects.  (I don't think I'll need to generate sounds on-device, so most of the remaining APIs aren't as urgent for me.)
I tested all of the functions in the simulator, and confirmed playback worked on device.  Docstrings detail any non-obvious behavior I found.  The only calls I found confusing were get_offset/set_offset, which were clearly getting through to the Playdate APIs OK (it did offset, and wouldn't go beyond my sample length) but weren't being set exactly to the values I passed through, even with rate 1.0.  I suspect my WAV files may be odd; I'm not very experienced with sound files or APIs.

Overall this seems good.
Storing the subsystem pointers seems fine, they are essentially static.
For handling errors, I think it is a bit of a mixed bag already, so whatever approach you use is fine. An interesting unrelated project would be to figure out what makes sense for error handling overall, and make sure that's at least possible for game authors.
I think there might be a soundness problem with AudioSample. What happens if you start playing it, then drop it? It appears the Rust code will free it while the audio system is still using it. This is an aspect the Playdate folks never document. I asked about it for bitmap tables.
If the samples can be immutable, I think we could use Rc. That way the Rust player Struct could keep the sample alive until it is done with it, regardless of what the caller does.
With Sprites I ended up using Rc<RefCell> to solve a similar issue, but hopefully that's not needed here.

I think there might be a soundness problem with AudioSample. What happens if you start playing it, then drop it? It appears the Rust code will free it while the audio system is still using it. This is an aspect the Playdate folks never document. I asked about it for bitmap tables.

@rtsuk I tried to test this.  I made a SamplePlayer in my game struct (so, long-lived) and an AudioSample in my update loop (so, short-lived), and added logging at the new/free points.  I used a long WAV file to be sure it was freed before it finished playing.  This is what happened:

player created
I take action to play the sound
sample is loaded
sample starts playing
sample is freed
sound continues to play for its remaining ~9 seconds OK
I take action again, twice in a row, so two samples are created and freed quickly, and I hear (a single copy of) the sample play all the way through OK
I set the crank spinner in the simulator to its max speed, playing the sound on each revolution, so it played a few hundred times quickly, and nothing bad happened, it kept playing

So it seems to work, but from what you said, I'm worried that perhaps it relies on undocumented behavior, or maybe it's a use-after-free because it's relying on nothing reusing the memory where the sample was loaded.  What do you think?  Would a Rc<RefCell> be the right approach for safety?

If the samples can be immutable

When the sample is passed as an argument, the bindings expect a mutable pointer, but I'm not sure what that means under the hood.  For our purposes, it's effectively immutable, because we're just using it to store a pointer, and we don't change that pointer.  But I can't say whether it's safe to treat it as immutable - maybe it's changing something in the underlying struct that makes it unsafe to load into two players, or something?  I'm not sure how to tell or test.

I think we can treat the sample as immutable from Rust's point of view, so we can get away with just Rc, but there's no harm to Rc.
I think it is best to assume the most dangerous possible behavior from the PlayDate firmware, so we should not free anything we've handed to them until they aren't using it anymore. If you look at the way Sprite works, that should provide maximal safety. Since the Rc is inside a cloneable struct it's not too hard on the game developer.

^ This push improves a few comments in the original commit based on what I learned building the next module, fileplayer, and adds #[derive(Debug)].

The second push adds two commits: one for using Rc for AudioSample, to address the concern above, and one that adds FilePlayer for music playback.

I tested the audio sample Rc change with short-lived samples in the update function, which played fine and were not freed until I changed the sample later, and with long-lived samples in the game struct, which worked as before.


Feature request: value indicators

It would be great to have value indicators.

Hello @savadodesigns. Thanks for your feedback! Feel free to make some changes and sumbit a PR.

Thanks. Keep watching here #7 .


Update types.py Address to enable relay get_node

I want to merge this change because...

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Hi @kayuapi

CI is failing
Can you add some description and explanation for this changes?

Closing due to inactivity.


PowerToys Run: URLs are lower in the list than "Warning: Not all drives are indexed."

ℹ Computer information

PowerToys version: v.0.21.1
PowerToy Utility: PowerToys Run
Running PowerToys as Admin: no
Windows build number: 19041.450

📝 Provide detailed reproduction steps (if any)

Go to PowerToys Run settings
Make sure that "Disable drive detection warning for the indexer plugin" is unchecked
Open PowerToys Run
Type example.com
See the result

✔ Expected result
example.com should be above "Disable drive detection warning for the indexer plugin"
❌ Actual result
example.com is below "Disable drive detection warning for the indexer plugin"
📷 Screenshots

You can disable the warning in PT Runs' settings.

not a bug, intentional


testadd join infer and derive test case

Proposed changes
testadd join infer and derive test case
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run p0


"@-ms-viewport" error "expected (css-rcurlyexpected)"

Shows an error for "@viewport" and "@-ms-viewport"
I'm trying to use @-ms-viewport because IE10 ignores the viewport metadata when it's snapped

@heart-of-code How did you solve it?

Closing as this package has been deprecated in favor of the official styled-components/typescript-styled-plugin fork
After updating to that fork, follow up in the styled-components repo if this is still an issue


Jason logged in homescreen redesign

I noticed some issues:

Newly created community won't display any member (need to refresh the page to see the members)

Admin link is not displayed properly

The UX of the Tab on community list is not good. When the community list is long I have to scroll up to see the member list, and when I scroll up I can't see the name of the selected community:

Suggestion: add search feature to the community list page to search by community name or member name.


Keda - Azure Eventhub - multiple resources - duplicate name / defined multiple times

Report
We currently have 2 azure eveny hub namespaces - 'dm-r1ew1-reporting-1-eh-s' and 'dm-r1en1-reporting-2-eh-s'.
Inside these namespaces, each one has an eventhub called 'reporting'. The consumergroup is called 'v2'.
They are in different namespaces so they should be able to be setup inside one scaledobject.
However, when deploying 1 scaledobject which consists of the above, we get an error:
'Error": "metricName azure-eventhub--v2 defined multiple times in ScaledObject azure-eventhub-campaignreportwriter.
Here is the YAML file:
`apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
name: {{ cat "azure-eventhub-scaled-" "xyz" | nospace }}
spec:
scaleTargetRef:
name: "xyz"
minReplicaCount: 2
maxReplicaCount: 4
triggers:
- type: azure-eventhub
metadata:
eventHubName: 'reporting'
eventHubNamespace: 'dm-r1ew1-reporting-1-eh-s'
storageConnectionFromEnv: ECCORE_ENV_EVENTHUB_STORAGE_CONNECTION
consumerGroup: 'v2'
unprocessedEventThreshold : 2000
blobContainer: appstorage
authenticationRef:
name: azure-eh-auth
kind: ClusterTriggerAuthentication

type: azure-eventhub
metadata:
eventHubName: 'reporting'
eventHubNamespace: 'dm-r1en1-reporting-2-eh-s'
storageConnectionFromEnv: ECCORE_ENV_EVENTHUB_STORAGE_CONNECTION
consumerGroup: 'v2'
unprocessedEventThreshold : 2000
blobContainer: appstorage
authenticationRef:
name: azure-eh-auth
kind: ClusterTriggerAuthentication
`
Here is what the scaled object looks like:

As you can see the HPA is missing (the ones already there are for other scaledobjects:

Here is the error on the Keda operator pod:

I tried to do some deep research and came across this:
https://github.com/kedacore/keda/blob/main/pkg/scalers/azure_eventhub_scaler.go#L261
If i am reading it correct - this seems to be a place where it generates the name (matches what we see in the error):

If i deploy the same scaledobject with one type:
apiVersion: keda.sh/v1alpha1 kind: ScaledObject metadata:   name: {{ cat "azure-eventhub-scaled-" "xyz" | nospace }} spec:   scaleTargetRef:     name: "xyz"   minReplicaCount: 2   maxReplicaCount: 4   triggers:     - type: azure-eventhub       metadata:         eventHubName: 'reporting'         eventHubNamespace: 'dm-r1ew1-reporting-1-eh-s'         storageConnectionFromEnv: ECCORE_ENV_EVENTHUB_STORAGE_CONNECTION          consumerGroup: 'v2'         unprocessedEventThreshold : 2000         blobContainer: appstorage       authenticationRef:           name: azure-eh-auth           kind: ClusterTriggerAuthentication
The scaledobject is looking better with the ready/active being 'True':

The HPA created:

And logs in the keda operator pod look better:

Expected Behavior
Create 1 scaledobject which can monitor the 2 eventhubs as they are not the same and are different living in different namespaces.
Actual Behavior
Fails to create the HPA - with an error in the keda operator pod:
'Error": "metricName azure-eventhub--v2 defined multiple times in ScaledObject azure-eventhub-campaignreportwriter.
Steps to Reproduce the Problem

Create a YAML config with 1 scaledobject but 2 eventhubs - keep the consumergroup the same in both types.
2.Deploy it.
3.Check the scaledobject/HPA/keda operator logs

Logs from KEDA operator
'Error": "metricName azure-eventhub--v2 defined multiple times in ScaledObject azure-eventhub-campaignreportwriter.

KEDA Version
2.3.0
Kubernetes Version
1.21
Platform
Microsoft Azure
Scaler Details
Azure EventHub
Anything else?
No response

Thanks @zroubalik I will wait this week for the release. Will let you know the outcome!


Any expectations to work with opsgenie?

Hi!
Any expectations to work with Opsgenie?
Erez

Hi @ErezWeiss, It should be possible to integrate using a similar approach but we've not tried it ourselves. Please let us know if you try it and how you get on.


Low resolution thumbnail

Hi, Thumbnail is in low resolution.
I'm using KDE Neon but same happens Ubuntu 16.10.

Thanks.

I was going to recommend a fix but it's actually impossible to do since the source code is built into electron when it's built. Unless you want to create a development environment to change source code then you may have to wait until the next version and I can always include a resolution setting. The reason the album art is small was to reduce overall cache disk space.
If you wish to change code, in ./windows/spotify/controller.js you can change the 0 from this.track.art = details.track.images[0][1]; to the length which will give you the biggest resolution (e.g. this.track.art = details.track.images[details.track.images.length][1];) as seen in [v1.0.30 controller.js Line 69](https://github.com/Quacky2200/Spotify-Web-Player-for-Linux/blob/master/windows/spotify/controller.js#L69

@develar how does electron-builder implement .desktop icons so that I could implement system icons first before my own as a backup?


Build failure: error[E0433]: failed to resolve. Use of undeclared type or module util

Compiling cuckoo_miner v1.0.2 (/home/weilo/Downloads/grin/grin-miner/cuckoo-miner)
error[E0433]: failed to resolve. Use of undeclared type or module util
--> cuckoo-miner/src/miner/consensus.rs:208:3
|
208 |         util::to_hex(self.0.to_vec())
|         ^^^^ Use of undeclared type or module util
error: aborting due to previous error
For more information about this error, try rustc --explain E0433.
error: Could not compile cuckoo_miner.
tried on ubuntu 16.04 and 18.04

please use latest rust for building


Specify that the workaround is applicable only when using token authentication mode

The documentation can explicitly specify that the the workaround is applicable only when using token authentication mode, as it does not work when using certificate authentication mode.

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

ID: 13c2fd54-8fa6-f60b-2dcb-ee672d61b561
Version Independent ID: 162e0362-b785-4373-e340-0d972c1bf4be
Content: Send APNS VOIP notifications with Azure Notification Hubs
Content Source: articles/notification-hubs/voip-apns.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@razvansalajan Thanks for the feedback! We are taking a look into this and will get back to you soon.

@razvansalajan Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.


deploy harrison views

[ ] Deploy the harrison view on Monarch OLS
[ ] add a harrison view subset to Mondo release

cc @sabrinatoro

need to revisit this after we implement the changes from the workshop on Nov 17, 2021

this is no longer needed. With the new high level classification efforts, the top level classification will look more like the Harrison view by default.


Implement #[bytecode_instruction]

#[bytecode_instruction] is a built-in attribute for native functions. It is applied only to the natives in 0x1::vector as an optimization. Calls to these functions do not emit bytecode for a native call but instead emit dedicated bytecode for these specific native functions.
These don't seem to be represented consistently in stackless bytecode:
While calls to these instructions have dedicated bytecode in normal move, in stackless bytecode they do not - they are just regular call instructions; but also the move model does not tell us these functions are called when we are making declarations - they do not show up in get_called_functions etc.
So it looks like our step of declaring called functions needs to have a special case of looking through the bytecode for calls to these functions in order to declare them.

move_compiler::natives::fake_naming is the code that deals with these special functions.

move_compiler::naming::fake_natives is the code that deals with these special functions.

 For the SBC path, it is pointless to lower them to specialized bytecode operations in the first place. What makes sense to me is to simply have the front-end ignore the directive for cfg Solana. If the SBC wants to turn them back into the calls the originally were, it should do so consistently. I'd actually call this a defect-- worse than a mere wart. It's all fixable in their code, but we're trying to avoid touching and/or fixing their problems.

It would be easier to just turn this feature off in the compiler, though we would have to modify the compiler to do it, which I am reluctant to do - it would be more desirable to operate on standard move bytecode and not require any compiler modifications.
A flag to disable this feature seems like a reasonable thing to push upstream since standard move can work without these special vector bytecodes; though I am not confident upstream is engaged enough to consider such a patch.
The hacks needed to work around this in the llvm code generator are not too onerous. And maybe there are still apis I'm not seeing in the move model to figure out which of these fake natives have been used and declare them.

It would be easier to just turn this feature off in the compiler, though we would have to modify the compiler to do it, which I am reluctant to do - it would be more desirable to operate on standard move bytecode and not require any compiler modifications.
A flag to disable this feature seems like a reasonable thing to push upstream since standard move can work without these special vector bytecodes; though I am not confident upstream is engaged enough to consider such a patch.
The hacks needed to work around this in the llvm code generator are not too onerous. And maybe there are still apis I'm not seeing in the move model to figure out which of these fake natives have been used and declare them.

Did you see my other comment above? Turning it off is simple and is a one-line change. The EVM compiler does the same thing-- guarding the builtin conversion with !cfg!(feature = "evm-backend"). See the code block in the comment above for resolve_builtin.

Closing issue-- this has been fixed by #199.


feat: add format support to toa export manifest

Adding --format option to toa export manifest:
$ toa export manifest --help
toa export manifest

Print manifest

Command options:
  -e, --error   Print errors only                                      [boolean]
  -p, --path    Path to component                        [string] [default: "."]
      --format  Exporting format     [choices: "yaml", "json"] [default: "yaml"]

Options:
      --log      Log level
      --env      Path to environment variables file (.env format)       [string]
  -h, --help     Show help                                             [boolean]
  -v, --version  Show version number                                   [boolean]

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

Comparison is base (1384ce4) 58.27% compared to head (48669fd) 58.17%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #483      +/-   ##
==========================================
- Coverage   58.27%   58.17%   -0.11%
==========================================
  Files         301      301
  Lines        9908     9908
  Branches     1244     1232      -12
==========================================
- Hits         5774     5764      -10
- Misses       4134     4144      +10

Flag
Coverage Δ

unit
58.17% <ø> (-0.11%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Change and document payment method and means in MX

Documents how payment method (MethodPago) is inferred from the presence or absence of advances
Expects the payment means keys to be given in the advances (instead of in the instructions) when an invoice is prepaid to map to a FormaPago value.
Doen't expect a payment means key in non-prepaid invoices. The FormaPago in that case must always be set 99 (Por definir)
Allows the payment means key to be given in the instructions temporarily until the users transition to use the advances

Thanks a lot @samlown! Copy suggestions are always very much appreciated!


Kresus never ending installation - HELP!!

I am having a big issue with Kresus, I just started installing it, and it is taking VERY long to install, the circle around the app just keeps on going round and round saying installing. i even rebooted the actual server and after that still saying the same thing. How do you stop this ?? or remove it and reinstall it? IT IS STUCK in a loop. help please?

Hello @Beast141 ,
could you provide more information?
Are you self-hosted? If so, did you install the dependencies listed here: https://github.com/bnjbvr/kresus#local-weboob-install-preferred-way ?
If not, please do, and restart Kresus, it should then work fine.
If it is still not working, could you provide server logs in a pastbin?
Than kou

Hi @bnjbvr ,
Ok when i clicked on the little trash icon while it was busy in a never ending loop installation, nothing happened, so i clicked that same icon again and the app disappeared thank goodness.  I reinstalled it and it installed properly this time, and its dependencies.
Though i have 2 minor questions which i hope i can quickly ask right now, What if your foreign bank isnt listed? Can you still use Kresus to list your expenses etc?
And the current banks listed, when you put the login details, does this pull the information from your login of your current listed bank account?

Hi @Beast141,
@bnjbvr is currently working on the installation process to make it quicker and safer :).
To investigate why your first installation failed and was stuck in a loop, the logs would be needed.
If your foreign bank is not listed I am afraid there is no ideal solution at the moment.
You will either have to:

develop a backend so that kresus is able to fetch your bank's data (see how in this comment: https://github.com/bnjbvr/kresus/issues/324#issuecomment-170251870)
wait for the possibility to create (a ticket is created for that purpose) an account manually and then insert your operations manually

I am not sure what you mean by your last sentence but Kresus uses the login information you provided to connect to your bank website (as you would do with your browser) and fetch the information needed (bank accounts, operations…).
I hope this answers your questions.

Hi @nicofrand ,
Oh thank you, yes you answered all my questions. the bank would be First national bank.
But how safe is it even to use a third party app to log into a bank?

Your bank password is encrypted inside cozy's database and is only decrypted in memory with your own cozy's password when you log into your Cozy.
If you are afraid that Kresus then uses this password then I guess you only have 2 choices: you trust Kresus or you read Kresus source code since it is opensource and avalaible on this very same URL and decides if everything is OK (the password is not sent to a third party, etc.).

Hi @Beast141,
Kresus behaves as if it were a browser (like Firefox) and goes log in into your bank's website to retrieve accounts and transactions. So you need to give it your password, and there is an intrinsic need to be able to have the raw password at some point, if you want automatic polling of your accounts. Trust comes from the fact it's open-source and you're assumed to have control on your hardware / software hosting Kresus.
If you could have an account not connected to a bank's website (so, no automatic polling), on which you add your own transactions by yourself, would it satisfy your use case?

Hi @nicofrand @bnjbvr ,
Yes that is true, it is open source and thank you for answering. I will have a look.


Bots: Add support for running packaged bots via entrypoints

The entrypoints approach seems like it may make it easier to produce third-party bots, and could potentially be extended to be the default way in which zulip_bots accesses built-in bots?
A test is included for the new approach, as well as:

removal of python 3.4 support and addition of 3.7 to travis (branch failed due to an existing bot requiring lxml with 2.7 or >=3.5)
addition of ImportError as an alternative to ModuleNotFoundError, which is only introduced in python 3.6
changes to improve indication to the user, where the bot module is being sourced

If this is a reasonable approach then we can add support for a 'sample' package, which I've done locally but not included so far.

I merged the first 2 commits.

Just adjusted the last commit to take into account 'editable' bot packages, which don't have a version - at least according to entrypoints - so just show the bot module_name details.

Closing, since we've now merged #708.


uport website down? Cannot register app...

https://developer.uport.me/myapps.html is not working ...

Thank you for reporting this issue we will get back to you shortly

@lazydino it's working fine for me!
could you provide a little more info about what you're seeing? what browser are you using? if you're trying to log in, what app version and device are you on?

@lazydino Can you try running in incognito mode?

Can you try going to appmanager.uport.me and let us know if you are able to register an app?

I sign in with "Connect with uPort" option using uport mobile running in my galaxy s7 phone. I tried to create the app but I need to sign my app in phone? What do I do? I need to past my public-encryption key?

Same problem. I am not able to register new app, already send ticket with more details to support,  but still no reply.

@lazydino yes, the application identity you create through our app manager is linked to your personal uPort ID, so you'll need to sign from the mobile app to set up the app.
@simeonemanuilov could you provide some more details here on what you're seeing? device type, browser, details of the bug?

We have not heard from you, so we are closing this.  If you still would like some help, please re-open it


Transforms w/ Value Operators in 2.0

Your system information

VelocityJS version: 2.0.1
Browser: Chrome 64.0.3282.167
Operating System: Mac OS

Checklist

Is this an issue with code?: No
Is this an issue with documentation?: Yes
Have you reproduced this in other browsers?: Yes
Have you checked for updates?: Yes
Have you checked for similar open issues?: Yes

Please describe your issue in as much detail as possible:
I'm currently attempting to add value operators to a translateY value however since the way transforms are handled 2.0+ I'm having difficulties properly formatting my code.
Steps for reproducing this issue (code):

$el.velocity({
  transform: 'translateY(-=200px)'
}, 100);

Value operators aren't supported in V2 yet - I want to get them in there again, but it's not something that's easy to do (we no longer just animate single numbers), and there will have to be changes in how it's specified. Currently I'm leaning towards something like translateY(*-200px).
While it would be relatively easy to add it for "normal" values, it wouldn't be future proof or very powerful compared to everything else. The code around that is in the process of being rewritten so ideas are welcome (it's in src/Velocity/tween.ts if you want to look).

Value operators exist on numbers now - you can use +=, -=, *=, and /= on any number in a string and Velocity will try to match it together.
Please note that while transform itself pretty much requires forcefeeding to work (browser standards etc), these may effectively kill values by putting them into nested calc strings, so it's strongly advised to use forcefeeding where possible.


Global elipse projections not compatible with central_longitude argument

https://github.com/SCECcode/pycsep/blob/f8f75a99431928b25a98f3585fba9ca04fda466e/examples/tutorials/plot_customizations.py#L120
Robinson (Ellipse) projections for global forecasts are not working when central_longitude is passed as argument. This is due to cartopy version update.

@pabloitu do we have a fix for this issue? see #148 for reasons i dont think we can pin cartopy=0.19.0


Gitea 1.3.0 fails to start

I just upgrade to gitea 1.3.0 from 1.2.3 and now the service fails to start.
The problem is that the oauth2 plugin seems to ignore the working directory:
$ echo $PWD
/home/gitea/gitea
$ /usr/bin/gitea web --port 10000 --config /home/gitea/gitea/custom/conf/app.ini
# ...
2017/12/01 16:17:18 [...uth/oauth2/oauth2.go:47 Init()] [E] Fail to create dir /usr/bin/data/sessions/oauth2: mkdir /usr/bin/data: permission denied

Gitea was build from source: https://github.com/klingtnet/klingt.net/blob/master/build/gitea/Makefile

This is where gitea crashes: oauth2.go

I think the root issue here is that by default Gitea wants to create its data directory next to executable file, what seems undesirable more often than not. IMHO, it should default to ${XDG_CONFIG_HOME}/gitea on Linux and similar directory on other supported operating systems.
@klingtnet Could you try to run Gitea the following way?
GITEA_CUSTOM='/home/gitea/gitea/custom' /usr/bin/gitea web --port 10000

I think the real issue is that by default Gitea wants to create its data directory next to executable file, what seems undesirable more often than not. IMHO, it should default to ${XDG_CONFIG_HOME}/gitea on Linux and similar directory on other supported operating systems.
@klingtnet Could you try to run Gitea the following way?
GITEA_CUSTOM='/home/gitea/gitea/custom' GODEBUG=netdns=go /usr/bin/gitea web --port 10000

This way you won't have to relate on PWD.
PS. The GODEBUG=netdns=go is not required, but seems to be a good choice for performance reasons, unless your networking setup is too complicated for Go's net library.
PS. II Please make backups before experimenting with the way you run Gitea. I don't take any responsibility. You know your setup the best and if you choose to follow my advices, you do it at your own risk.

I think the real issue is that by default Gitea wants to create its data directory next to executable file, what seems undesirable more often than not. IMHO, it should default to ${XDG_CONFIG_HOME}/gitea on Linux and similar directory on other supported operating systems.

@klingtnet Could you try to run Gitea the following way?
GITEA_CUSTOM='/home/gitea/gitea/custom' GODEBUG=netdns=go /usr/bin/gitea web --port 10000

This way you won't have to relate on PWD. This is also how it is done in Dockerfile and it's the recommended way override default custom directory placement.
PS. The GODEBUG=netdns=go is not required, but seems to be a good choice for performance reasons, unless your networking setup is too complicated for Go's net library.
PS. II Please make backups before experimenting with the way you run Gitea. I don't take any responsibility. You know your setup best and if you choose to follow my advices, you do it at your own risk.

@klingtnet Could you try to run Gitea the following way?
GITEA_CUSTOM='/home/gitea/gitea/custom' GODEBUG=netdns=go /usr/bin/gitea web --port 10000

This way you won't have to relate on PWD. This is also how it is done in Dockerfile and it's the recommended way to override default custom directory placement.
PS. The GODEBUG=netdns=go is not required, but seems to be a good choice for performance reasons, unless your networking setup is too complicated for Go's net library.
PS. II Please make backups before experimenting with the way you run Gitea. I don't take any responsibility. You know your setup best and if you choose to follow my advices, you do it at your own risk.

@0rzech Thank you for your detailed response.
I tried to set GITEA_CUSTOM—also double checked its value—but the error is still the same, i.e. gitea tries to write to /usr/bin/data.

@klingtnet You're welcome. Could you also post the value of APP_DATA_PATH from your app.ini?

You're a wizard, setting APP_DATA_PATH fixed the problem!
The key did not exist in my app.ini so I added APP_DATA_PATH = /home/gitea/gitea/data to the [server] section.

I have closed the issue but may consider APP_DATA_PATH to pwd if unset?

@klingtnet If APP_DATA_PATH was absent, then Gitea defaulted to 'next-to-binary location'. I think the best way is to run Gitea the way I presented in my second comment. Once you added the APP_DATA_PATH to your app.ini, it should work fine from now on. Just make sure all other paths in your app.ini are set to proper values and all existing data is in right place.

@klingtnet If APP_DATA_PATH was absent, then Gitea defaulted to "next-to-binary location".
I think the best way is to run Gitea using the command I presented in my second comment. Once you added the APP_DATA_PATH to your app.ini, it should work fine from now on. This means the conf/app.ini should be in data directory as well, resulting in /home/gitea/data/conf/app.ini.
Also make sure all other paths in your app.ini are set to proper values and all existing data is in right place.

@0rzech One note to GODEBUG=netdns=go, it looks like this is already the default behaviour and does not need to be set explicitly: https://golang.org/src/net/net.go

@klingtnet Yes, you're right. I have even pointed it out here. -.-'
Btw. it looks like your installation might become a bit messy, because you have /home/gitea/gitea/custom and /home/gitea/data now. GITEA_CUSTOM should have the same value as APP_DATA_PATH. If everything is kept default except for GITEA_CUSTOM itself, then I think your Gitea data directory should look like:
${GITEA_CUSTOM}/avatars
${GITEA_CUSTOM}/conf
${GITEA_CUSTOM}/indexers
${GITEA_CUSTOM}/log
${GITEA_CUSTOM}/sessions

etc.


Date error

I have make a Date extension like this:
Inputmask.extendAliases({
'date-input-mask': {
alias: "yyyy-mm-dd",
placeholder: "____...",
inputFormat: "yyyy.mm.dd.",
mask: "y.1.2.",
separator: ".",
leapday: ".02.29.",
insertMode: false,
showMaskOnHover: true,
max: 24
}
});
In the textbox I see: "y.1.2" instead of 2020.04.26.
What make I wrong?
I use 5.0.2 version. I used before 3.3.11 version and there was no problem with it.

@Fuegos83,
Have a read at the readme for dates.  The dtatime alias has been rewritten.


Add tutorial cards

As a tutorial TA, I want to add tutorial events so that I can keep track of when my tutorial sessions are

This issue is completed in PR #22.


Nesting classes into packages or simple bounding boxes

In order to use the class diagram in more complex projects (> 10 classes), we need to have some kind of boundary around some classes - e.g. bounded context for DDD-like approaches or packages referring to UML.
In a first step, a simple bounding box (e.g. dotted rectangle in grey) with some label (top, center) would be sufficient.
The definition could look like the package diagram, plus based on indented classes and namespaces:
[Context1]
   [MyClass]
   ...
[Context2]
   [MyOtherClass]
   [MyOtherClass]->[Context1.MyClass]
   [MyClass]-[note: this is a different class than MyClass from Context1 with different relations.]

Regards
Sven

For an idea of how to introduce different boundary or package styles, please refer to: PlantUml Package Style Types.
This could be defined like the color:
[Context1{style:cloud}]

Hi Sven,
I will consider it.
The indentation doesn't seem to be the "yUML way", though.

Comments regarding the yUML syntax should be done now in https://github.com/jaime-olivares/yuml-diagram


No errors message for missing file input

When the only thing entered during task creation is the task name and nothing else, and I click create new task, nothing happens, even though e.g. file not set error should be displayed

Done as a subpart of commit labeled "Closes #206"


nit (doc issue): remove reference to Zam in chapter 36 of unpriv ISA

It's a nit documentation issue.
In unpriv spec,
36.6. Additional Standard Extension Names says
If multiple "Z" extensions are named, they should be ordered first by category, then alphabetically within a
category—for example, "Zicsr_Zifencei_Zam".

Since Zam doesn't exist anymore, I don't think it makes sense to use it as an example here.

Indeed.  I'll replace it here with Ztso.


fix h-scrollbar cutoff issue

This PR fixes #21231
redo the layout after the content of taskbar changes since it could impact the height of the taskbar.
Before:

After:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.006%) to 42.157%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/query/browser/queryEditor.ts
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

Totals

Change from base Build<PHONE_NUMBER>:
-0.006%

Covered Lines:
28567

Relevant Lines:
62935

💛  - Coveralls


Fixes #784: Update arch db to 1.1.1

android.arch.persistence.db.SupportSQLiteProgram was updated to extend
java.io.Closeable instead of java.lang.AutoCloseable.
Updates the exception thrown by
com.squareup.sqldelight.SqlDelightStatement#close to java.io.IOException
to match java.io.Closeable#close.

Ah, right. This also adds a new interface method which we cannot return a valid value from. This is supported by 1.0 only. So db 1.0 is broken because they use an API 19 interface and 1.1 is broken because they added a method to an interface. Le sigh.

If a release for pre 1 will be cut would it also be possible to change the dependency of com.squareup.sqldelight:runtime:0.6.1 from compile to api or `implementation - whatever is consistent with 1.0.0.

I don't think this is going to happen, sorry. Lessons to be learned:

Don't ignore lint's NewApi check and use AutoCloseable for APIs supporting pre-19.
Don't add methods to interfaces without a major version bump or Java 8 default methods.
Don't depend on libraries which break rules 1 and 2.

3 was our fault...


Add subtitles for the what-is-monero video in home

Subtitles provided by @staff91
More subtitles are likely to arrive soon, i'll look into improving the videos. The subtitles format is .vtt, which is not supported by weblate, so at least for now translations must be submitted to this repo.
I'm having weird results on chromium, but that could be related to my local setup. So please test on your browsers. There shouldn't be compatibility issues.

Here's the translation to Italian for the video (.SRT format): https://gist.github.com/staff91/e64d6923dffb2b6162e23feca2d851af#file-what-is-monero-srt

@staff91 Thanks, but please open a new issue about it. Also, the format of the subtitles must be .vtt

Plays fine for me in Firefox, Chrome, and Edge.

Plays fine for me in Firefox, Chrome, and Edge.


ARROW-5401: [CI] Print ccache statistics on Travis-CI [skip appveyor]

Codecov Report

Merging #4393 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4393      +/-   ##
==========================================
- Coverage   88.36%   88.35%   -0.01%
==========================================
  Files         781      781
  Lines       98783    98783
  Branches     1251     1251
==========================================
- Hits        87288    87280       -8
- Misses      11259    11267       +8
  Partials      236      236

Impacted Files
Coverage Δ

cpp/src/arrow/csv/column-builder.cc
95.29% <0%> (-1.77%)
:arrow_down:

js/src/ipc/metadata/message.ts
92.99% <0%> (-1.28%)
:arrow_down:

cpp/src/arrow/io/readahead.cc
95.91% <0%> (-1.03%)
:arrow_down:

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 f82af62...c3ac74b. Read the comment docs.


Saved ROI should be loadable as a Mask into models

Currently, saved ROIs are store a variable called Mask which is a 1x1 cell of a struct with two properties: vol and color. Because of this, the saved roi *.mat cannot be loaded as a Mask for the model.

Hi @mathieuboudreau, I already handled this in the ROI tool. I removed the vol and color fields from the ROI cell array. But we are saving the whole ROI cell array as a mat,  do you think we should combine the saved ROIs when loading them as a mask or should we save each element (ROI) from the cell array as a separate mat file?

do you think we should combine the saved ROIs when loading them as a mask or should we save each element (ROI) from the cell array as a separate mat file?

I think if the user tries to load the ROI set as a mask, they would assumed that all ROI are combined to create a single mask (that's what I would want anyways). Maybe @HarrisNami could chime in, since he's the user who's used ROI tools in qMRLab the most.
Do you have a preference @TommyBoshkovski ? What behaviour would you want if you were using this feature?

I guess loading them in a single mask so all the ROI are loaded at the same time is more efficient and practical. Personally for me if I am looking at one ROI I would only like that one to be there, but since the tool shows only one ROI at a time anyway that isn't as a big of a deal.


skip functional tests for models that are not downloaded.

hi all, it seems that functional tests require two models to be downloaded: openai_whisper-tiny (because of testRealTimeFactorTiny()) and openai_whisper-large-v3 (because of testInitLarge() and testRealTimeFactorLarge()). This should probably be added to the CONTRIBUTING.md, however this PR does not make this change.
Another functional test, testOutputAll() - actually allModelPaths(), assumes that folders in ./Models/whisperkit-coreml, which are created from the make setup-model-repo, contain actual models, while in fact only models explicitly downloaded with make download-model MODEL=x or make download-models are present.
testOutputAll() fails if not all models have been downloaded, which is not ideal. Running all tests for all models (takes a long time and) makes sense in some scenarios, but probably not to test some code changes.
I've implemented some changes so that allModelPaths() only returns folders that contain downloaded models. This is done by checking if a proxy file (MelSpectrogram.mlmodelc/coremldata.bin) is a git lfs pointer file (starting with version https://…). If it is, then the folder is not included.

Moved the typo fix to its own branch (for a future PR).

It failed when building for VisionOS, which I don't have as a platform locally.
For the typo fix (langauges -> languages), as you prefer: I can add it back to this branch or create another PR just for it.

@metropol whatever is more convenient for you, probably just reverting the commit that removes it would be simplest.

@ZachNagengast  I've reverted the commit the removes the typo.


bug: cast run does not decode events on local node with chain id 1337

Component
Cast
Describe the feature you would like
doing cast run <tx> doesn't always fully decode calls and events. When executed against local node, it does parse methodIds, but not events.
However, when decoding transaction on external chains (event testnet, like sepolia), it parses both methods and events.
Apparently, it depends on the "chainId" of the local node, and when chainId is "1337", it fails to decode some information.
with Anvil, I can add --chain-id, but some of my tests require geth, which can't easily change the chainid.
suggestion
Either should be able to "push" symbol information to chain-id specific, or add a fallback, that if methodid/topicid are missing for a specific chain, then use them from another (after all, they can't differ much, the methodid/topicid depend on the method name and parameter types. the only thing that can differ between networks is the parameter names)
Additional context
No response

I think we're fetching that from etherscan API that is mapped to chain id (https://github.com/alloy-rs/chains/blob/main/src/named.rs#L1047), what you could try is to map the dev chainId "1337" (https://github.com/alloy-rs/chains/blob/main/src/named.rs#L170) to an Etherscan url / api, see https://book.getfoundry.sh/reference/config/etherscan
Pls lmk if this helps, thank you

possible related https://github.com/foundry-rs/foundry/issues/3473

marking as dupe of https://github.com/foundry-rs/foundry/issues/8606 and going to track impl there. thank you!


[docs] updated README.md

Fix some grammar and markdownlint issues.
Preview: https://github.com/cloudtty/cloudtty/blob/4234e6aac037ab1345cf02d1638e7c43a74df3af/README.md

@windsonsea  Thank you for your contribution,  can you help to update the chinese documents synchronously?

@windsonsea  Thank you for your contribution,  can you help to update the chinese documents synchronously?

boys, men, guys, and masters, could you give the suggestion, not always ask questions when you review a doc?

Click + before a line
Click the icon below

Directly modify the text, don't ask too many questions please.
Click Start a review if you have many comments. Click Add single comment if only one comment.

Thanks for efficiency.

@windsonsea Thank you for your contribution, can you help to update the chinese documents synchronously?

Sure, that's my pleasure. Will handle if available.

LGTM
@windsonsea thanks for your contribution


Bug in resize/fit

In some cases xterm fails because an off by n error in the render function.
Details

Browser and browser version: Atom 1.19.0 / Electron
OS version: Mac OS X 10.12.6
xterm.js version: 2.9.1

Steps to reproduce

Create the terminal of size rows: 40 cols: 80
Immediately call fit which makes it bigger, e.g. {terminal: Terminal, cols: 82, rows: 45}
Crash in Render.

      for (let i = 0; i < width; i++) { // i = 80, width = 82, lines has 80 things in it.
        // TODO: Could data be a more specific type?
        let data: any = line[i][0]; // lines[80] is undefined.

A quick hack to work around this is to make sure the initial size is always bigger than the first resize:
    @term = new Terminal {
      rows: 400
      cols: 800
      scrollback: atom.config.get('script-runner.scrollback'),
      useStyle: no
      screenKeys: no
      handler: (data) =>
        @emitter.emit('data', data)
      cursorBlink: yes
    }

Seems like a massive hack though.

The problem is reproducible with this branch:
https://github.com/ioquatix/script-runner/commit/494399261ae7a08d438c72a7f7c3b5d4e9411d2c#diff-ff2e3d1abc5ef3094441fd3d980abbe4R95

(but you need to change cols: 80 * 4 to cols: 1 to crash every time).
Once the terminal is working/displaying/refreshing correctly, calling resize works as expected.

Duplicate of https://github.com/sourcelair/xterm.js/issues/860, it's already fixed in master. Hopefully a release should come soon 😃

Cool can you let me know when it's released? Thanks :)

It is: https://github.com/sourcelair/xterm.js/releases/tag/2.9.2 😄


fix(nexus): handle IO errors per channel

IO errors are now handled differently such that errors on a channel, are
handled on a per channel basis.
When we are in submission, we simply remove the device from the
channel(s) and retry the IO by resubmitting it into the nexus.
Completion IOs are,  when completed, also retried. For each failed
device, we kick off one retire request which will pause the nexus.  This
prevents any new incoming IO, giving us the time to persist data if any.
Before we can pasue the nexus, all requests in the qpairs must be
completed. As the channel is removed from the nexus its io channels,
queued IO to the child will be aborted.
This has been tested with 400 volumes in total. The script used are part
of the commit. They test purely the data and gRPC parts.  Added to that,
kudos to @tiagolobocastro for helping out debugging an issue that I,
locally, could not reproduce.

bors try

bors try

bors try

bors try

bors try

bors try

bors try

Bors try

bors try

bors merge

bors merge


Auto-link URLs in markdown

I'm fascinated to learn that this isn't an option provided by Earmark. I'm upping this from tiny to small in lieu of that - however, it did give me the chance to add another feature we wanted to turn on from earmark as well as see how earmark prefers to be used alongside syntax highlighting libs

Fixed in #23


It does not work on Swift Project

I am trying to use your library but the activity indicator does not start animating. :(
Could you suggest for me? Why it is?

Hello, I'm not very familiar with swift, so I can't definetely say what's wrong. I've heard that you should generate some 'bridging header' to use objective c code in swift project, or something like that.
If you share some code I'll try to find a problem.


docs: update Callouts formatting

Why:
Closes: #32554

What's being changed (if available, include any code snippets, screenshots, or gifs):

Replace "danger" with "caution"
Use multi-line syntax instead of single-line:

> [!DANGER] Be extremely careful.

becomes
> [!CAUTION]
> Be extremely careful.

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@johnnygerard Thanks so much for opening a PR with these fixes! I'll update the branch and get this merged once tests are passing 🍏


post analysis: same names are used for saving figures (png) and parameters

In Post Analysis (time-to-first-binding or dwell-time), the same file names are used for saving the results from analyzing fits from different models (Cosmos or cosmos-hmm).  So the most recent analysis overwrites the files from the previous analysis.

Thanks for noticing  it @jc-brandeis ! It has been fixed in the latest branch and will be released in the next pip version.


Support calculations across the complex domain

Right now the use of fma() restricts the ability to calculate upon complex arguments, even though the functions are mathematically defined across the complex plane.
Figure out how to change the code and still maintain accuracy in the typical real-argument case.

Closed by #15.


Nodejs compatiblity policy

[ADD] Nodejs compatibility policy
[UPDATE] Update travis to match compatibility policy.

Coverage remained the same at 95.745% when pulling 4f6eee24ed4b49c48158858e4a0973cda9fb9463 on stan-nodejs-compat into 3077dc1cc2f7423ee84244ccce001fd3e5818740 on master.


Update logos to use new v3 ML logo

favicon too!

Here are the logo files (from Dropbox).

On white navbar pages, we now have the v3 logo:

This is now done. We have v3 logo in white navbar and the footer.
I kept the old logo on dark navbar because it looks better imo


(Bug) #1329 Transaction in feed not showing amount

Description
Verify all fields do not contain undefined values before putting the object into gun db.
About #1329
How Has This Been Tested?
Try to create a send/receive transactions.
There shouldn't be issues with empty amount on feed card.
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] 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
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

also you didnt fix the log with all empty fields

The fields were empty because the enqueueTx fn failed and lot of data was lost - after the fix enqueueTx should not fail again and we won't have empty fields in logs.


Do not swallow response validation errors

Previously we would be discarding actual validation results and only return fixed text

@turkerdev any chance you could publish a new version?

What version should I give it? A new feature? Fix?

I would consider it a fix, it's not a feature per se


popen: Do not suppress stderr

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Suppressing stderr makes troubleshooting failed commands difficult.

One test is failing:

expected block to not output to stderr, but output "Error: No such file or directory - /usr/bin/fgrep\nError: No such file or directory - /usr/bin/fgrep\nError: No such file or directory - /usr/bin/fgrep\n"

https://travis-ci.org/Homebrew/brew/jobs/269276988#L1745

Would it make sense to guard this with a check to see whether debug is enabled, rather than unilaterally disabling it? Otherwise, this seems to have the potential to dump a lot of confusing output into the average user's stderr, potentially complicating matters.
(If something like this is already in place, feel free to disregard.)

I've addressed the above test failure by silencing the stderr of fgrep. The -s option to silence stderr is portable to both BSD grep and GNU grep.

@JCount I feel that ignoring error messages wholesale is a dangerous default. When a particular tool is expected to output error message (like fgrep which caused a test to fail), its stderr should be selectively silenced.

I was advocating for selective silencing, not against it. Using the developer mode variable or the debug flag were merely a basic example to try to clarify what I meant.

I'm not sure about making this change. This is essentially a public formula API at this stage and will make formulae start outputting things they didn't before. I think it's a possibility if, as @JCount suggests, it's guarded on ARGV.debug? and/or ARGV.verbose?. CC @ilovezfs for some thoughts here, too.

I've modified the logic to be
$stderr.reopen("/dev/null", "w") unless ARGV.debug? || ARGV.verbose?

I'm still happy to receive comments and change it as desired.

So if we do foo = Utils.popen_read("blah") does foo get set to some mixture of stdout and stderr? If so, that is not good.

e.g. I'm pretty sure this will blow up https://github.com/Homebrew/brew/blob/master/Library/Homebrew/language/node.rb#L13

So if we do foo = Utils.popen_read("blah") does foo get set to some mixture of stdout and stderr?

No, the $stderr of blah does not go to foo. $stderr of the child process goes to $stderr of the parent process, which is typically the console for an interactive brew process.

Why did the fgrep need to be changed if this doesn't affect stdout?

expected block to not output to stderr, but output "Error: No such file or directory - /usr/bin/fgrep\nError: No such file or directory - /usr/bin/fgrep\nError: No such file or directory - /usr/bin/fgrep\n"

https://github.com/Homebrew/brew/pull/3099#issuecomment-325436813
It caused a brew tests failure that checked that the output of $stderr was empty when installing a bottle with a broken symlink.

codecov/patch — 0% of diff hit (target 50.49%)

@sjackman do you mind trying to raise that percent?

Both lines of this diff are hit by test/dev-cmd/bottle_spec.rb:20. I know because that test failed without adding the -s option to grep.
https://travis-ci.org/Homebrew/brew/jobs/269276988#L1745
https://github.com/Homebrew/brew/pull/3099#issuecomment-325436813
Doesn't that mean that 100% of the diff is covered? Or have I misunderstood?

I know because that test failed without adding the -s option to grep.

was that before or after you scoped it to debug and verbose?

This line is definitely being run.
Utils.popen_read("/usr/bin/fgrep", "-s", recursive_fgrep_args, string, to_s) do |io|

https://github.com/Homebrew/brew/pull/3099/files#diff-26acfe09641ad4e7cf4f22179625981eR99

$stderr.reopen("/dev/null", "w") if !ARGV.debug? && !ARGV.verbose?

https://github.com/Homebrew/brew/pull/3099/files#diff-83e8c13ed518191365f4edaa7339293dR16
This line is run too. I believe ARGV.debug? and ARGV.verbose? are both false when Jenkins runs, so only the one code path is being tested, where !ARGV.debug? && !ARGV.verbose? is false and $stderr is closed.

@sjackman It's unclear to me: are you saying the CI fails without that change being made?

CI passes. The fgrep -s change is required if brew tests is run with either ARGV.debug? or ARGV.verbose? enabled, but that doesn't seem to be the case.

The fgrep -s change is required if brew tests is run with either ARGV.debug? or ARGV.verbose? enabled, but that doesn't seem to be the case for CI brew test-bot.

:cool:, you can remove that change then, thanks.

Done!

Thanks again @sjackman!

Thanks for merging, Mike!


Remove Vagrant related files

The Vagrantfile has not been updated since 2016. The list of supported OSes do not match current reality.

Feel free to close out if someone is going to keep Vagrant going here.


Externalized PrimeVue Resolver

I have updated and externalized the PrimeVue resolver:

NPM
GitHub

I am not affiliated with Primefaces/PrimeVue in any way, and I understand if you don't want to link an unofficial resolver.
But it exists now, and I'll do my best to keep it updated, so feel free to link it wherever you deem it appropriate.

Ah yes, I am fairly new to publishing packages. I believe I fixed that. Do you mind checking again?


SLSA Level 1 compliance for Istio Release Process

Describe the feature request
SLSA is a security framework, a check-list of standards and controls to prevent tampering, improve integrity, and secure packages and infrastructure in your projects, businesses or enterprises. The objective of this issue is to track the progress of Istio Release Process to achieve SLSA Level 1 compliance. While most of the things seem to be already in place for Level 1 compliance, what is missing is a provenance attestation.
https://github.com/kubernetes-sigs/tejolote seems to be coming up with a provenance attestation framework which is soon going to support prow as well. It would be a good choice to add the attestation for Istio.
The compliance checklist for SLSA for Istio is tracked here.
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

cc @ericvn @howardjohn

/cc

@puerco let us know once the prow support is in place for tejolote. Another option is https://github.com/testifysec/witness , have you explored?

cc @fkautz

https://github.com/knative/test-infra/pull/3748 seems to be a good start for us, as this has prow integration. I will try this out and see how it goes

Looks like SLSA checklist has been updated: https://slsa.dev/spec/v1.0/whats-new

/not stale


Which protocol is this? I only get strange letters..

Hello!
I got a new GPS tracker from Amazon.de (sold by a marketplace seller).
SMS-Link works with "smslink123456". Setting APN and IP works also.
The tracker is connected to my traccar server and is sending data like this;
78781f12110712080b1ac905a70c9e011057bc003400010601572c00d9f31910fb120d0a

When I convert it, I get
xx...

I think, something it going wrong?
Can you identify the protocol or is it just nonsense? I already tried port 5000, 5001 and 5006 but it is not recognized,
Many thanks!

It's GT06 protocol, port 5023 in Traccar. It's a binary protocol, so you can't use hex decoder on it.

Hello!
Many thanks, the device is recognized now and I get the correct position :-)


Simplify examples to ease getting started

I may be wrong but the examples are trying to listen on port 0?

@andig Yes, this is intended:

In order to use a random port assignment, you can use the port 0:

https://github.com/reactphp/socket#server

@clue another thing learned, thank you!


Webpack compatible version

Hi,
This is a great lib, especially the infinite length feature.
Is it possible to get a published version that is compatible with webpack using ES5 exports?
Many thanks!

Hi James,
locally I work on a new export pattern, which allows all my node modules to work with webpack, typescript, AMD/UMD, ... I need to finish this work and bring it to the modules. If you know a quick fix to keep you running, let me know.

Robert,
Thanks so much for the prompt reply, that's great to hear.
I'm working on something today that requires bitset, and it works locally with imports but the build fails.
If you could publish a working copy that works with webpack on a separate branch, or even just a gist so I could include it as vendor code outside of node_modules that'd be fantastic.

From the code site, what changes are required? Export the module in ES6 style?

Exporting as ES5 style, npm modules as ES6 still tends to cause issues with build systems.

bitset.js is getting exported with ES5. What is your actual error?

tsconfig.json:
{
  "compilerOptions": {
    "baseUrl": "src",
    "outDir": "build/dist",
    "module": "es6",
    "target": "es5",
    "lib": ["es6", "dom"],
    "sourceMap": true,
    "allowJs": true,
    "jsx": "react",
    "moduleResolution": "node",
    "rootDir": "src",
    "forceConsistentCasingInFileNames": true,
    "noImplicitReturns": true,
    "noImplicitThis": true,
    "strictNullChecks": true,
    "suppressImplicitAnyIndexErrors": true,
    "noUnusedLocals": true,
    "allowSyntheticDefaultImports": true
  },
  "include": [
    "src/**/*"
  ],
  "exclude": [
    "node_modules",
    "build",
    "scripts",
    "acceptance-tests",
    "webpack",
    "jest",
    "src/*.test.tsx",
    "src/setupTests.ts"
  ],
  "types": [
    "typePatches"
  ]
}

error:
 npm run build

><EMAIL_ADDRESS>build /Users/xxxxx/Data/code/xxxxxx/client
> react-scripts-ts build

Creating an optimized production build...
Starting type checking and linting service...
Using 1 worker with 2048MB memory limit
ts-loader: Using<EMAIL_ADDRESS>and /Users/xxxxx/Data/code/xxxxx/client/tsconfig.json
No valid rules have been specified
Failed to compile.

Failed to minify the code from this file:

        ./node_modules/bitset/bitset.js:50

Read more here: http://bit.ly/2tRViJ9

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `react-scripts-ts build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Could you please try to configure bitset.js to be ignored by the minimizer of webpack?


Messaging Service Telegram not working anymore

Describe the bug
Using latest version 0.112.5.  I recognised that since version 0.111 telegram service is not working anymore. When I downgrade to version 0.106.5 it is working again with same evcc.yaml config.
Steps to reproduce

see description

...
Configuration details
messaging:
��events:
����start:
������msg: ${title} Wallbox started charging ${vehicl>
����stop:
������msg: ${title} Wallbox finished charging ${vehic>
��services:
����- type: telegram
…
(Rest of config deleted)

Log details
Feb  7 21:24:09 raspberrypi evcc[18548]: [push  ] ERROR 2023/02/07 21:24:09 invalid message template for start: wanted: [vehicleSoC], got: [tariffGrid chargeDuration vehicleDetectionActive vehicleIcon vehiclePresent homePower bufferSoc batteryConfigured savingsEffectivePrice phaseAction savingsSelfConsumptionPercent vehicleTitle savingsSince phasesConfigured auth chargedEnergy minSoc vehicleRange connectedDuration title savingsTotalCharged prioritySoc phasesActive chargePower chargeCurrents savingsGridCharged pvConfigured vehicleSoc pvAction savingsSelfConsumptionCharged connected gridConfigured tariffFeedIn residualPower pv loadpoint pvPower vehicleTargetSoc savingsAmount maxCurrent phaseRemaining climater planActive vehicleFeatureOffline enabled vehicles mode currency charging siteTitle vehicleCapacity chargeCurrent targetSoc minCurrent pvRemaining gridPower version vehicleOdometer phasesEnabled targetEnergy]
Feb  7 21:24:19 raspberrypi evcc[18548]: [lp-1  ] INFO 2023/02/07 21:24:19 stop charging <-
Feb  7 21:24:19 raspberrypi evcc[18548]: [push  ] ERROR 2023/02/07 21:24:19 invalid message template for stop: wanted: [vehicleSoC], got: [maxCurrent planActive auth prioritySoc chargeDuration phasesEnabled phaseAction chargeCurrents savingsGridCharged vehicles connectedDuration phaseRemaining vehicleFeatureOffline vehicleDetectionActive bufferSoc pvPower targetEnergy savingsSince savingsAmount charging vehicleSoc vehicleCapacity vehicleTargetSoc connected vehicleTitle minCurrent vehicleOdometer pvConfigured batteryConfigured vehicleIcon currency chargePower chargeCurrent phasesConfigured targetSoc savingsSelfConsumptionCharged pvRemaining mode tariffFeedIn climater minSoc title vehiclePresent chargedEnergy vehicleRange enabled siteTitle residualPower version tariffGrid pvAction homePower savingsSelfConsumptionPercent loadpoint savingsEffectivePrice gridPower phasesActive pv savingsTotalCharged gridConfigured]
Feb  8 02:35:53 raspberrypi evcc[18548]: [telegram] ERROR 2023/02/08 02:35:53 Post "https://api.telegram.org/bot***/getUpdates": read tcp [2a01:c23:9172:5400:d8c:b11d:84ea:290f]:40816->[2001:67c:4e8:f004::9]:443: read: connection timed out

What type of operating system are you running?
Linux
Version
evcc version 0.112.5

Use
vehicleSoc


Packaging with the DLL files bundled into the exe?

Heya!  I started from your source code to make a little Doom Eternal-specific overlay, and I got the functionality working quite well, but I'm wondering what approach you used to package your C# app without having 356 extra files in the same folder as the application.

You can see in .csproj file next properties:
<PublishSingleFile>true</PublishSingleFile>
<SelfContained>true</SelfContained>
<PublishTrimmed>true</PublishTrimmed>
<PublishReadyToRun>true</PublishReadyToRun>

Then I run from VS: right click on project in solution explorer -> Publish. It shows additional page with some settings. And there is Publish button for single publish profile.

Got it!  Thank you very much :D.
And a small teaser of a high end Doom Eternal guy using the new app:
https://streamable.com/2bl9o3
https://streamable.com/euzswz
Will make a proper fork in Github soon and release it when I have a chance.  Thank you very much for your help and the base code to make this possible - very slick little project you made :)


Fix build-XLA (3.7, nightly)

What does this PR do?
Noticed it was failing due to an outdated version mapping
Failure: https://github.com/PyTorchLightning/pytorch-lightning/runs/2404985974
Before submitting

[n/a] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[n/a] Did you make sure to update the documentation with your changes? (if necessary)
[n/a] Did you write any new necessary tests? (not for typos and docs)
[n/a] Did you verify new and existing tests pass locally with your changes?
[n/a] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

The version issue is now fixed, but it still fails because a package is uninstalling torch nightly and installing 1.8.1
Marking as draft until resolved


match query doesn't support search syntax, simple_query doesn't support synonyms

From the note on multiword synonyms:

… because the query_string query supports a terse mini search-syntax, it could frequently lead to surprising results or even syntax errors.

One of the gotchas of this query involves multiword synonyms…

As part of this parsing process, it breaks up the query string on whitespace, and passes each word that it finds to the relevant analyzer separately. This means that your synonym analyzer will never receive a multiword synonym. Instead of seeing United States as a single string, the analyzer will receive United and States separately.

So our multi-word synonyms like "close management" will break because queries are parsed into tokens that won't match the synonym mapping.
How to fix?

Mapping multi-word phrases to single-word phrases works some of the time:
"use force,officer-involved shooting,death custody,arrest-related death=>useofforce",

That appears to work for "death in custody" as it becomes useofforc. Does not work without quotation marks. Oddly, "officer-involved shooting" and others do not get this behavior.

Perhaps the solution is to do away with the synonym mapping and do expansion on tags when they are put into the search index or something.
A prepare_tags method could check that items tagged with "close management" or "solitary housing unit" had "shu", etc. added when the index is created or updated. As long as the tag expansion happened, then the possibilities would be captured, with the downside being a larger search index.

Wherein I record how I am dumb on Github for all to see. Essentially, there are additional points of override that SimpleESBackend should override because searches like q=juvenile justice are getting transformed into "(juvenile justice)" syntax (query string syntax) and we are passing that into a match search.
Stack for processing/building query that gets sent to build_search_kwargs:

run (ElasticsearchSearchBackend)

build_query (BaseSearchQuery)
as_query_string (BaseSearchQuery)
build_query_fragment (ElasticsearchSearchQuery)

Possible solution in 66f4e469e629d8e91ed5725589a26e441a138aae.

Calling this done.


TerraformPlanResultsPRPlugin: simplify configuration

Repo slug can be constructed from the scm URL
The base URL can be constructed from the scm URL
Remove landscape functionality from the plugin - there's a separate Landscape plugin

This functionality can be assigned to the original issue


Update data augmentation docs.

This PR closes this issue #9691 and only updates the Data Augmentation section of the policy docs page.
Proposed changes:

Make it explicit that augmentation is a crucial parameter when trying to speed up Core training, but less augmentation can make TED's performance worse
Correct the docs to state the correct default augmentation factor (they say it's 20 but it's 50).
Make it explicit that augmentation affects only TED (out of the policies we provide)

Status (please check what you already did):

[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

@samsucik I created this draft PR to discuss the issue since we can see the proposed edits side-by-side with the original.
Regarding these optional changes:

If your config contains TEDPolicy and no memoization policies,  start with story augmentation turned on (e.g. the default value of 50)
Otherwise (if a memoization policy is used too), try out also with story augmentation turned off -- to potentially speed up training

For the first item, I think we don't need to say this because by default augmentation is on at 50.
The second point I've folded into the tradeoff between speed and prediction performance in final paragraph:

We recommend using a memoization based policy along with TEDPolicy when reducing the amount of data augmentation to compensate.

WDYT?


Remove example configs from repo root

The config files in configs/ in this repository have no purpose except as examples, this keeps confusing people so it's best to either delete them or move them into the docs. If the latter option is chosen there should still be tests to make sure they don't go out of date.

connected to #352 where imports from ophyd are used as demonstrations of example_devices.
alternatively a new repo with blueapi examples could be made

just delete, examples are already in https://gitlab.diamond.ac.uk/daq/d2acq/examples/daq-deployments

there's 1 small test that relied on the defaults.ymal - can it be deleted  - @callumforrester ?