Acrylic Effect in the terminal is not working in Windows High Contrast

Environment
Windows build number: Microsoft Windows [Version 10.0.18362.592]
Windows Terminal version (if applicable): 0.8

Any other software?

Steps to reproduce
Change the Windows theme to High Contrast (black). Open Windows Terminal. Notice how the acrylic effect just disappear and scrolling the mouse wheel while pressing the SHIFT key doesn't  change the opacity either.
Expected behavior
Changing the Windows theme to High Contrast (black) should allow to still enjoying the acrylic effect.
Actual behavior
Changing the Windows theme to High Contrast (black) is not allowing to have acrylic effect in the terminal.
When you scroll the mouse wheel while pressing the SHIFT key, it doesn't  changes the opacity either.

High contrast is an accessibility feature to aid the visually impaired. Acrylic is very difficult for that group of people to perceive well, so it is disabled on all Windows applications when high contrast is enabled.
This one is by design. Thanks!

Well, this is a shame. It would be great Windows Terminal could implement it as the old cmd is doing now:

It could be argued that the legacy console host should suppress transparency in HC mode.


feat(sheets-data-validation): add validator service allowing developer to get status of data-validation

close #2404

Pull Request Checklist

[x] Related tickets or issues have been linked in the PR description (or missing issue).
[x] Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
[x] Unit tests have been added for the changes (if applicable).
[x] Breaking changes have been documented (or no breaking changes introduced in this PR).

import { SheetsDataValidationValidatorService } from '@univerjs/sheets-data-validation';

// get injector from you univer instance
const injector = univer.__getInjector();
const validatorService = injector.get(SheetsDataValidationValidatorService);

validatorService.validatorCell(unitId, sheetId, row, col);

validatorService.validatorWorksheet(unitId, sheetId);

validatorService.validatorWorkbook(unitId);


[Instagram] Failed to load resource: The file “embed.js” couldn’t be opened because there is no such file.

I am faced with Failed to load resource: The file “embed.js” couldn’t be opened because there is no such file. instagram embed, the rest of social medias work fine.
Device: iPhone 12
IOS version: 17.1.2
The project uses Capacitor, so it's reproducible on Application based on Capacitor cross-platform native runtime and only on iPhone with new IOS version.

I found solution.
https://github.com/justinmahar/react-social-media-embed/blob/master/src/components/embeds/InstagramEmbed.tsx#L10
'//www.instagram.com/embed.js' value should be 'https://www.instagram.com/embed.js'

@justinmahar thank you so much for quick reaction, I appreciate it. For me, everything works correctly with internal testing.
I will inform you if I have any cases regarding this issue but right now it works.


[Documentation] "Option to set forward DNS servers when enabling DNS"

According to the 1.20 release notes there is an "Option to set forward DNS servers when enabling DNS".
Would absolutely love to use this, but it's not documented.

I think you can do this microk8s enable dns:/etc/resolv.conf

I think you can do this microk8s enable dns:/etc/resolv.conf

Just updated the doc page at https://discuss.kubernetes.io/t/add-on-dns/11287

Just updated the doc page at https://discuss.kubernetes.io/t/add-on-dns/11287

Thanks @Ramblurr the documentation has been updated.

Thanks @Ramblurr the documentation has been updated.


API-NUMPY-METADATA

Replace this paragraph with a description of your proposed feature.
Please be sure to describe some concrete use cases for the new feature -- be as specific as possible.
Provide links to existing issues or external references/discussions, if appropriate.

I don't think that the issue is clear enough to keep it open, while the description looks copy-pasted from a template. Feel free to reopen with more details provided.

It Recovered DP-1 of Modular apple swift RLP


after backpacking cloud requests sprite and moving it to another project, it is unable to connect to scratchattach

ive tried multiple times and the only way i can get it to work is if i make a new project and then import the cloud requests template into scratch and then move all my sprites into that project.

Maybe the cloud variables were converted to normal variables (convert them back to cloud variables) or the @timeout variable is zero (put it to 10)

Backpacking the sprite will break the cloud variables. Instead of backpacking the cloud requests sprite, backpack all other sprites and move them to the cloud requests project (there are other possible solutions too, you could create all necessary cloud variables before importing the CR sprite from a sprite3 file, then it would work too)


Create a Navbar

using bootsrap add navbar.

go ahead

please check my pr

merged @nilaygit-10721

my pr https://github.com/Ctoic/CalHub/pull/99

can you please add a git ignore file to add the node modules ?

@nilaygit-10721

i made it please check my pr


Call out CRDs in default discovery roles

@philips PTAL

I am OK with this

/lgtm

/lgtm

/approve


DEV - Failed to start since new upate

Expected Behavior
Actual Behavior
after updating nothing works..
configs are in configs-dir
and for what is the "source bin/activate" command? it doesnt work on windows...
C:\Users\User\Desktop\PokemonGo-Bot>python pokecli.py
[2016-07-25 12:50:38] [x] No config argument specified, checking for /configs/config.json
Traceback (most recent call last):
File "pokecli.py", line 228, in
main()
File "pokecli.py", line 205, in main
config = init_config()
File "pokecli.py", line 65, in init_config
load.update(json.load(data))
File "C:\Python27\lib\json_init_.py", line 291, in load
**kw)
File "C:\Python27\lib\json_init_.py", line 339, in loads
return _default_decoder.decode(s)
File "C:\Python27\lib\json\decoder.py", line 364, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "C:\Python27\lib\json\decoder.py", line 380, in raw_decode
obj, end = self.scan_once(s, idx)
ValueError: Expecting , delimiter: line 17 column 2 (char 467)
Steps to Reproduce
Other Information
OS: windows 10 x64
Git Commit: b8606bc509ca89902441b213f5ee7996cca29137

Can you copy paste your config.json file (take our your username and password)?

sure
{
"auth_service": "google",
"username"<EMAIL_ADDRESS>"password": " ",
"location": "Schloss Johannisburg 63739 Aschaffenburg",
"gmapkey": "",
"max_steps": 7,
"mode": "all",
"walk": 4.16,
"debug": false,
"test": false,
"initial_transfer": 0,
"location_cache": true,
"distance_unit": "km",
"item_filter": "101,102,103,104",
"evolve_all": "NONE",
"use_lucky_egg": false
}

Devs, information on the bug here : http://stackoverflow.com/q/28223242/4884034 (I can't code at the moment).
For your question about source/bin/activate it is used to have a separate virtual environment, you can read about it here https://virtualenv.pypa.io/en/stable/

okay so i was having a similar problem on line 16.
this is my solution
{
"auth_service": "google",
"username": "YOUR_USERNAME",
"password": "YOUR_PASSWORD",
"location": "SOME_LOCATION",
"gmapkey": "GOOGLE_MAPS_API_KEY",
"max_steps": 5,
"mode": "all",
"walk": 4.16,
"debug": false,
"test": false,
"initial_transfer": 0,
"location_cache": true,
"distance_unit": "km",
"item_filter": [""],
"evolve_all": "NONE",
"use_lucky_egg": false,
"evolve_captured": false,
"catch": {
"any": {"catch_above_cp": "0", "catch_above_iv": "0"},
"logic": "or"
},
"release": {
"any": {"release_under_cp": "0", "release_under_iv": "0"},
"logic": "or"
}
}


No 'Access-Control-Allow-Origin' header

I was trying to consume your Bible web AP via AJAX and and i got this error :
No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8100'
Please kindly assist.

This issue is due to CORS blocking the request, maybe @seven1m can enable cross-origin requests on the server. For now you could use a free CORS proxy service such as crossorigin.me


Fails to reload script or load world after closing world. Client restart is needed.

I added a lot of thaumcraft mods to FTB Infinity. I had more than 18 mods, so thoumonicon (Probably spelled it wrong) tabs overlapped. Other suggested using mod tweaker. So i did some moving. I even removed some research, because it was a copy of a duplicate and created new connection to original.
Upon testing, my script failed to reload because of script modifying research tabs for the mod Thaumaturgical Knowledge. Script works, expect i can not reload or even reload world after the use.
In logs, this part is intresting:
Intresting part: http://pastebin.com/1GA7E2bM
Complete log: (Too large for pastebin)
My Script: http://pastebin.com/aVL8RKXN

Thaumcraft cannot be reloaded. End of story.


Use go 1.11 and alpine 3.8

Pull Request Test Coverage Report for Build 238

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.727%

Totals

Change from base Build 232:
0.0%

Covered Lines:
128

Relevant Lines:
176

💛  - Coveralls


adding stationary distribution code

Calculates the stationary distribution given a transition matrix.

Hello @shak360! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file lineage/Analyze.py:

Line 81:75: W291 trailing whitespace
Line 83:80: E501 line too long (85 > 79 characters)
Line 84:80: E501 line too long (85 > 79 characters)
Line 86:51: W291 trailing whitespace
Line 87:1: W293 blank line contains whitespace
Line 89:1: W293 blank line contains whitespace
Line 91:1: W293 blank line contains whitespace
Line 93:1: W293 blank line contains whitespace
Line 96:80: E501 line too long (86 > 79 characters)
Line 97:80: E501 line too long (88 > 79 characters)
Line 99:1: W293 blank line contains whitespace
Line 106:1: W293 blank line contains whitespace
Line 112:1: W293 blank line contains whitespace
Line 116:1: W293 blank line contains whitespace
Line 122:1: W293 blank line contains whitespace
Line 124:64: W291 trailing whitespace
Line 127:1: W293 blank line contains whitespace
Line 129:1: W293 blank line contains whitespace
Line 130:60: W291 trailing whitespace
Line 135:28: E261 at least two spaces before inline comment
Line 136:20: E231 missing whitespace after ','
Line 136:31: E231 missing whitespace after ','
Line 136:36: E261 at least two spaces before inline comment
Line 137:25: E231 missing whitespace after ','
Line 138:22: E231 missing whitespace after ','
Line 138:26: E261 at least two spaces before inline comment
Line 139:8: E231 missing whitespace after ','
Line 140:29: E231 missing whitespace after ','
Line 140:43: E231 missing whitespace after ','
Line 141:53: E231 missing whitespace after ','
Line 141:75: E231 missing whitespace after ','
Line 141:80: W291 trailing whitespace

This is the right stationary distribution if our phenotype of interest isn't growth. However, if the phenotype influences the relative abundance of each subpopulation, then we have to account for that. For example, if you have the same transition matrix, but state A grows 10X faster, that should change the abundance of A. I think it's probably something as simple as scaling each population by its relative growth rate, but I'm not sure.

