Allow defining credentials from variable environment

Description
It is not clear how one can best deploy Kedro with credentials for production. A good practice in traditional applications is to put sensitive credentials using environment variables.
In Kedro, it looks like it relies only on credentials.yml, located somewhere except in conf/local/ folder.
(PS: If I'm missing something with this, the problem is probably bad or missing documentation since I'm not able to find this information in the doc)
Context
Using environment variables will help standardize the deployment of Kedro like any app, thus reduces learning curve for developers.
Possible Implementation
Possible change could be to look for environment variables first, before looking for the content of credentials.yml. Changes will mainly be located in ConfigLoader class.
Possible Alternatives
I create credentials.yml file in config/base/ folder, because this folder ignored by Git, but is still packaged in the Kedro Docker. This is still not good because the credentials are now located in the Docker images repository: Anyone can pull that image and get prod credentials!

If this is an interesting implementation (which I think it is), I would be happy to contribute by implementing it. I first want to open the discussion about the best implementation given Kedro orientation, before making any pull request.
Regards,

@htaidirt, thank you for your contribution. Please find some comments below.

It is not clear how one can best deploy Kedro with credentials for production.

Currently Kedro supports deploying credentials for production via configuration environments. Default project template contains 2 of them - conf/base is intended for storage of non-sensitive shareable configuration, and conf/local, where you can put sensitive credentials. conf/local is in .gitignore by default. Please find more information on how kedro configuration module works in this section of documentation.

I create credentials.yml file in config/base/ folder, because this folder ignored by Git, but is still packaged in the Kedro Docker. This is still not good because the credentials are now located in the Docker images repository: Anyone can pull that image and get prod credentials!

conf/base is indeed copied into Docker image, however, as documentation suggests, it is not intended to store any credentials. You should rather store your credentials in conf/local/credentials.yml, which is in .dockerignore by default.

Using environment variables will help standardize the deployment of Kedro like any app, thus reduces learning curve for developers.

Currently you can manually construct/enrich your credentials dictionary in src/<package_name>/run.py with any data, including one coming from the environment variables.
In long term we consider adding templating capability for kedro configs, which may, possibly, handle environment variables, however exact specification hasn't been finalised yet.

We have updated our docs regarding credentials in https://github.com/quantumblacklabs/kedro/commit/5f4325f88d9b84bbf463b67cccd2dd785669f4c0. So I am closing this for now, but feel free to re-open it if this is still an issue :)

Hello,
I am trying to implement the following solution.

Currently you can manually construct/enrich your credentials dictionary in src/<package_name>/run.py with any data, including one coming from the environment variables.

When I clone the project, the file conf/local/credentials.yml is not present,
I want it to be created from env variables in the following situations:

Running kedro run from command line
Running context = load_context(MY_PATH) from a notebook (not a kedro notebook)

For this I modified run.py as follows:
import yaml

class ProjectContext(KedroContext):
    project_name = "my_project"
    project_version = "0.16.1"
    package_name = "my_package"

    def __init__(self, project_path, **kwargs):
        super().__init__(project_path,  **kwargs)
        self._set_credentials()

    def _set_credentials(self):
        kedro_project_dir = self.project_path
       credential_file = os.path.join(kedro_project_dir, "conf", "local", "credentials.yml")

        credentials = {
            "my_credential" = os.getenv("MY_ENV_VAR")
        }

        with open(credential_file, "w") as file:
            yaml.dump(credentials, file)

    def _get_pipelines(self) -> Dict[str, Pipeline]:
        return create_pipelines()

This does the job for now but is it future proof ?
(I guess the init argument of the super class KedroContext might change in the future).
Is there a better way to do this?

@HugoPerrier Can you please open a new issue and cross-link to this thread in there? Otherwise we won't get a visibility on thee updates on the closed issues.
Regarding your question: There might be a better way of handling your credentials using TemplatedConfigLoader. You can create conf/local/credentials.yml manually and add templated credentials in there (similar to catalog.yml in the example above). That way the file won't contain any static secrets and can be committed to the repository. The actual secret values will come from the corresponding environment variables, which will be resolved by TemplatedConfigLoader at runtime.

I posted my solution in
How do I fill the credentials from environment variables #403
Thanks for the help

I am not sure how I can use the crendtials for CI environment, I cannot just put in the YAML in the local/ folder.


multilayer does not show used layer names on plot

When I select multiple layers the plot title keeps the first one.
The UI shows in a line what was selected, this can also be optimized, because we don't have much space there to get many names concatenated in a row.
On the UI the selection in the ML view is where we have this information on the correct place,
The information in the row of the server/layer button could show "multilayer"
On the plot we maybe could enhance the title or put another documentation above the lines where we write the flight track name. The plot is the region one looks on after multilayer were selected. So we have to add that information there.

I recognized that the UI did not show in the line what was selected but an indicator that more data is used.
Currently I think we should do the same on the plot. This makes it easier to see that more data is involved.
For showing what is  selected we need a better idea than using the title for this information.


Better schedule-importing and task-completion handling.

Features:

Only show a task in the "incomplete stuff" list on the first day it becomes "incomplete"

If we get a new schedule for a task, and that task is a completed, one-time-only task, we ignore the new schedule

Decomposed the import process into smaller, more maintainable components, moving it out of APCScheduler.m.  Now, we have:
a)  ScheduleImporter:  top-level logic for importing schedules and tasks
b)  ScheduleFilter and its subclasses:  analyze schedules during the import process
c)  ScheduleTaskMap:  stores the results of CoreData queries needed by many of those Filters

@YuanZhu-apple — Just wanted to add your name to this, to make sure you saw it.  —ron

@RonConescu Sure, I am looking now.

@YuanZhu-apple — Hi!  I just wanted to check in.  Are you waiting for me to fix the above, before you continue your analysis?  Or can you tell me:  once I fix the above, will this pull request be considered acceptable?  —ron

@RonConescu
Once the fix is there, I accept this PR

Terrific.  Thanks.  In progress.

@YuanZhu-apple - Good comments. Thanks. Here you go.

Good.


PHP 8 Support for Glide v2

Check this to accept merge request for v2

Hi @nagibmahfuj !
2.0 isn't released yet and the current branch for 2.0 is going to be deleted at some point as we will release Glide 2.0 as the same library as Glide 1 but with Flysystem 2.0 support. But thanks for the PR!


Renaming files imported with alias/absolute paths does not update the path

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.90.2
OS Version: macOS 14.1.1 (23B81)

Steps to Reproduce:

Create a project with alias imports (e.g. setting up a new Nextjs project and configure to use @/ alias imports)
Create a new typescript file
Import this file, and ensure to import it using your @/ alias
Rename the created typescript file
The error is that the reference in the import is not updated, and now vscode is complaining it cannot find the file

See this short video:
https://github.com/microsoft/vscode/assets/3786627/8fb2ffd2-af46-4664-a44a-675649b17543
tsconfig:
{
  "compilerOptions": {
    "lib": [
      "dom",
      "dom.iterable",
      "esnext"
    ],
    "allowJs": true,
    "skipLibCheck": true,
    "strict": true,
    "noEmit": true,
    "esModuleInterop": true,
    "module": "esnext",
    "moduleResolution": "bundler",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "jsx": "preserve",
    "incremental": true,
    "plugins": [
      {
        "name": "next"
      }
    ],
    "baseUrl": "./src",
    "paths": {
      "@/*": [
        "./*"
      ]
    },
  },
  "include": [
    "next-env.d.ts",
    "**/*.ts",
    "**/*.tsx",
    ".next/types/**/*.ts"
  ],
  "exclude": [
    "node_modules"
  ]
}

Possibly related:

https://github.com/microsoft/TypeScript/issues/27959
https://github.com/microsoft/vscode/issues/97135

I noticed you invoked the rename via a third-party extension called File Utils. If you rename using only VS Code's built-in functionality, do you still experience the issue?

@andrewbranch I can confirm that it happens also using VS Code's built-in functionality to rename

@andrewbranch  i had the issue as well

I experience the same issue. Alias path does not work but relative path will update accordingly after file path changed.

Confirming this is a real bug (regression) and @bastiankistner's workaround is valid:

Disable the setting "TypeScript › Tsserver › Experimental: Use Vs Code Watcher"
The default seems to be enabled (checked), which sets equivalent:
"typescript.tsserver.experimental.useVsCodeWatcher": false
Important! Before dismissing this, see below why this might affect some people and not others -- There is a gradual rollout of this happening right now!

As part of diagnostics, I tried both alternate choices for "TypeScript: Select TypeScript Version...":

"VS Code's Version 5.5.2": Bug still applies. Workaround works.
"Use Workspace Version 5.5.2": Bug still applies. Workaround works.

For context, this seems to have been introduced around April/May 2024. See the VS Code 1.89 release notes which include:

A new experimental setting typescript.tsserver.experimental.useVsCodeWatcher controls if the TS extension is using VS Code's core file watching support for file watching needs. TS makes extensive use of file watching, usually with their own node.js based implementation. By using VS Code's file watcher, watching should be more efficient, more reliable, and consume less resources. We plan to gradually enable this feature for users in May and monitor for regressions.

Hope this can get to the right people doing that monitoring before they roll it out any further!

The 2024 VS Code regression with @aliased imports no longer updating on file moves has been identified. There is a short-term workaround, but your feedback and comments are essential to getting this escalated properly for a real fix.
For background, root cause, why it's complicated, related issues, workaround, and next steps, see the full analysis of the regression here:

https://github.com/microsoft/TypeScript/issues/51763#issuecomment-2282329357

Join the discussion there and let's get this resolved!

Amazing work @dkamins, thanks!

This was fixed by #59418.

As part of diagnostics, I tried both alternate choices for "TypeScript: Select TypeScript Version..."

You only tried 5.5.2—the fix is in nightly TS versions and will be released in 5.6 stable.

You only tried 5.5.2—the fix is in nightly TS versions and will be released in 5.6 stable.

FYI there is a video by @Bunkerbewohner in the new centralized issue concerning the import updating problem:

https://github.com/microsoft/TypeScript/issues/59603#issuecomment-2284919153

It shows the problem still occurring with TypeScript 5.6.0-dev.20240811.
(I also suggest moving the discussion there, where the issue is open, there is more attention, and clear repro steps exist).


Different writing behaviors before and after reading the navigation collection

