Running a local terminal using Terminal: Create New Terminal (Local) fails if a local terminal is specified

Issue Type: Bug
When running Terminal: Create New Terminal (Local) in a WSL remote, it throws the error The terminal process failed to launch: Path to shell executable "\mnt\c\Windows\System32\WindowsPowerShell\v1.0\powershell.exe" does not exist. This seems strange, since for some reason it's overriding the local terminal with the one specified in terminal.integrated.profiles.linux.
If I remove the WSL Host profiles specified in terminal.integrated.profiles.linux, then this command correctly runs a local PowerShell with the alert ⚠ : This shell is running on your local machine, NOT on the connected remote machine
This seems to be an issue with the Terminal: Create New Terminal (Local) command, rather than with WSL as far as I can tell.
Extension version: 0.64.2
VS Code version: Code 1.65.2 (c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1, 2022-03-10T14:33:55.248Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8086K CPU @ 4.00GHz (12 x 4008)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.84GB (13.92GB free)

Process Argv
--crash-reporter-id 75d2bff3-b4ef-4cb2-9b19-e7eeeba2b392

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-8086K CPU @ 4.00GHz (12 x 4008)

Memory (System)
15.55GB (14.70GB free)

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vsclayoutctrt:30451275
dsvsc009:30452663
pythonvsnew555:30457759
vscscmwlcmc:30438804
vscgsvid2:30447481
pynewfile477:30450038

@Jademalo I can't reproduce this one, can you share all profile related settings you have set? (or just your whole settings file)

    "terminal.integrated.profiles.linux": {
        "PowerShell (WSL Host)": {
          "path": "/mnt/c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe",
          "icon": "terminal-powershell"
        },
        "Command Prompt (WSL Host)": {
            "path": "/mnt/c/Windows/System32/cmd.exe",
            "icon":"terminal-cmd"
          }
    },```

This is the relevant terminal stuff in settings. If I break `PowerShell (WSL Host)` by changing the filepath to be invalid, `Terminal: Create New Terminal (Local)` works correctly.
I don't recall adding the WSL host things myself, as far as I'm aware they were added when the addon was installed.

I have the exact same issue.
I have a dev container connected and running. A valid path in terminal.integrated.profiles.linux fails to start with this message:
The terminal process failed to launch: Path to shell executable "\bin\bash" does not exist.

I added an invalid path entry:
"terminal.integrated.defaultProfile.windows": "WSL",
"terminal.integrated.profiles.windows": {
        "WSL": {
            "path": [
                "C:\\WINDOWS\\System32\\wsl.exe"
            ],
            "args": [],
            "icon": "terminal-ubuntu-wsl"
        }
    },
"terminal.integrated.profiles.linux": {
   "bash":  {
      // NOTE invalid path
     "path": "/invalid/path/bash"
   }
}

And the local terminal (bash in my case) starts. (probably because I have the setting "terminal.integrated.shell.windows":"C:\\WINDOWS\\System32\\wsl.exe"). If I comment out the terminal.integrated.shell.windows setting, it still works, but it is a powershell shell that is started, even if I have  "terminal.integrated.defaultProfile.windows": "WSL",

I am having the same issue as well.

Same issue but on OSX.
Running in a linux devcontainer
default config (no manual configuration for terminal)
Terminal: Create New Terminal (Local) results in
The terminal process failed to launch: Path to shell executable "/usr/bin/zsh" does not exist.
which is correct, because zsh is at /bin/zsh
adding to settings
   "terminal.integrated.profiles.osx": {
        "zsh": {
            "path": "/bin/zsh"
        },
    },
    "terminal.integrated.defaultProfile.osx": "zsh"

does not change behaviour
adding to settings
    "terminal.integrated.profiles.linux": {
        "zsh": {
            "path": "/bin/zsh"
        },
    },
    "terminal.integrated.defaultProfile.osx": "zsh"

causes correct terminal to be opened
Extension version: v0.245.0
VS Code version: Code 1.70
OS version: macOS Monterey 12.4


Error: Request is missing the pull request id.

Describe the bug
I'm using the automatically generated SWA Gihub actions on pull request closing and am getting the following error:
The content server has rejected the request with: BadRequest
Reason: Request is missing the pull request id.

For further information, please visit the Azure Static Web Apps documentation at https://docs.microsoft.com/en-us/azure/static-web-apps/
If you believe this behavior is unexpected, please raise a GitHub issue at https://github.com/azure/static-web-apps/issues/
Exiting

My workflow file looks like:
name: Close Pull Request

on:
  push:
    branches:
      - main
  pull_request:
    types: [ closed ]
    branches:
      - main

jobs:
  close_pull_request_job:
    runs-on: ubuntu-latest
    name: Close Pull Request Job
    steps:
      - name: Close Pull Request
        id: closepullrequest
        uses: Azure/static-web-apps-deploy@v1
        with:
          azure_static_web_apps_api_token: ${{ secrets.MY_TOKEN }}
          action: "close"

The staging deployment is successfully deleted from Azure even though I receive the error.

Resolved.  It was actually a problem with which branch was running toe job


[BUG] Can't show in jupyter pdf

In jupyter notebook, the rich render correctly

But when I downloads the pdf via LaTex, it can't show

It also can't show in sphinx pdf (buecause it build by latex)

I think this is an issue for the Jupyter devs. Rich generates HTML according to the Jupyter spec, I would have expected Jupyter to include the output in the pdf.

It is hard to implement. HTML is hard to render in LaTeX file.

I don't think there is much Rich can do to fix this. You might want to report it to the Jupyter devs.


mpldatacursor and webagg

Dear Joe,
First of all, thank you so much for the last feedback about mpldatacursor on Jupyter notebook!
This time I am trying a bit different thing, which is using matplotlib webagg backend. It seems that the backend is a very nice one to build a web UI for matplotlib and I would like to use mpldatacursor with it, but when I naively put mpldatacursor.datacursor() in the source code of https://github.com/matplotlib/matplotlib/blob/master/examples/user_interfaces/embedding_webagg.py, that is,
def create_figure():
    """
    Creates a simple example figure.
    """
    fig = Figure()
    a = fig.add_subplot(111)
    t = np.arange(0.0, 3.0, 0.01)
    s = np.sin(2 * np.pi * t)
    a.plot(t, s)
    mpldatacursor.datacursor()
    return fig

It doesn't complain but nothing happens in the plot. Then I tried something different,
def create_figure():
    """
    Creates a simple example figure.
    """
    fig = Figure()
    a = fig.add_subplot(111)
    t = np.arange(0.0, 3.0, 0.01)
    s = np.sin(2 * np.pi * t)
    a.plot(t, s)
    mpldatacursor.datacursor(
        axes=fig.axes,
    )
    return fig

Then I get the following error:
Traceback (most recent call last):
  File "embedding_webagg.py", line 242, in <module>
    figure = create_figure()
  File "embedding_webagg.py", line 46, in create_figure
    axes=fig.axes,
  File "/home/chan/anaconda/lib/python2.7/site-packages/mpldatacursor/convenience.py", line 168, in datacursor
    return DataCursor(artists, **kwargs)
  File "/home/chan/anaconda/lib/python2.7/site-packages/mpldatacursor/datacursor.py", line 207, in __init__
    self.ax_timer[ax] = ax.figure.canvas.new_timer(interval=interval,
AttributeError: 'NoneType' object has no attribute 'new_timer'

Will it be a huge work to make mpldatacursor work with webagg? If that can be done then it would be really awesome!
Best,
Chan

Thanks for the bug report, and sorry for my delay in responding!
As for the first problem, any time the pyplot interface isn't used, there's no way to detect which figures/axes exist.  Therefore, anytime you bypass pyplot (e.g. embedding in a gui), you'll have to explicitly specify which axes or artists mpldatacursor should work with.  I should make this more clear in the documentation, though.
For the second part, it looks like timers aren't implemented at all for the webagg backend.  This is a bug in matplotlib, but I should be handling it in mpldatacursor regardless.  I've added a workaround in 7531fe287eb774bbeaed8a7e1c0371a41bd7d86b that should mask the problem.  You'll have slightly worse performance for backends that don't implement timers, but webagg is going to have poor performance regardless.
If you have a chance, try installing from git master and let me know if this fixes your problem (it does for me, but you may hit other issues).  I'll probably release another point release soon that will include this fix as well as a couple of others.

Thank you so much for the help! I will try it and get back to you. By the way, could you report to the matplotlib community that there is such a bug in webagg? It would be nice if webagg can grow into one of the standard backends.


Roadmap Requested

We need to have a roadmap on what will be there for version 0.6.0 or even 1.0.0. With out a roadmap, we will lose our focus.
Making oras-go stable is more and more important as it is getting more and more adopters.
Related issues:

#33

Closing as we have milestones.


Use plugin bat when installing on Windows

This commit updates the InstallPlugins task to use the plugin batch file as the binary when installing plugins. Previously,
plugin installation used cmd as the binary, delegating to the batch file as an argument. This can cause issues where a plugin
writes warnings to stderr, such as ingest-attachmeent, which triggers the InstallPlugins task to throw an exception.
Remove IsMono property and replace with IsWindows property, using RuntimeInformation. Ultimately, this property is used to determine
the platform running on, specifically whether Windows or not.

I'm going to merge this in, to unblock efforts on https://github.com/elastic/elasticsearch-net


fix(client-presence): Rename ISessionClient so that function/property naming is consistent

Description
ISessionClient functions/properties must be consistent. From what I can gather from API naming discussion, getStatus and getClientConnectionId is the preferred naming.
We also create a SessionClient class to get and set connection ID and connection status

Superseded by #22973


Bad request : Invalid wait time and multiple Get request while trying to acquire leader

Hi,
We are using this package (version <IP_ADDRESS>) for Leader election using distributed lock.
It is based on blog : https://codeblab.herokuapp.com/high-availability-and-leader-election-with-consul-c-part-1/
It works pretty fine (thanks for great work).
We have multiple servers with  our application (which using playfab nuget) + with consul.io agent on the same server and additional cluster of consul.io servers for high availability
Anyways, once in a few days we getting exception on Non leader application:
Consul.ConsulRequestException: Unexpected response, status code BadRequest: Invalid wait time
Also,
I see in consul agent logs:
2019/03/12 04:54:08 [DEBUG] http: Request GET /v1/kv/service/scm/leader?index=934475&wait=1.999s (1.9967478s) from=<IP_ADDRESS>:56777
2019/03/12 04:54:08 [DEBUG] http: Request GET /v1/kv/service/scm/leader?index=934475&wait=ms (0s) from=<IP_ADDRESS>:56777
Usually we try to acquire lock every 2 seconds, so we see every to seconds GET log with wait=...s..
But when problem happens we see 2 GETs when second has wait=ms, which causes that exception.
I checked playfab code,  public async Task Acquire(CancellationToken ct) method. but I didn't find good explanation why it happens? Some rare race condition?
10x in advance

It looks like the timeout there is getting formatted as ms, probably a bug in the code that converts a timespan to a go duration string. Check out https://github.com/PlayFab/consuldotnet/blob/master/Consul/Utilities/Extensions.cs if you want to take a crack at fixing it, but otherwise I'll look at it when I can.

Actually the best fix is in code:
class Lock
public async Task Acquire(CancellationToken ct)
Instead of   if (elapsed > qOpts.WaitTime)
it should be  if (elapsed >= qOpts.WaitTime)
actually it looks like the only place where qOpts.WaitTime is updated and could be 0.
So i would like to do pull request with that fix


Docs Refresh

:tada: This PR is included in version 1.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update testing.md

You can return the promise directly and not have to deal with the callback. Mocha will wait for the promise to resolve or reject.

Nice, thanks for pointing this out. I just used it to simplify the live-libs tests: https://github.com/ConsenSys/live-libs/commit/4f6d246b907ebf51d6476bf4ece0898a393f18a0

What, go mocha! I didn't know this either. Thanks!


ARROW-12310: [Java] ValueVector#getObject should support covariance for complex types

9964

 Quality gate: Failed
4 Issues

 4 Code issues

Details


Modify data model to use DB

All of the persistent data is stored in dictionaries and shared between modules. Use of a sqlite or postgres DB might be cleaner. Would make internet play more viable in a future release

Use a config file to use a sqlite DB for local play and a postgres DB for network play


One typo

Coverage remained the same at 95.12% when pulling 6027f2922144dfde90c57500516310c8d4c1fd7b on kthyng:patch-1 into 82c8f54d4c04e67dae751552d9fa55d9123b2b4b on richford:main.


⚠ TorrentLeech has degraded performance

In 2e69408, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 4304 ms

Resolved: TorrentLeech performance has improved in 4c37031 after 10 minutes.


Add  REDMINE_RMAGICK_FONT_PATH Parameter

#228 Add  REDMINE_RMAGICK_FONT_PATH Parameter to set rmagick_font_path for the png export function of GANTT to work

I'd also prefer if you squashed your two commits together as the first ones commit message is still confusing.

I want REDMINE_RMAGICK_FONT_PATH parameter, too.
But I have two things on my mind.

I don't know how to proceed PR which I didn't create
The latest Redmine uses minimagick

@jcormier
Would you let me know what I have to do to proceed this PR or how to accomplish is other ways?

I want REDMINE_RMAGICK_FONT_PATH parameter, too.
But I have two things on my mind.

I don't know how to proceed PR which I didn't create
The latest Redmine uses minimagick
Do you know if this changes the config option?

@jcormier
Would you let me know what I have to do to proceed this PR or how to accomplish is other ways?

Hi @urimaro, I'm not familiar with the rmagick vs minimagick config.  But I'll try to help.
If you can provide me with instructions on how to test if the font path is working, I can look into updating this pull request and merging it.  Does that help?

Setting this path isn't mentioned in the redmine install guide. https://www.redmine.org/projects/redmine/wiki/RedmineInstall

@jcormier
Thanks for your reply.
I try to update this PR.

Any updates?

I'm sorry for no response.
I'm working on this but I have not been able to have enough time these days.
I'd appreciate it if you could wait with your warm eyes.


Inline demos that don't use JavaScript, still need an empty

I wondered why my inline demo didn't work:
{{<demo>}}

Great stuff

Just some text

    More…

{{</demo>}}

Then I figured out I needed to add an empty :
{{<demo>}}

Great stuff

Just some text

    More…

{{</demo>}}

Perhaps you can make it so that this is not possible, since not all inline demos will require JavaScript to work.

@roelvangils Yep, that was an oversight. Will take a look at fixing ASAP.

@roelvangils Fixed with b7d0f50


netlify addons:create fauna Internal Server Error

- Do you want to request a feature or report a bug?
bug
- What is the current behavior?
I run netlify addons:create fauna in a new project and it shows an error wit the following output:
Creating addon
Error: Error {"error":"{/"message/": /"Internal server error/"}"}
    at createAddon (~/.npm-packages/lib/node_modules/netlify-cli/node_modules/netlify/src/addons.js:24:11)

I looked into addons.js:
async function createAddon(settings, netlifyApiToken) {
...
if (response.status === 422) {
    throw new Error(`Error ${JSON.stringify(data)}`)
  }

return data

- If the current behavior is a bug, please provide the steps to reproduce.
Just run netlify addons:create fauna
- What is the expected behavior?
That it successfully creates fauna
- Local Environment Information
netlify-cli/2.11.15 darwin-x64 node-v10.18.1

I was able to run netlify addons:create fauna successfully. Maybe this was temporary down?
Closing as stale. Please comment if still relevant


DBSCAN code move, notebook, benchmark

README fix to add DBSCAN to supported algo

Notebook added in

DBSCAN benchmark script support

Move DBSCAN to spark_rapids_ml.clustering

build

Overall looks good. minor comments and questions.

build

build

build

build


Support video files

Add support for video:video elements

https://developers.google.com/search/docs/advanced/sitemaps/video-sitemaps


Text composables: Android Basics with Compose - setter does not work in main

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-classes-and-objects?authuser=1&continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-2-pathway-1%3Fauthuser%3D1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-classes-and-objects#4
In which task and step of the codelab can this issue be found?
step 5
Describe the problem
I define the setter like described:
var speakerVolume = 2
set(value) {
if (value in 0..100) {
field = value
}
}
but if I use the setter in main like
smartTvDevice.set(5)
I get an error message
Unresolved reference. None of the following candidates is applicable because of receiver type mismatch: public inline operator fun kotlin.text.StringBuilder /* = java.lang.StringBuilder */.set(index: Int, value: Char): Unit defined in kotlin.text
I tried as well to define a variable of type Int and set that as well as I defined the type of  speakerVolume : Int
But that did not work neither
Steps to reproduce?

Go to...
Click on...
See error...

Versions
kotlinplayground from developer.android.com
Additional information
class SmartDevice{
val name = "Android TV"
var speakerVolume = 2
set(value) {
    if (value in 0..100) {
        field = value
    }
}

fun turnOn(){
    println("Smart Device $name is turned on with Speaker volume $speakerVolume")
}

}
fun main() {
val smartTvDevice = SmartDevice()
val setVolumeto : Int = 5
smartTvDevice.turnOn()
println("This was a message from ${smartTvDevice.name}")
println("The volume is set to ${smartTvDevice.speakerVolume} and not to $setVolumeto")
smartTvDevice.set(setVolumeto)
smartTvDevice.turnOn()
println("This was a message from ${smartTvDevice.name}")
}

I learned that the getter and setter are implicit and it does not work to invoke them explicitly.

Hi @developerpeet
Glad you were able to solve your own issue. That is an important step to learn as a developer.
If you are still experiencing any issues with the course, please do not hesitate to create a new issue.
Hope you are enjoying the course.
Regards.


Getting error when building Apache Drill

I am getting the following error when building Apache Drill: docker build -t apache-drill .
Step 2/8 : RUN apt-get update && apt-get install -y base-files lsb-release lsb-base
 ---> Running in 604d2852db53
Get:1 http://security.debian.org jessie/updates InRelease [44.9 kB]
Ign http://deb.debian.org jessie InRelease
Ign http://deb.debian.org jessie-updates InRelease
Ign http://deb.debian.org jessie-backports InRelease
Get:2 http://deb.debian.org jessie Release.gpg [2420 B]
Ign http://deb.debian.org jessie-updates Release.gpg
Ign http://deb.debian.org jessie-backports Release.gpg
Get:3 http://deb.debian.org jessie Release [148 kB]
Ign http://deb.debian.org jessie-updates Release
Get:4 http://security.debian.org jessie/updates/main amd64 Packages [824 kB]
Ign http://deb.debian.org jessie-backports Release
Get:5 http://deb.debian.org jessie/main amd64 Packages [9098 kB]
Err http://deb.debian.org jessie-updates/main amd64 Packages

Err http://deb.debian.org jessie-updates/main amd64 Packages
  404  Not Found
Err http://deb.debian.org jessie-backports/main amd64 Packages
  404  Not Found
Fetched 10.1 MB in 6s (1559 kB/s)
W: Failed to fetch http://deb.debian.org/debian/dists/jessie-updates/main/binary-amd64/Packages  404  Not Found

W: Failed to fetch http://deb.debian.org/debian/dists/jessie-backports/main/binary-amd64/Packages  404  Not Found

E: Some index files failed to download. They have been ignored, or old ones used instead.
The command '/bin/sh -c apt-get update && apt-get install -y base-files lsb-release lsb-base' returned a non-zero code: 100

At the moment I fixed it by using openjdk:8-jdk as base image instead of java:openjdk-8-jdk

It's a general issue with Ubuntu Jessie release.
Created issue here
https://github.com/docker-library/docs/issues/1457

https://stackoverflow.com/a/55411551/363455


Add Bulk Data Load

Bulk data loading should be added to the CLI.

Is it a known issue that the force bulk query retrieve [job id] [batch id] (return the data from the query) only returns the first result file, and not any subsequent files? It seems like integrating the Bulk API may still be a WIP, so I didn't want to open another issue if this is already known.

Open the issue, I don't think we have support for multiple result files.
Big gap I think.
On Wed, Feb 18, 2015 at 9:35 AM, mickle00<EMAIL_ADDRESS>wrote:

Is it a known issue that the force bulk query retrieve [job id] [batch
id] (return the data from the query) only returns the first result file,
and not any subsequent files? It seems like integrating the Bulk API may
still be a WIP, so I didn't want to open another issue if this is already
known.
—
Reply to this email directly or view it on GitHub
https://github.com/heroku/force/issues/59#issuecomment-74908599.

--
Thanks,
Dave

Closing this out since bulk api support has been implemented.  We can open new issues for further enhancements or bug fixes.


How to retrieve items from a cart?

I have added several products in the cart. But I want to get the list of all added products back during a confirmation stage. Kindly help me to get array of all the added products in the cart

@cart.shopping_cart_items.map(&:item) should do the trick, right?

no, I have tried this. It gives me no method defined error

@zaingz can you provide some sample? tks!

i'm facing the same issue
@shopping_cart.shopping_cart_items.map { |i| i.item.store.name }
and i'm getting: undefined method `shopping_cart_items' for nil:NilClass
thx!

   items = @shopping_cart.shopping_cart_items#.map(&:item)     items.each do |item|        puts "item id is"	        puts	item.item_id        puts "item quantity is"        puts item.quantity     end


Add section for debugging slow and cpu intensive queries

In our debugging docs we can add a section explaining how to debug slow queries by appending SQL queries with EXPLAIN ANALYZE (for PostgreSQL) and studying the query execution plan.
We can describe similar techniques for other databases as well.

We now have a product - Optimize to provide these recommendations. We can close this.


Fix a typo in the aws_s3_bucket.rb resource

See also https://github.com/rapid7/convection/commit/782208968f#diff-7126ecd8a0f873d6bdeffcaf269fb9a7R15 where this typo was introduced.
cc @jmanero-r7 @athompson-r7 @simonirwin-r7 @abegley-r7

@jmanero-r7 @bturner-r7 @abegley-r7 as discussed adhoc — d24cfaf resolves the issue of any previous uses of the method.


Sample for Go using gRPC response service fail

I trying to deploy a service using go and grpc. I am using this tutorial https://github.com/GoogleCloudPlatform/golang-samples/tree/master/endpoints/getting-started-grpc and everything was complete successful until I have to call the service.
In the moment that I use the client to call the service I get this error:
could not greet: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:80: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.

Hi @SantyTaco. Were you running the server locally, on GCE, or GKE? I tried it locally and on GCE and it worked.

@SantyTaco, if you were running on GCE, would you mind trying with the changes from #845?

Closing this as it's been ~1 month. Thank you for filing this issue and hopefully you got it working!


zk: set main.Version build flag

Description of changes

Things done
Added -X=main.Version to ldflags so zk --version is accurate.
Previously, zk --version would just show the default value of dev

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 319124 run on x86_64-linux 1

1 package built:

zk


How to show full column data while exporting

I'm having product name in listing and will greater than 255 characters. While showing in the list used str_limit function to sub string.
While exporting the data I need full product name details.

You can set protected $exportColumns = [] array and manually list the columns to be exported.

It is possible to change column name? For example set my_long_column as column1
Thanks


Week 3 - Backend Development Review Ticket

fastpages blog w/ links

[x] Lesson on ifs and expressions
[x] Deployment on AWS
[x] Focus / Habits

Crossover Grade: 2.7/3. Fulfilled all the requirements. Basic if else without an input. Deployed Java Website. Focus/Habits was minimal but fulfilled requirements.


One click Open

What feature or improvement do you think would benefit Files?
Please make this avail for Details view/layout as it's only avail for Columns right now. Tx
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z

Files Version
<IP_ADDRESS>
Windows Version
10.0.22000.2538
Comments
No response

Thanks for the feedback, Files already has this in Files settings > Files & Folders > Open items > Open items with single click.


Preview features window displaying underneath

Describe the bug
When clicking preview features it isn't showing above the settings panel
To Reproduce
Steps to reproduce the behavior:

Go to the right side profile settings selector
Click on preview features

Expected behavior
The module to appear above, so it can be manipulated
Screenshots
Above
Desktop (please complete the following information):

OS: MacOS Sonoma Version 14.2 Beta
Browser: Chrome
Version: 115.0.5790.170

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

I'd say clicking on "Preview Features" should close the user menu

cc @gustavlrsn

It's happening for news and updates as well

And on Safari

I'd say clicking on "Preview Features" should close the user menu

Agree, opened issue here: https://github.com/opencollective/opencollective/issues/7143


response is not returned

In ipython:
    In [13]: restkit.__version__
    Out[13]: '3.3.2'

    In [11]: restkit.Resource("http://<IP_ADDRESS>:9037/render").get(rawData='true',target='keepLastValue(system.house.temp.livingRoom)', **{'from':'-60minutes'}).body_string()
    Out[11]: ''

strace shows this, which includes the output I was expecting:
    connect(3, {sa_family=AF_INET, sin_port=htons(9037), sin_addr=inet_addr("<IP_ADDRESS>")}, 16) = 0
    sendto(3, "GET /render?from=-60minutes&target=keepLastValue%28system.house.temp.livingRoom%29&rawData=true HTTP/1.1\r\nHost: <IP_ADDRESS>:9037\r\nUser-Agent: restkit/3.3.2\r\nAccept-Encoding: identity\r\n\r\n", 183, 0, NULL, 0) = 183
    recvfrom(3, "HTTP/1.1 200 OK\r\nServer: gunicorn/0.10.0\r\nDate: Fri, 25 Nov 2011 09:36:23 GMT\r\nConnection: keep-alive\r\nVary: Accept-Encoding\r\nContent-Type: text/plain\r\n\r\nkeepLastValue(system.house.temp.livingRoom),1322210280,1322213880,120|66.7625,66.65,66.5375,66.425,66.3125,66.2,65.4125,66.0875,65.975,66.0875,67.6625,68.3375,69.125,68.9,68.7875,69.6875,69.6875,69.2375,69.575,69.575,69.6875,68.7875,67.775,68.225,67.8875,67.6625,67.6625,67.55,67.2125,66.9875\n", 8192, 0, NULL, NULL) = 447

github keeps mangling my description. See it at http://pastebin.com/LPWygczL

what did you get ? Anything that could help me to reproduce ?

It's kind of unbelievable how much github is eating my text, even in a triple-backtick block. I have misspelled http to htp to make it work better.
I tried proxying that resource to a public url for a repro case, but that makes it start working.

In [5]: restkit.Resource("htp://<IP_ADDRESS>:9037/render").get(rawData='true', target='keepLastValue(system.house.temp.livingRoom)', **{'from':'-60minutes'}).body_string()
Out[5]: ''

In [6]: restkit.Resource("htp://bigasterisk.com/restkit78/render").get(rawData='true',target='keepLastValue(system.house.temp.livingRoom)', **{'from':'-60minutes'}).body_string()
Out[6]: 'keepLastValue(system.house.temp.livingRoom),1322275680,1322279280,120|70.8125,70.8125,70.7,70.7,70.5875,70.7,70.5875,70.5875,69.6875,70.7,70.7,70.7,70.5875,70.5875,70.7,70.7,70.1375,70.5875,70.475,70.475,70.475,70.3625,70.3625,70.3625,70.3625,69.8,70.25,70.1375,69.0125,69.0125\n'

With curl, both addresses are fine:

curl "htp://bigasterisk.com/restkit78/render?from=-60minutes&target=keepLastValue%28system.house.temp.livingRoom%29&rawData=true"
keepLastValue(system.house.temp.livingRoom),1322275680,1322279280,120|70.8125,70.8125,70.7,70.7,70.5875,70.7,70.5875,70.5875,69.6875,70.7,70.7,70.7,70.5875,70.5875,70.7,70.7,70.1375,70.5875,70.475,70.475,70.475,70.3625,70.3625,70.3625,70.3625,69.8,70.25,70.1375,69.0125,70.1375

curl "htp://<IP_ADDRESS>:9037/render?from=-60minutes&target=keepLastValue%28system.house.temp.livingRoom%29&rawData=true"
keepLastValue(system.house.temp.livingRoom),1322275680,1322279280,120|70.8125,70.8125,70.7,70.7,70.5875,70.7,70.5875,70.5875,69.6875,70.7,70.7,70.7,70.5875,70.5875,70.7,70.7,70.1375,70.5875,70.475,70.475,70.475,70.3625,70.3625,70.3625,70.3625,69.8,70.25,70.1375,69.0125,70.1375

In case it helps, here's the response in the broken case:
In [2]: restkit.Resource("htp://<IP_ADDRESS>:9037/render").get(rawData='true',target='keepLastValue(system.house.temp.livingRoom)', **{'from':'-60minutes'}).__dict__
Out[2]:
{'_already_read': False,
 '_body': <_io.BufferedReader>,
 '_closed': False,
 '_resp': <http_parser.http.HttpStream object at 0x21efa10>,
 'connection': <restkit.conn.Connection object at 0x21ef990>,
 'final_url': 'htp://<IP_ADDRESS>:9037/render?from=-60minutes&target=keepLastValue%28system.house.temp.livingRoom%29&rawData=true',
 'headers': IOrderedDict([('Server', 'gunicorn/0.10.0'), ('Date', 'Sat, 26 Nov 2011 03:48:40 GMT'), ('Connection', 'keep-alive'), ('Vary', 'Accept-Encoding'), ('Content-Type', 'text/plain')]),
 'headerslist': [('Server', 'gunicorn/0.10.0'),
                 ('Date', 'Sat, 26 Nov 2011 03:48:40 GMT'),
                 ('Connection', 'keep-alive'),
                 ('Vary', 'Accept-Encoding'),
                 ('Content-Type', 'text/plain')],
 'location': None,
 'request': <restkit.wrappers.Request object at 0x21ef950>,
 'should_close': False,
 'status': '200 OK',
 'status_int': 200,
 'version': (1, 1)}

should be fixed in 4.0 .


Fix misuse of overly verbose log levels and unify use of log and tracing

This uses the appropriate log levels for our logging.
The recommendation is:
info is surfaced to, and used as a means to communicate with the users, debug is intended for lower priority and implementation oriented messages, and trace is for very verbose and detailed information, most commonly includes debugged structs, or very implementation specific information that is almost entirely useful for internal devs and never users.
Additionally, this means the prefixed log filtering that were a common source of surprising behavior has been almost entirely removed. The only remaining targets are external crates that don't use logging appropriately.
The split usage of log, and its superset replacement tracing has been fixed, and tracing is used throughout. Tracing has lot of more useful features, and is especially good at capturing spatially temporal information through the use of spans.
Tracing also has support for further detailed and structured logging, async, and much more, and is what is the new standard in the wider rust ecosystem.
Fixes: #584
Fixes: #459

Old

Note: this uses our meticulously crafted specifc filtering rules that makes enabling additional logs very inconvenient as you can't differentiate between logs
New

Update:
A slightly more compact format

Additionally, there is a feature flag tracing-tree cargo run --features tracing-tree -- run ... that makes it much clearer which logs come from which scope and how they are grouped both spatially and temporally.

I mean, just look at this 😁

This is so useful for debugging contexts and knowing how logs related to each other

This <unknown time> is fixed as well


Log fixes

A bunch of log fixes.

@lxbsz  please review.
It is probably easiest to just apply all the patches and review it, because there is so much churn with the patchset.

@mikechristie
Your changes look handsome :-).
I test it and all work well. Except the above changes all looks good to me and I do not reject that changes,.
Thanks.