Linted. Should be ready for review/merge @aarmey .


Add support for Float in MovementDeltaPacket

As the new protocol uses both ints and floats this preserves both cases and should be backwards compatible.

Duplicate of #81


follow channel function appearing twice

Thanks @catuss-a! this change was fixed in #11 also and I hadn't merged it yet.


The new helm chart is missing the additionalVolumes property that the old one had

Hello !
Here is the property from the old helm chart I'm talking about
https://github.com/neo4j-contrib/neo4j-helm/blob/master/values.yaml#L153
I was using that to add more volumes to the pod.
With this I was able to mount a configmap script into a init-container that automatically restore the latest neo4j backup taken if the PVC is empty. That was pretty useful to me.
Are you guys thinking of implementing this back in the new one ?
Thanks !

@org-ci-cd this is added to the latest release of 4.4.10 helm charts
https://github.com/neo4j/helm-charts/releases/tag/4.4.10


[4.1] Fix for getting table name with prefix.

refs: #3119

A new inspection was created.

The inspection completed: 5 updated code elements

Thank you! Waiting for your final thumbs up @pxpm & @mgralikowski !

This PR is only partial solution. This and https://github.com/Laravel-Backpack/CRUD/pull/3126 must be accepted to final fix.

Ready to merge with #3126

Awesome. Thanks for the help (and the patience) everyone. Merged into master. It'll be available later today as 4.1.18.


[Feature/extensions] Add contentParser method to ExtensionRestRequest

Signed-off-by: Daniel Widdis<EMAIL_ADDRESS>Companion PR: TBD
This PR does not change the API and be reviewed/merged before the SDK portion of the linked issue. This will also enable AD integration work!
Description
Adds a contentParser() method to the ExtensionRestRequest class to enable extensions to parse REST requests with bodies containing JSON, SMILE, YAML, or CBOR.
Issues Resolved
OpenSearch part of SDK #178
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

Codecov Report

Merging #4760 (c60eddf) into feature/extensions (dfeefa6) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@                   Coverage Diff                    @@
##             feature/extensions    #4760      +/-   ##
========================================================
- Coverage                 70.71%   70.66%   -0.06%
+ Complexity                57896    57884      -12
========================================================
  Files                      4706     4706
  Lines                    277831   277834       +3
  Branches                  40389    40389
========================================================
- Hits                     196481   196330     -151
- Misses                    65114    65250     +136
- Partials                  16236    16254      +18

Impacted Files
Coverage Δ

...ensearch/extensions/rest/ExtensionRestRequest.java
77.61% <100.00%> (+2.61%)
:arrow_up:

...ava/org/opensearch/action/NoSuchNodeException.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...cluster/coordination/PendingClusterStateStats.java
20.00% <0.00%> (-48.00%)
:arrow_down:

...arch/search/aggregations/pipeline/LinearModel.java
23.07% <0.00%> (-46.16%)
:arrow_down:

...opensearch/persistent/PersistentTasksExecutor.java
22.22% <0.00%> (-44.45%)
:arrow_down:

...nsearch/discovery/InitializeExtensionsRequest.java
0.00% <0.00%> (-40.00%)
:arrow_down:

.../admin/cluster/reroute/ClusterRerouteResponse.java
60.00% <0.00%> (-40.00%)
:arrow_down:

...h/action/ingest/SimulateDocumentVerboseResult.java
60.71% <0.00%> (-39.29%)
:arrow_down:

...luster/routing/allocation/RoutingExplanations.java
62.06% <0.00%> (-37.94%)
:arrow_down:

...cluster/coordination/PublishClusterStateStats.java
33.33% <0.00%> (-37.51%)
:arrow_down:

... and 482 more

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


API Documentation

This is a placeholder for when the Product/Engineering teams indicate that the API is ready to be documented, at which point specifics can be added.
These are the subtasks currently targeted for the initial iteration:

[x] #857
[x] #920
[x] #996

Add a API quickstart page under the (to be added) API section, SURE-6847

Hi @stefanlasiewski, the Swagger component is under the v2.8 (preview) docs set at https://ranchermanager.docs.rancher.com/v2.8/api/api-reference. Once v2.8 is released, it'll be mirrored to the latest docs set as well.


Move external picosha2 library to extern folder

The extern folder holds third party libraries. This file https://github.com/etternagame/etterna/blob/develop/src/Etterna/Globals/picosha2.h is a header only library and should be in extern. This will involve making a folder for it in extern, writing the cmake for it (Which as a single header file should be simple) and then changing the #include's to point to the new location (I'm not sure what path we should use, might want to search for an example usage of the library)

#830


mysqli_real_escape_string Cant fully prevent SQL Injection

mysqli_real_escape_string CAN PREVENT SQL Injection
BUT
it cant FULLY PREVENT SQL Injection.
So,Its Safe But Unsafe? xD
https://stackoverflow.com/questions/5741187/sql-injection-that-gets-around-mysql-real-escape-string

Fo FIx It (Copied From Stackoverflow xD)
Use Modern Versions of MySQL (late 5.1, all 5.5, 5.6, etc) AND mysql_set_charset() / $mysqli->set_charset() / PDO's DSN charset parameter (in PHP ≥ 5.3.6)
Or
Don't use a vulnerable character set for connection encoding (you only use utf8 / latin1 / ascii / etc)

Mofhy is expired

🤦

🙄

This time it is permanently closed and will no longer receive any updates from vendor

why bro?

There are some issues i can't resolve either.

what are the issues? the community is there 🤔

yep

its easy to resolve :) just use PDO

Xss and some tokenized system issue.
I'm thinking to make a new application based on codeigniter in order to make it work smoother and secure.

Go for it!

Well the client, and admin area login needs to be redone as well. Testing on my own install, both can be hacked. Not sharing details since, well, that should be obvious.

wtf it just 1 day then lot of comment

wtf it just 1 day then lot of comment

So what?

Can something like this being implemented correct? Reference:
https://youtu.be/I4JYwRIjX6c?t=996

Can something like this being implemented correct? Reference: https://youtu.be/I4JYwRIjX6c?t=996

Yes can be.

Hey! I found a fix, it works on my local install. I tried with OOT and works! Will send a link to the new Repo since here it is no more maintained.

I am making fix for prepared stmts. https://github.com/MofhyDevs/MofhyCommunity/

Um, there already is a community version...

which no one cares to edit.

Well nobody cares whatever you are doing for community. They will just tell where you are wrong and will never try to fix it by there own.

which no one cares to edit.

Well nobody cares whatever you are doing for community. They will just tell where you are wrong and will never try to fix it by there own.

That's kinda sad honestly...

That's kinda sad honestly...

agreed

I mean you are not editing it either...

Who are you mentioning

https://github.com/MyOwnFreeHostCommunity/Mofhy 🤔

I am talking to @jaikt
Well the Community version here here: https://github.com/MOFH-Dev
You just made another one, when one already exists.

Well well well. That's a point

imo, rewriting the whole client area is the best we can do.
Ensuring all the security issues this time.


Feature Request: Clicking links in descriptions. Also bot to visit channel homepage and browse?

Is there an existing issue for this?

[X] I have searched the existing issues

I'm submitting a ...

[ ] bug report
[X] feature request
[ ] support request --> Contact me over mail for support https://github.com/MShawon

Description
Brilliant app btw.... I hope that this is the correct place for this kind of message. I am loving the software. But would it be possible to set up the bot to click links (to other youtube playlists only not external links) that are in the description on the video.
Also it would be nice to have the bot sometimes just to query channel name (ie. Open up YT and search my given query (eg. "My Full Channel Name")), then click my actual channel. Once there it could browse for a random video or even a whole playlist.
Thanks again for this great software (I am one day going to study the code to see how I can understand it and maybe even add some feature in the distant future :D)
Environment
- OS : Win10
- Python : ?
- Script version : ?

config.json
N/A

it's a good suggestion to search channels but I didn't want to make it any more complex hence most of the people don't understand the search.txt ..so I implemented the function to visit the channel automatically after watching a video
implementing playing a playlist needs to be done first for what you asked..but there seem to be a couple of problems with the playlist feature which could break the script
clicking certain links from the description would need more information from user which can again make it more complex
however if you use multiple videos you will get suggested views so no need to click description link..you could just add the video link to urls.txt

it's a good suggestion to search channels but I didn't want to make it any more complex hence most of the people don't understand the search.txt ..so I implemented the function to visit the channel automatically after watching a video
implementing playing a playlist needs to be done first for what you asked..but there seem to be a couple of problems with the playlist feature which could break the script
clicking certain links from the description would need more information from user which can again make it more complex however if you use multiple videos you will get suggested views so no need to click description link..you could just add the video link to urls.txt

This is all very fair comment. I wish I had a good enough understanding to help you coding out the problems. But sadly I am miles off! :D
Btw I have noticed that it does have some AI to find the Suggested Videos already, which was what gave me the ideas I mention. However I take on board that these new features aren't so simple to actually build.
Thanks again for the superb app.


morph: 1.1.0 -> 1.2.0

Motivation for this change
A new release of morph has been cut. Should be fully backwards compatible.
Things done

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

[x] NixOS
[ ] macOS
[ ] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build morph

The added ldflag ensures morph --version outputs something useful.


npm gulp-sass Variables and Mixins Not Working from @import

Same here. I'm developing an ionic project and i need to use some compass mixin. Is there a way to use it?
Thanks.

+1
Providing an example.
Using gulp-sass v2.0.4
site.scss - main file that just imports files:
@import 'utils/mixins';
@import 'pages/home';

utils/_mixins.scss:
.v_home {
  @include vertical-align;
}

pages/_home.scss:
@mixin vertical-align {
  position: relative;
  top: 50%;
  transform: translateY(-50%);
}

But when compiling i get an error message:
Error in plugin 'sass'
Message:
    src/scss/pages/home.scss
  7:12  no mixin named vertical-align

I've PR'd ionic to fix their broken dependency https://github.com/driftyco/ionic/pull/4449

The PR in mention has been merged.
https://github.com/driftyco/ionic/pull/4449

Just a heads up, this can be closed. It's been fixed in our starter projects, as well in the framework. No need to wait until we cut a release, since user would need to clone the repo for use sass there.

Thanks for the update @mhartington


Change in the order of Hooks called by I18nProvider

Hello!

I am having this issue:

This happens after any change inside _app.tsx.

After every warning I am getting the next error with process fall:

Can you give a reproducible example to see if there is something wrong? Thanks

