[Python] ImportError: DLL load failed while importing lib

Describe the bug, including details regarding any error messages, version, and platform.
Describe the bug
I am using the package sktime that uses pyarrow and I am getting the following error:
ImportError: DLL load failed while importing lib: The specified procedure could not be found.

Traceback
import pyarrow
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\Usuario\anaconda3\Lib\site-packages\pyarrow\__init__.py", line 65, in <module>
    import pyarrow.lib as _lib
ImportError: DLL load failed while importing lib: The specified procedure could not be found.

Version
pyarrow 13.0.0 installed using pypi channel
Platform
I am using the following OS:
Edition	Windows 11 Pro
Version	22H2
Installed on	2023-06-26
OS build	22621.3155
Experience	Windows Feature Experience Pack 1000.22684.1000.0

With python 3.11.8 and conda 24.1.2
Component(s)
Python

Hi @luifrancgom, thanks for the report. Can you provide a set of steps that reproduce the error, preferably using a fresh virtualenv or conda environment? I tried just now and am not having any trouble importing pyarrow 13.0.0.

You are right. I try the following configuration in yaml file called environment.yml:
#    conda env create -f environment.yml
#    conda env update --file environment.yml --prune
#    conda env list
#    conda activate test_pyarrow
#    conda env remove --name test_pyarrow

name: test_pyarrow
channels:
  - anaconda
  - conda-forge
  - defaults
dependencies:
  - python=3.11.8
  - pip
  - pip:
      - pyarrow==13.0.0
      - sktime==0.27.0

And when running this script
import sktime
import pyarrow

It works. I am going to close the issue and find the problem because it is not related to pyarrow or sktime

Hi @amoeba. I want to ask you something if you know something about it. The error I am having is happening in the base environment. I remove pyarrow and installed again using conda:
conda remove pyarrow
 conda install pyarrow

But I still have the same errow in relation to   ImportError: DLL load failed while importing lib: The specified procedure could not be found.
Do you know what file of files do I need to have in the DLLs folder in the case of anaconda so pyarrow can be imported?

Hi @luifrancgom, I'm not very sure but my best guess at the moment is that your shell environment isn't set up correctly for your base environment and that creating a new conda environment and activating sets things up correctly. If that's true, updating your PATH environment variable may fix things. Can you try out what's described in https://superuser.com/questions/1406191/getting-the-error-dll-load-failed-the-specified-module-could-not-be-found-wh?

Thank you for your help @amoeba. I follow the solution described in the link and it works. I really appreciate your help. Again thank you!!!


Not able to change the caption for SubPart pages

Hello,
I tried to change the field caption "No." in the "Sales order subform" page but it is not working, and i tried to do it in the other subforms like sales quote subform, purchase quote subform, i observe that for cardpart and listpart pages it is not working.
Thanks,
SainathP.

Please follow the template and provide adequate information to reproduce the issue:

AL extension version
BC version
AL Code reproducing the issue
Eventually, some screenshots showing the problem you mention


faucet pattern library

Can we simplify how devs create a faucet?
import * as std from "capi/std";

// Some web server endpoint
export async function main({ address }: Payload) {
  await std.faucetDeposit({
    address,
    signer,
    amount: 1000,
    every: 1000 * 60 * 60 * 24,
    lastDeposit(address) {
      return someLookup.get(address)
    },
    async setLastDeposit(address) {
      await someLookup.set(address, Date.now());
    }
    fundsLowThreshold: 1000000,
    async onFundsLow() {
      await someNotifService.notify("Faucet funds running low");
    }
  }).run(config);
}

Under the hood, this can handle checking the validity of the request, ensuring the faucet has enough funds, executing a low-balance callback (for instance, notifying the maintainer), misc.

Notes:

https://github.com/paritytech/substrate-matrix-faucet/blob/main/src/server/services/Actions.ts

Upon revisiting, this strikes me as more of an example than a pattern library. Closing in favor of #798


Add DUP Instruction

Add the DUP intrinsic.

I'm not sure why this would be useful given the SSA IR has no notion of a evaluation stack as raw IL code does, what is the use case here?

CIL completeness

I can't think of any way adding this as an intrinsic would make sense. We could add a few peepholes in ILAssembler to try and replace duplicate loads as a code size optimization, but I personally don't think that is worth the effort.


[WEB] - Navigator 2.0 - URL #fragment is removed when page is reloaded

In the web app with Navigator 2.0 and PathUrlStrategy the #fragment part of the URL/URI is removed when the page is reloaded.
URL Fragments should be preserved as they can be used to navigate inside the page, or invoke an action (show Alert dialog, etc).
Query params are preserved as expected.

@stenlee
Please provide flutter doctor -v and a complete minimal reproducible code sample along with actual vs expected behavior.
Thanks.


Always getting status 405

Since today morning, i have only been getting this error. Even the simplest inserts fail. However drizzle migrations are working for some reason, so the credentials are correct. Yesterday my setup was working, so i believe this error isn't on my end

@penberg @honzasp @giovannibenussi

fixed in https://ptb.discord.com/channels/933071162680958986/1249964350437851156

Hey, we've seen some issues on different environments, for example, when you set the database URL and token using something like import.meta or similar.
I'm glad that you were able to solve this issue! This solution helps us to understand this issue better, thanks for sharing your it!

This is an issue with bun's fetch override. It is fixed by either not using bun --bun run dev (just bun run dev, which respects the shebangs in scripts and executes with node most of the time). Thanks for the feedback! Glad i could help!

I found an even better solution without the need for additional libraries
`import { DB_URL } from '$env/static/private';
import { drizzle } from 'drizzle-orm/libsql';
import { createClient } from '@libsql/client';
const client = createClient({ url: DB_URL, fetch: async (req: Request) => {
return fetch(req.url, req);
} });
export const db = drizzle(client);`
The main issue is this one. For some reason bun doesn't accept raw Request object and in order for it to work the url is needed, so you use url from request object itself. I tested it with all methods (select, insert, delete, etc) and it works fine. I hope it helps


Layers not working

Layers from right side is not working properly. where v-sortable is used.

fixed it.


HTML STYLE ATTRIBUTE

Description 📜
This would be a documentation explaining Html style attribute

Domain of Contribution 📊

[x] Frontend Dev HTML/CSS/JS

/assign

/assign


Cannot flush logs before process ends

I have noticed that when a process ends, not all the logs are written because they are still in the buffer.  File transport has a flush event you can hook on to prevent this.  This transport does not offer the same,  so I have had to do something like this...
//allow papertrail to flush the buffer
setTimeout(() => {
	logger.info('lambda#handler: success');
	context.succeed('success');
}, 1000);

Yes, I have the same issue. Would be great if you can hook this flush event


C# Build "Language version: Latest Minor Version" not Honored (at least for UWP builds)

Should Document: Setting the C# Build Language Version to "Latest Minor Version" does not appear to be honored by AppCenter builds. I was receiving complaints about "default literal" not be valid for C#7.0, specify C#7.1, when my setting was "Latest Minor Version" (which works fine on VS).  I explicitly set all configurations for all platforms to C#7.3 and AppCenter builds stop complaining.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: be2ace35-a6ef-349a-e009-ca55e3778602
Version Independent ID: 85a92e95-828a-54f8-b7e7-4f86b8e76abc
Content: Configure a C# UWP app in App Center - Visual Studio App Center
Content Source: docs/build/windows/uwp/first-build.md
Service: vs-appcenter
GitHub Login: @siminapasat
Microsoft Alias: siminap

@kburgoyne I'll have the SDK team take a look.

@jwargo It is related to App Center Build.

Hi @kburgoyne, this seems like something our support can help investigate rather than a docs issue. App Center support is provided directly from within the App Center portal. Any time you need help, just click the blue chat button in the lower-right corner of any page and our dedicated support team will respond. Feel free to open up a conversation with support from our portal regarding this. For additional information, see the App Center Help Center.


Update configure-l2-stretched-vmware-hcx-networks.md

Corrected VMware product names.

@rvandenbedem : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Reloader not working in Flask Docker container

I am running multiple docker containers (Node, Nginx, Flask, ElasticSearch) using a docker compose file. Whenever I change something in my flask files, flask outputs to terminal saying it detected the change, but then it never reloads. It just outputs this line and nothing else:
"flask_1          |  * Detected change in '/var/flask/flaskr/api.py', reloading"
then all my api endpoints in flask don't work after.

You need to provide a Minimal, Complete, and Verifiable example demonstrating this, otherwise there's nothing we can do. I personally don't use Docker, so your best bet for a fix is investigating this and submitting a patch.

Anecdotally, I know there are plenty of Docker examples out there that do reload.


Use a single fetch handler per Router

R: @addyosmani @gauntface
The diff is pretty busy, but this is basically just shifting the fetch handler into the Router constructor, and storing the list of routes in a private member variable.
Fixes #354

PTAL at the revised approach to confirm you're still 👍


[nrf fromlist] scripts/Kconfig: add dt_node_array_prop Kconfig functions

The motivation is to allow accessing array elements from device tree properties in Kconfig.

Add dt_node_array_prop and _node_array_prop functions to extract the array elements from properties.
Add 'dt_node_array_prop_int' and 'dt_node_array_prop_hex' keys to use in Kconfig.

Signed-off-by: Eivind Jølsgard<EMAIL_ADDRESS>(cherry picked from commit 3865e08)

In case of problems with Jenkins CI for the future, please rebase your PR branch to the newest sdk-zephyr main, due to the significant changes in CI process, which requires the newest version of sdk-zephyr main (especially this commit: https://github.com/nrfconnect/sdk-zephyr/commit/a1b281e16961fcfd04fbb6bb691dd1ee4d0123bf). Sorry for the inconvenience.

@gopiotr No worries, I have rebased the PR and reran CI now :)


Azure Media Services REST API - Create Channel JSON Inaccurate

Attempting to use the "live encoding" Create Channel JSON example from https://docs.microsoft.com/en-us/rest/api/media/operations/channel#create_channels results in a 400 Bad Request with the following response body:
{
"odata.error": {
"code": "",
"message": {
"lang": "en-US",
"value": "Parsing request content failed due to: Make sure to only use property names that are defined by the type"
}
}
}
Note:  To get this Bad Request response, you first have to fix the missing comma on line 32.
Additionally, more complete documentation on the semantics of the JSON parameters (both for this request and for things like the List Channels response) would be really helpful.
The non-live-encoding example works as is.

