Support to perform test set validation during the training evolution

The acc and loss reported during the training process in trainer.py seems to be the training data metrics.
And is there any easy-to-implement options to calculate the testing set acc during the training evolution in the current framework?
thank you~

Hi, none of the datasets I used provides a validation set, so I'm not able to obtain validation acc and loss during training.

And is there any easy-to-implement options to calculate the testing set acc during the training evolution in the current framework?

I don't think accessing test set during training stage is reasonable. But if you still want to do that, I guess you can simpily copy the test function in test.py to trainer/trainer.py, then modify the following code
https://github.com/Renovamen/Text-Classification/blob/4a4aa4001c402ed4371ebaabe1393b27794e5992/trainer/trainer.py#L205-L233
to
def run_train(self):
    start = time.time()

    # epochs
    for epoch in range(self.start_epoch, self.num_epochs):
        # trian an epoch
        self.train(epoch=epoch)

        # time per epoch
        epoch_time = time.time() - start
        print('Epoch: [{0}] finished, time consumed: {epoch_time:.3f}'.format(epoch, epoch_time=epoch_time))

        self.test()

        # some other code...


Run python image_to_animation.py drawings/garlic.png garlic_out Error,  Have you encountered the same problem, please advise

win10 system, I got an error after running the command from image to animation, can any of you help to solve it?

https://github.com/facebookresearch/AnimatedDrawings/blob/ceda092a5da380569d6793ef974710e7cc9c7ec1/examples/image_to_annotations.py#L50-L52
Please make sure your docker_torchserve is running and the returned resp.status_code should be 2xx.
You can add the following code for testing:
if resp.status_code >= 300:
    print("Failed to get bounding box", resp)
    return None

https://github.com/facebookresearch/AnimatedDrawings/blob/ceda092a5da380569d6793ef974710e7cc9c7ec1/examples/image_to_annotations.py#L50-L52
请确保您docker_torchserve正在运行并返回的resp.status_code应是2xx。
您可以添加以下代码进行测试：
if resp.status_code >= 300:
    print("Failed to get bounding box", resp)
    return None

Thank you very much for your help, but because I am not familiar with docker and the program, I am a novice, I don't understand how to do what you mentioned, is there any similar documentation or can you tell me the steps to help me to solve this error, thank you very much.

You can find the docs on the home page (the README.md)

哦，我明白了，在你跑之前image_to_animation.py，你必须先跑TorchServe。 您可以在主页上找到文档（README.md）

The official command given keeps getting stuck at docker build here, I don't know if it's because of the network, but I used docker_torchserve provided by a partner and started it manually in docker, then ran curl http://localhost:8080/ping此命令是正确的, just in the next step I get an error when running python image_to_animation.py drawings/garlic.png garlic_out command. I don't know why

Is there any error log with docker_torchserve?
You can use the following command:
docker logs -f --tail=500 docker_torchserve

Is there any error log with docker_torchserve?
You can use the following command:
docker logs -f --tail=500 docker_torchserve

/

Is there any error log with docker_torchserve?
You can use the following command:
docker logs -f --tail=500 docker_torchserve

docker_log.txt
This is the log I ran out, the content is rather long, if you have time then you can trouble to help me look at it, I really do not know these very well

Oops, it seems that docker_torchserve can't run on WSL, #136 they got the same problem.

糟糕，似乎docker_torchserve无法在 WSL 上运行，#136他们遇到了同样的问题。

Thank you very much, I will take a good look first, thanks again


multi-touch-viewer: add the surface selection optional

only a wl_shell or ivi surface is created base on wl_surface. we should select of the type of surface via input argument.
-s optional is added to example, if the input is 1, client will run with wl_shell. if option don't set or other value, the ivi surface is set.

Reviewed-by: Harsha M M<EMAIL_ADDRESS>


Custom Emojis not displaying in Android at all

Description
I have over 800 custom emojis on my server. And Rocket Chat Android app doesn't display any of them.
Can we please try to figure out why that is the case?
Thanks
Suleman.
Devices and Versions
Google Pixel
Your Rocket.Chat.Android version: (e.g. 2.1.0)
Your Rocket.Chat Server version: (e.g. 0.63.1-develop)

Mobile device model and OS version: (e.g. "Nexus 7 - Android 6.0.1")
Steps to reproduce

Logs

I am getting the exception:
chat.rocket.common.RocketChatInvalidResponseException: Expected a string but was BEGIN_OBJECT at path $.emojis[9]._id(url='https://www.thepen.chat/api/v1/emoji-custom')

It looks like some of your custom emojis are returning with the _id property as an object. Checked and got "_id": {"_str":"5bc429276e3a9221cee9fb03"} when it should have been "_id": "5bc429276e3a9221cee9fb03". Have you changed something on the backend side of your instance? I'm investigating this issue within our backend.

I didn't change anything in the backend. But I used the following script to upload custom emojis:
https://github.com/amtypaldos/rocketchat-emoji-upload

@real19 I'm closing this since it was disclosed that the bug was on the aforementioned script.
Feel free to reopen the issue if you feel there's something else we can do on our side.


Skip open and close testing tracks when releasing

Task/Issue URL: https://app.asana.com/0/488551667048375/1207235055389847/f
Description
Release directly to prod at 0.1% to minimise PS review time.
Steps to test this PR
Code review

#4514  👈
develop

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

Join @aitorvs and the rest of your teammates on  Graphite


fix: Move "coverage" key to top-level in codecov.yml

While trying to figure out why the annoying "annotations" keep showing up in our diffs (https://discord.com/channels/990354215060795454/1075857804692295720/1092933724003377203) I found a PR to another project which disabled annotations. It does this:
github_checks:
  annotations: false

We had an extra 2 spaces before annotations so removed those. I also noticed they had coverage at the top-level whereas we had it nested under codecov. So I updated ours to be the same. This is also what example from the docs does.
If this doesn't get rid of the annotations, I don't know what will ...
One other difference is that this other project has .github/.codecov.yml and we have just codecov.yml in the root. But the docs clearly say to put codecov.yml in the project root so not changing that.

Just to test this, merged in commits from https://github.com/fedimint/fedimint/pull/2109 which has a ton of annoying annotations. Will see if they disappear here.

This seems to fix it. First image is with annotation from https://github.com/fedimint/fedimint/pull/2109. Second image is after grabbing commits from https://github.com/fedimint/fedimint/pull/2109 on top of this codecov change. Going to drop those change and mark ready for review.


Kernels 2021-03-22

Motivation for this change
Updates
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[X] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X]  Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@ofborg test kernel-latest kernel-lts latestKernel.login kernel-latest-ath-user-regd latestKernel.hardened

Port to 20.09 in 6968ac2b07cac7909840254914f0e5c520787d27...eb73dded7d168750c08ee2d697de4bb15dbb01d8


Remove submodelserver deployment on int/dev environments

As ...
I want ...
so that ...
Hints / Details
Acceptance Criteria
Out of Scope

...

won't do


Accepting dynamic IDs?

Is it possible to accept a dynamic webhook ID?
I want to be able to accept a POST request like /hooks/path/to/{DYNAMIC_ID} of-course without knowing the DYNAMIC_ID ahead of time. Looking around I'm not sure if this is possible but would appreciate any resources that point in the right direction.

Not currently possible.  What system are you using that sends such a request?

CopyLeaks API

Sadly I think due to this issue and #421 I'll have to look at another webhook library for my needs

Thanks for the link.  I guess the CopyLeaks folks thought that was clever, but it really complicates the hook receiver.  Oh well.
Thinking out loud about how we might add support for dynamic hook paths like CopyLeaks...
#421 is a prerequisite only if you need multiple path segments.  Assuming you would in more cases.
I think we would need to allow the hook ID to contain mux path variables.  One downside: it could expose the mux path parser syntax to the user, which would make it harder to switch away from Gorilla later.
"id": "{status}/{scan_id}"

We could match that request, but each hook would likely need its own HTTP handler func since we wouldn't necessarily know which route the mux matched.
We would also need a way to pass the individual variables (status & scan_id) down the stack.
Hmm.  Sounds like a lot of refactoring.  Something to ponder...


UX for error messages to display tracing ID

UI/UX design for error messages to include tracing ID and any other useful information.
This addition to the error message will assist developers in tracing the issue and providing more guidance to users.
### Open Questions
- [ ] What is the standard format for the tracing ID?
- [x] Is it possible to provide a link or button to make it easier for users to share their issue with the appropriate ID?
- [ ] Any suggestions on what the rest of the message should say to improve the users understanding of the problem?

### Tasks
- [ ] Create new error message design to display tracing ID

@esaminu @hcho112 , can you help me with some of the open questions above? Basically just want to know what format the ID should be in issue-433 for example, and if it would make sense to allow users to report directly from the message to save time and avoid confusion.

From discussion with @DavidM-D, seems like this is more of a reference if the user decides to follow up rather than a get immediate help and clog our system with tickets. What subtext would make sense on the message to give the user a little more confidence their issue will be fixed.


Atmel SAM0 ADC not working

Describe the bug
Setting up analogs on a custom board using an ATSAMD21G part.
reduced DTS:
zephyr,user {
	io-channels = <&adc 0>;
};

&adc {
	#address-cells = <1>;
	#size-cells = <0>;
	status = "okay";

	channel@0 {
		reg = <0>;
		zephyr,gain = "ADC_GAIN_1";
		zephyr,reference = "ADC_REF_INTERNAL";
		zephyr,acquisition-time = <ADC_ACQ_TIME_DEFAULT>;
		zephyr,resolution = <12>;
		zephyr,input-positive = <0>;
		status = "okay";
	};
};

Through some debugging I've found that the adc_sam0_init function calls pinctrl_lookup_state which returns -ENOENT and causes everything to break.  so must be an issue with pinctrl?
Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using?

Atmel SAM0

What have you tried to diagnose or workaround this issue?

debugged code, it appears that the pinctrl function is not working

Is this a regression? If yes, have you been able to "git bisect" it to a
specific commit?
...

To Reproduce
Steps to reproduce the behavior:

configure ADC