Short description
Writing to the database behaves differently based on whether or not I read the navigation collection property of the entity.
Steps to reproduce
Solution to replicate the bug: EntityFrameworkTesting.zip
Solution description
So, there is PizzaShop entity, Deliveryman entity and  PizzaShopDeliveryMan linking entity (for many to many relationship).
My goal is to create new version of PizzaShop entity every time when any changes are made but both the original PizzaShop and new modified one are pointing via PizzaShopDeliveryMan to the same Deliveryman. I achieve this by 3 steps:

Get entity from database and made changes.
Detach modified entity and set Id to 0 so the database will think this is new entity.
Add entity to the database.

In the first approach I didn't get anything from pizzaShop.PizzaShopDeliveryMans. I just set new collection with the same Deliveryman. Result of this approach is desired and that is original PizzaShop and new PizzaShop has both their own PizzaShopDeliveryMan linking entity connecting them to one Deliveryman.
In the second approach I get (lazy loaded) entities from pizzaShop.PizzaShopDeliveryMans. Result of this approach is that original PizzaShop lost his PizzaShopDeliveryMan linking entity (the linking entity is still there but the foreign key of PizzaShop is null), still the new PizzaShop correctly has his own new PizzaShopDeliveryMan linking entity pointing to Deliveryman.
!Note that result of the second behavior could be achieved also in the first approach if you in the step mod get (lazy load) pizzaShop.PizzaShopDeliveryMans.
So the problem is really sneaky because the writing behavior really change when you read the navigation property in any way. You can really have different changes in database just by reading property of the entity in debug mod.
Further technical details
EF Core version: 2.2.6
Database Provider: Npgsql.EntityFrameworkCore.PostgreSQL
Operating system: Windows 10 Pro
IDE: Visual Studio 2019 16.2.1

@LFusek I'm having trouble understanding what you are trying to do. Can you explain a bit more what you mean by, "My goal is to create new version of PizzaShop entity". What is a "new version" in this context?

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Examples updates

Fixes bugs, updates to newer techniques, and adds some new examples. In addition, I've replaced the old examples.md file with an examples.html that loops over a new data file, examples.yml, to generate our list of examples.

Updated descriptions for all examples.
Cover:

Fixes a bug where the nav would be left aligned on mobile devices.
Replaces table hack for vertical centering with flexbox, simplifying the HTML and CSS.

Sign-in:

Vertically centered the form with flexbox.
Added example image and a footer to round it out.

Offcanvas:

[ ] Update visuals, adjust grid, fix footer

Added new Pricing example. This closes #18296.
Updated descriptions for all navbar examples.

Still have some more to do here, so leaving as WIP for v4 stable.

@mdo: when you are ready, please squash any patches that can be squashed.


encoding not respected on read_msgpack

as discussed on https://groups.google.com/forum/#!topic/pydata/ngROaML_hLI
encoding does not seem to be respected on reading a msgpack, below i am expecting to get back what
I put in as utf8
In [17]: s
Out[17]: u'\u2019'

In [18]: s = pd.Series({'a' : u"\u2019" })

In [19]: s.values[0]
Out[19]: u'\u2019'

In [20]: pd.read_msgpack(s.to_msgpack(encoding='utf8')).values[0]
Out[20]: u'\xe2\x80\x99'

in stepping through, part of the problem seems to be that in the call to unpack on https://github.com/pydata/pandas/blob/master/pandas/io/packers.py#L134 that there is no encoding argument passed and so it defaults to latin1 in https://github.com/pydata/pandas/blob/master/pandas/io/packers.py#L558
changing L134 to :
l = list(unpack(fh, **kwargs))

and passing the encoding like:
pandas.read_msgpack(m, encoding='utf8')

makes it work for me, however i don't have en environment set up to submit this as a pull request via GH, and we're still using 0.14.1 due to compatibility issues.

even after making this change, this surprisingly raises a UnicodeDecodeError in msgpack.cpp:
pandas.read_msgpack(pandas.DataFrame([[401L, u'a']], index=[0], columns=['k', 'v']).to_msgpack(encoding='utf8'), encoding='utf8')

but i'm having trouble stepping through as my PyCharm environment crashes on me when inspecting


Various FIXME notes in test

also a wobbly test on 04-uni.t

did a bunch of these ... many remain


Modify shuffle to use identical algorithms for any number of dimensions

For reasons unknown at the moment, std::uniform_int_distribution generates different sequences for linux and non-linux environments while std::shuffle generates identical permutations. In order to produce identical permutations for arrays of arbitrary rank the one-dimension specialization should execute an algorithm analogous to the multi-dimension specialization. This code should have little if any compromise on performance (as it should pretty much be what the STL implements).
Additions to the test code were made to compare the results of shuffling arrays with the same leading dimensions.

I've added a comment. Maybe the integer type difference can explain the difference in sequences?

great PR!


⚠ ClamAV has degraded performance

In 7a7b9e2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 77 ms

Resolved: ClamAV performance has improved in 6125731.

Resolved: ClamAV performance has improved in c9ebdd2.


Add Provenance / fix permission

/release batch

From Release Bot
The /release command requires one of major, minor, or patch.
/release major
/release minor
/release patch

/release patch


Notation support - feature request

Some DAWs support notation, and exchange between them and notation software has always been very annoying with MIDI
If this project included some way of including metadata that would help those daws and notation software make the midi easier to turn into readable scores, we could improve that process. Just some general way to add notation metadata to notes as an optional thing.
Metadata like
"this note is to be notated as a quarter-note" (even though it is off-time due to humanizing)
"this note is to be notated as a staccato quarter note" (and not an 8th note and some silence)
"there is an indication to play using this other articulation from this note on"
"there is a chord symbol above that note"
"use a sharp and not a flat when notating this note" (enharmony)
It would be a lot of work to standardize that, but that could be the aim of another project if you just provide the ability to have such metadata

An example
<clips>
          <!-- note clip -->
          <clip time="8" duration="8">
            <notes id="5">
              <note time="3" duration="0.5" key="55" vel="0.8">
                <notation duration="quarter" enharmony="#"/>
            </notes>
          </clip>

musicXML repo: https://github.com/w3c/musicxml


Make server port configurable via environment variable

Add a check for environment variable named SERVER_PORT that allows you to specify the port number the server will use.
Should default to 8080.

Hi! Could you assign me to this task? 🙇


Feature/conduit support

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: michaelcfine:white_check_mark: kschrief:x: kschri201kschri201 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Implementing Phone- and Tablet-specific Styling could use a better UI

https://play.nativescript.org/?template=play-ng&id=uJIT6r&v=2

Minor changes: https://github.com/NativeScript/code-samples/commit/9098f5f0e7ad72a7c63b5e7cb3848bdec16bbdf2
Probably good enough for now :)


Add desktop.ini to default .gitignore

Is this a bug report?
No
I just noticed that the default .gitignore includes the .DS_Store  stuff that MacOS creates but doesn't include Thumbs.db or desktop.ini. I like to customize my icons and this makes Windows create the desktop.ini file.
It's a very very minor thing - I can make a PR for this if it's okay.

Well, .DS_Store is in the default .gitignore :D Why not also add Windows-defaults as well ?

@bogdan-calapod because it's my personal one, so I only keep the entries I need.

I was talking about the .gitignore that create-react-app automatically creates when you run create-react-app myapp


Remember Me fix

This properly unravels the _.forEach construct in order to accurately mimic the behavior of the async.series which it replaced.

 [uploader] [VCA] [bug] After clicking remember me, then closing and re-opening app, app gets stuck on "Loading..."

ping @gniezen for PR review


Add manager info in user endpoint payload

Add the manager full name as user endpoint payload.
Note: We need to verify whether we have the data correctly in manager.

cc @ttannis @jinhyukchang @danwom


Add support for NoReturn in auto-return-typing

Summary
Given a function like:
def func(x: int):
    if not x:
        raise ValueError
    else:
        raise TypeError

We now correctly use NoReturn as the return type, rather than None.
Closes https://github.com/astral-sh/ruff/issues/9201.

Does this use typing.Never in Python 3.11+?

Does this use typing.Never in Python 3.11+?

They're effectively identical in regards to type-checkers. Pyright (for Python 3.11+) infers the correct type for a function like the one given in the example as NoReturn. typing.Never` is used as a bottom type in situations like this:
def int_or_str(arg: int | str) -> None:
    never_call_me(arg)  # type checker error
    match arg:
        case int():
            print("It's an int")
        case str():
            print("It's a str")
        case _: # type checkers considers this unreachable
            never_call_me(arg)  # Pyright would say `arg` is of type `Never`.

Does this use typing.Never in Python 3.11+?

They're effectively identical in regards to type-checkers. Pyright (for Python 3.11+) infers the correct return type for a function like the one given in the example as NoReturn. typing.Never is used as a bottom type in situations like this:
def int_or_str(arg: int | str) -> None:
    never_call_me(arg)  # type checker error
    match arg:
        case int():
            print("It's an int")
        case str():
            print("It's a str")
        case _: # type checkers considers this unreachable
            never_call_me(arg)  # Pyright would say `arg` is of type `Never`.

[https://docs.python.org/3/library/typing.html#typing.Never](Official Python docs) state that starting in 3.11 typing.Never should be used instead of typing.NoReturn`.


Elevate the log level for messages that record which toolchain was used for a document

This is some fairly essential information for troubleshooting and should be logged at the same level as the build settings for a file.

@swift-ci Please test


[feature request] make the preprocessing more generic

I'm trying this great work to generate music from other midi files but there is some kinda "magic" parts in the preprocess.py file for me.
some code parts use magic numbers like
melody_stream = midi_data[5]
partIndices = [..]
other require strong knowledge of the example midi file & music21 library.
I manage to achieve the __parse_midi function to return values without exception but got an error later in __get_abstract_grammars (return empty list)
after change I changed 4 more thing and finally the training started but got a value error in __generate_grammar after a few iterations.
Basically I would like to make the generator to work with a preprocessed midi file with single channel .

I got the same problem when i try use my own MIDI file.
I think there is some knowledge about music waiting to learn.:）

Pretty late to the party here, and maybe this issue has already been solved, but did either of you figure this out? I'm having the exact same problem here. I took one of my own midi files that is just one single part, but it won't go through the preprocessing. If either of you still have the code around, what did you end up doing about this? Thanks and sorry for being a few years late...

im even later and now the code doesn't even work on the original file :(


literal_unroll: allow setitem in loop; also allow zip, enumerate, et al. w/ literal_unroll?

Feature request
In the end, it would be nice to be able to dynamically assign values arbitrary fields of a struct in a structured array (records array / record array). See, e.g., issue #5756. Another way to solve this problem is hinted at in the docs, where literal_unroll is used to unpack field names in  a struct array, but this only works for getitem while it fails for setitem. E.g., one could imagine an alternate code to what @marnixhoh would read as:
from numba import njit, literal_unroll
import numpy as np

struct_dtype = np.dtype([
    ('one', np.float64),
    ('two', np.float64)])

columns = struct_dtype.names  # note: tuple, so legal to use w/ literal_unroll

@njit
def my_test():
    struct_array = np.empty((2, 1), dtype=struct_dtype)
    for column in literal_unroll(columns):
        struct_array[column][0] = 1234
    return struct_array

result = my_test()
print(result)

This currently fails, with
TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Invalid use of Function(<built-in function getitem>) with argument(s) of type(s): (unaligned array(Record(one[type=float64;offset=0],two[type=float64;offset=8];16;False), 2d, C), Literal[str](one))

It would further help to make code more dynamic and concise (and avoid ugly code-generation type solutions) if that could be encapsulated in the function itself:
@njit
def my_test():
    struct_array = np.empty((2, 1), dtype=struct_dtype)
    for column in literal_unroll(struct_dtype.names):
        struct_array[column][0] = 1234
    return struct_array

One common issue I have is populating a structured array's fields with individual values that come from outside such a structured array (i.e., not with the structured array's same dtype). This is solve-able in plain python/numpy via
import numpy as np

