No Minified Version on npm

There is no minified version of bootstrap when installed with npm. To see what I mean, run npm install angular-ui-bootstrap in any empty directory, then inspect node_modules/angular-ui-bootstrap/dist, which holds the files built for the repository. You will see that there is a ui-bootstrap-tpls.js and a ui-bootstrap.js, but no ui-bootstrap-tpls.min.js or ui-bootstrap.min.js.
However, on bower, if you do the same thing (run bower install angular-bootstrap in an empty directory, then inspect bower_components/angular-bootstrap/), you will note that there are minified JavaScript files. Specifically, there is a ui-bootstrap-tpls.min.js and a ui-bootstrap.min.js. I tried with version 2.2.0 for both npm and bower.
First of all, was this intentional, or an oversight? Are we expected to minify the npm version as part of the build? If so, is there an npm script in this repository that can be hooked into, or at least a sample script, or will just a simple uglify build work fine?

This was intentional, the idea being that when using the npm version, a build process is used and smarter strategies are used to only require/import what is needed. This has been discussed in a prior issue, and will not be supported.

OK, thanks. I did not see the earlier issue, could this be put into the README or somewhere? Thanks.


A way to fill "location" field with an address

Is it possible to fill calendar event with an adress ?
Currently location property seems to be related to timezone.

I still can't see a venue in the event, even though I've filled in the location field in the Event object


give context about purpose of this section, #2421

Otherwise it's unclear why you should be installing at all (the quickstart describes a hello.py file that doesn't require any installation). Resolves #2421.

But this doesn't clarify why you would install it. It just says it's customary, which was basically the same issue you had with the original.

Sorry, maybe I poorly explained what my original problem was, or used poor choice of words.
I had already read the quickstart, and was following along with the tutorial, and then hit a chapter that started talking about how to install my webapp. I had no idea why this would be necessary, since in the quickstart all I had to do was set some env vars and do flask run. Further, I had no idea that it was even possible to install a package into my venv and have auto-reload automatically work (pip --editable .). I just wanted to set some env vars and run flask run and have it work. What I was shooting for in the paragraph I added in the PR is: "don't panic, when your webapp is a package (instead of a top-level hello.py script), you must install it and that's just part of the dev process". I'm fine with that explanation, and I think the reader will be as well (at least, where they are for now, in chapter 3 of the tut).
I'm guessing that later, during deployment, when the instructions say something like, "so, install your webapp package into...", then the reader will be pleased to know that they already have their webapp set up as a package, and that this is one of the reasons it was useful to set that up in chapter 3 of the tut.

This intent of this PR resonates with me, as I had pretty much the same reaction when I first started working with flask and wasn't very familiar with python packaging of "why would I do this, flask run does what I need..."
but I also agree with @davidism that the "customary" wording isn't helpful to n00bs--it would be much better to actually explain the benefits of creating your own python package.

Closing in favor of a more thorough rewrite. Thanks for bringing this up.


[request] Add onControllerReady callback to expose its controller

Add onControllerReady callback to expose its controller after init controller

There actually used to be a onInit callback, but I couldn't come up with a use case that wasn't already covered by onPlay, so I removed it.
Please feel free to reopen this with concrete examples of what you'd use this for that onPlay doesn't work for.

Hi @gskinner.
in my example. setState will cause it rebuild and stop() will be called.

import 'package:flutter/material.dart';
import 'package:flutter_animate/flutter_animate.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});
  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  AnimationController? btnCtl;
  bool loading = false;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            if (loading) Text('Loading'),
            IconButton(
                    onPressed: () {
                      setState(() {
                        loading = true;
                      });
                      btnCtl?.repeat();
                      Future.delayed(3.seconds, () {
                        btnCtl?.stop();
                        setState(() {
                          loading = false;
                        });
                      });
                    },
                    icon: const Icon(Icons.new_releases))
                .animate(onPlay: (c) {
              btnCtl = c;
              btnCtl?.stop();
            }).rotate(duration: 1.seconds, curve: Curves.fastOutSlowIn),
          ],
        ),
      ),
    );
  }
}


Synopsys VCS support for Google DV in ubuntu

Hello, I'm trying to install google DV in Red Hat, because I only have Red Hat version of  Sysnopsys VCS. But facing issues in installing git, there are some mirror issues.

Hello,
Can you please share the specific error you get with us?

when I try to install git "yum install git", it shows this error.
"This system is not registered with an entitlement server. You can use subscription-manager to register".
Then on registering through  subscription-manager, I get this
"Unable to verify server's identity: certificate verify failed"

is it possible to run google RISC-V DV without simulator like vcs or questa. Is it possible to just copy the output files of google RISC-V DV process  to windows and run those files in simulator in windows.


issue-56581-resolved

Fixes #
https://github.com/microsoft/TypeScript/issues/56581

This PR doesn't have any linked issues. Please open an issue that references this PR. From there we can discuss and prioritise. comment Addressed

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree

This is most definitely not correct. The "built" dir is not shipped to users. The actual package is produced via hereby lkg
If anything, we should remove the start script.

@jakebailey shall I remove start script ? As I have checked with 'hereby log' it is working fine.

@jakebailey @typescript-bot @rictic @Peeja Can you please look into this issue.

@PUNAMRAMPUKALE Sorry, what's this about? Was this an issue I was involved with?

@Peeja Not sure but this issue tagged under 'Infrastructure' label.

Please don't ping random people. Half of who you pinged don't work on TypeScript, and the other is a bot.

@jakebailey I will keep in mind.can you please confirm this PR as it is pending since a few days


Problems with barrels, BOP woods and logistical sorters

