Feature suggestion (resolver)

Hello, this is a really great cheat, but I am missing 1 thing here and I do not have enough knowledge to do it myself. Could you please add AA Resolver to the cheat.
Thank you.

I'll make a pull.

I'll make a pull.

quick bro we need this feature to beat spinner

yes

I mean there's an alternative, just spam body

bro its been 2 hours

I'll make a pull.

when pull sar

bro please pull

Sorry, I got school. I'll make the pull later on when I have time. And when I say "I'll make a pull"... I mean a LegitAA Resolver. Why the heck would you put a full AA resolver in a full legit cheat smh

the cheat isn't full legit cheat lmao
Osiris could be actually good for HVH if there was a resolver. The cheat can tap bad configs if you set bodyaim

É cada coisa que eu leio, não vou nem gastar meu inglês ruim com isso

Sorry, I got school. I'll make the pull later on when I have time. And when I say "I'll make a pull"... I mean a LegitAA Resolver. Why the heck would you put a full AA resolver in a full legit cheat smh
A full resolver would be a lot of code, that I don't have time to write.
EDIT: there's no point having a rage AntiAim resolver because you're still gonna get tapped first; either their ragebot will shoot quicker, the resolver won't work the first time (and then ya fucked) or you'll miss their fakelag

Can u make it with player options like dont resolve everyplayer we can choose the player
cuz it make legit players unhittable

Sorry, I got school. I'll make the pull later on when I have time. And when I say "I'll make a pull"... I mean a LegitAA Resolver. Why the heck would you put a full AA resolver in a full legit cheat smh
A full resolver would be a lot of code, that I don't have time to write.
EDIT: there's no point having a rage AntiAim resolver because you're still gonna get tapped first; either their ragebot will shoot quicker, the resolver won't work the first time (and then ya fucked) or you'll miss their fakelag

Can u make it with player options like dont resolve everyplayer we can choose the player
cuz it make legit players unhittable

I can try, might take a bit cause I have to find time.

Sorry, I got school. I'll make the pull later on when I have time. And when I say "I'll make a pull"... I mean a LegitAA Resolver. Why the heck would you put a full AA resolver in a full legit cheat smh
A full resolver would be a lot of code, that I don't have time to write.
EDIT: there's no point having a rage AntiAim resolver because you're still gonna get tapped first; either their ragebot will shoot quicker, the resolver won't work the first time (and then ya fucked) or you'll miss their fakelag

Can u make it with player options like dont resolve everyplayer we can choose the player
cuz it make legit players unhittable

I can try, might take a bit cause I have to find time.

So when?

Sorry, I got school. I'll make the pull later on when I have time. And when I say "I'll make a pull"... I mean a LegitAA Resolver. Why the heck would you put a full AA resolver in a full legit cheat smh
A full resolver would be a lot of code, that I don't have time to write.
EDIT: there's no point having a rage AntiAim resolver because you're still gonna get tapped first; either their ragebot will shoot quicker, the resolver won't work the first time (and then ya fucked) or you'll miss their fakelag

Can u make it with player options like dont resolve everyplayer we can choose the player
cuz it make legit players unhittable

Only just read this, but, no that isn't right. Resolving players that don't have anti-aim will just make them stay the same.

Sorry, I got school. I'll make the pull later on when I have time. And when I say "I'll make a pull"... I mean a LegitAA Resolver. Why the heck would you put a full AA resolver in a full legit cheat smh
A full resolver would be a lot of code, that I don't have time to write.
EDIT: there's no point having a rage AntiAim resolver because you're still gonna get tapped first; either their ragebot will shoot quicker, the resolver won't work the first time (and then ya fucked) or you'll miss their fakelag

Can u make it with player options like dont resolve everyplayer we can choose the player
cuz it make legit players unhittable

Only just read this, but, no that isn't right. Resolving players that don't have anti-aim will just make them stay the same.

find a way to check for the enemies eye angles to see if theyre pointing down or not or check if youre on a vac secured server could be a way to chose whether or not to resolve someone


Using protractor with typescript creates tsc warnings

tsc                                                                                ⏎
node_modules/protractor/globals.ts(22,37): error TS2304: Cannot find name 'global'.
node_modules/protractor/globals.ts(23,31): error TS2304: Cannot find name 'global'.
node_modules/protractor/globals.ts(24,37): error TS2304: Cannot find name 'global'.
node_modules/protractor/globals.ts(25,31): error TS2304: Cannot find name 'global'.
node_modules/protractor/globals.ts(26,31): error TS2304: Cannot find name 'global'.
node_modules/protractor/globals.ts(27,51): error TS2304: Cannot find name 'global'.
node_modules/protractor/globals.ts(28,57): error TS2304: Cannot find name 'global'.
node_modules/protractor/globals.ts(30,5): error TS2304: Cannot find name 'global'.

This might not be a bug if typings.json includes the DefinitelyTyped node ambient file. Should investigate post 4.0.2 release.

After some investigation, this is not an issue. In the cookbook, the typings.json file was missing node. Having node in typings allows for 'global' to exist.

Hi @cnishina ,
I have node in typings.json and it is still not finding 'global'.

There is a fix. Although your typescript compiler is complaining, it should still work. https://github.com/angular/protractor/pull/3485

@ravitej-klair It should work, I faced similar warnings and fixed them. Try uninstalling node typings and install it again. There is a catch- typings uninstall doesn't work same as install and it is not documented as well. I do it like this:
//create typings.json file in project
typings init

//for installing ambient types
typings install --global --save dt~node

//for uninstalling
typings uninstall --global --save node


Add support for different shell prompt characters

Add support for undollar to 'eat' different shell prompt characters such as and '%' and '#' (though this may warrant a confirmation prompt because it represents a superuser).

This whole program (with installation) could be reduced to this bash oneliner:
echo '"$@"' >/usr/local/bin/$ ; chmod +x /usr/local/bin/$

You could do the same with any special character you wanted.

You're absolutely right. This would work in the vast majority of cases. It is definitely kind of crazy to spin up an entire JavaScript virtual machine just to pass on a command to the system, and it does feel pretty gross doing so.
That being said, the number one priority for this project is that it is extremely easy for anyone to use, including across operating systems and shells. What you've given is a great base, and I am probably going to use it to replace the JavaScript with an installation shell script that handles permissions and offers uninstallation across operating systems as easily for users as it is with npm.
Thank you!

Here you go:
#!/bin/sh
exec "$@"


Combining more code

Several native-wgpu extensions apply to RenderPassEncoder and ComputePassEncoder, but not to RenderBundleEncoder.   Push Constants will soon join this group.
Even though there is no Class that is the ancestor of just these two, we put the code into GPUBindingCommandsMixin.

Closing.  This accidentally got some code it wasn't supposed to.


Add Trinity School NYC to JetBrains catalogue

https://www.trinityschoolnyc.org/

@DerkTerk First stage of verification passed, official domain is trinityschoolnyc.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Thank you!


More tests and fixing some minor pylint errors

Codecov Report

Merging #88 (0eeac4f) into master (dc1346a) will increase coverage by 2.93%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   83.68%   86.61%   +2.93%
==========================================
  Files          12       12
  Lines         521      523       +2
  Branches       63       62       -1
==========================================
+ Hits          436      453      +17
+ Misses         67       55      -12
+ Partials       18       15       -3

Impacted Files
Coverage Δ

src/ui/app.py
67.05% <0.00%> (+7.74%)
:arrow_up:

src/services/reading_tip_service.py
97.50% <100.00%> (+0.06%)
:arrow_up:

src/services/tags_service.py
100.00% <100.00%> (ø)

src/ui/console_table.py
100.00% <100.00%> (+5.55%)
: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 dc1346a...0eeac4f. Read the comment docs.


Unable to import