struct_dtype = np.dtype([
    ('one', np.float64),
    ('two', np.float64)])

def my_test():
    struct_array = np.empty((2, 1), dtype=struct_dtype)
    values = (0, 1)  # e.g., values = get_from_another_func()
    for column, val in zip(struct_dtype.names, values):
        struct_array[column][0] = val
    return struct_array

result = my_test()
print(result)

and might look like the following, using zip with literal_unroll to make it concise in Numba:
from numba import njit, literal_unroll
import numpy as np

struct_dtype = np.dtype([
    ('one', np.float64),
    ('two', np.float64)])

@njit
def my_test():
    struct_array = np.empty((2, 1), dtype=struct_dtype)
    values = (0, 1)  # e.g., values = get_from_another_func()
    for column, val in zip(literal_unroll(struct_dtype.names), values):
        struct_array[column][0] = val
    return struct_array

result = my_test()
print(result)

Using zip can be avoided via a counter that is manually incremented in the loop, but this is more error-prone, takes more lines of code, and is less Pythonic.
One "basic but practical" use case that has arisen for me is selectively copying particular values from one array to another. The following pure Python/numpy example works as desired:
import numpy as np
from numba import njit, literal_unroll

def select_and_populate(input_array, output_array, x_thresh):
    for input_row, output_row in zip(input_array, output_array):
        if input_row['x'] >= x_thresh:
            for field in ('x', 'z'):
                output_row[field] = input_row[field]

dtype = np.dtype([('x', np.float32), ('y', np.float32), ('z', np.float32)])

input_array = np.ones(10, dtype=dtype)
input_array["x"] = np.arange(10)

output_array = np.zeros(10, dtype=dtype)

select_and_populate(input_array, output_array, x_thresh=5)
print(output_array)

while something analogous with literal_unroll might look like:
import numpy as np
from numba import njit, literal_unroll

@njit
def select_and_populate(input_array, output_array, x_thresh):
    for input_row, output_row in zip(input_array, output_array):
        if input_row['x'] >= x_thresh:
            for field in literal_unroll(('x', 'z')):
                output_row[field] = input_row[field]

dtype = np.dtype([('x', np.float32), ('y', np.float32), ('z', np.float32)])

input_array = np.ones(10, dtype=dtype)
input_array["x"] = np.arange(10)

output_array = np.zeros(10, dtype=dtype)

select_and_populate(input_array, output_array, x_thresh=5)
print(output_array)

Obviously that is simple to solve in advance with an extra line of code writing values to specific field names, but being able to more dynamically define which field names are assigned to and not having to write 100 lines of code to assign to 100 fields is where a fix to this becomes a "big deal."

@jllanfranchi thanks for starting this discussion, I believe it will be important for Numba to include more mature support for structured arrays. Please be advised however, that we are currently preparing to ship a release candidate (RC) and will begin reviewing PRs and tending to discussions again once the RC and the final release is out. We thank you in advance for your patience!

@jllanfranchi I think you raise some very good and interesting points. In my opinion, it would be ideal if Numba supports setting an entire row of a structured array in one line using a tuple:
from numba import njit
import numpy as np

struct_dtype = np.dtype([
    ('one', np.float64),
    ('two', np.float64)])

@njit
def my_test():
    struct_array = np.empty((2, 1), dtype=struct_dtype)
    struct_array[0] =(0, 1)
    return struct_array

result = my_test()
print(result)

This syntax is supported by Numpy, but unfortunately Numba throws an error. Allowing this syntax would make your zip and literal_unroll code even neater. I believe that this is a very common pattern when working with Numpy structured arrays. In my experience, more often than not, one would set rows, not columns.

@esc this is not time critical for me, I am not paying you any money, and I understand the constraints we all work within, so I appreciate your acknowledging the eventual priority of this even if it is an "eventuality" and not an "immediacy."
@marnixhoh, I 100% agree with you, and that's actually where I started my questioning on the Numba gitter channel, asking if I could populate a struct with a tuple of values (also, I asked if I could instantiate a new numpy struct array using a tuple). For posterity, my post and the subsequent discussion is here.
If you could even just do the latter (instantiate an array with a tuple), then you could do the former with just a little more effort than should be necessary (but too bad):
...
    struct_array[0] = np.array((0, 1), dtype=struct_dtype)
    ...

note that it already works to assign one struct from another with same dtype within a numba function; what doesn't work is the instantiation of a new struct as I show it above. (Yet; @luk-f-a has a PR where he's been trying to get this to work here: https://github.com/numba/numba/pull/5510; he says in the gitter discussion that it's on the back burner right now, but still on his radar.)
If he does get that to work, I think the above syntax I wrote should work straight away. And then maybe it's not too hard, from there, to implement the casting of a tuple of values into the appropriate struct type (or simply populate the fields of the target struct directly?).

@jllanfranchi setitem has been merged.
I also have something that can be used in place of zip. @stuartarchibald I took your example and expanded it. It looks useful, but I wonder if there's a place in Numba for it. examples maybe?
from numba import types
from numba.extending import intrinsic
from numba.core.cgutils import unpack_tuple
from numba.core.errors import TypingError

@intrinsic
def tuple_zip(tyctx, *tys):
    if len(tys) > 1:
        tys = types.StarArgTuple(tys)
    elif len(tys) == 1:
        raise ValueError("Only one argument received. Tuples to be zipped must be passed as individual arguments")
    if not all(isinstance(x, types.BaseTuple) for x in tys):
        raise TypingError("All inputs must be tuples")
    nitems = min((x.count for x in tys))
    tuples = [types.Tuple(inner_ty) for inner_ty in zip(*tys)]
    ret = types.Tuple(tuples)

    def codegen(cgctx, builder, sig, args):
        assert len(args) == 1  # it is a vararg tuple
        args_tup = unpack_tuple(builder, args[0])
        values = []
        for i in range(nitems):
            inner_vals = [builder.extract_value(x, i) for x in args_tup]
            inner_tup = cgctx.make_tuple(builder, tuples[i], inner_vals)
            values.append(inner_tup)
        return cgctx.make_tuple(builder, sig.return_type, values)

    sig = ret(tys)
    return sig, codegen

And here is a tuple_enumerate based on the tuple_zip above:
@intrinsic
def tuple_enumerate(tyctx, ty):
    nitems = ty.count
    tuples = [types.Tuple([types.int32, inner_ty]) for inner_ty in ty]
    ret = types.Tuple(tuples)
    from numba.core.cgutils import unpack_tuple
    def codegen(cgctx, builder, sig, args):
        args, = args
        values = []
        for i in range(nitems):
            inner_vals = [
                cgctx.get_constant(types.int32, i),
                builder.extract_value(args, i)
            ]
            inner_tup = cgctx.make_tuple(builder, tuples[i], inner_vals)
            values.append(inner_tup)
        return cgctx.make_tuple(builder, sig.return_type, values)
    sig = ret(ty)
    return sig, codegen

@nb.njit()
def test(xs):
    r = 0
    for p in literal_unroll(tuple_enumerate_intr(xs)):
        i, x = p
        r += i + x

    return r

test((1, 2.)) # 4.0

Unfortunately it seems that both tuple_zip and my tuple_enumerate segfault if there is a numpy array in the arguments. For example:
@nb.njit
def f(xs, ys):
    for p in literal_unroll(tuple_zip(xs, ys)):
        c, codes = p
        return c

f(
    (1, 2),
    (np.zeros(2), np.zeros(2))
)

Or:
@nb.njit
def f(codess):
    for p in literal_unroll(tuple_enumerate(codess)):
        c, codes = p
        return c

f(
    (np.zeros(2),)
)

Both fail, but replacing np.zeros(2) with 0 works fine.


docs: Add tools_and_agents.ipynb

@willbakst
I introduced an example of basic chatbot.
Could you please check if this is the deployment you are expecting.


TSServer doesn't send configFileDiag event after tsconfig.json file changes (saved to disk)

TypeScript Version:  2.0.6
Code
tsconfig.json
{
    "compilerOptions": {
        "module": "commonjs",
        "target": "es5",
        "aaaca": true
    }
}

open a TS file covered by above tsconfig.json

Observe: you get an event
[Trace - 2:35:24 PM] Event received: configFileDiag (0).
Data: {
    "configFile": "c:/Users/dirkb/Projects/mseng/VSCode/Playgrounds/ts-2.0/tsconfig.json",
    "diagnostics": [
        {
            "text": "Unknown compiler option 'aaaca'."
        }
    ]
}

