Extension fails to activate unnamed conda environments

Description:
Extension is unable to activate unnamed conda environments created using the conda create -p command after the latest update. Previously, these environments were activated correctly. No changes were made to the environments themselves, only VS Code and the Python extension were updated. Named conda environments continue to activate without issue.
Steps to reproduce:

Create an unnamed conda environment using the following command:conda create -p ./env

Open a project folder in VS Code that uses this unnamed environment.
Attempt to select the environment as the Python interpreter for the project using the Python extension.

Expected behavior:
The Python extension should successfully activate the unnamed conda environment and correctly load its packages, as it did before the update.
Actual behavior:
The Python extension fails to activate the environment and displays an error message, indicating "EnvironmentLocationNotFound". The wrong Python interpreter may be selected, leading to issues with code execution and package resolution.
Logs:
2024-09-12 01:04:26.645 [error] getActivatedEnvironmentVariables Error: Command failed: conda run -n env --no-capture-output python /home/<USER>/.vscode-server/extensions/ms-python.python-2024.14.0-linux-x64/python_files/get_output_via_markers.py /home/<USER>/.vscode-server/extensions/ms-python.python-2024.14.0-linux-x64/python_files/printEnvVariables.py

EnvironmentLocationNotFound: Not a conda environment: /home/<USER>/miniconda3/envs/env

    at genericNodeError (node:internal/errors:984:15)
    at wrappedFn (node:internal/errors:538:14)
    at ChildProcess.exithandler (node:child_process:423:12)
    at ChildProcess.emit (node:events:531:35)
    at maybeClose (node:internal/child_process:1105:16)
    at Socket.<anonymous> (node:internal/child_process:457:11)
    at Socket.emit (node:events:519:28)
    at Pipe.<anonymous> (node:net:338:12) {
  code: 1,
  killed: false,
  signal: null,
  cmd: 'conda run -n env --no-capture-output python /home/<USER>/.vscode-server/extensions/ms-python.python-2024.14.0-linux-x64/python_files/get_output_via_markers.py /home/<USER>/.vscode-server/extensions/ms-python.python-2024.14.0-linux-x64/python_files/printEnvVariables.py'
}

Environment:

VS Code version: Version: 1.93.0 (user setup)
Python extension version: v2024.14.0
Conda version: 24.7.1
Operating System: Ubuntu 20.04.3 LTS in WSL 2

Additional information:

This issue only occurs with unnamed conda environments. Named environments activate correctly.
The issue appeared after updating VS Code and the Python extension today. Previously, unnamed environments worked as expected.
It seems the problem might lie in the conda run command used by the extension. The logs show it's attempting to activate an environment named "env", but the unnamed environment doesn't have a name and is located at "./env", not "/home/USER/miniconda3/envs/env".

I have tried restarting VS Code and reinstalling the Python extension, but the issue persists.
Please investigate this issue and provide a fix to restore the ability to activate unnamed conda environments within the Python extension.

Hi @F1bos Thanks for filing this issue.
Can you try adding:
"python.experiments.optOutFrom": ["pythonTerminalEnvVarActivation"],

to your settings.json and see if that works?
Thanks

Hi @F1bos Thanks for filing this issue. Can you try adding:
"python.experiments.optOutFrom": ["pythonTerminalEnvVarActivation"],

to your settings.json and see if that works? Thanks

This is what i have in Python extension logs:
2024-09-18 00:33:41.208 [info] Experiment 'pythonTerminalEnvVarActivation' is inactive
2024-09-18 00:33:41.208 [info] Experiment 'pythonRecommendTensorboardExt' is active
2024-09-18 00:33:41.208 [info] Experiment 'pythonTestAdapter' is active
2024-09-18 00:33:41.208 [info] Native locator: Refresh started
2024-09-18 00:33:41.208 [info] Default formatter is set to ms-python.black-formatter for workspace /home/user/projects/some_project
2024-09-18 00:33:42.549 [info] > pyenv which python
2024-09-18 00:33:42.549 [info] cwd: .
2024-09-18 00:33:42.567 [info] Conda environment manager found at: /home/user/miniconda3/bin/conda
2024-09-18 00:33:42.613 [info] Native locator: Refresh finished in 1548 ms
2024-09-18 00:33:42.627 [info] Python interpreter path: ./env/bin/python
2024-09-18 00:33:45.256 [info] Starting Pylance language server.
2024-09-18 00:33:45.268 [info] > ~/miniconda3/bin/conda info --json
2024-09-18 00:34:00.532 [info] Send text to terminal: conda activate env

This is what i got in terminal:
conda activate env
(miniconda3)user@DESKTOP:~/projects/some_project$ conda activate env

EnvironmentNameNotFound: Could not find conda environment: env
You can list all discoverable environments with `conda info --envs`.

I get the same issue running v2024.14.1 on Windows 11.
It appears that the folder name containing the unnamed environment is now being passed to the conda activate command in isolation. For example, if the environment is created in a subfolder (e.g. conda create -p ./local/env), then the command run in the terminal is still conda activate env rather than the expected conda activate ./local/env.

Can you try setting the setting for "native locator" to js instead of native?

That fixed the issue for me @anthonykim1. It now runs conda activate <absolute path to environment>.


Front endtweeks

Fixes #140
Fixes #147
Fixes #142
Fixes #146
Fixes #145
Fixes #141

Looks good to me


Add opts default value before usage

Super tiny bug fix:
Give the opts object a default value before trying to use it.