@dend Can you take a look at this issue?

Assigning to @Juliako for follow-up, as the content driver for Media Services.


Added some built_in keywords.

Added some built_in keywords which are part of the C++ standard.
see http://en.cppreference.com/w/cpp/io/manip/endl
Added some built_in keywords which are part of the C++11 standard.
see http://en.cppreference.com/w/cpp/utility/initializer_list
see http://en.cppreference.com/w/cpp/memory/unique_ptr

Thanks!


ethers-v5 gives TypeError: Cannot read property 'map' of undefined

I am seeing the following error after upgrading to typechain 7.0.0 and @typechain/ethers-v5 9.0.0:
Creating Typechain artifacts in directory types/contracts for target ethers-v5
An unexpected error occurred:

TypeError: Cannot read property 'map' of undefined
    at new Ethers (/home/adam/my-project/node_modules/@typechain/ethers-v5/src/index.ts:53:4)
    at TypeChain.findRealImpl (/home/adam/my-project/node_modules/typechain/src/TypeChain.ts:50:12)
    at new TypeChain (/home/adam/my-project/node_modules/typechain/src/TypeChain.ts:22:26)
    at OverriddenTaskDefinition._action (/home/adam/my-project/node_modules/@typechain/hardhat/src/index.ts:44:7)
    at async Environment._runTaskDefinition (/home/adam/my-project/node_modules/hardhat/src/internal/core/runtime-environment.ts:217:14)
    at async OverriddenTaskDefinition._action (/home/adam/my-project/node_modules/hardhat-contract-sizer/tasks/compile.js:6:3)
    at async Environment._runTaskDefinition (/home/adam/my-project/node_modules/hardhat/src/internal/core/runtime-environment.ts:217:14)
    at async Environment.run (/home/adam/my-project/node_modules/hardhat/src/internal/core/runtime-environment.ts:129:14)
    at async main (/home/adam/my-project/node_modules/hardhat/src/internal/cli/cli.ts:203:5)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed.
Exit code: 1

Adding debug reveals that allFiles is undefined in the Ethers constructor.

Turns out I had an ancient @typechain/hardhat (1.0.1) - upgrading to 4.0.0 fixed it.


Fix : Send Seen Error

Work around by adding empty getLastMsgKeyForAction() to chat object

Fixes #837

Closing as change is included in #880


HDDS-6689. New checkstyle: NoLineWrap

What changes were proposed in this pull request?
Add new checkstyle rule NoLineWrap, which checks no line wrap in pacakge, import and static import statements.
https://checkstyle.sourceforge.io/config_whitespace.html#NoLineWrap
Fix violations by running the following command multiple times.
grep -rlE '^import[^;]*$' --include '*.java' | xargs sed -szi 's/\(\nimport[^;\n]\+\)\n \+/\1/g'

P.S. Each iteration changes
import abc      import abc.def
    .def    =>      .ghi;
    .ghi;

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6689
How was this patch tested?
Existing CI.

Thanks @kaijchen for the patch.  I think it's a nice improvement.  However, there are a few feature branches out there, some planned to be merged soon.  I would like to avoid causing conflicts or failures for them due to this change.  So let's re-visit this once those branches are merged.


Unable to select single date in DatesRangeInput

It is impossible to select a single date when using DateRangeInput. When I click for the second time on a specific date I'm expecting this to be handled as an end date. Instead it simply deselects the date.
If this is desired behavior perhaps an option can be added to choose between the two types of behavior.
Version: 0.14.2
Browser: Chrome/Firefox/Safari

Yes, that was a desired behavior. Want to make a PR for this?

Sure, I added the feature to choose between types of behavior as you can see here
https://github.com/syntext/semantic-ui-calendar-react/commit/0a96c699022779a95b3e44a50c5363528bba79e6
Although I can build and see my change works, I'd like to write some proper tests for it but I when I run 'yarn test' it complains about an unexpected token in {allowJs:true}. Could you perhaps explain in a few words how to set up my environment properly? Apparently 'yarn install' isn't sufficient. Near zero TS experience here 😄

I was struggling with this line TS_NODE_COMPILER_OPTIONS=\"{\"\"allowJs\"\":true}\" too once. I was just experimenting and found a configuration that works. I don't understand why it doesn't work now.
As a quick solution you can try the following:

change allowJs to true in tsconfig.json at project root
change test command in package.json to this "test": "npx mocha -r ts-node/register ./test/setup.js ./test/**/*.{js,jsx,ts,tsx}"

Try yarn test again.

Thanks! Perfect solution, I've created a pull request.
Btw, I noticed the tests in inputs/pickers/dayPicker don't run when using yarn to execute all the tests at once. You might want to change ./test/**/*.{js,jsx,ts,tsx} to ./test/**/**/*.{js,jsx,ts,tsx}

How about using recursive flag npx mocha -r ts-node/register --recursive ./test/setup.js ./test? On my machine all of the variants work equally well with npm run test and yarn test.

PR merged


Integration: Alter TestContainerPids for Windows

The point of this test is to see that we successfully can get all of the pids running in the container and they match the number expected, but for Windows this concept is a bit different. Windows containers essentially go through the usermode boot phase of the operating system, and have quite a few processes and system services running outside of the "init" process you specify. Because of this, there's not a great way to say "there should only be N processes running" like we can ensure
for Linux.
With all that said, on Windows check that we're at least greater than one ("init"+system services).

Can we not expect and test for >=N+1?

Yea we could do that. I've been thinking maybe we change the windows longCommand to 'cmd /c ping -t localhost' so there's two processes we launched, OR we launch an exec for windows in this test and just check that the two (init + exec) processes we launched exist by their IDs.


Rename a gem to reportportal

Rename a module to ReportPortal.
This name is more meaningful and consistent with names of other clients.

I decided to name a gem "reportportal" instead of "report_portal" mainly for consistency with organization name and clients for other languages.


Allow custom logging in docker2aci lib.

Something to consider before releasing 1.0 in a separate PR - replace the pkg/log with a Logger field in the CommonConfig struct, so users can pass their own logger. The Logger fields would be of some interface type like:
type Logger interface {
    Debug(i ...interface{})
    Info(i ...interface{})
}

so the implementer could implement the Debug function in the same way as currently pkg/log does without resorting to some global boolean variable set by the InitDebug() function. Possibly export two implementation like DefaultLogger with configurable debugging function and DevNullLogger.

logrus defines a generic StdLogger interface and a more rich FieldLogger for this purpose.

My proposition is to use a minimalistic stdlib-compatible logger interface, not imposing log levels on the user of the library, since this concept is unknown in stdlib. I have a local branch working with the following (also logrus compatible) signature:
package log

type Logger interface {
	Print(...interface{})
	Printf(string, ...interface{})
	Println(...interface{})
}

We provide a StdLogger instanciating a stdlib logger using https://godoc.org/log#New used by the CLI main.go.
The user of the library is free to inject any Logger where we introduce two new members in CommonConfig:
type CommonConfig struct {
	Squash      bool               // squash the layers in one file
	OutputDir   string             // where to put the resulting ACI
	TmpDir      string             // directory to use for temporary files
	Compression common.Compression // which compression to use for the resulting file(s)

	Info  log.Logger
	Debug log.Logger
}

To retain backwards compatibility we initialize the logger properly in case they are undefined:
func (c *CommonConfig) initLogger() {
	if c.Info == nil {
		c.Info = log.NewStdLogger(os.Stderr)
	}

	if c.Debug == nil {
		c.Debug = log.NewNopLogger()
	}
}

The above method is called inside ConvertSavedFile, and ConvertRemoteRepo. The signatures of the previously mentioned methods remain unchanged, only two fields are added, retaining backwards compatibility .... at least for this issue.


[WIP] v1.0.1

Organized the files
Connect and disconnect info
User badges
Same user message join
Channel will change color once streamer has said something in chat (no auth hack).
Config dialog:

Multiple channels
Bionic style toggle
Show badge toggle
Fidden users

badges list MUST be updated, they are a year old and lack many sub badges, missing badges are hidden
Hosted here

thanks for your hard work stoney! i've got most things covered from here for now and can come back to it if needed.


Allow View containers to specify custom measure functions

This PR allows ViewManagers that are containers (extending ViewGroupManager) to specify it's custom CSSNode.MeasureFunction.
ViewManagers that don't need to specify this can extend SimpleViewGroupManager instead

Hey @obipawan, what's the motivation for this?

Hey @mkonicek
This change is basically to allow custom Views that are also containers(ViewGroup) to other views to be able to specify a measure function if they wanted to. With the current ViewGroupManager, it's not possible to specify a custom LayoutShadowNode.
It also brings this in parity with iOS where one can create any native view from RCTViewManager and then specify a custom RCTShadowView.

OK thanks for the explanation! @kmagiera, @foghina are you OK with this?

Why can you not provide a custom LayoutShadowNode by extending ViewGroupManager?

I don't see this change being necessary to support the use case you're describing. If you want your view group manager to use custom class as a shadow node you can simply override createShadowNodeInstance and getShadowNodeClass (note that both methods are not marked as final in ViewGroupManager). TBH I'd rather avoid adding yet another base class to the namespace

Hey @kmagiera It's possible to override createShadowNodeInstance, but as for getShadowNodeClass you'd have to make the return type to an unchecked Class type for it to work. Though this is yet another base class, it would make it safer to have the return type checked IMO.

@obipawan  Are you sure that is the case here? The return type on getShadowNodeClass in ViewGroupManager is Class<? extends LayoutShadowNode> (not Class<LayoutShadowNode>) which should allow you to return class object of any class that extends LayoutShadowNode.

@kmagiera Apologies! you're absolutely right. I missed that this was changed for 0.18 and missed this even while submitting this PR. My bad!


ConnectionPool.lockConnection() hangs under certain conditions.

When using ConnectionPool.lockConnection() in parallel in multiple tasks (single thread), under certain conditions it hangs.
On my CentOS 7 x86_64 + vibe.d 0.7.29 this code
/+ dub.json:
    {
        "name": "test",
        "dependencies": {
            "vibe-d:core": "0.7.29"
        }
    }
+/

import std.stdio;
import std.socket;
import std.datetime;
import std.functional;
import vibe.core.core;
import vibe.core.log;
import vibe.core.concurrency;
import vibe.core.connectionpool;

class Conn {}

