./bootstrap-vcpkg.sh  does not build vcpkg on  WSL

Below is the output of ./bootstrap-vcpkg.sh
rajnish@Audi-A10:/mnt/c/proj/vcpkg$  ./bootstrap-vcpkg.sh
Downloading cmake...
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 32.6M  100 32.6M    0     0  1140k      0  0:00:29  0:00:29 --:--:-- 1494k
Downloading cmake... done.
Extracting cmake...
Extracting cmake... done.
Downloading ninja...
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   604    0   604    0     0    270      0 --:--:--  0:00:02 --:--:--   270
100 77854  100 77854    0     0  18950      0  0:00:04  0:00:04 --:--:--  172k
Downloading ninja... done.
Extracting ninja...
Extracting ninja... done.
-- The C compiler identification is GNU 4.8.5
-- The CXX compiler identification is GNU 6.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/g++-6
-- Check for working CXX compiler: /usr/bin/g++-6 -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- Configuring done
-- Generating done
-- Build files have been written to: /mnt/c/proj/vcpkg/toolsrc/build.rel
[10/59] Building CXX object CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o
FAILED: CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o
/usr/bin/g++-6  -DDISABLE_METRICS=0 -I../include -O3 -DNDEBUG   -std=c++1z -pthread -MD -MT CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o -MF CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o.d -o CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o -c ../src/vcpkg/binaryparagraph.cpp
../src/vcpkg/binaryparagraph.cpp:27:5: error: function ‘vcpkg::BinaryParagraph::BinaryParagraph()’ defaulted on its redeclaration with an exception-specification that differs from the implicit exception-specification ‘’
BinaryParagraph::BinaryParagraph() noexcept = default;
^~~~~~~~~~~~~~~
[15/59] Building CXX object CMakeFiles/vcpkg.dir/src/vcpkg/commands.autocomplete.cpp.o
ninja: build stopped: subcommand failed.
cp: cannot stat ‘/mnt/c/proj/vcpkg/toolsrc/build.rel/vcpkg’: No such file or directory
rajnish@Audi-A10:/mnt/c/proj/vcpkg$
Thanks,
--Rajnish Chauhan

I was able to build successfully on Ubuntu just now. I'm using GCC 7 as compared to your GCC 6, but I doubt that is causing the issue.
It looks like you're building on the "mounted" windows volume. What if you try building on the Linux filesystem? The Linux user's home folder for instance.
You may also try git pull && git reset HEAD --hard and try the build again. I know the Microsoft team fixed a few things in the bootstrap script earlier.

I can't seem to repro this issue with g++-6 or g++-7.
Since the compiler seems to be complaining, you can try installing g++-7 like this:
sudo add-apt-repository ppa:ubuntu-toolchain-r/test -y
sudo apt-get update -y
sudo apt-get install g++-7 -y

(this is in fact suggested by bootstrap if no g++ >= 6 is found, but in your case something was found)
Then run bootstrap again (g++-7 will be preferred over g++-6).

Update to g++-7 using steps below , still got error.
sudo add-apt-repository ppa:ubuntu-toolchain-r/test -y
sudo apt-get update -y
sudo apt-get install g++-7 -y
rajnish@Audi-A10:/mnt/c/proj/vcpkg$ ./bootstrap-vcpkg.sh
-- The C compiler identification is GNU 4.8.5
-- The CXX compiler identification is GNU 7.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/g++-7
-- Check for working CXX compiler: /usr/bin/g++-7 -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- Configuring done
-- Generating done
-- Build files have been written to: /mnt/c/proj/vcpkg/toolsrc/build.rel
[11/59] Building CXX object CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o
FAILED: CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o
/usr/bin/g++-7  -DDISABLE_METRICS=0 -I../include -O3 -DNDEBUG   -std=c++1z -pthread -MD -MT CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o -MF CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o.d -o CMakeFiles/vcpkg.dir/src/vcpkg/binaryparagraph.cpp.o -c ../src/vcpkg/binaryparagraph.cpp
../src/vcpkg/binaryparagraph.cpp:27:5: error: function ‘vcpkg::BinaryParagraph::BinaryParagraph()’ defaulted on its redeclaration with an exception-specification that differs from the implicit exception-specification ‘’
BinaryParagraph::BinaryParagraph() noexcept = default;
^~~~~~~~~~~~~~~
[16/59] Building CXX object CMakeFiles/vcpkg.dir/src/vcpkg/commands.ci.cpp.o
ninja: build stopped: subcommand failed.
cp: cannot stat ‘/mnt/c/proj/vcpkg/toolsrc/build.rel/vcpkg’: No such file or directory
Thanks.

Did 'git pull' again. Then build was successful.
rajnish@Audi-A10:/mnt/c/proj/vcpkg$ ./bootstrap-vcpkg.sh
-- The C compiler identification is GNU 4.8.5
-- The CXX compiler identification is GNU 7.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/g++-7
-- Check for working CXX compiler: /usr/bin/g++-7 -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- Configuring done
-- Generating done
-- Build files have been written to: /mnt/c/proj/vcpkg/toolsrc/build.rel
[59/59] Linking CXX executable vcpkg
Thanks.


Add react-native run-macos command

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

This change adds a react-native run-macos command similar to run-ios and run-android.   It will build the macOS app and then launch it.
The change also modified the macos-generator command to output an updated final message that informs the user to run via npx react-native run-macos.
A minor bug in the macos xcodeproj template is fixed: it was creating an App bundle Id in the form org.reactjs.native.AppName.AppName instead of org.reactjs.AppName.AppName.
The new run-macos command is registered in react-native.config.js for macos platform.
scripts/packager.sh was updated so that when the "Launch Packager" custom build step in React runs, it correctly starts the packager in react-native-macos mode similar to what yarn start:macos does.
This command is patterned after the run-ios command which is implemented at https://github.com/react-native-community/cli/tree/master/packages/platform-ios/src/commands/runIOS.
Focus areas to test
npx react-native init AwesomeApp
cd AwesomeApp
npx react-native-macos-init
npx react-native run-macos

Microsoft Reviewers: Open in CodeFlow

Nice ✨


feat(providers): adding the Aurora native mainnet and testnet RPC

Description
This PR adds the Aurora Mainnet and Testnet RPC as a new Aurora provider.
Resolves #463
How Has This Been Tested?

Manually:

Start the server locally by cargo run.
Request the Aurora RPC locally by

curl -X POST "http://localhost:3000/v1?chainId=eip155:1313161554&projectId=XXX" --data '{"id":"1","jsonrpc":"2.0","method":"eth_chainId","params":[]}'

CI

Integration test for the Aurora provider.

Due Diligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

Can we merge this one? :)

Can we merge this one? :)

Hello @Karkunow!
We are finishing testing our updated providers CI tests and this PR will be merged asap. ETA Jan.18


Add a response bar to the bottom of the timeline when you tap on a cell

Dev Notes:
https://stackoverflow.com/questions/31818169/expand-and-contract-tableview-cell-when-tapped-in-swift

Should add "Like, Repost, Reply, Share and More" options.
Like and Repost create instant Micropub Requests
Reply does the same thing as swipe right
Share opens up the iOS Action Menu
More opens up a context menu based on https://github.com/GitHawkApp/ContextMenu
The context menu should include additional options such as:
"Remove post"
"Mark all posts below this as read"
"Mark all posts above this as read"
probably more as I think of them

If a response is used from the response bar, it should mark the post as read


The translation for Turkey in French should be Turquie not dinde (which is the bird)

Sorry, some of the translations may be incorrect) Most of names I trying to verify by myself... But it's very hard to verify all country names on all langs by one hand. If you native French speaker you can review all countries and make a PR to update those which wrong.

Will do in the next days. Thanks for the quality package.

Up


Layout Cycle Crash in ItemsRepeater inside ListViewItem

Describe the bug
An ItemsRepeater inside of a ListViewItem datatemplate causes a crash when trying to scroll due to a layout cycle.
Its easy to repro if the window height is small like shown in the video and there are 2 (more than 1?) items in the item repeater that both are larger than the window height.
Steps to reproduce the bug

Steps to reproduce the behavior:

Run the test app.
ItemsRepeaterCrash.zip

Make the height of the window small like shown in the video.

Scroll down and you should hit the crash.

Expected behavior
Should scroll normally. No Crash.
Screenshots
ReproCrash.mp4.zip
callstack.txt
Version Info
NuGet package version:
[Microsoft.UI.Xaml 2.2.190830001]

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2019 Update (18362)
Yes

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context

ListView and ItemsRepeater do their own corrections and anchoring which at times can conflict with each other especially if the growth is in the same direction. Please nest ItemsRepeater instead which knows how to work when nested.


Ftp active mode not work

I can't connect to an FTP server using an active mode i get message "read timed out" but when i try other ftp apps it worked ! , i use 4g network .

What FTP service you are using? Can I get a demo user account for testing purposes?


bug with strings

dotize.convert("foo") ==> {0:"f",1:"o",2:"o"} // expect "foo"
dotize.convert("foo","foo") ==> { foo.0: "f", foo.1: "o", foo.2: "o" } // expect {foo:"foo"}

Hi,
I updated the plugin for that support and added some mocha tests, take a look at that.

Yep, looks good! Well done!

Thank you for pointing out.
Cheers


Improve error handling for wrong formatted products

Sadly, people do not understand the importance and beauty of book keeping and thus bad formatted products are present in our organization. For the time being a fairly basic error handling are present in the Zettle parser, this could use some touch ups and improvements.

Is there a guide for naming products (both for zettle and swish) available for the different utskotts? Maybe that could be a first step?
Could one make a program that checks each product in out inventory and reports those that are badly formatted?

Is there a guide for naming products (both for zettle and swish) available for the different utskotts? Maybe that could be a first step?
Could one make a program that checks each product in out inventory and reports those that are badly formatted?


mobx demo of the groupchats tab

This is the same as the mobx-state-tree demo, but without the mobx state tree. Biggest differences:
action methods are not bound to models, so they need to be imported separate from the state
no obligation to mutate in 'actions'
no runtime type checking
less opinionated regarding to structure of the model layer
discuss in #399

Is the idea here that whenever you need state to work with async operations (in our case, mostly grpc API calls with the clients in the service layer), you create an observable bit of state for those group of components. Then the components that are wrapped in observers will then update whenever the relevant state changes?

Whether the operations is async or not does not matter. It will work. The rest is correct: The changed value needs to be observed. The component needs to be wrapped in an observer. One can work with pieces of (local component) state, or make one big local state just like redux and wrap that in observe().

And indeed only state being used in a component will cause rerender on change.

@navandstokes do you have any thoughts on this vs redux?

@navandstokes do you have any thoughts on this vs redux?

Just went through the issue discussion and responding there!

Same closing reason as #398


FaceMesh jawline moves significantly when (partly) occluded

