match dashboard master detail process selected item behavior for both Win.Web

In Windows the processing will be done on CurrentObjectChanged in addition to the default Xaf gridlisteditor double click.

Installer lab build 18.1.603.06 includes commit match dashboard master detail process selected item behavior for both Win.Web #216 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS

Installer lab build 18.1.603.21 includes commit match dashboard master detail process selected item behavior for both Win.Web #216 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Refactor to only use "publish" when referring to announcement channels

The word "publish" is often use synonomously with "send embed", however "publish" refers to announcement channels in the context of discord.
Tweak phrasing wherever possible.
ex. --publish-on-startup arg.

Completed


Added DB2 database driver

This Pull Request solves #5.
Tested on Ubuntu 14.04 x64 "DB2 v<IP_ADDRESS>", "s141128", "IP23633", and Fix Pack "5"
PHP 5.5.9, PDO_IBM-1.3.3
Tests: 212, Assertions: 1366, Errors: 1, Failures: 1, Skipped: 2.
pdo_ibm is buggy, e.g. using PDO::PARAM_NULL throws error.
It would be nice, if someone update pdo_ibm.
test config
$config['databases']['ibm'] = [
    'dsn' => 'ibm:yiitest',
    'username' => 'vagrant',
    'password' => 'vagrant',
    'fixture' => __DIR__ . '/ibm.sql',
];

Remaning issues:

testExplicitPkOnAutoIncrement failed. In db2 it seems to be a problem.
problems with blob (empty result string) and clob (segmentation fault error). I wrote test for that testClobBlobValues
Need to implement checkIntegrity. Solution from https://github.com/vbsessa/yii2-db2 doesn't work in my DB2 version.

I can send DB2 and pdo_ibm installation guide below, because it's tricky.
Now we have some beginning :)

Nice! Good initiative.
Actually, my work is based on @edgardmessias' one. I don't have a dev environment properly configured so a can't test it, but i think the checkIntegrity function has a syntax error. You could try SET INTEGRITY FOR $tableName IMMEDIATE $enable" instead.

@vbsessa your work based on @edgardmessias, my work based on yours - evolution :)
btw, your suggestion about checkIntegrity function doesn't work.
I've tried this example and got FOREIGN KEY error.
SET INTEGRITY FOR "order" ALL IMMEDIATE UNCHECKED;
INSERT INTO "order" ("customer_id", "created_at", "total") VALUES (2000,<PHONE_NUMBER>, 40.0);
SET INTEGRITY FOR "order" IMMEDIATE CHECKED;

Great job, thanks for starting this! Is there a possibility that I could install a DB2 instance for testing? Do they have a version that I could download for testing purpose?

See in my code: https://github.com/edgardmessias/yii2-ibm-db2
It has script to install the database for testing


fix #5021 by prepending path with "/docs"

the thumbnails in docs compiled by pxt are missing the /docs prefix, leading to images not being displayed, this fix adds that prefix

I'm closing this unless we missed something that needed to be done here.


Disable bottom tab nav

Replaces the component directly with the dashboard screen.

@the-kenneth Looks good


JSON CRDT move

[ ] Add move support: remove JSON CRDT node from old location and insert into a new one.

Move requires storing writeId for object keys in ObjectLww and in ValueLww. Instead moves can be implemented in userspace, by storing pointers to objects.


Datatype mismatch: 7 ERROR:  column "breakpoint" is of type boolean but expression is of type integer

I am using PostgeSQL and after running commands migrate|status I am getting this error:
[PDOException]                                                                                SQLSTATE[42804]: Datatype mismatch: 7 ERROR:  column "breakpoint" is of type boolean but expression is of type integer     LINE 1: ...tructure', '2016-07-14 02:42:52', '2016-07-14 02:42:52', 0);                                                                                ^                                                      HINT:  You will need to rewrite or cast the expression.
I tried to create new database and run init there and the migrations table has a new column breakpoint of type bool, the old one doesn't, so it's probably connected to some internal phinx upgrade...

Please upgrade to V0.6

👍  thanks. I updated composer, but it has been holding the 0.5.x version so I had to manually edit it to 0.6.x and now it's working fine

I think I'm going to have to back port a fix for 0.5 to allow it to work with postgres.

No idea how much work it is. The problem is that the issue is not reported anywhere. I have been searching first and I could not find any resource, especially the error message is not super clear, after I played with the empty database and I saw 'breakpoint' in migration table I realised that will be connected. Nothing on stackoverlow etc. That's why I finally used github... So for other users could be a 👍

What I'm not sure about is how a v0.5 got breakpoint support in any way as that was added in 0.6.
Are you able to check in your local composer cache what version/commit of phinx you had that generated the problem? If not, don't worry. I think I MAY have an answer, but if so, you were somehow loading a non-released version.


Add JSON API

Please add a JSON API that accepts an access token retrieved from the account dashboard. In particular it would be nice to:

fetch ladder including rank, name, ELO score
create new matches (with optional scores)
list recent matches

thanks!

[ ] Fetch Ladder (username, rank, rating, last game time, last game id?)
[ ] Submit match (winnerid, loserid) -- current logic doesnt play nicely with allowing arbitrary game times
[ ] Recent Matches for company
[ ] Recent Matches including player
[ ] Recent Matches including two players


Post ID in folder name

I migrated over from the DC script to this and it's working great. One thing that I did have in the DC script was that I had posts go into their own folder.
In the DC script I had:
       "file_directory_format": "OnlyFans-new2/{model_username}/{api_type}/{media_type}/{post_id} ({date})",
I currently have the following in this script:
       "dir_format": "{model_username}/{responsetype}/{mediatype}/",
Is this possible or something that could be added? I couldn't see the post_id in the docs here: https://github.com/datawhores/OF-Scraper/wiki/Customizing-save-path#dir_path

post id is in the filename section

Apologies, forgot about this one. I did see that the post_id is in the filename section.
I'd like to put them into folders:
    "file_format": "{post_id} ({date})/{post_id}_{filename}.{ext}",

This gives the correct path, but needs to create the folder otherwise it fails:
023-05-29 21:45:26 - DEBUG - Media:2108644840 Post:95447788407 Downloading with normal downloader
2023-05-29 21:45:26 - DEBUG - Media:2115786067 Post:102494066850 [attempt 1/5] exception [Errno 2] No such file or directory: 'G:\\Scripts\\ofscraper\\<model>\\Messages\\Images\\102494066850 (22-05-2021)\\102494066850_1847x3024_35f480e70c3d010e51581e3257a2532d.jpeg.part'
2023-05-29 21:45:26 - DEBUG - Media:2115786067 Post:102494066850 [attempt 1/5] exception Traceback (most recent call last):
  File "C:\Python\Python311\Lib\site-packages\ofscraper\utils\download.py", line 135, in download
    return await main_download_helper(ele,path,file_size_limit,username,model_id,progress)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Python\Python311\Lib\site-packages\ofscraper\utils\download.py", line 166, in main_download_helper
    with open(temp, 'wb') as f:
    ^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'G:\\Scripts\\ofscraper\\<model>\\Messages\\Images\\102494066850 (22-05-2021)\\102494066850_1847x3024_35f480e70c3d010e51581e3257a2532d.jpeg.part'

I added in a quick and nasty solution, which is just a check on whether the folder exists, if not just create it. Probably a better way of doing it, but in utils\download.py before line 166 I just added this:
                        folder_path = str(temp).rsplit('\\', 1)[0]
                        log.debug(f"Path for creation: {folder_path} ")

                        if not os.path.exists(folder_path):
                            os.makedirs(folder_path)

Probably an easier solution.

pathlib.mkdir is a little better
pathlib.mkdir(parents=True, exist_ok=True)
Something like that
We can add post_id to dir

Oh, slightly interestingly, this ends up with:
71160139506_1 (13-04-2021)
71160139506_2 (13-04-2021)
When a post has multiple images. Would it be possible to get them into one folder?

It because filenames need something to make them unique
the script uses a counter
By the way I added post_id to dir
since the uniqueness requirment is no longer needed it does not use a counter

https://github.com/datawhores/OF-Scraper/commit/7c3a1ad54a707affbb886f928b2a6aa2535bf833


chore: import repository https://github.com/ramaraok/samplegradle.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ramaraok: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
5797548207339a1e8bdf0524218e7d8d897bccff
link
/test verify

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Develop detailed instructions for setting up kubernetes cluster for running these tests

As described by @nbren12 in https://github.com/pangeo-data/pangeo-integration-tests/issues/1#issuecomment-839063503, we will run these tests as a cron job in a kubernetes cluster. Something like this:
apiVersion: batch/v1
kind: CronJob
metadata:
  generateName: pangeo-integration-test
spec:
  schedule: "*/1 * * * *"
  jobTemplate:
    spec:
      template:
        spec:
          serviceAccountName: accountWithNeededPrivs
          containers:
          - name: hello
            image: pangeo-integration-test:latest
            imagePullPolicy: IfNotPresent
            # all logic happens here
            # and publishes a integration report to a static webpage
            command: ["/bin/bash", "run_tests.sh"]

          restartPolicy: OnFailure

How should this kubernetes cluster be configured? How does one set up such a cluster from scratch on AWS, GCloud, and Azure? Could we use terraform? Can we run it in existing Pangeo clusters?
The easier we can make this step, the easier it will be to get different testing environments set up.

A related question is: how do we want to handle Dask. Is a distributed cluster required? Can we get away with a LocalCluster, or do we need an actual Dask Kubernetes / Gateway cluster?

Once #3 is pushing images, I can start experimenting with our cluster, and try to figure out what if any specific configurations need to change.

how do we want to handle Dask.

Since the cronjob will be running in a pod on the cluster it should be possible to use https://kubernetes.dask.org/en/latest/kubecluster.html#dask_kubernetes.KubeCluster. I did a few K8s jobs like this a while ago, and can dig up the yamls. My recollection is that this worked pretty well.