After adding import { XMasonry, XBlock } from "react-xmasonry"; to a component, I get the following error:
Uncaught Error: Cannot find module 'React'
    at makeMissingError (modules-runtime.js?hash=8587d18…:231)
    at require (modules-runtime.js?hash=8587d18…:241)
    at n (modules.js?hash=2355269…:220189)
    at index.js (modules.js?hash=2355269…:220189)
    at fileEvaluate (modules-runtime.js?hash=8587d18…:343)
    at require (modules-runtime.js?hash=8587d18…:238)
    at PinBoard2.js (PinBoard2.js:1)
    at fileEvaluate (modules-runtime.js?hash=8587d18…:343)
    at require (modules-runtime.js?hash=8587d18…:238)
    at routes.js (routes.js:1)

Any insight on why this might be happening?

Hello! What's the version of XMasonry you're using? What's the bundler you are using to bundle JavaScript? In the last package version, I've added React to peer dependencies, the error message looks strange to me...

This should fix the issue anyway: meteor npm install --save react

Yeah I'm using Meteor with React, and I was able to import many other react components from NPM but react-xmasonry is throwing that error for some reason.

Hm. Could you please post any of the components you've succeeded with? I will be happy to compare the code and fix the problem if it is on XMasonry side.

Also, try import { XMasonry, XBlock } from "react-xmasonry/dist/index.js" — will this work properly? Thanks!

Thanks for the quick response @ZitRos.
This component was working import Masonry from 'react-masonry-infinite';
And import { XMasonry, XBlock } from "react-xmasonry/dist/index.js" didn't do the trick :(.

You're always welcome! I see that almost the only difference is that in my package.json, there is "module" property which points to ES6 classes instead of bundled distributive. I am wondering if this can be the issue. If you are very motivated to use XMasonry in your project, could you please try to go to your node_modules/react-xmasonry/package.json and manually remove the line with "module" property from there? Will this solve the problem?

That didn't solve it either :( for now I just copied the source code to the project and imported manually so I can give the library a spin.
I'm not sure if this is something specific to my project, or Meteor or the way the library is packaged, I just wanted to let you know..

OK! Once I come back to my laptop (4-5 days ☀) I will try XMasonry with Meteor and let you know. Thanks!

I've just launched your example. Everything seems to work as expected:

Am I missing something?

I think I've found a problem. Try the newest XMasonry version 4.2.1.
The issue seems to be in case-sensitive file system: XMasonry UMD module was requiring "React" while it should require "react". Please confirm if the issue is solved and close it.
Thank you very much for helping to make XMasonry better!

That did the trick! thanks @ZitRos
I've evaluated many masonry implementation and I think this one out performs all, great work buddy.


Use profanity list

Messages containing profanity need deleted immediately. Even if you're not comfortable with using a filtering package on the actual repository, you could always use Replit and store the name of your package in an environment variable, then run a server from /tmp.

Profanity filter is handled by Scratch already.

Profanity filter is handled by Scratch already.

But they still missed that F attack. Shouldn't you strip non-UTF-8 characters and check then?

If they contain those characters it's likely they're already attempting to do this. Also, people are encouraged to change the disallow list

If they contain those characters it's likely they're already attempting to do this. Also, people are encouraged to change the disallow list

Good point. You might want to normalize homographs before processing.


GradCam Implementation

I am using a MultiModal predictor to do binaryclassification. I trained my model and load it back to reach forward and backward I am using this commands:
predictor = MultiModalPredictor.load(path)
predictor._model.forward(input_tensor)
However I am getting an error like this:
TypeError: TimmAutoModelForImagePrediction.forward() missing 1 required positional argument: 'image_valid_num'
When I search for image_valid_num I only could find this command predictor._model.image_valid_num_key which returns this as string:
'timm_image_image_valid_num'
My question is how can I find what is this image valid num and where can I find in my saved model?

Here is an example to get the image_valid_num from processed input batch (dict of tensors): https://github.com/autogluon/autogluon/blob/bf4940ee3d9a20e2d99015aa62afd18e493d3c5a/multimodal/src/autogluon/multimodal/models/clip.py#L141 during training The image_valid_num for each data entry is computed here: https://github.com/autogluon/autogluon/blob/bf4940ee3d9a20e2d99015aa62afd18e493d3c5a/multimodal/src/autogluon/multimodal/data/process_image.py#L306 and collated as a tensor here: https://github.com/autogluon/autogluon/blob/bf4940ee3d9a20e2d99015aa62afd18e493d3c5a/multimodal/src/autogluon/multimodal/data/process_image.py#L179 This is the forward function you called: https://github.com/autogluon/autogluon/blob/bf4940ee3d9a20e2d99015aa62afd18e493d3c5a/multimodal/src/autogluon/multimodal/models/timm_image.py#L132 The root cause seems to be that the data is not preprocessed.


⚠ Adacta-fintech.com has degraded performance

In be7bf53, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 502
Response time: 528 ms

Resolved: Adacta-fintech.com performance has improved in 690ca29 after 7 minutes.


Add libvvenc VVC encoding support of FFmpeg

Only master version of FFmpeg supports VVC encoding via external feature libvvenc. Later we still wait for next new FFmpeg release version to be released.

Martin Eesmaa

@BtbN, I fixed it, please check again! Thanks! :)

Martin Eesmaa

Any updates, @BtbN?

Martin Eesmaa

I'm still on vacation and don't have the time nor facilities to test this.
Will get to it once I am back home.

Oh, I see... okay thank you for noticing to me! :)
Take your time when you needed. Also I forgot to disable FFmpeg 6.1 version I didn't know that, so I will make a new change.

Martin Eesmaa

Is there a particular reason you disable ARM SIMD entirely?
If it's broken, is that issue reported upstream?

Did some ammendmends and merged.


Use Docker instead of Vagrant

It would be very nice, if tool for orchestration Docker would actually use Docker instead of Vagrant :)

Docker has restart policies (now), which would keep it running.
My question would be how you'd manage upgrades of helios (as entertaining as it'd be, I'm not sure it makes sense to have the helios agent managing the helios agent - you probably don't want to ever be able to undeploy, for example!).
In theory it should be as simple as persisting the agent state directory. I'll have to try it and see.

With helios-solo, we now have Helios-in-Docker for development purposes. helios-solo brings up a simple one-agent cluster in a Docker container.
@aidanhs, with helios-solo, upgrades are done by running helios-use latest. However, this is a destructive operation that wipes your whole helios-solo instance. Persisting the state directory is definitely a possibility, probably would be a pretty small change to the docker run command in helios-up.
In addition to helios-solo, we actually also build a helios-master and helios-agent Docker image with the build-images Maven profile:
mvn -P build-images package

These are more flexible and could, in theory, be configured to run in a cluster configuration. We use them for integration testing, but I don't see us recommending them as the blessed way to install Helios and run a cluster.

Closing this issue since we no longer use or recommend Vagrant.


Forks of repos have license issues

Origin page

Reported to GitHub, its a known issue which they're looking into.


fzf: 0.43.0 -> 0.44.0

Diff: https://github.com/junegunn/fzf/compare/0.43.0...0.44.0
Changelog: https://github.com/junegunn/fzf/blob/0.44.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

LGTM


composable either #1

Hello,
it seems to work, I'm a beginner though, I just copypasted and hacked out "left" either param with kind projector. Is it the best way?
[info] Running com.example.Hello
List(Right(Some((9,Some(5)))), Right(Some((9,None))))

Hi Tomaszym,
This does really look absolutely right to me. Yes, the kind projector is what I wanted to use in the first place. I'll check this in my computer quickly and merge afterwards

Your implementation looks really great, specially for a beginner! Many, many thanks! :)
I've added a really minor styling comment in one of the lines. If you can fix that, I'll merge straight away.

Nice one! I'll update the README to include Either as one of the supported types.


Python extension loading forever

I'm using VSCode within a corporate network where all traffic is routed via a proxy. The latest versions of Python extension (>= 2021.4.765268190) cannot load most likely due to proxy issues (aka. certain requirements cannot be connected/downloaded via the proxy). As a result, Python extension hangs with "loading python extension" forever.
A temporary fix is to downgrade to an earlier version (2021.3.680753044) but this means we cannot get constant improvements and bugfixes pushed out by this extension.
I need to figure out what URL the extension is trying to access so I could report this to the internal team.
I've also enabled python.logging.level = "info"
Environment data

