Clarify openfire validation set creation

This PR aims at:

clarifying the name to set validation ratio (between 0 and 1) instead of percentage (between 0 and 100)
raising exception when this ratio is not within correct range. Before, it was possible to set wrong number (ie: setting 21.5 instead of 0.215 would not raise any warning nor exception)

Any feedback is welcome,
cheers

Closing this as the valid_pct argument has been removed since the train/test split is not fixed directly in the downloaded annotations.
See #32


Got "Trust anchor for certification path not found" error with installed certificate

I got this error
javax.net.ssl.SSLHandshakeException: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.
after installed CA certificate.
What I missed ? running on android 6

installed ca-certificate-rsa.cer instead of ca-certificate-ec.cer and worked well


Add hosts for Draw Joust

Add multiple hosts for ads and trackers used by the Draw Joust game
by VOODOO.

Thank you for this Mackus @marbens-arch!
37 category-specific domains with zero overlaps. What's not to love ❤ about that?
Merging!

Thank you for this Mackus @marbens-arch!
37 category-specific domains with zero overlaps. What's not to love ❤ about that?

I think it's actually 26.


QueryRewrite: implement include and re-enable include tests

[x] nav expansion reference include
[x] nav expansion collection include
[x] nav expansion string based include
[ ] tracking
[ ] reference include translation
[ ] collection include translation
[ ] async

Tracking is inbuilt in materialization. Reference include merged. Collection include work is tracked by #15611


Change Event.srcElement type to EventTarget and expose it to Worker

This PR changes Event.srcElement type to EventTarget and exposes it to Worker. Accepting this PR should close https://github.com/Microsoft/TypeScript/issues/28252, however I would like to hear from @mhegazy first as to why Element type was chosen. Maybe there's a specific reason I don't know and now I'm ruining everything :).

 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.:x: FrogTheFrog sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@saschanaz Could you merge this if there are no other problems with this?

Pinging @sandersn @DanielRosenwasser for this, as I have no write access.


command not found: pyftsubset

I have set what is needed, but there is an error
zsh: command not found: pyftsubset

how did you install fonttools?

pip install fonttools

If repeat, I get it
Requirement already satisfied: fonttools in /Library/Python/2.7/site-packages

which python are you using? The macOS built-in python or homebrew or what?
Try to do this to get the full path to the binary called "python":
$ which python

Also, echo the content of your $PATH to check that /usr/local/bin is in there.
If you pip install from within a virtual environment, the installed console scripts' folder is automatically added to your shell $PATH once you activate that virtual environment.

brew install fonttools or, if already installed, brew link fonttools

brew install fonttools

no, that's not supported (not by us at least).
pip install fonttools is the official way.

Hello,
i have the same problem.
When i do echo $PATH in terminal i have : /Library/Frameworks/Python.framework/Versions/3.6/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/local/git/bin

Also, echo the content of your $PATH to check that /usr/local/bin is in there
i suppose that it's ok because usr/local/bin is here.

i tried to reinstall fonttools like this
pip install --ignore-installed fonttools
but i still can't use subset :(
Cheers,
bernard

looks like you do have another python available, a python3.6 from the official python.org installers.
just use that one to install.
python3.6 -m pip install fonttools

It works fine now, thanks a lot!
i noticed that this msg below was about python 2.7 :
"Requirement already satisfied: fonttools in /Library/Python/2.7/site-packages"
and the echo $PATH was ...Python.framework/Versions/3.6....

if you still want to install fonttools with the built-in macOS python, you have to avoid sudo and pass the --user option, like so:
$ python -m pip install --user fonttools

When you use --user, the scripts will not go to the global /usr/local/bin directory, but to a user's specific folder (for my user "clupo" this is /Users/clupo/Library/Python/2.7/bin). This folder is not in your $PATH by default, so you have to configure your environment to add this folder to the search path for executables. Google or stackoverflow how to do that.


4.0.3

[X] Completed PR - Has all features or fixes that it was meant to and isn't half done
[X] Follows the style guide - Style guide found here
[X] All changes have been tested

Added
Usage command
Changed
Logic on fix
Ordering of perms
Fixed
Autoassign bug
Purge ConcurrentModificationException
Permissions

@WalshyDev I'm confused why you open a PR for this..? xD

Only way to merge on mobile, plus we are investigating more bugs now :p

True


Implement test with multiple different root tables

