Bug Report / Support Request: Azure Active Directory Domain Services  (ADDS) deployment

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Hello team, This is much more question / looking for recommendation where we should deploy AD DS in Enterprise Scale. From my point of view, natural place is Identity subscription, but deployment is not allowing to separate network resources into different resource group / subscription. So everything is landing in same subscription. From caf-enterprise-scale we have policy which is blocking deployment of subnets without NSG and public IPs, which is not allowing to deploy AD DS.
Questions:

Is it on purpose, to deploy AD DS somewhere else?
Do we want to extend ES to support this use case? E.g. config to automatically deploy AD DS in identity sub.
Do we just want to change policy on Identity to have exception for AD DS.

Thanks!
Steps to Reproduce

1.Deploy AD DS in Identity Subscription

I have implemented this @mlomat  :)
Solved by doing the following (using terraform for everything):

Create an exclusion on the identity archetype for public IP, since AADDS demands an public IP.
Create a dedicated virtual network for the AADDS resources
Create an management VM (could be in the same virtual network or any other virtual network, based on your design)
Peer the dedicated virtual network to every other virtual network. You can't route the traffic through the firewall using UDR, not supported (or not supported when I was implementing it). See here for more information about the network configuration: https://learn.microsoft.com/en-us/azure/active-directory-domain-services/network-considerations
Deploy the AADDS resource
Create Entra ID groups for RBAC
I will also recommend creating an Azure Policy for automatically join VMs to the AADDS. Makes management easier.

If you are using terraform you can easily deploy subnet with the NSG, not sure how it works if you do it through the portal or Bicep.

Hi @hlokensgard, thanks for instruction ;) I know how to achieve it. Would be nice to have it out of the box and will be happy if @matt-FFFFFF or someone from MSFT team can comment. Thanks!

Hi @mlomat
I think @hlokensgard's suggestion is a good one here.
Essentially you have to create an exemption/exclusion for the policy that is preventing the deployment.
We don't have plans to extend this module to support AADDS. Let me know if you need anything else.

Thanks @matt-FFFFFF ok I understand, no issue from my side. I will create exception.


term merge? nucleosome remod. comp./chromatin remod. comp.

GO:0005679
nucleosome remodeling complex
A protein complex that is required for the induced
transcription of many genes; alters  chromatin
structure so as to allow the binding of specific RNA
polymerase II transcription factors.
GO:0016585
chromatin remodelling complex
undef
are the same?
At least all the things annotated to it appear to be
chromatin remodelling complexes
The definition needs rewording because not all
chromating remodelling complexes (for example the RFC
complex) remodel chromatin to allow transcription
Reported by: ValWood
Original Ticket: "geneontology/ontology-requests/1573":https://sourceforge.net/p/geneontology/ontology-requests/1573

thanks www.movied.org


[2pt] All data should be preserved by saving collections in .json files - Student A

All data should be preserved by saving collections in .json files.

https://github.com/jsug9/Catalog-Capstone/pull/65


Be more flexible about local h2 databases to allow distributing read-only copies for offline maven builds.

Is your feature request related to a problem? Please describe.
We are looking into pre-creating the databases used by the dependency check maven plugin to allow being run as part of an offline CICD maven build.
My idea so far is to have a SNAPSHOT build which uses src/main/resources as the datafolder (instead of sneaking in a corner of .m2) which then by default puts the ovc.mv.db file in the root of the generated jar file which is then in turn managed completely by Maven simply by installing it and then use it as a dependency and then tell the plugin to use a "jdbc:h2:zip:..." url instead.
Unfortunately the plugin logic explicitly checks in isH2Connection if the prefix is "jdbc:h2:file" so this breaks.
If I can get this to work, there will be another way of running the plugin in a locked down environment which will be reasonably fast and not require having a database server running.
Is this something that would be interesting to the project?

First thing that springs to mind is: why would you create such a complex setup that requires a mvn install? Why not simply use the datadir to get it out of your .m2 repo to some stable location?

With the database in a zip-file you will suffer performance-wise per H2 documentation.
Also note that the isH2Connection is only used during the database update scenarios (to trigger defragmentation of the database after the updates and to create the database schema if the H2 database was not yet present), which are not possible anyhow for an H2 database in a zip-file.
So for your scenario the isH2Connection can safely return false eventhough the JDBC URL you configure is in fact a read-only h2 connection.

@aikebah Thank you for your comments.
The basic problem is that the build servers that need to run mvn dependency-check:check as part of building whatever we want it to, do not have internet access and cannot reach the official servers.  Also they run on an empty .m2 folder, so the work to download and construct the database needs to be done in every run as they are not there, which takes several minutes at best, and the documentation does not cover this particular case.   A way around this is to provide data files from a previous run.  Unfortunately previous runs did not have access to the internet either, so they have to be brought in from a job running outside the build server enclosure.   I was looking into using Maven
Since I wrote this I found that it is not enough to just have the database file available with a JDBC connection, the whole set of data files are needed, which makes this approach somewhat moot.
I ended up so far with using the src/main/resources as the data dir for the online building step, and then deploy the artifact containing the datafiles to an Artifactory server visible from the inside of the offline CICD building enclosure, and run dependency:unpack to a folder in target/ just before running dependency-check:check with AutoUpdates disabled.   This has worked for our proof-of-concept.
That in turn means that we do not need anything from you to get this to run, and I'll therefore close this issue.


Implement ability to edit now playing songs

This PR is possible replacement of #719. I know that #719 brings more features, but it's unmergeable, and seems like no one wants to update it.
The editing mode differs from how it's made in #719, so I'd be glad to get comments on current implementation.
Resolves #245.
Resolves #371.
Resolves #619.
Resolves #1021.

Looks nice.
Some proposals:

I think something like "edited" should be added to the popup if song is edited by user
There should be (optional?) button like "Revert" or "Resubmit original info"
Saving these edits in storage would be really nice(OK, that's not about popup UI but anyway)

There should be (optional?) button like "Revert" or "Resubmit original info"

Ok, I'll look into that.

I think something like "edited" should be added to the popup if song is edited by user

Well, I think, the 'edited' label is redundant, when the 'revert' button is visible. And the latter should be visible only for edited songs.

Saving these edits in storage would be really nice(OK, that's not about popup UI but anyway)

It should works fine with #1167, in theory.

Well, I think, the 'edited' label is redundant, when the 'revert' button is visible. And the latter should be visible only for edited songs.

Yes, that's the best option for that.

I have some questions:

Should we close the popup when user clicks on 'Submit' button?
Should we close the popup whne user clicks on 'Revert' button?

The easier option is 'yes' for both questions. In this case it's not necessary to update popup when the song is processed by pipeline after editing (tbh, I don't know how to do that).

Revert is finally done + metadata info update on the fly.


[WIP] Support new protocol for gestures

Summary
This is a frontend implementation for the protocol changes in xi-editor/xi-editor#1072.
Related Issues
Discussed in xi-editor/xi-editor#826.
Review Checklist

[ ] I have responded to reviews and made changes where appropriate.
[x] I have tested the code
[ ] I have updated comments / documentation related to the changes I made.
[x] I have rebased my PR branch onto xi-mac/master.

This looks good to go, just needs the submodule update.

@cmyr I've updated the submodule.


changes to fix issue 246

All Submissions:

[ X] Have you followed the guidelines in our Contributing document?

[ X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

[ X] Does your PR follow our Code of Conduct?

[X ] Have you added an explanation of what your changes do and why you'd like us to include them?

[ ] Does each method or function "do one thing well"? Reviewers may recommend methods be split up for maintainability and testability.

[ ] Is this code designed to be testable?

[ ] Is the code documented well?

[ ] Does your submission pass existing tests (or update existing tests with documentation regarding the change)?

[ ] Have you added tests to cover your changes?

[ ] Have you linted your code prior to submission?

[ ] Have you updated the documentation and README?

[ ] Is PII treated correctly? In particular, make sure the code is not logging objects or strings that might contain PII (e.g. request headers).

[ ] Have secrets been stripped before committing?

This PR fixes #246 . The missing forward slash in the input reader and output writer docker files causes the Azure devops builds to fail.
Changes done

I have modified the docker file templates of input reader and output writer to include the correct path which fixes issue.

Azure Devops build is successful after adding these changes

Discussed offline with @maniSbindra. Adding the /, breaks local docker builds (as a container reg is not set or expected) and testing.


feat(readwise): init

🎉 Theme for Readwise 🎉
Readwise helps you get the most out of what you read by making it fun & easy to revisit your highlights from all your favorite reading platforms in one place. Quickly synchronize your highlights from Kindle, Apple Books, Instapaper, Pocket, Medium, Goodreads, and even paper books.

🗒 Checklist 🗒

[x] I have read and followed Catppuccin's submission guidelines.
[x] I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.

[x] I have ensured that the new directory is in lower-kebab-case.
[x] I have followed the template and kept the preprocessor as LESS.

[x] I have made sure to update the
userstyles.yml
file with information about the new userstyle.
[x] I have included the following files:

catppuccin.user.css - all the CSS for the userstyle, based on the
template.
preview.webp - composite image of all four individual flavor screenshots stitched together,
generated via Catwalk.

Closes https://github.com/catppuccin/catppuccin/discussions/2479

@uncenter Thank you for the review, I've approved all the changed, the only doubt is about the domain change, as I've described in the thread there.

Please don't update this branch unnecessarily!

@uncenter @isabelroses all the comments are resolved, preview image is updated both in the PR content and description

@uncenter @isabelroses PR is updated, comments are resolved

No need to force push FYI. We squash merge so your commits in PRs don't matter much.

@uncenter thank you for your help and careful review, I will resolve those design issues soon and will update my PR

@uncenter changes are applied

Just noticing in the last image you sent some things appear unthemed. Is there a reason for this?

@isabelroses You are right, sorry for skipping that. Went through different UI elements on the page: search icon, chevron icon, scroll bar and styled them according to the style guide

That bar with the tabs above it looks unthemed, as well as the background color of the details of that one book you have selected.

@uncenter thx for the feedback, issues are resolved

@uncenter @isabelroses Hi! Is there is anything I can help with to make it merged?


Unable to pass tests with official docker containers(igibson/igibson:lates，igibson/igibson-vnc:v2.0.6)

To whom it may concern.
I installed the officially released docker container on my own Linux server. And tested it according to the iGibson documentation([(https://stanfordvl.github.io/iGibson/quickstart.html)]). But except for simple commands like import igibson, there are other commands involving the GUI.
Currently the container can be accessed and the gpu detected as follows：

But when I run the command: python -m igibson.examples.environments.env_nonint_example,  the following error is reported:

Please ask how to solve this problem. Thank you very much.

Using the command sudo glxinfo | grep "OpenGL" , I found out that there was a problem with the nvidia driver on the server, where the driver for opengl was not downloaded.

Re-download the driver files and install them, don't bring "-no-opengl-files" in the command, so that only the driver files will be installed, not the OpenGL files.

Rerun the command "sudo glxinfo | grep "OpenGL"" and found that opengl supports 4.6

Although the opengl version issue was resolved, the iGibson GUI still cannot be displayed.
Follow the igibson instructions documentation to open the docker container:
docker run --env="DISPLAY" --volume="/tmp/.X11-unix:/tmp/.X11-unix:rw" --env="QT_X11_NO_MITSHM=1" --gpus all -ti --rm igibson/igibson:latest
Run the command and the following error is reported:

Is it a headless server (i.e. the server isn't connected to a display monitor)? If so, this behavior is expected. You can set mode="headless".


fixed bug in mergesort function of sorting.py

Found and fixed small bug in mergesort function of sorting.py. Changed standard division to floor division to fix.
Results of running sorting.py BEFORE change:
Trying:
    bubblesort([6, 4, 8, 2, 1, 9, 10])
Expecting:
    [1, 2, 4, 6, 8, 9, 10]
ok
Trying:
    insertionsort([6, 4, 8, 2, 1, 9, 10])
Expecting:
    [1, 2, 4, 6, 8, 9, 10]
ok
Trying:
    mergesort([5, 4, 1, 6, 2, 3, 9, 7])
Expecting:
    [1, 2, 3, 4, 5, 6, 7, 9]
**********************************************************************
File "c:/Users/parker/Documents/GitHub/Algorithms/sorting and basics/sorting.py", line 4, in __main__.mergesort
Failed example:
    mergesort([5, 4, 1, 6, 2, 3, 9, 7])
Exception raised:
    Traceback (most recent call last):
      File "C:\Users\parker\Anaconda3\lib\doctest.py", line
1329, in __run
        compileflags, 1), test.globs)
      File "<doctest __main__.mergesort[0]>", line 1, in <module>
        mergesort([5, 4, 1, 6, 2, 3, 9, 7])
      File "c:/Users/parker/Documents/GitHub/Algorithms/sorting and basics/sorting.py", line 12, in mergesort
        a1 = mergesort(arr[:n/2])
    TypeError: slice indices must be integers or None or have an __index__ method
Trying:
    mergesort([3, 2, 4, 2, 1])
Expecting:
    [1, 2, 2, 3, 4]
**********************************************************************
File "c:/Users/parker/Documents/GitHub/Algorithms/sorting and basics/sorting.py", line 7, in __main__.mergesort
Failed example:
    mergesort([3, 2, 4, 2, 1])
Exception raised:
    Traceback (most recent call last):
      File "C:\Users\parker\Anaconda3\lib\doctest.py", line
1329, in __run
        compileflags, 1), test.globs)
      File "<doctest __main__.mergesort[1]>", line 1, in <module>
        mergesort([3, 2, 4, 2, 1])
      File "c:/Users/parker/Documents/GitHub/Algorithms/sorting and basics/sorting.py", line 12, in mergesort
        a1 = mergesort(arr[:n/2])
    TypeError: slice indices must be integers or None or have an __index__ method
Trying:
    quicksort([6, 4, 8, 2, 1, 9, 10])
Expecting:
    [1, 2, 4, 6, 8, 9, 10]
ok
Trying:
    selectionsort([6, 4, 8, 2, 1, 9, 10])
Expecting:
    [1, 2, 4, 6, 8, 9, 10]
ok
2 items had no tests:
    __main__
    __main__.merge
4 items passed all tests:
   1 tests in __main__.bubblesort
   1 tests in __main__.insertionsort
   1 tests in __main__.quicksort
   1 tests in __main__.selectionsort
**********************************************************************
1 items had failures:
   2 of   2 in __main__.mergesort
6 tests in 7 items.
4 passed and 2 failed.
***Test Failed*** 2 failures.

Results of running sorting.py AFTER change:
Trying:
bubblesort([6, 4, 8, 2, 1, 9, 10])
Expecting:
[1, 2, 4, 6, 8, 9, 10]
ok
Trying:
insertionsort([6, 4, 8, 2, 1, 9, 10])
Expecting:
[1, 2, 4, 6, 8, 9, 10]
ok
Trying:
mergesort([5, 4, 1, 6, 2, 3, 9, 7])
Expecting:
[1, 2, 3, 4, 5, 6, 7, 9]
ok
Trying:
mergesort([3, 2, 4, 2, 1])
Expecting:
[1, 2, 2, 3, 4]
ok
Trying:
quicksort([6, 4, 8, 2, 1, 9, 10])
Expecting:
[1, 2, 4, 6, 8, 9, 10]
ok
Trying:
selectionsort([6, 4, 8, 2, 1, 9, 10])
Expecting:
[1, 2, 4, 6, 8, 9, 10]
ok
2 items had no tests:
main
main.merge
5 items passed all tests:
1 tests in main.bubblesort
1 tests in main.insertionsort
2 tests in main.mergesort
1 tests in main.quicksort
1 tests in main.selectionsort
6 tests in 7 items.
6 passed and 0 failed.
Test passed.

Never mind, although this change makes the code pass all test cases, strangely it doesn't produce a sorted list. This is obviously very confusing since the the code passes the 2 test cases for mergesort.
Here's some example code to show that it doesn't produce a sorted result:
def mergesort(arr):
    """ perform mergesort on a list of numbers
    >>> mergesort([5, 4, 1, 6, 2, 3, 9, 7])
    [1, 2, 3, 4, 5, 6, 7, 9]
    >>> mergesort([3, 2, 4, 2, 1])
    [1, 2, 2, 3, 4]
    """
    n = len(arr)
    if n <= 1: return arr
    a1 = mergesort(arr[:n//2])
    a2 = mergesort(arr[n//2:])
    return merge(a1, a2)

def merge(arr_a, arr_b):
    arr_c = []
    i, j = (0, 0)
    while i < len(arr_a) and j < len(arr_b):
        if arr_a[i] <= arr_b[j]:
            arr_c.append(arr_a[i])
            i += 1
        else:
            arr_c.append(arr_b[j])
            j += 1
    if arr_a[i:]: arr_c.extend(arr_a[i:])
    if arr_b[j:]: arr_c.extend(arr_b[j:])
    return arr_c

array = [3, 44, 38, 5, 47, 15, 36, 26, 27, 2, 46, 4, 19, 50, 48]
print(array)
mergesort(array)
print(array)

Outputs:
[3, 44, 38, 5, 47, 15, 36, 26, 27, 2, 46, 4, 19, 50, 48]
[3, 44, 38, 5, 47, 15, 36, 26, 27, 2, 46, 4, 19, 50, 48]


Added Back button to Compose.

Added Back button to Compose. Goes to MessageList displaying Inbox. #2238

Updated with changes requested @philipwhiuk #2238

We'll go with another pull request for this.


Add byte_mapped_address to processor spec

Is your feature request related to a problem? Please describe.
The only way to currently add a byte mapped memory block is by implementing a loader. The bit_mapped_address attribute exists in processor_spec.rxg for a default memory block but none exists for byte mapped addressing.
Describe the solution you'd like
Add byte_mapped_address attribute to processor_spec.rxg
Describe alternatives you've considered
Doing it in a loader.

This will require some rework of the MemoryBlockDefinition class.  In addition, byte-mapped blocks should allow specification of the byte-mapping ratio.

See /Ghidra/Framework/SoftwareModeling/data/languages/processor_spec.rxg for XML specification changes

See /Ghidra/Framework/SoftwareModeling/data/languages/processor_spec.rxg for XML specification changes affecting pspec memory_block tag.

Thank you.


COMP: Add testing of IsCongruentImageGeometry IsSameImageGeometryAs

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

Very happy to see another GoogleTest unit test in ITK, thanks @blowekamp ! Shall I still review in detail?


AlgorithmicDarkeningAllowed behavior exception in Target SDK >= 33

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I tried to set the value of algorithmicDarkeningAllowed to change the dark night mode of the webview, but it didn't work.
Then I created an Android native project to experiment and it worked.
I suspect that the reason it doesn't take effect is because setAlgorithmicDarkeningAllowed needs to change the native activity
theme to take effect, and this activity should inherit from AppcompactActivity, but FlutterActivity inherits the activity, so it doesn't take effect.
https://developer.android.com/reference/kotlin/androidx/webkit/WebSettingsCompat#setAlgorithmicDarkeningAllowed(android.webkit.WebSettings,boolean)

Expected Behavior
algorithmicDarkeningAllowed work fine.
Steps with code example to reproduce

Steps with code example to reproduce
test flutter code in https://github.com/pichillilorenzo/flutter_browser_app/blob/master/lib/pages/settings/android_settings.dart, it not work！
      SwitchListTile(
        title: const Text("Algorithmic Darkening"),
        subtitle: const Text(
            "Set the algorithmic darkening in targetSdkVersion > T for this WebView."),
        value:
            currentWebViewModel.settings?.algorithmicDarkeningAllowed ?? true,
        onChanged: (value) async {
          currentWebViewModel.settings?.algorithmicDarkeningAllowed = value;
          webViewController?.setSettings(
              settings: currentWebViewModel.settings ?? InAppWebViewSettings());
          currentWebViewModel.settings = await webViewController?.getSettings();
          windowModel.saveInfo();
          setState(() {});
        },
      ),

but in android native project, work fine！but if is Activity not AppCompatActivity ,it not work!
public class MainActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        WebView webView = findViewById(R.id.webview);
        WebSettings webSettings = webView.getSettings();
        webSettings.setJavaScriptEnabled(true);
        webView.setWebViewClient(new WebViewClient());
        webView.loadUrl("https://pub-web.flutter-io.cn/");
        Button button = findViewById(R.id.button);
        button.setOnClickListener(v -> {
            if (AppCompatDelegate.getDefaultNightMode() == AppCompatDelegate.MODE_NIGHT_YES) {
                AppCompatDelegate.setDefaultNightMode(AppCompatDelegate.MODE_NIGHT_NO);
                webSettings.setAlgorithmicDarkeningAllowed(false);
            } else {
                AppCompatDelegate.setDefaultNightMode(AppCompatDelegate.MODE_NIGHT_YES);
                webSettings.setAlgorithmicDarkeningAllowed(true);
            }

        });
    }
}

Stacktrace/Logs

Stacktrace/Logs
<Replace this line by pasting your stacktrace or logs here>

Flutter version
v3.24.4
Operating System, Device-specific and/or Tool
Android 33+
Plugin version
v6.1.5
Additional information
No response
Self grab

[X] I'm ready to work on this issue!

@pichillilorenzo

That's something related to Flutter itself and not this plugin: https://github.com/flutter/flutter/issues/71208

@pichillilorenzo I don't know if my speculation is correct. The current solution is to set the target SDK 32, but this will prevent Google Play from being listed

@pichillilorenzo I solved it, the reason is that the setTheme method of the activity should be used instead of the MyAmpatDelegate.setDonfirtNightMode method, I wrote a plugin to switch the theme of the activity, attach the code to those in need
<?xml version="1.0" encoding="utf-8"?>
<resources>
    <!-- Theme applied to the Android Window while the process is starting when the OS's Dark Mode setting is off -->
    <style name="LaunchTheme" parent="@android:style/Theme.Light.NoTitleBar">
        <!-- Show a splash screen on the activity. Automatically removed when
             the Flutter engine draws its first frame -->
<!--        <item name="isLightTheme">true</item>-->
        <item name="android:windowBackground">@drawable/launch_background</item>
    </style>
    <!-- Theme applied to the Android Window as soon as the process has started.
         This theme determines the color of the Android Window while your
         Flutter UI initializes, as well as behind your Flutter UI while its
         running.

         This Theme is only used starting with V2 of Flutter's Android embedding. -->
    <style name="NormalTheme" parent="@android:style/Theme.Light.NoTitleBar">
        <item name="android:windowBackground">?android:colorBackground</item>
    </style>

    <style name="DarkLaunchTheme" parent="@android:style/Theme.Black.NoTitleBar">
        <item name="android:windowBackground">@drawable/launch_background</item>
    </style>
</resources>

package cc.lycstar.pluvia;

import android.app.Activity;
import androidx.annotation.NonNull;
import io.flutter.embedding.engine.plugins.FlutterPlugin;
import io.flutter.embedding.engine.plugins.activity.ActivityAware;
import io.flutter.embedding.engine.plugins.activity.ActivityPluginBinding;
import org.jetbrains.annotations.NotNull;

import static cc.lycstar.pluvia.GeneratedNativeTheme.NativeThemeApi.setUp;

public class NativeThemePlugin implements GeneratedNativeTheme.NativeThemeApi, FlutterPlugin, ActivityAware {

    Activity activity;

    @Override
    public void onAttachedToEngine(@NonNull @NotNull FlutterPlugin.FlutterPluginBinding flutterPluginBinding) {
        setUp(flutterPluginBinding.getBinaryMessenger(), this);
    }

    @Override
    public void onDetachedFromEngine(@NonNull @NotNull FlutterPlugin.FlutterPluginBinding flutterPluginBinding) {
        setUp(flutterPluginBinding.getBinaryMessenger(), null);
    }

    @Override
    public void onAttachedToActivity(@NonNull @NotNull ActivityPluginBinding activityPluginBinding) {
        activity = activityPluginBinding.getActivity();
    }

    @Override
    public void onDetachedFromActivityForConfigChanges() {
        activity = null;
    }

    @Override
    public void onReattachedToActivityForConfigChanges(@NonNull @NotNull ActivityPluginBinding activityPluginBinding) {
        activity = activityPluginBinding.getActivity();
    }

    @Override
    public void onDetachedFromActivity() {
        activity = null;
    }

    @Override
    public void setDarkMode(@NonNull @NotNull GeneratedNativeTheme.NativeDarkMode mode) {
        if (activity == null) {
            return;
        }
        if (mode == GeneratedNativeTheme.NativeDarkMode.LIGHT) {
            activity.setTheme(R.style.LaunchTheme);
        } else {
            activity.setTheme(R.style.DarkLaunchTheme);
        }
    }
}

     Future<void> _enableDarkMode() async {
    if (Platform.isAndroid) {
      for (var element in webviewConfigs) {
        InAppWebViewSettings? inAppWebViewSettings =
            await element.controller?.getSettings();
        if (inAppWebViewSettings != null) {
          await NativeThemeApi().setDarkMode(NativeDarkMode.dark);
          inAppWebViewSettings.algorithmicDarkeningAllowed = true;
          await element.controller
              ?.setSettings(settings: inAppWebViewSettings);
        }
      }
    }
  }

  Future<void> _disableDarkMode() async {
    if (Platform.isAndroid) {
      for (var element in webviewConfigs) {
        InAppWebViewSettings? inAppWebViewSettings =
            await element.controller?.getSettings();
        if (inAppWebViewSettings != null) {
          await NativeThemeApi().setDarkMode(NativeDarkMode.light);
          inAppWebViewSettings.algorithmicDarkeningAllowed = false;
          element.controller?.setSettings(settings: inAppWebViewSettings);
        }
      }
    }
  }


[DOCS] Linkedin OAuth link

Hi there, we have been trying to setup the account in Linkedin:
Add the URL https://.firebaseapp.com/popup.html to the OAuth 2.0 > Authorized Redirect URLs of your LinkedIn app.
the issue is the url is not being accepted, we got the application id and the final looks like this:
https://1:312666799144:web:b7f11067d1fc976ded.firebaseapp.com/popup.html
when we try to submit it in the OAuth 2.0 settings in Linkedin we cant save with that url.
would have any suggestions?
Thanks

I think the application id is simply your Firebase project id.


Remove LiveMetric and LiveMetricMixin

These were returned by a number of Middleware/Hawkular models but all users have since been deleted (https://github.com/ManageIQ/manageiq/pull/19378)

cc @kavyanekkalapu

Do we have to remove app/javascript/oldjs/controllers/live_metrics from ui-classic? or can that be done separately?

Do we have to remove app/javascript/oldjs/controllers/live_metrics from ui-classic? or can that be done separately?

UI passes specs, but that certainly doesn't mean it isn't broken :)
I'm not sure how that code was working before, maybe @kavyanekkalapu can weigh in here on what backend methods are/were being called?

@Fryguy I found few files. not sure if these are related.
https://github.com/ManageIQ/manageiq-ui-classic/blob/e03b401aa2e142b0b3f0dd91bf03e5546542fb09/app/services/hawkular_proxy_service.rb
https://github.com/ManageIQ/manageiq-ui-classic/blob/e03b401aa2e142b0b3f0dd91bf03e5546542fb09/app/javascript/oldjs/controllers/live_metrics/util/metrics-http-factory.js#L102 ( i think we are calling many functions from above file here).

https://github.com/ManageIQ/manageiq-ui-classic/blob/e03b401aa2e142b0b3f0dd91bf03e5546542fb09/app/services/hawkular_proxy_service.rb

:scream: what is that?
I don't think any of those are directly related to the LiveMetrics here since they go to Hawkular directly and don't appear to wrap the results in anything.
Seems we have two different live metrics?

That looks like a service to just wrap the MetricsCapture class, so I don't think it's a second live metrics?  Strange that it's HawkularClient specific though.

That looks like a service to just wrap the MetricsCapture class, so I don't think it's a second live metrics?

Well it is called live metrics and is unrelated to this live metrics

EDIT: Oh I see what you mean - it's hitting Hawkular directly - wtf

Ha, yeah hence the :scream:

So, I don't have anything against going to the provider for "live" metrics
I do have an issue with it being Hawkular specific, not going over the API, and hitting the provider directly from the UI worker (not respecting the zones).  IMO we should just remove that current implementation and reevaluate if we need live metrics and redesign if the answer is yes.


Memory management and allocation

It seems like memory allocation is not in the sdk. Could it be possible to have a sbrk syscall and a simple allocator in the example source code ?

Thanks for the suggestion! The function void eadk_heap_range(u32 * start_address, u32 * end_address) will write the start and the end of an available heap in the addresses pointed by start_address and end_address.

Shouldn't it be better to have as parameter void ** than a pointer to a numeric variable?


SvelteKit authentication with static adapter

The client-side auth flow requires writing server-side hooks according to the docs, but it's not available with the static adapter. Is it possible to somehow solve authentication in a static site?

You would do the same as for pure Svelte: https://supabase.com/docs/guides/getting-started/tutorials/with-svelte

Could you explain how? As I previously mentioned, the static adapter does not support server-side hooks.

The tutorial doesn't use any server-side hooks, as it's tutorial for svelte therefore no hooks exists. Just go from top to bottom of the linked tutorial and you will have working auth flow.

Sorry for the confusion, I haven't notice there are separate pages for pure Svelte and SvelteKit.
However, SvelteKit does not use window.fetch but event.fetch, so it's not compatible with the SvelteKit tutorial. #889


feat: support devServer middleware hook

PR Details
support beforeDevServer、afterDevServer server hook.
onBeforeSetupMiddleware, onAfterSetupMiddleware can be used as syntactic sugar.
Description
can config in jupiter.config.js like this:
{
  tools: {
    devServer: {
      onBeforeSetupMiddleware: [
        async (ctx, next) => {
          console.log('before');
          next();
        },
      ],
      onAfterSetupMiddleware: [
        async (ctx, next) => {
          console.log('after');
          next();
        },
      ],
    },
  },
}

next step is to implement more configurations, such as specifying the middleware writing of the route
Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[x] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[x] I have updated changeset
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

对应的文档可以加一下哈

对应的文档可以加一下哈

7f7c6ef2a21da14323e61d3c40f28a4b9ce8e236 这个提交


[Feature Request]  Enable/Disable "Recording Voice Message" Service

I am writing to request an improvement to our application to include a service that allows users to activate and deactivate the "Voice Message Recording" feature based on existing services [startTyping and stopStyping]. This functionality is crucial to improving user control and flexibility when sending voice messages.
The main requirements for this improvement are the following:
Implementing this feature will greatly improve the user experience and provide greater flexibility in managing voice communications, as it truly simulates human interaction.
Thank you for considering this request. I look forward to your positive response and will be happy to discuss further details if necessary.
Yours sincerely,

Hi! Do you mean "recording" status? Try this one
https://waha.devlike.pro/docs/how-to/presence/#set-presence
POST /api/{session}/presence

{
  "chatId"<EMAIL_ADDRESS>  "presence": "recording"
}

and PLEASE :pray:  stop using chatgpt, it generates plenty of not important text that you could just translate using the same chatgpt
The rule of thumb would be to formulate the text in your language and just ask it to translate.


Loop Distribution

Use loop distribution/fission to increase parallelism.

I have done serious work on this one.  One big missing piece is generating different variants of distribution and sequentialisation (and while I'm at it - the kernel bodies are not presently sequentialised at all).  Furthermore, the code is not used in the main pipeline.
I am going to let this rest for now and prepare other important parts we need for our efficient code generator.  When ready, I can turn the distribution code into a fairly powerful kernel extractor with little extra coding effort.


changed key and server

updated apt_percona_repo_key and apt_percona_repo_key_server since key server was not available any more and key had changed, tested on ubuntu 14.04

I fixed problems with repo server, key and updated tests accordingly. An update to rspec-puppet >= 2.4.0 was causing tests to fail because of STRICT_VARIABLES, the problem was mitigated by adding STRICT_VARIABLES=no to env in travis.yml, but this could probably be handled better but I'm lacking the time to investigate further at the moment see: https://github.com/rodjek/rspec-puppet/issues/373

Hey thanks for the patches, I'm not maintaining this module very actively any more since I'm not using it at my current job. If you're interested, I suspect the entire module could be remade as a patch to puppetlabs-mysql with a bit of work.


Notification drawer accordion does not work consistently

Was talking with @jeff-phillips-18 about some issues, he noted height calculation.
I've been experiencing issues where items are truncated.  For example, I have 15 items in this list, but only 7 are visible & it will not scroll consistently:

Another example, this time the accordion collapsed entirely:


Parallel MFEM build in Appveyor [appveyor-mpi-build]

Add parallel MFEM build in Appveyor yaml file.  MS-MPI, Hypre, and METIS built.

Yes, the parallel run fails with MS-MPI.  This PR is just to get the hyper, metis, and compile portion of mfem checked in.

Sent from VMware Boxer
On August 8, 2017 at 5:49:25 PM PDT, Veselin Dobrev<EMAIL_ADDRESS>wrote:
@v-dobrev commented on this pull request.

In .appveyor.ymlhttps://github.com/mfem/mfem/pull/253#discussion_r132068551:

after_build:
-- cmake --build build --config %BUILD_TYPE% --target check
+#- cmake --build build_parallel --target check
Does the check of parallel build fail? Why is it disabled?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/mfem/mfem/pull/253#pullrequestreview-55110724, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARa7c-DfCQT4tWuG1aCP7880AeYEa94dks5sWQIBgaJpZM4OxVUo.


[Campaign] Ensure Github workflow runs on docker image used by Production Distribution Build

Hi All,
This is coming from the campaign here:

https://github.com/opensearch-project/opensearch-build/issues/3966

Overview
We would like your CI check (specifically plugin build) in GitHub Repo to run on top of the Build Docker Images from production distribution pipeline.
This is to ensure every plugin repo will use the exact docker images we used in Jenkins build, to check their PRs and run tests before merging the code, so that issues can be detected earlier, and environment can be identical across teams.
Solutions
The Build Team has created a simple script to dynamically retrieve the current docker image name/tag, so everyone can easily pull the images for their CI checks.
We have a trial run of the above with k-NN team. The script retrieves the docker image dynamically, save output, and use it as the docker image to pull for the upcoming run:

https://github.com/opensearch-project/k-NN/pull/1099

Note that GitHub Actions only support LINUX docker container at the time of this writing, so we will add Windows containers later on as well as macOS.
Implementation Notes
We would like you to review above PR and implement similar changes. Note on line 33 of the above k-NN PR, -u and -p parameters needs to assign values accordingly.

OpenSearch Plugin:

          CI_IMAGE_VERSION=`opensearch-build/docker/ci/get-ci-images.sh -p centos7 -u opensearch -t build | head -1`

OpenSearch-Dashboards Plugin:

          CI_IMAGE_VERSION=`opensearch-build/docker/ci/get-ci-images.sh -p rockylinux8 -u opensearch-dashboards -t build | head -1`

Note that in the above k-NN PR, despite it being OpenSearch plugin, it still uses rockylinux8, as we initially plan to upgrade to rockylinux. We have since revert back to centos7 to support older versions of systems running k-NN lib. As a result, all OpenSearch plugins still uses centos7 for the time being, and all OpenSearch-Dashboards plugins can go to rockylinux8.
Completion Date
The above should be implemented by Nov. 1, 2023 (2023-11-01) by Plugin Owners to their repository.
And backport the changes to 2.x branch after merging in main branch.
Contacts
Please contact @peterzhuamazon for any questions on this campaign.
cc: @bbarani
Thanks.

@peterzhuamazon, @bbarani completing this by 11/1 is a short notice with all the ongoing work. What OpenSource release are we targeting this for?  cc @sean-zheng-amazon

@peterzhuamazon the OSD CI image version doesn't seem to exist - see https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/actions/runs/6472097461/job/17572014571?pr=620#step:4:10

@peterzhuamazon the OSD CI image version doesn't seem to exist - see https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/actions/runs/6472097461/job/17572014571?pr=620#step:4:10 as the echo statements are blank.

That echo is to save value to github env vars, so you for sure will not see it.
          echo "ci-image-version-linux=$CI_IMAGE_VERSION" >> $GITHUB_OUTPUT

@peterzhuamazon oh, copying over it looks like I missed that part, just updated, thanks. It looks like I will need to add container/image keywords for running the linux tests as well.

We have a new approach established here.
Please see this sample workflow file from Alerting for example:
https://github.com/opensearch-project/alerting/blob/main/.github/workflows/multi-node-test-workflow.yml
Thanks.

@peterzhuamazon changing to this approach has caused some weird behavior where we are getting failures trying to install yarn with the new approach. I confirmed by reverting the change and seeing it is passing with the previous approach.
Ref (passing with old approach): link
Ref (failing with new approach): link
Any idea? Thanks.


cache token created during DCOS ACS auth flow

Looks awesome @tamarrow. One small comment. :shipit:


Added nice titles to atom feeds

Originally titles in the atom feed used the activity type. This gave no useful information. The user had to open the news item to see what was going on. Now, the title is the activity's message rendered to html.
This fixes #481

I think the travis build failed because of issues with typesafe's repos.
[info] Resolving com.typesafe.sbteclipse#sbteclipse-plugin;2.4.0 ...
[info] Resolving com.typesafe.sbteclipse#sbteclipse-plugin;2.4.0 ...
[info] Resolving com.typesafe.sbteclipse#sbteclipse-core;2.4.0 ...
[info] Resolving com.typesafe.sbteclipse#sbteclipse-core;2.4.0 ...
[error] SERVER ERROR: Internal Server Error url=https://repo.typesafe.com/typesafe/ivy-releases/com.typesafe.sbteclipse/sbteclipse-core/scala_2.10/sbt_0.13/2.4.0/ivys/ivy.xml

Could someone run it again?

@beraboris Amend your commit, make sure you don't have any other changes.
Here's how to amend, git commit --amend -m <message> then you have to use git push -f instead of git push

Looks like there's still something that screws up when installing dependencies. I'll try again later see if it wants to work then.

Looks like the build problem has been resolved.

@beraboris Thanks. I added small fix to remove html tags from title according to GitHub's atom feed.


g.e-hentai的链接会自动询问调用程序打开，exhentai的不会

I got the same issue 29  days ago (https://github.com/seven332/EhViewer/issues/181)
Last release was on Jul 28, so you'll probably have to build it yourself.


Trying to specify a container for acorn logs or acorn exec is confusing

When using acorn logs or acorn exec, it is unclear how to specify the container name.
Like, to get logs for an app named happy-tree, you'd do acorn logs happy-tree. If you want to get the logs for a container named web within happy-tree's acorn image, you'd think the command would be something like acorn logs happy-tree.web, but that's not how it works. You need to get the container name from the output of acorn containers and the command is going to look like:
acorn logs happy-tree.my-app-7657d9bbbf-hmhz6

I think we just need some tweaks to the help commands of logs and exec to explain container names can be obtained from the output the containers command. It would also be nice to have a few examples, like we do in the help text of the acorn run command.

Tested with build from master - jan/16 -  acorn version v0.4.1-112-gb7531fe+b7531fe7
Auto complete is available for containers.
So we should be able to use auto complete feature for fetching the desired containers in acron logs and acorn exec commands.
Autocomplete  defaults to apps , so if the intent was for containers we need to backtrack one space and then enter "." to get to the list of containers with in the app.


Add mapping for ".htm" files

For these old school guys like me who like 3 letters extensions files.
Fixes issue #739.

Thanks!


M1 Mac arm64 issue with AgoraCI Extension

Describe the bug
On a M1 Mac building for our flutter application fails with the following message
building for iOS Simulator, but linking in dylib built for iOS, file '/Users/morne/Documents/Work/repos/drrabbyt/ios/Pods/AgoraIrisRTC_iOS/AgoraCIExtension.framework/AgoraCIExtension' for architecture arm64
To Reproduce
Steps to reproduce the behavior:

Add agora_rtc_engine: ^5.0.1 to the pubspec file
run flutter pub get
run pods install
Click build with a simulator selected
See the error building for iOS Simulator, but linking in dylib built for iOS, file '/Users/morne/Documents/Work/repos/drrabbyt/ios/Pods/AgoraIrisRTC_iOS/AgoraCIExtension.framework/AgoraCIExtension' for architecture arm64

Expected behavior
For the build to complete
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [ iOS 15.4 ]
Host OS: Mac OS Monterey 12.3
Architecture: arm64 (M1 Mac chip)
Version Agora RTC Engine v5.0.1
XCode 13

Additional context
Xcode Build fails with the following message
building for iOS Simulator, but linking in dylib built for iOS, file '/Users/morne/Documents/Work/repos/drrabbyt/ios/Pods/AgoraIrisRTC_iOS/AgoraCIExtension.framework/AgoraCIExtension' for architecture arm64

.

This issue will be fixed in the next release.

This issue is fixed, pls try upgrade agora_rtc_engine: 5.1.0
https://pub.dev/packages/agora_rtc_engine


Add tests

Hi there, maybe a long time ago since, but if you want tests on the app, let me know: acceptance tests with selenium. Maybe too late for unit testing, but sure it'd come out something


Leaflet does not correctly render in chrome with CSS columns

[X] I've looked at the documentation to make sure the behavior is documented and expected
[X] I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x] I've searched through the issues to make sure it's not yet reported

Steps to reproduce
Steps to reproduce the behavior:

Create a div and give it a "column-count: 1" css attribute.
Create a "map" div and render the leaflet inside it

Expected behavior
I expected leaflet to render normaly inside the "map" div.
Current behavior
Leaflet does not render inside this div. Instead, the "cached" tiles around the currently viewed one are also displayed, around the "map" div.
If you zoom in or out, the cached tiles are still displayed, even though they are no longer inside of the div the map is rendered into.
Environment

Leaflet version: >= 1.4
Browser (with version): Google Chrome Version 75.0.3770.100 (64 Bit)
OS/Platform (with version): Independent (verified in Ubuntu 18.04, Mac 10.14.5 and Windows 10)

Additional context
The unexpected behaviour is triggered if the map is found inside of a css grid. Any other display property does what was expected. This isue also does not surface in firefox or safari. Only Chrome appears to have this issue, which leads to the conclusion, that either chromes webkit has a bug, or some -webkit- property is missing in the css, or something similiar.
MVE
http://jsfiddle.net/dz8L3us5/ (Leaflet 0.7.2)
edit: Same bug can be found in Leaflet 1.5.1. Link to the jsfiddle: http://jsfiddle.net/9eqc1fpm/
To see the difference, remove "column-count" from "theBug".

[x] this example is as simple as possible
[x] this example does not rely on any third party code

Changed MVC => MVE (Minimal Verifiable Example)

That fiddle is loading Leaflet 0.7.2. Please update it to 1.5.0 and see if it's still reproducible.

I added a jsfiddle for the new version. Bug still exists. Sorry, i did not doublecheck the version.

@IvanSanchez Does this issue need more infos?

I have the same issue, it seems to be caused by Chrome/Chromium.
Chrome 75.0.3770.142 (the current version) and 76.0.3797.0 (beta version) have the same bug; Chrome 74.0.3729.169 and 77.0.3840.0 (dev version) don't.
Let's hope the fix will be backported to v75 soon.

The unexpected behaviour is triggered if the map is found inside of a css grid. Any other display property does what was expected.

@ThorbenKuck I'm not seeing CSS grid in use, only CSS columns?
Anyhow, I'm not seeing any issues in your fiddles, but it sounds like what you're describing is similar to https://github.com/Leaflet/Leaflet/issues/7699. If that's the case and you still have this problem, can you try the solution described in https://github.com/Leaflet/Leaflet/issues/7699#issuecomment-950251395?

@ThorbenKuck If you're still able to reproduce the bug; can you try page-break-inside: avoid and/or break-inside: avoid (or avoid-column) and see if that solves the problem?
I can't reproduce the bug, and the above https://github.com/Leaflet/Leaflet/issues/6717#issuecomment-511953262 suggests that it was fixed in Chrome shortly after this issue was opened.
I don't think there's anything Leaflet can do here, even if break-inside works to solve the browser bug, it'd be an opinionated change that changes the behavior for authors who actually want to use CSS columns with Leaflet maps.


feat: add some unexpanders

These unexpanders make the Lean 4 prettyprinter behave more like Lean 3. You might think trans e is barely any different to e.trans, but the issue is that if the relevant namespaces aren't open then without the unexpander it's the more ungainly Equiv.trans e.

This is not any attempt to be exhaustive -- these were just the missing unexpanders which I ran into when trying to debug a failing simp in the port; it was just much easier for debugging purposes to make the lean 3 and lean 4 terms look as similar as possible.

LGTM. I added awaiting-review for you, hopefully that is correct.

I love the effect of this PR!
But I wish Kevin wouldn't have to write 5 lines of repetitive code after every defn.
Could this be golfed to @[derive unexpander] def FooBar using some meta-programming?

#3811 gives a possible metaprogramming solution that can replace these with one liners

I'll mark this wontfix for now, as #3811 is a more complete solution. @kbuzzard, perhaps you can close this once #3811 is merged.

Now that we have pp_extended_field_notation in mathlib, I went ahead and updated this PR and removed wontfix, since the dot notation for these things is opt-in. I also added in a couple more missing ones. I hope that's ok @kbuzzard and @semorrison.
I didn't update the PR summary.

bors merge


feat: socket manager and  better application startup

Pull Request Test Coverage Report for Build 400735479

55 of 214 (25.7%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 27.887%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ZwaveClient.js
0
1
0.0%

lib/Gateway.js
1
5
20.0%

lib/SocketManager.js
54
87
62.07%

app.js
0
121
0.0%

Totals

Change from base Build 400262976:
0.5%

Covered Lines:
1855

Relevant Lines:
6770

💛  - Coveralls


Ignore telemetry events when we await a once output message

The telemetry changes broke the await p.dap.once('output') logic because it returned for the telemetry event instead of the output event. Filtering out the telemetry events in once fixes that issue

It looks like there's a couple more locations where this happens, e.g. manually adding onceOnOutput to the "logpoints basic" breakpoint test makes that pass. looking for 'output' in the test file should show you any possible issues.
Alternately maybe we add a telemetry switch or filter at a lower level to avoid this altogether?

Yeah it seems better to just disable sending telemetry entirely


pythonPackages.ppft: use custom checkPhase

Motivation for this change
Fixes #56908
doCheck = true;  fails on python3
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@FRidh thanks for taking a look. You're right. It looks like the project uses python -m ppft.tests to run tests. I'll update the pull request with a custom checkPhase.

Backported in dcaf6f48b8207f5b92330a01ebd5e8fc2665a2fc


First city should automatically have a palace

Buildings (including Palace) have not yet been implemented.

This features has been implemented.


tests: drivers: pwm_api and pwm_loopback tests failed on frdm_k64f boards

Describe the bug
The tests/driver/pwm/pwm_api and tests/driver/pwm/pwm_loopback failed, I found this during the migrating it to the new ztest API.
The test failed due to device is not ready. Seem like it is not configurated correctly on frdm_k64f.
To Reproduce
Steps to reproduce the behavior:

west build -p always -b frdm_k64f tests/drivers/pwm/pwm_api  (also the tests/drivers/pwm/pwm_loopback)
west flash
sudo picocom -b 115200 /dev/ttyACM0
See error

Expected behavior
The test should pass.
Impact
The test is broken on frdm_k64f
Logs and console output
Failed log of pwm_api test:
E: Unsupported clock name
E: Could not get clock frequency
E: Unsupported clock name
E: Could not get clock frequency
*** Booting Zephyr OS build zephyr-v3.1.0-4745-g5c27067dcd9b  ***

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/pwm/pwm_api/src/main.c:20: test_main: (device_is_ready(dev) is false)
PWM device is not ready
E: ***** BUS FAULT *****
E:   Imprecise data bus error
E: r0/a1:  0x00000000  r1/a2:  0x00000000  r2/a3:  0x00000000
E: r3/a4:  0x20000060 r12/ip:  0x00000043 r14/lr:  0x00008d5b
E:  xpsr:  0x61000000
E: Faulting instruction address (r15/pc): 0x000089bc
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
E: Current thread: 0x200001d0 (main)
E: Halting system

Failed log of pwm_loopback test:
E: Unsupported clock name
E: Could not get clock frequency
E: Unsupported clock name
E: Could not get clock frequency
*** Booting Zephyr OS build zephyr-v3.1.0-4745-g5c27067dcd9b  ***

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/pwm/pwm_loopback/src/test_pwm_loopback.c:29: get_test_pwms: (device_is_ready(out->dev) is false)
pwm loopback output device is not ready
E: ***** BUS FAULT *****
E:   Imprecise data bus error
E: r0/a1:  0x00000000  r1/a2:  0x00000000  r2/a3:  0x00000000
E: r3/a4:  0x200000c0 r12/ip:  0x00000043 r14/lr:  0x00009b3f
E:  xpsr:  0x61000000
E: Faulting instruction address (r15/pc): 0x000097a0
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
E: Current thread: 0x20000230 (main)
E: Halting system

Environment (please complete the following information):

OS: Linux
Toolchain:  Zephyr SDK 0.15.0 and 0.14.2
Commit SHA:  In any recent commits. bisecting it.

Additional context

debug found the error is at ftm initial
mcux_ftm_init (dev=<optimized out>) at /home/shared/disk/zephyr_project/zephyr_test/zephyr/drivers/pwm/pwm_mcux_ftm.c:398
398             if (err != 0) {
(gdb)
402             if (config->channel_count > ARRAY_SIZE(data->channel)) {
(gdb)
407             if (!device_is_ready(config->clock_dev)) {
(gdb)
412             if (clock_control_get_rate(config->clock_dev, config->clock_subsys,
(gdb)
414                     LOG_ERR("Could not get clock frequency");
(gdb)
415                     return -EINVAL;

@galak , @dleach02 , @mmahadevan108


Github Pre-Release Trigger failure

Description
The "pre-release" trigger for Github Actions is not working as expected. However, we discovered that changing the trigger to "published" in our .github/workflows/release_trigger.yml file should start triggering the workflow. The problem with the current behavior is that when deploying and creating a pre-release, it doesn't trigger a deploy for staging, which requires manual work.

[x] discuss with QA

Steps to Reproduce

Create a pre-release
Publish the pre-release
Observe that the "deploy to staging" workflow is not triggered

Workaround
Change the trigger in the .github/workflows/release_trigger.yml file from [release, pre-release] to [published].
Impact/Urgency
This bug has a medium impact on our system and team, as it causes delays and requires a manual step in our process.
Expected Behavior
When a pre-release is created, the "deploy to staging" workflow should be triggered automatically.

[x] The "deploy to staging" workflow should run automatically
[x] An engineer should not have to manually deploy to staging
[ ] If this does not fix it, create a new ticket

QA Considerations

[x] create a draft pre-release for staging
[x] after the draft has been saved, publish the pre-release to staging
[ ] If the code is deployed, the fix works, else we need a new approach.

Additional Info & Resources

Link to the .github/workflows/release_trigger.yml file where the trigger can be modified: https://github.com/department-of-veterans-affairs/notification-api/blob/master/.github/workflows/release_trigger.yml#L4
The Github Actions Documentation that indicates the change that will likely fix the issue

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @jakehova @justaskdavidb2 @k-macmillan @kalbfled @ldraney

@cris-oddball Is this still happening/intermittent? It's been in next up for a while but we previously weren't ready to prioritize it due to the flakiness of the issue and insufficient data. Should it be in the backlog?

@mjones-oddball It's intermittent. We should probably work it, since the change is minor and it would be good to know if the fix actually works.

Created a tag from master, after I merged Lucas's code into master that contained the potential fix for the -pre-release trigger.
The Action did kick off - so in that sense it 'worked' but the deployment failed as shown above.
Link to failed deployment:
https://github.com/department-of-veterans-affairs/notification-api/actions/runs/5326447256

specifically it's failing because of:

The task definition has an invalid path.

the problem is this:
          script: |
            if ('${{ github.event.action }}' === 'prereleased') {
              core.setOutput('environment', 'staging')
            }
            if ('${{ github.event.action }}' === 'released') {
              core.setOutput('environment', 'prod')
            }

needs to be changed to something like this:
          script: |
            if (context.payload.release.prerelease) {
              core.setOutput('environment', 'staging')
            } else {
              core.setOutput('environment', 'prod')
            }

But the problem with the above is it will deploy to prod every time staging isn't correctly called!  Prod should not be the default.
I'm considering something like:
          script: |
            if (context.payload.release.prerelease) {
              core.setOutput('environment', 'staging')
            } else if (context.payload.release) {
              core.setOutput('environment', 'prod')
            } else {
              core.setOutput('environment', '')
            }

But I first need to confirm payload.release will work, but I'm having a hard time finding the relevant documentation.

I'm reading this documentation to try and make sure context.payload.release will work

Also perhaps a relevant stackoverflow

Also this webhook events  is probably also relevant

I should be able to make a gh cli command to verify this, but this is taking me time to figure out

At first I thought Github context is probably not what I’m looking for because the trigger is an action, not a previous workflow from a push event. A push event occurs when a commit or tag is pushed.
However, “this object is identical to the webhook payload of the event that triggered the workflow run, and is different for each event. The webhooks for each GitHub Actions event is linked in "Events that trigger workflows." For example, for a workflow run triggered by the push event, this object contains the contents of the push webhook payload.
Note: The prereleased type will not trigger for pre-releases published from draft releases, but the published type will trigger. If you want a workflow to run when stable and pre-releases publish, subscribe to published instead of released and prereleased.

I realized the best way to test triggers is on a personal repo, so I set up some actions similar to notification-api:

I've confirmed that this trigger isn't working yet:
name: Release Trigger
on:
  release:
    types: [published, released]

jobs:
  setup-release:
    runs-on: ubuntu-latest
    outputs:
      git-tag: ${{ steps.tag.outputs.version }}
      env: ${{ steps.setup.outputs.environment }}
    steps:
      - name: Check release
        run: |
          echo "Release tag: ${{ github.event.release.tag_name }}"
          echo "Release name: ${{ github.event.release.name }}"
          echo "Is pre-release (boolean): ${{ github.event.release.prerelease }}"
          echo "Is release (string): ${{ github.event.action }}"
      - name: Setup Environment
        id: setup
        uses: actions/github-script@v6
        with:
          script: |
            if (github.event.release.isPrerelease) {
              core.setOutput('environment', 'staging')
            }
            if ('${{ github.event.action }}' === 'released') {
              core.setOutput('environment', 'prod')
            }

Playing with it in my personal repo it seems that the REST documentation is the most useful

My solution looks like this:
name: Pre-release Trigger
on:
  release:
    types: [published, released]

jobs:
  setup-release:
    runs-on: ubuntu-latest
    outputs:
      git-tag: ${{ steps.tag.outputs.version }}
      env: ${{ steps.setup.outputs.environment }}
    steps:
      - name: Check release
        run: |
          echo "Release tag: ${{ github.event.release.tag_name }}"
          echo "Release name: ${{ github.event.release.name }}"
          echo "github.event.release.prerelease: ${{ github.event.release.prerelease }}"
          echo "github.event.release.isPrerelease: ${{ github.event.release.isPrerelease }}"
          echo "github.event.action: ${{ github.event.action }}"
          echo "github.payload: ${{ github.event.release.release }}"
      - name: Setup Environment
        id: setup
        uses: actions/github-script@v6
        with:
          script: |
            if ('${{ github.event.release.prerelease }}' === 'true') {
              core.setOutput('environment', 'staging')
            }
            if ('${{ github.event.action }}' === 'released') {
              core.setOutput('environment', 'prod')
            }

  run-deployment:
    needs: [setup-release]
    uses: ./.github/workflows/deployment.yml
    with:
      environment: "${{ needs.setup-release.outputs.env }}"

Pros: actions now work as intended
Cons: The workflow runs twice when deploying to prod, though it will effectively only be one, just may confuse the developer who is deploying.

I meant to post the above sooner, sorry about that.
Kimball, a fellow Oddball engineer, suggested adding:
  # A workflow run is made up of one or more jobs that can run sequentially or in parallel
jobs:
  init:
    runs-on: ubuntu-latest
    # This filters out pre-release releases
    if: "!github.event.release.prerelease"
    # when a non draft PR is created TODO test actor on new repo!!!!
    steps:
      - name: Init
        run: |
          echo Deploy starting
          echo ${{ github.actor }}
***
  setup-release:
    needs: [init]

Based on how they addressed this problem, but it still shows up in our run actions, just grayed out:

But at least greyed out is a little better than appearing to both run or one run one fail.  I think I will incorporate this since it does add some clarity to our Actions tab.

I think this is our best solution:
on:
  release:
    types: [published, released]

  # A workflow run is made up of one or more jobs that can run sequentially or in parallel
jobs:
  init:
    runs-on: ubuntu-latest
    # This filters out pre-release releases
    if: github.event.release.prerelease == true || github.event.action == 'released'
    # when a non draft PR is created TODO test actor on new repo!!!!
    steps:
      - name: Init
        run: |
          echo Deploy starting
          echo ${{ github.actor }}

  setup-release:
    needs: [init]
    runs-on: ubuntu-latest
    outputs:
      git-tag: ${{ steps.tag.outputs.version }}
      env: ${{ steps.setup.outputs.environment }}
    steps:
      - name: Check release
        run: |
          echo "github.event.release.prerelease: ${{ github.event.release.prerelease }}"
          echo "github.event.action: ${{ github.event.action }}"
      - name: Setup Environment
        id: setup
        uses: actions/github-script@v6
        with:
          script: |
            if ('${{ github.event.release.prerelease }}' === true) {
              core.setOutput('environment', 'staging')
            }
            if ('${{ github.event.action }}' === 'released') {
              core.setOutput('environment', 'prod')
            }
            else {
              console.log('Pre-release is false. Stopping workflow.')
              process.exit(0)
            }

It works on prerelease as intended with draft.
and it shows a skipped prerelease trigger like such (ignore that the workflow is called pre-release):

Lucas created PR, reviewed it with me line by line, we merged it to master, we cut a tag and pushed that tag. Created a pre-release as a draft. Exited the draft. Returned to the draft and published it. The deployment started to staging.


Allow binary fixtures

Fixtures works well for template files, but don't allow binary files (or any other kind of non-template files).
A typical use case is when testing a front-end app relying on binary files downloaded over the network (e.g. a PDF downloaded by pdfjs)
It appear that the issue originate from commit 1931adc300892b9266986f58e837c8803d8e2e2b, which is forcing the fixtures to be valid, UTF8-encoded templates.
I can submit a PR for a fix, but I wanted to open the discussion first.
I'm thinking about either doing some file type recognition to render the correct type of fixture, or creating a new fixture endpoint designed for binary files. Of course, we'll have to keep issue #264 in mind, making sure it won't create a new vulnerability.

I'm not sure if "fixture" is the right word here, but I also could make use of a way to access static binary files from within teaspoon. I have some javascript that needs to reference some image files and I don't currently have a good place to put those files.
Can you recommend a solution or workaround?

I'd say you can put them in any place that you are serving assets from -- remembering that you can add any paths to your assets path, including those that are within specs (obviously not in prod environments though). Typically you would need to load those files differently than a fixture would load (as in, it doesn't make sense to put them in the DOM or try to parse them, so go ahead and implement what you need in your project. I'd say a simple ajax request should give you a binary file, or in the case of images, putting the url to it as the source of an image tag that's in a fixture. In general though, that's all that makes sense, and adopting more code for a semi-rare use case probably wouldn't be useful.
So, to summarize, if you want it as a "fixture", put an image or anchor tag with a reference to the file you want to load as a src, or href attribute, and then load them within your test, however you think you should.
The one outlier to this is the file input, but since you're within the confines of javascript, you can't do anything here anyway.


Upgrade terraform-provider-aws to v5.20.1

This PR was generated via $ upgrade-provider pulumi/pulumi-aws --upstream-provider-name=terraform-provider-aws --kind=provider.

Rebased patches to solve go.sum merge conflicts.
Upgrading terraform-provider-aws from 5.19.0  to 5.20.1.
Fixes #2869
Fixes #2860

LGTM except those TODOs, looks like a small clean update.


October Month BUG

Hi,
The October month is with a Saturday repeated.
This ocurred in everey year.
Atencious,
Lucas Machry

please check out the development branch.
duplicate of #855 #881


Update how hmi-server interacts with sciml kickoff and mq

Description of Issue
Update HMI-Server to give our simulation services a queue name per user when a calibration is run.

[ ] Modify the calibration feignclient and controller to give our user specific queue name
[ ] Generate the correct queue name for the user in question.
[ ] Update simulation service to use correct queue name

Note this issue will need to be done in sync with #2238


DEMO-51/DEMO-52: Add Link to Article Source

Adds a simple link element next to the difficulty tag which dynamically links to the source of the currently loaded article. Includes an icon to indicate link will open in another tab.

Huly®: GITHB-106

Done!


Provide a way to mute notification for specific channels of a team

Currently, all messages in channels towards which an user is subscribed produce a notification. On the other hand both Slack and Discourse allow the user to manage which channels are supposed to produce notifications.
It would be great if the Keybase team chat would have similar ways to manage notifications on a per-channel basis.

This exists now. On the desktop app if you click the "i" logo and then choose "Settings" you should see notification settings for that channel.

Does this only exist on the desktop? What about iOS/Android?

The same option is available on iOS

The same option is available on iOS


feat(routing): add a "single index" compatibility mode

This mode has as goal to flatten the URL structure to avoid the indexName of the root index in the URL when there's only one URL.
Usage:
import { history } from 'instantsearch.js/lib/router';
import { singleIndex } from 'instantsearch.js/lib/stateMappings';

instantsearch({
  // ...
  indexName: "blabla",
  routing: {
    router: history(),
    stateMapping: singleIndex("blabla"),
  },
});

Right now it's not yet possible to use this router in TS because InstantSearch itself isn't generic, and trying to make this happen was causing me a cascade of Generic issues. For now you need to cast it (stateMapping: singleIndex("blabla") as StateMapping<UiState>).
BREAKING CHANGE: Default URLs have changed, and now will contain the full state, which includes the index name. You can use the old behaviour by using the singleIndex stateMapping, and passing the index to synchronise in the argument.
https://algolia.atlassian.net/browse/IFW-941

I found opt-in the most elegant. We could make a version which has access to the instance & the tree, but the logic seemed too inflexible to me


Interpretation of max_len in graph sampling

Hi,
Thanks for the great work!
I am going through the sEH_frag task, and am confused by the 'max_len' parameter.
My intuition is the sampler (at least initially) should be able to sample across all possible graphs (ie. different total nodes). However, when I run the code in a purely online setting with offline_ratio=0, graphs of length equal to max_len are the only ones being sampled initially.
I tested and observed that the sampler is capable of producing graphs that are smaller than max_len (if mandated by the reward function) but only after a majority of samples around max_len.
Is this expected behavior or are there tunable parameters that can help me produce graphs of different node lengths in the first few batches?

Hi, this is somewhat expected behavior if the reward doesn't penalize large objects.
One way to think about it is that if there are 1000 actions, then there are (roughly) 1000 times more objects of size n+1 than of size n. If the reward is mostly independent of size, then the correct reward-proportional sampling policy should sample objects of size n+1 1000 times more often than objects of size n. This of course compounds exponentially.
Typical strategies to mitigate this are to penalize size, or to "early stop" trajectories with some probability (this is something we've done in the past but is not implemented here; although it should be easy to add to GraphSampler).


JSON.Dump converts float numbers to string

Thank you for your fantastic work. Your project has been an immense help to me during my recent AHK project, and I'm grateful for the time and effort you put into it.
However, I wanted to bring a small issue to your attention. I noticed that the method JSON.Dump automatically converts float numbers to a string. While this works in most cases, some servers expect the parameter to be a float instead of a string. This can cause issues for users who are trying to send data to these servers.

Please refer to the type conversions table in the Readme. It explains how float conversions work, and how to make them do what you've asked here. If you have any further questions, feel free to re-open the issue.

Thanks @G33kDude! This solves my issue.


More advanced syntax highlighting

I've noticed that syntax highlighting is a bit too simple.
Just to compare unison share and vscode highlighting:
Unison share:

VSCode:

Yeah, the color schemes are different, but the highlighting itself seems to be completely different too.
Is it hard to make it work similarly to unison share (and/or the haskell vscode extension)
Here's a sample for Haskell

In the unison file I somehow expected None and helloHandler to have different colors
But I like how function definitions differ from function declarations (this is what I am used to see it in haskell)
Maybe some more things I just can't spot right away

I think likely the best way to do this would actually be through the Semantic Tokens LSP request, since in Unison we can't know the difference between a term, type, or constructor name through syntax alone, you need access to the codebase.
It should be relatively straight-forward to analyze the Haskell type for a parsed file and produce the semantic tokens response, but we're a bit busy lately so it's unlikely to happen in the short term.
In the meantime though we'd accept reasonable updates to the Regex-based highlighting, or we'd love to have a Tree-sitter grammar if anyone's up to that task 😄

For anyone reading this, I'm pretty close to being done with a tree sitter grammar at https://github.com/kylegoetz/tree-sitter-unison
(I came here because I was looking at how the VS Code plugin does highlighting for doc blocks)


Make use of data more by devising subsampling

When we use a certain memory_allocation [^1] in subsampling, we reduce the number of samples until we reach the memory limit.
However, we need to come up with an appropriate value for this as when we set it too high, the training fails due to memory error while when we set it too low, we waste memory.
For now, we circumvent this issue by measuring the memory consumption when using the default config.
[^1]: The definition of the memory_allocation is the following:
Absolute memory in MB, e.g. 10MB is "memory_allocation": 10.
The memory used by the dataset is checked after each reduction method is performed.
If the dataset fits into the allocated memory, any further methods listed in "methods" will not be performed.

Those passed when I ran with example for the tabular classification.
mem_limit, N, D = 3000, 500, 10000  # 3GB
mem_limit, N, D = 4000, 2500, 10000  # 4GB
mem_limit, N, D = 5000, 4500, 10000  # 5GB
mem_limit, N, D = 6000, 6500, 10000  # 6GB
mem_limit, N, D = 7000, 8500, 10000  # 7GB
mem_limit, N, D = 8000, 11500, 10000  # 8GB
mem_limit, N, D = 9000, 15000, 10000  # 9GB

Note that since neural networks become larger when using a larger input size, I used a large fixed D and this D is determined via the feature size in automlbenchmark.
Since when we use memory_limit=2000, APT does not run, we use the following equation to calculate the memory_allocation and raise an error when we get memory_allocation < 0:
memory_allocation = (memory_limit - 3000) / 1000.0 * 120 + 500  # for 64bit
memory_allocation /= 2  # for 32bit

The information is based on the runs on common modification branch.
mem_limit, N, D = 3000, 500, 10000  # 3GB
mem_limit, N, D = 4000, 3000, 10000  # 4GB
mem_limit, N, D = 5000, 5500, 10000  # 5GB
mem_limit, N, D = 6000, 8500, 10000  # 6GB
mem_limit, N, D = 7000, 11500, 10000  # 7GB
mem_limit, N, D = 8000, 15500, 10000  # 8GB

memory_allocation = (memory_limit - 3000) / 1000.0 * 200 + 40


Replace progress bar implementation for file upload

The current solution for showing a progress bar on file upload is no longer maintained.
We need a replacement. This is maybe a good solution, just based on vuejs?
https://serversideup.net/file-upload-progress-indicator-with-axios-and-vuejs/

Btw: This error sometimes happens during upload:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.9/site-packages/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/opt/conda/lib/python3.9/site-packages/django/core/handlers/base.py", line 171, in _get_response
    response = middleware_method(request, callback, callback_args, callback_kwargs)
  File "/opt/conda/lib/python3.9/site-packages/django/middleware/csrf.py", line 297, in process_view
    request_csrf_token = request.POST.get('csrfmiddlewaretoken', '')
  File "/opt/conda/lib/python3.9/site-packages/django/core/handlers/wsgi.py", line 102, in _get_post
    self._load_post_and_files()
  File "/opt/conda/lib/python3.9/site-packages/django/http/request.py", line 362, in _load_post_and_files
    self._post, self._files = self.parse_file_upload(self.META, data)
  File "/opt/conda/lib/python3.9/site-packages/django/http/request.py", line 322, in parse_file_upload
    return parser.parse()
  File "/opt/conda/lib/python3.9/site-packages/django/http/multipartparser.py", line 264, in parse
    chunk = handler.receive_data_chunk(chunk, counters[i])
  File "/opt/conda/lib/python3.9/site-packages/progressbarupload/uploadhandler.py", line 37, in receive_data_chunk
    data['received'] += self.chunk_size

Exception Type: KeyError at /manager/surface/1780/new-topography/
Exception Value: 'received'

When looking here at the code of the progress bar handling, it seems that it's buggy and does not handle
the case when a cache value is not available. Another argument for replacing the library.

We could replace this in conjunction with implementing #173

Resolved with #1012


Abracadabra Arbitrum: subgraph stopped syncing

Indexing errors on abracadabra
arbitrum: 44305314

Just commented this on an open Abra-avax issue, but posting it here since it looks to affect Abra-arb as well
Abracadabra Arbitrum marketDailySnapshots do not have any data in the rate fields, regardless of which market.
https://api.thegraph.com/subgraphs/name/messari/abracadabra-money-arbitrum/graphql?query={
++markets+{
++	dailySnapshots+(first%3A+1)+{
++++++id
++++	rates+{
++++++	id
++++	}
++++}
++++hourlySnapshots+(first%3A+1)+{
++++++rates+{
++++++++id
++++++}
++++}
++}
}

This is on the graph side looks like its fixed now. I moved the 0 marketDailySnapshots to this issue: https://github.com/messari/subgraphs/issues/1293. Going to close this


WebSocket open error

So this happened. Everything worked well yesterday, now I can't see the room url or connect with people.

Latest push seems to have broken signalling, I noticed the room name is not appended to the URL anymore.

0.304: Initializing; server= undefined.
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 0.304: Initializing; room=982495439.
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 0.348: Got access to local media with mediaConstraints:
'{"audio":true,"video":{"optional":[{"minWidth":"1280"},{"minHeight":"720"}],"mandatory":{}}}'
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 0.348: User has granted access to local media.
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 0.513: Retrieved TURN server information.
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 1.685: Opening signaling channel.
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 1.831: Joined the room.
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 WebSocket connection to 'wss://apprtc-ws.webrtc.org/ws' failed: Error in connection establishment: net::ERR_TIMED_OUT
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 31.821: WebSocket open error: WebSocket error.
apprtc.debug.js.pagespeed.jm.tFx8UtP6de56uECGa3-X.js:1 31.822: WebSocket register error: WebSocket error.

I do not think server= should be undefined.

This commit 5a287975f2b132c4b3eca63b8d5eadfd5fe85fd7 changes:
WSS_HOST = 'apprtc-ws.webrtc.org'
-WSS_PORT = '8089'
To:
WSS_HOST_PORT_PAIR = 'apprtc-ws.webrtc.org:443'
Could it be a just a connection issue to the websocket server?

So when I try using https://www.websocket.org/echo.html I get ERR_CONNECTION_REFUSED for wss://apprtc-ws.webrtc.org:8089 while I get ERR_TIMED_OUT for wss://apprtc-ws.webrtc.org:443.
Was this port change intentional?

server=undefined is expected for apprtc, it is only defined in the Chrome app.

The room is not appended because of what you later found, the websocket connection is timing out:
Error from console: WebSocket connection to 'wss://apprtc-ws.webrtc.org/ws' failed: Error in connection establishment: net::ERR_TIMED_OUT

I suppose this is related to the collider prober errors?

The service has recovered. The code bug in Collider is tracked in a separate bug.

Go to  src/app_engine/constants.py remove the old line and replace it with the line below instead, it should work:
WSS_HOST_PORT_PAIRS = ['apprtc-ws.webrtc.org:8089', 'apprtc-ws.webrtc.org:8089']
I got this from the android app sample

Hello pleas is apprtc  open source cos am getting this WebSocket register error: Failed to join the room: Status=404, response


Typos in simple-application.md

Description:
Trying to setup the simple application I found some typos in simple-application.md:
Actual
Now, let's modify the UserList view so that we can navigate from there to a UserForm:
// src/views/UserForm.js
..

Expected
Now, let's modify the UserList view so that we can navigate from there to a UserForm:
// src/views/UserList.js
..

Actual
Let's create a file src/views/Layout.js:
var Layout = {
         ..
}

Expected:
Let's create a file src/views/Layout.js:
var m = require("mithril")
var Layout = {
          ..
}
module.exports = Layout

Actual
The form itself still doesn't save when you press "Save". Let's make this form work:
// src/views/UserForm.js
var m = require("mithril")
var User = require("./model/User")
..

Expected
The form itself still doesn't save when you press "Save". Let's make this form work:
// src/views/UserForm.js
var m = require("mithril")
var User = require("../model/User")
..

Actual
Let's change the router in index.js to add our layout into the mix:
// index.js
var m = require("mithril")

var UserList = require("./view/UserList")
var UserForm = require("./view/UserForm")

m.route(document.body, "/list", {
..
})

Expected:
Let's change the router in index.js to add our layout into the mix:
// index.js
var m = require("mithril")

var UserList = require("./view/UserList")
var UserForm = require("./view/UserForm")
var Layout = require("./view/Layout")

m.route(document.body, "/list", {
..
})

Thanks for catching those! Fixed


Change pron display

Currently, alternate prons for the same word are listed side by side even though they are separated by \n in the data file

I think it would be good to do a line separated display e.g.,
/bEN/
/beIN/

But that would probably mean having to place this somewhere else inside of beside the word? Or maybe something like
/bEN/, /beIN/

Wiktionary does the latter:
/ˈkɑːsəl/, /kɑːsl̩/


handleClickOutside doesn't work when providing own input element

handleClickOutside relies on refs to the input element and the picker modal, which are not available when using an individual input element.

You could add a blur listener to set the "show" state to false as a workaround here, I guess. Hope this helps :)

@carstenblt I've created a PR to pass inputRef with options: https://github.com/OMikkel/tailwind-datepicker-react/pull/53

You could add a blur listener to set the "show" state to false as a workaround

I tried this but that made the picker disappear before a value is chosen


Use cstdint types across the board

There's a mixture of numeric types used in UHDM and Surelog - int, unsigned, unsigned int long, long long, etc
Can we enforce use of standard defined types like int32_t, uint32_t, int64_t, etc??
For whatever isn't available, define an explicit type in something like config.h.

Sure, please submit respective PRs.

Fixed by #3580


Prevent double waitgroup decrement

A user in discourse has discovered that under heavy loads it is possible to cause a double decrement in the server connection waitgroup.
This PR prevents this.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d795206). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master   #10170   +/-   ##
=========================================
  Coverage          ?   43.41%
=========================================
  Files             ?      576
  Lines             ?    79642
  Branches          ?        0
=========================================
  Hits              ?    34580
  Misses            ?    40786
  Partials          ?     4276

Impacted Files
Coverage Δ

modules/graceful/server.go
49.28% <100%> (ø)

Continue to review full report at Codecov.

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

The tests that failed before the latest commit (*int32 vs. int32) show that this was a real problem.


Return the characteristic object when resolving start/stopNotificatio…

…ns promise
Fixes issue #55

Superceded by #61


Bump serverleass k8s dependency to a common version

Description
Changes proposed in this pull request:

Bump serverleass k8s dependency to a common version to simplify go.mod merge afterwards
fix failing test

Related issue(s)

#533

/retest

/retest


Task - Custom eor and eof for BLM fie generation

Description:

Few properties has ~ symbol in their media urls and some properties has ~ or '|' or '^' in their description.  Using these characters as eor or eof in blm might result in parsing error. To Prevent such errors, we need to make sure that the symbols used as eor or eof should not conflict with property data.

Actions:

Changes made to accept input values for eof and eor fields and use those values in blm file generation to avoid conflicting with property data containing these symbols

Nice, that was a big oversight in the implementation, thanks for fixing !


Send clarification email through Notify

Summary
When a supplier submits a clarification question, the user who submitted
it will receive email confirmation that it has been submitted
successfully. This is currently sent through Mandrill, but we want to
try to move more of our notification emails over to Notify.
Ticket
https://trello.com/c/7dRVK9Yt/68-send-asked-clarification-question-to-supplier-emails-through-notify
Question
One thing to think about is if this is how we want to handle instantiating the Notify client or if we should do it in a similar way to the API clients. If the latter, we'll need to re-work how the Notify client is constructed (i.e. give it an init_app thing and allow it to take the tokens outside of the initializer). I suspect probably not. I started looking into https://github.com/alecthomas/flask_injector as a way of injecting dependencies into views, but haven't dug down into this much yet.

TODO: Do not merge until functional tests have been checked/updated.


add .editorconfig

http://editorconfig.org/
I had some issues conforming to existing coding styles :smile:

Cool, thanks!


hyperion-x11 - FLATBUFCONNECTION Unable to parse message

Hi,
hyperion-x11 is chrashing very often with [hyperion-x11 X11GRABBER]  Unable to parse message
I tried to attache gdb but it looks like the QT handling stops my from getting a stack dump.
I am a C/C++ developer but I have never worked with QT, if any more Input is needed feel free to ask.
Starting program: /home/links/build/hyperion.ng/hyperion/build/bin/hyperion-x11 -a <IP_ADDRESS>:19400
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
hyperion-x11:
	Version   : 1.0.0 (master (Paulchen-Panther-722915a4/4c0c102f-1573403044) Git Remote: https://github.com/hyperion-project/hyperion.ng.git)
	build time: Nov 10 2019 18:33:32
[hyperion-x11 X11GRABBER] <INFO> Update of screen resolution: [0x0]  to [1920x1080]
[hyperion-x11 X11GRABBER] <INFO> Using XRender for grabbing
[hyperion-x11 X11GRABBER] <INFO> Update output image resolution: [0x0]  to [240x135]
[hyperion-x11 X11GRABBER] <INFO> Capture interface is now enabled
[hyperion-x11 FLATBUFCONNECTION] <INFO> Connecting to Hyperion: <IP_ADDRESS>:19400
[hyperion-x11 X11GRABBER] <INFO> Update of screen resolution: [1920x1080]  to [1920x1080]
[hyperion-x11 X11GRABBER] <INFO> Using XRender for grabbing
[hyperion-x11 X11GRABBER] <INFO> Update output image resolution: [240x135]  to [240x135]
[hyperion-x11 FLATBUFCONNECTION] <INFO> Connected to Hyperion: <IP_ADDRESS>:19400
Qt has caught an exception thrown from an event handler. Throwing
exceptions from an event handler is not supported in Qt.
You must not let any exception whatsoever propagate through Qt code.
If that is not possible, in Qt 5 you must at least reimplement
QCoreApplication::notify() and catch all exceptions there.

[hyperion-x11 X11GRABBER] <ERROR> Unable to parse message
[Inferior 1 (process 342) exited with code 0377]

I will close this, if you need still help for this, please reopen it.
thanks


tf_prefix param: fix slashes and add to IMU Broadcaster

Related with #1102, rabase to master.

@saikishor FYI

Hi @christophfroehlich @saikishor,
After the initial exchange of knowledge, the topic died, has something happened, are we waiting for something?

@rafal-gorecki is still necessary, if yes, could you please resolve the conflicts introduced from #1420?


moreores and technic should be handled for mineclone* games

both of them gained support for mineclone* games recently:
https://github.com/minetest-mods/moreores/pull/55 (silver, mithril and tin)
https://github.com/mt-mods/technic/pull/328

feel free to make a pr


Add RocksDB log implementation

RAFT log stored in RocksDB database. The RocksDB dependency is optional
so users need to add it manually if they want to use it.

Didn't replace the LevelDBLog with RocksDBLog because it the RocksDB performance is worst (35,715 reqs/sec/node) than LevelDB (42,301 reqs/sec/node).
On other hand, the LevelDB performance tests are full of exceptions like:
87442 [ERROR] RAFT: B: failed advancing commit_index (844775) to 921570: java.lang.IllegalStateException: B: log entry for index 844776 not found in log

and
55472 [ERROR] RAFT: A: resending of 917566 failed; entry not found

While the RocksDB performance tests do not have any exception at all.

@belaban when you have time, can you check the RocksDB performance in your machine?
I'll forward-port to 5.x if we decided to integrate this :+1:

Yes, let's definitely add this; it doesn't change existing code...

Don't forget to add the dep to ivy.xml, too

I added a new perftest (CounterPerf); if you submit the forward-port to master before our meeting today, I can demo this as well!

Why don't you apply the change to IspnPerfTest? It would be nice to have a direct comparison...

I'm implementing the Cache interface where cache.put() increments the counter and cache.get() returns the cache value. Other methods aren't implemented. That's why I didn't open a PR against your repo.
https://github.com/pruivo/IspnPerfTest/blob/t_counter_perf_test/src/main/java/org/counter/jgroups/JGroupsCounter.java

So why not provide interface Counter and copy the perftest? This would be a new perftest only for counters...

That was the plan but I was lazy and did the fast code :)

OK, no problem! At some point, I should beef up IspnPerfTest anyway: newer JGroups versions, Infinispan etc..


Functionality to download the html and css code directly of the respective buttons. 🤔

Need some functionality to download the code directly into your local machine with ease.

can I have a go at this?
please assign this to me...

Yes Sure.

Hey, @Nevish-302 What are the updates on this issue?
I had assigned it to you.

It's India so that was 3 o clock at night... I'm working on it not. Well get done shortly

i have added this funtionality to one button tell me if i need to use another method or need to change the style.

@Nevish-302 It is amazing. 🚀
But can we download both files by clicking on just download button rather than downloading HTML and CSS files separately?
It will be a good user experience.

I have modified it accordingly. Check it in my forked repository

I think there are some problems with the code I will strongly suggest you to recheck it. @Nevish-302

I've fixed'em please recheck it

@Spyware007 any updates?

@Nevish-302 Would you like to become a maintainer of this repository?
Lets meet here. https://discord.gg/ZsMAyV6e

Merged.


Base64DataUrl ngf-pattern validation error

Hello,
Have an issue with validation on a Base64Data image.
If i put on the model a base64 image I get this error:
TypeError: files.splice is not a function
    at validateSync (ng-file-upload.js:1231)
    at upload.validate (ng-file-upload.js:1247)

I get this error when I drag or select a new image and I convert the image to base64 which is the format I need to send to the backend.
Created an example with the issue Plunker
Let me know if you need more info.

Should  be fixed at 11.2.2

Thank you :D


Log version

Log the version upon startup
Last git tag is used as version
time="2018-11-15T12:27:35Z" level=info msg="Service Manager Version: v0.1.1-11-gc06cba6 (c06cba6ed82255fc577377d9ef8e5be209237d6a)"

If the last commit is tagged
time="2018-11-15T12:27:35Z" level=info msg="Service Manager Version: v0.1.1 (c06cba6ed82255fc577377d9ef8e5be209237d6a)"

Pull Request Test Coverage Report for Build 1361

0 of 5 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 91.491%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

main.go
0
2
0.0%

pkg/version/version.go
0
3
0.0%

Totals

Change from base Build 1346:
-0.2%

Covered Lines:
2215

Relevant Lines:
2421

💛  - Coveralls


Documentation Request for a Rails 6.x app with 5.2 defaults

Documentation Request
Stimulus Reflex looks awesome and the documentation appears to have been crafted with great care and loving attention to detail!
However, my particular use case has not been mentioned, and it's unclear to me if one that has been mentioned is directly applicable.  I have a Rails 6 app that still has Rails 5.2 defaults (config.load_defaults 5.2).
It would be really helpful if you could clarify whether the Rails 5.2+ Support section is directly applicable, or whether I would have to modify the directions there in some way.
https://github.com/hopsoft/stimulus_reflex/blob/master/docs/setup.md
The features I'm working on in my app were just reprioritized, so I'm no longer in a position (at the moment) to take time to verify my hunch that the real question is simply: are you using webpacker or sprockets?, and the version of Rails is immaterial.  But it would be nice if this was specifically mentioned in the README, to make things clear for this situation.
Thanks!

Hi Brandon!
Everything that you're saying makes sense on a high level. I should be clear that I really wasn't doing much development during the v4-5.x time period, so I'm really only familiar with load_defaults in the same way that I've never seen "The Godfather" but I feel like I could describe it to you.
Realistically, while I genuinely love that writing documentation forces me to get more familiar with corners of Rails I wouldn't otherwise get the opportunity to work with, realistically I am not gunning for certification-level intuition about every detail. That means that you're actually far closer to being the person to drive this part of the documentation than I am. (I certainly don't feel like I'm holding anything back, so if stuff is missing...)
What I can say is that it would be great to have a dialogue in this issue where you make suggestions and I transmute them into the documentation. I am not realistically going to have time to set up a Rails 6 app that is running with 5.2 load defaults and figure out all of the subtleties. I am, at the end of the day, a volunteer... but the bigger issue is that there's honestly just too many priorities (even for documentation!) waiting for my attention in this project, and I don't want to build any false hope.
If you or someone else that comes along is able to give me a solid list of changes, I will definitely treat them with respect.

Okay, I understand a little better now.  I (wrongly) assumed that it was simply a matter of not thinking to mention that situation, rather than not actually knowing precisely how it would work.  I expect to come back to this in the future, so I'll try to contribute my experiences to clarifying this.

Hey @brandondrew I just wanted to follow-up and ask if you'd unblocked yourself? Happy to work your experiences into the documentation.