Project Details page - preliminary provenance values

User Story
As a Data Sponsor, when I review my project's details, I need to see event markers for submission requests, along with the current status, so that I can fulfill my duty of project monitoring and decide whether any further action is required.
Notes
We don't have all the details worked out yet for recording provenance in project metadata. The goal with this issue is to make a little progress by having a place to show key details to Data Sponsors on the Project Details page. I don't want this to be a blocker for #287, and it looks like some headway was already made in #264 and #302.
As of Dec. 13, I don't see a provenance section in staging yet:

For new project requests covered by #113, we certainly have a way to record who submitted the request, and presumably we can record a timestamp of when that happened (please, always use the ISO 8601 standard for all dates and times in metadata). When a new project is requested, the first status it receives is always "Pending".
We do not yet have a way to interoperate with other systems to automatically record project request approvals or update statuses from "Pending" to "Active". I suggest the way to keep moving forward in the meantime is to focus only on recording the submission request provenance.
Completion Criteria

[ ] Create a rails class called ProvenanceEvent where these will be stored
[ ] The preliminary "Project Details" page includes a section with the heading "Provenance" below the "Automatic Settings" section
[ ] The ProvenanceEvent has the following Fields:
- Event Type [ "Submission", "Status Update" ]
- Event Details [ see below criteria for submission ]
- Event Person [ netid of the creator ]
- Event timestamp [ timestamp on creation ]
[ ] The "Provenance" section on the display includes 2 fields (in order): "Submission" and "Status"
- Status is the current status
- Submission is the list of Event Details for the Request and the Approval only
-
[ ] Submission Event Details: The "Submission" field displays, for now, "Requested by [data sponsor name], [timestamp]", where [data sponsor name] is filled by the full name of the person who submitted the request form for that project and [timestamp] is filled by the ISO 8601-standardized date and time with time zone offset, as recorded by the server at the time of project request submission
[ ] Status Event Details: The Status was updated from [original] to [new]
[ ] When a Project form is submitted two Provenance Events are created Submission & Status

Hey team! Please add your planning poker estimate with Zenhub @hectorcorrea @JaymeeH @jrgriffiniii @kelynch @leefaisonr @bess


Problem on Yosemite

This does not work on Yosemite for me. Followed MakeFile instructions. Using example Lua file. Not sure what Error 3 is in plist part. I tried activating and deactivating use Option as Meta. Any ideas?
$ make output
cc -Ilua-5.2.1/src -MMD -Wpointer-arith -Werror -Wall -Wextra -W -Wunused -Wno-unused-parameter -Wno-unused-function -Wuninitialized -std=c99 -O3 -c -o cmd-key-happy.o cmd-key-happy.m
cc -Ilua-5.2.1/src -O3 -c -o lua-5.2.1.o lua-5.2.1.c
cc -g -o cmd-key-happy cmd-key-happy.o lua-5.2.1.o -framework Foundation -framework AppKit -framework Carbon
$ sudo make install
/usr/bin/install -d /usr/local/bin
/usr/bin/install -m 555 cmd-key-happy /usr/local/bin
/usr/bin/install -m 555 cmd-key-happy-restart.sh /usr/local/bin/cmd-key-happy-restart
$ make install-rcfile
cp example-rcfile.lua ~/.cmd-key-happy.lua
$ make install-plist
mkdir -p /Users/Joel/Library/LaunchAgents
launchctl stop com.frobware.cmd-key-happy
make: [install-plist] Error 3 (ignored)
launchctl unload /Users/Joel/Library/LaunchAgents/com.frobware.cmd-key-happy.plist
/Users/Joel/Library/LaunchAgents/com.frobware.cmd-key-happy.plist: No such file or directory
rm -f /Users/Joel/Library/LaunchAgents/com.frobware.cmd-key-happy.plist
sed -e 's~%INSTALL_ROOT~/usr/local~' com.frobware.cmd-key-happy.plist > /Users/Joel/Library/LaunchAgents/com.frobware.cmd-key-happy.plist
chmod 644 /Users/Joel/Library/LaunchAgents/com.frobware.cmd-key-happy.plist
launchctl load -S Aqua /Users/Joel/Library/LaunchAgents/com.frobware.cmd-key-happy.plist
launchctl start com.frobware.cmd-key-happy
I set my permissions as specified.
$ cmd-key-happy-restart

The "Error 3" is probably a first install issue - there was nothing to do because cmd-key-happy wasn't already running.  You have to also open "System Preferences", "Security & Privacy", look in the "Accessibility" section and enable "cmd-key-happy". After that, run "cmd-key-happy-restart".

Yes I have followed those instructions completely. Here's a photo of my System Preferences to show. I have checked Activity Monitor and it shows "cmd-key-happy" is running but if I open a new Terminal.app and type Command-V for example, I still get paste.

The sample file explicitly excludes "cmd-v". cmd-key-happy will swap cmd for alt except where specified. that's the purpose of the "excludes = { ... }" section. Try removing cmd-v - and don't forget to run cmd-key-happy-restart fore each file changed.

Sorry I meant Command-F. I have also tested with Command-S and other key combinations not in excludes and Terminal.app displays normal behavior. Below is my .cmd-key-happy.lua file if it helps.
function Set(t)
local s = {}
for _,v in pairs(t) do s[v] = true end
return s
end
function set_contains(t, e)
return t[e]
end
-- The set of global shortcuts we don't want to swap cmd/alt.
global_excludes = Set{ "shift-cmd-tab",
"cmd-tab" }
-- The set of apps we want to consider swapping keys for, with some
-- notable exclusions. The exclusion means that a "cmd-c" will do the
-- normal OS Terminal behaviour. If you omit items then you would
-- have to use "alt-c" to close a terminal window.
apps = {
Terminal = { exclude = Set{ "shift-cmd-[",
"shift-cmd-]",
"cmd-c",
"cmd-v",
"cmd-1",
"cmd-2",
"cmd-3",
"cmd-t",
"cmd-n",
"cmd-`",
} },
Eclipse  = { exclude = {} },
Xcode    = { exclude = {} },
TextMate = { exclude = Set { "cmd-1",
"cmd-2",
"cmd-3",
"cmd-4",
"cmd-t" ,
"cmd-fn-right",
"cmd-fn-left",
} },
["NX Player for OS X"] = { exclude = Set{} },
["Parallels Desktop"] = {},
}
debug = false
dbgprintf = function(s,...)
if debug then io.write(s:format(...)) end
end
-- Return true to swap cmd/alt, otherwise false.
-- This function is passed a table comprising the following keys:
--   key_str_seq	key sequence (e.g., "shift-cmd-e")
--   alt		true if the alt key was pressed
--   fn                 true if the fn key was pressed
--   control            true if the control key was pressed
--   shift              true if the shift key was pressed
--   cmd                true if the command key was pressed
--   keycode		numeric virtual keycode (e.g., 48)
--   appname            the frontmost application (e.g., Terminal)
-- The order of the modifier keys in key_str_eq is always:
--   shift control alt cmd fn, separated by a hyphen ("-").
function swap_keys(t)
-- for i,v in pairs(t) do print(i,v) end
-- print(t.appname)
if set_contains(global_excludes, t.key_str_seq) then
return false
end
if not apps[t.appname] then
return false
end
local excludes = apps[t.appname]["exclude"]
if (excludes ~= nil and set_contains(excludes, t.key_str_seq)) then
dbgprintf("-- %s: "%s", explicitly ignored\n", t.appname, t.key_str_seq)
return false
end
s, n = t.key_str_seq:gsub("cmd", "alt")

Could you uncomment the two lines towards the top of the swap_keys function and then run cmd-key-happy in the foreground.
$ pkill cmd-key-happy
$ /usr/local/bin/cmd-key-happy
This will at least help us see if the install went OK as you should get the app name printed together with all the parameters to the function.

Asks me to set permission for Terminal. I set allow. Nothing gets printed unless I use a Command key combo in application other than Terminal. For example Command-F in Safari prints this.
key_str_seq	cmd-f
cmd	true
appname	Safari
control	false
keycode	3
Safari
shift	false
fn	false
alt	false

Could you try the following:
$ make uninstall
Go to "Security & Privacy" preferences, then "Privacy". From there remove entries for 'cmd-key-happy' and 'Terminal'.
Back in your shell:
$ make install
$ make install-plist
That should pop-up a dialog, click "Open System Preferences" in that dialog and grant "cmd-key-happy" access. It should be listed there, but not checked.
$ cmd-key-happy-restart
$ pgrep cmd-key-happy
And see if that works. I just tried that sequence - it's important after the uninstall steps that "cmd-key-happy" is not listed in the Privacy list.

I followed exactly and made sure that cmd-key-happy was removed from the Privacy list after uninstall. Then I did the reinstall and ran the two commands. Returns "23656" but still does not work.

I'm at a loss...
When you did the 'make install-plist' step, did you see the dialog pop up?

Yes indeed I did.

Reboot? (bleh!)

A couple things I noticed. Weirdly it works for Safari but not Terminal. T-T

Thanks for all the detail. And I like your separator in the terminal windows. :)
Do you have "Use Option as Meta Key" checked in your Terminal profile?

I know (quite some time ago) somebody had similar problems with Terminal - though not exactly sure he was trying this in any other app but we ended up deleting his Terminal preferences:
$ cp ~/Library/Preferences/com.apple.Terminal.plist ~/Terminal-Prefs.orig
$ rm ~/Library/Preferences/com.apple.Terminal.plist
quit and start Terminal, add back "Use Option as Meta". Really don't understand what's going wrong here.
One other thing: what do you set your terminal type to?
$ echo $TERM

Haha thanks! I have tried with both options for my profile and done a restart for both changes. Still not working.
I will try the preferences in a second.
$ echo $TERM
xterm-16color

I have 'xterm-256color'. I would be genuinely shocked if that made any difference...

And... going back to basics a bit... if it works for Safari then perhaps something else is stealing the 'Event Tap' for Terminal. What does "Spectacle" do? I see that listed in your accessibility list.

Hmm yeah removing the preferences file did nothing. It might actually be Spectacle. I use that to map window size changing. Let me try removing that for a bit.

Darn still nothing with Spectacle removed.

I need to head to bed... (just gone midnight here).
I have a dev branch which you could try...

Ah thanks for the help so far. Yes if you could let me know how to checkout that branch.

git clone http://github.com/andymcd/cmd-key-happy -b dev
cd cmd-key-happy/cpp
make
make local-lua-install
./lua2rc ~/.cmd-key-happy.lua > ~/.cmd-key-happy.rc
make uninstall
make install
make install-plist
cmd-key-happy-restart
Good luck!