Would be good to verify, should be possible: [(http://stackoverflow.com/questions/35937976/multiple-root-tables-in-flatbuffers)]

IIUC, we should codegen
	public static func fromByteArray(data : UnsafeBufferPointer<UInt8>, config : BinaryReadConfig = BinaryReadConfig()) -> T
	public static func fromRawMemory(data : UnsafeMutablePointer<UInt8>, count : Int, config : BinaryReadConfig = BinaryReadConfig()) -> T

for all tables, not only the declared root one?

I am not sure it is a good idea because it interferes with file_identifier I guess #48 is a better solution where you can specify different root_type corresponding to different file_identifier

#48 solves a slightly different problem - the ability to merge multiple schemas - this case was about being able to decode i.e. FoobarContainer or Foobar with the classes that were in this case in the same file.
The use case is e..g. when you get multiple different messages sent to you over a network connection, it works to define each in a separate .fbs, but perhaps a bit cumbersome. I could live with #48 though.


New host instances can't start tasks using ECS ami-3fc1b928

I have been using weave with three hosts however two of these instances are spot. They go down and back up few times but lately, all tasks in these instances are started but they are not fully initialized, simply there is no logs for these tasks.
Checking weave container logs I could see, It is trying to connect a dead instance. Any ideias what could be wrong or how to solve it?
ca420ecc9f70        weaveworks/weave:1.6.2           "/home/weave/weaver -"   10 minutes ago       Up 10 minutes                            weave
[ec2-user@ip-10-0-102-15 ~]$ docker logs ca420ecc9f70
INFO: 2016/10/26 20:42:23.109116 Command line options: map[dns-effective-listen-address:<IP_ADDRESS> dns-listen-address:<IP_ADDRESS>:53 http-addr:<IP_ADDRESS>:6784 ipalloc-ran
ge:<IP_ADDRESS>/12 port:6783 resolv-conf:/var/run/weave/etc/resolv.conf datapath:datapath name:4a:f3:9e:4b:61:a2 nickname:ip-10-0-102-15]
INFO: 2016/10/26 20:42:23.114180 Communication between peers is unencrypted.
INFO: 2016/10/26 20:42:23.125634 Our name is 4a:f3:9e:4b:61:a2(ip-10-0-102-15)
INFO: 2016/10/26 20:42:23.125689 Launch detected - using supplied peer list: [<IP_ADDRESS> <IP_ADDRESS>]
INFO: 2016/10/26 20:42:23.128557 Docker API on unix:///var/run/docker.sock: &[GoVersion=go1.5.3 Os=linux Arch=amd64 KernelVersion=4.4.19-29.55.amzn1.x86_64 Version=1.11
.2 ApiVersion=1.23 GitCommit=b9f10c9/1.11.2]
INFO: 2016/10/26 20:42:23.129696 [allocator 4a:f3:9e:4b:61:a2] No valid persisted data
INFO: 2016/10/26 20:42:23.152445 [allocator 4a:f3:9e:4b:61:a2] Initialising via deferred consensus
INFO: 2016/10/26 20:42:23.164245 Listening for DNS queries on <IP_ADDRESS>
INFO: 2016/10/26 20:42:23.164352 Sniffing traffic on datapath (via ODP)
INFO: 2016/10/26 20:42:23.169327 Listening for HTTP control messages on <IP_ADDRESS>:6784
INFO: 2016/10/26 20:42:23.200582 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:42:23.200740 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:42:23.205764 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:42:23.207641 ->[<IP_ADDRESS>:6783|86:7b:00:0c:f2:8e(ip-10-0-101-5)]: connection ready; using protocol version 2
INFO: 2016/10/26 20:42:23.207812 overlay_switch ->[86:7b:00:0c:f2:8e(ip-10-0-101-5)] using fastdp
INFO: 2016/10/26 20:42:23.207968 ->[<IP_ADDRESS>:6783|86:7b:00:0c:f2:8e(ip-10-0-101-5)]: connection added (new peer)
INFO: 2016/10/26 20:42:23.221856 EMSGSIZE on send, expecting PMTU update (IP packet was 60028 bytes, payload was 60020 bytes)
INFO: 2016/10/26 20:42:23.222017 overlay_switch ->[86:7b:00:0c:f2:8e(ip-10-0-101-5)] using sleeve
INFO: 2016/10/26 20:42:23.222091 ->[<IP_ADDRESS>:6783|86:7b:00:0c:f2:8e(ip-10-0-101-5)]: connection fully established
INFO: 2016/10/26 20:42:23.222286 overlay_switch ->[86:7b:00:0c:f2:8e(ip-10-0-101-5)] using fastdp
INFO: 2016/10/26 20:42:23.223776 sleeve ->[<IP_ADDRESS>:6783|86:7b:00:0c:f2:8e(ip-10-0-101-5)]: Effective MTU verified at 8939
INFO: 2016/10/26 20:42:23.834597 Weave version 1.7.2 is available; please update at https://github.com/weaveworks/weave/releases/download/v1.7.2/weave
INFO: 2016/10/26 20:42:24.823201 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:42:24.823539 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:42:29.256167 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:42:29.256576 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:42:35.863440 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:42:35.863976 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:42:42.595814 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:42:42.596230 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:42:49.290667 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:42:49.291078 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:43:08.272476 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:43:08.272786 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:43:33.728925 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:43:33.729304 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:44:17.556602 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:44:17.569108 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:45:28.611499 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:45:28.612065 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:47:03.677807 Discovered remote MAC c2:02:40:94:6c:05 at 86:7b:00:0c:f2:8e(ip-10-0-101-5)
INFO: 2016/10/26 20:47:06.797489 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:47:06.798023 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:48:17.435153 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:48:17.435563 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:50:12.123765 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2016/10/26 20:50:12.124367 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: getsockopt: connection refused
INFO: 2016/10/26 20:51:00.419966 [allocator]: Allocate request for 2f52bd30d5c49116a8f62bb789f08e5742a9d46cf1f79b1b474c91ccbc480322 cancelled

Hi @dextervip!
The peer list <IP_ADDRESS> <IP_ADDRESS>: were those, at the time of launch, running Weave Net?
The logs indicate that they refused connection.

Also, can you run weave status on the node where you have tasks that won't start.


Fixed test workflow pull request number

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1007  👈
#1006
main

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

Join @CatChen and the rest of your teammates on  Graphite


Replace @metahub/conventional-commit-types with conventional-changelog-metahub

Codecov Report

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

@@          Coverage Diff          @@
##           master     #3   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      4    +1
  Lines          41     47    +6
=====================================
+ Hits           41     47    +6

Impacted Files
Coverage Δ

src/index.js
100% <100%> (ø)
:arrow_up:

src/aliases.js
100% <100%> (ø)

Continue to review full report at Codecov.

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


Add instruction opcode retrieval

This, along with the values in the LLVM headers, is very brittle but at some point we have to depend on the raw values themselves so there's not much choice.

⛵


add Histogram lineWidthValue and lineWidthSignal props

The necessary guide is explained in the overlay-object.md file

What is the main goal of this PR? seems like it making several unrelated changes

What is the main goal of this PR? seems like it making several unrelated changes

Adding more histogram and range customization, as well as adding custom icons to the  legendژontroller to change the chart variables at the moment.

What is the main goal of this PR? seems like it making several unrelated changes

Adding more histogram and range customization, as well as adding custom icons to the legendژontroller to change the chart variables at the moment.

What is the main goal of this PR? seems like it making several unrelated changes

Adding more histogram and range customization, as well as adding custom icons to the legendژontroller to change the chart variables at the moment.


Table entries

Parse table entries and embed them in IR.

Note: there are protobuf definitions for what a table entry looks like in pdpi https://github.com/google/p4-pdpi/blob/master/p4_pdpi/ir.proto.
We can reuse that, or reuse the one in p4runtime https://github.com/p4lang/p4runtime/blob/f324cba9b213c617de94ffeb0e1bd65d3aa32e19/proto/p4/v1/p4runtime.proto.
I am leaning more towards the p4runtime one, since it can parse all the data formats for us (e.g. ipv4, ipv6) as bit strings, which we can use directly. Another consideration is which can parse the entries out of the box for us.
I am planing to do this switch later on, for now I want to use this current code as a shortcut to get the end-end prototype working. When I do this switch, this will essentially change the proto definition from this PR to use one of the two options above, and will likely change table_entries.cc to be more like pdpi_driver.cc (i.e. a wrapper around calls to the library we decide on to parse the data for us). The rest of the code will remain similar.

Address most comments, blocked on RET_CHECK getting added to pdpi.

All comments have been address: main cleanup is on #30.


React-Native user authentification check takes a lot of time

✅ Prerequisites

[✅] Did you perform a cursory search of open issues? Is this question already asked elsewhere?
[✅ ] Are you reporting to the correct repository (magic-sdk)?

❓ Question
We have several questions regarding the authentification process with magic, in particular how to handle the authentification status check on app startup (react-native app). Do you have any best practices about that?
Here is how we do it now when the app is loaded:

Check if user is logged in via magic.user.isLoggedIn()
If yes -> magic.user.getIdToken() and magic.user.getMetadata() in parallel

The problem is that the first request takes from 3 to 5 seconds and the second pair also takes from 3 to 5 seconds to receive data. So users have to wait ~10 secs every time they open the app. Is any chanse we can speed up that process / change the flow?
🌎 Environment

Software
Version(s)

magic-sdk
"@magic-sdk/react-native": "^8.2.1"

Operating System
react-native / iOS / Android

Hi @pisarevkir, thank you for your feedback. Generally, calls combined should only take around 2~3s. Do you mind sharing your rough location so we can check if it's related?

Hi @Ethella!
We tried to execute those requests from Romania, Russia, Ukraine.
And the first step - magic.user.isLoggedIn() takes from 2s to 5s.
The second step for authorized users takes slightly less time, from ~1.5 to 4 secs:
Promise.all([
      magic.user.getIdToken(),
      magic.user.getMetadata(),
])

Just wonder if it's viable solution to get rid of the magic.user.isLoggedIn part and make a decision if user is logged-in depending on either magic.user.getIdToken returns error with some specific code or actual token string?


Update service version SDK to 3.2-preview.1

Update changelog and Readme with new service version
Update service version enum
Re-record all tests

Fixed by https://github.com/Azure/azure-sdk-for-net/pull/23078.


Remove traits with unconstrained Time associated types.

As discussed in https://github.com/rust-embedded/embedded-hal/issues/201 and
https://github.com/rust-embedded/embedded-hal/issues/177#issuecomment-910396479 ,
the traits that have an unconstrained type Time associated type end up
not being usable for HAL-independent drivers in practice, since there is too
much variation across HALs of what the actual Time type is.
These should be bound to something, probably a Duration trait, so that
drivers can actually create and use them. Alternatively embedded-hal could have
actual struct Duration and so.
Either way, it's unclear what the final solution would look like. We shouldn't leave
these traits in the 1.0 release, since fixing them later will be a breaking change.
We should temporarily remove them, and add them back once we have figured this out
which won't be a breaking change.

r? @eldruin
(rust-highfive has picked a reviewer for you, use r? to override)

Uh, this removes a few traits.
Before considering this, I would like to see an assessment of which public HALs implement them and which drivers use any of them to see how much downstream code would break.

I've created this source graph query on the bases of https://github.com/rust-embedded/embedded-hal/issues/177#issuecomment-910396479 to see, which crate is implementing these traits and also to give an overview, which Time types are used.
Based on the curated drivers list on awesome-embedded-rust and a quick and dirty script,
I also created a similar [query](timeout:3m0s repo:github.com/(rahul-thakoor/mma7660fc|JoshMcguigan/tsl256x|eldruin/kxcj9-rs|braun-embedded/rust-dw1000|dbrgn/shtcx-rs|japaric/enc28j60|danielgallagher0/hts221|rainbow-hat-rs/|aw/sram23x|sousandrei/iis2mdc|jamwaffles/ssd1306|japaric/l3gd20|eldruin/tmp1x2-rs|redpfire/usbd-webusb|e-matteson/keypad|kallemooo/adafruit-7segment|mvniekerk/microchip-eeprom-25x|eldruin/tmp006-rs|almindor/st7789|mvniekerk/microchip-tc72-rs|master/tcn75a|cs2dsb/adafruit-alphanum4.rs|eldruin/ads1x1x-rs|daschl/tsic-rs|eldruin/ds1307-rs|leshow/rotary-encoder-hal|japaric/lsm303dlhc|JitterCompany/adxl355-rs|chocol4te/aht20|jamwaffles/sh1106|eldruin/opt300x-rs|NeoBirth/ADXL343.rs|sousandrei/ism330dhcx|mvirkkunen/usbd-serial|twitchyliquid64/usbd-hid|agalakhov/usbd-hid-device|michaelbeaumont/dht-sensor|thejpster/embedded-sdmmc-rs|pcein/adc-mcp3008|eupn/bno055|flott-motion/stepper|eldruin/mcp794xx-rs|JoshMcguigan/shift-register-driver|dbrgn/mcp3425-rs|rust-iot/rust-radio-sx128x|lonesometraveler/max6955|eldruin/embedded-ccs811-rs|eldruin/veml6030-rs|eldruin/veml6075-rs|mvniekerk/adxl313-rs|danielgallagher0/bluenrg|ryankurte/rust-radio-sx127x|eldruin/ad983x-rs|eldruin/eeprom24x-rs|eldruin/pwm-pca9685-rs|dbrgn/sgp30-rs|eldruin/si4703-rs) lang:rust \b(WatchdogEnable|CountDown|Pwm|Capture)\b) for drivers the driver part (which sadly times out).
But [even](timeout:3m0s repo:github.com/(JoshMcguigan/tsl256x|agalakhov/usbd-hid-device|mvirkkunen/usbd-serial|redpfire/usbd-webusb|eldruin/veml6030-rs|twitchyliquid64/usbd-hid|eldruin/si4703-rs|rahul-thakoor/mma7660fc|braun-embedded/rust-dw1000|eldruin/tmp006-rs|dbrgn/sgp30-rs|eldruin/veml6075-rs|eldruin/opt300x-rs|master/tcn75a|mvniekerk/microchip-tc72-rs|eldruin/pwm-pca9685-rs|jamwaffles/ssd1306|rainbow-hat-rs/|leshow/rotary-encoder-hal|ryankurte/rust-radio-sx127x|kallemooo/adafruit-7segment|eldruin/tmp1x2-rs|aw/sram23x|almindor/st7789|jamwaffles/sh1106|JoshMcguigan/shift-register-driver|dbrgn/shtcx-rs|daschl/tsic-rs|flott-motion/stepper|rust-iot/rust-radio-sx128x) lang:rust \b(WatchdogEnable|CountDown|Pwm|Capture)\b) smaller  queries did not lead to any matches at all. So this might be the wrong way to get an idea, which driver uses these traits. (Or there are just no matches at all)
But hopefully this creates a basis for discussion.

thanks for the analysis! there's definitely room for improvement on em, which either way will be a breakage i guess, but, i still feel a little wary of removing all these (also particularly interested in whether this has flow-on impact on RTIC timer definitions?)
i'm not sure this achieves all goals, but, what if instead of associated types we elected here to define a really basic timestamp / duration type (maybe something like Ts{ s: u32, us: u32 } inspired by the classic struct timespec?) and then to bound the traits by Into<Ts>. that way we're providing a simple primitive and if embedded-time (or any other approach) want to work with this it can be implemented outside the hal.

I feel like, in previous discussion and  PRs it was made clear, that the most important priority e-h should solve, is to provide a consistent interface between drivers and MCUs, especially to be able to independently implement a half of the interface<->driver combination.
Everything, that misses that goal, should not be part of this crate. And, as of now, the traits which will be removed in this PR do miss the goal with the unconstrained Time type.
The fear of breaking many downstream crates while removing these traits is very real and very reasonable, so this problem should be solved.

i'm not sure this achieves all goals, but, what if instead of associated types we elected here to define a really basic timestamp / duration type [...]

Even though this sounds like an interesting idea, I propose we should not go this route.  Any discussion about an alternative simple Time type sets us back to square one, where we have to come to an agreement how this simple solution should look like. With a discussion like that started, this is still a blocker for a v1.0 release, which this PR is deliberately trying to solve, by just removing these traits for now.
IMHO, more important right now is to solve the breakage issue for downstream HAL implementations and maybe inter-op between HAL impls and RTIC apps. First of, I don't think this is a blocker for this PR in particular, but we should have a solution for that concern before v1.0 is released.
I have a few ideas, but I'm not really particularly happy with anyone of these (maybe someone else can iterate ontop of these ideas):

Break the world. Just remove them. This breakage could either lead to a big hurdle to move to v1.0 or to the situation, that the implementations just vendor / copy & paste these traits into the HAL implementations. This could lose the uniformity between these implementations over time, as adjustments to these traits can be made anytime. I'd argue this problem already exists, because of the widely different Time types used.
Move these traits behind a unstable feature flag, for which the semantic versioning rules are loosened, and breaking changes are ok, at least between minor versions. This reminds me of the unproven flag, so this might be a bad idea. Anyway, this would allow the HAL implementations to go-on their lives to keep their current Time based implementations and the churn is not unnecessarily higher as it is already to move to v1.0.
Move these traits to a companion crate (could still be part of this repository)  to solve point 1., which is advertised as temporary from the beginning until the Time situation is solved. This crate could also be used as a testing bed to iterate on the  Time design, including breaking changes until the WG is comfortable to move these into e-h again.

I'm out of ideas, but maybe there are other solutions as well.

Need testers https://github.com/stm32-rs/stm32f4xx-hal/pull/381

Have anyone saw this?
https://github.com/BlackbirdHQ/atat/blob/e5d509bf3871f540fe88f069b409555bee93ab3e/atat/src/lib.rs#L281

PwmPin does not contain associated Time type. Why it was removed?

@burrbull it was designed to work together with Pwm which does have the type Time problem. IMO the safest is to remove both,  I'm not sure the PwmPin trait makes sense "standalone".
We can redesign and add it back later, either as a "Pwm" peripheral with multiple pins/channels, or a standalone "PwmPin" that just allows setting duty % without "caring" about the full peripheral...

it was designed to work together with Pwm

It is not true. PwmPin was added much more earlier than Pwm

I stand corrected then!
I still think it suffers from a similar issue: it has an unconstrained type Duty; that can be integer, float... How can a HAL-independent driver make effective use of it?

Yeah, we should start some kind of "Get me those traits back" initiative and detail what the types could be :)