I built a sorting system that tries to sort wood items in to barrels from the JABBA mod. This works perfectly with vanilla logs and also with BOP (Biomes O' Plenty) when it tries to sort them into chests. But when I try to get the BOP logs in to barrels, the game kind of freezes. I can still move around and place blocks etc, but I can't interact with blocks (by rightlick). The chest stays open too. I added some images to make it more clear what I am trying and what is happening. When I try to close the world it freezes at the "shutting down internal server..." screen. If you are unable to reproduce this bug, I will try to isolate the issue more, so please tell me.

Would you mind providing your log?

There is nothing interesting in it, I already looked. http://pastebin.com/a0Hvd7Wn
Or are there other log files I/you can take a look at?

This is also happening on my modpack, here is a warmroast stack taken during when the server was hanging. http://i.imgur.com/u05tNcE.png
The offending setup involved logistical transporters, pipes and JABBA Barrels.

What version are you all using?

I am using <IP_ADDRESS>

@Xaethon, do me a favor and report this to @ProfMobius and give him that report - after looking through my code I don't know where this could be happening on my end.  Feel free to report back if he confirms it's due to Mekanism.

Please check that you are using the latest version of Jabba.  The most recently released version has a fix that may resolve this.  The log that was linked to earlier was not a complete log and did not include the first part where the mod filenames/versions are included.

You are right, I didn't note there was a new version. Sorry about that and thanks for the help!


Fix(RWX):  long remount time when share manager pod restart

Which issue(s) this PR fixes:

Issue longhorn/longhorn#8999
What this PR does / why we need it:
This PR addressed a regression where remounting NFS volume hangs for an extended period before succeeding. The regression is a side effect from  https://github.com/longhorn/longhorn-manager/pull/2815, and the way the share manager's Service is cleaned up caused the remounting to hang after the share manager Pod restarts.
This PR proposes moving the responsibility of cleaning up the Service and Endpoint to the setting controller. This is a more logical fit since the setting controller handles the storage network change, which aligns better with the purpose of this cleanup process. This change should eliminate the race condition.
Special notes for your reviewer:
None
Additional documentation or context

https://github.com/longhorn/longhorn/issues/8999#issuecomment-2228183416

The failure from CodeFactor is non-related to the changes proposed in this PR.

Test WIP.

The regression is a side effect from https://github.com/longhorn/longhorn-manager/pull/2815, and the way the share manager's Service is cleaned up caused the remounting to hang after the share manager Pod restarts.
Is it because we reference the service using a cluster IP, and now the cluster IP has changed?

@mergify backport v1.7.x


chore: remove convert

Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:
CI Switch
ci-runs-only: [Chaos CI, Cargo Clippy, Coverage Test, E2E Tests, Code Format, Unit Tests]

Codecov Report

Merging #347 (adab28d) into main (821fa64) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #347      +/-   ##
==========================================
+ Coverage   35.97%   35.98%   +0.01%
==========================================
  Files         119      119
  Lines        8228     8228
==========================================
+ Hits         2960     2961       +1
+ Misses       5268     5267       -1

Impacted Files
Coverage Δ

common/memory-tracker/src/lib.rs
0.00% <ø> (ø)

core/mempool/src/tx_wrapper.rs
89.28% <0.00%> (+1.78%)
:arrow_up:


Training Freeze after "Shuffle merged datasets" (and adding position ids)

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
Training should proceed without issue when accelerate launch --use_deepspeed -m axolotl.cli.train axolotl_bittensor_llama3_finetuning.yaml is run
Current behaviour
The datasets tokenize but invariably training will fail to begin, as axolotl freezes up after datasets are being shuffled:

This is on a second or third run. The first one, axolotl froze after the adding position ids step.
Config:
base_model: meta-llama/Meta-Llama-3-8B
# Heralax/bittensor-mistral-pretrained-base-1
#mistralai/Mistral-7B-v0.1
# Heralax/bittensor-mistral-pretrained-base-1
#mistralai/Mistral-7B-v0.1
model_type: LlamaForCausalLM
tokenizer_type: AutoTokenizer
is_mistral_derived_model: false

load_in_8bit: false
load_in_4bit: false
strict: false

datasets:
  - path: json
    data_files: ./essays_annotation_syspromptvaried.jsonl
    ds_type: json
    type: sharegpt
    conversation: chatml
  - path: json
    data_files: ./tweets_annotation_syspromptvaried.jsonl
    ds_type: json
    type: sharegpt
    conversation: chatml
  - path: json
    data_files: ./autometa_4_percent.json
    ds_type: json
    type: sharegpt
    conversation: chatml
  # - path: json
  #   data_files: paul_graham_essays_completion.json
  #   ds_type: json
  #   type: completion

dataset_prepared_path: last_run_prepared
output_dir: ./paulgraham-finetune-out

sequence_len: 4096
sample_packing: true
pad_to_sequence_len: true
shuffle_merged_datasets: true

wandb_project: pg-test
wandb_entity:
wandb_watch:
wandb_run_id:
wandb_log_model:

gradient_accumulation_steps: 6
micro_batch_size: 2
eval_batch_size: 1
num_epochs: 7
optimizer: paged_adamw_8bit
lr_scheduler: cosine
learning_rate: 0.000029
weight_decay: 0
# Gradient clipping max norm
max_grad_norm: 1.0
noisy_embedding_alpha: 0

train_on_inputs: false
group_by_length: false
bf16: true
fp16: false
tf32: false

gradient_checkpointing: unsloth
early_stopping_patience:
resume_from_checkpoint:
local_rank:
logging_steps: 1
xformers_attention:
flash_attention: true

# fsdp:
  # - full_shard
  # - auto_wrap
# fsdp_config:
  # fsdp_offload_params: false
  # fsdp_state_dict_type: FULL_STATE_DICT
  # fsdp_transformer_layer_cls_to_wrap: LlamaDecoderLayer
# warmup_steps: 10
warmup_ratio: 0.5
auto_resume_from_checkpoints: false
#warmup_ratio: 0.5
eval_steps: 10
saves_per_epoch: 1
eval_sample_packing: false
save_total_limit: 2
debug:
deepspeed: deepspeed_configs/zero2.json
chat_template: chatml
special_tokens:
  bos_token: "<s>"
  eos_token: "</s>"
  unk_token: "<unk>"
tokens:
  - "<|im_start|>"
  - "<|im_end|>"

Oddly enough I do not see a last_run_prepared folder.
This is on 6x A40s rented using RunPod, using the official axolotl docker image.
Steps to reproduce

Run config (presumably using any sharegpt sets + any completion set will cause problems)
Wait for datasets to tokenize
Observe freezing

rolling back to 5f58555bd0dbf15cae25fc021eb00421e53e47b2 does not seem to have helped.
Config yaml
base_model: meta-llama/Meta-Llama-3-8B
# Heralax/bittensor-mistral-pretrained-base-1
#mistralai/Mistral-7B-v0.1
# Heralax/bittensor-mistral-pretrained-base-1
#mistralai/Mistral-7B-v0.1
model_type: LlamaForCausalLM
tokenizer_type: AutoTokenizer
is_mistral_derived_model: false

load_in_8bit: false
load_in_4bit: false
strict: false

datasets:
  - path: json
    data_files: ./essays_annotation_syspromptvaried.jsonl
    ds_type: json
    type: sharegpt
    conversation: chatml
  - path: json
    data_files: ./tweets_annotation_syspromptvaried.jsonl
    ds_type: json
    type: sharegpt
    conversation: chatml
  - path: json
    data_files: ./autometa_4_percent.json
    ds_type: json
    type: sharegpt
    conversation: chatml
  # - path: json
  #   data_files: paul_graham_essays_completion.json
  #   ds_type: json
  #   type: completion

dataset_prepared_path: last_run_prepared
output_dir: ./paulgraham-finetune-out

sequence_len: 4096
sample_packing: true
pad_to_sequence_len: true
shuffle_merged_datasets: true

wandb_project: pg-test
wandb_entity:
wandb_watch:
wandb_run_id:
wandb_log_model:

gradient_accumulation_steps: 6
micro_batch_size: 2
eval_batch_size: 1
num_epochs: 7
optimizer: paged_adamw_8bit
lr_scheduler: cosine
learning_rate: 0.000029
weight_decay: 0
# Gradient clipping max norm
max_grad_norm: 1.0
noisy_embedding_alpha: 0

train_on_inputs: false
group_by_length: false
bf16: true
fp16: false
tf32: false

gradient_checkpointing: unsloth
early_stopping_patience:
resume_from_checkpoint:
local_rank:
logging_steps: 1
xformers_attention:
flash_attention: true

# fsdp:
  # - full_shard
  # - auto_wrap
# fsdp_config:
  # fsdp_offload_params: false
  # fsdp_state_dict_type: FULL_STATE_DICT
  # fsdp_transformer_layer_cls_to_wrap: LlamaDecoderLayer
# warmup_steps: 10
warmup_ratio: 0.5
auto_resume_from_checkpoints: false
#warmup_ratio: 0.5
eval_steps: 10
saves_per_epoch: 1
eval_sample_packing: false
save_total_limit: 2
debug:
deepspeed: deepspeed_configs/zero2.json
chat_template: chatml
special_tokens:
  bos_token: "<s>"
  eos_token: "</s>"
  unk_token: "<unk>"
tokens:
  - "<|im_start|>"
  - "<|im_end|>"

Possible solution
No response
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.11
axolotl branch-commit
main/c86c32a
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Nevermind, might've been an issue with hte specific instance. Got a new one and made a new HF token for paranoia -- it worked.


Unable to modify type union field through ref function

The following code fails to compile even though modification of a trn field through a ref function is allowed when its type is not a union:
class trn Foo
  let foo: (Array[I64] trn | None) = None

  fun ref a() =>
    match (consume this).foo
    | let ns: Array[I64] trn => ns.push(1)
    end

Output:
theodus@linux-xec3 ~/src/theodus/stuff/wat2 $ ponyc --version
0.14.0-f70dba09 [release]
compiled with: llvm 3.8.1 -- clang version 4.0.0 (tags/RELEASE_400/final 297347)
theodus@linux-xec3 ~/src/theodus/stuff/wat2 $ ponyc
Building builtin -> /usr/local/lib/pony/0.14.0-f70dba09/packages/builtin
Building . -> /home/theodus/src/theodus/stuff/wat2
Error:
/home/theodus/src/theodus/stuff/wat2/main.pony:7:7: this capture violates capabilities
    | let ns: Array[I64] trn => ns.push(1)
      ^
    Info:
    /home/theodus/src/theodus/stuff/wat2/main.pony:3:13: match type: (Array[I64 val] tag | None val)
      let foo: (Array[I64] trn | None) = None
                ^
    /home/theodus/src/theodus/stuff/wat2/main.pony:7:7: pattern type: Array[I64 val] trn
        | let ns: Array[I64] trn => ns.push(1)
          ^

Based on the sync discussion of changing the way as works, this will be possible with as (and it's correctly not compiling with the match).


[BUG] Crash on 1.19.3

Describe the bug
Game launches but when pressing open to lan, it crashes the game.
To Reproduce
Steps to reproduce the behavior:

Press Open to lan

Expected behavior
Should open the lan menu and let users start server

Could you provide the crash log?

crash-2022-12-17_01.42.26-client.txt
Thanks for looking at it

Oh I misread the title. This mod hasn't been updated to 1.19.3 yet. I'll upload the latest version within the next hour.
I will mark this as closed when it has been uploaded

Hello, I have updated this mod
You may find it now on Modrinth or on Curseforge when the file gets approved (usually takes up to an hour)


Force click on OS X

When using a touchpad on OS X it is possible in many apps to use a force click -- which is a light press followed by a hard press.  For example, in Finder, a light press selects a file, and a force click (pushing harder) edits the filename.  MacVim could (and in my opinion should) use the force click to begin insert mode.

this is an interesting feature for MacVim GUI only.


Default server config disables Internet access from client

Server:

OS: Ubuntu 18.04
Cloud provider: AWS

Client:

OS: Mac OSX 10.12.6
OpenVPN client: Tunnelblick 3.7.6a

Issue:
When connected to the VPN server after installing, I cannot access external Internet sites from my client. I resolved the issue by commenting out the following line from /etc/openvpn/server.conf:
#push "redirect-gateway def1 bypass-dhcp"

That makes sense. We have a custom DNS config on our clients which can't be routed through the VPN. I'll just leave the line commented out for now, thanks!


GC+: UncommittedGarbageCollector Unit Test

Implement unit test

Decided to test the main logic as part of integration testing. Using this issue to add more test to listObjects


feat: Added my Own Enode

Just added new enode from my stable server. hopefully it helps for faster sync

Appreciate it but not looking to add bootnodes atm!


init java sdk

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: huan:x: diaozxin007You have signed the CLA already but the status is still pending? Let us recheck it.

It's great to see our very first Java Wechaty Pull Request is submitted, thank you so much @diaozxin007 !
Please sign the CLA by clicking the [CLA | not signed yet] button in the first reply under this PR before we can merge it, I appreciate!

It's a HUGE step for our Java Wechaty with this PR merged because you have got such a lot of works done!
Thank you very much for the amazing work, and we can discuss the linting, file structure, unit testing in the future PRs.
I apprecate for your effort!

I have invited you to the @wechaty/contributors team, please accept it because this repo is current under the Wechaty org.

You've invited @diaozxin007 to Wechaty! They'll be receiving an email shortly. They can also visit https://github.com/wechaty to accept the invitation.


Merge latest changes of dev. - [closed]

In GitLab by @wknet123 on May 24, 2018, 08:39
Merges updatek8s18 -> updatek8s18

In GitLab by @wknet123 on Nov 13, 2019, 11:02
closed


feat(exporter-trace-otlp-http): fallback to XHR if sendBeacon fails

Which problem is this PR solving?
sendBeacon fails if the given payload is too big. It's browser-specific.
If user specifies a large limits in batch processor, all send spans will be dropped.
Short description of the changes
This PR handles such situations and fallbacks to XHR if sendBeacon returns false.
It also provides a mechanism which prevents further tries on payloads larger than previous fails.
It will minimize number of console errors.
Type of change
New feature (non-breaking change which adds functionality)
How Has This Been Tested?
Tested in UTs.
Checklist:

[X] Followed the style guidelines of this project
[X] Unit tests have been added
[ ] Documentation has been updated

@legendecas @svetlanabrennan please review again because I had troubles with merging master and had to move resetSendWithBeacon as a private property of the OTLPExporterBrowserBase (the test file used to be in the same package, but now it's two different packages).

Apparently I can't use @ts-ignore because of linter. Do you have any other ideas how to manage it?


Add option to truncate disassembly

Quick change to produce a truncated output. This may not be worth merging in as it is not our long term solution.

@yurydelendik WDYT? This could also be used in Firefox DevTools then, instead of generating the full disassembly and truncating it afterwards (which for big modules means that it ran out of memory before).

Looks good. FWIW, we have something like that on FF side: https://searchfox.org/mozilla-central/source/devtools/client/debugger/src/utils/wasm.js#156-161

This is different, as mentioned before, since the solution in FF DevTools runs out of memory before it gets to truncate the string.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Name license type in gemspec

Name the license type offered by the  LICENSE file, BSD 2-clause.

This would be incredibly useful as this populated rubygems with data.

Thought?


Which version of node is required?

Can all node versions (0.x, 4.x, 5.x) work with azure-documentdb-node? Thanks in advance.

We have tested with 0.x and 4.x but not 5.x. Let us know if you run into any issue with any of the versions you have installed.

I haven't had any issues running this on 5.x or 6.x.  However, I would recommend running on at least 5.4 to avoid issue #102

Thanks @rnagpal and @digitalclover .
We actually hit an issue on 4.x and 0.x, but we did not note the exact error message. We are trying to reproduce the error.
CC: @sebastus

@bingosummer : Feel free to open an issue with the repro steps if you run into any problem. azure-documentdb-node itself doesn't enforces any specific version and gets the latest version from npm.


client libs tags fix

updating client library generation to reflect the view used for the tags property of a node
@RackHD/veyron @king-jam

BUILD on-http #176 : FAILURE

BUILD on-http #176  Error Logs  ▼Test Name: Http.Api.Skus.2.0 "before all" hook: start HTTP server
Error Details: timeout of 5000ms exceeded. Ensure the done() callback is being called in this test.
Stack Trace: Error: timeout of 5000ms exceeded. Ensure the done() callback is being called in this test.
Test Name: Http.Api.workflowTasks.2.0 "before all" hook: start HTTP server
Error Details: timeout of 5000ms exceeded. Ensure the done() callback is being called in this test.
Stack Trace: Error: timeout of 5000ms exceeded. Ensure the done() callback is being called in this test.
Test Name: Http.Api.workflowTasks.2.0 "after all" hook: stop HTTP server
Error Details: Cannot read property 'send' of undefined
Stack Trace: TypeError: Cannot read property 'send' of undefined
at StatsDService.Client.send (node_modules/on-core/node_modules/node-statsd/lib/statsd.js:221:16)
at StatsDService.Client.sendAll (node_modules/on-core/node_modules/node-statsd/lib/statsd.js:181:10)
at StatsDService.Client.increment (node_modules/on-core/node_modules/node-statsd/lib/statsd.js:76:8)
at StatsDService.stop (node_modules/on-core/lib/services/statsd.js:52:14)
at node_modules/on-core/lib/services/core.js:47:28
at arrayMap (node_modules/lodash/index.js:1406:25)
at Function.map (node_modules/lodash/index.js:6710:14)
at CoreService.stop (node_modules/on-core/lib/services/core.js:46:26)
at app.js:9:2859
at bound (domain.js:287:14)
at runBound (domain.js:300:12)
at tryCatcher (node_modules/on-core/node_modules/bluebird/js/main/util.js:26:23)
at Promise._settlePromiseFromHandler (node_modules/on-core/node_modules/bluebird/js/main/promise.js:510:31)
at Promise._settlePromiseAt (node_modules/on-core/node_modules/bluebird/js/main/promise.js:584:18)
at Promise._settlePromises (node_modules/on-core/node_modules/bluebird/js/main/promise.js:700:14)
at Async._drainQueue (node_modules/on-core/node_modules/bluebird/js/main/async.js:123:16)
at Async._drainQueues (node_modules/on-core/node_modules/bluebird/js/main/async.js:133:10)
at Immediate.Async.drainQueues [as _onImmediate] (node_modules/on-core/node_modules/bluebird/js/main/async.js:15:14)

jenkins, test this please

BUILD on-http #177 : FAILURE

There were two builds going for this PR at the same time so I aborted one of them to free the resources.  The failure above was that abort.  There is still a test running.


feat: Use spf13/cobra

Codecov Report

Merging #14 (2777d02) into main (402d276) will decrease coverage by 2.74%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
- Coverage   81.39%   78.65%   -2.75%
==========================================
  Files           3        3
  Lines          86       89       +3
==========================================
  Hits           70       70
- Misses         14       17       +3
  Partials        2        2

Impacted Files
Coverage Δ

cmd/root.go
76.31% <0.00%> (ø)

cmd/version.go
100.00% <ø> (ø)

main.go
0.00% <0.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 402d276...2777d02. Read the comment docs.


kcp transport fails on unity linux server build

Describe the bug
I build server on windows machine for linux (pictures attached)
With network manager object having kcp transport linux server fails to start
With network manager object having simple web transport linux server starts and active
On windows all works fine with both transport types
How to reproduce the issue, step by step

It can be reproduced by means of ping-pong game, just build it as server for linux having kcp as transport

Expected behavior
Server fails with logs as in file attached (kcp_logs_with_failure)
kcp_logs_with_failure.txt
websocket_logs_successful_startup.txt
Screenshots
Sure

Desktop (please complete the following information):

OS: Windows
Build target: Linux
Unity version: 2020.2.1f1
Mirror branch: 35.1.0 I believe, as downloaded from unity assets store on 04/05/2021

Does this help?
https://www.coburnsdomain.com/2019/12/google-cloud-unity-magic-number-wrong-542/

Closing for lack of response.

Any solution or workaround? The same exact thing is occurring to me

Does this help?
https://www.coburnsdomain.com/2019/12/google-cloud-unity-magic-number-wrong-542/

Doing this command before launching the server turned out to be the workaround.
export TERM=xterm
then
echo $TERM
to make sure the first command worked


Prefixes in Pocket label

Is there a way to add label prefixes in the pocket label? I try custom and add a + in front but didn't work.

@ssLiu01, including label prefixes on pocket labels is not supported. Sorry.


Remove SelectPanel's backdrop

What are you trying to accomplish?

The SelectPanel component should not render a backdrop. See: https://github.slack.com/archives/C06F2US6HU6/p1724361897840679
Screenshots

Before
After

Integration

No changes necessary in production.
List the issues that this change affects.

Closes https://github.com/github/primer/issues/3878
Risk Assessment

[x] Low risk the change is small, highly observable, and easily rolled back.
[ ] Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
[ ] High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

Accessibility

No new axe scan violation - This change does not introduce any new axe scan violations.

Merge checklist
~- [ ] Added/updated tests~
~- [ ] Added/updated documentation~
~- [ ] Added/updated previews (Lookbook)~

[x] Tested in Chrome
[x] Tested in Firefox
[x] Tested in Safari
[x] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

@siddharthkp you're absolutely right 😄 We actually don't support a modal variant at the moment, but we'll need to consider adding a backdrop if/when we do.


Build fails with LTO

x86_64-pc-linux-gnu-gcc  -march=native -fstack-protector-all -O2 -pipe -fdiagnostics-color=always -frecord-gcc-switches -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3 -fstack-clash-protection -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing  -Wformat -Werror=format-security -Werror=implicit-function-declaration -Werror=implicit-int -Werror=int-conversion -Werror=incompatible-pointer-types -o iseltime iseltime.o libipmiutil.a -Wl,-O1 -Wl,--as-needed -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing -Wl,--defsym=__gentoo_check_ldflags__=0  -lpthread  ../lib/libipmi_lanplus.a -lcrypto
x86_64-pc-linux-gnu-gcc -march=native -fstack-protector-all -O2 -pipe -fdiagnostics-color=always -frecord-gcc-switches -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3 -fstack-clash-protection -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing  -Wformat -Werror=format-security -Werror=implicit-function-declaration -Werror=implicit-int -Werror=int-conversion -Werror=incompatible-pointer-types -o ipmi_sample ipmi_sample.o libipmiutil.a -Wl,-O1 -Wl,--as-needed -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing -Wl,--defsym=__gentoo_check_ldflags__=0  ../lib/libipmi_lanplus.a -lcrypto
ipmicmd.c:197:12: error: type of ‘ipmi_cmdraw_ia’ does not match original declaration [-Werror=lto-type-mismatch]
  197 | extern int ipmi_cmdraw_ia( uchar cmd, uchar netfn, uchar lun, uchar sa,
      |            ^
imbapi.c:863:5: note: type mismatch in parameter 7
  863 | int ipmi_cmdraw_ia(BYTE cmd, BYTE netfn, BYTE lun, BYTE sa, BYTE bus,
      |     ^
imbapi.c:863:5: note: type ‘BYTE’ should match type ‘int’
imbapi.c:863:5: note: ‘ipmi_cmdraw_ia’ was previously declared here
ipmicmd.c:216:12: error: type of ‘ipmi_open_direct’ does not match original declaration [-Werror=lto-type-mismatch]
  216 | extern int ipmi_open_direct(char fdebug);
      |            ^
ipmidir.c:392:5: note: type mismatch in parameter 1
  392 | int ipmi_open_direct(int fdebugcmd)
      |     ^
ipmidir.c:392:5: note: type ‘int’ should match type ‘char’
ipmidir.c:392:5: note: ‘ipmi_open_direct’ was previously declared here
ipmicmd.c:197:12: error: type of ‘ipmi_cmdraw_ia’ does not match original declaration [-Werror=lto-type-mismatch]
  197 | extern int ipmi_cmdraw_ia( uchar cmd, uchar netfn, uchar lun, uchar sa,
      |            ^
imbapi.c:863:5: note: type mismatch in parameter 7
  863 | int ipmi_cmdraw_ia(BYTE cmd, BYTE netfn, BYTE lun, BYTE sa, BYTE bus,
      |     ^
imbapi.c:863:5: note: type ‘BYTE’ should match type ‘int’
imbapi.c:863:5: note: ‘ipmi_cmdraw_ia’ was previously declared here
ipmicmd.c:216:12: error: type of ‘ipmi_open_direct’ does not match original declaration [-Werror=lto-type-mismatch]
  216 | extern int ipmi_open_direct(char fdebug);
      |            ^
ipmidir.c:392:5: note: type mismatch in parameter 1
  392 | int ipmi_open_direct(int fdebugcmd)
      |     ^
ipmidir.c:392:5: note: type ‘int’ should match type ‘char’
ipmidir.c:392:5: note: ‘ipmi_open_direct’ was previously declared here
itsol.h:61:5: error: type of ‘open_sockfd’ does not match original declaration [-Werror=lto-type-mismatch]
   61 | int open_sockfd(char *node, SockType *sfd, SOCKADDR_T  *daddr,
      |     ^
ipmilan.c:531:5: note: type mismatch in parameter 2
  531 | int open_sockfd(char *node, int port, SockType *sfd, SOCKADDR_T  *daddr,
      |     ^
ipmilan.c:531:5: note: ‘open_sockfd’ was previously declared here
ipmilan.c:531:5: note: code may be misoptimized unless ‘-fno-strict-aliasing’ is used
isol.c:145:13: error: type of ‘lan_keepalive’ does not match original declaration [-Werror=lto-type-mismatch]
  145 | extern int  lan_keepalive(int type);
      |             ^
ipmilan.c:2218:6: note: type mismatch in parameter 1
 2218 | int  lan_keepalive( uchar bType )
      |      ^
ipmilan.c:2218:6: note: type ‘unsigned char’ should match type ‘int’
ipmilan.c:2218:6: note: ‘lan_keepalive’ was previously declared here
ievents.c:629:12: error: type of ‘GetSensorType’ does not match original declaration [-Werror=lto-type-mismatch]
  629 | extern int GetSensorType(int snum, uchar *stype, uchar *rtype);
      |            ^
isensor.c:1111:5: note: type mismatch in parameter 1
 1111 | int GetSensorType(uchar snum, uchar *stype, uchar *rtype)
      |     ^
isensor.c:1111:5: note: type ‘unsigned char’ should match type ‘int’
isensor.c:1111:5: note: ‘GetSensorType’ was previously declared here
ievents.c:627:14: error: type of ‘get_unit_type’ does not match original declaration [-Werror=lto-type-mismatch]
  627 | extern char *get_unit_type(uchar iunit, uchar ibase, uchar imod, int flg);
      |              ^
isensor.c:628:7: note: type mismatch in parameter 1
  628 | char *get_unit_type(int iunits, int ibase, int imod, int fshort)
      |       ^
isensor.c:628:7: note: type ‘int’ should match type ‘unsigned char’
isensor.c:628:7: note: ‘get_unit_type’ was previously declared here
ievents.c:784:13: error: type of ‘get_MemDesc’ does not match original declaration [-Werror=lto-type-mismatch]
  784 | extern int  get_MemDesc(int array, int dimm, char *desc, int *psz); /*mem_if.c*/
      |             ^
mem_if.c:617:5: note: type mismatch in parameter 1
  617 | int get_MemDesc(UCHAR array, UCHAR dimm, char *desc, int *psize)
      |     ^
mem_if.c:617:5: note: type ‘UCHAR’ should match type ‘int’
mem_if.c:617:5: note: ‘get_MemDesc’ was previously declared here
make: TMPDIR value /tmp/iu/ipmiutil-3.1.9: No such file or directory
make: using default temporary directory '/tmp'
make: TMPDIR value /tmp/iu/ipmiutil-3.1.9: No such file or directory
make: using default temporary directory '/tmp'
lto1: some warnings being treated as errors
lto-wrapper: fatal error: x86_64-pc-linux-gnu-gcc returned 1 exit status
compilation terminated.
/usr/lib/gcc/x86_64-pc-linux-gnu/13/../../../../x86_64-pc-linux-gnu/bin/ld: error: lto-wrapper failed
collect2: error: ld returned 1 exit status
make[2]: *** [Makefile:1033: libipmiutil.so.1] Error 1
make[2]: *** Waiting for unfinished jobs....
ipmicmd.c:197:12: error: type of ‘ipmi_cmdraw_ia’ does not match original declaration [-Werror=lto-type-mismatch]
  197 | extern int ipmi_cmdraw_ia( uchar cmd, uchar netfn, uchar lun, uchar sa,
      |            ^
imbapi.c:863:5: note: type mismatch in parameter 7
  863 | int ipmi_cmdraw_ia(BYTE cmd, BYTE netfn, BYTE lun, BYTE sa, BYTE bus,
      |     ^
imbapi.c:863:5: note: type ‘BYTE’ should match type ‘int’
imbapi.c:863:5: note: ‘ipmi_cmdraw_ia’ was previously declared here
ipmicmd.c:216:12: error: type of ‘ipmi_open_direct’ does not match original declaration [-Werror=lto-type-mismatch]
  216 | extern int ipmi_open_direct(char fdebug);
      |            ^
ipmidir.c:392:5: note: type mismatch in parameter 1
  392 | int ipmi_open_direct(int fdebugcmd)
      |     ^
ipmidir.c:392:5: note: type ‘int’ should match type ‘char’
ipmidir.c:392:5: note: ‘ipmi_open_direct’ was previously declared here
ipmicmd.c:197:12: error: type of ‘ipmi_cmdraw_ia’ does not match original declaration [-Werror=lto-type-mismatch]
  197 | extern int ipmi_cmdraw_ia( uchar cmd, uchar netfn, uchar lun, uchar sa,
      |            ^
imbapi.c:863:5: note: type mismatch in parameter 7
  863 | int ipmi_cmdraw_ia(BYTE cmd, BYTE netfn, BYTE lun, BYTE sa, BYTE bus,
      |     ^
imbapi.c:863:5: note: type ‘BYTE’ should match type ‘int’
imbapi.c:863:5: note: ‘ipmi_cmdraw_ia’ was previously declared here
ipmicmd.c:216:12: error: type of ‘ipmi_open_direct’ does not match original declaration [-Werror=lto-type-mismatch]
  216 | extern int ipmi_open_direct(char fdebug);
      |            ^
ipmidir.c:392:5: note: type mismatch in parameter 1
  392 | int ipmi_open_direct(int fdebugcmd)
      |     ^
ipmidir.c:392:5: note: type ‘int’ should match type ‘char’
ipmidir.c:392:5: note: ‘ipmi_open_direct’ was previously declared here
lto1: some warnings being treated as errors
lto-wrapper: fatal error: x86_64-pc-linux-gnu-gcc returned 1 exit status
compilation terminated.
/usr/lib/gcc/x86_64-pc-linux-gnu/13/../../../../x86_64-pc-linux-gnu/bin/ld: error: lto-wrapper failed
collect2: error: ld returned 1 exit status
make[2]: *** [Makefile:1106: iseltime] Error 1
lto1: some warnings being treated as errors
lto-wrapper: fatal error: x86_64-pc-linux-gnu-gcc returned 1 exit status
compilation terminated.
/usr/lib/gcc/x86_64-pc-linux-gnu/13/../../../../x86_64-pc-linux-gnu/bin/ld: error: lto-wrapper failed
collect2: error: ld returned 1 exit status

Exhaustive build log: build.log
Downstream report: https://bugs.gentoo.org/863590

This is odd.  Apparently the typedef of BYTE as unsigned char is not included in certain cases,  I'll address this in the upcoming ipmiutil release.

I'm hoping that this is resolved in ipmiutil-3.2.0, but I cannot verify since I don't have an LTO environment to test.

You should be able to reproduce it anywhere by just adding a couple of GCC compile arguments.
In my case, I tried to build with the following {C,CXX,FC,LD}FLAGS (for ipmiutil I suppose that only CFLAGS and LDFLAGS are relevant though): -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing
Any GCC version should reproduce -- I usually test with GCC 14 but can confirm even GCC 10 will detect this issue when using the above mentioned CFLAGS for ipmiutil 3.1.9.

I just attempted to build the new 3.2.0 release. It still fails, e.g.
$ export CFLAGS="${CFLAGS} -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing"
$ export LDFLAGS="${LDFLAGS} -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing"

$ ebuild ipmiutil-3.2.0.ebuild clean install

[...]

make[2]: Entering directory '/var/tmp/portage/sys-apps/ipmiutil-3.2.0/work/ipmiutil-3.2.0/util'
x86_64-pc-linux-gnu-gcc  -pipe -march=native -fstack-protector-all -O2 -fdiagnostics-color=always -frecord-gcc-switches -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing  -Wformat -Werror=format-security -o iseltime iseltime.o libipmiutil.a -Wl,-O1 -Wl,--as-needed -Wl,-z,pack-relative-relocs -flto=4 -Werror=odr -Werror=lto-type-mismatch -Werror=strict-aliasing -Wl,--defsym=__gentoo_check_ldflags__=0  -lpthread  ../lib/libipmi_lanplus.a -lcrypto
make: TMPDIR value /tmp/iu/ipmiutil-3.2.0: No such file or directory
make: using default temporary directory '/tmp'
ipmicmd.c:197:12: error: type of ‘ipmi_cmdraw_ia’ does not match original declaration [-Werror=lto-type-mismatch]
  197 | extern int ipmi_cmdraw_ia( uchar cmd, uchar netfn, uchar lun, uchar sa,
      |            ^
imbapi.c:863:5: note: type mismatch in parameter 7
  863 | int ipmi_cmdraw_ia(BYTE cmd, BYTE netfn, BYTE lun, BYTE sa, BYTE bus,
      |     ^
imbapi.c:863:5: note: type ‘BYTE’ should match type ‘int’
imbapi.c:863:5: note: ‘ipmi_cmdraw_ia’ was previously declared here
lto1: some warnings being treated as errors
lto-wrapper: fatal error: x86_64-pc-linux-gnu-gcc returned 1 exit status
compilation terminated.
/usr/lib/gcc/x86_64-pc-linux-gnu/14/../../../../x86_64-pc-linux-gnu/bin/ld: error: lto-wrapper failed
collect2: error: ld returned 1 exit status
make[2]: *** [Makefile:1105: iseltime] Error 1


fix unit tests

Now there's a yarn error. I'm using only npm, so not sure what's the best fix (how to update the yarn lock file).

After fetching the latest code, installing via yarn install --frozen-lockfile, the install failed with an error "Your lockfile needs to be updated, but yarn was run with --frozen-lockfile." Same error as the build.
When debugging it, the packages the install complained about were :
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Removing the node_modules folder and running the install again and now the install completed.
I see in the test build, that the node_modules are retrieved from the cache. (from previous runs?)
Probably there are some other versions in that cache
For the tests in the CI we could either not retrieve the node_modules from cache or run the yarn install without the --frozen-lockfile
For that you need to update the .github/workflows/test.yml file.
There are a number of install commands for different environments, all with --frozen-lockfile and using the node_module cache.
You could also update the cache keys used, so that it results in a cache miss
According to github documentation, GitHub will remove any cache entries that have not been accessed in over 7 days.
But there is currently no way to manually clear the cache.


Typescript types

Hi, Can you please add Typescript types to this project. That would be very helpful. Thanks in advance.

This is not planned at the moment, sorry.

Perhaps this issue could be re-opened, I'd be happy to help out here and contribute types to the player. It would be extremely useful :)


python3Packages.blockdiag: 2.0.1 -> 3.0.0, python3Packages.{nw,act,seq}diag: 2.0.0 -> 3.0.0

Motivation for this change
Update to latest upstream releases and enable tests.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

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

24 packages built:

actdiag (python39Packages.actdiag)
asciidoc-full-with-plugins
blockdiag (python39Packages.blockdiag)
hy
nwdiag (python39Packages.nwdiag)
python38Packages.actdiag
python38Packages.blockdiag
python38Packages.funcparserlib
python38Packages.graphite_beacon
python38Packages.hy
python38Packages.nwdiag
python38Packages.seqdiag
python38Packages.sphinxcontrib-actdiag
python38Packages.sphinxcontrib-blockdiag
python38Packages.sphinxcontrib-nwdiag
python38Packages.sphinxcontrib-seqdiag
python39Packages.funcparserlib
python39Packages.graphite_beacon
python39Packages.hy
seqdiag (python39Packages.seqdiag)
python39Packages.sphinxcontrib-actdiag
python39Packages.sphinxcontrib-blockdiag
python39Packages.sphinxcontrib-nwdiag
python39Packages.sphinxcontrib-seqdiag

nixpkgs-review happy, macos 10.15, nixos x86_64 & aarch64


Fixes #32359

Removes the expression that returns 'red' as a color if the 'string' parameter is not set.
I was using a theme that is compiled from a .yml file to a .json file. The issue was that if I had a variable whose value was not set in the .yml file, the generated .json file would contain a null value. Looking through the source code, I found the method fromHex that would return Color.red if if the parameter passed to it couldn't be resolved.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

We want all invalid colors to be rendered red. As discussed in #32359, it's a bug of the theme to use 'null' for colors.


Problems about visualizing Oakink-Image dataset

Thank you for the amazing work on the OakInk dataset! It is a tremendous resource for the community, and I truly appreciate all the effort that has gone into making it so comprehensive and accessible.
I am currently trying to visualize the OakInk-Image dataset using the MANO layer implementation from this repository. While the code works without any errors, I have encountered an issue where the generated hand and the object do not seem to align correctly in the visualization as the picture I put below. I have reviewed the dataset documentation and my code multiple times, but I am unable to identify where the problem lies.
Below is the code I am using for visualization:
import torch
import mano
from mano.utils import Mesh
import pickle
from scipy.spatial.transform import Rotation as R
import numpy as np
import trimesh
general_info = "/Users/lyx/Downloads/dataset/OkaInk/anno/general_info/A01001_0001_0000__2021-09-26-19-59-58__0__16__0.pkl"
obj_path = "/Users/lyx/Downloads/dataset/OkaInk/obj/A01001.obj"

with open(general_info, "rb") as file:
    data = pickle.load(file)

# load object and apply transformation
obj_anno = data["obj_anno"]
obj_mesh = trimesh.load(obj_path)
obj_mesh.apply_transform(obj_anno)

# cam_extr = data["cam_extr"]
betas = data["hand_anno"]["hand_shape"].unsqueeze(0)
transl = data["hand_anno"]["hand_tsl"].unsqueeze(0)

rotation = R.from_quat(data["hand_anno"]["hand_pose"][:, [1, 2, 3, 0]])
axis_angle = rotation.as_rotvec().astype(np.float32)
global_orient = torch.from_numpy(axis_angle.flatten()[0:3]).unsqueeze(0)
pose = torch.from_numpy(axis_angle.flatten()[3:]).unsqueeze(0)

model_path = '/Users/lyx/Downloads/dataset/OkaInk/MANO/models'
n_comps = 45
batch_size = 1
rh_model = mano.load(model_path=model_path,
                     is_rhand= True,
                     num_pca_comps=n_comps,
                     batch_size=batch_size,
                     flat_hand_mean=False)

output = rh_model(betas=betas,
                  global_orient=global_orient,
                  hand_pose=pose,
                  transl=transl,
                  return_verts=True,
                  return_tips = True)

h_meshes = rh_model.hand_meshes(output)
j_meshes = rh_model.joint_meshes(output)

#visualize hand and joint and object meshes
hj_meshes = Mesh.concatenate_meshes([h_meshes[0], j_meshes[0]])
hjo_meshe = Mesh.concatenate_meshes([obj_mesh,hj_meshes])
hjo_meshe.show()

I would be very grateful if you could provide any guidance or point out potential mistakes I might have overlooked. Thank you again for this incredible dataset and for making it available to the research community!

Thank you for the amazing work on the OakInk dataset! It is a tremendous resource for the community, and I truly appreciate all the effort that has gone into making it so comprehensive and accessible.
I am currently trying to visualize the OakInk-Image dataset using the MANO layer implementation from this repository. While the code works without any errors, I have encountered an issue where the generated hand and the object do not seem to align correctly in the visualization as the picture I put below. I have reviewed the dataset documentation and my code multiple times, but I am unable to identify where the problem lies.
Below is the code I am using for visualization:

import torch

import mano

from mano.utils import Mesh

import pickle

from scipy.spatial.transform import Rotation as R

import numpy as np

import trimesh

general_info = "/Users/lyx/Downloads/dataset/OkaInk/anno/general_info/A01001_0001_0000__2021-09-26-19-59-58__0__16__0.pkl"

obj_path = "/Users/lyx/Downloads/dataset/OkaInk/obj/A01001.obj"

with open(general_info, "rb") as file:

    data = pickle.load(file)

# load object and apply transformation

obj_anno = data["obj_anno"]

obj_mesh = trimesh.load(obj_path)

obj_mesh.apply_transform(obj_anno)

# cam_extr = data["cam_extr"]

betas = data["hand_anno"]["hand_shape"].unsqueeze(0)

transl = data["hand_anno"]["hand_tsl"].unsqueeze(0)

rotation = R.from_quat(data["hand_anno"]["hand_pose"][:, [1, 2, 3, 0]])

axis_angle = rotation.as_rotvec().astype(np.float32)

global_orient = torch.from_numpy(axis_angle.flatten()[0:3]).unsqueeze(0)

pose = torch.from_numpy(axis_angle.flatten()[3:]).unsqueeze(0)

model_path = '/Users/lyx/Downloads/dataset/OkaInk/MANO/models'

n_comps = 45

batch_size = 1

rh_model = mano.load(model_path=model_path,

                     is_rhand= True,

                     num_pca_comps=n_comps,

                     batch_size=batch_size,

                     flat_hand_mean=False)

output = rh_model(betas=betas,

                  global_orient=global_orient,

                  hand_pose=pose,

                  transl=transl,

                  return_verts=True,

                  return_tips = True)

h_meshes = rh_model.hand_meshes(output)

j_meshes = rh_model.joint_meshes(output)

#visualize hand and joint and object meshes

hj_meshes = Mesh.concatenate_meshes([h_meshes[0], j_meshes[0]])

hjo_meshe = Mesh.concatenate_meshes([obj_mesh,hj_meshes])

hjo_meshe.show()

I would be very grateful if you could provide any guidance or point out potential mistakes I might have overlooked. Thank you again for this incredible dataset and for making it available to the research community!

I discover that the hand pose in the picture seems to be incorrect, but the relative position seems right. Am I use the mano incorrectly? I had little problems forming the axis-angle from the dataset and I don't know whether if I do it right

try to use this script to visualize:
https://github.com/oakink/OakInk/blob/main/scripts/viz_oakink_image_seq.py.
e.g:
python scripts/viz_oakink_image_seq.py --draw_mode mesh --view_id 0

I got:

seems the results is correct.

@lixiny Thank you for your prompt response and for providing the script! I appreciate your help and will try it out.


use_constraints example is incomplete and out of date

intended purpose for this example is to demonstrate to an on-boarder how to use constraints.
It should, therefore:

define a dm that includes one of each type of constraint (except ALARM_TABLE)
deliver a GUI for DB to render using a .grid file
demonstrate constraining behavior - i.e. setting an out of bounds value using Postman should preserve the value set within the constraints.

thinking more about this - we shouldn't make this a service, just show the interface that a connection manager would use i.e. Device::setValue, Device::getValue to keep the example as simple as possible.


Fix Start-Process -Credential -Wait to work on Windows

PR Summary
Per the docs, job objects can only contain processes within the same session, so if a credential is used, we simply WaitOnExit() instead of trying to use a job object.
Validated manually since you have to wait with a different credential.
PR Context
Fix https://github.com/PowerShell/PowerShell/issues/17033
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@PowerShell/wg-powershell-cmdlets discussed the issue and agreed that in the credential case, it's ok to not support the job object case with child processes

Docs say:

All processes associated with a job must run in the same session.

So I believe it is safe to remove try-catch and immediately call process.WaitForExit() if Credential parameter present.

@iSazonov yes, that makes sense.  I'll update the code.

Per the docs, job objects can only contain processes within the same session,

That may be true but specifying a credential does not result in the process being run in a new session, it's still the same one. The problem that you get today with Start-Process -Credential -Wait is due to PowerShell not using the hProcess object returned by CreateProcessWithLogon on the AssignProcessToJobObject stage. What is happening is:

PowerShell sees a -Credential was specified
It calls CreateProcessWithLogonW
It uses the process id returned from the above to call Process.GetProcessById <-- this is the problem
It uses that process object and it's internal Handle when calling AssignProcessToJobObject

Because dotnet is trying to open a new process handle rather than use the ALL_ACCESS handle returned by CreateProcessWithLogonW this will fail for users who don't have permission to open the process (non-admins)

Now this actually works today when you are running as admin, it will wait for the process, as well as any child processes the new process spawns as expected. Where this fails is if you try and run it as a non-administrator as while the user was able to spawn the object, process permissions on the new one will not allow the caller to get the Handle needed for AssignProcessToJobObject. I've mentioned this problem and possible solutions for it in https://github.com/PowerShell/PowerShell/issues/17033#issuecomment-1073150791.
The problem with this fix is that while it might work in some scenarios it is technically a breaking change as -Wait will no longer wait for any sub processes the new one spawns.
Just to show that this works even for non-admins if done correctly you can run this code.
# Uses Ctypes - https://www.powershellgallery.com/packages/Ctypes/0.1.0

ctypes_struct STARTUPINFOW -CharSet Unicode {
    [int]$CB
    [MarshalAs('LPWStr')][string]$Reserved
    [MarshalAs('LPWStr')][string]$Desktop
    [MarshalAs('LPWStr')][string]$Title
    [int]$X
    [int]$Y
    [int]$XSize
    [int]$YSize
    [int]$XCountChars
    [int]$YCountChars
    [int]$FillAttribute
    [int]$Flags
    [short]$ShowWindow
    [short]$Reserved2
    [IntPtr]$Reserved3
    [IntPtr]$StdInput
    [IntPtr]$StdOutput
    [IntPtr]$StdError
}

ctypes_struct PROCESS_INFORMATION -LayoutKind Sequential {
    [IntPtr]$Process
    [IntPtr]$Thread
    [int]$Pid
    [int]$Tid
}

[Flags()] enum ProcessCreationFlags {
    NONE = 0x00000000
    CREATE_SUSPENDED = 0x00000004
    CREATE_NEW_CONSOLE = 0x00000010
    CREATE_UNICODE_ENVIRONMENT = 0x00000400
}

$si = [STARTUPINFOW]@{
    CB = [System.Runtime.InteropServices.Marshal]::SizeOf([Type][STARTUPINFOW])
}
$pi = [PROCESS_INFORMATION]::new()

$advapi32 = New-CtypesLib Advapi32.dll
$kernel32 = New-CtypesLib Kernel32.dll

$commandLine = [System.Text.StringBuilder]::new("C:\Windows\System32\cmd.exe")
$res = $advapi32.SetLastError().CharSet('Unicode').CreateProcessWithLogonW[bool](
    "vagrant",
    $null,
    "vagrant",
    0,
    $commandLine.ToString(),
    $commandLine,
    [int][ProcessCreationFlags]'CREATE_SUSPENDED, CREATE_NEW_CONSOLE, CREATE_UNICODE_ENVIRONMENT',
    $null,
    "C:\Windows",
    [ref]$si,
    [ref]$pi
)
if (-not $res) {
    throw [System.ComponentModel.Win32Exception]$advapi32.LastError
}

$job = $kernel32.SetLastError().CreateJobObjectW[IntPtr]($null, "MyJob")
if ($job -eq [IntPtr]::Zero) {
    throw [System.ComponentModel.Win32Exception]$kernel32.LastError
}

$res = $kernel32.SetLastError().AssignProcessToJobObject[bool]($job, $pi.Process)
if (-not $res) {
    throw [System.ComponentModel.Win32Exception]$kernel32.LastError
}

$res = $kernel32.SetLastError().ResumeThread($pi.Thread)
if ($res -eq -1) {
    throw [System.ComponentModel.Win32Exception]$kernel32.LastError
}

$kernel32.CloseHandle[void]($pi.Process)
$kernel32.CloseHandle[void]($pi.Thread)
$kernel32.CloseHandle[void]($job)

I even do this in my ProcessEx module with no problems.

@jborean93 Can you create a new issue about this?

@PaulHigin there is one already https://github.com/PowerShell/PowerShell/issues/17033. Why would we merge a change where it changes the existing behaiour? It should be fixed properly in the first place.

@jborean93 would you be willing to submit a PR to fix this to work with a job object?

Opened https://github.com/PowerShell/PowerShell/pull/19096, just need to figure out a good way to test it out.

@doctordns What do you mean?

This issue was raised to the cmdlet working group, and we felt the issue was concluded via #19082. @SteveL-MSFT agreed, so I labeled this issue as resolved-fixed.

You write in 19082 - it is not issue, it is PR.


Add support for natural language like sentences

Most common user actions in browsers include:

click
type
select
drag-and-drop

A natural sentence say:
goto github.com and type "abc" in user name

This sentence can be easily broken down into actions which automation-for-humans understands. These can be done using a natural language processing algorithms.

@MadaraUchiha-314 Are we looking to extract the actions out of the given natural sentence?

@iriyagupta Yes.

Hi @MadaraUchiha-314 , can I take this issue for hacktoberfest ?

@cemeiq Sure!

@MadaraUchiha-314  can you assign me this issue ?


vehicles-q0x2v: metadata correction

Thank you for providing this great dataset!
While I playing around this dataset, I noticed metadata on vehicles-q0x2v is missing due to typo on labels_names.json.
This commit corrects the data manually.
You can easily check from the original data: https://universe.roboflow.com/roboflow-100/vehicles-q0x2v
Description
Please include a summary of the change and which issue is fixed or implemented. Please also include relevant motivation and context (e.g. links, docs, tickets etc.).
List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

How has this change been tested, please provide a testcase or example of how you tested the change?
By comparing manually.
Any specific deployment considerations
For example, documentation changes, usability, usage/costs, secrets, etc.
Docs

[ ] Docs updated? What were the changes:

Hi @kbumsik thank you very much. Weird bug indeed since these files were autogenerated, and thank you for catching it :heart:


[preset] update airbnb

remove disallowEmptyBlocks: nothing about it in the Airbnb JavaScript Style Guide
add new rules:

disallowSpaceBeforeComma (https://github.com/airbnb/javascript#commas)
disallowSpaceBeforeSemicolon (https://github.com/airbnb/javascript#semicolons)

/cc @markelog

remove disallowEmptyBlocks: nothing about it in the Airbnb JavaScript Style Guide

There is non-empty blocks :-), sometimes you gotta look at the code and code-examples, not everything is mentioned in the code-styles but only hints on some behaviour so i would leave that in.

Okay, i'm gonna merge this without disallowEmptyBlocks changes, if you like to continue this discussion further we would be open to that.

Thanks


Flash On Off Param in StoreCameraMediaOptions

A Request to add the Flash On/Off Param  in StoreCameraMediaOptions , please?
Thank you

No new features.


multipoint constraints

@bhaveshshrimali, from #436:

I am interested in applying scikit-fem to solve problems with general periodic boundary conditions, which can be implemented using multipoint constraints. I have done this at a higher level in Abaqus, where the kinematics of nodes on the boundary is described using some reference nodes. However I am still looking at how to implement these within a FE code written in pure python (and also, sooner than later in scikit-fem).

I wanted to know if there are plans to support these through a higher level interface such as in dolfinx_mpc, i.e. assembling FE matrices for problems with constrained kinematics.

What's a ‘general periodic boundary condition’? Is that the value at some point on the boundary is equal to that on another plus a jump? (Like maybe in a viscous incompressible fluid problem with the pressure in a length of duct, with velocity being periodic from outlet to inlet but the pressure being more or less periodic but with an overall longitudinal gradient?) Or is it more general than that, with some functional relation between the values (and normal gradients?) at the two points? Or are more than two points coupled?

Thanks for such a prompt reply. Yes that's exactly what I was referring to by periodic. By general periodic, I meant something like here.
I used the term probably a bit more loosely than should be. By general I wanted to denote a set of linear constraints that tie the dofs on opposite facets of a domain. For instance in elasticity, on a simple unit square, it would amount to saying
# u_R: displacement on the right edge (x=1)
# u_L: displacement on the left edge (x=0)
# u_T: displacement on the right edge (y=1)
# u_B: displacement on the left edge (y=0)

# u_BR: displacement of the bottom-right corner (1,0)
# u_TL: displacement of the top-left corner (0,1)

# u_o: displacement of the origin (0,0)

u_R - u_L = u_BR - u_o
u_T - u_B = u_TL - u_o

More often than not, I specify Dirichlet BC on u_o (can very well be another point in the domain), and treat u_BR/u_TL as control points, either (or both) of them could have prescribed (Dirichlet) values.
Implementation-wise this would be slightly more general than simply periodic boundary conditions (u_L = u_R and u_T = u_B), as it would even allow us to impose a combination of dirichlet and neumann bcs.  If, say, the material is incompressible and I apply u_BR = lmbda (Dirichlet) and nothing else (which is equivalent to Neumann BC on the top edge), then u_TL = 1./(1+lmbda)**(0.5) and accordingly the displacements at the top and right edges.

Like maybe in a viscous incompressible fluid problem with the pressure in a length of duct, with velocity being periodic from outlet to inlet but the pressure being more or less periodic but with an overall longitudinal gradient?

Yes. Infact if I am thinking properly, this would be analogous to the incompressible elasticity problem (with mixed Dirichlet/Neumann BCs) that I sketched above.
Thanks again for your reply. Sorry if this sounds too niche. However once I pick up some practice, I can maybe assist in some implementation too.
I will start this week by implementing a simple compressible hyperelasticity (like FEniCS demo), then hopefully incompressible hyperelasticity and eventually revisit this issue.

No, I don't think this is too niche.  I haven't needed it myself yet, but doubtless will sooner or later.
For identifying the periodic boundaries, my first thought would be to make use (in two dimensions) of Periodic Curve in Gmsh.  I think that part of a Gmsh MSH is read by meshio.read, but not yet by skfem.io.meshio.from_meshio.

Thanks! I was starting to take a look at implementing the hyperelasticity demo. The most natural examples to follow seem to be linear elasticity, nonlinear poisson and laplace with inhomogeneous bcs.
What would be the best place to look for functions like log, inv, det (like in UFL) when writing the weak forms? For instance the stress would have an expression that looks like the following in UFL:
def firstPKStress(u):
    F = Identity(len(u)) + grad(u)
    J = det(F)
    return mu * F - mu * inv(F).T + J * (J-1) * inv(F).T

I can see the helper functions eye and grad, which should help in defining F as eye(1, u.shape[0]) + grad(u), but how about the determinant (det) and inverse (inv) ?

For identifying the periodic boundaries, my first thought would be to make use (in two dimensions) of Periodic Curve in Gmsh. I think that part of a Gmsh MSH is read by meshio.read, but not yet by skfem.io.meshio.from_meshio.

I see; will take a look. This should save any post-processing like looking up matching vertices. Thanks!
For a simple 2D domain, generating a periodic mesh itself shouldn't be a problem as Christophe mentioned in a discussion sometime back. I have tested this with FEniCS/Abaqus. For generating the linear constraints however, I always did a simple lookup on the generated mesh (using numpy.where) to get the coordinates of matching points on opposite facets. For 3D meshes I have used Netgen which gives the corresponding node-pairs and moreover the list of linear constraints

I'll attempt to answer the question on writing forms in #439.

I once enforced a periodic solution like this:
    # periodic boundary condition
    Nleft=m.nodes_satisfying(lambda x,y:x==0.0)
    meps = 1e-12
    Nright=m.nodes_satisfying(lambda x,y:(x<2.0*np.pi+meps)*(x>2.0*np.pi-meps))
    Nbottom=m.nodes_satisfying(lambda x,y:y==0.0)
    Nleft=np.setdiff1d(Nleft,N1)
    Nright=np.setdiff1d(Nright,N1)
    Nelse=np.setdiff1d(I,Nleft)
    Nelse=np.setdiff1d(Nelse,Nright)

    I1=np.arange(Nelse.shape[0])
    I2=np.arange(Nleft.shape[0])+I1[-1]+1

    def periodify_matrix(Z):
        Zxx=Z[Nelse].T[Nelse].T
        Zx1=Z[Nelse].T[Nleft].T
        Z1x=Zx1.T
        Zx2=Z[Nelse].T[Nright].T
        Z2x=Zx2.T
        Z11=Z[Nleft].T[Nleft].T
        Z12=Z[Nleft].T[Nright].T
        Z21=Z12.T
        Z22=Z[Nright].T[Nright].T

        Zp1=spsp.hstack((Zxx,Zx1+Zx2))
        Zp2=spsp.hstack((Z1x+Z2x,Z11+Z12+Z21+Z22))
        return spsp.vstack((Zp1,Zp2)).tocsr()

    def periodify_vector(y):
        return np.hstack((y[Nelse],y[Nleft]+y[Nright]))

    def unpack_periodic(y):
        Y=np.zeros(f.shape[0])
        Y[Nelse]=y[I1]
        Y[Nleft]=y[I2]
        Y[Nright]=y[I2]

    Ap=periodify_matrix(A)
    x0=f*0.0
    x0[Nbottom]=other_BC
    fp=periodify_vector(f-A.dot(x0))

    up[0]=fsol.direct(Ap,fp,use_umfpack=True)
    # unpack periodic
    U=unpack_periodic(up[0])

It's from the numerical example of this paper: https://arxiv.org/abs/1711.04274. I think it works only for linear elements though. Idea is to eliminate the matching DOF's.
There are several other options:

Lagrange multipliers
weak enforcement (penalty or Nitsche's method)
modify mesh.t so that matching vertices have same index (haven't tested this so there might be caveats)


Add GS and TS

Signed-off-by: Chris Aniszczyk<EMAIL_ADDRESS>Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 250 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~10 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
item Fannie Mae (Adopter) has an invalid repo https://github.com/FannieMaeOpenSource
item Goldman Sachs (Adopter) has an invalid repo https://github.com/goldmansachs
item Two Sigma (Adopter) has an invalid repo https://github.com/twosigma


Error: 13 INTERNAL: Received RST_STREAM with error code 2

After upgrading to 0.23.0 (from 0.21.1) I see my subscriber services connecting to pubsub, receiving a few messages, then some time later experiencing the following error:
Error: 13 INTERNAL: Received RST_STREAM with error code 2
    at Object.exports.createStatusError (/app/node_modules/google-gax/node_modules/grpc/src/common.js:91:15)
    at ClientDuplexStream._emitStatusIfDone (/app/node_modules/google-gax/node_modules/grpc/src/client.js:233:26)
    at ClientDuplexStream._receiveStatus (/app/node_modules/google-gax/node_modules/grpc/src/client.js:211:8)
    at Object.onReceiveStatus (/app/node_modules/google-gax/node_modules/grpc/src/client_interceptors.js:1306:15)
    at InterceptingListener._callNext (/app/node_modules/google-gax/node_modules/grpc/src/client_interceptors.js:568:42)
    at InterceptingListener.onReceiveStatus (/app/node_modules/google-gax/node_modules/grpc/src/client_interceptors.js:618:8)
    at /app/node_modules/google-gax/node_modules/grpc/src/client_interceptors.js:1123:18

Here are some example logs and timestamps from service start until the error:
"Subscriptions ready" "2019-01-22T09:39:35.890Z"
"Handling message" "2019-01-22T09:39:42.701Z"
"Handling message" "2019-01-22T09:40:21.545Z"
"Handling message" "2019-01-22T09:40:22.276Z"
"Handling message" "2019-01-22T09:40:59.133Z"
"Handling message" "2019-01-22T09:52:01.244Z"
"Handling message" "2019-01-22T09:52:10.978Z"
"Handling message" "2019-01-22T09:52:11.854Z"
"Handling message" "2019-01-22T09:52:22.544Z"
"Handling message" "2019-01-22T10:03:55.492Z"
"Handling message" "2019-01-22T10:04:03.351Z"
"Handling message" "2019-01-22T10:05:45.136Z"
"Handling message" "2019-01-22T10:06:07.888Z"
"Handling message" "2019-01-22T10:06:17.679Z"
"Error emitted from subscription" "2019-01-22T10:09:42.682Z"

It seems to take about half an hour from a service starting to receiving the error, reasonably consistently I think.
See https://github.com/googleapis/google-cloud-node/issues/2619 for a similar, previous error.
Environment details

OS: Alpine Linux
Node.js version: 10.15.0
npm version: 6.6
@google-cloud/pubsub version: 0.23.0

Steps to reproduce

Use version 0.23.0
Wait half an hour?

Sorry I don't have any better repro steps.

That seems to have fixed it, thanks.

I still see the issue happening once a while with 2.5.0
https://github.com/googleapis/nodejs-firestore/issues/1023


Add unreducedEdits property

unreduced edits are edits without converting insertion/deletion pairs into .Move edits
unreduced edits is also good for manual changes in a UITableView or UICollectionView, since UITableView's only allow insertion, deletion, and update on rows.
converted tabs into spaces

I added a test case as well.

Thanks for showing your interest in the project, @chrisamanse.
If I understand you correctly, you want to be able to get the “unreduced” edits on a Changeset, i.e., edits where .Deletion and .Insertion of the same element are not converted into a .Move, the reason being that neither UITableView nor UICollectionView supposedly supports moving cells…
Two comments:

As a matter of fact, both view classes support moves. UITableView has moveRowAtIndexPath(_:toIndexPath:), UICollectionView also has moveRowAtIndexPath(_:toIndexPath:). You should use those.

The feature you request could be implemented in a much less intrusive manner.

I may include the feature (but again, are you sure you even need it?). And if I do, it could be done in the following way, defaulting to the current behaviour:
// Add reduceMoves argument to initializer:
public init(source origin: T, target destination: T, reduceMoves: Bool = true) {
	// ...
	self.edits = Changeset.editDistance(source: self.origin, target: self.destination, reduceMoves: reduceMoves)
}

// Add same reduceMoves argument to editDistance function:
public static func editDistance(source s: T, target t: T, reduceMoves: Bool = true) -> [Edit<T.Generator.Element>] {
	// ...

	// Check flag before reducing:
	if reduceMoves == true {
		return reducedEdits(d[m][n])
	} else {
		return d[m][n]
	}
}

Less that 10 lines of change, compared to the +266/-160 you propose.
If you want to contribute to this project (and probably others as well), I encourage you to study the style and conventions of the existing code. Go back and look at pull requests, browse the discussions. (Quickly done on this project.)
Your pull request changes pretty much every line in the file, converting tabs to spaces. That not only makes it very difficult to see the effect of your changes, it also discards the project’s convention to use tab indentation. There is even a recent commit (0eb375e85659fc118a52cd9657a333b2061b9153) which specifically enforces tabs. This project uses tabs.
Also, instead of making only one commit containing all of your changes at once, you should consider making several smaller commits, each containing an atomic change. So for example one commit would convert tabs to spaces, another contain the unreducedEdits property and function, and finally one adding the test. That would have made it easier for me understand your intentions, and I could potentially pick just some of your commits.
I will close this pull request.

Sorry about converting the tabs into spaces, I used a playground to test out the code, and simply copy and pasted the source file so it wasn't able to use the project's settings. I'll be mindful of my changes next time.
I've tried both the moveRowAtIndexPath(_:toIndexPath:) methods, and it worked. I got it confused with the UITableViewDataSource method tableView(_:moveRowAtIndexPath:toIndexPath:), where it is meant for observing movement caused by the user only. Thank you! And thanks for this useful framework.


Languages page translation

To be frank, im not sure in translating the title ... already writen somewhere else as Nyelvek, Fordítások sounds a little less weird. "Más nyelvek" maybe?


Handle function arguments parsing errors

This PR introduces error handling for function arguments parsing in the parseOutput function.
Changes:

A try-catch block has been added around the parsing of function_call.arguments with JSON.parse in the parseOutput function.
If an error occurs during parsing, an error message is thrown. This message includes the text that failed to parse and the original error message.

Rationale:
The previous code did not handle errors when parsing function_call.arguments. This could lead to unclear feedback to the user when incorrect argument formats were provided. This change addresses this issue and provides clearer feedback when an error occurs.
Note: If this contribution is included in a release, I'd be happy to be recognized for my work. My Twitter handle is @ywkim.

Thanks for opening this - not opposed, but how did you actually encounter this issue?

Thanks for opening this - not opposed, but how did you actually encounter this issue?

Hi @jacoblee93 ,
I ran into this issue when GPT was generating function arguments and hit the max token limit. The JSON creation was cut off midway, which led to a JSON parsing error. The error message was a bit vague and didn't really help me understand what was going wrong.
I figured if the error message could tell me what text it was trying to parse, it would've made debugging a lot quicker. That's why I proposed this change. Hope this makes sense!

Gotcha, thanks! Will get this merged tonight.


Implicit block call instead explicit block call. Faster code.

More info at https://github.com/JuanitoFatas/fast-ruby#proc--block

Thanks, merged!


Update GOV.UK Frontend to v3.12.0

Closes #1589

You can preview this change here:
Built with commit 1142f14991db9ab634327098a31afddcacac25f8
https://deploy-preview-1667--govuk-design-system-preview.netlify.app


asap7: add v28 .lib md5 hashes

Complementary to https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/577

We can then probably after the merge remove the v27 .lib files.


Improve parallel search performance

This PR improves the parallel search performance by

Selecting a proper number of threads
Shuffling the orders for search tree exploration for each thread for a better reuse of existing z3 invocations

Another modification included in this PR is solving memory leakage in search. I do not use smart pointers because the operators should be removed in a reversed order in the graph, which is not considered by the pointer reference manager. So currently I free the pointers in the destructor of graphs and disable the copy construction.


Fixed reading nested stats

Closes #1239 - thanks @ritchie46 for the report!

Thank you @jorgecarleitao!


[#26] Auto iterating results for MBEVENTS, MBQUERY, MBCUSTOMQUERY

Resolves #26
If a limit parameter of those functions is bigger than 30 or "all" which means infinity, API call will iterate with specific offset and limit and unify all multiple results automatically.
As bonus, cleaning up as well.

Regarding tests, I think they're not running successfully in CI?

If that is the case, in addition to fixing them, we should also make sure our CI test code causes the CI test to fail if the test fails.

Regarding tests, I think they're not running successfully in CI?

If that is the case, in addition to fixing them, we should also make sure our CI test code causes the CI test to fail if the test fails.

Good call! I added exit(1) when that case happen. Now I am seeing an error. I will cherry pick to other PRs as well and fix this issue soon.

@shoenseiwaso I fixed no tests running but this PR is come from a parent PR so can you review this first and other PRs?


When/if this website is approved by the lab, how do we redirect the MIMIC URL?

Once (/if) the website is finalized and approved by the group, what is the process for redirecting the MIMIC URL (https://mimic.physionet.org) to the new content?

I think we just change the entry in dns database, no?
K
On 9/2/15 12:56 AM, Tom Pollard wrote:

Once (/if) the website is finalized and approved by the group, what is the process for redirecting the MIMIC URL (https://mimic.physionet.org) to the new content?

Reply to this email directly or view it on GitHub:
https://github.com/MIT-LCP/mimic-website/issues/9

Depends where we host it.  We could either replace on the current server, or I think github allows for dns forwarding. Not sure if it has to be a root github.io repository or if it can be a separate repository.

We will be replacing the current website with the new website, so no redirect should be needed. Closing issue...


Dialin numbers for Teams conferences

I ended up on this page. I don't have phone numbers assigned to users, I'm just looking for a local dialin number. Teams consistently gives me a San Diego number, and I'm based in Los Angeles. I can't seem to fix this. This page doesn't tell me how, exactly.
This is not a service request issue - it's a documentation issue. There are several critical things this page doesn't address.

How to get additional toll numbers for dialin users. You don't cover all countries; Monaco, for example, isn't listed.

What costs are involved if you "place an order".

What happens if you have advanced conferencing, and not voice.
All I wanted was a dialin number that's local for some of my external users that are attending my conferences. This page implies it's for this purpose, but seems to be left out.

So fine, I figured I'd just request a toll free number for Monaco. I was worried I'd have to pay for it, but let's go for it. So then I get an error: "We can't do this because Communication Credits haven't been set up. "

This page doesn't address Communication Credits; if it's a prerequisite, it should be mentioned somewhere on this page.

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

ID: 7e37f13b-cb86-955e-147c-fa916da063f9
Version Independent ID: 4dd77188-abcb-fae2-3cf3-12b82b591bad
Content: Getting phone numbers for your users - Microsoft Teams
Content Source: Teams/getting-phone-numbers-for-your-users.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@jgwinner Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @jgwinner, thank you for your feedback.
This article is to assign phone numbers to users. If you want a phone number to call into a meeting, please check this article:
https://docs.microsoft.com/microsoftteams/set-up-audio-conferencing-in-teams

@jgwinner Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

How is the situation resolved? My numbers are still San Diego, this document still pops up on search, the linked documentation still doesn't tell you how to change numbers, and everything is "open a service request?"
Why is something that is so essential and should be simple, turn into a service request?
Do you guys get payment from long distance calls and are attempting to obfusticate this via tech support?

Hi @jgwinner,
My apologies if the article I sent you no helps. I think the article has all the steps to acquire services numbers for Audio Conferencing.
If the numbers you acquired does not service numbers you won´t be available to assign them for Audio Conferencing. I think you should remove them and acquire new numbers.
But I cannot help you here, please, you need to open a support ticket to get more assistance.


[NX-npm] requires project graph, which cannot be found

Feature id and version
ghcr.io/devcontainers-contrib/features/nx-npm:1
Devcontainer base Image
mcr.microsoft.com/devcontainers/dotnet:1.1.0-8.0
What happened?
In my combined dotnet and angular container, I wanted to try and start a nx serve action, which triggered this error
Relevant log output
ERROR (1) when writing
ENOENT: no such file or directory, open '/workspaces/DigitaalMeten/.nx/cache/project-graph.json~1f86e734'
at writeFileSync (node:fs:2352:20)
    at writeJsonFile (/workspaces/DigitaalMeten/node_modules/nx/src/utils/fileutils.js:55:28)
    at writeCache (/workspaces/DigitaalMeten/node_modules/nx/src/project-graph/nx-deps-cache.js:101:43)
    at buildProjectGraphUsingProjectFileMap (/workspaces/DigitaalMeten/node_modules/nx/src/project-graph/build-project-graph.js:65:40)
    at async buildProjectGraphWithoutDaemon (/workspaces/DigitaalMeten/node_modules/nx/src/project-graph/project-graph.js:72:27)
    at async createProjectGraphAsync (/workspaces/DigitaalMeten/node_modules/nx/src/project-graph/project-graph.js:121:25)
    at async Object.runOne (/workspaces/DigitaalMeten/node_modules/nx/src/command-line/run/run-one.js:21:26)

This project is not associated with the devcontainer project and appears to have been abandoned https://github.com/devcontainers-contrib/features/issues/628 . The official project has removed all devcontainer-contrib features from their listing. https://github.com/devcontainers/devcontainers.github.io/issues/451. There is a fork that is being worked on to update and maintain the features https://github.com/devcontainers-extra/features


[v2] stuck on isLoading when I setQueryData for initial load

Describe the bug
For my app, as the initial data was saved in local storage, I do not wish to make network call and show spinner. After upgrade to v2, I noticed that the app stuck at isLoading equal to true
https://codesandbox.io/s/react-query-2-setquerydata-18k47
To Reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/react-query-2-setquerydata-18k47
See the screen stuck at "Loading..." (you might need to click the browser refresh on the right panel)

Expected behavior
It should load the data in queryCache
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]: macOs catalina
Browser [e.g. chrome, safari]: chrome
Version [e.g. 22]: 83

Additional context
I tried another approach using initialData, but it is not working as expected as well: https://github.com/tannerlinsley/react-query/issues/650

I think this is what you're looking to do: https://codesandbox.io/s/react-query-2-setquerydata-5765w?file=/src/index.js


add android 12 support

targetSdkVersion 33
and
compileSdkVersion 33

I am facing the issue for the same. Not working on compileSdkVersion 33.

Got added in the newest release.


The terminal process failed to launch: path to shell executable "path"

After updating powershell via microsoft store, it shows me this error while I am trying to open terminal: The terminal process failed to launch: Path to shell executable "C:\Program Files\WindowsApps\Microsoft.PowerShell_<IP_ADDRESS>_x64__8wekyb3d8bbwe\pwsh.exe" does not exist. Now I can't access my terminal. Please quickly fix this. I have a deadline tomorrow. Before update, my default shell was powershell.

@mimamulhassankhan to unblock you, update the "terminal.integrated.shell.windows" to "C:\Program Files\WindowsApps\Microsoft.PowerShell_<IP_ADDRESS>_x64__8wekyb3d8bbwe\pwsh.exe"
I'll look into this

@mimamulhassankhan do you remember how you set this as your default shell? If you click the drop down for the terminals and select "Select Default Shell" that should discover the Store install and set your default shell to something like:
"terminal.integrated.shell.windows": "C:\\Users\\YOURUSERNAME\\AppData\\Local\\Microsoft\\WindowsApps\\Microsoft.PowerShell_8wekyb3d8bbwe\\pwsh.exe",

in the settings

/needsMoreInfo


Adding telemetry command extension to Agent

@TingluoHuang if there are not any major concerns, can merge the PR.

@TingluoHuang

Area/Command are Class variables like other extensions. https://github.com/Microsoft/vsts-agent/blob/master/src/Agent.Worker/TestResults/ResultsCommandExtension.cs#L29
I believe we invoke each instance per each command so this shouldn't be a problem. Correct me please.

LoadTelemetryInputs - do inline - it's nit and it's good to have well defined short methods

JSOn - I addressed it. we are no longer using ; separated data. We are using 'JSON' data.


chore: bdd-sbhg-1587321854 to 0.0.1

chore: Promote bdd-sbhg-1587321854 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


add exports to packages with multiple entry points

Fixes #87 and makes vocab more inline with Vanilla-Extract.

Hi 👋
Thanks for this PR!
The way vanilla-extract defines exports isn't ideal for compatibility with bundlers and node, so I'd prefer using import and require instead. Unfortunately, even this isn't perfect as vocab uses preconstruct for building which doesn't support outputting .cjs and .mjs file, though we're working on a solution for this that should be ready quite soon, so for now this will do.

Thank you for your explanation. I also think that the workaround should be merged for now, but of course a proper solution is much appreciated. I appreciate your affords.

Sorry for the delay. Happy to approve, though I'm not sure what the lockfile changes are for. Would probably be best to remove them.

If have removed them.


Collection partion method not returning correct generic types

Laravel Version
11.29.0
PHP Version
8.3.12
Database Driver & Version
Sqlite 3.43.2
Description
The partition() method of the Illuminate\Database\Eloquent\Collection, as defined in the Illuminate\Support\Traits\EnumeratesValues class possibly does not have the correct generic return types.
https://github.com/laravel/framework/blob/11.x/src/Illuminate/Collections/Traits/EnumeratesValues.php#L501
The issue goes away when I disable and remove the cache of the LaravelIdea plugin. However, I'm not sure if it's an issue with Laravel's definition of the generics, or with how the plugin is parsing the generics.  If it's deemed that Laravel is correct, I'll go open this issue in the plugin repo.
Steps To Reproduce
Given the following user table:

id
sex
birthday

1
male
2002-01-12

2
female
1991-05-05

3
male
2010-12-25

and the following code:
//get users
$users = User::all();

//partition by sex
[$men, $women] = $users->partition(fn ($user) => $user->sex === 'male');

//group men by age
$menGroupedByAge = $men->groupBy(fn ($item) => $item->birthday->age);

dd($users, $men, $women, $menGroupedByAge);

all 4 dumped variables will be of type Illuminate\Database\Eloquent\Collection. However, the IDE is currently thinking $men, $women, and $menGroupedByAge are all of type App\Models\User.
Both our Collection and User classes also have a groupBy() method available to them, but with different signatures. Because the IDE believes $men is a User, it's also warning that it is unacceptable to pass a closure to groupBy().

@crynobone am I missing something? It looks like the PR is still open.

now it's merged, all good!


Map Point Tool is always enabled (ArcGIS Pro)

with build 1513 the map point tool is always enabled in coordinate conversion (didn't seem to be happening in distance and direction). The tool is always enabled, but the button does not highlight as though it is enabled. Even though the tool appears disabled, it is always enabled and you need to select a different tool, such as the explore tool to disable it.

Spoke with @dfoll and issue will be looked at immediately.

Ready to verify, this should be tested with the combined/toolbar & single add-in - to make sure both work as expected

looks like it works as expected now with build 1526. i click the map point tool and it shows as blue/enabled and i can click points. When i click it again, it goes back to gray/disabled, and no longer makes points.

reopening, because we realized that this was giving us some unforseen behavior, will close again when we see that appropriate issues have been logged to cover this

@dfoll Changes were added to address this issue in #388


[SAPRK-15220][UI] add hyperlink to running application and completed application

What changes were proposed in this pull request?
Add hyperlink to "running application" and "completed application", so user can jump to application table directly, In my environment, I set up 1000+ works and it's painful to scroll down to skip worker list.
How was this patch tested?
manual tested
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@carsonwang

Test build #58126 has started for PR 12997 at commit c4cbc29.

Test build #58126 has finished for PR 12997 at commit c4cbc29.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58126/
Test PASSed.

Good idea. Merging into master 2.0.


Improve Property Resolution

Codecov Report
Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (db9fcf3) 80.12% compared to head (d18dd96) 77.27%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #54      +/-   ##
============================================
- Coverage     80.12%   77.27%   -2.85%
+ Complexity       69       65       -4
============================================
  Files             9        9
  Lines           327      330       +3
  Branches         56       57       +1
============================================
- Hits            262      255       -7
- Misses           39       45       +6
- Partials         26       30       +4

Files
Coverage Δ

...adle/versioncatalogs/service/DependencyResolver.kt
0.00% <ø> (-92.31%)
:arrow_down:

...versioncatalogs/service/LocalDependencyResolver.kt
100.00% <100.00%> (+19.04%)
:arrow_up:

...catalogs/VersionCatalogGeneratorPluginExtension.kt
76.92% <0.00%> (ø)

...kotlin/dev/aga/gradle/versioncatalogs/Generator.kt
79.03% <75.00%> (-0.65%)
:arrow_down:

...ersioncatalogs/service/GradleDependencyResolver.kt
82.75% <77.50%> (-8.16%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Added vuejs environment

Added Vuejs environment + nginx config in https://github.com/wardenenv/images/pull/2

This looks like a simple static not-even-PHP environment. I think a different approach for this that's not vue-specific (since no part of it seems to be) would be in order.
I've always thought Warden should have a generic php environment, and perhaps a generic static environment would be useful too (that does nothing but stand-up Nginx and mount into it)

@navarr Agree added simple php environment https://github.com/wardenenv/warden/pull/590 will add the static later.


prettier workflow: Use default GITHUB_TOKEN

Switch the prettier workflow to use the default GITHUB_TOKEN (automatically available to all workflows) rather than a repository-specific token.
This allows the script to be copied as-is and reused in other Sourcegraph repos. With the current version, that's not possible because it relies on a repository-level secret, which means that you'd have to add a secret with the same name to every repository where you want to reuse this.

We need to use a different token, see context here: https://github.com/sourcegraph/handbook/pull/71


do not bind rendering a page to controllers

So for form processors it could be nice to show the original form with the errors.

Since v1.0.0-M1 controllers are merely coordinators with very few logic. They do not handle HTML, just "HTTP Content", so it is completely flexible.


Dead animals are still punchable

When animal are dead you are still able to punch then. Until you stop punching them while already dead they will not be trigerred to disappear from world.
P.S.: Maybe this bug should have been opened in creatura, I'm not sure

Fixed in latest commit.


refactor(ssr): use ssrPrefetch

Blocked by: https://github.com/vuejs/vue/pull/9017

Vue is currently at version 2.6.10. Is this working yet?

Yes


Add an option to not fork when creating a pull request

This creates a branch in the origin repository instead.

Absolutely not tested, please check it out.

Pull-request updated, HEAD is now 25c21dcc99540e2f4022d6d5127e65a57514ded8

@therve I've fixed the pep8 error in master, if you can just rebase that should be fine now.

Pull-request updated, HEAD is now 83dc74e6dfbd5b3d5a2535186e531bbaa3f57717

Pull-request updated, HEAD is now ba21b286bb75a879f07122bf951bdfc0970a82cb


Typo in description

Typo in https://docs.microsoft.com/en-us/powershell/module/hyper-v/add-vmremotefx3dvideoadapter?view=win10-ps#description
it should be RemoteFX, not RemovteFX

@officedocsbot assign @e0i

@imamkhaira Thank you for your feedback. I was not able to identify the typo in the document. Either it was already fixed or I'm looking at the wrong doc. Can you check and confirm if the URL for the doc is correct https://docs.microsoft.com/en-us/powershell/module/hyper-v/add-vmremotefx3dvideoadapter?view=win10-ps#description ?

Hello, @imamkhaira
It seem the issue had already been addressed with the following commit,
[e840853 ] typo: Removte -> Remote
Changes can also be observed live.
We are thus closing this issue since it seems to be resolved.
Thank you for taking the time to provide feedback.

@officedocsbot close