colocate main.py with the central directory record of the zipapp

When the Python interpreter executes a zipapp, it will import __main__.py, while __pex__/__init__.py will be loaded if the pex file is on the PYTHONPATH. This file is loaded from the zip file handle with zipimport, and the PEX bootstrap script at that location will then import some of the code in .bootstrap/ before the zipapp is unpacked and compiled. Because a zip file's central directory records are located at the end of the file, placing this code at the end of the zip file list will reduce the amount of seeking the Python interpreter needs to perform in order to execute the bootstrap script. This was previously discussed in https://github.com/pantsbuild/pex/pull/2175#issuecomment-1665710187 and https://github.com/pantsbuild/pex/pull/2175#issuecomment-1665800458.
Changes involved:

Make Chroot#zip() respect the relative ordering of labels when collecting from self.filesets.
Put "main" and "importhook" labels at the end of the zipapp.

@cosmicexplorer I'm a huge non-fan of Problem / Solution formulas for writing commit messages. We're adults and I think you can just write a good message. Quoting people - not helpful. Just explaining the change and its reason should stand on its own.

I'm a huge non-fan of Problem / Solution formulas for writing commit messages.
Ok, I will keep that in mind for the future.

Quoting people - not helpful. Just explaining the change and its reason should stand on its own.
In this case it's not actually documented anywhere else that the python interpreter accesses __main__.py or __pex__/__init__.py from the same zip file handle (which I confirmed by looking through the CPython source), so I thought the quote was precisely the rationale needed for this change. However, given this feedback, I have rewritten the OP to be much more concise while simply linking to instead of directly quoting the prior discussions. Is that closer to what you're looking for?

You're right; I have fixed the wording to clarify that __pex__/__init__.py is loaded if the pex file is on the PYTHONPATH.


Fix issue where vault not ready on init

If vault is used for security provider and is not ready on startup then
the security provider will fail and will be replaced by the
BlackholeSecurityProvider. This change means that whenever a security
provider fails, the DelegatingSecurityProvider will attempt to
re-initialize it on subsequent authorization attempts.
This change does mean that, when failing over to the BlackholeSecurityProvider, it could be slower to let no one authenticate.
I have added a second change as I noticed a fairly impressive memory leak in how the security provider was used.  Again I'm pretty sure no behaviour was changed here.

Looks good.


Add webhook support

Adds webhooks and also fixes a few permissions issues identified during the course of testing the former item.  Also fixes finalizers so that deletion works properly given what is currently deployed via the Barbican resource.

Once this merges, using the operator locally will follow this process (as all other NextGen operators already do): https://github.com/openstack-k8s-operators/docs/blob/main/webhooks.md#using-webhooks-locally


Add a way to filter/toggle columns to easier compare browsers

If there were a way to filter/toggle columns it would be easier for someone to compare just the 2-3 browsers that they are interested discarding information of the browsers discarded a priori.
Alternatively a way to move and re-arrange the columns would also work.

Cool idea!


iOS Notification Actions not fired from "dead" state

I'm using the latest version of this package and I'm encountering a problem with notification actions on iOS when the app is in the "dead" state, when it's in background everything works fine.
If I attach the remote debugger to the app, when waking up from the "dead" state the app seems to bypass the fact that a notification action was pressed.
Any clue on what might be causing this?
I've attached my notification event listeners outside the component and straight into the index.ios.js file.

I have the same issue, actions aren't triggered when app is turned off.


Can we have an option to turn on/off low-power mode

Is your feature request related to a problem? Please describe.
It'd be of great  convenience to have a menu option to turn on/off low-power mode, it's especially useful for m2 chips and the system settings are buried too deep to use.
It'd be even better to have a keyboard short-cut associated with it, and corresponding badge on the menubar icon to indicate whether the low-power mode is enabled.
Describe the solution you'd like
mac commands:

to get status: pmset -g|grep "lowpowermode"|tail -c 2
to set low-power mode: sudo pmset -a lowpowermode 1
to unset:  sudo pmset -a lowpowermode 0

What is the advantage of doing this? I'm not familiar with this setting.

Duplicate of #110, please continue discussion there


Further abstraction

As in #34.

Added a very simple example.

As of the docs, once you think the PR is good to go, I'll add :)

@zawadz88 Done and merged :)

@leonardo2204 thanks for the changes!
This will be published to JCenter in version 2.0.0 as there are some breaking API changes.
I'm planning to release it by the end of January as there are some other improvements I want to add + update the documentation.

Sure, really appreciate all the help and attention !


Incomplete profile helper

We get a bunch of bug reports from people not knowing what to fill in (#5050), this attempts to fix that:

That all being said: I'm pretty sure I've done it in the wrong way, and that there's some big performance bug here... so could some React pro please help me figure out what I've done wrong? (I think one way to fix this is to use a <Controller/> then do this with control/render/onChange..? But that seems rather complicated compared to this... Maybe there's a better way?)
Web frontend checklist

[ ] Formatted my code with yarn format
[ ] There are no warnings from yarn lint --fix
[ ] There are no console warnings when running the app
[ ] Added any new components to storybook
[ ] Added tests where relevant
[ ] All tests pass
[ ] Clicked around my changes running locally and it works
[ ] Checked Desktop, Mobile and Tablet screen sizes

Superseded by #5378


CMake: Support static-only SDL2

When SDL2 is built with -DBUILD_SHARED_LIBS=OFF, it does not define an SDL2::SDL2 target.
Use SDL2::SDL2-static and set POSITION_INDEPENDENT_CODE if BUILD_SHARED_LIBS is OFF.

Hi @sezero, should we need this ?
why isn't this needed in other libs: SDL_image and SDL_mixer ?

First off: I'm not good with cmake (and I don't like cmake, I detest cmake),
so, as far as cmake features are concerned I may be wrong.
That aside, why do we need PIC with static builds?
As for cmake in SDL_mixer: it is very much incomplete (IIRC targeting only Android),
so having this in SDL_mixer should be the least of concerns.

Also, why can we not support both static and dynamic builds in one shot with cmake, like autotools?

Ok, I was just wondering, I am not a cmake expert neither, and I don't use it for SDL. It seems that all the sdl cmake files need some re-engineering.

@1bsyl

why isn't this needed in other libs: SDL_image and SDL_mixer ?

SDL_image does this already:
https://github.com/libsdl-org/SDL_image/blob/d026716b63e761f3fc9f64725480cf37a9dac81f/CMakeLists.txt#L74-L78
https://github.com/libsdl-org/SDL_image/blob/d026716b63e761f3fc9f64725480cf37a9dac81f/CMakeLists.txt#L14-L16

That aside, why do we need PIC with static builds?

If you want to link shared libraries against static SDL_ttf, it needs to be PIC

@glebm, this is true, that was added recently here https://github.com/libsdl-org/SDL_image/commit/f51f5f696abd75c82e12a527a4d8deb9c2ae9cc5
( I grepped on an older version )


progress reporting for Direct Image Migration

This will need a mig-operator PR for the CRD changes.

Required mig-operator PR: https://github.com/konveyor/mig-operator/pull/526


[Doc] release tad freatures

Release the extracted features of THUMOS14 and FineAction.

And refactor the code about building dataset.


❌ DO NOT MERGE (bad commit history): decorative img

creates 2 distinct entries for decorative vs informative img
expands/edits some of the writing in each entry (please edit/revise as you see fit!)
adds a demo page http://localhost:4000/demos/image-decorative/ to test out aria-hidden="true" on decorative images. Lindsay thinks alt empty alone should work in all modern and even post-modern version browsers/screen reader combos

closing as this branch has bad commit history that cannot be reverted/cleanly merged. See new PR here: https://github.com/tmobile/magentaA11y/pull/30


Vickel: Watch maestro[\W]+encrypter\W]+financier

Vickel requests the watch of the watch_keyword maestro[\W]*+encrypter\W]*+financier. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

maestro[\W]*+encrypter\W]*+financier has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


splitting-samples completely tears the mesh apart.

truck-polymesh/examples/splitting-samples.rs outputs about 3000 files with only one triangle registered. It is not correct.

There is a bug in Splitting::into_components.
This bug is fixed in truck-polymesh v0.2.1. Close this issue.


Star's imageView does not correctly animate when triggered via menu

⌘ + delete does the animation as expected.

fixed by #18


Context is lost

When a component is rendered into a <Gateway /> the subtree context is lost.

It seems to be passing the context through fine: https://github.com/cloudflare/react-gateway/blob/master/test/index.js#L72-L125
Can you create a case that reproduces this issue?

Yeah, bit will have to be offline.
On Jan 25, 2016 10:09 PM, "James Kyle"<EMAIL_ADDRESS>wrote:

It seems to be passing the context through fine:
https://github.com/cloudflare/react-gateway/blob/master/test/index.js#L72-L125
Can you create a case that reproduces this issue?
—
Reply to this email directly or view it on GitHub
https://github.com/cloudflare/react-gateway/issues/4#issuecomment-174850186
.

the problem occurs when you rewrite the test case such that the Child doesn't render the Gateway but the Parent does:
class Parent extends React.Component {
      static childContextTypes = {
        textContent: React.PropTypes.string.isRequired
      };

      getChildContext() {
        return {
          textContent: 'Hello from context'
        };
      }

      render() {
        return <Gateway into="dest"><Child/></Gateway>;
      }
    }

