Strange formatting of @, |, and parens patterns

In a rust-lang/rust PR, I saw some pretty strange formatting of pattern matching with @, |, and parens patterns: https://github.com/rust-lang/rust/pull/70906/files#r405165086
The diff was:
-                    category: category @ ConstraintCategory::Return,
+                    category:
+                        category
+                        @
+                        (ConstraintCategory::Return
+                        | ConstraintCategory::CallArgument
+                        | ConstraintCategory::OpaqueType),

which seems odd because it is putting @ on its own line and doing ( on the same line as a pattern and then ) also on the same line as a pattern.
I would have expected:
-                    category: category @ ConstraintCategory::Return,
+                    category: category @ (
+                        ConstraintCategory::Return
+                        | ConstraintCategory::CallArgument
+                        | ConstraintCategory::OpaqueType
+                    ),

Not a duplicate but similar report in #4031. Fixes will likely be similar (if not identical)

Did this pattern exist before, or is it something new that has been introduced recently? Either way, we should fix this.

I believe it's a new-ish change, but not sure precisely when it was introduced

category: category @ Record { ... }

has existed since Rust 1.0 I think, so that isn't new, but p0 | ... | pn nested within another pattern is new and currently an unstable feature.

Oops, I misunderstood the question anyway. I don't know definitively but I believe that the poor binding formatting described in #4031 was a change rustfmt started doing relatively recently.
Not sure what, if any, impact that may have on the formatting here