System information

OS Platform and Distribution: Mac OS 10.15.7
MediaPipe version: 0.8.5
Solution (e.g. FaceMesh, Pose, Holistic): FaceMesh
Programming Language and version ( e.g. C++, Python, Java): Python 3.8.10

Describe the expected behaviour:
Ideally, the face landmarks should be stable when (partly) occluded, while tracking. Unfortunately, if an occlusion is introduced, the landmark positions jump around. This is especially evident close to the jawline.
Standalone code you may have used to try to get what you need :
Just used the Python code available here
Other info / Complete Logs :
If more information is needed, I can attach pictures.

Hi @Grieverheart, Could you please record a video while reproducing the above behavior to understand the issue.

@sgowroji I updated the original post with an animated gif showing the issue.

There are many situations that may increase the error of face landmarks. Occlusion is definitely one of them. Others like some strange behaviour may also increase the error.  This face landmark model also does not recognize eye closure.
I think this is mainly caused by the shortage of corner samples in the training dataset. It will take a lot of time to refine the model as it is very hard to cover every corner situation in the training dataset.
Meanwhile, I don't think accurate landmark localization is a major target of mediapipe.


Be more explicit when we are running a CPUs older then 10 years

Describe the bug
Due to the instructions needed by DPDK, we only support icore7 and upwards. When we receive SIGILL kubectl logs is silent about it (as we crash hard) During startup, it would be nice if we were to detect this and crash more friendly in away.
To Reproduce
start mayastor on non supported CPU
Expected behavior
Nice log message describing the problem right now we need to look at exit code from k8s

@proegssilb we have updated our docs and wanted to check if there anything left for this issue based on your findings.

I've yet to succeed in getting the code running as described. Go ahead and close this out if it's not a huge concern.


Angular app converted to @angular/ssr will not proxy requests to existing API

Command
other
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Angular app converted @angular/ssr application to dynamically render pages of type /page/:id and optimize SEO. I have an existing local API made with Go and Gin Routing, and it is running at http://localhost:8081.
I have a server.ts file which was created by @angular/ssr with a proxy middleware from the http-proxy-middleware package. Here is the file:
import 'zone.js/node';

import { APP_BASE_HREF } from '@angular/common';
import { CommonEngine } from '@angular/ssr';
import * as express from 'express';
import { existsSync } from 'node:fs';
import { join } from 'node:path';
import AppServerModule from './src/main.server';
const { createProxyMiddleware } = require('http-proxy-middleware');

// The Express app is exported so that it can be used by serverless Functions.
export function app(): express.Express {
  const server = express();
  const distFolder = join(process.cwd(), 'dist/smartassreviews/browser');
  const indexHtml = existsSync(join(distFolder, 'index.original.html'))
    ? join(distFolder, 'index.original.html')
    : join(distFolder, 'index.html');

  const commonEngine = new CommonEngine();

  server.set('view engine', 'html');
  server.set('views', distFolder);

  // Example Express Rest API endpoints
  // server.get('/api/**', (req, res) => { });
  // Serve static files from /browser
  server.get('*.*', express.static(distFolder, {
    maxAge: '1y'
  }));

  // All regular routes use the Angular engine
  server.get('*', (req, res, next) => {
    const { protocol, originalUrl, baseUrl, headers } = req;

    commonEngine
      .render({
        bootstrap: AppServerModule,
        documentFilePath: indexHtml,
        url: `${protocol}://${headers.host}${originalUrl}`,
        publicPath: distFolder,
        providers: [{ provide: APP_BASE_HREF, useValue: baseUrl }],
      })
      .then((html) => res.send(html))
      .catch((err) => next(err));
  });

  return server;
}

function run(): void {
  const port = process.env['PORT'] || 4000;

  // Start up the Node server
  const server = app();
  const pathFilter = function (path: any, req: any) {
    return path.match('^/api');
  };
  server.use('/api',
    createProxyMiddleware({
      target: 'http://localhost:8081/api',
      pathFilter: pathFilter,
      changeOrigin: true,
      logLevel: 'debug',
      logger: console,
      onProxyReq: function(proxyReq: any, req: any, res: any) {
          //  log the req
          console.log('RAW REQUEST from the target', JSON.stringify(req.headers));
      },
      onProxyRes: function (proxyRes: any, req: any, res: any) {

          //  log the response
          console.log('RAW Response from the target', JSON.stringify(proxyRes.headers));
      }
    }));
  server.listen(port, () => {
    console.log(`Node Express server listening on http://localhost:${port}`);
  });
}

// Webpack will replace 'require' with '__webpack_require__'
// '__non_webpack_require__' is a proxy to Node 'require'
// The below code is to ensure that the server is run only when not requiring the bundle.
declare const __non_webpack_require__: NodeRequire;
const mainModule = __non_webpack_require__.main;
const moduleFilename = mainModule && mainModule.filename || '';
if (moduleFilename === __filename || moduleFilename.includes('iisnode')) {
  run();
}

export default AppServerModule;

As you can see, requests of type "http://localhost:4000/api" should be proxied to "http://localhost:8081/api". However, this is not what happens, in fact my local api does not get any of these requests at all. Instead, I get errors of this type:
ERROR RuntimeError: NG04002: 'api/pages/products'
    at Recognizer.noMatchError (/Users/kacylombard/Desktop/smartassreviews/client/smartassreviews/dist/smartassreviews/server/main.js:1:2773346)