This is where unstable_renderSubtreeIntoContainer would come in handy. (Event though it's unstable for good reasons)

Here's why this issue is problematic for my use case: I wanna render an indefinite (= n) amount of portals.  Since one GatewayDest can have only one child due to recent changes, I render one Gateway within the Parent. This gateway then renders n absolutely positioned children.
However those children now don't have access to the context defined within the Parent.

I'm concerned about this too. Here's a quick workaround I did:
class PassContext extends React.Component {
  getChildContext() {
    return {
      dragDropManager: this.props.dragDropManager
    };
  }

  render() {
    return this.props.children;
  }
}

PassContext.childContextTypes = {
  dragDropManager: T.any
}

function X() {
  return (
    <Gateway into="modal">
      <PassContext dragDropManager={this.context.dragDropManager}>
        <Modal2>...</Modal2>
      </PassContext>
    </Gateway>
  );
}

X.contextTypes = {
  dragDropManager: React.PropTypes.any
}


Update JSON null value to Ballerna type conversion

Purpose
This PR contains an improvement to JSON value to Ballerina converter tool on handling null value conversions. The improvement required is explained in the issue $37368.
Fixes #37368
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[x] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing this PR with the new one https://github.com/ballerina-platform/ballerina-lang/pull/37420


n8n: 0.210.1 -> 0.210.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for n8n is: Free and source-available fair-code licensed workflow automation tool. Easily automate tasks across different services.
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.210.2 with grep in /nix/store/5f5b808c7j7fkacg90ma5fkmcjnn8f3y-n8n-0.210.2
found 0.210.2 in filename of file in /nix/store/5f5b808c7j7fkacg90ma5fkmcjnn8f3y-n8n-0.210.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
n8n

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5f5b808c7j7fkacg90ma5fkmcjnn8f3y-n8n-0.210.2 \
  --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 n8n https://github.com/r-ryantm/nixpkgs/archive/b2aae06ba1fb6ff329e48d8d9d05b87e9e1845e0.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/5f5b808c7j7fkacg90ma5fkmcjnn8f3y-n8n-0.210.2
ls -la /nix/store/5f5b808c7j7fkacg90ma5fkmcjnn8f3y-n8n-0.210.2/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:

n8n

Maintainer pings
cc @freezeboy @K900 for testing.

Test passes locally. /shrug


Improve options for compact read-only JSON display.

Right now our options for JSON display for small read only snippets is a bit lacking.

You can pretty print the json and then style yourself in a div -- this lacks coloring, consistency and requires some boilerplate.

You can use a JSONEditor in readonly mode.  This isn't terrible, but it is quite bulky, with a very large left gutter that does not go away even if you turn off line numbers

If we would be willing to do without support for internal coloring (e.g. different colors for keys vs. values vs. braces etc) we could expose a simple component that could take care of:

Rendering an outer div, with layout support + internal scrolling as needed
Accepted either a JSON string or suitable JS object, then (re)parsed and pretty-printed for display
Styled contents with monospace font

If we want the syntax highlighting - or other features like the inline search / copy button - we could push further with codemirror customization to see if we can knock out the gutter.  But as you say, that might feel a bit heavyweight anyway in certain circumstances.  Happy to discuss.


Unknown DB type: timestamp without time zone [PostgresQL]

Realoquent Version
0.6.0
Laravel Version
11.27.2
PHP Version
8.2.24
Operating System & Version
Debian 11
Database Type & Version
Postgres 16.4 (Debian 16.4-1.pgdg110+2)
Description
Step in setup schema then i got bellow exception:
[2024-10-13 20:27:35] development.ERROR: Unknown DB type: timestamp without time zone {"exception":"[object] (RuntimeException(code: 0): Unknown DB type: timestamp without time zone at /var/_www/fitly/vendor/nova-horizons/realoquent/src/TypeDetector.php:115)
[stacktrace]
#0 /var/_www/fitly/vendor/nova-horizons/realoquent/src/TypeDetector.php(22): NovaHorizons\\Realoquent\\TypeDetector::fromPostgreSQL()
#1 /var/_www/fitly/vendor/nova-horizons/realoquent/src/DatabaseAnalyzer.php(28): NovaHorizons\\Realoquent\\TypeDetector::fromDB()
#2 /var/_www/fitly/vendor/nova-horizons/realoquent/src/DataObjects/Table.php(262): NovaHorizons\\Realoquent\\DatabaseAnalyzer::getColumns()
#3 /var/_www/fitly/vendor/nova-horizons/realoquent/src/SchemaManager.php(28): NovaHorizons\\Realoquent\\DataObjects\\Table::fromDB()
#4 /var/_www/fitly/vendor/nova-horizons/realoquent/src/RealoquentManager.php(84): NovaHorizons\\Realoquent\\SchemaManager->rebuildSchema()
#5 /var/_www/fitly/vendor/nova-horizons/realoquent/src/RealoquentManager.php(75): NovaHorizons\\Realoquent\\RealoquentManager->generateSchema()
#6 /var/_www/fitly/vendor/nova-horizons/realoquent/src/Commands/GenerateSchema.php(36): NovaHorizons\\Realoquent\\RealoquentManager->generateAndWriteSchema()
#7 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(36): NovaHorizons\\Realoquent\\Commands\\GenerateSchema->handle()
#8 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Container/Util.php(43): Illuminate\\Container\\BoundMethod::Illuminate\\Container\\{closure}()
#9 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(95): Illuminate\\Container\\Util::unwrapIfClosure()
#10 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(35): Illuminate\\Container\\BoundMethod::callBoundMethod()
#11 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Container/Container.php(690): Illuminate\\Container\\BoundMethod::call()
#12 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Console/Command.php(213): Illuminate\\Container\\Container->call()
#13 /var/_www/fitly/vendor/symfony/console/Command/Command.php(279): Illuminate\\Console\\Command->execute()
#14 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Console/Command.php(182): Symfony\\Component\\Console\\Command\\Command->run()
#15 /var/_www/fitly/vendor/symfony/console/Application.php(1047): Illuminate\\Console\\Command->run()
#16 /var/_www/fitly/vendor/symfony/console/Application.php(316): Symfony\\Component\\Console\\Application->doRunCommand()
#17 /var/_www/fitly/vendor/symfony/console/Application.php(167): Symfony\\Component\\Console\\Application->doRun()
#18 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Foundation/Console/Kernel.php(197): Symfony\\Component\\Console\\Application->run()
#19 /var/_www/fitly/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(1203): Illuminate\\Foundation\\Console\\Kernel->handle()
#20 /var/_www/fitly/artisan(13): Illuminate\\Foundation\\Application->handleCommand()
#21 {main}

here is one of my table have timestamp column that existing on DB.
CREATE TABLE accounts (
    id BIGINT GENERATED ALWAYS AS IDENTITY PRIMARY KEY,
    site_id BIGINT NOT NULL, -- Liên kết với bảng bet_sites
    username VARCHAR(255) NOT NULL,
    pwd VARCHAR(255) NOT NULL,
    email VARCHAR(255),
    phone VARCHAR(50),
    reg_at TIMESTAMP,
    last_login_at TIMESTAMP, -- Thời gian đăng nhập lần cuối
    created_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
    updated_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
    FOREIGN KEY (site_id) REFERENCES bet_sites(id)
);
CREATE INDEX idx_accounts_site_id ON accounts(site_id);
CREATE INDEX idx_accounts_username ON accounts(username);
CREATE INDEX idx_accounts_email ON accounts(email);
CREATE INDEX idx_accounts_phone ON accounts(phone);

Apologies for the delay, just published v0.7.0. Let me know if that resolves the issue. I haven't done much real-word testing with Postgres yet, so let me know if any other problems pop up, or any other thoughts you have about this library.

Hello.
I were tested with version 0.7.0 and still occured the error.
Do you need more dump, please let me know.
Here is console commands & output:
web@zuko:/www/fitly$ composer8.2 req nova-horizons/realoquent --dev
./composer.json has been updated
Running composer update nova-horizons/realoquent
Loading composer repositories with package information
Updating dependencies
Lock file operations: 0 installs, 1 update, 0 removals
  - Upgrading nova-horizons/realoquent (v0.6.0 => v0.7.0)
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 0 installs, 1 update, 0 removals
  - Downloading nova-horizons/realoquent (v0.7.0)
  - Upgrading nova-horizons/realoquent (v0.6.0 => v0.7.0): Extracting archive
Generating optimized autoload files
> Illuminate\Foundation\ComposerScripts::postAutoloadDump
> @php artisan package:discover --ansi

   INFO  Discovering packages.

  barryvdh/laravel-debugbar ................................................................................................................... DONE
  irazasyed/telegram-bot-sdk .................................................................................................................. DONE
  laravel/sail ................................................................................................................................ DONE
  laravel/sanctum ............................................................................................................................. DONE
  laravel/tinker .............................................................................................................................. DONE
  nesbot/carbon ............................................................................................................................... DONE
  nova-horizons/realoquent .................................................................................................................... DONE
  nunomaduro/collision ........................................................................................................................ DONE
  nunomaduro/termwind ......................................................................................................................... DONE
  spatie/laravel-ignition ..................................................................................................................... DONE

85 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
> @php artisan vendor:publish --tag=laravel-assets --ansi --force

   INFO  No publishable resources for tag [laravel-assets].

No security vulnerability advisories found
Using version ^0.7.0 for nova-horizons/realoquent
web@zuko:/www/fitly$ php8.2 artisan realoquent:generate-schema

   RuntimeException

  Unknown DB type: timestamp without time zone

  at vendor/nova-horizons/realoquent/src/TypeDetector.php:120
    116▕             'time(0) without time zone' => ColumnType::time,
    117▕             'timestamp(0) without time zone' => ColumnType::timestamp,
    118▕             'timestamp(0) with time zone' => ColumnType::timestampTz,
    119▕             'uuid' => ColumnType::uuid,
  ➜ 120▕             default => throw new RuntimeException('Unknown DB type: '.$type),
    121▕         };
    122▕     }
    123▕
    124▕     private static function fromSqlite(mixed $type): ColumnType

      +20 vendor frames

  21  artisan:13
      Illuminate\Foundation\Application::handleCommand()

web@zuko:/www/fitly$

i see in your pull request: There some types were added, but not mine. I know it will need more tests. But just leave it here. In case you need it.
timestamp without time zone is $type (string)

Doh, released 0.7.1 with the additional type, not quite sure at the moment why there's subtle differences in the returned types compared to tests, but will look into it more.

It's seem setup process passed.
Then there's a "warning".
Since i have mutilple php version on my server. Then Main command class of realoquent will failed if i does not set active version = requirement of realoquent. (I used update-alternatives to manage symlink of /usr/bin/php)
Env setup : primary PHP : 7.4
command was ran by using php8.2.
See console log bellow
web@zuko:/www/fitly$ php8.2 artisan realoquent:generate-schema
Realoquent Schema file generated successfully: /var/_www/fitly/database/realoquent/schema.php

   RuntimeException

  Failed to run code style fixer. Check `cs_fixer_command` config value.