Solved! There was a permissions issue with Terminal key events. If you click Terminal in top menu options it gives a drop down and there is an option called "Secure Keyboard Entry" that completely blocks some key events. Unchecking it lets it work otherwise cmd-key-happy must be run as root. Good game.

I did not have "Secure Keyboard Entry" in my terminal, and the master also does not work for me.  However, the dev patch works for me.  Thanks!


npm start fails

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository
Do you want to request a feature or report a bug?
Bug
What is the current behavior?
npm start fails
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
On clean project when I run npm start:
...
[759] ./src/app/app.style.css 160 bytes {4} [built]
[0] 4274ms -> [549] 52ms -> [542] 270ms -> factory:640ms building:970ms = 6206ms
[760] ./src/app/home/home.style.css 51 bytes {4} [built]
[0] 4274ms -> [552] 23ms -> [378] 68ms -> [375] 21ms -> [544] 60ms -> [543] 749ms -> factory:889ms building:127ms = 6211ms
Child html-webpack-plugin for "index.html":
chunk    {0} index.html 525 kB [rendered]
[0] ./~/html-webpack-plugin/lib/loader.js!./src/index.html 2.06 kB {0} [built]
factory:33ms building:40ms = 73ms
[1] ./~/lodash/lodash.js 523 kB {0} [built]
[0] 73ms -> factory:433ms building:769ms = 1275ms
[2] (webpack)/buildin/module.js 251 bytes {0} [built]
[0] 73ms -> [1] 1202ms -> factory:127ms building:717ms = 2119ms
webpack: bundle is now VALID.
[default] Checking started in a separate process...
[default] D:/Sources/asr/angular2-webpack-starter/node_modules/@angular/common/src/directives/ng_class.d.ts:81:35
Cannot find name 'Set'.
[default] D:/Sources/asr/angular2-webpack-starter/node_modules/@angular/common/src/facade/async.d.ts:34:33
Cannot find name 'Promise'.
[default] D:/Sources/asr/angular2-webpack-starter/node_modules/@angular/common/src/facade/async.d.ts:35:45
Cannot find name 'Promise'.
[default] D:/Sources/asr/angular2-webpack-starter/node_modules/@angular/common/src/facade/lang.d.ts:11:17
Cannot find name 'Map'.
...
My steps:
git clone --depth 1 https://github.com/angularclass/angular2-webpack-starter.git
cd angular2-webpack-starter
npm install
npm start

I'm having this same issue; however, when I actually go to localhost:3000, the site works; I just get this message spew as well. I suspect it is a 3rd party issue (duh), but haven't the faintest idea where to start looking.

Same for me.
Child html-webpack-plugin for "index.html":
chunk    {0} index.html 525 kB [rendered]
[0] ./~/html-webpack-plugin/lib/loader.js!./src/index.html 1.96 kB {0} [built]
factory:21ms building:33ms = 54ms
[1] ./~/lodash/lodash.js 523 kB {0} [built]
[0] 54ms -> factory:195ms building:442ms = 691ms
[2] (webpack)/buildin/module.js 251 bytes {0} [built]
[0] 54ms -> [1] 637ms -> factory:31ms building:179ms = 901ms
webpack: bundle is now VALID.
[default] Checking started in a separate process...
[default] /home/callum/1_develop/angular2-webpack-starter/node_modules/@angular/common/src/directives/ng_class.d.ts:81:35
Cannot find name 'Set'.
[default] /home/callum/1_develop/angular2-webpack-starter/node_modules/@angular/common/src/facade/async.d.ts:34:33
Cannot find name 'Promise'.
[default] /home/callum/1_develop/angular2-webpack-starter/node_modules/@angular/common/src/facade/async.d.ts:35:45
Cannot find name 'Promise'.
[default] /home/callum/1_develop/angular2-webpack-starter/node_modules/@angular/common/src/facade/lang.d.ts:11:17
Cannot find name 'Map'.
[default] /home/callum/1_develop/angular2-webpack-starter/node_modules/@angular/common/src/facade/lang.d.ts:12:17
Cannot find name 'Set'.
[default] /home/callum/1_develop/angular2-webpack-starter/node_modules/@angular/common/src/facade/lang.d.ts:68:59
...
It continues for other 90 lines, if you guys wants the whole log I can send it.
Btw the server runs

Update -
I've switched to branch material2 and I've done a npm install ... npm start worked with no errors

@scrapcupcake I go to localhost:3000 and get Connection broken error.

@david1983 David, material2 this is older code branch. There's still used typings. In the latest version of the master branch that is already do not have.

Confirmed on fresh install. What happened to typings?

I've got 2 different versions running. the older works fine ( typings 2.0.0-beta). this new one using 1.8.0 typescript has this same issue as above.

This is resolved by https://github.com/AngularClass/angular2-webpack-starter/commit/f71d4e6495d0cbc8e42549be355e024d6d78f518.

@sky-coding Thank you


Add filter send message

🤷♂ LGTM

Sorry I didn't add any context it's for a new feature on core side I just wanted you to check if I wasn't making AS problems or non idiomatic code

okay all good it looks go for me we can merge it


swap actual and expected value in error message

This fixes #49.

Thanks! However, the build is failing. I guess this might have something to do with the fact that .travis.yml contains no "script" key (which should point to vendor/bin/phpunit instead of a pre-installed phpunit executable. Could you maybe look into this @xabbuh ?

@matthiasnoback Let's see if #51 fixes that issue.

rebased to take into account the changes from #53

Thanks for your perseverance!

Thanks for teaching me a new word. :)

Hehe :)


[MdAutocomplete] cant select items in IE11

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Items in autocomplete shouldbe "mouse-selectable"
What is the current behavior?
When u try to select item with a mouse, panel just closes, and nothing happens.
But u can select items with keyboard interaction.
What are the steps to reproduce?
Open material official demo site with IE11
https://material.angular.io/components/component/autocomplete
and try to select something from dropdown with a mouse.
What is the use-case or motivation for changing an existing behavior?
Its not working for IE users.
Which versions of Angular, Material, OS, browsers are affected?
Tried Angular v.4.0.x, Windows 7, IE11 (11.0.9600.18537CO)
Is there anything else we should know?
I think something related to similar problem Promact/md2#43

This should've been fixed by https://github.com/angular/material2/pull/3188. Can you try installing the latest build from master?
npm install https://github.com/angular/material2-builds.git

Closing since this appears to have been fixed, feel free to open a new issue if the PR @crisbeto linked doesn't solve this for you


improve error messages on development

this also fixes an unrelated api issue

last commit has a chance of magically fixing everything btw


Entered unreachable code

I get
src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_support_map_support_map.rs:201:40:
internal error: entered unreachable code

This should not happen. it's rather hard to provide a repro
  0: std::panicking::begin_panic_handler
             at /rustc/a2d9d73e608f1b24eba840c4fd2d68dbe3b65e01/library\std\src\panicking.rs:647
   1: core::panicking::panic_fmt
             at /rustc/a2d9d73e608f1b24eba840c4fd2d68dbe3b65e01/library\core\src\panicking.rs:72
   2: core::panicking::panic
             at /rustc/a2d9d73e608f1b24eba840c4fd2d68dbe3b65e01/library\core\src\panicking.rs:144
   3: parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_support_map_support_map::compute_toi<parry2d::query::default_query_dispatcher::DefaultQueryDispatcher,parry2d::shape::ball::Ball,parry2d::shape::ball::Ball>
             src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_support_map_support_map.rs:201
   4: parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::impl$1::visit
             src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_composite_shape_shape.rs:143
   5: parry2d::partitioning::qbvh::qbvh::GenericQbvh<u32,parry2d::utils::array::DefaultStorage>::traverse_best_first_node<u32,parry2d::utils::array::DefaultStorage,parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::Nonline
             src\partitioning\qbvh\traversal.rs:163
   6: parry2d::partitioning::qbvh::qbvh::GenericQbvh<u32,parry2d::utils::array::DefaultStorage>::traverse_best_first
             src\partitioning\qbvh\traversal.rs:121
   7: parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::nonlinear_time_of_impact_composite_shape_shape
             src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_composite_shape_shape.rs:36
   8: parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::nonlinear_time_of_impact_shape_composite_shape
             src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_composite_shape_shape.rs:55
   9: parry2d::query::default_query_dispatcher::impl$0::nonlinear_time_of_impact
             src\query\default_query_dispatcher.rs:406
  10: parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::impl$1::visit::closure$0<parry2d::query::default_query_dispatcher::DefaultQueryDispatcher,dyn$<parry2d::shape::composite_shape::SimdCompositeShape> >
             src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_composite_shape_shape.rs:160
  11: parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::impl$1::visit
             src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_composite_shape_shape.rs:158
  12: parry2d::partitioning::qbvh::qbvh::GenericQbvh<u32,parry2d::utils::array::DefaultStorage>::traverse_best_first_node<u32,parry2d::utils::array::DefaultStorage,parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::Nonline
             src\partitioning\qbvh\traversal.rs:163
  13: parry2d::partitioning::qbvh::qbvh::GenericQbvh<u32,parry2d::utils::array::DefaultStorage>::traverse_best_first
             src\partitioning\qbvh\traversal.rs:121
  14: parry2d::query::nonlinear_time_of_impact::nonlinear_time_of_impact_composite_shape_shape::nonlinear_time_of_impact_composite_shape_shape
             src\query\nonlinear_time_of_impact\nonlinear_time_of_impact_composite_shape_shape.rs:36
  15: parry2d::query::default_query_dispatcher::impl$0::nonlinear_time_of_impact
             src\query\default_query_dispatcher.rs:393
  16: rapier2d::dynamics::ccd::toi_entry::TOIEntry::try_from_colliders<dyn$<parry2d::query::query_dispatcher::QueryDispatcher> >
             \rapier2d-0.17.2\src\dynamics\ccd\toi_entry.rs:131
  17: rapier2d::dynamics::ccd::ccd_solver::impl$1::predict_impacts_at_next_positions::closure$0
             \rapier2d-0.17.2\src\dynamics\ccd\ccd_solver.rs:317
  18: core::ops::function::impls::impl$3::call_mut<tuple$<ref$<rapier2d::geometry::collider_components::ColliderHandle> >,rapier2d::dynamics::ccd::ccd_solver::impl$1::predict_impacts_at_next_positions::closure_env$0>
             at /rustc/a2d9d73e608f1b24eba840c4fd2d68dbe3b65e01\library\core\src\ops\function.rs:294
  19: parry2d::query::visitors::bounding_volume_intersections_visitor::impl$1::visit
             src\query\visitors\bounding_volume_intersections_visitor.rs:42
  20: parry2d::partitioning::qbvh::qbvh::GenericQbvh<rapier2d::geometry::collider_components::ColliderHandle,parry2d::utils::array::DefaultStorage>::traverse_depth_first_node_with_stack
             src\partitioning\qbvh\traversal.rs:86
  21: parry2d::partitioning::qbvh::qbvh::GenericQbvh<rapier2d::geometry::collider_components::ColliderHandle,parry2d::utils::array::DefaultStorage>::traverse_depth_first_node<rapier2d::geometry::collider_components::ColliderHandle,parry2d::utils::array::Default
             src\partitioning\qbvh\traversal.rs:44
  22: rapier2d::dynamics::ccd::ccd_solver::CCDSolver::predict_impacts_at_next_positions
             \rapier2d-0.17.2\src\dynamics\ccd\ccd_solver.rs:281
  23: rapier2d::pipeline::physics_pipeline::PhysicsPipeline::run_ccd_motion_clamping
             \rapier2d-0.17.2\src\pipeline\physics_pipeline.rs:341
  24: rapier2d::pipeline::physics_pipeline::PhysicsPipeline::step
             \rapier2d-0.17.2\src\pipeline\physics_pipeline.rs:593
  25: PhysicsWorld::step_physics