VS Code version: 1.55.2
Extension version (available under the Extensions sidebar): v2021.5.842923320
OS and version: Windows 10 - version 1909
Python version (& distribution if applicable, e.g. Anaconda): python 3.7.9
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions: XXX
Relevant/affected Python-related VS Code extensions and their versions: XXX
Value of the python.languageServer setting: Pylance

Expected behaviour
Python extension being able to load normally and quickly
Actual behaviour
"python extension loading..." forever
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]
Hard for outsiders to reproduce if you're not on the internal network

Open VSCode
Open a python file
"python extension loading..."

Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

User belongs to experiment group 'pythonaacf'
User belongs to experiment group 'pythonJoinMailingListVar3'
User belongs to experiment group 'pythonSendEntireLineToREPL'
User belongs to experiment group 'pythonNotDisplayLinterPrompt'
User belongs to experiment group 'pythonDiscoveryModuleWithoutWatcher'
User belongs to experiment group 'pythonTensorboardExperiment'
User belongs to experiment group 'pythonSurveyNotificationcf'
User belongs to experiment group 'PythonPyTorchProfiler'

@ptmminh Can you try and get a CPU profile for the extension?
Can you see if you get any more details if you set python.logging.level = "debug"?
Add this to your user settings to unblock from this, but please do provide the requested info.
    "python.logging.level": "debug",
    "python.experiments.optOutFrom": [
        "pythonDiscoveryModule",
        "pythonDiscoveryModuleWithoutWatcher"
    ]

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


dense node format

Hi all,
I was wondering if I could get your thoughts on what sort of level of effort it would require to allow for extracts in dense node format. I see that it's mentioned in the readme, and the question derives from an interest we've got in starting to build and run an on demand extract system. I know we'd discussed this a while back, and it got derailed, but there's interest here in picking it up again.
Right now, in the format you're outputting, it looks like there's limited usefulness for a lot of standardized tooling that expects dense nodes. We're still doing a POC, but if it gets accepted I think we could contribute some code to make this happen if you think it's something that makes sense.
Ultimately we'd be looking to run an API based on vex, behind an api key based system (only for the purposes of rate limiting and being able to get in touch with users when necessary), as well as a front end for ease of use (poc: http://hkrishna.github.io/ode/#12/40.7261/-73.9807).
Let me know your thoughts...

A few weeks back I completed minutely update support in our Java version of Vanilla Extract. After some tweaks to the inner workings the performance of the Java version is completely acceptable, and it will be much less challenging to maintain and extend. Therefore, at this point all development will focus on the Java version.
Note that while Vanilla Extract keeps all imported OSM data, its indexing focuses entirely on ways. Geographic extracts contain only nodes that are referenced by ways. More so than lack of dense nodes in PBF output, this makes the extracts unsuitable for most people.
The reason for this is that it is focused on routing. VEX could probably be extended to track “lone” nodes that are not in any way without too much difficulty, but these lone nodes were just not a high priority for our use cases.
I increasingly dislike the PBF format and hope to see it replaced in the future, but it is of course the current standard and needs to be supported. It would be straightforward to produce dense-nodes output in the Java version of Vanilla Extract, it’s just a matter of setting aside some time to add it to Conveyal’s osm-library.


Reset geneva to 3 on (Referee)Halt

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

probably not really necessary because we send the geneva state continuously


No file ./launchers/launch_create_sdf_pix3d.sh

in preprocess dir, there is no file ./launchers/launch_create_sdf_pix3d.sh, can you provide the script? thanks so much!

same issue here, there is no launch_create_sdf_pix3d.sh file

Any updates on this?