make the tsconfig.json file correct (e.g. remove the "aaaca" property.

Observe: no configFileDiag event is sent. No event is send either if you do changes that make the tsconfig.json file still incorrect.

@mhegazy @zhengbli could you please investigate this for VS Code 1.7. I disabled the tsconfig file error checking for now.

@mjbvz same thing: https://github.com/Microsoft/vscode/blob/master/extensions/typescript/src/typescriptMain.ts#L374


[Doc][SQL] add explanations for how to get last message in topics

Fix https://github.com/apache/pulsar/pull/8635
Preview looks good:

@zymap @MarvinCai @momo-jun could you please help review? Thanks


Encounter some problems when simulating wake steering control using FAST.Farm

Hi everyone,
Recently I'm learning to use FAST.Farm to simulate the wake steering control of wind farm. What I do firstly I use FLORISSE_M to solve a set of optimal yaw angle to maximze the total power generation of WF, and secondly I set static yaw angle values for each wind turbine in FAST.Farm to test the control performance. However, the result is not ideal.
There are 3 WTs in my case, and arrange in a row along the wind direction and the distance is 5D. The results of FLORISSE_M are as follows:

However when I simulate it in FAST.Farm, the total power generation of WF is less than the case without yaw angle misalignment. The total power and power of each WT under the two methods are shown in the following figures:

The first figure is the total power and the rest 3 figures are power of WT1~3. The blue line represents the control without yaw. We can see that after the yaw control, the total power actually decreased.
At the same time, I would like to ask a question. Each of my FAST file uses the default DISCON controller, and I  modify the parameters related to yaw in DISCON.IN and set YCMode=5 in ServoDyn, but the yaw of the wind turbine was not activated. I set the initial value of NacYaw in ElastoDyn based on the answers to other questions, and set YawNeut=NacYaw and YawManRat=0.2 in ServoDyn, and in this way, yaw misalignment has been achieved both in OpenFAST and FAST.Farm.
The ideal results would be to increase the power of the entire WF through the yaw of the upstream WT, but this cannot be achieved in my case. Even if I set the yaw angle of WT1 to 10 ° or 5 ° and set the other WTs to 0 °, the total power still cannot be improved. I really want to know if there is anything wrong with my settings? Or is it due to inaccurate optimization results of yaw angle?
Yours Sincerly,
Venus

Dear @VenusJinxin,
Can you clarify:

What wind turbine are you modeling, e.g., the NREL 5-MW baseline or the IEA Wind 15-MW RWT?
What DISCON controller are you using?
Which wake formulation (Mod_Wake) are you using in FAST.Farm?  FYI: The curled wake model (Mod_Wake = 3) is more accurate for wake-steering applications.
What inflow conditions are you using in FAST.Farm?
Are you using DEFAULT wake-dynamics parameters or did you calibrate them in some way?

Regarding yaw control, YCMode = 5 in ServoDyn is only need if you want active yaw control.  You can use YCMode = 0 with YawNeut = NacYaw in ServoDyn if you want fixed yaw when YawDOF is enabled in ElastoDyn.
Best regards,

Dear @jjonkman ,
Thanks for your detailed advice. I'm not quite sure why I made the modification to C_NearWake, maybe it was due to my mistake.
Yesterday I downloaded a new r-test document and made modifications according to my previous simulation. I also conducted simulations under steady wind conditions. I found that when the wind speed is constant, the power increase caused by yaw is very significant; However, under turbulent wind conditions, it is not satisfactory. This may be because when the wind speed fluctuates too much, and the optimal yaw angle at different wind speeds is also inconsistent.
Perhaps my problem comes from an inappropriate control scheme used, not FAST Farm :)
Best Regards,
Venus


Profile a callsite after the call returns

In the interpreter, we profile a call site first and then call the callee. It may happen that the callee was defer-parsed and the its bytecode generation failed due to some reason (OOM?) upon undeferral. Since, the call won't execute any code in this case, we should not record call site information - during work item creation, we would erroneously think that this a potential inlining candidate. Fix is to record the call site info after the call returns

This change is

Submitting a different fix for the issue. This one will impact recursive inlining


chore(proto): remove unused messages

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
as title

@mergifyio requeue


Change Forge version

Is there a way to change the Forge version.  Because when i use -e TYPE=FORGE it downlaods the wrong Forge version (<IP_ADDRESS>68).  I want to install the version <IP_ADDRESS>46 .

Thanks @wlp7s0 for answering that.


fix: update icon for svelte

Use the proper icon for Svelte. Fixes #91

I recently synced my plugins using PackerSync, and the Svelte icon is no longer showing properly. I'm new to neovim, so did I miss a step? Here's what my svelte icons look like currently.

I recently synced my plugins using PackerSync, and the Svelte icon is no longer showing properly. I'm new to neovim, so did I miss a step? Here's what my svelte icons look like currently.

I think your nerd font is missing some glyphs. Please try changing or updating your nerd font.

Thank you for the suggestion. I switched to Hack Nerd Font, and my svelte icons show now!


Is [holding out the last H observations of each time series as a test set] implemented in the fine-tuning code?

Hi Chronos team,
I notice in the paper that you used the last H observations of each time series as a held-out test set (all models are judged by the accuracy of their forecast on such held-out set, which no model had access to for training purposes).
However, I haven't found the implementation of this part in file train.py. Could you please guide me on which part of the code implements this?