FlowRampManager feature?

is there any docs mention flowRamp ? can anyone tell me anything about it?

are you linkedin user?
kai xu

From: xianyouQ<EMAIL_ADDRESS>Sent: Monday, June 29, 2020 5:06 AM
To: azkaban/azkaban<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [azkaban/azkaban] FlowRampManager feature? (#2563)
is there any docs mention flowRamp ? can anyone tell me anything about it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fazkaban%2Fazkaban%2Fissues%2F2563&data=02|01|kxu%40linkedin.com|56bdf3510a7e430c0c3208d81c24de41|72f988bf86f141af91ab2d7cd011db47|1|0|637290291990668494&sdata=9h5s4S2Fh3P6T3KlZvQfDBqoHqlzNQdJ9WPU1%2B%2FT8pk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABRXIAMPDMZ5CUNCFE2XDOTRZB7UPANCNFSM4OLENLFQ&data=02|01|kxu%40linkedin.com|56bdf3510a7e430c0c3208d81c24de41|72f988bf86f141af91ab2d7cd011db47|1|0|637290291990668494&sdata=CLftZIXZ01%2FFP6Xi96N1OYHDntfA4%2F8nBSFnF5%2Bvh7Q%3D&reserved=0.


Replay with additional feature

The replay mode is a nice feature, where you can watch the game moves of the last game later. But it still could be improved by an additional mode, with which it is possible, to repeat the game with the same cards but making new moves. So you could compare different game moves with the same cards and better learn from your mistakes.

This is a very nice idea! Should be not too complicated to implement for local games.


Fix integration test on linux

Its still unclear to me why this issue only presents when running on linux, maybe the helm process returns faster. Waiting for cert-manager to start + 10 secs fixes it though, waiting for cert-manager alone still results in failures. The bottom issue is some form of deadlock on the tls-secret object, where cert-manager is unable to update because the object was modified while cert-manager was running.
Other cleanup:
Create namespace was unnecessary.
process.StandardError.ReadLineAsync() was occasionally throwing exceptions, process.OutputDataReceived += new DataReceivedEventHandler(OutputHandler) seems to be better.

Thanks for digging deep on this one!


Ability to disable/enable spellchek with a command

Is there a way I can disable/enable spellcheck of an already opened file?
I can run setlocal aspell.check on/off but that seems to behave rather odd - it only starts happening when I change the buffer.
I guess adding a custom command that does the above + calls spellcheck(bp.Buf) might work. I can try submitting a PR if this would be acceptable.

yeah, I would merge that. This has come up before https://github.com/priner/micro-aspell-plugin/issues/11#issue-925647899 but that one doesn't work correctly with aspell.check = auto.

Well, my idea was not a toggle command, but a disable/enable commands. So it would override auto I guess. A toggle might be nice, but harder.

Oh, I see b9e43770bc9a4b86cc092b4546c87c5c441b9cb0 -- thanks!


We may soon be able to drop all mamba-related info from our install docs

Proposed documentation enhancement
The mamba solver will become the default solver for conda soon, hence no need for ordinary users to explicitly touch mamba anymore:

The upcoming, special 23.10.0 release will be dedicated to the switch of the default solver to libmamba.

https://github.com/conda/conda/releases/tag/23.9.0

It will be nice to purge that language!  I think we should still say something like "if you have problems with the "solving environment" step failing or being slow, make sure you're on conda version 23.10.0 or newer".

Mamba is / will be deprecated as the solver is now fully integrated into conda. See this PR:
https://github.com/mamba-org/mamba/pull/3054
This means that we should recommend users install "conda >=23.10" and remove all references to mamba from the docs.
For CI, we should switch from Mambaforge to Miniforge.
cc @larsoner @cbrnr

The mamba CLI is / will be deprecated as the solver is now fully integrated into conda. See this PR:

That's not quite what is happening. Mamba is still a first class citizen. What is happening is mamba.py which was glue between conda and libmamba has been removed in favor of a full C++ CLI client.

@AntoinePrv since the Micromamba CLI is not compatible with Mamba, we are in fact deprecating Mamba.
I think what you call "mamba.py" in your comment is typically called "Mamba", and what you call "Mamba" is typically called "libmamba". IMO we should stick to the following terminology:

Mamba: The Conda-compatible Python CLI that will be deprecated.
Micromamba: The C++ binary based on libmamba.
libmamba: The C++ library.
libmambapy: The Python binding to the C++ library.

With this terminology, "Mamba" will be deprecated.

Yes this is what I meant! The mamba binary is deprecated. The mamba solver library and micromamba binary are not. Thanks for the clarification, @jonashaag

Yes, except that we will ship a mamba executable that uses the same code as micromama. The name will remain.

Ah okay, this is good to know!

@0xbe7a as in ln -s micromamba mamba (and micromamba is a conda-compatible frontend, we are interested in pairing any reported missing feature)

Based on git grep I think we're good here already!


'center_size' is not defined in output_utils.py

The center_size function is needed when 'preserve_aspect_ratio' is set to True.
This requires the center_size in the output_utils which is not defined currently.

Oh yeah huh. However, preserve_aspect_ratio is deprecated, so it has more problems than just that. See #51 for how to fix.

@dbolya is this dependent on the input resolution of the custom dataset since the output resolution i want to test is on is 2K resolution

oh yeah pixel scales are definitely dependent on resolution. Make sure to convert from relative coordinates to absolute coordinates (by multiplying by the image size) before calculating the area.

Since this has been open so long, I'm going to close it. Feel free to reopen if you have any updates.


df-chat-enhance: cannot update chat archive name

Module
df-chat-enhance 3.0.1
Describe the issue
chat archives name changed do not save permanently
To Reproduce
Steps to reproduce the behavior:

Go to 'Manage chat archive'
Edit name of archive and save
Close chat archive display
Chat Archive Manager does not display updated name

Expected behavior
Once saved, chat archive manager should retain saved name update.
Screenshots (Optional)

I noticed today you also can't update the visibility checkbox either. It doesn't save.


fix background overflow bug

removed cellbody drawbackgroud function to prevent backgroud overflow when scaling cellview.

@waizui Thanks very much for this pull request! But I think the cell's background should be painted correctly. If we remove the code it will remains an empty white space when cell's rendering. So I'm going to close this and find another way to fix this.


Datafile Model

Create a sequelize model to store the zip file uploads. The model should store the username, and the file. The model should be created when the file upload UI #11 finished the upload.

It is redundant to upload a geospatial file twice into the DB. We should do the file checking on the fileUploadController instead.

Re-opening this issue. Based on what we talked, the file uploader #11 should save a DB object with the following:

A time stamp
The user that uploaded the file
The filename
The location of the unzipped file folder in the db

I'll create the model, then we can just use it in the file uploader.

Added the dataFile model and the one-to-many relationship to users


⚠ Spryr has degraded performance

In 60d5725, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 10270 ms

Resolved: Spryr performance has improved in 6e5d52f after 9 minutes.


feature/databricks-sql-warehouse-compatibility

PR Overview
This PR will address the following Issue/Feature: Issue #120
This PR will result in the following new package version: v1.7.1

This will not impact existing users who are not using Databricks SQL Warehouse runtimes. A SQL Warehouse runtime user will never have seen success. Therefore, this fix is not breaking and should ensure they may now see success.
Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:

Bug Fixes

Users leveraging the Databricks SQL Warehouse runtime were previously unable to run the fivetran_platform__audit_table model due to an incompatible incremental strategy. As such, the following updates have been made:

A new macro is_databricks_sql_warehouse() has been added to determine if a databricks runtime is a SQL Warehouse runtime for Databricks. This macro will return a boolean of true if the runtime is determined to be SQL Warehouse and false if it is any other runtime or destination.
The above macro is used in determining the incremental strategy within the fivetran_platform__audit_table. For Databricks SQL Warehouses, there will be no incremental strategy used. All other destination runtime strategies are not impacted with this change.

For the SQL Warehouse runtime, the best incremental strategy we could elect to use is the merge strategy. However, we do not have full confidence in the resulting data integrity of the output model when leveraging this strategy. Therefore, we opted for the model to replicate a full create or replace behavior for the time being.

Under the Hood

Added integration testing pipeline for Databricks SQL Warehouse.
Applied modifications to the integration testing pipeline to account for jobs being run on both Databricks All Purpose Cluster and SQL Warehouse runtimes.

PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:

[X] dbt run –full-refresh && dbt test
[X] dbt run (if incremental models are present) && dbt test

Before marking this PR as "ready for review" the following have been applied:

[X] The appropriate issue has been linked, tagged, and properly assigned
[X] All necessary documentation and version upgrades have been applied
[X] docs were regenerated (unless this PR does not include any code or yml updates)
[X] BuildKite integration tests are passing
[X] Detailed validation steps have been provided below

Detailed Validation
Please share any and all of your validation steps:

For validating these changes I wanted to ensure the following in relation to the fivetran_platform__audit_table model:

No incremental strategy is being used for SQL Warehouse destinations
The incremental strategy created for all other warehouses is unchanged

See the validations below:

SQL Warehouse doesn't have any incremental strategy associated with it. You can see there is no incremental strategy used as the incremental run compiled code does not include the where statement that would identify the incremental strategy is being used.

I was then able to verify for all other warehouses that the incremental strategy was working as expected. You can see the compiled code for all the warehouses is returning the expected results.

BigQuery

Databricks (All Purpose Cluster)

I have been unable to run this locally. However, we can see from buildkite that this looks to be using the incremental strategy as expected. We can see this in the finished statement where it mentions there was 1 incremental model executed.

Postgres

Snowflake

Redshift

SQL Server

I am unable to run this locally, but same as Databricks (All Purpose Clusters) we are able to see the incremental model is being captured in the BuildKite run

If you had to summarize this PR in an emoji, which would it be?

🧱

Thanks @fivetran-catfritz! I agree with your suggestion and committed it to the branch. I also really appreciate your README updates 🙏. I made one small wording change to be consistent with the terminology of the runtime name (Datarbricks All Purpose Cluster). Lastly, I regenerated the docs as the incremental file format change needed to be picked up in the docs.
Let me know if there are any other comments needed before approving.


Application is Scaling up but not Scaling down

Hi Team,
I am unable to scale down the Instance, even if it has reached the minimum memory utilization.
Please find attached policy file
policy_file.txt
Also current
cf autoscaling-metrics bigapp2 memoryutil

Metrics Name            Instance Index          Value           At
memoryutil              0                       5%              2018-07-30T10:31:09Z
memoryutil              1                       3%              2018-07-30T10:31:23Z
memoryutil              2                       2%              2018-07-30T10:31:23Z
memoryutil              0                       5%              2018-07-30T10:31:24Z
memoryutil              3                       2%              2018-07-30T10:31:24Z
memoryutil              1                       3%              2018-07-30T10:31:38Z
memoryutil              2                       2%              2018-07-30T10:31:38Z
memoryutil              0                       5%              2018-07-30T10:31:39Z
memoryutil              3                       2%              2018-07-30T10:31:39Z
memoryutil              1                       3%              2018-07-30T10:31:53Z
memoryutil              2                       2%              2018-07-30T10:31:53Z
memoryutil              0                       5%              2018-07-30T10:31:54Z
memoryutil              3                       2%              2018-07-30T10:31:54Z

The Instance seems to be still in count of 3 and not Scaling down.

@ashwithahn , according to your policy definition, the scale-in will happen when the average memory util is <= 2%.
Looking at your instance metric below on 2018-07-30T10:31:38 ~ 2018-07-30T10:31:39.
memoryutil              1                       3%              2018-07-30T10:31:38Z
memoryutil              2                       2%              2018-07-30T10:31:38Z
memoryutil              0                       5%              2018-07-30T10:31:39Z
memoryutil              3                       2%              2018-07-30T10:31:39Z
The avg memoryutil of the 4 instances are (3% + 2 % + 5% + 2% )/4 = 3% .  So, app-autoscaler won't trigger scale-in under this situation.

@cdlliuy : Thank you very much :) This is resolved. I would like to close.