Whether the dask/k8s integration is something we need to test though, I don't know. There are other ways to (e.g. dataflow) to put these libraries through some serious workloads if that is what we want.

We also may be able to reproduce the main failure modes through a local distributed cluster.

Does github workflows have nodes of that magnitude?

No definitely not. That would have to be inside kubernetes. Just saying we don't necessarily need dask_kubernetes.

Ok. It would be cool if this repo provided a simple helm install or kustomization based template that we could use to install the cronjob into an existing cluster. We can start with a single k8s manifest and scale up from there.

I don't have enough knowledge to contribute much here, and in general we are quite short handed in Pangeo in terms of devops expertise. (This will change in a month or two once @sgibson91 starts with 2i2c! 🎉 )
Thus a helm chart or other "for dummies" level instructions will be very helping. Thanks Noah for taking the lead here.

How about we run a github self-hosted runner on a Kubernetes cluster that also has dask gateway configured? That way, we get all the goodness of GitHub actions without having to deal with nasty auth stuff.
@rabernat if you have credits, I can try setup a 64G / 16 core self hosted runner

or @nbren12 could too! doesn't have to be me :) But I think that's a neat way to get started here.

I do have credits! I would like to set up a new Google Cloud project for this, and to do that I need to go through my university to access the billing account linked to our NSF funding. May take a few days.

@yuvipanda That sounds like a good idea! I didn't know about github self hosted runners. Would this allow for multiple different clusters to all trigger based on PRs from this repo? The federated model is something I would like to pursue, without tying this repo to one specific cluster.

That way, we get all the goodness of GitHub actions without having to deal with nasty auth stuff.

Why is dask gateway required for this? This is why I proposed running these tests within the cluster, so that no additional authentication/networking patterns are needed beyond K8s RBACs, which the person installing needs to have anyway. Provided the runner is in a Pod with the correct service account, it should be able to create worker pods without needing any added authentication layers. We don't use dask gateway or jupyterhub so it would be nice to keep the footprint of these tests as small as possible.
I'm envisioning that anyone could add the tests to their cluster like this:
kubectl -f https://github.com/pangeo-data/pangeo-integration-tests/ci-runner.yaml -f site-specific-configmap.yaml

and make a PR here sharing the link to the reports.

Why is dask gateway required for this?

Oh yeah, dask gateway is definitely not necessary here. You can run things with just LocalCluster on a big machine, or just have the runner in a pod in a kubernetes cluster with appropriate service accounts. Whatever needs to work.

The federated model is something I would like to pursue, without tying this repo to one specific cluster.

Absolutely. You can have many different runners, and have different workflows trigger different runners. So we can have a script or helm or kubeconfig that people can apply, and it'll spin up a GitHub actions runner that can connect back to this repo. Then we can label which tests should run there, and they'll be run on the next round. It's fairly flexible.
Agree on the simplest possible setup. In this case, dask gateway isn't what's being tested, so we don't have to use it unless we are specifically testing that.

Github does caution against using self hosted runners on public repos for security reasons:
https://docs.github.com/en/actions/hosting-your-own-runners/about-self-hosted-runners#self-hosted-runner-security-with-public-repositories
This is good food for thought in general about how to do this in a secure way. Should we always assume that the code on master is is safe to run in a privileged environment?
Jenkins X and Argo CD are alternative CD systems we could look into: https://argoproj.github.io/argo-cd/, but I think the cronjob is a relatively simple way to get started that we could build on top of.

GitHub very recently got a feature called environments that lets users manually approve runs if needed. However, I think if code is on master it should be considered trusted - code in PRs should not be. This is their rationale too - This is because forks of your repository can potentially run dangerous code on your self-hosted runner machine by creating a pull request that executes the code in a workflow.

I agree that something like Argo might also be nice though, if things get complex. I do want to try out GitHub environments though - at least I think they give me the one missing piece that was a big draw for moving towards something like Argo...

@yuvipanda I don't have a ton of bandwidth to work on this now, so if your stoked on this in the short-term, then I'd be happy to test w/e you come up with in our cluster. I'm just mindful to keep the tech stack as thin as possible since I have no idea how to configure/run any of these in cluster CI/CD frameworks. I'm excited to see what we come up with and if we can emulate this pattern in our current test suites.
With github actions I assume we wouldn't need to host any web visible http service? This is workable with things like K8s ingress, but fairly complicated and maybe cloud provider specific. I really like the idea of github workflows just sending a message to github.com when it completes.

Sorry if I seemed out of line or anything, @nbren12. I did get pretty stoked and have a lot of short term energy to put into this, since I've been meaning to find a use case for github self hosted runners. I apologize if my communication style came out wrong.

I'm just mindful to keep the tech stack as thin as possible since I have no idea how to configure/run any of these in cluster CI/CD frameworks.

Me too!

With github actions I assume we wouldn't need to host any web visible http service? This is workable with things like K8s ingress, but fairly complicated and maybe cloud provider specific.

Indeed, no network traffic comes from GitHub to the runner - the runner polls GitHub for everything. This makes the networking setup easy across cloud providers, since no ingress needs to happen!

I apologize if my communication style came out wrong.
Not at all! I wasn't miffed at all! Just stoked that you interested, and happy to help.

Just a little update: @yuvipanda now has credentials for a new, Columbia-owned GCP project devoted to this. Yuvi, let us know what we can do to help with the next steps.


Write package size

Summary of Problem
The device to which I write, has a buffer size of only 64 Bytes. It is possible to write unlimited data to it, but the packages cannot be bigger than 64 Bytes.
In my C# application, I added some code that divides the packages to 64 Bytes and sleep so that the device can clear the buffer for the next data.
Question: Is such a method implemented already?
Code to Reproduce the Issue
C# code I use to write chunks of data to the serialport
        /// <summary>
        /// Write data to the COM Port basestream.
        /// </summary>
        /// <param name="data">Byte array: data to send.</param>
        private async Task Write(byte[] data)
        {
            var loops = (int)Math.Ceiling((double)data.Length / DeviceBufferLength);

            for (var iteration = 0; iteration < loops; iteration++)
            {
                // Take the Device Buffer Length, or the remainder length.
                var size = (iteration * DeviceBufferLength) + DeviceBufferLength > data.Length ? data.Length - (iteration * DeviceBufferLength) : DeviceBufferLength;

                await port.BaseStream.WriteAsync(data, iteration * DeviceBufferLength, size);

                if (size == DeviceBufferLength)
                {
                    Thread.Sleep(1);
                }
            }
        }

Versions, Operating System and Hardware

SerialPort@latest
Node.js@latest
Windows

@JBtje If you check out the Node.js Buffer documentation, you can see methods like #byteLength and #alloc that can help you manage the size of the data you're sending to your device.


Remove calls to promisifyAll

The promisifyAll are bluebird only calls, these need to be removed for future use.

PromisifyAll has been removed and will be deployed shortly.


Navbar

Pull Request
Description
Updated navbar with following:
Added a new theme change button with icons.
fixed some bugs
Closes #572
Type of change

[x] Website enhancement or fixes

Checklist:

[x] I read carefully CONTRIBUTING.md
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Screenshots (if appropriate):

We're going to redesign this, so I will close this PR for now


onPress in ToastProps

Hi! I built a custom wrapper for the toasts in my app using this lib, it's been pretty much straight forward, so thanks for your work.
Since I need to add listeners on the single toast instance, without using onToastPress on Toasts, I wanted to ask if this feature is on the development roadmap and what's the reason why you didn't include it on the single toast options.
Other than that, I guess that for now I'd need to manually add the onPress property in my wrapper and build a handler that executes that function in onToastPress using ids to match the right one.
The need of decentralized handlers comes from the lack of onToastPress to access component-specific methods or values such as local react hooks, custom hooks or generic functions. Thanks in advance for your time

You’re absolutely right that having decentralized handlers tied to the single toast instance would be super helpful. The main reason this isn’t included in the toast() API currently is due to potential issues that arise since toast() doesn’t depend on React or its lifecycle. If a component that calls toast() unmounts, it could lead to messy side effects, especially if the onPress function relies on local state, hooks, or context from the originating component.
To illustrate the issue, imagine this scenario:
// Inside library's Toast Component
<Toast
  onPress={(id) => {
    if(toast.onPress) toast.onPress(toast);
  }}
/>

// SomeComponent using toast()
useEffect(() => {
  toast("Hello!", {
    onPress: () => {
      console.log("Do something specific here");
    },
  });
}, []);

Now, if SomeComponent unmounts before the toast is pressed, the handlePress function is no longer part of the React tree, potentially leading to errors or unintended behavior.
Here are 2 solutions that I thought up:

Let developers manage side effects themselves:

Implement the onPress behavior as-is and allow developers to handle side effects manually. This could involve ensuring that handlers don't rely on local component state or hooks that might be invalidated on unmount.

Expose an onToastPress listener in the useToaster() hook:

This approach allows developers to handle toast presses centrally within a React component, ensuring access to the necessary React lifecycle and context. For example:
const MyToastListener = () => {
  const { onToastPress } = useToaster();

  React.useEffect(() => {
    const unsubscribe = onToastPress((id) => {
      console.log(`Toast with ID ${id} pressed`);
      // Access hooks, state, or context here safely
    });

    return unsubscribe; // Clean up when the component unmounts
  }, [onToastPress]);

  return null; // This component only handles side effects
};

This second approach could make it easier to track IDs, manage state, and avoid lifecycle issues while still allowing flexibility.
I’d love to hear your thoughts on these solutions or if you have specific needs that might make one approach preferable. Thanks again for your thoughtful feedback—it’s incredibly valuable in improving the library!

I prefer to have onPress as a param at toast.success() method. Let the dev manage the side effect by themselves.


Refactored GraphPlot and fixed renderer ordering

This PR refactors the GraphPlot._init_glyphs method so it is a bit less unwieldy and then adds a method which reorders the renderers on the plot depending on the defined draw_order.

[x] Adds unit tests

Before:

After:

Thanks for the fix! Merging.


fix up the loop protection to not die on just for

Was this resolved by one of the final I/O commits?

Actually yes! Thanks!


Utility method for replacing built-in object renderer

Hello! I'm trying to use Markdig to replace MarkdownDeep. Currently I pre-process GitHub-style code fences via a regular expression, as MarkdownDeep does not support them natively. I'd like to remove this as Markdig is nice and supports code fences out-of-the-box, which is awesome. However, I want to output some different HTML to support my current configuration using highlight.js.
Currently I'm doing something like this to replace the built-in CodeBlockRenderer with my own one
var originalCodeRenderer = renderer.ObjectRenderers.FindExact<CodeBlockRenderer>();
if (originalCodeRenderer != null)
{
	renderer.ObjectRenderers.InsertAfter<CodeBlockRenderer>(new HighlightCodeBlockRenderer());
	renderer.ObjectRenderers.Remove(originalCodeRenderer);
}

This works, but it seems a bit verbose. It would be nice to have a ReplaceWith utility method so I could do something like:
renderer.ObjectRenderers.ReplaceWith<CodeBlockRenderer>(new HighlightCodeBlockRenderer());

I guess I could just have an extension method for this though. Just an idea :)
Thanks for the awesome project!

This is the extension method I wrote, seems to be working fine so far:
public static ObjectRendererCollection Replace<TOriginal>(
	this ObjectRendererCollection renderers,
	IMarkdownObjectRenderer replacement
) where TOriginal : IMarkdownObjectRenderer
{
	var index = renderers.FindIndex(renderer => renderer.GetType() == typeof(TOriginal));
	if (index == -1)
	{
		throw new InvalidOperationException($"Could not find original object renderer for {typeof(TOriginal)}");
	}
	renderers.RemoveAt(index);
	renderers.Insert(index, replacement);
	return renderers;
}

Usage:
renderer.ObjectRenderers.Replace<CodeBlockRenderer>(new HighlightCodeBlockRenderer());

It makes sense. Feel free to open a PR for OrderedList.cs

Thanks, I'll do that soon! :)

Note that I have reverted back the changes related to VS2017 on master. Make sure to use the latest

Submitted as #103


Add bdist

Create bdist for Mac, Windows, and UNIX to speed up installation.

Would it be better to just create a whl and have users install it via pip install harvest-0.1.0-py3-none-any.whl or pip install harvest-0.1.0-py3-none-any.whl[Robinhood]

If I understand it right, bdist and wheel are somewhat synonymous. So yes, you can just create a whl. I'm thinking after v0.2 is released, or possibly sometime earlier, I will upload harvest to PyPI.


feat: add rule repository

There are some conflicts. I don't know why I cannot view them here.

There are some conflicts. I don't know why I cannot view them here.

I moved a file in branch A and changed it in Branch B. It's fixed now.

I shall merge after CI is green for sanity.

Perfect, the next step is moving the API and sonar away from the core to 2 subprojects.

We keep them inside Sorald as individual modules, right?

Yes


Private Accounts

Users should be able to make their accounts private, their libraries wouldn't be accessible to anyone other than themselves (and friends, if we implement that?). This'd be a boolean toggle in the user settings.
Admins (and maybe moderators?) would still be able to see the user's profile regardless.

Make sure to handle the compare page, since it doesn't currently have a policy method.


Ignore trailing dot

I went to http://go/foo. and got a 404. But http://go/foo works.
On 404, try to trim the trailing dot and try that? For people copy/pasting links with a trailing period in a sentence.
/cc @willnorris

Probably all punctuation characters. I can imagine ) being pretty common

It's probably handled by extracting the path only but header links like go/foo#my-heading-(parenthetical) would break if punctuation is removed from the full URL without first extracting the path.


arbitrary_types_allowed no longer works with root models in V2

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
In V2 root models arbitrary_types_allowed no longer works. In V1 it used to work. Is it intended not to work anymore? Example below showing an use case that used to work in V1. I guess the issue is somehow related to generic models handling in V2.
Example Code
import numpy

from pydantic import BaseModel

try:
    from pydantic import ConfigDict, RootModel
    PYDANTIC_VERSION = "v2"
except ImportError:
    PYDANTIC_VERSION = "v1"

def test_arbitrary_root():
    if PYDANTIC_VERSION == 'v2':
        # Now it raises suggesting using arbitrary_types_allowed. But it is set!
        # Exception:
        # pydantic.errors.PydanticSchemaGenerationError: Unable to generate pydantic-core schema for <class 'numpy.ndarray'>.
        # Set `arbitrary_types_allowed=True` in the model_config to ignore this error or implement `__get_pydantic_core_schema__` on your type to fully support it.
        class MyNdarray(RootModel[numpy.ndarray]):
            model_config = ConfigDict(arbitrary_types_allowed=True)
            # defines deserialization of list to ndarray etc...
    else:
        # In v1 it works fine.
        class MyNdarray(BaseModel):
            __root__: numpy.ndarray

            class Config:
                arbitrary_types_allowed = True

            # defines deserialization of list to ndarray etc...

Python, Pydantic & OS Version
pydantic version: 2.0.3
        pydantic-core version: 2.3.0 release build profile
                 install path: /Users/markussintonen/Library/Caches/pypoetry/virtualenvs/leak-test-VvUvdXxD-py3.10/lib/python3.10/site-packages/pydantic
               python version: 3.10.12 (main, Jun 20 2023, 17:00:24) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
                     platform: macOS-13.4.1-x86_64-i386-64bit
     optional deps. installed: ['typing-extensions']

Selected Assignee: @lig

@MarkusSintonen while I think ideally this wouldn't error, I think it might be hard to get it to work properly, since it's erroring while trying to create RootModel[np.ndarray], which you are inheriting from. Instead, you can do:
from typing import Any

import numpy

from pydantic import ConfigDict, RootModel

class MyNdarray(RootModel[Any]):
    root: numpy.ndarray
    model_config = ConfigDict(arbitrary_types_allowed=True)
    # defines deserialization of list to ndarray etc...

I was worried this might not give the right mypy behavior, but I wasn't able to produce any typing issues using the above.
I'd be open to PRs fixing it for RootModel but I don't think the right answer is to make arbitrary_types_allowed=True for RootModel by default, and I don't see another good way to achieve this.
Open to debate/other ideas

@dmontagu I think that both forms should work identical in general despite what type is being used for RootModel.
class MyNdarray(RootModel[numpy.ndarray]):
    model_config = ConfigDict(arbitrary_types_allowed=True)

and
class MyNdarray(RootModel):
    root = numpy.ndarray
    model_config = ConfigDict(arbitrary_types_allowed=True)

I agree that arbitrary_types_allowed shouldn't be True by default.
I will focus in this issue just on making arbitrary_types_allowed=True work as expected.

After thorough review of this issue I'm tempting to call this known behavior and add a note regarding this to the docs.
The issue here is that RootModel[numpy.ndarray] creates a specialized subclass of RootModel that could be used as a full functional model already. Two code examples below produce mostly identical results.
class Model(RootModel):
    root: numpy.ndarray

and
Model = RootModel[numpy.ndarray]

When creating a RootModel subclass explicitly we can set model_config for it and when creating using an implicit subclass as in RootModel[numpy.ndarray] we cannot.
This means that in the code below we attempt creating a RootModel subclass without model_config defined before setting model_config for MyNdarray.
class MyNdarray(RootModel[numpy.ndarray]):
    model_config = ConfigDict(arbitrary_types_allowed=True)


needs.json schema? How are links stored?

Is it currently correct to store custom link options of a need flat, e.g.
                "ACTIVITY_ThisAndThat": {
                    "description": "",
                    "id": "ACTIVITY_ThisAndThat",
                    "implements": [],
                    "has_in_port": ["...", ...],
                    "has_out_port": ["...", ...],
                    "instantiates": ["...", ...]
                 },

or nested below "links", like this:
                "ACTIVITY_ThisAndThat": {
                    "description": "",
                    "id": "ACTIVITY_ThisAndThat",
                    "implements": [],
                    "links": {
                        "has_in_port": "...,...",
                        "has_out_port": "...",
                        "instantiates": "...",
                    },
                }

I could not find anything in the docs. I'm writing a needs.json generator right now, so I needs to know :laughing:

The first one is correct, so it's just a "flat" dictionary.

I think this can be closed with #1230


Update RELEASE.md

Removed extra "for".

Can one of the admins verify this patch?

Jenkins, test this please


[Workaround] Fix for newly added project folder having git related view issues

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I've noted a high number of git related issues marked as bug with the following files/folders.

.gitignore
.git folder

This workaround is for newly created project folders not adding in git visuals in the treeview interface.
Steps to Reproduce

Create a directory
Add the newly created directory as a new project folder

File > Add Project Folder

Navigate to the directory via CLI and execute git init

First, note that the directory has not been recognized as a git repo in the tree view.
Second, the .git directory has not been recognized as a hidden folder
Third, if you proceed as if nothing is wrong, .gitignore will not "darken/hide" files properly.

To fix this, remove the project folder (not the actual directory) in the tree view and add it back.

😄 All fixed

Expected behavior: [What you expect to happen]
Tree view recognizes the project folder being initialized to a git repo and updates
Actual behavior: [What actually happens]
Tree view does not recognize the project folder being initialized as a git repo and does not implement additional git functionality as a result.
Reproduces how often: [What percentage of the time does it reproduce?]
100% of the time
Versions
Windows 10 Pro 10.0.15063 Build 15063
Atom    : 1.19.7
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0

Notes
While this functionality is a bit of a feature and the implementation required to detect a change as to whether or not the folder has been initialized git repo after addition may prove impractical, I believe this process may prove invaluable to further debug actual issues.

Thanks for reaching out and for sharing this workaround!  We'll go ahead and close this as a duplicate of https://github.com/atom/tree-view/issues/1040 since the described problem is the same (and that issue was in turn closed as a duplicate of https://github.com/atom/tree-view/issues/1036 so you can subscribe there for updates if you'd like).