```TypeError: Cannot read property 'position' of undefined````

Notes:

[x] Update in source
[x] Run grunt

Thanks!

New release done. This is in 2.1.3. Thanks again!


Add more descriptive exceptions

We're currently handling a significant portion of our exceptions as AssertionError. This is bad practice because it means we end up with unit tests that look like this:
https://github.com/omisego/plasma-mvp/blob/cf479aa6a9ec5baf64b0ca64db521f657dfa78a4/tests/child_chain/test_child_chain.py#L45-L60
There's absolutely no way to determine if the AssertionError being caught in test#1 is actually the correct AssertionError. We should create better and more descriptive exceptions.
There's a few ways to do this properly:

Continue to use AssertionError, but attach messages to the assert statement and verify those messages (samples thanks to @boolafish):

Modify the assert:
assert valid_signature, 'invalid signature'

Modify the test:
with pytest.raises(AssertionError) as e:
    child_chain.submit_block(block)
assert str(e.value) == 'invalid signature'

Extend Exception with more descriptive errors, e.g.:

class InvalidBlockSignatureError(Exception):
    """Raise when a block is signed by anyone but the operator"""

I'm personally a fan of (2), but I couldn't find any sort of guide on which model to generally stick to.

as for reference: https://stackoverflow.com/questions/20076182/what-are-the-implications-of-using-assert-vs-raise-exception

Fixed in #62


Allow for non-comma separators

As it currently stands, a CSV file with a non-comma separator will return "no valid row found in the CSV file".  Given that Excel uses a semicolon as CSV separator by default when comma is used the decimal separator, as is the case in e.g. the default Dutch language settings, being able to parse such CSV files without having to replace all semicolons with commas would increase user friendliness, especially for less tech-savvy users.

We are aware of this Microsoft problem, but for now I would advise to use LibreOffice or Google Sheets. We will point this out in the wiki.
We are using na already existing npm module to parse csv's that only supports comma seperated values documents. Maybe in the near future, we could look how we could support the Excel CSV. (Excel is just not a good tool for creating CSV's).

I've done some playing around, and found that with csv-parse you can set the separator character via the "delimiter" option. This opens up the possibility of letting the user choose the separator, perhaps under the advanced options, with comma as the default value.


Dispatch a broadcast event all time fails when it is sends from a MessageReceived listener

Reverb Version
1.0.0-beta12
Laravel Version
11.9
PHP Version
8.2.8
Description
Hi everybody and sorry for my terrible English.
From the last week I started to work with Laravel Reverb and I'm getting an error that is making me crazy. At time to call an event dispatch function from a MessageReceived listener, I'm receiving this error message:
Pusher error: cURL error 28: Operation timed out after 30009 milliseconds with 0 bytes received (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for http://localhost:8080/apps/751596/events?auth_key=ygvwwztqu7pi8egjbq8b&auth_timestamp=1718031505&auth_version=1.0&body_md5=ce1a06ea08584ba2c0e0777a13c875ef&auth_signature=f7d413d1d1d7881ddbcce5a40f565022899d710c39ccd678b3e39214915f19a2

Steps To Reproduce

I created a new Laravel project and installed the following packages: laravel/reverb, mongodb/laravel-mongodb, php-open-source-saver/jwt-auth and laravel/sanctum.
Created a new broadcast event with the following code:

<?php

namespace App\Events;

use Illuminate\Broadcasting\InteractsWithSockets;
use Illuminate\Broadcasting\PresenceChannel;
use Illuminate\Contracts\Broadcasting\ShouldBroadcastNow;
use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Queue\SerializesModels;

class DbReplication implements ShouldBroadcastNow
{
    use Dispatchable, InteractsWithSockets, SerializesModels;

    /**
     * Create a new event instance.
     */
    public function __construct(
        private string $accountId,
        public string $message
    ) { }

    /**
     * Get the channels the event should broadcast on.
     *
     * @return PresenceChannel
     */
    public function broadcastOn(): PresenceChannel
    {
        return new PresenceChannel("DbReplication.{$this->accountId}");
    }
}

Created a new listener with the following code:

<?php

namespace App\Listeners;

use App\Events\DbReplication;
use Illuminate\Support\Str;
use Laravel\Reverb\Events\MessageReceived;

class DbReplicationListener
{
    /**
     * Create the event listener.
     */
    public function __construct()
    {
        //
    }

    /**
     * Handle the event.
     */
    public function handle(MessageReceived $event): void
    {
        $message = json_decode($event->message);
        if (Str::startsWith($message->channel ?? '', 'presence-DbReplication.'))
        {
            $accountId = (string) str($message->channel)->match('/\w+$/');
            // This line is throwing the error
            DbReplication::dispatch($accountId, 'hello world');
            /**
             * I tried use too:
             * DbReplication::broadcast($accountId, 'hello world')
             * broadcast(new DbReplication($accountId, 'hello world'))
             *
             * All are throwing the same error
             */
        }
    }
}

I registered the listener in App\Providers\AppServiceProvider class:

<?php

namespace App\Providers;

use App\Listeners\DbReplicationListener;
use Illuminate\Support\Facades\Event;
use Illuminate\Support\ServiceProvider;
use Laravel\Reverb\Events\MessageReceived;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Register any application services.
     */
    public function register(): void
    {
        //
    }

    /**
     * Bootstrap any application services.
     */
    public function boot(): void
    {
        Event::listen(
            MessageReceived::class,
            DbReplicationListener::class
        );
    }
}

And I added the broadcast channel using the following code:

<?php

use Illuminate\Support\Facades\Broadcast;

Broadcast::channel('App.Models.User.{id}', function ($user, $id) {
    return (int) $user->id === (int) $id;
});

Broadcast::channel('DbReplication.{account}', fn () => true);

I'm using the following environment variables for Laravel Reverb.
BROADCAST_CONNECTION=reverb

# I included pusher values because I'm not sure if this could produce the error
PUSHER_APP_ID=
PUSHER_APP_KEY=
PUSHER_APP_SECRET=
PUSHER_APP_CLUSTER=mt1

REVERB_APP_ID=751596
REVERB_APP_KEY=ygvwwztqu7pi8egjbq8b
REVERB_APP_SECRET=xg2sadjwdqynmuw6j3t1
REVERB_HOST="localhost"
REVERB_PORT=8080
REVERB_SCHEME=http

You'll need to queue the listener in order to prevent a deadlock.
The issue with broadcasting (without queuing) is that you are making an HTTP request to the WebSocket server, which can't accept the request until the current process is complete. Of course, the current process cannot complete as it's waiting on the HTTP request so you end up in deadlock.

It's not as pretty, but I found if your goal is to broadcast something immediately you can use the existing connection.
public function handle(object $event): void
{
    $response = json_encode([
        'channel' => 'hello',
        'event' => \App\Events\SomeEvent::class,
        'data' => json_encode([
            // ...
        ]),
    ]);

    $event->connection->send($response);
}

Thank you very much, I will try apply your code.

It's not as pretty, but I found if your goal is to broadcast something immediately you can use the existing connection.
public function handle(object $event): void
{
    $event->connection->send(json_encode([
        'channel' => 'test',
        'event' => \App\Events\Example::class,
        'data' => [
            // ...
        ],
    ]));
}

You are the best, I couldn't send a reply for a week


Move where the HttpSM setup for redirect before going to origin

This allows plugins like conf_remap.so to set these settings:
    @pparam=proxy.config.http.number_of_redirections=3 \
    @pparam=proxy.config.http.redirect_use_orig_cache_key=1

(as an example). Before this, you would need to set the enabled flag before
remap (in read-request-header hook). With this change, we can modify the
overridable configuratios in e.g. conf_remap, or any other remap plugin.
In addition, this removes proxy.config.http.redirection_enabled, and
instead setting proxy.config.http.number_of_redirections to 0 (default)
disables this feature. I also cleaned out the special case for the POST,
which ignores the max-retries setting. This was inconsistently done
anyways, and likely should be done via configuration / plugins.
This is an incompatible change, and should be properly documented.

Note that I initially considered moving this much later, but that's at the risk of breaking many existing plugins. I think these changes (albeit incompatible) are less risky, and achieves the desired goal (making conf_remap.so work with these configs).


feat(spanner): add samples for creating instances with processing units

Hi @dwsupplee, do you have any idea why the Lint check fails? It doesn't show me any error info.

My bad. ; is missing at the end. 😞

@dwsupplee Can you please take a review? Thanks!

@bshaffer Please take a look again! Thanks.

@bshaffer Gentle ping.


Avoid using promauto for metrics registration

Recently, https://github.com/grafana/dskit/pull/342 migrated packages from weaveworks/common to dskit.
One of the changes was to replaced the use of prometheus with promauto which in cases of downstream users who try to instantiate multiple instances of objects (eg. due to configuration reloads) triggers "duplicate metrics collector registration attempted" panics, for example here and here.
We should allow downstream consumers of packages to pass in their own Prometheus registerers for these metrics, or default to the global one if nil is passed.

Avoid using promauto for metrics registration

Let's use promauto.With(reg)... with supplied registrerer.

More detail required on why you think promauto is involved.
Most of the errors you point to don't even include dskit in the stack.


KafkaConnector1.0.0-beta.4Release

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


Failed to retreive the Service Connection - Sync-ElasticPool.ps1

Hi,
I have been hitting an issue lately, its related to the section to Register the scale set into ADO.
The service connection isnt getting picked up
The service connection running the task has full access to the ADO project&ORG as a build administrators so its not rights related...
Im trying to run the code locally to validate the underlying of the error but not much getting very far :\
Any ideas why its failing ? the rest is very great. Good work btw.

thanks

Holy moly,
hey @jessicalavoie. Please excuse the delay of merely a few months. The creation of the issue must have gotten lost under the flood of notifications from GitHub.
That being said - if the issue still persists (I hope note), then I'd ask you to double-check if the pipeline agent (i.e., the principal used by the pipeline) is configured so that it can ready the service connections as described here (Initial configuration / 1.1 (Optional) Configure the agent pool parameters & environment / 1.1.2 Configure the Azure DevOps environment to enable the pipeline to read & register the resources)
If you can confirm that the permissions are configured correctly, my next best test would be to create a dummy pipeline with a simple AzureCLI task and try to retrieve the service connections using an implementation like here.
Please let me know how it's going.

Hi Alexander,
not a problem for the delay, i dit it manually for the time being.
Im going to test this out again this week - if I remember correctly my
service connection has the full access required for the org-project in ADO.
Ill test with the dummy get-endpoint.ps1 see if i get any output and might
get back to you in any cases!
thanks a lot
Jessica
On Thu, Jul 4, 2024 at 3:28 PM Alexander Sehr @.***>
wrote:

Holy Molly,
hey @jessicalavoie https://github.com/jessicalavoie. Please excuse the
delay of merely a few months. The creation of the issue must have gotten
lost under the flood of notifications from GitHub.
That being said - if the issue still persists (I hope note), then I'd ask
you to double-check if the pipeline agent (i.e., the principal used by the
pipeline) is configured so that it can ready the service connections as
described here
https://github.com/Azure/DevOps-Self-Hosted/wiki/Self-hosted Azure DevOps Virtual Machine Scale Set Agents#initial-configuration:~:text=Enable the build service to list service connections
(Initial configuration / 1.1 (Optional) Configure the agent pool parameters
& environment / 1.1.2 Configure the Azure DevOps environment to enable the
pipeline to read & register the resources)
If you can confirm that the permissions are configured correctly, my next
best test would be to create a dummy pipeline with a simple AzureCLI task
and try to retrieve the service connections using an implementation like
here
https://github.com/Azure/DevOps-Self-Hosted/blob/main/constructs/azureDevOpsScaleSet/scripts/registerAgentPool/Get-Endpoint.ps1
.
Please let me know how it's going.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/DevOps-Self-Hosted/issues/36#issuecomment-2209485935,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABU45YHHMLQCJMW5BOVC6PDZKWO7PAVCNFSM6AAAAABG6PIGXCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBZGQ4DKOJTGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

On that note (and I haven't tested it yet, so please take it with a grain of salt), there seems to be a new Resource Type on the Horizon: DevOps Infrastructure Pool.
A colleague is currently working on an implementation for the Azure Verified Modules, to which I also aim to contribute the image building construct (at least to an extend). Ideally, the two will complement each other and it will simplify the 2 constructs in this repository quite a bit. Nothing to leverage yet, but a heads up in case it's of interest to you 😉

Hey @jessicalavoie,
if of any interest, please note that I just release a new major version of the IP that not only upgrades the codebase to AVM, but also switches the Agent-Pool implementation from 'VMSS + custom code to register it as an agent pool' to the new 'Managed DevOps Pool' service with has a different, better approach towards the registration.

PS: While refactoring the solution I was wondering if maybe this parameter caused you the headache: https://github.com/Azure/DevOps-Self-Hosted/blob/f9ee6fd753f49642179cf86965b359827fc93733/constructs/azureDevOpsScaleSet/scripts/misc/REST.jsonc#L5
I've recently worked more with other Enterprise Applications and can't help but suspect this GUID is tenant-specific (i.e., different in your tenant) and would need to be updated too.

Hey @AlexanderSehr ! i wasn't aware of the Managed DevOps pool in preview! will definitely keep an eyes on this ;)
Might return on this project in the upcoming weeks, so ill pull the new file you mentioned about the PrincipalId and will retest the pool deployment process to see how it behave!
Thanks a lot for the information's :)
Jessica

Sounds good. Let me know how it is going when you get the chance 💪


VMAF option for log_path not working in ab-av1

When adding the option '--vmaf log_fmt=json --vmaf log_path=vmaf.json', the output file does not get written. This appears to be caused by the method used for monitoring the progress with tokio_process_stream. I haven't been able to figure out how to get this to work while being able to monitor the progress via the progress bar. In order to implement a clumsy fix, you can place a simple sleep ahead of line 83 in src/command/vmaf.rs that is longer than the total time for the calculation. This results in the output file being written as expected.
I've been modifying my own fork recently to add an SSIM calculation option. When running it, I get an output file whether running directly through ffmpeg CLI or via ab-av1. This appears to work because SSIM writes to the file as each frame is processed. With libvmaf, they output the file after all of the calculations have finished. I tried placing a sleep after the calculations finished, but that doesn't work. I'm probably not understanding how tokio is working on the process. Seems like it gets killed before it can write the file. Maybe there is a simple way to keep the process alive long enough to get the output file?
I wanted to add some of the capability of looking at more than just the mean VMAF as well as other metrics. This capability is necessary in order to get those calculations. I wanted to include some of these metrics in the crf-search to help fine tune it for more precise quality metric searches.

I've found a way to make this work. By adding a 1 second delay prior to bar.finish() in this same file. This does seem like a bit of a crude way to make it work, but I'm not sure how else to get it to wait for the file to complete being written. I will submit a pull request with this update. If you think it's worthwhile, you can accept it or if you don't, reject it.
This solution solves #168. It just requires that the user supply the requisite log file output switches, as mentioned above.


Validate Contact Form

Portfolio Milestone: Validate Contact Form
For this milestone I:
✅ Used Gitflow
✅ Used linters
✅ Used basic JavaScript syntax
✅ Used JavaScript to manipulate DOM elements
✅ Used JavaScript events
✅ Used objects to store and access data
✅ Processed user input according to business rules
✅ Used client-side validation to catch and throw errors in the UI

Thanks for your code review and feedback @PinkMoon25! I'm doing my best.


autocompletion for imports in emacs still unimplemented/unfixed?

related to #332

nsf commented on Mar 9, 2016
It definitely works with godit and with vim out of the box. My own sublime text 3 plugin fails to handle it properly. Because it ignores returned partial length and just goes back to last word. Will see if I can fix it. Also in emacs company-mode doesn't work at all in string literals for some reason.
Oh, I see: https://github.com/nsf/gocode/blob/master/emacs-company/company-go.el#L191
I guess we'll have to remove that line maybe one day.

was this ever implemented or is it still unavailable for emacs?

Eh.. sorry for delayed response. Yes, it was never implemented. Not even planning to.


Can not run the example app on Android device

I'm trying to build the example app in this repo. When I run react-native run-android without any options, the app successfully runs on the Android emulator, but it fails with --deviceId option to run it on a real Android device. Is there any solution?
Here is the log.
$ git clone https://github.com/shaqian/tflite-react-native.git
$ cd tflite-react-native/example
$ npm install
$ react-native run-android --deviceId XXXXXXXXXXXXX

Starting JS server...
Building the app...

> Configure project :app
WARNING: The specified Android SDK Build Tools version (28.0.2) is ignored, as it is below the minimum supported version (28.0.3) for Android Gradle Plugin 3.2.1.
Android SDK Build Tools 28.0.3 will be used.
To suppress this warning, remove "buildToolsVersion '28.0.2'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

> Configure project :react-native-image-picker
WARNING: The specified Android SDK Build Tools version (28.0.2) is ignored, as it is below the minimum supported version (28.0.3) for Android Gradle Plugin 3.2.1.
Android SDK Build Tools 28.0.3 will be used.
To suppress this warning, remove "buildToolsVersion '28.0.2'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

> Configure project :tflite-react-native
WARNING: The specified Android SDK Build Tools version (23.0.1) is ignored, as it is below the minimum supported version (28.0.3) for Android Gradle Plugin 3.2.1.
Android SDK Build Tools 28.0.3 will be used.
To suppress this warning, remove "buildToolsVersion '23.0.1'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

> Task :tflite-react-native:compileDebugJavaWithJavac
Note: /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/src/main/java/com/reactlibrary/TfliteReactNativeModule.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

> Task :app:bundleReleaseJsAndAssets
warning: the transform cache was reset.
Loading dependency graph, done.
bundle: Writing bundle output to: /Users/john/development/tflite-react-native/example/android/app/build/generated/assets/react/release/index.android.bundle
bundle: Done writing bundle output

> Task :tflite-react-native:compileReleaseJavaWithJavac
Note: /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/src/main/java/com/reactlibrary/TfliteReactNativeModule.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

> Task :tflite-react-native:verifyReleaseResources FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':tflite-react-native:verifyReleaseResources'.
> java.util.concurrent.ExecutionException: com.android.builder.internal.aapt.v2.Aapt2Exception: Android resource linking failed
  Output:  error: resource android:style/TextAppearance.Material.Widget.Button.Borderless.Colored not found.
  error: resource android:style/TextAppearance.Material.Widget.Button.Colored not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values-v26/values-v26.xml:7: error: resource android:attr/colorError not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values-v26/values-v26.xml:11: error: resource android:attr/colorError not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values-v26/values-v26.xml:15: error: style attribute 'android:attr/keyboardNavigationCluster' not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values-v28/values-v28.xml:7: error: resource android:attr/dialogCornerRadius not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values-v28/values-v28.xml:11: error: resource android:attr/dialogCornerRadius not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:954: error: resource android:attr/fontStyle not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:955: error: resource android:attr/font not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:956: error: resource android:attr/fontWeight not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:957: error: resource android:attr/fontVariationSettings not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:958: error: resource android:attr/ttcIndex not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:1039: error: resource android:attr/startX not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:1042: error: resource android:attr/startY not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:1045: error: resource android:attr/endX not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:1048: error: resource android:attr/endY not found.
  /Users/john/development/tflite-react-native/example/node_modules/tflite-react-native/android/build/intermediates/res/merged/release/values/values.xml:1056: error: resource android:attr/offset not found.
  error: failed linking references.

  Command: /Users/john/.gradle/caches/transforms-1/files-1.1/aapt2-3.2.1-4818971-osx.jar/a4f31520f4d94325cc40dcb108789e7e/aapt2-3.2.1-4818971-osx/aapt2 link -I\
          /Users/john/Library/Android/sdk/platforms/android-23/android.jar\
          --manifest\ ...
...

My environment

tflite-react-native: 0.0.5
Platform: Mac OS X 10.13.6
Android version (real device): 8.0.0
Android version (emulator): 8.0.0

$ react-native info

  React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.6
      CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
      Memory: 47.34 MB / 8.00 GB
      Shell: 5.5.1 - /usr/local/bin/zsh
    Binaries:
      Node: 10.13.0 - ~/.nodebrew/current/bin/node
      Yarn: 1.17.3 - ~/.nodebrew/current/bin/yarn
      npm: 6.4.1 - ~/.nodebrew/current/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 23, 24, 25, 26, 28, 29
        Build Tools: 28.0.3, 29.0.0, 29.0.2
        System Images: android-26 | Google Play Intel x86 Atom, android-28 | Google Play Intel x86 Atom, android-29 | Google Play Intel x86 Atom
    IDEs:
      Android Studio: 3.5 AI-191.8<IP_ADDRESS>00203
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.3 => 16.6.3
      react-native: 0.58.4 => 0.58.4
    npmGlobalPackages:
      react-native-cli: 2.0.1

Thanks.

The error is similar to
https://github.com/facebook/react-native/issues/19239#issuecomment-393822322
Can you please try the suggestions in the comment?
Thanks,
Qian

@shaqian Wow, that solved my problem! Thanks a lot!
I found a little bit more sophisticated solution by another one. I hope it helps someone who has the same problem.
https://stackoverflow.com/a/53301978/1369228


feat: Alt text on GIFs

Describe the feature you'd like to request
I acknowledge this could be due to lack of protocol support, but I'd like the ability to add alt text to GIFs.
Describe the solution you'd like to see
Pretty much the same as what happens for images, an alt text button/option for a GIF
Additional information
No response

Currently the description from Tenor is used - I will add the option to modify it

Whilst you're at it, there's no way to view the alt text in the app, you get the little gif indicator, but no alt indicator, so even if tenor did have a description, which it often doesn't, there's no indication in the feed view, you'd have to click and "zoom" to see it as a label for the gif.


feat: add server-info support and versioning to MonoVertex

2024-08-09T22:17:43.333371Z  INFO Server info file: ServerInfo { protocol: "uds", language: "go", minimum_numaflow_version: "1.2.0-rc4", version: "v0.7.0-rc2", metadata: None }
2024-08-09T22:17:43.333398Z  INFO Version_info: VersionInfo { version: "latest+unknown", build_date: "1970-01-01T00:00:00Z", git_commit: "", git_tag: "", git_tree_state: "", go_version: "unknown", compiler: "rustc", platform: "linux/aarch64" }
2024-08-09T22:17:43.333805Z  INFO Server info file /var/run/numaflow/sinker-server-info is not ready, waiting...
2024-08-09T22:17:44.335138Z  INFO Server info file /var/run/numaflow/sinker-server-info is not ready, waiting...
2024-08-09T22:18:06.373836Z  INFO Server info file: ServerInfo { protocol: "uds", language: "go", minimum_numaflow_version: "1.2.0-rc4", version: "v0.7.0-rc2", metadata: None }
2024-08-09T22:18:06.373859Z  INFO Version_info: VersionInfo { version: "latest+unknown", build_date: "1970-01-01T00:00:00Z", git_commit: "", git_tag: "", git_tree_state: "", go_version: "unknown", compiler: "rustc", platform: "linux/aarch64" }

Python
skohli@macos-JQWR9T560R numaflow % kubectl logs simple-mono-vertex-mv-0-c8a2z | grep ServerInfo
2024-08-09T22:29:55.715214Z  INFO Server info file: ServerInfo { protocol: "uds", language: "python", minimum_numaflow_version: "1.2.0-rc4", version: "0.7.3", metadata: Some({}) }

Testing incompatible versions (For testing)
skohli@macos-JQWR9T560R numaflow % kubectl logs simple-mono-vertex-mv-0-qa2bh
2024-08-09T22:50:52.855975Z  INFO Server info file: ServerInfo { protocol: "uds", language: "go", minimum_numaflow_version: "1.2.0-rc4", version: "v0.7.0-rc2", metadata: None }
2024-08-09T22:50:52.855988Z  INFO Version_info: VersionInfo { version: "latest+unknown", build_date: "1970-01-01T00:00:00Z", git_commit: "", git_tag: "", git_tree_state: "", go_version: "unknown", compiler: "", platform: "linux/aarch64" }
2024-08-09T22:50:52.856006Z  WARN Error waiting for source server info file: ServerInfoError("SDK version 0.7.0-rc2 must be upgraded to at least 1.7.0-rc2, in order to work with the current numaflow version")
2024-08-09T22:50:52.856009Z ERROR Application error: ForwarderError("Error waiting for server info file")

One good, one bad container with server info
2024-08-09T22:59:57.796619Z  INFO Server info file: ServerInfo { protocol: "uds", language: "go", minimum_numaflow_version: "1.2.0-rc4", version: "v0.7.0-rc2", metadata: None }
2024-08-09T22:59:57.796692Z  INFO Version_info: VersionInfo { version: "latest+unknown", build_date: "1970-01-01T00:00:00Z", git_commit: "", git_tag: "", git_tree_state: "", go_version: "unknown", compiler: "", platform: "linux/aarch64" }
specifiers: ">=0.7.0-rc2"
sdk_version_semver: Version { major: 0, minor: 7, patch: 0, pre: Prerelease("rc2") }
2024-08-09T22:59:57.796922Z  INFO Server info file: ServerInfo { protocol: "uds", language: "python", minimum_numaflow_version: "1.2.0-rc4", version: "0.7.3", metadata: Some({}) }
2024-08-09T22:59:57.796926Z  INFO Version_info: VersionInfo { version: "latest+unknown", build_date: "1970-01-01T00:00:00Z", git_commit: "", git_tag: "", git_tree_state: "", go_version: "unknown", compiler: "", platform: "linux/aarch64" }
specifiers: VersionSpecifier { operator: GreaterThanEqual, version: "1.7.0a1" }
sdk_version_pep440: "0.7.3"
2024-08-09T22:59:57.796938Z  WARN Error waiting for sink server info file: ServerInfoError("SDK version 0.7.3 must be upgraded to at least 1.7.0a1, in order to work with the current numaflow version")
2024-08-09T22:59:57.796941Z ERROR Application error: ForwarderError("Error waiting for server info file")

All green versions
skohli@macos-JQWR9T560R numaflow % kubectl logs simple-mono-vertex-mv-0-8urnj | head -50
2024-08-09T23:09:44.855804Z  INFO Server info file /var/run/numaflow/sourcer-server-info is not ready, waiting...
2024-08-09T23:09:45.856767Z  INFO Server info file: ServerInfo { protocol: "uds", language: "go", minimum_numaflow_version: "1.2.0-rc4", version: "v0.7.0-rc2", metadata: None }
2024-08-09T23:09:45.856802Z  INFO Version_info: VersionInfo { version: "latest+unknown", build_date: "1970-01-01T00:00:00Z", git_commit: "", git_tag: "", git_tree_state: "", go_version: "unknown", compiler: "", platform: "linux/aarch64" }
specifiers: ">=0.7.0-rc2"
sdk_version_semver: Version { major: 0, minor: 7, patch: 0, pre: Prerelease("rc2") }
2024-08-09T23:09:45.857192Z  INFO Server info file: ServerInfo { protocol: "uds", language: "python", minimum_numaflow_version: "1.2.0-rc4", version: "0.7.3", metadata: Some({}) }
2024-08-09T23:09:45.857205Z  INFO Version_info: VersionInfo { version: "latest+unknown", build_date: "1970-01-01T00:00:00Z", git_commit: "", git_tag: "", git_tree_state: "", go_version: "unknown", compiler: "", platform: "linux/aarch64" }
specifiers: VersionSpecifier { operator: GreaterThanEqual, version: "0.7.0a1" }
sdk_version_pep440: "0.7.3"


Representing opaque structs like HINSTANCE

Hello everyone,
I do not know how necessary this is but I will ask about representing opaque types like HINSTANCE, HBRUSH, HICON etc. Any type that is defined with DECLARE_HANDLE macro. Here is the definition of the macro in winnt.h
#ifdef STRICT
typedef void *HANDLE;
#if 0 && (_MSC_VER > 1000)
#define DECLARE_HANDLE(name) struct name##__; typedef struct name##__ *name
#else
#define DECLARE_HANDLE(name) struct name##__{int unused;}; typedef struct name##__ *name
#endif
#else
typedef PVOID HANDLE;
#define DECLARE_HANDLE(name) typedef HANDLE name
#endif

In STRICT mode handle types are defined as an opaque struct. The Rustonomicon explains about how to represent opaque structures.
With that I think we can represent handles like this,
macro_rules! declare_handle {
    ($i:ident) => {
        #[repr(C)]
        pub struct $i {
            _data: [u8; 0],
            _marker: ::core::marker::PhantomData<(*mut u8, ::core::marker::PhantomPinned)>,
        }
    };
}

declare_handle!(HINSTANCE__);
pub type HINSTANCE = *mut HINSTANCE__;

I am curious about your thoughts and suggestions.

Can you share a bit more as to what you're looking for @emirhantasdeviren? These types are in the crate today.

Sorry for not clarifying the purpose of this question.
In the crate HINSTANCE is defined as,
pub type HINSTANCE = isize;

This does not provide type safety. In The Rustonomicon way, it provides type safety with not being able to instantiate outside of the module. Also compiler marks the type !Send, !Sync and !Unpin.

@emirhantasdeviren Sounds like you're using the raw bindings provided by windows-sys. Take a look at the windows crate.
Here's the same type over there https://microsoft.github.io/windows-docs-rs/doc/windows/Win32/Foundation/struct.HINSTANCE.html
Also check out (the FAQ for a bit more info).

Yes, I have been looking into windows-sys. In my opinion the definition of HINSTANCE in windows crate can be also be more idiomatic likeso,
#[repr(C)]
pub struct HINSTANCE__ {
    _data: [u8; 0],
    _marker: ::core::marker::PhantomData<(*mut u8, ::core::marker::PhantomPinned)>,
}

pub struct HINSTANCE(pub ::core::ptr::NonNull<HINSTANCE__>);

Forgive me, but I'm struggling to understand how this is an improvement over the existing definitions.


Inconsistent use of CL Classification hierarchy to drive search

Describe the bug
Inconsistent use of CL Classification hierarchy to drive search (I'm assuming this is a bug, although I guess it could be a feature)
To Reproduce
Here's the CL hierarchy under kidney epithelial cell (courtesy of CellGuide)

Here is a list of collections found by search with 'kidney epithelial cell'

The results do not include collections with datasets annotated with subclasses of 'kidney epithelial cell' e.g.
e.g.
https://cellxgene.cziscience.com/collections/bcb61471-2a44-4d00-a0af-ff085512674c

Expected behavior
Search with 'kidney epithelial cell' should return all datasets with annotations to types of  'kidney epithelial cell', e.g. the Lake dataset above.

The current behavior is by design.

If there is a manually constructed hierarchy, is there somewhere I can access it?  I'm curious to compare where it differs with CL (we have an automated process for this). I'd also be very interested in any record of the rationale for the current structure.  Thanks.

See Cell Type Constants in https://github.com/chanzuckerberg/single-cell-data-portal/blob/2505cefa0408cb7b0efd5df75649fbd477fc8fa4/scripts/compute_tissue_and_cell_type_mappings.ipynb. The constants were hand curated by @jahilton and @pablo-gar. Much of the original conversation was either face-to-face or on slack.

@dosumis - did you have further comments or questions? Otherwise, I will close this issue.

No response. Closing.


Refactor input types to indicate and control user input on clients (affects API)

See also https://github.com/kort/kort-core/issues/9 .

inputType.name is an enum.
name (string, optional): name of the input type = ['number', 'select', 'text']stringEnum:"number", "select", "text",
see also swagger doc for model description:
https://kort.dev.ifs.hsr.ch/v1.0/ui/#!/Mission/api_missions_get_missions


SyntaxError: Cannot use import statement outside a module

I don't know why I'm getting this error. Please help!

Are you using create react app?

yes
[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
02/27/22,
11:43:32 PM
On Sun, Feb 27, 2022 at 7:10 PM Shahriar Shojib @.***>
wrote:

Are you using create react app?
—
Reply to this email directly, view it on GitHub
https://github.com/shahriar-shojib/React-bKash/issues/17#issuecomment-1053551844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFVAVEOUGMQK5HMRSSKJDUDU5IPEPANCNFSM5PO5G5NA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Hello,
I was able to create a payment without any issues.
Try upgrading to latest version.
Checkout this codeSandbox for example: https://codesandbox.io/s/fast-water-z79gm6?file=/src/App.tsx


Use HTTP GET for list requests

Closes #109
This is the branch we are currently using, I’ll fix it up

I was not sure how to add default GET parameters to a request model - if there’s a better way let me know.
I’ll remove the extra commits later on

Thanks for your PR. I would prefer to add an option that allows the developer to control if he would like to use the LIST or the GET method to accomplish this. It could also be that a reversed proxy only allows the LIST method to prevent that secret data are leaked.

Thanks @Haennetz - I've added kv2::list_with_http_get and kv1::list_with_http_get methods.

Thanks @orf and nice work there are only some formatting issues. @stormshield-gt is it okay for you if we Marge this PR and you add the tests to #107?

I've rebased @stormshield-gt <3
It's a tricky issue: on one hand, switching from LIST to GET is supported by the API and is contractually the same. On the other hand it feels like a breaking change.
However perhaps that's over-thinking things: we don't need to be constrained by hyrum's law if we don't want to be, and it's unlikely that this would cause major issues for users given that the Vault API explicitly supports this as an alternative to LIST. The specific case would be someone restricting things to only LIST requests, but... that's unlikely.

Hi, sorry for the late response. I agree we shouldn't constrain ourselves to the hyrum's law. I think it mostly applies when you have a lot of users, and that's not the case currently.

Perhaps we should just switch things to be GET requests and remove LIST entirely?

That's the direction I want to go. What do you think @Haennetz?
I will be on vacation for a few weeks.

Sure we can switch completely to GET requests.


Moves Docs to Website

Closes https://github.com/opendatacam/opendatacam-website/issues/74

Okay, somehow git tries to delete everything. Don't know how that happened. Will fix ASAP.


change order of homebrew mono and official mono

Change the priority order of homebrew mono and official Xamarin mono.
Homebrew mono typically has a higher priority on the PATH than xamarin mono... so I think this should be the expected order to use.
Might lead to less issues as people have latest homebrew installed but an old version of the official package.

I can't test it, but if it works for you :shipit:

It does work... but the downside to this is that anyone who got omnisharp-atom to work via updating the official installer, and who has an old homebrew mono installed, will now have a bust installation.
It should be this way around though :)


New issue with Discovery

Hi Joel and Tux, have you experienced and issues with Discovery in the last day or two? Mysteriously it all the sudden stopped working. This morning all was working fine, then when I got home and fired it up again, no device is discovered. First I thought it was my app so I stepped through it and didn't see anything wrong. So then I tried the python xbox-discovery and also your guys' SmartGlass.Cli v1.0.5 and they also fail to find a device now. The console is discoverable and I see the packets in wireshark, and I can see it under casting devices in (Netflix, YouTube, etc). So then I tried my other app that uses Microsoft.Xbox.Smartglass.dll and I was able to discover, connect, etc just fine. No network issues, and no recent updates installed. any thoughts?

Can you attach a wireshark pcap please?

Attached pcap. I've been stepping through MessageTransport and I can see that it is receiving a message back from the right ip for the console. everything good until
MessageReceived?.Invoke(this, new MessageReceivedEventArgs(message));
So I think it might be in the filtering of the message, since it does deserialize and I see all the Xbox info.
discovery.zip

I think I may have found the issue. In MessageTransport.cs changed:
_client.Client.Bind(new IPEndPoint(IPAddress.Any, 0));
to
_client.Client.Bind(new IPEndPoint(IPAddress.Parse(GetLocalIpAddress()), 0));
and it came back to life. Come to find out that Virtual NICs from VMWare/VirtualBox can interfere with what address your binding to. So rather than disabling those NICs I wrote a little method to get the 'real' IP address filtering out the interfaces no needed. This works for me, unless you've got another approach.


getThumbnail - TypeError: Failed to construct 'URL': Invalid URL

Version
nuxt-directus : 5.0.1
nuxt : 3.2.3
Steps to reproduce
Simply :
<img :src="img(fileId)">

const fileId = "b40911b1-e4e0-4fc0-a080-21b85ed921ff";
const { getFiles, getThumbnail: img } = useDirectusFiles();

What is Expected?
Show image.
What is actually happening?
Uncaught (in promise) TypeError: Failed to construct 'URL': Invalid URL
    at Proxy.getThumbnail (useDirectusFiles.mjs?v=4eb507e2:24:17)

In :
const getThumbnail = (fileId, options) => {
    const url = new URL(`${directusUrl}assets/${fileId}`); <---

Another question, how to use getFiles to v-bind= file like video/mp4 from Directus ?
Thanks ! :)

Looks like a bit related to my issue, I have the same error if I'm not explicitly adding directus url to the config, in production it doesn't catch DIRECTUS_URL env variable, but works in dev mode.

@paulvonber Thanks. I create a new issue

Looks like a bit related to my issue, I have the same error if I'm not explicitly adding directus url to the config, in production it doesn't catch DIRECTUS_URL env variable, but works in dev mode.

Fixed with 5.1.3 👯


DaouTech.DaouMessenger version 3.5.10

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Manual Validation ended with:

Faulting application name: DaouOffice.exe, version: <IP_ADDRESS>, time stamp: 0x5c9d6400
Faulting module name: KERNELBASE.dll, version: 10.0.19041.1288, time stamp: 0x3e55bd0b Exception code: 0xe0434352 Fault offset: 0x0012b5b2
Faulting process id: 0x22ec
Faulting application start time: 0x01db0dd422835f99
Faulting application path: C:\Program Files (x86)\DaouOffice\DaouOffice.exe
Faulting module path: C:\Windows\System32\KERNELBASE.dll Report Id: a95ee4e5-6672-487e-8afd-87abd15f0994
Faulting package full name:
Faulting package-relative application ID:   Application: DaouOffice.exe Framework Version: v4.0.30319 Description: The process was terminated due to an unhandled exception. Exception Info: System.IO.FileNotFoundException    at GroupOffice.App..ctor()    at GroupOffice.App.Main()

(Automated response - build 896.)

Verify:

Changing lanugage with the drop-down in the upper right corner reopens the window:

Clicking the "⚙ Preferences" section reopens the server selection window. I couldn't find how to have the application display its version number.

Registry:


Directed event/listener framework

Currently the only way to broadcast an event is utilizing Node.emit().  This will walk the graph to the root scene node where the event it dispatched to all legs and all nodes.  This is not always desirable and inefficient at best.
Each node should be able to dispatch events to all of their own children directly without notifying any other legs of the scene.  Similarly, each node should be able to "tell" their parent something directly.
The idea is to have a node/component tell their parent of some event.  Once the parent receives the event, the parent would then tell all of their children about the event.  After telling all of their children of the event, the parent would not propagate (bubble) the event any further up the scene.

While I don't think that this behavior belongs on the core Node.  I think we could provide subclasses that can we used when needed.
The reason for not wanting to include it on the core Node is that it doesn't support any features of the rendering pipeline. Its only use is in application logic.

The type of eventing described in this thread is available.  However the eventing framework is intended for use with DOM/GL events and not application events.  It is possible to use the eventing framework for non generated events if you really want to.


Suggestion: allow configuration of a base directory for components to be created in

Currently
ng generate component my-new-component
Will always create the new component directory in the root of the app. Many developers like to have all components nested under a shared directory, example: app/components/
I'd propose a new property in the angular-cli.json under apps named componentsDir
  "apps": [     {       "root": "src",       "outDir": "dist",       "componentsDir": "components"

The command should generate the component from the current working directory within the project.

This works:
ng generate component shared/my-new-component

@kibbled Running command ng g component components/Test would generate components in the specified directory.


bad pip url for BiblioPixelAnimations

When launching run-pixelweb it runs a pip install on BiblioPixel's packages. The bibliopixelanimations url is incorrect.
$ sudo run-pixelweb
Installing BiblioPixelAnimations
and then it fails, attempting to manually install BiblioPixelAnimations gives more feedback:
# pip install --no-deps BiblioPixelAnimations -v
Collecting BiblioPixelAnimations
  Getting page https://pypi.python.org/simple/bibliopixelanimations/
  Starting new HTTPS connection (1): pypi.python.org
/usr/local/lib/python2.7/dist-packages/pip/_vendor/requests/packages/urllib3/util/ssl_.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropri
ately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
  "GET /simple/bibliopixelanimations/ HTTP/1.1" 404 28063
  Could not fetch URL https://pypi.python.org/simple/bibliopixelanimations/: 404 Client Error: Not Found - skipping
  1 location(s) to search for versions of BiblioPixelAnimations:
  * https://pypi.python.org/simple/bibliopixelanimations/
  Getting page https://pypi.python.org/simple/bibliopixelanimations/
  "GET /simple/bibliopixelanimations/ HTTP/1.1" 404 28063
  Could not fetch URL https://pypi.python.org/simple/bibliopixelanimations/: 404 Client Error: Not Found - skipping
  Could not find a version that satisfies the requirement BiblioPixelAnimations (from versions: )
Cleaning up...
No matching distribution found for BiblioPixelAnimations
Exception information:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/pip/basecommand.py", line 211, in main
    status = self.run(options, args)
  File "/usr/local/lib/python2.7/dist-packages/pip/commands/install.py", line 294, in run
    requirement_set.prepare_files(finder)
  File "/usr/local/lib/python2.7/dist-packages/pip/req/req_set.py", line 334, in prepare_files
    functools.partial(self._prepare_file, finder))
  File "/usr/local/lib/python2.7/dist-packages/pip/req/req_set.py", line 321, in _walk_req_to_install
    more_reqs = handler(req_to_install)
  File "/usr/local/lib/python2.7/dist-packages/pip/req/req_set.py", line 461, in _prepare_file
    req_to_install.populate_link(finder, self.upgrade)
  File "/usr/local/lib/python2.7/dist-packages/pip/req/req_install.py", line 250, in populate_link
    self.link = finder.find_requirement(self, upgrade)
  File "/usr/local/lib/python2.7/dist-packages/pip/index.py", line 571, in find_requirement
    'No matching distribution found for %s' % req
DistributionNotFound: No matching distribution found for BiblioPixelAnimations

The path does not exist on https://pypi.python.org/simple, the correct pip location would be:
pip install  https://github.com/ManiacalLabs/BiblioPixelAnimations/archive/master.zip --upgrade
BiblioPixelAnimations should either be added under https://pypi.python.org/simple/bibliopixel/ or given its own entry, or the script should be updated to point to the correct url.

What is the actual error it gives when you call run-pixelweb and it tries to install BiblioPixelAnimations? The path in the code is correct and working on my end. The error you did post is expected because BPA is not on PyPi and will not be for a variety of reasons. Mainly, it's an ever changing repository and the intent is to always just install it directly from github.

I have to work around a proxy at work, so I manually specify --proxy= to pip to get things installed. I invoked run-pixelweb and when it would state that it was fetching a package, I would cancel run-pixelweb and execute a "pip --proxy install" with that package name. When I tried doing a pip install BiblioPixelAnimations however it failed because https://pypi.python.org/simple/bibliopixelanimations/ doesn't exist. But now that I investigated a little, I see that line 28 in bootstrap.py correctly calls the long url for animations, so this is not a bug, it's just my failure to read through the code when pip install failed. If I'd looked, I could have found the correct path. Sorry about that!

No worries... Documentation is coming soon!
On Oct 30, 2015 5:14 PM, "praeluceo"<EMAIL_ADDRESS>wrote:

I have to work around a proxy at work, so I manually specify --proxy= to
pip to get things installed. I invoked run-pixelweb and when it would state
that it was fetching a package, I would cancel run-pixelweb and execute a
"pip --proxy install" with that package name. When I tried doing a pip
install BiblioPixelAnimations however it failed because
https://pypi.python.org/simple/bibliopixelanimations/ doesn't exist. But
now that I investigated a little, I see that line 28 in bootstrap.py
correctly calls the long url for animations, so this is not a bug, it's
just my failure to read through the code when pip install failed. If I'd
looked, I could have found the correct path. Sorry about that!
—
Reply to this email directly or view it on GitHub
https://github.com/ManiacalLabs/PixelWeb/issues/1#issuecomment-152652973
.


update links from v1.5 to v1.6

update links v1.5 to v1.6

NOTE: Please check the “Allow edits from maintainers” box below to allow
reviewers fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

@chenopis After my checks, there are six files which need update from v1.5 to v1.6. I will update them one by one. Hope that you won't mind the number of PR. Currently I haven't known how to pull requests by batch. I need to study that. :)


✨ Feature Request: Profanity Filter

Is your feature request related to a problem?
No - new feature
Describe the solution you'd like
It would be great to add in a profanity filter so any words that are not "clean" or general use aren't part of the generation.
Describe alternatives you've considered
No response
Additional context
No response
Other Labels

[ ] Documentation
[X] UX
[X] API

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Closed


D1436 grid resize

Did you mean to change gridster.min.js?  If so, you should put a comment at least in the commit message about why you upgraded.

I'm not sure why it included the index.html in the diff. I did mean to change the minified build, to include the changes I made to the unminified javascript.

LGTM


Some util functions to support bpfix

Change-Id: If53f696fbe937e007b902434f62d0d92435846dd

Ping

Reminder that this is a prerequisite for the current implementation of bpfix

Thanks! :)


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/931 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit c053d6f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


ESTIMATE * FROM PAIRWISE results in exception

I ran this command trying to figure out what fields "FROM PAIRWISE" returns by default, when not extended with any model estimators:
ESTIMATE * FROM PAIRWISE satellites_cc

This results in an exception:
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/bayesdb.py", line 158, in execute
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/bql.py", line 59, in execute_phrase
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/bql.py", line 644, in execute_wound
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/bayesdb.py", line 176, in sql_execute
sqlite3.OperationalError: near "AS": syntax error

I suppose the fact that "*" cannot be used related to the fact that "expression" instead of "columns" is used in the documentation:
ESTIMATE <expression> FROM PAIRWISE <generator>

However, something like "*" here would be helpful. The alternative that I used instead to reveal the returned fields is:
ESTIMATE "foo" FROM PAIRWISE satellites_cc

which I don't think should be necessary.
Also, perhaps the BQL documentation should define the fields that are returned and their relationship to the fields in the data table, e.g.:
Returns a table with three fields:
1. rowid0: The value of field "rowid" in the data table for the first row.
2. rowid1: The value of field "rowid" in the data table for the second row.
3. value: The result of evaluating the expression on the two rows.

Also, the equivalent for columns also is not supported, e.g.:
ESTIMATE * from PAIRWISE COLUMNS OF satellites_cc
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/bayesdb.py", line 158, in execute
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/bql.py", line 56, in execute_phrase
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/compiler.py", line 229, in compile_query
  File "build/bdist.macosx-10.9-x86_64/egg/bayeslite/compiler.py", line 723, in compile_estpaircols
IndexError: tuple index out of range

For the pairwise columns case, knowing that the output fields are name0 and name1 are important if the user is interested in only a subset of the columns:
estimate dependence probability as dp from pairwise columns of satellites_cc where (name0 = "Anticipated_Lifetime" OR name0 = "Contractor") AND (name1 = "Purpose" OR name1 = "Eccentricity"

The first symptom, that ESTIMATE * FROM PAIRWISE t raises sqlite3.OperationalError, indicates a bug -- the BQL->SQL compiler should never generate broken SQL, but obviously it is generating broken SQL.  The bayeslite sql_trace should reveal what SQL was generated.  Can you find that?
The second part, of what * means in ESTIMATE ... FROM PAIRWISE t, is...well, it has never come up before, partly because (a) there's only one thing you typically do with it, namely SIMILARITY, and nobody's tried to explore that before; but also because (b) the number of ways you can use SIMILARITY has a combinatorial explosion: nrows * 2^ncols.
As a postscript: note that "foo" is how you quote identifiers.  What you wanted to do is write strings, which are written 'foo' instead.  This didn't fail immediately for two reasons:

The BQL->SQL compiler isn't equipped to detect this mistake at the moment -- that is the subject of issue #38.

sqlite3 intentionally lets you conflate the two where unambiguous, likely for historical compatibility with other broken SQL engines -- but this may come back and bite you later, if a column is ever added to the table you're selecting from whose name happens to coincide with the text of a string you were trying to write.

The fact that the space of possible model estimator expressions for use in ESTIMATE is large is interesting, and suggests a possible tension between the type of analysis this tool is supporting and this SQL-based framework. Here we have a combinatorial explosion in the space of similarity estimators. However, one could even imagine an infinite set of model estimators, parameterized by some real number, such as the indicator function of whether the entropy of the predictive distribution is greater than some number:
E_{\theta \sim P(\dot | X_{observed})}\left[ \mathbf{1}[H(P(x' | \theta) > \alpha] \right]

Are there other projects where the semantics of SELECT * are similarly troubled by an infinite space of possible things to select from?

Um, what does combinatorial explosion have to do with it?  Standard SQL permits an infinite number of possible expressions for SELECT ... FROM, and also defines that SELECT * FROM produces the columns.  Methinks the real issue with defining ESTIMATE * FROM PAIRWISE is that the point of the PAIRWISE is to introduce implicit arguments for functions, so there is no sensible meaning for "all the ground information which you may recombine with function applications".

What about having ESTIMATE * FROM PAIRWISE COLUMNS defined as producing the two column IDs and their statistical types, and having ESTIMATE * FROM PAIRWISE (ROWS) defined as producing the row IDs? Something like this would help the user quickly understand the type of collections FROM PAIRWISE returns, and also give access to important information like the statistical types in the case of columns. The import of having the row and column IDs beyond making it clear what the command is doing is less obvious. I'm guessing these row and column IDs are linked to other tables?


New Assist: To/From Block Comments

I would like an assist on converting /// <-> /**, and //! <-> /*! comments to toggle an entire section of comments between the line and block styles. For example, a series of /// lines documenting a struct could be converted to a single /** … */ block, and then back again.
I don't see committing code in the block style to be the main use here, but rather this would allow switching to block comments for easier editing, and then switching back to line comments when done.
By converting to a block comment temporarily, you're easily able to copy a code example to the clipboard without manually removing the ///<space> from each line, or to paste a code example from another source into the doc comment. It would also be helpful when updating word wrapping or code example formatting. When done, it only requires a couple keystrokes to switch back.
I would likely author new medium-length doc comments in the block form if it were easy to switch to line comments after.
Examples
/// This is the first line of the comment.
///
/// # Example
/// ```
/// let a = 1;
/// let b = 2;
/// let n = if a < b {
///     // Space indented
///     a + b
/// } else {
///     // Tab indented
/// 	b - a
/// };
/// ```
/// Etc...
pub struct S;