Library for Backends

Hi Nate,
just a quick question: would this library be useful for writing transpilers/backends for PureScript?
Thank you,
Michael

This library only parses the surface exact syntax of PureScript. I generally expect it to be used for syntactic tooling (eg. I used it to write purescript-tidy and purescript-tidy-codegen). Most backends consume the compilers corefn target, which is what you get after typechecking and elaboration.

Ok! Got it! Thanks a lot for clarification!


Op:BatchMatMul is not supported

Hi,
I am working on a project which has an Op of BatchMatMul, but it seems not to be supported by OpenVINO so far. Do you plan to support it in the future release or is there any alternative Op I can use to workaround it currently?

hi @xhuan28
have you fix the problem? have you create sumtom layers or ops?
I have the same problem, if you fixed it, please tell me how do you do, thank you.

Dear @WilliamZhaoz
If it's Tensorflow, MatMul will not be supported til 2019R4 (likely the end of the year-ish timeframe). I'm sorry about this.
Is it Tensorflow MatMul that you're after ?
Thanks,
Shubha

Hi shubha-ramani,
yes, it is an op in Tensorflow.
so before the new version, the only way fix it is I have to write the
custom layer?
That can work, a custom layer, yes?
shubha-ramani<EMAIL_ADDRESS>于2019年10月1日周二 上午2:37写道：