Box Requirements Checker
========================

> Using PHP 7.4.33
> PHP is using the following php.ini file:
  /etc/php/7.4/cli/php.ini

> Checking Box requirements:
  E.....

 [ERROR] Your system is not ready to run the application.

Fix the following mandatory requirements:
=========================================

 * The application requires a version matching "^8.1.0".

  at vendor/nova-horizons/realoquent/src/RealoquentManager.php:136
    132▕         }
    133▕
    134▕         foreach ($commands as $command) {
    135▕             $result = Process::path(base_path())->run($command);
  ➜ 136▕             throw_unless($result->successful(), new \RuntimeException('Failed to run code style fixer. Check `cs_fixer_command` config value. '.$result->errorOutput()));
    137▕
    138▕         }
    139▕     }
    140▕

      +14 vendor frames

  15  artisan:13
      Illuminate\Foundation\Application::handleCommand()

I did not read the code yet, but rely on php in PATH is not really good pratice

You can adjust your config by updating the cs_fixer_command value to use the proper version of PHP on your system. By default it is using ./vendor/bin/pint {file}, but you can prefix that with the correct PHP path.

From now please let me use "our package" to ref to realoquent
I known that there a configuration var for that. But i just want more powerful to the package. See bellow
Background
Currently, when running commands like pint through our package, it may use the system's default PHP version (e.g., PHP 7.4) even when project is running on PHP 8.2. This happens because the binary uses a shebang (#!/usr/bin/env php) which picks up the system PHP instead of your project's PHP version.
Important: This value:  ./vendor/bin/pint of cs_fixer_command  is published from our default configuration.
What it leads to

Complexity to get this package setted-up specially newly installed user (who just a "USER", not understand muchmore in system relate)

To relsove: users needed extra steps to configure the correct PHP binary
OR: Causes unexpected errors when system PHP version doesn't meet requirements, so the setup process can not be done.
Solution:
Here just a first think in my mind, and i'm too lazy to create a fork or pull request :D
First, try to read about 20 first line of configurated cs_fixer_command . if there is start of php code block. using PHP_BINARY magic contant to excute that binary file.
Example of code (i taken this from modified RealoquentManager::runCodeStyleFixer() ):
// Check if binary is a PHP script
        $binaryPath =  str_starts_with($this->csFixerCommand, './') ? base_path(trim($this->csFixerCommand, '"\'./')) : $this->csFixerCommand;
        $isPhpScript = false;
        $binaryPath = explode(' ', $binaryPath)[0];
        if (file_exists($binaryPath)) {
            $content = file_get_contents($binaryPath, false, null, 0, 1000);
            $isPhpScript = str_contains($content, '<?php') ||
                str_contains($content, '#!') && str_contains($content, 'php');
        }

        // Prepare command with PHP_BINARY if needed
        $command = $isPhpScript
            ? PHP_BINARY . ' ' . $this->csFixerCommand
            : $this->csFixerCommand;


[Bug]: Filtering option in Filter Staged Images does not work

Steps to reproduce the bug
1.Open filter staged images

Screenshot

Expected behavior
A Filter button is missing, data can not be filtered.
Additional Screenshots
No response
Environment
Production
Sandbox Used
No response
Version
No response
Browser Used
chrome

What does it mean blocker? The function still not working, is there a maintenance issue here?
Thanks!
István

Oh, I see, do you know when the update will be?

thnx!
i
On Tue, Apr 2, 2024 at 4:07 PM Matt @.***> wrote:

hopefully this evening. a reminder that closing issues here means they are
fixed in development, but until the CHANGELOG.md reflects a release they
are not yet live
—
Reply to this email directly, view it on GitHub
https://github.com/SpeciesFileGroup/taxonworks/issues/3901#issuecomment-2033005699,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGX6JLHCIOVVAB3KO3FIXTY3MFWVAVCNFSM6AAAAABFRY225CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZTGAYDKNRZHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
István Mikó PhD
Collection Manager
Department of Biological Sciences
117 Spaulding Life Sciences Building
38 Academic Way, Durham, NH 03824


add INSTALL_PATH

I couldn't get it to run from hyprland directly because of the hardcoded relative shader paths, this makes them absolute

this be better to just use INSTALL_PATH"/main.vert" instead of sprintf which is unsafe. also it would be a good idea for installing it to copy the files to a specific directory like:
/usr/local/share or something


Add AboutCard component to EntityPageOverview

Hey, I just made a Pull Request!
Replacing the EntityMetadataCard with new design.

fix #2039
:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

If we display the tags here in gray, I would probably switch the tags on the catalog list to gray, too.

They feel a bit like screaming at one 😆 I can send a PR if you are interested.

@Fox32 yeaaaah, please do! I've been meaning to bring that up as well 😄

Here: #2205


v2: Improves CSS style for ActionSheet in iOS

Top padding
Title font size
Background default color
Background active color
Button border when active.

Left: native iOS 9 Action Sheet
Right: ionic 2
Before fix:

After fix:

Thank you for this @manucorporat!

@bensperry you are welcome!


actions_popover: Fix copy link not working on enter keypress.

discussion: https://chat.zulip.org/#narrow/stream/9-issues/topic/Cannot.20copy.20link.20to.20message.20with.20keyboard

Hmm, wouldn't navigate_link_on_enter take you to another place rather than copying the link?

Yeah, but the copy link anchor tag doesn't have a href so, it doesn't try to go anywhere. Maybe we need a different class name for it?

Merging this, since the bug is fairly important, but also posted a comment that probably is best replied to in #frontend about what's going on here. It's possible the right answer is to rename the class, but also possible we could just tweak that logic to be smarter, or have a on_enter_click_target selector that the popover_menus.js code picks out to click on, rather than just clicking on the li.


File-level Parallelization for multipart upload of dataset and checkpoint

This PR makes the multipart upload parallelized at a file level.
Now min(32, (os.cpu_count() or 1)  + 4) // 2 threads are used to handle each file upload, and each thread spawns min(32, (os.cpu_count() or 1)  + 4) // 2 threads again to upload a split chunk of multipart uploading.
The optimization boosts up the upload throughput from 15MB/s to 50MB/s in my laptop with 10 vCPUs.
@ktaebum @kdh0102 It will be helpful if you can suggest a better concurrency option(e.g., the number of max thread pool workers).

@kdh0102 @ktaebum So, I fix the concurrency logic like this.
Now the outer thread pool, which parallelizes the upload of each file, and the inner thread pool, which parallelizes the upload of parts of a file, are shared. The ThreadPoolExecutor is initialized only once and it is passed to _multipart_upload_file method as an argument.
And how about getting a --num-threads options from the user as a CLI option?


[Automation] Generate Fluent Lite from appconfiguration#package-2023-03-01

[Automation] Generate Fluent Lite from appconfiguration#package-2023-03-01

version is not correct, will use local generation.

API change check
API changes are not detected in this pull request.

close by #34709


Feat request: Ability to specify default accessibility

Search Terms
typescript option default accessibility
Suggestion
It'd be neat to be able to configure the default accessibility of a class member. For example, if the option is set to private, then any members without accessibility explicitly defined would be private.
I like the idea of being require to explicitly defining what is public, and what is protected, with everything (except maybe the constructor) defaulting to private. It'll be a stricter mode that forces people to think about what API they expose to others, and also prevents bugs from methods accidentally not being called because the default public accessibility doesn't catch unused vars/functions.
Use Cases
Basically prevent errors, and forces people to explicitly define what APIs are meant to be exposed, and what parts are internal implementation.
Examples
Just imagine an option (f.e. defaultAccessibility or similar name) is set to private, and that writing the following results in an error:
class One {one() {}}
const o = new One
o.one() // ERROR, one is private

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

We generally don't add flags that radically change the interpretation of a given block of code.


feat: Add InterPlanetary Wayback image

This PR adds a logo for the InterPlanetary Wayback project.
I ran both make build and npm run build:readme, which causes a change in the README.md but per @VictorBjelkholm in #172, "readme should contain no changes", so I have not included it in the commit.

Please include the changes to the Readme file


Updated to use python3 and PyObjC 8.4

Switched print function to Python 3. Added import of ColorSync framework module from pyobjc. Removed import of objc and deprecated parseBridgeSupport code. Cleaned up some formatting using PyCharm.
To get all of the modules to import correctly on my newly updated macOS 12.3 installation, I needed to reinstall pyobjc entirely using:
% python3 -m pip freeze | grep pyobjc | while read p; do python3 -m pip uninstall -y "$p"; done;
% sudo python3 -m pip install pyobjc

Tested and working with Python 3.8.9 on macOS 12.3 installed on an iMac M1 2021. Fixes Issue #6.

Thanks so much for sending this in! This was needed. I'm not in a location where I can take a closer look, but I should be able to tomorrow.

I came back to this repo today and realized I'd never merged your modernization work! Thanks, it's much appreciated!
I'm going to look at incorporating a bit of pyinstaller as a way to offer a pre-compiled release, as well, from an older experiment branch I had.


extractCSS: true   css-flash?

not dev mode.
nuxt rc6
Deployed to server

Please give more informations, also, this is normal in dev mode, see https://nuxtjs.org/faq/css-flash

not dev mode.
nuxt rc6
Deployed to server

@Atinux
not dev mode.
Deployed to server

Do you have a link to the website @gamebody ?

@Atinux
http://test.fenmio.com/

Make sure to import your css into nuxt.config.js at the css property, see global-css example.

@Atinux
extractCSS: true
This setting is to extract CSS to a single CSS file？ I didn't find the CSS file

@Atinux
The CSS in the Vue component is handled by the vue-style-loader，So, at first, there will be CSS flash

@gamebody
I met the same problem, how can you fix this bug?

@ummo new folder and reinstall npm package。
run build

@gamebody could you please describe in detail? I have the same problem and nothing seems to work. Npm Build and then npm start starts the production Server right? I reinstalled all npm packages. I have the latest stable node version. I extract my CSS and my bootstrap grid is in the globals. Any idea?