@iksent ping on this. Without a reproducible example is difficult to fix! Thanks 🙏

Sorry, my code is private, can't share it.
I don't think, that the problem inside next-translate, because I saw this error with another's lib trace log.
I'll reopen it, if I have same problems again, sorry for inconvenience!


NetworkManager depends on GNOME wallpapers

Issue description
NetworkManager pulls in GNOME desktop backgrounds as a transitive dependency via the GNOME applet via the VPN plugins. The VPN plugins are built with withGnome by default. I use plasma5, which ships with its own NM applet. Building the -gnome versions of the VPN plugins is just weird.
Steps to reproduce

networking.networkmanager.enable = true;
nixos-rebuild switch

# nix-env -qa --out-path gnome-backgrounds
gnome-backgrounds-3.24.0  /nix/store/5kgxddmj7sf5lf1ax44myhs8vaxj1bcv-gnome-backgrounds-3.24.0

# nix-store -q --referrers-closure /nix/store/5kgxddmj7sf5lf1ax44myhs8vaxj1bcv-gnome-backgrounds-3.24.0
/nix/store/5kgxddmj7sf5lf1ax44myhs8vaxj1bcv-gnome-backgrounds-3.24.0
/nix/store/g5mckqqinkc3icszmssbv44n2xxzdxma-gsettings-desktop-schemas-3.24.0
/nix/store/7rvr2rzpwqxa12jbbv7jdmnwmkl0v7rz-network-manager-applet-1.4.6
/nix/store/1qd55958w7g7ix2fvdk6922nab6ghyvk-NetworkManager-fortisslvpn-gnome-1.2.4
/nix/store/35gcijhd0g3gvy9a93dal2n8wffsq4wp-NetworkManager-pptp-gnome-1.2.4
/nix/store/d4k20fv0znc0ik2rbli533yq91apjxdz-NetworkManager-l2tp-gnome-1.2.4
/nix/store/fbpk1m9mkk177kjr7kf0h3fjby1hd70s-NetworkManager-iodine-gnome-1.2.0
/nix/store/hvn19q4fz2llga7rk40wawmzhbfpam6w-NetworkManager-openvpn-gnome-1.2.8
/nix/store/rkfm8sh04hvrrp0ss1mairm1g03qpspl-NetworkManager-vpnc-gnome-1.2.4
/nix/store/3gnarj93bk7xr7glxdkyc4gxbrz3y9nd-system-path
/nix/store/4x8cz29cwlj1hxyb5443h2rkmgsk1yqp-unit-polkit.service
/nix/store/b5gk91cicxqw4qff09pr2igh3xrsqnvs-dbus-1
/nix/store/50ngv6mx65dfcfyvc1kskf0yh2knv370-unit-dbus.service
/nix/store/kwxhar3g32ixxh8p1rdnimdzrbar52rb-user-units
/nix/store/xx4r2qfvsxlz7m376bzsv9y6nm76997r-NetworkManager-strongswan-1.4.1
/nix/store/rsnigpgj3ghk6ackqijrr1x4q3h8dq08-unit-systemd-fsck-.service
/nix/store/yjhr9fx7ajfj66w6ihjdh5r4jj4ij5ya-unit-systemd-udevd.service
/nix/store/zsxbdm0bv9ml6v7vfbz1h4zi99qn7i7p-system-units
/nix/store/g0a41gr6hv4qq9p9nzl5i6r5gcxfp1q0-etc
/nix/store/7iwjxxwvv9wkf025v0yypg4nm1h2ndb3-nixos-system-nerull-18.03pre116347.272c0bf88c

Technical details

System: 18.03pre116347.272c0bf88c (Impala)
Nix version: 1.11.15
Nixpkgs version: 18.03pre116347.272c0bf88c
Sandboxing enabled: no

gnome-backgrounds are a dependency of gsettings-desktop-schemas. The schemas need path to backgrounds so they could set a default background.
I agree that withGnome should be false by default and pkgs/desktops/gnome-3/default.nix should override it.

Thanks!


When planning a route for a vehicle, take into road traffic

You can read more about the intricacies of using global and local traffic data providers in the discussion "Integrate Traffic Data from different sources (e.g. from the German bundes API)" of the Organic Maps project.

Thanks for opening issues here. You might want to see if these can be integrated in the graphhopper-maps repository as this app is only a wrapper for that. https://github.com/graphhopper/graphhopper-maps

@boldtrn, is this project on Github correctly listed as the main project for the GraphHopper Maps app on F-Droid?

It is, when you look at the code you will see what I mean.


Polygon validators and code refactoring

Added:

Validator interface and entry point which are compatible with polygon platform.
Implementation of InStream that can be constructed directly from some InputStream (instead of File object).
Utilities methods for convenient checks for string and numbers.

Changed:

Implementations of InStream are splitted to CharSource and implementation of interface. CharSource is a class which is responsible for reading input stream on low level.
All token-related methods now can be bounded with max length of token which allows to pass default stress tests of polygon (like 1 mln. or random ASCII characters in a single line).
All token-related methods are now can be parametrised with Predicate's which make them more flexible (an a little bit slower, but who cares about that during solution check?).
Fixed some bugs (like seekEoLn () actually founds the EoF).

Reasons for non-merging immediately:

Add to README information about validators.
Add some more utilities methods.
Test a bit more.

I'm not sure if it's really necessary since Java 8 is considered as obsolete.
Sorry, I didn't run run-checker-unittests target before PR. Now it's fixed.
Don't pretend to any special dates, check asap


RFC - add rawBody to NextApiRequest

Feature request
As per @timneutkens request, this is an RFC to add rawBody to NextApiRequest.
Is your feature request related to a problem? Please describe.
Currently, there is no way to use both a body parser and recuperate the raw body of a request. This makes things like securing github webhooks, which rely on calculating a hash based on a raw body, difficult in NextJS.
Describe the solution you'd like
The solution I proposed in the PR above is to add a raw body field to NextApiRequest.
Describe alternatives you've considered
The alternative we're currently using is deploying a separate microservice to handle our github webhook. It could be that this feature is outside the scope of nextjs, in which case the separate microservice makes sense.

Would disabling the bodyParser be an appropriate action?
This allows you to read the body however you'd like.

Hey! This definitely helps - we were hoping to use the body parser as the body is json, but what we can do as a temporary measure is disable the body parser and then parse the JSON ourselves. Thanks!

I would add to this, I had the same issue with stripe webhooks.
My solution was to reverse the process as described here https://github.com/stripe/stripe-node#testing-webhook-signing
JSON.stringify(payload, null, 2).

The stringify solution did not work for me. Also, one disabling the bodyParser, req.body is undefined, how would one parse the body?


rkt: run from a docker registry URL

rkt run docker://localhost:5000/mything should work. This depends on #142.

This now depends on endocode/docker2aci#3

/cc @iaguis

appc/docker2aci#3

:+1:

This is ready to go now that docker2aci is a library. If anyone wants to help out :) http://godoc.org/github.com/appc/docker2aci/lib

@philips this is already done - #489

Fixed by #489


Fix: Refresh same page - forbidden quick redirect

Fix #587

[x] All continuous integration tests are green
[x] Tests are added where necessary
[x] Documentation is added/updated where necessary
[x] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

thanks for your hard work @qmonmert


App Crash: unknown Fatal Unhandled Segfault

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
None that we know of.
We get the following issue reported to us via Sentry on occasion:
https://hunter-industries-mz.sentry.io/share/issue/d27c32bd6e4c40d19f0802d95ebe85c1/
Expected results
No app crash
Actual results
App Crash
Code sample
n/a
Screenshots or Video
n/a
Logs
https://hunter-industries-mz.sentry.io/share/issue/d27c32bd6e4c40d19f0802d95ebe85c1/
Flutter Doctor output
C:\git\mob> flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.13.6, on Microsoft Windows [Version 10.0.19045.3570], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Enterprise 2022 17.6.4)
[√] Android Studio (version 2022.3)
[√] VS Code, 64-bit edition (version 1.83.1)
[√] Connected device (4 available)
[√] Network resources
• No issues found!
C:\git\mob>

@MobIS1User
Looking at the log, it seems to be mostly coming from native SDK. So there's very little info to properly address the issue. If you could provide more details like, a pattern or any other logs would help to properly investigate this issue.


Triage impact related to CAI field name changes for sourceRange/sourceRanges

[ ] Please determine and open issues if there are changes required for Forseti CAI Inventory Image
[ ] Please determine and open issues if there are changes required for Forseti Firewall and other scanners
[ ] Please determine and open issues if there are changes required in the Policy Library

Hello Cloud Asset Inventory Customer,  We are writing to let you know that Cloud Asset Inventory is releasing some asset field name updates starting on September 10, 2019. Initially, both updated field names and existing names will be returned together. The previous naming of the re-named asset fields will no longer be returned in about 6 months. We suggest you review whether these field names are compatible with your tools to avoid possible service disruption.  What do I need to know?  We are making these name changes because in a number of Cloud Asset Inventory resources, some field names in the asset returned by Cloud Asset Inventory differ from their corresponding public APIs. Take the compute.googleapis.com/Firewall asset for example:                   Cloud Asset Inventory export result   firewalls.get API result       "sourceRange":["<IP_ADDRESS>/9"]}   "sourceRanges":["<IP_ADDRESS>/9"]}       In order to be consistent with each service’s public API format, we are updating those inconsistent field names. You can review the mapping table for more information.  We are making this change in phases so you will have time to adjust to the updated field names:  Phase 1  From September 10, 2019, we will be adding the updated field names alongside with existing fields. The content of the fields is identical to the current ones, only field names will be modified. We expect this change to be backwards compatible for all our customers.  Phase 2  We will remove the fields with deprecated names in about 6 months, we will send you another notification to remind you of the deprecation.  What do I need to do?  Although we are expecting the asset field names change to be backwards compatible with all our customers, please review the list of updated asset field names.  If you have any current data processing with dependencies on any field names to be updated, you should update these fields as soon as possible to avoid possible service disruption.  If you have any questions or require assistance, please reply to this email to contact Google Cloud Support.  Thank you for being a valued Cloud Asset Inventory customer.
Cloud Asset Inventory export result | firewalls.get API result | "sourceRange":["<IP_ADDRESS>/9"]} | "sourceRanges":["<IP_ADDRESS>/9"]}

Cloud Asset Inventory export result | firewalls.get API result
"sourceRange":["<IP_ADDRESS>/9"]} | "sourceRanges":["<IP_ADDRESS>/9"]}

