FIle permission issue

Hi
I was trying run the simple puts "hello World" program on my Lubuntu 32 bit with Clang as the compiler and I am getting the following issue:
rishav@rishav-VirtualBox:~/Workspace$ ls -l
total 4
-rwxrwxrwx 1 rishav rishav 18 Aug 19 12:55 test.cr

rishav@rishav-VirtualBox:~/Workspace$ crystal run test.cr
Error opening file '/home/rishav/Workspace/.crystal/home/rishav/Workspace/test.cr/bc_flags' with
mode 'w': Permission denied

*Errno::new<String>:Errno +305 [0]
*File::new<String, String>:File +787 [0]
*Crystal::Compiler#codegen<Crystal::Compiler, Crystal::Program, Crystal::Expressions,     Array(Crystal::Compiler::Source), String>:(Nil | Int32 | Bool) +4475 [0]
*Crystal::Compiler#compile<Crystal::Compiler, Array(Crystal::Compiler::Source),     String>:Crystal::Compiler::Result +5008 [0]
*Crystal::Command::run_command<Array(String)>:Nil +132 [0]
*Crystal::Command::run<Array(String)>:(Nil | Bool | FileDescriptorIO+ | Crystal::Compiler::Result | Array(Crystal::Init::View+:Class)) +17811 [0]
main +17803 [0]
__libc_start_main +222 [0]
_start +33 [0]

Error: you've found a bug in the Crystal compiler. Please open an issue, including source code that will allow us to reproduce the bug: https://github.com/manastech/crystal/issues

That is strange, did you create /home/rishav/Workspace with a different user or ran crystal run test.cr with a different user prior? You can try chown -R rishav /home/rishav/Workspace/.crystal, as root if you get permission errors on that one too.

I have only 1 user on the system. Its a fresh VM which i just installed today.
So this is the sequence of things as they happened:
I installed Crystal using the shell script and apt-get command.
when I tried to run it, i got a missing cc exception.
Thinking that it is because lubuntu is very minimalistic and may not have a c compiler installed, i installed clang.
then i tried to run by test program.
it ran fine and i got the output in the console. but it also gave me some warning: sigreturn is not implemented and will always fail
I ignored that as the output came on the console.
then i shut down my VM. I made some changes to the VM. increased memory to 786 mb.
changed chipset to ICH9 and enabled i/o APIC.
This was because my VM was acting sluggish.
then i restarted.
i tried to run the program again and now I am getting this error.
all operations were done using the same user rishav

Did you try the fix anyway? Does it work after rm -rfing .crystal? What are the permissions of the files and folders (ls -l)?

chown worked. it compiled and my console now shows Hello World.
so, what happened there? some random OS issue where the folder permission changed for reason?

I guess so, I doubt it's a Crystal issue.

Thanks. Shall i close the issue now?

Feel free to reopen if you find a reliable way to reproduce it :)


Take Control warnings while OP not engaged & when stopped, but looking away

Describe the bug
#1 - Getting ‘Take Control’ warnings while driving, but OP is not engaged.  #2 - Also, get ‘Take Control’ warnings while stopped (i.e. at a stop light), but I’m looking away (i.e. face recognition timeout).

How to reproduce or log data
As described above.  #1 happens if I’m driving, OP is not engaged, but I drift over a line a small amount or enter a turn too fast.  #2 happens every time I’m stopped, with OP engaged, but I’m looking away.

Expected behavior
#1 - One would expect to not get ‘Take Control’ warnings if you do not have OP engaged.
#2 - One would expect OP to know that you are stopped and that it is OK to look away and therefore not give a ‘Take Control’ warning.

Device/Version information (please complete the following information):

Device: comma two
Dongle ID:  6ffdd0db49c2f5e0
Route: [e.g. 77611a1fac303767|2020-05-11--16-37-07, can be found in my.comma.ai/useradmin]
Timestamp: [When in the route the bug occurs (e.g. 4min 30s into the drive)]
Version:  0.7.9
Car make/model: 2018 Honda Accord Hybrid Touring

Additional context

Can you post specific instances (routes with timestamps of the events)? I looked through a couple routes from your device and didn't find anything.

Both of these sound normal. Driver monitoring shows the green alert if you are distracted while at a stop but won't beep at you, and the other one is the lane departure warning alert which has a toggle in settings.

@VirtuallyChris  @adeebshihadeh
I will agree on the driver monitoring alert.  It doesn’t beep at you, but does have the alert when you are stopped.  One could argue either way whether that is necessary.
The lane departure alert in settings turns off ALL alerts (I assume).  Mine is going off when OP is not engaged.  Again, one could argue this is a ‘feature’ and not a ‘bug’.  ;)  I would land on the side that thinks if I do not have OP engaged, I do not want alerts from it.  I would recommend the lane alerts be disabled when OP is not engaged or at least make it a toggle.
I notice the lane departure alerts on today’s drives around 0508 and 1509.
Thanks for both of your help.

I don’t agree that this is ‘Closed’.  George Hotz specifically said in a video on YouTube that OP would not alert you if not engaged.  Will have to find the exact reference, but was in interview with Lex Fridman.  Please reopen and address #1 above.  Thanks.  @VirtuallyChris @adeebshihadeh

By default, lane departure warnings are enabled and will alert you while not engaged, but they can be disabled in the settings.

https://youtu.be/-qf1NzI1I7E
“If the car is not moving, we do not actively enforce driver monitoring.”  George Hotz at 0:52
And again at 1:52.
@adeebshihadeh  When you say it can be disabled in settings, does that just disable it when OP is not engaged or all the time?  I’ll admit that I didn’t try to disable the setting because I don’t want to disable them all the time.  Just when OP is not engaged.  (Which seems logical.....at least to me.). Thanks!

To clarify, in video, George is really talking mostly about #2.  I’m OK with the OP ‘silent’ alerts if I’m stopped.  My issue is still with #1.  I.e. getting alerts from OP when it is not engaged.  I think George would agree with me.  ;)