would convert to/from the following, any number of times...
/**
This is the first line of the comment.

# Example
```
let a = 1;
let b = 2;
let n = if a < b {
    // Space indented
    a + b
} else {
    // Tab indented
	b - a
};
```
Etc...
*/
pub struct S;

Additional Details
The above examples are actually imperfect. Each of the line comments likely has a space after the ///, but isn't required, and that decision isn't represented in the block comment. This allows easier editing at the cost of being potentially but rarely lossy.
I would suggest that relative indentation is preserved, i.e. the least amount of whitespace after the /// or //! on any line being transformed (except if the rest of the line is blank) be trimmed when converting to a block comment, and one space of separation be entered when converting back to line comments. It seems like it would be unusual for this to be an issue, but it would in theory be possible to resolve if people disagree (say, starting the comment with /** <!-- indent: none -->).
I'm unsure whether or not line comments that are themselves indented should preserve the whitespace before the /// when converted to block comments. Either way, you can use the least indented non-blank line when converting back to line comments, and indent it appropriately for the context.
It might also make sense to do this for the non-documentation // <-> /* comments, for completeness.
I think this would be a generally useful assist to have available, especially when writing or editing library documentation.

Yup, we should have this

Will try to implement this :)

Just bumping this, I was thinking of suggesting the same thing but found this request first.