void main() {
    runTask({
        // create pool with 2 max connections
        auto pool = new ConnectionPool!Conn({ return new Conn; }, 2);
        auto task = Task.getThis(); // main task
        void worker(int id) {
            {
                auto conn = pool.lockConnection(); // <-- worker(4) hangs here
                sleep(1.msecs); // <-- important, without sleep everything works fine
            }
            task.send(id); // send signal to the main task
        }
        // run 4 tasks (2 * pool max connections)
        runTask(&worker, 1);
        runTask(&worker, 2);
        runTask(&worker, 3);
        runTask(&worker, 4);

        // wait for first signal and run one more task
        writefln("recv  %s", receiveOnly!int);
        runTask(&worker, 5);

        // wait for other signals
        writefln("recv  %s", receiveOnly!int);
        writefln("recv  %s", receiveOnly!int);
        writefln("recv  %s", receiveOnly!int);
        writefln("recv  %s", receiveOnly!int);

        exitEventLoop();
    });
    runEventLoop();
}

outputs
recv  1
recv  2
recv  3
recv  5

and never exits the even loop.
Where is recv 4?

Appears to be already fixed by #1563. I'll add this as a regression test.


Correct custom_system_prompt for Mac

I'm using a 15-inch MacBook Air (M3).
The GUI agent told me "at the start of our conversation, there was a specific note stating "NOTE: you are operating a Linux machine".
That's because of this code in app.py > def setup_state:
device_os_name = "Windows" if platform.platform == "Windows" else "Mac" if platform.platform == "Darwin" else "Linux"

It should be:
device_os_name = "Windows" if platform.system() == "Windows" else "Mac" if platform.system() == "Darwin" else "Linux"

With this change, the GUI agent gets it right, with "NOTE: you are operating a Mac machine".
A good way to check is to ask the GUI agent to show the window "About This Mac". It didn't work until I changed the code.

Thank you for your suggestion! This bug is now fixed. More suggestions are welcomed:)


MAC timestamp should be in milliseconds now

So says http://tools.ietf.org/html/draft-ietf-oauth-v2-http-mac-05

This was fixed by #509

MAC Token support is being dropped from the OAuth2 gem.
See #589


Update multi-repo-checkout.md

Typo on line 165. Change MyAzureReposGitRepo to MyAzureReposGitRepository

@proudmonkey : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@steved0x : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@proudmonkey I made a tweak and I am merging this PR, thank you for your contribution!

#sign-off


Update Beam Wallet app icon

Please update app icon for
Mainnet - https://drive.google.com/drive/u/1/folders/1iPBzeULjeZmOiZocBOYjTshnfDKb_9Xz
Testnet - https://drive.google.com/drive/u/1/folders/1U8pEk1UULpF5wzh8axFwq6x8ypXPS3F3
Masternet - https://drive.google.com/drive/u/1/folders/1zLfGefd-vvBrNrbH8uXIP0ANUvtcLiYQ

checked on masternet


Fix the "zeroise" spelling error, and use the feature directly

It was not possible to activate the zeroizing feature in this crate. Because neither "zero" nor "zeroize" actually activated the implementation due to a spelling error.
Skip creating an extra feature "zero" and instead just use the one called "zeroize" by having the dependency optional. This is more idiomatic and avoids creating a set of basically duplicate features.
It should however be noted that the zeroize functionality is currently somewhat limited. Yes the Uake and Ake types can be cleared out. But the SharedSecret produced by the library is just a raw array without Zeroize. I'd suggest that more long term we look into doing something similar to what I added to classic-mceliece in https://github.com/Colfenor/classic-mceliece-rust/pull/20. This means that types containing secrets can no longer just be type definitions translating to arrays. Rather they should probably be structs owning the secret data, and clear on drop. But all of this can be done later of course.

This is technically a breaking change. Since removing a feature is breaking the API. But the previous feature was not "usable" anyway. One might consider this and #26 bugfixes, merge them, release 0.3.1 and yank 0.3.0? Up to you.


Update ubuntu version from 18.04 to 22.04

Context
Ubuntu 18.04 has reached end of life.
Suggested solution
Ubuntu images should be upgraded to the latest LTS version, 22.04.
Considered alternatives

Additional details
Vulkan doesn't work out of the box in ubuntu:18.04 image.
$ vulkaninfo
===========
VULKAN INFO
===========

Vulkan Instance Version: 1.1.70

/build/vulkan-UL09PJ/vulkan-1.1.70+dfsg1/demos/vulkaninfo.c:2700: failed with VK_ERROR_INITIALIZATION_FAILED

I agree, it's time.
We're gonna need to sort out which dependencies are needed for 20.04 or 22.04. Ideally the latter, but also depending on what the requirements of Unity Editor are.

Latter is good. Anything I can help with (e.g. which dependencies to look into)? I need it for my CI and I'd rather not have a custom image.

Sure.
First of all, it's important to understand how to iterate quickly. You can build images locally by following instructions in DEVELOPMENT.md

We'll need an upgrade of the base image, so that it's based on 22.04.
After that, the hub and editor images can be created. They will have to pass the build process.
Finally we'll need to ask a few people to test their use cases based on the changes from the PR. For example it'll need to work with all targetPlatforms as well as with ECS and burst compilation and the default projects with the different render pipelines should run without a hitch as well.

If any good changes were made that are also useful for other platforms:

We'll need to repeat the same steps for Windows.
MacOS does not yet have images as per https://github.com/containerd/containerd/discussions/5525

Once done, we'd have to do a major version bump for the images, so that old installations don't automatically start failing.


Is any other code to calculate mask’s mAP and mAR?

hi, i saw some code about calculating mAP and mAR in mmdet.evaluate but, i dont know that we use the same code to calculate mask and bbox mAP and mAR or not using MASK R CNN
is used same code ? or we have any other code for mask? thanks

We use pycocotools to calculate mask AP and mAR following common practices. See related code here: https://github.com/open-mmlab/mmdetection/blob/master/mmdet/datasets/coco.py#L440


Exporting a project does not save or even prompt you that there are unsaved changes

I have searched for previous bugs with similar text.
Godot version:
3.2.2 stable
OS/device including version:
linux
Issue description:
If you have any unsaved changed (e.g. project settings), selecting Export does not auto-save or even a prompt to warn you there are unsaved changes. You would expect either the same behaviour as 'run' or at least a prompt warning you before showing the export dialog.
I know, because I spent a while debugging a fault and it was simply I had changed my project stretch mode :)
Steps to reproduce:

make a change to the game or the project
select export
Note, this was for html, I presume the same for other exporters

Hello,
I'll try to solve this issue.
I think a prompt asking if it needs to apply or drop unsaved changes (or cancel the action) might be a good answer to this.
Please tell me if you have any other idea ^^

I don't. I'd love to help out more, but works take me forever to get up to
speed.
On Fri, 7 Aug 2020, 11:59 Antoine Félix<EMAIL_ADDRESS>wrote:

Hello,
I'll try to solve this issue.
I think a prompt asking if it needs to apply or drop unsaved changes (or
cancel the action) might be a good answer to this.
Please tell me if you have any other idea ^^
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/40963#issuecomment-670460649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADCERRKB2KYEBOXDICWZBF3R7PNCFANCNFSM4PSNOKPQ
.

This should hopefully resolve this issue.
And don't worry, making a clear and argumented issue like you did is already helping a lot! :D
If you have any remarks or if you think the pull request doesn't solve the issue in a satisfying way, don't hesitate ^^

Closing in favor of https://github.com/godotengine/godot-proposals/issues/3605, as feature proposals are now tracked on the Godot proposals repository.


Loosen the rate limit restriction in IPAMController

TODO
Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.ougjguo 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.

/sem-approve


Dev-CMS

Describe the Problem
As a site dev contributor I should be able to submit pictures and links of/for myself.
References
Will link screenshots to design.
Describe the Proposed Solution
Should expose "Submit" button for designated "Meet the Team" or "Contributors" section that can perform CRUD operations to the contributors table (users & UserModel?) in the DB. This should be available in the corresponding page view

Endpoint Admin/Dev

Add photo
Add Links

Key
Value

Site Name
URL

This data structure (object) will eventually be passed to the src/lib/components/icon-parser.svelte to parse out the correct icon based on the keys

@codingMustache lmk if this lines up with your vision

This is perfect. As for the tech to add photos. We should use cloudinary, use the SDK and not the API. there is a lot of functionality the SDK offers that the API doesn't and it cleans up a lot of code. When you install this package make sure you read the docs bc most packages should be installed with the -D flag.


[Bug]: Switching groups resets quest changes

Bug Description
When editing a quest in the sidebar, switching groups will discard all changes.
How to Reproduce?

Select a quest with the Move/Select tool.
Modify something in the sidebar.
Click on another group.
Return to the group the quest is in. The changes will be reverted.

Expected Behavior
The changes are saved automatically, as they are through every other form of exiting editing.
Version
0.0.2
Mod Loader Version
1.20, 0.14.21
Mod Loader
Fabric
Logs or additional context
This also rarely happens when toggling edit mode or closing, but I've been unsuccesful in consistently replicating it in those cases. I've noticed however that whenever it does happen, it's the first edit after enabling the menu. If i find a consistent cause of the problem, I will report it.
Code of Conduct

[X] I have searched the issue tracker and confirmed that the issue has not been reported
[X] I have checked the FAQ (if one is present) and my issue is not listed
[X] I have verified that I am using the latest version of the mod
[X] I have verified that I have installed all the required dependencies for the mod
[X] I have verified that I do not have Optifine installed. We will close issues if we detect Optifine is in your pack. We cannot fix Optifine issues. Please do not report issues with Optifine present.

tested in 0.0.3, still happens


Updating library versions

Upgrading library versions

the library should remain compatible with Java8 so some of the changes were reverted.
thanks for the contribution


Feat: add a11y for the count down component

Summary

Feeds the count down information to the SR panel
Refactor the Countdown to a functional component
The Countdown component initiates the A11yManager when mounted, clean up the when unmounted, and calls the update every 1 sec.
The A11yManager decides how frequent to update the SR panel, and it also handles the translation part

Tested scenarios

Added tests for the Countdown component
All tests passed

Relates to ticket: COWEB-1232 & a11y instanceId:<PHONE_NUMBER>

The advice from LevelAccess was that the countdown message in the SRPanel should clear after a few seconds (5 or 10), but I can see that it remains there until the next update