Expected behavior
A clear and concise description of what you expected to happen.
Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)
Logs and console output
If applicable, add console logs or other types of debug information
e.g Wireshark capture or Logic analyzer capture (upload in zip archive).
copy-and-paste text and put a code fence (```) before and after, to help
explain the issue. (if unable to obtain text log, add a screenshot)
Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows)
Toolchain (e.g Zephyr SDK, ...)
Commit SHA or Version used

Additional context
Add any other context that could be relevant to your issue, such as pin setting,
target configuration, ...

Hi @cperkulator, thanks for the report... apologies for my delayed response.
Have you provided the pinctrl info as well? (example below)
&adc {
    #address-cells = <1>;
    #size-cells = <0>;

    status = "okay";

    pinctrl-0 = <&adc_default>;
    pinctrl-names = "default";

    adc_vbat: channel@6 {
        reg = <6>;
        zephyr,gain = "ADC_GAIN_1";
        zephyr,reference = "ADC_REF_VDD_1";
        zephyr,acquisition-time = <ADC_ACQ_TIME_DEFAULT>;
        zephyr,resolution = <12>;
        zephyr,input-positive = <6>;
        zephyr,vref-mv = <ADC_REF_VDD_1_MV>;
    };

    adc_vsys: channel@17 {
        reg = <17>;
        zephyr,gain = "ADC_GAIN_1";
        zephyr,reference = "ADC_REF_VDD_1";
        zephyr,acquisition-time = <ADC_ACQ_TIME_DEFAULT>;
        zephyr,resolution = <12>;
        zephyr,input-positive = <17>;
        zephyr,vref-mv = <ADC_REF_VDD_1_MV>;
    };
};

&pinctrl {
    adc_default: adc_default {
        group1 {
            pinmux =
                <PA6B_ADC_AIN6>,
                <PA9B_ADC_AIN17>;
        };
    };
};

Hi @cperkulator,
Did comments from @attie-argentum solve your issue?

!@#$% I just spend 2 days chasing down the same problem, no pinctrl! Doh! Added it in and it works.
None of the (sam) ADC sample overlays that I looked at had it, I just now see that some do and some don't.

Hi @PeeJay ,
Good to know that it worked. We encourage people to contribute so feel free to send overlays patches to help keep everything in shape. Those are the good ones to start contribute : )


[Feature Request]: Contextual Embeddings and Retrieval  (anthropic)

Feature Description
Anthropic has posted an impressive approach for reducing RAG errors by 67% and keeping costs reasonable in the process.
Blog post: https://www.anthropic.com/news/contextual-retrieval
Cookbook: https://github.com/anthropics/anthropic-cookbook/blob/main/skills/contextual-embeddings/guide.ipynb
Reason
To implement this in llamaindex would require some crafty adaptation into a Node Parser I’m guessing.. maybe even a whole new NP?
Value of Feature
from the anthropic blog post:
“ This method can reduce the number of failed retrievals by 49% and, when combined with reranking, by 67%. These represent significant improvements in retrieval accuracy, which directly translates to better performance in downstream tasks “

This is extremely similar to stuff like the Summary metadata extractor that we already have. The cool part is using prompt caching here
In any case, there's a notebook here for how to do this
https://docs.llamaindex.ai/en/stable/examples/cookbooks/contextual_retrieval/


Using new import path for go/html package

code.google.com is being 'put down', so it's best to start moving away. Cheers

Thanks!


supabase-cli: 1.50.13 -> 1.55.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for supabase-cli is: A CLI for interacting with supabase
meta.homepage for supabase-cli is: https://github.com/supabase/cli
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.55.0 with grep in /nix/store/v4yq8dvaapsbxwvbad0dm56qrs8ddjbc-supabase-cli-1.55.0
found 1.55.0 in filename of file in /nix/store/v4yq8dvaapsbxwvbad0dm56qrs8ddjbc-supabase-cli-1.55.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
supabase-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v4yq8dvaapsbxwvbad0dm56qrs8ddjbc-supabase-cli-1.55.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A supabase-cli https://github.com/r-ryantm/nixpkgs/archive/c4b8bb723a75427e4190f58f51bc6e4db7315d0a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v4yq8dvaapsbxwvbad0dm56qrs8ddjbc-supabase-cli-1.55.0
ls -la /nix/store/v4yq8dvaapsbxwvbad0dm56qrs8ddjbc-supabase-cli-1.55.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

supabase-cli

Maintainer pings
cc @Gerschtli for testing.

Result of nixpkgs-review pr 230054 run on aarch64-darwin 1

1 package built:

supabase-cli


Fix for material-textarea (#42 and #43)

#43 was not compatible with riot v3
Close issue #42

@Joxit Yes, thank you for fixes


GitHub flavored markdown

Just installed Markoff and am really liking it! Feels slick!
Not sure if at all feasible or a good idea, but it'd be very cool if GitHub flavored markdown were to be supported. Especially features like checklists.
Thanks for considering!

Pretty sure a lot of people will request this feature.
I'd like to stick to using cmark as the amrkdown parser, so if you really want your favorite feature to be implemented, head over here and lobby for it.


client.auth.user() is null when using next.js auth helper

Description
I do not know if it's normal behavior or not but when I'm using auth helper of next.js with the setup provided in the readme when I do client.auth.user() the result is null. But the right user is injected into the page via props when I'm connected.
So with using tools of auth helper to get the user it works but when using supabase client to get the user it does not work anymore

Ok, this seems normal, we should use getUser when we connect via access_token.


Update index search preference

Description
Update index search preference to _primary
Issues Resolved
https://github.com/opensearch-project/observability/issues/1562
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@ps48 this should be backported to 2.x as well, right?


Use AST to transform code

Currently the code is made async by direct manipulation of the code string. This is not robust, we should use the ast module instead.

Done in #11


[move-lang] Extending parser and expansion by generic spec vars.

This PR adds the syntax name<t1, .., tn> resp. mod::name<t1,..,tn> to the asts and syntax of move lang. Move prover needs to have this expression forms to deal with global specifications of generic resources (like the new generic coin type).
As with other expression constructs only allowed in specs, we parse this form and then reject it in expansion if it doesn't appear in a spec block.
There is some change in error reporting implied by this change. Previously, if we had name<t1,..,tn>, an error would have been produced that either a (..) or a {..} needs to follow. Instead, now we accept this during parsing, but reject it in expansion. The existing test cases for this have been taken over, but now produce different baseline output. Because those tests where also the only case where we test the hint 'Perhaps you missed a space before <' a new test case was added to cover this error.
Motivation
Global specifications
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Added tests.
Related PRs
NA

Hi, I had to update the PR to cover one more use case, namely generic variables in assignments (v<T> = E). This was handled the same way as now done for Name, by extending the existing Bind::Var variant, then rejecting the extended version in expansion if not in in a spec block.

You've addressed all of my concerns. Thanks! I'd still like Todd to look at the parts outside the parser, so I'll leave it for him to approve.

@bors-libra r=tnowacki

:pushpin: Commit c614fc0 has been approved by tnowacki

:hourglass: Testing commit c614fc0d4f3e28c2dcc831d377132511749af1f1 with merge a4fe6b659ae4c4a167df7754101aadcb9a5115be...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: tnowacki
Pushing a4fe6b659ae4c4a167df7754101aadcb9a5115be to master...


Enter key does not work to jump to tests, test results, or code coverage results

The Enter key and numpad Enter key do not work to jump to tests, test results, or code coverage results. In order to open these items from the AxoCover window, a programmer currently has to context switch away from the keyboard-focused zone of productivity, grab the mouse, double click, release the mouse, then context switch back to the keyboard.
Tests:

Test results:

Code coverage results:

On master.

Released.


feat(gitbrowse): add separate getremotes

Description
Similarly to opening in github, I would like to be able to yank the current url. For that I thought that allowing to get the remotes would be useful.
Related Issue(s)
(not filed / tried) In the yank case at least for me, it'd be nicer to yank the file url without the line and in visual mode to yank the current line or range. I can yank the current file with this, but it's a bit inconvenient:
print(Snacks.gitbrowse._getremotes({ url_patterns = { ["github%.com"] = { file = "/blob/{branch}/{file}" } } })[1].url)

Screenshots

You can already do this:
Snacks.gitbrowse({ open = function(url) vim.fn.setreg("+", url) end })


chore: Do not accept similar code with more than 10 lines in files

Signed-off-by: Tibor Dudlák<EMAIL_ADDRESS>

Can do


Cannot read property '*' of undefined / using property of var inside includes

I use Pug in two ways. For development I run Express and Pug as view engine and for distribution I use gulp-pug. Works perfectly fine till today.
I tried to run gulp-pug on a file which includes the following concept:
- var partialData = [{title: "Partial 1"}, {title: "Partial 2"}];

each item in partialData
	span=item.title
	include ../partials/partial

and partials/partial.pug includes only:
span=item.title

Works perfectly fine in development, but fails with gulp-pug
events.js:160
      throw er; // Unhandled 'error' event
      ^
 TypeError: C:\D-drive\_vsp\Boilerplate-Frontend\dev\html\components\partials\pa
rtial.pug:1
  > 1| span=item.title

Cannot read property 'title' of undefined
    at eval (eval at wrap (C:\D-drive\_vsp\Boilerplate-Frontend\dev\node_modules
\pug-runtime\wrap.js:6:10), <anonymous>:7:61)
    at template (eval at wrap (C:\D-drive\_vsp\Boilerplate-Frontend\dev\node_mod
ules\pug-runtime\wrap.js:6:10), <anonymous>:7:118)
    at DestroyableTransform.compilePug (C:\D-drive\_vsp\Boilerplate-Frontend\dev
\node_modules\gulp-pug\index.js:33:49)
    at DestroyableTransform.Transform._read (C:\D-drive\_vsp\Boilerplate-Fronten
d\dev\node_modules\readable-stream\lib\_stream_transform.js:159:10)
    at DestroyableTransform.Transform._write (C:\D-drive\_vsp\Boilerplate-Fronte
nd\dev\node_modules\readable-stream\lib\_stream_transform.js:147:83)
    at doWrite (C:\D-drive\_vsp\Boilerplate-Frontend\dev\node_modules\readable-s
tream\lib\_stream_writable.js:313:64)
    at writeOrBuffer (C:\D-drive\_vsp\Boilerplate-Frontend\dev\node_modules\read
able-stream\lib\_stream_writable.js:302:5)
    at DestroyableTransform.Writable.write (C:\D-drive\_vsp\Boilerplate-Frontend
\dev\node_modules\readable-stream\lib\_stream_writable.js:241:11)
    at write (C:\D-drive\_vsp\Boilerplate-Frontend\dev\node_modules\vinyl-fs\nod
e_modules\readable-stream\lib\_stream_readable.js:623:24)
    at flow (C:\D-drive\_vsp\Boilerplate-Frontend\dev\node_modules\vinyl-fs\node
_modules\readable-stream\lib\_stream_readable.js:632:7)

I hope someone can help me out.

Found out what was wrong. Gulp tried to compile the partial.pug as a standalone file, but ofcourse it has no variables / locals set in that case. So it fails.
I want to distribute also partials for our back-end party; so they can easily check and implement the small html components without struggeling into the plain html template.
I created a fallback with dummy data.
'''jade
if item === undefined
- var item = {title: "Title"}
'''
Works like a charm.

However you are using gulp, you can try ignoring the specific file, i.e. gulp.src(['src/**.pug', '!src/partial.pug']).  Glad you solved it.


Patch 1

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

Hi,
Thanks for the PR.
I've switched this PR to point at @katarzyna-koltun-mx 's original PR.
Is this likely to be published this week, or can it wait until Kasia is back from leave?
Feel free to push more changes to this branch if there is anything else needed.
Could you also get @mx-awsdevteam to sign the CLA so that we get a clean oversight of what work is needed.
Thanks
Mark

@mx-awsdevteam
Can you let me know when this PR is ready to be merged and I'll do a final review.
Thanks

The purpose of this PR is to review documentation that will first be published on the Marketplace. I will keep it open until the Marketplace record is published, then I will merge into https://github.com/mendix/docs/pull/6401/files.


Wrap migration in a transaction

See https://github.com/blueapron/data-migrate/pull/1 for details and discussion

Changes look good to me. :+1:

I'll create a branch to reference this in ba and verify this solves the problem with migration log output before I merge

:+1: nice stuff.


DII dev

Proposed changes
Small improvements to DII functions and example notebook

Codecov Report
Attention: Patch coverage is 90.66667% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 81.34%. Comparing base (4ab6d46) to head (6958b5f).

Files
Patch %
Lines

dadapy/_utils/differentiable_imbalance.py
89.79%
5 Missing :warning:

dadapy/feature_weighting.py
92.30%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
+ Coverage   81.32%   81.34%   +0.01%
==========================================
  Files          20       20
  Lines        3299     3302       +3
==========================================
+ Hits         2683     2686       +3
  Misses        616      616

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


Refactor the authorization instrumenter

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines         295    303    +8
=====================================
+ Hits          295    303    +8

Impacted Files
Coverage Δ

lib/graphql-pundit/instrumenters/authorization.rb
100% <100%> (ø)
:arrow_up:

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 b62c6c1...426e417. Read the comment docs.


[AIRFLOW-XXX] Fix PythonVirtualenvOperator tests

The recent update to the CI image changed the default python from python2 to python3. The PythonVirtualenvOperator tests expected python2 as default and fail due to serialisation errors.
Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3968 into master will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #3968      +/-   ##
=========================================
+ Coverage   74.69%   74.9%   +0.21%
=========================================
  Files         200     200
  Lines       15974   15974
=========================================
+ Hits        11932   11966      +34
+ Misses       4042    4008      -34

Impacted Files
Coverage Δ

airflow/models.py
88.62% <0%> (-0.05%)
:arrow_down:

airflow/www/views.py
68.85% <0%> (+0.06%)
:arrow_up:

airflow/www_rbac/views.py
72.04% <0%> (+0.14%)
:arrow_up:

airflow/bin/cli.py
64.59% <0%> (+0.23%)
:arrow_up:

airflow/www_rbac/app.py
97.77% <0%> (+1.11%)
:arrow_up:

airflow/task/task_runner/base_task_runner.py
79.66% <0%> (+1.69%)
:arrow_up:

airflow/hooks/dbapi_hook.py
79.67% <0%> (+3.25%)
:arrow_up:

airflow/hooks/hdfs_hook.py
92.5% <0%> (+60%)
:arrow_up:

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 010924a...8069817. Read the comment docs.

Awesome job @johnhofman . Thanks for chipping in and fixing this.


New Adoptable Pets

https://www.petfinder.com/developers/api-docs

This issue was moved to office-of-civic-innovation/IFTTT-Smart-Louisville#4


feat: update profile settings

Summary
Please summarize your changes, describing what they are and why they were made.
Changelog
Please write an accurate and concise changelog for your changes.

Relevant Issues
Closes #3213
Closes #3214
Closes #3215
Type of Change
Please select any type below that is applicable to your changes, and delete those that are not.

[ ] Breaking - any change that would cause existing functionality to not work as expected
[ ] Chore - refactoring, build scripts or anything else that isn't user-facing
[ ] Docs - changes to the documentation
[ ] Fix - a change which fixes an issue
[ ] New - a change which implements a new feature
[ ] Update - a change which updates existing functionality

Testing
Platforms
Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[x] Linux
[ ] Windows

Mobile

[x] iOS
[x] Android

Instructions
Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.
Checklist
Please tick all of the following boxes that are relevant to your changes, and delete those that are not.

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

I merged the bottom navigation fix, so maybe the TransactionTabs conflict can be overwritten with the current state on develop?
Apart from that, just two minor things
Edit: We have to add the related issues.

The issues with locales should be solved once 1.5 will be merged, we should finish this PR asap, I just added a new task for this.
Also I added the latest TransactionTabs updates from the latest alpha release.


Check trace and span ID lengths. Warning if incorrect

We know from the opentelemetry specification that:

A trace ID must be 16bytes (32 characters) long
A span ID must be 8bytes (16 characters) long

If trace and / or span ID are provided, logpusher should warn when the user provides input that does not equal those lengths.
In other words, if a user tries to send a trace ID that is anything other than 32 characters long, logpusher should warn them because we know an error will occur during the push to the collector.
Similarly, if a user tries to send a span ID that is anything other than 16 characters long, logpusher should warn.
Examples
# Trace ID is too short (31 characters). Collector will fail to accept it.
% docker run gardnera/logpusher:v0.1.0 -ep http://<IP_ADDRESS>:4318 -c "This is my log line" --trace-id "1234567890123456789012345678901"
<Response [400]>

# This trace is 32 characters long. Collector will accept it.
% docker run gardnera/logpusher:v0.1.0 -ep http://<IP_ADDRESS>:4318 -c "This is my log line" --trace-id "12345678901234567890123456789012"
<Response [200]>

# This span ID is the correct length (16 characters). Collector will accept it.
% docker run gardnera/logpusher:v0.1.0 -ep http://<IP_ADDRESS>:4318 -c "This is my log line" --span-id "1234567890123456"
<Response [200]>

# This span ID is only 15 chars long. Collector will fail to accept it.
% docker run gardnera/logpusher:v0.1.0 -ep http://<IP_ADDRESS>:4318 -c "This is my log line" --span-id "123456789012345"
<Response [400]>

Documentation pages should also be included to leave a hint as to this "rule".

@MilijanBogdanovic this is basically the same fix as you did for tracepusher. Fancy taking it on?

Hey Adam,
Yes of course sorry for the late reply I was on vacation.
Will get it done in a day or two, if thats ok
@agardnerIT

Hey @agardnerIT
Just a follow up for this, sorry for the late reply, I was/ am unable to patch this in the next 7,8 days, as my laptop
is away from me. However, I will try with another PC, and will probably just patch this in 3,4 days, if that is not too late for you to wait.
Once again sorry for the delayed message and update

No worries at all. Take your time. No rush :)

Hey @agardnerIT,
Sorry for being late on this!
Please find the my PR attached on this repo, with the simple change and doc update.
LMK if you envisioned this a bit differently or wanted smth else, more than happy to change.
Thanks!

Thanks! All good! Merged.


[BUG]: Scan and Relay Satellites Mission Control Bug

Which Mod?
Scan & Relay Satelites
Description of the bug/crash
The mission control station extend functionality appears to be broken, the dupes will get an errand calling them to the station for a research task but they never get a progress bar and the rocket never gets the boost. They are repeatedly called for this errand and essentially just end up idling in front of the mission control station. I have tried replacing the satellites in several different locations and rebuilding the mission control station but the same behavior persists. Looking at the log it is repeatedly trying to add a satelite node every several ticks which is probably where this is failing, I am running quite a few mods so there definitely could be a conflict.
Steps To Reproduce
Place Satellite and mission control station, fly rocket within range of satellite
Player.log
Player.log

thx for the report, there was an additional range check in the workable I missed, causing the worker task to become invalid
fixed in 1.2.3


Removed the cap of level difference when Ingenuity is used.

When Ingenuity is used it caps the level difference value to between -5 and 20. I am not really sure why this is the case, especially since level difference isn't capped at all when it isn't used. This can produce weird bugs on recipes far below your level, like causing adding Ingenuity reducing how much progress you gain.
The -5 below level cap might be accurate, but I don't know when you are able to craft recipes that you will still be 5 levels below the post Ingenuity value.

Also added another commit which fixed the implementation of Aspected recipes.  Not just a bonus 100% on a matching brand.

With regards to the Ing level difference caps, that's how it used to work -- the caps were necessary to make the formulas work at extremes.

Have you been able to verify that removing the level caps on Ingenuity improves the accuracy at large level differences?

Well the above level cap definitely isn't in place, Its easy to test, craft a level 1 recipe at max level with and without Ingenuity and they give the same progress values.
For the -5 below level cap. Looking at the table the only place it was relevant was a level 50 crafting 3 and 4 star recipes. Besides that I don't see a place where it applies, since Ingenuity now drops the recipe level enough. So probably should leave that in until can get contradicting data.

Looks good to me. I like the way you reworked the elemental aspect check. Ready for merge?

Yea it's good to merge.


Update Building.md

Really? the system provided jsoncpp still doesn't work? I spent like a week on the findjsoncpp trying to get it just right so that even ancient jsoncpp shipped with distros worked fine so people didn't have to do this nonsense anymore...

Really, at least when I tried it.


Missing compatible arch even though compatible arch is there.

Hello, everyone.
I'm receiving this weird "incompatible architecture" error when I look at the console:
Error loading /Library/CoreMediaIO/Plug-Ins/DAL/CMIOMinimalSample.plugin/Contents/MacOS/CMIOMinimalSample:  dlopen(/Library/CoreMediaIO/Plug-Ins/DAL/CMIOMinimalSample.plugin/Contents/MacOS/CMIOMinimalSample, 0x0106): missing compatible arch in /Library/CoreMediaIO/Plug-Ins/DAL/CMIOMinimalSample.plugin/Contents/MacOS/CMIOMinimalSample
However, when I run lipo -info on the file, I get:
Architectures in the fat file: CMIOMinimalSample.plugin/Contents/MacOS/CMIOMinimalSample are: x86_64 arm64
Which should cover all arches supported by macOS.
Anyone ever encounter anything similar? I've tried chowning the entire .plugin to be root owned and 777 permissions, but to no avail.

To anyone experiencing this: on ARM MacBooks, I'm not getting the usual library validation errors, but it appears to be just that. Trying on a custom-wrote software using dlopen to open the library inside the plugin, it gave a more correct error message and testing on x86 confirmed that this was the issue.
Does this merit an update to the README? I'd be up to writing and PR-ing it @johnboiles

I have the same issue, plugin can not be loaded in arm64 Mac, can any one help?

I compiled (PTPWebcam)[https://ptpwebcam.org] for my M1 Mac Mini and run into this Issue as well. I can only say that it fixed itself (aside of the usually validation issue) after I change some Xcode settings. I change the build mode to "Any Mac" and compiled in Release mode instead of Debug the console started to show the validation error instead of the arch error. Maybe try this here as well.


DLP-1223: Add DLP dataset support

thanks for the PR but due the number of known internal issues with terraform-plugin-sdkv2 (most notably handling of zero values), we are no longer recommending using it and instead, advise using terraform-plugin-framework exclusively for all future data sources and resources. can you  port this over please?

Sure, where should the new data source live? A directory under https://github.com/cloudflare/terraform-provider-cloudflare/tree/master/internal/framework/service?

yep! some prior art - https://github.com/cloudflare/terraform-provider-cloudflare/pull/2961

acceptance tests all looking good now after taking into account empty datasets.
TF_ACC=1 go test ./internal/framework/service/dlp_datasets -v -run "^TestAccCloudflareDLPDatasets_" -count 1 -timeout 120m -parallel 1
=== RUN   TestAccCloudflareDLPDatasets_DataSource
=== PAUSE TestAccCloudflareDLPDatasets_DataSource
=== CONT  TestAccCloudflareDLPDatasets_DataSource
--- PASS: TestAccCloudflareDLPDatasets_DataSource (6.41s)
PASS
ok  	github.com/cloudflare/terraform-provider-cloudflare/internal/framework/service/dlp_datasets	7.369s


Fixed all bug

Related Issuse
fix #1607
Describe the changes you've made
One div was not closed in index.html and semicolon was not closed in complexlogic.js and added nth term of hp in AP,GP,HP section
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] My changes generate no new warnings.

@sairish2001 Resolved all conflicts :)

@sairish2001 actually its button was removed so i moved it to its right place i have deleted it and added in Sum of nth term of AP,GP,HP

@sairish2001


Fix for issue 24

This pull request fixes issue 24.
The first iteration of the function update_mesh() calls calculate_error() before the border state has been assigned. Therefore, the behavior of det != 0 && !border is not predictable.
The updated code simply changes the order of operations: identifying borders prior to calculating the initial edge error terms.

Happy New Year!
Thank you for the fix, I merged it.

Happy New Year!
Thank you for the fix, I merged it.


Referenced Class OrderNotFoundException does not exist

Shopware v<IP_ADDRESS> - Symfony Flex template
If the Webhook requests a Order which does no longer Exist the following Error will be returned:
Attempted to load class OrderNotFoundException from namespace Shopware\Core\Checkout\Cart\Exception.
Did you forget a use statement for Swag\PayPal\OrdersApi\Administration\Exception\OrderNotFoundException
This is because the referenced Class no longer exists in Shopware Core.

Dear memoworker
We will check this.
Best regards


compile issus!

Hello, everyone!!
I have an issue during building the repo.

ubuntu 16.04
cuda 8.0
caffe2 is working well in python

bg@bg-CGI:~/Desktop/caffe2_cpp_tutorial$ mkdir -p build
bg@bg-CGI:~/Desktop/caffe2_cpp_tutorial$ cd build
bg@bg-CGI:~/Desktop/caffe2_cpp_tutorial/build$ cmake ..
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found Protobuf: /usr/local/lib/libprotobuf.so
../caffe2: warning: directory does not exist.
-- Configuring done
-- Generating done
-- Build files have been written to: /home/bg/Desktop/caffe2_cpp_tutorial/build
bg@bg-CGI:~/Desktop/caffe2_cpp_tutorial/build$ make
Scanning dependencies of target caffe2_cpp
[  1%] Building CXX object CMakeFiles/caffe2_cpp.dir/src/caffe2/util/net_operator.cc.o
In file included from /usr/local/include/caffe2/core/logging.h:11:0,
                 from /usr/local/include/caffe2/core/blob.h:13,
                 from /usr/local/include/caffe2/core/operator.h:12,
                 from /home/bg/Desktop/caffe2_cpp_tutorial/include/caffe2/util/net.h:4,
                 from /home/bg/Desktop/caffe2_cpp_tutorial/src/caffe2/util/net_operator.cc:1:
/usr/local/include/caffe2/proto/caffe2.pb.h:230:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:353:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:665:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:863:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:976:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:1140:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:1253:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:1483:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:1674:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:1950:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:2180:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:2458:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:2602:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:2800:10: error: ‘PROTOBUF_CONSTEXPR’ does not name a type
   static PROTOBUF_CONSTEXPR int const kIndexInFileMessages =
          ^
/usr/local/include/caffe2/proto/caffe2.pb.h:6878:109: error: expected class-name before ‘{’ token
 template <> struct is_proto_enum< ::caffe2::TensorProto_DataType> : ::google::protobuf::internal::true_type {};
                                                                                ^
/usr/local/include/caffe2/proto/caffe2.pb.h:6883:99: error: expected class-name before ‘{’ token
 template <> struct is_proto_enum< ::caffe2::DeviceType> : ::google::protobuf::internal::true_type {};
                                                                                ^
CMakeFiles/caffe2_cpp.dir/build.make:62: recipe for target 'CMakeFiles/caffe2_cpp.dir/src/caffe2/util/net_operator.cc.o' failed
make[2]: *** [CMakeFiles/caffe2_cpp.dir/src/caffe2/util/net_operator.cc.o] Error 1
CMakeFiles/Makefile2:256: recipe for target 'CMakeFiles/caffe2_cpp.dir/all' failed
make[1]: *** [CMakeFiles/caffe2_cpp.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

need helps!!! plz!

I have solved it!!!
Just uninstall protobuf(version 3.5.1), then install 3.5.0.
It works well.


Dashboard Refactor - Move Distributions by County functionality [#3722]

Resolves #3722
Description
Small change to move "Distributions by County" button from dashboard tile to the side nav under "Reporting & Auditing". It also adds a systems test to ensure the link appears for org admins, fixes tests that assumed the previous location, and makes the submenu + available reports visible for non-admin users.
No new dependencies required, and no backend changes.

Type of change

Bug fix (non-breaking change which fixes an issue) (really a UI change...)

How Has This Been Tested?
Verified both visually (see screenshot) and via systems specs (one extra test for "Distributions by County" menu item, and verified that other Distributions by County report specs still pass.)

Screenshots

Per discussions with @awwaiid,  the making reports visible to non-admin is something that is definitely only an intermediate solution.


Added missing invalidate parameter to onValidSubmit/onInvalidSubmit

Fix #77

Hi @Semigradsky, thanks for the pull! I chose to handpick and implement myself, but you will get credit for it, thanks for the pull! ;-)

:+1:


How to install it?

no clue

It probably needs some more documentation, but mhsendmail should be installed in the same way as MailHog, e.g. using the binary release from the releases page, or by using go get github.com/mailhog/mhsendmail if you have go installed

I think it's a good idea to follow this Rules
http://githubengineering.com/scripts-to-rule-them-all/
So it's easier to use this project for others.

It appears that the code has been updated more recently than the releases. Any chance of getting a new release fileset?

This is still really unclear/difficult for someone who is not experienced in doing things like this. (like me)
Installing it the same way as mailhog: brew install mhsendmail: didn't work, doesn't exist...
Installing it with go: nothing happend, binary is not available, not found in command line...
Downloading the binary file from the releases: great, but ... which one do I need, where do I place it?
Follow the rules at: http://githubengineering.com/scripts-to-rule-them-all/: Didn't even bother with this option, just looks like complete alien language to me... :)
Should really be made much easier and documented better.

Here's an example https://gist.github.com/shakhmehedi/bbeb643079171be322eb6e9d08c7c020
On a CentOS env. install it with:
yum install go
go get github.com/mailhog/mhsendmail
ln  ~/go/bin/mhsendmail /usr/bin/mhsendmail
mhsendmail -h

HTH
Adrian

yum install go
go get github.com/mailhog/mhsendmail
ln  ~/go/bin/mhsendmail /usr/bin/mhsendmail
mhsendmail -h

I had to add a hyphen s to the ln command:
sudo ln -s ~/go/bin/mhsendmail /usr/bin/mhsendmail

How to install it on Mac OS X? Installing Mailhog is okay via brew

Dark.local ➜  ~ go get github.com/mailhog/mhsendmail
fatal: not a git repository (or any of the parent directories): .git

not a Go user:
Installing MailHog was fine, but here go get github.com/mailhog/mhsendmail returns with nothing, and I dont have mhsendmail in /usr/bin

comment installer mailhog et sendmail sur windows
sans oublie la configuration

How to install on ubuntu?

Use this to install the current HEAD state of this repository:
go install github.com/mailhog/mhsendmail@9e70164f299c9e06af61402e636f5bbdf03e7dbb

It includes the fix for the environment variables #25
Use this snippet to include the latest mhsendmail in your container:
FROM golang:latest AS builder
WORKDIR /go/src
RUN go install github.com/mailhog/mhsendmail@9e70164f299c9e06af61402e636f5bbdf03e7dbb

then, later on:
COPY --from=builder /go/bin/mhsendmail /usr/bin/sendmail

For me, this was the missing documentation to get mhsendmail working as a sendmail drop-in replacement.

With most of the comments from the last two years in this 5+ year old issue I wonder if it would be better to use something else.

For me, I have to type in
$ go install github.com/mailhog/mhsendmail@latest

And then get this output
go: finding module for package github.com/ogier/pflag
go: found github.com/ogier/pflag in github.com/ogier/pflag v0.0.1

But... it doesn't seem to put it anywhere on my system, so I'm at a loss of what to do. Any help would be appreciated!

OK I got it, and maybe this will help other people. It doesn't put it in a /go/ folder, it creates a go folder in your user folder. So I had to modify the php.ini file on my local setup to use that path:
sendmail_path = "/Users/phoopee3/go/bin/mhsendmail"
This article helped me figure it out: https://jonchristopher.us/blog/mylocaldev-part-3-mailhog-mhsendmail-os-x/

My notes:

To catch emails on localhost.
brew install mailhog
Follow this guide: MyLocalDev Part 3: MailHog and mhsendmail on OS X - Jon Christopher with the following changes:

When installing Go using HomeBrew, change the GOPATH to $HOME/.go. Refer to older .bash_profile file.
Install mhsendmail using go install github.com/mailhog/mhsendmail@latest
In php.ini, apart from adding the sendmail_path, comment out the SMTP and smtp_port options that are for Win32 only.

Then run source ~/.bash_profile and brew services restart httpd


when i upgrade answer from 1.1 to 1.2.1，a question happend，【check table not exist】

Describe the bug
when i upgrade answer from 1.1 to 1.2.1RC1，a question happend，【check table not exist】
when i view my site，i got 50x
To Reproduce
Steps to reproduce the behavior:

upgrade answer form  1.1 to 1.2.1RC1，i use docker & sqlite

Expected behavior
ok
Screenshots
Logs：
[upload-dir] try to install...
[upload-dir] install success, upload directory is /data/uploads
[i18n] try to install i18n bundle...
[i18n] find i18n bundle 42
install all initial environment done
config file exists, try to read the config...
config file read successfully, try to connect database...
check table not exist
[SUCCESS] answer installation service will run at: http://localhost:80/install/
Platform (please complete the following information):

Device: docker&sqlite
OS: linux
Browser all
Version 1.2.1

this is my config.yaml
debug: false
server:
  http:
    addr: <IP_ADDRESS>:80
data:
  database:
    driver: sqlite3
    connection: /root/data/docker_data/answer/answer.db
  cache:
    file_path: /data/cache/cache.db
i18n:
  bundle_dir: /data/i18n
service_config:
  upload_path: /data/uploads
swaggerui:
  show: true
  protocol: http
  host: <IP_ADDRESS>
  address: :80

the reson is the db config should use container's path, not host path.


unable to use ngAnimate

I have added the ngAnimate dependency and angular-animate.min.js(v-1.4) as well but the classes on the slider DOM element are not added or removed by the ngAnimate. please help me to enable animation on your module.
Thanks!
Tarun

ngAnimate require on angular-slider
//in the module
var sotosAngularSlider=angular.module("sotos.angular-slider",["ngAnimate","sotos.tmp"])
in your js
var app = angular.module('app', ['ngAnimate','sotos.angular-slider','ngSanitize']);
check the demo files if helps you
thanks

We have already added this in both the module as well as our js file but nothing is happening!
We are at lost and even replaced the ng-animate js file but of no help either. Is there anything else we are missing out?
It would be a great help if you could kindly zip the working demo.

check full source


No tests found related to files changed since last commit.

Hi!
I've installed the plugin, made sure I got this in package.json:
"test": "jest --watchAll --config node_modules/ca-ui-module-test/__jestDist__/jest.config"
But the runner keeps telling me all the time:
No tests found related to files changed since last commit.
Press `a` to run all tests, or run Jest with `--watchAll`.

I can do yarn test and everything works normally. Am I missing something? What should I look for? Is there anything I can activate that gives more more info?
Thanks!

Just to confirm, you understand how the watch mode works ( in that it only looks at tests related to the current git diff)?
So that I know whether this is a configuration problem or not

I just recently upgraded vscode to 1.8.1, and I am getting a similar response when I save my test files.
Here is my launch.json
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Tests",
            "type": "node",
            "request": "launch",
            "program": "${workspaceRoot}/node_modules/jest-cli/bin/jest.js",
            "stopOnEntry": false,
            "args": ["--runInBand"],
            "cwd": "${workspaceRoot}",
            "runtimeArgs": [
                "--debug-brk",
                "-i",
                "--colors"
            ],
            "env": {
                "NODE_ENV": "development"
            },
            "console": "internalConsole",
            "sourceMaps": true,
            "outFiles": ["${workspaceRoot}/dist/"]
        }
    ]
}

Here are my scripts from package.json:
"scripts": {
    "build": "npm run clean && webpack -p --config ./webpack.config.prod.js",
    "clean": "rm -rf ./dist",
    "start": "webpack-dev-server --config ./webpack.config.dev.js",
    "start:prod": "npm run build && NODE_ENV=production node ./src/server",
    "test": "jest",
    "test:watch": "jest --watch",
    "test:coverage": "jest --coverage",
    "test:coverage:ci": "jest --coverage --runInBand"
  }

I'm also experiencing this with VS Code Version 1.13.1.
You can test with a new app created with https://github.com/wmonk/create-react-app-typescript.
The first screenshot shows that the file is seen by Git as dirty, but the VSCode-Jest does not register the changes.
Running the command straight from terminal (second screenshot) run the appropriate test.
Thanks for the wonderful plugin, and let me know if I can help in further debugging this issue.

^ This is like an issue at the Jest level, this extension is only running jest --watch with some extra CLI args at this pint

@orta It was my mistake. I had to change pathToJest inside the workspace settings.
I initially didn't realize that I could set it to a command.
"jest.pathToJest": "npm test --"

If you'd like, I can make a PR making a note of this in the README.
Thanks again for your time!

Sure, yeah - better docs is always a great improvement!

Closing this stale issue. The Troubleshooting section of the README mentions pathToJest.

npm test a


[Cinder] report on empty backends

This patch adds:

The ability to add custom config items for each collector.
See the example config file for cinder collector config entries.

Allow cinder to have a list of configurable expected backends to look
for in the scheduler stats API call.  If there is a backend that is
expected to have stats, but isn't seen then the backend is down and
is reported as having 0 capacity.

The cinder collector now handles backends that don't support sharding
as needed by the kvm testing in qa-de-1.

A side note: These changes will detect not reported backends in a shard only if at least one backend in the shard is still reported. It's an improvement, but there's still a blind spot left, which we cannot solve by just looking at Cinder-reported data.

I think we would need to generate a list of shards in each region and have that list of shards get populated in the config file, so that the exporter knows what shards to expect to get reported.


proposal: net/mail: interpret comment after address as display name

In https://github.com/golang/go/issues/21257, we implemented skipping comments, resulting in e.g<EMAIL_ADDRESS>(Foo Bar)” parsing into Address{Name: "", Address<EMAIL_ADDRESS>I’d like to propose interpreting comments as display name, as various fairly common mail user agents still use them like that these days. Here are a few examples:

From<EMAIL_ADDRESS>(Marco d'Itri)
Date: Thu, 5 Oct 2017 14:42:30 +0200
User-Agent: NeoMutt/20170609 (1.8.3)
From<EMAIL_ADDRESS>(Debian Bug Tracking System)
Date: Sat, 07 Oct 2017 05:21:04 +0000
From<EMAIL_ADDRESS>(Yuriy M. Kaminskiy)
Date: Tue, 01 Aug 2017 22:42:55 +0300
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux)

Thoughts?
cc @minaevmike, who worked on this code last

I assume this would be a fallback - so Foo <email> (Bar) would parse as {Name: "Foo", Address: "email"}. Or do you have something else in mind?
Are there any cases where the name is missing and the comment is used for a different purpose?
I am also wondering if it would be useful to somehow make this difference visible to the developer. Otherwise, Foo <email> and <email> (Foo) would seem the same when parsing.

I assume this would be a fallback - so Foo  (Bar) would parse as {Name: "Foo", Address: "email"}.

Correct

Are there any cases where the name is missing and the comment is used for a different purpose?

I haven’t seen any so far, but of course that doesn’t mean such cases don’t exist.

I am also wondering if it would be useful to somehow make this difference visible to the developer. Otherwise, Foo  and  (Foo) would seem the same when parsing.

I’m not sure why a developer would need to concern themselves with this subtle detail of email parsing? :)

My knowledge of email address formats is limited, so I was simply trying to clarify the proposal a bit and see if there would be any noticeable disadvantages to accepting it.
It also seems like putting display names in the comments has been discouraged since 1982:
https://wordtothewise.com/2014/12/friendly-email-addresses/:

Never do this. The text in parentheses isn’t really a display name at all, rather it’s a human readable comment. Many MUAs will display that comment as though it were the display name, but it’s not been the right way to include a display name at least since RFC 822 was released in 1982.

RFC: http://tools.wordtothewise.com/rfc/822
Nevertheless, if this practice is still common, it seems okay to me to support it as a fallback.

Yep, I’m aware of the RFC :). We can of course also file bugs against mutt, neomutt, gnus and debbugs to use a more modern addr-spec, but that won’t fix the messages in my archives, which is why I’m starting to work on this end of the issue first.

That makes sense. I guess the only drawback of doing it would be that we'd be encouraging the bad behavior - but it's clearly too late for that :)
The point about being able to use email archives is useful, though. golang.org/s/owners just lists @bradfitz as maintainer - any thoughts, Brad? I realise the package is frozen, but this would not increase or change its API.

Wouldn't be dangerous to put in Address.Name something that is sometimes used as the display name?
Even if ( ... ) is used for the name, say, 90% of the time, we would be returning a completely bogus value in Address.Name in the remaining 10%.
I mean, it would be pretty reasonable to assume that Address.Name != "" implies that the parsed address has form display-name <addr>, has a name in it, and I can retrieve that name at Address.Name.
This is true now, but if we start parsing
<EMAIL_ADDRESS>(comment)"
as
Address{Name: "comment", Address<EMAIL_ADDRESS>that assumption breaks. Address.Name is not empty, but we just parsed an email address with no name in it.

Wouldn't be dangerous to put in Address.Name something that is sometimes used as the display name?

RFC822 states “Many MUAs will display that comment as though it were the display name”.
I understand this as: one should expect comments in the From header to commonly be interpreted as display name, provided no actual display name is present in the addr-spec.
I can’t quite see any danger in implementing what an RFC states is common practice, and only affects a fallback code path. Even in the unlikely event that this case actually exists anywhere (do you have any real-world examples of this edge case?), I would maintain that treating such a comment as display name is actually sensible — most likely it will identify the author of the message.

If, when present, the ( ... ) comment is pretty much always intended to be used as display-name, I guess it could be reasonable to implement your proposed change. I don't have any hard data about how frequently this assumption does not hold in the wild.

Per @golang/proposal-review, if there's just one comment after the address, sure, let's make it the Name. CL welcome for Go 1.11.

Thanks! I’ll try to send a CL tomorrow.

It looks like handling comments at all is new in Go 1.10. Given that, we might as well fix this now and avoid having a release that handles but drops comments.


[BUG] Hamburger Menu is not working properly

Description
Describe the bug:
A clear and concise description of what the bug is.
Expected behavior:
A clear and concise description of what you expected to happen.
Actual behavior:
A clear and concise description of what actually happened.
Reproduction Steps
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected result:
A clear and concise description of what you expected to happen at each step.
Actual result:
A clear and concise description of what actually happened at each step.
Screenshots

If applicable, add screenshots to help explain your problem.
Additional Information
Device (please complete the following information):

OS: [e.g., Windows, macOS, Linux]
Browser [e.g., Chrome, Safari, Firefox]
Version [e.g., 22]

Additional context:
Add any other context about the problem here.

Go for it!

Go for it!
but please synk with current changes in repo

ok,sure

Hello maam  i will be very grateful if i get the oppurtunity to work on this.Please assign me on this.


Change injector and sentry to GET daprsystem Configuration

In Kubernetes mode, when the Injector is patching a Pod to determine whether mTLS is enabled and Sentry on startup] fetches the global daprsystem Configuration, they do so by listing all Configurations in all namespaces and then match on the first Configuration with the name daprsystem. As Namespaces are sorted alphabetically when listed, the Configuration chosen by these services may not be the one located in the Dapr System namespace. This means that a malicious actor, or by accident a user of a Dapr enabled Kubernetes cluster, with write permissions to Configurations in a namespace which is alphabetically higher than the Dapr system namespace is able to override the global config for Sentry and Injector. We can expect that users of Dapr in Kubernertes would be able to have permissions to Configurations in order for them to control their Dapr deployment configuration.
PR updates the injector and sentry services to GET the daprsystem Configuration in the Dapr control plane namespace.
Fixes #7114

@JoshVanL Do we need this PR, Can't we merge 1.12 back into master instead?

@JoshVanL can 1.12 branch be merged back into master for this?

@mukundansundar sure thing.


Loading vectors - exception ValueError?!

Adding new language, created vector file with word_vectors.py, everything seems ok on file level.
When trying to create Vocab with init.py, this error pops up on vectors part.
  File "init.py", line 121, in write_vectors
    write_binary_vectors(vectors_src.as_posix(), (dst_dir / 'vec.bin').as_posix())
  File "spacy/vocab.pyx", line 565, in spacy.vocab.write_binary_vectors (spacy/vocab.cpp:12199)
  File "spacy/vocab.pyx", line 572, in spacy.vocab.write_binary_vectors (spacy/vocab.cpp:12067)
ValueError: could not convert string to float: b'c_codecs'

Any ideas?
Thanks!

Hey,
Looks to me like the data file you're trying to load is incorrectly formatted. Do you have whitespace in the word that's keying that line?
Matt

It seems so.
Should Doc handle somehow these?
doc = nlp(text) in word_vectors.py gives many "words" as spaces or similar.
When changed to this in Corpus in word_vectors (to skip those):
def count_doc(self, doc):
     # Get counts for this document
     for word in doc:
         w = str(word).strip()
         if w != str(word):
             print("'"+str(word)+"'")
             continue
         self.counts.inc(word.orth, 1)
         self.strings[word.orth] = word
     return len(doc)

I get this in init.py:
File "init.py", line 121, in write_vectors
write_binary_vectors(vectors_src.as_posix(), (dst_dir / 'vec.bin').as_posix())
File "spacy/vocab.pyx", line 565, in spacy.vocab.write_binary_vectors (spacy/vocab.cpp:12199)
File "spacy/vocab.pyx", line 572, in spacy.vocab.write_binary_vectors (spacy/vocab.cpp:12067)
ValueError: could not convert string to float: b'\x86q'

It looks like there are tokens with some sort of whitespace inside them. Maybe try if len(word.split()) != 1 as a condition.
Agree this is unideal.

Added if len(str(word).split()) != 1 but now some other valueError.
ValueError: could not convert string to float: b'K\x04X\x06\x00\x00\x00sampleq'
What is interesting, I don't know what this string is.
Google:
\x00 (0x00) = NULL
\x04 (0x04) = EOT (End of transmission)
\x06 (0x06) = ACK (Acknowledge)
P.S. have to add str() around word, otherwise gives
AttributeError: 'spacy.tokens.token.Token' object has no attribute 'split'

With this:
if not str(word).strip() or len(str(word).split()) != 1:
    print("'"+str(word)+"'")
    continue

gives:
ValueError: could not convert string to float: b'\x00\x00\x00null_wordq\x03K\x00X\x08\x00\x00\x00__scipysq\x04]q\x05X\x02\x00\x00\x00sgq\x06K\x00X\x04\x00\x00\x00loadq\x07cgensim.utils'

Maybe its something with bz2?
After running word_vectors.py it creates some file which I compress with:
bzip2 -z <file>
because init.py expects .bz2 file.
?

OK, I think I found it.
word_vectors.py uses word2.vec for saving model, and it is using utils.py "save" method that does pickle.dump.
On the other hand, init.py uses "write_vectors" method which calls "write_binary_vectors" in vocab.pyx that uses bz2 and simply iterates over lines:
with bz2.BZ2File(in_loc, 'r') as file_:
        for line in file_:

So, it is obviously not intended to be used like that. But following https://spacy.io/docs/usage/adding-languages, I didn't catch that it should be used in some specific manner except running word_vectors.py and init.py (plus Brown clusters), so either the instructions are not correct or I missed some key element in between.

This handles problems (add this to word_vectors.py):
def export_for_spacy_vocab_init(word2vec_model, location):
    string = ""

    for word, vocab in word2vec_model.vocab.items():
        string += word + " " + " ".join(str(x) for x in word2vec_model[word].tolist()) + "\n"

    with bz2.open(location, "wt") as bz2_file:
        bz2_file.write(string)

and add this to the bottom of "main" method:
export_for_spacy_vocab_init(model, out_loc)


Wrong percentage of languages. Ignores Golang

The repository: https://github.com/kriox26/rt-slackclone has 4 .go files which are not taken into account when showing the percentage of each language. The number of files are displayed correctly here:

And the colored bar is showing this:

The repository: https://github.com/kriox26/rt-slackclone has 4 .go files which are not taken into account when showing the percentage of each language. The number of files are displayed correctly here:

Hi @kriox26. They are being taken into account, the issue here is that you have 731KB of Javascript and  4.5 KB of Go. The repository stats are based on lines of code and so the Javascript files are dwarfing your repository statistics.

I see, sorry for the confusion. Thanks! :+1:


Extend automated end-to-end computation testing

Maps to https://github.com/trendscenter/coinstac/pull/1191

This needs to be broken up into more digestible issues as part of our DevOps initiatives.

We realize this is not feasible in the current sprint as defined.

Note: there is a PR that itself has issues to address.


Fixes #3423 | Report kills due to failed healthcheck.

Health check kills are published as an event and stored as task failure.
They appear in UI showing cause of last failed health check and host
just as normal Mesos task failures.

@janisz Thanks for this contribution!
I added some comments and questions.

Changes Unknown when pulling bd197bfb8c0134f48e47a57f7a903b16529c4691 on janisz:show_unhealthy_as_a_failure into ** on mesosphere:master**.

Changes Unknown when pulling 4278a88fb9e90a34d8bfbb4638d41653df56b68d on janisz:show_unhealthy_as_a_failure into ** on mesosphere:master**.

@janisz Looks good to me. Thanks!


Need to add link to the explore now button to open the product section

Fixed #IssueNumber
Check (put an 'x' between the square brackets) everything which applies:

[ ] I have added the issue number for which this pull request is created.

@nitin-pandita, please let me know about this update.

Hi @nitin-pandita ,
I would want to contribute to this issue, can you assign this to me ?
To clarifiy: I only need to add the same link as the 'Product' link in the navbar, right ?
Thank you.

Hey @lionelritchie29 , yes you need to add the same product link as the link in the navbar to the button


Enchancements by Tishka17

Add missing autoincrement=False to model
Simplify cmd_play() and miss() functions

(from me) Also use client-side cache_time for callbacks


Adding InputObject casing strategy

Add new casing strategy option for InputObjects

Closes apollographql/apollo-ios#3257

@BobaFetters Can you please do a follow up PR to add this option to the documentation for the codegen configuration?

@BobaFetters Can you please do a follow up PR to add this option to the documentation for the codegen configuration?

Good catch, thank you!


Support getting crate name and version from CLI args

We have CrateMeta struct for passing crate metadata, which (currently) supports setting package name, version and authors. We should initially support setting package name and version through the CLI (we can look at authors separately).
The changes should be done in src/bin/main.rs.

Hi, I am new to Rust. I have done rustling exercises but I think I need more time. I work with API based services using Python in my day job. I saw your profile in the awesome-rust-mentors list. Thanks for the opportunity. I would like to get involved in Rust community and I hope to learn Rust in the process.

Hello! This is definitely a good place to start if you wish to learn Rust and start contributing to community projects. Let me know if you're planning to work on this or if you have any questions! :smile:

Lets work on this issue then and see how far I can help! I will clone the repo and start lookin ginto the codebase. Do you have any pointers where I can start?

Sure! The change will live in src/bin/main.rs.
You can start with building this crate using cargo build --features cli. This will emit the executable in target/debug/paperclip. If you try running it, you can see that it'll show a number of options, which correspond to this struct.
We're setting metadata in this line, using this struct. As you can see in the struct definition, it supports setting a name and version (optionally). What we should do is add two more fields to Opt struct (name and version) which are both Option<String>. Then, in parse_args_and_run function, we'd check if any of them are set, and if they are, we'd set them in CrateMeta struct.
Does this make sense?

It makes sense.
I tried running the cargo command and encountered the following error:
cargo build --features cli
error: 'cargo.exe' is not installed for the toolchain 'nightly-2019-08-23-x86_64-pc-windows-msvc'
To install, run `rustup component add cargo --toolchain nightly-2019-08-23-x86_64-pc-windows-msvc`

For nightly, I get the following message:
λ rustup default nightly
info: using existing install for 'nightly-x86_64-pc-windows-msvc'
info: default toolchain set to 'nightly-x86_64-pc-windows-msvc'

  nightly-x86_64-pc-windows-msvc unchanged - rustc 1.40.0-nightly (22bc9e1d9 2019-09-30)

info: note that the toolchain 'nightly-2019-08-23-x86_64-pc-windows-msvc' is currently in use (overridden by '\\?\C:\Localdata\extra\rust\paperclip\rust-toolchain')

I'm not sure about Windows (sorry, I use linux), but what if you cd into the repository and run rustup override set nightly-2019-08-23 and try building again? If that still doesn't work, could you ask this in the Rust discord server?

It is building with the following command:
rustup run nightly cargo build --features cli

Cool! So, can you work on this?

Hi,
I did the build successfully. I am getting the following output:
λ target/debug/paperclip.exe
error: The following required arguments were not provided:
    <spec>
    --api <api>

USAGE:
    paperclip.exe [FLAGS] [OPTIONS] <spec> --api <api>

For more information try --help

Is that correct? It does not show Output and cli.

Did you run it with --help? It should show the optional arguments too.

Ahh, now I get a bigger output. Thanks. I will try to look into the code now.


Implement the dynamic foraging task as an example

[ ] subclass the abstract Task schema
[ ] save a file in examples/dynamic_foraging.py

As part of this implementation we should add version numbers to the task repository: https://github.com/AllenNeuralDynamics/dynamic-foraging-task/issues/218

This is the pattern i would suggest using the Vr Foraging task as an example:

# For now it assumes the following dependencies that can be pip installed using:
# pip install git+https://github.com/AllenNeuralDynamics/aind-behavior-curriculum.git@refs/pull/9/merge
# pip install git+https://github.com/AllenNeuralDynamics/Aind.Behavior.VrForaging.git@bc-dev

from typing import Dict, Literal

from aind_behavior_curriculum.behavior import GenericModel, Task
from aind_behavior_vr_foraging.task_logic import (EnvironmentStatistics,
                                                  ForagingSettings,
                                                  NumericalUpdater,
                                                  OperationControl,
                                                  TaskStageSettings)
from pydantic import Field

class VrForagingTaskParameters(GenericModel):
    """Class that represents the parameters necessary to instantiate the task logic of the Vr Foraging task"""

    updaters: Dict[str, NumericalUpdater] = Field(
        default_factory=dict, description="List of numerical updaters"
    )
    environment_statistics: EnvironmentStatistics = Field(
        ..., description="Statistics of the environment"
    )
    stage_settings: TaskStageSettings = Field(
        ForagingSettings(),
        description="Settings of the task stage",
        validate_default=True,
    )
    operation_control: OperationControl = Field(
        ..., description="Control of the operation"
    )

class VrForagingTask(Task):
    """Model for a VrForagingTask"""

    name: Literal["VrForaging"] = "VrForaging"
    version: Literal["0.0.1-preview01"] = "0.0.1-preview01"
    description: str = Field(default="This is a task schema for the VR task")
    task_parameters: VrForagingTaskParameters = Field(
        ..., description=VrForagingTaskParameters.__doc__
    )

print(VrForagingTaskParameters.model_json_schema())


Integrating all 59, 60 and 61 reported issues

The commit includes fixes for:
Support for base64 encoded buffers #59
Set child name (rootProperty) #60
Direct access to buffers and accessors #61

Build is failing..

Closing due to inactivity. Please reopen and fix the issues if you still want to merge this change.


Usage from node

I installed pdf.js to my node application, but it's version was only 0.1 but the current is 0.8 in the repository. Maybe it only needs an npm publish?
The problem that with version 0.1 the package.json requires cofeescript with the latest version, but the latest version doesnt auto register.
require('coffee-script');
module.exports = require('./lib/document');

Even after calling register here the code dies and complains for has no method 'getDocument'

The pdf.js package on NPM has nothing to do with this project.
If you want to install PDF.js, use npm install pdfjs-dist, see #5777.

Am i right then it could not be used from Node.js just from the browser?

It's not trivial to setup, but it's possible to use it from the node.js. See https://github.com/mozilla/pdf.js/tree/master/examples/node
We asked node.js community about API they want to see and what use cases they have in mind. No feedback was provided yet.

@yurydelendik is there an open discussion about that? I can't seem to find it.
I would be very happy to provide feedback on how one would want to use Pdf.js from node.


spelling: premitive to primitive

Addressing Azure Docs 646
Addressing Application Insights Docs 843

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

xml/Microsoft.Azure.Documents.SystemFunctions/TypeCheckFunctionsExtensions.xml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@CamSoper , whom do I need to ping about accepting this PR?

@Dmitry-Matveev I'm the approver, but this PR will get overwritten by the next CI build so I generally don't accept changes to the XML files.  If changes were made to the /// comments in the public API, then the change will automatically propagate to the site. That's why I opened an issue in the ApplicationInsights-dotnet repo and closed the one in this repo - Because changing the XML file in this repo won't fix it for longer than a day.


What is the different between the two files truth and truth_complete?

Hello kszuyen,
Thank you for your interest in DeepSeg.
Truth is the ground truth annotation provided by the BraTS Challenge (including all tumor labels l1, l2, l4), while Truth_complete is the whole tumor region (i.e. all tumor labels are converted into label 1).
Good luck!
Best regards,
Ramy

Hello, so I am recently reading your paper DeepSeg, and trying to implement your code again on my own. Maybe because I’m a beginner in deep learning with images, I encountered some problems, and I really hope you could reply.I’m now in the step to preprocess 2D images.
The first question is that the preprocess to 2D make 155 images for each case into a single file. Do you use all of these images to train your model? Or how to choose which image to train the model for each case?
Second is that for the label(segmentation or truth) images, am I comprehending it right that for the tumor region you assign it to 1, and for the others you assign it to 0? Because when I visualize the images I can only see black. I’m not sure if I am on the right track.
Sorry for bothering you with these basic or simple questions, but it’ll mean a lot to me if you could reply.

Ramy Zeineldin @.***> 於 2022年4月13日 上午10:30 寫道：
Hello kszuyen,
Thank you for your interest in DeepSeg.
Truth is the ground truth annotation provided by the BraTS Challenge (including all tumor labels l1, l2, l4), while Truth_complete is the whole tumor region (i.e. all tumor labels are converted into label 1).
Good luck!
Best regards,
Ramy
—
Reply to this email directly, view it on GitHub https://github.com/razeineldin/DeepSeg/issues/5#issuecomment-1097487811, or unsubscribe https://github.com/notifications/unsubscribe-auth/AW3FICHFTFLBV5PSVYQ77S3VEYWTNANCNFSM5TFFQLNA.
You are receiving this because you authored the thread.

Hi @kszuyen,
Thanks for your questions.
Firstly, the training of DeepSeg took only the images with tumor into consideration. This is one of tge strategies we followed to compensate for the data imbalance problem.
Secondly, you are right, 1 for the tumor segmentation while 0 for the background.
Good luck!


Paths as keys for Matrices

Hey Karel,
since the humble beginnings of this script, the read_key function only supported keys without a / symbol. I was just wondering, is there a reason why it is like that, since kaldi itself supports keys with / in them, e.g. audio/file.wav, but kaldi_io does not ?
Otherwise, I just propose to change line 115 to:
assert(re.match('^[\/.a-zA-Z0-9_-]+$',key) != None)

Okay, yes, there is no particular reason for disallowing slashes in ark-keys. This issue is now resolved in e7c94bc


During deserialization of pipeline the status "canceling" is not known and an error occurs.

Error message:  "Requested value 'canceling' was not found."
In my opinion the enum JobStatus must be extended.

See eg. https://gitlab.com/gitlab-org/gitlab-runner/-/merge_requests/2377 for where "canceling" was introduced to GitLab. It is listed at least at https://gitlab.com/gitlab-org/gitlab-runner/-/blob/main/network/remote_job_state_response.go#L16.

Also: https://gitlab.com/gitlab-org/gitlab/-/blob/master/lib/gitlab/ci/status/canceling.rb

Thank you very much for the quick help


pd: report write flow through store heartbeat

What is changed and how it works?
Ref #11718
What's Changed:

New tick to report write flow to pd worker
Report write flow through store heartbeat

kvproto: https://github.com/pingcap/kvproto/pull/845
Check List
Tests

Unit test
Integration test

Release note
Report write flow through store heartbeat

cc @bufferflies

/run-all-tests


chore: cleanup workspace deps

What's the blocker here? @tvolk131

What's the blocker here? @tvolk131

My squirrel-brain distraction with gateway stuff 😬

Now merge conflicts :cry:

Now merge conflicts 😢

Rebased!


Feature Time Picker

This is the continuation of #236

Thank you @julioflima  for this contribution.

Hi @julioflima!
A general refactoring of the code has been done to facilitate the evolution of the package. But this contribution is likely to cause a lot of conflicts at the current stage. So it will be removed so that we can add this refactoring. Then it can be added.

we need this function XD


remove() Function without Modal ID just like hide()

https://github.com/eBay/nice-modal-react/issues/58
Cause this issue, I don't want to use useModal outside modal.
So want to use NiceModal with NiceModal.show(), hide(), remove() outside modal(Not using useModal).
show, and hide function has no problem. But remove function doesn't accept modal and only accept modal Id.
I know I could declaratively set modalId with ModalDef or register, but It defeat the pros of NiceModal.
So I wonder is it possible to use remove() with just modal component something like this.
NiceModal.remove(SomeModal)

I'm not sure why it was not supported, seems a simple improvement: 7fa8bad .

Thanks!


fix(cookies): cookie header format

Spec
It seems incorrect to have a trailing semicolon.

awesome, thanks for this!


Deprecate @nonnull

Using @nonnull now yields:
> Task :integration-tests:generateNonnull-kotlinApolloSources
w: /Users/martinbonnin/git/apollo-kotlin/tests/integration-tests/src/commonTest/kotlin/test/nonnull/graphql/extra.graphqls: (1, 19):
Apollo: Using `@nonnull` is deprecated. Use `@semanticNonNull` and/or `@catch` instead. See https://go.apollo.dev/ak-nullability.
w: /Users/martinbonnin/git/apollo-kotlin/tests/integration-tests/src/commonTest/kotlin/test/nonnull/graphql/operations.graphql: (3, 12):
Apollo: Using `@nonnull` is deprecated. Use `@semanticNonNull` and/or `@catch` instead. See https://go.apollo.dev/ak-nullability.

Closes #5409

@BoD the intelli plugin fails on ApolloClientDirectiveStripperTest
     Actual [[query MyQuery {
      person {
        identity {
          firstName @catch
          lastName
        }
      }
    }
    ]]
    Expected [[query MyQuery {
      person {
        identity {
          firstName
          lastName
        }
      }
    }
    ]]

Do we need to add @catch to a list somewhere?

@BoD the intelli plugin fails on ApolloClientDirectiveStripperTest

Ah, I think I found it, need to import the directive, trying it out...


Use DirectML half-floating point

While using DirectML half-floating-point arithmetic, I found that convolutions require a huge amount of extra temporary resources


Notify user about disk space availability

When a certain volume of data is loaded in OpenRefine is reached, it stops loading the files. This is solved by deleting some files from the history.
Proposed solution
Generate an available space notification or indicator

I think this would make sense, if it is not too expensive / too involved to do cross-platform (I don't think it should be).

This could also be the occasion to check for available RAM, which we currently do in the launcher script. By doing it in the backend we could make this information more visible to the user (displayed in the frontend) and get rid of system dependencies.
Mentioned on the mailing list:
https://groups.google.com/g/openrefine-dev/c/eJbPPYnqJiw/m/_iQqCOdbAwAJ

Could it be displayed on some part of the front-end always indicating the change? Or just when memory usage reaches a certain thresh-hold?

Could it be displayed on some part of the front-end always indicating the change? Or just when memory usage reaches a certain thresh-hold?

@elroykanye: I personally think both would be great.
For the display, I would put this information in the Preferences page. I think we could also put more information here, like the path to the Workspace, the gData key & secret, the proxy IP & port, etc.
But, like you said, it could be good to have a warning when you get to, less say, 95 % of memory…
Regards, Antoine


[Item Conduit] Connectors don't have a texture?

Ender IO version: <IP_ADDRESS>5
Forge version: 1291

Latest build fixes this


Add Bazel and XLS to docker image

Adds Bazel and XLS to the docker image. XLS is quite large so it takes some time to build the first time. Was having issues building Bazel on my Mac, but it works fine on Havarti.

@gabizon103 looks like the docker image is failing to find the bazel package in apt. We can debug this later since this is not a blocker for experiments right now.

@UnsignedByte mentioned that this has been merged into #398


remove markdown

should fix unresponsive slang command

Coverage remained the same at 91.603% when pulling e3af969886d809f8fb5a4a5625d6821f4a8969e4 on remove_markdown into b83aec840f977ed1a7f99969c13f0779bf62418f on master.


Check new translations strings in PR

Questions
Answers

Description?
Add command to check new translations strings in PR. Migration from Prestonbot.

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes https://github.com/PrestaShop/PrestaShop/issues/34271

Sponsor company
PrestaShop SA

How to test?

Really tempting !


Redundancy error in case..of could be clearer

This is a continuation from this conversation
Consider the following SSCCE
import Html exposing (text)

a = "hello"

const1 = "test"

const2 = "hello"

main =
  let
    val =
      case a of
        const1 -> "test constant match"

        const2 -> "hello constant match"

        _ -> "no match"
  in
    text val

The error received is
The following pattern is redundant. Remove it.

16|         const2 -> "hello constant match"
            ^^^^^^
Any value with this shape will be handled by a previous pattern.

From a beginner's perspective, it would seem reasonable that I should be able to use a constant as a match expression in a case..of. However, this isn't valid and Elm interprets const1 as matching anything and does the same for const2, hence the error mentioning redundant patterns.
What's going on here isn't obvious to a beginner. An error message describing that any lowercase name, such as const2, will match any pattern might help. There could be better ways of wording that though.

This also confused me (a beginner) when I first encountered it today.


CASMUSER-3190

Summary and Scope

Implements CASMUSER-3190

Issue Type
Docs Pull Request

Replaces link to Getting Started Guide with a link to its replacement. The GSG doesn't exist in the next CSM recipe release.
Breaks up UAN_Ansible_Roles.md file into separate files, one for each role.
updates both ditamaps and the hugo_toc.json file for the refactoring
replaced links to the deleted file with links to the specific role.

Prerequisites

[x] I have included documentation in my PR (or it is not required)
[x] I tested this via local builds on my WSL2/Ubuntu instance and uploading to the test ITG portal. No errors from DITA-OT. Content on ITG test portal looks good.

Idempotency
N/A
Risks and Mitigations
Slight risk that I updated the hugo_toc.json file incorrectly since I can't test that build process.

waiting for @keopp-hpe 's approval before merging so that he can assess any impact to the Hugo-generated docs.


Https certificate for website is now invalid

Hi
The certificate for https://www.fluttericon.com is invalid

Sorry about that!  Fixed now.


Do we need a mechanism to provide a version of gist without extensive annotations?

Extensive documentation within the ontology is a double-edged sword - it provides scope for additional clarification of intention within the ontology, including examples etc., but it also provides greater bulk for situations where the only purpose of the ontology is to provide material for computation over the OWL and where the annotations are not going to be seen.  In this latter the bulk of the file with annotations is unnecessary and could be done better with a thinner file.

I feel quite strongly that we should not do this. If onto-tool provides the means to remove annotations, users can do so themselves. I see no obligation on our part to provide a "gist lite" with human semantics stripped out, and I can see how this could cause a good deal of confusion.

Programmatic manipulation of an OWL ontology via e.g. OwlAPI allows you to strip out annotations you don't want/need anyway, so this issue is unnecessary.

@rjyounes, @pwin PROPOSAL: to close due to lack of demand. Anyone who needs this functionality can do it themselves.

Closing. For the record, I think this is an awful idea.


Support other clouds?

Feature Description
It'd be nice to have basti available for GCP, Azure, etc. Would you consider adding this (or taking PRs)?
Use Case
We use GCP, can't use basti. :)
Proposed Solution
[If you have any suggestions on how to implement the feature, provide details or a high-level overview here. If not, that's okay too!]
Related Issues/PRs
[If there are any related issues or pull requests, please link to them here.]

When developing Basti, I considered possible future support for other clouds. At the moment, I won't be able to implement this functionality on my own but I will be happy to collaborate on incoming PRs!
In case you consider implementing a major piece of functionality like this, I will be happy to provide an introduction to the repository and discuss the implementation ideas on a meeting.


Sound Booster

Description of the new feature / enhancement
Nothing fancy, just boost the sound to 200% - 500% or more. You might add an equalizer.
Scenario when this would be used?
Sometimes users may need to boost volume. Similar app: FXsound
Supporting information
No response

This isn't a utility PowerToys would build out.  Windows feedback hub would a better spot to do this.

For Windows 10/11 related issue (enhancement), please file this on the Feedback hub


Generation artifacts

Issue https://github.com/aws-controllers-k8s/community/issues/809
Description of changes:
This patch adds a cobra PostRunE function that writes generator
parameters, output metadata and generated files checksum, to a
file within the API versions directory.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Output file example:
ack_generate_info:
  build_date: "2021-06-16T22:31:45Z"
  build_hash: e278e1aef5abc644e681ac733cc23b424c136063
  go_version: go1.16.4 linux/amd64
  version: v0.0.0
apis_directory_checksum: 4ffda010495d8ff251f0959fbc4b23d22d4a4615
apis_version: v1alpha1
aws_sdk_go_version: v1.35.35
generator_config_info:
  file_checksum: f31970ae673c4e8d77bf66ed09ec7a34f76569f2
  original_file_name: generator.yaml
last_modification_reason: API generation
last_modification_time: 2021-06-16 22:35:03.297998193 +0000 UTC

@jaypipes @RedbackThomson  if all looks good, can we give this PR a manual merge? i think that we should keep the LDFLAGS modifications separated from the generation metadata stuff

/lgtm


Empty case improvements

This modifies the fix for #3019.
It seemed a little weird to me to be using .any? in an if condition. I think that using || is a more standard way of writing these conditions.
I also added a test case using then.
@owst, let me know what you think of the changes.

Nice - I wasn't really happy with the any? solution when I made the original change, but wanted to ensure I covered all cases.
In my code, I think I would use [a, b, c, d].any? rather than a || b || c || d, especially if there are > 2 conditions, but I'm fine with this change.

In my code, I think I would use [a, b, c, d].any? rather than a || b || c || d, especially if there are > 2 components, but I'm fine with this change.

Notice that those two solutions are not exactly the same - || is short-circuiting, but the all the array's elements will always be evaluated.

A very good point @bbatsov. My inner Haskeller took over for a bit there!
Let's wrap all the conditions in lambdas and do any?(&:call)...  only joking! :-)

Anyways, I'll merge this - feel free to improve it further down the road.


Cannot initiate or respond to off-realm whispers

Ever since 10.2.7 hit, whispering people on another server stopped working. I can receive whispers just fine, but I cannot respond or initiate; I always get the error message that the recipient does not exist.
Issue is only present for character whispers, bnet communication works just fine.

There is a problem, namely someone writes in the /Trade of the server Mal'ganis, the server is not inserted in the WIM window with so coincidence Mal'ganis, Wim tries to whisper coincidence, which of course does not exist on the server Blackhand, then this is very annoying because when people try to write to players from Mal'ganis they get the message "coincidence" is not online.

@Yuraky does that happen to every server user or just Mal'ganis? I wonder if its something as simple as an expression update that isn't accounting for the apostrophe in the server name.

I can supplement that this (still) happens to whispers going off-server - single word, apostrophe or two words.

Thank you for the details.

I'm on the french server "confrérie du thorium" and it doesn't work too

and it works when whispering a player on the same server as mine.