@eldruin please point to the issues you mentioned. I have several drivers I'd like to publish in the next months that depend on timer traits (especially CountDown). I'm using embedded-time with https://github.com/drogue-iot/drogue-embedded-timer as a compatibility layer to circumvent the problems mentioned here. I want to speak up to get (reasonably bound) timer traits back in!

Thanks for the reminder @niclashoyer.
Here is the issue with the overall roadmap for these traits: #357


fix(playbook/init): ensure hostvars exists

Create the missing folder.

Adapt to match mono && multi


upgrade to latest sbt 1.3.x version (1.3.13)

[error] /Users/tisue/scala-ssh/src/main/scala/com/decodified/scalassh/ScpTransferable.scala:52:33: not enough arguments for constructor LoggingTransferListener: (x$1: net.schmizz.sshj.common.LoggerFactory)net.schmizz.sshj.xfer.LoggingTransferListener.
[error] Unspecified value parameter x$1.
[error]   private def defaultListener = new LoggingTransferListener
[error]                                 ^
[error] /Users/tisue/scala-ssh/src/main/scala/com/decodified/scalassh/SshClient.scala:63:44: implicit numeric widening
[error]         case Some(timeout) => channel.join(timeout, TimeUnit.MILLISECONDS)
[error]                                            ^

I saw this same error in the Scala community build and thought I should report it, so this PR is a way of reporting it.

Thank you, Seth!

I should probably have said: I don't know what's going on here. My best guess would be that the newer sbt is using newer Coursier and that something changed in the handling of transitive dependencies, leading to a different, unexpected set of library versions on the classpath.

This is indeed a strange problem.
No idea, what's happening underneath.
The fix I applied now is easy, but I still don't quite understand, where the version conflict comes from.
Anyway, should be working now...
Thanks again for the PR, Seth!

https://github.com/scala/community-build/commit/25d9ddee9bad1d2e5c808be17e708bbb97eceb04 👍


Allow compiling w/ autoconf 2.68

This version is part of Ubuntu 12.04, so used by at least one major supported distribution at the moment.
This change should have no negative impact on systems with newer autoconf.

I'm not going to test this. :) Agreed this likely won't cause any issues.
Thanks.


[TypeScript] LocaleRoute should return Location

Version
nuxt-i18n: 6.12.2
nuxt: 2.13.1
What is the problem ?
From vue-router type definitions:

push(location: RawLocation): Promise<Route>
replace(location: RawLocation): Promise<Route>

Those methods expect type RawLocation = string | Location but localeRoute from nuxt-i18n returns Route object which is slightly different from Location
Solution expected

localeRoute(route: RawLocation, locale?: string): Route | undefined
should become localeRoute(route: RawLocation, locale?: string): Location | undefined

nuxt-i18n doc is referring to localeRoute along side with $router.push which will return a type error

For reference: https://github.com/vuejs/vue-router/blob/f0d9c2d93cf06b5b2202bd77df6281cf47970b20/types/router.d.ts#L138-L158
I suppose it should work to just change the type. The object will still technically be a route and will potentially contain some extra properties but that should hopefully not cause any issues.

@rchl this issue is still relevant as the types have not been changed in the types declaration here

That got accidentally reverted during types refactoring.
But now I'm having seconds thoughts about whether I should change it. The method is called localeRoute and it does return a Route object. The issue seems more to be in the example itself.
But at the same time, it does make it inconvenient to use with router.$push when it's a Route so not sure what should be done about that. Could possibly provide a helper function to transform one to the other, maybe.

@rchl I am not sure what are all the use cases for this function but for me so far I only use it primarily with $router.push() and $router.replace() which in this case both expect a RawLocation type and I get a type error. However, if there are other use cases that require this function to return a Route object the I guess something like this would work:
localeRoute('myroute') // returns a Route
localeRoute('myroute').toRawLocation() // return a RawLocation