@Xiao-congxi this is indeed nowhere in the code. This happened prior to applying TSMixup (see Algorithm 1 in Appendix A https://arxiv.org/pdf/2403.07815): the last H observations from each series of each dataset (note that H is dataset dependent) were sliced out before calling the TSMixup routine. The mixed data was stored (you find it at https://huggingface.co/datasets/autogluon/chronos_datasets/tree/main/training_corpus) and used directly with train.py, which at that point requires no further slicing of the data, for training the model.

Thank you for your prompt response! This approach is indeed convenient. With that in mind, I have another question. In your paper, when conducting in-domain evaluation on a specific dataset in benchmark1 (for example, electricity_hourly), since it has already been mixed into the pre-training data, does that mean there’s no need for additional fine-tuning, but directly making the prediction?

For the in-domain evaluation (datasets in Benchmark I) we did not do any fine-tuning, but directly predicted as you say.
One could do additional fine-tuning, and potentially see improvements, but we did not run that experiments. We only tested fine-tuning on Benchmark II datasets (see Figure 6).

I understand. Thanks again for your help!


[UX][Admin] Allow users to select permissions in subscription policies

Description:
Not fulfilling checklist items:
User control and freedom
Related task:
Add subscription-level/application-level throttling policies
Issue:
When adding a subscription policy the user needs to type the full name of the policy. And user is not given details to add multiple roles.
The permission radio buttons need to be before the input field.
"This tier is Allowed for above roles" help tag "Allowed", "Deny" needs to be changed according to the selected permission.
Proposed Solution:
Add select 2 multi value select boxes https://select2.org/tagging#tagging-with-multi-value-select-boxes
Suggested Labels:
UX, Improvement
Affected Product Version:
2.2.0

Closing the issue as this is older than 3 years. Please reopen if the issue is valid still.


[BUG] Relayer Fails to Reset When Users Switch to a Different Bridge

Currently, in the withdrawal/transfer process, the Hubble bridge automatically selects a relayer based on the initial state. However, when users switch to a different bridge (e.g., opt for a different chain or pool), the relayer neither resets nor updates. This results in an issue when users select a relayer that is not supported by the chosen bridge.
Expected Behavior
Whenever users switch to a different bridge, the relayer should reset, or alternatively, select a relayer that is supported by the new bridge.
Current Behavior
The relayer neither resets nor reselects when a different bridge is chosen; it merely remains in the state of the previously selected bridge.
Possible Solution
Implement a watch function to monitor changes in the chain or pool, and accordingly, reset or reselect the supported relayer.

Closed in https://github.com/webb-tools/webb-dapp/pull/1610.


Bosh Deployment Status Spec reconciler

Motivation and Context
Implements the Bosh Status Spec reconciler
#171854899
Checklist:
Check item if necessary.

[ ] Review PRs of changed Quarks dependencies
[ ] Update this PR after the release of Quarks dependencies

Very cool @mudler .
@nwmac maybe there's a cool integration with Stratos to show this information?


Trailing commas for match cases

Feature request
Right now TrailingCommaInMultilineFixer can fix missing trailing commas in arrays, arguments and parameters. It should be appliable to match cases in PHP >= 8.0.
For example the following code:
$message = match ($statusCode) {
    200, 300 => null,
    400 => 'not found',
    500 => 'server error',
    default => 'unknown status code'
};

should be fixed into:
$message = match ($statusCode) {
    200, 300 => null,
    400 => 'not found',
    500 => 'server error',
    default => 'unknown status code',
};

Thanks for this suggestion,. if you can please try out https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/6376 and let me know what you think :+1:


Infra server release for Amazon Linux 2023

Description
[Please describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/main/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

Adhoc is green: https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/6115#_


Basic example code - improve the code dealing with tasks

Ref #256, the example code covering tasks is a bit hidden and may be improved.  See https://github.com/python-caldav/caldav/issues/256#issuecomment-1416494723 for details

Hi,
I would like to work on this issue.
Just wanted to check my understanding of what has to be done -
Examples have to be added in the basic_usage_examples for new argument in complete method, uncomplete method, set_duration, get_due and set_due method.
Is that correct?

Yes.  There is already a little bit of code at the bottom of search_calendar_demo, it should be moved out to a separate function (tasks_demo or something like that).
As for set_duration, get_due and set_due, I'm a bit hesitant if it should be in the examples, I'm not much happy with the API so it may change in version 2.0.  At least it's needed with a warning.  (the logic should be moved to the icalendar library, ref https://github.com/collective/icalendar/issues/496 - and the logic should also apply to events, not only tasks - but since events have DTEND instead of DUE, the current code cannot simply be moved up to the base class)

Okay I will work on this issue and then create a PR


model and training code for the AR variant

Thanks for open-sourcing this amazing project!
I wonder it is possible to also release model and training code for the AR baseline

Thank you in advance!

To keep this repo clean, we don't have a plan to release the AR code in this repo. However, it is very easy to reimplement it using the current repo -- almost all hyper-parameters remain the same as MAR. The only difference is the causal attention mask and the teacher-forcing loss.

"Hi @LTH14, in the AR variant, is it necessary for the attention mechanism within the MAE encoder to be causal? Alternatively, should we consider removing the MAE encoder altogether in this variant?"

In the AR variant, we don't need the MAE encoder. A single decoder is enough (similar to GPT).

Thanks! Do you double the depth of MAE decoder?

Yes we keep the total number of parameters unchanged


Better auto-tuning strategy for multi-machine training performance?

Description
The performance of multi-machine training is greatly affected by this MXNET_CPU_WORKER_NTHREADS variable. It seems that MXNet will auto-tune this variable, and it can lead to very different performances even if the user does not do anything else. Maybe the auto-tuning strategy can be improved so that it won't deliver bad performance just because of something that users have no control of?
Package used (Python/R/Scala/Julia): python
Environment info (Required)
centos 6.9
cuda 9.0
cudnn 7.0
network card: 100Gb/s infiniband
GPU: Quadro P4000
CPU: 28-core Xeon E5-2860
MXNet commit hash: fb50257aeb3281d7fc90abc38162edbbbe4a5cb2
(Paste the output of git rev-parse HEAD here.)
Minimum reproducible example
example/image-classification
Steps to reproduce
1, launch the distributed training of ResNet without specifying MXNET_CPU_WORKER_NTHREADS using the following script:
export DMLC_INTERFACE="ib1"

python3 ../../tools/launch.py \
        -n 2 --launcher ssh -H hosts \
        '(export DMLC_INTERFACE="ib1"; python3 train_imagenet.py --data-train /scratch/imagenet1k_no_val_resize240.rec --gpus 0 --batch-size 32 --kv-store dist_device_sync)'

The per-machine training throughput is sometimes 4 samples/s, sometimes 62 samples/s.
2, launch the distributed training of ResNet with MXNET_CPU_WORKER_NTHREADS=25 using the following script:
export DMLC_INTERFACE="ib1"
MXNET_CPU_WORKER_NTHREADS="25"
export MXNET_CPU_WORKER_NTHREADS

python3 ../../tools/launch.py \
        -n 2 --launcher ssh -H hosts \
        '(export DMLC_INTERFACE="ib1"; export MXNET_CPU_WORKER_NTHREADS="25"; python3 train_imagenet.py --data-train /scratch/imagenet1k_no_val_resize240.rec --gpus 0 --batch-size 32 --kv-store dist_device_sync)'

The per-machine training throughput is sometimes 62 samples/s, sometimes 98 samples/s.
What have you tried?
1, We found that when per-machine throughput is 4 samples/s, the CPU usage of the worker process (If I remember correctly) is exactly 100%, and one CPU core is fully busy, others are idle. When the throughput is 62, the CPU usage is around 500%.
2, With MXNET_CPU_WORKER_NTHREADS initialized with a large number, we have not observed any case which throughput is 4 again.
3, We couldn't find the reason why the performance is sometimes 98 samples/s. Do you have a clue to this?

@sandeep-krishnamurthy  could you help to add label Distributed, Performance? Thanks!


Escape >, <, and & in PR title

This is escaping necessity is explained here:
https://api.slack.com/reference/surfaces/formatting#escaping
I tested it under the 'mrkdwn' section here:
https://app.slack.com/block-kit-builder/T025FUEFN
Closes https://github.com/iFixit/PullBot/issues/10

A little easier to read the change with https://github.com/iFixit/PullBot/pull/11/files?diff=unified

QA
Pullbot runs out of cominor at /opt/pullbot/.
You can

clone this repo, and copy the config over from /opt/pullbot/config.json into your clone.
from your clone root run the exec/build.sh script
from your clone root run the exec/restart-with-latest.sh script

Check messages in the Slack channel #pull-requests
Suggestion:

open a test PR with the titles from the issue.

CR 🌽

dev_block ☁
It seems that < and > break the hyperlink formatting in the Slack message

With single >

With single <

With single &

Steps Taken:

In my Cominor home directory I cloned this repo, and copied the config over from /opt/pullbot/config.json into the cloned repo
Then ran the exec/build.sh script in the cloned repo
Then ran the exec/restart-with-latest.sh script
I then checked out the test-branch already in this repo
I created a new file and followed the process to pushing the change
I then created a new pull request with the title containing the escape characters https://github.com/iFixit/PullBot/pull/12
When I created the Pull Request I confirmed that the PR title showed up in the #pull-requests slack channel
I noticed that the hyperlink only covered the first character in the title so tested the escaping characters independently and found that < and > break the hyperlink syntax

I did this by editing the tile before closing or reopening the same pull request

I also tested that a regular title still works without issues 👍

QA 👍

[x] Titles with < , > , and/or & characters no longer break the slack syntax

Steps Taken:

In my Cominor home directory I cloned this repo, and copied the config over from /opt/pullbot/config.json into the cloned repo

I then checked out this branch in the repo

Then ran the exec/build.sh script

Then ran the exec/restart-with-latest.sh script

I then checked out the test-branch already in this repo

I created a new file and followed the process to pushing the change

I then created a new pull request with the title containing the escape characters https://github.com/iFixit/PullBot/pull/12

When I created the Pull Request I confirmed that the PR title showed up in the #pull-requests slack channel

I confirmed each character was escaping properly

I did this by editing the title before closing or reopening the same pull request

I also tested that a regular title still works without issues 👍


Object.prototype.hasOwnProperty takes undefined when no arg passed in

function o() {return this} // this `this` contains all global objects
o().hasOwnProperty() // false, should true as other browsers return
o().hasOwnProperty(undefined) // true

close #6477

Unfortunately, the solution for those intermittent failures has been "turn it on and off again". I will try to close the PR and reopen right away, otherwise we will have to contact somebody in MSFT who has the rights to restart CI.

Kicking tires on CI, take two

Ping @pleath @rhuanjl
This seems to me like a straight-forward PR. We've ran CI on the change three times - got Win 7 (disk full) failures, but also alternating between Win 8 and OSX failures on last two runs. Do either of you have any thoughts on how to clear this?

<EMAIL_ADDRESS><EMAIL_ADDRESS>do you know who the right person to address the Win7 disk failure is?
From: Petr Penzin<EMAIL_ADDRESS>Sent: Thursday, September 3, 2020 10:35 PM
To: microsoft/ChakraCore<EMAIL_ADDRESS>Cc: Paul Leathers<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/ChakraCore] Object.prototype.hasOwnProperty takes undefined when no arg passed in (#6497)
Ping @pleathhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpleath&data=02|01|pleath%40microsoft.com|c730d4f724c74262035a08d850943dfc|72f988bf86f141af91ab2d7cd011db47|0|0|637347944914180772&sdata=HX06elc1IkYzC4jRUJz6lJ1AW1PY1M1NNDvvvoLP42U%3D&reserved=0 @rhuanjlhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frhuanjl&data=02|01|pleath%40microsoft.com|c730d4f724c74262035a08d850943dfc|72f988bf86f141af91ab2d7cd011db47|0|0|637347944914190764&sdata=MnoB443%2Bi1MASiQMHxplbeemayLu7oBkohrx5sW%2Bd1U%3D&reserved=0
This seems to me like a straight-forward PR. We've ran CI on the change three times - got Win 7 (disk full) failures, but also alternating between Win 8 and OSX failures on last two runs. Do either of you have any thoughts on how to clear this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FChakraCore%2Fpull%2F6497%23issuecomment-686919114&data=02|01|pleath%40microsoft.com|c730d4f724c74262035a08d850943dfc|72f988bf86f141af91ab2d7cd011db47|0|0|637347944914190764&sdata=7aLBjP%2BupKWl93ZANn0EvzxbR5kljSEk8U8LO6%2FkxiU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADYXYRHMJZB2Q5XC2P65EULSEB37TANCNFSM4QOX2EFA&data=02|01|pleath%40microsoft.com|c730d4f724c74262035a08d850943dfc|72f988bf86f141af91ab2d7cd011db47|0|0|637347944914200757&sdata=PtZFlxBr5mG4PGv%2BVZWORF0PZvIYTaJB0fEuul4K8A0%3D&reserved=0.

I'll see if I can get it fixed.
From: Paul Leathers<EMAIL_ADDRESS>Sent: Friday, September 4, 2020 10:32 AM
To: microsoft/ChakraCore<EMAIL_ADDRESS>microsoft/ChakraCore<EMAIL_ADDRESS>Taylor Woll<EMAIL_ADDRESS>Atul Katti<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Subject: RE: [microsoft/ChakraCore] Object.prototype.hasOwnProperty takes undefined when no arg passed in (#6497)
<EMAIL_ADDRESS><EMAIL_ADDRESS>do you know who the right person to address the Win7 disk failure is?
From: Petr Penzin <notifications<EMAIL_ADDRESS>Sent: Thursday, September 3, 2020 10:35 PM
To: microsoft/ChakraCore <ChakraCore<EMAIL_ADDRESS>Cc: Paul Leathers <pleath<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [microsoft/ChakraCore] Object.prototype.hasOwnProperty takes undefined when no arg passed in (#6497)
Ping @pleathhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpleath&data=04|01|Taylor.Woll%40microsoft.com|ee7eec910b2a4580968908d850f87b04|72f988bf86f141af91ab2d7cd011db47|0|0|637348375438283589|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Czzdk%2FxTm%2Fs9l0vWvyU3rAbWMCK45JvyXKu0YCSEPUI%3D&reserved=0 @rhuanjlhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frhuanjl&data=04|01|Taylor.Woll%40microsoft.com|ee7eec910b2a4580968908d850f87b04|72f988bf86f141af91ab2d7cd011db47|0|0|637348375438293592|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=UR22rqLjNThOteEJbQmQFFlGevr0kAqq2JmEPbbPd48%3D&reserved=0
This seems to me like a straight-forward PR. We've ran CI on the change three times - got Win 7 (disk full) failures, but also alternating between Win 8 and OSX failures on last two runs. Do either of you have any thoughts on how to clear this?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FChakraCore%2Fpull%2F6497%23issuecomment-686919114&data=04|01|Taylor.Woll%40microsoft.com|ee7eec910b2a4580968908d850f87b04|72f988bf86f141af91ab2d7cd011db47|0|0|637348375438303578|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=MjaN8dCeaF0aiEgELu%2FkWsoGRTEJeiGzZXSG5UsFsB8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADYXYRHMJZB2Q5XC2P65EULSEB37TANCNFSM4QOX2EFA&data=04|01|Taylor.Woll%40microsoft.com|ee7eec910b2a4580968908d850f87b04|72f988bf86f141af91ab2d7cd011db47|0|0|637348375438303578|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=iAqfv3kEd%2F3%2BwHklnHFgmZDZbUmih%2FLUhS5ozWKOxGQ%3D&reserved=0.

Thank you @boingoing!

@unbyte thank you for your contribution!

@unbyte thank you for your contribution!

thank you, too 😀


WIP: Minimise GCP permissions needed by the operator

Can we get this merged?

👍
I can't remember why this is WIP. Probably it was never tested. Unless you are ready to test, I will get this bumped on my todo.


m64p - No ROMs Load

After following the N64 guide, I cannot get any game to load, it just boots to the emulator. Looks like others on the Reddit thread are having the same issue.

Hmm, could you try manually running flatpak run io.github.m64p.m64p <path-to-rom> (replacing <path-to-rom> with your game's path) in a terminal? It works for me, so I'm not sure why it isn't for some people.

What's weird about this bug is that I can load ROMs using the CLI on my Linux PC but not on Deck. I'm still investigating.

The latest version of the m64p flatpak seemed to introduce this issue. The bug is being tracked at https://github.com/flathub/io.github.m64p.m64p/issues/55.
In the meantime, you can run this in a terminal to downgrade m64p to a working version:
sudo flatpak update --commit=534c9e389049cb1ef8bd4206fbb5042c4d18be5363ae15e2ab979fd10b78f6c8 io.github.m64p.m64p

The latest version of the m64p flatpak seemed to introduce this issue. The bug is being tracked at flathub/io.github.m64p.m64p#55.
In the meantime, you can run this in a terminal to downgrade m64p to a working version:
sudo flatpak update --commit=534c9e389049cb1ef8bd4206fbb5042c4d18be5363ae15e2ab979fd10b78f6c8 io.github.m64p.m64p

Thanks for your help! Hopefully future updates address for flatpack.

A more permanent fix, which should work with the latest version, is giving the Flatpak access to your ROMs folder.
flatpak override --filesystem=<roms directory>:ro io.github.m64p.m64p

I'll add this to the guide soon.

The m64p Flatpak has been fixed - go ahead and update it from Discover.

Still not fixed in latest version from Discover, build date Mar 15 2022. m64p still needs --filesystem=host:ro to load any ROM via CLI.


Feature request: Allow adding extra permission types to per-object permissions table

Currently the "per object" permissions include "change", "delete" and "add" but "view" is also a default permission in django.contrib.auth. Is it possible to add this as well? I think it's nothing more than adding an entry to this array: https://github.com/wagtail/wagtail/blob/fdd8ddc2b183421eeb61231e8a724cdb34aa490c/wagtail/users/templatetags/wagtailusers_tags.py#L60
Wagtail version: 2.6

Adding the checkboxes to the 'edit group permissions' form would be easy enough, but that might be misleading to users, as those permissions aren't enforced anywhere in Wagtail, and we'd have to decide what they mean in the context of Wagtail, especially since Wagtail has a separate concept of 'view restrictions' on pages and documents.
One requested feature that might align well with this is a 'choose' permission (#4288, #4488) to determine which items appear in the 'choose page' / 'choose image' etc popups.
We also need to keep in mind that view permissions were added to Django in 2.1 - we need to ensure that things continue to work correctly in Django 2.0, or wait for a future release where we drop Django 2.0 support.

Hmm, maybe I can open a PR which simply allows adding more permissions next to "change" & "delete"? Note that I created this issue specifically for permissions of user-defined django models (and not pages/images).

@dmeijboom Sure, that sounds like a worthwhile addition.

Resolved via #7488


change default port of nativerpc server

Currently, if you would try to start new heroic cluster according to documentation, then cluster will not listen for nativerpc port and no error would be generated about reasons. The problem is that by default it bounds to port number 0. So, let's make it compliant with documentation. Otherwise, changes needed in documentation.

This looks reasonable. We just need to assert that the integration tests survive this.

Integration tests have stopped depending on this behavior. I'll merge this as soon as the tests pass.
Thanks!


minor changes to index optimus projects

Can you update the transformers tests to get the emptydrops result too?


Install as package

Is it possible to install it as a package?

Nope, still not possible. (https://github.com/pimusicbox/pimusicbox/issues/16)

it would be great...
closing this issue


Changing Version in Playground Leaves it Hanging

@noalak  Ref https://forum.babylonjs.com/t/playground-breaks-when-changing-babylon-version/5241

I'm on this one. We moved the compileAndRun function but didn't change the references to it.
On my way to the PR


Is this tool still compatible?

I noticed it's been a few years since the last release. Not sure if I'm doing something wrong in the installation. Using Cargo I get an error with funty
failed to select a version for the requirement "funty" = 1.2

Seems like i didn't check in a lockfile and therefore the yanked funty release caused issues.
I've updated the dependencies, can you see whether this works for you?

It installs fine now. Glad it's updated. Have a few questions about using the app though. I can move my questions to the closed issue that you asked people to direct questions too if you want. Mainly just about getting results from it. I can run reverse on some files i have but I don't really see any information output

Sure, just ask any questions here. If there is no output, it means that there are no solutions given the input parameters.

Ah that's good to know. I'll have to keep trying to refine the criteria. I'm mainly trying to use this to overcome checks on PS1 game saves. Most titles I've seen have anti-corruption checks within the saves themselves, and not every title uses the same methods. Which is a bit of a barrier for anyone who wants to tinker with them.


New strings don't show up after deployment

After new features were deployed this week, none of the new strings appeared in the UI. After the CDN was purged and folks cleared their browser cache they did appear. Let's use this issue to track a good solution to this that doesn't require action on the part of users.
See screenshot of the print icon showing up without the word "Print".

and blanks for "start a service". This is because we changed the strings in these buttons in this release

Is this still a thing or do we have confidence it won't happen again?

This is still a problem, and it has to do with how the files that have the localized text get cached. The first part of the cache issue is on the CDN side, where the time to live setting dictates how long the CDN will take before reloading the latest version. Purging the CDN on every deployment would solve that cache, but then local caching on user devices will still serve cached content unless a hard refresh is done.
To fix this, we should attach a version to the generated text files and have the client always request the latest version. Since the version would uniquely identify a resource file, no caching issue would exist.
One possible approach to achieving this:

When generating the localized files, the script could use the last modified date of the json text files as version number
On startup, the system could determine the latest version of the generated localized files to use and set a config value (if not otherwise set)
When looking up a localized string, the configured version number would be used as part of the path to make sure the proper version gets used


Unable to parse operator ..<

Expected Behavior
Code with operator ..< should be parsed properly.
Observed Behavior
Throws a exception when running gradlew ktlintFormat:
Caused by: com.pinterest.ktlint.core.ParseException: 354:37 Expecting an element
	at com.pinterest.ktlint.core.KtLint.prepareCodeForLinting(KtLint.kt:200)
	at com.pinterest.ktlint.core.KtLint.lint(KtLint.kt:150)
	at com.pinterest.ktlint.core.KtLint.lint(KtLint.kt:138)
	at org.jlleitschuh.gradle.ktlint.worker.LegacyParamsInvocation.invokeLint(KtLintInvocation.kt:63)
	at org.jlleitschuh.gradle.ktlint.worker.KtLintWorkAction.execute(KtLintWorkAction.kt:107)

Line 354: if (behind in profile.rank..< 1000000) { Source
Steps to Reproduce
Write any line of code with Kotlin's new operator ..<, then run gradlew ktlintFormat.
Your Environment

Version of ktlint used: 0.48.2
Relevant parts of the .editorconfig settings

[*]
insert_final_newline = true
charset = utf-8
indent_style = space

[{*.kt,*.kts}]
disabled_rules = no-wildcard-imports, import-ordering

indent_size = 4
max_line_length = 120

[*.{bash,sh,zsh,fish}]
indent_size = 2

[*.yml]
indent_size = 2

[*.md]
indent_size = 2

Name and version (or code for custom task) of integration used (Gradle plugin, Maven plugin, command line, custom Gradle task):

ktlint-gradle 11.1.0
Kotlin 1.8.0

Version of Gradle used (if applicable): 7.6
Operating System and version: Windows 11 22H2 22623.1250

I can not reproduce the problem with Ktlint CLI 0.48.x with code sample below:
val foo = 42 in 1..<100

However, with sample above, the problem is reproducable with Ktlint CLI 0.47.x though. Are you sure that ktlint 0.48.x was invoked by the ktlint gradle-plugin?

I recently switch to use spotless and use rangeUntil directly (there is a related commit). I guess its my misconfigured cause this error. Anyway, I will close this issue now since can't reproduce, Thanks for help.

for(i in 0 ..< m) {
}
it still doesn't work version 0.50.0
i have to change to operator name (rangeUntil)

for(i in 0 ..< m) { } it still doesn't work version 0.50.0 i have to change to operator name (rangeUntil)

What do you mean? The code you wrote is invalid. The spaces around ..< should not be there. This is flagged by ktlint 0.50.0 as you can see below:
$ ktlint-0.50.0 --stdin
14:23:33.262 [main] INFO com.pinterest.ktlint.cli.internal.KtlintCommandLine - Enable default patterns [**/*.kt, **/*.kts]
fun fooaasdasdsa() {
    val m = 99
    for (i in 0 ..< m) {
        println(i)
    }
}
<stdin>:3:17: Unexpected spacing around "..<" (standard:range-spacing)

Summary error count (descending) by rule:
  standard:range-spacing: 1

for (i in 1 ..<m) {
      for (j in 1..n) {
        if (str1CodePoints[i - 1] == str2CodePoints[j - 1]) {
          distancePath[i][j] = distancePath[i - 1][j - 1]
        } else {
          distancePath[i][j] =
            minOf(distancePath[i - 1][j], distancePath[i][j - 1], distancePath[i - 1][j - 1]) + 1
        }
      }
    }
    r

Expecting an element , Unable to parse operator ..<
But , use rangeUntil directly it work
   for (i in 1.rangeUntil(m)) {
      for (j in 1.rangeUntil(n)) {
        if (str1CodePoints[i - 1] == str2CodePoints[j - 1]) {
          distancePath[i][j] = distancePath[i - 1][j - 1]
        } else {
          distancePath[i][j] =
            minOf(distancePath[i - 1][j], distancePath[i][j - 1], distancePath[i - 1][j - 1]) + 1
        }
      }
    }

Ktlint is not responsible for compiling your project to executable code. The compiler in ktlint is used only to parse the file. The error that you report is related to the compiler provided by your build tool and/or IDEA. That configuration is not related to the ktlint configuration.


  Object literal may only specify known properties, and 'styleActiveLine' does not exist in type 'EditorConfiguratio
n | undefined'.

if i remove the styleActiveLine or other addon its working ok
import {inject} from 'aurelia-framework';
import {DOM} from 'aurelia-pal'
import * as CodeMirror from 'codemirror';
import 'codemirror/mode/javascript/javascript';
import 'codemirror/lib/codemirror.css';
import 'codemirror/addon/selection/active-line';
@inject(DOM)
export class CodeConsole {
   DOM: any = DOM;

  attached(){
    let myTextArea : any;
     myTextArea = this.DOM.getElementById('demotext');

     CodeMirror.fromTextArea(myTextArea, {
      styleActiveLine: true,
      lineNumbers: true,
      lineWrapping: true,
      mode: "javascript"
    });
  }
}

This looks like an issue in the TypeScript definitions (or maybe a limitation of TypeScript, I'm not sure), in that it assumes no unknown properties are valid in EditorConfiguration, whereas addons can define their own options.
I don't maintain TypeScript definitions, so I'd suggest you take this up with whoever supplied the definitions you're using (probably https://github.com/DefinitelyTyped/DefinitelyTyped)

In case someone (like me a few minutes ago) ends up here with the same problem, here's a fix:
To make the addons you imported work you have to "extend" CodeMirror's type definitions. You can do this in the same file where you imported CodeMirror and its addons:
import * as CodeMirror from 'codemirror'
import 'codemirror/addon/selection/active-line'

// add styleActiveLine to CodeMirror.EditorConfiguration:
declare module "codemirror" {
    interface EditorConfiguration {
        //  when set to true, adds 'CodeMirror-activeline' and
        // 'CodeMirror-activeline-background' to the current line
        styleActiveLine?: boolean;
    }
}

// this doesn't cause compilation errors anymore
const cm = CodeMirror.fromTextArea(..., { styleActiveLine: true })

import * as CodeMirror from 'codemirror';

//I've got an error:
ERROR in D:/demo/mytools/src/app/codemirror/codemirror.component.ts (2,29): Cannot find module 'codemirror'.

But the file is existence.  The tsconfig:
{
  "compilerOptions": {
    "baseUrl": "",
    "declaration": false,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "lib": ["es6", "dom"],
    "mapRoot": "./",
    "module": "es6",
    "moduleResolution": "node",
    "outDir": "../dist/out-tsc",
    "sourceMap": true,
    "target": "es5",
    "typeRoots": [
      "../node_modules/@types"
    ]
  }
}

ah, I'm back~
I've go an idea to resolve the question of myself.
import { Component, OnInit } from '@angular/core';
declare var $: any;
declare var CodeMirror: any; // instead of   import * as CodeMirror from 'codemirror';

done~


Create a dataset: Date formatting mask required

Currently there is no way to specify a date formatting mask in the Create a dataset window. Please add one.

Are you asking for the ability to provide your own CSV files for test data sets with your own format, instead of using the Kettle tools for creating CSV files?
If so, what other features might be required?

Yes, we‘d usually prepare the CSV files externally. I was here actually only after a way to define the format (especially date). So all that would have to be added to the existing dialog is an extra column for specifying the format for each field.

You seem to allude to other options as well?

No not really. All I primarily would like to have is a format column in the table just below The data set fields and their column name in the table:.


registry secrets do not populate correctly for project owners

Setup

Rancher version: v2.6.4-rc2
Browser type & version: chrome

Describe the bug

as a project owner, I can create a registry secret, but dropdowns for secrets do not populate them. The same secret will populate for a cluster owner
To Reproduce

create a cluster, and give a user project-owner permissions in the default project
create a registry secret (secrets -> registry)
create a workload using the registry secret as an environment variable

Result
user is unable to create a workload using the registry secret as an environment variable
Additional context
as project owner:

as cluster owner:

same result for secret key and secret
related issue: https://github.com/rancher/dashboard/issues/3940

This should be addressed by this PR: https://github.com/rancher/dashboard/pull/5124
The issue is the user does not have permissions to read/write secrets - with the fix above, the user will not be able to edit the environment variable.

Ticket #5084
With Docker and EKS hosted cluster:
Reproduced on 2.6.4-rc2:

Fresh install of rancher v2.6.4-rc2
As admin, create a new standard user
As admin, create a cluster (EKS, or any other)
As admin, give standard user project-owner permissions in the default project
Log out admin, then log in as standard user
As standard user, create a registry secret
As standard user, create a workload using registry secret or secret key as an environment variable
Issue reproduced - registry secrets and secret keys are not available for project-owner, but are available for cluster-owner

As project-owner:

As cluster-owner:

Verified on v2.6-e4af2465c5b4d3ed4ad02694b5fd939b81f6b76a-head:

Fresh install of rancher 2.6-head
As admin, create a new standard user
As admin, create a cluster (EKS or any other)
As admin, give standard user project-owner permissions in the default project
Log out admin, then log in as standard user
As standard user, create a registry secret
As standard user, create a workload using registry secret or secret key as an environment variable
Resolution Verified - registry secrets and secret keys are available for both project-owner and cluster-owner

As project-owner:

As cluster-owner:

Upgrade/Verified on 2.6.4-rc2 >> 2.6.4-head:
W.I.P.  (estimated completion by 3/10/22)


Catalan translation

Collaborators welcome.
https://github.com/darmng/anxiety
I'm going to start working on the translation this winter. If anybody wants to collaborate please reply to this issue and @mention me and we'll get organized.
Thanks

Hi!
Count on me, @darmng!
I'm also on Twitter (@MAFuentesArjona). Feel free to get in touch!

Thanks @miquelangelfuentes ! I'll get in touch. I want to start the translation this December ^^

Yeah, that'd be a good time since I'll be on holiday break. Do not hesitate to get in touch with me!

Hi. I think it's pretty obvious, but I haven't been able to start the translation and I won't be doing it for the next few months.
I hope someone else can do it, because it's an interesting game to be accessible to as many people as possible.


Move Playground QuickLook support  => PlaygroundSupport

@swift-ci Please test and merge

@swift-ci Please test Linux platform

@swift-ci Please smoke test Linux platform

@swift-ci Please test and merge

Que pinches abre? Usted ya no open ny mother's jajaja
Sent from my BlackBerry® by Boost Mobile
-----Original Message-----
From: Dave Abrahams<EMAIL_ADDRESS>Date: Sun, 07 Aug 2016 21:10:19
To<EMAIL_ADDRESS>Reply-To: apple/swift<EMAIL_ADDRESS>Subject: Re: [apple/swift] Underscore playgroundquicklook (#4089)
@swift-ci Please test Linux platform

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apple/swift/pull/4089#issuecomment-238138072


Fix respond.js to not use asset host

Coverage remained the same at 100.0% when pulling 8bdb8d13bc1514a8122d259adc8af6f51d769f4d on respond_js_fix into 98ff75187b4faa2839bae4dbe42d79f4cc773045 on master.


Illegal file name problem

Hi.
I tried to make bob-the-rapper skin for bfdance skin by TQualizer, but when I'm trying to save the spritesheet, the "Illegal file name" pops up. Please, fix this.
Sincerely, KubaPL2004 Studio's Grzymkowski.

Hi! This pop up will show if your file name either contains a special character (such as €%&!/ etc.) or if your file name is empty (i.e you've entered nothing into the filename section). Try to only use letters and number when naming your file, and it should work.
However, I do realise that the error message is very generic and actually does not tell you what the problem is with the file name. I will try to clarify this message to better explain the problem on a case by case basis (i.e the error message will change depending on the problem). This will be included in the next version.

I realize that filenames are handled by the OS anyways, and thus checking the filename is redundant. I have decided to completely remove the option to give a filename in the app itself, as you can name it trough the download dialog box. Solved by 17409e26165bfc557ff7173ae3986f6370c6a951


convert interaction package to TypeScript

Description of change

So the previous attempt at this, #6474, seems to have stalled.
It had a few issues where it was rewriting parts of the code base so i have done a more straightforward conversion.
Didn't run into many issues so was a fairly easy conversion.
Pre-Merge Checklist

[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Codecov Report

Merging #6656 into dev will decrease coverage by 5.75%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #6656      +/-   ##
==========================================
- Coverage   82.59%   76.84%   -5.76%
==========================================
  Files          35       29       -6
  Lines        1724     1127     -597
==========================================
- Hits         1424      866     -558
+ Misses        300      261      -39

Continue to review full report at Codecov.

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


Fix for buoyancy + weight incompatibility

For some reason, in my experience setting the mass of an object in gmod after you have changed its buoyancy ratio resets its buoyancy to default until you physgun the object. Being stored as modifiers, buoyancy always comes before weight and thus for any boat or other floating craft, you'd have to physgun every prop with modified buoyancy every time you spawn it. It took many hours of trial and error and broken dupes to figure this out lol
As a dodgy fix, I simply copied over the special code for setting the mass prop and added buoyancy after like so (line 720 sv_clipboard.lua):
if (Ent.EntityMods["mass"] and duplicator.EntityModifiers["mass"]) then 		local ok, err = pcall(duplicator.EntityModifiers["mass"], Player, Ent, Ent.EntityMods["mass"]) 		if (not ok) then 			if (Player) then 				Player:ChatPrint('Error applying entity modifer, "mass". ERROR: ' .. err) 			else 				print('Error applying entity modifer, "' .. tostring(Type) .. '". ERROR: ' .. err) 			end 		end 	end 	if(Ent.EntityMods["buoyancy"] and duplicator.EntityModifiers["buoyancy"]) then 		local ok, err = pcall(duplicator.EntityModifiers["buoyancy"], Player, Ent, Ent.EntityMods["buoyancy"]) 		if (not ok) then 			if (Player) then 				Player:ChatPrint('Error applying entity modifer, "buoyancy". ERROR: ' .. err) 			else 				print('Error applying entity modifer, "' .. tostring(Type) .. '". ERROR: ' .. err) 			end 		end 	end
I am sure that there is a better way to do this, given that this is not a problem experienced with advdupe 1 or the default duplicator. For now, this solution works for me.

wow it did not format that well - maybe a quote?

if (Ent.EntityMods["mass"] and duplicator.EntityModifiers["mass"]) then
local ok, err = pcall(duplicator.EntityModifiers["mass"], Player, Ent, Ent.EntityMods["mass"])
if (not ok) then
if (Player) then
Player:ChatPrint('Error applying entity modifer, "mass". ERROR: ' .. err)
else
print('Error applying entity modifer, "' .. tostring(Type) .. '". ERROR: ' .. err)
end
end
end
if(Ent.EntityMods["buoyancy"] and duplicator.EntityModifiers["buoyancy"]) then
local ok, err = pcall(duplicator.EntityModifiers["buoyancy"], Player, Ent, Ent.EntityMods["buoyancy"])
if (not ok) then
if (Player) then
Player:ChatPrint('Error applying entity modifer, "buoyancy". ERROR: ' .. err)
else
print('Error applying entity modifer, "' .. tostring(Type) .. '". ERROR: ' .. err)
end
end
end

I think I can add this.

 I thought I traced this to being a garrybug back in 2009 or so.
If we can verify its a garrybug, let's get a bug report out.

well, its nice to have boyancy modifier support anyway, but yeah they really ought to fix that


image-layout: Drop manifest ref example description

A ref → descriptor example caption gained “blobs the manifest references” wording with 2f24791, #136, but I don't see a direct relationship between descriptors and those deeper-ancestor blobs.  I think we either want to revert the changes 2f24791 made to this line or drop the line.  In this commit I drop the line, because the other points covered by the sentence are already covered in other descriptor docs.  And it's pretty clear just from the example command and output that this is showing the content of a ref pointing at a manifest list.
Fixes #427.

lgtm

This example is completely bare of context if you drop this line. I don't think this is clear at all for new readers of this specification. The content needs to be reinforced.

On Tue, Dec 20, 2016 at 11:39:19AM -0800, Stephen Day wrote:

This example is completely bare of context…

There's a whole bunch of context in the “Refs” section which contains
this “Example Ref” subsection.

I don't think this is clear at all for new readers of this
specification. The content needs to be reinforced.

Which content needs reinforcing?  The line I remove says:
a. “This illustrates the expected contents of a given ref” (I think
this is covered by the “Example Ref” header and ‘cat ./refs/v1.0’
command).
b. “the manifest list it points to” (I think this is covered by the
‘mediaType": "application/vnd.oci.image.manifest.list.v1+json"’
output).
c. “and the blobs the manifest references” (I think this is wrong,
#427).
Are you pushing back on my reasoning for any of those?  For all of
them?

@wking This block of json just needs some human language introductory text. Just say what it does, reinforcing the language above.

On Tue, Dec 20, 2016 at 07:17:11PM -0800, Stephen Day wrote:

@wking This block of json just needs some human language
introductory text. Just say what it does, reinforcing the language
above.

shrug I added something closer to what we had originally with
909ecab → 1143e26.

Ehh, this language doesn't add anything, and the document remains inconsistent: "Example Blobs" has no intro text, and "Example Layout" just has the vague/unhelpful "This is an example image layout:". Can we just strike all of them?

On Tue, Dec 20, 2016 at 11:59:14PM -0800, Jonathan Boulle wrote:

Can we just strike all of them?

That's my preference :).  But I don't really care.  Let me know what
to do once you've sorted it out with @stevvooe ;).

@stevvooe, @wking, @jonboulle - can you get to consensus on this?

Looking back at this, I think this change is fine, but I'd like to see it followed up by PR that introduces the examples in a way that clarifies.
LGTM with follow up.

On Wed, Jan 18, 2017 at 03:48:46PM -0800, Stephen Day wrote:

Looking back at this, I think this change is fine…

I'm ok with all of the proposals that let us drop the current “blobs
the manifest references”, although I'd be happier if we could drop
1143e26a.  I don't really care though (which is why I pushed 1143e26a
when you asked for it ;).  As long as you and @jonboulle (who also
sounded like he wasn't particularly excited about 1143e26a 1) are
both happy with things as they stand, then lets get this thing landed.
If you need any further tweaks (short of restoring “blobs the manifest
references”), just tell me what to do.

… but I'd like to see it followed up by PR that introduces the
examples in a way that clarifies.

If other folks have different wording they want to add to clarify any
examples, that sounds fine to me.  I don't have any ideas myself for
those follow-up PRs though.

LGTM
I don't understand what @stevvooe wants in the followup so I will let him make the call on merging.

LGTM


Companydelete

Current coverage is 50.57%

Merging #131 into master will increase coverage by +0.78% as of 656ed0b

@@            master    #131   diff @@
======================================
  Files           16      17     +1
  Stmts          249     259    +10
  Branches         0       0
  Methods          0       0
======================================
+ Hit            124     131     +7
  Partial          0       0
- Missed         125     128     +3

Review entire Coverage Diff as of 656ed0b

Powered by Codecov. Updated on successful CI builds.


It takes a few seconds for the stop signal to be sent to the run stop

The stop time seems to be consistent with the connect timeout，However I don't know why it took so long to stop。
loop = config.asyncio.get_running_loop()
loop.add_signal_handler(
signal.SIGTERM, loop.create_task, websocket.close())
I just want a quick reboot

Unfortunately there isn't enough context in your question to help you. If I just copy paste what you wrote, I get an instant reboot.

import asyncio
import traceback
async def test():
import websockets
from json import dumps
import signal
uri = "wss://fstream.binance.com/stream"
while True:
try:
print(f"ws重启完成")
async with websockets.connect(uri, timeout=3, ping_interval=20, ping_timeout=5 * 60) as websocket:
asyncio.create_task(websocket.send(
dumps({
"method": "SUBSCRIBE",
"params": ["!bookTicker"],
"id": 12}
)))
            loop = asyncio.get_running_loop()
            loop.add_signal_handler(
                signal.SIGTERM, loop.create_task, websocket.close())

            async for message in websocket:
                print(message)
    except:
        msg = f"{traceback.format_exc()}"
        print(msg)

import asyncio
import traceback
async def test():
import websockets
from json import dumps
import signal
uri = "wss://fstream.binance.com/stream"
while True:
try:
print(f"ws重启完成")
async with websockets.connect(uri, timeout=3, ping_interval=20, ping_timeout=5 * 60) as websocket:
asyncio.create_task(websocket.send(
dumps({
"method": "SUBSCRIBE",
"params": ["!bookTicker"],
"id": 12}
)))
            loop = asyncio.get_running_loop()
            loop.add_signal_handler(
                signal.SIGTERM, loop.create_task, websocket.close())

            async for message in websocket:
                print(message)
    except:
        msg = f"{traceback.format_exc()}"
        print(msg)

import asyncio
import traceback

async def test():
    import websockets
    from json import dumps
    import signal
    uri = "wss://fstream.binance.com/stream"
    while True:
        try:
            print(f"ws重启完成")
            async with websockets.connect(uri, timeout=3, ping_interval=20, ping_timeout=5 * 60) as websocket:
                asyncio.create_task(websocket.send(
                    dumps({
                        "method": "SUBSCRIBE",
                        "params": ["!bookTicker"],
                        "id": 12}
                    )))

                loop = asyncio.get_running_loop()
                loop.add_signal_handler(
                    signal.SIGTERM, loop.create_task, websocket.close())

                async for message in websocket:
                    print(message)
        except:
            msg = f"{traceback.format_exc()}"
            print(msg)

The above code cannot be restarted immediately, it needs to wait for 3 seconds

@aaugustin

https://github.com/aaugustin/websockets/commit/9e960b508988c4049eb9f3377c505f506a3af060 might help.
If you need more, unfortunately, https://websockets.readthedocs.io/en/stable/project/contributing.html#cryptocurrency-users


Modified all strcpy to use parStrncpy wrapper and added unit test for parodus git commit hash

Codecov Report

Merging #120 into master will decrease coverage by 0.12%.
The diff coverage is 96.32%.

@@            Coverage Diff             @@
##           master     #120      +/-   ##
==========================================
- Coverage   88.27%   88.14%   -0.13%
==========================================
  Files          39       39
  Lines        3403     3417      +14
  Branches      261      263       +2
==========================================
+ Hits         3004     3012       +8
- Misses        153      158       +5
- Partials      246      247       +1

Impacted Files
Coverage Δ

tests/test_conn_interface.c
93.97% <100%> (ø)
:arrow_up:

tests/test_downstream.c
100% <100%> (ø)
:arrow_up:

src/client_list.c
78.35% <100%> (ø)
:arrow_up:

src/upstream.c
78.31% <100%> (ø)
:arrow_up:

src/partners_check.c
75.55% <100%> (ø)
:arrow_up:

tests/test_ParodusInternal.c
100% <100%> (ø)
:arrow_up:

tests/test_createConnection.c
100% <100%> (ø)
:arrow_up:

tests/test_partners_check.c
100% <100%> (ø)
:arrow_up:

tests/test_token.c
94.46% <100%> (ø)
:arrow_up:

tests/test_seshat_interface.c
93.15% <100%> (ø)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Restart server each time I make changes to HTML page

Love Middleman but each time I make a change to my html files, I need to quit and then rerun middleman server.
I get this issue when I run the server.
Unresolved specs during Gem::Specification.reset: minitest (~> 5.1)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.
I assume this shouldn't be happening. Thanks!

What kind of environment are you using? OS? Vagrant? Docker?
This definitely isn't happening for anyone else.
Why did you file this bug under middleman-blog?

I apologize. I guess I didn't realize where I submitted this. I found the page
when googling for the answer. I'm working on a mac. Not sure out how I can find
out that additional info.
On Sun, May 8, 2016 5:42 PM, Thomas Reynolds<EMAIL_ADDRESS>wrote:
What kind of environment are you using? OS? Vagrant? Docker?
This definitely isn't happening for anyone else.
Why did you file this bug under middleman-blog ?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
Drew Lepp User Experience/Product Designer
Portfolio: drewlepp.com Blog: drewlepp.com/blog
Social: @ drewlepp LinkedIn Instagram

Hi Drew, so, the stuff I mentioned are very complicated tools certain types of developers setup so a group of devs can work with exact same environment. It sounds like you probably setup this project yourself, so those are unlikely to be in use. Let me know if someone else setup your environment.
Can you post the Gemfile.lock file in your project folder?

Closing as stagnant


Stub new HTTP cache hostcalls (witx only)

Adds the hostcalls for the forthcoming HTTP cache API, stubbed as "not available".
This will enable releasing the Rust SDK using these features. For the sake of time, the Component bindings are left for follow up.

I've manually verified that code compiled against the new SDK with HTTP caching will run against this Viceroy.


first argument in apply_formatter_if now boolean, updated codebase

#382
What was wrong?
Order of arguments in apply_formatter_if needed to be flipped around to put the boolean first
How was it fixed?
grepped for instances and changed  'em
Cute Animal Picture

I have no explanation for why github required two conflict resolutions with exactly the same details before it recognized that the PR was mergeable. I pulled c8e958e local, and it merged into master just fine.

@wschwab Sorry about all the churn, the formatting was changing pretty rapidly, but should be settled now. Would you update the PR?
If you want a hand on any of these pieces, please @ mention me here or in gitter:

Pull the latest master
In your updated master: git diff 51b28e6bb4fe4d19760044be961cbaa75f2875cb to take note of any new usages of apply_formatter_if
Checkout your branch
Rebase your commit on master
Flip the args of any new usages from # 2.
Run flake8 web3/ tests/ and resolve any warnings/errors.
Force push the change here.

@carver I can't figure out what's going on at my machine. (I seem to have a little talent for creating git anomalies...) If it's okay with you, I've updated my master, and I ran through it all again. I don't have any experience with flake8, but I got it and ran it - 0 violations. I'll throw it into a new PR, if that's okay.