Copy without render

This continues the work started in PR #132. Looking forward to your feedback.

So, is this idea dead?  REALLY need this for ansible scripts

@ianshark, This is is not dead at all. I want this to work! However, getting this to work on all platforms is a must for Cookiecutter pull requests. Unfortunately, this PR is not passing all tests on all platforms and fixing that takes free time that we just don't have. Even evaluating PRs takes time we don't have. The issue is that we have to focus on jobs and gigs that pay the bills. What time is left for open source is unfortunately not that much, and usually dedicated to emergencies or smaller tasks that we can resolve in minutes rather than hours.
We've considered adding a donation button in order to have paid time for Cookiecutter. Do you think that would be a good idea? Would you or your company be willing to donate for professional quality work on this particular issue? If so, then I'll add it today and we'll be able to get on the problem right now.

Hmm,
Hi Danny!
Wow - didn't expect to get an answer that quick!  I certainly understand
the problem.  I'm a full-time contractor too!  Donations are always a
good idea...but don't expect to get a lot of money that way, unfortunately.
I would be willing to work on the problem.  Is there a branch that's
attempted to merge this?  What tests are failing?  I have the time to
dedicate, and it's that important to me.  But I didn't want to step on
toes, since it looked like it was in progress.
Let me know how I can help!
-Scott
On 3/3/15 1:22 PM, Daniel Greenfeld wrote:

@ianshark, This is is not dead at all. I want this to work! However,
getting this to work on all platforms is a must for Cookiecutter pull
requests. Unfortunately, this PR is not passing all tests on all
platforms and fixing that takes free time that we just don't have.
Even evaluating PRs takes time we don't have. The issue is that we
have to focus on jobs and gigs that pay the bills. What time is left
for open source is unfortunately not that much, and usually dedicated
to emergencies or smaller tasks that we can resolve in minutes rather
than hours.
We've considered adding a donation button in order to have paid time
for Cookiecutter. Do you think that would be a good idea? Would you or
your company be willing to donate for professional quality work on
this particular issue? If so, then I'll add it today and we'll be able
to get on the problem right now.
—
Reply to this email directly or view it on GitHub
https://github.com/audreyr/cookiecutter/pull/184#issuecomment-77003073.

@lanshark - it's pretty frustrating because myself, @audreyr, and @michaeljoseph would love to put more time into this project, especially in regards to the review of pull requests. We're trying to figure out a way to fund development, since this is a very favorite project of ours.
In any case, this PR is failing on Windows tests (what @appveyor tests). That is what needs to be fixed AFAIK.

@hackebrot Everyone would love to see this work. The challenge, of course, is windows.

@pydanny Yeah I understand and I agree 100% that we should try to get this in. I thought about closing #132 and focussing on #184 instead. :relaxed:

:bell: Copy without render is now available on master! :smiley:
See http://cookiecutter.readthedocs.org/en/latest/advanced_usage.html#copy-without-render

:trophy:

Closing this with many thanks to @LucianU!

Happy to have contributed and happy to see this merged in.

Danny, I realize doing releases is always work, but it'd be great if this were released to pypi.
Do you know when you'll have a point release?  We use this feature for our cookie-cutters.
Thank you so much.

Hopefully this week. We have a period of slowdown between now and the arrival of what is hopefully the last proof of our book. Which means maybe tomorrow.


Adding elements nonsequentially

It would be nice, especially when using loaded documents, to be able to manipulate a document  in some manner beyond appending elements to their parent element. Two possibilities that leap to mind:

Add an index kwarg to the various add methods so that they can be used in lieu of insert. Possibly include a delete method as well.
Subclass the paragraphs and runs properties of the Document and Paragraph classes, respectively. Overwrite the various list methods to appropriately modify the underlying Etree elements. For instance, d.paragraphs[3] = "Hello world" would replace the 4th paragraph with a new hello world paragraph. This could be powerful and flexible, but it also feels hackish. I'm not really sure.

Thoughts?

I ended up needing to work around this limitation, and here's what I did:
r = CT_R.new()
para._p.insert(1, r)
run = Run(r)
run.style = 'my style'
run.add_text('my text')
Pretty brittle because I needed to insert the run before everything else but after the paragraph's styling item. But it worked! Would be great if this were part of the python-docx API.

I cannot import CT_R . How to do it?
from docx.oxml.text.paragraph import CT_R does not work...

It's in docx.oxml.text.run.
https://github.com/python-openxml/python-docx/blob/master/docx/oxml/text/run.py#L22


Account for relative_url_root in FailureApp's recall method

Closes #3738

+1

@josevalim Can you please review this? Thank you.

it seems that this fixes the issue

Friendly ping: @carlosantoniodasilva @josevalim, can one of you review?

Sorry for the delay! I have added some comments. :) Thank you!

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:

Thanks, @josevalim!


BUILD: Create a build step for browser extension files

The files can now be written independently in src/ where they
can also be unit tested. A build step grunt build now concatenates
the files and inserts them into a browser-extension template for
the content script.
Please note:

All src/ files should be wrapped with IIFE (function(){...}()). This IIFE will be taken out when concatenated.
If the IIFE needs to be kept through the concatenation process, add a comment to its top (ex ( function () { // don't remove)
Files should be added to the Gruntfile.js for the concatenation to continue. All files must be in order.

See https://phabricator.wikimedia.org/T227522

I get Npm module "grunt-concat-with-template" not found when running ./node_modules/.bin/grunt build. Is this expected?
I also see a mix of spacing / tabs, etc. in Gruntfile.js and contentScript.js, though eslint passes for everything in src/, so maybe we don't care :)
Finally, not really an issue with this PR, but we might want to add a .nvmrc. I got a different package-lock.json after running npm install.

Whoops, I forgot to add extension.json with the package! fixing that now.
I don't see the mix of spacing/tabs in Gruntfile.js, but I fixed the contentScript.js file.

And I added a .nvmrc file with node 11.5.

So:

I added generated.pageScript.js in; I think that as long as it's clear that it's autogenerated (so on one should manually change it) it's probably okay to keep it in the repo. We can change that later.
I added contentScript.js to .eslint rules, and fixed to make it pass
I made the activation button an actual link somewhere just to test it out and added a 'test.js' file with an OO.ui.alert to test the behavior and timing (to make sure ooui is available)
I cleaned up manifest.json to fit our code styles.
I tested the extension/ directory as an unpacked extension in chrome -- and made sure it works :)

Awesome! This looks good to me, though I haven't tested it.

Awesome. I'm merging in. We should take this repo under /Wikimedia (there's a task for that)


test: provide stop sequence for test inference request

@pacman82, I also wonder if it is sensible to set Self::NO_TOKEN_LIMIT as default for Stopping ? This specifies neither maximum_tokens nor stop_sequences.

We also updated some examples (e.g. in the ReadMe) to neither specify max tokens nor stop sequences.


vulkaninfo: Set missing sType and pNext in surf_formats2 array.

This fixes #109

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Project Completion

Hey Reviewer, this is our complete project
We've worked individually and collaboratively as needed based on the tasks at hand to finalize our project.

Teammate 1: @JonahKayizzi
Teammate 2: @eng-mohamed-eldeeb
Teammate 3: @rebaz36

we'd love to receive any constructive feedback, 👨🏼💻

A new video that is less than 5 minutes has been shared in the README


Fix MESSAGE OUT handling for initiator mode (#1283)

This PR can wait until right after the first bookworm release, because in this release MESSAGE OUT in initiator mode is not yet used.
It is going to be needed by the upcoming improved scsidump.

@rdmark Even though we do not need this change for the bookworm release I would like to merge it. It would make my life a bit easier, because currently I have to keep track of several pending PRs and issue changes.
In case you agree I will request a review on this draft.

@rdmark Yes, it is not yet used nywhere. The only application running in initiator mode is scsidump, but only the upcoming scsidump version (for the release after the first bookworm release) makes use of it.


All other VSCode tasks disappear

Type: Bug
All automatically detected tasks provided by extensions (including built-in VSCode extensions) disappear when this extension is loaded. The smallest config I noticed this with an extension bisect, but I was able to reproduce it with only the task extension and a language pack enabled.
Extension version: 0.3.2
VS Code version: Code 1.85.1 (0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2, 2023-12-13T09:48:16.874Z)
OS version: Darwin arm64 23.2.0
Modes:

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
5, 4, 4

Memory (System)
16.00GB (0.39GB free)

Process Argv
. --crash-reporter-id 08a9bcfc-08ca-400f-be73-4872b73843c5

Screen Reader
no

VM
0%

To clarify, this happens before the task extension has even loaded somehow.
Honestly no idea how, this is one of the weirdest issues I've ever run into.


No tangible results with pong or NeonRace within VNC environment

I have read through #52 and #45 both being closed without an explanation of resolution. I have run many iterations of NeonRace and gym-core.PongDeterministic-v0 over a couple weeks utilizing many different configurations of workers and vnc containers.

This screenshot is has been running about 8 hours with 8 workers on a processing server and 8 VNC environments on another server locally networked (0.1ms pings), both have 16 cores and haven't exceed a load average of 15 on either machine. I have tried with tf0.12, tf0.12.1, tf1.2.0, tf1.2.1, tf1.3.0 all with the same results and utilizing the version of all the dependencies listed in the README (you can see my install instructions here to verify: https://gist.github.com/dtcarls/03c23fca2fba804cfd4ac99ef4780a29). I continue to max out around 35k score in NeonRace (just hold forward essentially) and episode reward average of about 0 in pong. I have checked that all workers are running, I get reaction times sub-150ms, not a lot of reaction_time:None responses (see my poor attempt at a script here: https://github.com/dtcarls/universe-checkpoints/blob/master/reaction_time.sh). I have not been able to get either of these games to work in a VNC environment. I have been able to get PongDeterministic-v4 without VNC to work fine and quickly. I feel I am checking all of the diagnostic boxes yet cannot replicate the results of this repository, even with a longer timeline given that I am using 8 (or less) workers vs 16 workers.
If someone could point me in the right direction I would be grateful.

Update. 100 million steps made it to about +7. That is 7 workers running for 2.5 days (64.5 hours) While the readme claims: "If you run this experiment on a high-end MacBook Pro, the above job will take just under 2 hours to solve Pong." Now I understand those numbers are for non-VNC version and using a different core count, but to extrapolate a laptop in a non-VNC environment vs 2 servers with 16 cores each in a VNC environment getting a score of 7 over 32 times as long doesn't seem to add up.

Again, any assistance in finding what I may be doing wrong, or if this repo is simply out of date, would be much appreciated.


Analyzer hangs when analyzing some files

This issue was originally filed by @astashov

What steps will reproduce the problem?

Clone the repo https://github.com/astashov/analyzer-hanging
pub get
dart main.dart /path/to/dart-sdk /path/to/packages

What is the expected output? What do you see instead?
I'm expecting to get a resolved CompilationUnit. But it just hangs forever while trying to resolve it on the last line.
What version of the product are you using?
Dart 1.9.1
On what operating system?
MacOSX 10.9.5
What browser (if applicable)?
Server-side
Please provide any additional information below.

@bwilkerson - is this still relevant?

@bwilkerson, this is looking a little stale. Is this still being worked on?

Yes, this will be addressed as part of landing the new task model.

The new task model has now been enabled in the bleeding edge builds and should be in the next dev build.

@bwilkerson can I find any details somewhere about that new task model (like, what this is)?

The new task model is a different internal implementation of the tasks / steps / phases in the analyzer. The old implementation was more coarse-grained. In order to analyze a single library L, we ended up having to analyze all libraries that L was circularly dependent on as a single large task. This ended up conflicting with some memory management support we had added, causing the problem you were seeing. The new task model is much finer-grained, which, in addition to solving the problem you ran into, also means that we don't have to do nearly as much work before we can get analysis results for a single library (which should improve the performance of tooling such as IntelliJ or Atom).
Because it's an implementation detail the only documentation currently available is in documentation comments in the code.

Got it, thanks for the explanation!


Cannot download any file from Chromium browser

name: buster
encrypted: no
Entering /home/linux/chroots/buster...
crouton: version 1-20181001133934~master:38012fdf
release: buster
architecture: amd64
xmethod: xorg
targets: xfce
host: version 11151.17.0 (Official Build) beta-channel auron_yuna
kernel: Linux localhost 3.14.0 #1 SMP PREEMPT Tue Oct 30 00:45:31 PDT 2018 x86_64 GNU/Linux
freon: yes

Please describe your issue:
every time I try to save a file to the system it fails...
If known, describe the steps to reproduce the issue:
downloading anything from chromium using xfce.

@Carl393,
Where are you trying to save the file(s), in ~/Downloads?
Can you save a file in Chrome OS but not in your crouton chroot?
-DennisLfromGA

Yes Sorry for the delay.. when in xfce4 i cannot save a file in the shared downloads folder... under /home/carl/downloads/ but I can save a file on the xfce desktop.... is this a permissions problem? I never had this issue before.

I get this message... any clue what it means?

It seems like it belongs to Crouton?
How do i make it also belong to me? (username carl)
I find this kind of funny I realize it's a shared folder but some programs like SKYPE pick this folder by default. it would be nice if it just worked by default.
I can save any file to this folder using chrome OS but not when I'm in crouton.

@Carl393,
I don't believe the ownership of the ~/Downloads directory is the issue, mine are shown below and I have no problem when saving to ~/Downolads in the chroot -
(xenial)denny@localhost:~$ ll
total 24K
drwxr-xr-x.  3 denny denny   4.0K Dec 26 16:22 bin/
drwxrwxr-x.  2 denny denny   4.0K Oct 13  2016 Desktop/
drwx--x---. 15 denny crouton  12K May 18 09:04 Downloads/

(xenial)denny@localhost:~$ mount| grep /home
/dev/sda1 on /home/denny/Downloads type ext4 (rw,nosuid,nodev,noatime,seclabel,resgid=20119,commit=600,data=ordered)

I can't reproduce the issue myself but you may find some help or clues here as it appears to be an upstream bug in 'buster':

Cannot create files in ~/Downloads · Issue #3860 · dnschneid/crouton
https://github.com/dnschneid/crouton/issues/3860

Required key not available · Issue #3261 · dnschneid/crouton
https://github.com/dnschneid/crouton/issues/3261

Hope this helps,
-DennisLfromGA


Add Support for profiling in Tabular form

Hi,
Currently, with this tool after doing profiling of a process, we can only get the details in a flame graph which is in SVG Format, the main issue of the flame which is a good option but the main issue is, it is really hard to read and do rapid analysis due to nearly not readable flame graphs.
So if we could provide a good way to profile the same after generating the data, it will be really helpful.
Best,
Vibhoothi

Upvoted; the flamegraph is pretty useless I think :) I would want output that is a histogram of each source row's PC sample count. Anybody know how to analyze the perf output to produce something like that?


Test if code coverage is stable

As this is only a change to the README and is still valid Markdown then CI should still pass and (more importantly for this PR) not have any reported change to code coverage.

Codecov Report

Merging #8308 into master will decrease coverage by 0.37%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8308      +/-   ##
==========================================
- Coverage   59.32%   58.95%   -0.38%
==========================================
  Files         504      504
  Lines       23087    23087
  Branches     3736     3731       -5
==========================================
- Hits        13697    13610      -87
- Misses       8498     8576      +78
- Partials      892      901       +9

Impacted Files
Coverage Δ

src/client/testing/serviceRegistry.ts
48.64% <0%> (-46.85%)
:arrow_down:

src/client/testing/codeLenses/main.ts
44.44% <0%> (-33.34%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
66.66% <0%> (-33.34%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
75% <0%> (-25%)
:arrow_down:

...ience/jupyter/jupyterDebuggerRemoteNotSupported.ts
75% <0%> (-25%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
60% <0%> (-20%)
:arrow_down:

src/client/common/utils/icons.ts
83.33% <0%> (-16.67%)
:arrow_down:

src/client/api.ts
78.57% <0%> (-14.29%)
:arrow_down:

src/client/datascience/cellFactory.ts
75.86% <0%> (-13.8%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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

I expect Coverage to change (drop).
So the problem had nothing to do with stability (it's a misconception).
We run lots of tests on nightly and CI with coverage. These tests do NOT run on PRs.
Hence there would be a drop in coverage when ever we run the PR pipeline.

@DonJayamanne I have opened #8310 to compare two PRs to see if they are at least consistent that way (basically an A/A experiment). But if we are reporting coverage from CI and nightly which are not run as part of PRs it will mean we can't get on any coverage drop but only drops that go beyond some margin of error as based on the difference between what a PR runs and what CI/nightly run.


Not loading plugin due to different version of package internal/cpu

I've been trying to create a plugin to handle backend responses and transform the data coming from the back-end in a way that currently the default API doesn't provide. While loading the krakend.json on start/restart of Krakend I am getting the following error:
[00] 2021/03/21 17:11:55  WARNING: [loading plugins: plugin loader found 1 error(s):
[00] opening plugin 0 (plugin/clientPlugin.so): plugin.Open("plugin/clientPlugin"): plugin was built with a different version of package internal/cpu]
[00] 2021/03/21 17:11:55  INFO: [total http executor plugins loaded: 0]
[00] 2021/03/21 17:11:55  WARNING: [loading plugins: plugin loader found 1 error(s):
[00] opening plugin 0 (plugin/clientPlugin.so): plugin.Open("plugin/clientPlugin.so"): plugin was built with a different version of package internal/cpu (previous failure)]

After googling around, I am not sure exactly why this is happening. Code looks solid so for me it's either I am missing some configuration, generating the .so in the wrong GoLang version/architecture... I already tried with GoLang 1.13, 1.15.7 and now 1.14
krakend.json
{
  "version": 2,
  "name": "Gateway POC",
  "port": 8080,
  "cache_ttl": "300s",
  "timeout": "3000ms",
  "plugin": {
      "pattern":".so",
      "folder": "./plugin/"
  },
  "extra_config": {
    "github_com/devopsfaith/krakend-gologging": {
      "level": "DEBUG",
      "prefix": "[KRAKEND]",
      "syslog": true,
      "stdout": true,
      "format": "default"
    },

    "github_com/devopsfaith/krakend-metrics": {
      "collection_time": "60s",
      "proxy_disabled": false,
      "router_disabled": false,
      "backend_disabled": false,
      "endpoint_disabled": false,
      "listen_address": ":8090"
    },
    "github_com/devopsfaith/krakend-opencensus": {
      "sample_rate": 100,
      "reporting_period": 1,
      "exporters": {
        "jaeger": {
          "endpoint": "http://jaeger.local:14268/api/traces",
          "service_name": "sd-gateway"
        }
      }
    },
    "github_com/devopsfaith/krakend-cors": {
      "allow_origins": [
        "*"
      ],
      "allow_methods": [
        "POST",
        "GET"
      ],
      "allow_headers": [
        "Origin",
        "Authorization",
        "Content-Type",
        "Accept",
        "X-Requested-With"
      ],
      "expose_headers": [
        "Content-Length",
        "Accept",
        "X-Requested-With",
        "Content-Type"
      ],
      "max_age": "12h"
    }
  },

  "endpoints": [
    ...
    {
      "endpoint": "/merged/test",
      "headers_to_pass": [
        "Authorization",
        "User-Agent"
      ],
      "method": "GET",
      "output_encoding": "json",
      "whitelist":["data"],
      "backend": [
        {
          "url_pattern": "/api/galleries",
          "sd": "static",
          "method": "GET",
          "host": [
            "http://galleries.local"
          ],

          "extra_config": {
            "github.com/devopsfaith/krakend/transport/http/client/handle": {
                "name": "clientPlugin"
            }
          }
        }
      ]
    }
  ],
  "output_encoding": "json"
}

This is my plugin.go that is generating the clientPlugin.so in the plugin folder with the command /home/thiago/sdk/go1.14/bin/go build -i -buildmode=plugin -a -installsuffix cgo -ldflags -w -s -o /home/thiago/go/src/github.com/thiduzz/krakend/./plugin/clientPlugin.so /home/thiago/go/src/github.com/thiduzz/krakend/plugin.go
plugin.go
package main

import (
	"context"
	"errors"
	"fmt"
	"net/http"
)

var ClientRegisterer = registerer("clientPlugin")

type registerer string

func (r registerer) RegisterClients(f func(
	name string,
	handler func(context.Context, map[string]interface{}) (http.Handler, error),
)) {
	f(string(r), r.registerClients)
}

func (r registerer) registerClients(ctx context.Context, extra map[string]interface{}) (http.Handler, error) {
	name, ok := extra["name"].(string)
	if !ok {
		return nil, errors.New("wrong config")
	}
	if name != string(r) {
		return nil, fmt.Errorf("unknown register %s", name)
	}

	return http.HandlerFunc(func(w http.ResponseWriter, req *http.Request) {
		fmt.Println("clientPlugin message")
	}), nil
}

func init() {
	fmt.Println("clientPlugin client plugin loaded!!!")
}

func main() {}

Commands used
I am currently using a docker-compose configuration and utilizing the devopsfaith/krakend:config-watcher image which is loading the krakend.json from a docker volume.

Nevermind - got it working. Here is the deal, when creating a plugin, make sure to compile them with the exact Go version that your Krakend is running. Where do you find that? Access Dependency Validation page, select the Kraken version and you will see in the top of the list the correct Go runtime you will have to run, in my case, 1.15.8.


Screenshot flexibility request

Are you requesting a feature or reporting a bug?
Feature
What is the current behavior?
Screenshot naming conventions are very restricted, command line parameter -S has no control at all.
What is the expected behavior?

Screenshots should have flexible naming conventions.  For example, Junit Attachment Plugin uses the classname xml attribute to associate the artifacts with the package, or uses absolute references

Fixtures and/or tests should have the ability to individually take screenshots on fail.  Maybe a hook that can be defined per test?  If I have a fixture with:
test("Test A", async => {});
test("Test B", async => {});
test("Test C", async => {});

Then I should be able to specify which tests I want to take a screenshot on failure.  Example:
test("Test A", async => {});
test
  .screenShotOnFail(
	//Some kind of path structure here maybe?
  )
  ("Test B", async => {});
test("Test C", async => {});

Specify your

operating system:  Windows 7
testcafe version:  0.16.0
node.js version:  6.10.1

Similar issues:  #1515 and #1595

closed in favor of #2152


[Memo] Change notifications?

Maybe I'm missing something obvious, but is there an API (or plans for an API) to be notified of changes to the work tree. In particular if the work tree gets modified due to remote ops being applied how do I learn when that happens and what has changed.

Our original plan was to allow you to detect changes just by inspecting the state. So if you're rendering a tree view, you can just retrieve the entries and re-render. If you have 3 buffers open, you can call changesSince for just those buffers whenever operations get applied to see if any have changed. We're open to iterating on this plan based on feedback. Definitely would be interested in what might work better for you in your use case.

I think the changesSince API is good for buffers. Could something similar be added to work_tree to report file changes in the work tree?
It's true I can just determine this all by inspecting state. But if I want to update the view in an animated fashion then I need to know which exact files were inserted/removed/etc. I could do that by diffing states, but it gets expensive and complicated. Especially when all the information is already known by memo, seems like just exposing exactly what's changed would be nicer.
My use case is that I'm building a macOS file tree view as a learning project. The view can be updated in two ways. Simple way is to just reloadData... which reloads the entire view. Works, but it's somewhat expensive and doesn't provide any animation. On the other hand if I know exactly which files are removed/inserted/changed then I can just reload those portions of the view that have changed. It's faster and you get animation.


[cuda]Build：building cuda:x64-windows failed with: BUILD_FAILED

-- Executing NVCC-NOTFOUND --version resulted in error: 1
CMake Error at ports/cuda/vcpkg_find_cuda.cmake:69 (message):
Could not find CUDA.  Before continuing, please download and install CUDA
(v10.1.0 or higher) from:
  https://developer.nvidia.com/cuda-downloads

Call Stack (most recent call first):
ports/cuda/portfile.cmake:9 (vcpkg_find_cuda)
scripts/ports.cmake:175 (include)

Please follow the error message to check whether cuda is installed?

OK，thank you.

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


translate to chinese

when i learned this repo, i think it really gem！
so i want to translate it into chinese for more readers. it's not done yet, but i'd like to complete it in 1-2 weeks.

Awesome. Please merge all commits into one (one PR - one commit), and I'll integrate it.

@cpq ok, i'll have a try.


Absorb https://github.com/bryal/dxgi-rs into dxgi-sys

AFAIK, this adds all functions, structures, enumeraitions, constants (GUIDs), and interfaces of dxgi.
Also add some macros from dxgi-rs to winapi for more convenient construction of representations of C++ classes

Unfortunately this completely breaks with the existing style of how things are done in this project.


Refresh button is shown two times in the References pane

VSCode Version: 1.33.1
OS Version: Windows 10

Steps to Reproduce:

npx create-react-app . --typescript (or any other TypeScript codebase)
Go to index.ts and right-click on App, select Find All References
Observe how only a single disable refresh button is shown during search
Observe how two Refresh buttons are shown when the search finishes
Observe that the right one doesn't seem to initiate refresh (no loading indicator)

I don't know if this happens without extensions disabled, nor do I know if this is TypeScript specific, nor do I know if it happens in Insiders, haven't had time to check.

I have never seen something like that... @TomasHubelbauer what are those other icons? Where does this magnifier icon come from? Is that another viewlet/extensions?
cc @sandy081

Oh shoot, these are from one of my own extensions! I am sorry, not sure how that didn't connect before I posted this 🙄 How did they find their way into this viewlet though? Should it be possible to contribute buttons here programatically?


FIRAuthUI.authUI authentication workflow doesn't work on iOS 10

Swift:
iOS 10:
Firebase 3.3.0, FirebaseUI 0.4.0

Explanation of your issue
On iOS 10, the flow to authenticate with FIRAuthUI.authUI() is broken. After presenting authUI?.authViewController(), and clicking on Signin With Google, authViewController dismiss immediately, going in the delegate method authUI(_ authUI: FIRAuthUI, didSignInWith user: FIRUser?, error: NSError?), with an error

(Error Domain=FIRAuthUIErrorDomain Code=1 "(null)" UserInfo={NSUnderlyingError=0x79ff7420 {Error Domain=com.google.GIDSignIn Code=-2 "keychain error" UserInfo={NSLocalizedDescription=keychain error}}, FIRAuthUIErrorUserInfoProviderIDKey=google.com})

(without passing in the app delegate application:openUrl: method)
However, on iOS 9, the same code works.
Steps to reproduce

Configure a new project with Firebase, and add the pod FirebaseUI
Add a button in the project created viewController with this IBAction:

    @IBAction func authenticate() {
		let authUI = FIRAuthUI.authUI()
		authUI?.delegate = self
		 let googleAuthUI = FIRGoogleAuthUI(clientID: "*******")
		authUI?.signInProviders = [googleAuthUI!]
		let authViewController = authUI?.authViewController()

		self.present(authViewController!, animated: true, completion: nil)
	}

Conform to FIRAuthUIDelegate and add this delegate method

func authUI(_ authUI: FIRAuthUI, didSignInWith user: FIRUser?, error: NSError?) {
		if error != nil {
			print(error)
			return
		}
		if (user != nil) {

		}
	}

Launch the app and test the workflow (on iOS 9 it works, on iOS 10, it breaks)

Sincerely

Thanks for the report. This appears to be a bug in iOS 10 beta 2 and beta 3. https://openradar.appspot.com/27422249
We will keep track of the issue internally and make sure everything works with the final release of iOS 10.


Attach artifacts to the pose graph

Added ARL's artifact message

Can one of the admins verify this patch?

ok to test


Update http4s-blaze-server, http4s-circe, ... to 0.21.19

Updates

org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.21.4 to 0.21.19.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

dependency:v0_21@org.http4s:http4s-scalafix:0.21.5

Documentation:

https://github.com/http4s/http4s/releases/tag/v0.21.5

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch, scalafix-migrations

Superseded by #67.


Fix rotate when use labelsFilter

the rotation of labels is calculated with all label length,
can you make the rotation depends on the label length after filter?

hey sorry for the late reply. I had tried this when i was first developing the filter and when there is a lot of data I found that without rotation the label covered several datapoints (in all of my use cases for creating it the reason a filter is there is because they can't all fit in so most of the time there is a lot of data on the graph). With this overlap it became hard to distinguish the original datapoint the label was trying to show. With the rotation this is obvious because the label ends at the point where the data is on the graph. Because of that i decided against re adjusting the rotation after the filter was applied.


Add an index flush parameter

I would be nice to have a parameter to have the record indexed after a POST, PUT and DELETE (elasticsearch flush index). This is very useful for a single app application.

This is indeed very useful for SPA. Right now the only way is to add delays in the JavaScript app.
To be considered for invenio-records-resources.


Library is not picking up access token in implicit flow

Describe the bug
Access token is never taken into the OAuthService. I may initiate an implicit flow where I am redirected to keycloak, log in, and redirected back to the app. However the access token is never grabbed by the library though I am pretty sure (90%) it does get passed back from keycloak when looking at the network responses. So I have a secured backend but want to use implicit flow to get identity claims and be able to use guards to know if users should be allowed access to certain locations on the app.
Configuration & Code
authConfig
export const authConfig: AuthConfig = {
  issuer: environment.keycloakIssuer,
  redirectUri: window.location.origin + "/admin/callback",
  clientId: 'spa-client',
  scope: 'openid profile email offline_access',
  responseType: 'token',
  disableAtHashCheck: true,
  showDebugInformation: true,
  requireHttps: false
}

Do note I have also tried with 'code' response.
AppModule
OAuthModule.forRoot({
      resourceServer: {
        allowedUrls: [environment.baseApi],
        sendAccessToken: true
      }
    }),

AppComponent
import { Component } from '@angular/core';
import { authConfig } from './core/auth/auth.config'
import { OAuthService } from 'angular-oauth2-oidc';
import { JwksValidationHandler } from 'angular-oauth2-oidc-jwks';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.scss']
})
export class AppComponent {
  title = 'testAngular';

  constructor(private oauthService: OAuthService) {
    this.configure();
  }

  private configure() {
    this.oauthService.configure(authConfig);
    this.oauthService.loadDiscoveryDocument();
    this.oauthService.tokenValidationHandler = new JwksValidationHandler();
  }
}

AdminComponent
export class AdminComponent implements OnInit, OnDestroy {

  subs: Subscription[] = [];

  constructor(private oauthService: OAuthService) {}

  ngOnInit() {
    this.subs.push(this.oauthService.events.subscribe(e => console.log(e)));
  }

  ngOnDestroy() {
    this.subs.forEach(sub => sub.unsubscribe())
  }

  public login() {
    this.oauthService.initImplicitFlow();
  }

  public logoff() {
    this.oauthService.revokeTokenAndLogout();
  }
}

The guard that consistently tells me that I do not have a valid access token
export class CreateGuard implements CanActivateChild {

  constructor(private oAuthService: OAuthService) {}

  canActivateChild(childRoute: ActivatedRouteSnapshot, state: RouterStateSnapshot): boolean | UrlTree | Observable<boolean | UrlTree> | Promise<boolean | UrlTree> {

    let resp = this.oAuthService.hasValidAccessToken();
    console.log('this.oAuthService.hasValidAccessToken() -> ' + resp);
    return resp;
  }
}

Expected behavior
Expect to be library to automatically pick up the access token as described in the docs (end of https://github.com/manfredsteyer/angular-oauth2-oidc#logging-in) so I can use the methods in OAuthService to get identity claims and attach access token to http calls.
Desktop (please complete the following information):

OS: Windows 10
Browser: Brave, Firefox, Opera
Version: Brave 1.13.82, Firefox 80.0.1

Additional context
I can see nonce and PKCE verifier in local storage after logging in to keycloak, but no access token.
Note I have looked at other issues such as #876 #891 #658 but none of them seemed to help. Stack overflow suggested changes were less so: https://stackoverflow.com/questions/48369306/angular-5-oauth2-token-not-getting-set-with-libary-angular-oauth2-oidc?rq=1

You're right! The library should do that. But the point at which it does so is an application concern. You need to tell the library when to do that.
You either need to:

call this.oAuthService.tryLogin(); or
instead of this.oAuthService.loadDiscoveryDocument() call this.oAuthService.loadDiscoveryDocumentAndTryLogin()

the TryLogin bit will extract the access token Keycloak puts in the hash fragment for Implicit Flow.
See for example/reference my sample implementation's use of loadDiscoveryDocument() and tryLogin.
Hope that helps (and is applicable to your situation)?

Thanks for the feedback, I managed to fix it!
I made 3 changes:

in the authConfig I changed responseType from token to code
in the AdminComponent I changed the login from initImplicitFlow to `initLoginFlow``
in the CallbackComponent I added this.oauthService.loadDiscoveryDocumentAndTryLogin()

An aside, in the example you linked to I could not see how the oauthService was configured at any point before loading the discovery document.

example you linked to I could not see how the oauthService was configured at any point before loading

I configure everything through Angular's DI system.
Glad to hear you got everything fixed!


Problem: can't process an unzipped folder or single file transfer

Describe the bug
When an unzipped folder or single file transfer is uploaded to Minio to start processing, Enduro fails with an error in the bundle activity.
To Reproduce
Steps to reproduce the behavior:

Upload the unzipped transfer directory https://github.com/artefactual/archivematica-sampledata/tree/master/TestTransfers/small to Minio

OR

Upload the single file https://github.com/artefactual/archivematica-sampledata/blob/master/TestTransfers/small/small.txt to Minio

The Enduro workflow fails with an bundle activity error like:
non retryable error: error moving file (from /tmp/blob-853182088 to /home/a3m/.local/share/a3m/share/c/990965350/objects/small.txt): rename /tmp/blob-853182088 /home/a3m/.local/share/a3m/share/c/990965350/objects/small.txt: invalid cross-device link"

Expected behavior
The bundle activity should create a valid Archivematica standard transfer, which a3m would then process and create an valid AIP.
Screenshots

Additional context
As indicated by the error, the os.Rename() method does not work if source and target names are on different logical devices.
Ref: https://groups.google.com/g/golang-dev/c/5w7Jmg_iCJQ?pli=1

It think that Enduro is treating an uploaded directory as multiple single file uploads rather than a single transfer - which would be a separate bug.


error converting from keras to caffe

Platform (like ubuntu 16.04/win10):ubuntu 16.04
Python version:Python 2.7.16
Source framework with version (like Tensorflow 1.4.1 with GPU):tensorflow 1.9.0
Destination framework with version (like CNTK 2.3 with GPU):caffe 1.0.0
Pre-trained model path (webpath or webdisk path):custom file
Running scripts: mmconvert -sf keras -iw model_076720.h5 -df caffe -om mycaffemodel
Using TensorFlow backend.
Traceback (most recent call last):
File "/home/taeheej/work/nobkup/anaconda2/envs/deephipy2f/bin/mmconvert", line 10, in
sys.exit(_main())
File "/home/taeheej/work/nobkup/anaconda2/envs/deephipy2f/lib/python2.7/site-packages/mmdnn/conversion/_script/convert.py", line 102, in _main
ret = convertToIR._convert(ir_args)
File "/home/taeheej/work/nobkup/anaconda2/envs/deephipy2f/lib/python2.7/site-packages/mmdnn/conversion/_script/convertToIR.py", line 46, in _convert
parser = Keras2Parser(model)
File "/home/taeheej/work/nobkup/anaconda2/envs/deephipy2f/lib/python2.7/site-packages/mmdnn/conversion/keras/keras2_parser.py", line 120, in init
'DepthwiseConv2D': layers.DepthwiseConv2D
File "/home/taeheej/work/nobkup/anaconda2/envs/deephipy2f/lib/python2.7/site-packages/keras/engine/saving.py", line 417, in load_model
f = h5dict(filepath, 'r')
File "/home/taeheej/work/nobkup/anaconda2/envs/deephipy2f/lib/python2.7/site-packages/keras/utils/io_utils.py", line 197, in init
'Received: {}.'.format(type(path)))
TypeError: Required Group, str or dict. Received: <type 'unicode'>.

Hi @TaeheeJeong, maybe this workaround could solve your issue.
Thanks!


Update release-4.10 branch with latest from main

What this PR does / why we need it:
Merge the latest from main branch into release-4.10

/lgtm

/retest

/test e2e-aws


Goated Experimental changes

quantum ssm-multiverse code changes

Integrated Maven into the file-code system, to handle libraries, setup, shading and compiling.
Refactored the codebase to correctly implement git maven hierarchy.
Cleaned the readme file with proper instructions and information.
Fixed inconsistencies and errors which existed due to the original project using Java 14 instead of Java 8.
Minor tweaks and code fixes (mainly getting rid of the copy-pasted spahetti if else statements)

I appreciate the work you're trying to do but this is too much for me to review, if we can talk about it further I'd be possibly open to merging it.

If you reduce the files changed I'd be open to looking into it, but this is just too much to ask when you didn't even review the files yourself.

Had to delete the template server, pretty sure we shouldn't be hosting server jars they need to be built for a reason?


HomePage UI is broken

There is a white box apearing in the bottom out of nowhere, i think it is because of improper styling of frontend, can i fix the frontend? @muruga21

@samar12-rad i too saw that.. i dont have much time to fix that.. could you do that..
and remove the header / navbar at the loading screen

I will do that the most priority if you assign me that issue @muruga21

Started work

os the part you should with german words scrollable ?
can you share a video ?

@muruga21

@samar12-rad its not about the scroll... when you click the generated link.. you can see hello in multiple languages right.. i that page i dont want header / navbar

i hope im clear.. if not let me know

let me check .
Can you explain on clicking where does it happen?
As it is not happening on my side.
https://github.com/user-attachments/assets/c4109d16-9b36-4e50-9c6c-f9297c8600f8

is it the loading?  i couldnt see that, because the loading is happening fast on my side. What changes do you want there ?

@samar12-rad  in app/layout you can see Navbar component.. [ it makes the navbar global to all the pages ]  you have to remove that and in app/(default)/layout add the Navbar.. [ if makes navbar only for the home page ]

@samar12-rad  in app/layout you can see Navbar component.. [ it makes the navbar global to all the pages ]  you have to remove that and in app/(default)/layout add the Navbar.. [ if makes navbar only for the home page ]

Ok


Ripper compatibility layer

We want to replace Ripper with prism within the community as the default choice when a developer needs access to a parser. It's possible that given enough time we could replace all of the public uses ourselves, but we won't be able to migrate private repositories/applications that are using it. Even though Ripper has been marked as "early alpha" since it was first introduced, it has been around a long enough time that people have been relying on it for years.
We want to provide a compatibility layer such that people can continue to use Ripper for as long as they need to, but have the parsing be backed by prism. This means replacing all of Ripper's surface area, which is no small task. That surface area can loosely be broken up into two different groups: Ripper core and the Ripper API.
Ripper core is a SAX-style Ruby parser. This means it calls out to Ruby methods whenever a rule is reduced and uses the results of calling those methods as arguments to more methods higher in the tree. For example:
class Foo < Ripper
  def on_binary(left, op, right) = [:binary, left, op, right]
  def on_int(value) = [:@int, value]
  def on_stmts_new = []
  def on_stmts_add(stmts, stmt) = stmts << stmt
  def on_program(stmts) = stmts
end

Using Ripper.new("1 + 2").parse you will get back [[:binary, [:@int, "1"], :+, [:@int, "2"]]]. The results of our on_int method get passed to on_binary, which get passed to on_stmts_add, which get passed up to on_program.
We're not going to put hooks into prism's parsing for every rule reduction. We would not be able to maintain it, and it would clutter the entire codebase. Instead, we're going to replicate Ripper by parsing the entire tree with prism and then walking the tree in the same order as Ripper to generate the same output. Note that we're going to be calling out to methods that are user-defined, not providing the implementation of those methods ourselves. The order of the methods that are called therefore very much matters. You can see the beginnings of this approach here: https://github.com/ruby/prism/blob/main/lib/prism/ripper_compat.rb.
Unfortunately, because Ripper exposes so much of the internals of the Ruby parser, you can also access various state on the parser. This includes the current line number, column, and parser state. We're not going to be able to replicate the parser state very accurately, so I don't think we should try. The lines and columns we can though. This will involve changing the line and column numbers to be in a specific state before calling out to the user-defined hooks.
The second half of this work is the ripper API. This is Ripper::SexpBuilder, Ripper::SexpBuilderPP, and Ripper::Lexer. Because all of these are just Ruby libraries, they can come after Ripper core and we can mostly copy-paste. We'll see if there are other blockers later, but it looks like it will be manageable.
Success/deliverables for this ticket can come in multiple stages. In loose order, they would be:

Parsing various codebases (start with prism itself, then ruby/ruby, etc.) with Prism::RipperCompat::SexpBuilder and Ripper::SexpBuilder and comparing the outputs.
Ensuring all of the fixtures in prism pass with the same comparison.
Running ruby-syntax-tree/syntax_tree's test suite against Prism::RipperCompat until it passes (this is the biggest project I know of based on ripper).

This would be awesome!
Also for TruffleRuby which currently reuses the Ripper C code, which is slow and uses way too much internals (including two copies of parse.y, it's not a proper C extension but basically CRuby core C code behind a require).

Mentioned so we don't forget: in #2439, I broke up Translation::Ripper into two files, one inheriting from the other. The encapsulation is pretty messy -- it's mostly the same as it was before in terms of everything seeing everything. We can't break it up in quite the way we'd like (ref: https://github.com/ruby/prism/pull/2439#issuecomment-1948916567) by doing it this way. The hard part is the on_* methods, which are used from the compiler to emit Ripper output, but we want to redefine them to change where that output goes (e.g. SexpBuilder).
I think we could fix this by having a RipperEmitter (or some other name) that turns on_* methods into Ripper output. Then the RipperCompiler would just know how to turn Prism nodes into Ripper output names, but we could separate out things like combining lists of raw ripper (SexpBuilderPP) or just emitting as arrays (SexpBuilder) as an emitter. I think this would yield a cleaner design. In any case, we'd like a cleaner design.

Right now we primarily test Translations::Ripper output against Ripper's own output using sexp_raw. That's not a bad start. Tom Enebo of JRuby appreciates that we've found at least one incompatibility in their Ripper implementation. Benoit Daloze would rather just drop Translations::Ripper into TruffleRuby to replace their Ripper implementation as soon as reasonable, and so isn't as bothered about fixing minor Ripper incompatibilities. Fair.
We still want Prism and Translations::Ripper to run well on both JRuby and TruffleRuby. But especially for Truffle it makes sense to collect Ripper fixtures from at least Ruby 3.3 and 3.4. This would allow us to test against reference Ripper on TruffleRuby, and against specific CRuby-version Ripper on different CRuby versions. That may also let us enable tests on more TruffleRuby and/or JRuby versions since we can test against known Ripper output.

This is now on main. There are a couple of incompatibilities that I'll work through, but it should work for the vast majority of applications.


feat: oci image support for multiple src files

Support setting additional SrcFile component properties when same OCI image is used across multiple src files

@setchy any tests you can add? Would love to see this in action!

@setchy what is left for this PR?

@setchy what is left for this PR?

i think it was just tests...

@setchy, setup-swift is known to fail at least once a week, and it is always the Swift team's fault. Usually, restarting the job in a few hours would work; if not, we may have to move the swift tests to a separate job and make them optional.

bingo - the re-run did the trick @prabhu


[OODT-1018] OPSUI throwing NotSerializableException on several classes

OPSUI attempts to serialize classes that have not implemented the Serializable interface. This pollutes the logs with a large number of NotSerializableException stack traces.
I added transient to the Transceiver and Proxy objects in Client classes to prevent attempts at Serialization, which solves the issue.

@yasithmilinda Thanks for identifying and fixing this issue. Merging.


After eight hours of training, the model can synthesize natural speech!

code branch position PR https://github.com/lifeiteng/vall-e/pull/46
egs https://github.com/lifeiteng/vall-e/tree/position/egs/ljspeech
synthesised speech https://github.com/lifeiteng/vall-e/tree/position/egs/ljspeech/demos
The next step is to verify in-context learning capabilities of VALL-E on a large dataset.

Hi, @lifeiteng
Thanks for your work!
Btw current vall-e repo supports prosody transfer feature?
In official demo I can see "Speaker’s Emotion Maintenance".
Looking forward to your reply.
Best Regards!
Petar

Hi, @lifeiteng
I have trained model with the latest position branch and finally have got epoch-100.pt file (162MB).
But the results are different from the result yous shared.
My result -> 0.wav: https://drive.google.com/file/d/1HBeubaOxPClPbR7HntxA4708raXZt5eY/view?usp=share_link
1.wav: https://drive.google.com/file/d/1HtQRdJRz4yWm-MikyBngpjNB6CUcXgba/view?usp=share_link
Please try to check and let me know about the solutions.
Thanks very much.
Petar

how many gpus you trained with?

I used one GPU (Tesla V100 GPU).

Hi, @lifeiteng
When we train with LJSpeech dataset, we need to keep all .wav files in /wavs folder.
If we use another data instead of LJSpeech dataset, we can not keep all files in several folders?
Looking forward to your reply.
Best Regards!
Petar

@lifeiteng , I've been unable to reproduce good quality results. For training, I've used the exact same arguments as mentioned in the egs/ljspeech/README.md, with the addition of the world-size argument for training it on multiple GPUs.
I'm also using the provided arguments for inference as well, using the 99th epoch checkpoint instead of the 100th. The audio files that are generated are very garbled and have unintelligible speech.
Training Arguments
python3 bin/trainer.py --max-duration 72 --filter-max-duration 14 --num-buckets 6 --dtype "float32" --save-every-n 10000 --model-name valle --norm-first true --add-prenet false --decoder-dim 256 --nhead 8 --num-decoder-layers 6 --prefix-mode 0 --base-lr 0.05 --warmup-steps 200 --num-epochs 100 --start-epoch 1 --start-batch 0 --accumulate-grad-steps 1 --exp-dir /ssd_scratch/cvit/skosgi242/valle_ckpts/ljspeech --world-size 4

Inference Arguments
python3 bin/infer.py --output-dir demos --top-k -1 --temperature 1.0 --model-name valle --norm-first true --add-prenet false --decoder-dim 256 --nhead 8 --num-decoder-layers 6 --prefix-mode 0 --text-prompts "In addition, the proposed legislation will insure." --audio-prompts /ssd_scratch/cvit/skosgi242/24K_samples/en_ljspeech_LJ049_0124.wav --text "To get up and running quickly just follow the steps below.|During the period the Commission was giving thought to this situation." --checkpoint /ssd_scratch/cvit/skosgi242/valle_ckpts/ljspeech/epoch-99.pt

The drive link contains the following information:

Generated samples
The checkpoint used
The training log.

Please help me figure out where it went wrong. Let me know if you require further information on this.

Hi, @lifeiteng When we train with LJSpeech dataset, we need to keep all .wav files in /wavs folder. If we use another data instead of LJSpeech dataset, we can not keep all files in several folders? Looking forward to your reply. Best Regards! Petar

Hi, have you got the answer?

Hey, can anybody pls release their BEST model please? I need a free model for this pls, got no money to spend on GPUs, need to make a good ai thing to make me money to have to spend on GPUs

I used one GPU (Tesla V100 GPU).

so 8 hours of training on a single GPU?


fixing OS partition detection

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

#sing-off

vm repair

@zhoxing-ms I am not added as part of the automatic reviewers despite me being code-owner for this repo.
https://github.com/Azure/azure-cli-extensions/blob/main/.github/CODEOWNERS#L63

Could you please add description in the history notes for this change?

fixing OS partition detection for RHEL ADE encrypted VMS and changing version to 0.4.9

＠jofrance It is recommended to write this version change into the HISTORY.rst

Done
/ceschi - Jose Francisco Franceschi Vargas
Escalation Engineer, Microsoft Azure Linux Support
Office: +506 411 60004
<EMAIL_ADDRESS>0800 - 1700 GMT -6
If you have any feedback about my work, please let either me or my manager Kurt Davidson know at<EMAIL_ADDRESS>From: Xing Zhou @.>
Sent: Sunday, November 13, 2022 8:15 PM
To: Azure/azure-cli-extensions @.>
Cc: Francisco Franceschi (CESCHI) @.>; Mention @.>
Subject: Re: [Azure/azure-cli-extensions] fixing OS partition detection (PR #5507)
＠jofrance It is recommended to write this version change into the HISTORY.rst

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli-extensions%2Fpull%2F5507%23issuecomment-1312948117&data=05|01|Jose.Franceschi%40microsoft.com|95e18a5dfdb24978e93a08dac5e5f7cf|72f988bf86f141af91ab2d7cd011db47|1|0|638039888744936819|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=hJxGFzAU5CpYEEojb9wdofOf%2B0bZ%2BZynqtaLWP5KpMI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALQPQRBFWSLVLQ755HD5MMDWIGOAPANCNFSM6AAAAAARUHVIXA&data=05|01|Jose.Franceschi%40microsoft.com|95e18a5dfdb24978e93a08dac5e5f7cf|72f988bf86f141af91ab2d7cd011db47|1|0|638039888744936819|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=zS1m09eVhtNX2c3lRSosqcRkjFzcrkUMeNtTWO%2BbQ4E%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


best result on nuscenes

hi,
i noticed that your rank in nuscenes lidar track is 2
CenterPoint-VID
mAP: 0.674
NDS: 0.718

What config should I use to train the model to achieve this effect?

this one is not my entry

thanks.
Which ranking on the nuscene list is yours now? Can you share the training strategy and related configuration?
@tianweiy

the leaderboard uses old version (and worse). Currently we get 66.8 / 68.5 (single / flip) https://github.com/tianweiy/CenterPoint/issues/249 on val set. The test set should be similar or higher. The configs are already provided.
If you want to get to SOTA performance, you will need fusion based methods

ok, thanks!

the leaderboard uses old version (and worse). The most recent models get 66.8 / 68.5 (single / flip) #249 on val set. The test set should be similar or higher. The configs are already provided.
If you want to get to SOTA performance, you will need fusion based methods (e.g. https://github.com/tianweiy/MVP)

Hi, i want to know does the result of 68.5 use dcn?
thanks!

the leaderboard uses old version (and worse). The most recent models get 66.8 / 68.5 (single / flip) #249 on val set. The test set should be similar or higher. The configs are already provided.
If you want to get to SOTA performance, you will need fusion based methods (e.g. https://github.com/tianweiy/MVP)

Hi, i want to know does the result of 68.5 use dcn? thanks!
@tianweiy

No, I don't think dcn helps much

I basically just run this config with pretrained model https://github.com/tianweiy/CenterPoint/blob/master/configs/mvp/nusc_centerpoint_voxelnet_0075voxel_fix_bn_z_scale.py

I basically just run this config (w/ w/o flip test) with pretrained model https://github.com/tianweiy/CenterPoint/blob/master/configs/mvp/nusc_centerpoint_voxelnet_0075voxel_fix_bn_z_scale.py

can you provide the per-trained model? thanks!

i just use this model https://github.com/tianweiy/CenterPoint/blob/master/configs/nusc/README.md (first one)

so you trained totally 40 epoch?
could you provide your training command? like
python -m torch.distributed.launch --nnodes=1 ./tools/train.py configs/mvp/nusc_centerpoint_voxelnet_0075voxel_fix_bn_z_scale.py --resume_from epoch_20.pth

thanks!

i don't get it. I only trained 20 epochs, I didn't change anything (the config is exactly what i used)

run config < i mean testing

the 68.5 is with flip aug, 66.8 is without flip aug. Flip definitely helps at the cost of 4x latency

does the flip mean double flip?
should i add the double_flip=ture during training and testing

we always do flip aug during training.
By default, we don't do flip aug during testing.
To enable it, we need to add multiple flip flags in the config. See this file for an example https://github.com/tianweiy/CenterPoint/blob/master/configs/nusc/voxelnet/nusc_centerpoint_voxelnet_0075voxel_dcn_flip.py

does the flip aug during testing help improve performance?

yes

the 68.5 is with flip aug, 66.8 is without flip aug.

the 68.5 is with flip aug, 66.8 is without flip aug. Flip definitely helps at the cost of 4x latency

could you provide this test config with flip aug?
thanks!

will do in a bit (after i finish graphics hw, probably 1 to 2 hrs)

i noticed that in https://github.com/tianweiy/CenterPoint/blob/master/configs/mvp/nusc_centerpoint_voxelnet_0075voxel_fix_bn_z_scale.py config.
there is no voxel_generator config, why?

there is no voxel_generator config, why?

it uses dynamic voxelization, so we can remove the old voxelizer

there is no difference, we use separate files for diff paper

will do in a bit (after i finish graphics hw, probably 1 to 2 hrs)

up now


CodePush

Check https://github.com/start-react/native-starter-kit

Not going to add CodePush?

I'm done with open source. I hate it. Send pull request if you want. I can review it.

It's like painting canvas every single day and then burn in. There is no joy in it anymore.

Dude, I get it. Look up redux framework. People just don't care. I'll help add a pull request as I dig in. ;)


#558 - Workaround for enum mapping

Description

Workaround for DelegationChangeType enum
Removed CTE from insert query

Related Issue(s)

#558

Developer/Reviewer Checklist

[ ] Your code builds clean without any errors or warnings
[ ] No changes to config/appsettings or environment variables created in separate linked PR
[ ] Manual testing done (required)
[ ] Relevant integration test added (required for functional changes)
[ ] Relevant automated test added (required for functional changes)
[ ] All integration tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Setting it to close after consent from Marius


[Feature] Creating contact us page.

Description

To create a "Contact Us" page that complements the overall theme and styling of your website.
I will ensure the design is cohesive with your website's color scheme, typography, and layout to maintain brand consistency.
The link to the contact us page will be provided in the footer.

I want to work on this issue kindly assign it to me under GSSOC'24.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Congratulations, @yashi-15! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

One issue at a time @yashi-15
Let's start with #163

I can work on this.

@gdhyani
Okay do one thing. Create a single PR that closes both #165 and #174
I'll add level3 to it.
TODO  :

Maintain the Folder Structure of the Project.
Use Components like Navbar, Footer if necessary.
Contact Us page must maintain the theme of the Popshop.
Make is as Minimalist as possible.
Contact Us page should have only Name, Email, Message with a Send button.

@gdhyani
Okay do one thing. Create a single PR that closes both #165 and #174 I'll add level3 to it.
TODO :

Maintain the Folder Structure of the Project.
Use Components like Navbar, Footer if necessary.
Contact Us page must maintain the theme of the Popshop.
Make is as Minimalist as possible.
Contact Us page should have only Name, Email, Message with a Send button.

Done with all the changes. Created a Pull Request #176.


windows only it seems, right?

add_tags.py gets included, but that one says
requires pywin32 (See requirements.txt. Supports Windows only obviously)
in line3. So, this is Windows only, correct?
In Linux web-ui launches with the usual message
Aesthetic Image Scorer: Unable to load Windows tagging script from tools directory.
probably a reaction to the failed import of add_tags.py
ubuntu 22.04
regardless if installed via UI extensions or git pull.

I added additional logging for this error: Aesthetic Image Scorer: Unable to load Windows tagging script from tools directory. See if it provides anymore insights. pywin32 for windows tags, but it shouldn't cause any problems if you don't try to use windows tags.

I'm also not on Windows and get the same error. Here's the full traceback in case you're still interested in fixing this:
ERROR:root:Aesthetic Image Scorer: Unable to load Windows tagging script from tools directory
Traceback (most recent call last):
  File "/data/apps/stable-diffusion-webui/extensions/stable-diffusion-webui-aesthetic-image-scorer/scripts/image_scorer.py", line 26, in <module>
    from tools.add_tags import tag_files
  File "/data/apps/stable-diffusion-webui/extensions/stable-diffusion-webui-aesthetic-image-scorer/tools/add_tags.py", line 6, in <module>
    import pythoncom
ModuleNotFoundError: No module named 'pythoncom'

I get this error when loading the WebUI which causes the extension to not show up at all. So not caused by trying to use Windows tags. Looks like pythoncom is part of pywin32 which is Windows only and gets loaded on start up even if you're not using any Windows features.

+1 pythoncom windows only
Running Linux:
ERROR:root:Aesthetic Image Scorer: Unable to load Windows tagging script from tools directory
Traceback (most recent call last):
  File "/stable-diffusion-webui/extensions-builtin/stable-diffusion-webui-aesthetic-image-scorer.git/scripts/image_scorer.py", line 26, in <module>
    from tools.add_tags import tag_files
  File "/stable-diffusion-webui/extensions-builtin/stable-diffusion-webui-aesthetic-image-scorer.git/tools/add_tags.py", line 6, in <module>
    import pythoncom

https://pypi.org/project/pywin32/
Per Readme:
After restarting the ui, see settings for options

I did not see anything added to UI under settings or any new tabs.
This project, at least, needs a warning that it is Windows only.


Resolve Issue of Global Object Dependency by Implementing Local Collection in Converter Function

Having a global object was causing a problem in running the code multiple times in series.
This pull request, solves this by creating a collection in converter function and pass it through each create function.

I checked it for PEP8 using autopep8, but maybe it have a problem. can tell me which lines doesn't follow PEP8.


The "Web site settings" page referenced by this page is out of date and missing DevOps Server 2019 configuration options

In step 6 under subsection "Configure using the Basic options", this page references the "Web site settings" page located here: https://docs.microsoft.com/en-us/azure/devops/server/admin/websitesettings?view=azure-devops.
That page was last updated 08/07/2016 and only references "TFS 2017 | TFS 2015 | TFS 2013". Furthermore, on the "Web site settings" page, the options shown do not match what is available for Azure DevOps Server 2019. Please update the "Web site settings" page to reflect Azure DevOps Server 2019 (and TFS 2018, if applicable) configuration options.

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

ID: c41e05dc-914c-cae4-1d16-b4e2fced0915
Version Independent ID: f629cea4-6a41-e3e6-4942-af0caf304255
Content: Install and configure a single server - Azure DevOps Server & TFS
Content Source: https://mseng.visualstudio.com/DefaultCollection/TechnicalContent/_git/tfs-docs-pr?path=%2fdocs%2finstall%2fsingle-server.md&version=GBlive&_a=contents
Product: devops-server
Git Login: aaronhallberg
Microsoft Alias: aaronhallberg

The configure search hyperlink in step 7 points to nowhere.


More info on errors

I've been using Dave Cheney's errors pkg to add stack traces to all my errors. In dev, when I'm testing queries in graphiql/my app, I like to return the error directly in the response. I was wondering if there was any way to return the %+v version of errors back instead of the %s version, because right now, I'm not getting the line numbers of various errors and the rest of the stack trace.

See #49. I think the same solution should be possible in your case. If yes, then I should add some documentation to the readme since this question came up several times now.

Oops, I referenced the wrong issue in the commit. Reopening.

@EmperorEarth Any response to my comment above?

Not to hijack the issue, but the addition of query paths on errors to the specification seems near. See: https://github.com/facebook/graphql/pull/230
At first I was thinking of adding the path value after the query has executed and before we respond to the user, but I think this might be more appropriate in the library.

@neelance So sorry! I'll spend some time on this today and tomorrow.
I've been working on my startup prototype solo so long, I've gotten out of the habit of checking on Github/Slack.

@neelance, if I understand it right, it doesn't solve what I'm trying to do because the param is still %s in Printf, not %+v. But I don't really understand it at all, so?
As a basic example, as I'm rapidly writing code, sometimes I forget to check err == sql.ErrNoRows() before I check err != nil to return a nil resolver. With Cheney's errors package, if I wrap all my errors with the func WithStack, I'll get a line number.
I return all errors, and in my non-resolver code, only log them at the root. With graphql, however, we aggregate errors and proceed on the rest of the resolution as best we can. This is great, except when we return back to the client, in the client's fetch().catch(), console.error(response.error) gets an array of strings with very little contextual information. It gets very tedious to go through the response to see what did/didn't get returned. It would be helpful if the response to the client had the errors printed using %+v instead of %s, so the strings would have a line numbers.
Does that provide more context?

@neelance, if I understand it right, it doesn't solve what I'm trying to do because the param is still %s in Printf, not %+v. But I don't really understand it at all, so maybe it's better if I try to explain the yak I'm shaving?
As a basic example, as I'm rapidly writing code, sometimes I forget to check err == sql.ErrNoRows() before I check err != nil to return a nil resolver. With Cheney's errors package, if I wrap all my errors with the func WithStack, I'll get the full stack trace with line numbers.
I return all errors, and in my non-resolver code, I have an obvious endpoint to log the errors. I don't see that with your library. This is especially relevant because with graphql we aggregate errors and proceed on the rest of the resolution as best we can. What I have right now is an array of minimally helpful error strings sent back to the client as %s.
I guess what I need is threefold:
An understanding how to write a middleware layer just before the aggregated output is sent back to the server, giving me a chance to log my contextualized errors
A way to massage those errors at the end perhaps to not leak too many details back to the client, while still logging them all at my end
The flexibility to continue just returning errors at each of many resolvers, and choose a logging solution at a single place (probably makes sense to make it as late as possible before sending back to the client)

To me this still sounds like you need the same solution as #49. Take the list of errors returned by Schema.Exec and do with them whatever you want before encoding them as JSON and returning them to the client.

@tonyghita, we can close this now
ResolverError solves this, can't remember the PR #


Empty atProvider for loadBalancer

What happened?
 All sub-resources of loadBalancer such as availabilityZone, dnsName, loadBalncaerARN etc under status.atProvider are empty
apiVersion: elbv2.aws.crossplane.io/v1alpha1
kind: LoadBalancer
metadata:
  name: xlb
spec:
  deletionPolicy: Delete
  forProvider:
    loadBalancerType: network
    name: x-lb
    region: us-east-1
    subnets:
    - subnet-02234e0485760df45
  providerConfigRef:
    name: default
status:
  atProvider: {}  <--------
  conditions:
  - lastTransitionTime: "2022-05-16T16:58:48Z"
    reason: Available
    status: "True"
    type: Ready
  - lastTransitionTime: "2022-05-16T16:56:46Z"
    reason: ReconcileSuccess
    status: "True"
    type: Synced

How can we reproduce it?
 Just create a loadBalancer resource with the provider-aws version v0.25.0
What environment did it happen in?
Crossplane version: v1.6.4
Cloud provider: AWS
Kubernetes version:
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.4", GitCommit:"e6c093d87ea4cbb530a7b2ae91e54c0842d8308a", GitTreeState:"clean", BuildDate:"2022-02-16T12:38:05Z", GoVersion:"go1.17.7", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.7", GitCommit:"1f86634ff08f37e54e8bfcd86bc90b61c98f84d4", GitTreeState:"clean", BuildDate:"2021-11-17T14:35:38Z", GoVersion:"go1.16.10", Compiler:"gc", Platform:"linux/amd64"}

Same issue in v0.27.0, it's blocking us as we rely on it to get dns of alb


Edit (Fix): Extract context from diagnostic message markdown

Description
Diagnostic error messages contain useful information. For example:
Argument of type 'Element | null' is not assignable to parameter of type 'Element | DocumentFragment'.\nType 'null' is not assignable to type 'Element | DocumentFragment'.
Currently we just provide this message to Cody, and hope that it is useful for the fix. However we can do a lot more here.
Improvement:

Extract symbol like information from the message, e.g. the above would be ['Element', 'DocumentFragment']
Search current file symbols for an exact match
(If step 2 fails) Search workspace symbols for an exact match
If we have found a symbol from step 2 or 3, include it as context

We should be able to measure if this has a positive impact in persistence rates for "Ask Cody to Fix"

We can do https://github.com/sourcegraph/cody/issues/3519 as part of this too

For context, prototype of this work is here: https://github.com/sourcegraph/cody/pull/3706


Delete Terraform job from Travis

Deleting the Terraform build from travis cron jobs.
However, the same job will stay on our internal Jenkins.
@marstr @jhendrixMSFT

@tombuildsstuff, just a heads up if you didn't see this thread :)
As @mcardosos says above, we're just changing up the procedure of where we run these tests, not actually disregarding Terraform.


Fix exception causes in server.py

I recently went over Matplotlib, Pandas and NumPy, fixing a small mistake in the way that Python 3's exception chaining is used. If you're interested, I can do it here too. I've done it on just one file right now.
The mistake is this: In some parts of the code, an exception is being caught and replaced with a more user-friendly error. In these cases the syntax raise new_error from old_error needs to be used.
Python 3's exception chaining means it shows not only the traceback of the current exception, but that of the original exception (and possibly more.) This is regardless of raise from. The usage of raise from tells Python to put a more accurate message between the tracebacks. Instead of this:
During handling of the above exception, another exception occurred:

You'll get this:
The above exception was the direct cause of the following exception:

The first is inaccurate, because it signifies a bug in the exception-handling code itself, which is a separate situation than wrapping an exception.
Let me know what you think!

Looks like the mypy error in the CI is unrelated to my change.

raise ... from None would be even better in this case, wouldn't it?

The mypy behavior changes are a constant source of happiness.
In this case:

it looks like a bug was fixed, so I no longer need to tell mypy to ignore things it got wrong; progress!
we're hit by a regression caused by a change in typeshed: https://github.com/python/typeshed/pull/4079; perhaps making the code in websockets conditional on sys.platform, like the typeshed code, could help.

Coming back to the original issue, I'm in favor of making changes wherever websockets changes the type of an exception.
We should choose between raise ... from exc or raise ... from None depending on whether the original exception is likely to add useful information for debugging.

Regarding the mypy issue: I'll rebase and see what happens.
Regarding from None: I personally hate it, and I think that when people are debugging they have a very different mindset than when they are writing code. When you're debugging, and you get a very limited amount of information that may have been copy-pasted in an email by a non-technical person, you're very desperate for the information.
In this case, I would have wanted to see the cast(Origin, headers.get("Origin")) traceback. I would maybe get a hint on what happened. If it's a dead end for exploration, I at least want to know its content before I rule it out as a dead end.
Another issue is that some error-reporting systems show not only the traceback, but all the local variables on each of these levels. These supposedly irrelevant levels could have variables that make it clearer what this exception is about.
My policy is to never add from None. I'm okay with you marking the places where you'd prefer from None, and I'll just remove them from my PR(s) and keep the lines we agree about, so you could do the from None on a separate PR.

I fixed the mypy failures in master just now.

Style detail: I'd rather use as exc / from exc than as e / from e, for consistency with the current style.
Happy to look at a patch fixing other instances besides this one :-)

Fixed and rebased. Would you be okay with the other instances being fixed in a separate PR after this one is merged?

Cool :) Expect another PR soon.


Asynchronous task timeout issue

I have deployed a django app on using Zappa and I have an asynchronous task which is executed from admin actions. Sometimes when I does the action I get "endpoint request timeout" error in the browser and the task still gets executed in the background. But sometimes no error is shown and the task executes as expected.
Please let me know what could be this intermittent issue where timeout error occurs

If you are using API Gateway, the API gateway has a 30 second timeout in front of the lambda.
If your lambda process takes longer than 30 seconds, you'll see a timeout from the browser, but the process will continue and not timeout as long as it finishes within the max set lambda execution time.

instead of adding @task decorator to the main action function, use it on the inner function you are calling as below.
def my_action():
    some_func()

@task
def some_func():
   ...

If you share a bit more of your code, we may be able to help a bit more.
What is the request being made that results in calling your async function?

def update_video_visibility_task(video_id_list, visibility):
"""
Zappa async task to execute visibility update function
:param video_id_list:
:param visibility:
:return:
"""
update_video_visibility(video_id_list, visibility)
@task
def update_video_visibility(video_obj_id_list, visibility_to_update):
"""
Updates the privacy status of passed video ids in youtube, (private/public/unlisted)
:param video_obj_id_list:
:param visibility_to_update:
:return:
"""
video_queryset = YoutubeVideo.objects.filter(id__in=video_obj_id_list)
for video in video_queryset:
account = video.user
api_service_name = "youtube"
api_version = "v3"
credentials = Credentials(token=account.access_token, refresh_token=account.refresh_token,
client_id=account.client_id, client_secret=account.client_secret,
token_uri=account.token_uri)
client = build(api_service_name, api_version, credentials=credentials, cache_discovery=False)
request = client.videos().update(
part='status',
body={
"id": video.video_id,
"status": {
"privacyStatus": visibility_to_update
}
}
)
request.execute()
video.visibility = visibility_to_update
video.save()
def make_videos_private(modeladmin, request, queryset):
    """
   admin action which calls the function that executes async task
    """
    update_video_visibility_task(list(queryset.values_list('id', flat=True)), YoutubeVideo.PRIVATE)
    messages.info(request, "Videos are being updated in background, please check in few seconds")

This is my code where I'm making youtube api requests within the async task. But even calling the async function results in timeout error but the task gets executed in the background.

Ok, you code doesn't look like it should take long.
Is your lambda in a VPC?  If so you will need a NAT to make calls to the internet (youtube).
If you don't these calls will timeout because they can't reach the destination.

Not enough information to resolve.  Closing.
It seems like the task requested task is exceeding the max set lambda execution time, causing the timeout.
First configure your lambda to timeout at the max 15 minutes.
If it exceeds this limit, your lambda will and is expected to timeout.
In which case, you need to adjust your code so that it finishes in the max time allotted to lambda.


3609: omit idps with non supporting profiles on picker page

If an IDP is not able to support any of the evidence profiles that a user has declared to us then we will no longer list them on the /choose-a-certified-company page. If we feel that an IDP supports some of the user's evidence profile and is likely to verify the user then we mark the IDP as recommended, but if we don't feel they are  likely verify while supporting some of the user's evidence profiles then the IDP is display within the unlikely to verify group.
To do this we have created two tiers of profiles (recommended and non-recommended) which allow for the grouping stated above. If an IDP is not found to be in either of these groups when the user's evidence is supplied then it is omitted from the page.
In addition to these changes we have performed a small refactor of the code primarily aimed at renaming rules to profiles.

There are a few instances of rules left actually, might be worth a quick grep to clean them up.

LGTM when it's rebased and any rules -> profiles clean up that can be done is

Great, thanks for doing all that rebasing!


Add .npmignore

Source is not currently intended to be consumed from a Node.js application and should be excluded - dist folder is all that should be included in the package.

https://github.com/Azure/azure-mobile-apps-js-client/issues/158 and https://github.com/Azure/azure-mobile-apps-js-client/issues/88 are related issues. Plan is to stop bundling the code in the dist directory and instead make it possible to consume the repository as a nodejs package.

Ahh... We should still package a built version so that web can consume it. We should also be updating the CDN, something we haven't done since 1.3.2. I can't remember exactly where it's located, I'll dig it up.

https://github.com/Azure/azure-mobile-apps-js-client/pull/197 closes this.


Fail to update Maven packages with properties in their names

If a Maven dependency uses a property in its artifactId or groupId, the writer does not find the original definition of the package and will end up creating a new section for the patch.
e.g. Starting with this:
<properties>
  <artifact>foo</artifact>
</properties>
<dependencies>
  <dependency>
    <groupId>com.xyz</groupId>
    <artifactId>${artifact}</artifactId>
    <version>1.0.0</version>
  </dependency>
</dependencies>

A patch to com.xyz:foo will end up looking like:
<properties>
  <artifact>foo</artifact>
</properties>
<dependencies>
  <dependency>
    <groupId>com.xyz</groupId>
    <artifactId>${artifact}</artifactId>
    <version>1.0.0</version>
  </dependency>
</dependencies>
<dependencyManagement>
  <dependencies>
    <dependency>
      <groupId>com.xyz</groupId>
      <artifactId>foo</artifactId>
      <version>2.0.0</version>
    </dependency>
  </dependencies>
</dependencyManagement>

Which, in this case, does not actually override the version of the package (despite the override claiming it would).

"If a Maven
dependency uses a property in its artifact Id
or group Id,
the writer does not find the
original definition of the package and
will end up
creating a new section
for the patch.
Starting with
</this><properties>
  <artifact>foo
</artifact></properties>
<dependencies>
  <dependency><groupId>
<{com.xyz}></groupId>
<artifactId>
</ $ artifact></artifactId>
<version> 1 </version>
</dependency>
</dependencies>A patch to com.xyz:foo
will end up</looking like:>
<properties>
<artifact>
foo </artifact>
</properties>
</dependencies>
<dependency>
<groupId>
</com.xyz></groupId>
<artifactId></$artifact</artifactId>
<version>: 1 </version>
</dependency>
</dependencies><dependencyManagement>
<dependencies><dependency>
<groupId>com xyz</groupId>
 <artifactId>
</foo></artifactId><version>
             2 </version>
</dependency>
</dependencies>
</dependencyManagement>Which,
in this case, does not actually override
the  version of the package </despite
 the override claiming it would

let me know if this was the answer you were looking for


Ivez won an award ...


My twitter (@newberlin) proof has failed

Hello, I've tried this twice and it's still not showing. Please see twitter.com/newberlin

The screenshot shows your twitter account is locked/private.
You need to have a public twitter profile for Keybase to be able to "see" the verification tweet

Thank you! I tried that and it worked ;)
On Thu, May 14, 2020 at 6:12 AM Joshua<EMAIL_ADDRESS>wrote:

The screenshot shows your twitter account is locked/private.
You need to have a public twitter profile for Keybase to be able to "see"
the verification tweet
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/24148#issuecomment-628625130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3QEAAA2QAWV2VHDIW3ESTRRPU3FANCNFSM4NAGGSIQ
.

--
PRIVILEGED AND CONFIDENTIAL COMMUNICATION: This E-mail, and any attachments
thereto, is intended only for use by the addressee(s) named herein and may
contain legally privileged and/or confidential information. If you are not
the intended recipient of this E-mail, you are hereby notified that any
dissemination, distribution or copying of this E-mail, and any attachments,
is strictly prohibited.


scrape the restaurant logo icons too

many restaurants has their logos only on social media profiles.
goal: the daily menu app needs to make sure the logo is uptodate

first it would make sense to check if the old logo is still available: https://github.com/GoogleChrome/puppeteer/issues/1205
if it isn't => {restaurantName} + " restaurant logo" Google image search

since the publishing of the tool no restauran logos were change, if one would: it can be replaced manually


Bonfire: Convert HTML Entities - assert test doesn't call function

http://www.freecodecamp.com/challenges/bonfire-convert-html-entities
the test never passes because it's not calling the Convert() function

pull request submitted to fix the issue


FIX BUG 63394

Description
Hello,
UbikLoadPack team is happy to contribute this bug fix.
Qian Qian and Philippe
Regards
@ubikloadpack
https://ubikloadpack.com
Motivation and Context
Fixes:
https://bz.apache.org/bugzilla/show_bug.cgi?id=63394
How Has This Been Tested?
Manual and JUnit tests
Screenshots (if appropriate):
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

[X ] My code follows the code style of this project.
[] I have updated the documentation accordingly.

Codecov Report

Merging #458 into trunk will increase coverage by 0.11%.
The diff coverage is 70.73%.

@@             Coverage Diff              @@
##              trunk     #458      +/-   ##
============================================
+ Coverage     61.18%   61.29%   +0.11%
- Complexity    11464    11491      +27
============================================
  Files          1220     1221       +1
  Lines         77978    78019      +41
  Branches       7527     7528       +1
============================================
+ Hits          47714    47825     +111
+ Misses        27563    27497      -66
+ Partials       2701     2697       -4

Impacted Files
Coverage Δ
Complexity Δ

src/core/org/apache/jmeter/JMeter.java
42.93% <0%> (+5.19%)
50 <0> (+5)
:arrow_up:

test/src/org/apache/jmeter/JMeterTest.java
74.35% <74.35%> (ø)
4 <4> (?)

src/core/org/apache/jmeter/util/JMeterUtils.java
46.21% <0%> (+0.5%)
66% <0%> (+2%)
:arrow_up:

...org/apache/jmeter/engine/StandardJMeterEngine.java
58.47% <0%> (+0.69%)
41% <0%> (+1%)
:arrow_up:

src/core/org/apache/jmeter/save/SaveService.java
72.58% <0%> (+2.15%)
40% <0%> (+1%)
:arrow_up:

...c/core/org/apache/jmeter/reporters/Summariser.java
90.83% <0%> (+4.58%)
22% <0%> (+3%)
:arrow_up:

.../jorphan/org/apache/jorphan/util/JOrphanUtils.java
70.4% <0%> (+5.2%)
67% <0%> (+5%)
:arrow_up:

src/core/org/apache/jmeter/util/JMeterVersion.java
78.94% <0%> (+5.26%)
3% <0%> (+1%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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

A few other comments on this.

The security manager doesn't get removed after use.
The tests should probably be run in isolation, e.g. implement JMeterSerialTest.
They are quite verbose, as the errors are currently printed to System.out

Hi Felix,
We have taken into account your feedback.
Philippe on behalf of UbikLoadPack Team
@ubikloadpack


[HUDI-2763] Metadata table records - support for key deduplication and virtual keys

What is the purpose of the pull request

The backing log format for the metadata table is HFile, a KeyValue type.
Since the key field in the metadata record payload is a duplicate of the
Key in the Cell, the redundant key field in the record can be emptied
to save on the cost.

HoodieHFileWriter and HoodieHFileDataBlock will now serialize records
with the key field emptied by default. HFile writer level relies on the
callers to tell about the key field in the schema.

HoodieHFileReader when reading the serialized records back from disk,
it materializes the missing keyFields if any. It relies on the callers
to tell about the key field in the record schema.

WriteHandles and all its derived classes rely on the table properties
for the key field when constructing the file and log readers. This way
base file creation, append and merging all work seamlessly irrespective
of data or metadata table.

NOTE: There is a simplified version of this PR at https://github.com/apache/hudi/pull/4449
where the HFile reader and writer would do key deduplication and key materialization only if
the record has the metadata payload schema field 'key' (the hardcoded version).
Verify this pull request

Tests have been added to verify the default virtual keys and key
deduplication support for the metadata table records.

Manually verified the serialized records on the disk are trimmed
off the key field

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

We have decided to go with https://github.com/apache/hudi/pull/4449 approach instead this PR approach.


JSCSFormatter can't exclude specific files configured in the .jscsrc file.

I have a .jscsrc file on my project root path, I installed the newest JSCS  globally, I am using the jscs-formatter plugin of sublime text 3, but the JSCS just can't skip the exculdeFiles each time I press ctrl + s key to save file , any other rule works well. Here is my config detail.
{
"preset":"airbnb",
"fileExtensions": [".js", ".jsx"],
"fix": true,
"requireSemicolons": null,
"disallowMultipleSpaces": true,
"disallowSemicolons": true,
"disallowSpaceBeforeComma": {"allExcept": ["sparseArrays"]},
"excludeFiles": ["dist/",  "coverage/"]
}

Hmm.
When you run jscs on explicitly that file from the command line, does it fix it even if it is in the excludeFiles?
If so, this sounds like a JSCS bug. JSCS-Formatter just simply calls JSCS to fix the currently opened file. It sounds like for this to work, you want JSCS when given a specific file and the --fix flag to not fix the file if it is in the ignored things in excludeFiles

The JSCS CLI works well to exclude files what I confiured in the .jscsrc file , I think It is the jscs-formatter's bug.

Does this occur when you manually call the formatter? As in, not using the format on save feature?

Yes!

Hey @handane123, can you supply the console output from Sublime text: View -> Show Console after you run jscs-formatter?

Using node.js path on 'windows': node.exe
Using jscs path on 'windows': %APPDATA%/npm/node_modules/jscs/bin/jscs

I just found that  jscs just can be used on the project root path to function properly,if used in subdirectory, then jscs can't recognize the excludeFiles rule. So my suggestion is that jscs-formatter should call jscs on the project root path.

JSCS should probably work relative to the config file it finds instead of the formatter using a different CWD. @mrjoelkemp, do you agree?

It took me two hours to figure out! The problem is that you pass the string data to jscs,it looks like

So the jscs just ignored the config rules.
I make some little changes of  the ""JSCS-Formatter.py""

line 80: cdir = os.path.dirname(self.view.window().project_file_name())
add one line code below line 80: cmd.append(self.view.file_name())
line 158: stdout, stderr = p.communicate(input=None)
can remove the argument data of the function get_output()

I tried jscs in the subdirectories which I excluded in the .jscsrc file , it  now skips the file what I want.
And when used  upon the the normal file , it also works well to format code.
Everything is done!

Yeah, I'm not sure how we want to go about this. If we pass the file, we will be unable to format only the selected text. It would also cause JSCS to save the file which would cause sublime to reload it. I think that experience would be a little less than ideal.
@handane123 Can you open a PR with your changes? It would be good to take a look and have them to consider.

Thanks for investigating @handane123!
Jscs is doing the right thing in my opinion; we pass a portion of a file and it doesn't know that the origin should have been excluded.
Perhaps the plugin should prevent sending portions of an excluded file. Otherwise, jscs needs a special behavior when given piped text and a file argument.

I think this plugin trying to find and parse a configuration file is a recipe for pain and I know JSCS isn't adding new functionality since they are merging with eslint. I'm not sure there is a path forward here.

Finding the jscsrc file is straightforward but parsing (non-json) configs would definitely be a headache.
Unfortunately, I also don't see a worthwhile path here. Sorry @handane123.


[request] PowerToys Run, execute hotkeys, switch to desktop

Description of the new feature / enhancement
Hello,
would be nice if a new feature could be implemented to send key or hotkeys, example: you type "win+d" => switch to desktop, "win" => opens start menu, "ctrl+shift+esc" => opens task manager, and so on.
Then also for the window switch, could you also please implement a "switch to desktop" / minimize all in the list.
Scenario when this would be used?
This would be really handy to use, especially if you command the PC through remote or game streams, and could then easily maneuver through Windows, where some hotkey bindings are not natively available.
Supporting information
No response

/dup #5668

Ah, I read that as you key "win+d" and all of those actions take place, like a macro. Are you asking for the ability to search for and execute commands like: show desktop, open start, open Task Manager, etc. from PT Run?
Could also be a duplicate of #11048

Here is a use case for this request:
I am using Moonlight/Sunhine to game streams from a host PC to client PCs. On the host I have PowerToys with module Run. So I can trigger alt+space through Moonlight (remote client) on the host. It would be handy to execute hotkeys on the host via the Run module this way, for example you tyle in (some modifier like++) "++alt+d" in the Run gui and this will execute alt+d on the host, or any other key combination, or "++win" and executes win key. It would also be handy to have important hotkeys alreday build into the Run gui for the window switch feature, like having a button for window key.


calendar usage changes

Change control to swipe left/right for next/previous month instead of touch left/right side
Add up/down swipe for next/previous year
Display scheduler events for current month on touch

Thanks!


[RFC]: Add @stdlib/random/array/rayleigh

Description
This RFC proposes adding support for creating an array containing pseudorandom numbers drawn from a Rayleigh distribution. This is the array creation analog of @stdlib/random/base/rayleigh.
Package: @stdlib/random/array/rayleigh
Alias: rayleigh
Related Issues
None.
Questions
No.
Other

A package with a similar implementation/API design is @stdlib/random/array/exponential.

Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

Closed by 2dc4be898dd31d35c294c7bc8f35753313e64332.


CLI: remove --no-logging flag

The listen command at one point supported a mode where it would skip logging requests in Play's UI.
Today, this doesn't seem to be supported (Play rejects the request because it says the token is invalid).
Rather than work to repair this in Play, we're just going to remove the flag.

This one is ready for review, but sadly this is the 3rd PR in a chain. Will likely need another ✅ after the others have landed.


PrivateFile::stream_content doesn't honor the index for inline content

PrivateFile::stream_content() takes an index parameter that is passed to the private forest content when the file content is external. However in the case of inline content, the whole vector is yieled in https://github.com/wnfs-wg/rs-wnfs/blob/6df4a6bf89a71010b7bc15c132512da979ebc438/wnfs/src/private/file.rs#L489
I think we should instead yield data[index..].to_vec()

Sorry. The index actually refers to the block index, not the byte index. We should fix the docs and provide an abstraction on top that does the conversation between block and byte indices.


Missing documentation

More documentation is needed to use this package in a comfortable way. For example:

Nothing about configuring the JSON schema loading is explained in the README file.
Nothing about which "phrases" provide the contexts of this package

Hello,
I guess you're right ! may be you could send your proposition through a PR and will discuss it next ?