Link to the field name announcement documenting the changes that were made live on 9/10/2019.


onConnected callback not called on iOS

Hej, I'm writing a plugin which utilizes nativescript-bluetooth and I ran into an issue on iOS, where the onConnected callback is not triggered after the connection to a peripheral is established. About 20 seconds after the connections is established the peripheral is disconnected. The same behavior appears with the demo app.
iOS Versions tested: 10.1.1, 10.3.1
Xcode Version: 8.3.3
Here's the console log of the demo app:
CONSOLE LOG file:///app/tns_modules/nativescript-bluetooth/bluetooth.js:239:16: ----- delegate centralManager:didDiscoverPeripheral: BOT @ -35
CONSOLE LOG file:///app/main-view-model.js:41:16: !!&&&&***** Clicked item with index 0
CONSOLE LOG file:///app/main-view-model.js:43:16: --- peri selected: 77571063-AFD5-4F8D-BE5E-E0A108938FF4
CONSOLE LOG file:///app/tns_modules/nativescript-bluetooth/bluetooth.js:426:20: Connecting to peripheral with UUID: 77571063-AFD5-4F8D-BE5E-E0A108938FF4
CONSOLE LOG file:///app/tns_modules/nativescript-bluetooth/bluetooth.js:264:16: ----- delegate centralManager:didConnectPeripheral: <CBPeripheral: 0x1702ea280, identifier = 77571063-AFD5-4F8D-BE5E-E0A108938FF4, name = BOT, state = connected>
CONSOLE LOG file:///app/tns_modules/nativescript-bluetooth/bluetooth.js:268:16: ----- delegate centralManager:didConnectPeripheral: cached perio: <CBPeripheral: 0x1702ea280, identifier = 77571063-AFD5-4F8D-BE5E-E0A108938FF4, name = BOT, state = connected>
CONSOLE LOG file:///app/tns_modules/nativescript-bluetooth/bluetooth.js:275:16: ----- delegate centralManager:didConnectPeripheral, let's discover service

The demo app was build with following versions:
node: 6.10.1
nativescript: 3.1.3
tns-core-modules: 3.1.1
tns-android: 3.1.1
tns-ios: 3.1.0

In my plugin I've tested this with nativescript-bluetooth version 1.1.6 and 1.2.0
Here's a part of the package.json:
{
  ...
  "nativescript": {
    "platforms": {
      "android": "3.0.0",
      "ios": "3.0.0"
    }
  },
  ...
  "dependencies": {
    "nativescript-bluetooth": "1.1.6",
    "nativescript-platform": "^1.1.1",
    "nativescript-zip": "^1.3.2"
  },
  "devDependencies": {
    "tns-core-modules": "~3.0.0",
    "tns-platform-declarations": "^3.0.1",
    "typescript": "~2.2.1",
    "prompt": "~1.0.0",
    "rimraf": "~2.5.0"
  }
}

The app in which I'm using my plugin was build with:
node: 6.10.1
nativescript: 3.1.3
tns-core-modules: 3.0.1
tns-android: 3.0.0
tns-ios: 3.0.1

The console log is identical to the log of the demo app (except the lines of the view-model of course).
On Android everything is working fine.
Any idea how to solve this? Could version conflicts be the cause?
Thanks in advance!

Bump for this. My app had been working for a while and now it has suddenly stop working and I am not sure why that is! I might have updated {N} but the onConnected callback is not triggered any longer. :( If someone has an idea in how to fix this I have a show on Tuesday where I really would need to use it. Any help is greatly appreciated.a

Can you try using an older release and narrow it bit more down?

older release of the library? or {N}?

Ok, I played a bit around and found out that the peripheral in question does not advertise its service(s). Therefore the peripheralDidDiscoverServices delegate is not triggered.
With a peripheral which advertises its service and everything is working fine there.

@jimport can you elaborate on what's missing? I'm not sure how to modify the source. This entails that we have always control over the bluetooth unit that we are interacting with, which is not the case..

This could be an "issue" with CoreBluetooth. I've tried to connect with the LightBlue Explorer app, without success. It displays just No services and disconnects after a few seconds.
@mm-ns Have you tried to connect with LightBlue or something similar?

no i haven't because before today my app was working just fine and I am not sure I updated anything actually. I have been running a couple of different {N} apps on my device but that's about it. :/ I found a zipped package of my project from last week (when it still was working) and if i try to run that on my device it still doesn't work. The only thing that changed from a few days ago I see im now running 3.0.3 and when i made this app i was running 3.0.1.

So this is bizarre. The whole thing actually "works", if it wasn't for the fact that when I am trying to connect to my ble device from a list, it takes probably about 1 min for the callback to come back as "connected". Nothing changed on the ble device side. Any idea why this may be?

Tried to revert everything to 3.0.1, but still same issues. My ble devices (arduino) are also the same code that used to work. No idea what's changed..

It seems that this is not possible with CoreBluetooth. To discover services, they need to be advertised. So this is no issue with nativescript-bluetooth!


ol.Sphere.haversineDistance is not a function, also metersPerUnit doesn't work inside ol.Projection

Hello!
I want to increase ol.controls.scaleline distance because it shows wrong distance
I tried using metersPerUnit inside my ol.Projection like this:
      var mapProjection = new ol.proj.Projection({
        code: 'EPSG:3857',
        extent: worldBoundary,
        units: 'm',
        metersPerUnit: 5 // default 1
     });

I haven't noticed any change, so I tried overriding getPointResolution, but there are problems with some internal functions (they aren't available). Basically I want to recreate the original function logic so I could modify it later.
      var mapProjection = new ol.proj.Projection({
        code: 'EPSG:3857',
        extent: worldBoundary,
        units: 'm',
        getPointResolution: function(resolution, point) {
          // Estimate point resolution by transforming the center pixel to EPSG:4326,
          // measuring its width and height on the normal sphere, and taking the
          // average of the width and height.
            var vertices = [
              point[0] - resolution / 2, point[1],
              point[0] + resolution / 2, point[1],
              point[0], point[1] - resolution / 2,
              point[0], point[1] + resolution / 2
            ];
            vertices = ol.proj.transform(vertices, 'EPSG:3857','EPSG:4326');
            console.log(vertices);
            var width = ol.Sphere.haversineDistance(
              vertices.slice(0, 2), vertices.slice(2, 4)
            );
            var height = ol.Sphere.haversineDistance(
              vertices.slice(4, 6), vertices.slice(6, 8)
            );
            console.log(height);
            var pointResolution = (width + height) / 2;
            var metersPerUnit = this.getMetersPerUnit();
            if (metersPerUnit !== undefined) {
              pointResolution /= metersPerUnit;
            }
          return pointResolution;
        }
      });

This throws an error:
TypeError: ol.Sphere.haversineDistance is not a function

But I've addet it into my  ol.custom.json build file for export:
"exports": [
  "ol.Map",
"ol.Map#*",
"ol.Map#getView",
"ol.View",
"ol.View#getResolution",
"ol.Feature",
"ol.format.GeoJSON",
"ol.format.GeoJSON#*",
"ol.coordinate.createStringXY",
*"ol.Sphere",* // Sphere
"ol.Sphere#haversineDistance",
"ol.proj.Projection",
"ol.proj.transform",
    "ol.control.defaults",
"ol.control.MousePosition",
"ol.control.ScaleLine",
"ol.style.Style",
"ol.style.Icon",
"ol.style.Fill",
"ol.style.Circle",
"ol.style.Stroke",
"ol.style.Text",
"ol.layer.Vector",
"ol.layer.Vector#set",
"ol.source.Vector",
"ol.source.ImageStatic",
"ol.layer.Image",
"ol.source.ImageStatic#get",
"ol.Image#getImage",
"ol.geom.Point",
"ol.Collection#*"
  ]

ol.Sphere#haversineDistance is not a static function. You need to create an ol.Sphere instance with your desired radius, and then call #haversineDistance on that.
var sphere =  new ol.Sphere(6378137);
var distance = sphere.haversineDistance(whatever);

That aside, it would be good to know more about your intended use of ol.control.ScaleLine, and how it failed to show the correct scale. What view projection were you using? How was it defined? Maybe you could share your code in a JSFiddle.

Closing due to lack of activity.


Command "python setup.py egg_info" failed with error code 1

I'm having trouble installing, I get
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/54/qx83hf8s6nj9vm3f0vqd7crwmkzn0f/T/pip-build-iauSHc/pygobject/

Full output:
src ❯ pip install neovim-gui
Collecting neovim-gui
  Downloading neovim_gui-0.1.3.tar.gz (65kB)
    100% |████████████████████████████████| 71kB 2.1MB/s
Requirement already satisfied (use --upgrade to upgrade): neovim>=0.1.3 in /usr/local/lib/python2.7/site-packages (from neovim-gui)
Collecting click>=3.0 (from neovim-gui)
  Downloading click-6.6-py2.py3-none-any.whl (71kB)
    100% |████████████████████████████████| 71kB 4.5MB/s
Collecting pygobject (from neovim-gui)
  Downloading pygobject-2.28.3.tar.bz2 (889kB)
    100% |████████████████████████████████| 890kB 1.3MB/s
    Complete output from command python setup.py egg_info:
    ********************************************************************
    * Building PyGObject using distutils is only supported on windows. *
    * To build PyGObject in a supported way, read the INSTALL file.    *
    ********************************************************************

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/54/qx83hf8s6nj9vm3f0vqd7crwmkzn0f/T/pip-build-cLy1Ap/pygobject/

what OS are you on? As the message says, pygobject can't be installed by pip, rather by the system package manager. On debian/ubuntu for instance the package python3-gi

@bfredl (That was my other account). I'm in macOS Sierra. Let me see if I can find where to get that from...

@bfredl Got it neovim-gui installed after brew install pygobject3, but now running pynvim says:
~ ❯ pynvim
Traceback (most recent call last):
  File "/usr/local/bin/pynvim", line 9, in <module>
    load_entry_point('neovim-gui==0.1.3', 'console_scripts', 'pynvim')()
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/neovim_gui/cli.py", line 63, in main
    from .gtk_ui import GtkUI
  File "/usr/local/lib/python2.7/site-packages/neovim_gui/gtk_ui.py", line 8, in <module>
    gi.require_version('Gtk', '3.0')
  File "/usr/local/lib/python2.7/site-packages/gi/__init__.py", line 118, in require_version
    raise ValueError('Namespace %s not available' % namespace)