Calling fn(Box<T>) from C++ doesn't compile

I'm trying to instrument the entry- and exit-points of a C++ function with cxx-generated code to measure its runtime.
Rust side:
use std::time::{Instant, Duration};

#[cxx::bridge]
mod ffi {
    extern "Rust" {
        type Clock;
        fn timer_start() -> Box<Clock>;
        fn timer_end(clk: Box<Clock>);
    }
}

/// Wraps [Instant] so as to be usable by [cxx].
struct Clock {
    heart: Instant,
}

impl Clock {
    fn new() -> Self {
        Clock {
            heart: Instant::now()
        }
    }

    /// Returns the elapsed time since the [Clock]
    /// was created in the form of a [Duration].
    fn tick(&self) -> Duration {
        self.heart
            .elapsed()
    }
}

/// Creates a new [Clock] and wraps it around a [Box],
/// so as to be passable across [cxx]'s FFI bridge.
fn timer_start() -> Box<Clock> {
    Box::new(Clock::new())
}

/// Consumes a boxed [Clock] and prints the time elapsed
/// since its creation to stdout.
fn timer_end(clk: Box<Clock>) {
    let dur = clk.tick();
    println!("Elapsed time: {} μs", dur.as_micros());
}

C++ side:
#include "RusToy.h"