charm: 0.12.1 -> 0.12.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/charmbracelet/charm/releases.
meta.description for charm is: "Manage your charm account on the CLI"
meta.homepage for charm is: "https://github.com/charmbracelet/charm"
meta.changelog for charm is: "https://github.com/charmbracelet/charm/releases/tag/v0.12.3"
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.12.3 with grep in /nix/store/s55yzh1lgndz49isbvbmf7a66qgw1daf-charm-0.12.3
found 0.12.3 in filename of file in /nix/store/s55yzh1lgndz49isbvbmf7a66qgw1daf-charm-0.12.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
charm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s55yzh1lgndz49isbvbmf7a66qgw1daf-charm-0.12.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A charm https://github.com/r-ryantm/nixpkgs/archive/5b3aa061f217a2feb3f4af39faeacbe0986e6028.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s55yzh1lgndz49isbvbmf7a66qgw1daf-charm-0.12.3
ls -la /nix/store/s55yzh1lgndz49isbvbmf7a66qgw1daf-charm-0.12.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

charm

Maintainer pings
cc @penguwin for testing.

Result of nixpkgs-review pr 185930 run on x86_64-darwin 1

1 package built:

charm

Result of nixpkgs-review pr 185930 run on x86_64-linux 1

1 package built:

charm


Fix the file selection bug

for some reason your selection never 'sticks' first time. You have to select any image, then select the image you want. You can confirm your choice has been made by seeing the name in the 'this image/copy' box below matches the image you selected the second time

this is related to the timer that checks for consistency between images in assets and labeled folders


[QUESTION] Broadcast message when import is completed

Package version: 2.1.3
Laravel version: 5.2.41
I would import a large file using chunks and Laravel queue with Redis.
After this I would notify the user that the import is completed firing an event with broadcasting.
Suppose that the file contains 5000 rows and I use chunks of 250 rows at a time (20 jobs).
How can I know when all chunks are processed?
Thoughts?
Thanks in advance

Try putting the import process on the queue, and then pushing a "FinishImportJob" onto the queue. Let the FinishImportJob fire the event.

Seems a good idea! Thank you @patrickbrouwers

Hi guys, can you show me an example about that. Something like that, could be correct:
dispatch(new FinishImportJob($user));
or i need to specify the queue by onQueue method ?
I mean, right now i'm using only one queue with db driver.


Update Software Versions for Protocol 21: Mainnet

Preview is available here:http://developers-pr685.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr685.previews.kube001.services.stellar-ops.com


An extra choose providers pops up

A minor bug, but pacuar -S python-geopy pulls in the dependency python-oauthlib, which in turn requires python-pyasn1. I'm given the following option:
:: There are 2 providers available for pyasn1:
   0) python2-pyasn1 (extra)
   1) python-pyasn1 (extra)

Enter a number (default=0):

Clearly I was very worried because the default option seems to be a python2 package. A bit of digging shows that the two are in fact precisely the same binary in the repos. The correct behaviour which pacman -S python-oauthlib, pacaur -S python-oauthlib and even downloading the geopy PKGBUILD and using makepkg -sri give is to recognise that these are actually precisely the same packages and just install them without asking.

Thanks for the report. The python2-pyasn1 and python-pyasn1 packages are actually distinct packages (split), but something is definitely fishy here. The prompt shouldn't appear at all since the dependency chain is explicitly defined with the dependency set on python-pyasn1 (python-oauthlib -> python-cryptography -> python-pyasn1).

Fixed in 35ffc401c0ac20575ef546af308ae6beb929e8b0. This was actually a bug that stayed under the radar for a very long time.

To be fair, I don't think most users would notice - it's only because I just adopted the python-geopy package that I was very twitchy about packages the install pulled in.

Good catch then. Only the prompt was faulty, the correct dependency was always installed even if you chose the "wrong" choice beforehand.


facing issue while using "ExcelDataReader"

I got this message "System.Xml.XmlException: 'Root element is missing.'" while the code below:
DataTableCollection tableCollection;
private void BtnOpenFile_Click(object sender, EventArgs e)
{
        using (OpenFileDialog openFileDialog = new OpenFileDialog() { Filter = "Excel Files|*.xlsx" })
        {
            if (openFileDialog.ShowDialog() == DialogResult.OK)
            {
                TBoxFilePath.Text = openFileDialog.FileName;
                FileInfo fi = new FileInfo(openFileDialog.FileName);
                string extn = fi.Extension;

                using (var stream = File.Open(fi.FullName, FileMode.Open, FileAccess.Read))
                {
                    //System.Text.Encoding.RegisterProvider(System.Text.CodePagesEncodingProvider.Instance);
                    using (var reader = ExcelReaderFactory.CreateReader(stream))
                    {
                        var result = reader.AsDataSet(new ExcelDataSetConfiguration()
                        {
                            ConfigureDataTable = (_) => new ExcelDataTableConfiguration()
                            {
                                UseHeaderRow = true
                            }
                        });

                        tableCollection = result.Tables;
                        SheetDdl.Items.Clear();
                        foreach (DataTable table in tableCollection)
                            SheetDdl.Items.Add(table.TableName);//add sheet to combobox
                    }

                }
            }

        }
    }

Can you open the file in Excel? If Excel can open the file, could you provide the file for us to reproduce the issue?

your reply was so useful ... i just realized that the Excel file has issues to open. thanks for your smart reply. 👍
But still have different error when I tried to open another Excel-File, which is:
"System.IO.IOException: 'The process cannot access the file 'G:\EFILE\DPA\Book1.xlsx' because it is being used by another process.'"
I tried find out why it's showing that error, by separate Read-file-path Process with a button & read Content with another Button... but still same that error ... knowing the ms-excel is closed already & no other processes on the same is running!

You can use for example Process Explorer to find out what process is holding the file open.

yea... you are right again ... thank you very much ❤


Update Explorer URL

Looks good, but this pull request probably should have been opened against the dev branch instead of master because dev is the default branch (the one that users will see when they visit this repository) and master doesn't get updated frequently. I could cherry pick this commit onto dev but will wait for input from @Jenova7 first.