ValueError: Namespace Gtk not available


typo: recieve -> receive

Added code fence languages as well

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/create-the-receive-location-and-send-port-programmatically.md
:white_check_mark:Succeeded
View

biztalk/core/how-to-configure-a-wcf-custom-receive-location.md
:white_check_mark:Succeeded
View

biztalk/core/how-to-configure-a-wcf-customisolated-receive-location.md
:white_check_mark:Succeeded
View

biztalk/core/how-to-configure-a-wcf-netmsmq-receive-location.md
:white_check_mark:Succeeded
View

biztalk/core/how-to-configure-a-wcf-netnamedpipe-receive-location.md
:white_check_mark:Succeeded
View

biztalk/core/how-to-configure-a-wcf-nettcp-receive-location.md
:white_check_mark:Succeeded
View

biztalk/core/how-to-configure-a-wcf-wshttp-receive-location.md
:white_check_mark:Succeeded
View

biztalk/core/transactional-adapter-biztalk-server-sample.md
:white_check_mark:Succeeded
View

biztalk/core/using-the-unit-testing-feature-with-pipelines.md
:white_check_mark:Succeeded
View

biztalk/core/walkthrough-custom-message-processing-with-the-wcf-nettcp-adapter.md
:white_check_mark:Succeeded
View

biztalk/reference/biztalk-2016-fp1-rest-administration.json
: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.


Modify Makefile to allow DOCKER_HUB_ID to be overridden by env var

In the future we plan to have a more advanced CI/CD process and allowing the DOCKER_HUB_ID to be overridden by an environment variable will needed.
We should change
DOCKERHUB_ID:=ibmosquito

to the following
DOCKER_HUB_ID ?= ibmosquito

This ticket is still open.  After cloning the repo and running make in the folder, I get an error:
% make
docker build -t /"web-hello-python":"1.0.0" .
invalid argument "/web-hello-python:1.0.0" for "-t, --tag" flag: invalid reference format
See 'docker build --help'.
make: *** [build] Error 125

Scanning the Makefile, I see line 3 shows the variable DOCKER_HUB_ID but lines 16 and 22, 29, 35, 41 and so on still use DOCKERHUB_ID.
Please modify all of the other occurrences from DOCKERHUB_ID to DOCKER_HUB_ID to match.  The script is currently broken.


Modern Authentication

We are moving all our scripts for automation away from basic authentication, The last one we have to change is using the SharePoint module, When will this be updated as December is fast approaching.

@smitz1 The SharePoint Online PowerShell module already supports modern authentication. SharePoint Server does not nor does it need to.

@smitz1 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also


Add the titiler service to the deployment stack