So essentially, my basic API callouts in my Angular components are not working. For example, this callout does not work:
 async ngOnInit() {
    this.spinner.show();
    try {
      const data = await this.exploreService.getAllReviewPages(this.pageNum)

In fact, this API callout returns an entire HTML file representing the current page I'm on, when it's actually supposed to return a list of JSON objects.
Minimal Reproduction
@angular/ssr package added 17.0.9
Exception or Error
Angular ssr not proxying to existing local backend API via http-proxy-middleware.

Your Environment
17.0.8

Anything else relevant?
No response

The problem here is that the /api route is declared after '*', which causes the /api requests not the catched by the proxy middleware.
See https://expressjs.com/en/guide/routing.html for more information about express server routing.
Closing as this is working as expected.


When will FP8 be available for Mixtral?

Could you guys share rough timeline on the support of FP8 quantization for Mixtral (MoE) model?
cc: @Tracin

Mixtral with FP8 is almost done for now, will be released in version 0.9.0

Does Mixtra support AWQ 4-bit?

Mixtral with FP8 is almost done for now, will be released in version 0.9.0

Hi, it seems that I still can't export an fp8 quantized version of Mixtral 8x7B in version 0.9.0
Is there something I missed in the doc or is this still not supported ?
When using the python script quantized.py like this:
python /app/tensorrt_llm/examples/quantization/quantize.py \
   --model_dir mistralai/Mixtral-8x7B-Instruct-v0.1  \
   --output_dir /workspace/checkpoint/mistralai_Mixtral-8x7B-Instruct-v0.1/tllm_checkpoint_fp8 \
   --qformat fp8 \
   --tp_size 1  \
   --pp_size 1  \
   --kv_cache_dtype fp8

it exit with an error:
Warning: this is an old NPZ format and will be deprecated soon.
Traceback (most recent call last):
  File "/app/tensorrt_llm/examples/quantization/quantize.py", line 52, in <module>
    quantize_and_export(model_dir=args.model_dir,
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/quantization/quantize_by_ammo.py", line 360, in quantize_and_export
    with safetensors.safe_open(f"{export_path}/rank0.safetensors",
FileNotFoundError: No such file or directory: "/workspace/checkpoint/mistralai_Mixtral-8x7B-Instruct-v0.1/tllm_checkpoint_fp8/rank0.safetensors"

It did generate two files though, but I can't use them an input for trtllm-build:
$ ls /workspace/checkpoint/mistralai_Mixtral-8x7B-Instruct-v0.1/tllm_checkpoint_fp8/
unknown:MixtralForCausalLM_tp1.json  unknown:MixtralForCausalLM_tp1_rank0.npz

The generated json file does not look like the ones usually generated with convert_checkpoint. Please advise ! :)

Hi!
I'm sorry, I couldn't understand it from release notes, but is it released in 0.9.0?
I can see that it now has OOTB support and is under the unified workflow, but does that mean that it works in fp8 now?

@hawkeoni @Marlinski Sorry guys, I think Mixtral-FP8 is delayed a little bit and will be released very soon.

@hawkeoni @Marlinski Sorry guys, I think Mixtral-FP8 is delayed a little bit and will be released very soon.

Hi @Tracin, I suggest to keep somewhere a compatibility table between models and quantization techniques, so that at a glance it's easy to understand what is supposed to work and what not.

@hawkeoni @Marlinski Sorry guys, I think Mixtral-FP8 is delayed a little bit and will be released very soon.

Hi @Tracin, I suggest to keep somewhere a compatibility table between models and quantization techniques, so that at a glance it's easy to understand what is supposed to work and what not.

Good idea! You can check it here https://github.com/NVIDIA/TensorRT-LLM/blob/main/docs/source/precision.md, we will keep this updated in time.

@hawkeoni @Marlinski Sorry guys, I think Mixtral-FP8 is delayed a little bit and will be released very soon.

Hi @Tracin, I suggest to keep somewhere a compatibility table between models and quantization techniques, so that at a glance it's easy to understand what is supposed to work and what not.

Good idea! You can check it here https://github.com/NVIDIA/TensorRT-LLM/blob/main/docs/source/precision.md, we will keep this updated in time.

Thanks for your answer. Is there any reason why Mixtral is not present in the table? Also, is the column for "Int4-FP8 AWQ (W4A8)" not present because it's intended to be supported when both "FP8" and "W4A16 AWQ" are supported?

@fedem96 I think the table is not up to date. I was able to export and run Mixtral in FP16 as well as in W4A16 but it does not appear in the matrice.

@fedem96 I think the table is not up to date. I was able to export and run Mixtral in FP16 as well as in W4A16 but it does not appear in the matrice.

Yes, I also think so, I'm able to run it in W4A16 by using the convert.py script, but I'm not able to quantize it with using the quantize.py script.

@Tracin I noticed that documentation for Mixtral FP8 has been added: https://github.com/NVIDIA/TensorRT-LLM/tree/main/examples/mixtral#fp8-post-training-quantization! Thank you very much for your hard work on this feature!
I am able to quantize, build, and run Mixtral 8x7B-Instruct v0.1 in FP8 on H100s following these instructions, however I'm running into two different errors with beam search:

When using beam_width > 1, an assertion error is thrown:

python {quantize_path} --model_dir {model_dir} \
                                --output_dir {checkpoint_dir} \
                                --dtype float16 \
                                --qformat fp8 \
                                --kv_cache_dtype fp8 \
                                --calib_size 512 \
                                --tp_size 2

trtllm-build --checkpoint_dir {checkpoint_dir} \
                 --output_dir {deploy_dir} \
                 --gemm_plugin float16 \
                 --workers 2 \
                 --strongly_typed

mpirun --allow-run-as-root -n 2 \
          python3 /triton-llm/trtllm-0.11.0.dev20240521/tensorrtllm_backend/tensorrt_llm/examples/run.py \
          --engine_dir {SCRATCH_MODEL_REPO}/{MODEL_NAME}/{MODEL_NAME}-tensorrt_llm/1 \
          --tokenizer_dir {SCRATCH_RAW_MODELS}/{HF_MODEL_NAME} \
          --max_output_len 1024 \
          --input_text "{prompt}" \
          --no_prompt_template \
          --temperature 1.0 \
          --length_penalty 1.0 \
          --repetition_penalty 1.0 \
          --num_beams 2 \
          --early_stopping 0

[TensorRT-LLM][WARNING] cache_indirection: expected dim[1] = 1, provided dim[1] = 2
[TensorRT-LLM][WARNING] cache_indirection: expected dim[1] = 1, provided dim[1] = 2
[TensorRT-LLM][ERROR] 3: [executionContext.cpp::setInputShape::2037] Error Code 3: API Usage Error (Parameter check failed at: runtime/api/executionContext.cpp::setInputShape::2037, condition: engineDims.d[i] == dims.d[i] Static dimension mismatch while setting input shape.)
[TensorRT-LLM][ERROR] 3: [executionContext.cpp::setInputShape::2037] Error Code 3: API Usage Error (Parameter check failed at: runtime/api/executionContext.cpp::setInputShape::2037, condition: engineDims.d[i] == dims.d[i] Static dimension mismatch while setting input shape.)
[TensorRT-LLM][ERROR] Encountered an error in forwardAsync function: [TensorRT-LLM][ERROR] Assertion failed: Tensor 'cache_indirection' has invalid shape (1, 2, 2048), expected (1, 1, -1) (/app/tensorrt_llm/cpp/tensorrt_llm/runtime/tllmRuntime.cpp:178)
1       0x7fbf5f47640a tensorrt_llm::common::throwRuntimeError(char const*, int, std::string const&) + 102
2       0x7fbf6121000e tensorrt_llm::runtime::TllmRuntime::setInputTensors(int, std::unordered_map<std::string, std::shared_ptr<tensorrt_llm::runtime::ITensor>, std::hash<std::string>, std::equal_to<std::string>, std::allocator<std::pair<std::string const, std::shared_ptr<tensorrt_llm::runtime::ITensor> > > > const&) + 2318
3       0x7fbf6141a817 tensorrt_llm::batch_manager::TrtGptModelInflightBatching::setupContext(std::vector<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest>, std::allocator<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest> > > const&, std::vector<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest>, std::allocator<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest> > > const&, int) + 359
4       0x7fbf6141aa5c tensorrt_llm::batch_manager::TrtGptModelInflightBatching::executeBatch(tensorrt_llm::batch_manager::ScheduledRequests const&) + 492
5       0x7fbf61427664 tensorrt_llm::batch_manager::TrtGptModelInflightBatching::forwardAsync(std::list<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest>, std::allocator<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest> > > const&) + 2164
6       0x7fbf6144b7e4 tensorrt_llm::executor::Executor::Impl::forwardAsync(std::list<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest>, std::allocator<std::shared_ptr<tensorrt_llm::batch_manager::LlmRequest> > >&) + 100
7       0x7fbf6144da6c tensorrt_llm::executor::Executor::Impl::executionLoop() + 380
8       0x7fc096ab0253 /usr/lib/x86_64-linux-gnu/libstdc++.so.6(+0xdc253) [0x7fc096ab0253]
9       0x7fc215956ac3 /usr/lib/x86_64-linux-gnu/libc.so.6(+0x94ac3) [0x7fc215956ac3]
10      0x7fc2159e7a04 clone + 68

Settings that worked for Mixtral in FP16 (v0.9.0) cause a runtime CUDA malloc error when I attempt to run Mixtral in FP8 (v0.11.0.dev2024052100). Note that for both FP16 and FP8, I'm using 2xH100:

# This works using v0.9.0
python {convert_checkpoint_path} --model_dir {model_dir} \
                                --output_dir {checkpoint_dir} \
                                --dtype float16 \
                                --tp_size 2

trtllm-build --checkpoint_dir {checkpoint_dir}
                                --output_dir {deploly_dir}
                                --gemm_plugin float16
                                --workers 2
                                --tp_size 2
                                --pp_size 1
                                --gpt_attention_plugin float16
                                 --context_fmha enable
                                --remove_input_padding enable
                                --use_custom_all_reduce disable
                                 --paged_kv_cache enable
                                --max_input_len 32768
                                --max_batch_size 60
                                --max_output_len 1024
                                --max_beam_width 5
                                --max_num_tokens 393456
                                --strongly_typed
                                --use_paged_context_fmha disable

mpirun --allow-run-as-root -n 2 \
          python3 /triton-llm/trtllm-0.11.0.dev20240521/tensorrtllm_backend/tensorrt_llm/examples/run.py \
          --engine_dir {SCRATCH_MODEL_REPO}/{MODEL_NAME}/{MODEL_NAME}-tensorrt_llm/1 \
          --tokenizer_dir {SCRATCH_RAW_MODELS}/{HF_MODEL_NAME} \
          --max_output_len 1024 \
          --input_text "{prompt}" \
          --no_prompt_template \
          --temperature 1.0 \
          --length_penalty 1.0 \
          --repetition_penalty 1.0 \
          --num_beams 5 \
          --early_stopping 0

# This does not work in v0.11.0.dev20240521
python {quantize_path} --model_dir {model_dir} \
                                --output_dir {checkpoint_dir} \
                                --dtype float16 \
                                --qformat fp8 \
                                --kv_cache_dtype fp8 \
                                --calib_size 512 \
                                --tp_size 2

trtllm-build --checkpoint_dir {checkpoint_dir}
                                --output_dir {deploly_dir}
                                --gemm_plugin float16
                                --workers 2
                                --tp_size 2
                                --pp_size 1
                                --gpt_attention_plugin float16
                                 --context_fmha enable
                                --remove_input_padding enable
                                --use_custom_all_reduce disable
                                 --paged_kv_cache enable
                                --max_input_len 32768
                                --max_batch_size 60
                                --max_output_len 1024
                                --max_beam_width 5
                                --max_num_tokens 393456
                                --strongly_typed
                                --use_paged_context_fmha disable

mpirun --allow-run-as-root -n 2 \
          python3 /triton-llm/trtllm-0.11.0.dev20240521/tensorrtllm_backend/tensorrt_llm/examples/run.py \
          --engine_dir {SCRATCH_MODEL_REPO}/{MODEL_NAME}/{MODEL_NAME}-tensorrt_llm/1 \
          --tokenizer_dir {SCRATCH_RAW_MODELS}/{HF_MODEL_NAME} \
          --max_output_len 1024 \
          --input_text "{prompt}" \
          --no_prompt_template \
          --temperature 1.0 \
          --length_penalty 1.0 \
          --repetition_penalty 1.0 \
          --num_beams 5 \
          --early_stopping 0

Traceback (most recent call last):
  File "/triton-llm/trtllm-0.11.0.dev20240521/tensorrtllm_backend/tensorrt_llm/examples/run.py", line 571, in <module>
    main(args)
  File "/triton-llm/trtllm-0.11.0.dev20240521/tensorrtllm_backend/tensorrt_llm/examples/run.py", line 420, in main
    runner = runner_cls.from_dir(**runner_kwargs)
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/runtime/model_runner_cpp.py", line 981, in from_dir
    return ModelRunnerCppExecutor.from_dir(engine_dir, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/runtime/model_runner_cpp.py", line 110, in from_dir
    executor = trtllm.Executor(
RuntimeError: [TensorRT-LLM][ERROR] CUDA runtime error in ::cudaMallocAsync(ptr, n, mCudaStream->get()): out of memory (/app/tensorrt_llm/cpp/tensorrt_llm/runtime/tllmBuffers.h:118)
1       0x7fe04ed78555 void tensorrt_llm::common::check<cudaError>(cudaError, char const*, char const*, int) + 149
2       0x7fe0509459e4 tensorrt_llm::runtime::BufferManager::gpu(unsigned long, nvinfer1::DataType) const + 324
3       0x7fe050a10814 tensorrt_llm::runtime::TllmRuntime::TllmRuntime(void const*, unsigned long, float, nvinfer1::ILogger&) + 644
4       0x7fe050c21f6d tensorrt_llm::batch_manager::TrtGptModelInflightBatching::TrtGptModelInflightBatching(int, std::shared_ptr<nvinfer1::ILogger>, tensorrt_llm::runtime::ModelConfig const&, tensorrt_llm::runtime::WorldConfig const&, std::vector<unsigned char, std::allocator<unsigned char> > const&, bool, tensorrt_llm::executor::SchedulerConfig const&, tensorrt_llm::batch_manager::TrtGptModelOptionalParams const&) + 1309
5       0x7fe050be27e0 tensorrt_llm::batch_manager::TrtGptModelFactory::create(std::vector<unsigned char, std::allocator<unsigned char> > const&, tensorrt_llm::runtime::GptJsonConfig const&, tensorrt_llm::runtime::WorldConfig const&, tensorrt_llm::batch_manager::TrtGptModelType, int, tensorrt_llm::executor::SchedulerConfig const&, tensorrt_llm::batch_manager::TrtGptModelOptionalParams const&) + 976
6       0x7fe050c49767 tensorrt_llm::executor::Executor::Impl::createModel(std::vector<unsigned char, std::allocator<unsigned char> > const&, tensorrt_llm::runtime::GptJsonConfig const&, tensorrt_llm::runtime::WorldConfig const&, tensorrt_llm::executor::ExecutorConfig const&) + 727
7       0x7fe050c4a381 tensorrt_llm::executor::Executor::Impl::Impl(std::filesystem::path const&, tensorrt_llm::executor::ModelType, tensorrt_llm::executor::ExecutorConfig const&) + 2097
8       0x7fe050c40622 tensorrt_llm::executor::Executor::Executor(std::filesystem::path const&, tensorrt_llm::executor::ModelType, tensorrt_llm::executor::ExecutorConfig const&) + 50
9       0x7fe0c6428a5d /usr/local/lib/python3.10/dist-packages/tensorrt_llm/bindings.cpython-310-x86_64-linux-gnu.so(+0xaaa5d) [0x7fe0c6428a5d]
10      0x7fe0c63cdd33 /usr/local/lib/python3.10/dist-packages/tensorrt_llm/bindings.cpython-310-x86_64-linux-gnu.so(+0x4fd33) [0x7fe0c63cdd33]
11      0x5648af1cb10e python3(+0x15a10e) [0x5648af1cb10e]
12      0x5648af1c1a7b _PyObject_MakeTpCall + 603

Separately, I'm wondering if it should be possible to run a TensorRT-LLM engine for Mixtral 8x7B in FP8 on a single H100. I think that should be possible (the compiled engine is ~44 GB), but I'm running into an error message similar to (2) above.
Thank you again for the work you've put into this feature!

Hi @Pernekhan  do u still have further issue or question now? If not, we'll close it soon.

No questions. Thank you.


The video swap is not consistent

Thanks for your excellent work. I ran face_swap for each frame in the video but found the results were not smooth.
But in your video demo, the results are very nice, could you please tell me how to fix it?
Thanks for your help!

Are the correct swaps working? ignoring the smoothness. Whichever frame is swapping, are they good?


[PROPOSAL] add Dependabot to the project for dependency monitoring

Purpose:
The LFAI requires that we employ some mode of dependency monitoring. GitHub offers a built-in solution called Dependabot. It automatically opens PRs to update dependencies when there is a known security issue and/or when a newer version is available. We are already using it in Marquez for security updates. It is very easy to deploy but would require some configuration to be effective for OpenLineage.
Proposed implementation:
All configuration is done in a single dependabot.yml file in the .github directory. There, one can specify particular package ecosystems to be monitored (e.g., gradle), the location of their manifests (e.g., client/java), the desired frequency of monitoring (e.g., weekly), and dependencies to be allowed or ignored.
If we decide to implement this solution, we would need to determine the package managers that need monitoring and the desired frequency, the dependencies to include or exclude, etc. I propose that these determinations could easily be made in a PR.
Resources:

The Dependabot docs include a list of package managers currently supported by the tool.
An approach to configuring the tool for monitoring a monorepo is covered by this blog post.

Looks good!


Database

We do not have permission to modify the database

Now it may work, have a try?

got it


(feat): add modalType option to address validation event

The type of this PR is: feat

The goal of this PR is to add an option and subject property to the ValidationAddressViewed tracking event so that analytics can differentiate which modal (address suggestions or general verification error) is viewed by user. We will better be able to analyze triggers here, per @starsirius's notes on this PR: https://github.com/artsy/cohesion/pull/443
@daytavares please let me know if I have things in the right place or if changes are requested. My goal was to make some quick edits to the schema so that we can keep progressing on analytics tracking.

PR Checklist (tick all before merging)

[ ] If I've added a new file to the tree I've exported it from the common index.ts
[ ] I've added comments with examples for any new interfaces and ensured that they're in the docs
[ ] No platform-specific terminology has been used outside of click and tap (platform is inferred by the DB storing events)

It looks good! Thanks!

:rocket: PR was released in v4.135.0 :rocket:


The feedback/error callbacks are not working

It seems like the feedback/error callbacks are not working properly.
When I first connect I get this info:
[debug] [APNS] connected to gateway.sandbox.push.apple.com:2195
[debug] [APNS] connected to feedback.sandbox.push.apple.com:2196
[debug] [APNS] Feedback socket was closed. Reconnect in 10000s.

If I try to send something to a wrong id, Apple just closes the gateway.sandbox.push.apple.com connection and there's no error. Also next time callback connects - there's no feedback either.
What is the expected behavior?

That is weird.
The expected (and actual for me) behaviour is this:

Can u provide any more relevant information?

I just did
{:ok, conn}=APNS.start :dev
APNS.push conn, "0000000000000000000000000000000000000000000000", "test"

and received :ok, no other messages except
[debug] [APNS] Apple socket was closed
[debug] [APNS] connected to gateway.sandbox.push.apple.com:2195

Running on MacOS
elixir --version
Elixir 1.1.1

erl --version
Erlang/OTP 18 [erts-7.1] [source] [64-bit] [smp:8:8] [async-threads:10] [hipe] [kernel-poll:false] [dtrace]

I think I know why. The size of the key matters! ;)
If I do APNS.push pid, "0000000000000000000000000000000000000000000000000000000000000000", "Hello world!" I get an error back. But if I do APNS.push pid, "00", "Hello world!" I get nothing.

Yeah this makes sense. APNS receives incorrect packet in that case and closes the connection.

I think it would be better to call error callback in this case as well. As a developer working with a library I want to have one callback to handle a case where the token was wrong. Token is the only thing that I don't control since it comes to me from a device. What do you think?

ok, I think I'll do this

Thanks!

Done. Also I've added error callback trigger on payload size error. Changelog

Thanks, will test today.

Tested 0.0.6 - :+1:

I still can't get the feedback callbacks to work at all. Below is my configuration:
config :apns,
  certfile:         "MyCertAndKey.pem",
  callback_module:  APNS.Callback,
  timeout:          30000,
  feedback_timeout: 1200,
  reconnect_after:  1000

@kroucis hi, can u share logs? Maybe even complete code? As of now circumstances when it fails for you are not obvious.

Here is the call to APNS
{:ok, pid} = APNS.start :dev
allTokens = Repo.all from pt in PushToken, select: pt.token
for token <- allTokens do
  APNS.push pid, token, "Hello!"
end
# I would call APNS.stop(id) here, but that kills the process before it can actually send anything to Apple.

Is it possible that calling APNS.start/1 and then sending notifications is somehow not giving the service enough time to start up the feedback connection?
Here are the logs for that code:
[debug] [APNS] connected to gateway.sandbox.push.apple.com:2195
[debug] [APNS] connected to feedback.sandbox.push.apple.com:2196
[debug] [APNS] Feedback socket was closed. Reconnect in 1200000s.

As a note: notifications appear to be working correctly.

Is it possible that calling APNS.start/1 and then sending notifications is somehow not giving the service enough time to start up the feedback connection? no it shouldn't be the case
But I want to ensure you are clear on feedback purpose:
The Apple Push Notification service includes a feedback service to give you information about failed remote notifications. When a remote notification cannot be delivered because the intended app does not exist on the device, the feedback service adds that device’s token to its list. Remote notifications that expire before being delivered are not considered a failed delivery and don’t impact the feedback service. By using this information to stop sending remote notifications that will fail to be delivered, you reduce unnecessary message overhead and improve overall system performance.
Are you sure feedback should be received? Can you also test with any wrong token and confirm error callback is properly triggered?

Ah! Fair enough. I was mistaken on the intent of the feedback system. When attempting with an invalid token, I get the following in red:
[error] [APNS] Error "Invalid token size" for message <<142, 231, 22, 86>>

This seems to be in line with what you are saying. Sorry for the false alarm.


When setting the mouse on somewhere within the string, it goes immediately goes to the end of the string

Expected: the cursor will remain at the same place and you'll be able to type.

can you provide a plunkr of this?

This is intended behaviour to sepearte user input field from autocomplete input field. I am not sure this is worth to fix unless there are many upvotes.


pr

 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.:x: miguelillo sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Avoid multiple web services calls

We could unify the calls to /synset, /get-comments, and /get-suggestions into one single call to avoid all the round trips between the Lisp code and Bluemix, therefore reducing the response time of the application even further.

or, alternatively, make all the different web service calls asynchronously.

No longer critical.


Allow raw JSON to be passed into GOOGLE_APPLICATION_CREDENTIALS env variable

... or into new env variable.
Usecase is using service in bitbucket pipelines:

While this isn't the exact useless you described, you can use the -token flag to pass in credentials to the proxy when it starts (which could rely on a env var).

Is there a way to store a file to disk using the bitbucket configuration?
That's seems like a pretty basic feature it should support (never used it,
haven't checked, just a guess). If you can manage that, then the Proxy
doesn't need anything extra from what already exists: store that file to
disk somewhere, and then point to it using the existing flags/env variables
for authentication.

I don't think -token works for this: the json credentials include a private
key for obtaining oauth tokens (via Google auth APIs), but -token should
take the oauth token itself (implying something else has to call the Google
APIs).
On Mon, Oct 28, 2019, 10:36 AM Kurtis Van Gent<EMAIL_ADDRESS>wrote:

While this isn't the exact useless you described, you can use the -token
flag to pass in credentials to the proxy when it starts (which could rely
on a env var).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/cloudsql-proxy/issues/323?email_source=notifications&email_token=AAELF362XOYSZABY6PS6JIDQQ4IKPA5CNFSM4JF4WJX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECNRGPQ#issuecomment-547033918,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAELF34JR4Q4MYXB4WDJSITQQ4IKPANCNFSM4JF4WJXQ
.

Is there a way to store a file to disk using the bitbucket configuration?

Sure, and I'm using it now to run cloud-proxy in background

But, if I want to have it as stand-alone service (similar to docker-compose items) I need to define it in special separate section and only env variables are available there...

+1 for this, seems like a great idea to be able to pass the json content. Since many faas/paas seem to support using secrets in key => value form. So then the content of the json could be stored once, as a secret, and then that file then never needs to be kept in a file system.

+1 for this. Requiring that the file be present on the filesystem makes cloud_sql_proxy difficult to use with terraform. This code search of GitHub shows that the only way anybody has figured out how to deploy cloud_sql_proxy with terraform is to use k8s volume mounts.
I think the scarcity of terraform+cloud_sql_proxy usage on GitHub points to a serious usability problem. It is, of course, up to you whether you want to support terraform. If you do, allowing us to set the entire service account key as an env var would make cloud_sql_proxy much easier to use. Thanks for keeping this issue open and considering this feature.

We don’t control GOOGLE_APPLICATION_CREDENTIALS and so can’t change how it works. However, there is a CredentialsFromJSON method that we could in theory expose as a CLI flag. That said I don’t understand the use case here and why using a credential file isn’t a reasonable option.

The use case is: it's easier to pass the credentials into a container as an env var instead of mounting them as a volume. Look at all the templating hoops I had to jump through to deploy Cloud SQL Proxy with Terraform. Also, lots of container environments are restricted (e.g. CI/CD) and don't let you mount volumes. But just about every container environment lets you set env vars.

This is on main now and will be available in the next preview release of v2 (preview 2).


TileGrid origin causes missing and partial tiles

Describe the bug
When an oddball origin is set on a TileGrid, not all of the tiles will be drawn. At some zoom levels, partial tiles will be drawn.
To Reproduce
Steps to reproduce the behavior:

(Go to https://jsfiddle.net/kay99yor/1/ to see what it should presumably look like)
Go to https://jsfiddle.net/6hfs9tbx/6/
Zoom out
A portion of the south-west corner of the map is missing
A portion of the north-east corner is black

Expected behavior
Regardless of origin,

I would expect to see all tiles which are within the view, regardless of zoom level.
I would expect areas for which there is no data to be transparent - not black.

A little background
I modified the above fiddle from one I found at https://stackoverflow.com/questions/47612523/how-to-configure-openlayers-for-a-flat-11-pixel-coordinate-system?rq=1 in order to demonstrate what is happening in my somewhat more complex application.
In my real application, I have a Vector layer (with a pixel based projection) and an MVT layer that may be slightly misaligned from that Vector layer. I am using the TileGrid's origin to bring the MVT layer into alignment with the Vector layer. The problem is that the MVT layer has partial and disappearing tiles. I suspect (?) that the same problem affects this fiddle as affects my MVT layer.

The origin needs to match a tile grid corner. Did you verify that this is the case?

I am a little unclear as to what you are asking.
The documentation says:
origin The tile grid origin, i.e. where the x and y axes meet ([z, 0, 0]).
Doesn’t this mean that the origin is the coordinate at which the map’s x and y axis meet the tile’s [z, 0, 0]?
If so, whatever I set the origin to, defines where the top left grid corner will appear. In other words, it matches by definition. No?

Doesn’t this mean that the origin is the coordinate at which the map’s x and y axis meet the tile’s [z, 0, 0]?

No. It is the projected coordinate at the top left corner of the tile with x=0 and y=0. The x and y axes in the documentation refer to the tile grid, not the map.
You cannot set arbitrary values here. Those need to match the tile grid layout that the server provides.

So, this is what you are saying?
origin The tile grid origin, i.e. where the x and y axes of the tile grid meet the tile addressed by ([z, 0, 0]).


inference on automotive dataset

@donydchen @QianyiWu  thansk for sharing the wonderful work , just wanted to knw if we can test it for images like automotive like cityscapes , bdd100k dataset or have u tested it in your work ?
If not should we re-train the model on the new dataset ?

Thanks in advance

Hi @abhigoku10 , we appreciate your interest in our work.
We have never attempted similar datasets before. As stated in our paper, Sem2NeRF mainly intends to consider "taking as input only one single-view semantic mask of a specific category". The main reason is that Sem2NeRF relies on a pre-trained NeRF-based generative model to decode the latent code, hence the current version is limited to data that can be modelled by the corresponding generative model, i.e., pi-GAN.
Our suggestion is that you might consider finding a more powerful decoder that generates 3D-aware contents similar to cityscapes, and then re-training our encoder to map the semantic mask to the generated 3D contents. Hope this helps.


Fixed a bug in DeviceAllocator

Fixed a bug in DeviceAllocator

This pull request does a little bit more than solve that one bug

Device allocator fixes is now a separate pull request. I'm closing this.


Fix refcount handling for None object

Py_None is a singleton, but its refcount has to be managed like a
regular object, that, it needs to be Py_INCREF'd before returning it or
assigning it to a structure and it needs to be Py_DECREF'd when the
containing object goes away.

Again, many thanks!  Wasn't aware of refcount handling wrt Py_None.

Glad I could help!


Plot error bars in sample plot

From @mortonjt's comment in #218—

Would this add in the possibility of pseudocounts?  There are some scenarios where this can be very helpful. (#208)
An even better option would be to add in uncertainty via Dirichlet bootstrapping.  One possibility is you perform bootstrapping (with like 100 replicates), and then pass in the means / stds.  That way you can also plot error vars in the balance scatter plots to help measure the uncertainty.
Bootstrapping will look something like

import numpy as np
boots = []
table = np.array( ... ) // your biom table
pseudo = 1
for _ in range(100):
    boots.append(np.random.dirichilet(table))

boots = np.dstack(boots)
m, s = boots.mean(axis=2), boots.std(axis=2, ddof=1)

Was thinking about this today, but Vega-Lite supports error bars -- see https://vega.github.io/vega-lite/docs/errorbar.html. I don't think it'd be too much work (...in the future) to change the rank or sample plot to show confidence intervals / error bars / etc., provided this data exists. The main challenge will just be testing stuff, I think.
If we have a measure of uncertainty in the differentials (as in the figure here), adjusting the rank plot to show error bars might actually be easier since we don't have to worry about formatting as much as in the sample plot.
just a thought :)

Relevant to BIRDMAn; looping in @gibsramen.


Build tag format changed between Go 1.16 and 1.17

Go 1.16 is entirely happy with the build tags here:
https://github.com/atc0005/elbow/blob/ffb206e99c2f0c2df1f82b0b64c22ad55120c4d1/logging/logging_unix.go#L1
but the gofmt tool from Go 1.17 flags the file as not being properly Go formatted.
Here is a diff after running gofmt -s -w logging/logging_unix.go:
diff --git a/logging/logging_unix.go b/logging/logging_unix.go
index 6be6ce7..00a3854 100644
--- a/logging/logging_unix.go
+++ b/logging/logging_unix.go
@@ -1,3 +1,4 @@
+//go:build !windows
 // +build !windows

 // Copyright 2020 Adam Chalkley

Notice the additional build tag entry. Presumably this makes the builds compatible with both Go 1.16 (and earlier versions?) and Go 1.17 and later.

Also, I suspect that this line will need to be updated as well:
https://github.com/atc0005/elbow/blob/ffb206e99c2f0c2df1f82b0b64c22ad55120c4d1/logging/logging_windows.go#L1

Also, I suspect that this line will need to be updated as well:

Confirmed. After running gofmt -s -w against the logging/logging_windows.go file it showed the same change applied:
diff --git a/logging/logging_windows.go b/logging/logging_windows.go
index 3cfab54..81b76cd 100644
--- a/logging/logging_windows.go
+++ b/logging/logging_windows.go
@@ -1,3 +1,4 @@
+//go:build windows
 // +build windows

 // Copyright 2020 Adam Chalkley


Proposal to split up into a monorepo

I propose we split this up into a monorepo solution, which will publish one package per adapter type, plus shared packages.
As an addition, I propose to use projen for managing the repo, and switch to using pnpm as package manager (better DX for monorepos).
I can volunteer to implement this.

appreciate the enthusiasm! 😄
currently it's not that big of a repo and devs can easily switch from one adapter to another, for example upgrade from apigwv1 to apigwv2 without installing new packages. so I'm not sure there is a need to split into multiple npm packages 😅
if we did, we would have a bunch of packages with 1 relatively small file, might be overkill. we're providing mappers from 1 model to another, preferable it doesn't get more complicated then that 😄
pnpm would be a nice addition, npm is a lightweight tool and standard in every node.js evironment, I hope in the future npm would take over some features from pnpm but that doesn't always happen so bringing pnpm to remix-aws would be nice 👍


Incorrect/ incomplete Permissions listed for enabling Defender Plans

While a Contributor role for a subscription can enable the plan, changing or enabling additional components such as the Azure Monitor Agent seems to require Owner rights on the subscription. This is not well reflected in the documentation.
Thanks

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

ID: df8d64db-117b-a6ae-61d6-809906c9ebb9
Version Independent ID: 92edcbb4-a2bd-f40a-14fd-e495d88180ea
Content: Permissions in Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/permissions.md
Service: defender-for-cloud
GitHub Login: @ElazarK
Microsoft Alias: elkrieger

@jrohrich
Thanks for your feedback! We will investigate and update as appropriate.

@jrohrich Thank you for bringing this to my attention. I am checking with the appropriate people and will have an answer in the next few days

@jrohrich you are indeed correct and the documentation is being updated and will be reflected in the next few days.

#please-close


extension points for plugin

Do you have any plans to allow extending the existing plugin? Or maybe it is possible already, and I just couldn't find it...
For example we'd like for each and every proto message to generate builders, test builders (with test values) and probably jest matchers.
Of course we can take the plugin framework and write our own but that means duplicate codegen execution and plenty of code copy & paste from the plugin.
Thanks!

I have never used extension points, but I've heard they might be unpleasant to use. Is it an option to do what you need in runtime?
For example, the existing reflectionCreate() takes a IMessageType<T> and returns an instance of message T, all via reflection ops:
https://github.com/timostamm/protobuf-ts/blob/fbce12b7259cacea5d45ebf3503c8ac8a0200c73/packages/runtime/src/reflection-create.ts#L10
I imagine it should be possible to create a jest matcher like isMessageOfType(received, type: IMessageType), for example.
Could you elaborate a bit what you want to do?

@timostamm thanks for your reply.
The runtime magic is an option indeed, however the devex isn't great (IDE support and compilation errors are cryptic). Thus I prefer to do a codegen. Currently having some post-codegen with ts-morph.
Another use case I have is message type refinement (due to custom options) or method signatures (think something like CallContext argument).
I was thinking about some decorators support for all those tasty generators you have there.
Since I couldn't find any programmatic API for the plugin, possible API could be another plugin parameter with absolute path to .js file with decorators factory:
exports = {
  enumGeneratorDecorator: {...},
  methodInfoGeneratorDecorator: {...},
  // etc
}

The plan is to switch to the base types provided by protobuf-es. protobuf-es includes a much improved plugin framework with @bufbuild/protoplugin, which makes it a whole lot easier to write your own plugins.
I think this should be the best way forward for your use case as well (generating builder, test builders, jest matchers). It might not give you all the capability extension points give you, but they are a double edged sword for sure. You can see the code for a simple plugin here. I hope we can simplify it even more (not needing to type out .js, .ts and .d.ts code for example).


Keypoints Dataset - flip_keypoint,missing size issue

Hi,
During training phase for keypoints just after reading them from .xml files there is a method invoked on every loaded keypoint to flip it (utils.masks.py):
def _flip_keypoint(id_kp, x_kp, y_kp, input_shape=(320, 320, 3))

In general it takes 4 arguments: keypoints id, x, y and.. size of an image. But keypoints_dataset class call to this method is missing this last one, that is why every single time images used to train keypoints model need to be in 320x320x3 format:
for id_kp, v in six.iteritems(keypoints):
    new_id_kp, x_kp, y_kp = _flip_keypoint(id_kp, min(v[0],image.shape[0]-1), min(v[1],image.shape[1]-1))
    new_keypoints[new_id_kp] = (x_kp, y_kp)

Think this call should be made like that or there will be no possibility to train model on differently sized pictures:
for id_kp, v in six.iteritems(keypoints):
    new_id_kp, x_kp, y_kp = _flip_keypoint(id_kp, min(v[0],image.shape[0]-1), min(v[1],image.shape[1]-1), image.shape)
    new_keypoints[new_id_kp] = (x_kp, y_kp)

Hey,
This is resolved in the following commit.
I also changed two functions where the same problem might happen in the future.
Some shapes are still hardcoded in the PlayerBallTracker class but that should not cause any issues.
Thanks


chore: Use multi-arch oc client for RHACM install

Signed-off-by: Denilson Nastacio<EMAIL_ADDRESS>Contributes to: #315
Description of changes:

Use multi-arch images for sync hooks

Output of argocd app list command or screenshot of the Argo CD Application synchronization window showing successful application of changes in this branch.
 argocd app list
NAME                                  CLUSTER                         NAMESPACE                PROJECT               STATUS  HEALTH   SYNCPOLICY  CONDITIONS  REPO                                    PATH                                  TARGET
openshift-gitops/argo-app             https://kubernetes.default.svc  openshift-gitops         argocd-control-plane  Synced  Healthy  Auto-Prune  <none>      https://github.com/IBM/cloudpak-gitops  config/argocd                         315-power-rhacm
openshift-gitops/cp-shared-app        https://kubernetes.default.svc  ibm-cloudpaks            default               Synced  Healthy  Auto-Prune  <none>      https://github.com/IBM/cloudpak-gitops  config/argocd-cloudpaks/cp-shared     315-power-rhacm
openshift-gitops/cp-shared-operators  https://kubernetes.default.svc  ibm-cloudpaks            default               Synced  Healthy  Auto-Prune  <none>      https://github.com/IBM/cloudpak-gitops  config/cloudpaks/cp-shared/operators  315-power-rhacm
openshift-gitops/rhacm-app            https://kubernetes.default.svc  open-cluster-management  rhacm-control-plane   Synced  Healthy  Auto-Prune  <none>      https://github.com/IBM/cloudpak-gitops  config/argocd-rhacm                   315-power-rhacm
openshift-gitops/rhacm-cloudpaks      https://kubernetes.default.svc  openshift-gitops         rhacm-control-plane   Synced  Healthy  Auto        <none>      https://github.com/IBM/cloudpak-gitops  config/rhacm/cloudpaks                315-power-rhacm
openshift-gitops/rhacm-seeds          https://kubernetes.default.svc  open-cluster-management  rhacm-control-plane   Synced  Healthy  Auto        <none>      https://github.com/IBM/cloudpak-gitops  config/rhacm/seeds                    315-power-rhacm

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


Background uploader

Provide a background upload mechanism that

doesn't rely on lots of http requests for
uploading large numbers of (potentially large) files in parallel
from diverse storage systems (s3 and other cloud storage, sftp, nfs, ...)

There's something usable from HZDR already:
https://www.hzdr.de/db/!Publications?pNid=head&pSelMenu=0&pSelTitle=27057
And we have a migration task developed by @egabancho working on our zenodo-clone that does something like that in a well defined scenario.

Tracked on the product board in https://github.com/inveniosoftware/product-rdm/issues/67 https://github.com/inveniosoftware/product-rdm/issues/60 https://github.com/inveniosoftware/product-rdm/issues/59


spec seeding

Change the resulting pdf name to riscv-brs-spec.pdf
Use brs.bib as the bibtex file instead of example.bib
Ignore vim swap files.
Change bibtex format to IEEE to make citings smaller.
Seed Glossary and Bibliography with some expected terms.
Add BRS-I and BRS-B names
Add RISC-V Profile to bibliography & glossary
Add dependencies in table form to BRS-I and BRS-B recipes
Add initial introduction stab.

looks good to me


'ArduinoFFT' was not declared in this scope

Hi, when I try to compile the sketch, this is the error I'm getting. Do you know what could be happening? All libraries needed have been installed including the arduinoFFT library.
line 122   auto fft = ArduinoFFT(real, imag, SAMPLE_COUNT, SAMPLE_RATE_HZ, weighingFactors);
Thanks for your help!

Hi wilson3682. You need to use the development version of the ArduinoFFT library.

There's also some files still missing due to problems with the submodules. I will update the code in the next few days when I find time and respond here.

Thanks HorsBaerbel for your help! I deleted the arduinoFFT library installed through arduino and placed the dev library manually and now it compiles. Thanks again!

I'd be surprised if it worked because you're probably missing some submodule code. I've updated the repos with the submodules and my current version of the code, but this is WIP. Will update the code in the next weeks. I've killed my ESP32 and am currently waiting for an replacement...


SWP-1932] Hotfix Changes. Introduce findReviews().

• Add a new function that allows to search reviews with any options.
• This allows us to find reviews with different options that behave differently between Trustpilot and ReviewsIO api.
• Needed because OT needs to search reviews via $installation->uuid, not its reviewId.

Codecov Report

Merging #20 (fca5d34) into master (da257ac) will decrease coverage by 1.95%.
The diff coverage is 61.11%.

@@             Coverage Diff              @@
##             master      #20      +/-   ##
============================================
- Coverage     93.26%   91.30%   -1.96%
- Complexity       92       99       +7
============================================
  Files            25       26       +1
  Lines           282      299      +17
============================================
+ Hits            263      273      +10
- Misses           19       26       +7

Impacted Files
Coverage Δ
Complexity Δ

src/Providers/ReviewsIo/MerchantActions.php
80.00% <0.00%> (-20.00%)
5.00 <1.00> (+1.00)
:arrow_down:

src/Providers/ReviewsIo/ProductActions.php
42.85% <0.00%> (-7.15%)
5.00 <1.00> (+1.00)
:arrow_down:

src/Providers/Trustpilot/MerchantActions.php
94.73% <0.00%> (-5.27%)
5.00 <1.00> (+1.00)
:arrow_down:

src/Providers/Trustpilot/ProductActions.php
42.85% <0.00%> (-7.15%)
5.00 <1.00> (+1.00)
:arrow_down:

...s/ReviewsIo/Request/Merchant/FindReviewRequest.php
100.00% <100.00%> (ø)
3.00 <0.00> (ø)

.../ReviewsIo/Request/Merchant/FindReviewsRequest.php
100.00% <100.00%> (ø)
3.00 <3.00> (?)

Continue to review full report at Codecov.

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


Fix navigation breadcrumbs

Fixed issue #55

Codecov Report

Merging #57 (3498916) into master (88d37ab) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##            master       #57   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          285       285
  Branches        58        31   -27
=========================================
  Hits           285       285

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Exporting RadixUI Portal to Avoid Hidden Hover Content

Related: #1940
In current Tooltip, HoverCard components, the Portal component from radix UI isn't exported:
https://github.com/shadcn-ui/ui/blob/6f3050248c80480188176d505d873dafadc4d160/apps/www/registry/default/ui/tooltip.tsx#L30
https://github.com/shadcn-ui/ui/blob/6f3050248c80480188176d505d873dafadc4d160/apps/www/registry/default/ui/hover-card.tsx#L29
And it's also not used in the official example.
https://github.com/shadcn-ui/ui/blob/6f3050248c80480188176d505d873dafadc4d160/apps/www/content/docs/components/hover-card.mdx#L61-L66
However, the portal is of crucial importance to the hover related components, without which the hover content will be constrained within the wrapping component if the wrapping component itself has limited size, with everything exceeding the wrapping component boundary hidden.
People new to the library (and Radix UI) will likely be stepping into this issue. It might be good to have the export and official example corrected.
Please let me know if a PR is preferred. I can help create one.

Agreed. We should wrap the <TooltipContent /> in a portal by default. https://github.com/shadcn-ui/ui/blob/6f3050248c80480188176d505d873dafadc4d160/apps/www/registry/new-york/ui/tooltip.tsx#L18 like we did for <AlertDialog />: https://github.com/shadcn-ui/ui/blob/6f3050248c80480188176d505d873dafadc4d160/apps/www/registry/new-york/ui/alert-dialog.tsx#L34
Happy to review and merge a PR. Thank you.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.

Hi @shadcn, a new PR (#5409) has been made for this issue. Please review, thanks!


Fetch IMDb ratings, genres using Web Scraping for movies present in a directory (path given)

Description
A short summary of what is included in your Pull Request.
Fixes #245
Type of change
Choosing one or more options from the following as per the nature of your Pull request.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Documentation Update

Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[x] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[x] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[x] I have included a requirements.txt file (if external libraries are required.)
[x] My changes do not produce any warnings.
[x] I have added a working sample/screenshot of the script.

any use of this file .DS_Store ?

No, my bad, I forgot to delete

any use of this file .DS_Store ?

No, my bad, I forgot to delete

Remove >DS_Store file so we can merge it


Add helpers in Azure.Core to map between ServiceVersion enums and REST API versions

Investigate if there's anything we can do to mitigate pain of keeping version numbers in sync with our ServiceVersion enums.

I think this was a concern raised by Keith...
@seanmcc-msft does something need to happen here?  If so what would you like to see?  Otherwise, will you please close this.


Create groups in canvas

Can you create all of the project groups in canvas and also create two master groups for the two sections which will be different than their registered section. See the spreadsheet in the "projects" folder. We need this to make all future assignments.
Use the Project ID as the group name.
Also, please add to the instructor guide if not there.

This is in the guide.


docs: depricated Marking on outdated docs - Local dev setup superset

SUMMARY
I have spent multiple hours in figuring out setting up my local superset (development) setup.
I have followed both docs https://github.com/apache/superset/blob/master/CONTRIBUTING.md
and https://github.com/apache/superset/blob/master/docs/docs/contributing/local-backend.mdx
Ended up with a bunch of errors. (tried fixing, but chain it continuing) lot of breaking stuff.
I have even tried PyPI installation methods (even that method breaks) while delivering frontend content. Marked deprecated
Current way is https://superset.apache.org/docs/installation/docker-compose#option-1---for-an-interactive-development-environment
Updated same in other places.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

While docker-compose is probably easiest (and smoother/faster than ever), the other methods of installation are not deprecated, and people use them all the time. Maybe you can file issues about the problems you ran into, or connect with some folks on slack to troubleshoot?

If we can help whatever situation is happening on your machine, maybe we can add some helpful documentation steps rather than kill off these installation methods.

When I left installation steps, these were issues

From local setup docs - 1 errors was superset_config.py should be set in evn variable - that instruction was mission

It would install apache-superset as pip dependency.
1 was specific to brew - where is expected mysqClient should be brew. (it was complaining since I have installed MySQL via official installer)

when I did https://superset.apache.org/docs/installation/pypi, when did
superset run -p 8088 --with-threads --reload --debugger

Server starts - (UI is not identical). After you give creds which you created while init, you see blank screen only two boxes are show.

@surapuramakhil I made some updates to the PyPI instructions, do you have feedback on this PR: https://github.com/apache/superset/blob/58c25238fee96c89203184dd2badda7a4f06aa35/docs/docs/installation/pypi.mdx

Last week I tried to install from PyPI following Superset's docs and it did not work.  That is a bad state of affairs, I agree.  And this PR helped bring my attention to that.  But I think the appropriate fix is to improve those instructions, not to mark those installation methods as deprecated.  I'm going to close this PR in light of that.  @surapuramakhil your feedback on my linked PR was already helpful, thank you!  And I welcome further suggestions from you or anyone else as to how to improve those installation instructions.


Explore creating unique directories for sorted WAL recovery

When a tablet server dies, its write ahead logs need to be sorted.  A random tablet server is chosen to sort the write ahead logs.  If the random tablet server doing the sort dies or times out, the sort may be started on another write ahead log.  This may lead to multiple tablet servers sorting the write ahead logs, possibly to the same directory.  Would probably be better if each tserver doing a sort wrote to a unique dir.

@ctubbsii I would like to take a look at this.

I think the unique directory would have to be added to the destination path here:
https://github.com/apache/accumulo/blob/3fd5cad92f9b63ac19e4466f3f2d5237b905262c/server/base/src/main/java/org/apache/accumulo/server/master/recovery/RecoveryPath.java#L45
It looks like that method getRecoveryPath(Path walPath) takes the WAL path and returns the recovery path for sorting.  So given something like:
source = hdfs:/accumulo/wal/ip-10-113-12-25+9997/bd9a7597-b51f-415a-9648-0363913749d0
it will return dest = hdfs:/accumulo/recovery/bd9a7597-b51f-415a-9648-0363913749d0

Closing based on conclusion in https://github.com/apache/accumulo/pull/1803#issuecomment-771883813


feat: Run gov-indexer on cloud

Feature Request
Describe the Feature Request

Describe Preferred Solution

Describe Alternatives

Related Code

Additional Context

Estimate points suggestion: 8


Template format error: The Value field of every Outputs member must evaluate to a String and not a Map.

When referencing a list instead of a value in a stack that spans multiple accounts the following error is produced:
ERROR: Template format error: The Value field of every Outputs member must evaluate to a String and not a Map.
Steps to reproduce:

Create a hosted zone for "somedomain.com"  in account A.
Create a single template that defines the following resources:

A hosted zone for "sub.somedomain.com" in account B
An NS resource record for domain B in the "somedomain.com" zone in account A using the "!GetAtt SubHostedZone.NameServers" value.

Example of template for step 2:
  # hosted zone in account B
  HostedZone:
    Type: AWS::Route53::HostedZone
    OrganizationBindings:
      Regions: eu-west-1
      Accounts: !Ref AccountB
    Properties:
      HostedZoneConfig:
        Comment: Domain Hosted Zone
      Name: !Sub '${AWSAccount.Tags.subdomain}.${rootHostedZoneName}.'
      HostedZoneTags:
        - Key: Application
          Value: cso
        - Key: Component
          Value: domains
        - Key: Aspect
          Value: addressing

  # Parent NS record in root zone in account A
  ParentNsRecord:
    Type: AWS::Route53::RecordSet
    OrganizationBindings:
      Regions: eu-west-1
      Accounts: !Ref AccountA
    Properties:
      Type: NS
      HostedZoneName: !Ref rootHostedZoneName
      Name: !Sub '${AWSAccount.Tags.subdomain}.${rootHostedZoneName}.'
      TTL: 86400
      ResourceRecords: !GetAtt HostedZone.NameServers

The "HostedZone.NameServers" returns a List. As CloudFormation outputs are used to handle cross-account references the error most likely is caused by that.
Note that there's probably an easier way to reproduce this but this is what I've got...

fixed specifically for NameServers.


winform Vlc crash witchout any hint while  Fast switching the url

I have an issue / a question (pick one) about Vlc.DotNet.
Generic information

Vlc.DotNet version : 3.0.0
Vlc.DotNet project used : WinForms
libvlc version : (Auto Select the Program version) (x86/x64) + Nuget
.net version : (Net472)
Project language : (C#)
Project build architecture : (AnyCPU (prefer 32bits))
Operating system : (Windows 10 Home) (x64)

Summary
(What happens, when, how? Steps to reproduce (with sample codes))
winform Vlc crash witchout any hint while  Fast switching the url
I started a thread, switching a set of links every 10 seconds, and switching a url every 10 seconds inside the custom vlc control. However, they always crash for no reason. and the Windows Event Viewer also cannot find the reason
there are Demo: https://github.com/wutangyuan/VlcWinformCrashDemo

[ ] I confirm that my issue doesn't happen in VLC itself.

Can you narrow down your repo to the minimum? I see different things in your UI like DataTemplates and that makes things harder for me to analyze.

Can you narrow down your repo to the minimum? I see different things in your UI like DataTemplates and that makes things harder for me to analyze.

thank you for yor reply!  I had Push the minimum Code to the repo .

hello,i have found that When I call  “stop()” and “play()” frequently, it will report the exception recycled by GC. But I didn't call GC .Finally, the program crashed.

you are checking VlcProvider and its media player outside of your Task.Factory.StartNew(() => ), which can cause race conditions, because you don't know how long does it take for a thread to be available.
As a general advice, I would use Task.Run() instead of Task.Factory.StartNew
Do you really need it ? you don't seem to be inside a libvlc callback.
Also, your try/catch will never catch anything except task scheduling errors. Since you don't await the task, this function will continue and you will be out of the try/catch before the callback executes.
Please try to simplify your code, there seems to be something wrong with it.

Hello,I simplified my code,and run the code. but The exception of kernelbase is often thrown.

Why did you close this @wutangyuan ? How did you solve your issue ?

Why did you close this @wutangyuan ? How did you solve your issue ?
with some suggestions,I found that I could solve the problems by modifying the code.

Thanks so the issue was on your side then? Good to know you found an answer !

Thanks so the issue was on your side then? Good to know you found an answer !

This problem is caused by the resource competition of VLC player caused by multithreading.
thank you so much for your help!


Scripts exiting with an error.

I get this error on Linux Ubuntu 14.04:
./dashman install
dashman version 0.1.20 (0.1.20) - Fri Sep  2 06:12:23 EDT 2016
gathering info, please wait...
Could not find latest version from https://www.dash.org/downlaods/ -- Exiting.

I don't find that url in the source. It looks like a local edit.
try doing: ./dashman sync to remove any local edits.
Then try ./dashman install

Problem still exists after the sync:

That is the correct url.
What does curl https://www.dash.org/downloads/ | grep linux show?

It can find the file, but the literal 'linux' is not in downloads HTML

Maybe it is because he asks to complete the CAPTCHA.

It may be cloudflare requesting human proof.  Try again now.

Still stuck in requesting human proof.

Please capture the page for me and place it in a zerobin paste.  https://zerobin.net/
curl https://www.dash.org/downloads/ > downloads.html ; less downloads.html

I've created a rule for that page. Please try again.

Synced again, but still captcha problem.
I send you the output later.

Please try again now.
The issue is with cloudflare and I cannot reproduce.

Doesn't work. The problem is right now i cannot copy paste from the Linux to my PC. I can only send you this screenshot with the tail of the HTML:

Mabe it's because that IP is blacklisted somehow?

I'm not finding that ip or Ray ID.
Where is this server located?  What is it's ip address?

Its IP is <IP_ADDRESS>, in Montreal Canada.(https://www.pulseservers.com)  It is a VPS

My VPN previously run a tor exit node. That might be the reason why CloudFlare requires Capcha now.
See:
[https://support.cloudflare.com/hc/en-us/articles/203306930-Does-CloudFlare-block-Tor-]
On the other hand, tor server is stopped since 3 weeks.  When does Cloudflare unblock those IPs?


Add tests with various types

Add an integration test with

record types
ADT
union types
equivalence types
subset types

Fixed by @blitz-1306


home-assistant: ./parse-requirements.py fails to update on master

Describe the bug
I can't add a new home-assistant component because ./parse-requirements.txt fails for reasons unrelated to my changes:
> ./parse-requirements.py
$ mypy --ignore-missing-imports ./parse-requirements.py
git status
Success: no issues found in 1 source file
Generating component-packages.nix for version 0.107.7
Component androidtv is missing True dependencies
missing requirements: ['adb-shell', 'androidtv', 'pure-python-adb']
Component arwn is missing True dependencies
missing requirements: []
Component default_config is missing True dependencies
missing requirements: ['home-assistant-frontend']
Component doorbird is missing True dependencies
missing requirements: ['doorbirdpy', 'home-assistant-frontend']
Component dsmr_reader is missing True dependencies
missing requirements: []
Component google is missing True dependencies
missing requirements: []
Component huawei_lte is missing True dependencies
missing requirements: ['huawei-lte-api', 'url-normalize']
Component logbook is missing True dependencies
missing requirements: ['home-assistant-frontend']
Component logi_circle is missing True dependencies
missing requirements: ['logi_circle']
Component manual_mqtt is missing True dependencies
missing requirements: []
Component mcp23017 is missing True dependencies
missing requirements: ['RPi.GPIO', 'adafruit-blinka', 'adafruit-circuitpython-mcp230xx']
Component mqtt is missing True dependencies
missing requirements: []
Component mqtt_eventstream is missing True dependencies
missing requirements: []
Component mqtt_json is missing True dependencies
missing requirements: []
Component mqtt_room is missing True dependencies
missing requirements: []
Component mqtt_statestream is missing True dependencies
missing requirements: []
Component plex is missing True dependencies
missing requirements: []
Component safe_mode is missing True dependencies
missing requirements: ['home-assistant-frontend']
Component smartthings is missing True dependencies
missing requirements: ['pysmartapp', 'pysmartthings']
Component snips is missing True dependencies
missing requirements: []
Component telegram is missing True dependencies
missing requirements: []
Component telegram_bot is missing True dependencies
missing requirements: []
Traceback (most recent call last):
  File "./parse-requirements.py", line 182, in <module>
    main()
  File "./parse-requirements.py", line 149, in main
    attr_path = name_to_attr_path(name, packages)
  File "./parse-requirements.py", line 126, in name_to_attr_path
    req, attr_paths
AssertionError: tensorflow matches more than one derivation: {'python3Packages.tensorflow-build_2', 'python3Packages.tensorflow-bin_2', 'python3Packages.tensorflow'}

@colemickens Fixed in 02becde898971983b0382a89409fa94b97021001, can you confirm?


Project name change

After discussion with a number of people, it seems that template-Coq is not the best name to use for the whole project. @betaziliani is ok with us reusing the name MetaCoq (which was used for a former MTac version) and people seem to like it. To me it sounds better as well, given the we provide much different functionality than just reifying / reflecting terms, with the template monad to script the vernacular and the typing/checker work to provide a kind of Coq-in-Coq implementation. In my opinion, Template-Coq / MetaCoq-Template could still refer to the core reification/reflection framework we have, while MetaCoq-Gallina (or some better name) could refer to the current template monad, MetaCoq-Checker, MetaCoq-Extraction and the various translations should also be separate projects under a larger MetaCoq umbrella. If you agree, I can create a new organization and work on separating the various parts into different projects, which should be easy as they are already in separate directories. So, what do you think?

@tabareau @SimonBoulier @yforster @CohenCyril @gmalecha @aa755 I’d like to have your approval before starting to do that.

I approve. Thanks for this effort.

I think this would be a great thing as well.

Sorry, I was distracted by the CPP deadline. I agree as well!
(If there would be a better alternative, I would vote for the better alternative to avoid confusion of MTac and Template-Coq-now-MetaCoq. But I don't have a good alternative name, and I think MetaCoq is strictly better than Template-Coq)
Ah, and we're settling on MetaCoq (no dash, no space)? For Template(-)Coq, the repo and the published paper did not agree.

Yes MetaCoq, no dash nor space. I'm about to start by changing the organization name. Apparently (according to https://help.github.com/articles/renaming-an-organization/) this should support repository links for a while. I think apart from opam packages and developers .git/config files nothing else mentions the github.com address (except the ITP paper of course but I guess that's what we have to pay). Raise your hand if you are not ok.

My plan is to then rename the repository itself to metacoq but keep everything in the same repository for the time being and rather produce separate opam packages for the different components.

Ok, the organization name was just changed, please update your repository links to use github.com/MetaCoq now.

Pinging @tabareau @SimonBoulier @yforster @CohenCyril @gmalecha @aa755 @TheoWinterhalter in case you don't see this. I expect problems with this change :)


Alt arrow keys print extra characters

Terminal Version: 0.5.2762.0
Microsoft Windows [Version 10.0.18362.356]
Steps to reproduce
Open terminal.  Press alt-up, alt-down, alt-right, alt-left.
(For byobu:  start byobu with 'byobu'.  Press F2 to add a new tab.  Press alt-right and alt-left to switch between tabs.)
Expected behavior

When not running something like byobu perhaps the letters ABCD should be printed out.

When using byobu to manage console sessions it should switch between tabs.

Observed behavior

c:\windows\system32\bash.exe: ABCD is printed out as expected.
c:\windows\system32\bash.exe in byobu: Byobu tabs are switched on alt-left and alt-right. (Only a single session is running so alt-up and alt-down just error out as expected.)  No additional characters are printed out, as expected.
Windows Powershell terminal (run->powershell): Nothing is printed out. (Expected? I don't know.)
Terminal App powershell: ☻♠♦ is printed out.
Terminal App ubuntu: A◘B☻C♠D♦  is printed out.
Terminal App ubuntu in byobu: Byobu tabs are are switched as expected, BUT when alt is released one of ◘☻♠♦ is printed out depending on which arrow key was pressed last.

I'm having the same issue. Happens in both fish and bash.
I am also on Terminal version 0.5.2762.0. Windows 10 version 18362.388 (1903).
I use alt-left and alt-right to navigate through words on a command.
Terminal version: 0.5.2762.0
Windows 10 version 18362.388 (1903)

Confirmed! Possible regression from #2836 (/cc @lhecker)?

@DHowett-MSFT This issue is caused by the use of ToUnicodeEx(), which has the following warning...

As ToUnicodeEx translates the virtual-key code, it also changes the state of the kernel-mode keyboard buffer. This state-change affects dead keys, ligatures, alt+numpad key entry, and so on. It might also cause undesired side-effects if used in conjunction with TranslateMessage (which also changes the state of the kernel-mode keyboard buffer).

I'll open a PR.
@paulwhiting I was not able to exactly replicate your observed behavior. Is there a way to switch between "keypad mode" on/off? I'm "Press[ing] alt-up, alt-down, alt-right, alt-left" (in that order) and I'm not really certain why I'm getting these different results:

C:\Windows\System32\bash.exe: Alt+Up/Down causes a beep (or "bing") and Alt+Left/Right jumps between words
Terminal App powershell: ◘☻♠♦ is printed out.
Terminal App ubuntu: ♦◘☻♠ is printed out.

I believe num lock should do what you want?  My keyboard doesn't have a numeric keypad, just standalone arrow keys.

In my case, the extra keystroke will happens when I "release the Alt key" after pressing Alt+Left/Right/Up/Down.
If I doesn't release Alt key, the control key \cA \cB \cC \cD won't come out.

can confirm, I too face this. Within tmux, if i switch panes using Alt+direction keys, i get funny chars on the command line

Hi folks, sorry for stating the obvious.  Please review/approve the PR so this can be addressed.  Thanks!

Thanks, everyone!


unify execution events

it should fix #37

@antonpod please quix/issues/37 branch for integration


will support other game files? (name,skel,geom,anim)

Thank you for the wonderful plugin. This works magically for DSCS.
By the way, I know other games that use files such as mvgl and skel, geom. For example, megido 72.
This plugin is for Digimon, but are there any plans to support it for other games? If you are interested, I will upload a sample.

I don't have any plans to update the plugin for other games at the moment (it's not finished for DSCS yet, after all) but if there are minor changes between the file versions I may add experimental support for other game files. Feel free to upload a sample - I can't promise I'll implement support for it, but I will take a look and see how difficult it would be to extend to tools for it.
Glad you're enjoying the plugin so far. :)

thanks reply, Pherakki.
Send a sample file to your gmail account. mvgl can be decompressed with the same tools as DSCS.
And you too, thank you for making the plugin!
If you are interested in other games and have time, please go. It is kindness, not obligations.

For Megido, looks like there's only a few differences in the geometry file to look at, so I'll take a look at this. I got a test case importing correctly already, but I need to put those changes into the code properly. The animation files throw some unpleasant errors though, so that might take a bit longer.

Thanks for the check, Pherakki!
I'm looking forward to the follow-up report! Do not hurry, proceed at your own pace.

Fixed the bug that was in the priority queue before this, now working on implementing this feature. Thanks for being patient. :)

The latest commit should implement a loader for Megido72 models. The textures are PVR though, and Blender can't load those by default... and I haven't had any luck finding a GitHub repo that can read them, so you might have to do some research to find something that can convert the textures. I've tested all the models you uploaded, and I can load them all in and they seem to work correctly, so let me know if you find any that don't work - you can also upload some more and I can test a few more if you want.
I'm going to keep the issue open for now because

We should check for any issues with other models,
The unresolved texture issue,
Animations don't load yet.

Thank you for updating the plugin!
I tried it immediately. I haven't checked all the models yet,
but most of the humanoid models can be read. An error will occur for the monster model, so I will send a sample URL from gmail again.
The known problems are that the bones are oriented differently on the left and right,
and the UVs on the mesh are disabled. I hope these are fixed in the future!
I'll send you all the mvgl related to the model file later, but I'll continue to test the file reading!
Textures have PVR magic headers, so I'll try to convert them with some software!

I can now load up all models; there was an additional way of storing vertex normals I hadn't seen in any of the models yet that was missing. Not pushed this change yet though.
The bone orientations are actually correct - you'll see that the animations only work with these orientations (models like chrXXXX_skXXXX will load up animations currently if you import in QA mode, if you want to checkthis).
The UVs are a trickier problem. They seem to use a kind of data compression I'm not familiar with, but I'm throwing more things at it to figure out the compression technique. At least I know that what I'm doing at the moment results in the right shape, just the wrong scale...

Well, I semi-figured out the UVs. I got something that works, but I think it's a hack which might destroy some important data... or the data isn't important at all. Latest commit should make all models loadable, according to my test, and also "fix" the UVs. Last thing to go is the animations (which already work for some models, as I noted above).

Ok, and I've got that done much quicker than I expected too. The animation files now load for all characters and monsters.
There's still some issues - some models have incorrect rotations and translations in the animations, and some animations don't load due to errors. These issues are also present in the DSCS models, but to a lesser extent. Therefore, I think that any further fixes to the Megido animations do not require any Megido-specific code, and they will be fixed when the DSCS models are also fixed. The annoying thing is that lots of the Megido animations seem to suffer from these errors.
It would be good to hear if you spot anything wrong with the models after these changes - if the models all seem to load fine, I think we can probably close this issue now. I think the animations will get fixed when I find and fix the bugs with the DSCS models. You can also re-open the issue or start new ones if you have further issues. :)

Great update! thank you very much! I'm sorry I've been a little busy since then and couldn't do many tests.
I don't have much time now, but I plan to test it again next week.
Thank you for a really great update!

Most of the animation issues have now been fixed. There's still a few issues; weapons tend to be displaced from the characters (and monster 61 has a related issue with its hands), but other than those displacement issues everything that I've manually checked seems to work good.

Did you find the time to test out the new features?

I'm sorry to have kept you waiting! I've finally finished my work and will check tomorrow and the day after tomorrow!
I haven't been able to check it at all since I last contacted you, so I'm really looking forward to a fixes and loading of 3D models!

Sorry I made you wait! Currently checking. First, UV loading is now accurate! Now you don't have to fix it manually.
I'll also test other models and animations!

As is often seen in other models, the positive and negative of the location information may be incorrect on the left and right of the animation.
The Megido72 model has different bone orientations on the left and right, which may be the cause.
If the position is wrong, it seems that the correct information is usually obtained by simply inverting the positive and negative directions.

*In the case of the sample, the right arm is below the ground, which can be fixed by changing the Y position from 0.711 to -0.711.


Add more info from youtube.

currently get_video only returns "videoPrimaryInfoRenderer" part from "ytInitialData". although for some other applications having "ytInitialPlayerResponse" would be useful. like storyboards thumbnails.
def get_video(
    id: str,
) -> dict:

    """Get a single video.

    Parameters:
        id (``str``):
            The video id from the video you want to get.
    """

    session = get_session()
    url = f"https://www.youtube.com/watch?v={id}"
    html = get_initial_data(session, url)
    client = json.loads(
        get_json_from_html(html, "INNERTUBE_CONTEXT", 2, '"}},') + '"}}'
    )["client"]
    session.headers["X-YouTube-Client-Name"] = "1"
    session.headers["X-YouTube-Client-Version"] = client["clientVersion"]
    data = json.loads(
        get_json_from_html(html, "var ytInitialData = ", 0, "};") + "}"
    )
    ytInitialPlayerResponse = json.loads(
        get_json_from_html(html, "var ytInitialPlayerResponse = ", 0, "};") + "}"
    )
    returning = next(search_dict(data, "videoPrimaryInfoRenderer"))
    returning['ytInitialPlayerResponse'] = ytInitialPlayerResponse
    return returning

something like this would be appreciated.

forgot to mention. it should also do similar on get_videos too


[manila-csi-plugin] unknown flag: --runtime-config-file in nfs-nodeplugin container

/kind bug

What happened:
I'm testing the new latest version of Manila CSI plugin to test the changes in this PR, but when I try to deploy the helm chart nfs-nodeplugin containers are unable to run and show the error above. I've checked the image version is the latest one
What you expected to happen:
Pods start running
How to reproduce it:
Deploy the NFS driver from https://github.com/kubernetes-csi/csi-driver-nfs
cd <path/to-csi-driver-nfs>
kubectl -f deploy/kubernetes create
Deploy the helm chart for Manila CSI plugin:
cd <path/to/cloud-provider-openstack/charts/manila-csi-plugin>
helm install . --name= --namespace=

It looks like the docker images in my K8s nodes were not updating. I've logged into the machines, removed the images and launched the Helm chart again so they were freshly downloaded.
Sorted