GPL-nnn As a developer or a user I would like to be able to add tags to a plate automatically

User story
As a developer or a user I would like to be able to add tags to a plate automatically. Currently you need to add tags through libraries via a pipeline. R&D need to be able to tag samples so they can be run through an ONT pipeline in traction without running it through a high throughput pipeline.
Who are the primary contacts for this story
e.g. Steve.
Acceptance criteria
To be considered successful the solution must allow:

[ ] I want to be able to add tags to a plate of samples automatically
[ ] ...

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Would a library plate manifest with tag group and indexes work here, (Instead of the current one which is I believe by sequence) or do the samples need to exist in Sequencescape in an untagged form as well.

Would a library plate manifest with tag group and indexes work here, (Instead of the current one which is I believe by sequence) or do the samples need to exist in Sequencescape in an untagged form as well.

I think the point is here that users (R&D / Long Read) don't want to be adding the tags at all except by automation. For example they have been creating the plates in Sequencescape and these plates are then imported into traction. The samples need to have tags but the libraries are created in Traction. They know the strategy i.e. ONT PCR tags need to be added by column and it seems a waste of time for them to then add those tags manually by index or sequence in a manifest. Hope that makes sense?


Crashes when using latest 2.0 release for 1.16

Minecraft: 1.16_combat-6
Mod: 2.0.1+1.16
Details
The game crashes when I use the latest 2.0 release but does not crash when I use a 1.0 release
Please run the game with -Dextrasounds.debug=true added to your Java arguments and attach a log file as well.

@stashymane why was this closed? it isn't resolved yet

May have closed this when mass-closing the 1.19 issues, my bad

Yes, I may have compiled with Java 16, but there are a few other issues blocking an update, the main one being a severe lack of time. I'll try my best to work on the mod in a week or two.


Disable algin in the query not work, when use session pool in java

Describe the bug
Disable algin in the query not work, when use session pool in java.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

MacOS 11.4
IoTDB 0.12.1
Additional context
Add any other context about the problem here.

Hi, this feature is under developing. See https://github.com/apache/iotdb/issues/2770 .

ok, waiting for release.


Place Y axis on Right

Hi when viewing financial charts, it can be useful to show the Y axis on the right, closest to the latest data point. It looks like currently this isn't possible, from this code:
        public static void FigureTicks(Settings settings)
        {
            if (settings.dataSize.Width >= 1 && settings.dataSize.Height >= 1)
            {
                settings.ticks.x.Recalculate(settings);
                settings.ticks.y.Recalculate(settings);
                RenderTicksOnLeft(settings);
                RenderTicksOnBottom(settings);
                RenderTickMultipliers(settings);
            }
        }

Would be great to have! Library is very nice nontheless.

Hi @Linchy,
Multiple axes is a is a new feature in ScottPlot 4.1. To make this easier to discover I added a new example to the cookbook:
OHLC[] prices = DataGen.RandomStockPrices(null, 30, TimeSpan.FromMinutes(5));
plt.AddCandlesticks(prices);
plt.XAxis.DateTimeFormat(true);

plt.YAxis.Ticks(false);
plt.YAxis2.Ticks(true);
plt.YAxis2.Label("Price (USD)");

You can see additional multi-axis examples in the cookbook: https://swharden.com/scottplot/cookbooks/4.1.2-beta/#multi-axis
Hope it helps!
Scott


Refactor with NEAR standard NEP141

Overview
To significantly simplify the contract, as well as to avoid potential errors and pitfalls associated with the implementation of the NEP-141, the contract is being refactored with a library near-standard dependency.
What is removed
All implementations related to Fungible TOkens. And introduced dependency from near-standard.
Breaking changes
NEP-141 basic (ft_transfer, ft_transfer_call) methods are now public, without access right control.
All related methods were also removed.
All types, that were related to NEP-141 implementation are deleted.
Tests
Tests refactored qith new types dependencies and with keep in mind of new changes
Gas cost
Gas cost increase x2 (twice) for migrations. It relates to a new organization of storage key.

@birchmd unfortunately, gas costs for migration increased twice (!). AFAIK it depends on how currently storage key is organized. Specifically proof keys.

A factor of 2 is a large increase. It's not clear to me why this increase happens. You say it's because of the proof keys, but it looks like both before and after this change they were contained in a LookupMap<String, bool>. Could you elaborate more on what difference causes the gas increase?
Also, is this factor of 2 increase a problem for this migration?

Could you elaborate more on what difference causes the gas increase?
Even with increased gas, all migration costs only 1.5 NEAR (in testnet).

What was changed: before refactoring, proof keys were part of FungibleToken struct. And yes it was LookupMap. For some reason, after moving it to the root of the Contract struct - gas costs increased.

Are you compiling the contract with link-time optimization enabled?

Yes, it is:
https://github.com/aurora-is-near/aurora-eth-connector/blob/feat/use-near-standard-nep141-implementation/Cargo.toml

@birchmd @mrLSD gas cost possibly may have increased with Workspace 0.7.0?

Tests failing :(, please fix.

@birchmd @mrLSD gas cost possibly may have increased with Workspace 0.7.0?

It was changed after the proposal in: https://github.com/near/workspaces-rs/issues/255 ,
https://github.com/near/workspaces-rs/issues/253
Opened 2 issues about that. Currently no success.

Whats the status on this?


View Transfers window

There should be listed a name of the file or directory which is being transfered in view transfers window...

It is very likely that this feature will be added to the next release.

Received my first file from Nitroshare and already miss this feature. )

I'll see if I can get that feature added this week.

I want to see filename to know if I should accept or skip it in https://github.com/nitroshare/nitroshare-desktop/issues/121 so if Transfer Window showed files currently in "quarantine", that could make a good experience. Like:
filename            [Accept][Remove]
filename            [Accept][Remove]
filename            [Open]

Hmm... this isn't possible using the current transfer protocol since the filename and other metadata aren't transferred until immediately before the payload. In other words, it would need to prompt for each file one-at-a-time as it is received. However, it is certainly possible to display the current item being transferred.

I think about quarantine after the file is transferred. So Accept here means moving from unsafe area to download destination. No need to keep pusher waiting for transfer.

Oh, okay. Yes, that's definitely possible.

I implemented a quarantine option as you suggested. It still needs a bit of adjustment but for the most part it works. To enable the feature, ensure the following checkbox is checked:

Once enabled, you will receive a prompt when a transfer completes:

Clicking "yes" will move them to the correct location and clicking "no" will simply ignore the files.

I'd prefer not to see popup windows, but click the icon myself to open the transfer window and accept them. Another problem with the window that there is no list of files that are going to be accepted.

"I'd prefer not to see popup windows, but click the icon myself to open the transfer window and accept them."

This is a hard decision to make. On the one hand, it makes sense to inform the user that a transfer has completed and must be accepted. On the other hand, I can understand how this would quickly get out of control if a large number of transfers were received at once. Perhaps the behavior can be controlled by a separate setting.

"Another problem with the window that there is no list of files that are going to be accepted."

Yes, I plan to have that added very soon.

Both the list of items and popup window request have been completed now.

@nathan-osman : Just for clarification - is the client on the receiver side blocking the transfer until the receiver accepts the files or are the files already transferred before the prompt pops up?
It should ask before it accepts the transfer - otherwise anybody will be able to fill up the drive of the receiver regardless of the prompt.
Thanks alot!


Instance Options

Summary

the exhaustive list of the current instance options (90% copied/pasted from the official doc)
2 new options --max-indexing-memory (MEILI_MAX_INDEXING_MEMORY) and --max-indexing-threads (MEILI_MAX_INDEXING_THREADS)

Information To Reviewers
📣 If you only are interested in both new options, or you want to do a quick review, I recommend you check only the paragraphs regarding both options.
Misc

[x] Update telemetry-policies.md spec to describe newly added options.

@ManyTheFish @MarinPostma @bidoubiwa and @gmourier I've updated the naming according to your feedback and to the @dichotommy help regarding the naming.

It looks like we're ready for one last round of approval and then I merge on to develop.


Mixscape reproducibility

PR Checklist

[X] Referenced issue is linked (closes #671)
[X] If you've fixed a bug or added code that should be tested, add tests!
[X] Documentation in docs is updated

Description of changes

As described in #671, running Mixscape multiple times can produce varying results due to the non-deterministic GaussianMixture model. I added a seed parameter to pt.tl.Mixscape.mixscape.
In the original Mixscape implementation, there is a parameter called n_dims, which specifies the number of dimensions from the chosen representation (usually PCA) to be used for determining the nearest neighbors. I have added this parameter to our implementation of pt.tl.Mixscape.perturbation_signature and set its default value to 15, which is also the default used in the original implementation.
I added a test for pt.tl.Mixscape.perturbation_signature that not only verifies that a layer has been added to the AnnData object but also checks the correctness of the computed scores
Input required: The perturbation signature was previously calculated by subtracting the average expression of the N nearest control (NT) cells from the observed expression. But, in the original implementation, this subtraction is performed in the reverse order (see here). This only affects the sign of the result, but aligning our calculation with the original implementation would make it easier to compare scores. I suggest adopting our calculation to the original approach, but happy to discuss this further.

The test fixes are unrelated and I'm hoping to fix them next week. Just ping me if you want a review, please.


Split lights into classes

Our if statements are getting too long! Need to split them out into subclasses for each different fixture.

Would it be an idea to allow light device configuration to specify a channel layout? I need RGBD, for instance, and I can imagine other users have lights that require yet again different layouts. Each channel could then receive a type indicator, and a modifier, such as r,g,b,d (RGB + dimmer), 'r,g,b,w(RGB + white value), andr,g,b,wa` (RGB + auto white value).


Building for Moto G4 with older Flutter versions gives "Bad CPU type in executable"

I tried building the following sample code with "flutter run --release", when synced to the v1.2.1 stable release, and got the error "Bad CPU type in executable" when building release. It also failed identically in v1.7.8+hotfix.4.
Yes, we could ask people to upgrade (this builds on v1.9.1+hotfix.6, for instance), but it does hinder our ability to bisect issues.

import 'package:flutter/material.dart';
void main() => runApp(MyApp());
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Material App',
theme: ThemeData.dark(),
home: Home(),
);
}
}
class Home extends StatelessWidget {
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
title: const Text('Material App Bar'),
),
body: Center(
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: const [
TextField(
autofocus: true,
),
Text(
'You have pushed the button this many times:',
),
],
),
),
floatingActionButton: FloatingActionButton(
child: const Icon(Icons.add),
onPressed: () {},
),
);
}
}