This involves updating the stack.yml file (docker-compose.yml for Docker Swarm) to include an additional server (that is titiler). Here is the file that needs to be updated: https://github.com/SAEON/mnemosyne/blob/stable/deploy/stable/stack.yml
For a reference on how to add a service that uses a pre-built image available on Docker Hub - https://github.com/SAEON/elk-stack/blob/main/v8/stack.yml#L55-L80.
(https://hub.docker.com/r/developmentseed/titiler)

For interest sake, how would I test this while developing or before I send a PR? At present, my understanding is that I would have to deploy the Mnemosyne stack to a test machine

you can deploy the mnemosyne stack on your local machine - create a single node Docker swarm: https://docs.docker.com/engine/reference/commandline/swarm_init/
Running docker swarm init will just register your docker engine as a manager node, meaning you can run docker swarm CLI commands on it. This doesn't effect your Docker install in any way so far aw I know (at least I have never noticed a difference in the way that docker works when registered as a single node swarm compared to not).
docker compose up and docker stack deploy are very similar.
With Docker Compose you have a yml file that specifies a number of containers that go up together and should be on the same network - that's the same with Docker Swarm. The compose file specifies a number of containers that go up together. Just the terminology is different. So with compose the result is a number of containers.
Likewise with stack deploy, you get a number of containers. But each container belongs to a service. And each service belongs to a stack.

Is this still needed?

Moved the titler service to the titiler branch for now

I remember now - this was moved to here: https://github.com/SAEON/titiler


fix: add support for Neovim v0.7+

This should be tested, but I think this should fix with older Neovim compatibility.
I can test this fully early next week. Thanks to the maintainers for this great project!
Closes #658

@lukas-reineke just got around to testing this and it's working perfectly on neovim v0.8! Thanks for considering this PR 🙏🏼

hm looks like the types for 0.9.2 just don't include it at all anymore. Just ignore all errors for that line I guess
---@diagnostic disable-next-line


Add support for passing script type in more methods

PR improves the following methods by adding the option to pass the type of a script instead of string name (just like pc.ScriptComponent#create and pc.ScriptRegistry#add):

pc.ScriptComponent#has
pc.ScriptComponent#get
pc.ScriptComponent#destroy
pc.ScriptComponent#swap (marked as private)
pc.ScriptComponent#move
pc.ScriptRegistry#has
pc.ScriptRegistry#remove

This allows script managing to be more safe and maintainable as well as making the API more unified and consistent.
Note: PR also optimizes the code a bit in both pc.ScriptComponent#create and pc.ScriptRegistry#add.
I confirm I have signed the Contributor License Agreement.

@Maksims Any comments on this one?

@willeastcott Fixed. Promise to run tests before submitting next time :)

No objections, nice one.

Awesome. Thanks @Maksims! Let's meeeeeerge! 😄


[backport] consensus decode from finite decoder.

Backport for #1023. Required for #997. Addresses issues like #1359

Perhaps, we can get a review here and make 0.28.2 with the bug fix. Since @apoelstra is offline for the next two weeks, I think @TheBlueMatt has permission to push to crates io.

duh, this is more involved because we changed the MSRV in between.

I thought there were three patches in the hong-fuzz fix, they can be seen on the 0.29.2 backport. Can we have the version bump as a final patch after all the backport patches as well please?

@tcharding , working on it

CI cleared, ready for review/publication.

ACK a0489d42b966ef8eb86d659674575c563b53f946
I verified that the first patch is the same as the original patch but with import statement changes.


Tracking issues of RFC-2758: Merge Append Into Write

RFC: https://github.com/apache/incubator-opendal/pull/2758
Tasks

[ ] Add append in OpWrite
[ ] Add new capability write_with_append
[ ] Implement append (or return errors) based on append flag
[ ] Remove all append and appender related code

Will append support abort?

Will append support abort?

Yes. abort's semantic will become aborting not committed (or flushed) data.

I will take over it.

I will take over it.

Hi, we could use separate issues to mark sub tasks.

Yep, it's better to assign with sub tasks instead of the tracking issue itself. We can create new issues from task list and assign self.

I'm working on this now.


fs_fat32.c fat_seek maybe return a error that shoud ok

Use case:
open a file with readonly,then lseek to the end with offset 0
there will return a error( ENOSPC  28 "No space left on device")
so i can't get the file size with this.
TestFile:13,107,200 Byte
Code:
static off_t fat_seek(FAR struct file filep, off_t offset, int whence)
---------1164
if ((ff->ff_oflags & O_WROK) != 0)
{
/ Extend the cluster chain (fat_extendchain
* will follow the existing chain or add new
* clusters as needed.
*/
          cluster = fat_extendchain(fs, cluster);
        }
      else
        {
          /* Otherwise we can only follow the existing chain */

          cluster = fat_getcluster(fs, cluster);
        }

---------1178
when i debug i found that fat_getcluster(fs, cluster) will retrun the "0x0fffffff" that means FAT cluster end.
but in fat_seek ,there is no end deal,so there will be a error.
The break way is less than clustersize,in the loop， position Just coincidentally equal to clustersize,so it continue find next cluster chan.
---------1142
if (position < clustersize)
{
break;
}
---------1145
Solution:
add FAT end deal or make break condition wiht "position <= clustersize"

Hi @elmagnificogi, this is the old repo no longer used, but kept for reference.
The current PX4 NuttX repository is here - https://github.com/PX4-NuttX/nuttx


cannot get UltraSSD_LRS storage to work with the instructions provided in the documentation: " UltraSSD_LRS is not supported with caching type ReadWrite"

I tried to follow the instructions here to set up a k8s pod with a ultra ssd disk:
---
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: ultra-disk-sc
provisioner: kubernetes.io/azure-disk
volumeBindingMode: WaitForFirstConsumer # optional, but recommended if you want to wait until the pod that will use this disk is created
parameters:
  skuname: UltraSSD_LRS
  kind: managed
  cachingmode: None
  diskIopsReadWrite: "4800"
  diskMbpsReadWrite: "1200"
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: documents
spec:
  accessModes:
  - ReadWriteOnce
  storageClassName: ultra-disk-sc
  resources:
    requests:
      storage: 16Gi
---
...
      containers:
...
          volumeMounts:
            - mountPath: /data
              name: documents

I get the following error:
 "error": {
    "code": "InvalidParameter",
    "message": "Storage Account type UltraSSD_LRS is not supported with caching type ReadWrite. Please use caching type None.",
    "target": "dataDisk.caching"
  }

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

ID: f9cef1a6-012b-2750-c016-b3ebddbcd706
Version Independent ID: 424ee453-2459-af29-8a1f-65ad333ce454
Content: Enable Ultra Disk support on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/use-ultra-disks.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@oocx
Thanks for your Question! We are currently investigating and will get back on this.

@oocx , thank your or your feedback.
We could only reproduce this error when explicitly setting the storageClass.parameters.cachingmode to ReadWrite. When this was set to None as in the examples from the linked document, this error wasn't reproducible.
@zr-msft can we add this limitation to the document?

@SrijitBose-MSFT In my example, I have set it to "None" (at least I think so, see my example above), but I still get the error, that's why I am confused. Is it supposed to work the way I have configured it? Or am I missing something?
Thanks!

Any Update? I am facing same issue.

Hi all,
I was able to reproduce the issue and there seems to be a discrepancy with the case of "cachingmode" within the storageClass - it should I believe be "cachingMode" instead.
Please can you try the below, and re-create the storageClass, PVC and pod?
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: ultra-disk-sc
provisioner: kubernetes.io/azure-disk
volumeBindingMode: WaitForFirstConsumer # optional, but recommended if you want to wait until the pod that will use this disk is created
parameters:
  skuname: UltraSSD_LRS
  kind: managed
  cachingMode: None
  diskIopsReadWrite: "2000"  # minimum value: 2 IOPS/GiB
  diskMbpsReadWrite: "320"   # minimum value: 0.032/GiB

As mentioned by @AdamSharif-MSFT the issue is fixed here #86177 .
Closing this thread now.


feat(CSV_verifications): Add public endpoint for CSV verifications

Fixes #<INSERT ISSUE NUMBER HERE>
Checklist

[ ] Up to date with main
[ ] All the tests are passing

[ ] Delete all resources created in tests

[ ] Prettier
[ ] Spectral Lint
[ ] npm run bundle outputs nothing suspect
[ ] npm run postman outputs nothing suspect

Changes
Areas of Concern (optional)

@Thexumaker Do you have an ETA on when this should have merge conflicts resolved and be shipped?

@Thexumaker Do you have an ETA on when this should have merge conflicts resolved and be shipped?

Will be taking a look in a couple min, completely forgot I opened this for a sec XD


redesign of overview, new colour palette, and tweak to explorer visuals

Continued development of dashboard

Hey @ThomasHP-EECA . I reckon if this is a version that has been shared with comms etc recently it's probably worth reviewing and merging now and then it will be deployed to the dev dashboard. You can then do further PRs to refine as you get feedback.


Accessing individual pixels without copying the buffer

Is there a way to directly access random pixel data with a non-iterative random access pattern without copying into a Vec?
I want to use the bitmap as a kind of simple data structure.
For that I need random access to pixels. The iterator approach doesn't work and collect()ing the iterator wastes memory.
I'm thinking of an API such as:
impl RawBmp {
    fn get_pixel(&self, x: u32; y: u32) -> Pixel {
        ...
    }
}

Thanks a lot for your help.

I've just opened #34 to add this feature.

@rfuest Thanks.
But you still seem to be using Iterator instead of direct O(1) access to the raw data.
Isn't Iterator::nth() horribly expensive for random accesses?


test framework: centralize pod dump and include envoy configs

DRY up logic to fetch pods and dump logs
Move the deployment.DumpX functions to namespace. As they were, they operated at a namespace scope, having them in deployment (with no usages in that package) was confusing. namespace isn't a great place either, lots of interdependency so it causes import cycles.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
ef2263b0a3a5d50a1b271e244ec916d093b54adc
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
ef2263b0a3a5d50a1b271e244ec916d093b54adc
link
/test integ-local-tests_istio

lint_istio
ef2263b0a3a5d50a1b271e244ec916d093b54adc
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
ef2263b0a3a5d50a1b271e244ec916d093b54adc
link
/test integ-local-tests_istio

lint_istio
312b463736805692cc92c3b3252d4fd2cea6459a
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
ef2263b0a3a5d50a1b271e244ec916d093b54adc
link
/test integ-local-tests_istio

lint_istio
312b463736805692cc92c3b3252d4fd2cea6459a
link
/test lint_istio

gencheck_istio
312b463736805692cc92c3b3252d4fd2cea6459a
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl mind waiting for my refactoring to land? https://github.com/istio/istio/pull/23586

@stevenctl #23586 is in now. I think you just have to move your changes to deployment/util.go over to the test/kube package

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
b9dd72ba3bd7ed5f9630de6531cdf186b08d959c
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Start on team composition form

Based on https://trello.com/c/wXa0dJqb/23-mockups-version-1, this adds a simple CompositionForm React component with a form that is completely not hooked up or functional yet. My thinking is we can start by getting the basic page structure in place, then hook things up in follow-up branches.
Here's what it looks like -- is it everything you ever dreamed of, @RobThePM? 😀

I hope @smbriones, @justbarreto, and @summasmiff can build upon this to make it look good. 🙏 It's fine with me if y'all want to work on this branch or in a separate branch, whichever.

Looks great so far. The labels "Offense Payload 1", "Offense Payload 2", etc. when the user catches maps, right (e.g. they switch the map to a king of the hill map)?

The "Offense Payload 1" labels don't change at all right now, but that's the intention. Think of this pull request like a slightly more real mock-up: it's not Balsamiq, but it's not the real thing either.
That perhaps reveals another table we should have in the database: each segment of a map where the team organizer might want to have a particular role.

Hi there! I'm having some fun playing with the visual design a bit and curious what your thoughts are on this. Did I leave off anything that is important to see? What does everyone think of something like this? Any feedback/thoughts would be great! cc @cheshire137 @RobThePM

Ahhh! It's so pretty! 🌈  Thank you, Stephanie. 😀 You've managed to take a page with a lot of information to convey and make it not intimidating. I love the colors, it's very friendly. I also think having the hero faces really helps for visual recognition. It's easier and faster for me to parse Sombra's face than to read "Sombra" and conjure up the hero in my mind.
I like that the name of the comp isn't always presented in an input field. It's clearly editable from the icon next to it, but there's no box around it to clutter up the design from being in an input.
Having the slots where no hero has been selected have a gray background instead of white also draws attention to them, like "hey you need to do something here." That's helpful.
The gradient up top is pretty and I can see it comes from the Gibraltar screenshot. This puts in mind all kinds of fun ideas about programmatically detecting the best color range to use based on a given image, but that's probably a bit unnecessary at this point. 😀 I'm sure we can just hardcode some gradients based on our static list of maps. I think each map will have a "slug" anyway, so "Watchpoint: Gibraltar" -> "watchpoint-gibraltar", and we can make use of that slug for CSS classes like .watchpoint-gibraltar { } for setting map-specific colors and images.
@siege911 you're a Reinhardt main now. 😆

Looks very nice but in order to be a Reinhardt main, I think I need to learn to right click occasionally. Apparently my teammates like it when I put that big blue rectangle up.

Ah one omission I'm seeing, and it might fine, is the different map sections. You have "Offense 1-3" when Rob before had them separated out as "Offense Payload 1," "Offense Payload 2," etc. Depending on what map sections we have, they might not be summarizable like in your design. For a capture map like Hanamura, we'd have Point 1 and Point 2, which can be shortened to "Points 1-2". For Eichenwalde, first you capture Point 1, then you escort the payload to point 2, then you escort the payload to point 3. So maybe for that one the columns in your design would keep separate names, like Capture Point 1, Payload Point 2, Payload Point 3.

@smbriones
I agree with Cheshire on lots of points:

portraits next to the names help recognize the hero faster
i like the edit button for the team comp name

I also love the way you've chosen to display the Offense and Defense labels. Very slick. Hopefully it can somehow still be used for hybrid maps like King's Row, where there's a capture point and payload, as Cheshire pointed out.
I posted a couple of screenshots for sample design stuff here: https://trello.com/c/wXa0dJqb. These are purely suggestions and ultimately it's up to the designers to determine what would look best for the product. If they help give you ideas, then great. If not, no worries.
As a side note, we will be deciding which features will be included in our first version of the product tomorrow. This should help better focus development and design efforts. That being said, everything you have in that sample will most likely make it into the first version.
Thanks Stephanie!

Ooh your design brings to mind another feature: we could indicate visually when there's a duplicate hero. So if Rob tried to put me and Rick both on Mercy for the same segment, the app could highlight both of us in red. There could also be a subtle checkbox that's unchecked by default for "Allow duplicate picks?" for if you wanted to comp up with a team comp for No Hero Limits in the arcade.

everything you have in that sample will most likely make it into the first version

That was my thinking when I started on this page. This composition form seemed like the core of your original app idea, Rob, so I figured it'd be fine to get it underway. 👍

So maybe for that one the columns in your design would keep separate names, like Capture Point 1, Payload Point 2, Payload Point 3

Hopefully it can somehow still be used for hybrid maps like King's Row, where there's a capture point and payload, as Cheshire pointed out.

@cheshire137 @RobThePM That makes sense! Thank you for explaining. I'll update that in the mockup.

we could indicate visually when there's a duplicate hero.

@cheshire137 Good idea! I'll work on how to show that.

we could indicate visually when there's a duplicate hero

i totally forgot about duplicate heroes, great idea!

@cheshire137 @RobThePM, based on the above points, here are some updates! Not sure if I got this data completely right, but here are examples of how we could potentially show duplicates and allowed duplicates, and also different map sections like with Hanamura and Eichenwalde.

@smbriones nice! i like how the points are displayed for the hybrid map. data looks pretty much right; we might change the label for "point 1" to "capture point 1" or something, but overall everything looks good. what determines if the red/blue line below each point label is solid or faded?

what determines if the red/blue line below each point label is solid or faded?

@RobThePM Good question! I was thinking that it could indicate that there were still selections that need to be made in that column. It's not too important, but just another visual queue. Does that make sense?

@smbriones ahhh okay, gotcha, i see that now! totally makes sense.

Control map! 😄 @cheshire137 @RobThePM

Very nice! I just now noticed the bars under each name have rounded corners on the start and end of the three segments to signal that each bar is part of a larger whole. 😀
Since this is a control map, there won't actually be attack/defend on it -- everyone's attacking. So you could lop off the three blue columns on the right and just keep the first three red columns. I don't know if that changes the layout any, since you'll have much more horizontal space available.
The 'duplicate' checkbox styling is just how I pictured it. 👌 The only thing I'd maybe change is since it's listed right after the list of players, you might think it's referring to players and not heroes. That area is where we have the most free whitespace, so I agree with your choice to stick the option there. Maybe a header and some space between the players and the checkbox, though, like "Composition Options"? There might be other settings we think of per-composition in the future that could also go there.
Another detail we might incorporate somewhere is a way to share this composition with your friends. I think initially it would just be copying the URL to the page and sending it to people outside of the app. So maybe the URL with a little copy icon next to it. Here's how that sort of thing looks on Gfycat:

This could perhaps go in that whitespace on the left, just after the 'duplicate' checkbox. We can truncate the URL with ... to fit the space, if we show the URL. Maybe we just have a copy icon and 'Share' as the clickable text.

Looking good!

"Composition Options"

I agree that we might be able to fit a "Composition Options" section somewhere.

Maybe we just have a copy icon and 'Share' as the clickable text.

I like this (or even the arrow icon for sharing from Facebook).
Also, no biggie, but for Control Maps we only need three columns since there's only three sections to the map. In my original Google sheet (https://docs.google.com/spreadsheets/d/1zstpx2-vZfGX9sgfHy5vtOmJUFOaYowtlp3l1xWbhE8/edit#gid=823800667) I went with a violet color because red + blue = violet... right? riiiiiight? 😄 However, I am NOT an artist/designer by any means so please feel free to use whatever color you like.

@cheshire137 @RobThePM

Maybe a header and some space between the players and the checkbox, though, like "Composition Options"? There might be other settings we think of per-composition in the future that could also go there.

Got it! There may be a better spot for these settings at some point, but this is probably good for now.

Maybe we just have a copy icon and 'Share' as the clickable text.

Sounds good! I'll add that as well.
Would it be helpful if I created a mockup for each map to reflect the column count and specific headers? Love that spreadsheet, @RobThePM. So helpful to see everything mapped out like that. ❤

Also, no biggie, but for Control Maps we only need three columns since there's only three sections to the map.

Since this is a control map, there won't actually be attack/defend on it -- everyone's attacking.

@RobThePM @cheshire137 you're both right, and this makes sense. I am mostly to blame for why @smbriones' mock up has the three additional columns, but I don't think the idea is communicated in the mock ups alone. The idea was that on a control map, having the flexibility to define an "attack" focused comp and a "defense" focused comp allows a team to fluidly switch focus during respawn from attack to defense and back to attack throughout a match.
For example, at the beginning of the match, we might go with the attack comp to try to capture the control point. As we hold that point, the enemy team makes adjustments to counter. Depending on the success of those counter picks, we have the option of migrating the team to the preset defense comp as the team respawns. Later in the match, despite our best defense efforts, the enemy team snowballed three ults on us and took control of the point, we can switch back to the attack comp.
I understand having both attack and defense comps defined for a control map is not currently an idea built into the game, and the two team comps for a single map would be unique to control point maps. We could think of it is "alternate comp" rather than attack and defense. I'm thinking since the UI for all other maps includes 6 columns, we could continue with that here, but would be a slightly different meaning. Curious what y'all think of this idea?

In order to accommodate multiple team comps for the same point, I thought they could added below each other. Please see the Trello card with mockup: https://trello.com/c/GNyojHlC
I call each Team Comp a "Plan".
The thought behind this mockup is several things.

I thought scrolling up and down would be an easy and quick way to navigate through the different Plans.
buttons for adding and deleting plans as needed
by setting the columns as the "static" value (i.e. The points don't change), and have the rows/names be the dynamic value, it allows you to change people'a names in case there's more than 6 people in your group and you have different Plans for the different team comps

That being said, I'm open to other ideas. :)

I thought they could be added below each other*
Also if we expand to other games I think this format will also work
(Sorry on phone)

Would it be helpful if I created a mockup for each map to reflect the column count and specific headers?

I don't think that'd be necessary. Having the general design in place is enough to start implementing it in this branch! How would you like to do this, @smbriones? I can start translating your image mockup into JSX and CSS in this branch, and you can join in, making commits to this branch as well.

The idea was that on a control map, having the flexibility to define an "attack" focused comp and a "defense" focused comp

I think I prefer Rob's idea for having these be two different compositions rather than built into one composition that overloads the meaning of the attack/defend split.

@RobThePM @cheshire137 allowing multiple team comp plans for a single map is great and maintains consistency for the data model across all map types. I'm 💯 on this rather than the dual purpose attack / defense comps for control point maps 👍

And sorry, I didn't mention it earlier, but the idea of having an attack comp and a defense comp for control maps can also be accommodated by the format in the Trello card.

We are essentially thinking of the same idea. Great minds think a like! :smile:

How would you like to do this, @smbriones? I can start translating your image mockup into JSX and CSS in this branch, and you can join in, making commits to this branch as well.

That would be perfect! 😄

It's coming along!

Wow, you are fast! 😄 Looking great, @cheshire137!

This now fixes #27 by adding the Clear Sans font.

Ever closer!

I think we could go ahead and land this and continue work on 1) styling and 2) actually hooking up these static HTML elements in a follow-up branch. The diff size of this pull request is getting large enough that I don't want to make it a bear 🐻 to review.


Toolbar button sample

Set up UiFramework in the SampleShowcase to allow AppUi samples to be added.
Add a sample to show configuration of a frontstage with a viewport.
Add a sample that uses a uiprovider to add a toolbar button at runtime.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

It looks like the convention from the UI component gallery is to leave off the prefix.  So its just Button.tsx, CheckListBox.tsx., etc.
From: NancyMcCallB<EMAIL_ADDRESS>Sent: Thursday, July 9, 2020 11:32 AM
To: imodeljs/frontend-sample-showcase<EMAIL_ADDRESS>Cc: Josh Schifter<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [imodeljs/frontend-sample-showcase] Toolbar button sample (#47)
@NancyMcCallB commented on this pull request.

In src/frontend-samples/app-ui-samples/toolbar-button-provider-sample/index.tsxhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_imodeljs_frontend-2Dsample-2Dshowcase_pull_47-23discussion-5Fr452305823&d=DwMCaQ&c=hmGTLOph1qd_VnCqj81HzEWkDaxmYdIWRBdoFggzhj8&r=nCl9AtI4SeyzQw45ZqkFTDOMf9xUcbw9RGDltaNb_vs&m=Fc7X_cG3oJ2gV1G_FMRTkNTMm6r387LNXNcInv8wAGw&s=TByn5G9vmHF8UaQWrJdtvrlZf2oFHwsXU7J5gjexfSU&e=:

@@ -0,0 +1,45 @@

+/*---------------------------------------------------------------------------------------------
I didn't notice that. When I started, all the app files were named "index" so I followed suit. Since all the apps are to show how to do UI, should I name them App or UI?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_imodeljs_frontend-2Dsample-2Dshowcase_pull_47-23discussion-5Fr452305823&d=DwMCaQ&c=hmGTLOph1qd_VnCqj81HzEWkDaxmYdIWRBdoFggzhj8&r=nCl9AtI4SeyzQw45ZqkFTDOMf9xUcbw9RGDltaNb_vs&m=Fc7X_cG3oJ2gV1G_FMRTkNTMm6r387LNXNcInv8wAGw&s=TByn5G9vmHF8UaQWrJdtvrlZf2oFHwsXU7J5gjexfSU&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AJEFFOE5W5AYSNDSO5GLOB3R2XPGRANCNFSM4OS3HXSQ&d=DwMCaQ&c=hmGTLOph1qd_VnCqj81HzEWkDaxmYdIWRBdoFggzhj8&r=nCl9AtI4SeyzQw45ZqkFTDOMf9xUcbw9RGDltaNb_vs&m=Fc7X_cG3oJ2gV1G_FMRTkNTMm6r387LNXNcInv8wAGw&s=YG82pK5PRWyJYo5tKhkwHuRYs2XmG7elO9uyOx1Fiw0&e=.


Readme edits

lots of edits that have been meaning to be done.

adds shiny, new circleci build shield
adds table for where mapzen product docs live
describes how to use Precog for testing docs
links to the writing style guide!
how to add redirects to a doc config

I removed the 'Rapid bootstrapping' section as I think we want to view changes on Precog now?
@rmglennon would love a glance-over

Thanks for the changes! This looks OK. We should figure out the relationship with the style guide. It seems reasonable to keep the actual writing style content (although it needs to be added into the table of contents) in the style guide and the build process info here. It might be getting a bit long for a readme.
One thing, I believe the section on not being able to build multiple sections at once is no longer true. I think you can build the whole help locally by running make. If I'm making a big change, I'll build locally to avoid so many precog rebuilds. You need to clear out the dist folders each time you build, otherwise it won't catch your changes and will instantly say the build is done.

I agree. Shortening this readme and maybe making a separate set of instructions for installation or updating the docs might be worthwhile (like mapzen.js's build.md).
That updating section needs love 💖 (and correct information!). I've tried the virtualenv build and couldn't get it to work personally, so revisiting how we describe that would be great.

can we get some forward momentum on this?

This looks OK and I think is correct! There is more detailed content https://github.com/mapzen/styleguide/blob/master/src/site/guides/doc-instructions.md that should get integrated at some point. Probably best to make a new page for the other stuff, as this one is getting pretty long.

closing for edits done in #244 and #250.


Move custom field settings into org settings

How are you running Flagsmith

[ ] Self Hosted with Docker
[ ] Self Hosted with Kubernetes
[X] SaaS at flagsmith.com
[ ] Some other way (add details in description below)

Describe the bug
Custom fields are defined per organisation, but the UI to configure them lives under Project Settings. This should be under Organisation Settings instead.
Steps To Reproduce
Go to Project Settings > Custom Fields. The fields shown here are global to the org, not specific to the current project
Expected behavior
Fixing this will probably break existing links like https://app.flagsmith.com/project/16945/settings?tab=custom-fields. I think this is okay given how little Custom Fields are currently used, and because the links are project-specific and not global to all users.
Screenshots
No response

I've taken a look at this, and for some reason it looks like creating/editing the fields is done on an organisations level but making a field "required" is done on the project level via a is_required_for "object_id" which is a project id. I don't think this makes sense, we should either be creating a field at the project level or making the required attribute organisation wide.
It seems in order to move this to organisation settings the easiest approach would be to just remove the object_id parameter.


treewide: remove lib.meta.enableIfAvailable

Motivation for this change
This seems like a better approach than "enableIfAvailable" to me. The issue with "enableIfAvailable" is it combines two concepts: optional dependencies (usually configured by a flag) and availability, that are better left separate. This may seem convenient but it gives users less control over what packages are being built. For both packages (Nix & Systemd) using "enableIfAvailable" right now, it's pretty clear that they are being used as we normally use optional dependencies. Both "libseccomp" & "kexectools" can be left out and they will run correctly, whether each is broken or not. So, it makes more sense to make these configurable with a flag. We can still provide libseccomp.meta.available & kexectools.meta.available as defaults.
I don't know if anyone has strong feelings in favor of enableIfAvailable. I definitely think it's an overloaded concept that shouldn't be used extensively in Nixpkgs. If necessary packages can always define their own version of enableIfAvailable, but I definitely don't think it should be included in lib.

@Ericson2314

I disagree with this foo.meta.available thing if it means specifying config.allowUnsupportedSystem effectively breaks nixpkgs my making it do unpredictable things.


Default supported verbs/options should not return non-zero exit code

This library should not return a non-zero exit code for the default supported verbs and options:

help
version
--help
--version

using System;
using CommandLine;

public class Program
{
    [Verb("add", HelpText = "Add file contents to the index.")]
    class AddOptions
    {
        [Option(Default = false, HelpText = "Prints all messages to standard output.")]
        public bool Verbose { get; set; }
    }

    [Verb("commit", HelpText = "Record changes to the repository.")]
    class CommitOptions
    {
    }

    static int Main(string[] args)
    {
        var exitCode = Parser.Default.ParseArguments<AddOptions, CommitOptions>(args)
            .MapResult(
                (AddOptions opts) => 0,
                (CommitOptions opts) => 0,
                errs => 1
            );

        Console.WriteLine($"exitCode = {exitCode}");
        return exitCode;
    }
}

Executing this code results in the following outputs:
> ConsoleApp1.exe version
ConsoleApp1 <IP_ADDRESS>
exitCode = 1

> ConsoleApp1.exe help
ConsoleApp1 <IP_ADDRESS>
Copyright c  2022

  add        Add file contents to the index.

  commit     Record changes to the repository.

  help       Display more information on a specific command.

  version    Display version information.

exitCode = 1

> ConsoleApp1.exe --version
ConsoleApp1 <IP_ADDRESS>
exitCode = 1

> ConsoleApp1.exe --help
ConsoleApp1 <IP_ADDRESS>
Copyright c  2022

  add        Add file contents to the index.

  commit     Record changes to the repository.

  help       Display more information on a specific command.

  version    Display version information.

exitCode = 1

> ConsoleApp1.exe help add
ConsoleApp1 <IP_ADDRESS>
Copyright c  2022

  --verbose    (Default: false) Prints all messages to standard output.

  --help       Display this help screen.

  --version    Display version information.

exitCode = 1

> ConsoleApp1.exe add --version
ConsoleApp1 <IP_ADDRESS>
exitCode = 1

> ConsoleApp1.exe add --help
ConsoleApp1 <IP_ADDRESS>
Copyright c  2022

  --verbose    (Default: false) Prints all messages to standard output.

  --help       Display this help screen.

  --version    Display version information.

exitCode = 1

As far as I can tell, those errors can be handled by the consumer as in your code example or by doing it like suggested in #660.
However, since those verbs/options are supported they should not result in an error and therefore also should not require to be handled by the consumer.

This behaviour is very weird, no stablished command line I know would return != for version/help options


Add XInputToZMQPub, processor, commands and zaber to control the stage with Xbox controller

According to the software diagram, these devices communicate with each other via zmq ports. Zaber device also communicates through forwarder in order to receive tracking commands later.


Bugs preventing reproducing results

The code is broken in a number of places, making it impossible to reproduce results:

There is no way to specify args.mode in run.py other than editing it into the script

passing --mode train to run.py does not work, because that argument will be stored in args.train.mode, not args.mode.
The same is true for args.checkpoint_dir for inference.

In gen_summary/inference.py:38, you assign to self.bart.cfg.dataset.batch_size_valid = bsz. I'm not sure what this is supposed to do, but the bart object comes from fairseq and has no cfg member.
in stage 2, run.py does not actually use the output from stage 1 and will just create the same input/output as stage 1 again

the others are one-line fixes but for this one I am not sure how to fix

For others who are also trying to reproduce:

The required fairseq version is actually 0.10.0, there is no 1.10.0

You also need to copy the examples folder somewhere and add its parent folder to PYTHONPATH. You must move it out of the main fairseq dir, or the fairseq logging module will override the Python builtin.

Since AnyROUGE does not provide any installation instructions: Clone the library and add its root folder to the PYTHONPATH.
Then clone https://github.com/pltrdy/rouge into the ThirdParty folder inside AnyRouge. If you put it somewhere else or pip install it, you will need to edit the scripts to change imports.

Hello, can you share your modified code? thanks.

in stage 2, run.py does not actually use the output from stage 1 and will just create the same input/output as stage 1 again

I think the at line 100 of run.py, it is getting the output data from stage 1 with args.cur_stage still being 1 at that time:
source_path = os.path.join(args.train.output_path, f"stage_{args.cur_stage}")
cur_source = load_split_aslist(source_path, suffix='hypo')

But the other mentioned problems do exist and also at line 87 and 123 or run.py, the path to the checkpoint should be something like:
os.path.join(training_args.checkpoint_dir, f"stage_{args.cur_stage}/trainer_output")

if we want to use the checkpoints after training without any file/folder movements.

I only achieved around 70%  paper rouge score on AMI dataset.
I used fairseq 0.12.2 or 0.10.0 + comment out codes about self.bart.cfg.xxxxx in inference.py
Is there something except @felix-schneider & @xianglous said should be modified?

Hi @ CAH9487, I would like to know if you produced the result. do we need to change anything about the configure file? Thanks!


IPv4 multicast datagrams can't be received for mimxrt1064_evk board (missing ethernet API)

Describe the bug
IPv4 multicast datagrams can't be received for mimxrt1064_evk board (missing ethernet API).
In order to support IPv4 multicast datagrams, it is necessary to modify the mimxrt1064_evk ethernet driver API to join a multicast group.
For mimxrt1064, it is the file eth_mcux.c, which could provide the configuration function set_config to add or remove a multicast source adress filter to the interface.

Open the file $ ZEPHYR_BASE / zephyr / drivers / ethernet / eth_mcux.c
the following static function could be defined

static int set_config (struct device * dev,
                           enum ethernet_config_type type,
                           const struct ethernet_config * config)
{
    int result = -1;
    switch (type)
    {
    ETHERNET_CONFIG_TYPE_FILTER box:
    {
        switch (config-> filter.type)
        {
        ETHERNET_FILTER_TYPE_SRC_MAC_ADDRESS box:
        {
            / * Adds or leaves the ENET device to a multicast group. * /
            uint8_t multicastMacAddr [6];
            for (uint32_t i = 0; i <6; i ++)
            {
                multicastMacAddr [i] = config-> filter.mac_address.addr [i];
                if (config-> filter.set)
                {
                    printk ("\ r \ nENET_AddMulticastGroup \ r \ n");
                    ENET_AddMulticastGroup (ENET, multicastMacAddr);
                }
                else
                {
                    printk ("\ r \ nENET_LeaveMulticastGroup \ r \ n");
                    ENET_LeaveMulticastGroup (ENET, multicastMacAddr);
                }
                result = 0;
            }
        }
        break;
        default:
        {
            // Nothing
        }
        break;
        }
        break;
        default:
        {
            // Nothing
        }
        break;
    }
    }
    return result;
}

... and added to the api by modifying the api_funcs structure:

static const struct ethernet_api api_funcs = {
    .iface_api.init = eth_iface_init,
    .get_capabilities = eth_mcux_get_capabilities,
    .send = eth_tx,
    .set_config = set_config, / * Adding the set config function to the API * /
};

To Reproduce
Try to receive UDP ipv4 multicast datagrams.
Environment (please complete the following information):

OS: linux
Toolchain: zephyr sdk v0.1.13
board: mimxrt1064_evk
SHA: 979e124e21fb05cfedf8ed12078983d40e2f0b6c

For IPv6 there is net_if_mcast_mon_register() that can be used to register a function that will be called if a multicast address is added/removed to the network interface. That function should have IPv4 address support and then we could enhance the IPv4 support in the driver side. I am not convinced that we should have this implemented via set_config().

@jukkar i've looked at the driver and adding the driver part is not difficult we have to re-purpose net_if_mcast_cb and remove ipv6 configuration, also re-purposing net_if_mcast_mon_register() is not difficult.
However I need to ask you don't we need to add some new ipv4 functions/events/handles that handle joining/leaving ethernet multicast groups? I.e. networking stack support.

However I need to ask you don't we need to add some new ipv4 functions/events/handles that handle joining/leaving ethernet multicast groups? I.e. networking stack support.

Yes, we need to join/leave multicast group when IPv4 address is added/removed to/from network interface. Sorry for not being more clear in my words above. So this would need to be implemented, basically the functionality should be similar as with IPv6.

Is this still a problem with tcp2?

On the issue of adding join/leave API, is this something that is generically needed across all drivers?

Probably yes, typically (at least for IPv6 mcast msg) the driver needs to program the Ethernet controller so that it can receive the multicast packets.

Also, if we add the .set_config would that be enough to close this ticket?

We have an API where a driver can register a function to be called when net stack joins/leaves a multicast group. Right now that API supports only IPv6 addresses. That API would need to be extended to support also IPv4 addresses. See for example drivers/ethernet/eth_mcux.cand search for net_if_mcast_mon_register() to see how it is used.
But this is only part of the story as also network stack would need to support IPv4 multicast handling similar way as what have been done for IPv6. Right now there is an API to add a IPv4 multicast address to the network interface net_if_ipv4_maddr_add(() but it does not trigger a call to multicast monitoring which would inform the driver that the multicast group needs to be joined.
Changes to the code to support this are not very big, it is just that someone needs to join the dots to make this work and test it properly.

@jukkar, If I'm understanding you then it seems like this isn't a platform specific bug?

@dleach02 First of all, this is not a bug, we are just missing a feature. And second, this is not a platform specific issue.

Cool... so @jukkar we can convert this to an enhancement request then?

we can convert this to an enhancement request then?

Yep, done.


Feat/sqlalchemy 2 support

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (8d7ff43) 100.00% compared to head (968fd7f) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #10   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          292       292
=========================================
  Hits           292       292

Impacted Files
Coverage Δ

sqlalchemy_filterset/filters.py
100.00% <ø> (ø)

sqlalchemy_filterset/filtersets.py
100.00% <100.00%> (ø)

sqlalchemy_filterset/strategies.py
100.00% <100.00%> (ø)

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

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

Any update for sqlalchemy 2 support?

Bump

If you use poetry, you can add dependency this way:
sqlalchemy-filterset = { git = "https://github.com/sqlalchemy-filterset/sqlalchemy-filterset.git", branch = "feat/sqlalchemy-2-support" }

Sketchy but ok 😄 Thanks for the response!


System.DllNotFoundException: libskia_osx.dylib

It looks like the NuGet package of SkiaSharp is failing to copy libskia_osx.dylib to Mac applications during the build. To repro:

Create a MonoMac (Open Source) Cocoa Project.
Add the SkiaSharp Nuget package.
Add some code that uses SkiaSharp.
Run the application.
Application crashes with System.DllNotFoundException: libskia_osx.dylib.

After manually copying SkiaSharp.<IP_ADDRESS>/build/net45/mac/libskia_osx.dylib to bin/Debug/MyApp.app/Contents/MonoBundle/ the app runs perfectly.
I was able to reproduce in both SkiaSharp.<IP_ADDRESS> and SkiaSharp.1.49.4-beta.

MonoMac is no longer maintained, and this is one of those side effects, it has not caught up with the various developments in the space.
Migrate your application to Xamarin.Mac, which is the evolved version of Xamarin.Mac and is also fully open source now.