still no updates for this?? =( seems like a few files missing in this open source project.


feat: Add generic data quality checks

Summary of Changes
WIP - Need to add a placeholder API handler in the flask app
Add support for generic data quality checks
Tests

Documentation

CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

Any idea why the python tests are failing?
error with MailTest.test_notification_endpoint_calls_send_notification. I haven't touched these files.
https://github.com/amundsen-io/amundsen/pull/1073/checks?check_run_id=2587126169
@Golodhros @dkunitsk @allisonsuarez @youngyjd

IS there an RFC for this?

IS there an RFC for this?

@dorianj no there is not. We were planning to do this as a Lyft only change, but Mark had requested that we put this in OSS. I realize that I have skipped a few steps, and there may need to be further changes from what I have. I'm not sure how much more time I can spend on this as I have other pressing tasks at the moment.

@Golodhros this kind of blew up with additional work. I added the data quality client and icons

IS there an RFC for this?

@dorianj no there is not. We were planning to do this as a Lyft only change, but Mark had requested that we put this in OSS. I realize that I have skipped a few steps, and there may need to be further changes from what I have. I'm not sure how much more time I can spend on this as I have other pressing tasks at the moment.

I think we should create a quick RFC, move it fast and get this merged once the RFC is merged.

IS there an RFC for this?

@dorianj no there is not. We were planning to do this as a Lyft only change, but Mark had requested that we put this in OSS. I realize that I have skipped a few steps, and there may need to be further changes from what I have. I'm not sure how much more time I can spend on this as I have other pressing tasks at the moment.

I think we should create a quick RFC, move it fast and get this merged once the RFC is merged.

Yeah, a quick RFC would be great. We are exploring this too and it'd be great to share broader context. Overall, we are pretty supportive of this but if there are certain design decisions (both UX and arch) that are being made, that'd be good to see. Thanks!

@danwom @Golodhros does this pr align with https://github.com/amundsen-io/amundsenfrontendlibrary/pull/495 which has an extra link for DQ reports?


Unable to build when using BottomSheetOverlay

Not able to build when using BottomSheetOverlay with onDismiss param, getting below error.
overlay/picklist/PickItemBottomSheet.kt:31:15 This material API is experimental and is likely to change or to be removed in the future.

But when I comment out onDismiss and run, it builds fine. Also I tried to suppress the warning, still fails.

This is an opt-in annotation like any other, we use experimental material APIs in that implement under the hood so you have to opt in to use it.


Incorrect error message

primitive Foo
    fun prev(n : Bar) : String =>
        n.n2.string() + ", " + n.n1.string()

class Bar
    let n1: U8
    let n2: U8

    new create() => (n1, n2) = (1, 2)
    fun reverse() : String => Foo.prev(this)

actor Main
    new create(env: Env) =>
        let n = Bar
        env.out.print(n.reverse())

The above results in the error message
Error:
main.pony:10:40: argument not a subtype of parameter
    fun reverse() : String => Foo.prev(this)
                                       ^
    Info:
    main.pony:10:40: argument type is this->Bar ref
        fun reverse() : String => Foo.prev(this)
                                           ^
    main.pony:2:14: parameter type is Bar ref
        fun prev(n : Bar) : String =>
                 ^
    main.pony:10:40: Bar box is not a subtype of Bar ref: box is not a subcap of ref
        fun reverse() : String => Foo.prev(this)

which isn't at all helpful. to be helpful, I think it should say
    main.pony:10:40: argument type is this->Bar box
I'm assuming this is a bug. It looks like a bug to me.

Here's the relevant section of code that generates the error:
https://github.com/ponylang/ponyc/blob/master/src/libponyc/expr/call.c#L243

I have the feeling this is related to https://github.com/ponylang/ponyc/issues/3568#issue-623794991

Hmm, while I agree that this error message is not perfect, I don't know about the fix: I think that the current this->Bar ref is more "correct", but may be unclear. This box method may result in something which is mutable, since it's polymorphic, it's just that in this case it's not. So the "box" part is handled by the viewpoint adaptation this->..., when this is instantiated to box.
What we may want to do is expand out the viewpoint adaptation with error messages, e.g.:
"has type this->Bar ref", but must be Bar ref.
But if this has capability box, then this->Bar ref will be Bar box
If this has capability val, then this->Bar ref will be Bar val

Yeah, @jasoncarr0's explanation is correct - the error message is fully correct - it's just not obvious to everyone who reads it that this->ref implies a possibility of box or val depending on the cap of this.

It was agreed during sync that while the error message is technically correct, it isn't a helpful error message. Changing this is a rather deep change that involves a lot of context and knowledge of the existing compiler implementation and how and when various parts of type reification are done.
At the moment, this is a bad error message that we are mostly stuck with unless someone wants to undertake a reasonably large amount of work.

I wonder if we could get an easy win by making the compiler print this{ref, val, box}->Bar ref instead of this->Bar ref as the type - which would at least point out that the cap of this can vary and the code must account for all three capabilities.

Another option would be to add to the typechecking logic in which this->Bar ref failed to be a subtype of Bar ref, some logic that detected the this-> on the prospective subtype, and tried stripping the this-> and trying the typecheck again - if the former fails and the latter succeeds, we could print a message that says "this would be possible without the this-> viewpoint adaptation - did you mean to originate the value from a fun ref instead?". Or possibly a better wording on that same theme.


2037: separate the TCAP functionality from the TCA (tca.go -> tcap.go)

Description
A quick separation of the external gRPC (public) interface for TCA.
Part of the breaking down of #2034 into smaller, gradual, issues as requested.
Motivation and Context

Fixes #2037
How Has This Been Tested?

Full build via make all, running all tests.
Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[x] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

The continuous integration build process will run make checks to confirm that tests pass and that code quality meets minimum standards. You may optionally run this locally as PRs will not be accepted until they pass.
Signed-off-by: JonathanLevi<EMAIL_ADDRESS>

LGTM thanks

Thanks, LGTM


Fix upload button on comment textareas

Fixes #1242.
Explanation: https://github.com/sindresorhus/refined-github/issues/1242#issuecomment-381403293

The code looks a bit hacky, but works.
Open to any other suggestions or modifications!

Works as intended!


Update splash screen

Fixes issue #0
Brief summary of changes
New splash screen
Testing I've completed
CHANGELOG.md (choose one)

no need to update because...
updated...

Added Nick to 4.0 team

We removed the beta and added Nick to the 4.0 team.


Callout

adds Callout, CalloutSuccess, CalloutWarning, CalloutError component
adds Callout scss styling
adds warning svg (closes #65)
adds color prop based on SystemColors

Closes #61.

@ParmeJon, I'm going to merge this but I wanted to recommend that you read through my commits a bit since I made some small and some not-so-small changes.
Thanks for all your work on this, your first component in the design system! :tada: :beers:


v2.0.0-rc2: .on('notification') event not fired when background notification in Android

Expected Behaviour
Notification received in background should fire event after tapping on it
Actual Behaviour
After tapping the notification, app opens but event is not fired
Reproduce Scenario (including but not limited to)
The app is made with Ionic 2.0.1 and the code launched within platform.ready)=
Steps to Reproduce
Send notification with the payload shown below.
Tap it from the notification bar and when the app is launched nothing happens, but it should.
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 6.0.1
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Samsung Galaxy S7
Cordova CLI version and cordova platform version
cordova --version 6.5.0
cordova platform version android 6.1.2

Plugin version
cordova plugin version | grep phonegap-plugin-push 2.0.0-rc2

Sample Push Data Payload
{
    "to": "/topics/_all_",
    "notification": {
    	"title": "title notification",
        "body": "body notification"
    },
    "data": {
    	"title": "title data",
        "message": "message data",
      	"app": "MOVAPP_GEN",
        "content-available":  "1"
    }
}

Sample Code that illustrates the problem
let push = PushNotification.init({
	android: { sound: 'true', vibrate: 'true', topics: ['all', '_all_'] },
	ios: { alert: 'true', badge: true, sound: 'true', fcmSandbox: true, topics: ['all', '_all_'] }
});

push.on('registration', (data) => { this.userData.setTokenFire(data.registrationId); this.userData.setPushToken(data.registrationId); });

push.on('notification', (data) => {
	console.log("NOTIFICATION: " + JSON.stringify(data));
	if (data.additionalData.app == constants.APP_ID) { events.publish('app:notification', data); }
	else { utils.sendNotification(data); }
});

push.on('error', (error) => { console.log(error.message); });

Logs taken while reproducing problem

I've just found that if you just send this payload:
{
    "to": "/topics/_all_",
    "data": {
    	"title": "title data",
        "message": "message data",
      	"app": "MOVAPP_GEN",
        "content-available":  "1"
    }
}

The notification event is fired twice in android, and the notification doesn't reach iOS at all.

@markos92i yes, that is the way that it works:
https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#notification-vs-data-payloads
If you send:
{
    "to": "/topics/_all_",
    "notification": {
    	"title": "title notification",
        "body": "body notification"
    },
    "data": {
    	"title": "title data",
        "message": "message data",
      	"app": "MOVAPP_GEN",
        "content-available":  "1"
    }
}

on Android it will not work. As you discovered, sending:
{
    "to": "/topics/_all_",
    "data": {
    	"title": "title data",
        "message": "message data",
      	"app": "MOVAPP_GEN",
        "content-available":  "1"
    }
}

will work. The reason you are getting two notifications is because of the "content-available":  "1" setting. This is a signal for your the push plugin to fire the event in the background.
Using FCM on iOS you will need to send:
{
    "to": "/topics/_all_",
    "notification": {
    	"title": "title data",
        "message": "message data",
      	"app": "MOVAPP_GEN"
    }
}

It sucks the payload is different on Android and iOS but that is the way Google made it.

It turns out that this plugin "https://github.com/arnesson/cordova-plugin-firebase" DOES fire the event even when you send the "notification" attribute in android. And debugging in the phonegap plugin, it also receives the payload, so its possible to fire the event too.
This has to be a bug, because it makes us to differentiate 2 payloads depending on the platform.
Another problem I'm facing is that if you don't send the "notification" attribute in android, and you receive several notifications, they are all override with the last one, instead of stacking them in the notification bar.

@markos92i you can use inbox stacking as detailed here: https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#inbox-stacking

That works for now, thank you. But still if you take a look at that plugin it manages to work even with the notification attribute.


dnf-json: use the default connection timeout

By default timeout is 30 seconds, but we had it set to 5. Drop
the override and use the default.
This has two effects: it increases the time before we give up on
connecting (as it says on the tin), and it also increases the time
download has to be slow for before we give up.
Internally, we were seing failures in downlaoding metadata from ODCS
and similar issues have occurred in CI too.
The potential downside to this is in case of having several mirrors
this means it takes longer before giving up on a bad one and trying
a better one. But slow is better than broken, so for now rever to
the default behavior.
Signed-off-by: Tom Gundersen<EMAIL_ADDRESS>This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

See also https://github.com/osbuild/osbuild/pull/990


Podcast talk

Added a page covering my Lightning Talk on 8/20, with video and embeded slides.

Looks good @buddhaplex! Great job, the only thing that I see as a slight issue is the measurements of the slide show but that's easy enough to fix so I can change that quick once I merge into master.
Otherwise great job! Welcome to the open source contributor community!

You rock, Eric! :) :+1:

Thanks, Colin! Appreciate the tweak. And glad that was the only issue. :) good to be officially on board as an Open Source contributor!
E.
Eric Andrade
the barbarian group
917.942.9940
sent by science/magic

On Aug 28, 2015, at 11:33, Colin Rubbert<EMAIL_ADDRESS>wrote:
Looks good @buddhaplex! Great job, the only thing that I see as a slight issue is the measurements of the slide show but that's easy enough to fix so I can change that quick once I merge into master.
Otherwise great job! Welcome to the open source contributor community!
—
Reply to this email directly or view it on GitHub.