Because we treat our issues list as the Atom team's backlog, we close issues that don't represent a task needing to be completed.


Add: Mega TV

Channel Name
Mega TV
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
No response
Country
PY
Subdivision (optional)
No response
City (optional)
No response
Broadcast Area
c/PY
Languages
es
Categories (optional)
No response
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://megacadena.com.py/mega-tv/
Logo
https://megacadena.com.py/wp-content/uploads/2022/05/MegaTV-logo-10.png
Notes
It's a free IPTV channel

Added in https://github.com/iptv-org/database/commit/674e9922d943746b4b02e8b2a151358201f77bec


enforce java style

https://github.com/google/google-java-format
are there similar things for js, ts, and python?
my ideal is these run as tests.

Python
Python has historically used a tool called pep8, which is a reference to Python Enhancement Proposal 8. This proposal is a style guide from Python's creator.
Example: pycodestyle
The tool pep8 was renamed recently to pycodestyle.
#!/usr/bin/env python

def ThisIsCamelCase(x):
    print("this is a really, really, really, really, really, really, really, really long line")
def thisFunctionIsTooClose(x):
    pass

$ pycodestyle bad_python.py
bad_python.py:3:1: E302 expected 2 blank lines, found 1
bad_python.py:4:80: E501 line too long (95 > 79 characters)
bad_python.py:5:1: E302 expected 2 blank lines, found 0

Example: pylint
Python also has a linter called pylint that is worth knowing about too.
$ pylint bad_python.py
No config file found, using default configuration
************* Module bad_python
C:  1, 0: Missing module docstring (missing-docstring)
C:  3, 0: Invalid function name "ThisIsCamelCase" (invalid-name)
C:  3, 0: Invalid argument name "x" (invalid-name)
C:  3, 0: Missing function docstring (missing-docstring)
W:  3,20: Unused argument 'x' (unused-argument)
C:  5, 0: Invalid function name "thisFunctionIsTooClose" (invalid-name)
C:  5, 0: Invalid argument name "x" (invalid-name)
C:  5, 0: Missing function docstring (missing-docstring)
W:  5,27: Unused argument 'x' (unused-argument)

-------------------------------------
Your code has been rated at -12.50/10

TypeScript
According to this github issue it seems VSCode uses typescript-formatter.
It also uses tslint for linting.
Javascript
There seem to be a lot of choices for JS, so it's less clear to me what the right choice might be. However, js-beautify looks promising.
I've heard from friends that they love ESLint for JS linting.


Updates to Korean translation of the deeds

(From @soohyunpae via email )
Could you have a look at the Korean Deeds page and let me know how I can get the untranslated texts updated?
(and)
I just updated some strings for Korean Deeds. Could you help me get them live?
https://www.transifex.com/nkinkade/CC/viewstrings/#ko/deeds-choosers/2858697

@sarahpearson @mattl what is the procedure for this, please? :-)

Good question! @mattl should know how we push those live.


Packages, consumer

@h1alexbel take a look, please
here are some changes for packages and I tried to create a Consumer interface. I'm not sure about Consumer, what do you think?

Codecov Report
Base: 93.10% // Head: 93.10% // No change to project coverage :thumbsup:

Coverage data is based on head (de657cd) compared to base (83a8e75).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #48   +/-   ##
=========================================
  Coverage     93.10%   93.10%
  Complexity       12       12
=========================================
  Files             6        6
  Lines            29       29
=========================================
  Hits             27       27
  Misses            2        2

Impacted Files
Coverage Δ

src/main/java/org/eocqrs/kafka/data/KfData.java
100.00% <ø> (ø)

...rc/main/java/org/eocqrs/kafka/data/KfDataized.java
100.00% <ø> (ø)

...ain/java/org/eocqrs/kafka/producer/KfProducer.java
100.00% <ø> (ø)

.../org/eocqrs/kafka/producer/KfProducerSettings.java
100.00% <ø> (ø)

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

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

@h1alexbel take a look, please

@l3r8yJ the puzzle #49 is still not solved.

@l3r8yJ thanks for contribution !

@l3r8yJ the puzzle #54 is still not solved; solved: #49.

@l3r8yJ 5 puzzles #294, #295, #296, #297, #298 are still not solved; solved: #49, #54.

@l3r8yJ 4 puzzles #294, #295, #297, #unknown are still not solved; solved: #296, #298, #49, #54.

@l3r8yJ 2 puzzles #297, #unknown are still not solved; solved: #294, #295, #296, #298, #49, #54.

@l3r8yJ all 8 puzzles are solved here: #294, #295, #296, #297, #298, #49, #54, #unknown.


Does anybody know why this number is used here - "4294967295"? Why not a simple value like "1"?

https://github.com/gitbrent/PptxGenJS/blob/506af72a7a0e2d8694ca012d374521d7d1c7de6c/dist/pptxgen.es.js#L1917

My guess would be, that this is the last possible index.
Why the last?
It seems the index here is an unsigned 32bit number, starting at 0 and<PHONE_NUMBER> = 2³²-1 is the highest (last) representable number here.
Probably this is used to ensure, that no previous index is re-used or overwritten.
Just my 2 cents.

Thanks, @koddistortion, I know about the upper bound of unsigned int type, but I've checked it on real pptx files and behavior a bit different with idx='4294967295'  than, for example with idx='1' etc, and I'm trying to understand how does it work...

I’m unable to recall where the value is derived from off-hand. I may have simply mimicked what PowerPoint produced.

@gitbrent I've understood how to get this value in PP. For example, you have an inheritance chain: master slide placeholder with idx='1', slide layout placeholder with idx='1', and slide placeholder with idx='1'. if you remove the slide layout's placeholder the inheritance chain between the slide and the slide layout will be broken, but PP still has to inherit properties from parent entities in this case from the master slide, and it marks slide placeholder with idx='4294967295'. It's like a mark. If PP reads this value it inherits properties directly from the master slide, excludes the slide layout and it understands that placeholder which belongs to the slide layout has been removed.


Add ability to read svgoptions inside loader options

I would like to be able to do this:
{
    loader: 'external-svg-sprite-loader',
    options: {
        name: 'img/svg-sprite.[hash].svg',
        prefix: '',
        svgo: {
            plugins: [
                { removeTitle: true },
                // Remove stuff related to colors because they are unecessary
                { removeStyleElement: true },
                { removeAttrs: { attrs: '(fill|stroke|class)' } },
            ],
        }
    },
},

I think that changing https://github.com/Karify/external-svg-sprite-loader/blob/master/index.js#L52 to account for query.svgo would do it. Thoughts?

You can do:
{
    module: {
        rules: [
            {
                loader: 'external-svg-sprite-loader',
                options: {
                    name: 'img/svg-sprite.[hash].svg',
                    prefix: '',
                },
            },
        ],
    },
    svgoOptions: {
        plugins: [
            { removeTitle: true },
            // Remove stuff related to colors because they are unecessary
            { removeStyleElement: true },
            { removeAttrs: { attrs: '(fill|stroke|class)' } },
        ],
    },
}

Or is this not desirable / logic? Let me know what you think.

