[MRTK3] Poke scrolling - WIP

Overview
Introduces poke scrolling to ScrollRects. Very WIP.
Fixes https://github.com/microsoft/MixedRealityToolkit-Unity/issues/10736

[ ] Ray scrolling is... no bueno, if not using the native UGUI raycast route. XRI doesn't bubble up, sooooo.... hard challenge
[ ] Gazepinch scrolling has the same issue as above
[ ] Thinking about a good way to cancel button press visuals during a scroll (easier to cancel the events, not so easy to cancel the visuals)

@SimonDarksideJ, here's the experimental scrolling work I've been doing. It's focused on finger scrolling first and foremost, mostly because doing bubble-up ray scrolling is extra tricky. We'll track work on scrolling through this PR, so, stay tuned :)

Hi @Zee2
I'm sorry if this isn't the best spot to post about this.  I am using your awesome VirtualizedScrollRectList (borrowed from the MRTKDeveTemplate project and the VirtualizedScrollRectList example scene) in my project, and I really like it! It's really slick. Although I barely understand how it works. :)  I wish you would make a youtube video explaining how it works.
Anyhow, a capability I wanted to add to it was that when the user clicked on one of the buttons, the Icon would show to indicate that was the button the user had selected.  And obviously the icon would disappear on any other button that was previously selected.
However, this seems to be something I can't figure out. When I enable the Icon on the button, because the button is shared in the pool, when the object gets reassigned by going out and coming back into view as another button, the Icon still shows up, but it's not the right button anymore. So, it appears like it's getting enabled on multiple buttons, but I understand that it's really the same button.
I've spent several hours using different methods trying to make this work. First with an event trigger that all the buttons listen too, and then by having the Container test it's children to compare with the last button clicked.  No dice.
Do you have any suggestions for how to make this work?

CC @maluoi as well, who worked on VirtualizedScrollRectList, if I remember correctly

I apologize for the misattribution. @keveleigh As it turns out, I just figured out how to make this work using a simple private string. Not sure why I was struggling so much with it yesterday.  Just one of those things.  Let me know if you want, I can delete that post.

@jasonhbartlett It's all good! I just wanted to ping as many people as I thought might be able to help!
Glad to hear you were able to figure it out 🙂 no need to delete the post: I find it better to leave things like that up in case somebody else does a search or something in the future with the same (or a similar) issue!

Hello :), any news on this awesome feature ?

This was completed with the following

https://github.com/microsoft/MixedRealityToolkit-Unity/pull/11642


Cannot find module './build/Release/nodejieba.node'

nodejieba 1.0.3 node 6.2.1 CLI version: 2.3.0 GitBook version: 3.1.1

When using gitbook serve
PluginError: Error with plugin "search-jieba": Cannot find module './build/Release/nodejieba.node'

I believe it's a gitbook's bug. After you run gitbook install, please also run npm rebuild in the book dir. In this way, the native code of nodejieba will be compiled.

New bugs after npm rebuild
(node:48114) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.

><EMAIL_ADDRESS>install /Users/tianshuo/sources/docs/node_modules/nodejieba
> node-gyp rebuild

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

  CXX(target) Release/obj.target/nodejieba/lib/index.o
  CXX(target) Release/obj.target/nodejieba/lib/nodejieba.o
  SOLINK_MODULE(target) Release/nodejieba.node

><EMAIL_ADDRESS>install /Users/tianshuo/sources/docs/node_modules/gitbook-plugin-search-pro/node_modules/nodejieba
> node-gyp rebuild

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

  CXX(target) Release/obj.target/nodejieba/lib/index.o
In file included from ../lib/index.cpp:1:
In file included from ../lib/nodejieba.h:4:
In file included from ../lib/utils.h:6:
../node_modules/nan/nan.h:602:20: error: no type named 'GCEpilogueCallback' in
      'v8::Isolate'
      v8::Isolate::GCEpilogueCallback callback
      ~~~~~~~~~~~~~^
../node_modules/nan/nan.h:608:20: error: no type named 'GCEpilogueCallback' in
      'v8::Isolate'
      v8::Isolate::GCEpilogueCallback callback) {
      ~~~~~~~~~~~~~^
../node_modules/nan/nan.h:613:20: error: no type named 'GCPrologueCallback' in
      'v8::Isolate'
      v8::Isolate::GCPrologueCallback callback
      ~~~~~~~~~~~~~^
../node_modules/nan/nan.h:619:20: error: no type named 'GCPrologueCallback' in
      'v8::Isolate'
      v8::Isolate::GCPrologueCallback callback) {
      ~~~~~~~~~~~~~^
4 errors generated.
make: *** [Release/obj.target/nodejieba/lib/index.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:270:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)
gyp ERR! System Darwin 15.2.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/tianshuo/sources/docs/node_modules/gitbook-plugin-search-pro/node_modules/nodejieba
gyp ERR! node -v v6.2.1
gyp ERR! node-gyp -v v3.0.3
gyp ERR! not ok

npm ERR! Darwin 15.2.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "rebuild"
npm ERR! node v6.2.1
npm ERR! npm  v3.3.12
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the nodejieba package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls nodejieba
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/tianshuo/sources/docs/npm-debug.log

Looks like you have mac. you need to install XCode to compile the module.

Looks like this bug: https://github.com/nodejs/node-gyp/issues/569


Follow system theme option on linux (freedesktop color scheme)

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
The follow system theme option doesn't work on Linux, Bruno stays always in light mode.
Most Linux distros follow this preference: https://gitlab.gnome.org/GNOME/Initiatives/-/wikis/Dark-Style-Preference
.bru file to reproduce the bug
No response
Screenshots/Live demo link
System in light mode:

System in dark mode:

Bruno in dark mode:

Interresting Bruno currently uses the Browser API to determine the correct theme. This works well on windows not sure why it does not on Linux.
But electron offer an API for this: https://www.electronjs.org/docs/latest/api/native-theme maybe it works better with that.

System theme detection was fixed in Chromium 114 iirc, so it might be required to update electron version, too.
Another example of an electron app recently fixing theme detection is VS Code.

This got fixed with #2668, since electron got updated to v 25+, which has theme changing fixed 🎉


DLS-8167 added in reg pending page and minor rosm reg writes change a…

…s was not writing line 1 properly
https://github.com/hmrc/soft-drinks-industry-levy-ui-tests/pull/103

@JayenKerai I have fixed the cosmetic issue.


Add yuanchen8911 as approver

What type of PR is this?

What this PR does / why we need it:
This PR makes a request for adding @yuanchen8911 as an approver.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/assign @wzshiming

/cc @Huang-Wei

/retest

/lgtm
/assign @Huang-Wei

@yuanchen8911 thanks for the contribution! Would you mind starting with reviewer? Or, do you mean a reviewer but just a typo?

@yuanchen8911 thanks for the contribution! Would you mind starting with reviewer? Or, do you mean a reviewer but just a typo?

Whatever makes sense to @wzshiming and you. I just wanted to help with the project maintenance and reviews as well.

Changed the request to reviewer from approver.

/approve
/lgtm


Shader errors now opens a message box

Any errors in a shader now shows in a message box in addition to the program console.
Since the message box blocks until it has been closed the assert on shader load has been changed to a return statement.

Nice.


consistent seed not generating reproducible responses

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
When I call gpt-4-turbo streaming with a constant seed (1234), There is still variations in the output, espeically as the prompt and answer get longer. Is there any way to avoid this or somehow force it to be even more deterministic?
To Reproduce
code below, problem occurs specifically with in-debth thought provoking prompts.
Code snippets
client = AsyncOpenAI(timeout=30)

async def send_openai_request(prompt, engine="gpt-4-1106-preview"):
    try:
        stream = await client.chat.completions.create(
            messages=[{"role": "user", "content": prompt}],
            stream=True,
            model=engine,
            seed=1234,
            temperature=0.0001,
        )
        collected_messages = []

        async for part in stream:
            print(part.choices[0].delta.content or "")
            collected_messages.append(part.choices[0].delta.content or "")

        all_messages = ''.join(collected_messages)
        return all_messages

### OS

macOS

### Python version

Python 3.10.12

### Library version

openai 1.3.3

Thanks for reporting!
This sounds like an issue with the underlying OpenAI API and not the Python library, so I'm going to go ahead and close this issue.
Would you mind reposting at community.openai.com?


fix typo

Summary
Fixes minor typo in getting started doc.
Short checklist

[x] Did you run bundle exec rubocop -a to fix linter issues? – Does not apply
[x] If relevant, did you create a test? – Does not apply
[x] Did you confirm that the RSpec tests pass? – Does not apply

Nice catch. Thanks @HLFH ! 🙌


Extract alias logic into its own implementation.

What
Extract alias logic into its own implementation.
Why
So we can easily reuse it in other commands (e.g. contract bindings typescript will also load the contract id alias).
Known limitations
N/A
Storing alias locally
$ soroban contract deploy --wasm ./target/wasm32-unknown-unknown/release/hello_world.wasm --network standalone --source alice --alias alice
CDMCQHHE356XOGBXXUTZ3AVVARRTRYKHFVYAC6UYWM7RYPKSYV2FUPDS

$ cat .soroban/contract-ids/alice.json | jq
{
  "ids": {
    "Standalone Network ; February 2017": "CDMCQHHE356XOGBXXUTZ3AVVARRTRYKHFVYAC6UYWM7RYPKSYV2FUPDS"
  }
}

$ soroban contract invoke --network standalone --source alice --id alice -- hello --to fnando
["Hello","fnando"]

Storing alias globally
$ soroban contract deploy --wasm ./target/wasm32-unknown-unknown/release/hello_world.wasm --network standalone --source alice --alias alice --global
CDNSYIZ7IUPFGP72DATK4DYOU5ZKPUBBHBTAIFUBTCEDK3RR5WRQOP7F

$ cat ~/.config/soroban/contract-ids/alice.json | jq
{
  "ids": {
    "Standalone Network ; February 2017": "CDNSYIZ7IUPFGP72DATK4DYOU5ZKPUBBHBTAIFUBTCEDK3RR5WRQOP7F"
  }
}

$ soroban contract invoke --network standalone --source alice --id alice --global -- hello --to fnando
["Hello","fnando"]

Looks good so far. One suggestion is to move to it's own file under config. Then you can name the types Error and Data. Then externally, it would be alias::Error and alias::Data.

Looking good! Just one more suggestion of moving the logic to config this way we later if it becomes it's own crate we can use it to load_contract_ids from aliases to import them in smart contracts.

@leighmcculloch @fnando One more thought designwise, is it possibly  confusing to users that --contract-id could be an --contract-alias and move this into its own Arg? This commands that need a contract_id
 Cmd {
    #[flatten]
    contract_id: contract_id::Arg,
    /// ...
    }

and it could be
struct Arg {
    #[arg(long = "contract-id", required_unless_present = "contract-alias")]
    pub contract_id: Option<String>,
    #[arg(long = "contract-alias", conflicts_with = "contract-id")]
    pub contract_alias: Option<String>,
}

impl Arg {
    pub fn resolve(&self, config: &config::Args) -> Result<Strkey::Contract, Error>
}

Then for all commands that use the contract id it would be self.contract_id.resolve(&config)?.
I'm totally fine either way just always am a fan of putting shared logic into flattened Args and not overloading args.

@willemneal as for --contract-alias, I think we shouldn't add it, just because is yet another thing to keep in mind when using the CLI. But one I think we should do is improving the documentation for --contract-id mentioning the support for aliasing.
As soon as the alias work is done, I plan to update the docs with a section for contract id aliasing, which should also help with adoption.

@willemneal can i get a 👍 ?


Browser cross-site scripting filter misconfiguration

Cross-site scripting (XSS) filters in browsers check if the URL contains possible harmful XSS payloads and if they are reflected in the response page. If  such a condition is recognized, the injected code is changed in a way, that it is not executed anymore to prevent a succesful XSS attack. The downside of these filters         is, that the browser has no possibility to distinguish between code fragments which were reflected by a vulnerable web application in an XSS attack and these which are         already present on the page. In the past, these filters were used by attackers to deactivate JavaScript code on the attacked web page. Sometimes the XSS filters itself are vulnerable in a way, that web applications which were protected properly against XSS attacks became vulnerable under certain conditions. It is considered as better practice to instruct the browser XSS filter to never render the web page if an XSS attack is detected.
No X-XSS-Protection header was set in the response. This means that the browser uses default behavior that detection of a cross-site scripting attack never prevents rendering.
All 8 occurrences of this finding were on a Error page (in this case, 301).
Need to add HTTP Security Headers to Error Pages.
Aha! Link: https://chef.aha.io/epics/SH-E-337

Hit http://a2-dev.test and check the network tab for 301

@kalroy
Code changes are tested. "x-xss-protection" added in response header. FYI - Response header of http://localhost, it redirects to https://a2-dev.test.

GET / HTTP/1.1
Host: localhost
User-Agent: curl/7.68.0
Accept: /

Mark bundle as not supporting multiuse
< HTTP/1.1 301 Moved Permanently
< Server: nginx
< Date: Wed, 07 Jul 2021 12:44:44 GMT
< Content-Type: text/html
< Content-Length: 162
< Connection: keep-alive
< Location: https://a2-dev.test:443/
< Strict-Transport-Security: max-age=63072000; includeSubDomains
< x-xss-protection: 1; mode=block
< X-Content-Type-Options: nosniff

Added x-xss-protection in Response header #5286


fetch rewards on page load

#1572
Removes the componentDidReceiveProps hook from the <RewardsPage> component, which was firing repeatedly.
Passes in claimed rewards through the container to differentiate between "none available" and "you claimed them all".

@daovist won't be able to test this in our live environment until next week's LBRY cast. Other option is to run a local internal-api server. I know @seanyesmunt has one setup...I haven't got around to trying it on Linux yet (failed on Windows in the past).

Thanks for the fix @daovist, now it fires.
There is a slight loading delay, which isn't a big deal. Are the rewards cachable similar to how we handle some of the other pages?

@tzarebczan yes, I want to rewrite the rewards page (the whole component is a mess rn) to look and function like the new channel page in terms of caching and loading but I'm thinking we should iron out the details there before using it as a model for other pages. This may include re-fetching new data while a user stays on a page. @seanyesmunt

I think this is good for now, but I agree it would be nice to add styling similar to how the channel pages work now.


Create a Standalone/Maya version of the shader

Right now the shader is only for 3ds Max, but I should create a version for V-Ray Standalone/V-Ray for Maya.

Just started on this also ... I'm new to Vray plugins so just small steps for me.   If you need someone to test it out let me know.


Multi Node Distributing - RuntimeError: Connection reset by peer

Hi,
Am trying to use multi node in AWS Kubernetes (EKS). Its working in Single Node (current machine without the hostfile) but when i try to connect different nodes, its not)
hostfile
gpu2 slots=1
ssh gpu2 date is returning the date.
In Terminal when i execute the following commands, its throwing the error given below. "RuntimeError: Connection reset by peer"
deepspeed --hostfile=hostfile cifar10_deepspeed.py --deepspeed_config ds_config.json