Woohoo! Thanks, Ken. Back into some Flixter goodness and coding challenges this weekend!
Eric Andrade
the barbarian group
917.942.9940
sent by science/magic

On Aug 28, 2015, at 11:45, ken mazaika<EMAIL_ADDRESS>wrote:
You rock, Eric! :)
—
Reply to this email directly or view it on GitHub.


Incorrectly states "Intune: Not available" for Block persistence through WMI event subscripton rule

[Enter feedback here]
section for Block persistence through WMI event subscription says Intune name: Not available however this is configurable in the Intune ASR policy.

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

ID: 71a17626-d89b-a06f-9cc6-308b80a84acc
Version Independent ID: 71a17626-d89b-a06f-9cc6-308b80a84acc
Content: Attack surface reduction rules reference
Content Source: microsoft-365/security/defender-endpoint/attack-surface-reduction-rules-reference.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @jweston-1
Microsoft Alias: dansimp

@jrhi Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/10954
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @jrhi Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

This was fixed as part of: https://github.com/MicrosoftDocs/microsoft-365-docs/pull/10954


Combining control flow operators

Does Solid support combining control flow operators? I can't get it to work, not sure what I'm doing wrong. I removed code bit by bit and I think this example captures where the error occurs:
import { createState, createRoot, onCleanup } from 'solid-js'

const CountingComponent = () => {
  const [state, setState] = createState({ counter: 0, data: [1, 2, 3], flag: 3 });

  const interval = setInterval(() => {
    setState({ counter: state.counter + 1, flag: state.counter % 3 })
  }, 1000);

  onCleanup(() => clearInterval(interval));

  return (
    <$ each={state.data} >
      <>
        <$ when={state.flag === 1}>

            Flag is 1.

        </$>
        <$ when={state.flag === 2}>

            Flag is 2.

        </$>
      </>
    </$>
  )
}

createRoot(() =>
  document
    .getElementById("app")
    .appendChild(<CountingComponent />)
);

Once data has been rendered once, it throws the same error 3 times (cannot read property nextSibling of null, in removeNodes), then goes on to get the next error (cannot read property __rGroup of null), where it enters a steady state and keeps throwing this error.
The error is thrown when trying to dismount the last item from state.data. The childnodes of .app are then [#comment, div, #comment], and it passes in the last comment to removeNodes and null as the end marker. Looks like it's either sending in a remaining comment node that should have been removed from a previous item, or passing in the wrong comment node? Not sure.

Thank you for the detailed bug report. This is an area that I've been tinkering with the last few releases (reducing number of comment nodes). I probably introduced something, some newer hole in the logic. I will try to get this fixed as soon as I can, probably later today.

Alright, there was a bug around clear code getting the wrong markers. Fixed v0.8.4. See it working here: https://codesandbox.io/s/combining-flow-xnt7m


docs: Add multi-language inline docs page

I had some difficulty with this page, as the Python examples were different from the others. I have tried to make them all consistent now, but even so differences persist (see inline comments) and the snippets should be reviewed again.
Flagging this for re-review for each code snippet @kpenfound @helderco @TomChv

Deploy preview failed, so I'm rebasing from the web to get it built.

Ok, should have checked the error. Looks like reference docs are failing.

@vikram-dagger, let me know when the preview is fixed. And to ask you to update the outputs from --help based on latest dagger version. 🙏

@vikram-dagger, let me know when the preview is fixed. And to ask you to update the outputs from --help based on latest dagger version. 🙏

Preview is fixed. --help output fixed for first example. Still not able to get the second one to display the descriptions properly, see my inline replies with code and output.


Add "convenience" keys

See my pull request https://github.com/dcrespo3d/ZX-ESPectrum-Wiimote/pull/2

Thanks for merging


FW Script did not ask me to backup ROM

I have an HP Chromebook 11 G6, which corresponds to "snappy / alan", and after removing the battery and running the script, it successfully installed coreboot, however it did not ask me to backup my original ROM like it did on my octopus. Considering I literally don't have the original ROM, how do I even obtain it?

it did not ask me to backup my original ROM like it did on my octopus

it almost certainly did, you just skipped through it. The backup prompt is conditional on the device running stock firmware.

Considering I literally don't have the original ROM, how do I even obtain it?

if you want to go back to stock, then create a recovery USB for your device and then select the option from the script to restore from a recovery USB.

@MrChromebox with all due respect, it did not ask me, but I was moving from SeaBIOS rw_legacy to OpenCore UEFI.
I was asked to backup on octopus with OpenCore rw_legacy to OpenCore UEFI, but Idek.
Thanks for the tutorial on how to restore the stock ROM tho.

I have to assume that pic is several weeks old, since it shows an older firmware version being downloaded?
I don't see how the backup prompt got skipped, it doesn't depend on if you're booted via RW_LEGACY. Unless the firmware version was unable to be read.
FYI, my firmware has nothing to do with OpenCore, that's a MacOS thing IIRC.

I have to assume that pic is several weeks old, since it shows an older firmware version being downloaded?
I don't see how the backup prompt got skipped, it doesn't depend on if you're booted via RW_LEGACY. Unless the firmware version was unable to be read.
FYI, my firmware has nothing to do with OpenCore, that's a MacOS thing IIRC.

pretty sure he means coreboot

I have to assume that pic is several weeks old, since it shows an older firmware version being downloaded?
I don't see how the backup prompt got skipped, it doesn't depend on if you're booted via RW_LEGACY. Unless the firmware version was unable to be read.
FYI, my firmware has nothing to do with OpenCore, that's a MacOS thing IIRC.

hi yeah i meant coreboot, i have someone who messes with macOS a lot and i keep confusing the two


Blog reviews and set up PR

@lpgoldstein
I incorporated all of Maroya's edits into my blog. If you have time, could you please also review? Things I need help with are the title (I put everything in gpt but I didn't like any of its suggestions) and seeing if anything could upset comms approvals.
https://doimspp.sharepoint.com/:w:/r/sites/NRRD/Shared Documents/General/Blog/In Progress/ONRR-content-database_blog.docx?d=w6249943bc46c46c78b04c2b5fcddfaf9&csf=1&web=1&e=P6kUnH

@cthomasONRR of course! Will review this afternoon.


Could not find a version

Describe the bug

I'm can't installed of pythonfetch package. Currently there are 2 versions in PyPI.

Expected behavior

Being able to install the package.
To Reproduce

My Platform

OS: Debian Unstable
PIP Version: 22.0.2
Additional context

https://pypi.org/project/pythonfetch/

Hi, this issue tracker is for issues with the codebase behind https://pypi.org itself, not the projects hosted on PyPI.
You should report this issue to the tracker for the project in question instead.

Sorry, I don't understand. What is problem?


fix bug with HadoopJob constructed command args when command in None in Python client

Coverage increased (+0.03%) to 89.092% when pulling 47653b3d336c87c772c8a37f96f63a0b090d37ec on irontable:master into a077070e81b8f946dd186bd17f944367be4e0762 on Netflix:master.

Coverage remained the same at 89.057% when pulling 24a94ef8db130c7d25bd87f86ff78e23d2592e3e on irontable:master into 04b17724e417a8b5575316801141b184c0230a5a on Netflix:master.


404 on elasticsearch image

Expected behavior
install elasticsearch
Actual behavior
(HTTP code 404) no such image - no such image: elasticsearch:latest: No such image: elasticsearch:latest
Information about the Issue
using kitematic gui
Steps to reproduce the behavior

...
...

Try setting up the tag.
The reason why elasticsearch is getting a 404 is because Kitematic automatically requests for the image with :latest tag.
However, since elastic search does not have the 'latest' tag, it becomes an invalid tag.
Try manually changing the tag to something listed on https://hub.docker.com/_/elasticsearch?tab=tags .

Looks like that repository does not have a :latest tag, so you will have to specify a specific tag to use; here's the steps to do so:

Click on the ... button to view the image's details

Click on the Selected tag: latest to open the "tag selection" window

Click the tag / version of the image you want to run

Click the (X) to close the tag selector window