I created a new project. Copied all my project files. RC8. Used the starter template. Now it works.


print-dataset with options now returns a string

Was side effecting via println

The print here is intentional. It's a printing function.
Do you remember why did you want to remove println in the second case?

I think I just misread the 1 arity code. I think this is my mistake. I'm happy to close it (and I've not bumped into it in ages and don't remember why I suggested the change).

I remember we've discussed this but what was the conclusion? idk... :) Anyway - there are two functions one creates string and one prints to the out. Should be ok for all the cases. Closing it.


Case studies

Description:
Create detailed case studies that highlight how organizations have successfully leveraged open-source solutions to address challenges, innovate, and drive business growth. Share real-world examples of how open-source technologies have been implemented, the benefits they provided, and the lessons learned from these experiences. These case studies will serve as an educational resource for the open-source community, especially for organizations considering open-source adoption or looking to scale their use of Open-Source Software.
What You Can Do:

Identify Organizations Using Open-Source Solutions: Start by researching companies, non-profits, or government entities that have adopted open-source software (OSS) for their operations. These could be organizations of various sizes, from startups to large enterprises, across different industries.
Examples might include:

Netflix and its use of open-source for microservices architecture.
NASA leveraging open-source tools for scientific research.
The European Union promoting the use of OSS in public services.
Benefits and Impact: Detail the positive outcomes, such as reduced costs, improved performance, enhanced collaboration, or community engagement

Action to take:

Write a Script - Check out the  samples  and use this template
Create a Slide deck - Find a sample here, and use this template
Record a video - Sample

Additional context

Kindly go through the ReadMe.md file.
Contributing.md file contains information that will help you get started
Choose your preferred background for your video recording.
Resources :
Red Hat Open Source Success Stories
Apache Foundation Case Studies
CNCF (Cloud Native Computing Foundation) Case Studies
Open Source Initiative Case Studies
[Linux Foundation]:(https://www.linuxfoundation.org/) Case Studies
GitHub Case Studies
Open Source Initiative

Hello Peculiar, Trust your day went well? Here's the transcript and slide respectively https://docs.google.com/document/d/1nTXMfsO9dnVc5jV26H6aheh9yduCWa4uaLNQ5KIOxb8/edit?usp=sharing https://docs.google.com/presentation/d/1jg7O9J9rtjMAZX7cD9djrCc5SQXQEGuM4XWUsJouTd0/edit#slide=id.g2fb20b0af91_7_10 Please go through and revert Best Regards Maybell
…
On Wed, Oct 23, 2024 at 11:45 PM Peculiar C. Umeh @.> wrote: Assigned #107 <#107> to @frontEndDoctor https://github.com/frontEndDoctor. — Reply to this email directly, view it on GitHub <#107 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIO3GA2VOU3RF3ZRJ7ZQYKTZ5ARHNAVCNFSM6AAAAABOAZFLUKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUHAZTENZYGA4TQMQ . You are receiving this because you were assigned.Message ID: @.>

Thanks for your effort on this.
Could you please send this on a PR, so it will be easy to share with other reviewers?
Thank you so much.


Upload package to Package Control?

This will make the installation so much easier

It's an ST4 package, and I am hesitant to upload it until there is a public release for ST4.

Public release is here, would love this in package control :)

I'd probably switch to the https://github.com/python-lsp/python-lsp-server fork before releasing it since pyls seems kinda dead.
Also, did you guys consider LSP-pyright instead? The general opinion is that it's superior to this one when it comes to language support. It doesn't support linting and plugins stuff though.

How do I install this package? I've only used Package Control.

@StabbarN within your Packages directory you will need to run git clone https://github.com/sublimelsp/LSP-pyls.git and there you have it
Your packages directory can be found by running
sublime.packages_path()

In your sublime console

@TheSecEng Thank you! It worked perfectly. I didn't even have to restart sublime.

Yeah, we worked pretty hard on seamless installation.

Perhaps put this in the readme?

git clone https://github.com/sublimelsp/LSP-pyls.git

I'm working on switching out pyls for pylsp in https://github.com/sublimelsp/LSP-pyls/pull/23
Once that is done, will publish LSP-pylsp to package control.

Done.

Thank you. I'm new to PLS and just updated to the LSP-pylsp in package control. I'm able to disable pycodestyle and mccabe with the following, but I don't understand isort. First, should it be "pylsp.plugins.pyls_isort.enabled" and second, what does it do? Does it show a warning (I don't see it) or auto-format the code? I've set it to true and don't see either.
 // Settings in here override those in "LSP-pylsp/LSP-pylsp.sublime-settings"
{
    "settings":
    {
        "pylsp.plugins.pycodestyle.enabled": false,
        "pylsp.plugins.mccabe.enabled": false,
        "pylsp.plugins.pyls_isort.enabled": true,
    }
}

Actually I can get the isort via LSP: Format document, but it's different from running it in the terminal.
isort in the terminal yields:
import argparse
import logging
import os
import re
import string
import sys
import time
from collections import namedtuple
from datetime import datetime
from io import StringIO
from subprocess import Popen, call
from xml.etree.ElementTree import Element, ElementTree, SubElement, parse

from dateutil.parser import parse as dt_parse
from lxml import etree as l_etree

import thunderdell as td
from change_case import sentence_case, title_case
# personal utilities
from web_utils import escape_XML, get_HTML, get_JSON, get_text, unescape_XML

LSP: format file document yields:
import argparse
import logging
import os
import re
import string
import sys
import time
from collections import namedtuple
from datetime import datetime
from io import StringIO
from subprocess import Popen, call
from xml.etree.ElementTree import Element, ElementTree, SubElement, parse

import thunderdell as td
from change_case import sentence_case, title_case
from dateutil.parser import parse as dt_parse
from lxml import etree as l_etree

# personal utilities
from web_utils import escape_XML, get_HTML, get_JSON, get_text, unescape_XML

I'll stop this here, I think there's a confusing interaction between "LSP: Format File" and sublack running in real time.

Are you on https://github.com/sublimelsp/LSP-pylsp/releases/tag/2.0.2 ? This includes a fix for issue where black would run even when disabled.

Hi, yes, 2.0.2 . I have sublack enabled and running (reformats on every save; this is different than LSP) and it seems like there is a difference between it and pylsp_isort. (I don't need isort that often, so I'll just set _isort to false for now).


⚠ Bofrost.at has degraded performance

In 3450383, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2248 ms

Resolved: Bofrost.at performance has improved in 8d5ba88 after 48 minutes.


How to set env sandbox ?

Hi, I have read the documentation in the README.md file, but I haven't found a way to use sandbox mode.
I check the code and find that i have to set the prop "dev" to true to use the sandbox mode but it will be helpful if it is explained in the README.
Thanks

Hey, sorry if it isn't clear.
Use the component like this:
<paypal-checkout
  amount="10.00"
  currency="USD"
  :client="credentials"
  :dev="true"
  invoiceNumber="201701011000">
</paypal-checkout>

Here is where you specify your sandbox client id. If the prop dev is true, it should use sandbox mode, with the specified sandbox client id in your data().
If you need a sandbox client id, you should be able to get one from here https://developer.paypal.com/developer/applications/
export default {
  data () {
    return {
      credentials: {
        sandbox: '<sandbox key>',
        production: '<production key>'
      }
    }
  }
}

I'll take a look into updating the readme about using it in sandbox mode.


Page.GetCookiesAsync does not return all the cookies

The method Page.GetCookiesAsync() doesn't return all the cookies showed in the Chrome Dev Tools. After googling this issue, i found a nice explanation on Stack Overflow (https://stackoverflow.com/questions/49389775/puppeteers-page-cookies-not-retrieving-all-cookies-shown-in-the-chrome-dev-to/59604510). So basically the cookie i want is probably an httpOnly cookie, wich is not retrieved by the "Network.getCookies" command performed by the Page.GetCookiesAsync() method.
My humble suggestion would be adding an method called Page.GetAllCookiesAsync(), with this implementation:
public async Task<CookieParam[]> GetAllCookiesAsync(params string[] urls) => (await Client.SendAsync<NetworkGetCookiesResponse>("Network.getAllCookies", new NetworkGetCookiesRequest             {                 Urls = urls.Length > 0 ? urls : new string[] { Url }             }).ConfigureAwait(false)).Cookies;
In other words, just a clone of the already existing Page.GetCookiesAsync() method, but with the "Network.getAllCookies" call instead of the "Network.getCookies".

Thank you for the insight and the time to go through it.
I think that makes sense. I would like to implement that on the original puppeteer library so we are in sync.
We shouldn't have any API that doesn't match an upstream one.

Thanks for the feedback! Meanwhile, I found a workaround. Since the Page.GetCookiesAsync() accepts an array of urls as optional parameter, I've just inspected and passed all the urls of the cookies I needed.

Thank you for the insight and the time to go through it. I think that makes sense. I would like to implement that on the original puppeteer library so we are in sync. We shouldn't have any API that doesn't match an upstream one.

Thanks for the feedback! Meanwhile, I found a workaround. Since the Page.GetCookiesAsync() accepts an array of urls as optional parameter, I've just inspected and passed all the urls of the cookies I needed.

It depends on https://github.com/puppeteer/puppeteer/issues/6916


⚠ Bofrost.de has degraded performance

In fdf0866, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1837 ms

Resolved: Bofrost.de performance has improved in 400679b after 37 minutes.


Adding a list of OpenStack requirements for provider docs.

Adding list of openstack requirements.

@zreigz This takes care of listing the requirements per https://github.com/kubernetes/kubernetes.github.io/pull/74#issuecomment-217329346

LGTM :+1:


correct python code for db-connected function app

the affected python code did not work previously

testing the function app by executing it with vscode sends the name parameter via the body, we need to access that instead of the query params
uuid was not imported
uuid objects are not serializable and need to be converted to string explicitly
changed false -> False
changed url -> "url" for a valid dictionary key

@dzsquared
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@JetterMcTedder
#assign:JetterMcTedder

#sign-off

@Court72 if you can reassign it to me ill sign off.

#sign-off
if you're good with it I'll just drop the hashtag to get this done :)

#sign-off

#sign-off

#sign-off

#sign-off