It has to be pushed in Dev branch yes, you can cherry pick John 👍

Cherry picked into dev


skipIfAlreadyDeployed flag + proxyOptions not behaving the expected way

Describe the bug
skipIfAlreadyDeployed is intended to avoid redeployments. When paired with proxy options, it will skip updates even if the updated implementation contract is named differently from the current implementation. I understand this is because the plugin is checking on the proxy contract instead of the implementation one.
To Reproduce
Try to run this, with the variable contract pointing to a v1 version, and then run it again with contract pointing to a v2 version. It won't be deployed.
const proxyOptions: ProxyOptions = {
    proxyContract: 'OpenZeppelinTransparentProxy',
    owner: deployer,
    execute: {
      init: {
        methodName: method,
        args: args,
      },
    },
  }

  const deployment = await deploy(contract, {
    from: deployer,
    contract,
    log: true,
    proxy: proxyOptions,
    skipIfAlreadyDeployed,
  })

Expected behavior
I expect that when using skipIfAlreadyDeployed flag set to true, the plugin will make the check on the possible new candidate to implementation, not on the proxy. Thus if the implementation contract is different from the current one, it will go through.
versions

hardhat-deploy 0.9.29
hardhat 2.6.4
nodejs 14.18.1

This is to me expected.
as proxy, they won't be redeployed in any case, so skipIfAlreadyDeployed could not apply to the proxy itself.
This is why it is applied to the implementation instead.
If you want the upgrade to go through, you simply have to set skipIfAlreadyDeployed to false
But maybe you are thinking about contract name, and expect that if the contract name specified is different than previous it should be deployed.
If that was the idea, note that hardhat-deploy do not save the name, it just use it to get the artifact and so for hardhat-deploy it does not matter whether you specify the contract by name like you do or if you provide an artifact object.

But maybe you are thinking about contract name, and expect that if the contract name specified is different than previous it should be deployed.

Yes that was exactly what I was expecting. Gotcha then. I 'll close it.


mpxv temporal colouring scale

Note: this requires forthcoming PRs in augur and auspice, and I'll notify here when released etc. Because of this i'll keep this as a draft PR.
The end result is

Augur PR: https://github.com/nextstrain/augur/pull/969
Auspice PR: https://github.com/nextstrain/auspice/pull/1523

Both of the prereq PRs above have been merged. Auspice is released (2.37.3) and the augur change should be released tomorrow 🤞

Wait until the nextstrain image has been updated (should be automatic)

This has been done: nextstrain/base:build-20220616T211939Z

Added a commit to ensure we're running augur v16, which is causing CI to fail as nextstrain/base:branch-nextalign-v2 has yet to be updated...

I've restarted failing CI to see if the new image has propagated by now @jameshadfield

I'll merge because it's better than what we use currently. But is it on purpose that there's no cake for the map?

but I leave the button pushing to you

Thanks for merging this -- it's the nice thing about working across different time zones 😂

But is it on purpose that there's no cake for the map?

I've not head them called 🍰  but I like it. Continuous scales (and temporal is a continuous scale with small tweaks) get colour blending (e.g. https://next.nextstrain.org/zika/?c=num_date&d=map&p=full), however it's been a to-do to explore discretising these (e.g. via the binning in the legend) to produce a 🍰 . It would certainly make the map code simpler!


update unittest coverage to reach over 90%

Description
This PR includes the following proposed change(s):

update each test using junit.jupiter instead of Java default Assert;  run maven verify to check code coverage in jacoco; modify test code and source code to bring coverage above 90%;  add a new file OrdsErrorTests.java  to test all null/bad requests.

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactoring / Documentation

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

looks good. close it.


Adds message to the chat when someone joins or leaves the clan chat

resolves #2406

This is the chatbox experience with this setting enabled:

Updated the style to looks like a clan message.

That's nice, I was thinking about this not long ago. What happens if someone hops worlds 10x?

Unfortunately you will get lots of messages. The plan was to remove the message after a few seconds to stop clutter but I could not manage to do that.


Can't ssh into the VM on Windows

General information

OS: Windows 10 Pro
Hypervisor: Hyper-V
Did you run crc setup before starting it (Yes/No)? Yes
Running CRC on: Laptop

CRC version
CRC version: 2.20.0+f3a947
OpenShift version: 4.13.0
Podman version: 4.4.4

CRC status
DEBU CRC version: 2.20.0+f3a947
DEBU OpenShift version: 4.13.0
DEBU Podman version: 4.4.4
DEBU Running 'crc status'
CRC VM:          Running
MicroShift:      Running (v4.13.0)
RAM Usage:       1.17GB of 3.902GB
Disk Usage:      4.836GB of 16.1GB (Inside the CRC VM)
Cache Usage:     27.44GB
Cache Directory: C:\Users\demo\.crc\cache

CRC config
- preset                                : microshift

Host Operating System
# Put the output of `cat /etc/os-release` in case of Linux
# put the output of `sw_vers` in case of Mac
# Put the output of `systeminfo` in case of Windows

Steps to reproduce

Start CRC
Ssh into the VM

Expected
SSH enters the VM
Actual
The private key is accessible to current user (demo) and another one (None ?). Removing the last makes SSH working again
C:\Users\demo>ssh -i ~/.crc/machines/crc/id_ecdsa -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null<EMAIL_ADDRESS>-p 2222
Warning: Permanently added '[<IP_ADDRESS>]:2222' (ECDSA) to the list of known hosts.
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@         WARNING: UNPROTECTED PRIVATE KEY FILE!          @
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
Permissions for 'C:\\Users\\demo/.crc/machines/crc/id_ecdsa' are too open.
It is required that your private key files are NOT accessible by others.
This private key will be ignored.
Load key "C:\\Users\\demo/.crc/machines/crc/id_ecdsa": bad permissions
<EMAIL_ADDRESS>password:

Logs
Before gather the logs try following if that fix your issue
$ crc delete -f
$ crc cleanup
$ crc setup
$ crc start --log-level debug

Please consider posting the output of crc start --log-level debug  on http://gist.github.com/ and post the link in the issue.

Load key "C:\Users\demo/.crc/machines/crc/id_ecdsa": bad permissions

This is a local issue. That happens when you are using the 'wrong' ssh client.
CRC itself is not affected by this.
WONTFIX

https://superuser.com/questions/1296024/windows-ssh-permissions-for-private-key-are-too-open


azure: update docs with updated default param

Update --azure-image param documented that changed in
Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>

LGTM

this documentation check fails very often. :)

Yes I can't discern real failures from false alarms in that any more

let's try the magic incantation...

test this please

yay the magic worked 🐰 🎩


[docs] Specify which style convention is enforced

@gastonche can you specify which style guide conventions are enforced in the linter?

PS: you can add this information in https://github.com/openwisp/netjsonconfig-editor.js/blob/basic/README.rst#contributing (see the TODO note)


upgrade admin ui to 1.6.2

Codecov Report

Merging #6328 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #6328      +/-   ##
============================================
+ Coverage     83.26%   83.28%   +0.01%
+ Complexity    12821    12817       -4
============================================
  Files          1480     1480
  Lines         48489    48489
  Branches       5910     5910
============================================
+ Hits          40376    40383       +7
+ Misses         5358     5349       -9
- Partials       2755     2757       +2

Impacted Files
Coverage Δ
Complexity Δ

...exceptions/GroupByOnArrayUnsupportedException.java
66.66% <0%> (-33.34%)
1% <0%> (-1%)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
47.82% <0%> (-26.09%)
5% <0%> (-3%)

...ticsearch/indices/recovery/BlobRecoveryStatus.java
63.63% <0%> (-9.1%)
3% <0%> (-1%)

...io/crate/protocols/postgres/types/VarCharType.java
91.66% <0%> (-8.34%)
10% <0%> (-1%)

...xecutor/transport/TransportSchemaUpdateAction.java
83.87% <0%> (-4.84%)
13% <0%> (-2%)

.../projectors/BatchIteratorBackpressureExecutor.java
92.53% <0%> (-4.48%)
17% <0%> (-1%)

...er/src/main/java/io/crate/sql/tree/AstVisitor.java
30% <0%> (-2.31%)
39% <0%> (-3%)

...ticsearch/indices/recovery/BlobRecoveryTarget.java
52.04% <0%> (-2.05%)
1% <0%> (ø)

.../src/main/java/io/crate/blob/RemoteDigestBlob.java
78.57% <0%> (-1.79%)
8% <0%> (ø)

.../crate/executor/transport/task/UpsertByIdTask.java
89.31% <0%> (-0.77%)
37% <0%> (-2%)

... and 13 more

Continue to review full report at Codecov.

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


add azure mention

Signed-off-by: craig<EMAIL_ADDRESS>rh-pre-commit.version: 2.2.0
rh-pre-commit.check-secrets: ENABLED

minor doc update


micro: add syntax highlighting for nix

Issue description
For nano, we specifically grab .nix syntax highlighting files from somewhere. I wanted to adapt these to micro and patch them into our derivation as well, but the repo doesn't have a license specified, so I'm fuzzy on the details there.
Steps to reproduce

Open a .nix file in micro

Technical details
n/a

Still relevant

I would also like this feature

micro added syntax highlighting for nix in June 2021, released in version v2.0.10 (August 2021) upstream.
See https://github.com/zyedidia/micro/issues/1769, https://github.com/zyedidia/micro/pull/2024 and #132953


gh-442 Return original url from HTML link parser for http element types

gh-442 Added processing for http element types when parsing text to html

@jamesrwelch Yes, I can make a fix for email addresses as well in the new ticket.


DwdsInjector Finished with error: HttpException: Content size below specified contentLength.  0 bytes written but expected 1225., uri = /main_module.bootstrap.js

For me this project not working with flutter web..
Always the same errors:

Black Screen
Finished with error: HttpException: Content size below specified contentLength.  0 bytes written but expected 1225., uri = /main_module.bootstrap.js

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 2.3.0-2.0.pre.206, on Ubuntu 20.04.1 LTS 5.4.0-73-generic, locale en_US.utf8)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.2)
[✓] Connected device (1 available)

\cc @grouma

This should have been fixed with https://github.com/dart-lang/shelf/pull/183
Looks like that has been pulled into Flutter Tools already. Is this still an issue?

@fede1295 What's the output of flutter doctor -v?