command
flutter build aot --suppress-analytics --quiet --target /Users/gspencer/code/manual_tests/lib/simple_test.dart --target-platform android-arm --output-dir /Users/gspencer/code/manual_tests/build/app/intermediates/flutter/release --release
exception
ProcessException: ProcessException: Bad CPU type in executable
Command: /Users/gspencer/code/flutter.stable/bin/cache/artifacts/engine/android-arm-release/darwin-x64/gen_snapshot --causal_async_stacks --packages=.packages --deterministic --snapshot_kind=app-aot-blobs --vm_snapshot_data=/Users/gspencer/code/manual_tests/build/app/intermediates/flutter/release/vm_snapshot_data --isolate_snapshot_data=/Users/gspencer/code/manual_tests/build/app/intermediates/flutter/release/isolate_snapshot_data --vm_snapshot_instructions=/Users/gspencer/code/manual_tests/build/app/intermediates/flutter/release/vm_snapshot_instr --isolate_snapshot_instructions=/Users/gspencer/code/manual_tests/build/app/intermediates/flutter/release/isolate_snapshot_instr --no-sim-use-hardfp --no-use-integer-division /Users/gspencer/code/manual_tests/build/app/intermediates/flutter/release/app.dill
#0      runCommandAndStreamOutput (package:flutter_tools/src/base/process.dart:140:27)
<asynchronous suspension>
#1      GenSnapshot.run (package:flutter_tools/src/base/build.dart:68:12)
#2      AOTSnapshotter.build (package:flutter_tools/src/base/build.dart:194:55)
<asynchronous suspension>
#3      BuildAotCommand.runCommand (package:flutter_tools/src/commands/build_aot.dart:137:56)
<asynchronous suspension>
#4      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:545:18)
<asynchronous suspension>
#5      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:482:33)
<asynchronous suspension>
#6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#7      _rootRun (dart:async/zone.dart:1124:13)
#8      _CustomZone.run (dart:async/zone.dart:1021:19)
#9      _runZoned (dart:async/zone.dart:1516:10)
#10     runZoned (dart:async/zone.dart:1463:12)
#11     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#12     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:473:20)
#13     CommandRunner.runCommand (package:args/command_runner.dart:196:27)
<asynchronous suspension>
#14     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:396:21)
<asynchronous suspension>
#15     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#16     _rootRun (dart:async/zone.dart:1124:13)
#17     _CustomZone.run (dart:async/zone.dart:1021:19)
#18     _runZoned (dart:async/zone.dart:1516:10)
#19     runZoned (dart:async/zone.dart:1463:12)
#20     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#21     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#22     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:111:29)
#23     new Future.sync (dart:async/future.dart:224:31)
#24     CommandRunner.run (package:args/command_runner.dart:111:11)
#25     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#26     run.<anonymous closure> (package:flutter_tools/runner.dart:60:20)
<asynchronous suspension>
#27     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#28     _rootRun (dart:async/zone.dart:1124:13)
#29     _CustomZone.run (dart:async/zone.dart:1021:19)
#30     _runZoned (dart:async/zone.dart:1516:10)
#31     runZoned (dart:async/zone.dart:1463:12)
#32     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#33     runInContext (package:flutter_tools/src/context_runner.dart:48:24)
<asynchronous suspension>
#34     run (package:flutter_tools/runner.dart:51:10)
#35     main (package:flutter_tools/executable.dart:52:9)
<asynchronous suspension>
#36     main (file:///Users/gspencer/code/flutter.stable/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#37     _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:298:32)
#38     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