Dear @WilliamZhaoz https://github.com/WilliamZhaoz
If it's Tensorflow, MatMul will not be supported til 2019R4 (likely the
end of the year-ish timeframe). I'm sorry about this.
Is it Tensorflow MatMul that you're after ?
Thanks,
Shubha
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opencv/dldt/issues/80?email_source=notifications&email_token=AIWSN2IYIVHQBTG4VIJKE73QMJBO5A5CNFSM4GSBF7HKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD76U34Q#issuecomment-536694258,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIWSN2IEDYYGRZ6PZIMGUK3QMJBO5ANCNFSM4GSBF7HA
.

@WilliamZhaoz
Yes of course. You can write a Custom layer for MatMul in the meantime.  Here is some information regarding Custom layers:
http://docs.openvinotoolkit.org/latest/_docs_MO_DG_prepare_model_customize_model_optimizer_Extending_Model_Optimizer_with_New_Primitives.html
http://docs.openvinotoolkit.org/latest/_docs_IE_DG_Integrate_your_kernels_into_IE.html

Okay, thank you shubha-ramani.
shubha-ramani<EMAIL_ADDRESS>于2019年10月8日周二 上午1:09写道：

@WilliamZhaoz https://github.com/WilliamZhaoz
Yes of course. You can write a Custom layer for MatMul in the meantime.
Here is some information regarding Custom layers:
http://docs.openvinotoolkit.org/latest/_docs_MO_DG_prepare_model_customize_model_optimizer_Extending_Model_Optimizer_with_New_Primitives.html
http://docs.openvinotoolkit.org/latest/_docs_IE_DG_Integrate_your_kernels_into_IE.html
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opencv/dldt/issues/80?email_source=notifications&email_token=AIWSN2OLVRLFLWREHUGHSWDQNNUMHA5CNFSM4GSBF7HKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEARDDYA#issuecomment-539111904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIWSN2IRCW36TNJG3L76GSDQNNUMHANCNFSM4GSBF7HA
.