Click on the Create button to pull the image and start the container


Reflect Default for ComputedVisibility and Handle<T>

Objective

Reflecting Default is required for scripts to create Reflect types at runtime with no static type information.
Reflecting Default on Handle<T> and ComputedVisibility should allow scripts from bevy_mod_js_scripting to actually spawn sprites from scratch, without needing any hand-holding from the host-game.

Solution

Derive ReflectDefault for Handle<T> and ComputedVisiblity.

Changelog

This section is optional. If this was a trivial fix, or has no externally-visible impact, you can delete this section.

The Default trait is now reflected for Handle<T> and ComputedVisibility

Yes, both types already had Default implementations, they just aren't derived, they're implemented manually.

Merging as trivial.
bors r+


bulk transfer issue

Saved the domain list into csv using one user's account and tried to use this file to transfer a domain to a new registrar. Received "object does not exist"
sample file used in staging
Domains_2022-06-30_11.57.csv

OK


Fixed "Team members profile pictures were not showing"

Fixed "Team members profile pictures were not showing"
Fixes #251
Before

After

@smaranjitghose Please have a look at this PR.


Do not create 'tomcat.8080' directory under project root directory

Motivation:
UnmanagedTomcatServiceTest does not specify the base directory of the
Tomcat, which makes the Tomcat create an empty 'tomcat.8080' directory
under the project root directory. It could be created somewhere under
'/target' instead for less distraction.
Modifications:

Set the baseDir of Tomcat explicitly

Result:
Cleaner build

@blmarket PTAL

Good catch :) I just ignored tomcat.8080

@blmarket It's just an empty directory so it didn't bother Git, but it just doesn't belong there. :-)


Algorithm for Singly Linked List.

Okay, I'll do that.
On Wed, Oct 9, 2019, 20:20 Mayank Pathela<EMAIL_ADDRESS>wrote:

@starkblaze01 requested changes on this pull request.
Hi, thanks for the contribution @Anishukla https://github.com/Anishukla.
Can you do a small change? When we are taking K input can print there
K = int(input('choose an option between 1 to 9'))

In SinglyLinkedList/SinglyLinkedList.py
https://github.com/starkblaze01/Algorithms/pull/39#discussion_r333059233
:

       position = position+1

       c = 0

       if temp.data == value:

           c = c+1

           print(position)

   if c == 0:

       print("The value isn't present.")

+mylist = LinkedList()
+print("Choose 1.To append at start, 2.To append in between, 3.To append at end." )
+print("4.To delete from start, 5.To delete from between, 6. To delete from end.")
+print("7.To View Linkedlist, 8. Search a value's position, 9.To view and exit.")
+
+while(1):

K = int(input())

Hi, thanks for the contribution @Anishukla https://github.com/Anishukla.
Can you do a small change? When we are taking K input can print there
K = int(input('choose an option between 1 to 9'))
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/starkblaze01/Algorithms/pull/39?email_source=notifications&email_token=AIJH5W6NLMEB3L5MQJEHGE3QNXVSXA5CNFSM4I62MZ7KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHMZF6I#pullrequestreview-299471609,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIJH5W4GJ2SRMT7LBIODBODQNXVSXANCNFSM4I62MZ7A
.

I've made the changes. You can check it now!

 Will try and add some more algorithms soon!


User Management - Name Search not working

When the user searches for a name in the User management search, no results are showing.

Search bar must have a capital “N” in “name”
Private Zenhub Image

Private Zenhub Image

Verified.


Build Instructions

Can you provide brief build instructions somewhere? It would be preferable to include links to resources/software. Non-programmers can compile a program to access pre-release code features and even tweak a few things here and there.

I think porviding  a file named HowtoBuild.md just under top directory is nice.
The way is farely simple.
PreCondition:

The whole XstReader is placed in a PC by doing git pull or downloading source zip.

Procedure:

Download and install VisualStudio Community from https://visualstudio.microsoft.com/

Note that C# development is included.

Double click XstReader.csproj
Compile and run in the VisualStudio.

Could you try and contribute? > flywire

I agree the idea of programmer writing noob instructions is dumb. Assign it to me and I'll seek your guidance as required.

Simple instructions added. Thanks halueda!


Add a prompt about the slider.

@ajbozarth Actually I looked at the UI again and I think we should do something about it. In some cases no object is detected and the user might be very confused about what is going on in the app. In this scenario it might make sense to add a note besides the image to suggest the user to lower the threshold.

imho I disagree, the ui itself is simplistic. Once detecting is complete there is only one remaining UI feature (if nothing is detected) and that's the slider. The slider itself has a tooltip explaining this issue. I believe cluttering up the UI for an edge case that seems clear (to me) is not necessary.
If you were to propose a UI update that somehow highlights the threshold slider feature once detection is complete if no boxes are present at the current threshold, I would be open to that, but just adding text is poor UX.

I agree that we should not clutter the UI, and the added text may not be a good idea, but my main point of issue is the confusion. Suppose the user uploads a picture and no object is detected at the threshold of 50% (which is not so unlikely). Now the only change that the user sees is the uploaded image staying on the webpage. What should the user expect? Should the user be waiting? Should the user click on somewhere? I'm happy with any solution that dissolves this confusion, not necessarily adding text.

I understand and agree with your statement. So that raises two points from me.

I believe the current UI handles that confusion in an okay way. Specifically in the case described above, the user is left with a UI with just an image, but there is only one "interactive" UX element for them to use, that is the slider. So my argument is that they would go to that. Also the slider works even when the image is being processed.

If you think my UX argument above is too flimsy I will describe a good UX alternative that does not rely on text but is outside my current dev time. Add a UI function that "highlights" the slider in a way to draw attention to it. This could be changing the box border to red or another color, just something subtle but distinct to say " try this slider". This function would then be called if, on detection completion, the current threshold doesn't show any boxes. It would then be cleared on any UI interaction.

So again I believe the UI is simple enough that the confusion your raising wouldn't happen, but if you're willing to put in the dev time to add a highlighting feature like I've described above I would be willing to review (and even polish) it.


Output to quickfix

I would like to suggest to display the output in quickfix directly. Like in https://github.com/skywind3000/asyncrun.vim or https://github.com/Shatur/neovim-cmake.

Thats a good idea, but i think we should have a generic run_command setting so that we can support floatterm and this usecase / more

So i have an alpha version up and running, but I can't get the qflist to parse the errors using 'erroformat', would you have any idea how to do that? cause without it it's kinda useless to have it in the qflist

You probably want to set it to something like this: https://github.com/rust-lang/rust.vim/blob/4aa69b84c8a58fcec6b6dad6fe244b916b1cf830/compiler/rustc.vim#L32


Autocomplete

Two things in this pull request:

Addresses #39
An a11y fix for the text input. A placeholder is not enough to make it accessible, so now an aria-label attribute mirrors the placeholder.

You're awesome, Ray! Merging :dancer:


ci: run Crowdin every night

What
CI: run Crowdin every night
This will save me a few minutes each day

Codecov Report

Merging #1597 (5ebda30) into develop (2ea0da3) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           develop   #1597      +/-   ##
==========================================
- Coverage     8.86%   8.85%   -0.01%
==========================================
  Files          161     161
  Lines         6623    6628       +5
==========================================
  Hits           587     587
- Misses        6036    6041       +5

Impacted Files
Coverage Δ

...mooth_app/lib/generic_lib/widgets/smooth_card.dart
0.00% <ø> (ø)

...oth_app/lib/pages/onboarding/country_selector.dart
0.00% <0.00%> (ø)

...ges/smooth_app/lib/pages/product/summary_card.dart
0.00% <0.00%> (ø)

...es/smooth_app/lib/pages/scan/ml_kit_scan_page.dart
0.96% <0.00%> (ø)

Continue to review full report at Codecov.

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


Release a Firefox addon version?