The advice from LevelAccess was that the countdown message in the SRPanel should clear after a few seconds (5 or 10), but I can see that it remains there until the next update

I have some doubt about this 🤔 , because a new message will feed into this aria-live section after certain time and will be announced by the SR, and SR only announces additions text as we instruct it to do so, so it won't repeat the same message.
I suspect the advice is to avoid the same message being repeated, but with additions text it won't happen.
Hence I think we don't need to clear out the message..

So can the A11yManager component be deleted?

So can the A11yManager component be deleted?

A11yManager is renamed to CountdownA11yReporter. Do you mean move all the logic from CountdownA11yReporter to the custom hook?

So can the A11yManager component be deleted?

A11yManager is renamed to CountdownA11yReporter. Do you mean move all the logic from CountdownA11yReporter to the custom hook?

No I mean delete the A11yManager.tsx file. It still exists in the Countdown directory

So can the A11yManager component be deleted?

A11yManager is renamed to CountdownA11yReporter. Do you mean move all the logic from CountdownA11yReporter to the custom hook?

No I mean delete the A11yManager.tsx file. It still exists in the Countdown directory

Hm, it's not there Countdown directory. Maybe you didn't pull the latest?

So can the A11yManager component be deleted?

A11yManager is renamed to CountdownA11yReporter. Do you mean move all the logic from CountdownA11yReporter to the custom hook?

No I mean delete the A11yManager.tsx file. It still exists in the Countdown directory

Hm, it's not there Countdown directory. Maybe you didn't pull the latest?

It's still there in my project... weird

So can the A11yManager component be deleted?

A11yManager is renamed to CountdownA11yReporter. Do you mean move all the logic from CountdownA11yReporter to the custom hook?

No I mean delete the A11yManager.tsx file. It still exists in the Countdown directory

Hm, it's not there Countdown directory. Maybe you didn't pull the latest?

It's still there in my project... weird. Must just be WebStorm behaving oddly.

Sometimes git reset --hard to the latest commit might work.


CI passes even though job fails

https://dev.azure.com/ms/vscode-python/_build/results?buildId=56281&view=results
The failed exit code 1 is not being passed to the core process, due to which CI passes. As far as I've noticed none of the CI builds are actually passing since a while now.
The multiroot workspace and debugger tests for windows almost always have this issue.
@DonJayamanne Any idea why/where this might happen?

The problem is with the way we run the tests. We shouldn't be using npm run.... to run the tests.
We need to run the tests using raw node commands.
See here, I thought I fixed this in other pipelines as well. Guess I didn't. @karthiknadig /cc
https://github.com/microsoft/vscode-python/blob/ec90eff4c78146335c67c08ff7d6b2df1e0344ea/build/ci/templates/steps/uitest.yml#L138-L144

In testBootstrap.ts
we'll need to handle the error and exit with an exit code as follows:
start().catch(ex => {
    console.error(ex);
    process.exit(1);
});

Doesn't seem to be happening anymore