ok, i think we are merged

are you sure? maybe we should #sign-off again
:)

i double dare you...


Added GATT Hardware Revision String and Serial Number Characteristics

Issue
#58.
What does this PR Do?
Added GATT Hardware Revision String and Serial Number Characteristics
Where should the reviewer start?
GATTHardwareRevisionString.swift and GATTSerialNumberString.swift

Result of Integration 1
Duration: 2 hours, 16 minutes and 27 seconds
Result: -1 error, failing state: checkout-error

Result of Integration 1
Duration: 2 hours, 16 minutes and 27 seconds
Result: -1 error, failing state: checkout-error

Result of Integration 1
Duration: 2 hours, 16 minutes and 27 seconds
Result: -1 error, failing state: checkout-error

Result of Integration 1
Duration: 2 hours, 16 minutes and 27 seconds
Result: -1 error, failing state: checkout-error


nc_test4/tst_vars4.c is mostly commented out

Not sure when this got commented out, may be my fault. Anyway, tests don't do us much good it we comment them out! ;-) I will uncomment and see what happens.

OK, test code runs just fine. I will fix this in my next PR.


Doesn't seem to work

I wish I had a better message to report than "it doesn't work", but there's no feedback or error message.

I added the gem to the Gemfile as specified (in development) in the correct order
I double checked that I'm running in development,
updated the binding_of_caller and better_errors gems because they were a little out of date,
stopped spring,
stopped the server,
tried straight puma,
tried rails s,
reset the terminal,
reset the computer

... nothing. When better_errors loads, I still have the regular old boring non-Pry console.
Is there some kind of other installation or initializer that needs to be set up that isn't mentioned in the Readme?

Sorry @bbugh I was missing some documentation. This gem doesn't implicitly activate Pry in BetterErrors–BetterErrors does not currently support doing that. So you must still activate pry: https://github.com/charliesome/better_errors/wiki/Pry
I have a branch where I started to make this automatic, but it relies on changes I need to make in BetterErrors first, and I've been too busy to finish all of it. I've updated the readme in the meantime.

Excellent, thank you! The Readme is much more clear about the purpose of this library now. 👍


Ikuffour

Deleted the empty Frontend Directory.
Since our model is based on MVC,I moved all the VIEW components of the web app into view directory since that is how Jinja2 from Flask renders HTML pages. Same with Django, they all look for HTML files in a folder called templates, none other. Tried different methods to render outside templates, didn't work
Added some HTML files and CSS styles to render the home page with flask.
Home page screenshot below

Searched on stack overflow and chatgpt if there was a way to render HTML files from a different directory outside templates, I tried them but they al don't work. Unless I change the name to templates and it must be in the same folder as the flask app

Alright. But everything in the view is about routing the frontend or what
the user can see but yea you can check their docs out. or even stack
overflow
On Wed, 1 May 2024 at 8:04 AM, Kwabena Twumasi @.***>
wrote:

Merged #8 https://github.com/angelotheman/RenTopia/pull/8 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/angelotheman/RenTopia/pull/8#event-12668089305, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BBCPLI5XBWIOZLZXEJU4GGDZACO2DAVCNFSM6AAAAABHBNESROVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGY3DQMBYHEZTANI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Unable to initialize user keys: Value cannot be null.

Hi, so whenever i log in it spits out this error and i cannot open files on drive.

Extended error report shows this:
Unable to initialize user keys: Value cannot be null.
Parameter name: source
ProtonDriveSync
at ProtonDriveSync.StorageProvider.ProtonAPI.d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ProtonDriveSync.StorageProvider.ProtonDriveStorageProvider.d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ProtonDriveSync.ProtonDriveSyncExt.d__8.MoveNext()
Void MoveNext()
Thank you in advance for looking into this.

Same error here with keepass 2.55 and last version of protondrivesync on Windows 10 pro

Same error here with keepass 2.56 and last version 0.1.3 of ProtonDriveSync on Windows 11 pro 23H2

Unable to initialize user keys: Unable to refresh session: API response contains an error (10013) : Invalid refresh token

Same error here with keepass 2.56 and last version 0.1.3 on windows 10 22H2

Hi Guys,
I ran into the same problem detailed by grenmel, and found that although proton will allow you to login with just your username, this plugin didn't work until I specified my full email address -<EMAIL_ADDRESS>not just username. Works fine now.

This has been fixed with release 0.1.4. You can now login with username or email address.


Its not possible to have NO cssPrefix