Yea I know that works but I have multiple rules that use external-svg-sprite-loade :(

Hmm I see, I guess that makes sense. I'll see if I can look into it today. Otherwise you can also create a PR ;)


Update ns-wdm-_key_value_basic_information.md

Add REG_QWORD and REG_QWORD_LITTLE_ENDIAN, in Windows SDK:

#define REG_QWORD                   ( 11ul ) // 64-bit number
#define REG_QWORD_LITTLE_ENDIAN     ( 11ul ) // 64-bit number (same as REG_QWORD)

[like]  Barry Golden reacted to your message:

From: Ted Hudek @.>
Sent: Tuesday, October 15, 2024 4:55:26 PM
To: MicrosoftDocs/windows-driver-docs-ddi @.>
Cc: Barry Golden @.>; Assign @.>
Subject: Re: [MicrosoftDocs/windows-driver-docs-ddi] Update ns-wdm-_key_value_basic_information.md (PR #1585)
Assigned #1585https://github.com/MicrosoftDocs/windows-driver-docs-ddi/pull/1585 to @barrygoldenhttps://github.com/barrygolden.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-driver-docs-ddi/pull/1585#event-14661224394, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJOIVMP3YVNU2MSTQLI5G3Z3VCH5AVCNFSM6AAAAABP7HWJXCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGY3DCMRSGQZTSNA.
You are receiving this because you were assigned.Message ID: @.***>


Use named fields in Build and Version structs

Best use explicit names to clarity purposes.

CI shouls pass now. :) PTAL.

This is a good opportunity to work towards a style guide, which we haven't defined yet for the Enarx organization.
I'm in favor of named arguments and fields while I see these two possible cons: they require public fields, and they are prone to breaking compatibility (which might not be a con but rather intention). The latter is easily solvable by deriving Default and using the ..Default::default() notation at the end of each instantiation. The former is a limitation in Rust itself, which I haven't investigated in depth.
Since the structs we're dealing with here already uses pub fields, we could think about introducing the Default approach here, depending on whether we can foresee adding new fields which we expect to have defaults. In this case these could be patch and build version fields.

Thanks for your input on this @steveeJ .

depending on whether we can foresee adding new fields which we expect to have defaults

Although I'm a complete newbie to Enarx, I don't foresee addition of fields of any non-simple external type w/o a Default implementation in the future to these simple structs. But if that prediction turns out to be wrong, we can always create a wrapper type, implement Default for it and use that instead.
So I think I'll just add Default implementation here and it should be good for now. Unless @npmccallum or anyone disagrees?

@steveeJ Turns Default was already implemented through derive on both structs so I've now modified the patch to make use of it where it makes sense. PTALA.


Bug in Node hash function

The hash function for the node is just using source node and the edge to create a hash. In situations that we have something like following the add function won't add to set of nodes in the graph.
a --k1-->b
a --k1 -->c
one of the above gets deleted in the hash set!
here is the code:
    public int hashCode() {
        int result = this.name != null ? this.name.hashCode() : 0;
        result = 31 * result + (this.links != null ? this.links.hashCode() : 0);
        result = 31 * result + (this.attributes != null ? this.attributes.hashCode() : 0);
        return result;
    }

Should finally be fixed in 0.12.1


Allow Numeric with different base

Solve https://github.com/typst/typst/issues/5519 by copy the parse code for Int into Numeric

By the way, I recommend using separate branches for your PRs and not using main in your fork (only updating it to Typst's latest main before branching off for a PR). That not only allows you to open more than one PR at a time, but also ensures changes to this PR which may be requested by maintainers won't affect your fork's other branches in any way. (Also makes it easier to wipe past history as you start clean from upstream main each time.)

By the way (just a general tip for future PRs - this is not a blocker for your PR), I recommend using separate branches for your PRs and not using main in your fork (only updating it to Typst's latest main before branching off for a PR). That not only allows you to open more than one PR at a time, but also ensures changes to this PR which may be requested by maintainers won't affect your fork's other branches in any way. (Also makes it easier to wipe past history as you start clean from upstream main each time.)

Thank you.
I just use main for ease. I will use other branch later.

See https://github.com/typst/typst/issues/5519#issuecomment-2526210351


Updated for xcode 7 beta 4

Just a one line change required in ToJSON.swift, + a nimble update for swift 2.0. All tests passed.
Let me know what you think.

:+1:


Instrumentation page

Moved an image from the libraries page to the instrumentation page.

@DIANAAJAMBO please take a look at this comment: https://github.com/open-telemetry/opentelemetry.io/pull/5523#discussion_r1835117995

@DIANAAJAMBO please take a look at this comment: #5523 (comment)

Hello @svrnm , I made this change in my last commit, as seen here: https://github.com/open-telemetry/opentelemetry.io/pull/5523/files. However, the original image referenced was merged into the main repository, and I currently lack the necessary rights to update it.
Could you please review my last commit and confirm if everything looks good? If so, we can proceed with the merge.
Thank you.

@DIANAAJAMBO please take a look at this comment: #5523 (comment)

Hello @svrnm , I made this change in my last commit, as seen here: #5523 (files). However, the original image referenced was merged into the main repository, and I currently lack the necessary rights to update it. Could you please review my last commit and confirm if everything looks good? If so, we can proceed with the merge. Thank you.

You copied the file but you didn't move the file: https://github.com/DIANAAJAMBO/opentelemetry.io/blob/librariesPage/static/img/libraries-instrumentation.svg is still existing, so what you need to do is to remove that file and then commit that as a change, git(hub) will then detect that this is a move.

@DIANAAJAMBO please take a look at this comment: #5523 (comment)

Hello @svrnm , I made this change in my last commit, as seen here: #5523 (files). However, the original image referenced was merged into the main repository, and I currently lack the necessary rights to update it. Could you please review my last commit and confirm if everything looks good? If so, we can proceed with the merge. Thank you.

You copied the file but you didn't move the file: https://github.com/DIANAAJAMBO/opentelemetry.io/blob/librariesPage/static/img/libraries-instrumentation.svg is still existing, so what you need to do is to remove that file and then commit that as a change, git(hub) will then detect that this is a move.

Okay thank you. I will implement this today.

@DIANAAJAMBO please take a look at this comment: #5523 (comment)

Hello @svrnm , I made this change in my last commit, as seen here: #5523 (files). However, the original image referenced was merged into the main repository, and I currently lack the necessary rights to update it. Could you please review my last commit and confirm if everything looks good? If so, we can proceed with the merge. Thank you.

You copied the file but you didn't move the file: https://github.com/DIANAAJAMBO/opentelemetry.io/blob/librariesPage/static/img/libraries-instrumentation.svg is still existing, so what you need to do is to remove that file and then commit that as a change, git(hub) will then detect that this is a move.

Okay thank you. I will implement this today.

@svrnm All should be okay now. Help me review. Thanks

@svrnm & @DIANAAJAMBO - thinking about this more: how about putting this information in a table rather than an SVG? It would be easier to consult, visualize, and render (e.g., dark-mode would be handled automatically; no need to putts around with the SVG colors). WDYT?

Thank you for the suggestion, @chalin . I have already implemented the dark mode change along with the image update, but let’s hear @svrnm's thoughts before finalizing. This way, I can incorporate any necessary adjustments.

@svrnm & @DIANAAJAMBO - thinking about this more: how about putting this information in a table rather than an SVG? It would be easier to consult, visualize, and render (e.g., dark-mode would be handled automatically; no need to putts around with the SVG colors). WDYT?

Thank you for the suggestion, @chalin . I have already implemented the dark mode change along with the image update, but let’s hear @svrnm's thoughts before finalizing. This way, I can incorporate any necessary adjustments.

Proposal: we merge this PR with the image for now, since it is there already and it's an incremental update and then we can talk about turning it into a table instead. Another advantage of that is accessibility, since the SVG might not be readable easily as it is right now.


AzureML Studio Dataset as source

Hi,
In this current release only flat files and SQL Server query result sets can be used as source for IDEAR. Is there any plan to allow IDEAR to connect to a transient data set (after initial cleansing and transforming) in AML Studio ?
Best regards,
Carlos Abramo

Hi, Carlos,
Good point. We will add this request to our backlog and prioritize it in our future work.
Currently, you can use the AzureML R library to download the dataset from AML workspace to your local machine, and start using IDEAR to visualize and explore the data.
Thanks,
Hang


chore: bump ic-utils, ic-agent and ic-identity-hsm to 0.27.0

Description
This commit bumps ic-utils, ic-agent and ic-identity-hsm from version 0.26.1 to version 0.27.0.
How Has This Been Tested?
In OpenChat we have just started using dfx-core to load DFX identities and when using the version from this commit everything works as expected.
Checklist:

[x] The title of this PR complies with Conventional Commits.
[x] I have edited the CHANGELOG accordingly.
[ ] I have made corresponding changes to the documentation.

It is likely that the target branch is wrong, please let me know if so and I'll change it.

It is likely that the target branch is wrong, please let me know if so and I'll change it.

Yes, should be master branch

I've made that change


experiment: profile against leanprover/lean4#2097

Test a setup I discussed with @gebner for profiling mathlib4 against arbitrary Lean tags

!bench

!bench


Retire Anne and Darian from maintainers and codeowners

They have previously indicated they wished to be retired.

Should probably remove Kelly also since it looks like he's not a member of the hyperledger org anymore.
https://github.com/orgs/hyperledger/people?query=ineffectualproperty

Should probably remove Kelly also since it looks like he's not a member of the hyperledger org anymore. https://github.com/orgs/hyperledger/people?query=ineffectualproperty

Yes, we are working on rewritting the maintainer rules which will have more explicit rules about removing inactive maintainers but since that is still in progress I am currenlty only removing those who I have previously been asked to retire.


Allow Spectrum objects to support complex coefficients

Describe the new feature or enhancement
Discussed briefly with @larsoner.
We are considering changes in MNE-Connectivity where the spectral coefficients can be passed directly to the connectivity estimation functions in the form of EpochsSpectrum and EpochsTFR objects, as returned from Epochs.compute_psd/tfr(output="complex").
While the complex output is supported for TFR objects, Spectrum objects do not allow complex outputs from the various PSD functions: https://github.com/mne-tools/mne-python/blob/6ad28a461fe29d96e5d822e2de6c4c9d1b7c2537/mne/time_frequency/spectrum.py#L319-L323
Describe your proposed implementation
Remove the ValueError for output="complex" from Spectrum classes.
There are a couple places where tweaks would be required, e.g. in _check_values() and handling PSD func outputs in _compute_spectra(), but similar solutions are already present in the TFR classes.
Same situation for the plotting methods, but again TFR classes also have solutions for this already which could be followed:
https://github.com/mne-tools/mne-python/blob/0c369e0eaa1327e612137ba0e4a54ad776ea6e48/mne/time_frequency/tfr.py#L4326-L4328
One thing that would need to be added is storage for weights in the Spectrum class when complex output is requested from the multitaper mode, which as I understand is not the case for the TFR multitaper.
Describe possible alternatives
N/A
Additional context
No response

Addressed in #12747


Created landing page

I have created a basic landing page for the website.

Nice work @AshwinSriram11 🚀


fix: add regex to avoid api tokens creation with invalid name

This PR aims to add a validation to avoid API tokens creation with an invalid name.
closes #403

This is still a draft @athoscouto ;)


Section on call, apply and bind

I thought it might be good to add a section on call, apply and bind. These aren't covered explicitly anywhere, and have some unusual behaviour.
For example bind doesn't create a new function with bound arguments, it permanently binds arguments on that function, meaning future calls (or uses of bind) have no effect on those arguments.

@peterjwest did you want to add this? Reopen if you do!

I don't think I'll have time anytime soon, so will leave this.

It wasn't mentioned here, but although using bind on a bound function will not change the context on the original unbound function, it is still possible to partially apply more arguments:
var a = console.log.bind(console, 'lol');
a('rofl'); //=> lol rofl
var b = a.bind(null, 'omg');
b('wtf'); //=> lol omg wtf

(Also, likely because of scoping issues, the original unbound function cannot be accessed from a bound function.)


[testing-karma] How to handle ES6 bare module imports for local and CI testing?

Hi again, 👋
I'm using @open-wc/testing framework for testing my library of web components, though, it's failing for my local and TravisCI tests now.🤦
Directory Strucutre
├── src
│   ├── elements
│   │   ├── hx-busy
│   │        ├── **/*.index.js
│   │        ├── **/*.index.spec.js
│   ├── ...
├── dist
│   ├── css
│   │   ├── helix-ui.css
│   │   ├── helix-ui.min.css
│   ├── js
│   │   ├── helix-ui.js
│   │   ├── helix-ui.esm.js
│   │   ├── helix-ui.module.js
│   │   ├── helix-ui.cjs.js
├── node_modules
├── karma.conf.js
├── package.json
└── ...

karma.conf.js
/* eslint-disable import/no-extraneous-dependencies */
const { createDefaultConfig } = require('@open-wc/testing-karma');
const merge = require('deepmerge');

module.exports = config => {
  config.set(
    merge(createDefaultConfig(config), {
      files: [
        { pattern:
            config.grep ? config.grep : 'dist/js/helix-ui.module.js',
            type: 'module'
        },
        { pattern:
            config.grep ? config.grep : 'src/elements/**/*.spec.js',
            type: 'module'
        },
      ],
      esm: {
        nodeResolve: true,
        compatibility: "max",
        moduleDirs: ['node_modules', 'dist'],
      },
      proxies: {
        //  attempted to add proxies
      },
      middlewares: [
        // attempted to add routes
      ],
      // you can overwrite/extend the config further
      polyfillsLoader: {
          polyfills: {
            webcomponents: true,
            shadyCssCustomStyle: true,
          },
      },
      logLevel: config.LOG_DEBUG,
    }),
  );
  return config;
};

Example spec file
import { fixture, expect } from '@open-wc/testing';

import HelixUI from '../../../dist/js/helix-ui.module.js';

/**
 * <hx-busy> component tests
 *
 * @type HXBusyElement
 *
 */

describe('<hx-busy> tests', () => {
    const template = '<hx-busy>';

    it('should verify HelixUI is initialized', async () => {
        if (HelixUI.Elements === undefined ) {
            await HelixUI.initialize();
        }

        expect(HelixUI.Elements).to.not.be.undefined;
    });

    it('should be instantiated with hx-defined attribute', async () => {
        const el = /** @type {HXBusyElement} */ await fixture(template);

        expect(el.getAttribute('hx-defined')).to.exist;
    });

    it('should have aria-hidden attribute set to true', async () => {
        const el = /** @type {HXBusyElement} */ await fixture(template);

        expect(el.getAttribute('aria-hidden')).to.be.equal(String(true));
    });

    it(`the rendered light DOM should NOT equal simple template ${template}`, async () => {
        const el = /** @type {HXBusyElement} */ await fixture(template);

        expect(el).lightDom.to.not.equal(template);
    });

    it(`should NOT have a Shadow DOM`, async () => {
        const el = /** @type {HXBusyElement} */ await fixture(template);
        const sr = el.shadowRoot;

        expect(sr).to.be.null;
    });
});

I would like to import from dist/ for testing locally and TravisCI
I would prefer to import bare modules for web component testing, though this does not work.
import HelixUI from '../../../dist/js/helix-ui.module.js';  // Does not work

But, I had to do this (which is now failing)...
This temporary fix used a CDN (unpkg) to allow local and TravisCI builds.  Now it doesn't work.
import HelixUI from<EMAIL_ADDRESS> // No longer working

Thanks and appreciate any recommendations on a possible solution! 🤞 💯

@LarsDenBakker I think your work on import-maps-resolve is relevant here, no?

With import maps you will be able to set up aliases yes, though importing the dist folder should work as well.
What kind of errors are you getting? Are yous ure this folder is inside the root directory served by karma? Otherwise it can't reach the file.

Hey @bennypowers @LarsDenBakker 👋
Ok it looks my issue was that I had to clear the node module cache and re-install all of the dependencies to the latest release.  I was missing lit-element reference. 🤦
Thank you for the quick response and great work, as you helped resolve my issue!!! 💯

[UPDATE]: Issue updated to include a reference solution for others in our community.
SOLUTION: For Importing ES Modules for Testing
After much consternation, I realize I was missing a step in our testing process. 🤦I could not figure out how to import our web component library for testing across all of our tests.  After a number of different iterations, our team came up with a solution using a Karma start script.
Reference Karma start/initialization script as a pattern
module.exports = config => {
  config.set(
    merge(createDefaultConfig(config), {
      files: [
        { pattern: 'karma-init.js', // reference start script here
          type: 'module'
        },
 ...

karma-init.js (name can be different)
/**
 * @overview Karma start script.  Used to initialize HelixUI ES Modules for testing.
 */

import HelixUI from './dist/js/helix-ui.module.js

HelixUI.initialize();

That's it! 🙌 Now it works locally and on TravisCI.


[terra-popup] Prevent default error

Summary
Fixes https://github.com/cerner/terra-core/issues/1602
The Tab menu would flicker closed then back open when the menu bar was tapped on mobile devices. This was due to an error on preventDefault() in onClickOutside handler in popup, there were updates to react-onclickoutside which prevent this error we just needed to update popup to use the preventDefault prop instead of calling it ourselves.
This can be verified on mobile devices by opening a popup and pressing the button that opened it and confirming that popup gets dismissed.
Deployment Link:
https://terra-framework-deploye-pr-338.herokuapp.com/#/components/terra-popup/popup/popup
Please add your name to the CONTRIBUTORS.md file. Adding your name to the CONTRIBUTORS.md file signifies agreement to all rights and reservations provided by the License.
Thanks for contributing to Terra.
@cerner/terra

Functional verification completed. The popup examples display and function as expected.


Improve the member pics component in home page

Description
It is very long and has a lot of repetetive code,
refactoring is definitely necessary.
Screenshots
No response
Additional information
Please avoid using the @apply in global.css, it is not the way to make classes more efficient, as per tailwind docs

<3


Cannot implement healthcheck as described in docs

The docs say:

You can write your own custom health-checking code. It should reply to /_ah/health requests with a HTTP status code 200.

However, this code and this handler appear to override the /_ah/health handler after the app is configured, such that the app cannot override the health handler in any way that I can see...
Is there some way I should be implementing my own health handler?
The only thing I can think of is to implement my own Dockerfile ENTRYPOINT that uses my own app, that tries to import vmruntime.wsgi.meta_app and override it after the fact...are there any other options?
(Note: I checked this awhile ago and noticed this behavior...finally getting around to writing up a bug report. Haven't checked it specifically on the latest code, but the issue looks to be the same.)

Hi Mike,
I'm sorry that you're having this problem.  Looking at the code, it's really unfortunate the way wsgi.py is implemented.  I don't see an easy solution other than some aggressive monkey-patching.  It might work to define your own gunicorn.conf.py, where you 'from vmruntime import middleware' and then set 'middleware.health_check_middleware = [your own health check function name]'.
Second, I need to mention that the python-compat runtime is deprecated, and will not advance to General Availability with the rest of AppEngine Flex.  If at all possible, I would recommend migrating to the 'python' runtime and using a supported third-party Python web framework such as Flask.  More information is available here: https://cloud.google.com/appengine/docs/flexible/python/runtime#application_startup .  This allows you to set the ENTRYPOINT via an 'entrypoint' directive in 'app.yaml'.

Thanks for the detailed response. Hadn't considered the gunicorn.conf.py angle, but that makes sense, and I may go with that, since it seems a bit cleaner than my proposal (which might be operating on the wsgi app too late).
As far as the deprecation....I'm aware, but I'm in a tricky spot (like many python-compat users). I am a very-long-time GAE user, and migrated to Managed VMs (then Flex VMs), which means Google is pulling the rug out from me on a bunch of services I now depend on:

memcache: waiting on the yet-to-be-released "App Engine Flexible Memcache" service (just applied for the "alpha")
push queues: which may or may not be supported in the yet-to-be-released "Cloud Tasks - Alpha" (just applied for the alpha)
NDB: the "supported/recommended" python Datastore library, which is not "supported" outside of Standard VM. I'm waiting on https://github.com/GoogleCloudPlatform/datastore-ndb-python/issues/272
search: I need to migrate to a third-party platform (possibly ElasticSearch, though I'll be paying a lot more...)
mail sending: I need to migrate to a third-party platform
login: admin functionality: I need to migrate to a third-party platform

And of course, since I jumped on the Managed VM / Flex VM bandwagon, I've also added enough features that make it difficult to migrate back to stock GAE:

Python native C modules
a co-resident JS server for rendering React pages

I assumed "deprecated" just meant an undetermined end date, though poking around now I see it has a date of October 27th, 2017. Ugh, let's hope Google staffs the above "alpha" projects and open bug requests appropriately, so I have time to rewrite my code to use them (or work around them). :)


Add 30m timeout to CI

Related to https://github.com/photopills/photopills-ios-alpha/pull/434

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall first build on feature/ci-30min-timeout at 87.844%

Totals

Change from base Build<PHONE_NUMBER>:
87.8%

Covered Lines:
5779

Relevant Lines:
6481

💛  - Coveralls


Avoid suggesting star dependencies.

A * version constraint makes it hard/impossible to release a breaking
change (e.g. clap 2.0) without causing people's code to break.  cargo update with a * dependency will happily move from (say) 1.4 to 2.0,
even though semver says these may be a breaking change. Using a more
precise dependency constraint gives the crate author more flexibility in
making breaking changes, and makes users of the crate happier if/when
they do happen.
https://github.com/rust-lang/rfcs/pull/1241 contains a bit more
discussion about this.

Indeed, it should be done, thank you!
Also you can do a commit adding yourself to CONTRIBUTORS.md.

Done!

Thank you!
@homu r+

:pushpin: Commit 45b101b has been approved by Vinatorul

:hourglass: Testing commit 45b101b with merge edec5ae...

:sunny: Test successful - status


Feature Request: provide bash/zsh completions

Feature Request
Is your feature request related to a problem? Please describe:
No.
Describe the feature you'd like:
Generate completion files for bash/zsh. It makes much easier with using this cli tool.
Describe alternatives you've considered:
Hands-write a completion script. 🤔
Teachability, Documentation, Adoption, Migration Strategy:
Nothing here. 😁

Sorry, duplicated with #73


Retest

Week 1 assessment - re-assessment!

nice, well done. Would love to see you work out blackjack?.


Connect to mongo docker with authentication from other docker (node)

Hi,
How can I connect to a docker with authentication from another docker?
I have followed this steps:
docker run --name some-mongo -d mongo --auth
docker exec -it some-mongo mongo admin
db.createUser({ user: 'jsmith', pwd: 'some-initial-password', roles: [ { role: "userAdminAnyDatabase", db: "admin" } ] });
And now the command to connect is said to be
docker run -it --rm --link some-mongo:mongo mongo mongo -u jsmith -p some-initial-password --authenticationDatabase admin some-mongo/some-db
but I cant success on merge this command with the one I had launching nodejs and connecting to mongo:
docker run -it --name start -p 9000:9000 -v pwd:/usr/src/app --link some-mongo:mongo node:4.2.4 bash -c "cd /usr/src/app && node server/app.js"
(it's the command I had working when mongo docker had no authentication)

Your Node.js application will need to also use authentication when connecting, and how to configure that is going to depend on the specifics of your Node.js application.


Update Koruri

Thank you to check my commit and comment.
Fix commit will be late...(few hours maybe)

Thank you.


how tf do i make icons next to freeplay songs

uhhhhhhh how do i do it? is it just
(icon name) (songname) in freeplaySongList.txt (if yes, it doesnt work) (my icons are in assets/characters/icons btw)
https://cdn.discordapp.com/attachments/829551530163568670/982916329692594236/unknown.png

its based on the song.player2 if a normal chart exists and its not being added through the EngineData.hx
if its added through EngineData.hx, it's based on the specified character in there

ok thanks


RuntimeError: Internal psf_fseek() failed.

Hi,
I try to reprocess Clotho and AudioCaps dataset, and calculate the retrieval metrics for audio and text. But when I processed the datas as you released and run the eval code. There is an error, RuntimeError: Internal psf_fseek() failed. when I used either Clotho or AudioCaps.
I'm not sure what make this error.
Thank you!

Hi,
I guess this is related to the sound file I/O after your machine. What system (linux, mac, window) does you use?
You can refer this (https://github.com/bastibe/python-soundfile/issues/274) to see if this is related to the flac format reading and libsndfile.


[Feature] Get higher bitrates with the “Samsung TV” UA profile

Device
All devices
Suggestion
Hi …
Triggered by that post on Reddit …
https://www.reddit.com/r/xcloud/comments/1h6ehvj/398_mbps_bitrate_on_samsung_tv/
… I played around with the BxC UA settings.
And found out that I can also get bitrates above 15 Mb/s by using the “Samsung Smart TV” UA profile. But only if “target resolution” and “visual quality” is set to “default”. Like this I’m seeing bitrate peak values up to around 20 Mb/s.
I only have a 50 Mb/s internet connection. So I don’t know if users with a faster connection might be able to get even higher values this way.
But as mentioned: This only works if resolution and visual quality is kept on “default”. When setting anything else here, the 15 Mb/s cap seems to be in place again.
So I think at the moment BxC is forcing 1080p@High on unsupported devices by claiming to be a Windows PC. But maybe you want to change this, so BxC identifies as a Samsung TV. And all users can benefit from the higher bitrates.
Thanks.

Thanks for letting me know. Added in 6.0.0-beta-3

Also did a post about this on Reddit:
https://www.reddit.com/r/xcloud/comments/1h6voq6/get_higher_bitrates_with_better_xcloud_and_the/
Because I’m curious what bitrates others will get like this.
But do you happen to know if this is something new? Or maybe Samsung TVs always got higher bitrates?


added my name

as the title suggest, I added a file with my name using the command touch

Thank you for contributing to this project! :)


Use 'tabIndex' instead of 'tabindex'

Check that this is really a bug

[X] I confirm

Reproduction link
https://github.com/konstaui/konsta/blob/b2bfcf0fb1cdeb85d80bebb7350b937644783f99/src/react/components/Link.jsx#L100
Bug description
The Link component uses tabindex instead of tabIndex property
https://github.com/konstaui/konsta/blob/b2bfcf0fb1cdeb85d80bebb7350b937644783f99/src/react/components/Link.jsx#L100
Expected Behavior
Use tabIndex="0"
Actual Behavior
Uses tabindex="0"
Konsta UI version
3.0.1
Platform/Target and Browser Versions
Windows 11 & Chrome 116
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Konsta UI issue and not a framework-specific issue

Would you like to open a PR for this bug?

[x] I'm willing to open a PR

Ha what timing, this has been bugging me for a few days too now and I literally just visited this repo to open a PR fixing it: https://github.com/konstaui/konsta/pull/170
@nolimits4web, thanks for this awesome lib, when you get a chance is it possible to also merge in https://github.com/konstaui/konsta/pull/167? If you need, I can make a new issue for it.
Cheers!


[Docs]:  #11982 [Feature] HubSpot CRM API Saas Integration

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
No response
Describe the problem

TODO

[ ] Evaluate if this task is needed. If not add the "Skip Docs" label on the parent ticket
[ ] Fill these fields
[ ] Prepare first draft
[ ] Add label: "Ready for Docs Team"

Field
Details

POD
BE Coders Pod

Parent Ticket
#11982

Engineer

Release Date

Live Date

First Draft

Auto Assign

Priority

Environment

As discussed with @jnikhila, the HubSpot Docs could become huge (~40 commands). So, we decided to segregate the work on different issues.
My proposal is to have an issue that serves as an introduction (what's HubSpot, how to get a token, how to create a data source), and other issues for the different scopes of the HubSpot Integration (CMS stuff, CRM stuff, Settings, Domains, etc).
Proposed Structure / Organization of Issues/Drafts

13466 - [Docs] #11404 [Feature] HubSpot Integration

What's HubSpot?
Creating a Private App and getting a Token
Creating a Datasource
Making a basic Query.

13267 - [Docs] #13026 [Feature] HubSpot CMS Domains API Saas Integration

Everything related to the CMS Domains commands.

13266 - [Docs] #13028 [Feature] HubSpot CMS URL Redirects API Saas Integration

Everything related to the CMS URL Redirects commands.

13021 - [Docs] #12151 [Feature] HubSpot Settings API Saas Integration

Everything related to the CMS URL Redirects commands.

17817 - [Docs]: #11983 [Feature] HubSpot Files API Saas Integration

Everything related to the Files commands.

17818 - [Docs]: #11982 [Feature] HubSpot CRM API Saas Integration

Everything related to the CRM commands.

17819 - [Docs]: #11984 [Feature] HubSpot CMS HubDB API Saas Integration

Everything related to the CMS HubDB commands.

Duplicate of 1280


thread fix

this pr just repackages the fix and test submitted by @nobodyinperson (thank you!), and applies the edit requested by @rcolyer, so it can hopefully be merged. having meshing internal and external threads is pretty important!

Does this mean I just started a print where my internal and external threads aren't going to mesh?

After that print finished, the threads didn't mesh. I pulled in @ellemenno 's changes and reprinted and the threads still didn't mesh but they were a little better. I'm assuming the issue I'm having is related to the precision of my printer so I'm just increasing the diameter of the internal threads by 2%.


compiler complaints about third party with 1.0.0-rc.4

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.

@angular/cli: 1.0.0-rc.4
node: 7.4.0
os: darwin x64
@angular/common: 2.4.10
@angular/compiler: 2.4.10
@angular/core: 2.4.10
@angular/forms: 2.4.10
@angular/http: 2.4.10
@angular/material: 2.0.0-beta.2
@angular/platform-browser: 2.4.10
@angular/platform-browser-dynamic: 2.4.10
@angular/router: 3.4.10
@angular/cli: 1.0.0-rc.4
@angular/compiler-cli: 2.4.10
Repro steps.

Repo: https://github.com/Complex-Portal/complex-portal-view
Branch: CP-97
Command ng-serve
The log given by the failure.

mbp:complex-portal-view muUser$ ng serve
** NG Live Development Server is running on http://localhost:4200 **
Hash: d54f1439cfea0f1f2c06
Time: 19366ms
chunk    {0} polyfills.bundle.js, polyfills.bundle.js.map (polyfills) 232 kB {5} [initial] [rendered]
chunk    {1} main.bundle.js, main.bundle.js.map (main) 238 kB {4} [initial] [rendered]
chunk    {2} styles.bundle.js, styles.bundle.js.map (styles) 158 kB {5} [initial] [rendered]
chunk    {3} scripts.bundle.js, scripts.bundle.js.map (scripts) 648 kB {5} [initial] [rendered]
chunk    {4} vendor.bundle.js, vendor.bundle.js.map (vendor) 7.73 MB [initial] [rendered]
chunk    {5} inline.bundle.js, inline.bundle.js.map (inline) 0 bytes [entry] [rendered]

ERROR in /Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2758,30): ']' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2758,37): ')' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2758,38): Unexpected token. A constructor, method, accessor, or property was expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2758,39): Unexpected token. A constructor, method, accessor, or property was expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2869,28): ']' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2869,41): ';' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,35): ']' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,48): ')' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,59): '=' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,60): Expression expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,61): Unexpected token. A constructor, method, accessor, or property was expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,39): ']' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,52): ')' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,63): '=' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,64): Expression expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,65): Unexpected token. A constructor, method, accessor, or property was expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16136,23): ';' expected.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2719,42): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2758,18): A parameter initializer is only allowed in a function or constructor implementation.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2869,9): Type 'string' is not assignable to type 'C[]'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2869,18): All declarations of '['params']' must have identical modifiers.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2869,42): Duplicate identifier '['params']'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2869,42): Subsequent variable declarations must have the same type.  Variable '['params']' must be of type 'C[]', but here has type 'any'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2869,42): All declarations of '['params']' must have identical modifiers.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,22): A parameter initializer is only allowed in a function or constructor implementation.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,25): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,49): Duplicate identifier '['params']'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,49): Subsequent variable declarations must have the same type.  Variable '['params']' must be of type 'C[]', but here has type 'boolean'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2870,49): All declarations of '['params']' must have identical modifiers.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,26): A parameter initializer is only allowed in a function or constructor implementation.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,29): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,53): Duplicate identifier '['params']'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,53): Subsequent variable declarations must have the same type.  Variable '['params']' must be of type 'C[]', but here has type 'boolean'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,53): All declarations of '['params']' must have identical modifiers.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (2871,67): Duplicate identifier 'void'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (3210,23): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (3296,31): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (15528,43): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (15723,36): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16136,17): Cannot find name 'keyof'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16738,29): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16780,23): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16807,30): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16808,34): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16829,33): Cannot find name 'Partial'.)
/Users/myUser/IdeaProjects/Complex-Portal/complex-portal-view/node_modules/litemol/dist/js/LiteMol-plugin.d.ts (16840,33): Cannot find name 'Partial'.)