Hi @jeffra
ds_ssh hostname is returning worker nodes correctly. However when i execute deepspeed command it's throwing "Connection reset by peer"
I have the hostfile both in /job/hostfile as well in present working directory.
Full command is "deepspeed --hostfile=hostfile cifar10_deepspeed.py --deepspeed_config ds_config.json"
Here's my ssh config file at /root/.ssh/config
Host gpu2
Hostname xx.xxx.x.xxx
user ec2-user
IdentityFile /app/ri.pem
Port 22

Hi @jeffra
Do you any documentation for how to setup deepspeed in Kubernetes? Also, It's good to have setup documentation for kubeflow integration.

Hi @jeffra
Would love to see some information on running deepspeed in k8s as the others mentioned. Thank you!

Hi, possibly if anybody still searching something I can describe how I managed to setup DeepSpeed on kubeflow (EKS backend). It won't be full tutorial but mayby at least some people will save time.
So generally speaking you need something for setting up pods and connection between them. In most tutorials they guide to use mpi-operator but when you've never worked with MPI... I would say there are easier options you can choose. Especially PytorchJob. I would advice to create python function that will dynamically create this ( for names, or number of nodes and gpus sake).
Remember that you need to run your script both in master pod and all worker pods with different node. This operator will  create sufficient env vars for you, $RANK, $MASTER_PORT, $MASTER_ADDR etc.
Than you will probably want to use this operator as a pipeline component. You can use ResourceOp to achieve this.
Now, how to launch it. I've menaged to do it with torchrun and script looked like this:
sleep 20 \
        && export NCCL_SOCKET_IFNAME=eth0 \
        && torchrun \
            --nnodes={num_of_nodes} \
            --nproc-per-node={num_of_gpu} \
            --node_rank=$RANK \
            --master_addr=$MASTER_ADDR \
            --master_port=$MASTER_PORT \
            -m script

Sleep was mainly for debugging issues. Setting interface eth0 was because it sometimes was detecting wrong so I've set it manually.
Last part was connected with adjusting code. I've need to change deepspeed.init_distributed to
torch.distributed.init_process_group(
        backend='nccl',
        init_method=f'tcp://{os.getenv("MASTER_ADDR")}:{os.getenv("MASTER_PORT")}',
        world_size=os.getenv("WORLD_SIZE"),
        rank=os.getenv("RANK"))

    )
    torch.distributed.barrier()

I've couldn't menage to do it with either gloo backend or env init method ot without torch.distributed.barrier but I am not sure because it was kinda messy process of guessing for me so maybe it is not needed.
And what's most important. You don't need to change your Dockerfile at all!
So that's it. That worked for me. I hope it will help someone.


docker documentation enhancement

The documentation here http://nsq.io/deployment/docker.html needs a couple of enhancements.
1- nsqlookupd needs the -broadcast-address= otherwise nsqd will complain
2-the nsqd arguments should have 1 dash instead of 2
3-should add documentation on running nsqadmin also

these were just updated


Fixed parameter types

but default value is still null
On Wed, Sep 30, 2020, 16:14 Michał Bundyra<EMAIL_ADDRESS>wrote:

@michalbundyra commented on this pull request.
In src/AbstractValidator.php
https://github.com/laminas/laminas-validator/pull/72#discussion_r497498978
:

@@ -321,8 +321,8 @@ protected function createMessage($messageKey, $value)
}

 /**

* @param  string $messageKey

* @param  string $value      OPTIONAL

* @param  string              $messageKey

* @param  string|array|object $value      OPTIONAL

It's fine, see implementation:
if ($value === null) {
$value = $this->value;
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/laminas/laminas-validator/pull/72#discussion_r497498978,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWH7VCYUN3RBDZZ3GM6WDSIMVMLANCNFSM4R7BZPGA
.

@svycka Thanks for this PR.
To make travis pass, you can either temporarily add COMPOSER_ROOT_VERSION=2.14.0 to travis.yml or "version": "2.14.0" to composer.json in a single commit.
After this, travis should startup and execute tests.
When anything is green, we can add this to the next release 👍

@boesing I am A bit lost do you want something changed? I got an email but I guess you removed the comment.

Nah, everything is fine. Thats why I deleted my comment.
Just have to sort some things out regarding this project and merging stuff as we have commits in here which should already be part of the branch but isn't...


Uprev google/certificate-transparency to include Solaris support

This is a dependency for #23397.
Signed-off-by: Amit Krishnan<EMAIL_ADDRESS>

All green, merging

thanks @thaJeztah @vdemeester !


GemFire Blog post - TG4K8S Creating Basic GemFire client

Adds content/data/gemfire/blog/tg4k8s-basic-client/index.md, a tutorial for creating a Java-based GemFire client that connects to a TLS-secured GemFire for Kubernetes cluster

Hello @danielramirez0 , this PR is ready for your review. I've built the preview and the new content looks correct, please let me know if there are additional things I should do


Fix discriminator column defaults

DiscriminatorColumn mapping generated by all 3 metadata drivers (annot, yml, xml) was lacking length and type when only the name is specified in the mapping. Even though this results in a columnDef that generates a string column of length 255, the mapping for the discriminator column should have those fields set properly, and should be consistent with the behavior when no discriminator column mapping is specified at all.
Adjusted docs to properly reflect that the @DiscriminatorColumn annotation is optional.
Closes: #1601

The change is quite subtle, IMO: marked for 2.6.0 only therefore.

:+1:


[Feature Request]: Add support for DataScienceCluster CR in dashboard.

Feature description
Related to #1475
Currently dashboard uses KfDef CRs in the code base to enable Application CR. As we are moving away from KfDef CRDs, as part of migration , add support for the new operator CR DataScienceCluster.
Describe alternatives you've considered
Possible Solutions

Update Dashboard code to watch for DataScienceCluster CR.
Allow operator to update ODHApplication CR when a component is enabled/disabled

Anything else?
No response

We will migrate away from the KfDefApplications and thus don't need to follow through with the CR reading.


Training in Colab always gives an error of "FailedPreconditionError"

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Google Colab

Started training in Colab but unfortunately, I am always having this error:
WARNING:tensorflow:Unresolved object in checkpoint: (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.moving_mean W1202 07:14:44.360409<PHONE_NUMBER>98304 util.py:150] Unresolved object in checkpoint: (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.moving_mean WARNING:tensorflow:Unresolved object in checkpoint: (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.moving_variance W1202 07:14:44.360842<PHONE_NUMBER>98304 util.py:150] Unresolved object in checkpoint: (root).model._box_predictor._base_tower_layers_for_heads.class_predictions_with_background.4.10.moving_variance WARNING:tensorflow:A checkpoint was restored (e.g. tf.train.Checkpoint.restore or tf.keras.Model.load_weights) but not all checkpointed values were used. See above for specific issues. Use expect_partial() on the load status object, e.g. tf.train.Checkpoint.restore(...).expect_partial(), to silence these warnings, or use assert_consumed() to make the check explicit. See https://www.tensorflow.org/guide/checkpoint#loading_mechanics for details. W1202 07:14:44.361112<PHONE_NUMBER>98304 util.py:158] A checkpoint was restored (e.g. tf.train.Checkpoint.restore or tf.keras.Model.load_weights) but not all checkpointed values were used. See above for specific issues. Use expect_partial() on the load status object, e.g. tf.train.Checkpoint.restore(...).expect_partial(), to silence these warnings, or use assert_consumed() to make the check explicit. See https://www.tensorflow.org/guide/checkpoint#loading_mechanics for details. WARNING:tensorflow:FailedPreconditionError: models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint; Is a directory W1202 07:14:44.364649<PHONE_NUMBER>98304 checkpoint_management.py:295] FailedPreconditionError: models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint; Is a directory WARNING:tensorflow:models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint: Checkpoint ignored W1202 07:14:44.364827<PHONE_NUMBER>98304 checkpoint_management.py:296] models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint: Checkpoint ignored WARNING:tensorflow:FailedPreconditionError: models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint; Is a directory W1202 07:14:44.365584<PHONE_NUMBER>98304 checkpoint_management.py:295] FailedPreconditionError: models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint; Is a directory WARNING:tensorflow:models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint: Checkpoint ignored W1202 07:14:44.365750<PHONE_NUMBER>98304 checkpoint_management.py:296] models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint: Checkpoint ignored Traceback (most recent call last):   File "model_main_tf2.py", line 113, in <module>     tf.compat.v1.app.run()   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/app.py", line 40, in run     _run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)   File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 300, in run     _run_main(main, args)   File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 251, in _run_main     sys.exit(main(argv))   File "model_main_tf2.py", line 110, in main     record_summaries=FLAGS.record_summaries)   File "/usr/local/lib/python3.6/dist-packages/object_detection-0.1-py3.6.egg/object_detection/model_lib_v2.py", line 627, in train_loop     manager.save()   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/checkpoint_management.py", line 819, in save     self._record_state()   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/checkpoint_management.py", line 728, in _record_state     save_relative_paths=True)   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/checkpoint_management.py", line 248, in update_checkpoint_state_internal     text_format.MessageToString(ckpt))   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 570, in atomic_write_string_to_file     rename(temp_pathname, filename, overwrite)   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 529, in rename     rename_v2(oldname, newname, overwrite)   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 546, in rename_v2     compat.as_bytes(src), compat.as_bytes(dst), overwrite) tensorflow.python.framework.errors_impl.FailedPreconditionError: models/ssd_resnet50_v1_fpn_640x640_coco17_tpu-8/checkpoint.tmp1e96a55bc7b14447b94ca71713ebdea9; Is a directory
I don't have any idea why it keeps saying the checkpoint.tmp is a directory?
Appreciate your help.
This is my pipeline.config

@androuino
Request you to flll issue template.
Please, share colab link or simple standalone code with supporting files to reproduce the issue in our environment. It helps us in localizing the issue faster. Thanks!

Hi @ravikyram, thank you for your response. This is the colab link.
Please use this link for a simple set of data for training.
And this is the link I am following for the tutorial.
Thank you and have a nice day.

@androuino
Please, grant me the access for the colab link. Thanks!

Hi @ravikyram, thanks for your fast reply. Please kindly check if you already have access to my colab notebook.
Thank you.
Have a great day.

@androuino
This issue is more suitable for TensorFlow Models repo. Please post it on Tensorflow Models repo from here. Thanks!

@ravikyram, oh ok. Thanks for routing me to models repo.

@androuino
Please, close this thread here and we can track the issue in Tensorflow Models repo. Thanks!

@androuino
Please, close this thread here and we can track the issue in Tensorflow Models repo. Thanks!


add Ptr* functions for package gconv

Codecov Report
Base: 76.57% // Head: 76.57% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (fce4a2f) compared to base (2598745).
Patch coverage: 94.23% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2206      +/-   ##
==========================================
- Coverage   76.57%   76.57%   -0.01%
==========================================
  Files         580      581       +1
  Lines       48272    48310      +38
==========================================
+ Hits        36964    36992      +28
- Misses       9291     9299       +8
- Partials     2017     2019       +2

Flag
Coverage Δ

go-1.15-386
76.58% <94.23%> (-0.02%)
:arrow_down:

go-1.15-amd64
76.55% <94.23%> (-0.03%)
:arrow_down:

go-1.16-386
76.57% <94.23%> (-0.03%)
:arrow_down:

go-1.16-amd64
76.59% <94.23%> (-0.01%)
:arrow_down:

go-1.17-386
76.63% <94.23%> (+0.03%)
:arrow_up:

go-1.17-amd64
76.62% <94.23%> (+<0.01%)
:arrow_up:

go-1.18-386
76.52% <94.23%> (+0.01%)
:arrow_up:

go-1.18-amd64
76.54% <94.23%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

database/gdb/gdb_driver_wrapper_db.go
92.50% <50.00%> (-7.50%)
:arrow_down:

database/gdb/gdb.go
64.17% <100.00%> (-1.82%)
:arrow_down:

util/gconv/gconv.go
92.30% <100.00%> (ø)

util/gconv/gconv_ptr.go
100.00% <100.00%> (ø)

os/gfsnotify/gfsnotify_watcher.go
81.51% <0.00%> (-2.53%)
:arrow_down:

os/gfsnotify/gfsnotify_watcher_loop.go
83.19% <0.00%> (-1.69%)
:arrow_down:

database/gdb/gdb_core_structure.go
68.85% <0.00%> (-0.88%)
:arrow_down:

os/gcmd/gcmd_parser.go
92.19% <0.00%> (-0.71%)
:arrow_down:

contrib/drivers/mysql/mysql.go
69.38% <0.00%> (ø)

... and 1 more

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

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


Premake Fixes

Removed opengl32.lib link duplicate
Added ImGui include directory
Arranged premake5.lua spaces

Suggestion to delete redundant files at Core/Dependencies/imgui/main.cpp and Core/Dependencies/glm/glm/detail/dummy.cpp which causes 'main is already defined error' after including imgui and glm in premake

Thanks ! It is now merged :)


Applying academic discount doesn't update the default deposit amount

If I don't edit the default deposit amount, it stays at the old number after the AJAX realizes that it should be less. In this image, the deposit was $4 even, but then it realized there was a discount, and updated all numbers except the textbox. I suppose that if the user hasn't updated the number and the textbox isn't focused, we could update that amount automatically?

Ideally we should collect the same amount from them that they are filling and then give them virtual credits to run double the number of tasks. It should never increase their displayed $ value in credits. Otherwise, if they ask for full refund, that will mean we legally have to pay them more than they are entitled for.

umm im confused, why would the amount change? the "Your credit card will be charged" value is?

if they ask for a refund we only have to refund their credit card charges .

But what is the displayed value in their credits? As per what Michael is saying above, it appears like it will show $4 even though they got charged $2.38

the 2.38 is all that matters, sure they will be given 4 daemo dollars :D

i think what's missing is a refund policy on the page otherwise it's fine .. but we dont refund anyway :D

Sorry, I wasn't clear. I wanted to launch $4 in tasks. So I went to deposit. Originally the page said I had to deposit $4. Then it realized I got the academic discount via AJAX/Angular, and updated most of the page to reduce my cost. Except it left the amount that I was depositing the same. If I now need less in order to cover my costs, shouldn't it reduce that deposit amount?

ah just the default you mean, sure i guess (?) they can change it by themselves (?)

Yeah, I was mainly suggesting that the default # change if/when we realize that their costs have shifted.

@mbernst sorry, I'm still struggling with this one ... and i think because it's correct as it is and reason is this:
half off works like this: there's an amount of funds X required to post a project (or the requester wants to deposit a certain amount), now they pay half we pay the other half.
My point is that X doesn't change, it's not that we need less money for this project, it's just that you'll end up being charged half of that amount. And the "deposit amount" specifies how much Daemo credits ($) they'd like to purchase, if we change that to half they will end up being charged 1/4 of the required (original) amount which is not what we want.

Ah, I see. OK, let's close it.


[9.x] Add dispatchWith method to Dispatcher class

The Dispatcher class allows referencing any method as a listener for a given event, but there is an important assumption on the listeners.
And that is they should not throw exceptions or otherwise the execution of the other listeners will stop, and there is nothing that the dispatcher of the event can do about it. In other words, it can not set an exception handler for each and every listener.
For example in HomeController:
try {
    \Event::dispatch('user-registered);
} catch (\Exception $e) {
    // handle the exception
}

In the example above, if the first listener throws an exception, all the other listeners would not be executed and it goes to the catch block directly.

 $wrapper = function ($listener) {
    try {
        return $listener();
    } catch (\Exception $e) {
        // report the exception...
    }
}

\Event::dispatchWith('user-registered', wrapper: $wrapper);

But this one allows the caller to set a try/catch for each and every listener.

This is not needed if all the listeners are set by the person who fires the event, but for a package developer who is firing an event in the package code, it is always a matter of question if the listeners set by the users are healthy enough or not. In other words, you fire the events but you never know what is going to be executed as the listener.

In this implementation, the wrapper can not know about the details of the listener since it is already wrapped in a closure and can only call it and get the results.

You may also decide to add it to the Dispatcher interface as well.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


When two popovers are added one inside the other, it creates a flickering effect.

Encountering a bug: When integrating the calendar component within the popover component, the calendar shakes twice unexpectedly.

this may be a shot in the dark but have you tried wrapping your entire component in a Form?
There is an example on the website which does what you're trying to do I believe and the main difference (that I can see) is that its wrapped in a Form component
https://ui.shadcn.com/docs/components/calendar

@BahlulHasanli Can you help me reproduce this? Here's a CodeSandbox link: https://codesandbox.io/p/sandbox/github/shadcn/next-template-debug/tree/main
You can fork it and add the code above to it.

@BahlulHasanli Can you help me reproduce this? Here's a CodeSandbox link: https://codesandbox.io/p/sandbox/github/shadcn/next-template-debug/tree/main
You can fork it and add the code above to it.

Hi @shadcn you can try here

Hi @shadcn you can try here
https://codesandbox.io/p/sandbox/exciting-bardeen-lvmtrd

Hi @shadcn you can try here
https://codesandbox.io/p/sandbox/exciting-bardeen-lvmtrd

I'm also facing the same flickering when using the date picker inside a dialog.  It seems anywhere where I use animated components inside a dialog flickering happens. Alert Dialog inside a dialog also has same issue.

i have flickering problem on my popover, dialog and accordion.
https://github.com/shadcn-ui/ui/assets/107303813/e5c9d3b6-ec13-480e-8200-ff3a0efbacdc

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


Add tablefunc for use in dashboard queries

Add tablefunc for use in dashboard queries
This PR lets us use crosstab in some of the ltree based queries on the
dashboard.

LGTM


Reusable widget

When built as a Slicer extension, a loadable module is created.
When built as a standalone project, a CLI executable is built.

@jcfr i had a quick look and things look :thumbsup:
I am running a local build before merging and let you know

Great! thanks


Generic socialpic - Pulls from multiple social media services

Q
A

Bug fix?
no

New sample?
yes

Related issues?
N/A

What's in this Pull Request?
Provides a new sample that uses a Social Media Image service (http://avatars.io) to return the Social Media pictures from multiple Social Media Services (Twitter, Instagram, Etc.) in a dynamic way.
NOTE: This could be merged with the Generic TwitterPic example, but I figured I would provide it as a separate example first.
Guidance
Sample that provides a Social Media Image request for multiple Social Media services (Twitter, Instagram, Etc.) and displays the Social Media picture from each service.
Thanks for your contribution! Sharing is caring.

Great sample! Didn't even know this service existed but seems very powerful, thanks!


Binary search on answer and related problems for practice and solutions

hi i previously solved questions in gfg.I am intrested to take part in this issue.I am gssoc 23 contributor

Please assign me this issue.

Hello @aaryahjolia, I had previously solved many question on binary search in codechecf, codeforces, leetcode and gfg. I can make good contribution to this task, please assign me this one. :)


Setup github actions

WIP testing

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Submission audit log: return entity property in event details

Right now, the entity audit log endpoint returns details about the creating submission in the submission property of the details object of the entity.create event. That allows Frontend to determine the submission's instance name and whether it has been deleted. I think it'd be great if we could do something similar in the reverse direction: if the submission audit log could return details about the created entity. Those details could go in an entity property on the details object of the entity.create event returned in the submission audit log. Doing so would have the following benefits:

Frontend could show the entity's current label instead of its initial label.
If the entity label could be obtained from this new entity property, we wouldn't need to store it in the audit log. That would give us a path forward on #822.
If the entity has been deleted, Frontend could indicate that (most importantly, it would not link to the entity).

I think everything discussed above has been implemented.


Fundamentals of Modern Javascript

A blog series that covers the new ES6 features, es linters, etc. Embrace the new way to write Javascript code :).

Nope.


Allow Indexing for Exprs

As mentioned in the comments of commit 080e4d7 (related to #328), we run into errors when attempting to index a list (since [] expects an Int but the variables that need to be passed in are Exprs).

duplicate of #265 ?

Indexing added to Expr in  f682d82


HTTP integration - server selection not respect by Test Request

What happens?
After selecting a server from the list of available servers at the top of the page (I'm deploying this with regular HTTP, embedding the API spec in it), when using the "Test Request" feature on any endpoint, it still shows the URL of the first server in the list, and on hover says "variable not found"
What did you expect to happen?
It should ideally memorize the server selected at the top of the API spec and/or let me switch easily with something like a dropdown on click/hover of the server URL that already shows at the top of the Test Request dialog.
OpenAPI Document
No response

Yep known issue, we will have this one fixed tomorrow

Awesome, thanks!

Should have been fixed in https://github.com/scalar/scalar/pull/3138
release coming soon!

awesome! Great thanks

Hello, I am using scalar server via CDN.
Seems like changing the server from the dropdown still doesn't produce any effect on the testing client.
Am I missing something?


Build error on catkin_make_isolated --install --use-ninja

Hey! I am trying to build cartographer using these instructions but I get the following error. (my system is Ubuntu 14.04, gcc 4.8.4, ROS Indigo).
==> ninja -j1 in '/home/manos/Development/cartographer_ws/build_isolated/cartographer/install'
[2/296] Building CXX object cartographer/common/CMakeFiles/common_blocking_queue_test.dir/blocking_queue_test.cc.o
FAILED: /usr/bin/clang++    -O3 -DNDEBUG -I. -I/home/manos/Development/cartographer_ws/src/cartographer -isystem /usr/src/gmock/gtest/include -I/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/{Boost_INCLUDE_DIRS}     -pthread -std=c++11  -Wall -Wpedantic -Werror=format-security -Werror=missing-braces -Werror=reorder -Werror=return-type -Werror=uninitialized -O3 -DNDEBUG -MMD -MT cartographer/common/CMakeFiles/common_blocking_queue_test.dir/blocking_queue_test.cc.o -MF "cartographer/common/CMakeFiles/common_blocking_queue_test.dir/blocking_queue_test.cc.o.d" -o cartographer/common/CMakeFiles/common_blocking_queue_test.dir/blocking_queue_test.cc.o -c /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc:17:
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue.h:24:
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:60:7: warning: unknown attribute 'capability' ignored [-Wattributes]
class CAPABILITY("mutex") Mutex {
      ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:36:53: note: expanded from macro 'CAPABILITY'
#define CAPABILITY(x) THREAD_ANNOTATION_ATTRIBUTE__(capability(x))
                                                    ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:57: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                                        ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:67:26: warning: unknown attribute 'acquire_capability' ignored [-Wattributes]
    Locker(Mutex* mutex) ACQUIRE(mutex) : mutex_(mutex), lock_(mutex->mutex_) {}
                         ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:48:33: note: expanded from macro 'ACQUIRE'
  THREAD_ANNOTATION_ATTRIBUTE__(acquire_capability(__VA_ARGS__))
                                ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:57: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                                        ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:69:15: warning: unknown attribute 'release_capability' ignored [-Wattributes]
    ~Locker() RELEASE() {
              ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:51:33: note: expanded from macro 'RELEASE'
  THREAD_ANNOTATION_ATTRIBUTE__(release_capability(__VA_ARGS__))
                                ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:57: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                                        ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:75:46: error: invalid use of 'this' outside of a non-static member function
    void Await(Predicate predicate) REQUIRES(this) {
                                             ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:53: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
                                                    ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:57: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                                        ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:81:18: error: invalid use of 'this' outside of a non-static member function
        REQUIRES(this) {
                 ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:53: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
                                                    ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:57: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                                        ^
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc:17:
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue.h:110:33: warning: unknown attribute 'requires_capability' ignored [-Wattributes]
  bool QueueNotEmptyCondition() REQUIRES(mutex_) { return !deque_.empty(); }
                                ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:33: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
                                ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:57: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                                        ^
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc:17:
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue.h:113:32: warning: unknown attribute 'requires_capability' ignored [-Wattributes]
  bool QueueNotFullCondition() REQUIRES(mutex_) {
                               ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:33: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
                                ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:57: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                                        ^
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc:17:
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue.h:51:25: error: expected body of lambda expression
    lock.Await([this]() REQUIRES(mutex_) { return QueueNotFullCondition(); });
                        ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:3: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
  ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:42: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                         ^
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc:17:
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue.h:59:22: error: expected body of lambda expression
            [this]() REQUIRES(mutex_) { return QueueNotFullCondition(); },
                     ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:3: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
  ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:42: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                         ^
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc:17:
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue.h:70:25: error: expected body of lambda expression
    lock.Await([this]() REQUIRES(mutex_) { return QueueNotEmptyCondition(); });
                        ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:3: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
  ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:42: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                         ^
In file included from /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc:17:
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue.h:81:22: error: expected body of lambda expression
            [this]() REQUIRES(mutex_) { return QueueNotEmptyCondition(); },
                     ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:45:3: note: expanded from macro 'REQUIRES'
  THREAD_ANNOTATION_ATTRIBUTE__(requires_capability(__VA_ARGS__))
  ^
/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/mutex.h:31:42: note: expanded from macro 'THREAD_ANNOTATION_ATTRIBUTE__'
#define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
                                         ^
5 warnings and 6 errors generated.
ninja: build stopped: subcommand failed.
<== Failed to process package 'cartographer':
  Command '['/home/manos/Development/cartographer_ws/install_isolated/env.sh', 'ninja', '-j1']' returned non-zero exit status 1

Reproduce this error by running:
==> cd /home/manos/Development/cartographer_ws/build_isolated/cartographer && /home/manos/Development/cartographer_ws/install_isolated/env.sh ninja -j1

Command failed, exiting.

Any idea what goes wrong?
Thanks!

Actually, your commandline shows that you are using clang, not gcc:
FAILED: /usr/bin/clang++    -O3 -DNDEBUG -I. -I/home/manos/Development/cartographer_ws/src/cartographer -isystem /usr/src/gmock/gtest/include -I/home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/{Boost_INCLUDE_DIRS}     -pthread -std=c++11  -Wall -Wpedantic -Werror=format-security -Werror=missing-braces -Werror=reorder -Werror=return-type -Werror=uninitialized -O3 -DNDEBUG -MMD -MT cartographer/common/CMakeFiles/common_blocking_queue_test.dir/blocking_queue_test.cc.o -MF "cartographer/common/CMakeFiles/common_blocking_queue_test.dir/blocking_queue_test.cc.o.d" -o cartographer/common/CMakeFiles/common_blocking_queue_test.dir/blocking_queue_test.cc.o -c /home/manos/Development/cartographer_ws/src/cartographer/cartographer/common/blocking_queue_test.cc

Unfortunately, Cartographer does not build with clang right now. And ROS is kinda gcc centric too. If you fix the build, please send a PR.

Correct! Thanks ;)


Change bosco morning

Change "bosco morning" so that pulling docker image changes is the last task that's run. This is so that people can start working on the code before the docker images are pulled. There is a friendly message to let you know that you're (probably) ready to start working.

@jakehoward @vaneetkaur I've merged Dali's change, so please merge from master as part of the PR.

lgtm - thanks


Try to fix e2e with mapsId

Quick fix https://github.com/jhipster/jhipster-vuejs/issues/473
Not sure it's a correct solution

Please make sure the below checklist is followed for Pull Requests.

[ ] Travis tests are green

[ ] Tests are added where necessary

[ ] Documentation is added/updated where necessary

[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

cc @pmverma : when you have time, can you help us to improve the Vue.js part, about this, plz ?

@pascalgrimaud yes, it is not a correct solution, but I would have done the same. It was hard to create a proper test for mapsId thus I left it as a comment block.

Ok fine so. Thanks @pmverma for having a look :-)


Using SPMT Powershell cmdlets when SPOnline does not allow legacy sign-in methods

Trying to use SPMT PS cmdlets to migrate file shares to SPO.  Our sandbox tenant allows legacy sign-in methods, however the production build does not.  Our PS scripts are failing with "Cannot contact web site ‘https://tenant-admin.sharepoint.com/’ or the web site does not support SharePoint Online credentials. The response status code is ‘Unauthorized'” errors, which I am fairly confident is related to the sign-on method.
Does SPMT support AppOnly credentials?  I have not found anything indicating that it does.  Could we use Connect-PnPOnline somehow to make this work?  Note that our PS scripts are running under a windows service unattended, so no user intervention is possible.

Hello @GarthMartin
The current methods of authentication are the ones described in the article https://docs.microsoft.com/en-us/powershell/module/spmt/register-spmtmigration

-SPOCredential
The credentials are used to connect the SharePoint Online tenant. If no credentials are passed in, use ADAL OAuth2. Important: If you are using multifactor authentication on your tenant, do not use the -SPOCredential parameter. The ADAL dialog will prompt for authentication and MFA.

hi, i am trying to use register-spmtmigration without the -spocredentials flag but the command is just completed without the ADAL dialog.


Add ngx_http_geoip2_module

Adding ngx_http_geoip2_module module (https://github.com/leev/ngx_http_geoip2_module) would be useful for visualizing locations using docker images such as https://github.com/gilbN/geoip2influx

I agree integrating geo location for geoblocking is a big part of protecting an NGINX install.  I tried shoe horning it in through apk edge install and such but it resulted in bad nginx 'stream' config errors.  It would be great to have the GeoIP2 module.

+1

+1

That would be a great thing to have geoip maxmind is part of the Access List, options of restrictions + adding exceptions of IPs if needed

Duplicate of https://github.com/jc21/nginx-proxy-manager/issues/46


Help setting version (height related) correctly on branch builds for nuget packages

We're using GitVersioning in most of our projects with only one slight annoyance.  In the case of our internal NuGet packages we build pre-release versions for all branch commits and push them to nuget.   With the config we currently have unfortunately the pre-release version races ahead of our latest master, even when the branch is finally merged back to master, because of the git commit height on the branch.
Example:

Latest master version = 3.0.17
Start a branch

first commit builds/publishes version 3.0.18-g63d7b27d66
second commit builds/publishes version 3.0.19-g659f66fde5
third commit builds/publishes version 3.0.20-7c49eb8dde
etc

Merge branch, master builds/publishes version 3.0.18

However the latest version of the package when pre-release versions are included ends up being shown as 3.0.20-7c49eb8dde.  Worse still, if a branch from last year had lots of commits it will be shown as the latest version even though its very old:

Is there a version.json configuration that will avoid this problem, possibly by pushing the git height number to a 4th semver position (giving versions like <IP_ADDRESS>, <IP_ADDRESS> on the branch in the above example)?
Here's our current version.json:
{
    "version": "3.0",
    "nugetPackageVersion": {
      "semVer": 2
    },
    "publicReleaseRefSpec": [
      "^refs/heads/master$",
      "^refs/tags/v\\d+\\.\\d+"
    ],
    "cloudBuild": {
      "setVersionVariables": true,
      "setAllVariables": true,
      "buildNumber": {
        "enabled": true,
        "includeCommitId": {
          "when": "nonPublicReleaseOnly",
          "where": "buildMetadata"
        }
      }
    }
  }

There is no way that merging a topic branch versioned as 3.0.20 into a master branch versioned as 3.0.17 will result in master building 3.0.18. NBGV always takes the longest path in history, so merging 3.0.20 and 3.0.17 together will result in a 3.0.21 version.
Did you squash instead? That would cram all your individual commits into just one on master and result in a version height increment of just one, taking you from 3.0.17 to 3.0.18.
Don't squash commits that you've already published. There's no way for NB.GV to track history and keep versions incrementing when you do that.

Ok, I think this is simply because our standard workflow IS squash merge.  Publishing the pre-release builds from branch commits just helps us facilitate local testing of other apps that depend on the package (these nuget packages are not published publicly or indeed outside the dev team).  Ideally we want separate control of the version numbers on the branch, I'm feeling NBGV is not designed for this workflow (and yeah maybe we aren't following best practice for the nuget case).

I would question your squash policy. Why in the world would you want to erase history of commits that you actually published as nuget packages? You'd never be able to find/retrieve the source for those packages again.

I would question your squash policy. Why in the world would you want to erase history of commits that you actually published as nuget packages? You'd never be able to find/retrieve the source for those packages again.

Put simply, its because we don't release to a public nuget feed.  Pushing the "pre-release" versions from branch commits is simply the easiest way for us to test that version against apps that consume it without merging the branch into master, i.e. its just for convenience in our dev cycle.  None of our apps ever use pre-release versions of these packages in their master/prod versions so we have no need to keep the commits or the packages for that matter  (I guess one answer is that we should delete/unlist all "invalid" pre-release versions from the nuget feed after completing the squash merge, but that's something we'd need to look at automating).
I totally understand the the expected/standard usage and that NBGV is working as designed.


error: missing binary operator before token "("

Type: Feature Request
Don't compile anything
VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:32:34.589Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: Yes

/needsMoreInfo

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Customize symbols for the diagnostics component

Enable to set symbols for diagnostics.

Nice idea . I think it should support setting single symble and leaving rest to default . Like if I want to change the error symbol I and I set only the property symbol.error warn and info should use default symbols ....

I see. I'll rewrite with vim.tbl_extend() and enable to overwrite a part of symbols.

lgtm, thanks


Provider Error(?)

How am i supposed to do?
All line with builder: (_) => is error.

So i change builder with crate and it worked.


Broken notebooks

Describe the bug
Figure-9.4.ipynb crashes with:
RuntimeError                              Traceback (most recent call last)
Cell In[2], [line 21](vscode-notebook-cell:?execution_count=2&line=21)
     [19](vscode-notebook-cell:?execution_count=2&line=19)     else:
     [20](vscode-notebook-cell:?execution_count=2&line=20)         print("Exiting program ...")
---> [21](vscode-notebook-cell:?execution_count=2&line=21)         raise RuntimeError("Can not find %s" % nyhead_file)
     [23](vscode-notebook-cell:?execution_count=2&line=23) with h5py.File(nyhead_file, 'r') as f:
     [24](vscode-notebook-cell:?execution_count=2&line=24)     lead_field_normal = np.array(f["sa"]["cortex75K"]["V_fem_normal"])

RuntimeError: Can not find sa_nyhead.mat

Figure-9.9.ipynb crashes with:
ValueError                                Traceback (most recent call last)
Cell In[2], [line 19](vscode-notebook-cell:?execution_count=2&line=19)
     [15](vscode-notebook-cell:?execution_count=2&line=15)     return xis, corr_sig
     [17](vscode-notebook-cell:?execution_count=2&line=17) xis, corr_sig = calculate_all_xis()
---> [19](vscode-notebook-cell:?execution_count=2&line=19) np.save("xi_corr.npy", [xis, corr_sig])
     [20](vscode-notebook-cell:?execution_count=2&line=20) #xis, corr_sig = np.load("xi_corr.npy")

File <__array_function__ internals>:200, in save(*args, **kwargs)

File ~/miniforge3/envs/brainsignals/lib/python3.11/site-packages/numpy/lib/npyio.py:521, in save(file, arr, allow_pickle, fix_imports)
    [518](https://file+.vscode-resource.vscode-cdn.net/Users/espehage/Repositories/ElectricBrainSignals/notebooks/Ch-9/~/miniforge3/envs/brainsignals/lib/python3.11/site-packages/numpy/lib/npyio.py:518)     file_ctx = open(file, "wb")
    [520](https://file+.vscode-resource.vscode-cdn.net/Users/espehage/Repositories/ElectricBrainSignals/notebooks/Ch-9/~/miniforge3/envs/brainsignals/lib/python3.11/site-packages/numpy/lib/npyio.py:520) with file_ctx as fid:
--> [521](https://file+.vscode-resource.vscode-cdn.net/Users/espehage/Repositories/ElectricBrainSignals/notebooks/Ch-9/~/miniforge3/envs/brainsignals/lib/python3.11/site-packages/numpy/lib/npyio.py:521)     arr = np.asanyarray(arr)
    [522](https://file+.vscode-resource.vscode-cdn.net/Users/espehage/Repositories/ElectricBrainSignals/notebooks/Ch-9/~/miniforge3/envs/brainsignals/lib/python3.11/site-packages/numpy/lib/npyio.py:522)     format.write_array(fid, arr, allow_pickle=allow_pickle,
    [523](https://file+.vscode-resource.vscode-cdn.net/Users/espehage/Repositories/ElectricBrainSignals/notebooks/Ch-9/~/miniforge3/envs/brainsignals/lib/python3.11/site-packages/numpy/lib/npyio.py:523)                        pickle_kwargs=dict(fix_imports=fix_imports))

ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (2,) + inhomogeneous part.

I run it in VSCode with the jupyter extension etc. The issue is that while it does prompt for confirmation accepting "Enter" as input -- but this is a VSCode prompt -- so download will not start. We can close that particular case then.


Inconsistent error handling between communication protocol for state management

Expected Behavior
Whether you use HTTP or gRPC shouldn't matter for coding.
Actual Behavior
The save() method in the gRPC implementation throws an exception on failure, which is great
However, the same save() method in the HTTP implementation returns an object containing an error without actually throwing an exception.
Steps to Reproduce the Problem
You can see this in the code itself
Re-trowing the error in the HTTP implementation should fix this, and perhaps using the cause attribute on top of that to chain errors would also help with debugging.
I can help with this if needed.

/assign

/keep-alive


add user agent in getusermembership

add user agent in getusermembership
@ZhijunZhao @yaweiw @aovechki

@yungez,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #124 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #124   +/-   ##
=======================================
  Coverage   72.07%   72.07%
=======================================
  Files          58       58
  Lines        1221     1221
  Branches      118      118
=======================================
  Hits          880      880
  Misses        274      274
  Partials       67       67

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 ce72c3f...563a8e2. Read the comment docs.

still working with bi team to check weather this is right way to set user agent, close this pr for now.


SeriesGroupBy does not accept dd.Aggregates

It uses pandas’ validation function, which doesn’t recognize dask’s Aggregate class:
https://github.com/dask/dask/blob/99036c42083fa77fe974fc38a58391340566baaa/dask/dataframe/groupby.py#L2279

@flying-sheep, thanks for creating this issue. Please provide a minimal reproducer (see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports for guidelines) to allow us to investigate your problem.

Not really necessary, you can directly see the problem in the linked line, but sure.
import pandas as pd
import dask.dataframe as dd

custom_sum = dd.Aggregation(
    name='custom_sum',
    chunk=lambda s: s.sum(),
    agg=lambda s0: s0.sum()

)

df = pd.DataFrame(dict(a=[1, 1, 1, 1, 1], g=[5, 6, 6, 6, 7]))
ddf = dd.from_pandas(df, npartitions=2)

ddf.groupby('g')['a'].agg(sum="sum").compute()  # works
ddf.groupby('g')['a'].agg(sum=custom_sum).compute() # broken

Thanks for adding the reproducer! Please keep in mind that the time of contributors is limited, so having a reproducer ready allows them to move more quickly. It is also a great starting point for a regression test.


Any plan to support Hangout Chat ?

Recently (march 2018) google again changed its products lineup. They are now proposing 'hangout chat'.
which is somewhat compatible with Hangout, but offers many more features (essentially to fight slack, if you want my advice)
details for instance here https://www.techrepublic.com/article/google-hangouts-chat-the-smart-persons-guide/
For now Hangout Chat is missing a linux client,
that's why I'm asking : would yakyak be such a candidate for this or not at all ?
Note that I'm not asking for the feature, I'm just looking for such a client, and wondered what were the plans regarding yakyak.
thank you !

no plans, sorry

Google Hangouts will be terminated soon (Oct 2019) https://gsuiteupdates.googleblog.com/2019/01/upcoming-hangouts-service-consolidation.html
Any plan to migrate to Hangouts Chat now?


New-VenafiSession cannot be created with a valid AccessToken

Unlike with VenafTppPS's New-TppSession, I cannot generate a new session with a valid OAuth token.
Environment
Operating System: Windows 10
VenafiTppPS version: 20.4
PowerShell version: 5.1

Steps to reproduce
$mysession = New-VenafiSession -Server $vtp_host -AccessToken $token -PassThru
Expected behavior
I would expect to receive a TPPS object that I can then use to create a header object and pass into other commands.
Actual behavior
I am prompted for a credential.  This seems to completely defeat the whole purpose of OAuth in the first place.
Screenshots
PS C:\PowerShell> $token
gjTEwxxxxxxxxdK+kw==
PS C:\Powershell> $mysession = New-VenafiSession -Server $vtp_host -AccessToken $token -PassThru
Windows PowerShell credential request
Enter your credentials.
Password for user gjTEwxxxxxxxxdK+kw==:

And then I actually read the doc.  Sorry.


doesn't work at coin hive

please check this error when i try to shortlink using coin hive https://cnhv.co/6bk
OS: windows 10 FCU
Latest version: 0.4.14
Browser:chrome Version 65.0.3325.146 (Official Build) (64-bit)

Whitelist this or excempting it, I think would be a bad idea. they're using there site to check if script is loaded.
I think the site should warn user if their script is blocked, but not block access to site. Personally its serverside issue rather than list issue.

CheetosLover,
use NoCoin's built-in 'Whitelist for 1 Minute' or 'Pause' functionality.


With vertical split windows, when aerial window is opened, the invoking buffer gets squished up (becomes small in width than the irrelevant buffer without corresponding symbols))

Description
Let's say we have two files opened in vertical splits. If we open the symbols outline with any suitable :Aerialxxx command (or keymapping), that window gets squished up (i.e. becomes smaller than the other window which doesn't correspond to the aerial symbols).
The expected behaviour is that the two originally opened windows should retain their width proportions after opening/closing aerial window(s).
Examples:

If the two original windows were at 50% widths before opening 1 aerial window, they should remain equal in width after opening it. They should restore to 50% of screen widths after closing the single aerial window.
If the two original windows were at 50% widths before opening 2 aerial windows, they should remain equal in width after opening them. They should still be equal size after closing 1 aerial window. Each should restore to 50% of screen width after closing both the aerial windows.
If one of them was at 40%, and other at 60% width before opening 1 aerial window, they should remain at 2:3 proportion after opening the aerial window, and restore to the original widths after closing the aerial window.
The logic should scale to any number of regular (or aerial) windows.

Preliminary Ideas

Extract the currently open vertical window widths into an array. Ignore any horizontal splits in further width calculations.
Calculate the ratio/proportion of each vertical split
When an aerial window is opened, resize the remaining available screen area based on the above ratio (use the nearest integer multiple/rounding if necessary for the new widths). Of course, each aerial window will respect the max_width, width, and min_width setting tables as usual.
Repeat 3: above when additional aerial windows are used.

System information

OS: Linux
Neovim version:
AerialInfo:

Filetype: lua
Configured backends:
  treesitter (supported) (attached)
  lsp (not supported) [LSP client not attached]
  markdown (not supported) [Filetype is not markdown]
Show symbols: Array, Boolean, Class, Constant, Constructor, Enum, EnumMember, Event, Field, File, Function, Interface, Key, Method, Module, Namespace, Null, Number, Object, Operator, Package, Property, String, Struct, TypeParameter, Variable

Aerial config: defailt config

Screenshots
Before opening aerial window

After opening aerial window (the buffer from which the :AerialOpen was triggered gets squished up in width).

I agree that the windows (especially the active window) getting squished is not ideal. I've pushed up a change that should fix that. This won't produce the exact behavior you're asking for when windows have a manually-set width, but it's far simpler and should cover the common cases.

Hmm. If you have carefully constructed your window splits layouts (and saved them to a session using a session management plugin), equalising all the other splits after opening an aerial window might not be desirable at all.
This window equalising strategy will also not play well with plugins that provide a golden-ratio layout for the splits.
All in all, while preventing the squishing up is great, it would be respectful for the plugin to adhere to the pre-aerial window split proportions if its technically feasible to do so.


⚠ Auth has degraded performance

In b0607c9, Auth (https://auth.k2lin.com) experienced degraded performance:

HTTP code: 200
Response time: 11554 ms

Resolved: Auth performance has improved in 091ed8b.


fs.mkdir in ensureDirExistsForWritesAsync is not async

fs.mkdir in ensureDirExistsForWritesAsync is not async
This results in the testing that is being written to fail

May need to look at splitting the functionality out for this function. Either add in another param which is something like 'create dir if does not exist', or just flat up make the function return a boolean and that can be handled however is needed (even as another function within the utils file)

So the mkdir was async. I was just being blind. Problem is the jest test isn't logging things nicely without using fs.promises. Still might want to split the functionality out but that's not part of this issue.


Add a references section to async stack traces

Ref https://github.com/rust-lang/wg-async-foundations/issues/251
This adds references from other languages to the async stack traces section. Thanks!

This looks great, thanks!


Add missing types for dashboard 2.0 after adding translations to A

It looks the graphql types for dashboard 2.0 is not update-to-date.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] GraphQL schema and type definitions are up to date.
[x] Changes are mentioned in the changelog.

The failed CI https://travis-ci.org/mirumee/saleor/builds/501983687?utm_source=github_status&utm_medium=notification in https://github.com/mirumee/saleor/pull/3804 has similar issue.


Help with Python implementation

Hi there,
I would like to help with the progress of this project by starting off with Python support. Hopefully someone can help with the question I have but, I was wondering what is the goto command for running a Python app? For me I usually have the Python app run on app.py but, that's not the case for all projects. I would like to know what the Python equivalent for a Python app would be and if there are any additional steps. For example, in https://github.com/coollabsio/coolify/blob/main/api/packs/nodejs/index.js#L24 the run command is "yarn start"

Hey,
First of all, thanks for the contribution!
I have an idea of how to support Python. As I know - but my Python knowledge is not the best - it has no equivalent like for Node.
So I would place another field on the frontend where you can set the main python script—no better idea atm.
I will implement such a thing!

Basic Python support landed in #49. 🎉 I will extend it later on with different application-specific deployments, like Django.


[RFC] error reporting

Basically error reporting is not standardized through the library and also could possibly be made more descriptive and useful for the user.
I would like you to point out the use cases and scenarios where there is a room for improvement.
Pain points:

try/catch vs uncaught

Context: in React Native, if there's an exception inside the saga, the error log it spits out is completely useless and makes me wade through lines after lines of redux-saga internals.

Sounds like a first step would be to log the error.message here
https://github.com/redux-saga/redux-saga/blob/v0.14.3/src/internal/proc.js#L437
Because of the ||s, it only logs error.stack.

if there's an exception inside the saga

Worse, because yielding put calling the reducers, errors throws in any reducers also get the message-less RedScreen.

Would adding error.message to the log (as suggested bt @simlmx) would help a little?

Absolutely, though coming from React Native it would be nice if the error was formatted the same.
In fact, is there any reason the error object itself can't be logged? That would allow the platform to handle its display in the console.

The error.stack property is implemented inconsistently across browsers. Most notably, it includes the error message in Chrome, but not in Firefox. This makes debugging in Firefox a tad harder.
Since the stack property is non-standard, I would be in favour of dropping it completely, but as a quick fix, I would also welcome adding the error.message to the log, even if this would duplicate it in Chrome.

@Andarist @richardszalay What is blocking merging that simple change (adding error.message to the log)?

I've standardized error logs a little bit in latest release. Gonna close this issue - but if you feel there is any more room to improvement (beside implementing babel plugin for better stack traces like mentioned here) please create a new issue so we can discuss this further.


How can we use this model to train on real low resolution images without super-resolution images?

I am a new researcher in super-resolution work, for Blind Image Super-Resolution, I think we cannot get the SR image information and true kernel. But in your work, you have used this information, maybe this is not Blind Image Super-Resolution?

Although the ground truth kernels are required in training, we don't need them in testing. In other words, once the model is well-trained, we can directly apply it to any image, estimating the kernel and recovering a high-resolution version. So the inference is exactly the 'blind' super-resolution.

Although the ground truth kernels are required in training, we don't need them in testing. In other words, once the model is well-trained, we can directly apply it to any image, estimating the kernel and recovering a high-resolution version. So the inference is exactly the 'blind' super-resolution.

作者您好，感谢您做出的优秀工作，关于以上讨论我还有些问题想请教一下：我看您的论文和代码中，在每次训练过程中，关于模糊核的类型和尺寸都是特定的，这样如果在不确定测试集的模糊核类型和大小的情况下，是否效果就不行了？
进一步地，为了解决上述问题，如果在训练过程中让模糊核的类型和尺寸都是随机的，这样您的方法还是否适用呢？
刚接触这个领域不久，有理解不对的还请谅解。

理论上确实如此，测试集模糊核不确定的话效果肯定会变差。不过我没有试过随机类型或尺寸模糊核，这个也许可以参考下Real-ESRGAN。


[WIP] refactor: switch to fresh

Left to do:

[ ] module file search & manual searchbars
[ ] fix code blocks
[x] registry "add module"
[x] benchmarks page
[x] _app (waiting on fresh)
[ ] various minor css things

Closing in favour of #2016


MLPAB-1113, MLPAB-1114: Send SMS updates to paper and digital certificate providers

Purpose
Adds checks on what stage the donor and CP are at in their journeys and sends appropriate SMSs.
Also enum-ed how CPs carry out their role.
Fixes MLPAB-1113, MLPAB-1114

Branch names with + is a no-go.


[TEST MANIFEST Do Not Merge] Tgstation.Server version 6.1.999

Do not merge this pull request.
This manifest is a reversioned resubmission of Tgstation.Server.6.1.0
Recently, Tgstation.Server has begun failing winget validation (#132245, #133023, #135613). I have no reason to believe any change made to the package's code would cause this.
This PR is meant to test the last version that succeeded (#132002) against the validation pipeline to see if the problem lies in the pipeline itself.
I will close this PR on conclusion of the test.

Point proven, issue opened. https://github.com/microsoft/winget-pkgs/issues/135625


Crashing on macOS Sierra

Hi there, interesting looking client for macOS, but doesn't seem to behave well. Hanging at full utilization, etc.
What can I do to help?
Hardware Overview:
Model Name:	MacBook Pro
Model Identifier:	MacBookPro10,1
Processor Name:	Intel Core i7
Processor Speed:	2.8 GHz
Number of Processors:	1
Total Number of Cores:	4
L2 Cache (per Core):	256 KB
L3 Cache:	8 MB
Memory:	8 GB
Boot ROM Version:	MBP101.00EE.B0A
SMC Version (system):	2.3f36
System Software Overview:
System Version:	macOS 10.12 (16A304a)
Kernel Version:	Darwin 16.0.0
Boot Volume:	Macintosh HD
Boot Mode:	Normal
Computer Name:	BTopping-MBPro
User Name:	Brian Topping (btopping)
Secure Virtual Memory:	Enabled
System Integrity Protection:	Enabled
Time since boot:	15:16

@briantopping We don't have Sierra support yet, working on it.
CC @gabriel

No worries, I figured that may be true.
The last developer preview didn't have an update to Xcode command line tools, which indicates to me there were no breaking changes at the library level. Clock is ticking, they'll probably be releasing soon! :)

Can you try this release:
https://prerelease.keybase.io/darwin/Keybase-1.0.17-20160823110029%2Bdfc9854.dmg

We've found an issue where the kext isn't loading on some Sierra installs, so the the version posted above still may not work. I'll post an update when we have a fix for that.

We found an issue when upgrading to Sierra, this build should work:
https://keybase.pub/gabrielh/Keybase-1.0.17-20160826020144%2B61c0667.dmg

Okay thanks Gabriel, I will give it a look this weekend. :)

That new build works for me.

I ran the build for a while and was starting to have some nasty system stability problems where various processes, especially Finder, would lock up for minutes at a time. From sampling them, it looked like they were blocked in filesystem calls looking for volume information, so I started to suspect the fake (FUSE?) filesystem that Keybase mounts. I quit Keybase and haven't had a problem since.

Is this using the new build from the link above? (Also might require a restart, since the current public version is ultra hangy on Sierra.)

Hi all, so I gave that one a shot and it's better in that it's not hanging my machine. After install, it did not insist that I reboot, and it actually worked (i.e. /keybase/briantopping existed) somewhat until I did. Once I rebooted, it did not lock up, but it did go into 15% CPU mode (presumably one core on a 8 hyper core machine) for some time. After the CPU settled down, it's behaving as I might expect, but it is using more CPU (kernel_task hovering around 9%).
How do I gracefully uninstall?

@briantopping Sorry you are having issues. Before uninstalling can you do another keybase log send so we can look at the perf issues?
To uninstall:

Run keybase uninstall to remove services.
Then you can remove the placeholder /keybase directory: sudo rmdir /keybase
Then you can unload the kext: kextunload -b com.github.kbfuse.filesystems.kbfuse
Remove the kext: sudo rm -rf /Library/Filesystems/kbfuse.fs
Then remove the helper sudo rm /Library/PrivilegedHelperTools/keybase.Helper.

Thanks @gabriel!
Results from log: Success! Your log ID is: 36e6d19420a6f29c2b592b1c
I'll keep watching the thread to try new results. :)

Ok, just an update after uninstalling and rebooting to avoid ratholes on your side: Once I uninstalled and rebooted, the kernel was still at 9%, so I think this must have to do with the debug state of macOS and not Keybase.

Yeah it's hard to tell what in the kernel is causing cpu usage. Also sometimes the samples in Activity Monitor can be misleading cause they are snapshots. Energy usage for a process might be a greater indicator for measure if a process is using lots of resources over time.

no worries, I just wanted to make sure I didn't lead you guys astray...

FYI - I'm currently running 1.0.17-20160908190031+139f86e and have not yet had any CPU usage problems, and the folders are working (although the contents of folders loads very slowly).

thank you for the info on how to uninstall keybase, hoping that resolves my short term issues with sierra!!
i've had nothing but trouble after updating to sierra (i know self inflicted).  Opening finder & doing anything that deals with processes (ie. Activity Monitor) will hang forever.  Started looking for kernel extensions and realized that keybase had one, so i checked here.  Looking forward to trying this again when it supports Sierra.

@eswat2 Sorry about this!  It looks like it's an incompatibility between Keybase (really, any FUSE filesystem) and Time Machine on Sierra.  We'll disable KBFS on Sierra until it's fixed.


Log files location?

I installed this cartridge without problems - thanks for great software! But I can't find the log files anymore. Are there any, or if not, how can I enable some node logs similar to Openshift's originals, located in nodejs/logs ?
The reason I'm asking is that my application doesn't start automatically when pushing to Openshift, but it starts if I do it manually using the same command as in package.json.
During the automated startup, I see the following (logshifter not found):
remote: Preparing build for deployment
remote: Deployment id is a4df8975
remote: Activating deployment
remote: CLIENT_MESSAGE: Starting Node.js application...
remote: /var/lib/openshift/58ef631bf9d320e77axxxxxx/nodejs/bin/control: line 46: /usr/bin/logshifter: No such file or directory
remote: CLIENT_RESULT: Node.js application started.
remote: Result: success
remote: Activation status: success
remote: Deployment completed with status: success

However, even though it looks like everything went fine, the app is inaccessible (Service Unavailable) until I start it manually using "node server.js"

That's strange. Everything works fine on a test deployment I've just created minutes ago.
/usr/bin/logshifter is present and the logs are created in ${OPENSHIFT_HOMEDIR}/app-root/logs/.
Just out of curiosity, what kind of account are you using?
The cartridge is tested and works perfectly on OpenShift Online, but it probably doesn't on OpenShift Enterprise, and most likely never will, since OpenShift didn't really provide any kind of feedback for past issues raised by Enterprise customers trying to use this cartridge...

Yes, you're probably right. The account belongs to the company I'm working for, and it says "Openshift Origin" top left.
I'm thinking I could create a standard NodeJS app on that account, and copy the bin/control file from there into this account (possibly with some adaptations) ... Do you think that would work?

Got it working in my Openshift environment with some minor changes to your bin/control file. I added two environment variables and changed a couple of lines in the start function.
#
# Copy this file to $HOME/nodejs/bin/control  ("control" is the filename)
#
export OPENSHIFT_NODEJS_LOG_DIR="/var/lib/openshift/<insert account-id>/nodejs/"
export OPENSHIFT_LOG_DIR=$OPENSHIFT_NODEJS_LOG_DIR

source $OPENSHIFT_CARTRIDGE_SDK_BASH
source $HOME/nodejs/lib/util

START_COMMAND=$(node -e "var p = require('$OPENSHIFT_REPO_DIR/package.json'); console.log(p.scripts.start);")
STOP_COMMAND=$(node -e "var p = require('$OPENSHIFT_REPO_DIR/package.json'); console.log(p.scripts.stop || '');")

function is_running() {
  if [ ! -z "$(ps -ef | grep "$START_COMMAND" | grep -v grep)" ]; then
    return 0
  else
    return 1
  fi
}

function pre-repo-archive() {
  if [ -d ${OPENSHIFT_REPO_DIR}node_modules ]; then
    rm -rf ${OPENSHIFT_DATA_DIR}node_modules
    mv -f ${OPENSHIFT_REPO_DIR}node_modules ${OPENSHIFT_DATA_DIR}
  fi
}

function build() {
  update_nodejs
  local INIT_DIR=`pwd`
  cd ${OPENSHIFT_REPO_DIR}
  if [ -d ${OPENSHIFT_DATA_DIR}node_modules ]; then
    rm -rf ./node_modules
    mv -f ${OPENSHIFT_DATA_DIR}node_modules ./
    rm -rf ${OPENSHIFT_DATA_DIR}node_modules
  fi
  npm prune --production
  npm i --production
  cd ${INIT_DIR}
  client_result 'Node.js modules installed.'
}

function start() {
  logf="$OPENSHIFT_NODEJS_LOG_DIR/node.log"
  if is_running; then
    client_result 'Application is already running.'
  else
    client_message 'Starting Node.js application...'
    local INIT_DIR=`pwd`
    cd ${OPENSHIFT_REPO_DIR}
    #nohup ${START_COMMAND} |& /usr/bin/logshifter -tag nodejs &
    nohup ${START_COMMAND} >> $logf 2>&1 &
    # nodejs_context "nohup $node_cmd >> $logf 2>&1 &"
    cd ${INIT_DIR}
    i=0
    while ! is_running && [ $i -lt 60 ]; do
      sleep 1
      i=$(($i + 1))
    done
    if is_running; then
      client_result 'Node.js application started.'
    else
      client_result 'Warning! Could not start Node.js application!'
      exit 1
    fi
  fi
}

function stop() {
  if ! is_running; then
    client_result 'Application is already stopped.'
  else
    client_message 'Stopping Node.js application...'
    if [[ ! -z "$STOP_COMMAND"  ]]; then
        local INIT_DIR=`pwd`
        cd ${OPENSHIFT_REPO_DIR}
        $STOP_COMMAND > /dev/null 2>&1
        cd ${INIT_DIR}
    else
        kill $(ps -ef | grep "$START_COMMAND" | grep -v grep | awk '{ print $2  }') > /dev/null 2>&1
    fi
    i=0
    while is_running && [ $i -lt 60 ]; do
      sleep 1
      i=$(($i + 1))
    done
    if is_running; then
      client_result 'Warning! Could not stop Node.js application!'
      exit 1
    else
      client_result 'Node.js application stopped.'
    fi
  fi
}

function restart() {
  stop
  start
}

function status() {
  if is_running; then
    client_result 'Node.js application appears to be running.'
  else
    client_result 'Node.js application appears to be stopped.'
  fi
}

function tidy() {
  shopt -s dotglob
  client_message "Emptying logs in ${OPENSHIFT_LOG_DIR}..."


Bump bs-platform dependency to 4.x

Depending on bs-platform 4.x to keep up with the times. :slightly_smiling_face:
No errors or warnings from bsb in this process, just some minor changes in the compiled JS output.

Guess I should have looked at the already-open PRs first. :stuck_out_tongue:. Closing in favor of #19.


[WIP] Adds MaxAge Property to CookieOptions and CookieBuilder

#688
@Tratcher Not sure if there is more to do here than this.

And https://github.com/aspnet/HttpAbstractions/blob/ab0185a0b8d0b7a80a6169fd78a45f00a28e057d/src/Microsoft.AspNetCore.Http/Internal/ResponseCookies.cs#L63

@Tratcher Should Max-Age be a header name in HeaderNames.cs?
\

No, it's not a header name, it's a header property name.

@Tratcher Do you recommend doing a full E2E to verify max-age shows up on requests?

on requests? None of the cookie settings are echoed on the request, on the the name and value. It's worth doing some manual testing though to make sure major browsers and HttpClient honor this setting.

ping

@Tratcher I verified locally and the max-age header is sent in the response.  :shipit: ?

Did you verify that various browsers and clients honor it? Or at least don't choke on it? HttpClient, IE, Chrome, Firefox...

ping :)


O5.1.4 (coupler) Extend TempestRemap functionality to conformal cubed sphere

(WIP)
The Climate Modeling Alliance
Software Design Issue 📜
Purpose

Cost/Benefits/Risks

b: this will allow accurate generation between RLL, Gnomomic and conformal cubed spheres, and facilitate coupling to Oceananigans.
r: this will require a substantial time investment

People and Personnel

Components

translate TempestRemap's weight generation to Julia
design of the extension (Overleaf doc write up)
MWE
demonstration in ClimaCoreTempestRemap + unit tests
demonstration in ClimaCoupler + regression tests
documentation

Inputs

Results and Deliverables

Task Breakdown And Schedule

Other comments

there may be an existing preliminary (untested) example using xESMF (ask Chris,  #217 )

out of date


Example of db-conn secret?

What problem are you facing?

There is no example of how to create the db-conn secret for the ProviderConfig?
What is supposed to be in this secret?
How could Crossplane help solve your problem?

Give an example of what the secrets for Postgresql and Mysql

I made a secret like this with the following keys:
{
  "user": "postgres",
  "password": "password",
  "endpoint": "postgresql.svc",
  "port": "5432"
}

I found some hints in the code.
Unfortunately I can't get a DB to actually provision on Postgresql.
I put root credentials in the secret even, so permission shouldn't be an issue.
Here is the error I get on the DB resource in the status:
observe failed: cannot select database: pq: Could not detect default
        username. Please provide one explicitly

Here is my ProviderConfig, does it look right?
apiVersion: postgresql.sql.crossplane.io/v1alpha1
kind: ProviderConfig
metadata:
  name: default
spec:
  defaultDatabase: postgres
  credentials:
    source: PostgreSQLConnectionSecret
    connectionSecretRef:
      namespace: crossplane
      name: postgresql

OK, I found it. The issue is user should be username
Now I get this error??
observe failed: cannot select database: pq: SSL is not enabled on the server

How to disable the SSL? This is just for testing at the moment.