@jmagman
[✓] Flutter (Channel master, 2.3.0-2.0.pre.206, on Ubuntu 20.04.1 LTS
5.4.0-73-generic, locale en_US.utf8)
• Flutter version 2.3.0-2.0.pre.206 at /home/federico/Flutter/Master/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision ea09f4ecbc (4 days ago), 2021-05-14 15:09:02 +0800
• Engine revision c75ec8ea4d
• Dart version 2.14.0 (build 2.14.0-115.0.dev)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /home/federico/Android/Sdk
• Platform android-30, build-tools 30.0.2
• Java binary at:
/home/federico/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.
6953283/jre/bin/java
• Java version OpenJDK Runtime Environment (build
1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• Chrome at google-chrome
[✓] Android Studio (version 4.1)
• Android Studio at
/home/federico/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.
6953283
• Flutter plugin version 51.0.2
• Dart plugin version 201.9245
• Java version OpenJDK Runtime Environment (build
1.8.0_242-release-1644-b3-6222593)
[✓] Connected device (1 available)
• Chrome (web) • chrome • web-javascript • Google Chrome 87.0.4280.141
• No issues found!

This should have been fixed with dart-lang/shelf#183
Looks like that has been pulled into Flutter Tools already. Is this still an issue?

shelf 1.1.4 was updated with 3445cb86f2.  User is on ea09f4e from a few hours before.
@fede1295 would you verifying it works on current master?


Get current disk cache usage

Is it possible to get the current disk cache usage? By that, I mean the actual size that the disk cache is currently using for the app, not the maximum disk cache size.

There seems to be a method in DiskLruCache called size(). Can you expose this method in LruDiskCache?

Alternatively, you could use StorageUtils.getCacheDirectory(context) and then compute the directory size


[SECOAUTH-440] Add support for token revocation proposed standard

Priority: Minor
Original Assignee: Dave Syer
Reporter: Harleen Sahni
Created At: Tue, 7 Jan 2014 21:19:50 +0000
Last Updated on Jira: Wed, 8 Jan 2014 14:47:05 +0000

Add support for the proposed standard for oauth2 revocation, basically support requests such as:
POST /revoke HTTP/1.1
     Host: server.example.com
     Content-Type: application/x-www-form-urlencoded
     Authorization: Basic czZCaGRSa3F0MzpnWDFmQmF0M2JW

     token=45ghiukldjahdnhzdauz&token_type_hint=refresh_token

to revoke tokens.
See http://tools.ietf.org/html/rfc7009

+1

+1

+1

+1

+1

+1

+1

Is this on the roadmap yet?

+1

A DELETE method on the /auth/token endpoint would also be elegant

@dfcoffin As an FYI, we are limiting new features and only accepting bug fixes and minor enhancements as this project is in maintenance mode. Our efforts are focused on the new OAuth support we are building into Spring Security core project. We just released 5.0.0.M1 which provides support for the client-side and we hope to get to the server-side support soon.

Do we have any timelines on implementation of RFC 7009 in the near future?

@bas-velthuizen no it wouldn't. You cannot delete the /auth/token resource. It would need to be /auth/token/{token}.


Update incident-response-overview.md

Updating year of the latest digital defense report to reflect 2022 year

#sign-off

#please-close


How relates to queueMicrotask

I'd suggest adding some mention of queueMicrotask in the section of the README on macrotasks/microtasks: https://developer.mozilla.org/en-US/docs/Web/API/WindowOrWorkerGlobalScope/queueMicrotask
Not offering a specific PR, but seems it would be germane and of interest.

related and interesting comment about queueMicrotask, setImmediate and others... https://github.com/w3c/setImmediate/issues/3#issuecomment-342238397


Outdated screenshots

[Enter feedback here]
Screenshot outdated in the deploy to azure section:

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

ID: 47cb39a6-cd9e-4f02-b31b-1e09d6a7fa3f
Version Independent ID: eeb0f211-80b7-9006-72e1-b72b6a58fa8d
Content: Create a Java function from the command line - Azure Functions
Content Source: articles/azure-functions/create-first-function-cli-java.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@yucwan Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@yucwan Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.


changed the fragment_profile_stats to Constraintlayout

What this PR does

Changed the Layout from LinearLayout to ConstraintLayout

Open questions and concerns
UI remains the same before and after

@gconnect please remove the nested constraint layout and modify the layout IDs. Constraint layouts IDs should start with cl_

@ujjwalagrawal17 I have modified the layout ids. I left the nested Constraints because they are needed to differentiate AsStudent and ByStudent categories used to set the visibility.

Cool. I will review it soon.
On Tue 5 Nov, 2019, 2:29 AM gconnect<EMAIL_ADDRESS>wrote:

@gconnect https://github.com/gconnect requested your review on: #225
https://github.com/WikiEducationFoundation/apps-android-wikiedudashboard/pull/225
changed the fragment_profile_stats to Constraintlayout.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/WikiEducationFoundation/apps-android-wikiedudashboard/pull/225?email_source=notifications&email_token=AEVTAAYWMC5NIDY6KRFQXYLQSCEKNA5CNFSM4JID7G22YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUUHMC4A#event-2769207664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEVTAA4PYG5EHXUY7VGJFR3QSCEKNANCNFSM4JID7G2Q
.

@ujjwalagrawal17 I have fixed the conflict

okay will do that


Add trinary exercise

This adds a trinary exercise to the clojure track. The tests were ported
from xruby.

It might be cool to do this in one pass, rather than re-seq, num->digits, reverse map-indexed and apply. .compareTo can come in handy too.

I went ahead and made the changes and merged #90. Thanks!

Ah nice. compareTo avoids the need turn it into an integer. I like the destructuring too; it didn't occur to me to use it with strings.

Gotta love the seq abstraction.


【推荐】快速注册ChatGPT教程

快速注册ChatGPT完整教程：注册教程

It is a spam, commercial, against TOS (many) etc. Also untrue ;) , as one CAN se free virtual non-Chinese  #s for this.
Let us remove this pseudo isssue.
GTrans:

On the payment page, fill in 1 for the payable amount , charge 1 USD, then click the [Payment] button, and scan the QR code to pay on the Alipay page that pops up (I won’t take a screenshot here)....


"TOPLIȚA", "Călimănel", "Luncani", "Măgheruș", "Moglănești", "Secu", "Vale", "Vâgani" and "Zencani" are listed in the drop-down menu

Description:
"TOPLIȚA", "Călimănel", "Luncani", "Măgheruș", "Moglănești", "Secu", "Vale", "Vâgani" and "Zencani" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "TOPLIȚA" in the search bar.

Expected results
The location is listed in the drop-down menu as "Municipiul TOPLIȚA, HARGHITA" and "Oraș TOPLIȚA, HARGHITA (TOPLIȚA)".

Step 1
Type "Călimănel" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Călimănel, HARGHITA (TOPLIȚA)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Luncani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Luncani, HARGHITA (TOPLIȚA)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Măgheruș" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Măgheruș, HARGHITA (TOPLIȚA)".

Step 9
Type "Moglănești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Moglănești, HARGHITA (TOPLIȚA)".

Step 10
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 11
Type "Secu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Secu, HARGHITA TOPLIȚA)".

Step 12
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 13
Type "Vale" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Vale, HARGHITA (TOPLIȚA)".

Step 14
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 15
Type "Vâgani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Vâgani, HARGHITA (TOPLIȚA)".