Desired functionality.

Before I installed "@angular/cli": "^1.0.0-rc.4", I was using "@angular/cli": "^1.0.0-beta.30" and it did not give me this error. Why is this happening now? Basically this is now not allowing me to run my application.
Mention any other details that might be useful.

Updating to "typescript": "^2.0.10" solved it. I was on "typescript": "^2.0.3" before.


[FRS-47] Job can switch to new shuffle cluster only when the old cluster is unavailable

What is the purpose of the change
This solves #47 . Currently, a running job will switch to the new shuffle cluster if the new one has higher version. However, this may cause the shuffle data of one job is managed by multiple clusters which is complicated and easy to cause problem. By only switching to new cluster when the old cluster is unavailable. We can simplify the logic and further support upgrading without influence the running job if the old cluster is kept until all running jobs finish.
Brief change log

Job can switch to new shuffle cluster only when the old cluster is unavailable.

Verifying this change
This change added tests.

Hi @wsry Is this also solved by https://github.com/flink-extended/flink-remote-shuffle/pull/49 ?


Pardon the interruption 🙏🏻

Hello @SelviA, 👋🏻
First, I apologize for reaching out via a pull request. I could find no other means through which to contact you.
I am interested in the GitHub username you own (@SelviA). As you were the first to claim it, I fully respect your right to hold it for perpetuity. However, if you are open to an exchange, I am interested in obtaining it for my organization. I understand this request may incur some imposition upon you, so I'd be willing to compensate your effort should you be willing. I'll include my email below. Hope to hear from you.
Thank you for your consideration,
Andrew Selvia<EMAIL_ADDRESS>