@xhuan28 the OpenVINO support for BatchMatMul and MatMul has been added. Could you take the latest OpenVINO package or compile OpenVINO from sources and verify?

It seems that the issues is not actual anymore as no response. Closing it. Feel free to reopen it or create a new one.


Fix empty Zammad field in region form

Short description
Fix handling empty Zammad URL in region form. We allowed Null/None values for the zammad_url field in #3051 but forgot to consider this in the form sanitization.
Proposed changes

Check if the zammad_url field in the form is empty before doing string operations. If empty, return None (Null for database).

Side effects

I cannot think of any.

Resolved issues

Fixes: #3098

Pull Request Review Guidelines

Ah, one thing I forgot. I think it makes sense to write a test for region creation as we encounter issues with it every once in a while. Maybe we can make this part of this PR depending on how urgent it is


Can LDAP Search return the Group id of LDAP users ?

I tried this and it was working fine with haproxy. but is there a way where the binary can return the ldap group details of the ldap user for which the ldap search is being done ? so that we can differentiate between authenticated ldap users and take different action for different users ?

I am closing this.


How do I get access to the Private Preview version, so i can use -MembershipType and -Owner?

How do I get access to the Private Preview version, so i can use -MembershipType and -Owner?

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

ID: fd44b411-4b59-b90f-0b82-ad0a87652854
Version Independent ID: 2c52cc4e-47a1-be0d-8860-1011adf061fd
Content: New-TeamChannel (teams)
Content Source: teams/teams-ps/teams/New-TeamChannel.md
Service: teams-powershell
GitHub Login: @kenwith
Microsoft Alias: kenwith

@jimcantor - I believe @LanaChin has documented the process here: https://docs.microsoft.com/en-us/microsoftteams/private-channels-life-cycle-management

thanks!  wonderful news
Regards, Jim Cantor
<EMAIL_ADDRESS>0414 593 723
On Thu, 21 Nov 2019 at 09:42, Ken Withee<EMAIL_ADDRESS>wrote:

Closed #4414
https://github.com/MicrosoftDocs/office-docs-powershell/issues/4414.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/office-docs-powershell/issues/4414?email_source=notifications&email_token=ACNQYROMZI7NKD6T3IB57GDQUW4OZA5CNFSM4JP2IQR2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOU7URIVI#event-2817070165,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNQYRPZSTRMK2YQSB23METQUW4OZANCNFSM4JP2IQRQ
.

.


ML: Update to LangChain 0.1.1, and GA release of cratedb-toolkit 0.0.3

About
Adjust dependencies to reflect updates to LangChain 0.1.1, and cratedb-toolkit 0.0.3.