Step 17
Type "Zencani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Zencani, HARGHITA (TOPLIȚA)".


Vulnerability CRLF injection in RestSharp >= 107.0.0 && < 120.0.0

https://github.com/restsharp/RestSharp/security/advisories/GHSA-4rr6-2v9v-wcpc
Manually bumping to 120.0.0 throws MethodNotFoundException.

Hmm, the latest version of RestSharp per today is v112.1.0 (not 120.0.0)
The RestSharp component has got an update regarding RestSharp version + several others issues. See release v2<IP_ADDRESS>.
Therefore this issue is considered as closed.


Refactor PrepareDenominate, split it into 2 decoupled steps.

As a client, submit part of a future mixing transaction to a Masternode to start the process (SubmitDenominate):
step 1: prepare denominated inputs and outputs (PrepareDenominate, code moved from wallet.cpp, slightly refactored)
step 2: send denominated inputs and outputs (SendDenominate, slightly refactored)

Rebased after #980

Mixing works as expected for me with this (mainnet, downgraded protocol versions). Have anyone else tested? Any issues?

Since there is no nACKs, I'm assuming everyone is ok with this, merging


[Request] Unique handles

Hi.
Is it possible to get the code generators to generate unique handles like in vulkan.hpp?
For example, alloc.createBufferUnique(...) where alloc is a vma::Allocator.

I just tried to add them myself but just realized that it's not very straightforward to generate unique handles where vma returns pairs.
For example, the function 'vma::Allocator::createBuffer(..)' returns a pair that has to be deleted together.
Any ideas? Or is just a bad idea to add unique handles in vma?

Sure it's a good idea. But I believe they are not necessarily destroyed together (vmaDestroyBuffer = vkDestroyBuffer + vmaFreeMemory), so I think we can just return a pair of unique handles in that case.

I see.
I'm still very new to VMA, so I guess I'll give myself more time to understand it.


Remove support for GopherJS

WIP PR for removing the support for using GopherJS.

I was going to say that all other targets matched an operating system and that we ought to stick to wasm for consistency, but then I checked and wasm is the architecture and not the os. Technically we should do js, I guess, but maybe we can make web be an alias for that?

(So yes, I agree but maybe we ought to use js instead of web as the os target)

I think it would be very confusing if we move to "js" and yet are removing the GopherJS support.
Also JS is a language and could imply that we are transpiling which is not what the web/WASM is doing.
I guess I don't see why we should move from the existing os=web which has been in place for over a year...

Yeah, you are right. Using js would be more consistent with Go tooling but yeah (Go wasm builds actually have GOOS=js), it does give the wrong impression. I'll keep web and wasm around.

I think it should be possible to merge this and pin the repo to the organization after that :)

Is it intended to keep the "js" os value present?

Hmm. I forgot to remove hit but it is the actual GOOS for wasm so maybe we want to support it?

You can pin the repo now :)

Done :)


Create Offer Letter to Lead Scientist

Create an offer letter to a lead scientist that follows a similar pattern:
An executive level offer letter should be comprehensive, clearly outlining the terms of employment, benefits, and any other perks associated with the position. It's essential to ensure that the offer is attractive and competitive, reflecting the level of responsibility and expertise expected from the executive. Based on your draft, here are some additional elements and refinements you might consider including to ensure the offer letter is complete and standard for an executive role:
Heading and Introduction

Company Logo and Contact Information: Ensure the letterhead is professional and includes your company's logo and contact details.
Date: The date the offer is made.
Candidate's Name and Address: Personalize the letter to the candidate.

Opening Paragraph

Job Title and Start Date: Confirm the position being offered and the proposed start date.
Welcome Statement: A brief sentence welcoming them to the team or expressing excitement about the potential of working together.

Compensation and Benefits

Base Salary: Clearly state the annual base salary.
Bonus/Incentive Plans: Detail any performance-related bonuses, stock options, or equity plans.
Vacation Package: Your current offering is strong; ensure clarity on the number of vacation days per year.
Family Housing Provided: Confirm the details as you have, possibly adding any allowances for personalization or utilities coverage.
Boarding Provided: Clarify if this is for the entire family and any options or limitations.
Cash Compensation: Ensure the monthly amount is competitive for the role and location. Consider specifying the total annual compensation for clarity.
Vehicle Provided: Detail the type of vehicle, usage policies (personal vs. business use), and any maintenance coverage.
Bus Pass Provided: Confirm if this is for the executive only or includes family members.
Health Program Provided: Expand on the SILK Health program, including coverage details (e.g., dental, vision), deductibles, co-pays, and any wellness programs.
Retirement Plans: Include information on any 401(k) plans, pension plans, or other retirement benefits, including employer matching contributions.

Additional Perks

Relocation Assistance: If applicable, detail any support for relocating, including moving expenses and temporary housing.
Professional Development: Outline opportunities for professional growth, such as training, conferences, or education reimbursement.
Technology: Confirm the specifics of the AI Workstations and In-Home AI Research Lab, including any software or subscription services provided.
Work-Life Balance Programs: Mention any policies or programs supporting work-life balance, such as flexible working hours, remote work options, or wellness initiatives.

Terms of Employment

At-Will Employment Clause: If applicable, clarify that the employment is at-will.
Confidentiality/Non-Disclosure Agreements: Highlight any agreements the executive will need to sign.
Background Check: Mention if the offer is contingent upon a satisfactory background check.
Legal Right to Work: State that employment is contingent upon proving legal right to work in the country.

Closing

Acceptance: Provide instructions for how to accept the offer, including any deadlines.
Contact Information: Offer a specific contact person for any questions or clarifications.

Signature

Signature of Company Representative: Include a space for the signature of a company representative.
Signature of Candidate: Include a space for the candidate to sign, indicating acceptance of the offer.

Remember, the offer letter should be clear, concise, and free of any jargon that might confuse the recipient. It's also a good idea to have it reviewed by legal counsel to ensure compliance with employment laws and regulations.