Have you made a decision regarding my proposal?


Pokestop will become ??? if it's unicode...

[2016-09-12 22:59:12] [  SpinFort] [INFO] Spun pokestop ???????. Experience awarded: 50. Items awarded: 3x Pokeball
[2016-09-12 22:59:17] [MoveToFort] [INFO] Moving towards pokestop ???????? - 0.05km
[2016-09-12 22:59:23] [  SpinFort] [INFO] Spun pokestop ????????. Experience awarded: 50. Items awarded: 1x Pokeball, 2x Greatball
[2016-09-12 22:59:28] [MoveToFort] [INFO] Moving towards pokestop ???? - 0.12km
[2016-09-12 22:59:33] [MoveToFort] [INFO] Moving towards pokestop ???? - 0.11km
[2016-09-12 22:59:39] [MoveToFort] [INFO] Moving towards pokestop ???? - 0.10km

Not really. I becomes "????" because your stdout in not configured to display utf-8 encoded characters.
If you change that, you will see all the characters.

This is a buggy thing at windows. It's not possible to set default charset to UTF-8 because lots of application can stop working. But there is nice solution for this, before running python pokecli.py if you run this command then your UTF-8 problem will be resolved chcp 65001
Important thing is you should run the command everytim before you start the bot ;)
I think this will help you.