Hi, I used to use this extension on Edge. But when I switched to Firefox, I haven't found an alternative version on Firefox yet.
I found that if substituting the chrome object with browser, e.g. chrome.tabs to browser.tabs, then the extension can also work on Firefox.
Now I'm using this extension on Firefox by loading temporary add-on, but I think that it would be better if it's released in Firefox Add-ons.  Any plan for this? Thanks.

yes please make this work.


UniTensor constructor, name argument

I suggest to put "name" as an argument in the UniTensor constructor

we will add this in dev-master.

8977341


deleting bullet from normal makes things weird

- [XDG] compliance wherever possible to keep `$HOME` clean
  - See [Arch Linux wiki for XDG Base Directory Support]
  - See [Debian DotFilesList]
  - See [grawity's notes] and [environ notes]
- RC files for Lua, markdownlint, node, PHP, python, R, and others

cursor on first bullet, hit x to delete
expected: no change to virtual bullets
observed: virtual bullet is rendered on second line first column

Interesting, thanks for reporting this bug!
I found that it only happens if you go from the first item in a list directly to a nested list.
So you get the same bug with this list:
- List Item 1
  - Nested List Item

But not with this list:
- List Item 1
- List Item 2
  - Nested List Item

I'll see if I can figure out what's going on with the parser, I imagine something to do with starting with a nested list isn't being handled ideally, I can probably add some simple condition to at least avoid the weird rendering.

Issue should be resolved by: https://github.com/MeanderingProgrammer/markdown.nvim/commit/df98da81375e5dc613c3b1eaa915a847059d48d9


 This commit
relaxes the assert condition for that query.
Closes #284

This test isn't necessarily the best thing to check but if we do want to keep it with some relaxed constraints then maybe we allow the norm to go higher by a small amount iff the total loss is lower. So something along the lines of
norm2(l2.coef) < norm2(noreg.coef) OR
    ( (norm2(l2.coef)-norm2(noreg.coef))/norm2(noreg.coef) < 0.1 AND l2.loss < noreg.loss),

Thank you for the comment @iyerr3 , will relax the constraint as suggested.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/528/


feat(tier4_planning_rviz_plugin): add offset from baselink param

Description
Add param for offset_from_baselink in the footprint visualization.
The aim is to visualize the difference in footprint when following different coordinate systems for a trajectory.
This analysis is required in path planning, e.g. the obstacle_avoidance_planner has the same parameter that sets the offset length from baselink for the path optimization algorithm.

When the trajectory is followed in the baselink the footprint is shown as the yellow polygon, and vehicle center coodrinate makes the red one.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Why is this visualization necessary?

@yukkysaito I update the docs for the purpose

The aim is to visualize the difference in footprint when following different coordinate systems for a trajectory.
This analysis is required in path planning, e.g. the obstacle_avoidance_planner has the same parameter that sets the offset length from baselink for the path optimization algorithm.

The aim is to visualize the difference in footprint when following different coordinate systems for a trajectory.
This analysis is required in path planning, e.g. the obstacle_avoidance_planner has the same parameter that sets the offset length from baselink for the path optimization algorithm.

Just memo.
Footprints with offset_from_baselink ≒ 0 are kinematically infieasible, and the car cannot follow the footprints.
Obstacle avoidance planner solves this infeasiblity problem with optimizaiton.


On-Premises Exchange Security Configuration first commit

Change(s):

First PR of new solution "ESI - On-Premises Exchange Security Configuration" without package

Reason for Change(s):

Adding new REST API Connector with its parser and the first Workbook

Version Updated:

First version 1.0.0

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

No

@v-spadarthi Can you please look into this

@nlepagnez  Parser query is falling please take a look and resolve it. thanks!!!

@nlepagnez  Workbook query's is also failing could you please check it once. thanks!!!

@nlepagnez Parser query is falling please take a look and resolve it. thanks!!!

Hi @v-marimanda, as you can read at the beginning of the parser, that you need to save the parser as a function called "ExchangeConfiguration" and this function has parameters. SpecificSectionList is one of the 4th parameters.
Also for the Workbook, the "ExchangeConfiguration" parser function is needed. If you have this error, it's because you didn't follow the installation steps and you didn't savce the parser as function.
I just added a new instruction part on Connectors to be more explicit than just the previous "Note".

Hi @v-marimanda, @v-spadarthi, there is any update on that PR ?

@nlepagnez  I have followed all the steps you have mention on parser but i could only one sample data attached.
I have used this sample data for ESIExchangeConfig_CL.  can i use the same sample data for another table ESIExchangeOnlineConfig_CL, If not please provide sample data for ESIExchangeOnlineConfig_CL. Currently i am getting below error as i don't have any table with name  ESIExchangeOnlineConfig_CL.

@v-marimanda, yes you can use the same content for the other table.
But what you say is something interesting as we can have condition where one of the two tables doesn't exist and in this case, the parser has errors.
I have to work on the parser to see how to avoid that.

@v-marimanda you can find an update of the parsers with the fuzzy mode to be able to be agnostic of missing tables.
This can allow you to be able to finish your tests.

@nlepagnez  thank for update we are checking ...

@v-laanjana : Could you please provide an update on this.

@v-laanjana : Could you please provide an update on this.

Hi @v-laanjana,
Did you get a chance to check on this. Can you please provide an update.
Thanks

@v-mchatla  we are working ..

Hi @v-laanjana this is a long pending one. Can you please have a look?

@v-laanjana , @v-marimanda, is it possible to have any update ?

Hi @nlepagnez,
I have save the parsers  with parameter's and sending default value as well but still Workbook query's are failing can you please take a look once for your reference i am attaching screen shot below.
Workbook

Parsers

Hi @nlepagnez  can you please address the above comment. thanks!!!

Hi @nlepagnez can you please address the above comment. thanks!!!

Hi @v-marimanda, I think it's because the sample data are too old, as we can't use TimeGenerated for filtering but another property. How do you fill the sample data in your table ? I want to try in my side and be sure that the Sample data is correct before launching a new round.

@v-marimanda, I have just updated Sample Data, Parser and Workbook so that they can show you results on potentially old "Sample" data (which will never be the case in normal use).

Hi @nlepagnez,
Thanks for sharing the sample data. We will check and provide more details once done.
Thanks

Hi @nlepagnez,
Sorry for the delay. I'm going to start working on it today and provide you the update by EOD.
Thanks

Hi @nlepagnez, Sorry for the delay. I'm going to start working on it today and provide you the update by EOD. Thanks

@v-mchatla Perfect. Do not hesitate to ask me everything if you need.

ExchangeConfiguration Parser is working fine
Getting below issue for the other parser ESI_ExchConfigAvailableEnvironments

Can you please have a look at it and let me know if I'm missing something.
Thanks

ExchangeConfiguration Parser is working fine Getting below issue for the other parser ESI_ExchConfigAvailableEnvironments  Can you please have a look at it and let me know if I'm missing something. Thanks

Hi @v-mchatla, the error message indicates that the tables ESIExchangeConfig_CL and/or ESIExchangeOnlineConfig_CL are missing. Did you create one of them with sample data ?

Hi @nlepagnez
I have already dumped the data to Log Analytics as you can see below.

Thanks

Parsers are working fine. Need your help in fixing the below workbook related issues.

Workbook should be of json format, please update it accordingly.
Role Assignee dropdown should populate with some data but its not fetching any data while rendering the workbook page.

Getting below issue for ExchangeAdminAuditLogs

Workbook is referring data from watchlist (ExchCUSU), please share those watchlist data to test end to end.

server data is not rendering in the dropdown.

Thanks

Parsers are working fine. Need your help in fixing the below workbook related issues.

Workbook should be of json format, please update it accordingly.
Role Assignee dropdown should populate with some data but its not fetching any data while rendering the workbook page.

Getting below issue for ExchangeAdminAuditLogs

Workbook is referring data from watchlist (ExchCUSU), please share those watchlist data to test end to end.

server data is not rendering in the dropdown.

Thanks

Hi @v-mchatla,

For the point 1, I transformed the Workbook to JSON Extension file.
For the point 2 and 5, with gave you a sample data file but with only 2 to 3 entries. There are many and many entries from a production environment and Workbooks are made to display Real production data. I update the sample data with a complete simulated infrastructure data file.
For the point 3, we removed the part calling ExchangeAdminAuditLogs that is planned for a V2 of this solution package, we want to validate a simple V1 version before adding all connectors, workbooks, analytic rules, etc..
For the point 4, sorry, we have changed our strategy since August on how we want to manage this list. The new version of the Workbook that I commit reflect that.

I hope this new commit will be good for you.

Hi @nlepagnez,
With the Latest sample data Role Assignee details are auto populated. But help page is not rendering any data. Is that okay?

Rest all things looks good.
Please fix the validation errors to proceed with approval and merge.
Hope the below clues can help you in resolving the validation errors.

should not contain hardcoded values.

should not contain reference to any resources.

@v-mchatla,
I removed the hardcoded values and commit the new version.
It's normal that help page is empty for the moment.

@nlepagnez please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

Can you please respond to this with your company details

@microsoft-github-policy-service agree [company="Microsoft"]

@microsoft-github-policy-service agree company="Microsoft"

Hi @v-mchatla, I have accepted the CLA but it's marked as queued. Is it normal or do I have to do something ?

Hi @v-mchatla, I have accepted the CLA but it's marked as queued. Is it normal or do I have to do something ?

Hi @nlepagnez,
Unless and until the below check is passed, I can't merge the pr. Let me also check with my team if you can do anything for this case.

Hi @nlepagnez,
Can you try signing again, I will also check with team and provide you the update
Thanks

@microsoft-github-policy-service agree company="Microsoft"

Hi @nlepagnez, Can you try signing again, I will also check with team and provide you the update Thanks

Hi @v-mchatla I tried to sign again, but nothing changes. Is it because it's "Microsoft" in company name ?

Hi @nlepagnez,
Can you just try sending only the below command.
@microsoft-github-policy-service agree
Thanks

@microsoft-github-policy-service agree

Hi @nlepagnez,
Can you please go through the steps specified here
Thanks

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

@v-mchatla I think you can now approve the merge as CLA is validated.

Hi @nlepagnez,
Thank you for your continued support. It would be great if you can share the steps you followed to fix the CLA issue as lot of other GitHub users are also facing the similar issue, because of that we are not able to accept their contribution.
Thanks

Hi @nlepagnez, Thank you for your continued support. It would be great if you can share the steps you followed to fix the CLA issue as lot of other GitHub users are also facing the similar issue, because of that we are not able to accept their contribution. Thanks

Hi @v-mchatla, Using the link you sent me about the CLA application, I installed it on my Github account. I have created the ".Github" repository and begin to configure it, but I see that the CLA approval was validated before I finished to configure the .Github repo.
So to resolve the problem, for sure they need to install the CLA Application in their Github account, but after it's sure that they need to create the .Github repo or not.


Create kubeconfig: Respect namespace

Currently, the combined kubeconfig will always be for all namespaces.
With this change, it will be for the configured namespace only. In order
to allow getting kubeconfigs for clusters in all namespaces, the
--all-namespaces/-a flag is added.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm


Perform release-related activities for 20.06

A number of checks have to be performed on the evidence files submitted by data providers as they arrive and before the pipeline is run to see whether the agreed changes have been implemented.

TEPs checked on May 20th and there are no updates

EuropePMC evidence file received on May 23. It was checked and it looks good:

9,790,732 evidence strings (up from 9,655,854 in 20.04)
Valid JSON
1,287 evidence strings annotated with COVID-19 (MONDO_0100096). However, the Hoffmann et al. Cell paper is not among them, checking with Shyama why this might be.

PheWAS catalog evidence updated on 26th May:

A few traits for which new terms were requested to be added to EFO now have a mapping, e.g. Abnormal chest sounds - see updates
Unique association fields changed to reduce the number of duplicates (see commit):

phewas_string replaced by phewas_string_and_code, e.g. "Colorectal cancer" vs "Colorectal cancer [153]"
Variant id added (variant_id)

191k evidence strings in total (compared to 187k in 20.04)

ChEMBL file received on 27th May:

434878 evidence strings (up from 416,174 in 20.04)
Valid JSON
ChEMBL version in evidence strings (evidence.target2drug.provenance_type.database.version) changed from "26" to "27" as it was generated based on ChEMBL 27. New file called cttv008-27-05-2020.json.gz and uploaded to the ChEMBL Google bucket.
It contains 761 evidence strings annptated with COVID-19 (MONDO_0100096)

Baseline and differential expression files received on 12th May.

Baseline:

Data processed using Ensembl 79 and Ensembl 95 genome references depending on data set. Inconsistencies observed
Bins and z-scores computed

Differential:

Data from Expression Atlas release 35
375,338 evidence strings received, which end in 221,633 after filtering based on activity (up from 215,921 in 20.04)
Filtering still performed in-house
Valid JSON

Chemical Probes updated on 1st June:

One new probe (NVS-MALT1) in SGC, there are 77 in total.
12 new probes in Chemical Probes Portal: ABBV-744, AT1, BAY-885, CM11, Eleutherobin, Glyburide, ICI-199441, NI-57, PFI-3, RO2468, T-26c, UCSF7447. There are 135 probes in total.

Eleutherobin is a particular case as it targets tubulin, which is an structural protein formed by the alpha- and beta-tubulins. Given that protein complexes are not considered in the platform yet, it has been decided to assign this chemical probe to all the alpha- and beta-tubulins in Open Targets, which is a total of 17. A better solution for this will be thought of in the future.

There are still 65 chemical probes in Open Science Probes/Donated Chemical Probes portal, there not been any changes

The first 20.06 pipeline run has failed due to some ChEMBL evidence strings having the evidence.drug2clinic.date_asserted in the yyyy-dd-mm format instead of yyyy-mm-dd. ChEMBL have generated a new evidence file with the correct format and OT have changed the JSON schema to capture this issue (see #1090 and PR #87). The new ChEMBL evidence file looks good:

It has the same number of evidence strings as the previous one
It’s valid against the new json schema version (1.6.8)
The date of the offending 5 evidence strings that was 2016-25-11T00:00:00.000Z is now 2016-11-25T00:00:00.000Z

EVA file received on 22nd May:

125,138 evidence strings (up from 122,219 in 20.04)
Valid JSON
There are still 20 evidence strings where the condition is Alhzeimer disease (AD) and they are mapped to EFO_0000249  . These were manually mapped to Early-onset autosomal dominant Alzheimer disease (Orphanet_1020) - see #104
60 trinucleotide repeat expansions in evidence file thanks to new variant annotation pipeline developed by EVA, which identified 85 trinucleotide repeat expansions in ClinVar. According to Kirill, the majority of missing ones are because their clinical significance is bening or not provided. However, there are about 10 variants that lack trait-to-ontology mapping and need to be checked manually. This will be revisited in the next release.

Python script to generate metrics revamped and bash script included so that now all the information needed to fill the release metrics spreadsheets is created in one go and it requires very little editing. See PR #2


Getting Started with flakes instructions don't work

Describe the bug
The Getting Started guide for working with flakes says to run,
$ nix flake init --template github:cachix/devenv

First, the copy to clipboard button on the web page includes the dollar sign prompt, which produces an error if simply pasted into a terminal. After deleting that, we see another error,
error: cannot find flake attribute 'github:cachix/devenv#defaultTemplate'

Maybe the intended text should be,
nix flake init --template github:cachix/devenv#simple

but I encountered other errors with that, so perhaps it is not correct either.
To reproduce
Follow the getting started guide for users of flakes.
Version
The errors occur before devenv is available to run.

I have the same problem, my dev environment keeps printing command grep not found


CRUD Around User Inputs and Code snippets

After writings some of the create methods for the two controllers I realized I only needed the Index's, Edit, and Delete to accomplish what I want to.