By the end of my shift I will have an executive-level offer letter for a lead scientist position that is both comprehensive and competitive, ensuring it clearly outlines all terms of employment, benefits, and additional perks. My goal is to finalize and refine this document by the end of the day, including essential elements such as the company logo, detailed compensation packages, and terms of employment, among other key details. I will ensure the offer is attractive to the candidate and positions our company positively, while also adhering to all relevant employment laws and regulations. To accomplish this, I will organize and include all necessary information into a structured and easy-to-understand format, prioritizing clarity and succinctness throughout the letter.


Export-Excel slow when run in an Azure Automation runbook

When  I run the following in a local PowerShell session (Windows 10, PS 5.1), it executes within seconds:
$AllUserLicenses = Get-MsolUser -All -PipelineVariable MsolUser | Where-Object isLicensed
$AllUserLicenses | Export-Excel -Path $XlsxPath -WorkSheetname Licenses -AutoSize -TableName Licenses
When the same code is running inside an Azure Automation runbook which is executed on a Windows Server 2016 Hybrid Runbook Worker, the runbook uses 21 minutes from start to finish and Export-Excel seems to be the slowness based on Write-Output commands inside the runbook.
Any thoughts on why? And how to troubleshoot?

Wow, off the top of my head, I'm not sure. The Hybrid Worker? Are you running the Get-MsolUser in Azure and saving to an on premise file?