chore: update NPM dependencies

As reported in b/158734154 for Google-internal.

Note that the tests appear to be failing, but I honestly have no clue why :( Travis also doesn't seem to be running the tests, so I don't know what could have gone wrong. This is the failure message:
 FAIL  src/WasmDis.spec.ts
  ● NameSectionReader › Wasm module with bad names

    parseWat failed:
    test.wat:3:11: error: redefinition of function "$foo"
             (import "import" "function2" (func $foo))
              ^^^^^^

      251 |
      252 |   test("Wasm module with bad names", () => {
    > 253 |     const { buffer } = parseWat(
          |                        ^
      254 |         `test.wat`,
      255 | 	`(module
      256 |          (import "import" "function" (func $foo))

      at parseWat (node_modules/wabt/index.js:26:14344)
      at Object.<anonymous> (src/WasmDis.spec.ts:253:24)

 PASS  ./test.ts
 PASS  ./wabt-compatibility.spec.ts

Test Suites: 1 failed, 2 passed, 3 total
Tests:       1 failed, 876 passed, 877 total
Snapshots:   0 total
Time:        2.849 s, estimated 6 s

Test failures are related to the upgraded version of wabt. Downgrading to 1.0.13 fixes the issue. The regression is therefore in<EMAIL_ADDRESS>or<EMAIL_ADDRESS>(although the 1.0.14 is not published on NPM). Release notes are at https://github.com/WebAssembly/wabt/releases/tag/1.0.15
Likely an issue/change in Emscripten?

Travis also doesn't seem to be running the tests

I think I address that in #63 -- can you re-push this branch?

Done.

:tada: This PR is included in version 2.2.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


stuck in a world

I have a problem with multiverse when I create a world and tp me it works but when I want to return to my home world it does not work
[url=https://www.noelshack.com/2020-14-1-1585551049-2020-03-29-23-31-55.png][img]https://image.noelshack.com/minis/2020/14/1/1585551049-2020-03-29-23-31-55.png[/img][/url]
[url=https://www.noelshack.com/2020-14-1-1585551049-2020-03-29-23-31-55.png][img]https://image.noelshack.com/minis/2020/14/1/1585551049-2020-03-29-23-31-55.png[/img][/url]

I am not very sure is world names with spaces is supported, doesnt look like it is. Can you try creating 2 worlds with no spaces in its name and see if you can teleport between them?
If possible, then you will have to bakcup, remove, rename, then reimport your worlds without space.

It's best (way faster as we use GitHub to track bugs/suggestions) to receive support on the Multiverse Discord.


Degree Pathway Component v2

Redesigned degree pathway component and added new fields to display

yeah, so those are the yog fields, degree type fields, etc. once the pathway dialog is finished those should be properly collected and shown.

you can manually edit the localstorage object if you want and it will show correctly

Sounds good, that seems to work. Those fields don't seem to be naturally visible on the local storage info. Adding in the mappings seems to fix the issue.

yeah theyre not yet, i was gonna leave that to @ultralapse. He's the one working on the pathway dialog.

And im still gonna keep working on this, I realized I should probably implement update and delete operations for the pathways

Also, I think originally, the tags were meant as a way to track requirements like Arts and Humanites requirements, science with lab requirements, etc. When you create your courses, you would select the tags that a course would satisfy. Does that make sense?

I think we really need to outline the attributes of some of the internal objects. Would really define their behavior and subsequent UI. Backend Team Assemble!

Yeah we should talk more about the tags Thursday, I was a little confused on how thay was supposed to work


PHP cs-fixer setup

The fixer is required, but never setup.
Now it is. :smile:
Some points:

I took benefit of not release yet version to introduce risky fixers, especially for PHP and strict typing.
I introduced PHP 7.1 rule. So #264 must be discussed and merged before this one
I add the generated folder to the configuration. This may have to be modified after #262

I also add some rules. Feel free to comment to update/remove/add some. Here is the list: https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/2.7/README.rst

 [Symfony\Component\Process\Exception\ProcessTimedOutException]
 The process "vendor/bin/php-cs-fixer fix --dry-run --verbose --diff" exceed
 ed the timeout of 300 seconds.

Well.

Can you rebase this ? Would gladly accept

@joelwurtz done!

Thanks a lot


i18n(ko-KR): update api-reference.mdx

Description (required)

update api-reference.mdx
#9687

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
reference/api-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Once #9803, #9804 are merged, it is expected that all errors in this PR will be resolved.


.:16: maximum nested function level reached

os: ubuntu kylin 14.04
workerman version: 3.4.2
mutiple_process.php file like below:
$protocol_work = new Worker('tcp://<IP_ADDRESS>:8686');
$protocol_work->name = 'protocol';
$protocol_work->protocol = 'Workerman\\Protocols\\Http';
$protocol_work->onMessage = function($connection, $data)
{
    var_dump($_GET, $_POST);
    $connection->send("使用的协议是" . $protocol_work->protocol . "\n");
};

// transport 设置传输层协议
$tran_work = new Worker('text://<IP_ADDRESS>:8484');
$tran_work->name = 'transport';
$tran_work->transport = 'udp';
$tran_work->onMessage = function($conn, $data){
    $conn->send('transport is {$tran_work->transport}\n');
};...

When I stop the process by php mutiple_process.php stop , it said
.:16: maximum nested function level reached
.:5: maximum nested function level reached

Looks like an error thrown by the xdebug extension.
Try to disable it.

不是吧，我没启用这个扩展呢。


Please upload the following transformers library you are using.

I found some undefined elements in the official library of transformers.
Is it necessary to make a little modification to the source code of the transformers library?
Please upload the following transformers library you are using.

Yes, the author did not upload the modified code, transformers and rouge


Add MetricKit crash collection

Please review the release process for BrowserServicesKit here.
Required:
Task/Issue URL: https://app.asana.com/0/1200194497630846/1203353338544857/f
iOS PR:
macOS PR:
What kind of version bump will this require?: Minor
Optional:
Tech Design URL:
CC:
Description:
Adds pixel based Crash Collection.
Steps to test this PR:

See iOS PR https://github.com/duckduckgo/iOS/pull/1381

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16
[ ] macOS 10.15
[ ] macOS 11
[ ] macOS 12

Internal references:
Software Engineering Expectations
Technical Design Template

Trying to merge wrong downstream repo.