void dummy() {
    rust::cxxbridge1::Box<Clock> clk = timer_start();
    // ...
   timer_end(clk);
}

g++ throws the following error:
error: use of deleted function 'constexpr rust::cxxbridge1::Box<Clock>::Box(const rust::cxxbridge1::Box<Clock>&)'
  129 |   timer_end(clk);
      |   ~~~~~~~~~^~~~~

[...]/RusToy.h:14:7: note: 'constexpr rust::cxxbridge1::Box<Clock>::Box(const rust::cxxbridge1::Box<Clock>&)' is implicitly declared as deleted because 'rust::cxxbridge1::Box<Clock>' declares a move constructor or move assignment operator
   14 | class Box final {
      |       ^~~

[...]/RusToy.h:265:37: note:   initializing argument 1 of 'void timer_end(rust::cxxbridge1::Box<Clock>)'
  265 | void timer_end(::rust::Box<::Clock> clk) noexcept;
      |                ~~~~~~~~~~~~~~~~~~~~~^~~

Box<T> is a move-only type. There is no implicit copy constructor (Copy in Rust). You need to use timer_end(std::move(clk)).


git version --silent is no longer valid in git 2.14.0

I use librarian-puppet to manage my puppet modules. After upgrading Git to the latest version (2.14.0), librarian-puppet install fails because git version --silent is no longer valid as of 2.14.0.
The output of git version --silent is no different from that of git version, indicating the --silent has been having no effect for some time.

Same problem here

Same problem. But the development for this repository seems dead.

Same issue!


Use logging instead of print

There are a few instances where CmdStan prints to stdout. It would be convenient if the standard Logging macros were instead used (@info, @warn, etc). Then users could opt out of the messages without redirecting stdout. This is especially useful when running CmdStan in a documented example.

Hi Seth, seems like a good suggestion. I’ll have a look how much effort is involved as I’m pretty busy at the moment.

I'm happy to submit a PR if you'd like.

Absolutely! Thanks.

Sorry @goedman , this PR became a lower priority. I probably won't get to it immediately.


Add Thumb Label to Slider

Adds a thumb label (https://material.google.com/components/sliders.html#sliders-discrete-slider) to the slider. The thumb label is displayed when thumb-label is added to the slider. It will appear whenever the slider is active (a click or a slide).
R: @jelbourn @kara
CC: @hansl @robertmesserle

LGTM, just a few minor comments

@jelbourn: Addressed comments

LGTM


Broken link

Reached:
https://googlecloudplatform.github.io/google-cloud-php/#/docs/google-cloud/v0.24.0/not-found
Via redirect from:
https://googlecloudplatform.github.io/google-cloud-php/#/docs/latest/naturallanguage/naturallanguageclient
From:
https://cloud.google.com/natural-language/docs/reference/libraries#additional_resources
(PHP API Reference Documentation)

Thanks for the report, @jambrose42. This should be fixed for you now!


Question about reverse proxy with Apache/Nginx/etc

It's often said that a best practice for Node/Express is to reverse proxy it through Apache or some other server. This might potentially also speed up the serving of static files, allowing them to come directly from Apache rather than going through Express.
I'm not very knowledgeable about server setups, but I was thinking maybe you have some suggestions for this that might be relevant for the purpose of this example repo, in case you've done this before? When implementing a reverse proxy setup through Apache, is there anything in particular that the dev needs to do or keep in mind?

You could certainly serve the static stuff in dist and favicon as well as proxy the /api requests with any other web server. But that's a pretty late stage optimization concern to be dealt with once your application is built and popular. Certainly not worth addressing in a starter example. And it's an optimization that can be implemented painlessly later. It should be irrelevant to any design choices during app construction.
Does that help?


bug(cdk/overlay): Setting maxWidth on overlay messes up top and bottom position

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Setting maxWidth on the overlay messes up the positioning. If you instead set only width, it works fine. But maxWidth messes it up. It seems to work fine with some positions, but breaks top and bottom positioning.
Reproduction
StackBlitz link: https://stackblitz.com/edit/k6wprl?file=src%2Fexample%2Fcdk-overlay-basic-example.ts
Steps to reproduce:

Click "Display overlay" button

Expected Behavior
Overlay should be properly positioned.
Actual Behavior
Overlay doesn't have correct positioning, it is off to the side instead of being above the button that triggered it.
Environment

Angular: 16.2.3
CDK/Material: 16.2.3
Browser(s): Chrome

Is there any update on this? Seems like a pretty common use case that we'd want to enable.


fix incorrect documentation of sentPacketHistory.Iterate

We've been using this in https://github.com/lucas-clemente/quic-go/blob/bed802aee5732c8fe373bf4f71e789bea2f7f11f/internal/ackhandler/sent_packet_handler.go#L158-L166
I thought this was not a safe thing to do, but it turns out to be. Therefore, we can revise the documentation, and add a test case to make sure we don't introduce a regression at some point.

Turns out the documentation was right, and the test case I added is wrong. We need to test that we actually iterate over all packets instead.


label function support

It should work exactly the same way as parent

Ok, after a bit of search, I'm logging line 211, parseLabel(viewScope), and that runs a few times (more and more times each time I visit the page without reloading if that makes sense...), and on some runs, I get the result from my function but at the end run I do not. So I don't understand.
It goes like so:
Home
Forums:
Home
Forums: myForum
Home
Forums:
Home
Forums:
So I guess that because the last run gives an empty response, that's why my code isn't showing. But don't know why...

It is perhaps worth noting that using the above approach, I was only able to output the function after injecting $rootScope into the controller.
// controller
$scope.getBreadcrumbLabel = function() {
// Something with scope here
}
would become:
// controller
$rootScope.getBreadcrumbLabel = function() {
return $scope.parent.child;
}

I would also see the use for having a function for label.
function($scope, $state)
Maybe the function could return a promise that would be waited on before displaying.
Basically, I've got states that have have a parent state with a dynamic label.
And I would like for there to be a way to handle that.
Home > user name > current state
"user name" would be a state such as "site.user({id: 12})", and I would like in the label function to be able to perform a backend query to get the name of the user and output it there.
The backend query could be done in the current state and I could set some $rootScope var to print the user name. But I would prefer if the current state didn't have to handle that. I would like each state to handle its own labeling, even if it's not the current state.

I guess ideally, I would like to have a breadCrumb builder like:
.state('site.users', {
url: '/users/{id:[0-9]+}',
templateUrl: 'users.html',
controller: 'UsersController',
ncyBreadcrumb: {
builder: function($injector, $state) {
var someService = $injector.get('someService');
var $q = $injector.get('$q');
var deferred = $q.defer();
        someService($state.params.id).then(function(result) {
          deferred.resolve({ label : result.label, parent : result.parent });
        }

        return deferred.promise;
    }

}
})

@Flo-G did you figure out how to get around this with your proposed solution? Having the same issue

this would be helpful, i have multiple controllers using the same state and in order to render their own custom breadcrumbs I have to attach the same function in each controller vs being able to pass in a provider or service
ncyBreadcrumb:{
    label:  function(myService, $stateParams) {
                  return myService.getLabel($stateParams.id);
              }
 }

if there way I can accomplish this?

+1
ng-breadcrumbs seems to support it: https://github.com/ianwalter/ng-breadcrumbs#adding-dynamic-route-labels.

+1

+1 for ability to use $stateParams in label, as @bautistaaa suggested

+1

Related to #105

Also related to #146


tutorial

are there plans on how to write new widgets for the dashboards and extra pages?
thank you,

We're planning on doing a number of tutorials to show the versatility of the Filestack platform. We'll definitely touch on a number of ways to integrate the picker into your page, including how to implement and customize widgets.

Good question. There have always been plans for it but it has been one of those things that has fallen on the back-burner. However, I do not see it in the immediate future unless someone submits a PR to add it.
The current wiki gives an overview of the stacks. If you have any specific questions about the setup of the app internals, feel free to ask on the gitter channel.


fix: allow .(dot) literal in table name

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
It's common for OpenTSDB metrics to use .(dot) literal in their names.
Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

We should validate the table name before creating or renaming a table.


Syncing from upstream patroni/patroni (master)

bt_gitbot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.875%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
14328

Relevant Lines:
14346

💛  - Coveralls


recent changes break ie tests

See an example of the travis log.
Commits 57670384f3ac85322c557627eb93957250c707c4 and 39ed0650a32054117f388e1bcb6fa46be76c14aa  had an interdepenency on each other with respect to angular-bootstrap component, so it is difficult to unwind them. They seem to be breaking the e2e tests. In particular, testing hangs in IE9.
I spent quite a bit of time yesterday tracking this down.
I don't like to revert commits, especially ones where multiple issues are hoped to be fixed, so I am in the process of a triple-checking whether reversion solves the issue and what is the minimal amount of reversion to restore IE9 to health. I am hoping to salvage the tooltips work without major upheaval. If so, I will push the reversion and we need to re-examine the issues (#412, #413, #414, #469, #473) that the commits were intended to address.
cc @popzip @gghai @clockworked247

Latest theory is that the reason may be the difference between angular-ui-bootstrap v 0.11 vs. 0.12. If so, we may be able to keep our changes after tweaking Scott's customization of boostrap to be based on 0.11 instead of 0.12. @clockworked247 I recall you saying that 0.12 might be necessary for the current tooltip implementation -- is that correct?

@clockworked247 , @wooldridge , please examine the last command that ran under this test: https://saucelabs.com/tests/ce0fc470e88a48e29861890c2bad8911
It appears the "Post Your Question" button is never finishing and the result of the click never comes back.
Will you guys run this through in IE and figure out whether or not the app works here and see if you can figure out why this test might be failing?
Please note: I've set "back" browser-develop by several commits while I was trying to diagnose. Don't be alarmed, we will put back those commits which turn out to be absolved once we figure out what's wrong here.
Thanks!

I can confirm that with @laurelnaiad 's browser-develop, we are still seeing failures on saucelabs with the tooltip and datepicker changes removed.
Update on tests for SauceLabs tests for Win7 on IE9.   @wooldridge and I were able to reproduce an issue https://saucelabs.com/tests/3707a15f6c494b19bd280c89068b4135 which shows a hang after joe log into the app.
After talking with Gaurav, it seems that the exact same issue is being seen on appw7-brwt-3 where the app loads, but the results spinner just sits and spins.   I believe this is the cause of the timeout issue.   Since I messed with the previous test while it was running, I'm running just the IE9 test again here:
https://saucelabs.com/tests/09a75615db4e47cbb1e4fb6c9b47389c
While that runs, I'm looking at appw7-brwt-3 to see if I can debug why the spinner just sits there.  This is definitely one of the issues causing errors in the build.  I have yet to reproduce the "Post a Question" issue yet.
Will update this issue as we continue to debug this.

See if it's related to the ng-tags-input directive, please. I don't trust it in IE9 (e.g. https://github.com/marklogic/marklogic-samplestack/issues/457).

please ignore this one (https://saucelabs.com/tests/3707a15f6c494b19bd280c89068b4135) for now and let's focus on the one that has been hanging us most of all... we can come back to that one once we've settled "Post a Question".

Update:  When loading the app for the first time on appw7-brwt-3, the header loaded, but not the filter bar.  Observed spinner in below header just sitting.   On manual refresh, the page loaded fine.  This appears to be our timeout issue.
After page loaded, I manually tested the "Post a Question" and it worked fine in IE9 on appw7-brwt-3.
Will continue to investigate the timeout issue on appw7-brwt-3.

Make sure the middle tier has been warmed up before you have any expectation of getting off of that initial spinner.

Update:  I was able to run the W7 IE9 sauce command from the linux box and it passed.  https://saucelabs.com/tests/69559d27a2014ed5a370c064a24aa4c5
from rh6-intel64-80-qa-dev-2
So I wasn't able to repro what Travis was seeing.
So for both Mac and Linux, tests are passing fine for W7 IE9.   This may be a Travis specific problem.

When i ran the below command on the mac app installation with  latest develop code :
gulp e2e --sauce=win7-ie-9 --tags='~@broken'
in sauce screen cast i am seeing : Can you please confirm what can be causing it .

THhis is in the category I was describing in my email -- sometimes IE9 just bomb during tests. Please try a few times. If it continues to hang on the same test more than a couple times, let's dig deeper. Otherwise this is just the best we've been able to do with testing IE9.

@laurelnaiad  This is more of the travis issue which QA doesnt run the test on . I will verify one I have in my plate on saucelabs . Passing it back to you , you can close this one as it more of the dev task .


Add .Resource.PathInfo

Which would be a smaller and curated version of what we assign to every resource internally. Instead of adding lots of NameOriginal variants, you would be able to do:
.PathInfo.Name // => roses-are-red.en.jpg
.PathInfo.NameNoIdentifier // => roses-are-red
.PathInfo.Unnormalized.NameNoIdentifier // => Roses are red
.PathInfo.Path

I can outline such an interface if people think it's a good idea ...?
/cc @jmooring

Based on the context here I would personally find value in being able to get the extension of a resource without causing it to be published (e.g. because I'm going to rename it).
How would this interact with files that are compiled and have their extension change?
Given
{{ with resources.Get "main.ts" }}
    {{ $resource := . | js.Build }}
{{ end }}

Would $resource.PathInfo.Name be main.ts or main.js?


Build with mvn.  Allow app to run w/o model.  Show errors in UI.

Restructured to build with mvn.
We have no out-of-the-box modelId, so let the app run w/ the generic model.
(todo should have a message, at least, if using generic model)
Errors (bad model, auth, etc...) show up in the UI now to avoid confused users.

Thanks @markstur for cleaning this up


Add basic skeleton for developer docs

Create a reasonable skeleton for important topics to cover and write a bit about the various subjects. This mostly note-taking levels of detail, but I think a reasonable starting off point. I'm open to any and all suggestions from copy-edits to missing/incomplete/incorrect data.

Thanks! This is quite comprehensive already and the structure makes sense. I'm about halfway through reviewing.
On writing style, some parts feel sales-pitch-y in a way that I don't think is relevant in developer documentation. There are some inaccuracies that can be adjusted but also some nonsensical statements. (Did you use an LLM to help writing? That's fine, but I'd appreciate transparency on that)

Yeah the outline was generated with my dev notes and claude so the tone is classically LLM'd (sales pitch-y is a nice way to put it). I did a few of slop cleanup but I'll do another to tighten it up.

Alright! In that case I'd also cut down on the length of some of the pages especially around the "recommendations" and "design principles". They're reasonable for the most part, but too generic to add much value IMO.

Gave it another once over and removed some of the overly flowery blocks. It reads as a reasonable first-time-dev resource to me, but please let me know what else you'd like to see changed. Thanks!

Excellent! Thanks for the feedback, pedantry is always welcome when talking docs. Will take another pass at the branch this morning.

Accepted your suggestions, made some more copy edits, and rebased everything together. I only have the one remaining question as to how (if at all) you think we should document the intent of the StrWrite trait.

Thanks! I'm not seeing the rebase yet though, and it looks like the latest commit undoes the changes.

How about something like:

Custom writer implementations via implementing the StrWrite trait

This is the bit of the crate that I was personally the most interested int and it wasn't clear to me using the existing docs.

That isn't possible right now though. The only public entry points for the HTML module are:
pub fn push_html<'a, I>(s: &mut String, iter: I) {...}
pub fn write_html_fmt<'a, I, W: std::fmt::Write>(writer: W, iter: I) {...}
pub fn write_html_io<'a, I, W: std::io::Write>(writer: W, iter: I) {...}

StrWrite is mainly meant to be an adapter trait. An implementation detail we'd rather not have to expose. If you look at the definition for std::fmt::Write, it is very close to StrWrite:
pub trait Write {
    // Required method
    fn write_str(&mut self, s: &str) -> Result;

    // Provided methods
    fn write_char(&mut self, c: char) -> Result { ... }
    fn write_fmt(&mut self, args: Arguments<'_>) -> Result { ... }
}

So for all intents and purposes, a user of pulldown-cmark should be able to implement fmt::Write for their custom type and use that instead of StrWrite.
The reason we want StrWrite at all is it makes the Error type customizable as an associated type, so we can pass through both std::fmt::Result and std::io::Result without losing anything.
Reading the docs for std::fmt::Error, I notice it mentions that the way IO errors are passed through formatting in the standard library is with an adapter that stores the error:
struct Adapter<'a, T: ?Sized + 'a> {
    inner: &'a mut T,
    error: Result<()>,
}

We could consider something similar for pulldown-cmark in the future as well.
Previous discussions:
https://github.com/pulldown-cmark/pulldown-cmark/issues/492
https://github.com/pulldown-cmark/pulldown-cmark/pull/493
https://github.com/pulldown-cmark/pulldown-cmark/pull/870
https://github.com/pulldown-cmark/pulldown-cmark/pull/881
TL;DR: StrWrite is an implementation detail to abstract over the fmt and io machinery, since they have different error types. Users can provide custom writers by implementing either fmt::Write or io::Write, whichever is more appropriate.

Unrelated to this PR, that actually cleared up some mystery around the W generic 👍

Indeed, I had to read through those prior discussions again to remind myself of the nuances

Actually pushed the rebase now ;)

Thanks for the writing and reviewing work! I will try to review this in the next days.

It looks good to me! I think it can be merged when the last olipu's comment is fixed. Thanks!

Corrected that reference to be firstpass.rs 👍


[plot] Make sure there is no circular references between plot widgets and associated tools

It seems there is places where strong reference are kept instead of using weakref.
This can be the root of issues like "seg fault" at the end of the application and accessing a PlotWidget while the underlying C++ object has been destroyed..
@dnaudet, if you have simple sample code to reproduce that would be nice.

Quick example: in my app I use weakmethod to keep a ref to a callback, so that it doesn't get called when the object it belongs to is deleted  (and so I don't have to explicitly keep track of it myself). This doesn't work with PlotWidget.
(note that all slots/signals get disconnected, because the underlying C++ object is gone, but the python wrapper is still there)
import weakref
import datetime
from functools import partial

import numpy as np

from sip import ispyowned, isdeleted

from silx.gui import qt as Qt
from silx.gui.plot import PlotWindow
from silx.utils.weakref import WeakMethod

def ref_destroyed(obj, name=None):
    print('Weakref "{0}" destroyed! {1}'.format(name, obj))

class Scheduler(object):
    def __init__(self, callback):
        self._timer = Qt.QTimer()

        self._cb = WeakMethod(callback)

        self._timer.timeout.connect(self._timeout)
        self._timer.start(1000)

    def _timeout(self):
        cb = self._cb()
        if cb:
            cb()

class MyWid(Qt.QWidget):
    def someSlot(self, *args, **kwargs):
        print('MyWid SLOT!', ispyowned(self), isdeleted(self))

class PlotWithSlot(PlotWindow):
    def someSlot(self, *args, **kwargs):
        print('PlotWithSlot SLOT!', ispyowned(self), isdeleted(self))
        self.addCurve(np.arange(3) + datetime.datetime.now().second, np.arange(3))

class MainWindow(Qt.QMainWindow):
    def __init__(self, *args, **kwargs):
        super(MainWindow, self).__init__(*args, **kwargs)

        mdi = Qt.QMdiArea()
        self.setCentralWidget(mdi)

        subWin = MyWid()
        mdi.addSubWindow(subWin)

        plotWSlot = PlotWithSlot()
        mdi.addSubWindow(plotWSlot)

        self._weakQWwid = weakref.ref(subWin,
                                      partial(ref_destroyed, name='QWidget'))
        self._weakPlotWid = weakref.ref(plotWSlot,
                                        partial(ref_destroyed,
                                                name='plotWSlot'))

        self._sched_1 = Scheduler(subWin.someSlot)
        self._sched_2 = Scheduler(plotWSlot.someSlot)

if __name__ == '__main__':
    app = Qt.QApplication([])
    win = MainWindow()
    win.show()
    app.exec_()

Adding gc.collect() in _timeout solves the issue, this confirms the fact that there are some circular references somewhere.
For the record, the issue is also there with PlotWidget and the OpenGL backend.

As a workaround for now I embed the PlotWidget in a QWidget and only keep a weakref of the QWidget and its methods.

I'm on it.

PR #1874 is fixing this issue for 4 plot widgets (PlotWidget, PlotWindow, Plot1D and Plot2D).
I guess there are other plot and plot3d widgets (and maybe other widgets) which suffer from similar issues so it is worth checking...


Calling 'destroy' when a cropper is already destroyed (or not initalized) creates it.

$('#image').cropper('destroy') creates a cropper for the image if it hasn't been created already. If this is the intended behavior, it should at least be explicitly mentioned in the documentation.
I also don't see any way to find out whether a cropper is currently attached to a certain image - there is no method that returns this information.
This means that there is no foolproof way to "destroy" a cropper regardless of its current state.

I am currently doing an if statment on the image, but it is kind of buggy.
im doing:
if $("#image").hasClass('cropper-hidden'){
    $("#image").cropper("destroy");
}

it is not the best.

Destroy the cropper when it is built:
$('img').cropper({
  built: function () {
    // Do something
    // ...
    // Then destroy it
    $(this).cropper('destroy');
  }
});

I had improved this in the v0.7.8, you can try it now.


Added Marathi and Hindi readme.

धन्यवाद..

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

Thanks for taking the time to contribute this localization pull request. However, we are not able to accept contributions to this historical reference repository at this time.


fix: Double slash in links splash page

The site.url in config includes a trailing slash already

@smellems looks like you reverted everything, I'll let you re-apply if you want, but this was the only part that needed to be addressed

We tried just removing the / but for some reason the /en/index.html was not found (404).  Thanks!

Closing, since it currently works.  CI re added with #260


update tc.thermostat_basic_checkout to handle runtime variables

tc.thermostat_basic_checkout uses zone from module runtime input but thermostat_type is currently from function input parameters,
default mode of this function should use runtime inputs, but allow function overrides for both thermostat type and zone.

this ticket was resolved by #295


buildship cannot build project preview

I have a complex Gradle build that I tried opening with Buildship.
Eclipse complains that some property cannot be found.
Situation is as follows in this project:

In the root project gradle file there is

ext.foo = 'bar'

In buildSrc there is some plugin that uses project.foo or even project.rootProject.foo
In a sub-projects gradle file this buildSrc plugin is applied

Now when Buildship tries to build the project preview, it fails, complaining that there is no property foo on the sub-project, or the root project, depending on whether project.foo or project.rootProject.foo is used.
Normal Gradle builds via the commandline work fine with this 3.5 build.

Having Eclipse metadata already present from gradlew eclipse, the import dialog can correctly build the project preview.

Can you please move this issue to the Buildship repository? Also, please  provide a reproducible example project on GitHub.

Closing as it is better suited to be asked in the Buildship repo.


[FEATURE]Deploy modules from a AWS Region to a AWS LocalZones

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have to deploy a piece of CDK code e.g. EKS cluster to the Hamburg LocalZone, from the AWS Frankfurt region.
Describe the solution you'd like
Here is the architecture of seed-farmer and code-seed. The step 2 to 6 will run in the Frankfurt region. Only the deployment target in step 7 e.g. CDK for EKS clusters is the Hamburg LocalZone

Describe alternatives you've considered
There is no alternative solution to make a consistent developer experience.

We will take a look and evaluate LOE

@JunjieTang-D1
It appears that once the VPC in the region is configured to support a LocalZone, you just reference the subnet and all still works (EKS itself would probably need a self-managed node group in order to leverage the LZ, but that is not SF's concern).
I don't understand this comment, though:
I'm always frustrated when I have to deploy a piece of CDK code e.g. EKS cluster to the Hamburg LocalZone, from the AWS Frankfurt region.
I am not sure I understand this as the LZ must be associated with a VPC in a region.  Though I understand your use case, I am not understanding your request.  The LZ cannot exist without the region, so CDK code must be deployed int he region.
Please contact me