CI fails with exit code 1 (https://dev.azure.com/ms/vscode-python/_build/results?buildId=57947&view=results):

CI passes with exit code 0 (https://dev.azure.com/ms/vscode-python/_build/results?buildId=57872&view=logs&j=429bac49-2647-5fb0-aca5-26b3177b1562&t=0a0e6554-ca16-5d32-d945-2e3a2fc83193):


PR: Avoid visible_blocks to be empty when first visible block is wrapped

Fixes #3166
This was also causing fold panel to be empty.

Thanks @rlaverde! Could we add a test for it?


Add missing uninstantiate(ancestor)

Fixed #5281

Unfortunately CI is failed on downloading LLVM. It is completely not related to this.

@mverzilli or @bcardiff, can we get your seal of approval on this?
Thank you.
❤ ❤ ❤


Locations view UI tweaks

Ah yes, that only happens in the preview build, forgot to investigate. I'll try to fix.

In reply to:<PHONE_NUMBER>

got its not our issue then

In reply to:<PHONE_NUMBER>


Add quickstart template for user studies of blob storage using nodejs

Kyle , it looks good. One nice to have suggestion - A simple static HTML page which could let the developer test the endpoints(which could invoke the commands internally).


Typesafe router[✨]

Is your feature request related to a problem?
Hello, I was trying to type my routeLoader$ where I use params and found out, that I can't. It's not automatic nor can I use generic. What I ended up with is assertion via unknown which is not a good solution.
type RouteParams = {
  id: string;
};

export const use_get_movie = routeLoader$(async ({ params }) => {
  const response = await get_movie((params as unknown as RouteParams).id);

  return response.data;
});

Describe the solution you'd like
It would be awesome to have it automatically typed. Since it's file based, when I have a folder [id] it should be able to know, that id is a param that is a string
Another feature that would be great is typed <Link> component. That if I start typing in href attribute, it would suggest all the routes that are available in the project.
Describe alternatives you've considered
Inspiration could be Tanstack Router although there is a different API. But Next.js router has typed Link component, and I believe it's a similar router to Qwik router, since it's file based. So it should be possible.
Additional context
Version:
<EMAIL_ADDRESS>"0.103.0",
<EMAIL_ADDRESS>"0.103.0",

there is declarative routing and typed routes
https://github.com/ProNextJS/declarative-routing/blob/main/docs/qwikcity.md
https://qwik.dev/docs/labs/typed-routes/


Remove feature flag for 1.24

Issue #, if available:
Description of changes:
Remove feature flag for 1.24
Testing (if applicable):

make
make e2e

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.12

Changing the default kube version is from the cluster.go file. Will send out a pr for that.

/approve

/cherrypick release-0.12


[suggestion ] : A slap command command which virtually slap another person

A slap command which some cool and fun gif list

Is this issue solved?

@gamerrio is assigned to it

ok

@gamerrio Are you still working on this?
I saw that your version of the bot, gave output for the command.
So, if it's done, it would be nice if you made a PR.

Take your time 👍

Shifting this to Build v1.1
Might make a release soon

Are you working on this issue, or should I unassign you?

currently i am not working on this issue

@gamerrio you have been unassigned, since the period of response was too long
If you're still interested in contributing, you could ask and get re-assigned :+1:

@gamerrio you have been unassigned, since the period of response was too long
If you're still interested in contributing, you could ask and get re-assigned :+1:


update PPP Haiqel

Codecov Report
Patch and project coverage have no change.

Comparison is base (923f49a) 55.63% compared to head (9001cea) 55.63%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #148   +/-   ##
=========================================
  Coverage     55.63%   55.63%
  Complexity      505      505
=========================================
  Files           109      109
  Lines          2069     2069
  Branches        214      214
=========================================
  Hits           1151     1151
  Misses          847      847
  Partials         71       71

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

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


consolidate MetricsCloseable and IShutdown interfaces

originally filed as https://github.com/engineblock/engineblock/issues/130

IShutdown is no longer present in the codebase. This must be done.


output the MetadataBLOBPayload to plain-text JSON file

It would be nice if there was a way to have the MetadataBLOBPayload to be output to a (pretty-printed) plain-text JSON file.
and/or have a button that simply displays all the metadata entries in a single view, similar to the present view of an individual authenticator's metadata that one obtains by clicking on the authenticator name.

Hi Jeff
FYI, I updated my tool to decode the blob on the backend. The JSON can now be seen in https://github.com/opotonniee/fido-mds-explorer/blob/main/mds.js. The change history will thus be visible in git commits.


[AppleBug] Deleting the same index path twice crashes in batch updates

We disabled this test due to flakiness. It turns out that UICollectionView can't handle deleting from the same NSIndexPath twice in the same performBatchUpdates:completion: phase. (though it'll handle 2x inserts just fine...)
This really only manifests when a client deletes the head, e.g. {section = 0, item = 0}, and the updates are coalesced in the same runloop.
Discovered in https://github.com/Instagram/IGListKit/commit/a415ef5552cbfbf2d3884a705241973f4ea72dde and workarounds attempted in #657.

should file a radar

Going to punt this for next release, no reports about it and we don't run into this at all, more of a known bug that we might eventually find but is too rare to make hi pri.

Area:
UIKit
Summary:
If the same NSIndexPath is included in the array provided to -[UICollectionView deleteItemsAtIndexPaths:] more than once, eventually there will be an EXC_BAD_ACCESS in -[UICollectionViewData layoutAttributesForItemAtIndexPath:].
Say I have an NSMutableArray of data powering the data source of my UICollectionView, if I wanted to delete the head element twice, I might do something like:
[self.collectionView performBatchUpdates:^{
  NSArray *paths = @[
    [NSIndexPath indexPathForItem:0 inSection:0],
    [NSIndexPath indexPathForItem:0 inSection:0]
  ];
  [self.data removeObjectAtIndex:0];
  [self.data removeObjectAtIndex:0];
  [self.collectionView deleteItemsAtIndexPaths:paths];
} completion:nil];

If I repeat this action, or do other batch updates, eventually it will crash.
Steps to Reproduce:

Open sample app provided
Tap "Play" nav bar item a few times
Observe crash

Expected Results:
Delete the first two elements each time, don't crash.
Actual Results:
It will delete the first two elements, but after a few times tapping the "Play" item, it will crash.
Version:
10.3.1 (14E304)
Notes:
Configuration:
7 simulator, iPhone 7+

UICollectionViewMultipleDeletesSameIndex.zip

Please dupe:
https://openradar.appspot.com/radar?id=5021459084738560

Got a response from the big fruit. This is considered a bad update and crash is ok. 🤷♀

Weird 🤷♂


May I ask when this project will be open-sourced?

That's fantastic, thank you again for your excellent work.

I've just pushed the core code. I will still need to cleanup the code so there is an easy to use demo.
I will also release the encoder weights.
See https://github.com/aluo-x/BrainDiVE/blob/main/run_brain_guidance_category.py#L266 for a basic usage example for your own fMRI encoder & brain region.


Definition of canonical is confusing

The concept of canonical in YaNFD is confusing. For ethernetURI, fdURI, internalURI, nullURI, udpURI, canonical means well-formed:
https://github.com/named-data/YaNFD/blob/001755fc7bb1fb335704775f3bb910c6b14b4bc1/ndn/uri.go#L286-L302
For devURI  and unixURI, canonical means existing interface, which is related to specific machine and operation system:
https://github.com/named-data/YaNFD/blob/001755fc7bb1fb335704775f3bb910c6b14b4bc1/ndn/uri.go#L303-L306
This restricts that the unit test can be only executed on specific machines:
https://github.com/named-data/YaNFD/blob/001755fc7bb1fb335704775f3bb910c6b14b4bc1/ndn/uri_test.go#L172
Not every machine allows building program access to this path.
Note: ndn-cxx does not check the physical existence of interface in isCanonical.

This concept was inherited from NFD/ndn-cxx. I'd recommend looking at what they did (and trying to stay compatible) :-)

This concept was inherited from NFD/ndn-cxx. I'd recommend looking at what they did (and trying to stay compatible) :-)

ndn-cxx does not check physical existence of faces. For devURI, it always succeed if there is a path; for unixURI, ndn-cxx always fails (not supported).


Please 1.20.2 fabric version

PLS
+The issue was resolved by updating the mod version. sorry.

how to change version
1:jar -> zip
2:unzip the file
3:open file -> open fabric.mod.json
4:"depends": {
"fabric": "*",
"minecraft": "1.20.1"
"depends": {
"fabric": "*",
"minecraft":  [
"1.20.1",
"1.20.2"
]
5:zip file
6:zip -> jar


Pod restart is throwing false alarms

It appears that the pod restart checker is triggering on pods that have not restarted five times in the last hour.  This only happens once the pods have actually restarted five times previously.  Likely the counter is not being purged correctly.

The pod in the example was is part of a stateful set and had been restarted within the past hour (the timeframe the PRC looks for restarts over).  I believe the previous iteration of this pod WAS restarted more than 5 times in the last hour, but the pod was also terminated completely, wiping the restart count back to 0 when you check after the fact.
I just tested and terminating a pod with a restart count > 5 generates an alert - thats where I think this bug is. I'll do a bit more investigation.

The bug lies here: https://github.com/Comcast/kuberhealthy/blob/8f25870ae211caa721c6cd6ee2626a08882f2f88/pkg/checks/podRestarts/podRestarts.go#L207 .
We are collecting metrics based on pod names - since stateful set pods come back with the same name, kuberhealthy treats the terminated and newly created stateful set pod as the same pod.  In the linked line, Kuberhealthy is only measuring the delta between the largest recorded value and the smallest recorded value of restarts.  Since the new stateful set pod of the same name is created with a restart count of 0, if the outgoing pod had a restart count of more than 5, Kuberhealthy alerts that the delta between the max and min restart values was greather than 5.
We can handle this by either comparing something more specific than pod names (resource version?) Or detecting when the restart count is reset to 0 and clearing that pod's history upon reset detection.


Alert on change - radioButtons and checkBoxes

Description
Introduced a new prop, alertOnChange, which can be configured to trigger an alert when users attempt to select a different radioButton or uncheck a checkBox.
This should also enable handling through expressions, as desired by SSB. However, I haven't had time to test this with expressions.. :
If any adjustments are necessary (hopefully not too many for your sake) , I hope that you guys can make the necessary refinements while I'm on vacation. ❤
Arigatou gozaimasu!

Related Issue(s)

closes #1233

Verification/QA

Manual functionality testing

[x] I have tested these changes manually
[ ] Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
[ ] No testing done/necessary

Automated tests

[ ] Unit test(s) have been added/updated
[x] Cypress E2E test(s) have been added/updated
[ ] No automatic tests are needed here (no functional changes/additions)
[ ] I want someone to help me make some tests

UU/WCAG (follow these guidelines until we have our own)

[x] I have tested with a screen reader/keyboard navigation/automated wcag validator
[ ] No testing done/necessary (no DOM/visual changes)
[ ] I want someone to help me perform accessibility testing

User documentation @ altinn-studio-docs

[ ] Has been added/updated

[ ] No functionality has been changed/added, so no documentation is needed
[ ] I will do that later/have created an issue

Support in Altinn Studio

[ ] Issue(s) created for support in Studio

[ ] This change/feature does not require any changes to Altinn Studio

Sprint board

[x] The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
[ ] I don't have permissions to do that, please help me out

Labels

[x] I have added a kind/* label to this PR for proper release notes grouping
[ ] I don't have permissions to add labels, please help me out

I took the liberty of refactoring this logic into a reusable hook that can be used to control the DeleteWarningPopover and also updated the existing places where this was used.

Found a problem when testing this. In checkboxes, the dialog appears twice. Sometimes the first button (near the top) works, but not all the time. Radio seems OK. Also, I don't think we can close #1233 as long as this hasn't been implemented for dropdowns.


WIP: AdaptiveLasso and AdaptiveLassoCV

closes  #165

Codecov Report

Merging #169 (3329980) into master (13a61d9) will increase coverage by 0.02%.
The diff coverage is 86.27%.

@@            Coverage Diff             @@
##           master     #169      +/-   ##
==========================================
+ Coverage   84.81%   84.83%   +0.02%
==========================================
  Files          12       13       +1
  Lines         981     1029      +48
  Branches      137      139       +2
==========================================
+ Hits          832      873      +41
- Misses        108      115       +7
  Partials       41       41

Impacted Files
Coverage Δ

celer/dropin_sklearn.py
92.50% <65.00%> (-3.93%)
:arrow_down:

celer/__init__.py
100.00% <100.00%> (ø)

celer/homotopy.py
87.68% <100.00%> (+0.37%)
:arrow_up:

celer/tests/test_adaptive.py
100.00% <100.00%> (ø)

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 13a61d9...77e8eac. Read the comment docs.

It doesn't look that good so far:

@josephsalmon @agramfort @QB3
Here's the example : https://183-122246365-gh.circle-artifacts.com/0/dev/auto_examples/plot_adaptivelasso.html#sphx-glr-auto-examples-plot-adaptivelasso-py
Agree with the name for new parameter: n_reweightings ?
Docstrings may need to be polished, feedback welcome.  As you can see there's a lot of duplicated code/docstring with the Lasso class...

@josephsalmon @agramfort @QB3
Here's the example : https://183-122246365-gh.circle-artifacts.com/0/dev/auto_examples/plot_adaptivelasso.html#sphx-glr-auto-examples-plot-adaptivelasso-py
Agree with the name for new parameter: n_reweightings ?
Docstrings may need to be polished, feedback welcome. As you can see there's a lot of duplicated code/docstring with the Lasso class...

Maybe this is a little bit off topic but I put it here, just in case:

I am impressed that the adaptive Lasso does better in prediction for the CV
I am also impressed by the piecewise differentiability of the path for the adaptive.
From what I remember, when I played with the adaptive a while ago, the mse_path was highly discontinuous, and we thus we could not not use autodiff to set the regularization parameter. Your plot seems to suggest the opposite, this may worth investigate.

I don't think it does better in prediction, does it ?
If you put n_samples, n_features = 100, 160 the MSE curves go havoc for Adaptive:

This is interesting, and weird!
Do you confirm this is bug less on the AdaptiveLassoCV side?
Could it be due to lack of convergence?
The blue dotted line on the right is wild...is the black line the average of the 5 folds (seems median would be less damaged here)?

I made another experiment on Finance:

The part where it gets funny: AdaptiveLasso uses only one coefficient:
In [26]: (lasso.coef_ != 0).sum()
Out[26]: 267

In [27]: (adaptive_lasso.coef_ != 0).sum()
Out[27]: 1

In [28]: np.where(adaptive_lasso.coef_)
Out[28]: (array([0]),)

In [29]: X0 = X[:, 0].toarray().squeeze()

In [30]: np.dot(X0, y) / np.linalg.norm(y) / np.linalg.norm(X0)
Out[30]: 0.9941543404858482

In [31]: adaptive_lasso.intercept_, lasso.intercept_
Out[31]: (-0.6253583759665271, -1.125413246436354)

In [32]: X0_cent = X0 - X0.mean()

In [33]: y_cent = y - y.mean()

In [34]: np.dot(X0_cent, y_cent) / np.linalg.norm(y_cent) / np.linalg.norm(X0_cent)
Out[34]: 0.8085836249206475

In [35]: lasso.normalize
Out[35]: False

In [36]: sparse.linalg.norm(X, axis=0)
Out[36]:
array([436.72549038,<PHONE_NUMBER>3,<PHONE_NUMBER>7, ...,   1.20056613,
         1.47236374,   1.89979315])

On a different note, I think the way the adaptive path is computed is suboptimal: instead on doing
for alpha in alphas:
    for iter in range(n_reweightings):

we should do
for iter in range(n_reweightings):
   for alpha in alphas:

in order to benefit from warm start. Otherwise, for a fresh new alpha (without weights), initiliazing with the last reweighting solution for the previous alpha is not very useful. Makes sense @josephsalmon

sparsity =1 for AdaptiveLassoCV? really unexpected (normalization issue?).
I am also supprised that for alpha -> 0 the two methods reach different errors level (should be the OLS performance, but I agree in high dim, there might be different least-squares solutions).
And I agree with the loop inversion: very good idea!

I am not sure why the two paths should tend to same solutions as alphas goes to 0 : ot should be the basis pursuis performance for the Lasso, but for the adaptive lasso I don't know what the limit is expected to be.

You are right, I just thought they would be closer...
My guess would be the Adaptive Lasso with alpha->0 should converge to the solution of the interpolation Xcoef=y with constraints \norm{coef}_{0.5} minimized.

It's way faster this way but some gap scaling is wrong somewhere, we get negative ones in doc examples and on news20:
import time
from celer import AdaptiveLassoCV
from libsvmdata import fetch_libsvm
X, y = fetch_libsvm("news20")
t0 = time.time(); clf = AdaptiveLassoCV(verbose=1, cv=2, n_jobs=2, eps=1e-2).fit(X, y); dur = time.time() - t0


Comments on Lines with Dataflow Always Pushed to New Line

If you make a comment on a line that has dataflow information, it will always push the comment above the line. It will do this regardless of how large the value is that has been calculated. Kinda annoying when you want to be able to read down a series of comments later. Both linear and graph view are affected.

related to https://github.com/Vector35/binaryninja-api/issues/708

Closing in favor of #708


Move in-publish to devDeps

Could you do a patch release with this? 👼

Released as v1.0.1, sorry for the delay.

No worries, thanks!


Minor corrections in cpp.php

Updates for a couple of typos I noticed on this page:

Update working fibonacci clang invocation to reference good_fibonacci.c
replace returb with return in b.c examples

I don't have a good setup atm for pulling and validating these changes locally.  They were authored with the GitHub edit feature.  Feel free to ignore, edit, or merge as desired.

Thanks for your work over the years by the way.  I've enjoyed reading your blog since I discovered it around 2011.


also sanitize / from app name and label

mess app ids use /, which we also want to sanitize before sending to collectd

👍


Failed to minify code from external : @loadable/component

🐛 Bug Report
Build fails after using @loadable/components
Failed to minify code from external : @loadable/component
To Reproduce
Followed the same steps as per the examples.
Info
"@loadable/babel-plugin": "^5.13.0",
"@loadable/webpack-plugin": "^5.13.0"
"@loadable/component": "^5.13.1",
"@loadable/server": "^5.13.1"
Expected behavior
Link to repl or repo (highly encouraged)
https://gist.github.com/kri7217/8f87d573d92c9b7b3a94087c9e2d1f62
https://gist.github.com/kri7217/c1843f5f47e1440e7ed7bc310bde4918
Please provide a minimal repository on GitHub.
Attached my webpack config file and screenshot,

Run npx envinfo --system --binaries --npmPackages @loadable/component,@loadable/server,@loadable/webpack-plugin,@loadable/babel-plugin --markdown --clipboard
Paste the results here:

Attachment
webpackconfig.txt

tried only with
externals :[nodeexternals()]. Build passed
But namedChunks is always empty [ ]
*******scripttags

You might consider filing an issue for webpack or CRA as there is nothing loadable can fix.
Technically it was already solved - https://github.com/facebook/create-react-app/issues/1125


Warn if naive singularize method receives a word that doesn't end in "s"

Also fixes a nearby spelling errors in docs

Thanks @lukemelia!


Lol. I did the final level :D

I saw your post about react-repainting and came to your github. While checking repositories i saw this repository.
I STARTED PLAYING IT AND COULDN'T LEAVE IT UNFINISHED :D

IT TOOK ME AN HOUR :D

I'm glad you liked it, but the one in the image is a demo of another project. The real game is this
https://paol-imi.github.io/the-aviator/

The last level should take you more than one hour :)


feat: generate json schema for YAML and JSON configurations

Problem
IntelliSense works in VS code only on the .graphql files because of .tailcallrc.graphql. We propose generating a JSON-schema file so that YML and JSON also get a great developer experience.
Proposal
Autogenerate the json schema file which is equivalent to .tailcallrc.graphql.
Technical

Use the macro API to derive the auto-completions
Lint existing YML and JSON configurations in the project using the JSON Schema
Document how to use the IntelliSense features in VSCode and other IDEs.

/bounty 150$

/attempt #890

can you assign this to me


[2.93.2] Error when loading RIB archive

Hi,
The delayed RIB archive doesn't work for me, tested with several versions of Blender and the plugin.
I find a way to make it work though, rman_translator.py around line 128 :
            if rman_type in [
                    'DELAYED_LOAD_ARCHIVE',
                    'ALEMBIC',
                    'PROCEDURAL_RUN_PROGRAM',
                    'DYNAMIC_LOAD_DSO'
                ]:
                procprimid = hashlib.sha1(rman_sg_node.db_name.encode()).hexdigest()
                attrs.SetString('user:procprimid', procprimid)

user:procprimid seems to expect a string attribute, not a float (with RenderMan 24.1 anyway)
Best regards

I've pushed a fix for this. Let me know if it doesn't work for you.

It does work, thank you


fix: Drop domain name in mentions if chat members are on the same domain

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/32534
PROPOSAL: https://github.com/Expensify/App/issues/32534#issuecomment-1845888229
Tests

Create a room with 4 members: 2 on the same private domain, 1 on another private domain, and 1 on a public domain
As each member, send a message mentioning each other member
Verify that:

For email-based mentions:

Look at the email address in the mention
If it's on the same private domain as mine, render as @person. Otherwise render as<EMAIL_ADDRESS>Verify the tooltips(web/desktop) are correct
Click on the mention to go the profile page, verify  the informations are correct

For accountID-based mentions:

Look at the personalDetails object for the accountID
If we don't have a personalDetail for that accountID, render as @Hidden
If we have a personalDetail, but we don't have a display name or login in there, render as @Hidden
If we have a personalDetail with display name, but no login, render as @Display Name
If we have a personalDetail with login, render the same way as email-based mentions
Verify the tooltips(web/desktop) are correct
Click on the mention to go the profile page, verify  the informations are correct

[x] Verify that no errors appear in the JS console

Offline tests

Same as above
QA Steps

Create a room with 4 members: 2 on the same private domain, 1 on another private domain, and 1 on a public domain
As each member, send a message mentioning each other member
Verify that:

For email-based mentions:

Look at the email address in the mention
If it's on the same private domain as mine, render as @person. Otherwise render as<EMAIL_ADDRESS>Verify the tooltips(web/desktop) are correct
Click on the mention to go the profile page, verify  the informations are correct

For accountID-based mentions:

Look at the personalDetails object for the accountID
If we don't have a personalDetail for that accountID, render as @Hidden
If we have a personalDetail, but we don't have a display name or login in there, render as @Hidden
If we have a personalDetail with display name, but no login, render as @Display Name
If we have a personalDetail with login, render the same way as email-based mentions
Verify the tooltips(web/desktop) are correct
Click on the mention to go the profile page, verify  the informations are correct

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else

[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)

[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/113963320/94dd9c4b-a9b5-487c-a640-7ad8b1a7d7e3

Android: mWeb Chrome

https://github.com/Expensify/App/assets/113963320/2f533dc6-d888-4c93-b024-9634f5958efe

iOS: Native

https://github.com/Expensify/App/assets/113963320/1db0ef6a-51b9-4ada-b415-baf9eb1a1e35

iOS: mWeb Safari

https://github.com/Expensify/App/assets/113963320/9f7a95a2-9d1e-4f75-a5f2-d79461c8a841

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/113963320/ffaf00f7-cd56-4cd9-a3f5-de51e3a9dbac
https://github.com/Expensify/App/assets/113963320/13da9b58-2c55-4ddf-83f5-de036d2e4847

MacOS: Desktop

https://github.com/Expensify/App/assets/113963320/78170d55-ef89-4d37-8d8d-ef52841835f9

@tienifr 1) Even email section is Details page is not showing domain name.  2) Also, there is some issue in the own mention, the highlight color should be green. These issues were reproducible for newly created account. I cannot consistently reproduce the issue.

I am not sure if we want to remove the domain name while mentioning own user account, can you please confirm it in the issue?

@tienifr Can you please help me test different cases in Email-based Mention and Account ID-based Mention. I could only test accountID-based mention with email address. I am not sure how to test other cases.

@tienifr Were you able to reproduce these issues? Are they resolved?

@tienifr Were you able to reproduce these issues? Are they resolved?

Bump on this @tienifr

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/NikkiWines in version: 1.4.33-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
success ✅

@NikkiWines @francoisl  Would you be able to run this internally? This PR required multiple domains and we seem to be limited to Expensifail.com only at this point.

Looks like it works 🎊
@mvtglobally if this comes up in the future we should be able to use the applausecard.expensifail.com and expensifail.com accounts are different private domains and then any public domain email for the public domain test.

@NikkiWines Sure we will keep in mind regarding private domains. Just to be sure, it this tested internally?

Yep, this is all good! Though i'm not sure how to test the @hidden portion of the testing steps.

🚀 Deployed to production by https://github.com/francoisl in version: 1.4.33-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@NikkiWines Thanks! I was also not sure how to test. I used the account which does not have display name, but not showing as hidden in the chat.

Hmm yeah, I tried with a random account on a test domain and the behavior is quite weird - sometimes the tooltip doesn't match the (e.g. in the video below it shows<EMAIL_ADDRESS>as the tooltip for both users) and the intro blurb isn't correct. However, I'd be surprised if that behavior is tied to this since the mention seems to work fine (albeit I never got it to show @Hidden).
https://github.com/Expensify/App/assets/16747822/6eab8e77-ebe6-44ba-9043-9e73b68113ee
@tienifr can you confirm the steps to mention a @hidden user please?


feat: extend TO_CHAR to cover date, time and timestamp types

feat: extend TO_CHAR to cover date, time and timestamp field types
The change made for https://github.com/GoogleCloudPlatform/professional-services-data-validator/issues/623 only applies to Decimal and float values (see code changes below). Extending this to date, time and timestamp.
Sundar Mudupalli

Hi @sundar-mudupalli-work - Did you get a chance to test these changes?
Reason I am asking is - the function to_char in query_builder.py is returning NumericValue.to_char so I am kind of doubtful if it works for temporal data types
https://github.com/GoogleCloudPlatform/professional-services-data-validator/blob/c8daded621ace882e269c3a0c65e75d87cc50da1/data_validation/query_builder/query_builder.py#L313
@staticmethod
def to_char(config, fields):
fmt = ibis.literal(config.get("default_to_char_fmt", "FM90.099"))
return CalculatedField(
ibis.expr.api.NumericValue.to_char, config, fields, fmt=fmt
)

Yes, I definitely tested it, with the to_char success, without it failure
for date types.
I have it on a gcp vm, so we can both look at it together if you want. I am
at lunch at the moment.
Sundar Mudupalli
On Mon, Dec 5, 2022, 1:04 PM sramsrinivasan @.***>
wrote:

Hi @sundar-mudupalli-work https://github.com/sundar-mudupalli-work -
Did you get a chance to test these changes?
Reason I am asking is - the function to_char in query_builder.py is
returning NumericValue.to_char so I am kind of doubtful if it works for
temporal data types
https://github.com/GoogleCloudPlatform/professional-services-data-validator/blob/c8daded621ace882e269c3a0c65e75d87cc50da1/data_validation/query_builder/query_builder.py#L313
@staticmethod https://github.com/staticmethod
def to_char(config, fields):
fmt = ibis.literal(config.get("default_to_char_fmt", "FM90.099"))
return CalculatedField(
ibis.expr.api.NumericValue.to_char, config, fields, fmt=fmt
)
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/professional-services-data-validator/pull/641#issuecomment-1338159790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZK36WJNBQEB3KMSDES4TKTWLZKHDANCNFSM6AAAAAASUT6LRA
.
You are receiving this because you were mentioned.Message ID:
<GoogleCloudPlatform/professional-services-data-validator/pull/641/c1338159790
@github.com>

/gcbrun


Hit ip forwarding problem from macOS to container in v18.09.0 while v18.06.1-ce works well

1 Environment Description
Operation System : macOS majove version 10.14.1
Docker created by: "docker-machine" and "docker" CLI, NOT "docker for Mac" app.
2 Reproducing steps in v18.09.0

Create a docker machine "tryIt", command is:
docker-machine create tryIt --virtualbox-cpu-count "2" --virtualbox-memory "4096" --virtualbox-disk-size "100000"
In this step, hit problem #1347, just workaround it. And successfully created.

Create a common ubuntu container, now the situation is:
macOS------vboxnet1------<IP_ADDRESS>
tryIt---------docker0-------<IP_ADDRESS>
tryIt---------eth1-----------<IP_ADDRESS>
container----eth0----------<IP_ADDRESS>

Try to ping container(<IP_ADDRESS>) from macOS(<IP_ADDRESS>) with adding a route
Failed here in v18.09.0
Ping successfully in v18.06.1-ce with exactly the same above steps

3 Other information
With command cat /etc/issue, found that:
v18.06.1-ce is based on "Core Linux"
v18.09.0 is based on "TinyCoreLinux 8.2.1"

I'm not totally sure why this didn't exhibit sooner, but this appears to have been changed in https://github.com/moby/moby/pull/28257 (back in 2016), and is only now actually exhibiting in boot2docker for some reason: (this is comparing iptables from 18.06 to 18.09)
...
-Chain FORWARD (policy ACCEPT 3 packets, 232 bytes)
+Chain FORWARD (policy DROP 11 packets, 744 bytes)
...

Full iptables diff:
--- iptables-18.06.txt	2018-11-28 13:35:31.000000000 -0800
+++ iptables-18.09.txt	2018-11-28 13:39:03.000000000 -0800
@@ -1,16 +1,16 @@
-Chain INPUT (policy ACCEPT 2233 packets, 5158K bytes)
+Chain INPUT (policy ACCEPT 1721 packets, 5130K bytes)
  pkts bytes target     prot opt in     out     source               destination

-Chain FORWARD (policy ACCEPT 3 packets, 232 bytes)
+Chain FORWARD (policy DROP 11 packets, 744 bytes)
  pkts bytes target     prot opt in     out     source               destination
-   20  1361 DOCKER-USER  all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
-   20  1361 DOCKER-ISOLATION-STAGE-1  all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
-    8   515 ACCEPT     all  --  *      docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0            ctstate RELATED,ESTABLISHED
-    3   232 DOCKER     all  --  *      docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0
-    9   614 ACCEPT     all  --  docker0 !docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0
+   11   744 DOCKER-USER  all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
+   11   744 DOCKER-ISOLATION-STAGE-1  all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
+    0     0 ACCEPT     all  --  *      docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0            ctstate RELATED,ESTABLISHED
+   11   744 DOCKER     all  --  *      docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0
+    0     0 ACCEPT     all  --  docker0 !docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0
     0     0 ACCEPT     all  --  docker0 docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0

-Chain OUTPUT (policy ACCEPT 1396 packets, 130K bytes)
+Chain OUTPUT (policy ACCEPT 915 packets, 98123 bytes)
  pkts bytes target     prot opt in     out     source               destination

 Chain DOCKER (1 references)
@@ -18,29 +18,29 @@

 Chain DOCKER-ISOLATION-STAGE-1 (1 references)
  pkts bytes target     prot opt in     out     source               destination
-    9   614 DOCKER-ISOLATION-STAGE-2  all  --  docker0 !docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0
-   20  1361 RETURN     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
+    0     0 DOCKER-ISOLATION-STAGE-2  all  --  docker0 !docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0
+   11   744 RETURN     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0

 Chain DOCKER-ISOLATION-STAGE-2 (1 references)
  pkts bytes target     prot opt in     out     source               destination
     0     0 DROP       all  --  *      docker0  <IP_ADDRESS>/0            <IP_ADDRESS>/0
-    9   614 RETURN     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
+    0     0 RETURN     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0

 Chain DOCKER-USER (1 references)
  pkts bytes target     prot opt in     out     source               destination
-   20  1361 RETURN     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
-Chain PREROUTING (policy ACCEPT 15 packets, 880 bytes)
+   11   744 RETURN     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
+Chain PREROUTING (policy ACCEPT 20 packets, 1240 bytes)
  pkts bytes target     prot opt in     out     source               destination
-   12   648 DOCKER     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0            ADDRTYPE match dst-type LOCAL
+    9   496 DOCKER     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0            ADDRTYPE match dst-type LOCAL

-Chain INPUT (policy ACCEPT 12 packets, 648 bytes)
+Chain INPUT (policy ACCEPT 9 packets, 496 bytes)
  pkts bytes target     prot opt in     out     source               destination

-Chain OUTPUT (policy ACCEPT 64 packets, 4584 bytes)
+Chain OUTPUT (policy ACCEPT 32 packets, 2132 bytes)
  pkts bytes target     prot opt in     out     source               destination
     0     0 DOCKER     all  --  *      *       <IP_ADDRESS>/0           !<IP_ADDRESS>/8          ADDRTYPE match dst-type LOCAL

-Chain POSTROUTING (policy ACCEPT 67 packets, 4816 bytes)
+Chain POSTROUTING (policy ACCEPT 32 packets, 2132 bytes)
  pkts bytes target     prot opt in     out     source               destination
     0     0 MASQUERADE  all  --  *      !docker0  <IP_ADDRESS>/16        <IP_ADDRESS>/0

You should be able to "fix" this for your use case by enabling ip_forward explicitly before Docker starts via /var/lib/boot2docker/profile.
As for why this is exhibiting now and hasn't previously, the old boot2docker had https://github.com/boot2docker/boot2docker/commit/c2b60ab21080d15148948fc6536078251dcfa842#diff-5d40f3f9d874ac036eafd3052a2a1d31 in /etc/sysctl.conf, applied at boot, which is a really bizarre ancient relic from before Docker applying those necessary bits automatically (and in a much more secure-by-default way):
net.ipv4.ip_forward=1
net.ipv6.conf.all.forwarding=1

So, I'm sorry that this broke your workflow, but the workaround should be relatively simple (sysctl net.ipv4.ip_forward=1 in /var/lib/boot2docker/profile) and this is honestly a bad enough security issue that it's tempting to go backwards and remove all old boot2docker ISOs (but I don't honestly think that would really fix anything -- folks would simply complain they're gone). :scream:
(Yet another reason why we so strongly discourage use of boot2docker in any sort of production environment. :disappointed:)

Thanks @tianon
Actually, without any change, value of  net.ipv4.ip_forward is already 1 by default in v18.09.0.
The crux is about the changes of iptables you mentioned.
Check "sudo iptables  -L" in v18.09.0, it shows Chain FORWARD (policy DROP)
After executing sudo iptables -P FORWARD ACCEPT, I can ping the container from macOS.

Yep, that's as expected; go re-read through
https://github.com/moby/moby/pull/28257. 👍

@tianon
May I ask one more question that:  how to configure sudo iptables -P FORWARD ACCEPT into the virtual machine (here is "tryIt") ?
After trying, found files like: ~/.bashrc, ./.profile, /usr/local/etc/bashrc, /etc/profile will be restored after restart virtual machine. (Watched bootup log and get the idea why this happened)
/var/lib/boot2docker/profile will not be restored, but it doesn't run command during bootup.
So, is there any convenient method to allow a user to add a command running during bootup ?
Thanks in advance.


Create patch/content/primary branch and add primary content

e.g. avatar, name, job, contacts, intro, etc.

Experience

Recent

2018

Amongst many fighters of the past was featured in the movie "Ready Player One". Ryu ready always!

2017

United with unknown until then to me Power Rangers to defeat resurrected M. Bison. Damn good fighters they are, even not knowing karate ways!

2016

In "Street Fighter V" once and for all erased M. Bison from earth and tamed my inner Evil Ryu. Also spent some time on a side-project, mini-series "Street Fighter: Resurrection".

Back then

2014

Filmed in a martial arts tutorial "Street Fighter: Assassin's Fist", revealing to audience forgotten art of Ansatsuken.

2012

The gods of intenet concluded me in the animated movie "Wreck-It Ralph", but rewarded me with a good drink for it!

2008

My evil alter ego sneaked out of control and became a boss in "Street Fighter IV".

2006

Borrowed my famous ripped gi to Soki in "Onimusha: Dawn of Dreams", gods informed me he needed that for the battle.

1998

Defeated M. Bison in "Street Fighter Alpha 2", bastard wanted to use my heroic body for his evil intentions, but retreated with shame. Also this year I encountered Marvel universe in "Marvel vs. Capcom: Clash of Super Heroes" fighter, loyal to my employer, Capcom, of course.

1997

Reunited with Ken, my friend and opponent in "Street Fighter III". Men's friendship works mysteriously. Also helped with Arika project "Street Fighter EX"

1996

Took care of villainous Akuma in "Street Fighter Alpha 2". As a freelance on "Super Puzzle Fighter II Turbo" tried myself in a new genre, puzzle games. Let the wrath of seven heavens come upon these malicious enigmas, never again!

1995

Kept fighting in "Street Fighter Alpha: Warriors' Dreams".

1994

Screened in a movie "Street Fighter", so that the batllesof past would not be forgotten by millenial generation.

1991

Was promoted to "Street Fighter II". Didn't stay there for long, celebrations and office dwelling is not for me!

1987

Started working for Capcom project "Street Fighter". My partner was Ken Masters, the boss' name was Sagat.

Publications

2011

The true and humble story of me under pseudonim Tyler Wilde in an article "The evolution of Ken and Ryu"

2003 – 2018

For 15 years I have published comic books with Udon Entertainment depicting my adventures, bitter lessons and great victories.

Achievements

#1 in "Top 25 Street Fighter Characters" by IGN, who also called me "fancy", "icon" and "a testament to the virtue of simplicity"

#2 in "Top 20 Street Fighter Characters of All Time" by GameDaily, same council found me worth of #6 place in "Top 25 Capcom Characters of All Time", which I proudly share with my partner Ken

I was named "5th Most Powerful Street Fighter Character" by Screen Rant-san

#71 in "Top 100 Heroes of All Time" by UGO Networks, also recognized #2 in their list of "Top 50 Street Fighter Characters"

In a survey of 4000 online matches for Super Street Fighter IV, I got to be the most popular character, with 16.6% of players choosing my side


Lynda Member only video download

I just see great feature in youtube-dl. it allow to download from lynda.com.
But if video are paid it give message "Video %s is only available for members' % video_id"
Would it be possible to get userName and password in shell and login and download those video ?

From another forum, I find the following method to download Lynda videos:
"One of my friends recommends me to use Allavsoft download Lynda course.
After have a try, it really works great and can batch download lynda course, videos to MP4, MOV, AVI, WMV, 3GP etc on Mac or Windows at http://www.allavsoft.com/how-to/lynda-video-downloader.html "


Fix working with proxy servers

When you try to install this module using proxy, install.js will silently fail, because any normal proxy server will decline request which does not mention scheme and return HTTP 400.
This commit fixes this, this way will work either using proxy, or not.

This PR is duplicates of #22 and I suggest to close.


add option to turn the axis environment of

I tried to implement an option to disable the axis environment.
I hope the PR is ok  and is actually usefull for other people :)

C:289, 0: Unnecessary parens after u'if' keyword (superfluous-parens)
C:297, 0: Unnecessary parens after u'if' keyword (superfluous-parens)

Thanks for the PR! I see that it optionally discards the axis environment. What's this good for again?

I recently wanted to have a plot, where i zoom in one part, for this i needed a groupplot, but wantet all the drawing logic to be in a .tex file not in the python script.
This only worked after removing the axis environment.
So I thought it would be nice to have the possibillity to disable the axis environment in matplotlib2tikz so i can rerun the python part without having to edit the resulting .tex file.
Minimal example:
In the latex file:
\begin{tikzpicture}
\begin{groupplot}[options]
  \nextgroupplot
    \input{path_to_plot1}
    \legend {bla},
    \draw [red,thick] (0,0.5+101) rectangle (0.5,2.5e+101);
  \nextgroupplot
    \input{path_to_plot2}
\end{groupplot}
\draw [thick,blue,->,shorten >=2pt,shorten <=2pt]
(group c1r1.east) -- (group c2r1.west);
\end{tikzpicture}

and in the plot files:
\definecolor{color1}{rgb}{1,0.498039215686275,0.0549019607843137}
...
\addplot [semithick, color0]
table {%
...
};

Alright, thanks for the explanation. I guess that's a valid use case.
Could you please rename 'axenv to 'axis environment'? Also, note that there is no testing for this functionality, so it's not guaranteed that people won't mess around with it in the future. Given the relative shortness of the code, I suppose the risk is manageable.

Thank you, I changed it.
Also on a completely different topic, would it be possible to wrap pipdated in something like
try:
    import pipdated
    if pipdated.needs_checking(__name__):
        print(pipdated.check(__name__, __version__))
except Exception:
    print("consider installing pipdated")

The reasoning is, one does not really need pipdated, when installing with a packagemanager as the package manager already checks for updates.

Let's open a new issue for the pipdated thing.

Ok, thank you for the fast merging process


Build Windows test fixtures

Goal
Build e2e test fixtures for Unity 2017 to 2020 on Windows in the CI pipeline.
Design
New steps are currently behind a block step as we don't yet have permanent Windows build servers in place.  There's also low value in only building the test fixtures - running the e2e tests will come shortly.
To run these new build steps, a Buildkite agent can be set up temporarily on a Windows development machine very easily and the block step manually released through the Buildkite UI.
Changeset
New pipeline steps as described.  Existing concurrency group settings removed as they are unnecessary - only the number of Unity agents for the number of licenses that we have will be stood up in the first place.  If there's an agent free, it will be licensed.
Testing
Tested with an agent running on my Windows development machine.

This looks good to me at the moment. Are we looking at merging this down and then making changes when new test systems are provisioned for us, or will this be parked for a bit?

Yes, I'd merge it now so it doesn't go stale and update as soon as we get the infrastructure.  If engineers want they will also be able to set up a Buildkite agent on their dev machine easily and run the tests.


Fix tf table init on settings load

Type of PR

[x] BUG
[ ] FEAT
[ ] MAINT
[ ] DOC

Is your Pull Request linked to an existing Issue or Pull Request?
Addresses the issue outlined here - https://github.com/moj-analytical-services/splink/issues/802.
Give a brief description for the solution you have provided
Adds support for two new methods:

_queue_term_frequency_tables
_materialise_tf_tables

_materialise_tf_tables
This method is the simpler of the two, calling compute_tf_table on a selection of input columns defined by the user or alternatively all columns requiring term frequency adjustments identified in the settings. This function always materialises the table (as this is the functionality implemented in compute_tf_table).
_queue_term_frequency_tables
This also does what it says on the tin, queueing term frequency tables for future execution. I've set this to only queue those tables identified in the settings object, as I don't foresee any circumstances where we would want to expand this to tables outside of the settings (our existing methods/functions all rely on tf tables found in concat_with_tf).
This method works both when:

No tf tables exist
One or more required tf tables exist

In the former case, we simply create the concat_with_tf table and then infer all of the required term frequencies from this.
In the latter, we queue up the previously generated term frequency tables (first_name_tf, for example) before queueing up any missing tables as SQL CTEs.
It's worth noting that this method can be made to work without the concat_with_tf materialisation step (if you look at my commit history this was originally the case. I've opted to add the mat step in as concat_with_tf is a core table used throughout the whole of splink and also allows future runs of _queue_term_frequency_tables to execute more quickly.
PR Checklist
I will update the docs tomorrow w/ the above change.

[ ] Added documentation for changes - I will update the docs with a few guides over the next week
[ ] Added feature to example notebooks or tutorial (if appropriate)
[x] Added tests (if appropriate)
[x] Made changes based off the latest version of Splink
[x] Run the linter

I'm trying to write an example script to test this and it occurs to me that it's not really possible to use this function how we might want, becaues it's not possible to instantiate the linker without any data.
So suppose we have some settings
Scenario 1:
User has an existing settings.json and want to compare_two_records.
You can't do this:
linker = DuckDBLinker([], "compare_two_records.json")

because:
TypeError: reduce() of empty iterable with no initial value
Do they have to provide some arbitrary data:
import pandas as pd

from splink.duckdb.linker import DuckDBLinker

record_1 = {
    "unique_id": 0,
    "first_name": "Julia ",
    "surname": None,
    "dob": "2015-10-29",
    "city": "London",
    "email"<EMAIL_ADDRESS>    "cluster": 0,
}

record_2 = {
    "unique_id": 1,
    "first_name": "Julia ",
    "surname": "Taylor",
    "dob": "2015-07-31",
    "city": "London",
    "email"<EMAIL_ADDRESS>    "cluster": 0,
}

linker = DuckDBLinker(pd.DataFrame([record_1]), "compare_two_records.json")

linker.compare_two_records(record_1, record_2).as_pandas_dataframe()

But now the term frequency tables are incorrect because they're derived from the fake data.

Here's the scripts I used to try this out:

import pandas as pd
from splink.duckdb.comparison_library import (
exact_match,
levenshtein_at_thresholds,
)
from splink.duckdb.linker import DuckDBLinker
df = pd.read_csv("./tests/datasets/fake_1000_from_splink_demos.csv")
settings = {
"probability_two_random_records_match": 0.01,
"link_type": "dedupe_only",
"blocking_rules_to_generate_predictions": [
"l.first_name = r.first_name",
"l.surname = r.surname",
],
"comparisons": [
levenshtein_at_thresholds("first_name", 2),
exact_match("surname"),
exact_match("dob"),
exact_match("city", term_frequency_adjustments=True),
exact_match("email"),
],
"retain_intermediate_calculation_columns": True,
"additional_columns_to_retain": ["cluster"],
"max_iterations": 10,
"em_convergence": 0.01,
}
linker = DuckDBLinker(df, settings, connection=":memory:")
linker.estimate_u_using_random_sampling(target_rows=1e6)
blocking_rule = "l.first_name = r.first_name and l.surname = r.surname"
linker.estimate_parameters_using_expectation_maximisation(blocking_rule)
blocking_rule = "l.dob = r.dob"
linker.estimate_parameters_using_expectation_maximisation(blocking_rule)
linker.save_model_to_json("compare_two_records.json")

then

import pandas as pd
from splink.duckdb.linker import DuckDBLinker
record_1 = {
"unique_id": 0,
"first_name": "Julia ",
"surname": None,
"dob": "2015-10-29",
"city": "London",
"email"<EMAIL_ADDRESS>"cluster": 0,
}
record_2 = {
"unique_id": 1,
"first_name": "Julia ",
"surname": "Taylor",
"dob": "2015-07-31",
"city": "London",
"email"<EMAIL_ADDRESS>"cluster": 0,
}
linker = DuckDBLinker(pd.DataFrame([record_1]), "compare_two_records.json")
linker.debug_mode = True
linker.compare_two_records(record_1, record_2).as_pandas_dataframe()

In terms of how the API should work, I think it should be more like this:
import pandas as pd

from splink.duckdb.linker import DuckDBLinker

linker = DuckDBLinker(input_table_or_tables=None, "compare_two_records.json")

record_1 = {"first_name": "Julia ", "surname": None, "dob": "2015-10-29", "city": "London", "email"<EMAIL_ADDRESS>"cluster": 0, }

record_2 = {"first_name": "Julia ", "surname": "Taylor", "dob": "2015-07-31", "city": "London", "email"<EMAIL_ADDRESS>"cluster": 0, }

linker.compare_two_records(record_1, record_2)

Closing for now, a bit unclear what the best way forward on this one is


feat: allow console reference from elements panel

This improvement consists in creating a console reference from the DevTools Elements panel as well.
Currently, typing $$el in the browser console will only display the component selected in the Augury ComponentsTree.
However, sometimes it may be simpler to get this component reference by clicking on its corresponding element in the native DevTools Elements panel.
Selecting a component from the Augury ComponentsTree still works as before. The current console reference is simply the last selection from both the Augury ComponentsTree and the DevTools Elements panel.

@yannickglt First off, thanks for your contribution to augury! We'll be discussing whether this is a desired feature for an upcoming release. One concern that I have after testing it a bit is that the highlighted element in the augury tab and the value of $$el can now be out of sync (for example if you select a different element in elements and then switch back to augury).

Hi, thanks for your response and for taking the time to discuss about this feature. I gonna check the unexpected behavior you are talking about and suggest a fix (embedded in this pull request)  soon.