Correct. Does Export-Excel write to the verbose stream by default? It might be that the Hybrid Runbook Worker is streaming data back to Azure, including the verbose stream. I`ll try with -Verbose:$false to see if it makes a difference.

Wow. Glad you found it. Good to know. Sounds like a good blog post. :)


Empty repositories Always Show Quick-Start Guide after PUSH

Description
No response
Gitea Version
1.18.0+rc0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
The problem is still not solved!
https://github.com/go-gitea/gitea/issues/2898
https://github.com/go-gitea/gitea/issues/2665
Is there any solution?
Install GITLAB?
Database
SQLite

Hi @mariuszmaximus, this is likely due to git hooks not firing. This can be caused by many reasons, some are documented here: https://docs.gitea.io/en-us/faq/#push-hook--webhook-arent-running

You have right !  Thank you for the link !
I check my server and docker version
I have ubuntu 19.xx and very old docker
After update Ubuntu to 22.04  all works, many other issues have also been fixed

Glad to hear :)


Add ADPCM codec for Mac client

need to build with -DWITH_MACAUDIO

@freerdp-bot test this nice pr

@freerdp-bot test

the @freerdp-bot only listen to the voice of its master ;)

LGTM

LGTM +1
If you add a cmake_dependent_option for WITH_MACAUDIO the pull can be merged.

I would rather add a
set (WITH_MACAUDIO ON CACHE BOOL "Core audio"
in ci/cmake-preloads/config-macosx.txt

@DavBfr The idea would be to automatically activate the flag if the build is for OS X or iOS (but not others). The ci scripts are more of a specialized configuration for our automated build system.

yes but there is not such thing for linux or windows.

@davbfr well, one platform a time ;)

@DavBfr Something like cmake_dependent_option(WITH_MACAUDIO "Enable OSX/iOS sound backend" ON "APPLE" OFF) in channels/ConfigOptions.cmake just like the one for windows.


"System.FormatException: String was not recognized as a valid Boolean"

{
"#variable(var1)": "#('abc')",
"#variable(var2)": "#('def')",
"MyProp": "#(0 < 1 && 1 == 1 ? $var1 : $var2)"
}

Fixed


issues/634: orientdb.profile=orientdb_MEMORY for jetty-block profile

@carlspring it's ready to be merged

Thanks!


Fix warning messages

All the warnings were displaying the wrong stack level: now corrected to show user code rather than snakebids code.
The warnings relating to bids were poorly worded or slightly inaccurate. These have been corrected

Also just fixed the error message for when multiple paths are found. That one's been bothering me for a while


Pointing to the report page instead of the report PDF

I am the head of content at the DoKC. As a community, we need to track how our content performs and who reads it to better understand the impact of our work and ensure it is useful to our community. Pointing to the PDF directly means that we don't get to know the data we need.

Hi there,
My apologies for not replying earlier. Thanks for your PR along the given details.


feature_request: Add Drill-Down-Dimensions

For many users it would be helpful to add Drill-Down-Dimensions to the Governed Metrics Service. For now it's possible to add normal dimensions.

I would like to add my vote for this request.

Will be added to 2.0 release scheduled for end Jan begin Feb 2017.

Great, thanks!  :)

Resolved in https://github.com/eapowertools/GovernedMetricsService/releases/tag/2.0.0


[question] Force ecal to not use shm

Hello
I am trying to run ecal c++ sample apps from the docs in two different containers running in the same machine.
The containers are exposed to the host machine network but I am unable to make the subscriber app to  receive any data from the publisher app unless both containers are exposed to both IPC and PID of the host machine.
I am using the following  configuration in my ecal.ini file:
[publisher]
use_inproc = 0
use_shm = 0
use_tcp = 0
use_udp_mc = 2
share_ttype = 1

[network]
network_enabled           = true
multicast_ttl             = 2
shm_rec_enabled           = false
udp_mc_rec_enabled        = true

Is there a way to force ecal to not use shm even if both apps are running in the same machine?

Hi @odra,
I try to get your right. eCAL has a kind of "auto" mode to check if it should transfer data using shared memory or a network protocol like udp multicast. This is mode "2" (auto). The decision logic is quite simple. If the host names of publisher and subscriber are different it switches to shm, if not to udp multicast.
So inside your container .. do the hosts have the same name (from the underlying machine) ? If yes .. can you rename them ?
If you want to use shared memory in different containers you have to link the dev/shm from the hosting machine.
The last option you have is to force eCAL to use udp only and switch of any kind of automatic detection. To do so you need to configure the ecal.ini file that way.
[publisher]
use_inproc = 0
use_shm = 0
use_tcp = 0
use_udp_mc = 1            <<< use 1 instead 2 to switch of any automatic and force udp
share_ttype = 1

Yes, the hosts have different hostnames, I will try to set to "1" and report my results :-)

Btw. there was a kind of similar question, see issue #238.

I tried to set it to 1but i didn't work, the sample subscriber app can't receive/read messages sent by the pblisher.
I saw that issue and I am able to make it work when exposing both IPC and and PID to the host machine but that is not an option for me.
I also have my network  routes properly set:

Would you know why does ecal need access to the PID in the host machine? Would this have something to do with /dev/shm usage?

The first line of the subscriber app log shows the following message: shm_open failed : /hello_world_topic_81654748045_shm errno: No such file or directory

Can you please start the sample_monitoring_reg sample on one host and person_snd on the other one and check if the monitoring registration information are exchanged in the background ?
sample_monitoring_reg should log something like this

Closing, as the discussion seems to be resolved.


Support an alternative flexible allocation scheme that would submit unique copies of the same test on different nodes

This is different from the current flexible tests, which submits one test on multiple nodes. The problem with the current setup is that you probably need to change the benchmark you are running in order to include node information, which also makes much tougher the sanity checking as well as performance checking that would allow us to identify bad nodes.

In #2395 you gave some comments, but since we closed it as duplicate, let's continue here.

Even if you get the list of nodes of interest in a partition, you would still need to parametrize your test differently per partition.

I'm not sure I understand you correctly here. You mean we cannot simply do e.g.
parametrize_over_nodes = parameter(get_node_names())
valid_systems = ['sysA:part1', 'sysA:part2']

Because the get_node_names() function would need to return a different node set for part1 and part2 ?

but the workaround is to define the parameter to take the union of the parameter values and then filter in a post-init hook. For nodes, I agree, this is not so nice.

I guess that is the solution we essentially discussed on the Slack channel, right? To give an example, I now developed a test that does:
valid_systems = [
    'A:part1',
    'A:part2'
]

nodelist = {
    'A:part1': ['node1', 'node2', ...],
    'A:part2': ['node10', 'node11', ...],
}

node_list = []
for system in valid_systems:
    if system in nodelist:
        tmplist=[]
        for node in nodelist[system]:
            tmplist.append(([system], node)) # I guess this is what you meant by the union of parameter values?
         node_list.extend(tmplist)
parametrize_over_nodes = parameter(node_list)

   # And this is the post init-hook you refer to, to 'filter' the valid partitions?
   @run_after('init')
    def scope_systems(self):
        self.valid_systems = self.test_size_variant[1]

    @run_before('run')
    def prepare_singlenode_run(self):
        # Check if a single node name has been set
        nodename = self.test_size_variant[2]
        if nodename:
            self.job.options = [f"--nodelist={nodename}"]
...

(ignore the hardcoded nodelist for now, I'm sure we can do that much more elegantly when something like this is properly integrated in the framework). This works, but as you say, having to put the system as part of the tuple to make a parameter that defines essentially has already expanded all system + nodename combinations is not very elegant.
I would think that it might be possible to integrate this expansion in the framework though. I.e. the user would specify something in the class body to signal that this is a test of which one copy should be run on each node with a certain status, e.g.
one_per_node = 'IDLE'

And that the framework then generates copies of the test where it takes all (valid) permutations of valid_system + node name combinations. The case doesn't seem so different from e.g. generating the all valid permutations of valid_systems + valid_prog_environs. E.g. a test that does:
valid_prog_environs = ['foss', 'intel']
valid_systems = ['A:part1', 'A:part2']

and a ReFrame settings file that defines:
...
'name': 'part1',
'environs': ['foss', 'intel']
...
'name': 'part2',
'environs': ['foss']

Would also generate tests only for the valid combinations of systems + programming environments (i.e. A:part1-foss, A:part1-intel, A:part2-foss).
But then... I obviously don't know the framework as well as you guys, so maybe this is much more difficult for partition+nodename combinations than it is for partition+prog_env's... :)

because the get_node_names() function would need to return a different node set for part1 and part2? If so, I understand, and that is indeed clearly a challenge.

Exactly.

I guess that is the solution we essentially discussed on the Slack channel, right?

Right.

It seems like a lot of boiler plate code for something that is probably a pretty commonly desired execution pattern.

Indeed, but actually it is not so much code. You can write this in a much simpler way:

# NOTE: I'm using sets here for a quick look up in `find_systems` in case you have very long node lists.
nodelists = {
    'A:part1': {'node1', 'node2'},
    'A:part2': {'node10', 'node11'},
}

def find_system(node):
    for system, nodes in nodelists.items():
        if node in nodes:
            return system

class my_test(...):
    nodeid = parameter(itertools.chain(*nodelists.values()))

    @run_after('init')
    def scope_systems(self):
        self.valid_systems = [find_system(self.nodeid)]

In any case, I agree that trying to define a parameter based on the value of a variable, is a recurring pattern that also arises in test libraries, so it should be addressed by the framework.
I think the solution you propose is not going towards the right direction. First, I don't think that an additional test variable is needed. My idea is to allow users to run any single node test flexibly without having to change anything in the test. Users could use tags to mark tests that they will run flexibly. I am leaning towards something like this:
reframe -t <tag> --one-per-node -J reservation=foo ... -r

I don't like very much the --one-per-node name, but you get my point.
Internally, all we need to is to create programmatically a test with a different name for each of the system partitions, deriving from the original one, that will be parameterized on the node list of the corresponding partition. And those tests will be generated only if the current system is listed in the valid_systems of the original test.


trie.go: sort address is enough

Why change this?

This logic actually works as sorting address in order, so we don't need to encode addr&value to uint64 and decode it, it is useless in my concern.

I suspect the encoding is actually faster, since you don't have to fetch from the map each time. Considering it works fine and I don't think this is simpler, not going to merge.


Your Icons on nested items are not loading

You have nav items with nested content and the Icons "UserPen" are not rendering.

The icon component is missing in nested items. That's why!