Currently, I am creating my own types and importing them:
types/Location.ts:
// types taken from vue-router types https://github.com/vuejs/vue-router/blob/dev/types/router.d.ts#L188
// this is needed until this is fixed https://github.com/nuxt-community/i18n-module/issues/776#issuecomment-813201177
type Dictionary<T> = { [key: string]: T };

export interface Location {
  name?: string;
  path?: string;
  hash?: string;
  query?: Dictionary<string | (string | null)[] | null | undefined>;
  params?: Dictionary<string>;
  append?: boolean;
  replace?: boolean;
}

Changed my mind. Leaving localeRoute behavior unchanged and return type changed to reflect the reality (Route), and added localeLocation API that returns Location.


Update dubbo revision to 3.3.3-SNAPSHOT

What is the purpose of the change?
Checklist

[x] Make sure there is a GitHub_issue field for the change.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[x] Make sure gitHub actions can pass. Why the workflow is failing and how to fix it?

Please merge the relevant pr first.

Please merge the relevant pr first.

Which one?


display code samples when path has a parameter in it

I have a created a folder that is "v1@messages" which has a post.cs file in it for the csharp code sample, but when I put the get.cs file in there and try to build it produces the following error:
throw Error('Code sample for non-existing operation: "' + path + '",' + verb);
If I move it out of there into it's own folder it produces the same error. I have tried with this folder name:
"v1@messages@{messageId}"
and
"v1@messages@messageId"
What else should I try? Also, this should be updated in the readme for code samples.

Closing as stale. Feel free to reopen if this is still an issue.


Dropcol depends on OOB score

It seems that the dropcol importance depends on the oob_score which might not be available or accurate in many cases depending on the problem. Would it make sense to submit a PR that checks for the full model metric vs constrained model metric on a validation set and not OOB?

howdy! Yes, maybe we need to rename that oob_dropcol_importances and then make a general dropcol_importances that uses a validation set?

Sounds good, I will work on those 2 PRs in the upcoming few days. Also thinking of generalizing the metric to be a list or a single metric in case we'd like to monitor the change in multiple metrics at once? Not sure how much more overhead in terms of impl that would be, but might make sense.

Well, I think we can start simply and just pass in a single metric.


Enhanced Footer with Social Links

Description
The current README lacks an engaging and consistent footer section with social media links. The goal is to create a visually appealing footer at the end of the README that includes clickable icons for different social media platforms, encouraging better engagement and easy access to contact links.

@vishanurag ASSIGN ME THE TASK UNDER GSSOC AND HACKTOBERFEST

@vishanurag ASSIGN ME THE TASK UNDER GSSOC AND HACKTOBERFEST

@vishanurag


add daminisatya as en owner

This is to add 1.17 docs lead, @daminisatya, to the list of en owners as required by the role handbook.

/lgtm
/approved

/approve
Oops :)


consider renaming

you know, something that works better in variable names lol

Themely?
naw, 'cause this is for Scratch Blocks

Themey McThemeFace

I guess this name is fine. shrug


Detect Play event in SwiftUI

Hello,
I'm trying to detect the play event with SwiftUI
Here is my view :
struct PlayerVideoView: View {
    let videoId: String

    var youtubePlayer: YouTubePlayer {
        let player = YouTubePlayer(stringLiteral: "https://youtube.com/watch?v=\(videoId)")
        return player
    }

    var body: some View {
        YouTubePlayerView(youtubePlayer) { state in
            switch state {
            case .idle:
                ProgressView()
            case .ready:
                EmptyView()
            case .error:
                Text(verbatim: "YouTube player couldn't be loaded")
            }
        }
        .onAppear {
            youtubePlayer.statePublisher
                .sink { playbackState in
                    Logs.info(playbackState)
                }
        }
    }
}

The Logs.info(playbackState) never pass.
I've also tested :
.onChange(of: youtubePlayer.playbackState) { state in
            Logs.info(state)
        }

Did I miss something about this publisher or is this more related to my SwiftUI lifecycle ?
Thanks

Hi @Goule
Your example contains two issues:

The YouTubePlayer instance never gets retrained as you are using a computed property instead of a stored property
The closure which you are passing to the sink operation on the statePublisher never gets executed as the returned AnyCancellable instance is not retained and therefore automatically cancelled.

Thanks @SvenTiigi !
Look like I need to learn more SwiftUI and Combine :D
struct PlayerVideoView: View {
    private var cancellable: AnyCancellable?

    var youtubePlayer: YouTubePlayer

    init(videoId: String) {
        self.youtubePlayer = YouTubePlayer(stringLiteral: "https://youtube.com/watch?v=\(videoId)")

        self.cancellable = youtubePlayer
            .playbackStatePublisher
            .sink { playbackState in
                Logs.info(playbackState)
            }
    }

    var body: some View {
        YouTubePlayerView(youtubePlayer) { state in
            switch state {
            case .idle:
                ProgressView()
            case .ready:
                EmptyView()
            case .error:
                Text(verbatim: "YouTube player couldn't be loaded")
            }
        }
    }
}

👍  A tip I would recommend subscribing to the statePublisher in your SwiftUI View via the onReceive view modifier which automatically handle the subscription returned by the publisher.


Fixing security holes

https://tanzu.vmware.com/security/cve-2020-5405
when I upgraded the 2.1.7 version, I encountered the following error, dependency error,

The server starts normally. Only when the client starts to connect to the server, the server reports this error.
2.1.7 or 2.2.2

error.zip
Please check the log

You have mismatched versions of boot and cloud.

I have solved this problem with other versions


Implemented LQR solver with cross term matrix as of issue #9894

Implementation of issue #9894 with cross term matrix M i.e., x'Qx + u'Ru + 2x'Mu.
The necessary condition for the existence of a Riccati stabilizing solution will be added later.

Happy to help :)

The check of the necessary conditions is not yet implemented

Yes, and please create a separate PR for this.
On Mon, Oct 14, 2019 at 11:39 AM Precious, Ugo Abara <
<EMAIL_ADDRESS>wrote:

The check of the necessary conditions is not yet implemented
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ApolloAuto/apollo/pull/9909?email_source=notifications&email_token=ABEKLZKUVX5EUBQ4QKHDWQ3QOS4ELA5CNFSM4JAJJ672YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBF7FZI#issuecomment-541848293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEKLZMLOAVRIFLGGC6LINDQOS4ELANCNFSM4JAJJ67Q
.


toggle-files-button notice is misplaced when not on main branch

Description

How to replicate the issue
Go to https://github.com/refined-github/refined-github/tree/abort-delegates (this happen even with a full page load).
Extension version
22.7.20
Browser(s) used
Firefox

Love it. 😅
We should probably also hotfix it (hide it via banner + notice” selector) on 22.7.1 and 22.7.20

0 Fixed by https://github.com/refined-github/refined-github/pull/5980


Add install guide for nixos configuration with flakes

There is nix-profile install guide with flakes, but no configuration install guide with flakes.
Please add it.
Thanks

Done

Thanks! 👍


Unable to start application due to file number watchers limit

Is it an actual bug?
Yes.
Did you check if the bug has already been reported?
Yes.
Describe the bug
When I'm trying to start sleek it shows blank screen with System limit for number of file watchers reached error in logs:
$ flatpak run com.github.ransome1.sleek
[2 zypak-helper] Error retrieving supports property: org.freedesktop.DBus.Error.InvalidArgs: No such property 'supports'
[2 zypak-helper] Portal v4 is not available
Gtk-Message: 14:46:23.680: Failed to load module "appmenu-gtk-module"
User preferences located at: /home/user/.var/app/com.github.ransome1.sleek/config/sleek/user-preferences.json
libva error: /usr/lib/x86_64-linux-gnu/dri/intel-vaapi-driver/iHD_drv_video.so init failed
Success: Translation loaded for: en
Success: Window events setup
Success: Filewatcher is watching: /mnt/data/share/notes/todo/todo.txt
(node:2) UnhandledPromiseRejectionWarning: Error: ENOSPC: System limit for number of file watchers reached, watch '/mnt/data/notes/todo/todo.txt'
    at FSWatcher.<computed> (internal/fs/watchers.js:218:26)
    at Object.watch (fs.js:1525:34)
    at createFsWatchInstance (/app/sleek/resources/app.asar/node_modules/chokidar/lib/nodefs-handler.js:119:15)
    at setFsWatchListener (/app/sleek/resources/app.asar/node_modules/chokidar/lib/nodefs-handler.js:166:15)
    at NodeFsHandler._watchWithNodeFs (/app/sleek/resources/app.asar/node_modules/chokidar/lib/nodefs-handler.js:331:14)
    at NodeFsHandler._handleFile (/app/sleek/resources/app.asar/node_modules/chokidar/lib/nodefs-handler.js:395:23)
    at NodeFsHandler._addToNodeFs (/app/sleek/resources/app.asar/node_modules/chokidar/lib/nodefs-handler.js:629:21)
    at async /app/sleek/resources/app.asar/node_modules/chokidar/index.js:451:21
    at async Promise.all (index 0)
