Defaults updates for Biopower single owner cause nan IRR, negative equity

In the release candidate, the default PPA price of $0.16/kWh for Biopower, single owner causes a sculpted debt higher than the capital cost, which causes a nan IRR. Lower the PPA price to something that causes more reasonable financial numbers, similar to what we did for PV.
Testing with the target IRR mode, $0.12/kWh generates a 10.5% IRR.

Additional issues with sculpted debt:

Generic system, single owner has a negative debt percent to start. The default PPA price might need to be raised here.
Many financial parameters for fuel cell, single owner have been set to zero. Need to update to those from PV-battery, then assess impact. I suspect the PPA price might need to be raised here as well once that's done.

Fuel cell single owner did not get the update for the other FOM energy charges - need to change this to a single period with $0.045/kWh retail rate, and turn off demand charges.

Now I'm just using this issue as a catch all for remaining defaults issues I find. Wind-LCOE calculator needs to have it's costs updated to align with the ATB and single owner. Capital cost should be $1303/kW (to match single owner), O&M should be $42/kW-yr.

With the financial defaults lined up to PV-battery, the default debt fraction is negative. IRR target mode suggests a price of $0.75/kWh. That decimal is in the right place, so I'm inclined to change the default to debt fraction instead of DSCR for this configuration.

Closed via https://github.com/NREL/SAM/pull/823


Sass::SyntaxError in SabisuRails::Explorer#index

Running into this issue trying to set up sabisu, somewhere in the @import 'furrato', the site works (although very ugly) if I comment out that line
Undefined mixin 'display-flex'.
(in /home/nate/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/bundler/gems/sabisu-rails-af7499e87912/app/assets/stylesheets/sabisu_rails/application.css.scss:32)

Yeah this is due to compass renamed many methods, I'm adressing this to update even to the last version of Furatto.

+1

Have you tried updating the sabisu gem?

i found the same problems.
I just change my Gemfile to:
gem 'sabisu_rails', github: "IcaliaLabs/sabisu-rails"
gem 'furatto', github: "IcaliaLabs/furatto-rails"
gem 'compass-rails'

Did that solve the problem?


Sample custom resource generation

This change intends to solve the following problems:

Rapidly changing CustomResource structure's which are evolving with both additional features and changes in the Istio codebase.
Multiple services - The adapter is expected to be able to handle API management for multiple services on 3scale. This requires a unique identifier in the naming of these CR's
Lessen complexity on the end user. Users currently need to digest a lot of information from the doc's to correctly format and patch the sample CustomResource
Self-documentation - Changes in the CR structure result in a change to documentation. This feature will go somewhat to help making the docs static in this regards.

Solution is to expose a package that can be used to generate sample custom resources based on some input. These generated templates are tested against the testate run in the integration tests providing a (close as possible) e2e verification method.

Codecov Report

Merging #60 into master will increase coverage by 0.64%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
+ Coverage   86.69%   87.33%   +0.64%
==========================================
  Files           3        5       +2
  Lines         436      616     +180
==========================================
+ Hits          378      538     +160
- Misses         37       48      +11
- Partials       21       30       +9

Impacted Files
Coverage Δ

pkg/templating/internal.go
100% <100%> (ø)

pkg/templating/manifest_generator.go
88.23% <88.23%> (ø)

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 3129cd7...085c9e6. Read the comment docs.


Converted form buttons to go thru miqObserveRequest JS method.

Changed buttons on advanced search to call MiqAjaxButton JS method so transactions queue up and are sent upto server in an order they were clicked on to address an issue seen in automated tests.
https://bugzilla.redhat.com/show_bug.cgi?id=1388695
@dclarizio @himdel @mshriver please review/test.
Note: that this issue was only seen while running automated tests.

@h-kataria Hey.. just one thing.. miqAjaxButton does not go through the observe queue. But it does (mostly) keep the ordering, so it should work :).
Although I'm not sure users won't percieve this as a performance problem, given it always waits at least 0.7 seconds before doing anything.
Either way, this should fix the bug, and if you want the users to not to have to wait, replacing miqAjaxButton with miqObserveRequest would do the trick I think..

I have run several iterations of cfme.tests.infrastructure.test_advanced_search_providers.py against a patched build from 10/28 with no activity hangs.

@himdel i tried to use same method call that is being used for other form buttons, and that seems to be working  with automated tests. i am willing ot change it to call miqObserveRequest instead, let me know, i can have @mshriver run tests again with a new change.

@h-kataria I think I'll leave it up to you :) you're right that it's the right method, so.. does it feel sluggish to you or is it OK?
As I mentioned there is a slight possibility of this not preserving the ordering, but it can only be done with 3 or more actions, timed so that...

action 1 gets queued, waits 700ms
action 2 gets queued, waits 700ms
action 3 gets queued, waits 700ms
action 1 has finished waiting, gets sent to the server - now there is an active request
action 2 has finished waiting, there is still an active request, waits another 700ms
action 1 has received a response - no active requests now
action 3 has finished waiting, gets sent to the server, breaking the ordering

But maybe that's something we can fix for miqAjaxButton in general if it becomes a problem..

Euwe backport details:
$ git log -1
commit d5fbf5819599994f49a6ec5c5794b1131b9fc920
Author: Dan Clarizio<EMAIL_ADDRESS>Date:   Tue Nov 8 14:18:03 2016 -0800

    Merge pull request #12373 from h-kataria/automation_tests_adv_search_spinner_fix

    Converted form buttons to go thru miqObserveRequest JS method.
    (cherry picked from commit b91ec103422bc02e3775de0df08561e0bb2df887)

    https://bugzilla.redhat.com/show_bug.cgi?id=1396243


"syscall.Getuid()" returns "0" when run with "sudo"

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.10 linux/amd64
Does this issue reproduce with the latest release?
Yes (go version go1.10.2 linux/amd64)
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build507988449=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Pull docker image
docker pull golang:alpine

Run container
docker run -it --name test-1 --rm golang:alpine sh

Add sudo to docker container
apk update && apk add sudo

Activate wheel-group for sudo
sed -i -e "/%wheel ALL=(ALL) NOPASSWD: ALL/s/^# //" /etc/sudoers \
  && grep "^%wheel" /etc/sudoers

Add user
adduser -D user1

Add user to wheel-group
sed -i -e "/wheel/s/$/,user1/" /etc/group \
  && grep wheel /etc/group

Add test script
vi /tmp/test.go

package main

import (
	"fmt"
	"syscall"
	"os"
)

func main() {
	fmt.Printf("Syscall Real UID: %d\n", syscall.Getuid())
	fmt.Printf("Syscall Effective UID: %d\n", syscall.Geteuid())
}

Switch user
su user1

Run script
go run /tmp/test.go
sudo go run /tmp/test.go

What did you expect to see?
go run /tmp/test.go
# => Syscall Real UID: 1000
# => Syscall Effective UID: 1000

sudo go run /tmp/test.go
# => Syscall Real UID: 1000
# => Syscall Effective UID: 0

What did you see instead?
go run /tmp/test.go
# => Syscall Real UID: 1000
# => Syscall Effective UID: 1000

sudo go run /tmp/test.go
# => Syscall Real UID: 0
# => Syscall Effective UID: 0

Found while using docker build: https://github.com/docker/cli/issues/1109

Isn't this intended? Same thing happens in C and Bash. sudo should set environment variables if you want to get the user who is actually running it...
Demo (fish shell): test.fish
id -u
echo $SUDO_UID

 ><>  ~ > ./test.fish
1000

 ><>  ~ > sudo ./test.fish
0
1000

@deanveloper You're right, I got confused what EUID really means.


fix disable join reorder in bucket shuffle join test

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


modem_cellular: periodic_chat_script is only running once

Describe the bug