I got this too (in rapier3d after despawning a bunch of things)

In Rapier, this is typically caused by CCD, because that uses nonlinear_time_of_impact, which in turn uses nonlinear_time_of_impact_support_map_support_map for most shapes.
I don't have time to look into this too deeply right now, but for others wanting to debug this, here are some of my findings just based on looking at the codebase.
The problem
The actual issue here is interesting. The non-linear time of impact query calls closest_points with a max distance using the maximal real value, i.e. Real::MAX (or Bounded::max_value()):
https://github.com/dimforge/parry/blob/8823198d704a85b748b8c45ee2f54e35f24a080e/src/query/nonlinear_time_of_impact/nonlinear_time_of_impact_support_map_support_map.rs#L128-L130
Because the max distance is essentially the largest it can be, the result is expected to never be ClosestPoints::Disjoint (i.e. non-intersecting and above the max distance). This is why the Disjoint variant is specified as unreachable, but somehow it is reached regardless, which causes the panic.
https://github.com/dimforge/parry/blob/8823198d704a85b748b8c45ee2f54e35f24a080e/src/query/nonlinear_time_of_impact/nonlinear_time_of_impact_support_map_support_map.rs#L201
This indicates that the closest_point query seems to be the actual culprit for the crash, as it returns Disjoint in a case where it should never happen.
Potential reasons for returning Disjoint
One option is that the distance between two objects is somehow Real::INFINITY for whatever reason. I think this would qualify as larger than Real::MAX, which could make closest_points return Disjoint.
If the closest_points method uses GJK for the given shape pair (it depends on the shapes), a very likely reason it can return Disjoint could be that GJK returns NoIntersection, because closest_points_support_map_support_map returns Disjoint in that case:
https://github.com/dimforge/parry/blob/8823198d704a85b748b8c45ee2f54e35f24a080e/src/query/closest_points/closest_points_support_map_support_map.rs#L30
This either happens if (1) at any moment the minimum bound computed by GJK is larger than the maximum distance (it'd have to be Real::INFINITY in this case), or if (2) GJK runs out of iterations before converging on a solution.
Finally, if the specific shape pair doesn't use GJK for closest_points, it could be that its special implementation happens to have a bug that causes it to incorrectly return Disjoint.
Debugging
If someone wants to debug this, I would recommend finding what exactly returns Disjoint by adding logs in the appropriate places, like the "unreachable" match branch and the closest_points implementations. If it's caused by GJK returning NoIntersection, adding logs there to see what the shapes and isometries are could be useful.

Yep, it was changed temporarily to log an error instead of panicking in #192 until a proper fix is found


Backup-DbaDatabase with BackupFileName creates ps-errors

Bug Report

General Troubleshooting steps

[x ] Verified running latest release of dbatools?
[x ] Verified errors are not related to permissions?
[x ] Can duplicate in new/fresh PowerShell.exe session?

Version Information

Operating System (Name|Version): Windows Server 2008 R2
PowerShell Version: 5.0.10586.117
SQL Server (Edition|Version):  SQL Server 11.0.6567
dbatools 0.9.18

Steps to Reproduce
running command in powershell ($sqlcred was set before) like
Backup-DbaDatabase -SqlInstance 'localhost,6201' -SqlCredential $sqlcred -Databases MYDB -BackupDirectory D:\applications\prg\Backup\sql -BackupFileName MYDB.bak -Type Full -CompressBackup
$error[0]

creates the following output:
WARNING: [Backup-DbaDatabase][11:46:45] SQL Server cannot write to the location D:\applications\prg\Backup\sql\
New-Object : A constructor was not found. Cannot find an appropriate constructor for type System.IO.FileInfo.
At line:364 char:13
+ ...            $file = New-Object System.IO.FileInfo($FinalBackupPath[0])
+                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (:) [New-Object], PSArgumentException
    + FullyQualifiedErrorId : CannotFindAppropriateCtor,Microsoft.PowerShell.Commands.NewObjectCommand

Split-Path : Cannot bind argument to parameter 'Path' because it is an empty array.
At line:474 char:30
+                 BackupFile = (split-path $FinalBackupPath -leaf)
+                                          ~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidData: (:) [Split-Path], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorEmptyArrayNotAllowed,Microsoft.PowerShell.Commands.SplitPathCommand

It works, when running without the -BackupFileName option (and creates the filename with the date-time stemp fine). It seems the -BackupFileName option is broken somehow.

Thanks for the full error report. I'll have a look at it.

Hi,
Did the backup actually complete? the only way I could reproduce this was with a missing drive spec. Removing filename stopped the nasty warning, but the output wasn't for a completion:

I'm putting some better path checks in, but want to make sure it was that causing the issue rather than something else

Yep ... without the parameter (only directory) the job completes fine:
Backup-DbaDatabase -SqlInstance 'localhost,6201' -SqlCredential $sqlcred -Databases MYDB -BackupDirectory D:\applications\prg\Backup\sql -Type Full -CompressBackup

SqlInstance      : localhost,6201
DatabaseName     : MYDB
BackupComplete   : True
BackupFilesCount : 1
BackupFile       : MYDB_201707251331.bak
BackupFolder     : D:\applications\prg\Backup\sql
BackupPath       : D:\applications\prg\Backup\sql\MYDB_201707251331.bak
Script           : BACKUP DATABASE [MYDB] TO  DISK = N'D:\applications\prg\Backup\sql\MYDB_201707251331.bak' WITH NOFORMAT, NOINIT, NOSKIP, REWIND,
                   NOUNLOAD, COMPRESSION,  STATS = 1
Verified         : False
Type             : Full
```

Afraid I've just not been able to reproduce this. And we've not have any other reports, so afraid closing this as cannot repro,.
it was potentially an issue with the client machine as I can't imagine any machines without System.IO.FileInfo being available, fairly common .net assembly,

@Stuart-Moore I was running into the same issue on a server where the credentials used do not have permissions to read from the filesystem (but has permissions to write).

Only with BackupFileName set, the final backup path is checked using Test-DbaSqlPath here.
Test-DbaSqlPath executes EXEC master.dbo.xp_fileexist ... which will return 0 because the user does not have said permissions.
This causes Test-DbaSqlPath to return false, and leads to the failure SQL Server cannot write to the location.
The above error block doesn't set $FinalBackupPath either, so we're trying to call New-Object System.IO.FileInfo($FinalBackupPath[0]) which leads to the mentioned New-Object : A constructor was not found. exception.

Hi,
@waffel you're using a really old version of the module. Could you update
to the latest and try again please.
Andreas, that's a special case. I've added an ignorefilechecks switch in
the backupnames branch if you want to take it for a test drive
On Sat, 14 Apr 2018 at 15:29, Andreas Maechler<EMAIL_ADDRESS>wrote:

@Stuart-Moore https://github.com/Stuart-Moore I was running into the
same issue on a server where the credentials used do not have permissions
to read from the filesystem (but has permissions to write).

Only with BackupFileName set, the final backup path is checked using
Test-DbaSqlPath here
https://github.com/sqlcollaborative/dbatools/blob/bc77f84d5be7a16a01a2e149bafb036eb23fde08/functions/Backup-DbaDatabase.ps1#L322
.
Test-DbaSqlPath executes EXEC master.dbo.xp_fileexist ... which will
return 0 because the user does not have said permissions.
This causes Test-DbaSqlPath to return false, and leads to the
failure SQL Server cannot write to the location.
The above error block doesn't set $FinalBackupPath either, so we're
trying to call
https://github.com/sqlcollaborative/dbatools/blob/bc77f84d5be7a16a01a2e149bafb036eb23fde08/functions/Backup-DbaDatabase.ps1#L381 New-Object
System.IO.FileInfo($FinalBackupPath[0]) which leads to the mentioned New-Object
: A constructor was not found. exception.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/1890#issuecomment-381333019,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFWoZkL5lgi0locvN9G3TEfAWvNKeodPks5togfFgaJpZM4OiVlw
.

@Stuart-Moore I give your branch a quick test, but am still running into an error. If I'm not mistaken, setting IgnoreFileCheck to $true now causes $FinalBackupPath to not be ever set.

ok, works with dbatools 0.9.325.
There are other issues with the command itself, but here I will create a different issue.
This can be closed. So the BackupFileName parameter works and is honored.
Thanks for pinging me back!

Reopening this as @amaechler issue is different and has exposed some flaws in the path parsing, so will hang the PR off this
Good to meet you at PSConfEU @waffel

After some more digging, I believe this is expected behaviour from SQL Server, and hence how SMO works:
I created a WriteOnly folder for my SQL Service account. (c:\sqlwo3).
backup database master to disk='c:\sqlwo3\test2.bak' - works as expected
restore headeronly from disk='c:\sqlwo3\test2.bak' - fails as expected
backup database master to disk='c:\sqlwo3\test2.bak' - fails
As we're appending another backup to the file, SQL Server needs to read the headers so it knows the offset to start pushing data in for the new backup.
When a filename is provided we're going to explicitly write into the file, so it already exists then it's going to throw a big error no matter what we do as the underlying SMO method is going to die horribly
When you don't specify a filename (just a directory) we generated the filename with a timestamp on the end, this timestamp is down to the minute. So if you did 2 quick backups in succession it would also fail, a bit more time between them and it's all good.

@Stuart-Moore Good points and analysis.
I think the ability to provide your own filename (assuming you know that the file does not exist, as based on your analysis it will fail otherwise) is still beneficial, especially in the context of scripting something where Backup-DbaDatabase is just one step.
I initially discovered this issue while writing a script that takes a backup using Backup-DbaDatabase, copies the file somewhere else and then does some more modifications to it. Ideally, I could pass in the filename (maybe even creating the same timestamp filename format) and then use that to manipulate the file further.
As a workaround, I currently call
$tempBackupFileName = "${SourceDatabase}_$(Get-Date -Format yyyyMMddHHmm).bak"

immediatly before calling Backup-DbaDatabase to get a handle for the filename (which in very rare cases could fail if the call happens just before the new minute).

PR is in and will probably get merged later.
In your case, you can specify a file name which will be honoured. Either of:
Backup-DbaDatabase -SqlInstance server -database db1 -BackupFileName c:\backups\backup.bak
or
Backup-DbaDatabase -SqlInstance server -database db1 -BackupDirectory c:\backups\ -BackupFileName backup.bak
So you can pregenerate the filename before hand and pass it in
$tempBackupFileName = "${SourceDatabase}_$(Get-Date -Format yyyyMMddHHmm).bak"
Backup-DbaDatabase -SqlInstance server -database db1 -BackupDirectory c:\backups\ -BackupFileName $tempBackupFileName

Fixed with #3558


docs(demo): change favicon to svg

favicon.ico is changed to favicon.svg
SVG (Scalable Vector Graphics) images are infinitely scalable without losing quality, have a smaller file size and are supported by all modern browsers. Therefore .svg format is more effective as a favicon

Upd. Redundant and outdated .ico favicons are deleted


User should be able to add self as contributor to new work

Describe the bug
When a user tries to add themselves as a contributor to a Work, sometimes their name(s) go into incorrect fields, and sometimes they fail to be added at all.
To Reproduce
Steps to reproduce the behavior:

Click on "Add a Work"
Click on "Contributors and Funding"
Click on "Add myself"
Possible failure states:
a. Nothing happens
b. User's full name is added to the surname field

Expected behavior
User's surname, first name, affiliation (if present) and username added to form.
Additional context
See Issues #2 and #9 on the spreadsheet.

Rand Test is an example user whose full name is added to the surname field.


Change the format of Slack release messages

I had some thoughts on how to improve our slack messages while implementing slack-style attachments in the browser.
Currently, messages about releases a difficult to parse, especially now that the controller type is included in the message:

I think splitting each controller entry into an attachment would be easier to understand:

This provides the opportunity to include more interesting content, such as links to the release itself, individual services, and container images.
One drawback is the 20 attachment limit from Slack. We would need to handle cases where more than 20 services get updated.
Link to the Slack Message formatter
This json renders the message you see in the screenshot:
{
    "text" : "Automated Release: <https://cloud.weave.works/fluxv2/proud-wind-05/releases/3d6066eb-df32-4454-6ec3-1aac67ee63af|3d6066eb-df32-4454-6ec3-1aac67ee63af>",
    "attachments": [
        {
            "title": "<https://cloud.weave.works/fluxv2/proud-wind-05/services/notification%3Adeployment%2Feventmanager|notification:deployment/configmanager>",
            "text": "A new container image was released to your service",
            "color": "#36a64f",
            "fields": [
              {
                 "title": "Previous Image",
                 "value": "<quay.io/weaveworks/notification-sender:master-ee6a451|master-ee6a451>",
                 "short": true
               },
               {
                  "title": "New Image",
                  "value": "<quay.io/weaveworks/notification-sender:master-5a7b625|master-5a7b625>",
                  "short": true
                }
            ]
        },
        {
            "title": "<https://cloud.weave.works/fluxv2/proud-wind-05/services/notification%3Adeployment%2Feventmanager|notification:deployment/eventmanager>",
            "text": "A new container image was released to your service",
            "color": "#36a64f",
            "fields": [
              {
                 "title": "Previous Image",
                 "value": "<quay.io/weaveworks/notification-eventmanager:master-ee6a451|master-ee6a451>",
                 "short": true
               },
               {
                  "title": "New Image",
                  "value": "<quay.io/weaveworks/notification-eventmanager:master-5a7b625|master-5a7b625>",
                  "short": true
                }
            ]
        }
    ]
}

@awh @foot @squaremo LMK what you think. I would be willing to work on this if we agree its a good improvement. This would have have a nice side effect of making events render nice in the browser.

Slick!
Clicking release id through to the releases page for more details would be neat. (#782 related?).

@jpellizzari This is a much clearer display of the release information. I'm happy for you to adapt the notifications.
Do all the individual entries have to be in a different attachment? If we can do tables (like old image, new image above) perhaps we can just list all the updates out in one attachment -- or, since there's the possibility of failures, in two attachments -- and summarise at the top.
The screenshot above doesn't mention the image that got released (something like quay.io/weaveworks/notification-sender), or the container using that image. In many cases the container doesn't matter, mind you.

This isn't a Flux feature. I would recommend migrating this issue into the service repo.


Add support for events for Insights

I'm looking to use Insights in one of my apps, so I've created this quick PR to try to add some support for events. This is not complete yet, so I've marked it as draft, but I wanted to get some initial feedback on the approach.
Since Insights supports arbitrary events, I added a Honeybadger::Event type that takes arbitrary data and serializes it into JSON. The ts key is automatically supported, defaults to the current time, and has nanosecond precision. All other properties, included nested arrays and hashes, are implemented via JSON::Serializable::Unmapped and can be provided either in the constructor or via event[key] = value.
Since Honeybadger::Dispatch is focused on exceptions, I created an EventDispatch class specifically for sending events. When you call Honeybadger.send(event), it will buffer events before sending them to the Honeybadger events API. The buffer will continue until either 5MB of events have been collected or 1 second has passed, whichever comes first.
A few other things I'd like to do here:

Subscribe to an instrumentation framework for common operations, a la ActiveSupport::Notifications

Does Pulsar work for this?

Reuse HTTP::Clients between reports to the API

Open questions

What do we do if an event exceeds 100KB? I feel like there should be a warning for this, but I'm not sure how to surface that. Logging it seems … cyclic. 😄
I've implemented nanosecond precision for ts, but only because that's what OTel prescribes. Is that overkill for Insights?

Hey @jgaskins, thanks for this! Sorry for the delay in commenting — I have updated my notifications settings for this repo. :)

I've implemented nanosecond precision for ts, but only because that's what OTel prescribes. Is that overkill for Insights?

Yeah, we're only doing milliseconds.

The buffer will continue until either 5MB of events have been collected or 1 second has passed, whichever comes first.

5MB feels large and 1 second feels short to me. Even though we support 5MB payload sizes, I'd go with 1MB for the max size and 60 seconds for the timeout.
Other than that, this looks good to me!

5MB feels large and 1 second feels short to me. Even though we support 5MB payload sizes, I'd go with 1MB for the max size and 60 seconds for the timeout.

I don't disagree, but I do have a clarifying question. Are 5 1MB requests preferred over a single 5MB request? My assumption was that amortizing request overhead would be preferred (for both my client and your server), but I may be overlooking some aspects.
I agree that 1 second is kinda eager. I was going to argue that a 60-second timeout runs the risk of losing events when a service is redeployed, but maybe we can register an at_exit handler here (since Crystal lets you register any number of them) to try to flush the buffer before the container shuts down. 🤔

I don't disagree, but I do have a clarifying question. Are 5 1MB requests preferred over a single 5MB request? My assumption was that amortizing request overhead would be preferred (for both my client and your server), but I may be overlooking some aspects.

That's a good point... I suppose a single request is better than 5.

I agree that 1 second is kinda eager. I was going to argue that a 60-second timeout runs the risk of losing events when a service is redeployed, but maybe we can register an at_exit handler here (since Crystal lets you register any number of them) to try to flush the buffer before the container shuts down. 🤔

An at_exit flush sounds like a great idea!

I know it's gone unchanged, but I've been running this essentially unchanged since the beginning of October and it's going great.

Thank you for the reminder here. Things have been absolutely nuts lately and I've lost track of a bunch of the things I was working on.


Tweak Block Logging in JSON Mode

We should put the block height in a dedicated field.

done here https://github.com/CounterpartyXCP/counterparty-core/pull/2150/commits/a0d55c9d8615ec2d87d46aeb33d8133bf74849ba


Offset tag doesn't work as indicated in the documentation

Describe the issue
When I use the %{O} tag it moves the whole block and not only the text to its right.
Expected behavior:
The %{O} tag only move the text to its right as the documentation said:

Offset %{O}
%{O10} pushed 10 pixels to the right %{O-5} 5 pixels to the left

Actual behavior:
The whole block is moved. By example if I use "%{O10} move right %{O-5} move left", the whole block is moved 5 pixels to the right (10-5).
Was it working before?

Did you also experience this bug in an earlier version of polybar? Don't know
If no, what was the last version where this worked correctly? Don't know

To Reproduce
A minimal but complete config with which the problem occurs:
[bar/example]
width = 100%
height = 27

font-1 = fixed:pixelsize=10;1
font-0 = Inconsolata:style=Bold:pixelsize=13;2
font-2 = siji:pixelsize=10;1

modules-left = label_test
modules-center =
modules-right =

[module/label_test]
type = custom/text
content = %{O10} pushed 10 pixels to the right %{O-5} 5 pixels to the left

[module/xwindow]
type = internal/xwindow
label = %title:0:30:...%

Polybar Log
Post everything polybar outputs to the terminal when you run it and the issue occurs below
* Loading config: /home/loma/Code/repositories/polybar/doc/config
- config: Current bar section: [bar/example]
* Loaded monitor eDP-1 (1920x1080+0+0)
* Bar geometry: 1920x27+0+0; Borders: 0,0,0,0
- bar: Attach X event sink
- bar: Attach signal receiver
- controller: Install signal handler
- controller: Setup user-defined modules
* Starting application
- controller: Main thread id = 1
* Starting module/label_test
- module/label_test: Thread id = 2
* Eventqueue worker (thread-id=3)
* Entering event loop (thread-id=1)
- bar: Create renderer
- controller: Attach config watch
- renderer: Get TrueColor visual
- renderer: Allocate colormap
- renderer: Allocate output window
- renderer: Allocate window pixmaps
- renderer: Allocate graphic contexts
- renderer: Allocate alignment blocks
- renderer: Allocate cairo components
- renderer: Load fonts
* Configured DPI = 96x96
* Loaded font "Inconsolata:style=Bold:pixelsize=13" (name=Inconsolata, offset=2, file=/usr/share/fonts/TTF/Inconsolata-Bold.ttf)
* Loaded font "fixed:pixelsize=10" (name=DejaVu Sans, offset=1, file=/usr/share/fonts/TTF/DejaVuSans.ttf)
* Loaded font "siji:pixelsize=10" (name=DejaVu Sans, offset=1, file=/usr/share/fonts/TTF/DejaVuSans.ttf)
* Bar window: 0x3a00001
- bar: Reconfigure window
- bar: Set window WM_NAME
- bar: Set window _NET_WM_WINDOW_TYPE
- bar: Set window _NET_WM_STATE
- bar: Set window _NET_WM_DESKTOP
- bar: Set window _NET_WM_PID
- bar: Map window
- bar: Draw empty bar
- bar: Setup tray manager
* Disabling tray manager (reason: missing `tray-position`)
* module/label_test: Rebuilding cache
- bar: Force update
* Redrawing bar window
- renderer: flush(1 geom=473x27+0+0, falloff=0)
- bar: Received expose event

Screenshots

Environment:

WM: i3 version 4.16.1 (2019-01-27)
Distro: ArchLinux 5.0.2-arch1-1-ARCH x86_64
Output of polybar -vvv:

polybar 3.3.0

Features: +alsa +curl +i3 -mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) -render -damage -sync +composite +xkb +xrm +xcursor

Build type: RelWithDebInfo
Compiler: /usr/bin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -Wall -Wextra -Werror -Wno-noexcept-type -pedantic -pedantic-errors
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

or (current master)
polybar 3.3.0-44-gded7a4f-git

Features: +alsa +curl +i3 -mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) -render -damage -sync +composite +xkb +xrm +xcursor

Build type: Debug
Compiler: /usr/bin/c++
Compiler flags:  -Wall -Wextra -Werror -Wno-noexcept-type -pedantic -pedantic-errors
Linker flags:

Additional context
While this problem isn't resolves #1671 is blocked. If this is really a bug and not the standard behavior, I can try to fix this bug in #1671 (since I need to modify the %{O} tag to add pt support)

I don't see anything wrong with your example, consider:
%{O100} move right %{O-100} move left, here the block itself is moved to the right 100px while move left is moved back again 100px making it overlap with move right. This seems to be consistent with what lemonbar does.
But the O tag does have some problems, mainly that it doesn't use the current background color but rather the bar background color.
With the following string:
%{O100}Filler%{B#ff0000 O100}First%{O200}Second%{O-100}

I get this with polybar:

but it should actually look like this (as it does with lemonbar):

This happens because polybar just advances the render position instead of rendering a solid square.

Hum, my bad I figured the problem, I also noticed the problem with the color. I'll try something.


Build fails while getting Google Calendar API data

Both locally and in GitHub Actions, the npm run build command fails because the asynchronous call to the Google Calendar API fails. It's not an issue with the service account, we double-checked and replaced the credentials.json file.
Surprisingly, this doesn't occur when using npm run dev, the calendar data loads just fine in this case.
Needs further investigation. We fixed it temporarily by disabling the calendar integration altogether in #98 - if the issue is fixed, this commit can just be reverted to restore the calendar feature.
Issue Detail:
Error: Not Found
    at Gaxios._request (/home/runner/work/iwi-website/iwi-website/node_modules/gaxios/build/src/gaxios.js:127:23)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Unfortunately, this is not very helpful...

@Blizzardcool would be great if you could try to reproduce this locally and confirm I'm not mistaken here...

@mapelpapel Where can i find the credentials.json file? Thank you!

@dustinsommerfeld sent it to you on Slack


Remove README.md file from backend repo

Description
README.md file removed from backend repo.
Testing
Not applicable.
Reviewers
@wsedor
@mentions of 1 person at minimum to review the proposed changes

seems the changes in this PR are not needed. Please close as necessary or ensure changes here are not same as in other PR


Client timestamps for game events

Splitting from #372, idea is from @nomeata
It's unfortunate that games are basically always wrong about their local state after a UI event.  That's because the server gives the event its canonical timestamp.  But we trust the clients not to cheat, so we can have the clients set timestamps instead.
The trick is that the clients need to know a safe committed timestamp, before which no new events will be added.  That is no longer the latest server event, since there's clock skew between clients.  So clients will need to send pings with their current times, so that the server knows the true committed time, which is the oldest timestamp from any client.

It might be sufficient if  the client estimates the time stamp the server will give to the client’s event, (it could ask for the difference by sending a time-stamped event) and then using this difference to almost correctly assign timestamps to unconfirmed local events as they enter the queue. This way, the general server protocol does not need to change.

First of all, I think it's reasonable to expect that 95% of the time, the after-the-fact change in time won't matter.  Most events will commute with events from other players, except for the occasional critical point (e.g., both players try to move to the same spot, or try to click the same object).  Most events will also commute with time steps, except in a few mechanics such as "click this before it disappears")
So, if the client estimates the server timestamp, what happens then?  Does it wait for a tenth of a second or so before applying the event locally?  I'm afraid that would be even worse!  We'd add visible latency in the 95% of the time that it doesn't matter, in exchange for avoiding a glitch in the remaining 5%.  I think it would generally be better to leave things as is.

There is a middle way. Assume we estinate the round trip to be one second. State is s. Then, after the user presses a button (e)and half a second passed, then we'd display state handle e (step 0.5 s) instead of step 0.5 (handle e s). This way, whenever the events comes back and we want and have to go to handle e (step 1 s), there is no jump, but rather a smooth interpolation between the anticipated local state and the  eventual committed state.

This vaguely related to #385, which is a big about discrepancy of event times between client and server.  Though in that case, the client was  actually incorrect.

I think I want to just push ahead with the client-side timestamp plan.  I've made a few steps in that direction recently.  Some outstanding points here:

Clients do not yet send pings.  The changes in commit 2dfec1099bfaa6da578fee9bf0c9fa8df22656d6 make it easy to do, though.
Once that's finished, we need something (perhaps the game server, or perhaps each client individually) to keep track of the latest timestamp from each client.  The minimum of these is the committed time for the game.
The prediction code will need updates to support inserting events at arbitrary times without advancing the committed time.  This should be easy.

But does it make sense to sort events by timestamp when the timestamps come from different sources?

It's true that clock synchronization is definitely a concern.  It would be possible to add approximate (e.g., SNTP-like) clock sync to the protocol.  I strongly suspect, though, that the artifacts from different clocks will be much less noticeable than one's own events changing timestamps in history.  At least the lag will be attached to things that other players do, which is more understandable.

Hmm, ok. I agree that in the absence of conflicting events, the user experience should be as good as a local interaction, so it unavoidably has to be the local time stamp.

I am about to write the paper section on this, and noticed we still haven’t implemented this scheme. I guess I should so that now.
If now the timestamps ensure order, and no longer the server, there is no point for the server to transmit an event back to the client, right? But we still might need to do that for eventual consistency, so hat the timestamp (if a Double) gets mangled by serialization the same way for all clients.
Should the timestamps really be doubles? Or Integer with some high precision (picoseconds or so).
What about events from two clients with precisely the same time stamp. I guess we can order by player number (although that gives an unfair advantage).
This scheme does not have much in common with the existing scheme any more, where the server orders events. That is a bid unfortunate for the narrative of the paper, but not much we can do about it.
If all clients count their time stamp from the moment they received the “game start” event, I think that ought to be enough synchronization.

I would have no complaints at all if you implemented this!
I think our plan for this currently is to drop clients whose pings fall too far behind?  If so, I agree that there is no reason for clients to receive their own events back again.  The only reason for changing this might be if, in the future, we were to try to recover from longer-term network unavailability by a client by telling it to skip some of its own history when it becomes available again.  But this can be done later if and when the change is made.
Timestamps are doubles on the client to start with, because they come thay way from the JavaScript implementation (https://developer.mozilla.org/en-US/docs/Web/API/Performance/now).  If transmitting doubles over the network is lossy, then we have other problems!  I think it's okay though:
Prelude Test.QuickCheck> quickCheck $ \ (n :: Double) -> read (show n) == n
+++ OK, passed 100 tests.

I'm not at all concerned about events with identical timestamps.  Client timestamps should have microsecond resolution, so I just don't think it will ever be a noticeable problem.
I am still concerned about clock synchronization, though.  If clients count their start times from the moment they receive the game start event, that's a TCP packet, so it's possible it could be dropped and retransmitted, leading to very big variations (a second?) in time until it's received.  And when that happens, this client's events would be significantly delayed to all other clients for the entire game, because of that one packet arbitrarily far in the future.  That being said, though, I'm all for implementing this in the simplest possible way, then filing a bug to add clock synchronization later.

Good idea with the quickcheck test :-)

And when that happens, this client's events would be significantly delayed to all other clients for the entire game, because of that one packet arbitrarily far in the future.

True. But remember that this client will – from its local p.o.v. – start the game when it receives the start package, so at least its own events will look just in time. There are two non-local directions to consider:

Packages from other clients to the late clients. Assuming the TCP connection gets a bit faster, these might actually carry timestamps that are, from the late client’s p.o.v., in the future. My code does not do that now, but should probably simply not apply these events until the local time has progressed far enough. If they are a bit in the past, then it looks like a normal delay.
Packages from the late client to other clients. They will be very much in the past, causing the list of pending packages to grow, and causing the late client to appear to be quite jumpy. But shrug – that is just a quantitative difference to the normal delay, not a large one.

Not a big deal so far, I’d say.

Done.  I'll file bugs for the remaining steps.


Supply  didn't  work

Hi,
I am playing with a friend in a 1.18.1 serveur with mechanization, we download the datapack+utilities+resource pack and I cannot supply the (for exemple) the alloy furnace with battery+steam generator.
All the carft and the ore génération work
Tim
Ps: sorry for the spelling, I am french

@ICY105

I found the solution, I must put some cable...


Breaks with 1.5rc1 due to "internal" package name.

What's the error exactly?

package foobar
	imports github.com/gengo/grpc-gateway/internal: use of internal package not allowed

Ah, I understand.
Are you talking about the same one as this design doc?
https://docs.google.com/document/d/1e8kOo3r51b2BWtTs_1uADIA5djfXhPT36s6eHVRIvaU/edit


Remove extra logic from throwOdspNetworkError

It should be a oneliner throw createOdspNetworkError(...). Or don't bother and callers can just type that in-line.
It's doing two other things today:

~Munging the message to include response.statusText (see
https://github.com/microsoft/FluidFramework/pull/7788/files#r727503529_), which is arbitrarily bifurcating errors in aggregate telemetry~
Pulling the retry-after header to compute retryAfterSeconds

If we want an overload of createOdspNetworkError let's make one but without the throw semantics.  In fact, some callers wrap throwOdspNetworkError in a try-catch to achieve this!

I removed the use of response.statusText in error message with #9249, and the other difference is not a big deal.  createOdspNetworkError is only used in one other place, and the caller has retryAfterSeconds but no Response, so it kind of makes sense.  The signature could be tightened down to some helper type that has either retryAfterSeconds or Response etc, but not worth tracking - no harm no foul.


Mat/Pat - Selenium Other Scenario - Deferred leave

Flow -  Maternity + Standard parental leave + Deferred Allowance + Supervisor approval
AC

[ ] Test applicant is able to submit successfully
[ ] Test supervisor receives submission and is able to approve

Link to script - https://bcgov.sharepoint.com/:u:/r/teams/03991/Shared Documents/General/Selenium/Selenium Scripts/Maternity Parental L%26A/DeferredLeave_1879.side?csf=1&web=1&e=OVTWPX

@MikePSA Can you peer review this script?

After testing, some potential improvements to the script are:
•	Clean up disabled commands
•	Use search function for the Maternity/Parental form and relocate “Start new” button
•	Missing confirm employee information is required checkbox?
•	If/click Saturday checkbox, can we change that to just click?
•	Something going on with the dates after I edit the script? (But I didn’t change date commands) need to discuss
•	At end of script, delete the command "click on success intro" as it should be wait for element before assert
•	Observed 60 seconds not long enough to generate new Mat form in review and process, may need to extend wait time

@ayushdamani  to test

Script worked great! I just had to increase the duration between clicks, while moving from one action to another!
Moving to PO!

Tested again - thanks Fazil!

Sweet! this worked great the very 1st try  - yahoo closing, PO approved


Burp Pro file error when  imported a saved copy

Just try with my Burp Pro V2021.9.1 and can't import the saved scan
Tested with Defectdojo 2.3.1 and fresh -dev from this 2021-11-04 day.
Error is :
uwsgi_1         | [pid: 35|app: 0|req: 7/10] <IP_ADDRESS> () {58 vars in 2183 bytes} [Thu Nov  4 07:24:46 2021] GET /alerts/count => generated 13 bytes in 28 msecs (HTTP/1.1 200) 6 headers in 171 bytes (1 switches on core 1)
uwsgi_1         | [04/Nov/2021 07:24:46] ERROR [dojo.engagement.views:633] not well-formed (invalid token): line 1, column 1
uwsgi_1         | Traceback (most recent call last):
uwsgi_1         |   File "/usr/local/lib/python3.8/xml/etree/ElementTree.py", line 1693, in feed
uwsgi_1         |     self.parser.Parse(data, 0)
uwsgi_1         | xml.parsers.expat.ExpatError: not well-formed (invalid token): line 1, column 1
uwsgi_1         |
uwsgi_1         | During handling of the above exception, another exception occurred:
uwsgi_1         |
uwsgi_1         | Traceback (most recent call last):
uwsgi_1         |   File "/app/./dojo/engagement/views.py", line 618, in import_scan_results
uwsgi_1         |     test, finding_count, closed_finding_count = importer.import_scan(scan, scan_type, engagement, user, environment, active=active, verified=verified, tags=tags,
uwsgi_1         |   File "/app/./dojo/importers/importer/importer.py", line 350, in import_scan
uwsgi_1         |     parsed_findings = parser.get_findings(scan, test)
uwsgi_1         |   File "/app/./dojo/tools/burp/parser.py", line 29, in get_findings
uwsgi_1         |     tree = etree.parse(xml_output, etree.XMLParser())
uwsgi_1         |   File "/usr/local/lib/python3.8/site-packages/defusedxml/common.py", line 100, in parse
uwsgi_1         |     return _parse(source, parser)
uwsgi_1         |   File "/usr/local/lib/python3.8/xml/etree/ElementTree.py", line 1202, in parse
uwsgi_1         |     tree.parse(source, parser)
uwsgi_1         |   File "/usr/local/lib/python3.8/xml/etree/ElementTree.py", line 601, in parse
uwsgi_1         |     parser.feed(data)
uwsgi_1         |   File "/usr/local/lib/python3.8/xml/etree/ElementTree.py", line 1695, in feed
uwsgi_1         |     self._raiseerror(v)
uwsgi_1         |   File "/usr/local/lib/python3.8/xml/etree/ElementTree.py", line 1602, in _raiseerror
uwsgi_1         |     raise err
uwsgi_1         | xml.etree.ElementTree.ParseError: not well-formed (invalid token): line 1, column 1
I can send the tested file (but seems not be upload in git :-( )

Line 1, column 1 is at the very start of the file. Is there some kind of invalid BOM character or something? (just guessing at the moment).

Line 1, column 1 is at the very start of the file. Is there some kind of invalid BOM character or something? (just guessing at the moment).

Just done a save of a simple projet and try to upload. I don't be sure this is only the problem....

just having an issue on my repos, seems working well now


RenderTexture - preserve world transform

This addresses issue https://github.com/GoodBoyDigital/pixi.js/issues/1174 - as a bonus it also fixes and expands documentation.
The renderCanvas and renderWebGL modify the worldTransform, worldAlpha,
and update the transformations of the children. This code adds in a
parameter to the these methods (true by default now, but that can be
changed) to indicate that these transformations should be undone when the
rendering is complete.
While restoring the previous transformations does add a little bit of
overhead this is adventageous in two important ways:

The default (or requested) behavior works in an "unsurprising manner"
and does not leave a temporary transformation applied permamently when
the render operation completes.
Even if the restore/preserve operation is the documentation and addition
of method parameter makes this behavior more apparent.

:+1: I will merge this in to the rewrite branch shortly (see: #1296).


Add Ksponspeech pretrained model

I just noticed a test error. I updated only the table.csv, so I wasn't expecting this to happen.
Can anyone help me fixing this problem?

@ftshijt, could you take a look at this?
https://github.com/espnet/espnet_model_zoo/pull/43/checks?check_run_id=3281969284#step:8:28
It seems that the test requires kenlm, but I'm not sure why it happens.

I think it's because a recent PR directly import NgramScorer which has such dependency. see this.
I can fix that by dynamic import only when initializing the Ngramscorer. How do you think about the solution?

I think it's because a recent PR directly import NgramScorer which has such dependency. see this.
I can fix that by dynamic import only when initializing the Ngramscorer. How do you think about the solution?

Yes, this solution sounds good to me.
Thanks!

It's working now. Thanks for your effort @YushiUeda

@ftshijt Thanks so much for your help!


Error running job: black-forest-labs/FLUX.1-dev does not appear to have a file named config.json.

Same issue.

Same issue...

I grapped all files saved in .cash in my user profile on windows and made same folders as in this link https://huggingface.co/black-forest-labs/FLUX.1-dev/tree/main
then copied tha path to config file as model source path

I will try this tip with cache.. i always download all folders from huggie
because i use onetrainer and it cant read sd3 fto huggie
sob., 17 sie 2024, 13:55 użytkownik abozahran @.***>
napisał:

I grapped all files saved in .cash in my user profile on windows and made
same folders as in this link
https://huggingface.co/black-forest-labs/FLUX.1-dev/tree/main
then copied tha path to config file as model source path
—
Reply to this email directly, view it on GitHub
https://github.com/ostris/ai-toolkit/issues/47#issuecomment-2294837752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTDZ5THBLJQSEGABYTD54LZR422PAVCNFSM6AAAAABMN3OUC6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJUHAZTONZVGI
.
You are receiving this because you commented.Message ID:
@.***>

Closing for now. Let me know if not resolved.

Same issue... but after download files from huge this error disapear: https://huggingface.co/black-forest-labs/FLUX.1-dev/tree/main/transformer

To what specific folder did you place the downloaded files?


docs: add simple example for integration with gofiber

As requested and discussed in these Issues I created a small example for an integration of Templ into gofiber. The example currently goes without a README.md file, I oriented on the other examples you currently provide.
Issues:

https://github.com/a-h/templ/issues/349
https://github.com/gofiber/template/issues/302#issuecomment-1867958802

If there's any more you'd like me to do before this goes in, please go ahead. I could also provide other examples regarding CSRF and HTMX or a small Templ gofiber HTMX Todo app.

Thanks for that, much appreciated!


Google starts requiring a captcha over time

One of the flaws with this approach is that the random and algorithmically generated queries look random and algorithmically generated.
In other words, Google can tell that it is a robot, and eventually if the wrong keyword combinations appear it may trigger a forced captcha.
Some possible solutions to explore:

Find an alternative URL source
Find a way to access the feeling lucky button
Change the frequency or general quantity of calls
???

(One possible alternative source is duckduckgo)

@peterberbec you experimented with duckduckgo -- my main concern is that when I tried it I noticed a little bit more instances of attempted NSFW results (they didn't go through, but that could be disconcerting to users).  What was your experience with the experiment?

Tried startpage.com?

DDG paused if it hit NSFW if the safeoff tag wasn't on. not optimal.
Anyone know bing? Might be good to have M$ do something useful for once...

I lowered the number of tabs to just 2, which seems to be far less likely to start bothering google.

The lowered number of tabs seems to have significantly lowered the number of reports of captcha requests.  They still happen, but not as often (and I have personally experienced them a lot less as well)


vscode.dev get crashed while saving a untitled editor

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.76.2
OS Version: macOS Monterey 12.6

Steps to Reproduce:

New File:

Enter some code and enter Command + s

Chrome get crashed

I have debug vscode source code and get this error(Source code at: https://github.com/microsoft/vscode/blob/main/src/vs/workbench/services/dialogs/browser/fileDialogService.ts#L127):
Unhandled Rejection (SecurityError): Failed to execute 'showSaveFilePicker' on 'Window': Must be handling a user gesture to show a file picker.

I have found the key code which cause the Chrome's crashing:
var types = [
  	{
        "description": "JSON",
        "accept": {
            "text/plain": [
                ".json",
                ".bowerrc",
                ".jscsrc",
                ".webmanifest",
                ".js.map",
                ".css.map",
                ".ts.map",
                ".har",
                ".jslintrc",
                ".jsonld"
            ]
        }
    }
];
window.showOpenFilePicker({ suggestedName: 'Untitled_1', startIn: undefined, types });

This is a Chrome's issue, I have report issue for Chrome

Thanks for reporting to Chrome, can you put the link here for future reference?

https://bugs.chromium.org/p/chromium/issues/detail?id=1426323


Add node-selector & toleration to the redis nodes

I want to take this PR opportunity and add node-selector & toleration to the redis nodes (optional, only if exists). we will need to configuration for preproduction environment
something like this:
{{ if .Values.NodeSelector }}
      nodeSelector:
{{ toYaml .Values.NodeSelector | indent 8 }}
{{ end }}
{{ if .Values.Tolerations }}
      tolerations:
{{ toYaml .Values.Tolerations | indent 8 }}
{{ end }}

We also need to add the option to the pod (if exists) in our code
Originally posted by @shyimo in https://github.com/PayU/redis-operator/pull/52#discussion_r554532621

As part of this work I will make the CR more generic by adding a whole PodSpec field. This way the pods are completely customisable and the only special case we need to handle are the affinity rules. We need a particular configuration in order to ensure HA over AZs.
However, it might be the case that not all users need this, so I will set a default behind a flag 'enableAZDistrbution' that makes sure soft rules for leader/node anti-affinity are added on top of provided affinity rules.
The default config is specified on wiki.

As part of this work I will make the CR more generic by adding a whole PodSpec field. This way the pods are completely customisable and the only special case we need to handle are the affinity rules. We need a particular configuration in order to ensure HA over AZs.
However, it might be the case that not all users need this, so I will set a default behind a flag 'enableAZDistrbution' that makes sure soft rules for leader/node anti-affinity are added on top of provided affinity rules.
The default config is specified on wiki.


Implement page stores

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/7496/
Test FAILed.

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/7521/
Test FAILed.

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/7522/
Test FAILed.

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/7583/
Test FAILed.

alluxio-bot, merge this please


Angular packages won't install

Describe the bug
We have encountered a bug with Angular on Stackblitz when connecting to our github repository. It seems that some of our dependencies are not installed, thus producing errors.
Link to the blitz that caused the error
https://stackblitz.com/edit/stackblitz-starters-akfzl3?file=README.md
Steps to reproduce

Go to the stackblitz project
On loading the project the following error will produce:

Error in src/queries/getArticles.ts (1:21)
Cannot find module 'apollo-angular' or its corresponding type declarations.

Expected behavior
I would expect all the dependencies to be installed automatically, like is done with other frontend frameworks like react.
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform

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

Additional context
No response

Hey @prepr-io 👋 !
The problem here is that you probably started from a classic StackBlitz project, and not from a WebContainer based project. Classic projects (if I'm not mistaken) don't look for dependencies in the package.json file, only in the list of dependencies on the left.

So in order to fix that issue, you probably need to add the dependencies to the list on the left hand side. Or try to start from a WebContainer based project (https://node.new) and connect your repository again.


fixes delay on initial spinner firing

Ran into the same issue documented in #1. Calling animateSpinner() right away onAppear

@james-ingold
First of all, thank you so much for your contribution.
Change looks good to me and it works fine, I'm merging 😁


How to improve agent performance

I test JMeter 3.0 & nGrinder 3.4.1.
Hardware is 4C4G
use the same JDK 8, same JVM params
Both Controller use -Xmx16g -Xms16g -Xmn12g
JMeter Agent use -Xmx3g -Xms3g -Xmn2g
nGrinder Agent use default JVM params
test with the same REST API
and with the same QPS
but I found JMeter performed better than nGrinder:
When threads count is 500,JMeter is single process, nGrinder Agent 4 processes, 125 threads per processs.
JMeter Agent cpu usage percent is 152%, nGrinder Agent is 220%(55%*4)
When threads count is 1000, nGrinder Agent 4 processes, 250 threads per process.
JMeter Agent cpu usage is 330%(load is about 1.3), nGrinder is 340%, but has higher load(near 3.3)
======
I like nGrinder design pattern, hope to improve performance, thx

I found similar issue and hope to improve it:
On the same 4C16Gb virtual Linux machine, JMeter can create 12,000 concurrent RPS, but nGrinder's agent only can up to 3000!
I really want someone to improve it, please

What's RPS?

RPS is like QPS, which means it can generate 12000 access pressure to tested service.

Well.. A u make a HTTPS request or HTTP?

http is OK, but both will be better

This is due to underlying grinder nature.
grinder always disconnects connection when the test is repeated.
So that, the https handshaking always occurs in every test and it consumes the CPU power.
The easiest way to workaround this is sth like following.
@BeforeProcess
public static void beforeProcess() {
   ...
  test.record("doRealTest")
}

@Test
public void test() {
  for (int i = 0; i < 100; i++) {
     doRealTest()
  }
}

public void doRealTest() {
   // do test here..
}

This will make the connection is alive during test() is executed and the TPS will be increased whenever the doRealTest() is executed.


Improve how-to info for the examples

I don't want to double up on this info by having it both here and in the ES dev guide. GitHub should be The Place. The instructions are pretty good right now, but assume more knowledge than I'm comfortable with. I would like to either beef up the README or, if Justin and Charles prefer to keep the landing page minimal, create a project wiki page with more detailed info.
What I perceive as currently lacking:

A high level description of each example that enables the user to choose which one to explore for which purpose. There are descriptions in each example dir, but I shouldn't have to go tree climbing to grok the difference between them. This absolutely belongs in README.md.
More info about configuration. What, if anything, you HAVE to change. What you're most likely to choose to change (host, port, admin password). If there are gotchas about changing properties, what are they? A few words about the users that get created.
Required software. Again, belongs in README.md. I think maybe this is just ML and a Java 8 JRE, but we should say so.

I'm assigning this to myself, but tagging @grechaw and @jmakeig for any additional thoughts.

The README in the examples dir tells you to look at the info in each example dir for instructions on how to run it. And those READMEs say something like run "./gradlew runExamplePerson". That would all fine if only you didn't need to run that command from the parent dir. This is silly experience, even if the directions were correct:

go down into the example dir
read the instructions
go back up to the parent example and run the command

The parent README should either go ahead and tell you the name of the gradle task for each example, or it should be possible to run the example from its own dir. (Would "../gradlew blah" work?)

The little bit about code gen at the bottom is great, but a little confusing out of context. I'd like to reword it a bit, and correct a couple typos.

@kcoleman-marklogic here's another issue that maybe can be dispensed with.  I think we addressed.

I think I took care of most of this for 9.0-3, but let me dig up the bugtrack version of this and see.

I would not say we addressed it. AFAICT, the only thing that's been addressed is the bit about running the examples from the parent dir. I think everything I said in the original description still applies.
However, I can't claim this is a priority for me right now, either. I'm going to change the milestone. If you'd rather just close it, that is fine, too.


FAB buttons should move when a toast message is shown

on Ionic v2, when FAB button is displayed and toast message is shown, the FAB button must be animated and move at the same time the toast message does. Right now, the toast message is over the FAB button.
Ref: https://material.google.com/components/snackbars-toasts.html#snackbars-toasts-usage

Hello! Thanks for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to driftyco/ionic-feature-requests#152

I am curious, what is the status on this?

I have implemented a temporary solution. It is an Angular / Ionic 3 directive that allows Ionic toasts to animate the fab buttons and other elements in the fixed-content div of a page accordingly.
All you have to do is to include the directive in your project and then add a corresponding element in your app component template:
<ng-container move-fixed-content-with-toast></ng-container>
Please check out the code here:
https://gist.github.com/maumchaves/d7270b54afd5c8ae120f34b41436a119


iOS: All user dialogs become invisible in iOS 9 after opening and closing a QLPreview! Help?

Hey there @aritchie! I've come across a new issue in our app that coincides with iOS 9. All of the user dialogs in the app work perfectly up until a QuickLook preview window (displaying a PDF) is opened. As soon as you close the preview and continue using the app, the dialogs are no longer visible! They aren't rendered in any way but are still active and fully functional, I can actually tap on the invisible ActionSheets and they'll act as intended. I realized I was using the deprecated version of the user dialogs, and moved to using this NuGet package but the issue has not been resolved. Can you offer any insight as to what is going on?
Thanks in advance!

@mikey-mike please provide a full test-case/gist with your issue and I'll take a look

Never mind! Ended up having a lot of compilation issues with the build host along with trouble trying to update all of the NuGet packages. Those were resolved by reverting to a previous version of the app, manually going into the csproj and cleaning up duplicate asset catalogs, and then removing the old NuGet packages prior to using the new ones. You can mark this issue as resolved!


ImportError: cannot import name 'chatbot'

Got this:

from chatterbot import chatbot
Traceback (most recent call last):
File "", line 1, in
ImportError: cannot import name 'chatbot'

I'm using python 3.6.2. The problem occured on my friend's computer, too.

Did you installed chatterbot from pip install -U chatterbot or git cloned version
If yes, which version of chatterbot are you using , to know python -m chatterbot --version

The class name is ChatBot not chatbot

Thanks. Solved.

I'm having this issue to. I copy/pasted 'pip install chatterbot' from git, then attempted to run python -m chatterbot --version but I still get cannot import name ChatBot. Tried the test document to see if things are working and still the same error. I have a fresh install of linux, but I don't see any dependencies so I don't think Im missing anything.  Any thoughts on this would be appreciated.

@mikegustafson101 pip install chatterbot did you got any errors?

I'll have to reboot into linux to doublecheck, but Im 99% sure it installed correctly, and that's why I thought it was confusing that I couldn't even get the test script to run.


dotnet new project templates

Right now, there are project templates available for Visual Studio (https://marketplace.visualstudio.com/items?itemName=BotBuilder.botbuilderv4).
While nice, these only work for Visual Studio and on Windows. To support macOS and Linux users, having dotnet new templates instead would be much appreciated.

I also need this, as I'm using macOS and really want to leverage the bot framework templates for my development.  In the world of Data & A.I. where cross platform thrives, this would really make so much sense.


Fix for expose ports issue in UI

Fixes #797
Signed-off-by: Tom Barlow<EMAIL_ADDRESS>

lgtm thx!


cliprdr/server: Don't call CloseHandle on ChannelEvent

ChannelEvent is queried with WTSVirtualChannelQuery and calling CloseHandle on it will result in  crashes/exceptions.

@mfleisz
I think this is going to be a handle leak, so instead of just remove the CloseHandle call, we should move these lines of codes to function cliprdr_server_context_free() in order to make sure that the handle is released before the context memory is freed.

@mfleisz
Another off-topic matter I spotted: the CliprdrServerPrivate struct is allocated inside cliprdr_server_context_new() but is not freed in cliprdr_server_context_free() (only the context struct is freed, but the private struct is not). It looks like this is another memory leak.

@llyzs I'm not quite sure about the handle leak ... I'm not an expert with channel code but I just took a look at the other channels and they all never CloseHandle the handle that is returned by WTSVirtualChannelQuery(WTSVirtualEventHandle). So either they all leak that handle or cliprdr incorrectly closes it (in fact the value seems still ok - not like free'd memory but Windows throws an exception that the value doesn't point to a valid handle).
Of course you are right about the private struct (context->handle) leak!

@mfleisz
Ah, you are right, my mistake. The channel event handle is managed by VCM and should not be closed by the channel implementation. Looks good now!


Add a secondary value to the progress bar

This PR adds a secondaryValue prop to the progress bar that is displayed below the primary value. If the primary value surpasses the secondary value it would be hidden

Nice one 👍


Debian package description is cut off when installing on Ubuntu

Not sure whether there's a limit in the Ubuntu Software Center displayed length, or if it's getting trimmed earlier in the process.

If it's possible to change the name of the package from ka-lite to KA Lite that might be worthwhile as well?

Weird, your version to be putting the description below the name, while mine does this:

I don't know exactly how to get the name displayed correctly, it might be some meta data missing in the .deb package.

Alpha3:


iFacialMocap Support

Adds network tracking support via iFacialMocap. Do not merge yet as incomplete and will cause conflict with OpenSeeFace functions.
TODO:

[ ] Clean up code
[ ] Replace network tracking button with dropdown setting
[ ] Implement separate library for iFC mapping

https://discordapp.com/channels/853476898071117865/855985791984730112/888612851508461569


Implement celo_getLogs path for /call endpoint

First pass implementation of the celo_getLogs path for the /call Rosetta endpoint. This is mostly a passthrough to ethclient.FilterLogs with contract registry parsing on top.
Let me know if this interface makes sense or if it could be improved
cc: @yorhodes

@yorhodes I changed things around to look more like the Call Method API. There's a bit of similar (but not completely duplicated code) here, not sure if there's a great way to clean it up. Let me know what you think


feat: Replace fd-slicer dependency with fd-slicer2 fork

pick from https://github.com/thejoshwolfe/yauzl/pull/115

@SocketSecurity ignore<EMAIL_ADDRESS>


Update idc-isle-dc to reflect migration ID/reference updates

Update migration definitions, test data, etc to reflect the results of jhu-idc/iDC-general#334.

This is a necessary part of the migration work, so I'm assigning it to myself.

Addressed by https://github.com/jhu-idc/idc-isle-dc/pull/136


Polyfill does not satisfy complexity requirements

ES6 spec says:

Map object must be implemented using either hash tables or other mechanisms that, on average, provide access times that are sublinear on the number of elements in the collection.

The polyfill in this module does not satisfy this requirement, as it uses a linear-time probe into an array for get, has, and delete.

I see this is expected behavior per #3. I agree with the commentators in that issue that this is an important caveat that should be documented, but I'll close this as a duplicate.

@jfirebaugh thanks for understanding, it'll also be fixed with #2


increasing the service name length to 63 to reflect the updated limits in kubernetes v1.4.0

#206
https://github.com/kubernetes/kubernetes/issues/12463
https://github.com/kubernetes/kubernetes/pull/25041
https://github.com/kubernetes/kubernetes/pull/28005

Hi there, we were having issues with the plugin truncating service names and I noticed this was fixed in Kubernetes v1.4.0-alpha.3 (https://github.com/kubernetes/kubernetes/pull/29523) so it seems like there is no reason to keep that limited to 24 characters anymore.

Thanks ! looks good to me
[merge]


Basic Support for Camel

Let's try out camel and see how compatible it is with the DSC.

In consultation with @oxisto, the issue was closed until a clear concept was developed. In general, Camel is already supported by the DSC, see: https://github.com/International-Data-Spaces-Association/DataspaceConnector/wiki/Using-Camel


access log: add response code details to the access log formatter

Description: add formatting for the "response code details" string recently added to the StreamInfo (#6530)
Risk Level: low
Testing: unit tests
Docs Changes: updated
Release Notes: updated
Signed-off-by: Elisha Ziskind<EMAIL_ADDRESS>

@eziskind drive by comment: would it be possible to also add this so the gRPC access log? In general we try to keep parity here. Thanks so much for adding this.

+1 on gRPC access logger parity

@eziskind drive by comment: would it be possible to also add this so the gRPC access log? In general we try to keep parity here. Thanks so much for adding this.

Done

/retest


Added animateToNavigation method to MapView

animateToNavigation is a method to allow navigation-like moviment to a map that, for instance, follows the user position. (like google maps navigation or waze).
On android when trying to animateToRegion, animateToBearing and animateToViewingAngle simultaniously the methods would overwride each other's movement. animateToNavigation solves this problem.
CreateMap method is introduced to solve the react's MapBuilder limitation of key-value pairs.
This was already tested worked on android.

Great contribution! But could you add an example to the project please?

Please undo c819f45 and add an example here please.

Surely! c819f45 undone and added example here.

@GuilhermeEd thanks for this contribution, I tested it on android - it works, but it behaves weird if you set mapPadding. If you want the marker a little bit closer to the bottom, like navigation apps do, for example.

LGTM @rborn @christopherdro 🐽

@juergengunz What a great idea! I'll see what i can do.

Hey guys. I'm having trouble calculating the offset at AirMapView.java, lines 590-605. If you could take a look at my algorithm that would be great. I commented it so you could check it out.

@GuilhermeEd nice! Check out this comment, he made an example for iOS and this seems to work. Maybe you could just adapt the algorithm for android.
https://github.com/react-community/react-native-maps/issues/1726#issuecomment-347593978

@GuilhermeEd what is the offset being used for?
Wouldn't it be better to have this alignment stuff be relative to the map zoom (or lat/long delta)?  Ideally this just looks at what the coordinates of the edge of the screen are if you want to align the user's center point to the bottom of the screen.  Better to ship this without the offset logic for now ?

@EricPKerr The offset should be the distance from the current location to the bottom of the map. For instance, most of the navigation apps out there shows the driver position not at the center of the map, but a little to the bottom, instead.
Your idea is good, tough.
Unfortunely lately i've been overwhelmed with work. Maybe ship it without the offset logic would do the trick for now.

I've been watching this for the past month and a half, super excited for it to land. I think the offset is not super necessary, I'd mostly just love to be able to set the angle and bearing at the same time. Thanks for your work, hope I get to use it soon!

I have already used this functionality in one of my projects and the offset wasn't much of a must-have in this case because most of the time i needed to display some info at the bottom of the screen (which overlapped the bottom part of the map).
Back to the business, how can i submit it without the offset funcionality for now?
Pull > Resolve Conflicts > Pass Travis Build > Pull Request?

@GuilhermeEd yeah 🤗

There you go :)

@GuilhermeEd added few comments 🤗

Gonna be so excited when this lands.

@rborn I've deletes the IDE and .class files as you asked. Do you think there is something else that needs to be done? If you have a good solution for the Map function that we've discussed above that would be great as well.

@rborn Is there anything I can do to get this through?

@alvelig need some help here, can you take a look? 🐽

@rborn I am! Thanks for the reminder! LGTM
@equesteo @brunocascio have you tried the branch? Can you confirm that it's working ok for you? We'd be very grateful if you helped us with testing this PR.

@alvelig Tested and works for me.

Can i pass a object with all my pins locations to it? like in fitToCoordinates(objectWithLocations) ?

How to focus on the current location while using animateToNavigation? In the example also map position is fixed. Can anyone help?

I've been waiting on this functionality, I even wrote my own version for Android but didn't get around to the iOS code. Thank you so much!
Why is there no zoom parameter though? I know most of the package uses regions which negates the need for zoom, but this function just uses a center. It's just one number passed down to the GoogleMap API.
@ayushnawani It's up to you to supply the lat/long of where you want the map to be. It sounds like you need to access the GPS of your device and then pass the location results to the function.


[Refactoring] Merge identical cases of switch

This diff doesn't look quite right. I think the changes to delete the ResumeThrowingContinuationReturning case didn't come through.

fixed

ping

Can you do a rebase instead?

Can you do a rebase instead?

done

@swift-ci please smoke test


Unify README action versions

When copy and pasting the README examples, I got the error:
Failure: branch is not main

On CI runs. This was due to using an older version of the action.

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

Good catch!


101 for everyone: fix workflow configuration & licensing concerns

The customization of the scatterplot steps that needs to be done by the
user in the workflow run dialog was described incompletely, which caused
problems in trainings when users selected column 4 both for plotting and
for grouping. The updated version explains in more detail the purpose of
the customization and clearly states which columns need to be selected
for plotting and grouping.
In addition, the "4Cs of diamonds" image got removed from the tutorial
(and is now only linked to) because of licensing concerns.
fixes #1972

@annefou @jennaj does this look good to you?

Thanks!


AUDIO SYNC HELP !!

VERY BASIC
**trying to play multiple audio files simultaneously, but they play back at different times

also it has a pharrell-like 4 count start before playing..**

code:
var playerRate = 1;
function preload() {
song1 = loadSound('assets/melody1.mp3');
song2 = loadSound('assets/melody2.mp3');
song3 = loadSound('assets/melody3mp3');
song4 = loadSound('assets/melody4.mp3');
song5 = loadSound('assets/melody5.mp3');
}
function setup() {
createCanvas(windowWidth, windowHeight, WEBGL);
song1.playMode('restart');
song2.playMode('restart');
song3.playMode('restart');
song4.playMode('restart');
song5.playMode('restart');
}
function draw() {
if (key == '1') {
song1.loop();
song2.loop();
song3.loop();
song4.loop();
song5.loop();

}
}
Please Help
Thank U <3 X O

Hi nomiliveson,
For this scenario, you'll want to take everything out of the draw() loop (or delete the draw loop entirely) and put it in an event listener. Right now, the draw function is continuously re-triggering the loop method on each song at the frame refresh rate. Try making a keypressed() event listener by placing the following code at the very end:
function keyPressed() {
if (key === '1') {
song1.loop();
song2.loop();
song3.loop();
song4.loop();
song5.loop();
}
}

hey @datramt  - thank you
the audio files till play with a delay, however
-they are not syncronised

Hello @nomiliveson,
What browser are you using? (I am using Firefox 58.0.2, p5 v.0.6.0 and p5.sound v3.7.0)
To my ears, your files seem to be in sync, and they play pretty instantaneously when I click a key. I also let the patch play for 30 minutes to see if they would get out of sync over time but they seemed to be still aligned for me.
@datramt attached are a sample of @nomiliveson files
PLYR.zip

@datramt had the right suggestion to interactions out of draw (the animation loop) and into event handlers like keyPressed
Closing for now but feel free to reopen if it's still an issue, @nomiliveson !

closing for now, but feel free to reopen with browser info, @nomiliveson ...very cool sounds btw!

Hey I have the same Issue using Chrome 90.0.4430.93
In Firefox everything is perfectly in Sync but using Chrome there is a slight delay between each sound.

Hi,
I've been testing, uploaded it here https://joex.apps.dj/PLYR
when the page have been playing for a long time the audio begins to sound glitchy, why is that? is it have to do with the buffer? how do I fix it?


updated logic and added check for git longpath setting

Fixes #82
Added additional error output with link to remediation instructions in enterprise-scale repo

/azp run

/azp run


Backport SameSite Cookie changes 7103, 7433, 7442, 7516, 9614 and 9879

Changes look good, but the documentation build is failing - looks like you need to update docs somewhere for the new cookie API

@bbarkley Fixed! Made it all green now.