Hi even when setting cssPrefix to an empty string it still gives the default 'icon-' prefix.
changing the if (!options.cssPrefix) { to  if (options.cssPrefix === undefined) {
wil give the possibility to have an empty prefix

Yep, I just encountered this myself.
Try PR #17


quick fix for octopress/deploy#41

Makes S3 uploads publicly accessible to fix issue #41.
I tried to do this more properly with a config file option, but wasn't able to get it to actually work (see issue #42).

I just wanted to pop in quickly and say thanks for pitching in. I can't review this right now, but I will soon.

I ended up rebasing this on #42 and merging it into master. Testing this ended up helping me find a bug with pulling down an S3 site, so that was great. Thanks for your help!


Fix a couple of typos

Hey, I just made a Pull Request!
A couple of typos I noticed when reading the docs
:heavy_check_mark: Checklist

Nice thanks :tada:


Document data model

[x] Drawio in repo
[x] Script to extract SVG
[x] Makefile to install
[x] make commit
[x] first draft of model

[x] orgs spaces and owners
[x] add tables PKs and FKs

Or maybe planting?

Suppress transparent background

Generate list of fields for all csvs using csvkit - n

done enough for the time being


Clean up: common coordinate space

MINOR changes:

improved instructions
establish alphabetical order
establish correct order within a property

MAJOR changes:
none
SPECIAL ATTENTION:

added strain as allowed value for usedSpecies since those are in a way more specific species and we solved this similarly for the species of a specimen

@aeidi89 could you double check this PR and merge if you agree with my changes and don't find any other issues I overlooked.


nbqa: add passthru.withTools

Description of changes
You can now do fancy things like
(nbqa.override { python3 = python311; }).withTools (ps: [ ps.black ])

I didn't apply nixfmt for now in order to keep the diff minimal. Any suggestion for making both happen?

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I didn't apply nixfmt for now in order to keep the diff minimal. Any suggestion for making both happen?

A separate commit applying nixfmt (and only that) before the commit with the functional change would make it easy to review the functional change separately.

I didn't apply nixfmt for now in order to keep the diff minimal. Any suggestion for making both happen?

A separate commit applying nixfmt (and only that) before the commit with the functional change would make it easy to review the functional change separately.

Do you mean after? The problem is that introducing let ... in would indent everything by two spaces.

I didn't apply nixfmt for now in order to keep the diff minimal. Any suggestion for making both happen?

A separate commit applying nixfmt (and only that) before the commit with the functional change would make it easy to review the functional change separately.

Do you mean after? The problem is that introducing let ... in would indent everything by two spaces.

No, before. It's easy to ignore space changes when reviewing.

@l0b0 What you're saying doesn't make sense. The functional change (introducing let) is exactly what requires reformatting.

@l0b0 What you're saying doesn't make sense. The functional change (introducing let) is exactly what requires reformatting.

pkgs/tools/misc/blacken-docs/default.nix should be reformatted before moving it. But you're right that pkgs/by-name/nb/nbqa/package.nix doesn't need reformatting before the functional change. In that case, the reformatting should be done as part of the functional change commit, so that each commit passes linting.

pkgs/tools/misc/blacken-docs/default.nix should be reformatted before moving it

No point because git correctly identifies the file as being moved.

In that case, the reformatting should be done as part of the functional change commit, so that each commit passes linting.

That doesn't answer my question of how to do it with a minimal diff.

In that case, the reformatting should be done as part of the functional change commit, so that each commit passes linting.

That doesn't answer my question of how to do it with a minimal diff.

Making each commit valid is more important than having the minimal diff.

pkgs/tools/misc/blacken-docs/default.nix should be reformatted before moving it

No point because git correctly identifies the file as being moved.

There is no such commit and there won't be. Please stop.

There is no such commit and there won't be. Please stop.

When you said:

pkgs/tools/misc/blacken-docs/default.nix should be reformatted before moving it

No point because git correctly identifies the file as being moved.

That's not the only thing you need to do when moving a file. You also need to change anywhere that file is referenced. That's a functional change, which means the files involved must be nixfmted. By doing that in a separate commit before moving the file, you could keep each commit as simple as possible.

Alright, enough. I blocked you.

@dotlambda did you just block the current maintainer of this package for trying to help you (even though you originally explicitly asked for input about that)? Or am I horribly misreading the situation?

They were not helpful at all. They suggested I turn the commit for blacken-docs into two for no reason and didn't comment on how to improve the withTools commit.


Advanced configuration

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
yes

Tickets

License
MIT

The allows for a more advanced configuration, mainly per branch configuration and a local configuration file.

Configuration is now validated using the Symfony Config component
Per branch configuration (requires schema-version 2 is used)
Local configuration file usage for easier management (requires schema-version 2 is used in the main config file)

The per branch configuration allows defining a default configuration, per branch
and version ranges by allowing regexp.
Existing configurations will continue to work but will not be supported in Hubkit v2.0

Keeping a local configuration file might cause some trouble when switching branches, where the new configuration might be expected. To solve this it might be an idea to use a special _hubkit branch to keep configurations (and scripts for special operations).

Local configuration is now stored in a special branch named "_hubkit", the configuration is loaded once (using a special temporary repository) and it not effected by branch switching.
This new config branch can also be used in the future to keep special hook scripts for upmerge for example, as the script can be executed without the need to keep this in the lowest possible branch.

OK. I'm finally done! I added two additional commands to initialize the configuration (importing any existing configuration to ease the transition) and another one to synchronize the configuration with the upstream (pull/push/rebase when diverged).
All and all I'm really pleased with this feature!


Task Status Update

Added a feature for Turbiniactl status  to show live updates for queued/running tasks.

@aarontp PTAL.


Fix event dispatcher off error

Fix #1309

Codecov Report
Base: 36.55% // Head: 12.15% // Decreases project coverage by -24.40% :warning:

Coverage data is based on head (80228b1) compared to base (ab3c82c).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head 80228b1 differs from pull request most recent head ec46f9d. Consider uploading reports for the commit ec46f9d to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1314       +/-   ##
===========================================
- Coverage   36.55%   12.15%   -24.41%
===========================================
  Files         349      349
  Lines       16721    16728        +7
  Branches     2618     2619        +1
===========================================
- Hits         6113     2033     -4080
- Misses      10052    14482     +4430
+ Partials      556      213      -343

Impacted Files
Coverage Δ

packages/core/src/base/EventDispatcher.ts
71.42% <100.00%> (+60.71%)
:arrow_up:

packages/core/src/VirtualCamera.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

packages/math/src/SphericalHarmonics3.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

packages/core/src/physics/StaticCollider.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

packages/core/src/shadow/ShadowSliceData.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

packages/physics-lite/src/LiteStaticCollider.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

packages/core/src/material/PBRMaterial.ts
6.25% <0.00%> (-93.75%)
:arrow_down:

packages/core/src/material/UnlitMaterial.ts
0.00% <0.00%> (-90.91%)
:arrow_down:

packages/core/src/material/BlinnPhongMaterial.ts
1.85% <0.00%> (-90.75%)
:arrow_down:

packages/core/src/material/PBRSpecularMaterial.ts
5.55% <0.00%> (-88.89%)
:arrow_down:

... and 99 more

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

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


Cannot find nitro.json after build inside Docker

Environment

Operating System: Linux
Node Version:     v16.15.1
Nuxt Version:     3.0.0-rc.6
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr
Runtime Modules:  -
Build Modules:    -

Reproduction

Clone this repo: https://github.com/HendrikJan/nuxt3-problem
Follow the steps in the README
See that npm run build give this error inside Docker: ACCES: permission denied, stat '/root/.node_modules/file:/var/app/node_modules/unenv/runtime/mock'
See that npm run start gives this error: ERROR  Cannot find nitro.json.
See that you do not get an error outside Docker

Describe the bug
Nuxt (or Nuxi) can build and run without problems outside Docker, but inside docker we get errors and the server does not start.
Additional context
nuxt dev works with no problem, writing to the /.nuxt-folder, so permissions seem to be ok.
Logs
No response

Hi, any fix about this error ? I got the same.

No problem during npm run build

You should not run nuxi preview in production. It is just for a developer preview. Instead, run node .output/server/index.mjs.

I had the same problem, and for me, it worked by removing the node_modules and .nuxt folders, and then running yarn -i.


Could you please improve the support of local language ?

Hi,
In Vulcan, we can set locale to (fr, en, ru, etc.) in our settings.
It seems that the current version 1.8.9 doesn't support the local in:
moment date, we have to add the support to the local language manually
view sort, the sort use only the default "en" and not use collation( { locale: "fr" } ) if we have set our locale to "fr".
Thank You
Adalidda

What are the specific steps we should take to improve language support?

I suggest that
Step 1: Recommend to the language translators to use the local setting based on mongodb collation locale https://docs.mongodb.com/manual/reference/collation-locales-defaults/#collation-languages-locales
examples:
fr for French
fr_CA for French Canada
etc...
This is important because the local that mongodb used is not always the local that other software used ! Example for Khmer language, usually we use kh and not km !
Step 2: Add support for MongoDB 3.4 collation function, in all mongodb operation, based on the local language settings
https://docs.mongodb.com/manual/reference/collation/#collation-document-fields
Step 3: Make sure that Vulcan use moment with the local setting
Thank You
Adalidda

Oh so are you talking about internationalizing the actual documents stored in Mongo? Or something else?
And for moment, yeah that would be good. A lot of people have been migrating away from Moment recently though so maybe there's a better library we can use?


How can i convert the segment output from json file to mark the image?

I got the JSON output file. I found the counts field that is the marking information but its encode to ASCII code string (I think that from eval.py code.).
How can I convert the count's field to binary or NumPy type to mark in my image?
This is my sample output.

@beebrain can you tell me how you extracted the output mask polygon coordinates. I just want to extraxt the polygon/contour points of the detected/masked object from the output.

@beebrain
import cv2
import numpy as np
from pycocotools import mask

segmentation= {}
segmentation["counts"] = "gR_>3..."
segmentation["size"] = [1080, 1920]

segmentation= mask.decode(segmentation)

Hi @beebrain
I tried the approach mentioned by @PL-96, but got a blank image as output.
Did it work for you?

我得到了 JSON 输出文件。我发现 counts 字段是标记信息，但它编码为 ASCII 代码字符串（我认为来自 eval.py 代码。）。 如何将计数字段转换为二进制或 NumPy 类型以在我的图像中进行标记？
这是我的示例输出。

hii, how did you get this json datei

I got the JSON output file. I found the counts field that is the marking information but its encode to ASCII code string (I think that from eval.py code.). How can I convert the count's field to binary or NumPy type to mark in my image?
This is my sample output.

hii, how did you get this json file? can you share the code oder others.


Fix TS type definition

Typescript type definitions are wrong and unusable. I fixed them and added defaults variable.

@gstamac Thanks for the PR. Merged and just published an update.

@mauricedb great thanks. That was quick. I have some other ideas if you are willing to accept PRs.

@gstamac 🔥 Sure always happy to accept PR's. But if it's more than trivial I would recommend opening an issue first so we can discuss if it is something I would like to accept. Just to avoid doing a lot of work with the possibility of me wanting to go another direction,

Done.


Not sure how -d is supposed to work

Could you give an example in the readme or --help on how to use the -d argument?   I tried this but got an error:
./simplify -d ADD2S="(2+2)" "ADD2S + 4"

and got this error:
simplify: expected an expression in the form of VARIABLE=EXPRESSION

Probably because there's a number in the variable name.   So I removed that and got this:
/simplify -d ADD="(2+2)" "ADD + 4"

And got this:
2 + 2 = 4
2 + 2 + 4

So I guess the first one evaluates the variable expression and the second is just a replacement/simplification of the last expression.
Is that right?

It definitely shouldn't be printing that first line. I when was the last time you pulled the repo? I think this was just fixed a few hours ago commit d75eb1bdab62e24dca69011168e12ba5345ab8ab


lookup failed error

I get a lookup error in inventory and ship vendor some come from a string folder some is something else not yet found it seems

Details
weapon inventory, and ship vendor at new Atlantis with 132 esm mods the strings folder does cause some mods to not work correctly also so might be broken files that didn't update when Bethesda updated starfield
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
just playing the game it and it pops up in the dialog and inventory boxes

This is never an issue with the mod itself. It will be an issue on your end. With 132 esm files, you are almost certainly over the master index limit.

wait 132 is over the limit i had more mods in fallout 4 and that is including 7 that look like they are normal starfield files now that I look at them and it is running fine just the occasional look up failed error?


Mm 10601 - Use Enter Key to send message on Hardware Keyboards

Summary
Capturing Enter and Shift-Enter events on hardware keyboards so users can use it to add a new line  (Shift-Enter) or send the message draft (Enter).
These type of keyboard events can only be listened at a native level so most of the changes should be performed there. To reduce the amount of native code modified, I used an external library to handle the native event once it is captured.
I need some help understanding how you guys would place the native changes as I understand the AppDelegate and MainActivity are not meant for this kind of code but I was not sure how you guys would like to have them put.
Also, I was not able to test this on an Android as I don't own a hardware keyboard so it would be great if anyone could help me doing some QA there.
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/8797
Jira Ticket https://mattermost.atlassian.net/browse/MM-10601
Checklist
N/A
Device Information
This PR was tested on: iPhone XS, iPad (3rd generation)
Screenshots
N/A

One other issue: I don't have an external keyboard to test on Android, but when testing on an emulator pressing shift+enter submits the post.

Yup, I had the same issue. I read simulators won't send any modifier when pressing the enter key but it would work on real hardware keyboards. That's why I asked for help testing this on Android but if you don't own an external keyboard I'll order one so this can be properly QAed.

Ah, so nice to be able to hit the enter key to submit a post! I noticed one issue while testing on an iPad, though. Try the following:

Tap a message to open the Thread screen
Go back to the Channel screen
Attempt to post a message by tapping the Enter key

True, I'm sending a fix for this right now

Aside from the comments below, I'm wondering why NativeEventEmitter isn't used for Android?
I guess I was so desperate trying out ways of testing Android without a hardware keyboard I tried using DeviceEventEmitter and forgot to move back to NativeEventEmitter 😅

I changed the module back to using NativeEventEmitter on both platforms.

I managed to get a Bluetooth keyboard to test with a Galaxy S7 and I found that event.getAction() is always ACTION_UP :( I wonder if this will behave different on a tablet meant to be used with a keyboard accessory? I'll see if I can order one.

If this this behaviour is consistent on every keyboard we could just switch from KeyEvent.ACTION_DOWN to KeyEvent.ACTION_UP on dispatchKeyEvent. But yeah, it would be great if we could test on more keyboards to make sure that's the case.

Also is there a reason why the event handlers are being registered at the AppDelegate and MainApplication level instead of a component level? maybe extending the TextInput as we have done in the past? 0/5 on this, just want to understand the reasoning.

I put it in the most generic level I could because the I think the app could use other keyboard shortcuts in the future (e.g. I use Cmd + Shift + K all the time in slack to open the Direct Message browse window) so having all these shortcuts together on a generic level made sense for me. But you know the app much better than me so I'm 100% open to ideas.

You can follow the examples on this same file on how to do it as your lib does not have a ViewManager

I tried following the other examples but I always end up getting a java.lang.NullPointerException when trying to call HWKeyboardEventModule.keyPressed(java.lang.String) because HWKeyboardEventModule hasn't been initialised yet and therefore the instance from the Singleton is not there. Is it possible we cannot use TurboReactPackage for modules which are used in the MainActivity as the instance won't be there?

0/5 if we leave it without turboModule

@emilioicai Please resolve conflicts

@emilioicai Please resolve conflicts

Done!

@migbot I already tested this before. I don't think we're waiting on anything other than fixing the conflicts

And @saturninoabril needs to approve I guess

@josephbaylon conflicts should be resolved now

@josephbaylon Nope, I'm all good with your QA review.


how to truncate table?

how to delete all data in a table

To delete all data from the table you need to create a next @Query request:
@Query('DELETE FROM task')
Future<void> clearTable();

But note that Query actions doesn't trigger a change update.

Thank you！


Fix/add url, add some other config

Fixed the git-repository-url and added edit-url-template so they point to the book source. Added some config for the playground and search params.

LGTM. Thanks!


Implement redirect capability

NOTE: The syntax described below was not used in the final implementation.
This is my initial attempt to implement the ability to perform redirections (see #87). I decided to indicate a redirect by prefixing the destination with the HTTP status code to be used for redirection, followed by a pipe, followed by the specific URL you want to redirect to. There is no built-in protection against creating redirect loops currently. Here is how you might use it:
route add svc example.com:80/ 302|https://example.com
route add svc example.com:443/ <IP_ADDRESS>:5678

Note that these include explicit source ports. To use it on consul services, you have to use multiple service tags.
urlprefix-example.com:80/ 302|https://example.com
urlprefix-example.com:443/

You can create redirect routes to anywhere.
route add svc example.com/dothething/ 301|https://github.com/fabiolb/fabio

Looking for feedback.
(Signed CLA)

Thx a lot. A number of users have been asking for this. I’ll have a look tonight.

Yes, we were just discussing doing this last week but it looks like you beat us to it.  Thank you, this is one of the few things we're still using HAProxy for on the frontend.

Happy to help! I hope the syntax for indicating a redirect is acceptable. It's not even close to what was discussed in #87 but it enables you to redirect to any given URL which I find valuable.
Also what's up with travis? The failure doesn't seem legit.

Looks like golang tip and the Vault checks aren't getting along.

lets disable tip for now then. I've got some cycles in the coming weeks to clean out the issue list. So lets get this done and merged.

This is doing the right thing in the http_proxy.go but the config implementation is more complex than it needs to be IMO. It should be enough if you add a redirect=<code> option since the syntax parsing is mostly there. The notable exception is https://github.com/fabiolb/fabio/blob/master/registry/consul/service.go#L120-L131 which you have covered already and which could use some tests (cough). Then you don't need an addRedirectTarget method and can just add that one option to addTarget method.
I assume you need to make the changes for matching the ports in order to do the http -> https redirect. I'm wondering whether this wouldn't be more expressive
urlprefix-example.com/ https://example.com proto=http redirect=302
urlprefix-example.com/

I'd also like to see an integration test in https://github.com/fabiolb/fabio/blob/master/proxy/http_integration_test.go

It may also make sense to get #385 fixed first. I have the patch but need to add the test.

Ok I'll see what I can get done on this tonight. Understood regarding tests.. will add those too.

I assume you need to make the changes for matching the ports in order to do the http -> https redirect.

Correct. This makes it possible to define a route explicitly on http but not https (and vice versa), which was required for this feature.
My understanding is that the proto= option tells fabio what protocol to use for comms with upstream. I wouldn't recommend using it to define route-matching behavior.

The proto= option means whatever we want it to mean. It can mean different things in different contexts if its meaning can be unambiguously derived from the context. Lets get this down to the minimal and tested change first and then we can polish it.

I've just merged #388 so you want to rebase.

I implemented your suggestions, minus the proto=http one.
In writing the integration test I discovered some issues that I'm not comfortable resolving without feedback. The integration test fails... have a look at it and I think you'll see the problem. It's copying the Path part of the request URL over to the target. In many cases this will be desired, but not always. I am considering adding another opt like absolute=true. Thoughts?

Got the kids today. So this might be tomorrow.

You missed one spot:
route/table_test.go:480:48: cannot use tt.req (type *http.Request) as type bool in argument to normalizeHost

I made the requested changes but we're not done yet unfortunately. As I mentioned in an earlier comment the integration test fails due to the way the targetURL is constructed in http_proxy.go. With
route add mock  /foo   http://a.com/abc   opts "redirect=301", we get redirected to http://a.com/foo. So I was considering implementing some way to indicate you want the target URL to be absolute, in case this behavior is not desired. Can you comment on that?

Yeah, I'm not sure that matters since the target URL should always be absolute. There is no path mangling in fabio since the routing table is usually generated and the http://host:port/ is really just host:port. Originally, I wanted to keep this open and used URLs for that but this may be misleading. So in fabio you can't forward /foo to /bar/foo. This isn't supported and so far not many have complained about that.

I think the other thing that bugs me a bit but for which I don't have a good answer right now is the continued divergence between the syntax of the urlprefix- tags and the route commands.
urlprefix-/ redirect=302,http://www.google.com/

# translates to
route add svc / http://www.google.com/ opts "redirect=302"

I need to think about this a bit more.

That isn't something that I'd want to address in this PR anyway but I'd like to keep in mind. One approach could be to write a template instead of a special syntax.
fabio <templated route add command>

e.g.

fabio route add $svc / $addr opts "redirect=302"

Then you'd have to learn only one syntax.

I'm not so sure we always want it to be absolute. Suppose you have these routes
route add svc example.com:80 / https://example.com redirect=302
route add svc example.com:443 / <IP_ADDRESS>:5678

If you hit http://example.com/some/deep/link you get redirected to https://example.com/some/deep/link. This seems very useful, but I can also see how it might not be desired, depending on the purpose of your redirect route.

How do other load balancers address that?

Don't know.. I'll do a bit of research.

Thx. That'd be nice. It's getting a bit late here in NL. Going to hit the pillow since the kids get up early. We're getting there.

I can comment on how it's done with HAProxy where we do our http to https redirects.
    redirect scheme https code 301 if { hdr(Host) -i my.ena.com } !{ ssl_fc }

We have that syntax in our main frontend that binds on both 80 and 443 and redirects any request coming in with a Host header of  my.ena.com to the exact same URI but using the https scheme.  That means if I try to access http://my.ena.com/foo/bar/thing.html I will get a 301 redirect to https://my.ena.com/foo/bar/thing.html with the above configuration.

We could add some magic with a terminating slash or not. redirect=301,http://foo.com would append the path from the original request and redirect=301,http://foo.com/bar would redirect to /bar. That wouldn't solve the path mangling if you want to redirect /foo to /foo/bar or /bar/foo. Another approach could be to add a pseudo-variable like $path at the end. This would give you full control over the redirect target.
request uri: http://bar.com/foo

redirect=301,http://foo.com$path -> http://foo.com/foo
redirect=301,http://foo.com/ -> http://foo.com/
redirect=301,http://foo.com/bar$path -> http://foo.com/bar/foo
redirect=301,http://foo.com/bar/$path -> http://foo.com/bar/foo # ignore double slashes

The pseudo variable would be very flexible and is also less ambiguous than magic based on trading vs non-trailing slash paths.

I really like the pseudo-variable idea

Pushed changes. Have a close look at http_proxy because I rearranged some code in there since I am no longer using the targetURL var as the redirect target.

@ctlajoie I will. This looks decent so far but I'll let it bake for a couple more days since I want to think a bit more about edge cases. If me or you can't think of anything major then I'll merge this early next week (Mon or Tue). Sounds good?

Sounds good. I'm actually using it right now in a semi-production environment (accessed by people working in my office) so it's getting some real-world testing.

Even better.

I've merged this to master. I had to update the demo server to handle the comma in the redirect option but that was a good time to make this change anyway.
Thanks a lot for this. This has been a long requested feature.

@magiconair You might want to know that there was/is a problem with current versions of nomad/consul where if you have :// in a service tag, nomad is unable to deregister that service. See https://github.com/hashicorp/nomad/issues/3620
Should be fixed in the next nomad release.

@ctlajoie Thanks. That's good to know.


Error: " Build Warning: Layout 'default' requested in ... does not exist."

When I build my site, I'm getting all kinds of warning errors about default.html layout not found:
 Generating...
              Lunr: Creating search index...
     Build Warning: Layout 'default' requested in content/adtruth/archive.html does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/archive_getting_started.html does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/demo.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/demo_environment.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/faqs.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/hello_world.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/identifiers.md does not exist.
     Build Warning: Layout 'default' requested in index.html does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/make_requests.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/matchdevice.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/overview.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/questions.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/release_notes.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/rulesets.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/sample_scenario.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/scrollspy.md does not exist.
     Build Warning: Layout 'default' requested in search.html does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/troubleshooting_hmac.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/use_cases.md does not exist.
     Build Warning: Layout 'default' requested in content/adtruth/windows.md does not exist.
     Build Warning: Layout 'post' requested in _posts/2014-06-28-example-bootstrap.md does not exist.
     Build Warning: Layout 'post' requested in _posts/2014-07-27-exmaple-css.md does not exist.
     Build Warning: Layout 'post' requested in _posts/2014-07-28-introducing-shiori.md does not exist.
              Lunr: Index ready (lunr.js v0.4.5)

Removing the search plugin made these errors go away. Do you know why the errors are happening? They didn't occur in my plain, simple search integration site.

@tomjohnson1492 ,check this : #34 ,https://github.com/slashdotdash/jekyll-lunr-js-search/commit/11156ef5aa3170a13a1415d045fae1fe56c44e4d

Thanks, but those threads didn't help much. #34 is inconclusive. 11156ef is vague. Am I supposed to make those changes in the plugin source files and rebuild it or something?

@tomjohnson1492,yes change source as #11156ef,then jekyll build,jekyll server.

I commented in #34 ... I was able to fix it using the suggested change, but not sure if it works with the current version of the plugin. Commenting here also because this thread lead me to the fix.

I've just published an update to the plugin and gem that should fix this error. When extracting the content of documents for the search index the layout is unset.
gem install jekyll-lunr-js-search -v3.0.0

Hi, sorry to say that I observe the issue with jekyll 3.0 beta10 in combination with jekyll-lunr-js-search 3.0.0 too.
I have not worked on earlier jekyll versions, so I cannot tell differences.
Not sure if the following is the stack trace you need, but that's what I get (removed the absolute paths):
vendor/bundle/ruby/2.2.0/gems/json-1.8.3/lib/json/common.rb:155:in `parse': nesting of 151 is too deep (JSON::NestingError)
        from vendor/bundle/ruby/2.2.0/gems/json-1.8.3/lib/json/common.rb:155:in `parse'
        from vendor/bundle/ruby/2.2.0/gems/jekyll-lunr-js-search-3.0.0/lib/jekyll_lunr_js_search/javascript.rb:10:in `to_hash'
        from vendor/bundle/ruby/2.2.0/gems/jekyll-lunr-js-search-3.0.0/lib/jekyll_lunr_js_search/indexer.rb:90:in `generate'
        from vendor/bundle/ruby/2.2.0/gems/jekyll-3.0.0.pre.beta10/lib/jekyll/site.rb:155:in `block in generate'

ERROR:  Error installing jekyll-lunr-js-search:
ERROR: Failed to build gem native extension.
/usr/bin/ruby2.1 extconf.rb

mkmf.rb can't find header files for ruby at /usr/lib/ruby/include/ruby.h
extconf failed, exit code 1
Gem files will remain installed in /var/lib/gems/2.1.0/gems/nokogiri-1.6.8 for inspection.
Results logged to /var/lib/gems/2.1.0/extensions/x86_64-linux/2.1.0/nokogiri-1.6.8/gem_make.out

Still seeing this in jekyll-lunr-js-search 3.2.0.

I was having the error with 3.2.0, jekyll 3.3.0. This made the warning go away for me:
Change this line
from item.data.delete("layout")
to item.data["layout"] = nil

@bjgordon @slashdotdash  the item.data["layout"] = nil variant is the correct and working one for all users since you need  leave an explicit value for the layout to prevent jekyll from falling back to the default layout configured in the front matter defaults in _config.yml
The current approach with removing the property completely was probably not tested in a project that uses a default layout in front matter defaults.

I've just merged pull request #109 with that change.

Is this in the latest release? I'm seeing this in 3.3.0.


VSO Agent | OAuth  issue

Hi,
I am using VSO agent (installed in CentOS Lx) inorder to build my code from VSTS environment. When I initiate a build, I get the following error in the console. I have never configured OAuth , how to get rid of this , is there any way that I can specify my agent to work with basic authentications (i.e. username/PAT ) . I have tried to change the VSO_GIT_USERNAME/PASSWORD variables but didnt help , can someone advice.
I can clone the git repo using the same usercredentials on test machines , so this is to say my PAT cred's are working fine., appreciate your advice.

cwd: /home/******Admin/******agent/_work
 valid scm providers: git,github,svn,tfsgit,tfsversioncontrol
Repository type: TfsGit
using source provider: tfsgit
loading: /home/******Admin/******agent/agent/scm/tfsgit
Using auth scheme: OAuth
running clean
 skipping clean since repo does not exist
getting code
https://************.visualstudio.com/DefaultCollection/******/_git/products
srcVersion: 02830b18779dd35bac74**
srcBranch: refs/heads/***-master
Using ref: 02830b18779dd35bac74**
/usr/bin/git arg: clone
/usr/bin/git arg<EMAIL_ADDRESS>/usr/bin/git arg: --progress
/usr/bin/git arg: s
exec tool: /usr/bin/git
Arguments:
clone
<EMAIL_ADDRESS>--progress
s
Cloning into 's'...
fatal: Authentication failed for<EMAIL_ADDRESS>rc:128
success:false
/usr/bin/git failed with return code: 128

Use this agent:
https://github.com/Microsoft/vsts-agent
Specifically, see: https://github.com/Microsoft/vsts-agent/blob/master/docs/preview/latebreaking.md


Save to disk broken after resource split

fixes #5
As the storage currently takes a list of dicts I did it on an index basis, might be cleaner to change the storage to take a dictionary instead.

Dict does seem to be cleaner and more future proof to allow adding/omittin other elements, can you change?

I aggree but will have to wait till tomorrow.

The in-between solution, doesn't change the storage, but uses a dictionary.

Excellent, thanks!


Make requestId usage consistent everywhere in the docs

Incorrect:
Request ID
RequestID
Request Id
Correct:
requestId

Fixed


Support discovery of arbitrary Flutter instances

Background
Flutter is easily embeddable in a variety of platforms and environments. While the tool will still build out first class support for several platforms, there are a growing number of alternatives being developed as well. Ideally a developer that targets an alternative embedding would be able to benefit from as much of the existing flutter tooling as possible. This would both improve the usability of Flutter in different environments, and make it easier to stand-up new environment's tooling.
Originally this was proposed via a Dart API in the flutter tool. I investigated this approach in 2019 and found that problem had a much larger scope than we had resources to work on. It required defining and stabilizing APIs for building, installation, and doctor checks, which would be both difficult the design given the lack of current usage and difficult to change in the future if usage increased. Without ruling out this approach in the future, there are some incremental steps we could take to improve usability.
Arbitrary embedder discovery
flutter attach almost supports discovery of arbitrary embedders, and with a few adjustments could become generally usable for this.
Using mDNS, the tool can discover Dart VMs running on iOS devices, but there is no reason we could not generalize this to more platforms. The requirement for the embedder side would be to either advertise via the Dart VM automatically, or otherwise manually implement some variety of debug-only broadcast.
caveat: There were issues getting mDNS to work on Android devices, so this may be limited in other ways we're not aware of. FYI @dnfield
The tool could then easily attach to these instances and support feature discovery to determine build mode and dart platform (dart v fuchsia)
IDE integration
Currently Intellij and vscode have a drop-down selector for devices that can be targeted for flutter run. We could similarly implement an "instance dropdown" for Android, iOS, and other instances that could be attached to.

mDNS assumes that the Flutter instance is network discoverable from the host machine. That's not always the case, even on iOS (e.g. some combination of enabling hotspot or wifi or mobile data breaks it, as do some network configurations that are out of the user's control).
I think the biggest challenging to making this work is getting a stable, reliable interface to advertize and discover on. mDNS makes sense as long as we can have that. Alternatively, the biggest challenge is detecting and helping users who do not have the right kind of interfacing to make the connection work automagically.

The iOS case is particularly complicated. I would draw a line here and say that it would be the responsibility of the embedder developer (us in the case of iOS, ??? in the case of an arbitrary embedder) to ensure that their own tooling can setup in such a way that the network accessibility is reasonably stable.
In the cases today, the custom embedders are for desktop development so there should be no issues, though this might change in the future.

There's probably work that can be done on the embedder side to detect known-bad configurations, although I'm not sure how brittle that would be.
I don't think we can detect whether the network we're on just blocks peer connections (some WiFi networks do this to increase security or decrease traffic on the network).
The challenge will be on some platforms, it just plain might not be possible. E.g. Android connected via cable does not create a network bridge to the host, and even Android emulators do not support multicast packets to the host OS.

Why would we need to detect whether the network we're on blocks peer connections?

To be clear, no part of this proposal involves changing the current Android or other 1st party supported device's discovery implementations.

Would it be an idea to initially support this by exposing observatory URIs in the flutter run command?
In flutter_tools/resident_runner.dart#L199 we know the URI of the started app and thus the tool could expose this over mDNS.
This would at least give us the option to find/attach to 'local' applications started via flutter run (which will usually be in debug mode)


Add support for vimdoc

I've added Vimdoc support, the implementation is very similar to that of Norg's. I included headings and tags in the outline, just headings provide very little information.
There seem to be quite a lot of edge cases that could be improved. For example, sometimes there are empty headings or headings that are not actual document headings but table headings. Sometimes there are multiple tags in one line, sometimes in multiple lines (but they refer to the same thing). The formatting seems quite inconsistent, even in the builtin documentation (e.g. the main page marks modeline as heading, other pages seem to get it right...). We could come up with some heuristics for the most common cases, for now I ignored all that.
Example below.

I used Variable kind for tags so that they can be easily filtered out.
I've encountered on issue, not sure if my change caused it. When there is a window with help file open and another with, for example, a lua file then the outline sidebar will show outline for the lua file always. For some reason the outline for help files seems to work only for one file and if there is another buffer in the same tab it will take over. Even switching between two help files won't work, the first one will be always shown in the sidebar. Other filetypes seem to work fine. I can try to debug that, but maybe someone has an idea what could be the reason?
Resolves: #34

Thanks for this! I'll take a look at it soon.

I've encountered on issue, not sure if my change caused it. When there is a window with help file open and another with, for example, a lua file then the outline sidebar will show outline for the lua file always. For some reason the outline for help files seems to work only for one file and if there is another buffer in the same tab it will take over. Even switching between two help files won't work, the first one will be always shown in the sidebar. Other filetypes seem to work fine. I can try to debug that, but maybe someone has an idea what could be the reason?

Currently outline looks for buflisted option to determine whether the symbols should be refreshed when we switch to a new buffer.
https://github.com/hedyhli/outline.nvim/blob/028e0bc8ad3d511e9fc1f0678594a608eb908ee5/lua/outline/sidebar.lua#L341-L345
This check is only done on "refresh", ie, when the cursor moves or text changed (governed by outline_items.auto_update_events.items), which means it prevents the outline items from updating when the user navigates from the code to things like floating windows (eg, :LspStatus).
I believe it makes sense to special case the help ft here (for instance, only ignore refresh if buflisted is false and ft is not help), now that vimdocs are now supported in outline.nvim.

Thanks, changing this one line fixed the issue.
I got rid of "OutlineHelp" from the condition, because outline help buffer is created with 'buflisted' set to false.

Thanks for your thorough testing -- I didn't encounter these issues during mine. What are some help pages you used that can reproduce this issue?

These are not big issues to be honest and maybe not as common as I initially thought, here are some examples.
Heading with no text (except for the tag):

Three tags in the same line:

Two tags referring to the same thing in multiple lines:

Modeline is a header:

(you can see the help file name in the top left corner)
The examples with tags are not necessarily issues with the help file formatting. It's just that they take a lot of lines in the sidebar. Not a big deal probably.

Here is an example where some tags for a heading are in the line before that heading:


[BUG]: Search bar cursor position

Description
The cursor position of the search input isn't in the right place when the placeholder moves up
Actual result

The solution to this issue will already be included in the pr of the issue #1332