(Use `sleek --trace-warnings ...` to show where the warning was created)
(node:2) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:2) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Failed to open '/app/sleek/resources/app.asar/src/index.html': Not a directoryFailed to open '/app/sleek/resources/app.asar/src/index.html': Not a directoryFailed to open '/app/sleek/resources/app.asar/src/index.html': Not a directoryFailed to open '/app/sleek/resources/app.asar/src/index.html': Not a directoryFailed to open '/app/sleek/resources/app.asar/src/index.html': Not a directory[10 zypak-sandbox] Dropping 0x1243250 (3) because of connection closed
[10 zypak-sandbox] Host is gone, preparing to exit...
[10 zypak-sandbox] Quitting Zygote...

But it seems that I have enough  available watchers:
$ lsof | grep inotify | wc -l
58
$ sysctl fs.inotify
fs.inotify.max_queued_events = 16384
fs.inotify.max_user_instances = 1024
fs.inotify.max_user_watches = 8192

To Reproduce
Steps to reproduce the behavior:

Launch application.
See blank screen

Do you see any error entries in sleeks developer tools?
No.
Expected behavior
Application should display todo.txt.
Screenshots
No.
Desktop (please complete the following information):

OS: Ubuntu 20.04
Version of sleek: 1.1.1
Source: Flathub

@anlar : I improved the filewatcher setup and managed to get rid of this issue on my side. Can I ask you to give this pre release a try and let me know if this fixes anything on your side as well? https://github.com/ransome1/sleek/releases/tag/v1.1.2-rc.3

@ransome1 Thanks for the update.
I have updated to v1.1.2 from Flathub and right now I'm not able to reproduce that error.


ReleaseNotesGrabber Override HttpClient Global Injection

While it is nice that I can now override the CreateHttpClient in the ReleaseNotesGrabber, I have no particular way to do anything meaningful with that, as I cannot control when a ReleaseNotesGrabber is new'd up.  It would be nice if I could inject my own ReleaseNotesGrabber the way I can with a IAppCastDataDownloader.
My greater need for this is because I may desire to set my own User Agent.  My update files are in S3 buckets and there is currently a security restriction in our setup requiring User Agents to be set.

Probably related to #352

Hi @iamjplant,
Yeeeeahhh......the easiest way to do that right now is to create your own UIFactory, override CreateUpdateAvailableWindow, and then set the ReleaseNotesGrabber property on the window's data context (or similar, depending on which UI you are using):

    class MyUIFactory : UIFactory
    {
        public override IUpdateAvailable CreateUpdateAvailableWindow(SparkleUpdater sparkle, List<AppCastItem> updates, bool isUpdateAlreadyDownloaded = false)
        {
            var window = base.CreateUpdateAvailableWindow(sparkle, updates, isUpdateAlreadyDownloaded);
            if (window is UpdateAvailableWindow uaw && uaw.DataContext is UpdateAvailableWindowViewModel uawvm)
            {
                //uawvm.ReleaseNotesGrabber = new ReleaseNotesGrabber(...)
            }
            return window;
        }
    }

But that's a pain, I would agree. 🤪 It would be easier if there was a direct way to access/override this object from the UIFactory itself, yes.

You can test this out in 2.1.4-preview20221106001, rolling out now via CI/CD. Set UIFactory.ReleaseNotesGrabberOverride. Please let me know if that helps!

Yes, that looks like it will fit the bill.  Very cool.

When do you plan to have 2.1.4 deployed?  Thanks!

Released in v2.2.0


Feature Request: MLB Teams in each lineup and order range

Would you be able to add 2 constraints to the BASEBALL optimizer? I would like to be able to control how many teams are in my lineups (preferably just the hitters, but if it's easier to do all players that's fine). I would also like to control how far apart my stacks are from each other in the lineup. Ideally, I am hoping to be able to have the optimizer construct lineups with roughly 3 teams for the hitters and the hitters from those teams to be within x number of spots in the order from each other.

Hi, I just released new version of library with team stacking constraint, it probably what you want. If you want that all hitters will be selected from same team you can use it with another constraint set_positions_for_same_team. If you want to restrict selecting hitters only from 3 team you can remove other hitters from optimization or set max_exposure to 0 for those players.

Thank you for the quick response. I was thinking something a little different. I would want it so that the optimizer would be the one to handle limiting to x number of teams in each lineup. The goal would be for multi-entry, so I would want to cover a variety of team combinations in a single run. This wouldn't be possible with setting max exposure to 0 on the other players or removing them.  Also having the optimizer to use players within x lineup spots in the order from each other would help with grouping players together that might correlate together (player x gets a hit, then player y 2 spots later hits him in).

For now you optimizer hasn't constraint for restricting total number of different teams in lineup, for now you can try to use team stacking constraint that allows you to specify how many players from same team will appear in result lineup. For example: optimizer.set_team_stacking([3, 3, 4])  DK MLB has 10 players in lineup and you specify all 10 players in stacking, so you will get lineups with only 3 different teams.
According to second part of issue, I didn't understand idea of what you want, could you give me example how it should work? Probably you can achieve this with existed constraints.

Cool, thank you. I will try the team stacking constraint. The other option is less important for me, and is unique only to baseball. In baseball, it is common to make a stack of batters that are near each other in the lineup. It is not standard in DK's player pool to include the order, but could be added to the csv as a new column. the constraint i was thinking of would allow you to set a range for what spot in the batter order they hit in. So if I was to set it at 9, the optimizer could use any combination of hitters, but if I was to set it to 4, the optimizer would limit batters from the same team in a lineup to 1-4, 2-5, 8-2, etc. Again, less critical to me, but might be nice to have if it is not a significant amount of work. Thanks again.  Really like this optimizer.

Dima, thank you for all the hard work. I hope you're making money on DFS!
I noticed the solver completely bogs down after using the set_team_stacking feature. It seems to only be able to create one or two lineups. Is there anything we can do about this? Or is it just simply too much to solve for?

@dachel12 I think it's not hard, but I'm not sure what api I should implement for this, because dfs sites doesn't provide this information, so it's not clear where user can get this ordering.
@brente33 It strange because it works for me. I just set team stacks to [3, 3, 4] (hardest to optimize) for 600 players csv and it's take 2.5x more time then without stacking for generating 10 lineups. Are you using team stacking in combination with other constraint? Maybe some combinations can cause this, I'm not sure.

I deleted all constraints and it still only solved for 1 lineup. I updated to version 2.4 today, I’m not sure what is happening.
Sent from my iPhone
On Apr 22, 2019, at 12:37 PM, Dima Kudosh <notifications<EMAIL_ADDRESS>wrote:
@dachel12https://github.com/dachel12 I think it's not hard, but I'm not sure what api I should implement for this, because dfs sites doesn't provide this information, so it's not clear where user can get this ordering.
@brente33https://github.com/brente33 It strange because it works for me. I just set team stacks to [3, 3, 4] (hardest to optimize) for 600 players csv and it's take 2.5x more time then without stacking for generating 10 lineups. Are you using team stacking in combination with other constraint? Maybe some combinations can cause this, I'm not sure.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/DimaKudosh/pydfs-lineup-optimizer/issues/55#issuecomment-485469729, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKX3JKFLGVU2CY7JAR4ZVU3PRXSV5ANCNFSM4HHKYOUA.

Could you send csv file with players and exact parameters that you pass to team stacking method to my email? I'll try to investigate why it works slowly.

This is the stripped down code I'm running. When I use n=1, n=4, or n=5 it works fine. If I use n=2, n=3, or n>5 it stops after 1 lineup.
from pydfs_lineup_optimizer import Site, Sport, get_optimizer
optimizer = get_optimizer(Site.FANDUEL, Sport.BASEBALL)
optimizer.load_players_from_csv("fanduel-MLB.csv")
for lineup in optimizer.optimize(n=2):
print(lineup)
I had no problems at all running your optimizer until after I updated to version 2.4. I pulled it from GIT first. The I deleted everything and tried it new  but that didn't help either.