MODEM_CHAT_SCRIPT_CMDS_DEFINE(quectel_bg95_periodic_chat_script_cmds,
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGREG?", ok_match));

chat_script above only runs once and then completes.
[00:00:18.402,000] <dbg> modem_chat: modem_chat_script_next: quectel_bg95_periodic_chat_script: step: 0
[00:00:18.402,000] <dbg> modem_chat: modem_chat_script_next: sending: AT+CGREG?
[00:00:18.428,000] <dbg> modem_chat: modem_chat_log_received_command: +CGREG:  2 5 "FFFE" "F9715" 7
[00:00:18.428,000] <dbg> modem_cellular: modem_cellular_log_event: event registered
[00:00:18.428,000] <dbg> modem_cellular: modem_cellular_log_state_changed: switch from await registered to carrier on
[00:00:18.429,000] <dbg> modem_chat: modem_chat_log_received_command: OK
[00:00:18.429,000] <dbg> modem_chat: modem_chat_script_stop: quectel_bg95_periodic_chat_script: complete
[00:00:18.429,000] <dbg> modem_cellular: modem_cellular_log_event: event script success
[00:00:18.456,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Receiving PPP frame
[00:00:18.456,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Received PPP frame (len 29)
[00:00:18.456,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Receiving PPP frame
[00:00:18.456,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Received PPP frame (len 12)
[00:00:18.485,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Receiving PPP frame
[00:00:18.485,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Received PPP frame (len 13)
[00:00:18.510,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Receiving PPP frame
[00:00:18.510,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Received PPP frame (len 12)
[00:00:18.541,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Receiving PPP frame
[00:00:18.541,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Received PPP frame (len 12)
[00:00:18.541,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Receiving PPP frame
[00:00:18.541,000] <dbg> modem_ppp: modem_ppp_process_received_byte: Received PPP frame (len 9)

Expected behavior

It used to run fine, which resulted in the script being called periodically. I could not figure out what change impacted modem_cellular, as I am pretty sure I reverted most changes to test the behaviour again.
Environment (please complete the following information):
Zephyr: 3.7.0
Running a custom board with an RC76xx and I am doing the following:
#if DT_HAS_COMPAT_STATUS_OKAY(quectel_bg95)
MODEM_CHAT_SCRIPT_CMDS_DEFINE(quectel_bg95_init_chat_script_cmds,
				  MODEM_CHAT_SCRIPT_CMD_RESP("ATE0", ok_match),
				  MODEM_CHAT_SCRIPT_CMD_RESP("AT+IFC=2,2", ok_match),
				  MODEM_CHAT_SCRIPT_CMD_RESP("AT+COPS=2", ok_match),
				  MODEM_CHAT_SCRIPT_CMD_RESP("AT+COPS=0", ok_match),
				  MODEM_CHAT_SCRIPT_CMD_RESP("AT+KSRAT=5", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP_NONE("AT+CMUX=0,0,5,127", 300));

MODEM_CHAT_SCRIPT_DEFINE(quectel_bg95_init_chat_script, quectel_bg95_init_chat_script_cmds,
			 abort_matches, modem_cellular_chat_callback_handler, 10);

MODEM_CHAT_SCRIPT_CMDS_DEFINE(quectel_bg95_dial_chat_script_cmds,
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGDCONT=1,\"IP\","
							 "\""CONFIG_MODEM_CELLULAR_APN"\"",
							 ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP_NONE("ATD*99***1#", 0));

MODEM_CHAT_SCRIPT_DEFINE(quectel_bg95_dial_chat_script, quectel_bg95_dial_chat_script_cmds,
			 dial_abort_matches, modem_cellular_chat_callback_handler, 10);

MODEM_CHAT_SCRIPT_CMDS_DEFINE(quectel_bg95_periodic_chat_script_cmds,
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGREG?", ok_match));

MODEM_CHAT_SCRIPT_DEFINE(quectel_bg95_periodic_chat_script,
			 quectel_bg95_periodic_chat_script_cmds, abort_matches,
			 modem_cellular_chat_callback_handler, 4);

Any insight is greatly appreciated!

I really have no idea, just looking at the source code, it should work just fine. only thing I can speculate is that the timeout is missed somehow? try increasing the size of uint8_t event_buf[8]; to 32 or something, and keep an eye on modem_cellular_delegate_event() see if the timeout is actually added successfully

Thanks for the comment! I will look into it next week.
I think I was also a bit unclear in my original post. I have only shown the changes I made in modem_cellular, but my actual code is most likely the reason the periodic_script is not running. I reverted my code to a previous commit and modem_cellular was working just fine.
I will keep an eye on modem_cellular_delegate_event() while also investigating the changes in my code as well. Thanks!

Managed to narrow it down a bit.
I have a ZBus 'Module' called modem and an l4_cb. At the end of it I have a zbus_chan_pub to publish the network_status. If I were to remove it the script will run periodically again.

Sounds like it could be the system work queue being blocked, could you check which context calls the l4_cb?

That's likely, no blocking work may be performed from there, zbus_chan_pub can be blocking if there are zbus callback listeners registered which do something blocking, try following the call to zbus_chan_pub() and see if anything blocking is called, like taking a sem or mutex

The zbus_chan_pub indeed takes a sem with chan_lock.
But if this is the problem I will move the function out of the cb. Thanks for your help!


Add OpenSSL

Current build process skip OpenSSL. OpenSSL must be added as dependency and turn on XMRig CMake SSL Option

/cib


Issue https://github.com/h6ah4i/android-advancedrecyclerview/issues/1…

…42 fix

Reverted change in this method, my fault - sorry I have written fix for old version for library (which didn't had notifyItemRangeChanged signature with Object) - then I updated source and forgot to fix that.

Thank you for fixing the notifyItemRangeChanged() method. It looks good to me ; )
I am going to merge it manually, because I want to merge it into the develop branch.

Done. It will be included in v0.8.4. Thanks!


Stacked chart y-axis scaling

When stacking the charts the Y-axis is not recalculated and this causes some weird chart layout where a bar heigh for huge values still can be the same as a bar height for small values.

It seems like working for now


No bash control in shell; unable to use ctrl+c to get out of processes

Type: Bug
bash: no job control in this shell
VS Code version: Code 1.81.1 (6c3e3dba23e8fadc360aed75ce363ba185c49794, 2023-08-09T22:20:33.924Z)
OS version: Darwin x64 18.7.0
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3720QM CPU @ 2.60GHz (8 x 2600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
3, 2, 3

Memory (System)
8.00GB (0.06GB free)

Process Argv
-psn_0_13155467 --crash-reporter-id 49b716f6-b6e1-4ffc-8bd1-85243bc84f14

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

dart-code
Dar
3.70.0

flutter
Dar
3.70.0

awesome-flutter-snippets
Nas
4.0.1

LiveServer
rit
5.7.9

svelte-vscode
sve
107.10.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30803844
282f8724:30602487
89544117:30613380
showlangstatbar:30737416
03d35959:30757346
57b77579:30736110
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
gsofa:30804715
pythonnosmt12:30797651
pythonidxptcf:30805731
pythonnoceb:30805159
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30812749

Sorry for my short ticket description.
Here's more information.
I have a MacBook Pro running Mojave OS and am running Visual Studio Code 1.8.1.
When I open up my terminal, it says "bash: no job control in this shell". See the attached screenshot.
When I start a process such as "npm run dev", I am unable to stop it using ctrl+c. Instead, it just shows ^C and keeps the process running as shown in the attached screenshot.
To exit the process, I have to click the trash can next to the terminal which crashes VS Code and causes me to have to restart it.
I have searched on Google for a fix and tried adding "set -m" to my .bash_profile file and to my .bashrc file but neither of those solved the issue.
My standard Mac terminal works fine and ctr+c works there so it seems like an issue with VS Code. VS Code was working fine in earlier releases. I tried downgrading but I am unable to. When I download an earlier version and try to install it, it says I'm still on 1.8.1.
Appreciate any help you can provide.

try updating vs code

I clicked 'Check for updates' and it says there are currently no updates available. See screenshot.

Also, I'm on 1.81.1 and per the VS Code download site here, it looks like it's the latest version.

@deepak1556 any thoughts?

Merging to https://github.com/microsoft/vscode/issues/184713


fix W.tc Children W.sdt dosn't contain W.tc

fix W.tc Children W.sdt dosn't contain W.tc , Only W.sdtContent contain W.tc .

i can say W.tc Children W.sdt don't contain W.tc directly. W.tc is contained by W.sdtContent.

you can see this picture.

so find W.tcPr should add a stdContent.

i hope it can be merged.

@haozhouxu Can you add a test for this?

@twsouthwick should i put a test in here?or in project code?

@twsouthwick
before modify code , this output is like this:

after modify code ,this output is like this:

this is test file:
TemplateDocument.docx
Data.txt
is this test ok?

@haozhouxu There is a test project that should have a test added to it. Having a unit test with the specific elements that are expected as input and output (rather than a full file) is preferred. Unit tests can be targeted to the specific behavior while the test files end up being more integration tests as they are more complicated and end up using many components to run.


Using backslashes in paths is a D-move. Please don't do that anymore.

@PG85 ping 🏓

Already fixed for v8.1-r1, to be released on the Discord asap. Thanks tho!


Add only clause to mix_test_watch

{:mix_test_watch, "~> 1.0", only: [:test], runtime: false}
Not necessary to include in production.

Thanks for that, you're welcome to send a small PR through to remove it :)

Done

Will get included in the next release, thank you ❤


Separate logging query time and loading time

Taken from the discussion here: https://groups.google.com/forum/#!topic/elixir-ecto/Px92gzBNlkg
Me:

Recently I was messing around with a query that loaded around 10,000 records. It was a simple Repo.all(User) query. In the logs it said query: 600ms. This seemed odd to me that the query took so long to run. I tried the query in psql and it ran in 30ms.

It seems like the log is counting both the query time and the time it took to load the records. I think it would be helpful (and more accurate) to make the query time show how long the actual query took to run and then a separate load time. That way it's simpler to see if it's a slow running query or something else.

Jose:

Yes. That's because a good chunk of the processing happens in the adapter and we can't make a distinction between query time and load time. We hope to improve this in the future, James is already working on reading the socket data on the client, which should make things faster and easier to measure. Please open up an issue so we don't forget to retackle this

Added in https://github.com/elixir-lang/ecto/commit/ac8c9013fb12134dd7f0795a1a05080d532e7690

Right on! Thanks @fishcakez


Fix assert() so that it doesn't warn

Rewrite this macro so it doesn't cause a spurious
warning about braces.
Signed-off-by: Dan Cross<EMAIL_ADDRESS>

Dan, rebase it.

Done.
On Thu, Apr 20, 2017 at 4:31 AM, Álvaro Jurado<EMAIL_ADDRESS>wrote:

Dan, rebase it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Harvey-OS/harvey/pull/530#issuecomment-295633214, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABhLAP-jateCt7EZz4UYktoWJbhMqYAOks5rxxf8gaJpZM4NCPAJ
.


RX-28 is not detected

Servo model: RX-28
Setup: DynamixelShield on ArduinoUNO. Debug serial using another ArduinoUNO as UART-Serial bridge.
Situation: I recently purchased the DynamixelShield for a project to control a couple of RX-28s that I had from another project. This decision came after I was unable to even detect the motors using a self-made RS-485 to USB converter, though this same setup worked on that earlier project. I assumed that the converter is broken as the cables were pretty worn-out and bought the shield. Then when I tried to scan one of the motors using scan_dynamixel example code, nothing showed up. The power supply is set to 14V and the even the motor blinks once when powered on, which is normal. But after that, nothing works nor there are any LED indications of a particular error. Just as a control experiment, I connected an AX-12 that I borrowed from another robot, which got clearly detected with the same scan_dynamixel example code. It’s pretty evident that the issue is with the RX-28 motors. I’m not sure how to debug this to get them working as they are not even discoverable to maybe factory-reset them or something. Any help from the from the community will be appreciated, thank you.

Hi @Jayadev22
The DYNAMIXEL2Arduino library deactivates some discontinued DYNAMIXEL series in order to save the memory space in some boards.
In order to enable the RX-28, please set the ENABLE_ACTUTATOR_RX to 1 in the setup below.
https://github.com/ROBOTIS-GIT/Dynamixel2Arduino/blob/master/src/utility/config.h#L7
Thanks

Hi @ROBOTIS-Will
Thank you for replying. I tried what you suggested, but unfortunately it didn't make any difference. The RX-28 is still undetectable. Any other possibilities that you can think of.? I would readily try whatever is possible to get them running.
Thank you again.

If you have changed the buad rate of the RX-28, the scan_dynamixel example may not detect it as the example only detects as below.
const int32_t buad[MAX_BAUD] = {57600, 115200, 1000000, 2000000, 3000000};

If you have USB2DYNAMIXEL or U2D2 interface, I'd recommend recovering the firmware with DYNAMIXEL Wizard 2.0
Thank you.


Smoke detection not able to see when no "else" branch

When running boogie -smoke on the following file:
function {:identity} Lit<T>(x: T) : T;

axiom (forall<T> x: T :: {:identity} { Lit(x): T } Lit(x): T == x);

procedure problemMethod() returns (r#0: int);

implementation problemMethod() returns (r#0: int)
{
	if (Lit(true))
	{
	}
}

it finds me a smoke issue with the following trace:
found unreachable code:
implementation problemMethod() returns (r#0: int)
{

  0:
    goto anon0;

  anon0:
    goto anon2_Else;

  anon2_Else:
    assume {:partition} !Lit(true);
    assert false;
    return;
}

It is an incorrect trace because the branch anon2_Else was never written in the code, so it should not even try to add assert false to it.
This is causing the bug https://github.com/dafny-lang/dafny/issues/1500

The smoke trace looks reasonable to me.  The else branch of the if statement would have been generated in the unstructured Boogie that is desugared from the structured code.  You could try to see the unstructured code by using /printUnstructured or /printInstrumented (I forget exactly which one does the job).

The smoke trace looks reasonable for the unstructured code indeed. Thanks for sharing the option which is /printInstrumented.
Since users are not dealing with the unstructured code, it seems even more reasonable to either not even try assert(false) in the branches that did not appear in the original code, or discard them before announcing results, right?
I don't know how hard it would be.


fix: smoke test for webhook e2e

Fixes #N/A
Description
Fix e2e syntax for webhooks.
How was this change tested?
make test
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

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 78.692%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
5894

Relevant Lines:
7490

💛  - Coveralls


[PkgEval] OpenStreetMap may have a testing issue on Julia 0.3 (2015-02-13)

PackageEvaluator.jl is a script that runs nightly. It attempts to load all Julia packages and run their tests (if available) on both the stable version of Julia (0.3) and the nightly build of the unstable version (0.4). The results of this script are used to generate a package listing enhanced with testing results.
On Julia 0.3

On 2015-02-10 the testing status was Tests pass.
On 2015-02-13 the testing status changed to Tests fail, but package loads.

Tests pass. means that PackageEvaluator found the tests for your package, executed them, and they all passed.
Tests fail, but package loads. means that PackageEvaluator found the tests for your package, executed them, and they didn't pass. However, trying to load your package with using worked.
This issue was filed because your testing status became worse. No additional issues will be filed if your package remains in this state, and no issue will be filed if it improves. If you'd like to opt-out of these status-change messages, reply to this message saying you'd like to and @IainNZ will add an exception. If you'd like to discuss PackageEvaluator.jl please file an issue at the repository. For example, your package may be untestable on the test machine due to a dependency - an exception can be added.
Test log:
>>> 'Pkg.add("OpenStreetMap")' log
INFO: Cloning cache of OpenStreetMap from git://github.com/tedsteiner/OpenStreetMap.jl.git
INFO: Installing Cairo v0.2.24
INFO: Installing Color v0.4.0
INFO: Installing DataStructures v0.3.5
INFO: Installing FixedPointNumbers v0.0.6
INFO: Installing Graphics v0.1.0
INFO: Installing Graphs v0.5.0
INFO: Installing IniFile v0.2.4
INFO: Installing LightXML v0.1.10
INFO: Installing OpenStreetMap v0.7.0
INFO: Installing Tk v0.2.17
INFO: Installing Winston v0.11.7
INFO: Building Cairo
INFO: Building LightXML
INFO: Building Tk
INFO: Package database updated
INFO: METADATA is out-of-date a you may not have the latest version of OpenStreetMap
INFO: Use `Pkg.update()` to get the latest versions of your packages

>>> 'using OpenStreetMap' log
Julia Version 0.3.5
Commit a05f87b (2015-01-08 22:33 UTC)
Platform Info:
  System: Linux (x86_64-unknown-linux-gnu)
  CPU: Intel(R) Xeon(R) CPU E5-2650 0 @ 2.00GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Sandybridge)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3

>>> test log
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  322k  100  322k    0     0   348k      0 --:--:-- --:--:-- --:--:--  348k
testing read_data ...
elapsed time: 12.105333587 seconds (273812616 bytes allocated, 1.14% gc time)
testing simcity ...
elapsed time: 0.151198825 seconds (4118936 bytes allocated)
testing crop_map ...
elapsed time: 0.793757345 seconds (32035296 bytes allocated)
testing classes ...
elapsed time: 0.125040678 seconds (6366196 bytes allocated)
testing coordinates ...
elapsed time: 0.040347547 seconds (1124892 bytes allocated)
testing routes ...
elapsed time: 1.231563937 seconds (42102808 bytes allocated, 2.68% gc time)
testing plots ...

ERROR: `getgc` has no method matching getgc(::Canvas)
 in anonymous at /home/idunning/pkgtest/.julia/v0.3/Winston/src/tk.jl:23
 in display at /home/idunning/pkgtest/.julia/v0.3/Winston/src/tk.jl:35
 in display at /home/idunning/pkgtest/.julia/v0.3/Winston/src/Winston.jl:2735
 in display at multimedia.jl:149
 in plotMap at /home/idunning/pkgtest/.julia/v0.3/OpenStreetMap/src/plot.jl:173
 in include at ./boot.jl:245
 in include_from_node1 at ./loading.jl:128
 in anonymous at no file:56
 in include at ./boot.jl:245
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:285
 in _start at ./client.jl:354
while loading /home/idunning/pkgtest/.julia/v0.3/OpenStreetMap/test/plots.jl, in expression starting on line 21
while loading /home/idunning/pkgtest/.julia/v0.3/OpenStreetMap/test/runtests.jl, in expression starting on line 10

INFO: Testing OpenStreetMap
============================[ ERROR: OpenStreetMap ]============================

failed process: Process(`/home/idunning/julia03/usr/bin/julia /home/idunning/pkgtest/.julia/v0.3/OpenStreetMap/test/runtests.jl`, ProcessExited(1)) [1]

================================================================================
INFO: No packages to install, update or remove
ERROR: OpenStreetMap had test errors
 in error at error.jl:21
 in test at pkg/entry.jl:718
 in anonymous at pkg/dir.jl:28
 in cd at ./file.jl:20
 in cd at pkg/dir.jl:28
 in test at pkg.jl:67
 in process_options at ./client.jl:213
 in _start at ./client.jl:354

>>> end of log

I'm guessing someone will get to this on the Winston end. Lots of (great!) changes on 0.4 recently. If not, I'll try to take a look in a couple days.

Yeah, this is a weird one.
The test version (above) of Winston matches my local version (0.11.7) but the Tk version is one patch newer. But the supposedly missing method, getgc(::Canvas) is in file Tk/src/tkwidget.jl, and that file hasn't been changed since your commit back in October.
Definitely appears to be outside of this package.

Ah, looks like Winston had the same error in their tests, too. https://github.com/nolta/Winston.jl/issues/204

Cool, that sounds right.

Closing this since our tests are passing on Julia 0.3 now.


Register on dockerhub

You can register the docker file on dockerhub https://cloud.docker.com ~
So that we can deploy it without clone.

it's being planned


[New Feature]: Complete DISP-S1 Forward Processing Workflow

Checked for duplicates
Yes - I've already checked
Alternatives considered
Yes - and alternatives don't suffice
Related problems
No response
Describe the feature request
Forward processing, just like all other processing modes, needs to be retooled to align w the historical database json and other new findings in DISP-S1 triggering business logic.

Currently the code marks all CSLS granules stored in cslc_catalog as downloaded (aka triggered) However, we do not want to do this with k granules. We should make distinction between the granules that we are processing vs the supporting k granules and should only mark the processing granules. We now have information in a hashmap to support this so just need to utilize that information.


Issue : Provisioner log files created as nobody

One of our service was not able to read the provisioner log file, we noticed it was created as nobody.
Please don't make configuration or log files as nobody.
-rw-r----- 1 nobody nobody 575897 Apr  6 16:20 choria-provisioner.log

I am not a fan at all of these logs being world readable, but made it 644 which is the actual problem you encountered, also fixed the nobody though.


pm: procfs add pm prapare fail stats

Summary
Add more information in pm procfs, record when inside pm_prepall who cb->prepare ret failed.
Impact
Cost more time when Enabled pm_procfs and have prepare failed.
When procfs read and pmconfig can show more information.
Testing
CI test & qemu-v8a

same fail  with https://github.com/apache/nuttx/pull/12361


Turning off autotoggle doesn't work

When I start up Atom, even if I turn off autotoggle, it still runs at startup. What's weirder is that if I toggle it once, it still runs, but then if i toggle it a second time, it turns off. This makes me think that, whilst it is changing the boolean, it doesn't check that boolean's value at startup. Thanks!

@linky00 Is this issue still present? The best I can tell this shouldn't be happening since it does properly check the state/config on load.

fixed in 0.3.0 :tada:


Udev devices should be mounted through DeviceSpec

Describe the bug
Udev discovered devices are currently mounted through Mount, which does not allow for fine grained control of access permissions. We should switch to DeviceSpec and expose the desired permission control through Configuration CRD.

/// Mount specifies a host volume to mount into a container.
--
  | /// where device library or tools are installed on host and   container
  | #[derive(Clone,   PartialEq, ::prost::Message)]
  | pub struct Mount {
  | /// Path of the mount within the   container.
  | #[prost(string, tag = "1")]
  | pub container_path: std::string::String,
  | /// Path of the mount on the   host.
  | #[prost(string, tag = "2")]
  | pub host_path: std::string::String,
  | /// If set, the mount is   read-only.
  | #[prost(bool, tag = "3")]
  | pub read_only: bool,
  | }
  | /// DeviceSpec specifies a host device to mount into a   container.
  | #[derive(Clone,   PartialEq, ::prost::Message)]
  | pub struct DeviceSpec {
  | /// Path of the device within the   container.
  | #[prost(string, tag = "1")]
  | pub container_path: std::string::String,
  | /// Path of the device on the   host.
  | #[prost(string, tag = "2")]
  | pub host_path: std::string::String,
  | /// Cgroups permissions of the   device, candidates are one or more of
  | /// * r - allows container to   read from the specified device.
  | /// * w - allows container to   write to the specified device.
  | /// * m - allows container to   create device files that do not yet exist.
  | #[prost(string, tag = "3")]
  | pub permissions: std::string::String,
  | }

After some testing after a thread on the slack channel, using Mounts instead of DeviceSpecs also prevents the said devices to be added in the containers cgroups devices.allow list, thus denying access to the devices to non privileged containers.


data augmentation

whether to use data augmentation? and whether will improve performance when used? Have done ablation study

Yes. You could refer to this.

thiinks for your reply, does data augmentaion is used by default?


Msds1<IP_ADDRESS>4

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @harshitav-docusign,
I am approving this PR, however if you have any issues while uploading this connector to Connector certification portal (ISV studio). Kindly reduce number of characters of docusign demo info/description within 500 and upload.


⚠ Bofrost.be has degraded performance

In 3dda5c1, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1127 ms

Resolved: Bofrost.be performance has improved in f20ba1b after 6 minutes.


vcpkg is trying to reinstall git.

I do not need duplicate installs. :)
I installed git with the Community 2017 (VSC hereafter) installer (these details are from that install)
I also removed the version installed by VSC and installed the version from git.org.
In all cases I tried, git was on my path.
This was done on 02/21/2018 with a fresh clone of the vcpkg repository from that day.
Details:
D:\tmp\vcpkg>path
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0;C:\Program Files\Git\cmd;C:\Users\redact\AppData\Local\Microsoft\WindowsApps;c:\util;C:\Program Files\CMake\bin
D:\tmp\vcpkg>vcpkg install cpr
The following packages will be built and installed:
cpr[core]:x86-windows

curl[core]:x86-windows
libssh2[core]:x86-windows
nghttp2[core]:x86-windows
openssl[core]:x86-windows
zlib[core]:x86-windows
Additional packages (*) will be modified to complete this operation.
Starting package 1/6: zlib:x86-windows
Building package zlib[core]:x86-windows...
A suitable version of git was not found (required v2.16.2). Downloading portable git v2.16.2...

D:\tmp\vcpkg>which git
C:\Program Files\Git\cmd\git.exe
D:\tmp\vcpkg>git --version
git version 2.16.1.windows.4

vcpkg is always trying to use the latest git and cmake.
(But indeed I don't think it's necessary)

Thanks for posting this issue!
In this case, it was caused by an insufficient version:

A suitable version of git was not found (required v2.16.2)
git version 2.16.1.windows.4

According to the git website, version 2.16.2 was released on 2018-02-15 [1].
[1] https://git-scm.com/downloads

I guess I should have been more clear and paid more attention.  I looked at that version number at least 50 times and didn't notice the difference.  I apologize for my lack of attention.
What motivated me to report is that a very recently downloaded git-scm.com (from within the last 8 days or so) and the one installed by the latest Microsoft VS Community 2017 installer were not sufficient...
It seems unnecessarily strict, especially given the much slower pace of updates to the compiler and associated tools.  At least let me override the strictness.

Your concerns are perfectly valid and we did some work to improve the situation.
Latest master (after 7f5efbe14419412d8c24df836a0b3b5192661aa3) consolidates the tool versions in a single file. Previously, the versions were hard-coded in a 2-3 places which was bad on its own accord and it also made the required vs auto-downloaded version a bit unclear, especially when different versions of vcpkg.exe were involved.
With latest master, the required version and auto-downloaded version can be modified in an intuitive/obvious way in the xml. As a bonus, you can now override the version of a tool by modifying the requiredVersion:
    <tool name="git">
        <requiredVersion>2.16.2</requiredVersion>
        <exeRelativePath>MinGit-2.16.2-32-bit\cmd\git.exe</exeRelativePath>
        <url>https://github.com/git-for-windows/git/releases/download/v2.16.2.windows.1/MinGit-2.16.2-32-bit.zip</url>
        <sha256>322c727e482aa97522c64a5ac68bdda3780111e8670bcfb532beac8e11ece5da</sha256>
        <archiveRelativePath>MinGit-2.16.2-32-bit.zip</archiveRelativePath>
    </tool>

As for the update cadence, I am mostly the one pushing for aggressive updating, but I agree that the tools are pretty stable so sometimes it can be unnecessarily strict (I will reconsider!).

Thanks folks. ;)
Now, if I can just get the ports quirks worked out.... :)

Any way of stopping vcpkg from downloading Git, I already have it installed, I'm also behind a proxy, so it's download is failing anyways. Is there an option anywhere to bypass this?


[DOC UPDATE] Index.html Chinese Translation

[DOC UPDATE] Index.html Chinese Translation
Description
Translate Index.html to Chinese version which referred to READ.me of gh-pages and the timely instruction from @dzamo (James Turton).
Documentation
This is a translation PR, reference to drill/gh-pages/zh/index.html
Testing
This is a translation PR, no test here.

I see in that in the external links you remove the scheme "https:", probably by accidentally reverting to old code. I recently added "https:" to these URLs because otherwise Polyglot thinks it must mangle them for multilingual, and of course it must not. It is possible to use a special character to tell Polyglot to leave these URLs alone but instead I used an explict "https:" since that is a practice recommended in general these days. This could be what has messed with your page layout so could you put "https:" back into every URL starting with "//" and see if the layout problems are fixed?

Yes, I found the problem you mentioned, after I changed those thing to newest version you just submitted, the video switch framework bug I mentioned in slack back to normal. I will close this PR, and open a new one.

@kingswanwho YES, Don't need to open new pull request. Because Git is a very powerful tool, the condition is that we have to master these skills, but most developers will mess it up at the beginning.
Well, I'm going to address some suggestions (for zh-CN) tomorrow.

@kingswanwho just push a last commit with the fixes to kingswanwho:git-pages-zh-index.  This PR will automatically incorporate it.


Unnecessary Imports

Discussion 🗣

There are some files present in the codebase that have unnecessary imports.
These could be removed as they are not used.
Detailed Description

Particularly, I saw three such files (might have missed some).
They are:
1.4-ContractEditor.stories.js
2.3-ConcertoForm.stories.js
3.FormattingToolbar/index.js

What are the imports that are unnecessary? Could you like to the code?

What are the imports that are unnecessary? Could you like to the code?

@irmerk
These were the lines where imports are not being used.
FormattingToolbar/index.js
Line 4:import { Transforms } from 'slate';
4-ContractEditor.stories.js
Line 4:: import { Provider, connect } from 'react-redux';  (connect)
**Line 15 **:
import { text, select, boolean, object } from '@storybook/addon-knobs'; (text, object)
Line 16:
import { storiesOf } from '@storybook/react'
Line 18:
import { Editor, Node, Transforms } from 'slate'; (Node)
3-ConcertoForm.stories.js
Line 1:import React, { useState } from 'react'; (useState)

I think removing the unused imports works. However, this is pretty low priority, especially for the ...stories.js files because they are not published to npm.

I think removing the unused imports works. However, this is pretty low priority, especially for the ...stories.js files because they are not published to npm.

Well, I had no idea about the priority of the packages :sweat_smile:. I think we can close the issue then as it hardly affects the work


combine -o and --remote-wait

Currently, this is not possible, something like
nvr -o --remote-wait ~/code/somefile

gives
usage: /usr/local/bin/nvr [arguments]
nvr: error: argument -o: expected at least one argument

vim -o --remote-wait <file> doesn't work as well. (Although it acts a bit different: it throws no error and ignores the -o).
It's documented that -o must take at least one argument.

I don't say it's impossible, but basically you want -o to take optional arguments and when no arguments are given act as a hint for --remote-wait to use :split instead of :edit.
So, I'd rather make it consistent with Vim and ignore the -o in that case.
But I guess that doesn't solve what you're trying to do, since --remote-expr is executed after --remote-wait on purpose. Should we introduce --remote-split-wait and friends?

I agree, the general option sounds better.

Ehm, no idea why I was thinking of --remote-expr-before.. I meant the counterpart to -c. I called it -cc.

nvr -cc split --remote-wait <file> should do what you want now.


It's possible manage Long Running Worker using nats?

I have to run a process that takes at least 20 minutes
I want to understand if nats is suitable for managing this idea:

On the browser the user presses a button,
Node Js Express sends a message to nats
The worker listening on nats starts the process
Every 5 seconds, the worker updates the job progress status
On the browser, the user sees the progess bar

I found this article, which explains how to implement the idea using RabbitMQ
https://itnext.io/handling-long-running-api-requests-in-nodejs-403bd566d47
But I love NATS

Control-plane management is the origins of nats. This is a perfect application for nats. Take a look at this https://nats.io/blog/webinar-nats-for-modern-messaging-and-microservices/


Allow function-callback as filter for links

It should be also possible to use a filter-function instead of a *, to for instance breadth search only "next-archive"-links in a atom archive.

Fixed with ed53ade


[BUG] oiiotool jpeg output as Output-sRGB has dark halos around highlights

Describe the bug
A simple conversion of an EXR from aces_cg to output sRGB when saved as JPG has dark halos around highlights. saving as PNG or any other format looks correct. Tried all sorts of filters and settings and I'm unable to fix it.

To Reproduce
Steps to reproduce the behavior:
convert the attached EXR using this command:
oiiotool -i "attached_exr" --colorconvert "ACES - ACEScg" "Output - sRGB" --compression jpeg:99 -o "srgb-output.jpg"
oiiotool -i "attached_exr" --colorconvert "ACES - ACEScg" "Output - sRGB" --compression jpeg:99 -o "srgb-output.png"
oiio-jpg_comp_main_v0001.0001.exr.zip
Expected behavior
both files should look exactly the same (minus compression artefacts). the jpg however has clear dark halo around the bright shape in the middle.
Platform information:

OIIO branch/version: OIIO 2.4.0dev built for C++14/199711 sse2
OS: windows

Filters available: box, triangle, gaussian, sharp-gaussian, catmull-rom, blackman-harris, sinc, lanczos3, radial-lanczos3, nuke-lanczos6, mitchell, bspline, disk, cubic, keys, simon, rifman
Dependent libraries: FFMpeg 4.4 (Lavf58.76.100), libheif 1.12.0, jpeg-turbo 2.0.6/jp62, null 1.0, OpenEXR 2.5.0, libpng 1.6.37, LIBTIFF Version 4.3.0
OIIO 2.4.0dev built for C++14/199711 sse2
Running on 12 cores 63.7GB sse2,sse3,ssse3,sse41,sse42,avx,avx2,avx512f,avx512dq,avx512cd,avx512bw,avx512vl,fma,f16c,popcnt,rdrand
EDIT: tested with the latest and the problem is still there:
OpenColorIO 2.1.2
Filters available: box, triangle, gaussian, sharp-gaussian, catmull-rom, blackman-harris, sinc, lanczos3, radial-lanczos3, nuke-lanczos6, mitchell, bspline, disk, cubic, keys, simon, rifman
Dependent libraries: OpenEXR 3.1.5, LIBTIFF Version 4.4.0, jpeg-turbo 2.1.4/jp62, FFMpeg 4.4 (Lavf58.76.100), libheif 1.13.0, null 1.0, libpng 1.6.38
OIIO <IP_ADDRESS>dev built for C++14/199711 sse2
Running on 12 cores 63.7GB sse2,sse3,ssse3,sse41,sse42,avx,avx2,avx512f,avx512dq,avx512cd,avx512bw,avx512vl,fma,f16c,popcnt,rdrand

JPEG as a format was designed for photographic images. It actually behaves very poorly for images with ares of solid color and hard edges. At the very least, all of the default settings that are great choices for photos are all wrong for that kind of image. It's just not the kind of image that JPEG compression was designed to handle well.

@lgritz thank you. I was suspecting chroma but wasn't sure if there was a way to control it. This indeed fixed the issue. Thank you mate. Appreciate.
I know that jpg is not designed for solid colours etc. I made this simple example to demonstrate the issue as I cannot use production frames.


#147 - fixed blogs layout

added breakpoints in style
fixed layout style
fixed layout responsiveness
created ScrollToTop component and implemented in blogs layout

@kire21 The layout looks much better. Thank you for making the changes. Can you just make the back to top button smaller, and the animation should be smoother.(going to the top of the page and also the button disappearing.)

OK. I will make these changes. :)

@kire21 What do you think about adding some hover effect on the scroll to top btn?

It is a good idea!


update task description

Describe your problem and how to reproduce it:
In the description, it said "Nest three button elements within each of your well div elements."
I think it would be clearer to change it to the following:
Nest three button elements within each of your div elements with class well.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/front-end-libraries/bootstrap/add-elements-within-your-bootstrap-wells
Tell us about your browser and operating system:

Browser Name: Chrome
Browser Version: 87.0.4280.88
Operating System: Mac OS

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

What about the following?

Nest three button elements within each of your div elements that have the class name well.

That's even better, thank you.

@jasontwuk Would you like to create a PR with this change?

@RandellDawson Yes, I would like to do this.


If an URL path element contains %2F (an escaped "/")  Flask.route considers it as an unescaped slash.

Here's an example:
from flask import Flask
app = Flask(__name__)

@app.route('/<urlvar>')
def hello_world(urlvar):
    return 'Hello World! ' + repr(urlvar)

if __name__ == '__main__':
    app.run(debug=True)

When running:
curl http://localhost:5000/aaa%2Fbbb

I would expect to see:
Hello World! u'aaa/bbb'

Instead I get:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>404 Not Found</title>

Not Found

The requested URL was not found on the server.  If you entered the URL manually please check your spelling and try again.

I tested this against Flask 0.9 and Flask 0.10.1. Both had the same error.
While searching for existing bug reports, I came across this: https://github.com/mitsuhiko/werkzeug/issues/21
Is this related? If yes, what's the status?
In any case, being a WSGI spec problem or not, this is behaviour is incorrect!

Not to add more to a long-closed issue, but for what it's worth, Passenger's WSGI implementation does not unescape URL-encoded characters before forwarding it along in PATH_INFO. This would seem to be a bug in Passenger (assuming that WSGI says that PATH_INFO should be pre-urldecoded), but I wrote up a workaround on StackOverflow to make it behave the same in that context as in Flask's local server.

If you encode once, then the parameter you get in flask is encoded. if you encode three times in javascript and decoding twice in python it will work:
in javascript
export const multipleEncode = (name) => {
//flask decode one so if you send one encode / become %2F that becomes a / so we need double encoding
return encodeURIComponent(encodeURIComponent(encodeURIComponent(encodeURIComponent(name))));
};
you will send multipleEncode (nameWithSlashes);
in python
from urllib import parse as urllib_parse
def multiple_decode(parameter, info):
if parameter is None:
return parameter
parameter_unquoted=urllib_parse.unquote(urllib_parse.unquote(parameter))
return parameter_unquoted


fix(tiflash): user clang-format 15 after v7.4

Related to this PR https://github.com/pingcap/tiflash/pull/7927/files#diff-a847e281c62ad2738f1616a96cd3174ca1fa5eba8134bf6f5d802a425c92c8b2R86-R90
For versions greater than or equal to 7.4 and less than 8.2, use clang-format 15.

/cc @JaySon-Huang @Lloyd-Pottiger


Add in kube-hunter, kube-bench, and microscanner

Add Aqua Security open source Kubernetes and container tools.

Thanks for the contribution!

@slietz you're welcome. I was going to add Anchore for container scanning and policy enforcement similar to Aqua and Twistlock. They have an open source solution you can use without cost (https://anchore.com/opensource/), however I wasn't sure which section to put it under.


Add list of open source funding platforms

In addition to the examples, folks should know about the awesome OS funding platforms that are out there. Paraphrased the descriptions directly from content on the site's About pages.

!m @timothyfcook
Don't forget Liberapay!
* **[Liberapay](https://liberapay.com/),** provides a stable crowdfunded basic income to creators, enabling them to keep doing great work that benefits everyone

My concern is that this ends up becoming an always-expanding bullet list. Is there a way this could be refocused so that there's 2-3 and it's clear they are examples of different ways of solving the problem?

I agree with @MikeMcQuaid. The goal is to point people toward popular options. Patreon and Open Collective are already listed. We'd consider other examples if other platforms are more widely adopted.

I can't disagree.
Back to work, @timothyfcook!

In the current version those read like examples of individual projects receiving corporate funding, not examples of platforms that anyone can use to host their project and get support from the community.
Seems useful to point people to the tools, rather than individual projects. I expect the list isn't going to grow much beyond the above.


[ALLUXIO-1749] Make thrift largest frame size adjustable

Thrift v0.9.3 changes the default size of largest allowable frame from 1GB (v0.9.3) to 16MB. It could become an issue in current implementation. This PR is used to make largest frame size adjustable and help further optimizition.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8725/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8727/
Test PASSed.

@pfxuan thanks for the PR, left some minor comments

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8751/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8752/
Test PASSed.

@jsimsa, the comments have been addressed. Please help check.

LGTM


More changes

Output polishing, Ignore the database files, and Fix keyamount.

Still some errors, check this and the next same lines

Yeah its still wrong, as we discuss.
Im still investigate some things, searching here and there.


GH2746: Log task duration at the end of a task

See #2746 for details.

@GeertvanHorrik your changes have been merged, thanks for your contribution 👍


Avoid duplicate headers (and the warning in XCode 6) on pod install

This should solve #295
I was able to test this by pod installing from my local copy using the :path parameter, and the warning goes away.

https://github.com/mutualmobile/MMDrawerController/pull/306 also resolves this, in a slightly different way. Either one should be sufficient; as long as one of them is included!
One workaround is to only add MMDrawerController/Core instead of every subspec, or every subspec instead of subclass if you need those.

Any chance this or #306 will be merged soon? It seems like this project is not being maintained at all.

I took #306. Hopefully will get some updates from some other PR's in the next 7-10 days.
Thanks!
:beers:

Cool, didn't see it was a dup when I opened this. Thanks!


prisma-engines: add git to nativeBuildInputs

several dependencies invoke git to get the current head / revision Example:
https://github.com/prisma/prisma-engines/blob/main/prisma-fmt/build.rs#L4
How to reproduce the issue:
nix-build -A prisma-engines

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I think this is a duplicate of https://github.com/NixOS/nixpkgs/pull/211099


Pep8 fix

Refactored code to follow pep8 style guide.
Some lines were extremely long (didn't even fit into github source code viewer)
Also: wouldn't be great to add automatic code style checker (i.e. flake8) as travis job

going to leave the long lines in for now. they're easier.


Add .travis build

I had a go at prototyping a way to run automated builds & tests using continuous integration. Let me know what you think.
I've used a service called Travis to do this (which is free for open source projects). There are other services like it, but most large open source software projects seem to use Travis (for example QGIS).
Travis is essentially an online virtual machine which hooks into GitHub. So now, every time a new commit is made to GitHub

Travis downloads the latest code
looks for a .travis file
executes its instructions

I've set it up to build and test on every commit. To see what this looks like on Travis, check out https://travis-ci.org/jmettes/DynAdjust/builds/524874865.

Steps followed in the .travis file:

install system packages for the build dependencies

operating system packages change a lot, this might be a way to quickly find out when things break

build DynAdjust

if at any stage it fails to build, the whole pipeline is stopped
this gives a strong incentive to fix the bug

perform very crude system test

it compares the output of an adjustment to a previously run adjustment, given identical inputs
this shouldn't be different, unless either

a change in the commit broke the software (see example of me breaking it)
or the software was changed enough to warrant an update to the test

it would be ideal to have more granular unit tests to drill down to error sources. but for now, I'd say this is better than nothing

This whole process takes about 15 minutes. This might become too long in the future. I'm playing around with some ideas to shave off some of this time.

Further notes:

Builds on Ubuntu. I believe Travis also supports Windows builds

not sure yet how to go about testing on NCI

I've added a small subset of AUSGeoid2020_20170829 (bounding box of stations in sample data)

@jmettes, looks good. I think this is an important QA tool and 15 minutes is not too long.


Release v 0.2.0

There were a few fixes contributed recently, having 0.2.0 release to crates.io would be very my appreciated!

#9 bit us.   We'll switch to a commit hash version but other users might appreciate avoiding segfaults too.

released on crates.io :)


Update .nf-core.yml

.nf-core.yml prevented the automated PR for the template update
PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/sammyseq branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Also, linting will fail because it is expecting the updates form the last template version, so don't worry for this PR, they can be ignored 🙂 let's just fix the .nf-core.yml and afterwards you can do the template update to fix the rest

I didn't get what is the problem in the file preventing the creation of the template update PR, is it ok now?

@mirpedrol can you please help me in fixing or skipping the linting error and merge? I can't find which file triggers the error message

ERROR: Invalid requirement: 'nf-core==': Expected end or semicolon (after name and no valid version specifier)
nf-core==
^
Error: Process completed with exit code 1.

This error will be fixed with the template update 3.0.2. I would advice to merge this PR with failing test and do the template update in a different PR, to simplify things.
For the template update you can install the last version of nf-core/tools 3.0.2 and run nf-core pipelines syn
Alternatively you can also execute the GitHub Action we have with the Run workflow button and making sure to write your pipeline name in the textbox (replacing "all")

@mirpedrol thank you, I probably don't have the privileges to merge this PR with a failing required check :( the Merge button is still inactive despite your approval. Can you please force it?

Done, you can now do the template update, thanks for this fix!

Thank you for your help!


CORS requirements on web exporters with navigator.sendBeacon

[x] This only affects the JavaScript OpenTelemetry library

navigator.sendBeacon with non-simple content-type header like "application/json" requires CORS checks like Access-Control-Allow-Headers, Access-Control-Allow-Methods, Access-Control-Allow-Origin, and Access-Control-Allow-Credentials to be set with appropriate values on preflight requests.
Notably, sendBeacon initialize the request with credential mode set to true with a non-simple content-type, which means the server has to respond to preflight requests with Access-Control-Allow-Credentials true too. This is not configurable nor an option exposed with the sendBeacon API. That is to say, credentials like cookies, authorization headers or TLS client certificates are sent with sendBeacon in this case. However, the credential mode is configurable with fetch and XHR APIs.
We have the following use cases on sendBeacon:

OTLP exporter: Sent with application/json content type, not tested if OTLP server responds with proper CORS headers.
Zipkin exporter: actually, the Zipkin server accepts JSON data sent over content-type as text/plain, so the problem does not apply to Zipkin backends.
Zipkin exporter sending to Jaeger backend: Not supported without a reverse proxy, Jaeger server rejects text/plain content-type, and doesn't respond with Access-Control-Allow-Credentials CORS header.

Currently, we don't provide options on how those data are sent, like sent with fetch (environments that don't support XHR or beacon), XHR or beacon, and the CORS credential modes like omit or include. I think it may be worthwhile to add a package that shares all the necessary setups on the web platform.

For the Zipkin compatible mode of the Jaeger server, I'm if it is behaving in an expected way. Maybe @yurishkuro can chime in here.

In https://www.jaegertracing.io/docs/1.35/cli/ I see the allowed headers and allowed origins, but I don't see the allow credentials response header. Is there somewhere else I should look for cors configs?

Zipkin receiver in Jaeger has a config option for cors.

Yes, but as stated in the OP it is missing Access-Control-Allow-Credentials CORS header. This behavior is as same as Zipkin's. Yet, Zipkin supports interpreting simple content types like text/plain so navigator.sendBeacon (the API Zipkin exporter uses by default) can successfully send data to Zipkin servers. However, Jaeger (tested with 1.35) does not support simple content types and rejects requests that are not typed as MIME application/json.

but I don't see the allow credentials response header

Seems like easily added if needed. The full Options struct has a lot more fields than what Jaeger CLI flags currently support - https://github.com/rs/cors/blob/da52b0701de54d82f71776d634d4183b69c3a915/cors.go#L32

Having said that, maybe it's better to implement this in OTLP receiver, then Jaeger could use that as well. I prefer to spend less effort on supporting legacy formats like Zipkin.

This works fine for me for now ~
import { OTLPTraceExporter } from '@opentelemetry/exporter-trace-otlp-http';
const exporter = new OTLPTraceExporter({
  headers: {}, // magic here !!!
  url: '<your ot receiver host>/v1/traces',
});

After digging into the source code with debugger i found this, with the header: {} option, ot sender will use xhr instead of sendBeacon !

so without credentials: 'include',
this config on ot receiver will work
receivers:
  otlp:
    protocols:
      grpc:
      http:
        cors:
          allowed_origins: ["*"]
          allowed_headers: ["*"]

Maybe we should include a way to force xhr or sendbeacon by config

Maybe we should include a way to force xhr or sendbeacon by config

Yeah, this is what I have in my mind too. I'm thinking about introducing a new @opentelemetry/core-web package for web utilities that need to be shared across various web packages. Currently, those utils are located in the @opentelemetry/sdk-trace-web (e.g. https://github.com/open-telemetry/opentelemetry-js/blob/main/packages/opentelemetry-sdk-trace-web/src/utils.ts#L201).

Maybe consider having xhr (or fetch considering how it's available in node v18+, deno v1 and usually polyfilled in IE) as the default method in general and navigator.sendBeacon as backup when in browser document is being unloaded?

sendBeacon has a 64KB limit. In firefox this is per sendBeacon call (eg can do 64KB + 64KB + 64KB), but in chrome this limit also applies for requests queue so 64 KB + won't send + won't send

It's common to send RUM data directly to a vendor endpoint (eg. https://bam.vendor-data.net/). There is also a rule in EasyPrivacy list used by content blockers like adblock, ublock origin and such:

! Block ping
$ping,third-party

What this rule means is all sendBeacon calls to third-party domains should be blocked, meaning all data from users with this adblock list enabled would be missing

More of a future looking but should any format introduce support for retrying sending data, it would be necessary to read response from server

In my opinion, xhr or fetch is more predictable ~


Endpoint for embedding in the zeeguu homepage

I was looking today into integrating the news reader in the main site.
The first problem we have to solve is to allow the user who is logged in to the main site to not need to login again to the UMR.
To do this, I think the easiest way is that you provide an endpoint which can take the session as an GET or POST parameter.
The second problem is to see how do we make the integration more visually appealing :)

We could also require every endpoint in our system to require the session as a GET parameter? In the same manner all Zeeguu endpoints are tagged with an @with_session parameter?
This might be more consistent, and we could still leave a simple login endpoint for debugging purposes.

We should probably discuss the stylistic integration on a whiteboard sometime, it's not so easy to discuss design choices in text ;)

Yes, every endpoint requiring the session in a GET parameter would work.
But then we have to move the reader on the zeeguu server, which serves
everything over SSL. Otherwise the session travels in clear, and that's not
cool.
On Sun, Mar 12, 2017 at 10:10 PM, Luc van den Brand <
<EMAIL_ADDRESS>wrote:

We should probably discuss the stylistic integration on a whiteboard
sometime, it's not so easy to discuss design choices in text ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mircealungu/Unified-Multilanguage-Reader/issues/24#issuecomment-285976923,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcWhyNeNH0N0IndHCfDWz67kJ8l3Q36ks5rlF9jgaJpZM4Mao4a
.

I understand your point. We'll try to add an additional endpoint.

I'm not sure that adding a new endpoint is the solution. I think the solution is:

We could also have every endpoint in our system require the session as a GET parameter? In the same manner all Zeeguu endpoints are tagged with an @with_session parameter?

Maybe we should discuss this in person :)


Older Server, cannot connect

Using an older mysql 5.5. server.
It says I need to use the

But I cannot find where to set this.

(https://user-images.githubusercontent.com/6238443/42217023-f9c2a170-7ec3-11e8-8268-08aa8bce12b6.png)=True


enforceStates parameter managed incorrectly in BodyShape, BodyBox, BodyCylinder

Reported by fcasella on 1 Mar 2010 14:43 UTC
The models BodyShape, BodyBox, BodyCylinder in Modelica.Mechanics.Multibody.Parts contain position and velocity vectors, fixed to frame_a, as well as an instance of Body connected to frame_a.
If enforceStates = true, the position and velocity vectors are forced to be state variables
SI.Position r_0[3](start={0,0,0}, stateSelect=if enforceStates then StateSelect.always else StateSelect.avoid);

SI.Velocity v_0[3](start={0,0,0}, stateSelect=if enforceStates then StateSelect.always else StateSelect.avoid)

but so are the position and velocity vectors inside the Body instance
  Body body(
    ...
    enforceStates=enforceStates,
    ...)

As a result two aliases of the position and velocity vectors are both forced to be state variables, and this is of course not possible.
The following simple model demonstrates the problem
model Test
  inner Modelica.Mechanics.MultiBody.World world;
  Modelica.Mechanics.MultiBody.Parts.BodyBox link(
    r={1,0,0},
    enforceStates=true);
end Test;

Output from Dymola:
Error: The equation
link.v_0[1] = link.body.v_0[1];
constrains the continuous time variables:
The following variables have stateSelect = StateSelect.always:
link.body.v_0[1]
link.v_0[1]
Error: requiring them to be continuous time state variables.
It is necessary to modify the stateSelect attributes or the model.
Unless there are other side effects, the problem can be solved by removing the stateSelect modifiers on the r_0 and v_0 variables in the three models.

Migrated-From: https://trac.modelica.org/Modelica/ticket/300

Comment by otter on 25 Jul 2010 14:01 UTC
Fixed in 05addf8b338923628861762978e58780735fd49e


[Car Port] 2022 Honda HR-V e:HEV

co-work with @somkanae
opendbc PR: https://github.com/commaai/opendbc/pull/812
Checklist

[x] added entry to CarInfo in selfdrive/car/*/values.py and ran selfdrive/car/docs.py to generate new docs
[x] test route added to routes.py
[x] route with openpilot: 1e4baee1aa2687a0|2023-04-11--11-42-42
[x] route with stock system: 1e4baee1aa2687a0|2023-04-11--12-01-15

@sshane anything blocking this? looks like a straightforward car port

The user of this Thai car reports they get "Hill Descent Error" that I haven't had time to look into at all. The Brazilian HR-V also has issues in experimental mode (it fails to respond to our brake command at low speed), so I didn't want to merge any global HR-V yet until I looked into it more.

We shouldn’t block merging ports on experimental longitudinal control. Let’s just blacklist them from that and include a good comment.

I'll try to find a way to detect it, they also are slightly different in another way I forgot about, so probably a new platform will work.

thanks for the PR, but looks like this is also in https://github.com/commaai/openpilot/pull/27915, with proper detection


Using "limit" as a query parameter in a request results in a 500 error.

Running standalone process with wiremock-standalone-2.25.1.jar make any request(mocked or not) and include the query parameter "limit" in the request. The request works fine with other query parameters, but not with "limit".
sample json mapping
{
  "request": {
    "method": "GET",
    "urlPathPattern": "/test",
    "queryParameters": {
      "limit": {
        "matches": "^(.*)"
      }
    }
  },
  "response": {
    "status": 200,
    "headers": {
      "Content-Type": "application/json"
    },
    "jsonBody": {"some": "response"}
  }
}

Expected Result:
{
    "some": "response"
}

Actual Result:
java.lang.NullPointerException
	at java.lang.String.contains(String.java:2133)
	at com.github.tomakehurst.wiremock.matching.ContainsPattern.match(ContainsPattern.java:32)
	at com.github.tomakehurst.wiremock.matching.ContainsPattern.match(ContainsPattern.java:20)
	at com.github.tomakehurst.wiremock.matching.MultiValuePattern$1.apply(MultiValuePattern.java:79)
	at com.github.tomakehurst.wiremock.matching.MultiValuePattern$1.apply(MultiValuePattern.java:77)
	at wiremock.com.google.common.collect.Lists$TransformingRandomAccessList$1.transform(Lists.java:638)
	at wiremock.com.google.common.collect.TransformedIterator.next(TransformedIterator.java:47)
	at java.util.Collections.min(Collections.java:635)
	at com.github.tomakehurst.wiremock.matching.MultiValuePattern.getBestMatch(MultiValuePattern.java:83)
	at com.github.tomakehurst.wiremock.matching.MultiValuePattern.match(MultiValuePattern.java:55)
	at com.github.tomakehurst.wiremock.matching.RequestPattern$4.apply(RequestPattern.java:250)
	at com.github.tomakehurst.wiremock.matching.RequestPattern$4.apply(RequestPattern.java:248)
	at wiremock.com.google.common.collect.Iterators$7.transform(Iterators.java:750)
	at wiremock.com.google.common.collect.TransformedIterator.next(TransformedIterator.java:47)
	at wiremock.com.google.common.collect.ImmutableCollection$Builder.addAll(ImmutableCollection.java:409)
	at wiremock.com.google.common.collect.ImmutableList$Builder.addAll(ImmutableList.java:699)
	at wiremock.com.google.common.collect.ImmutableList.copyOf(ImmutableList.java:256)
	at wiremock.com.google.common.collect.ImmutableList.copyOf(ImmutableList.java:209)
	at wiremock.com.google.common.collect.FluentIterable.toList(FluentIterable.java:614)
	at com.github.tomakehurst.wiremock.matching.RequestPattern.allQueryParamsMatch(RequestPattern.java:246)
	at com.github.tomakehurst.wiremock.matching.RequestPattern.access$300(RequestPattern.java:44)
	at com.github.tomakehurst.wiremock.matching.RequestPattern$1.match(RequestPattern.java:83)
	at com.github.tomakehurst.wiremock.matching.RequestPattern$1.match(RequestPattern.java:80)
	at com.github.tomakehurst.wiremock.matching.RequestPattern.match(RequestPattern.java:178)
	at com.github.tomakehurst.wiremock.stubbing.InMemoryStubMappings$3.apply(InMemoryStubMappings.java:225)
	at com.github.tomakehurst.wiremock.stubbing.InMemoryStubMappings$3.apply(InMemoryStubMappings.java:223)
	at wiremock.com.google.common.collect.Iterators$6.computeNext(Iterators.java:617)
	at wiremock.com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:145)
	at wiremock.com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:140)
	at wiremock.com.google.common.collect.Iterators.getNext(Iterators.java:818)
	at wiremock.com.google.common.collect.Iterators.find(Iterators.java:688)
	at wiremock.com.google.common.collect.Iterables.find(Iterables.java:638)
	at com.github.tomakehurst.wiremock.stubbing.InMemoryStubMappings.serveFor(InMemoryStubMappings.java:71)
	at com.github.tomakehurst.wiremock.core.WireMockApp.serveStubFor(WireMockApp.java:204)
	at com.github.tomakehurst.wiremock.http.StubRequestHandler.handleRequest(StubRequestHandler.java:53)
	at com.github.tomakehurst.wiremock.http.AbstractRequestHandler.handle(AbstractRequestHandler.java:65)
	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet.service(WireMockHandlerDispatchingServlet.java:120)
	at wiremock.javax.servlet.http.HttpServlet.service(HttpServlet.java:790)
	at wiremock.org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:812)
	at wiremock.org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:587)
	at wiremock.org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1127)
	at wiremock.org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:515)
	at wiremock.org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1061)
	at wiremock.org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)
	at wiremock.org.eclipse.jetty.servlets.gzip.GzipHandler.handle(GzipHandler.java:479)
	at wiremock.org.eclipse.jetty.server.handler.HandlerCollection.handle(HandlerCollection.java:110)
	at wiremock.org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:97)
	at wiremock.org.eclipse.jetty.server.Server.handle(Server.java:499)
	at wiremock.org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:311)
	at wiremock.org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:258)
	at wiremock.org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
	at wiremock.org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
	at wiremock.org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
	at java.lang.Thread.run(Thread.java:748)

I'm struggling to replicate this one. I copied your stub example into my mappings and it worked fine with this curl command:
curl -v 'http://localhost:8888/test?limit=123'

Also, the fact that your pasted stack trace has the ContainsPattern class in it suggests that the stub given in the example isn't the one matching your request
Could you post your entire stub set and the HTTP request you're using?

Thank you for the response! I am re-looking at it based on your feedback and will update. I created a new sample project and it does indeed work(I thought for sure I had created a really simple project and it wasn't working which is what I pasted from). I deleted all my other mapping from my main project as well and was able get it working once I had only the trouble mapping left. This means it's likely something conflicting somehow between all my mappings in my client project. It's more likely than not an issue on my end and not a bug.

Okay, thank you again for responding. I have created a repo showing the possible issue:
https://github.com/drewtwo/wiremockExample
It is mostly on my side. If you look at getBadLimitRequest.json it has limit included as a query parameter but the contains value is a number. In the getSearchRequest.json the contains value is a string. The mock server spins up but the bad limit request breaks all other requests using the limit query parameter.
In my project I have ~40 requests mocked out in separate files and only a couple use the limit parameter and I didn't think about checking if the other requests might have had an error. I apologize for not posting a better example initially.

WireMock should probably do a better job of validating stub JSON in cases like this. It should reject a match operator with the wrong type on create (or startup if using files), rather than blowing up when a request is processed.
I'll mark this as a bug to be fixed, as I've just found out that (for some reason) the matcher gets coerced to an empty object when it can't be parsed, which is definitely dodgy.


EMoE Language Evaluation

Hello, during the Language training and testing process of EMoE, when I test after training, the following is displayed:

['cola']
Namespace(adaptive_experts=False, add_expert_size=0, aux_loss_weight=0.01, cache_dir='./.cache', capacity_factor=1.5, checkpointing_steps=None, disable_peft=False, expert_repeat=1, gate_noise=1.0, gate_type='top', gradient_accumulation_steps=1, hub_model_id=None, hub_token=None, ignore_mismatched_sizes=False, include_training=False, is_gshard_loss=False, key_gate=False, learning_rates=[2e-05, 3e-05, 5e-05], load_model=None, lr_scheduler_type=<SchedulerType.LINEAR: 'linear'>, max_expert_num=8, max_length=128, max_train_steps=None, model_name_or_path='/MyData/bert-large-cased', moe_drop=0.1, moe_layers=[10, 11], normalize_one_score_gate=False, num_experts=16, num_train_epochs=10, num_warmup_steps=0, one_score=False, one_score_gate_update_momentum=0.0, output_dir='test', pad_to_max_length=False, per_device_eval_batch_size=32, per_device_train_batch_size=64, push_to_hub=False, random_ cluster=False, random_init_gate=False, report_to='tensorboard', resume_from_checkpoint=None, save_model=False, seeds=[0, 1, 2], source_dir='/MyData/bert-large-cased_save/cola', task_name='cola', to_MoE=False, top_k=4, train_file=None, use_fp1 6=True, use_slow_tokenizer=False, validation_file=None, weight_decay=0.0, with_tracking=True) learn_gate_random_False_repeat16
test
No best results found
What is the problem?
As far as I can remember I only changed the following in search_glue_no_trainer.py line 544:
------------------------------------------------
data_collator = DataCollatorWithPadding(tokenizer, pad_to_multiple_of=(8 if accelerator.use_fp16 else None))
------------------------------------------------
there is an error (*** AttributeError: 'Accelerator' object has no attribute 'use_fp16'), so I changed it to:
------------------------------------------------
try:
pad_to_multiple_of = (8 if accelerator.use_fp16 else None)
except:
pad_to_multiple_of = (None)
data_collator = DataCollatorWithPadding(tokenizer, pad_to_multiple_of=pad_to_multiple_of)
------------------------------------------------

During training, EMoE does a grid search on seeds (0,1,2) and lr (2e-5, 3e-5, 5e-5), each combination will produce a result. And when grid search ends, a txt file will be saved at the output dir. You may see something like this:

This filename of this txt file contains the best learning rate found during training. In test_glue_no_trainer.py, you should see the follow lines of code, which extracts the best lr from the filename of the txt file. So I think you need to check whether the txt file is saved successfully during training.

During training, EMoE does a grid search on seeds (0,1,2) and lr (2e-5, 3e-5, 5e-5), each combination will produce a result. And when grid search ends, a txt file will be saved at the output dir. You may see something like this:  The filename of this txt file contains the best learning rate found during training. In test_glue_no_trainer.py, you should see the follow lines of code, which extracts the best lr from the filename of the txt file. So to find the bug, I think you need to check whether the txt file is saved successfully during training.

Thanks for your quick reply!
I have solved the above problem, but I have a new problem and I would like to ask if you know the reason, which also appeared in the testing stage:


Refactor/revert functionality for data augmentation order

This will (should) be addressed when refactoring ADS with higher level libraries like Keras/Pytorch (#141).
See discussion near the end of PR #150 for details of old/vs new functionality, and why we want to revert to the old method (or a "better" one that doesn't use values the key names are variables).
i.e. Something like this happened for all data augmentation key-val definitions in the config:
da-flipping: {'activate': True, 'order': 2} was redefined "recently" (months to a year ago) to da-2-flipping-activate: True
and the preferred way would be to revert back to something closer to the old way.

Closing this issue as training and data augmentation are now done on ivadomed's side.


Transfer view - encrypted message improvements

Right now when user put some text into the message and tick "Encrypt the message":

it's asked for a password
text is changed to the encrypted string

My proposition:

present to user always unencrypted text (don't change to encrypted string)
if possible don't ask the user for a password. User need to put anyway password when sending transaction.

The only possibility of not requiring a user password is to store the user password in a session or any other storage, which can weaken user protection.

Yes, saving the password is bad.
But why don't we encrypt the message right before sending, after the user enters his password in confirmation modal?

we have to have the user password to encrypt the message, there is no way around that.
But I agree with other comments, could that encryption be done at the signing the transaction stage when the user has already entered their password?
I think the problem around this is trying to calculate the fee or maybe know how much space is left in the message?
But basically the little checkbox that one clicks when the click "encrypted message" is kind of a faux encrypt checkbox, meaning it isn't encrypted in real time, but instead put on the list to do later after confirmation. Like I said, the biggest problem I can think of in this case is that it some how messes with the message length or fee structure because of that.


[draft] weird error message, T::of vs of

Seems that some packages are imported unexpectedly. I will take a look.


Separate test states to avoid any duplication

Actuall changes are ~ +-200 lines
the rest are updated tests.
Refers to but not doesn't close #750

Introduced changes

Separate TestRunner, Cheatnet and Blockifier states completely

Breaking changes

Checklist

[x] Linked relevant issue
[x] Updated relevant documentation
[x] Added relevant tests
[x] Performed self-review of the code
[x] Added changes to CHANGELOG.md

Feel free to review this one further, I am merging it to avoide excessive conflicts. Will address all comments in follow up PRs


expose FormControlFeedback (props?) in Form.Autosuggest

from https://github.com/openedx/frontend-app-authn/issues/1133#issuecomment-1940664097

In addition to the missing onErrorStateChange, the reason for not using the internal error state provided by Form.Autosuggest is the way it renders error. It is prefixed with "x" icon with no way to remove it. All other errors on Authn don't have it and this makes the form errors inconsistent.

This is happening because we are using default props for FormControlFeedback in Form.Autosuggest, meaning hasIcon is always set to true
https://github.com/openedx/paragon/blob/2801667f79d4a8edea101d35bc77d982a2f0c14e/src/Form/FormAutosuggest.jsx#L333-L343
https://github.com/openedx/paragon/blob/2801667f79d4a8edea101d35bc77d982a2f0c14e/src/Form/FormControlFeedback.jsx#L48-L54
The simplest solution to this exact use case would be to just expose hasIcon, but it's worth exploring how/how much of FormControlFeedback we want to expose here.

https://paragon-openedx.netlify.app/components/datatable/#paginated-selection example of bulk actions components being passed in was discussed in Paragon WG. I will try following that pattern to implement this