Deprecation Warnings
LangChain
LangChainDeprecationWarning: Importing (chat message histories|document loaders|vector stores) from langchain is deprecated. Importing from langchain will no longer be supported as of langchain==0.2.0. Please import from langchain-community instead:
from langchain_community.chat_message_histories import CrateDBChatMessageHistory.
from langchain_community.document_loaders import CrateDBLoader.
from langchain_community.document_loaders import UnstructuredURLLoader
To install langchain-community run pip install -U langchain-community.
Miscellaneous
test.py::test_notebook[vector_search.ipynb]
/opt/hostedtoolcache/Python/3.11.7/x64/lib/python3.11/site-packages/pytest_notebook/plugin.py:273: PytestRemovedIn8Warning: The (fspath: py.path.local) argument to JupyterNbCollector is deprecated. Please use the (path: pathlib.Path) argument instead.
See https://docs.pytest.org/en/latest/deprecations.html#fspath-argument-for-node-constructors-replaced-with-pathlib-path
return JupyterNbCollector.from_parent(parent, fspath=path)
test.py::test_notebook[conversational_memory.ipynb]
/opt/hostedtoolcache/Python/3.11.7/x64/lib/python3.11/site-packages/nbformat/init.py:93: MissingIDFieldWarning: Code cell is missing an id field, this will become a hard error in future nbformat versions. You may want to use normalize() on your notebooks before validations (available since nbformat 5.1.4). Previous versions of nbformat are fixing this issue transparently, and will stop doing so in the future.
validate(nb)
test.py::test_file[vector_search.py]
/opt/hostedtoolcache/Python/3.11.7/x64/lib/python3.11/site-packages/langchain_core/_api/deprecation.py:117: LangChainDeprecationWarning: The class langchain_community.embeddings.openai.OpenAIEmbeddings was deprecated in langchain-community 0.1.0 and will be removed in 0.2.0. An updated version of the class exists in the langchain-openai package and should be used instead. To use it run pip install -U langchain-openai and import as from langchain_openai import OpenAIEmbeddings.
warn_deprecated(
test.py::test_file[vector_search.py]
test.py::test_file[vector_search.py]
/opt/hostedtoolcache/Python/3.11.7/x64/lib/python3.11/site-packages/langchain_community/embeddings/openai.py:500: PydanticDeprecatedSince20: The dict method is deprecated; use model_dump instead. Deprecated in Pydantic V2.0 to be removed in V3.0. See Pydantic V2 Migration Guide at https://errors.pydantic.dev/2.5/migration/
response = response.dict()
test.py::test_file[vector_search.py]
test.py::test_file[vector_search.py]
/opt/hostedtoolcache/Python/3.11.7/x64/lib/python3.11/site-packages/pydantic/main.py:979: PydanticDeprecatedSince20: The dict method is deprecated; use model_dump instead. Deprecated in Pydantic V2.0 to be removed in V3.0. See Pydantic V2 Migration Guide at https://errors.pydantic.dev/2.5/migration/
warnings.warn('The dict method is deprecated; use model_dump instead.', DeprecationWarning)


KNN Base model

I have added kNN base model file please let me know if there are any changes I should do
@ashwani-rathee
issue #26

@Abheeshth Can you provide a example for testing in the comment,will be helpful for soemone using this later

@Abheeshth Can you provide a example for testing in the comment,will be helpful for someone using this later

Sure I will do that as soon as possible

I have  provided a example for testing in the comments. please have a look


Update COLT 2022 & Correct IJCAI 2022

Which conference does this PR update?

COLT 2022
Correct deadline timezone of IJCAI 2022

Related URL

https://learningtheory.org/colt2022/index.html
https://learningtheory.org/colt2022/cfp.html
https://ijcai-22.org/calls-papers/

LGTM


Memory manager errors when using FormLinearSystem

When forming the linear system using FormLinearSystem methods; objects such as X/B  may have different lifespans than x/b. The different lifespans can cause errors within the memory manager when X/B are destroyed before x/b.
Also: The verify at mem_manager.cpp:1255 needs to allow d_mt == MemoryType::DEFAULT

Addressed in #2203


[Update] Refactor setOverrides() call and IPFS URL resolution

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
The focus of this PR is to refactor the codebase to centralize and optimize the execution of setOverrides function on app load.
Detailed summary

Moved setOverrides function calls to centralize execution on app load
Added useEffect hooks to ensure setOverrides runs once on app load
Updated replaceIpfsUrl function to use resolveScheme with thirdwebClient
Imported setOverrides from vercel-utils in multiple files

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4587  👈
#4586
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on  Graphite


Ensure uptime monitoring is set for all sub-services used in production

Sub-services listed here:
https://github.com/biolink/biolink-api/blob/master/conf/config.yaml
However, only a subset currently used for production purposes:

http://golr.berkeleybop.org - GO function
https://solr.monarchinitiative.org/solr/golr - homology
scigraph-dev - ID mapping (see #108)

We should have either/or

periodic execution of behave tests, with a warm body to check if something goes wrong
uptime robot checks on all 3 sub-services, as well as api.mi itself

I'd actually expand to all of:

uptime robot checks on all 3 sub-services, as well as api.mi itself (setting this up now)
periodic execution of behave tests, with a warm body to check if something goes wrong (periodically run in Jenkins with email on failure, or have uptimerobot  look at the results)
nagios and greylog scanning for likely problems (harder and we'd need some bandwidth to do it, but important for catching some problem classes before outages occur)

I think number 1 for this ticket, at least, and then planning items for the other two.

tagging @jmcmurry for coordination of warm bodies

Okay, a first pass is done. Currently:

all warnings are fired to chris or chris and myself (in the case of BBOP GOlr dev--enjoy that extra chatter, Chris ;)
A beginning status page can be found here: https://stats.uptimerobot.com/Z44RnHOYG
added at 15min frequency (the second and third already being used for AGR):

http://golr.berkeleybop.org/solr/select?q=*:*&rows=0
https://api.monarchinitiative.org/api/bioentity/gene/NCBIGene:3630/homologs/?homology_type=O&fetch_objects=false ("ENSMUSG00000000215")
https://api.monarchinitiative.org/api/bioentityset/slimmer/function?&slim=GO:0003824&slim=GO:0004872&slim=GO:0005102&slim=GO:0005215&slim=GO:0005198&slim=GO:0008092&slim=GO:0003677&slim=GO:0003723&slim=GO:0001071&slim=GO:0036094&slim=GO:0046872&slim=GO:0030246&slim=GO:0003674&slim=GO:0008283&slim=GO:0071840&slim=GO:0051179&slim=GO:0032502&slim=GO:0000003&slim=GO:0002376&slim=GO:0050877&slim=GO:0050896&slim=GO:0023052&slim=GO:0010467&slim=GO:0019538&slim=GO:0006259&slim=GO:0044281&slim=GO:0050789&slim=GO:0042592&slim=GO:0007610&slim=GO:0008150&slim=GO:0005576&slim=GO:0005737&slim=GO:0005856&slim=GO:0005739&slim=GO:0005634&slim=GO:0005694&slim=GO:0016020&slim=GO:0031982&slim=GO:0071944&slim=GO:0030054&slim=GO:0042995&slim=GO:0032991&slim=GO:0045202&slim=GO:0005575&subject=SGD:S000002976 ("provided_by")
https://solr.monarchinitiative.org/solr/golr/select/?defType=edismax&qt=standard&indent=on&wt=json&rows=25&start=0&fl=*,score&facet=true&facet.mincount=1&facet.sort=count&json.nl=arrarr&facet.limit=25&facet.method=enum&fq=object_category:"gene"&fq=relation_closure:"RO:HOM0000001"&fq=subject_closure:"NCBIGene:23543"&q=*:* ("subject_gene_label":"RBFOX2")
https://scigraph-data-dev.monarchinitiative.org/scigraph/graph/neighbors?id=GO%3A0022008&depth=1&blankNodes=false&relationshipType=subClassOf&direction=BOTH&entail=false ("lbl":"cell differentiation")

TODO:

a list of people to be notified when the various services are down
additional tests to be sent to me
for status.monarchinitiative.org, add CNAME to stats.uptimerobot.com.


Patch 1

Removed the depreciate package
Remove the old meteor coding practice (NPM.require)
Added Prerequisite to clear the usage of third party package.

@dr-dimitru @amos-whitewolf @jeremywrnr can you guys please review this.

lgtm 👍

I currently can't verify due to an upcoming milestone but it looks good.


Problem with Gmx version check

https://github.com/bioexcel/biobb_md/blob/2844840ba23e7da6607d4022f9c3919d37e5bb79/biobb_md/gromacs/mdrun.py#L207-L211
When using mpi_bin and not using containers (e.g. in any supercomputer with gromacs installed), the code snippet checking gromacs version is crashing because self.gmx_version is not initialized (see lines 122 and 123). https://github.com/bioexcel/biobb_md/blob/2844840ba23e7da6607d4022f9c3919d37e5bb79/biobb_md/gromacs/mdrun.py#L122-L123
Possible solution is to just add and not self.mpi_binto the line 208.

Solved in 3.7.1


Private folder upload failure

[write something useful and descriptive here]
my log id: 50dd35f9ed5cd4becdb1651c

@maxtaco: looks like yet another identify timeout


STP - materialization is broken in latest version

EGL1: All _sp requests returns with X-CM-WELL-SG-RS : 422 when running STP Agent with local source (EGL1)
Works when configuring to work with PPE
Works when sending separate _sp to EGL1 (with fiddler)

Turned out to be an issue with missing namespace prefixes in egl1

Correct


Support for all transaction types, all common fields, and multi-sign

Summary
This change introduces a major rewrite with support for all transaction types, all common fields, and multi-sign. Along with the rewrite we have been required to make changes to the transport protocol used to communicate between a client and the device. These protocol changes are implemented in our fork of ledgerjs, and we are ready to open another pull request there whenever you find appropriate.
The changes introduced in this rewrite fixes the following issues:

https://github.com/LedgerHQ/ledger-app-xrp/issues/1
https://github.com/LedgerHQ/ledger-app-xrp/issues/2
https://github.com/LedgerHQ/ledger-app-xrp/issues/3
https://github.com/LedgerHQ/ledgerjs/issues/175
https://github.com/LedgerHQ/ledgerjs/issues/216
https://github.com/LedgerHQ/ledgerjs/issues/141
https://github.com/LedgerHQ/ledgerjs/issues/301

Additional details are available in the updated README.
How to test
We have prepared an external test suite containing tests for all of the supported transaction types. You can find this test suite along with usage instructions in our repository TowoLabs/ledger-tests-xrp.

Most commits in this PR were already present in LedgerHQ, only the new ones from 61dccb6 have been merged.


gccgo: internal compiler error: in mangled_name

Update
I change package domain from 9fans.net to fans.net, it can build without any error.
I think package domain can't start with number in gccgo.

gccgo 8.2.0 work well in most case at my SunOS 5.10/SPARC mechine.
But compiler error when I trying to build 9fans.net/go/draw
I try gccgo in Windows WSL, and the error can be reproduced.
What version of Go are you using (go version)?
go1.10.3 gccgo (GCC) 8.2.0 solaris/sparc
go1.10.3 gccgo (GCC) 8.2.0 linux/amd64(in windows WSL)
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
SunOS 5.10/SPARC
windows10 WSL(ubuntu)/amd64
What did you do?
cd $GOPATH/src/9fans.net/go/draw
go build

What did you expect to see?
success build
What did you see instead?
# 9fans.net/go/draw/drawfcall
go1: internal compiler error: in mangled_name, at go/gofrontend/names.cc:539
0x668549 Type::mangled_name[abi:cxx11](Gogo*) const
        ../../gcc-8.2.0/gcc/go/gofrontend/names.cc:539
0x710b8a Gogo::specific_type_function_names(Type const*, Named_type const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >*)
        ../../gcc-8.2.0/gcc/go/gofrontend/names.cc:290
0x74f897 Type::specific_type_functions(Gogo*, Named_type*, long, Function_type*, Function_type*, Named_object**, Named_object**)
        ../../gcc-8.2.0/gcc/go/gofrontend/types.cc:2014
0x74e395 Type::type_functions(Gogo*, Named_type*, Function_type*, Function_type*, Named_object**, Named_object**)
        ../../gcc-8.2.0/gcc/go/gofrontend/types.cc:1946
0x6e9a02 Specific_type_functions::type(Type*)
        ../../gcc-8.2.0/gcc/go/gofrontend/gogo.cc:2424
0x7378d2 Type::traverse(Type*, Traverse*)
        ../../gcc-8.2.0/gcc/go/gofrontend/types.cc:318
0x73a324 Forward_declaration_type::do_traverse(Traverse*)
        ../../gcc-8.2.0/gcc/go/gofrontend/types.cc:12059
0x7378e8 Type::traverse(Type*, Traverse*)
        ../../gcc-8.2.0/gcc/go/gofrontend/types.cc:326
0x6e9988 Specific_type_functions::type(Type*)
        ../../gcc-8.2.0/gcc/go/gofrontend/gogo.cc:2447
0x7378d2 Type::traverse(Type*, Traverse*)
        ../../gcc-8.2.0/gcc/go/gofrontend/types.cc:318
0x6ecedc Bindings::traverse(Traverse*, bool)
        ../../gcc-8.2.0/gcc/go/gofrontend/gogo.cc:7950
0x6ed101 Gogo::traverse(Traverse*)
        ../../gcc-8.2.0/gcc/go/gofrontend/gogo.cc:2498
0x6ed197 Gogo::write_specific_type_functions()
        ../../gcc-8.2.0/gcc/go/gofrontend/gogo.cc:2474
0x6e876a go_parse_input_files(char const**, unsigned int, bool, bool)
        ../../gcc-8.2.0/gcc/go/gofrontend/go.cc:162
0x6e456f go_langhook_parse_file
        ../../gcc-8.2.0/gcc/go/go-lang.c:329
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See <https://gcc.gnu.org/bugs/> for instructions.

@thanm The problem here is that the pkgpath starts with a digit.  Should we tweak the encoding scheme even farther by saying that hex-encode a leading digit in a name?

I'll take a look.

This may take a bit of work to fix -- this particular assert is easy to get rid of, but there are unfortunately a fair number of other places where the compiler forms symbol names by starting with "." and then tacking on the package path, which is problematic given the mangled type encoding machinery.


Manually Process Host Application On Error

Resolve https://github.com/opencollective/opencollective/issues/6408
test-osc.webm

After "reviewing the implications quietly", I came to the conclusion that it was better solved by https://github.com/opencollective/opencollective-api/pull/8619


fix pkg-config files

apparently some programs expect libdir to be a fully qualified path

@googlebot I signed it!

wait this isn't why it's failing in imagemagick static. it's the -R stuff in brotli!


Get tables to inherit font-size from body

Avoids problems when browsers thrown into quirksmode.

Closing due to impending deprecation


docker type: no recommendations produced

The current prometheus query for cpu, rate(container_cpu_usage_seconds_total{name=~"acmeair-mono-app1"}[1m]) does not return actual cpu usage data, resulting in not getting any recommendations at all.
Memory OTOH shows the correct actual usage, but still no recommendations produced.

Reproduced this even after using sum. Seems like some code changes needed, taking a closer look


Update zinc version to 1.5.8

Update zinc versionto 1.5.8
Output a warning message when using Scala 2.12.4
Change used scala version in test

Converted to draft. This PR was incomplete, but no longer necessary for next bump to zinc 1.5.9, as they reverted their chnage which introduced the issues. But later, when zinc 1.6.x comes out, we may still need this tweak, hence I keep this PR open for later use.


Side bar and status bar flickers when Code window maximized

VSCode Version: Code 1.6.1 (9e4e44c19e393803e2b05fe2323cf4ed7e36880e, 2016-10-13T15:43:46.054Z)
OS Version: Linux x64 4.4.0-43-generic
Extensions: DotJoshJohnson.xml-1.6.0, EditorConfig.EditorConfig-0.3.3, abusaidm.html-snippets-0.0.17, dbaeumer.vscode-eslint-1.0.8, donjayamanne.python-0.4.2, eg2.tslint-0.5.39, johnpapa.Angular2-1.0.2, ms-vscode.cpptools-0.9.2, ms-vscode.csharp-1.4.1, msjsdiag.debugger-for-chrome-1.2.0, rebornix.Ruby-0.9.0, robertohuertasm.vscode-icons-3.0.4

I'm on Ubuntu 64-bit 16.04 LTS with a multi-screen setup: VGA (22'') and DVI (27'') from a Lenovo x240. If the code window is maximized I get occasional flickering in the side bar and the status bar. This only happens when the VGA connected screen is active. Also in full screen (F11) or when the Code-window is not maximized the flickering does not seem to appear. Other programs are not affected by similar flickering either as it seems. I use the standard window manager for this version of Ubuntu. I did not have similar problems before today when I started to use the multi-screen setup.
Some hardware details
Intel® Core™ i7-4600U CPU @ 2.10GHz × 4 , 8GB Ram
Intel® Haswell Mobile
I realize that this probably is related to other things than Code, but the manifestation of these problems could at least be noted.
Steps to Reproduce:

Maximize window on a multi-screen setup
Observe occasional flickering

I think the flickering is gone when I use the --force-gpu-rasterization flag. After reporting the issue I noticed some of the same flickering issues in Chrome they also disappeared there with this flag.


Update config system to use a object rather than a module.

Implementing https://github.com/PyPSA/atlite/issues/45

The config system now uses a config object, located in the module "config".
The parent class is atlite.config.Config, access to the configuration is now achieved via atlite.config.config .
Getters and setters are implemented, the callback function for update hooks is mostly working, except for when the object atlite.config.config is overwritten directly with a new Config object.
Since we can now use getters and setters, manual calls for resolving the (relative) file path are obsolete and now automatically handled by the config object.
I could not figure out two problems related to direct overwrites of the atlite.config.config object, i.e. by
atlite.config.config = Config()

How to call the _UPDATE_HOOKS (listener pattern on module variable)
How to use the config in other modules as

from .config import config
...
config.windturbine_dir

without comprimising the integrity in the case of a direct assignment above.
The current version works in this case, i.e. the complexity
from . import config
....
config.config.windturbine_dir

is required for covering the case that the config object changes.

@euronion this relevant or necessary for v0.2?

It was, not sure if the changes by Jonas made it obsolete - I'll have to check it during review.

Obsolete

No config system, instead.


Add a test runner and template Enzyme test.

I put this together to show how we can add a test runner to the project. Discussion of exactly which test runner to use is ongoing, so I don't want to focus too much on that part. The Mocha-specific part of this is relatively minor, and could be swapped for a different framework relatively easily.
The main part is setting up a test config for Webpack and creating an entry point to bundle all the tests. Note that the first argument to require.context has to be a constant string (see issue here), which is why I made three different entry point files to handle where the file is in relation to the app's src directory. Definitely open to a more straightforward solution if anyone has suggestions.
The test plan: right now I'm planning to add on to the existing e2e.sh test, since the other scripts use that.
I think getting testing that "just works" is an important part of this project, and I'm excited to see where we can go with this!

Also important to note: in keeping with the project's purpose, it exposes zero config to the user prior to ejecting.

What are the pros and cons of using Webpack as test builder vs just using Node + Babel?

node + babel won't work if you want to test a component that imports css or images etc. because you cannot import those in node by default.

This depends on the runner, e.g. Jest has moduleNameMapper (it’s not quite “node” though so I see your point).

Jest only has moduleNameMapper to solve a Facebook specific problem though; it's not the norm.
Personally I'm strongly opposed to testing frameworks that integrate the totally separate things of:

running a bunch of JavaScript files in node/the browser
structuring tests and assertions within a file

My feeling is that we should provide 1 as part of create-react-app and maybe make a recommendation for 2.  This would rule out jest and mocha though.

Jest only has moduleNameMapper to solve a Facebook specific problem though; it's not the norm.

Importing CSS/images is not quite “the norm” either but we do it here (so we can have dependencies managed by a single tool), we do it in React Native, and many Webpack users in the community do it as well. So I think it’s fair to say at this point that if it’s not the norm, it’s at least a very common (and useful) deviation from the norm.

Importing css/images is (I think) rapidly becoming the norm.  What I meant was that it's not the norm for test runners to support anything other than the node.js resolution algorithm.  Jest does because we don't use the node.js resolution algorithm at Facebook.  We still use the node.js resolution algorithm for CSS dependencies and files here.  Using a different algorithm for resolving dependencies is exceptionally unusual.

Personally I'm strongly opposed to testing frameworks that integrate the totally separate things of:
running a bunch of JavaScript files in node/the browser
structuring tests and assertions within a file

Not sure I understand, exactly. I agree we shouldn't dictate what assertions to use—I used chai's expect in the template test, but the user could use other types of chai assertions, or add a different library. But in order to be useful, a test runner should show you the outcomes of the tests; that would require it to understand the structure of what a test is, which is why you have syntax like describe and it. I don't really see how something that just ran arbitrary Javascript code in node or the browser could really be called a testing framework.

Usually I have the following line in my tests setup
require.extensions['.css'] = function () {};

This code makes node.js module system ignore css imports and return empty module instead. As far as css-modules are not allowed (#98), I will be enough to deal with it.

What are the pros and cons of using Webpack as test builder vs just using Node + Babel?

@gaearon To your original question, a major benefit of using Webpack is that, when the project adopts Webpack 2 and ES modules, the test setup won't have to continue to rely on transpiling to CommonJS.

I don't really see how something that just ran arbitrary Javascript code in node or the browser could really be called a testing framework.

If you think in terms of continuous integration servers (e.g. travic-ci) the only thing they ever understand from tests is the exit code.  They show the stdout/stderr to the user for debugging, but all they use in any structured way is EXIT_CODE === 0 to tell if the test passed.  In this way a node.js module that exits with the correct code and loggs debugging info to stdout provides all the information that's needed.
The describe/it structure doesn't need to be provided by the same module that runs the JavaScript file (consider testit for example, which I wrote).
The issue with having tests which are not just files to be run is that you can't choose a different runner.  You can stack libraries for assertions, mocking, jsdom, structuring tests etc. as much as you like within a node.js module.  Meaning you can pick whichever of those are useful to you.  You can only have one test runner though.  This makes it really hard to add code coverage to runners like mocha and jest (unless they add support directly).  It makes it really hard to do arbitrary transpilation for babel/webpack etc.
By making sure that the runner's only job is finding the modules and running them (potentially in parallel) you can swap it out and use a different runner when it's convenient.  Not being able to run tests with node-inspector has been a huge extra challenge when testing with mocha.  Being defaulted into all the terrible auto-mocking stuff is the reason I don't use jest.  We need to use more focused testing tools.

@just-boris The require.extensions approach is neat.  It will break down completely if we adopt css-modules though, and you already need to make sure that all the image formats return a string.

@ForbesLindesay I see your point, and you're right about being able to separate the running from the structure of the test.
Given that, though, I don't see how running the test with node versus using mocha gives the user more flexibility to swap in a new test runner (without ejecting). Assume we've gone the less-opinionated route—when they run npm test, it does nothing more than run all the test files with node. There's still no simple way they could make it so that npm test runs ava or karma instead. They could try to use a different npm script like npm test:ava, they could eject, or they could rewrite npm test to not use react-scripts at all. But those are all the options they would have had if we included mocha (or any other runner) by default anyway.

Some random questions after implementing something similar for https://github.com/facebookincubator/create-react-app/pull/245 -
Does this webpack up the node_modules? I think it does (I could be missing something) but I don't think you have to do that for testing.
Is the convention that NODE_ENV is "test" for testing? I thought the convention was that NODE_ENV was always either "development" or "production" or unset.
I think it is better to put these intermediate files in the global temporary directory - then it can get automatically cleaned up if your system is administrated that way.

I believe that it does include node_modules in the bundle, which I think is necessary. They might be imported and used in some way that would affect the component's render output. Would the bundle be able to require them?
I'm really not sure about the NODE_ENV conventions. I'm not using it anywhere, so we can leave it unset.
I had no idea that os.tmpdir was a thing—definitely a better idea here.

I think it will be pretty slow if you are recompiling all of your node_modules every time you run tests, and normally in a node environment you would expect the node_modules to be shipped in ES5 so they can be run without compiling.

Still not sure I understand. This shouldn't include all of your node_modules, but it will include all the dependencies of your tests (which might include node_modules) in the bundle. Your node_modules won't get run through Babel (since, like you said, they're ES5 at this point), but if they're not in the bundle, how would the bundled test code know where to find them? Especially if the bundle is in your OS's tmpdir, how would it resolve the require statements to your project's node_modules?

You should be able to instruct it that requires go to node_modules somehow. On a different PR someone suggested https://www.npmjs.com/package/webpack-node-externals

That looks promising, I'll check it out

I don't think it is scalable to bundle all app code for every test run. This kinda kills iteration speed on any slightly bigger project.

Hey, thank you very much for the PR.
I’m inclined to go with #250 instead for these reasons: https://github.com/facebookincubator/create-react-app/pull/250#issuecomment-236148488.
We might reevaluate this choice in the future (luckily porting Jasmine 2 tests to another runner is no big deal).


Merge logs-agent into Datadog-Agent

What does this PR do?
Merged logs-agent source code into agent code:
https://github.com/DataDog/datadog-log-agent
Motivation
Reuse some logic, ease building and shipping, unify source code, etc.
Additional Notes
This is the very first step toward our main goal: running logs-agent as a go routine:
https://docs.google.com/document/d/1U02WcH1luUO2hBGzzDUj057ncjaH2W-xPuvOHyeeJqk/edit

see https://github.com/DataDog/datadog-agent/pull/920


QGIS 3.14 missing from ubuntu xenial repository

Description
Following these instruction: https://qgis.org/en/site/forusers/alldownloads.html#debian-ubuntus suggests QGIS 3.14 should be installed. However, the ubuntu xenial repository only has 2.18 (See here: https://qgis.org/ubuntu/dists/xenial/main/binary-amd64/Packages)
Can't tell if this is an intentional decision to keep xenial on an older version, a build failure with 3.14 on xenial which hasn't been resolved yet, or just an oversight.
Page URL: https://qgis.org/en/site/forusers/alldownloads.html#debian-ubuntu

Hi @Thra11
I don't remember the last supported version on UbuntuRecent versions of QGIS have dependencies (versions) that are not available on Ubuntu Xenial, so we can not provide packages. Supported distro for the active versions of QGIS are listed

I think we can close this one (xenial eol, 3.14 eoled)


Send a notification on Slack when an user is connected to a room and alone

Could happen also when the number of user goes beyond a value that would be set by an environment variable ?


Build on OS X Catalina fails

Building the wrk-update branch fails with
./luajit: cannot create state: not enough memory
Related issue:
https://github.com/LuaJIT/LuaJIT/issues/521
The fix should be to use LuaJit 2.1. The build then fails with the following at the moment:
src/script.c:30:37: error: array has incomplete element type 'const struct luaL_reg'
static const struct luaL_reg addrlib[] = {
                                    ^
src/script.c:30:21: note: forward declaration of 'struct luaL_reg'
static const struct luaL_reg addrlib[] = {
                    ^
src/script.c:36:38: error: array has incomplete element type 'const struct luaL_reg'
static const struct luaL_reg statslib[] = {
                                     ^
src/script.c:30:21: note: forward declaration of 'struct luaL_reg'
static const struct luaL_reg addrlib[] = {
                    ^
src/script.c:42:39: error: array has incomplete element type 'const struct luaL_reg'
static const struct luaL_reg threadlib[] = {
                                      ^
src/script.c:30:21: note: forward declaration of 'struct luaL_reg'
static const struct luaL_reg addrlib[] = {
                    ^
src/script.c:476:5: warning: implicit declaration of function 'gettimeofday' is invalid in C99
      [-Wimplicit-function-declaration]
    gettimeofday(&tv, NULL);
    ^
1 warning and 3 errors generated.
make: *** [obj/script.o] Error 1

Use the patched luajit sources from Homebrew.
brew unpack --patch luajit
rm -rf deps/luajit
mv luajit-2.0.5 deps/luajit
make


Deployment builds are failing

https://travis-ci.org/sindresorhus/refined-github/builds/253917298
> webpack
Hash: 7efb917c088b33fefd28
Version: webpack 3.2.0
Time: 436ms
        Asset     Size  Chunks             Chunk Names
   options.js  4.09 kB       0  [emitted]  options
   content.js  4.08 kB       1  [emitted]  content
background.js  4.09 kB       2  [emitted]  background
   [0] ./src/content.js 1.61 kB {1} [built] [failed] [1 error]
   [1] ./src/background.js 1.61 kB {2} [built] [failed] [1 error]
   [2] ./src/options.js 1.61 kB {0} [built] [failed] [1 error]
ERROR in ./src/content.js
Module build failed: Error: Couldn't find preset "babili" relative to directory "/home/travis/build/sindresorhus/refined-github"
    at /home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:293:19
    at Array.map (native)
    at OptionManager.resolvePresets (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)
    at OptionManager.mergePresets (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:264:10)
    at OptionManager.mergeOptions (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:249:14)
    at OptionManager.init (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/index.js:212:65)
    at new File (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/index.js:135:24)
    at Pipeline.transform (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/pipeline.js:46:16)
    at transpile (/home/travis/build/sindresorhus/refined-github/node_modules/babel-loader/lib/index.js:49:20)
    at Object.module.exports (/home/travis/build/sindresorhus/refined-github/node_modules/babel-loader/lib/index.js:174:20)
ERROR in ./src/background.js
Module build failed: Error: Couldn't find preset "babili" relative to directory "/home/travis/build/sindresorhus/refined-github"
    at /home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:293:19
    at Array.map (native)
    at OptionManager.resolvePresets (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)
    at OptionManager.mergePresets (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:264:10)
    at OptionManager.mergeOptions (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:249:14)
    at OptionManager.init (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/index.js:212:65)
    at new File (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/index.js:135:24)
    at Pipeline.transform (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/pipeline.js:46:16)
    at transpile (/home/travis/build/sindresorhus/refined-github/node_modules/babel-loader/lib/index.js:49:20)
    at Object.module.exports (/home/travis/build/sindresorhus/refined-github/node_modules/babel-loader/lib/index.js:174:20)
ERROR in ./src/options.js
Module build failed: Error: Couldn't find preset "babili" relative to directory "/home/travis/build/sindresorhus/refined-github"
    at /home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:293:19
    at Array.map (native)
    at OptionManager.resolvePresets (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)
    at OptionManager.mergePresets (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:264:10)
    at OptionManager.mergeOptions (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:249:14)
    at OptionManager.init (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/index.js:212:65)
    at new File (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/file/index.js:135:24)
    at Pipeline.transform (/home/travis/build/sindresorhus/refined-github/node_modules/babel-core/lib/transformation/pipeline.js:46:16)
    at transpile (/home/travis/build/sindresorhus/refined-github/node_modules/babel-loader/lib/index.js:49:20)
    at Object.module.exports (/home/travis/build/sindresorhus/refined-github/node_modules/babel-loader/lib/index.js:174:20)

Nevermind, I see this referenced in #617

babel-preset-babili is missing, it'd great if someone can add it and test npm run reduce-size because I don't have time right now.


Support FastPthreadMutex contention profiler && expose FastPthreadMutex to user

…ex to user
What problem does this PR solve?
Issue Number: resolve #1354
Problem Summary:
What is changed and the side effects?
Changed:

FastPthreadMutex支持contention profiler。
暴露FastPthreadMutex给用户使用。

Side effects:

Performance effects(性能影响):

Breaking backward compatibility(向后兼容性):

Check List:

Please make sure your changes are compilable(请确保你的更改可以通过编译).
When providing us with a new feature, it is best to add related tests(如果你向我们增加一个新的功能, 请添加相关测试).
Please follow Contributor Covenant Code of Conduct.(请遵循贡献者准则).

@wwbmmm @yanglimingcn 有空看看这个PR

LGTM


Rename any usage of “Hedera”, “Swirlds” or “Hashgraph” to “Hiero”

Problem

Solution

Alternatives
No response

The renaming is done for /sdk module:
https://github.com/hashgraph/hedera-sdk-java/pull/2045
We would need someone from devops to rename the gradle files in order to build the other modules.


Trouble building more than one recipes at once

Seems conda-build-all has had a wrench thrown in it with the recent conda-build release (1.21.7). Not really sure what is causing it, but it seems to have some issues identifying where source should be extracted. See this comment for one example. Also here is another that springs up in this build. It would be nice to figure out why source extraction is getting messed up in these cases.
cc @jjhelmus @patricksnape @msarahan

Updated the title to be more accurate. In short, we seem to be losing the -all part of conda-build-all.

Just a reminder about this one, @pelson and @msarahan.

I can't reproduce this @jakirkham.
find my_recipes -name meta.yaml -exec echo {} \; -exec cat {} \;
my_recipes/recipe_a/meta.yaml
package:
  name: recipe_a
  version: 2.4

requirements:
  run:
    - python
  build:
    - python

my_recipes/recipe_b/meta.yaml
package:
  name: recipe_b
  version: 3.2

requirements:
  build:
    - recipe_a
    - numpy x.x
  run:
    - recipe_a
    - python
    - numpy x.x

And
CONDA_NPY=10 conda build-all my_recipes --matrix-conditions 'python >=2.7'
Using Anaconda Cloud api site https://api.anaconda.org
Fetching package metadata .........
Resolving distributions from 2 recipes...
Computed that there are 9 distributions from the 2 recipes:
Resolved dependencies, will be built in the following order:
	recipe_a-2.4-py27_0 (will be built: True)
	recipe_a-2.4-py34_0 (will be built: True)
	recipe_a-2.4-py35_0 (will be built: True)
	recipe_b-3.2-np110py27_0 (will be built: True)
	recipe_b-3.2-np111py27_0 (will be built: True)
	recipe_b-3.2-np110py34_0 (will be built: True)
	recipe_b-3.2-np111py34_0 (will be built: True)
	recipe_b-3.2-np110py35_0 (will be built: True)
	recipe_b-3.2-np111py35_0 (will be built: True)

> conda info
Current conda install:

             platform : osx-64
        conda version : 4.1.9
    conda-env version : 2.5.2
  conda-build version : 1.21.5
       python version : 2.7.12.final.0
     requests version : 2.9.1
     root environment : /Users/pelson/miniconda  (writable)
  default environment : /Users/pelson/miniconda/envs/_dev_conda_build_all
     envs directories : /Users/pelson/miniconda/envs
        package cache : /Users/pelson/miniconda/pkgs
         channel URLs : https://conda.anaconda.org/t/<TOKEN>/conda-forge/osx-64/
                        https://conda.anaconda.org/t/<TOKEN>/conda-forge/noarch/
                        https://repo.continuum.io/pkgs/free/osx-64/
                        https://repo.continuum.io/pkgs/free/noarch/
                        https://repo.continuum.io/pkgs/pro/osx-64/
                        https://repo.continuum.io/pkgs/pro/noarch/
          config file : /Users/pelson/.condarc
         offline mode : False
    is foreign system : False

I'm just looking into why conda info isn't reflecting my local imported versions... 😞

OK, that seems to have done it. I am able to reproduce the problem now. Additionally, all of the unit tests are passing 😞 .


make useNativeDriver configurable

For the use with react-native-swipe-list-view I had to disable useNativeDriver in order to make it work (here's the reason why).
Therefore I thought it would be useful to make this property configurable in the CollapsibleStack options.

v5.2.0 is published with this feature.


[Suggestion] Redstone Pager

Last night I had an idea and I think it would be pretty cool so I figured I would throw it out there and see what you guys think.
The idea is something along the lines of the storage tablet, but that offers a way to monitor the state of RFTools Wireless Transmitters (maybe interact with but that seems OP).  The UI would have the Channel Number, a user-definable description, channel state (On/off, maybe signal strength if they support analog, not sure if they do), and a toggle that would let you set an alert when that channel turns on/turns off.  The alert could be something as simple as the icon changing in your hotbar, or an audio tone.
This came from wanting a way to know if I got a dimlet parcel in my RS system.  I was half asleep at the time, so the best I could come up with was putting the fireworks lens on a mana spreader or atomic reconstructor, then having an RS Detector turn on if there were parcels in the RS.  Then I thought "What if I'm not at my base?" and then the rest of this came together.
What do you think?

I really like the idea of being able to turn on a redstone signal remotely, really nice for base entrances.

maybe a wireless transmitter or receiver could be crafted into a sort of "remote" that lets you toggle or monitor channels, respectively? Could be crafted the same way as a network monitor but with a transmitter or receiver in the middle, perhaps

This has actually been on my todo for a long time. Maybe I should finally do it :-)

@deli73 I was thinking a module that you could slot into the tablet, but your suggestions make sense too.
@McJty Yes please :-D