From: Dima Kudosh<EMAIL_ADDRESS>Sent: Monday, April 22, 2019 1:41 PM
To: DimaKudosh/pydfs-lineup-optimizer
Cc: brente33; Mention
Subject: Re: [DimaKudosh/pydfs-lineup-optimizer] Feature Request: MLB Teams in each lineup and order range (#55)
Could you send csv file with players and exact parameters that you pass to team stacking method to my email? I'll try to investigate why it works slowly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/DimaKudosh/pydfs-lineup-optimizer/issues/55#issuecomment-485489315, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKX3JKDVF6V2X47JAGVPWL3PRX2FPANCNFSM4HHKYOUA.

Do you have hangs with simple optimization? On this release I changed some solver settings, maybe they can cause this, try init your optimizer like this:
from pydfs_lineup_optimizer.solvers.pulp_solver import PuLPSolver

class CustomPuLPSolver(PuLPSolver):
    LP_SOLVER = None

optimizer = get_optimizer(Site.FANDUEL, Sport.BASEBALL, solver=CustomPuLPSolver)

You are the man, that worked. Really appreciate you creating the optimizer and taking the time to help out!

From: Dima Kudosh<EMAIL_ADDRESS>Sent: Monday, April 22, 2019 2:44 PM
To: DimaKudosh/pydfs-lineup-optimizer
Cc: brente33; Mention
Subject: Re: [DimaKudosh/pydfs-lineup-optimizer] Feature Request: MLB Teams in each lineup and order range (#55)
Do you have hangs with simple optimization? On this release I changed some solver settings, maybe they can cause this, try init your optimizer like this:
from pydfs_lineup_optimizer.solvers.pulp_solver import PuLPSolver
class CustomPuLPSolver(PuLPSolver):
LP_SOLVER = None
optimizer = get_optimizer(Site.FANDUEL, Sport.BASEBALL, solver=CustomPuLPSolver)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/DimaKudosh/pydfs-lineup-optimizer/issues/55#issuecomment-485509517, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKX3JKFYP7MGZY6GXP7PNT3PRYBRZANCNFSM4HHKYOUA.

@DimaKudosh totally agree with you. That would make it difficult for people to use that feature. Would it make sense to write the code as if there would be an additional column in the players file, but comment it out so that it is not active by default? Otherwise I am 100% okay with leaving optimizer as is. Thank you very much for discussing these features with me.

@brente33  You are welcome! Strange that my changes slow-down your program. Can you say what version of python you are using and how many cores on your pc. Also what is output of import pulp; pulp.VERSION and import multiprocessing; multiprocessing.cpu_count()? I'll try to investigate reason of this issue, because I tried run new version on 2 different pc and it works faster than previous.
@dachel12 I can check for additional column when parsing csv as I do for max_exposure, but it's not clear, in your example you wrote that if we set spot spacing to 4 optimizer should limit players to 1-4, why it should limit to 1-4, it's 2 players between 1 and 4 not 4.

Sorry, the way I worded it may have been confusing. You're thinking about it correctly. If I was to limit the range to 4 then it could be (for example) any number of players between 1 and 4 in the order. Just that the range between where they are in the order should be whatever is set.

I taught myself python just to be able to use your solver so try not laugh too hard at my coding skills....
I'm using JetBrains PyCharm Edu 2018.2
This is what I get when I run pulp:
[cid:f4b52411-d653-4eb1-a746-7f6eb1925ef2]
I don't play a lot of money on DFS but if I ever hit anything big I'll pay you something for all of this!

From: Dima Kudosh<EMAIL_ADDRESS>Sent: Monday, April 22, 2019 3:23 PM
To: DimaKudosh/pydfs-lineup-optimizer
Cc: brente33; Mention
Subject: Re: [DimaKudosh/pydfs-lineup-optimizer] Feature Request: MLB Teams in each lineup and order range (#55)
@brente33https://github.com/brente33 You are welcome! Strange that my changes slow-down your program. Can you say what version of python you are using and how many cores on your pc. Also what is output of import pulp; pulp.VERSION and import multiprocessing; multiprocessing.cpu_count()? I'll try to investigate reason of this issue, because I tried run new version on 2 different pc and it works faster than previous.
@dachel12https://github.com/dachel12 I can check for additional column when parsing csv as I do for max_exposure, but it's not clear, in your example you wrote that if we set spot spacing to 4 optimizer should limit players to 1-4, why it should limit to 1-4, it's 2 players between 1 and 4 not 4.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/DimaKudosh/pydfs-lineup-optimizer/issues/55#issuecomment-485521683, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKX3JKFVNADQWQSCNB4Q6ITPRYGDZANCNFSM4HHKYOUA.

@DimaKudosh  Thank you for making this optimizer!  After upgrading to version 2.4 I experienced the same issue that brente33 described which was fixed by your suggestion in this thread.  I had a little luck from increasing the number of threads for LP_Solver, but changing to None got it running smoothly again.
I'm using Python 3.7 and the same IDE as brente.  CPU count is 4 and pulp version is 1.6.9.
Thanks again.

@kpm1988 I have same pc setup. What OS are you using?

@DimaKudosh I'm using Windows 10.

@kpm1988 I'm using ubuntu, probably it's problem with solver binaries for windows, I can't change it because it comes from PuLP, I'll remove additional solver settings in new minor release.

Thanks @DimaKudosh !  I have Ubuntu installed on another machine, would you recommend using that for now, or would it work the same with the LP_SOLVER = None code you recommended above?

@kpm1988 I think it will be same, I just released version 2.4.1 with remove default solver, so you can just update library instead of overriding solver class.

@DimaKudosh  Thanks for the help with setting up stacking. However, last night I noticed I had some teams with batters against my pitcher. Here is how I had the position constraint set up. My DK salaries page was the export from Draftkings with only the avg points column changed to update with the projections I was using.
optimizer.restrict_positions_for_opposing_team(['P'], ['1B', '2B', '3B', 'SS', 'C', 'OF', 'P'])
Any idea why some batters vs pitchers made it through?

I think it because DK split pitcher position into 2 separate positions SP and RP, so imported players actually has SP or RP positions instead of P, try to change your code to:
optimizer.restrict_positions_for_opposing_team(['SP', 'RP'], ['1B', '2B', '3B', 'SS', 'C', 'OF', 'SP', 'RP'])

Thank you, I believe that fixed it. I had 5/20 lineups initially with the P vs opp batter. After updating to that, there were 0.

optimizer.restrict_positions_for_opposing_team(['SP', 'RP'], ['1B', '2B', '3B', 'SS', 'C', 'OF', 'SP', 'RP'])
Is this actually correct?
Having no 'SP' vs '1B' from opposite team is understandable.
But having no 'SP' vs 'SP' from opposite team is not beneficial.
Isn't it should be like this:
optimizer.restrict_positions_for_opposing_team(['SP', 'RP'], ['1B', '2B', '3B', 'SS', 'C', 'OF'])

No SP vs SP allows both pitchers to get the win bonus. But yes, not needed. I added that to mine and I think it carried over to Dima's reply to me

Hi, how do you load the pitchers and hitters from CSV file? I was able to get an export for DraftKings MLB pitchers and batters as 2 separate files from rotogrinders.

Bryan,
The optimizer wants the player list in the Draftkings format, so go to
draftkings and export the player list for the contest you are entering. If
you are trying to load in the projections for the rotogrinders exports,
open up the draftkings player list and update the average points column
with the projections from rotogrinders and resave the draftkings csv.
Import that as your players into the optimizer.
I always save the draftkings export into the same location with the same
name so I don't have to update the code to tell it to open a different
file. Hope this helps.
Matt
On Sat, Jul 27, 2019 at 12:24 PM Bryan Choi<EMAIL_ADDRESS>wrote:

Hi, how do you load the pitchers and hitters from CSV file? I was able to
get an export for DraftKings MLB pitchers and batters as 2 separate files
from rotogrinders.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DimaKudosh/pydfs-lineup-optimizer/issues/55?email_source=notifications&email_token=AEXOIW72QX2TXQAOJDPLY4LQBSAEPA5CNFSM4HHKYOUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD26PNWY#issuecomment-515700443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEXOIW3Q4Z4S5WR4D6OPPQ3QBSAEPANCNFSM4HHKYOUA
.


error on install rabbitmq - undefined method `undent' for #String:0x00007f80a3f32778

install a depreciated version of rabbitmq

What happened (include command output)
rabbitmq: stable 2.8.7
http://www.rabbitmq.com
/usr/local/Cellar/rabbitmq/2.8.7 (165 files, 3.3MB) *
Built from source on 2019-03-31 at 22:29:08
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/rabbitmq.rb
==> Dependencies
Required: erlang ✔
Error: undefined method undent' for #<String:0x00007f80a3f32778> Please report this bug: https://docs.brew.sh/Troubleshooting /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/rabbitmq.rb:49:in caveats'
/usr/local/Homebrew/Library/Homebrew/caveats.rb:17:in caveats' /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/forwardable.rb:202:in empty?'
/usr/local/Homebrew/Library/Homebrew/cmd/info.rb:214:in info_formula' /usr/local/Homebrew/Library/Homebrew/cmd/info.rb:88:in block in print_info'
/usr/local/Homebrew/Library/Homebrew/cmd/info.rb:77:in each' /usr/local/Homebrew/Library/Homebrew/cmd/info.rb:77:in each_with_index'
/usr/local/Homebrew/Library/Homebrew/cmd/info.rb:77:in print_info' /usr/local/Homebrew/Library/Homebrew/cmd/info.rb:64:in info'
/usr/local/Homebrew/Library/Homebrew/brew.rb:102:in `'
[/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula]$ brew doctor rabbitmq
No check available by the name: rabbitmq
[/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula]$ less rabbitmq.rb
[/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula]$ less -N rabbitmq.rb
[/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula]$ brew test rabbitmq.rb
Error: rabbitmq defines no test

What you expected to happen
rabbitmq 2.8.7 install

Step-by-step reproduction instructions (by running brew install commands)

brew create http://www.rabbitmq.com/releases/rabbitmq-server/v2.8.7/rabbitmq-server-generic-unix-2.8.7.tar.gz

brew install rabbitmq

https://gist.github.com/bfca09266d86e6f445bf565ef3465a45

We don't support installing old versions in this way.
From the issue template for future issues:

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you do not read fill out the issue template repeatedly you will be blocked from ever submitting issues to Homebrew again.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter maxxi-charge for common errors and appropiate suggestions to keep this adapter up to date.
This check is based on the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.maxxi-charge
   -
ERRORS:

[X] :heavy_exclamation_mark: [E046] devDependency must not require a specific version. Use "~1.2.3" or "^1.2.3" syntax. Please update "@iobroker/testing:4.1.3"
[X] :heavy_exclamation_mark: [E507] missing size attributes [xs,sm,md,lg,xl] for select at admin/jsonConfig.json/items/apiMode
[X] :heavy_exclamation_mark: [E507] missing size attributes [xs,sm,md,lg,xl] for staticText at admin/jsonConfig.json/items/_localApiHint
[X] :heavy_exclamation_mark: [E507] missing size attributes [xs,sm,md,lg,xl] for text at admin/jsonConfig.json/items/localapiport
[X] :heavy_exclamation_mark: [E507] missing size attributes [xs,sm,md,lg,xl] for text at admin/jsonConfig.json/items/maxxiCcuName
[X] :heavy_exclamation_mark: [E606] Current adapter version 1.2.19 not found in README.md
[X] :heavy_exclamation_mark: [E901] .gitignore not found

WARNINGS:

[ ] :eyes: [W401] Cannot find "maxxi-charge" in latest repository

:thumbsup: No suggestions found
Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
or
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Sun, 08 Dec 2024 11:14:55 GMT based on commit fe3bddf2f7850cef0268c9095eea143bc68bfd72
ioBroker.repochecker 3.2.2

@ioBroker-Bot recreate


Fix issue #478

Proposed changes
Return a values field in /entities containing the last value
Types of changes
What types of changes does your code introduce to the project: Put
an x in the boxes that apply

[ x ] Bugfix (non-breaking change which fixes an issue)
[ x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after
creating the PR. If you're unsure about any of them, don't hesitate to
ask. We're here to help! This is simply a reminder of what we are going
to look for before merging your code.

[ x ] I have read the CONTRIBUTING doc
[ x ] I have signed the CLA
[ x ] I have added tests that prove my fix is effective or that my
feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in
downstream modules

Further comments
Fix for Issue

Gentle Reminder!!

this pr needs a rebase, but due to other changes operated in the code (see the pr from your colleague: https://github.com/orchestracities/ngsi-timeseries-api/pull/486) there will be conflicts to be solved.

this pr needs a rebase, but due to other changes operated in the code (see the pr from your colleague: #486) there will be conflicts to be manually solved.

@chicco785 , I have updated files. Also, please let me know if i missed something. Thank you :)

this pr needs a rebase, but due to other changes operated in the code (see the pr from your colleague: #486) there will be conflicts to be manually solved.

@chicco785 , I have updated files. Also, please let me know if i missed something. Thank you :)

as mentioned the union query is required, i fixed the code accordingly.

given that @pooja1pathak is working on setting up as "a datasource" endpoint, i will return to the basic version that returns only the last index.


CFE does not issue "native" syntax errors.

CFE is not issuing errors for syntactically invalid uses of the native syntax.
https://github.com/dart-lang/sdk/blob/master/tests/language_2/syntax_test.dart
class DOMWindow {}

class Window extends DOMWindow
native "*Window" //# 28: syntax error
{}

class Console
native "=(typeof console == 'undefined' ? {} : console)" //# 29: syntax error
{}

class NativeClass
native "FooBar" //# 30: syntax error
{}

abstract class Fisk {}

class BoolImplementation implements Fisk
native "Boolean" //# 31: syntax error
{}

class _JSON
native 'JSON' //# 32: syntax error
{}

class ListFactory<E> implements List<E>
native "Array" //# 33: syntax error
{
  noSuchMethod(_) => null; // Allow unimplemented methods
}

Previous issue on this test: https://github.com/dart-lang/sdk/issues/12158
cc @leafpetersen

Related to https://github.com/dart-lang/sdk/issues/28791


Indigo Device Name in list of UniFi devices

If I select to add a UniFi device and sometime later go back and edit, then in the list of UniFi devices (using the names from the Controller), the top name in the list is the Indigo device name and the actula associated device is listed but not selected. Ideally, I think it should indicate the name of the UniFi device selected for the Indigo device? Doesn't seem to cause a problem though, albeit, it is slightly confusing as I wouldn't expect the Indigo device name to be listed.

Yeah, that's there because if the device is currently off-line, if you try to edit the device the pop up will be blank and the device won't be in the available devices list.  I'll see if I can change that to the last known name for that device.

Fixed in 0.0.11, I think.


webpub manifest JSON spine and resource lists of links should be mutually-exclusive / non-overlaping?

https://readium2.feedbooks.net/Ym9va3MvY2hpbGRyZW5zLWxpdGVyYXR1cmUuZXB1Yg==/manifest.json
If I am not mistaken, cover.xhtml, nav.xhtml and s04.xhtml should only appear in the spine collection, not resources:
...
,
"spine": [
  {
   "href": "https://aldiko-stats.feedbooks.net/Ym9va3MvY2hpbGRyZW5zLWxpdGVyYXR1cmUuZXB1Yg==/EPUB/cover.xhtml",
   "type": "application/xhtml+xml"
  },
  {
   "href": "https://aldiko-stats.feedbooks.net/Ym9va3MvY2hpbGRyZW5zLWxpdGVyYXR1cmUuZXB1Yg==/EPUB/nav.xhtml",
   "type": "application/xhtml+xml",
   "rel": [
    "contents"
   ],
   "properties": {
    "contains": [
     "js"
    ]
   }
  },
  {
   "href": "https://aldiko-stats.feedbooks.net/Ym9va3MvY2hpbGRyZW5zLWxpdGVyYXR1cmUuZXB1Yg==/EPUB/s04.xhtml",
   "type": "application/xhtml+xml"
  }
 ],

 "resources": [
...
{
   "href": "https://aldiko-stats.feedbooks.net/Ym9va3MvY2hpbGRyZW5zLWxpdGVyYXR1cmUuZXB1Yg==/EPUB/cover.xhtml",
   "type": "application/xhtml+xml"
  },
  {
   "href": "https://aldiko-stats.feedbooks.net/Ym9va3MvY2hpbGRyZW5zLWxpdGVyYXR1cmUuZXB1Yg==/EPUB/s04.xhtml",
   "type": "application/xhtml+xml"
  },
  {
   "href": "https://aldiko-stats.feedbooks.net/Ym9va3MvY2hpbGRyZW5zLWxpdGVyYXR1cmUuZXB1Yg==/EPUB/nav.xhtml",
   "type": "application/xhtml+xml",
   "rel": [
    "contents"
   ],
   "properties": {
    "contains": [
     "js"
    ]
   }
  }
...
 ],
...

That's correct, we shouldn't have any overlap between spine and resources cc @banux.
I don't know which version of the streamer is running on our servers though.


missing windows component on sf cluster

i developed a .net service with c++ native components, those are using gulp32.dll, opengl32.dll and msvfw32.dll. After deployment on SF with virtual machine image "WindowsServerSemiAnnual Datacenter-Cor-1709" the service doesn't run because of missing opengll dlls and msftw32.dll. My question is how to proceed in this case and howto add the missing components?

This is an old thread and I imagine the context is no longer relevant, but for others hitting similar issues the thing to make sure is that the dlls are bundled in the application package and remain co-located at the service activation path in the data root. If you depend on the dlls being present at the global machine scope just make sure this doesn't conflict against other versions of the dlls also made available through PATH. You can run as the service executing user from the service directory to determine whether you are hitting a versioning conflict which often manifests as a dll not found error:

where msftw32.dll

Feel free to create another thread if you are still hitting issues.


Hotfix/create delete column template

Can we get this merged? Also I will be happy to update the nuget package as well if @jeffreyabecker doesn't want to manage this anymore.

I'm happy to hand off upkeep of this project. Could you provide a couple
links on how?
On Tue, May 26, 2020, 9:49 AM Virendra Singh<EMAIL_ADDRESS>wrote:

Can we get this merged? Also I will be happy to update the nuget package
as well if @jeffreyabecker https://github.com/jeffreyabecker doesn't
want to manage this anymore.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jeffreyabecker/FluentMigrator.NHibernate/pull/6#issuecomment-634073528,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABAZH7W53DITZWKIY2KRFDRTPJIPANCNFSM4KCCVOWQ
.

Well giving admin privileges on git repo is easy, possibly also transferring it to another user is fine. The main problem is the Nuget.org account which in your case must be an individual account. The only way is to make your account an organization account, and then add me in the organization.
https://docs.microsoft.com/en-us/nuget/nuget-org/organizations-on-nuget-org.
That way both of us will be able to manage the package and I will be able to add more people to the organization provided I have the rights.


WIP: store: stream watching with an outdated index returns a list of events

A watch with stream=true and an outdated waitIndex returns all events from the outdated index till current index and closes the watch, with the first commit. The watch is kept open for subsequent events with the second commit.
I've only tested this lightly. Looking for opinions before I proceed.
/cc @xiang90

@bprashanth This is exactly what I would expect.

Let me know when you are ready for a final review.

@xiang90 Simple ad-hoc testing with the client half (https://github.com/coreos/go-etcd/pull/215) worked out OK.
In trying to convince myself of the order of notifications sent down a stream watch I noticed writes are serialized on a stop the world lock. I'm a little concerned about the scan taking a lock that could potentially block notifyWatchers while the caller holds the stop the world lock. Wdyt?

Wdyt?

It is hard to tell until I benchmark it. But if there are O(100) watchers, I am not that concern about lock contention. We are not doing anything blocking with lock.

Yeah that's what I figured, the priority inversion should only happen when we're adding a new watcher which shouldn't be too often. Just bringing it up since I'm new to the code base, and who knows what's going to bite.

@xiang90 this isn't going to make our 1.0. Just checking that this is still useful to you guys/something you want before I re-up a pr (it will be useful to us post 1.0 depending on what the new etcd api provides).

@bprashanth We are going to have a design doc for the v3 watching api and its implementation. We would very like to see our opinion on that!

@xiang90 alright, thanks for the update

Probably do not need this anymore. etcd3 already supports this.


Only allow https as URL scheme

This closes: https://github.com/cds-snc/url-shortener/issues/148
Added validation to only allow https as a scheme.

Good work! Thanks for changing that in the unit tests as well.

Thx 👍


Convert RequestTimeoutException to gRPC Status using GrpcExceptionHandlerFunction

The logic converting RequestTimeoutException to gRPC Status is hard-coded so users can't override the message or customize the Status.
https://github.com/line/armeria/blob/d30db0c88d178468c3bb03a4e1d920b4c7a897b9/grpc/src/main/java/com/linecorp/armeria/server/grpc/FramedGrpcService.java#L323-L325
Before applying the default logic, we may first apply a user-defined GrpcExceptionHandlerFunction.

May I look into the issue? 😄


"SIGSEGV" error

Guzzle version(s) affected: 6.3.0
PHP version: 7.2.34 (hint: php --version)
cURL version: 7.64.0 (hint: php -i | grep cURL)
Description
When requesting other service interfaces, if the other party puts back the status code "999", the current process will report an error of "SIGSEGV", causing the process to restart
child 60 exited on signal 11 (SIGSEGV) after 67.959713 seconds from start
child 101 started
How to reproduce

Only when the status code is 999, other status codes are normal, and only online, but not local. How to solve this problem
linux core:Linux <IP_ADDRESS> 3.10.0-1160.el7.x86_64
Possible Solution

Additional context

Segfaults are a bug in PHP itself, and need to be reported to the PHP team. I'm sorry that there's nothing we can do here in userland.

Thank you


After changing to cargo chef the program stops right after execution

I recently changed my Dockerfile to cargo chef and after it runs successfully then my actix-web program stops immediatly with exit code 0.  What do I have to change in my Dockerfile?
# Build stage
FROM lukemathwalker/cargo-chef:latest-rust-1 AS chef

RUN apt-get update && \
  apt-get install -y pkg-config make g++ libssl-dev cmake libmariadb-dev-compat && \
  rustup target add x86_64-unknown-linux-gnu

WORKDIR /var/www/app

FROM chef AS planner
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

FROM chef AS builder
COPY --from=planner /var/www/app/recipe.json recipe.json
RUN cargo chef cook --release --recipe-path recipe.json

COPY . .

RUN cargo build --release

# Prod stage, removing the Rust toolchain
FROM debian:buster-slim AS runtime
WORKDIR /var/www/app
COPY --from=builder /var/www/app/target/release/isumis /usr/local/bin

CMD ["/usr/local/bin/isumis"]

I also started getting this recently. I was able to create a minimal example for you here.
To test I ran the build:
cd cargo-chef-demo
docker build -t test .

Then shell into the container to run the bin:
docker run -it test ash
./chef-builds

It should print "Hello,world!", but just exits.
Let me know if I can do anything to help

@lanklaas it doesn't print "Hello,world!".
cargo-chef-demo$ sudo docker run -it test ash
/app $ ./chef-builds
/app $

@LukeMathWalker I added it.

Sorry, think I worded it wrong. The example I gave is an example of the broken one. So it not printing Hello,world! is the problem. I pushed another dockerfile to the repo to show you the working example (without using cargo-chef). If you pull again you can build it like this:
docker build -f working.Dockerfile -t wtest .

Then run it:
docker run -it wtest ash
./chef-builds

This one prints Hello,world!
An interesting thing I noticed was that the binary file sizes differs just a little:
Broken Size:
/app $ ls -l
total 4372
-rwxr-xr-x    1 root     root       4475024 Aug 18 06:11 chef-builds

Working size
/app $ ls -l
total 4384
-rwxr-xr-x    1 root     root       4481912 Aug 18 06:56 chef-builds

And you think this should solve my problem?

I run into this as well. After a whole day of tinkering with things, I was able to narrow down the issue to cargo-chef. Removing it from the Dockerfile revealed the real issue, I had mistakenly excluded some necessary files through .dockerignore entries (including the main.rs). And yet, with cargo chef cook in the recipe, the Dockerfile compiled and completed without an issue! Fixing the ignore entries solved everything and hopefully, this case helps to narrow down the issue.

@lanklaas: pretty sure the issue is due to a swapped statement in your Dockerfile, see https://github.com/lanklaas/cargo-chef-demo/pull/1
I'm not at my laptop to confirm it works, but I'm fairly confident.

@LukeMathWalker thanks, Yes I forgot I swapped those because I got version issues before the swap. I added 2 branches on the repo correct-copy-order which is the same fix as your PR, and workspace in which I changed the repo to a cargo workspace to fix the error in correct-copy-order, just in case it helps someone else here.
I tracked down the original issue that caused me to do the swap, but it is not related to this issue. I will investigate and report it if I have some more concrete info

Perfect, I'll close the issue then 👍🏻


Add AWS credentials / region to boto session instantiation when fetch…

Description
Specifies AWS credentials and region when fetching the project zip via S3.
GitHub Issues
https://github.com/Miserlou/Zappa/issues/984

Coverage decreased (-0.2%) to 73.087% when pulling 78c6c8d65b8925bef525652e4a7708e9f06084e7 on bahoo:use_aws_credentials_in_creating_boto_session into b7c16d3ac62aa496dfce8ba533dc64cd20376269 on Miserlou:master.

I'm :-1:  on this since Boto implements this mechanism already.  If it's not working, we should figure out what's breaking the environment and/or file a bug with Boto.

Per discussions in #984 this is an edge case I'm running into and coding around. I'm going to close it.


Error after add firebase_messaging plugin

I'm using google_sign_in in my app. It works well
I wanted add notification to my app, and so i followed the Android integration like explained here:
https://github.com/flutter/plugins/tree/master/packages/firebase_messaging
If i try to build the app i receive this error:

What went wrong:

Execution failed for task ':app:transformClassesWithDexForDebug'.
> com.android.build.api.transform.TransformException: com.android.ide.common.process.ProcessException: java.util.concurrent.ExecutionException: com.android.dex.DexException: Multiple dex files define Lcom/google/firebase/iid/zzb;

How did you resolve this issue ?


fix: set min width on percentage uncontrolled side

Fixes: https://github.com/zakodium-oss/react-science/issues/496

Codecov Report
Patch coverage has no change and project coverage change: -0.04 :warning:

Comparison is base (e4ddf56) 48.88% compared to head (85388d6) 48.84%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #505      +/-   ##
==========================================
- Coverage   48.88%   48.84%   -0.04%
==========================================
  Files         138      138
  Lines        7510     7516       +6
  Branches      184      184
==========================================
  Hits         3671     3671
- Misses       3839     3845       +6

Impacted Files
Coverage Δ

src/components/split-pane/SplitPane.tsx
31.80% <0.00%> (-0.64%)
:arrow_down:

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


Abount UNPACK_ALIGNMENT

https://github.com/BabylonJS/Babylon.js/blob/e883bb339451cf7c9aaaaf7fb1618a3a4dff00c6/packages/dev/core/src/Engines/Extensions/engine.rawTexture.ts#L321
if (texture.width % 4 !== 0) {
this._gl.pixelStorei(this._gl.UNPACK_ALIGNMENT, 1);
}
webgl is a statemachine
maybe we need to revert to 4 after changing this parameter？

Hello! As it's stated on our template, please validate issues on the forum first https://forum.babylonjs.com/ before opening on Github.