flutter doctor
[!] Flutter (Channel unknown, v1.2.1, on Mac OS X 10.15.5 19F101, locale en-US)
    • Flutter version 1.2.1 at /Users/gspencer/code/flutter.stable
    • Framework revision 8661d8aecd (1 year, 4 months ago), 2019-02-14 19:19:53 -0800
    • Engine revision 3757390fa4
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)
    ✗ Downloaded executables cannot execute on host.
      See https://github.com/flutter/flutter/issues/6207 for more information

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/gspencer/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/gspencer/Library/Android/sdk
    • Java binary at: /Users/gspencer/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6514223/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[!] iOS toolchain - develop for iOS devices (Xcode 11.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4.1, Build version 11E503a
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    • ios-deploy 1.10.0
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Android Studio (version 4.0)
    • Android Studio at /Users/gspencer/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6514223/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.46.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.11.0

[✓] Connected device (1 available)
    • Moto G 4 • ZY223M6WQF • android-arm • Android 6.0.1 (API 23)

! Doctor found issues in 2 categories.

Might be related to dropping support of 32-bit binaries on recent versions of mac os.
The command above works fine on that version of flutter on linux.

Yeah, I don't think this is really actionable. You can't target ARM32 using old flutter releases on Catalina.


BUG: Update Navbar Background Color and Size

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
We need to update the navbar to ensure a consistent look and feel across all webpages. This includes changing the background color to match the navbar on other pages and decreasing its size for better visual alignment.web pages
What happened?
Requirements:
Background Color:
Change the navbar's background color to match the navbar color used on other webpages of the project. Ensure the color code matches exactly to maintain consistency.
Navbar Size:
Decrease the height of the navbar to match the size used on other pages. The new height should be consistent with the design guidelines provided or existing navbar dimensions on other pages.
Expected behavior
Follow the existing color scheme of the project.
Ensure the navbar's text and icons remain clearly visible and accessible against the new background color.
Ensure the navbar's size adjustment does not impact the usability or readability of the navigation items.
Screenshots
The navbar to be changed.

Navbar which is indifferent.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

please assign the issue


feat: use pyo3-asyncio to get a fresh tokio runtime

Description
This PR greatly reduces network connections and dns request volume by the delta-rs library when using Python bindings. The approach here is to utilize pyo3-asyncio's tokio-runtime feature as the source of the Runtime. This yields the same runtime across function calls which preserves connections in the connection pool. The previous code created a new runtime per python function call, which established all new socket connections and issued new DNS requests.
Related Issue(s)
Partly https://github.com/delta-io/delta-rs/pull/1315
Testing:
Ran a script that called hundreds of delta operations, and watched tcpdump. Only saw one dns request.

@PengLiVectra can you add the requested suggestion from Will?


Newbie Question - HTML encoded within XML

I want to pull some data out of HTML in the description fields of items in an RSS feed.
I can pull out the description OK, as per the example.  How do I then remove the ampersand encoding, and parse the result as HTML?  Where do I look for answers to such things?

You can use the unescape liquid filter and then send it to a WebsiteAgent for parsing.

I am closing this issue. Please feel free to reopen if you need further assistance!


Format CSS, JS

Fixes Issue

Fixes #1055
Fixes #1056

Changes proposed

Format CSS
Format JS

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

True, I got lots of emails saying it failed, Closing this. Thanks


Need to convert underscore to dash character for node name

fix a bug where getting the NodeInfo, not convert the underscore strings

LGTM.
Does not kubernetes support _ character ? How is it working so far ?

no one did '_' before, their testbed used to use "Dell-5810-2" like of name, now they started to be  creative.


a bunch of feedback on the beta

Here a bunch of smaller issues I discovered while testing
feature feedback

the "all commands" commands is a great idea, some other plugins would benefit from doing something like this
I am not sure what the usecase for "select previous block" would be, when you already have "select current block" and "expand to previous/next block"? If there is no usecase, you could maybe ditch those to reduce the number of commands?
I do not really like the implementation of the pull/push/copy commands. Selecting a note, selecting a beginning line, selecting an ending line... It feels a bit tedious, and I got the impression that working with copy/cut-paste and two panes would be quicker than doing it with this plugin. The "bookmarks" setting is a nice solution to speed up the selection of a note – maybe you could just use a value defined in the options (top of the file, bottom of the file, etc) to skip the selection of beginning/ending lines and so on? Like I press only one hotkey, select out of my 5 bookmarked notes, and the current line gets simply appended/prepended.
I get that bookmark+TOP/ BOTTOM does something similar, but I believe it feels a bit counterintuitive. And also means that until you use the plugin with some bookmarks, it would feel tedious. I dunno. Maybe just having a list of all lines is a bit too much, and having TOP/BOTTOM/Headers as options would reduce the tediousness a bit?

wording

"copy block as block as block reference" is phrased a bit weird.  I'd expect sth like "Copy embedded block reference"?
is there a reason it is called "select current line", but also "select next block"? Confuses me a bit, since I'd expect something like "next line" and "current block" as well then. (I never really understood the difference between lines and blocks in Obsidian, tbh.)
it only becomes clear upon testing that "copy block as block as block reference" and "Add block ref ID's to selection and Copy them to clipboard" do the same thing. Maybe it could also be phrased as "line/selection" as the other commands?
I think the "Alias Indicator" in the settings should rather be called "Alias Placeholder", if I am not misunderstanding the feature?
is it necessary to add ; to the end of every bookmark line, when not using TOP/BOTTOM/Heading? (add some info on that?)

Otherwise, as far as I can tell from testing this plugin a bit, I didn't encounter any bugs. (But I also could not test all features, since I do not use daily notes, for example.)

Awesome Awesome! Thank you so much for your help with testing and feedback. I agree, some aspects of the navigation feel tedious. I will do some experimenting on the flow. It is so hard to figure out the right balance of being fast, and having precision control.
will also improve the wording based on your feedback.

Just to mention, “ is it necessary to add ; to the end of every bookmark line, when not usingTOP/BOTTOM/Heading? (add some info on that?)”
you don’t need to use the semi-colon in this case. I will update the description.

I think I have tweaked some of these issues in an upcoming release. Others might wait while we do more experiments with the flow of the tool.


MWPW-146782: Add Description to Article richresults

Resolves: MWPW-146782
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/spandit/mwpw-146782-richresults?martech=off
After: https://MWPW-146782--milo--spandit22.hlx.live/drafts/spandit/mwpw-146782-richresults?martech=off
Check: Array.from(document.querySelectorAll("script[type='application/ld+json']")).map((e) => JSON.parse(e.innerText))[0]

According to https://github.com/adobecom/milo/pull/2505/checks?check_run_id=26618754935 we have to open and close this again. I don't see option to rerun check.


How to install on Ubuntu / Debian and other Linux disros?

I was looking for information about yazi on Ubuntu and I could not find any on https://yazi-rs.github.io/docs/installation/
I managed to install it using the  cargo install commands, but IMHO it would be nice to have some explicit information for the users of those (and maybe other) Linux distributions.

You should take a look at the packaging status page on the website to see if your distribution has packaged Yazi. You will find that most popular Linux distributions, like Ubuntu, Linux Mint and Debian, do not package Yazi.
Ubuntu and Debian both do not appear on the list, so you will have to either install Yazi via Rust's cargo package manager, or build it from source yourself.
There are no special installation instructions for these distributions because Yazi is just not installable using the distribution's package manager.


Please add preview for golang files as well

Provide a description of the new feature / enhancement
all the major programming languages have a preview on powertoys file explorer add on, but golang's .go files are not viewable like that . please add this feature as well.

Tracked via #1527


[Feature Request]: Add google translate feature

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to add google translate feature which translate the entire page with the given language.
Use Case
This would help the users to look out the page with their language.
Benefits
No response
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Not related issue


Implement lastb?

lastb is the same as last, except that by default it shows a log of the /var/log/btmp file, which contains all the bad login attempts.
man page

This is pretty useful for security audit, for example.
I'm willing to implement it. Though it might be a bit hard for a PAM module to record the final result after the PAM process..

The majority of applications do not support lastb, neither today nor in the past. Which means the current lastb output is pretty useless and not usable for a security audit.
For a security audit there is the audit framework.
And since there is no connection between wtmp and lastb, I did not implement it in wtmpdb and have no plans for this.
If somebody really needs lastb support, I suggest an independent implementation optimized for the lastb usecase.


Feature Enhancement: Allow users to manually restart, or manually start, bundled go daemon within the app (not only via tray icon)

Is your feature request related to a problem? Please describe.
go daemon startup is tightly coupled to app startup. If you start IPFS desktop and the node fails to start due to repo lock or other issue, users have no ability to get things working without a full app restart.
We're all familiar with this screen which is indicative that a full app restart, or separate daemon start (via CLI) is required:

Describe the solution you'd like
ipfs go daemon status should be toggle-able, restartable, stoppable, etc, from the ipfs desktop application
Describe alternatives you've considered
N/A
Additional context
Related: https://github.com/ipfs/ipfs-desktop/issues/1977

@juliaxbow restart is possible from tray menu, we should probably make this more accessible, if restarts are viable solution for daemon startup failure.

Did you want this to be available from just the "could not connect" screen? OR should users be able to access wherever they are in the app?
Assuming the latter based on your suggestion of "toggle-able" here are some options to include in the menu for initial feedback:
https://user-images.githubusercontent.com/58958327/195266734-d05cbfca-c60d-43bd-945f-bba9ec73c3d8.mov
The toggles would require addt'l help text to explain what's happening.
https://user-images.githubusercontent.com/58958327/195266743-28ed8675-05c9-42ab-89cc-51b4f05dd405.mov
https://user-images.githubusercontent.com/58958327/195267158-ff1899a7-c8da-48b3-b524-4db07c1e5502.mov

I feel like we should use the icon that's on the far right,

to toggle it instead of simply redirecting to the status page and opening the advanced view... open to other thoughts here though.


Incompatible CLI options for VS 2019 Debug build

🐛 Bug Report

When building in Visual Studio 2019 in Debug mode I get this error:

Only happens in Debug build; Release, MinSizeRel, RelWithDebInfo build fine
Expected Behavior
The project should build fine
Current Behavior
The project did not build :(
Possible Solution
There's this: https://stackoverflow.com/a/44416270
But having /O2 for a debug build seems incorrect. Also had an issue with CLion+Linux where Debug build skipped breakpoints, which can be proof of incorrect configuration.
Steps to Reproduce

git clone https://github.com/metacall/core.git
install VS 2019
mkdir build
cd build
cmake ..
Hit Local Windows Debugger here

Context (Environment)
I was trying to build the project

This needs to be documented. Basically.. MetaCall build system depends on this flag:
cmake -DCMAKE_BUILD_TYPE=Debug ..

The default option is Release. MSVC allows you to change after generation time the build type but metacall depends on the cmake option.. so each time you want to change the build type you need to set it up and re-generate the msvc files.

@viferga it makes sense. But this solution seems incorrect as an IDE user would expect the configurations to work fine out of the box. Apart from Visual Studio, I had an issue in Clion where breakpoints were skipped. Now that I added the flag you mentioned, it works fine for me. So if this hack is intended, we can close this issue and I can add this to the documnentation

I just discovered that since CMake 3.11, you can set up per configuration the options!
We should either check if this works: https://stackoverflow.com/a/32956785
Or this: https://stackoverflow.com/a/33831145
I think the second version may not work because generator expressions are handled at cmake generation time.
I'm leaving the issue open until it is solved.


add / edit SRV records

closes #1144

Coverage decreased (-0.004%) to 81.852% when pulling 2aa9a48c7b0b378b6b1105537ff831350deddfd0 on abemassry:dns-srv-record into f4c9117c25e8bcddb5103794e0c54be336b4b444 on linode:master.

Coverage increased (+0.2%) to 82.042% when pulling 1bda4b0c7c179a154d355b9c5cd772bcf27b91ae on abemassry:dns-srv-record into f4c9117c25e8bcddb5103794e0c54be336b4b444 on linode:master.

Coverage decreased (-0.04%) to 82.014% when pulling 5e3781f39abeb0568257c673fb584a7c6a2680be on abemassry:dns-srv-record into 4f1ff44e2e9b69d225f13f2318c542902048dbec on linode:master.

Coverage decreased (-0.04%) to 82.014% when pulling e58370bb456c48870ac0bed02189276939dc6531 on abemassry:dns-srv-record into 4f1ff44e2e9b69d225f13f2318c542902048dbec on linode:master.


✨ Add github workflow to update nix inputs

Closes #15. I'm not familiar with this, so feedback is welcome.

@ewen-lbh do you have an idea on how to test this?

Unfortunately CI is notoriously annoying/impossible to test locally 🥲
Apart from running the just script locally, there's not much you can do but commit and try ^^
Let me know when you tested the just recipe locally, i'll merge this in. If it doesn't work on the first try, that's fine

Unfortunately CI is notoriously annoying/impossible to test locally 🥲
Apart from running the just script locally, there's not much you can do but commit and try ^^
Let me know when you tested the just recipe locally, i'll merge this in. If it doesn't work on the first try, that's fine

The Justfile addition is fine.


Overlay covers the live chat

The overlay will need to be made movable and / or hideable

Fixed in release 0.0.4-alpha


[Doc] Update default roles in QSG

Description:
Under Add users and roles in QSG[1]
-> point 5 and 6, mention that creator, publisher roles exist by default.
-> point 8 and 12, update the screenshots
[1] https://docs.wso2.com/display/AM220/Quick+Start+Guide#QuickStartGuide-Creatingusersandroles
Suggested Labels:
Doc, improvement
Affected Doc Version:
2.2.0

Added a note including a screenshot of the roles available by default in the following pages.
[1] https://docs.wso2.com/display/AM2xx/Quick+Start+Guide
[2] https://docs.wso2.com/display/AM220/Quick+Start+Guide


can't access to local resources

hello, i'm trying to share a local input device from ubuntu pc to use it on remote pc through FreeRDP, i used thincast but the microphone is not running on client side.
i tried using remmina too but same (can't access local resources)
any idea would be appreciated..

@kerimdk ok, please try again, I don´t get what you´re trying to tell.
It might be helpful if you also add a description of what exactly you have tried already.

I have tried to connect to a windows pc remotely from my Ubuntu pc using both Remmina and Thincast.

On Remmina, I was unable to access any local resources such as (webcam-microphone) from my ubuntu pc on the remote PC side.
On Thincast, I was only able to access the webcam, but the microphone wasn't redirected. To resolve this issue, I added the line 'redirect-microphone:i:1' to the .rdp connection file. This allowed the remote  PC to see the microphone, but that was not enough to make it running correctly on the remote PC.

@kerimdk ok, so:

Remmina  and xfreerdp only support usb redirection, so your success with webcams depends on luck.

If you use usb redirection the whole USB device is redirected, so if your microphone is on that device it will not work.
The usb redirection only redirects one function of the usb device and windows filters some device classes (e.g. human input device, mass storage, audio devices and others) so your microphone will not work if it is on the same usb device

As for thincast client could you please be a bit more specific with not enough to make it running correctly? Hard to help if we do not know what exactly your problem is. (note: Both Remmina and Thincast Client have options to enable microphone redirection from the UI, take a look ;) Remmina in advanced tab/Redirect local microphone and Thincast Client in Local Resources Tab )

Thank you very much for your explanation. I got it very well now. So, it's not possible to use both the webcam and microphone on a common USB.
Yes, I know the options to enable microphone redirection from the user interface in Remmina and Thincast Client and they're awesome! but my problem is in running two different devices on a common USB, Could a USB hub solve the issue? Sorry if the question seems illogical. I have only three USB ports, and I am using them for the keyboard and the mouse, the remaining one is for either the webcam or the microphone.

@kerimdk yes, the problem with not having both on the same USB device is valid for xfreerdp, what was your issue with Thincast Client? (it uses [MS-RDPECAM] redirection which is not affected by these limitations

the issue is the Thincast Client is not available on armbian ubuntu22 system, i'm using orange thin client device, i tried the Thincast Client on my PC first but didn't found it on my thin client device then

when i run $ flatpak install flathub com.thincast.client
the output is
Looking for matches…
error: Nothing matches com.thincast.client in remote flathub

@kerimdk then your config is broken, see https://flathub.org/apps/details/com.thincast.client

Please help me find the Thincast Client on Armbian. I think I've tried everything in https://flathub.org/apps/details/com.thincast.client

@kerimdk there is no build for 32bit arm, only aarch64 . Install the 64bit os, then you have it.


Clarify supported Django versions

It would helpful to specify the versions of Django that we aim to support. There's some old compatibility code that could be removed.
Judging by the removal of south migrations, I assume it's >=1.7.
Tox configuration file specifies 1.7-1.9.
Proposed Travis configuration (#438) specifies 1.8-1.10.
Personally, I think we should aim for 1.8-1.10, since 1.7 upstream support has ended and 1.8 is an LTS.

yess thats the target.


[1.11.2] When right clicking a tank with lava in it, it does not yield a bucket of lava.

Hello,
When I right click a tank of lava with an empty bucket it removes the lava from the tank, but I don't get a bucket of lava.
This can be reproduced.
Mekanism: <IP_ADDRESS>

I've heard of this before. Something changed in the fluid integration of Forge.
I'll look back for the change.

Fixed!


Log number of ops that are not system ops

Fix for #9621 - log number of ops that are not system ops to help us understand how much user content is there.

            }

I could also take a different approach as vlad was worried about the performance of creating a temp array with map. An alternative could be :
let totalNonSysOps = 0;
snapshot.ops?.forEach((op) => {
if (!isSystemMessage(op)) {
totalNonSysOps++;
}
});
And use it when setting the event. Thoughts ?

Refers to: packages/drivers/odsp-driver/src/fetchSnapshot.ts:309 in 92d783f. [](commit_id = 92d783fa31ccf6a040ad3dea016bc975d5a5371e, deletion_comment = False)


Update Doc Build Process and Doc

The current build process is documented but not implemented anymore. This change adds a make target that uses Docker as an isolated environment for building the doc, and updates the doc accordingly.
Signed-off-by: Brett Logan<EMAIL_ADDRESS>

@Mergifyio rebase


fix docs (#1423)

for 1423.

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

Thanks @lu-xiaoshuang, I have no problem for fixing typos in code comments. As to the change in README, I think it's clear that AggregateFunction, the data type, is not supported.
Changing reviewer to Mark, who is the primary reviewer.

Thanks @lu-xiaoshuang, I have no problem for fixing typos in code comments. As to the change in README, I think it's clear that AggregateFunction, the data type, is not supported.
Changing reviewer to Mark, who is the primary reviewer.

Hi @zhicwu, we have already discussed whether clickhouse-java supports AggregateFunction in 1423.


NickAkhmetov/HMP-159 Upgrade vitessce-python dependency

This PR updates the vitessce-python dependency from 1.0.9 to 3.0.5; this was done as part of troubleshooting HMP-296. There are still failing tests etc, which will be updated accordingly once the core issue in 296 is resolved.

Current state:

All the assays on the portal which have vitessce hints display identical to prod
Potential issue: I can't get the obsLabelsType coordination that's supposed to take the place of the factors to work as expected right now. When the related code is uncommented and the duplicate obs sets are removed, the tooltips show up with blank values for all marker genes, regardless of whether the coordination value is prefixed with obs. Continuing to investigate this a bit further.

Without obs/:

With obs/:

Sample vitessce config with the marker genes issue

{
  "coordinationSpace": {
    "dataset": {
      "A": "A"
    },
    "embeddingType": {
      "A": "UMAP"
    },
    "obsLabelsType": {
      "A": "marker_gene_0",
      "B": "marker_gene_1",
      "C": "marker_gene_2",
      "D": "marker_gene_3",
      "E": "marker_gene_4"
    }
  },
  "datasets": [
    {
      "files": [
        {
          "fileType": "anndata.zarr",
          "options": {
            "featureLabels": {
              "path": "var/hugo_symbol"
            },
            "obsEmbedding": [
              {
                "dims": [
                  0,
                  1
                ],
                "embeddingType": "UMAP",
                "path": "obsm/X_umap"
              }
            ],
            "obsFeatureMatrix": {
              "initialFeatureFilterPath": "var/marker_genes_for_heatmap",
              "path": "X"
            },
            "obsSets": [
              {
                "name": "Leiden",
                "path": "obs/leiden"
              }
            ]
          },
          "url": "https://assets.hubmapconsortium.org/6c57274e7a40413dc042ec32442a228b/hubmap_ui/anndata-zarr/secondary_analysis.zarr"
        }
      ],
      "name": "6c57274e7a40413dc042ec32442a228b",
      "uid": "A"
    }
  ],
  "description": "",
  "initStrategy": "auto",
  "layout": [
    {
      "component": "scatterplot",
      "coordinationScopes": {
        "dataset": "A",
        "embeddingType": "A",
        "obsLabelsType": [
          "A",
          "B",
          "C",
          "D",
          "E"
        ]
      },
      "h": 6,
      "w": 9,
      "x": 0,
      "y": 0
    },
    {
      "component": "obsSets",
      "coordinationScopes": {
        "dataset": "A",
        "obsLabelsType": [
          "A",
          "B",
          "C",
          "D",
          "E"
        ]
      },
      "h": 3,
      "w": 3,
      "x": 9,
      "y": 0
    },
    {
      "component": "featureList",
      "coordinationScopes": {
        "dataset": "A",
        "obsLabelsType": [
          "A",
          "B",
          "C",
          "D",
          "E"
        ]
      },
      "h": 3,
      "w": 3,
      "x": 9,
      "y": 4
    },
    {
      "component": "obsSetFeatureValueDistribution",
      "coordinationScopes": {
        "dataset": "A",
        "obsLabelsType": [
          "A",
          "B",
          "C",
          "D",
          "E"
        ]
      },
      "h": 4,
      "w": 5,
      "x": 7,
      "y": 6
    },
    {
      "component": "heatmap",
      "coordinationScopes": {
        "dataset": "A",
        "obsLabelsType": [
          "A",
          "B",
          "C",
          "D",
          "E"
        ]
      },
      "h": 4,
      "w": 7,
      "x": 0,
      "y": 6
    }
  ],
  "name": "6c57274e7a40413dc042ec32442a228b",
  "version": "1.0.15"
}

I put together a local draft release of vitessce-python 3.0.7 using some of my pending PR's:

https://github.com/vitessce/vitessce-python/pull/272
https://github.com/vitessce/vitessce-python/pull/273
https://github.com/vitessce/vitessce-python/pull/274

I verified these changes work as expected locally; CI fails for this branch because 3.0.7 is not yet a released Vitessce version. Once those three PR's are merged and a new release is made, I will update the test fixtures again and open this PR for review.
I've also preemptively pushed a portal-visualization package version bump as part of this PR given the change in dependency version.

Vitessce 3.0.7 is released
Fixtures are updated
Tests pass
Smoke test of portal visualizations loaded as expected

Ergo, I'm opening this for review.

After resolving John's comments above and getting verbal approval from Mark via slack, I'm merging these changes in 🙌


Add style for last observed valid observation

Hi Vicky! Thanks for working on this. I have a few minor suggestions:

I wonder whether the black outline is the best choice? When fully zoomed out, the whole continent appears in black, which is quite different from the default extent styling. My suggestions would be to try either no lines at all, or lines that are a slightly darker, more saturated version of the colour of each category. I don't know what this will look like zoomed out, but it might be worth experimenting. If you try other options and the black outline still looks best (both zoomed out and zoomed in) then go with that.
In the category labels, you use "x" as the variable, but I think that might be confusing to the user. Perhaps something like "% wet" or "wet percentage" might be more clear?

Otherwise, I think it looks good!

Hi @caitlinadams, thanks for reviewing the style. For the polygons which have no valid observation , which includes very large waterbodies, should we mask them out when styling by area or leave them as grey.


NLS Enablement of Core Node Categories

Includes NLS enablement of core node categories. NLS enablement of user node categories is still in progress.

I've manually cherry picked the relevant commit from your repository and force-pushed the rebased version of the code. Which basically means, this PR is no longer needed, and you'll need to refresh your fork to the latest code. The quickest way is to delete your fork and create a new one.


Note in stack view should show more (... or complete?) content

As a user I want to see the complete (... or more of the) text of a note in the detail/stack view, if theres enough space available on my screen.
Alternatives how we could address this:

Maybe add expande/collapse toggle (to all notes)?

Show expanded state if theres only one note
Show collapsed note if there are multiple children in the stack

Increase space for parent note?
Show complete text and include parent note into the scrollable area?
Redesign the screen and put parent note on one side and the children on the other side?

New Design Stack view:
https://xd.adobe.com/view/1bdd7208-4f56-462f-8bd3-cbf3f36249f3-4e8e/

improved design:
https://xd.adobe.com/view/9c39ca3a-b4a6-4dfe-ba4e-e0d075a9acc6-c96a/

Hey @brandstetterm can you check whether you closed this issue on purpose or by accident? I did not find another issue related to this topic

Yes, I think that was by mistake because I confused it with another issue @bitionaire

I put this back into Todo because of time issues 🥲


evidences: split tendermint dependency

I wanted to run the storetestcases in Alice (on the Alice store wrapper) and it ended up importing almost all of Tendermint inside Alice, which is really not desirable.
I think it's good to create a clear boundary with everything Tendermint-specific in go-indigocore, to allow many pieces to be used outside without pulling any of the Tendermint dependencies.
It's a bit painful to have to remember to do a blank import on all the evidence packages in the cmds that use them, but I think that it's cleaner as a dependency management point of view (only import these in cmds' main.go file).

This change is

At some point we'll really need to have either cmd tests somehow or system tests!

Reviewed 26 of 27 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

cs/evidences/evidences.go, line 101 at r1 (raw file):

		return &p, nil
	}
	cs.DeserializeMethods[BcBatchFossilizerName] = func(rawProof json.RawMessage) (cs.Proof, error) {

For consistency wouldn't that make sense to move those initialization in their fossilizer package as well?

Comments from Reviewable

I agree, these would be useful!

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

cs/evidences/evidences.go, line 101 at r1 (raw file):
Previously, such (Adrien Montfort) wrote…
For consistency wouldn't that make sense to move those initialization in their fossilizer package as well?

I think it would make sense indeed. I'll do it in this PR to keep the change contained.

Comments from Reviewable

Reviewed 26 of 27 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

Reviewed 26 of 27 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

Reviewed 28 of 29 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 28 of 29 files at r2.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


New theme: Whiteplain

Hello Admins,
I made new Hugo theme "Whiteplain".
Could you please add this repository?
https://github.com/taikii/whiteplain
Thank you, Merry Xmas!

Hello @taikii
thank you very much for sharing this theme with the Hugo community. Your theme should be listed soon on the themes site.
Furthermore, I'll promote your theme on Hugo's official Twitter account.
Regards,
Digitalcraftsman

@digitalcraftsman Thank you so much!


[New Feature] Get-DbaRepPublication

Complete code and Pester Tests for Get-DbaRepPublication

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[X] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[X] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[X] Pester test is included
[ ] Nunit test is included
[X] Documentation
[ ] Build system

Purpose
Submitting new feature Get-DbaRepPublication #3891

I've made the suggested changes. Let me know if there is anything else to be done! Thanks!

super work! thank you, will be available in the PowerShell Gallery shortly 💃


Sample crashes with Flow serialization issue

I’m running the emoji search sample and it’s bitrotted.
2022-08-15 12:46:38.358 6126-6151/com.example.zipline.emojisearch.debug E/Zipline: java.lang.Exception: SerializationException: Class 'ChannelFlowBuilder' is not registered for polymorphic serialization in the scope of 'Flow'.
    Mark the base class as 'sealed' or register the serializer explicitly.
        at captureStack (../../../../../runtime/coreRuntime.kt:86)
        at SerializationException_init_$Create$_0 (kotlinx-serialization-kotlinx-serialization-core-js-ir.js)
        at throwSubtypeNotRegistered_0 (../../../../../Z:/BuildAgent/work/b2fef8360e1bcf3d/core/commonMain/src/kotlinx/serialization/internal/AbstractPolymorphicSerializer.kt:102)
        at throwSubtypeNotRegistered (../../../../../Z:/BuildAgent/work/b2fef8360e1bcf3d/core/commonMain/src/kotlinx/serialization/internal/AbstractPolymorphicSerializer.kt:113)
        at findPolymorphicSerializer (../../../../../Z:/BuildAgent/work/b2fef8360e1bcf3d/core/commonMain/src/kotlinx/serialization/PolymorphicSerializer.kt:109)
        at <anonymous> (../../../../../Z:/BuildAgent/work/b2fef8360e1bcf3d/core/commonMain/src/kotlinx/serialization/internal/AbstractPolymorphicSerializer.kt:32)
        at <anonymous> (../../../../../Polymorphic.kt:89)
        at <anonymous> (../../../../../Z:/BuildAgent/work/b2fef8360e1bcf3d/core/commonMain/src/kotlinx/serialization/encoding/AbstractEncoder.kt:80)
        at <anonymous> (../../../../../commonMainSources/libraries/stdlib/src/kotlin/util/Preconditions.kt:287)
        at <anonymous> (zipline-root-zipline.js)
        at <anonymous> (../../../../../Polymorphic.kt:89)
        at <anonymous> (../../../../../Z:/BuildAgent/work/b2fef8360e1bcf3d/core/commonMain/src/kotlinx/serialization/encoding/AbstractEncoder.kt:80)
        at <anonymous> (../../../../../Z:/BuildAgent/work/b2fef8360e1bcf3d/core/commonMain/src/kotlinx/serialization/encoding/Encoding.kt:287)
        at <anonymous> (zipline-root-zipline.js)
        at <anonymous> (../../../../../Polymorphic.kt:89)
        at encodeToDynamic_0 (../../../../../DynamicEncoders.kt:42)
        at encodeToDynamic (../../../../../Dynamics.kt)
        at encodeToStringFast (../../../../../jsonJs.kt)
        at <anonymous> (../../../../../CallCodec.kt:56)
        at <anonymous> (../../../../../OutboundCallHandler.kt:52)
        at <anonymous> (../../../../../SuspendCallback.kt)
        at <anonymous> (zipline-root-zipline.js)
        at <anonymous> (../../../../../InboundService.kt:713)
        at <anonymous> (../../../../../InboundService.kt)
        at l (zipline-root-zipline.js)
        at <anonymous> (../../../../../src/kotlin/coroutines_13/IntrinsicsJs.kt:163)
        at <anonymous> (../../../../../src/kotlin/coroutines_13/CoroutineImpl.kt:55)
        at <anonymous> (kotlin-kotlin-stdlib-js-ir.js)
        at <anonymous> (../../../../../DispatchedTask.kt:10131)
        at <anonymous> (../../../../../commonMainSources/libraries/stdlib/src/kotlin/util/Standard.kt:153)
        at <anonymous> (../../../../../JSDispatcher.kt:19)
        at apply (native)
        at <anonymous> (../../../../../GlobalBridge.kt:70)
        at <anonymous> (../../../../../platform.kt)
        at <anonymous> (zipline-root-zipline.js)
        at <anonymous> (../../../../../InboundService.kt:835)
        at <anonymous> (../../../../../Endpoint.kt:79)
        at <anonymous> (zipline-root-zipline.js)
        at <anonymous> (../../../../../GlobalBridge.kt)
        at <anonymous> (zipline-root-zipline.js)
        at app.cash.zipline.internal.JsPlatform$Companion$Adapter$GeneratedOutboundService.runJob(platform.kt:37)
        at app.cash.zipline.internal.CoroutineEventLoop$DelayedJob$run$1.invokeSuspend(CoroutineEventLoop.kt:59)
        at app.cash.zipline.internal.CoroutineEventLoop$DelayedJob$run$1.invoke(Unknown Source:8)
        at app.cash.zipline.internal.CoroutineEventLoop$DelayedJob$run$1.invoke(Unknown Source:4)
        at kotlinx.coroutines.intrinsics.UndispatchedKt.startCoroutineUndispatched(Undispatched.kt:55)
        at kotlinx.coroutines.BuildersKt__Builders_commonKt.startCoroutineImpl(Builders.common.kt:198)
        at kotlinx.coroutines.BuildersKt.startCoroutineImpl(Unknown Source:1)
        at kotlinx.coroutines.AbstractCoroutine.start(AbstractCoroutine.kt:134)
        at kotlinx.coroutines.BuildersKt__Builders_commonKt.launch(Builders.common.kt:56)
        at kotlinx.coroutines.BuildersKt.launch(Unknown Source:1)
2022-08-15 12:46:38.358 6126-6151/com.example.zipline.emojisearch.debug E/Zipline:     at kotlinx.coroutines.BuildersKt__Builders_commonKt.launch$default(Builders.common.kt:47)
        at kotlinx.coroutines.BuildersKt.launch$default(Unknown Source:1)
        at app.cash.zipline.internal.CoroutineEventLoop$DelayedJob.run(CoroutineEventLoop.kt:57)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:920)

Once fixed, if you write a quick Espresso test I'll set up the CI for it.

I think the core issue is we have special serialization support for Flow, and we have other special serialization support for the result of a suspend fun, but we don’t have anything for the combination of these features.
The frustrating part here is that Kotlin serialization doesn’t compose the way I want it to for type parameters. In particular I can’t find a way to mark the type parameter as contextual. I can probably find a work around but ugh.

Hmm. There should (almost) never be a reason to have a suspend function that returns a Flow as the async operation can be moved into the flow itself to occur upon collection.
Should we just ban it?

Good idea on the immediate problem. I think I’m still in a bad place with the way the serialization of types composes.

Fix will be the same as https://github.com/cashapp/zipline/issues/602


Simplify commands using aliases

@mikecroucher noted that having to repeatedly specify ports and IP addresses gets tiring and is error-prone.  Would be useful to update the Docker image to include aliases to simplify commands where possible.
Usage of aliases/wrapper scripts would need documenting in the slides and/or student setup guide (#12)

Done


[varLib.cff] Suboptimal packing of deltas, or flaw in CFF2?

With a CFF2 VF built from the attached designspace project (modified from MutatorSans), the S rendered at wdth=1000, wght=595 has a bump on the right hand side, even though its corresponding source does not:

The bump is at the start/end point of the contour.
The design space has five sources, one on each corner, and a support master between Light Wide and Bold Wide.
Light    Medium   Bold
Wide     Wide     Wide
o--------o--------o
|                 |
|                 |
|                 |
|                 |
|                 |
|                 |
O-----------------o
Light             Bold
Condensed         Condensed
(neutral)

The Medium Wide source is at the location shown (wdth=1000, wght=595)
What I think happens is:

The delta for that location is relative to the interpolation between Light Wide and Bold Wide
This interpolation has fractional coordinates
The delta's are calculated for relative coordinates (the differences between one point and the next), rounded to integers
Leading to an accumulation of rounding errors
To the extent that the final point is no longer coinciding with the starting point.
(The difference between start and end point is ≈3 units horizontally, and ≈1.5 units vertically.)

On the one hand, in theory the varLib CFF2 packer could do a better job, but looking at the code this seems far from trivial. On the other hand this shows that CFF2 has pretty terrible properties for cases like this.
mutator_sans_S.zip

Btw. props to @behdad for the wonderful main prog in svgPathPen.py:
python -m fontTools.pens.svgPathPen MutatorSans-VF.otf S --variations "wght=595 wdth=1000" > letter_S.svg

This fixes it:
diff --git a/Lib/fontTools/varLib/cff.py b/Lib/fontTools/varLib/cff.py
index b2fb51f1..2dac5123 100644
--- a/Lib/fontTools/varLib/cff.py
+++ b/Lib/fontTools/varLib/cff.py
@@ -502,7 +502,7 @@ class CFF2CharStringMergePen(T2CharStringPen):
        """
        def __init__(
                                self, default_commands, glyphName, num_masters, master_idx,
-                               roundTolerance=0.5):
+                               roundTolerance=0.1):
                super().__init__(
                                                        width=None,
                                                        glyphSet=None, CFF2=True,

The CFF2 table size goes from 624 bytes to 858 bytes with this change.
(glyf + gvar don't have this same problem despite not supporting floating-point numbers.)

A smarter encoder can work around this by accumulating the error while rounding. That's hard to implement. Let me think about it.

That would be very cool, but I'm currently in a "CFF2 is bad, please give me cubics in glyf" kind of mood :)

One thing I realized only when rewriting the CFF2 spec is that, if a Postscript-style outline ends with a curve, the start point is effectively specified twice. With statics this doesn’t matter but with variable, if the start point is moved, the “extra start point” needs to be moved too!  I wonder if the CFF2 compiler takes this into account.

if the start point is moved, the “extra start point” needs to be moved too

It does move, but because of the accumulated error over all relative coordinates + deltas, the end point doesn't always move the exact same distance as the start point. Which is what we're seeing here.

@justvanrossum what do you suggest we do here? I suggest we make the change, ie. prefer correctness over size.

Maybe yeah. I'm not having a good view of possible unintended side-effects, though.

possible unintended side-effects

Is only size.

Non-integer CFF was tested in the field in 2015:
https://blog.typekit.com/2015/10/01/smoother-curves-with-decimal-coordinate-values/

Cool, let's go for it then!

I just fixed a bug with fractional CFF coordinates in LibreOffice subsetter the other day, which suggests they work fine elsewhere.


Add support for curve ed25519

This adds support for ed25519 curve.

Need to bump version? @himanshuchawla009

Need to bump version? @himanshuchawla009

i will bump it during release, it doesnt seems to have any breaking versions right, all apis and response types are backward compatible?

Need to bump version? @himanshuchawla009

i will bump it during release, it doesnt seems to have any breaking versions right, all apis and response types are backward compatible?

Technically, there are some breaking changes:

The client now expects a key_type parameter upon initialization.
The client will now send a key_type parameter with every request to the server.

Need to bump version? @himanshuchawla009

i will bump it during release, it doesnt seems to have any breaking versions right, all apis and response types are backward compatible?

Technically, there are some breaking changes:
1. The client now expects a `keyType` parameter upon initialization.

2. The client will now send a `key_type` parameter with every request to the server.

(1.) could be made non-breaking by using secp256k1 as default if not keyType is provided. (2.) may be breaking or may not be depending on how the server handles unknown parameters: does it throw? or just ignore?

Just tested, server doesn't throws currently on passing unknown params.

So does this PR look good? Or you want me to change anything? @himanshuchawla009

@himanshuchawla009, have created alpha branch and targeted that one here. Please merge and publish when you are ready. Also, could you please set alpha branch as default?


might want to add a TL;DR to after 'git branch testing' 'git checkout testing'

I totally should have known better, as I've used git for a while, but had been using svn for a different project for a few weeks.  I simply went to the 'Git-Branching-Branches-in-a-Nutshell' page to verify I remembered the create branch syntax, saw that I remembered it properly, created the branch, checked it out and made changes.
Unfortunately, I checkout my 'master' branch, forgetting that without a commit being done on my new branch, 'master' and my new branch are pointing at the same revision, and the changes got merged into 'master'.

Still can't believe I did this. :(

not sure on how to proceed with this @peff ...

This is a suggestion for book content, so I think if anything is to be done, it would have to be via an issue or PR in https://github.com/progit/progit2 (the suggestion overall seems reasonable to me, though).


How to programmatically unfocus node(s)?

Is there a way to programmatically unfocus a node without resorting to setting a bogus one? This obviously isn't the right way to go about it, since I can't get rid of the console log (which definitely is not desired).

It also seems that I'm having to "unfocus" when deselecting nodes using tree.deselectAll(). Since there isn't an unfocus method, I'm having to use a bogus value which then outputs in the console.
Could we

Remove the console log
add an unfocus method

Yes, we can certainly remove that console.log.
And yes, we can add a .blur() method on the tree. I'm not sure whether it should return focus to the tree container, or to document.body or something else?
Can you explain your use case? What is it your trying to do by removing focus?

You can also remove focus by simply focusing on a different element.
Is it a styling concern? If you are styling the nodes with .isFocused, you can change the css selector to use :focus and that will only be applied when the tree has focus.

I removed that console.log in this PR: https://github.com/brimdata/react-arborist/pull/79

I apologize for the delay in getting back to you. I have been busy, but I will respond with more details shortly.


Hapi / Handlebars: languageHeaderField: 'accept-language' not working

I have this plugin registered with 'accept-language'. The README suggests language but that does not detect the browser's language.
{
    plugin: require('hapi-i18n'),
    options: {
        locales: ['en', 'es'],
        directory: __dirname + '/locales',
        languageHeaderField: 'accept-language'
    }
}

How can I get my app to automatically use the right locale depending on the user's language set in their browser?

Browser language was not supported, yet. I have opened PR #23 to add it. Can you please check if the updates work on your end? I will merge the changes into master and publish a new release then.

PR is merged and new version 2.1.0 published to npm.


question 3 layout in place

Smashed through a basic layout for question 3.
https://trello.com/c/DSG4HWLF/210-new-designs-in-question-3-content
This means that the cloud, the heading, and the content are roughly in place and can possibly be used as the template for the other pages that feature these same elements. I think it would be better to arrange this through another layout composition, but also time is against me and I have to move on to other things today.
Also, struggling to work out how to change the Next button to the right background colour.
Still, I think all in all, unless there are big things to change here, this one can go through and help out with people taking the future pages that have the same configuration. We can fine tune later.

@batteries76 I can't figure out why, but there's always a scrollbar, even when the content fits comfortably in the viewable height.

@ymloh , I can see the same issue with the scrollbar too; it's always there!
it seems to be this line of code on L16 on components/Layout/Layout.jsx that's causing the scrollbar to stick around.

2 areas need changes. The cloud missing features. Buttons need to meet the figma design. Button colour is fixed on question 4. Can get from there.

@Amutha37 , can you be more specific about what feature is missing from the cloud in question 3? I don't see any issues with the cloud icon. I've attached a screenshot of Question 3 so we can confirm that you and I are seeing the same thing:

@batteries76 ,
I there is an issue with the page, where if you:

have provided a number value in the text box, and click 'Next' to proceed to 'Question 4'
in 'Question 4', you click 'Back' to go back to 'Question 3'
in 'Question 3', your previously entered value is no longer in the text field (i.e. text field is empty), but you are allowed to proceed to 'Question 4' nonetheless. (see screenshot)

This can be confusing. Suggestions:

Either disable 'Next' button until a value is supplied in text field, or
(preferred) retrieve previously entered value, and have value appear in text field

Yes, will have a look into it. Almost tempted to make it a bug card and get the design part through just to have the template for the other pages that people can pull down. But I think I'll work to get that done and push it up.

Looking Ace @batteries76 ! :v:
I left a couple of nitpicks, and a suggestion for improvement;
but happy to approve it as-is :+1:


Curl script https not supported or disabled in libcurl

Attempting to use a curl script to upload a file to a remote location. It caused this error curl: (1) Protocol "https" not supported or disabled in libcurl. Using http is not an option. I assume that libcurl is somehow included with the agent when its installed. Can a version that supports https be included?

The agent doesn't distribute libcurl - it's written in javascript.  Issue is not with agent ...


Running multiple tests in one batch for better time-sharing between async tests

Hi, I remember nextest spawns child processes to run test cases one at a time (haven't checked the latest code, correct me if wrong). The performance overhead could be small for normal tests, but for async tests (such as #[tokio::test]), multiple cases cannot parallelize in this runner model.
I wonder if it's possible to run multiple test cases within one process? For example fetching small batches of cases from the job queue to execute.

Hi,
This is #27 and as listed there is a very large endeavor, especially to make work with all of nextest's features. However, in general Tokio is super quick to start up so I'm surprised this is a major issue.

Going to close this as a dup of #27, thanks for the report!

Oh, good to know. To clarify though, the issue is not tokio startup time. It's the parallelism between multiple test cases. When running async tests with cargo-test, multiple tests can run at the same time even if user specifies job=1.

Huh, I'd never heard of that. I'd consider that a bug to be honest.

Oh is this because -j1 controls build jobs for cargo test and test jobs for nextest?

Ach, looks like it. Used the wrong command when testing this before..


open previous window from command line?

Opening Atom without the command line (e.g. using the dock on OSX) now re-opens any previously opened windows

Would it be hard for me to do a PR to add a command-line option to open atom with the previous windows?  I know my use-case is rare but I open X11 windows from the command-line and I have no desktop at all.
I know this isn't going to happen but logically using atom would open previous windows and atom . would open the working directory.

Had a quick discussion with @maxbrunsfeld with this a couple of days back but didn't really have time to go about doing it.  This would be great :smile:.

This would be great

I assume you mean adding a command-line option, not changing what it does
with no options.
If the code is in javascript I'll look into the PR.  Is it?
On Tue, Apr 21, 2015 at 6:57 PM, Wliu<EMAIL_ADDRESS>wrote:

Had a quick discussion with @maxbrunsfeld
https://github.com/maxbrunsfeld with this a couple of days back but
didn't really have time to go about doing it. This would be great [image:
:smile:].
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/6464#issuecomment-94996845.

The Atom script is a shell script: https://github.com/atom/atom/master/atom.sh.
And yes, a command-line option.

That link is a 404 and the only atom.sh I could find was
https://github.com/atom/atom/blob/aad393618ea68c4992c7627fda1f624c31d2d8aa/resources/win/atom.sh
which is windows only.
I only care about linux myself.
On Tue, Apr 21, 2015 at 7:08 PM, Wliu<EMAIL_ADDRESS>wrote:

The Atom script is a shell script:
https://github.com/atom/atom/master/atom.sh.
And yes, a command-line option.
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/6464#issuecomment-94998604.

Ugh why did I try reformatting the URL...
https://github.com/atom/atom/blob/master/atom.sh

That looks like an easy PR, even for me.  Does anyone have a preference on
the option letter?  Unfortunately w is taken.  Maybe r for restart?
On Tue, Apr 21, 2015 at 7:13 PM, Mark Hahn<EMAIL_ADDRESS>wrote:

That link is a 404 and the only atom.sh I could find was
https://github.com/atom/atom/blob/aad393618ea68c4992c7627fda1f624c31d2d8aa/resources/win/atom.sh
which is windows only.
I only care about linux myself.
On Tue, Apr 21, 2015 at 7:08 PM, Wliu<EMAIL_ADDRESS>wrote:

The Atom script is a shell script:
https://github.com/atom/atom/master/atom.sh.
And yes, a command-line option.
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/6464#issuecomment-94998604.

-r is also taken for --resource-path, but I can't think of a better flag than --restore

I really want it to be quick and easy.    How about -o or --old-windows?

I know this isn't going to happen but logically using atom would open previous windows and atom . would open the working directory.

But that makes so much sense. :100:

As for an explicit flag perhaps -R for a shortcut to --restore

I still like -o for --old-windows.  It is a good complement to -n for --new-window.   Here it is in context...
Options:
  -1, --one             Run in 1.0 API preview mode.                                                         [boolean]
  -d, --dev             Run in development mode.                                                             [boolean]
  -f, --foreground      Keep the browser process in the foreground.                                          [boolean]
  -h, --help            Print this usage message.                                                            [boolean]
  -l, --log-file        Log all output to file.                                                              [string]
  -n, --new-window      Open a new window.                                                                   [boolean]
  -o, --old-windows     Open last-closed windows.                                                                   [boolean]
  -r, --resource-path   Set the path to the Atom source directory and enable dev-mode.                       [string]
  -s, --spec-directory  Set the directory from which to run package specs (default: Atom's spec directory).  [string]
  --safe                Do not load packages from ~/.atom/packages or ~/.atom/dev/packages.                  [boolean]
  -t, --test            Run the specified specs and exit with error code on failures.                        [boolean]
  -v, --version         Print the version.                                                                   [boolean]
  -w, --wait            Wait for window to be closed before returning.                                       [boolean]

Based on my experience looking at #6466, I think you can achieve the behavior you desire by running /usr/share/atom/atom instead of /usr/bin/atom.

That would be great.  Unfortunately  /usr/share/atom/atom is giving me this error ...
$ /usr/share/atom/atom
[28516:0422/125255:ERROR:nss_util.cc(94)] Failed to create /home/mark/.pki/nssdb directory.
[28516:0422/125255:ERROR:browser_main_loop.cc(170)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
[28516:0422/125255:ERROR:browser_main_loop.cc(216)] Gtk: cannot open display:

$ [28518:0422/125255:ERROR:nss_util.cc(94)] Failed to create /home/mark/.pki/nssdb directory.
[28518:0100/000000:ERROR:zygote_linux.cc(585)] write: Broken pipe

What is really weird is that something is causing the bash command line to show some of the error in a second prompt.  I didn't know this was even possible.
Is it possible that script only works when there is a desktop?

Is it possible that script only works when there is a desktop?

Yeah, that's possible. In any case, that binary is what the atom command uses, so if you use atom from the command-line, you should be able to use /usr/share/atom/atom.

if you use atom from the command-line, you should be able to use
/usr/share/atom/atom.

atom works great from the command line.   /usr/share/atom/atom doesn't.
On Wed, Apr 22, 2015 at 1:22 PM, Max Brunsfeld<EMAIL_ADDRESS>wrote:

Is it possible that script only works when there is a desktop?
Yeah, that's possible. In any case, that binary is what the atom command
uses https://github.com/atom/atom/blob/master/atom.sh#L77, so if you
use atom from the command-line, you should be able to use
/usr/share/atom/atom.
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/6464#issuecomment-95324792.

Looks like your DISPLAY environment variable isn't set in your shell, but is set when running /usr/bin/atom.  Do either of these work:
bash -c /usr/share/atom/atom

or:
DISPLAY=:0 /usr/share/atom/atom

Unfortunately they are giving the same error.
What do you make of the Failed to create /home/mark/.pki/nssdb directory.
error?  That is apparently inside chromium.
I don't see how this could be the same script run by the Atom command.
On Wed, Apr 22, 2015 at 2:27 PM, Dean Brettle<EMAIL_ADDRESS>wrote:

Looks like your DISPLAY environment variable isn't set in your shell, but
is set when running /usr/bin/atom. Do either of these work:
bash -c /usr/share/atom/atom
or:
DISPLAY=:0 /usr/share/atom/atom
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/6464#issuecomment-95342631.

Ah, that is the gnome-keyring.  I've had trouble with that before.  Now I just need to remember how I fixed it then.  There was some variable that overrode it.

I'm not sure what the nssdb error is but I got the same error when I tried running /usr/share/atom/atom without the DISPLAY env var set. With it set, both the gtk error and the nssdb error went away and the atom window opened. Please post the output from:
echo "DISPLAY=$DISPLAY"
DISPLAY=:0 /usr/share/atom/atom

You can also try this:

Copy the atom script locally with cp /usr/bin/atom . and edit ./atom as follows:

Change the line which says:
 USR_DIRECTORY=$(readlink -f $(dirname $SCRIPT)/..)

to say:
 USR_DIRECTORY=/usr

Change the line which says:
 nohup "$ATOM_PATH" --executed-from="$(pwd)" --pid=$$ "$@" > "$ATOM_HOME/nohup.out" 2>&1

to say:
 nohup "$ATOM_PATH" --pid=$$ "$@" > "$ATOM_HOME/nohup.out" 2>&1

Run your modified copy with ./atom.

First, the results of ...
echo "DISPLAY=$DISPLAY"
DISPLAY=:0 /usr/share/atom/atom

$  echo "DISPLAY=$DISPLAY"
DISPLAY=

$  DISPLAY=:0 /usr/share/atom/atom
[18702:0422/223935:ERROR:nss_util.cc(94)] Failed to create /home/mark/.pki/nssdb directory.
[18702:0422/223935:ERROR:browser_main_loop.cc(170)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
[18702:0422/223935:ERROR:browser_main_loop.cc(216)] Gtk: cannot open display: :0

$  [18704:0422/223935:ERROR:nss_util.cc(94)] Failed to create /home/mark/.pki/nssdb directory.
[18704:0100/000000:ERROR:zygote_linux.cc(585)] write: Broken pipe

I made the two mods and ran the local copy.  I got pretty much the same results ...
[20116:0422/224711:ERROR:nss_util.cc(94)] Failed to create /home/mark/.pki/nssdb directory.
[20116:0422/224711:ERROR:browser_main_loop.cc(170)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
[20116:0422/224711:ERROR:browser_main_loop.cc(216)] Gtk: cannot open display:

Thanks for all the trouble.  I would be happy to let you ssh into my machine, if you want.

I suspect you have a more general X-Windows configuration problem since it looks like you can't open display :0. The only thing I don't understand is why running "atom" works. Perhaps it is an alias or isn't actually /usr/bin/atom? What do you get for:
which atom

and what happens when you run:
/usr/bin/atom

? Can you confirm that you can run other X programs from shell where you are trying to launch atom?

$  which atom
/usr/bin/atom
$  atom
<Atom opens up in an X window>

When I enter atom the terminal emits about 100 lines of garbage but the window opens.

Can you confirm that you can run other X programs from shell where you are trying to launch atom?

Yes, I also have run xterm, SmartGit and the unity desktop.  I have been using Atom and SmartGit in X windows for almost all of my development for weeks.

I suspect you have a more general X-Windows configuration problem since it looks like you can't open display :0.

As I mentioned, I suspect some keyring problem, at least with the first error message.  In the past I had to add some env var to fix the keyring. Maybe one script can see the var and another cannot.   Unfortunately I can't find the old forum posts for that fix.  I'll keep looking.

What Linux distribution/release are you running?
Is SELinux enabled? If so, you could try turning it off temporarily to see if it's the cause. Use sudo setenforce 0 to turn it off, and then see whether /usr/share/atom/atom works. Don't forget to turn it back on with sudo setenforce 1. You might also want to check/fix the permissions on /home/mark/.pki as described here.
FWIW, I'm mystified as to how you are able to run any X programs without having the DISPLAY environment variable set.

What Linux distribution/release are you running?

I'm using ubuntu 14.0.4 inside a chroot using crouton on a chromebook pixel.  You can run different chroots.  I used to use unity desktop but I switched to a stripped chroot that only has xiwi.

Is SELinux enabled?

It is not installed.

check/fix the permissions on /home/mark/.pki as described here.

I changed the owner/group and that fixed my first error.  Progress!
I'm left with ...
$  DISPLAY=:0 /usr/share/atom/atom
[9426:0423/080408:ERROR:browser_main_loop.cc(170)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
[9426:0423/080408:ERROR:browser_main_loop.cc(216)] Gtk: cannot open display: :0

$ [9428:0100/000000:ERROR:zygote_linux.cc(585)] write: Broken pipe

Damn, I am so sorry to have wasted your time.
I had forgotten that in the beginning I made an alias for atom that did xiwi atom.  xiwi /usr/share/atom/atom loads up just fine.  Without xiwi no X env was being loaded at all.
Now I'll try re-opening the multiple windows.

After 24 hrs of wasting everyone's time I can verify that /usr/share/atom/atom loads the old windows just fine.  So there is no need for a command-line option.  This issue is resolved.
Thank you @brettle.  Without your persistence I would never have figured this out.

After 24 hrs of wasting everyone's time I can verify that /usr/share/atom/atom loads the old windows just fine. So there is no need for a command-line option. This issue is resolved.

Maybe, but I think a command-line option would still prove very useful and would be a lot more useable than remembering to launch from /usr/share/atom/atom every time you want to restore your last session from the command line.

You may be right, but I personally have no problem since I'll be using an alias to a script that makes the atom command behave as I want.  atom with no args will re-open windows.  atom . will open the current directory.

Yay! Glad you figured it out.
@mnquintana, I agree that a command line option would be useful.

@mnquintana A command line option would be strange to me the more I think about it. atom should work like subl did (open previous windows). atom . can open up the current directory.

@mehcode Ah, I forgot about that option – yeah, I like that one a lot better.

I know this isn't going to happen but logically using atom would open previous windows and atom . would open the working directory.

@mark-hahn @mehcode I think this should happen.
/cc @atom/non-github-maintainers @atom/core

@mark-hahn @mehcode I think this should happen.

Definitely :+1: from me

I think this should happen.

As I said, a simple script will do it.    Adding the script is easy.


RuntimeError: Failed to parse docstring of query function as LMQL code:

Platform: OSX
Model: AzureModel Codex (loads successfully no issues).
Interestingly this only happens when a python interpreter is used to execute the functions separately but not if it is run as part of a script. Hence it is not a major blocker but it is just something some people might run into.
Query I would like to define (but same error gets thrown at python example from documentation.)
@lmql.query(model=codex_model, decoder='argmax', top_p=1, temperature=0.0)
def query_code_single_line(varname, df, question='Count occurences of distinct input_source in df.'):
'''lmql
"""
#!/usr/bin/python3
import pandas as pd
Write a single statement of code using pandas to answer the following: {question}
df = {df}
The output should be a new dataframe containing the result.
Code : [CODE]
"""
'''
Error:
Traceback (most recent call last):
File "", line 2, in
File "/Applications/anaconda3/envs/llm_otd/lib/python3.10/site-packages/lmql/init.py", line 161, in wrapper
return query(fct, input_variables=input_variables, is_async=is_async, calling_frame=calling_frame, **extra_args)
File "/Applications/anaconda3/envs/llm_otd/lib/python3.10/site-packages/lmql/init.py", line 177, in query
code = get_decorated_function_code(fct)
File "/Applications/anaconda3/envs/llm_otd/lib/python3.10/site-packages/lmql/utils/docstring_parser.py", line 65, in get_decorated_function_code
raise RuntimeError("Failed to parse docstring of query function as LMQL code:\n\n" + str(source))
RuntimeError: Failed to parse docstring of query function as LMQL code:
After trying to execute the inspect method, I suspect it is an OS error related to Mac:
File "/Applications/anaconda3/envs/llm_otd/lib/python3.10/site-packages/lmql/utils/docstring_parser.py", line 25, in get_decorated_function_code
source = inspect.getsource(fct)
File "/Applications/anaconda3/envs/llm_otd/lib/python3.10/inspect.py", line 1139, in getsource
lines, lnum = getsourcelines(object)
File "/Applications/anaconda3/envs/llm_otd/lib/python3.10/inspect.py", line 1121, in getsourcelines
lines, lnum = findsource(object)
File "/Applications/anaconda3/envs/llm_otd/lib/python3.10/inspect.py", line 958, in findsource
raise OSError('could not get source code')
OSError: could not get source code

Thanks for reporting this. Can you go into more detail what you mean about "running the function separately"? There is a limitation, where Python's inspect will not be able to get the source for functions that are not defined in a file that can be found somewhere on disk, I think.

Yes exactly, that is the issue. I just wanted to mention that if people want to test functions directly in their interpreter they will maybe run into this issue.


[FEATURE] Add Playfair cipher in Ciphers - New Cipher

Detailed description
The Playfair cipher is a classical encryption technique that uses a 5x5 grid to encrypt pairs of letters (digraphs) in a message. This method helps obscure the frequency analysis of individual letters, making it more secure than simple substitution ciphers. The Playfair cipher works by substituting pairs of letters based on their positions in the grid, which is constructed using a keyword or phrase.
Context
I have previously implemented the Playfair cipher for a class project. While searching for existing code online, I found a lack of accessible implementations that would aid students in understanding and utilizing this cipher for their assignments. After spending two hours writing the code from scratch, I realized the importance of sharing this knowledge. By adding my implementation to an open-source platform, I hope to assist others who may find themselves in a similar situation. It would bring me great joy to see students benefit from my work while also aiming to earn the Hacktoberfest tag for this contribution.
Possible implementation
Here’s a possible outline of how to implement the Playfair cipher:

Grid Construction:

Create a function to generate the 5x5 grid from a keyword.
Remove duplicate letters from the keyword and add the remaining letters of the alphabet (omitting 'J' or treating 'I' and 'J' as the same letter).

Message Preparation:

Prepare the plaintext message by removing spaces and punctuation.

Split the message into pairs of letters (digraphs). If a pair contains the same letter, insert a filler letter (e.g., 'X') between them. If there's an odd letter at the end, pair it with a filler letter.

Encryption Logic:

For each pair of letters, find their positions in the grid.

Apply the Playfair rules:

If both letters are in the same row, replace them with letters immediately to their right (wrap around if necessary).

If both letters are in the same column, replace them with letters immediately below (wrap around if necessary).

If the letters form a rectangle, replace them with letters on the same row but at the opposite corners of the rectangle.

Decryption Logic:

Implement the reverse logic of the encryption process to retrieve the original message.

User Interface:

Create a simple command-line interface that allows users to enter a keyword and plaintext message for encryption and decryption.

Additional information

Languages and Tools: This implementation can be done in C++, Python, or any programming language of choice that supports basic string manipulations and data structures.

Educational Value: The Playfair cipher serves as an excellent introductory exercise for understanding more complex encryption algorithms and cryptography concepts.

Potential Extensions: Consider adding features such as handling special characters, integrating user authentication, or creating a graphical user interface (GUI) for better usability.

Pull Request