Happens equally as root for a match arm:
match head.packet_type()? {
    p
    @
    (PacketType::Connect
    | PacketType::ConnAck
    | PacketType::SubAck
    | PacketType::UnsubAck) => return Err(Error::WrongPacket(p)),

Manually, I would probably format it as:
match head.packet_type()? {
    p @ (
        PacketType::Connect
        | PacketType::ConnAck
        | PacketType::SubAck
        | PacketType::UnsubAck
    ) => return Err(Error::WrongPacket(p)),

Interestingly enough, we still end up with N+2 lines. But moving the output expression to the left is definitely a huge win.

"Fixing" this by locally by increasing max_width for now (not really a fix...) but this is definitely something that would make rustfmt much less annoying when using nested or patterns.
I wonder if this would be more worthwhile to investigate now that https://github.com/rust-lang/rust/issues/54883 is stabilized (?), etc.

This was fixed in #4978, and a test case was added for this issue


Test skip on and off in RDO loop

master@bf537 -> skip@14ce4
   PSNR | PSNR Cb | PSNR Cr | PSNR HVS |    SSIM | MS SSIM | CIEDE 2000
-0.7872 | -1.1801 | -1.2715 |  -0.1556 | -0.2205 |  0.4549 |    -0.3761

Great- I expect more gains comes from lower rates?

@ycho Yes, on the sequences where this makes a difference, the gains are mostly at the lower rates

Can you also share the change of encoding time with skip testing please?

@ycho The full data is at https://beta.arewecompressedyet.com/?job=master%40bf537&job=skip%4014ce4
Seems actually to run a bit faster (maybe because things can be skipped during final encode)

Let me do final testing of comparing enc and dec videos for nyan at -s 0 and 1.


i18n(es): translate tabs

Description

What does this PR change? Give us a brief description.
Translates tabs to spanish.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

es
components/tabs.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Installation on Windows ?

Hi,
I wanted to give a try to prql in duckdb on a Windows-10 laptop (duckdb-0.8.1).
I couldn't install it. Here are the steps:
> duckdb -unsigned
v0.8.1 6536a77232
Enter ".help" for usage hints.
D SET custom_extension_repository='welsch.lu/duckdb/prql/latest';
D FORCE INSTALL prql;
Error: HTTP Error: Failed to download extension "prql" at URL "http://welsch.lu/duckdb/prql/latest/v0.8.1/windows_amd64/prql.duckdb_extension.gz"

Candidate extensions: "parquet", "sqlite", "sqlite3", "sqlite_scanner"
D

Should it be possible as of now ? prql cli compiler works on Windows...

Thanks for your answer.
I have tried again. It went better, but I am still noticing things that I want to report.
I could install a first time, but another trial gave me this error. Probably Windows only
> duckdb.exe -unsigned
v0.8.1 6536a77232
Enter ".help" for usage hints.
D SET custom_extension_repository='welsch.lu/duckdb/prql/latest';
D FORCE INSTALL prql;
Error: IO Error: Could not move file: Impossible de créer un fichier déjà existant. (impossible to create an existing file)

D load prql;
> -- seems OK

I removed extenstions from ~/.duckdb/extensions/v0.8.1/windows_amd64
-a---          2023-09-06    09:36       21632000 prql.duckdb_extension
-a---          2023-09-06    09:41       21632000 prql.duckdb_extension.tmp-0fa5bddd-36f7-44aa-9b35-ce707671d01d
-a---          2023-09-06    09:45       21632000 prql.duckdb_extension.tmp-8f4eacf3-d03f-44a7-9e28-bda6e963299f

Indeed, FORCE INSTALL doesn't work on Windows as it leaves prql.duckdb_extension.tmp-xxx extensions
I made another simple test which gives me unexpected results (I'm just playing with PRQL, so might be my mistakes).
> duckdb.exe -unsigned
v0.8.1 6536a77232
Enter ".help" for usage hints.
D create table t (name text, age int, primary key(name));
D insert into t values('bob', 20);
D from t;
┌─────────┬───────┐
│  name   │  age  │
│ varchar │ int32 │
├─────────┼───────┤
│ bob     │    20 │
└─────────┴───────┘

D load prql;
D from t
> filter name == 'bob';
┌─────────┬───────┐
│  name   │  age  │
│ varchar │ int32 │
├─────────┼───────┤
│ bob     │    20 │
└─────────┴───────┘
D from t | filter name  == 'bob' | derive [ a = 10,  age10 = age + a, ];
Error: Parser Error: Error:
   ?─[:2:60]
   │
 2 │ from t | filter name  == 'bob' | derive [ a = 10,  age10 = age + a, ]
   │                                                            ───┬───
   │                                                               ?───── array expected type `int`, but found type `int || float || timestamp || date`
───?

Is it a duckdb or prql extension issue?

Re force install issue: The install should only be done the very first time, i.e., it's not recommended to run this command every time you run duckdb, as it will redownload the extension from the custom repo. After an initial installation, just doing a load prql is enough (no need to set custom extension repository either then). Windows is known to have issues with overriding files (which is implemented atomically with temp file + move in DuckDB) when these files have been mapped into a process before. This is not an issue specific to my extension (but applicable to all extensions). An alternative is not to use force install, but just regular install. In that case it will skip the installation when the extension is already present.
Re parser error:
PRQL has recently changed the syntax from square brackets to curly brackets, see https://prql-lang.org/book/reference/stdlib/transforms/derive.html
The example works if you write from t | filter name  == 'bob' | derive { a = 10,  age10 = age + a, }

Thank you.
Using FORCE INSTALL was just a test. I know it's a recurring issue on Windows.
Sorry for the syntax error. I read about the change from [] to {} in release 0.9.0, but didn't think about it when testing, nor did I check the official documentation.
Would you mind updating your README.md that I was following blindly?
Thanks again!

Would you mind updating your README.md that I was following blindly?

Thanks for pointing out the out-of-date README - I've updated it now.


Update virtual-machines-image-builder-overview.md

A few typos

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good. Thanks @TomHickling


Build an Automated Data Pipeline

Re-open for rearrange functions into classes for better understanding.

Pipeline.py modified into class layout.


Add dotenv configuration, update manual Installation steps in README.md

Haven't updated the Docker Installation steps, please make the required changes if necessary.

Is there something i need to change?

Please do not forget to shut down the firebase project associated with this project as the firebase options are still visible in the previous commit history. Leaving such secrets exposed might result in misuse of the your personal resources.


pytorch tutorial fixed position missing torch.manual_seed()

in pytorch tutorial, block :
Let's try with a fixed initial position
does not actually fixes the position, torch manual seed should also be used.
agent = DQNAgent(env.observation_space, env.action_space)
seed = 10
torch.manual_seed(seed)
env.run(agent, episodes=300, seed=seed)

ok. do you mind opening a pull-request for that ?


Defaultable configurable record variable throws error when value is not provided

Description:
It throws an error when a configurable variable with record type that contains defaultable fields and the configuration is not provided.
type Person record {
};

type Test record {
   Person person = {"age" :23};
};

configurable Test test = {};

Error:
error: {ballerina/lang.map}InherentTypeViolation {"message":"invalid value for record field 'person': expected value of type '(foo:Person & readonly)', found 'foo:Person'"}

Expected behavior :
A record value {person : {age : 23}} is expected for variable test
Related Issues (optional):
Related to spec issue https://github.com/ballerina-platform/ballerina-spec/issues/544

Same as https://github.com/ballerina-platform/ballerina-lang/issues/33773, except via configurable variable.


Fix copyright headers

Some copyright headers were pointing to wrong file

Should this be backported to 3.5.x?

Should this be backported to 3.5.x?

Can't imagine why - the affected files are all part of tests. Did you have a reason in mind?


Hermes crashes on sleep 10.11.2

Hermes_2015-12-25-222038_Herb-Rowes-iMac-2.crash.zip
Crashes either on sleep or when it wakes up, pretty much every time.

+1 yes for me as well

I see it crashing often as well after wake with segmentation fault 11.

Can you attach a crash log? Thanks

I have the same problem for which I have posted logs over on the Apple Community forums. I have been trying to troubleshoot the issue for a couple of weeks. The thread refers to both Hermes (current version) and HD Radio for Pandora under OS X El Capitan because I tried alternating (uninstalling/reinstalling) apps when I couldn't prevent them from crashing every 7-12 minutes. At this point I am leaning toward it being sleep related. The topic is titled "Two Different Pandora Apps Hang on MacBook (Crashlog Help)":
https://discussions.apple.com/thread/7805488
My crashlogs refer to the System "Audio Toolbox" and to a call for a sharing/remote function of some sort that doesn't appear to correspond to any of my system/library preference files. I'm at a total loss and about to reinstall the OS to see if I can fix it because I've tried everything else, including network diagnostics, renewing the DHCP lease, turning off file sharing and trashing power management, bluetooth preferences, etc.
Since I am rather new to Hermes one of my questions when running it on a MacBook Pro (mid-2012) is whether or not disconnect-on-sleep is inevitable short of totally disabling HD sleep in Energy Saver? A couple of times I had the app remain active, just suspended with the dialog that it requires an "active Internet connection" but most of the time the app just crashes outright on sleep, always at the conclusion of a song (before it's time to load more into the buffer, presumably). Most of the time the only feedback I get is a System message asking if I want to report the issue to Apple, which I have done numerous times.
I don't have any other wi-fi problems. I've tested it on and off ethernet. And I have another Mac running El Capitan with no issues during sleep on the same network, so not likely to be my network at issue.
I hope filling in this information (along with the logs on the Apple forum) can help get to the bottom of this!
Thank you & Happy New Year!

Here are the extracted portions of my latest Hermes/System crashlog that seems relevant; however, I do not know how to fix it. Will someone PLEASE jump in and tell me what this process is hanging up on?
Crashed Thread:        0  Dispatch queue: com.apple.main-thread
Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       EXC_I386_GPFLT
Exception Note:        EXC_CORPSE_NOTIFY
…
Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.audio.toolbox.AudioToolbox	0x00007fff995be285 MP3AudioStream::GeneratePackets(AudioFileStreamContinuation&) + 1527
1   com.apple.audio.toolbox.AudioToolbox	0x00007fff995a2630 AudioFileStreamObject::ParseBytes(unsigned int, AudioFileStreamContinuation&) + 108
2   com.apple.audio.toolbox.AudioToolbox	0x00007fff995a1ebf AudioFileStreamWrapper::ParseBytes(unsigned int, void const*, unsigned int) + 97
…
Thread 0 crashed with X86 Thread State (64-bit):
rax: 0xa00007fb3c8c4d69  rbx: 0x0000000000000688  rcx: 0x0000000000000000  rdx: 0x0000000000000688
rdi: 0x00007fb3c904c200  rsi: 0x0000000109908c5f  rbp: 0x00007fff56326a80  rsp: 0x00007fff563269a0
r8: 0x0000000000000006   r9: 0x0000000000000000  r10: 0x00007fb3c983ba00  r11: 0x00000001099263a8
r12: 0x0000000000000688  r13: 0x0000000000000004  r14: 0x00007fb3c8ca03e0  r15: 0x00007fb3c8ca03e8
rip: 0x00007fff995be285  rfl: 0x0000000000010246  cr2: 0x0000000105462000
Logical CPU:     0
Error Code:      0x00000000
Trap Number:     13

UPDATE
Good news…
Over at "AskDifferent" I was told to check for third-party audio drivers. Turns out there was a "Soundflower" extension loaded and when I removed that I resolved the Hermes "unexpectedly quit" app crashes upon wake. Now the app simply suspends when it loses wi-fi until I wake the MacBook (at which point I can resume playback).
I still need assistance (please, anyone… Help?):
Current problem is, I want an option in Hermes (and apps like it) to "keep awake" the network connection while active, and yet that checkbox/preference doesn't appear to exist.
One suggestion was to use "Wimoweh" or "Amphetamine" from the App Store to selectively "keep alive" wi-fi connectivity for Hermes (current version). Therein lies my question. Per the Wimoweh FAQ, there is a way to program apps like Hermes to make "assertions" that will allow the wi-fi network to remain active. The app has to be programmed to convey to OS X that it is indeed busy (streaming Pandora) and not to sleep the computer while running (at least not to the point where wi-fi disconnects). Here's the quote from the FAQ:
"If you’re using Wimoweh to stop the system sleeping to handle one specific app it might be worth asking the developer to check out IOPMAssertionCreateWithDescription in Apple’s developer documentation. In just a few lines of code they can have their application inform OS X when the application is busy and then you, in the best possible way, will no longer need to use Wimoweh. "
Now I wish that Apple had seen fit at any time in the history of OS X to place a tick-box in the Energy Saver pane to prompt the user to "decide" whether to sleep network connections while idle (yes/no). A "no" would mean that all sleep settings apart from the network connection are active but wi-fi/network connectivity does not terminate with sleep. But since Apple doesn't encode the OS this way, and the "Wake for Network Activity" is actually counterproductive — some discussions indicate that "Wake for Network Activity" has nothing to do with keeping wi-fi connections active for downloads, mail pickup of streaming and might actually trigger a network disconnect problem! — I do believe that it's Apple's "bad" not to have made it user-configurable within the OS to perform a TCP/IP "keepalive" function (short of messing around in Terminal).
Although coming straight from Snow Leopard to El Capitan I didn't appreciate as much, apparently it is well established that with the introduction of Mountain Lion OS X more aggressively enforced sleep settings. So either A) Apple needed to allow OS X to present the user with the option to sleep (or not sleep) network connections, or B) Hermes and apps similar to it, such as HD Radio for Pandora, need to encode ASSERTIONS to keep wi-fi/network connections OPEN as long as the audio streaming is active at the time the computer attempts to enter sleep.
It would appear that "Hermes" isn't actively maintained from some of the other messages I've read here on GitHub (which is sad). But SURELY a still-current user/fan can help me understand why Hermes has all these nifty features of being able to "pause" for screen saver or "pause" for screen lock" — and yet best I can tell there's nothing in the app or user-preferences to implicitly tell the OS to keep the wi-fi/network connection active?????
I am seeing from my previous crashlogs that audio streams terminate at sleep. So more than just being a conflict with some outdated third-party Audio driver, the audio-stream disconnects trace back to a timeout. Hermes would appear to be calling for the next batch of Pandora content to enter the buffer but because OS X (El Cap) has slept the network connection it can't reach the Pandora sever and then the disconnect error comes up (and in some cases this also triggers a wholesale crash of the app). It seems like a no-brainer that either the Assertion needs to be encoded per the Wimoweh developer advice or, if I am mistaken and the app should be able to keep running in the background even at idle than at least let me know that much (so that I can conclude that my OS X install is at issue rather than some inherent shortcoming of the Assertion encodings in the app).
To recap the above in less technical terms:
Hermes may not be actively maintained but surely there are people using the current version who can tell me if THEIR Macs lose the audio stream between Pandora/Hermes every time the system enters sleep.

Thanks for the research but I'm still having a bit of trouble understanding this.
Hermes doesn't do anything to prevent system sleep.  The issue you saw which resolved when you removed Soundflower is likely an OS bug and nothing to do with Hermes specifically.
I don't ever listen to Hermes when I'm not otherwise using my Mac so I haven't ever had this problem. Hermes doesn't keep a network connection open to Pandora constantly, instead it downloads a few songs at a time; this is just how Pandora works.
From reading your post on the Apple Support site, it seems you may actually be running into another issue (#285), which appears to be specific to some changes Apple made in 10.12, and unfortunately which I can't reliably reproduce to see if there's a way for Hermes to work around the problem.  I do have some vacation time next week and will try once again to see if I can reproduce it, but if anyone else is able to do some digging on how I can reproduce the crash/stutter reliably, that would be really appreciated.


Python unittest test cases with 96% coverage and unicode-only internal processing.

Sorry to disturb. It looks like a bug:
❯ pythonsed -h
usage: pythonsed [-h] [-H] [-v] [-c ENCODING] [-f file] [-e string] [-i [backup suffix]] [-n] [-s] [-p] [-r] [-l LINE_LENGTH] [-d DEBUG] [targets ...]

sed.py - python sed module and command line utility

positional arguments:
  targets               files to be processed (defaults to stdin if not specified)

options:
  -h, --help            show this help message and exit
  -H, --htmlhelp        open html help page in web browser
  -v, --version         display version
  -c ENCODING, --encoding ENCODING
                        input encoding
  -f file, --file file  add script commands from file
  -e string, --expression string
                        add script commands from string
  -i [backup suffix], --in-place [backup suffix]
                        change input files in place
  -n, --quiet, --silent
                        print only if requested
  -s, --separate        consider input files as separate files instead of a continuous stream
  -p, --python-syntax   Python regexp syntax
  -r, -E, --regexp-extended
                        extended regexp syntax
  -l LINE_LENGTH, --line-length LINE_LENGTH
                        line length to be used by l command
  -d DEBUG, --debug DEBUG
                        dump script and annotate execution on stderr

Option -c can be repeated multiple times. The first is going to be used as default encoding and for stdout output, writing to files (commands w and W) and reading from files (commands r and R). For script literals (-e), script files (-f) and input files the last preceding -c option is used to define that part's encoding.

Options -e and -f can be repeated multiple times and add to the commands executed for each line of input in the sequence they are specified.

If neither -e nor -f is given, the first positional parameter is taken as the script, as if it had been prefixed with -e.
Traceback (most recent call last):
  File "/home/wzy/.local/lib/python3.10/site-packages/PythonSed/sed.py", line 3400, in main
    args = ParseArguments()
  File "/home/wzy/.local/lib/python3.10/site-packages/PythonSed/sed.py", line 3365, in __init__
    self.args = parser.parse_args()
  File "/usr/lib/python3.10/argparse.py", line 1826, in parse_args
    args, argv = self.parse_known_args(args, namespace)
  File "/usr/lib/python3.10/argparse.py", line 1859, in parse_known_args
    namespace, args = self._parse_known_args(args, namespace)
  File "/usr/lib/python3.10/argparse.py", line 2068, in _parse_known_args
    start_index = consume_optional(start_index)
  File "/usr/lib/python3.10/argparse.py", line 2008, in consume_optional
    take_action(action, args, option_string)
  File "/usr/lib/python3.10/argparse.py", line 1936, in take_action
    action(self, namespace, argument_values, option_string)
  File "/usr/lib/python3.10/argparse.py", line 1100, in __call__
    parser.exit()
  File "/usr/lib/python3.10/argparse.py", line 2570, in exit
    _sys.exit(status)
SystemExit: 0

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/wzy/.local/bin/pythonsed", line 8, in <module>
    sys.exit(main())
  File "/home/wzy/.local/lib/python3.10/site-packages/PythonSed/sed.py", line 3447, in main
    traceback.print_exception(*sys.exc_info(), file=byteStream)
  File "/usr/lib/python3.10/traceback.py", line 121, in print_exception
    print(line, file=file, end="")
TypeError: a bytes-like object is required, not 'str'

And why when sed.no_autoprint = True, it still output a;? Is it a bug?
In [16]: from PythonSed import Sed
In [17]: sed = Sed()
In [18]: sed.no_autoprint = True
In [19]: sed.load_string("s/2/a/g;2aa\\;")
In [20]: sed.apply('/home/wzy/Desktop/a.txt')
a;
Out[20]: ["a;\n"]

At last. Why not upload this package to PYPI? Thanks.

@Freed-Wu
You were absolutely right! The exception for pythonsed -h was definitely a bug and it is fixed now.
The output "a;\n" is not a bug. The command a in sed will always print - otherwise that command does not have any effect. The GNU sed is doing exactly the same thing.
I also added the necessary GitHub actions to publish all of this as package pythonsed to PyPi.org, thus you can now install it with "python -m pip install pythonsed" and that will give you the Python package PythonSed as well as an executable script "pythonsed" to run the main function in sed.py from command line.


Fixed the file searching code for linux also

Please fix your conflicts so I can merge the PR

Please see #82


Help getting simple plugin working

I'm new to developing neovim plugins and I'm trying get a simple node plugin working, but when I run :UpdateRemotePlugins it throws this error:

Here is my plugin:
/* globals plugin, debug */
'use strict';

debug('In plugin');
var fmt = require('util').format,
	numCalls = 0;

function incrementCalls() {
	if ( numCalls === 5 ) {
		debug('too many calls');
	}
	numCalls++;
}

plugin.command('testCmd', {
	range: '',   // Line 16
	nargs: '*'
}, function (nvim, args, range) {
	debug('in testCmd');
	incrementCalls();
	nvim.commandOutput( fmt('Test cmd press: ', numCalls, ' times. ', range) );
});

You can find the rest of the project at neovim-autoformat.
I try to run :testCmd in nvim and it gives me the E492: Not an editor command: testCmd error.
Eventually I would like to have this run a shell command when a nvim : command is sent, but this plugin doesn't seem like it's even getting loaded properly.
nvim version:
NVIM 0.1.0 (compiled Nov  1 2015 16:15:29)
Commit: v0.1.0
Build type: RelWithDebInfo
Compilation: /usr/local/Library/ENV/4.3/clang -Wconversion -O2 -g -DDISABLE_LOG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -I/tmp/neovim20151101-33169-32i8gt/build/config -I/tmp/neovim20151101-33169-32i8gt/src -I/tmp/neovim20151101-33169-32i8gt/deps-build/usr/include -I/tmp/neovim20151101-33169-32i8gt/deps-build/usr/include -I/tmp/neovim20151101-33169-32i8gt/deps-build/usr/include/luajit-2.0 -I/tmp/neovim20151101-33169-32i8gt/deps-build/usr/include -I/tmp/neovim20151101-33169-32i8gt/deps-build/usr/include -I/tmp/neovim20151101-33169-32i8gt/deps-build/usr/include -I/tmp/neovim20151101-33169-32i8gt/deps-build/usr/include -I/usr/local/opt/gettext/include -I/usr/include -I/tmp/neovim20151101-33169-32i8gt/build/src/nvim/auto -I/tmp/neovim20151101-33169-32i8gt/build/include
Compiled by gregoryblock@Geebs

Optional features included (+) or not (-): +acl   +iconv    +jemalloc
For differences from Vim, see :help vim-differences

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/HEAD/share/nvim"

Nothing in the $NEOVIM_JS_DEBUG file and just this in my ~/.nvimlog:
2015/08/16 10:35:09 [info @ main_loop:558] 71322 - Starting Neovim main loop.
2015/08/16 10:35:21 [info @ main_loop:558] 71366 - Starting Neovim main loop.
2015/08/16 10:43:44 [info @ main_loop:558] 72402 - Starting Neovim main loop.
2015/08/16 10:44:04 [info @ main_loop:558] 72557 - Starting Neovim main loop.
2015/08/16 10:45:45 [info @ main_loop:558] 72910 - Starting Neovim main loop.
2015/08/16 10:46:07 [info @ main_loop:558] 74787 - Starting Neovim main loop.
2015/08/16 10:50:47 [info @ main_loop:558] 76445 - Starting Neovim main loop.
2015/08/16 10:51:03 [info @ main_loop:558] 78304 - Starting Neovim main loop.
2015/10/31 17:46:19 [info @ main_loop:558] 20212 - Starting Neovim main loop.
2015/10/31 17:46:37 [info @ main_loop:558] 20334 - Starting Neovim main loop.
2015/10/31 17:46:42 [info @ main_loop:558] 20415 - Starting Neovim main loop.
2015/10/31 17:46:50 [info @ main_loop:558] 20476 - Starting Neovim main loop.
2015/10/31 17:48:28 [info @ main_loop:558] 20883 - Starting Neovim main loop.
2015/10/31 17:49:28 [info @ main_loop:558] 21111 - Starting Neovim main loop.
2015/10/31 17:52:51 [info @ main_loop:558] 21906 - Starting Neovim main loop.
2015/10/31 17:54:20 [info @ main_loop:558] 22553 - Starting Neovim main loop.
2015/10/31 17:54:55 [info @ main_loop:558] 22786 - Starting Neovim main loop.
2015/11/01 13:41:43 [info @ main_loop:558] 89297 - Starting Neovim main loop.
2015/11/01 13:44:25 [info @ main_loop:558] 89749 - Starting Neovim main loop.

Any help would be greatly appreciated! Thanks!

Once I get back developing the infrastructure for remote plugins, I will look for a plan to removing the requirement for running :UpdateRemotePlugins, make things a little bit more automatic.

@tarruda Sounds great!
While you are working on that, is there any advice you can give to help me get this working?

@gblock0 The message suggests that the connection was closed due to an error, which I guess is caused by the lowercase command your plugin defines(vim/nvim only accept uppercase names for user-defined commands and functions).

@gblock0 did changing the command name to uppercase solve your issue?

No it did not.

I think that the problem is that you want commandOutput to send nvim the output of TestCmd, but what it actually does is asks nvim to execute a command (e.g. w) and return the output (e.g. "main.js" 25L, 507C written).
If you want to display an output message, try using nvim.commandOutput('echo "output!"')!

I changed it to:
plugin.command('TestCmd', {
	range: '',   // Line 16
	nargs: '*'
}, function (nvim) {
	nvim.commandOutput( 'w' );
});

but I am still getting the same error.

Weird. Your plugin works perfectly on my end. Perhaps you could try updating to the most recent version of Neovim? Otherwise, could you please paste the contents of .init.vim-rplugin~ (or ~/..nvim-rplugin~ if you haven't updated recently) after running :UpdateRemotePlugins?

I just updated Neovim with homebrew and it seems to be the same version. Where would the init.vim-rplugin~ file be? I don't see it in my home directory and I can't find it in the neovim homebrew folder.

Try taking a look in ~/.config/nvim!

I had checked there but forgot to add -a to the command...derp. Here is the output:
" node plugins

" python3 plugins
call remote#host#RegisterPlugin('python3', '/Users/gregoryblock/.config/nvim/plugged/vim-markdown-composer/rplugin/python3/markdown_composer.py', [
      \ {'sync': 0, 'name': 'ComposerOpen', 'type': 'command', 'opts': {}},
      \ {'sync': 0, 'name': 'ComposerPort', 'type': 'command', 'opts': {}},
      \ {'sync': 0, 'name': 'ComposerStart', 'type': 'command', 'opts': {}},
      \ {'sync': 0, 'name': 'CursorHold,CursorHoldI,CursorMoved,CursorMovedI', 'type': 'autocmd', 'opts': {'pattern': '*.md,*.mkd,*.markdown'}},
      \ {'sync': 1, 'name': 'ComposerUpdate', 'type': 'command', 'opts': {}},
      \ {'sync': 1, 'name': 'FileType', 'type': 'autocmd', 'opts': {'pattern': 'markdown'}},
     \ ])

" python plugins

Okay, my questions then are 1) whether you've installed this repo as a plugin and 2) whether you've done npm install inside of it.

I am using vim-plug and this is the line in my init.vim for this plugin Plug '~/neovim-autoformat'									. I have tried running :PlugInstall and then :UpdateRemotePlugins. Doesn't me getting remote/host: node host registered plugins ['main.js'] mean the plugin is trying to install?

I was not aware that I was supposed to run npm install inside the plugin. Does my plugin need to be aware of node-host even though Neovim is? Currently I do not have a package.json insde my plugin.

I did run npm install in the node-host plugin folder.


Add support for BDD features / stories

Hi there,
This pull request adds the support for BDD (behavior-driven development) features/stories discussed in issue #668.
We added keywords, punctuations, comments and numbers and etc for lexers to parse. It now should highlight those different parts correctly. Let us know if there are any bugs that need to be fixed or somewhere that needs to be changed.

Hi there,
This pull request adds the support for BDD (behavior-driven development) features/stories discussed in issue #668.
We added keywords, punctuations, comments and numbers and etc for lexers to parse. It now should highlight those different parts correctly. Let us know if there are any bugs that need to be fixed or somewhere that needs to be changed.

Thank you, any reply will be appreciated.

Hi, thanks for preparing this. A few notes:

Please add a test file with a known-good output.
Don't change the .gitignore
Please remove the .ds_store files which are part of this commit

Hi, thanks for preparing this. A few notes:

Please add a test file with a known-good output.
Don't change the .gitignore
Please remove the .ds_store files which are part of this commit

Thanks for your reply. We will fix it base on your suggestion.

Hi, thanks for preparing this. A few notes:

Please add a test file with a known-good output.
Don't change the .gitignore
Please remove the .ds_store files which are part of this commit

Hi, thanks for your reminder, already add test cases. let me know if there are any changes that need to make :)

Changes look mostly good, but you do highlight whitespace as Text. Please assign whitespace to things like   and \n as some style might highlight whitespace differently from text. Check the golden test output.

Merged, sorry it took so long.


Support for QueryExp

I noticed that Query.queryNames(mbeanServer) calls mbeanServer.queryNames(objectName, null) to fetch all the objectNames that matches the objectName in the query config. In my current use case, a Cassandra server, the number of objectName instances returned is too long. Some extra filtering may help a lot.
It would be interesting to have some basic support to populate the 2nd argument, which is a QueryExp. Of course that adds some complexity, like designing a DSL to be used in the JSON config file.
I am not too familiar with this part of JMX, but the javax.management.Query static factory methods seems to be very well documented. A simple DSL implementation can straightly "transpile" the JMX's query builder to a string version and keep the Polish notation, e.g.:
Query.match(Query.attr("Id"), Query.value("7[0-9]"))

becomes
match(attr('Id'), stringValue('7[0-9]'))

Another alternative would be matching the returned objectNames against a regular expression and let the regular expression validate they keys. This approach is implemented by the Datadog's JMXFetch
Does any of this ideas make any sense for you guys? Let me know what you think.
Cheers!

This does indeed look like a great idea! I'm pretty sure I won't have time to work on it, but feel free to submit a PR!


Pipe Atlas Rendering Issues

I'm working on a modification to the Pipe atlases for drawing 3 different types of pipes on the same tile.  In the current functionality the pipes draw on top of each other and it is hard to see what types of pipes are on the tile since they stack.
I'm having trouble with the rendering of the textures from the Atlas PNG.  How are these rendered?  When comparing to the default atlas (Power Conduit for example), the textures stretch out to the edge of the image and the image is divided into 16 equal squares.  There are 16 squares on your Pipe atlas PNGs but they don't really make sense when they're tiled out.  Would there be a way that you know of to change the way that they are drawn so it picks the texture out of the full grid without the white space on the edges and between pipe tiles?

Figured out the issue that I was having. I'll post the picture that solved my problem if anyone has questions.


Issue with the implementation of eye recoloring

I have changed the iris tracking code and filled region of Iris.
I need to add opacity to the filled region
To achieve Iris filling i have updated RenderIris method in (line no: 255) https://github.com/google/mediapipe/blob/master/mediapipe/graphs/iris_tracking/calculators/iris_to_render_data_calculator.cc
to

and update subgraph https://github.com/google/mediapipe/blob/master/mediapipe/graphs/iris_tracking/subgraphs/iris_and_depth_renderer_cpu.pbtxt
last node(removed face landmark annotation) to

For adding Opacity:
I tried to change DrawFilledOval in (line no 368)  https://github.com/google/mediapipe/blob/master/mediapipe/util/annotation_renderer.cc
to

But it didn't worked

Expected Result achieved by using MaskOverlayCalculator
https://github.com/google/mediapipe/issues/1170


Adding database modules with tests

Replaces PR https://github.com/elixir-cloud-aai/foca/pull/11.
Partly addresses https://github.com/elixir-cloud-aai/foca/issues/10
Only adding two functions in db_utils.py file. Others are specific to service and/or Celery.

@uniqueg
For adding the file register_mongodb.py, it has a dependency on some other module which has not been included till now in this archetype.
For example: https://github.com/elixir-cloud-aai/cwl-WES/blob/dev/wes_elixir/database/register_mongodb.py , if you see the imports, you'll see from wes_elixir.ga4gh.wes.endpoints.get_service_info import get_service_info this line.
If I include this line, it will give import cannot be resolved error.
So, should I create a new issue to include register_mongodb.py file later in database folder after this dependency has been added?

Thanks a lot @sarthak-sopho. I think that register_mongodb.py can in priniciple be added after some changes.
The line initiating the service info endpoint should be removed (and in the code of the actual services be imported and called from the main script instead). Also, some other code in that module has to be abstracted / generalized. In particular, the names and properties (which fields are indexed etc) of the database collections should be passed as arguments in order to make the functions independent of the actual service being built. Those arguments would then be passed from the main function, ideally via the app config.

As for this change, if you rephrase your PR, we can already merge it, as it seems to be complete for the db_utils.py module

The line initiating the service info endpoint should be removed (and in the code of the actual services be imported and called from the main script instead). Also, some other code in that module has to be abstracted / generalized. In particular, the names and properties (which fields are indexed etc) of the database collections should be passed as arguments in order to make the functions independent of the actual service being built. Those arguments would then be passed from the main function, ideally via the app config.

I think it would be better if we try to make this kind of change in one of the existing service and then replicate it here. Or should I send a PR according to your comment and then we'll work on it, if it needs improvement? That can also be done.


Create an interactive plot

Create a report in which a user can click on plots

interactive plots would be fun, and it's a good idea to discuss where we can use them, but I would put it on the backburner for now.


Log Cache does not have a metric for dropped envelopes

It looks like Log Cache logs how many envelopes were dropped, however it does not have anything on its health endpoint. This is a vital metric.
https://github.com/cloudfoundry/log-cache/blob/81850b0272c9e59fc7396aeb93a30cb90555fc18/internal/routing/batched_ingress_client.go#L40

Resolved. Here is the metric.


Refactor generation

Changes:

Add action for Generation
Add generate command
Add full-node, consensus, validator, execution, mevboost sub command to generate.

Types of changes

[x] New feature (non-breaking change which adds functionality)
[x] Refactoring (no functional changes, no api changes)

Codecov Report
Base: 56.59% // Head: 53.71% // Decreases project coverage by -2.88% :warning:

Coverage data is based on head (9f13413) compared to base (fd9728c).
Patch coverage: 41.68% of modified lines in pull request are covered.

:exclamation: Current head 9f13413 differs from pull request most recent head 905b386. Consider uploading reports for the commit 905b386 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #205      +/-   ##
===========================================
- Coverage    56.59%   53.71%   -2.89%
===========================================
  Files           54       57       +3
  Lines         3672     4070     +398
===========================================
+ Hits          2078     2186     +108
- Misses        1510     1801     +291
+ Partials        84       83       -1

Flag
Coverage Δ

unittests
53.71% <41.68%> (-2.89%)
:arrow_down:

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

Impacted Files
Coverage Δ

cli/actions/actions.go
100.00% <ø> (ø)

cli/generate.go
0.00% <0.00%> (ø)

cli/root.go
82.50% <ø> (-3.75%)
:arrow_down:

cli/sub_gen.go
0.00% <0.00%> (ø)

cli/cli_utils.go
70.89% <26.86%> (-11.58%)
:arrow_down:

cli/cli.go
59.10% <81.81%> (+0.71%)
:arrow_up:

cli/actions/generation.go
100.00% <100.00%> (ø)

internal/pkg/generate/clean_scripts.go
92.80% <100.00%> (ø)

internal/pkg/generate/generate_config.go
100.00% <100.00%> (ø)

internal/pkg/generate/generate_scripts.go
100.00% <100.00%> (+3.31%)
:arrow_up:

... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Following Warnings are Treated as Errors (Release Mode cannot compile)

FWOG_ASSERT() in Release mode can trigger warning as error for the following:

When using FetchContent() for my own fork that disables the Warning as Error (/WX) I was able to compile Fwog for my project in release mode. (https://github.com/ClementineAccount/Fwog/blob/main/CMakeLists.txt)

Hi @ClementineAccount. Try pulling and let me know if the fix worked.

Hi @ClementineAccount. Try pulling and let me know if the fix worked.

Works perfectly. Thanks for the fix!


⚠ T:Riza Api has degraded performance

In f02e0ce, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 200
Response time: 4566 ms

Resolved: T:Riza Api performance has improved in 7715b38.


More runtime support for subclass existentials

Also simplify SubstitutionMap a bit, as a follow-up to https://github.com/apple/swift/commit/26aa02324b27fa1422be7112ff7abbc6ef82de68.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

I suspect this breaks non-smoke tests. An unrelated PR test failed IRGen/subclass_existentials.sil on 32-bit iOS simulator: https://ci.swift.org/job/swift-PR-osx/7053/

Jordan is fixing the test in #9020.

Thanks for tracking down the original commit.


Add News Articles as a sample Data Source for CMS (internal contents)

with the following normalized schema:
{
"headline": {
"type": "String",
"required": true,
"showInList": true,
"schemaOrgProperty": "headline"
},
"date": {
"type": "Date",
"showInList": true,
"schemaOrgProperty": "datePublished"
},
"author": {
"type": "String",
"schemaOrgProperty": "author.name"
},
"image": {
"type": "ImageURL",
"schemaOrgProperty": "image"
},
"description": {
"type": "String",
"schemaOrgProperty": "description"
},
"category": {
"type": "String"
},
"body": {
"type": "HTML"
}
}

Sample internal component has been added!

OK, but should be named "News Articles" (label in the "Select a data source" right panel) / "newsArticles" (datasource name), not "internalContents".

"Internal Contents" renamed to "News Articles" in sample data source for CMS

schema.org type should be automatically set to "NewsArticle" for this sample data source.


fuzzy string match for incorrect pkg-specs

For example:
$ tea +node-js.org

Should error with the suggestion of +nodejs.org.

This has been merged, #160.


Fix/cur 417/incorrect ordering of units in visualiser

Description
Curriculum units are being displayed in the wrong order - See the ticket below.
https://www.notion.so/oaknationalacademy/Incorrect-ordering-of-units-in-visualiser-e4dd201f74f4424ebfec7be2296b82d6
Fix

The order field is now brought through with the MV and used within the getStaticProps to order the units correctly.

How to test

Go to https://deploy-preview-2320--oak-web-application.netlify.thenational.academy/teachers/curriculum/maths-secondary/units

Click on Year 10

You should see units in the correct order. Please consult the CAT for correct ordering.

Go to https://deploy-preview-2320--oak-web-application.netlify.thenational.academy/teachers/curriculum/english-primary/units

Year 1 - 'Ada Twist Scientist' should be in position 17.

Year 2 - 'Monster Pizza' should be in position 14.

Year 2 - 'Apostrophes' should be in position 3.

Screenshots
Not required
Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility

:tada: This PR is included in version 1.394.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Documentation for the buffered commands API

Documentation, including some inline examples, for the buffered commands API

We have AgonConsole8/agon-docs now, where this documentation has been integrated, so this is no longer needed


[Impeller] flutter debug runs falling back to opengl

flutter run --enable-impeller
...
E/flutter ( 7021): [ERROR:flutter/shell/platform/android/android_context_vulkan_impeller.cc(42)] Using the Impeller rendering backend (Vulkan with Validation Layers).
E/flutter ( 7021): [ERROR:flutter/impeller/base/validation.cc(49)] Break on 'ImpellerValidationBreak' to inspect point of failure: Requested validations but the validation layer was not found.
E/flutter ( 7021): [ERROR:flutter/impeller/base/validation.cc(49)] Break on 'ImpellerValidationBreak' to inspect point of failure: Requested validations but could not find the VK_EXT_debug_utils extension.
E/flutter ( 7021): [ERROR:flutter/impeller/base/validation.cc(49)] Break on 'ImpellerValidationBreak' to inspect point of failure: Device has insufficient capabilities.
E/flutter ( 7021): [ERROR:flutter/shell/platform/android/android_context_gl_impeller.cc(80)] Using the Impeller rendering backend (OpenGLES).
Syncing files to device Pixel 4a...                                 18ms

This is happening because the validations are not correctly loading.  They worked with locally built but something is happening with flutter/flutter archives.
doctor
[✓] Flutter (Channel main, 3.14.0-14.0.pre.222, on macOS 13.5.2 22G91 darwin-arm64, locale en)
• Flutter version 3.14.0-14.0.pre.222 on channel main at /Users/aaclarke/dev/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision e7058f568f (15 minutes ago), 2023-09-11 22:46:04 +0200
• Engine revision 0774ddcda9
• Dart version 3.2.0 (build 3.2.0-134.0.dev)
• DevTools version 2.27.0

The validations are missing from flutter/bin/cache/artifacts/engine/android-arm64/flutter.jar
$ find lib
lib
lib/arm64-v8a
lib/arm64-v8a/libflutter.so

I also verified that local engines have the necessary file structure:
$ find lib/
lib/
lib//arm64-v8a
lib//arm64-v8a/libflutter.so
lib//arm64-v8a/libVkLayer_khronos_validation.so
lib//arm64-v8a/libc++_shared.so

Are we missing some updates in flutter.gradle/flutter.groovy?

Are we missing some updates in flutter.gradle/flutter.groovy?

@jonahwilliams I'm not sure what that file is.  If it is building apps when you execute flutter run I don't think that is the problem because the actual flutter/bin/cache/artifacts/engine/android-arm64/flutter.jar file is wrong on disk.  It seems like the bot that archives these files is modifying the jar file or using some odd build path not used locally.

I don't think that jar file is used though, I believe we upload something to maven central and then the tool downloads this?

I don't think that jar file is used though, I believe we upload something to maven central and then the tool downloads this?
Hmm, hopefully if I fix that artifact it also fixes the maven copy.
It looks like we are uploading something in the bots like here: https://ci.chromium.org/ui/p/flutter/builders/prod/Linux Production Engine Drone/354235/overview
27. gsutil Upload /b/s/w/ir/cache/builder/src/out/android_debug_arm64/zip_archives/download.flutter.io/io/flutter/arm64_v8a_debug/1.0.0-fe5fa056a54043b08df27f4a0028da21a4df4687/arm64_v8a_debug-1.0.0-fe5fa056a54043b08df27f4a0028da21a4df4687.jar to gs://download.flutter.io/io/flutter/arm64_v8a_debug/1.0.0-fe5fa056a54043b08df27f4a0028da21a4df4687/arm64_v8a_debug-1.0.0-fe5fa056a54043b08df27f4a0028da21a4df4687.jar

That build appears to be compiling context_vk.cc but nothing from //third_party/vulkan-deps/vulkan-validation-layers.  I'll see if getting it showing up there will fix it.

If that is in fact the source of all the archives what is suspicious is that context_vk.cc is getting compiled, but --enable-impeller-vulkan is not being specified.


Reserve Entity ID Range

In networking, It's fairly common to synchronize some pieces of game state. Currently, almost all networking solutions use their own HashMap or other indirection to map entity IDs between a client and server. While this solution generally works, it adds an unnecessary layer of complexity and makes it harder to reason about networked logic.
I am proposing extending the bevy API to include a reserve_entities() method, or something to this effect. This would allow you to reserve a contiguous interval of entity IDs, and they would not be used by bevy. This would reduce indirection and complexity in writing networked games, and will likely have other non-networked benefits in other circumstances. The ability to have deterministic IDs is generally a useful pattern, and I think it has a lot of value in Bevy.
In the coming days I will begin working on a naive approach just to get some feedback, but I would love to hear other thoughts in the meantime.

Possibly related: #7335

A problem I see with this is that it would make it very easy to increase the amount of memory being used.
Currently there are a bunch of sparse sets indexed by entities' indexes, and their memory usage depends on the maximum index being used. So if for example you reserve a range of 1M entities when someone else reserves another entity they get an index bigger than 1M and those sparse sets will have to allocate space for 1M entities, even though there may exist a lot less.
This could be fixed by reserving many small ranges alternated with unreserved ranges, this way both normal entities and networked entities get to use small indexes, but this doesn't mean the footgun is not there.

Thanks for your input! I agree there is a lot of nuance to this issue, but I still think having some form of reservation capability would be useful. Perhaps the API can be constructed with this in mind.
It may sound unorthodox, but it might make sense to have the API reserve a fragmented 'slice' of the full range of indices like you suggested. ie, instead of reserving 0..10_000 you might reserve every odd ID, or maybe every 10 ids. The only downside of this I would see is that a potential server might only use ID 10,20,30,etc, and waste the entire range between each of these values.

You can technically already do this via the unsafe function World::entities_mut, and Entities::reserve_entities. This is explicitly made unsafe for a reasons. So long as you can satisfy the safety invariants, go nuts.


Error: tus: unexpected response while creating upload, originated from request (response code: 404, response text: )

i am trying to run AspNetCore_netcoreapp3.1_TestApp. iisexpress working fine. But after deploying on iis server i can't upload file.
You can try it from the url below.
http://ssltest.netcad.com.tr/tus/
I would be glad if you could help.

My fault. Moving the application to the site root directory was fixed.

My fault. Moving the application to the site root directory was fixed.


Dragon fix

This closes #960

I think you might need to rebuild CookieMonster.js after merging the dev branch. It complains about a difference between the current file in your branch and a temporary file created by running the build command again (ie., if you run the build command on your branch now you get a different file than is currently included on the branch).

Looks good! Going to test this when I get home, but LGTM!

I just realised something. In the last commit there is a change to the mana bar text because in steam it will print it 5 times.
this will more or less diable the timer in the web version

Are you able to also fix that?

Without having a seperate version or checking the version number in the script, I dont know how

Doesn't this work?
if (typeof Steam !== 'undefined') {
 // Steam function
} else {
 // Function for browser version
}

Isn't this simply going to calculate the price of all the buildings that have at least amount owned, rather than the cost of the bulding required for the current dragon level upgrade? Am I reading the code correctly?
See my attempt at #975

Thanks for your help guys, with #981 this has now been fixed!


Docker support

Did a quick draft for docker support, but it yet to be tested.
Fixes #1

Pipeline builds for amd64, armhf and aarch64. Should support most platforms besides really old RPis, but those are a pain to support since many dropped support for arm v6.

Also if you tag a tag like v1.0.0, it'll build an the images with the latest, 1, 1.0 and 1.0.0 tag so you might wanna do that after merge.

acmeproxy-1  | Configuration file "/app/acmeproxy.pl.conf" missing, maybe you need to create it?
acmeproxy-1 exited with code 0

Well it's searching in the wrong folder, no idea why.

Awesome, thank you!
I think I spotted and fixed the problem. Try re-basing this PR on the latest main and see if it works.

It now has an issue that cron is missing of course, I'll think of something.

To I think it's working now. I have some issues with ZeroSSL though (it's doesn't accept my + email, because their email validator seems just bad...) and there's no way currently to pass any options to ACME.sh or so, so I can't specify the --server flag to go to letsencrypt.
Not to mention that my configuration would currently reject ZeroSSL certificates anyway because I have CAA set up.

To I think it's working now. I have some issues with ZeroSSL though (it's doesn't accept my + email, because their email validator seems just bad...) and there's no way currently to pass any options to ACME.sh or so, so I can't specify the --server flag to go to letsencrypt.

acmeproxy.pl generates a certificate for itself using default acme.sh parameters. You can use acme.sh to provide your own certificate in $HOME/.acme.sh/acmeproxy.pl.key and $HOME/.acme.sh/acmeproxy.pl.crt . It will not overwrite a pre-existing certificate and will even re-load the certificate when it detects the files are modified. Thanks for showing this is required information, I'll see about adding this to the README.
And of course you can pass whatever parameters to acme.sh clients using the proxy without issue.
Is this PR in "works for me" state?

acmeproxy.pl generates a certificate for itself using default acme.sh parameters. You can use acme.sh to provide your own certificate in $HOME/.acme.sh/acmeproxy.pl.key and $HOME/.acme.sh/acmeproxy.pl.crt .

Yeah I think this isn't on the right volume yet, I might have to change the home folder or so, because I've tried that and it seems it tried to fetch it again after recreating the container.

Is this PR in "works for me" state?

Almost


[Feature] Support set password in new RBAC privilege framework

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #15045
Problem Summary(Required) ：

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:heart_eyes: pass : 13 / 14 (92.86%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/SetExecutor.java
10
11
90.91%
[89]

:large_blue_circle:
com/starrocks/authentication/AuthenticationManager.java
3
3
100.00%
[]

run starrocks_admit_test


Content-List publish causes 404 error

Publishing this tag-based Content List causes the homepage to return a 404 error:

Fixed!


Update API deprecation page for 1.26

This adds info about APIs shipped in 1.26 and updates language about the 1.25 and 1.26 releases
/hold for 1.26.0 release

Let's ship this after the release given that the repo is already frozen.

The milestone tracking is to highlight that this is time sensitive.

/priority important-soon

As the release is postponed by a day, we can probably land this OK if it gets a tech LGTM.

LGTM, for flowcontrol changes

The APF (flowcontrol) changes look good to me new.  The force-push to https://github.com/kubernetes/website/commit/8a2bcf65c67e6939650541e3e4c6f88993e3cc13 removed the remark about the new fields, which were added to all versions in the new release and are not a matter of "Removed APIs".

/hold cancel
Since v1.26.0 is released. Will let doc folks traffic direct this in after merge of dev-1.26

/lgtm

Thanks @liggitt
Let's have this one jump the usual merge queue - I am happy for a manual merge whilst the merge block issue label is present.
/approve


Error While Deploying CC "Failed to Create or update User app in AAD"

Hi, hope you are well.
I'm receiving the error bellow when deploying the CC:

Could you help me to fix this? I've checked the permissions, but the account that I'm using is Global Admin.

Hi @rcoleta ,
Thanks for raising the query.  Uninstall the existing Azure CLI version. Here is the link for required Azure CLI version and try the procedure again.

Hi @gsv022 !
Tks for the fast reply. I made the uninstall and install of the az cli version mentioned and the problem seens resolved. Sometime after running the application and resources are created, but I received these errors:

could it be access permition?

Hi @rcoleta ,
Could you please share the deployment logs. And meanwhile we recommend you to proceed with the manual deployment to unblock yourself

Hi gsv022.Hi gsv022.
Here is the logs from the issue that  @rcoleta is handling
logs.zip
Thank you

Hi!
Any update about this issue?

Hi @rcoleta ,

RoleAssignmentUpdateNotPermitted is the error observed in logs that has been shared. Please update the below 7places with new GUID's. Here is the link to generate new GUID's.

Then go for re-deployment. If there is any issue observed please share the logs. Meanwhile to keep yourself unblocked go for manual deployment.


Linux networking: Incorrect server handshake ACK message.

The last known good build should be 4175d13a084453bd2a2bf43adf1cc4e76f9fe417.
Investigating.

considering to add a build-time smoke test for minimal networking capabilities...

issue identified & repro captured in a test.
the pending_len is not correctly set during sock_t allocation.


Simplify assignment to avoid None

Fixes #34

Codecov Report

Merging #35 (22ff2f0) into main (d79b823) will increase coverage by 0.2%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main     #35     +/-   ##
=======================================
+ Coverage   92.7%   93.0%   +0.2%
=======================================
  Files          9       9
  Lines        331     329      -2
=======================================
- Hits         307     306      -1
+ Misses        24      23      -1

Impacted Files
Coverage Δ

reprexlite/cli.py
100.0% <100.0%> (+4.3%)
:arrow_up:


Link tensorflow.dll to kernels_experimental on Windows

Windows doesn't have an equivalent to libtensorflow_framework.so as part of its C API library, so it needs to include everything as a part of tensorflow.dll. Linking kernels_experimental to tensorflow.dll will allow pluggable devices to be used with the TensorFlow C API on Windows.

@penpornk @rishikasinha-tf  Just letting you know about this new PR.

@penpornk @rishikasinha-tf It's been 3 weeks since my last bump :) We'd like to get this in before TF 2.11. Thank you!

This PR will be in TF 2.11. Sorry again for the delay!


Export training model for MNIST training using MXNet

Currently, the MNIST training does not export the model. This needs to be exported so that it can be used during serving.
@gautamkmr will look at this

https://github.com/aws-samples/machine-learning-using-k8s/pull/75 closes this


Scatter Plots

increase font size
the axis labels should include the metric (either [time] or [cost])
drop title

Resolved in 6faa624 .


Add Name Transforms for Rust style

Adding transformations to Rust-style alongside the existing C-style transforms.
Co-authored-by: Ashutosh Parkhi<EMAIL_ADDRESS>

@tvm-bot rerun


Persist SQL state

They seem to have been broken already and were going to be fixed around the same time as getAllMentions() is implemented, which is soon.
Iterator[Mention] may need to be a CloseableIterator[Mention].

cool, thanks!
On Thu, Sep 24, 2020 at 8:42 AM Keith Alcock<EMAIL_ADDRESS>wrote:

External Email
They seem to have been broken already and were going to be fixed around
the same time as getAllMentions() is implemented, which is soon.
Iterator[Mention] may need to be a CloseableIterator[Mention].
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lum-ai/odinson/pull/208#issuecomment-698426234, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCLGQP3VRCOOPFCLIC3SHNSGNANCNFSM4RYNFVIQ
.


[Enhancement] Skip tablet schema in rowset meta during ingestion.

Why I'm doing:
Since version 3.2, Rowset has been designed to save its own schema, and the complete tablet schema is stored in the metadata. This can lead to the following issues:

When the import frequency is very high, and a large number of Rowsets are generated, the metadata in RocksDB grows, particularly in non-primary key (non-PK) table scenarios. This is because, in non-PK tables, each update to the tablet metadata rewrites the historical Rowset metadata, leading to a large amount of obsolete data in RocksDB.
When the tablet has a very large number of columns (e.g., 10,000 columns), the time taken to persist the Rowset metadata increases, especially when the imported data volume is small.

These two issues can eventually reduce the efficiency of real-time imports.
What I'm doing:
This PR attempts to solve the issue of reduced import efficiency caused by metadata bloat.
One feasible solution is to store the tablet schema only once for all Rowsets that share the same schema. Instead of saving the complete schema in each Rowset's metadata, a reference or marker would be saved in each Rowset’s metadata to point to the corresponding tablet schema.
However, the issue with this solution is compatibility. In previous versions, each Rowset generated its corresponding schema based on its own metadata. If the system is upgraded and then rollback to an older version, the older version would not be able to locate the schema using a reference or marker. This would lead to the generation of incorrect schemas, as the previous versions expect the full schema to be included in each Rowset's metadata.
So I choose a more conservative solution, and the main changes are as follows:

Skip the schema in Rowset meta during import if the Rowset's schema is identical to the latest tablet schema.
Update the RowsetMeta that does not store schema when updating the tablet schema.

If the BE  exits at any given time and restarts, those Rowsets that have not saved their own schema will be initialized using the tablet's current schema. Since the Rowset meta without schemas is updated each time the tablet schema is modified, it ensures that after the BE restarts, every Rowset can find its corresponding schema.
Moreover, this logic is backward compatible with older versions, so even after an upgrade and subsequent downgrade, the BE will still be able to retrieve the correct schema.
Compared to imports, DDL operations can be considered low-frequency tasks. As a result, in most cases, the Rowset meta generated during imports will not carry the schema, which helps alleviate metadata bloat.
However, there can still be some bad cases. For example, in non-PK tables, during the period between an alter operation and the deletion of outdated Rowset meta, if the number of outdated Rowsets is particularly large, the system will still rewrite all outdated Rowsets each time the tablet meta is saved. This can still lead to a decline in import performance.
To solve this issue, we need to resolve the problem of storing multiple copies of the same schema. I think we can first support downgrading and then resolve this issue, allowing for an iteration based on this PR.
Below is a test based on this PR:
a table with 200 columns, one bucket, writing one row of data at a time, with 10 concurrent threads, executed 1,000 times.

Branch
Table type
Total cost time

main-8f128b
Duplicate
1043.77(s)

this pr
Duplicate
178.49(s)

main-8f128b
Primary
188.46(s)

this pr
Primary
186.68(s)

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@mergify rebase

https://github.com/Mergifyio backport branch-3.3

https://github.com/Mergifyio backport branch-3.2


Current User is null but isSignedIn method returns true

value is true but current User is null
_googleSignIn.isSignedIn()
        .then((value){
            if(value){
              Future.delayed(Duration.zero, () async {
                _currentUser = _googleSignIn.currentUser;
                var googleKey = await _currentUser?.authentication;
                accessToken = googleKey?.accessToken;
                print("Access Token - $accessToken");
                print(_currentUser);
              });
            }
});

Hi @abeerfaiz, please provide a complete minimal reproducible code sample along with the output of flutter doctor -v.
Please also provide the steps to reproduce the issue as well.
Also, what version of google_sign_in are you using, and what device(s) are you experiencing this issue on?
Thank you


nullptr is not permitted as ctor argument for jobject like types

nullptr should always be viable for use with jobject and jstring, despite being of type nullptr_t.   Proxy definitions need to be updated to allow this.

This is actually not a bug.  nullptr_t could map onto multiple signatures making function resolution impossible.
For example, if two separate overloads accepted objects or strings, it would be impossible to differentiate which you wanted (despite both being able to accept null objects).
As such this isn't a bug, but required, at least for strings.


Require properties be tagged for multiparameter attribute support?

Rails has the concept of multiparameter attributes, which are properties that get broken into pieces for a view and then assembled magically by the model. datetime_select is an example, http://api.rubyonrails.org/classes/ActionView/Helpers/DateHelper.html#method-i-datetime_select.
Adding support for this was one of my earliest contributions to the gem, using code I found somewhere from someone who was trying to add support to another ORM. Looking at it now, I see that it's pretty expensive. https://github.com/neo4jrb/neo4j/blob/master/lib/neo4j/shared/property.rb#L77. At a minimum, that's two new hashes for every node that comes through, more than that if something has changed.
I propose that we add a multiparameter: true option to the property class method. We can then change this method to only try "reassembling" those multiple fields if it has reason to believe that they might be present. It puts the burden on the user, which might bug some Rails users, but I think that it's in everyone else's best interest.

No.


[dynamodb] Status always false when setting billingMode to PAY_PER_REQUEST

Describe the bug
setting billingMode to PAY_PER_REQUEST makes status always false after table is created
Steps to reproduce
deploy 2 tables using following yaml(modified the E2E test example)
apiVersion: dynamodb.services.k8s.aws/v1alpha1
kind: Table
metadata:
  name: ack-demo-table-pay-per-request
spec:
  keySchema:
    - attributeName: Id
      keyType: HASH
  attributeDefinitions:
    - attributeName: Id
      attributeType: 'S'
#  provisionedThroughput:
#    readCapacityUnits: 1
#    writeCapacityUnits: 1
  billingMode: "PAY_PER_REQUEST"
  tableName: "ack-demo-table-pay-per-request"
---
apiVersion: dynamodb.services.k8s.aws/v1alpha1
kind: Table
metadata:
  name: ack-demo-table-provisioned
spec:
  keySchema:
    - attributeName: Id
      keyType: HASH
  attributeDefinitions:
    - attributeName: Id
      attributeType: 'S'
  provisionedThroughput:
      readCapacityUnits: 1
      writeCapacityUnits: 1
  billingMode: "PROVISIONED"
  tableName: "ack-demo-table-provisioned"

after both tables have been created,

the conditions are

ack-demo-table-pay-per-request

    - lastTransitionTime: "2022-12-16T07:53:09Z"
      message: Resource not synced
      reason: ""
      status: "False"
      type: ACK.ResourceSynced

ack-demo-table-provisioned

    conditions:
    - lastTransitionTime: "2022-12-16T07:45:35Z"
      message: Resource synced successfully
      reason: ""
      status: "True"
      type: ACK.ResourceSynced

Expected outcome
the Status should become True after table is created
Environment

Kubernetes version: 1.23
EKS: 1.23
AWS service targeted: DynamoDB
DynamoDB controller: 0.1.7

log
2022-12-16T14:29:51.484Z	INFO	ackrt	desired resource state has changed	{"account": "647927084307", "role": "", "region": "eu-west-1", "kind": "Table", "namespace": "default", "name": "ack-demo-table-pay-per-request", "is_adopted": false, "generation": 1, "diff": [{"Path":{"Parts":["Spec","ProvisionedThroughput"]},"A":null,"B":{"readCapacityUnits":0,"writeCapacityUnits":0}}]}
{
   "account":"647927084307",
   "role":"",
   "region":"eu-west-1",
   "kind":"Table",
   "namespace":"default",
   "name":"ack-demo-table-pay-per-request",
   "is_adopted":false,
   "generation":1,
   "diff":[
      {
         "Path":{
            "Parts":[
               "Spec",
               "ProvisionedThroughput"
            ]
         },
         "A":null,
         "B":{
            "readCapacityUnits":0,
            "writeCapacityUnits":0
         }
      }
   ]
}

look  ProvisionedThroughput is different
but regarding to API reference

If you set BillingMode as PAY_PER_REQUEST, you cannot specify this property.

fixed by https://github.com/aws-controllers-k8s/dynamodb-controller/pull/55


Notebook document features

3.17 feature: https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#notebookDocument_synchronization
The Deno LSP would use this for jupyter notebook intellisense. Specifically having continuity for variables between TypeScript cells.
cc @ebkalderon You excluded this from #255 because it was handled by #229, but that PR was closed without merge.
P.S. It would be nice to have a #[serde(flatten)] custom: Option<Map<String, Value>> field in ServerCapabilities. With tower-lsp we could get ahead of not-yet-supported request methods using LspServiceBuilder::custom_method(). But there's no way of doing that with capabilities.

Ah there's a PR: #268.


feat(FloatingFocusManager): Optionally add the inert attribute on outside nodes when guards=true

Added the ability to add the inert attribute to nodes outside of a modal popup.
I introduced the inactiveElementAnnotation (feel free to propose a better name) prop to control this behavior and leave the current defaults intact.
I'm wondering if the opposite, aria-hidden when guards = false, makes any sense. I'm leaning towards not allowing it (and potentially simplifying the API with a boolean prop).
Fixes the issue outlined in https://github.com/floating-ui/floating-ui/discussions/3098

I'm wondering if the opposite, aria-hidden when guards = false, makes any sense. I'm leaning towards not allowing it (and potentially simplifying the API with a boolean prop).

guards=false relies on inert to work, so yeah that'd be an invalid state


Fix boundaries setting for "SampleBufferVideoRenderer" re-layouting

Hello @hiroshihorie!
I'm experiencing issues with the camera's front-to-back switch feature, which occasionally causes the video frame to be incorrectly oriented in landscape mode when it should be in portrait mode.
To resolve this issue, I've found that it's essential to update the frame of the AVSampleBufferDisplayLayer after applying the transformation matrix. This is because the transformation matrix also rotates the frame, which can result in incorrect orientation if the frame is not updated accordingly.

Thank you! Will test your fixes 🙏


[Bug] TCGC incorrectly creates a model for some types

For https://github.com/Azure/cadl-ranch/blob/main/packages/cadl-ranch-specs/http/payload/content-negotiation/main.tsp#L55-L58
I don't think this is a "real" model, but TCGC creates one for it.
Interestingly, I don't see one being generated for PngImage from the same tsp.

This one we should talk about more. The reason PngImage isn't created as a model is because all of the properties are explicitly parameters. In the case of PngImageAsJson, the content property could be a property of a body model. We should be digging more into the context that this model is used though. If it's only spread on a function's signature, then we shouldn't generate it


Conditional column formatting - Undo applied style if query engine error occurs

Description
If a conditional column condition + style are applied that results in a query engine error, the table gets into a non-recoverable state.
Steps to reproduce

Create a ticking table with a DateTime column

from deephaven import time_table

mylist = ["mars", "pluto", "saturn"]

simple_ticking = time_table("00:00:01").update([
    "Id=new Integer(i)",
    "Text=mylist[ (int) (3 * Math.random()) % 3 ]"
])

2 Apply a conditional style to a DateTime column with an is condition
3. Set the date value to "2020-01-01 America/New_York" (this can be parsed by UI but fails in query engine)
Expected results
The table should still be in a usable state. Ideally we can revert the last action on the undo stack or something comparable
Actual results

Versions
Engine Version: 0.22.0
Web UI Version: 0.30.1
Java Version: 17.0.6
Barrage Version: 0.5.0

May not be necessary with #1130 as it won't  be as likely to occur.

May not be necessary with #1130 as it won't be as likely to occur.

@dsmmcken  It also occurs to me that the reproduction steps for this bug will likely no longer work after #1130 is done. There may be other use cases where we can get the table in an error state that can't recover, but it may actually prove impossible to get into the state from conditional formatting.

Ok, closing then. Re-open if necessary.


Add breeze.js for caching

Would like to add Breeze.js to manage caching and use as a fun aurelia sample but the data structure isn't typical of relational data.  Any ideas @wardbell ?

There is no unique identifier (except for a url which appears to be unique)
All of the arrays are url references to other objects.
There are no identifiers showing relations such as species_id

I could add a json results adapter but it feels like it could get unwieldy quickly and might be not the best for a community sample.

I think I'd have to wire up the parent ids in the json results adapter if I chose to go that route.

If you show me a JSON data sample of a few related "entities", I might have some ideas. What are these entities anyway?

It's the swapi.co API serving Star Wars data :)
{
    "birth_year": "19 BBY",
    "eye_color": "Blue",
    "films": [
        "http://swapi.co/api/films/1/",
        ...
    ],
    "gender": "Male",
    "hair_color": "Blond",
    "height": "172",
    "homeworld": "http://swapi.co/api/planets/1/",
    "mass": "77",
    "name": "Luke Skywalker",
    "skin_color": "Fair",
    "created": "2014-12-09T13:50:51.644000Z",
    "edited": "2014-12-10T13:52:43.172000Z",
    "species": [
        "http://swapi.co/api/species/1/"
    ],
    "starships": [
        "http://swapi.co/api/starships/12/",
        ...
    ],
    "url": "http://swapi.co/api/people/1/",
    "vehicles": [
        "http://swapi.co/api/vehicles/14/"
        ...
    ]
}

which might have some related starships -
{
    "MGLT": "10 MGLT",
    "cargo_capacity": "1000000000000",
    "consumables": "3 years",
    "cost_in_credits": "1000000000000",
    "created": "2014-12-10T16:36:50.509000Z",
    "crew": "342953",
    "edited": "2014-12-10T16:36:50.509000Z",
    "hyperdrive_rating": "4.0",
    "length": "120000",
    "manufacturer": "Imperial Department of Military Research, Sienar Fleet Systems",
    "max_atmosphering_speed": "n/a",
    "model": "DS-1 Orbital Battle Station",
    "name": "Death Star",
    "passengers": "843342",
    "films": [
        "http://swapi.co/api/films/1/"
    ],
    "pilots": [],
    "starship_class": "Deep Space Mobile Battlestation",
    "url": "http://swapi.co/api/starships/9/"
}

The issue I see with why this won't work with Breeze.js is because the references are backwards to the traditional references.  The parent references it's children instead of the other way around.

Doesn't bother me that we're not dealing with a relational source. Most sources rarely are.
I suspect this is a read-only source. If not, talk to me and we can apply a variation of the Abstract REST Adapter in Breeze Labs.
As for modeling, I think I know what I'd do:

Create array properties for the URL strings (filmsUrl, speciesUrl, etc)

Write a JsonResultsAdapter that morphed incoming (outgoing) nodes from(to) the JSON field names

Define EntityTypes for the corresponding objects that I'll be using in my demo (e.g. Film, Species).

Add an Object.definedProperty for each "navigation" (xxx from set {film, species, ...}) that uses the xxxUrl array to deliver instances of the related xxx entity from Breeze cache.

Think about how i wanted to populate the cache with those related entities.  One trick might be to have the xxx property itself populate the result async ... a kind of lazy load.  Breeze doesn't do lazy loading (for good reasons) so this might seem weird; could fall back to an "eager fetch" or a "fetch-on-demand" approach.

All of this is fun speculation.  It depends upon what you're really trying to achieve and whether you think we'd be demonstrating value with these kinds of games.


Utils::Bottles::tag: ARM tag is arm64_big_sur

[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?

Consistent with the Linux bottle tags, aarch64_linux, armv7_linux, and x86_64_linux.
See https://github.com/Homebrew/brew/blob/d1f6296f0fef3c06ddd29630ca00f950a8f7b762/Library/Homebrew/utils/bottles.rb#L9

The canonical name of ARM64 is AArch64. See https://en.wikipedia.org/wiki/AArch64
Do you prefer

aarch64_big_sur
arm_big_sur

The canonical name of ARM64 is AArch64.

Should HOMEBREW_PROCESSOR be arm or arm64 or aarch64?

I’d prefer aarch64.
The arm prefix can be confused with the 32-bit ARM architecture, especially if you throw armv7_linux bottles into the mix.

https://github.com/Homebrew/brew/blob/b9e2b82f684b1f9abff813ed2672c25fb4dbd8d0/Library/Homebrew/brew.sh#L122
I feel like this line should be changed to
-HOMEBREW_PROCESSOR="$(uname -p)"
+HOMEBREW_PROCESSOR="$(uname -m)"

and HOMEBREW_PROCESSOR should be arm64.

I think standardizing on aarch64 sounds good, as it's the name of the instruction set just like x86_64 is. I'd like very much for linux's armv7 to be called aarch32 though :smile:

uname -m reports armv7 rather than aarch32. 🤷♂

uname -m reports armv7 rather than aarch32. man_shrugging

But I really like this bikeshed to be purple, cmon.

HOMEBREW_PROCESSOR is what it is on Intel because of the user-agent. I think we need to separate those concerns here.

In light of what @Bo98 wrote, I’d rather leave HOMEBREW_PROCESSOR as is.
We may want a separate environment variable for "$(uname -m)". Alternatively, how about we derive the tag from Hardware::CPU.arch?

But I really like this bikeshed to be purple, cmon.

Sorry, we're all sold out of purple. 💜 😉

In light of what @Bo98 wrote, I’d rather leave HOMEBREW_PROCESSOR as is.

HOMEBREW_PROCESSOR remains Intel on macOS and aarch64 or x86_64 on Linux. You prefer arm for macOS on Apple Silicon?

We may want a separate environment variable for "$(uname -m)". Alternatively, how about we derive the tag from Hardware::CPU.arch?

If it's done for Linux as well as macOS that'd change the bottle tag prefix
from armv6, aarch64, and x86_64
to arm, arm64, and x86_64.
Our naming is a bit inconsistent and crazy making that I've jumped down this rabbit hole. 🕳🐇

I'd like to change HOMEBREW_PROCESSOR from Intel to x86_64 on macOS, to match Hardware::CPU.arch identically. We could rename HOMEBREW_PROCESSOR to HOMEBREW_CPU_ARCH while we're at it.

HOMEBREW_PROCESSOR remains Intel on macOS and aarch64 or x86_64 on Linux. You prefer arm for macOS on Apple Silicon?

I don’t know semantics nor purpose of HOMEBREW_PROCESSOR so I’ll better leave that decision to people who do ☺

If it's done for Linux as well as macOS that'd change the bottle tag prefix
from armv6, aarch64, and x86_64
to arm, arm64, and x86_64.

Oh I see. So that’s not what we want, either.

I'd like to change HOMEBREW_PROCESSOR from Intel to x86_64 on macOS, to match Hardware::CPU.arch identically. We could rename HOMEBREW_PROCESSOR to HOMEBREW_CPU_ARCH while we're at it.

I wouldn't be against this - but I'd make sure the user-agent still uses Intel.

si_big_sur is clearly better. The text being unrenderable on Linux only emphasizes that this is a Mac-only architecture. /s

I'd like to change HOMEBREW_PROCESSOR from Intel to x86_64 on macOS

I'm -0 on this. It's not a major objection but I'd be concerned about the possibility of subtle bugs emerging, and I'm not sure if the benefits would outweight the potential of digging up weird bugs for awhile.

We may want a separate environment variable for "$(uname -m)". Alternatively, how about we derive the tag from Hardware::CPU.arch?

If it's done for Linux as well as macOS that'd change the bottle tag prefix
from armv6, aarch64, and x86_64
to arm, arm64, and x86_64.

@claui I forget that I'm modifying extend/os/mac/utils/bottles.rb here, so this change applies only to macOS and doesn't affect Linux.

I've changed this PR to use Hardware::CPU.arch rather than HOMEBREW_PROCESSOR on macOS. Linux continues to use HOMEBREW_PROCESSOR.

❌ codecov/patch — 66.67% of diff hit (target 73.06%)

https://codecov.io/gh/Homebrew/brew/compare/28f4a700acc1c5835f5565212a91b4541fcf469f...9a83e52ae479607084c6df198ec21a93d371cb7b/diff
😢 The ARM branch of this code isn't tested. Any suggestions?

😢 The ARM branch of this code isn't tested. Any suggestions?

That's fine, ignore it.

Do we want to use x86_64_big_sur or big_sur for x86-64?

Good job @sjackman 🍻

Do we want to use x86_64_big_sur or big_sur for Big Sur on x86-64?

I’d be ok with leaving as is, given that x86_64 bottles will organically bubble up and go away in a few years.


Hook with Authme

When reimplementing plugins, can you also add AuthMe Hook. Without this Hook users in my offline server are loosing items. I already tried writing the implementation code here:
https://github.com/cheddar262/Combat-Tag/pull/39

Yes if no one can damage I should not be able to damage anyone else.

Could you add this final utility please? It is very critical for my server.


Option to Have Resizable Widgets Snap to Grid

Using the jQuery UI Resizable widget...is it possible to gather the available width/heights that the gridstack_items can resize to and pass that through to the 'grid' option of the resizable UI widget??

Hi,
What do you mean? Can you describe a use case?

My personal use case is more of a UX/UI preference.  I would like for the widget being resized to be able to snap automatically, like Gridster does, to the next available spot while still in resizing mode.

In the image above, it just seems strange that the placeholder can be resized to below or above the min/max values and would be nice if the  placeholder 'image' doesn't resize past the min/max values.
If any of that makes sense...apologies if it doesn't.

It is fixed now. Min/max constrains are respected during resizing.


spago run does codegen

It looks like spago run does codegen in addition to spago build. For production apps, when startup time is important, this has a negative impact. Would it be possible to limit the codegen to only spago build?

Actually this is not pspy-related. It's a design of spago itself.
spago build will run $backend, while spago run will run $backend && $backend --run.
You can use pspy --run to avoid codegen.

Thanks!


Export the hex_psbt! macro

The hex_psbt helper macro is useful for testing, examples, and in rust-miniscript. Export it for others to use.
The first two patches are preparatory cleanup, patch 3 does the work.

I like the test_macros crate idea. I'll link the comment over in the crate smashing issue. I was also a bit hesitant about the unwrap. Closing this.

If we want to have macros that are meant for literals, I think they should be proc macros instead (so that they can verify the input).

I'd love that, but I don't want to do the work :)


regex test simplified

https://github.com/objectionary/eo-strings/issues/88

@levBagryansky @Graur please have a look

@EugeneDar sure

@yegor256 please have a look

@yegor256 please have a look

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 4min)


Add firmware for NF10.

The firmware source code for NF9 and 10 is identical. NF10's main.cpp is just a symbolic link to NF9's main.cpp. Valve support is compiled in by defining the macro USE_VALVE to 1. It's compiled out by setting USE_VALVE to 0. The platform.ini build_flags set this macro accordingly for each project.

@haydenroche5 would you mind resolving the conflicts in your branch and merging to the nf10 branch for final review before it goes public?


spawn on join when not wanted

people are spawning on reconnecting, and checking the config im not seeing anything to be able to change that. im running on 1.12.2 and thus far the plugins working apart from not wanting the spawn on join

running luck perms, random spawn, world edit, essentials X

There is a config for it: https://github.com/MatthewSH/spigot-RandomSpawn/blob/master/src/main/resources/config.yml#L12


SJICube.plot() Not Returning an Animation

I tried running SJICube.plot() to get an animation.  It says it returns a FuncAnimation but is only showing one image and has no sliders.  I'm running it in a jupyter notebook.  I also tried animating it manually using sunpy.visualization.imageanimator.Imageanimator and it worked fine.  So it's not a jupyter notebook issue.

I keep getting this error. It seems updatefig is having trouble updating the axes...
~/Developer/sunpy_dev/sunpy/map/mapcube.py in updatefig(i, im, annotate, ani_data, removes)
219             if wcsaxes_compat.is_wcsaxes(axes):
220                 im.axes.reset_wcs(ani_data[i].wcs)
--> 221                 wcsaxes_compat.default_wcs_grid(axes)
222             else:
223                 im.set_extent(np.concatenate((ani_data[i].xrange.value,
TypeError: default_wcs_grid() missing 2 required positional arguments: 'units' and 'ctypes'

@sjtimothy Can you supply the code you ran to get this error?  I didn't get this error, but there definitely seems to be something up with this function.  Perhaps @ehsteve can take a look when he's back from his travels.

Sure, this was using a locally saved fits file, not the sample data
from irispy.sji import SJICube, SJIMap
import matplotlib.pyplot as plt
import matplotlib.animation as animation
local='/Users/shelbe/Documents/IRIS/data/AR12661/'
import os
sji_flist=[]
for file in os.listdir(local):
if file.endswith(".fits"):
sji_flist.append(os.path.join(local, file))
mc=SJICube(sji_flist[0])
mc.plot()

I also tried plotting a slice of the map cube and got a different error:
mc[0:10].plot()
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 mc[0:10].plot()
~/Developer/irispy/irispy/sji.py in getitem(self, key)
210             return self._get_map(key)
211         elif isinstance(key, slice):
--> 212             return SJICube((self.data[key, :, :], self._meta[key]))
213
214     def len(self):
~/Developer/irispy/irispy/sji.py in init(self, input)
194         cmap = []
195         for i in range(number_of_images):
--> 196             norm.append(deepcopy(self[i].plot_settings['norm']))
197             cmap.append(deepcopy(self[i].plot_settings['cmap']))
198
~/Developer/irispy/irispy/sji.py in getitem(self, key)
208         work.  Otherwise, a mapcube is returned."""
209         if isinstance(key, int):
--> 210             return self._get_map(key)
211         elif isinstance(key, slice):
212             return SJICube((self.data[key, :, :], self._meta[key]))
~/Developer/irispy/irispy/sji.py in _get_map(self, index)
201
202     def _get_map(self, index):
--> 203         return SJIMap(self.data[index, :, :], self._meta[index], mask=self.mask[index, :, :])
204
205     def getitem(self, key):
AttributeError: 'SJICube' object has no attribute 'mask'

FYI: When typing out code in GitHub Issues, PRs, etc., you can bookend the code snippet with three tickmarks, `, (N.B. not quotation marks) to make it cleat that it's code.  For example
mc[0:10].plot()
Regarding the above plot error, I think you have to create a submap of the region you are interested in and then try plotting the submap.  Perhaps @ehsteve can give more advice on the next call.

When trying to run image coalignment on the map cube, I get this error:
'''AttributeError: 'SJICube' object has no attribute 'maps''''
Could this be related to the plotting errors? Should we add a maps attribute?

animation bug fix #62


New tokens do not stand out enough

As much as I love the HD tokens. Any chance we can get a  black or grey boarder around them (or a shadow under them). They bleed a bit too much into the background as it stands.
Originally posted by @scrubbless in https://github.com/NPBruce/valkyrie/issues/1074#issuecomment-465249862

@redwolf2 suggested :
We could add the same Normal Map to all token. https://www.youtube.com/watch?v=Up9zttIi_xg

Proof of concept:

The questionmark of the height map is a little bit off, thats why the edges are not yellow, but this should be an easy fix.

@redwolf2, I'm not convinced with the current idea of height and normal map and, therefore, the look of the token. I don't like the qurvature of the token nor the gradual loose of height on the edges.
@scrubbless said 'a black or grey boarder around them/tokens (or a shadow under them)'. I think a shadow wouldn't be good. I think the map could look something like that (I know colours may not be accurate; it's just a brief presentation):

The token has is a border around the edges so it should stand out better on the board.
@scrubbless, did I understand you correctly?

I like the normal map, but I want to see it rendered in game before sharing my thoughts.
Normal maps looks good when lighting is set so I'm curious what would be the end result.

A quick fix could be to increate the blackness of the border shadow and avoid using cubic interpolation when downsizing from the 512x512 to 256x256 image. This is what blured the resulting image a bit. That way the icon will come out more crisp and detailed. Tokens are a litte bit like arrows from a design perspective. Less directs to more. So having more roughness in the image will make the icon stand out better. Imaging a white room with only a wooden door in it.  The original designer or the token seem to be aware of this. I will match up the token closer to the original.

I've just found another problem with tokens:

I zoomed out the screen to max.

@chrismaliszewski That is not a new problem and does not seem to have something to do with the new icons. See: https://github.com/NPBruce/valkyrie/issues/712

@chrismaliszewski That is not a new problem and does not seem to have something to do with the new icons. See: #712

Unfortunately, the screenshot above is from valkyrie-windows-2.2.02b version so the new icons also have this problem. Or are they not included in the version yet?

The old version already have this problem, it is not the icons. You might want to double check with valkyrie 2.1.5.

Yeah, I know. I read the ticket you mentioned.
Actually, now I understand what you meant by "[...] problem [...] does not seem to have something to do with the new icons".
Sorry if I seemed pushy. Didn't want that to look like that. It was just a simple misunderstanding.

Hey, no offense taken ;), but you are right that it is a problem or simply: not so nice. I will link your comment in the other issue.
Created a pull request: https://github.com/NPBruce/valkyrie/pull/1090

Merged


About voice recording in Asterisk

Please, do not get mad at me for talkativeness, I just want  to recommend to add into manual information about recording voice in asterisk. I understand, that this is off topic, but with this (I don't know how to say it in English) "снизится порог вхождения".
For example, I configured voice recording with this way:

Make directory for voice records
mkdir -p /var/spool/asterisk/monitor
Install lame: download latest archive from here for example with wget , and after that:

tar zxvf lame-3.*.gz
cd lame-3.*
./configure
make
make install

Create file /usr/local/bin/conv2mp3.sh with the following content:

#!/bin/sh
LAME="/usr/local/bin/lame --noreplaygain"
filename="$1"
foldername="$2"
$LAME -V2 $foldername/$filename.wav $foldername/$filename.mp3
rm -f $foldername/$filename.wav

Use this script in /etc/asterisk/extensions.conf to convert recorded WAV file to MP3 (also, this is example of recording voice in format needed by webcdr ) :

exten => _8XX,1,Set(filename=${UNIQUEID})
exten => _8XX,n,Set(foldername=/var/spool/asterisk/monitor/${STRFTIME(,,%Y)}/${STRFTIME(,,%m)}/${STRFTIME(,,%d)}/)
exten => _8XX,n,MixMonitor(${foldername}/${filename}.wav,b,/usr/local/bin/conv2mp3.sh ${filename} ${foldername})
exten => _8XX,n,Dial(SIP/${EXTEN},30,t)

and to show webcdr, that voice recording is configured (step 8 of installation manual), user should change file /etc/asterisk/cdr_mysql.conf by adding alias, for example I added
alias  uniqueid => record

It's fine if it works for you. Even better, if somebody finds it helpful.
The problem is that there are too many options to formulate a single solution that fits all. Some Linux distributions have mp3 software in official repositories, for example Ubuntu. For others, there may be unofficial packages. Or some people may prefer using ffmpeg, not lame. Some might as well use Opus instead of MP3 (as I do).
The intended use of record column is to set it explicitly as an additional field using cdr_odbc_adaptive only on those calls that have been monitored. Setting an alias in conf file will show recordings for all calls, even for those that have never been recorded.
Still, there should be some basic manual(s) for most common installations, so I'm keeping the issue open. Thanks for the feedback once again.

Hello! Thank you for reply.
Can you explain, how I can change record fields only for that calls, which have been monitored?
I guess, that I should do it from Asterisk Dialplan =)

...
Set(CDR(record)=${UNIQUEID})
MixMonitor(...)
...

It only works with cdr_adaptive_odbc. cdr_mysql and other database cdr drivers will discard additional fields.

hello,
thanks for the how-to in previous comments, it was really helpful.
But I have a problem, I got "GET /api/recordings/undefined 500 3.844 ms - 147 invalid recording id" error every time when I try to play the recorded file.
The file in the correct folder ( like
pattern = /var/spool/asterisk/monitor/%YYYY%/%MM%/%DD%/*%uniqueid%.mp3) and I use the following way to make calls:
exten => _X.,1,Set(DIRNAME=${STRFTIME(${EPOCH},,%Y/%m/%d)})
exten => _X.,n,Set(FILENAME=${DIRNAME}/${UNIQUEID})
exten => _X.,n,System(/bin/mkdir -p /var/spool/asterisk/monitor/${DIRNAME})
exten => _X.,n,MixMonitor(${FILENAME}.wav,,)
exten => _X.,n,set(CDR(record)=${UNIQUEID})
exten => _X.,n,Dial(SIP/0011101${EXTEN}@trunk)

exten => h,1,StopMixMonitor()
exten => h,n,System(/usr/bin/lame /var/spool/asterisk/monitor/${FILENAME}.wav /var/spool/asterisk/monitor/${FILENAME}.mp3 )
exten => h,n,System(rm /var/spool/asterisk/monitor/${FILENAME}.wav)

The filename is: /var/spool/asterisk/monitor/2018/03/29/1522334172.3623.mp3
I added alias uniqueid => record to cdr_mysql.conf, but not helped.
In the database I checked and the record column is the same as uniqueid.
I hope the solution is very simple just somehow I missed it. Could you help me? Thank you!