When switch is turned off manually in the home app, but its status should be on, it is not updating until the status response changes or a getOn is called at home app refresh

The issue is that if you turn off a switch while its status response says it should be on, the switch will stay off until the home app is refreshed or the status response from the server changes.
To replicate:

setup a working switch with a valid onUrl & a valid offUrl.
ensure the off URL doesn't actually change the status response from the server when toggled.
Ensure the current status response says the switch should be "on" & that the switch is on in the home app.
Turn the switch off & watch it stay off even when it should come back on based on the response  status pattern.
Refresh the home app to see it update to the proper status, and/or change the status response from the server away from and then back to the proper response for switch status "on". Either will refresh it to be proper.

The expected behavior would be that the switch would turn itself back on basically immediately (at the next polling interval), but this does not happen. The only way to force it to change with the polling interval is to make the status response change and then change back to the expected "on" response for the switch.
I have ideas to fix this:

Prevent caching the status response, possibly leading it to believe it has changed when it gets the next one.
Feed it a fake response, clearing the last response, before grabbing the real response and updating the switch status. I believe this will make the home app think the status changed and then changed back, if the switch is still in the same state as the last status check, causing it to think it needs update the switch status when it normally wouldn't.
Other ideas...

This issue is now resolved. longpoll only updates the response when the response from the server changes, whereas poll updates the response each time the URL is called, even if the data in the response hasn't changed. Switches now work as expected.


Suggest to remove unnecessary dependencies

I'm considering to use this library to mock our custom wrapper around go-github. Would you consider removing the github.com/go-kit/log dependency, e.g., just print using fmt or the std lib logger?
I'm also not sure about the rationale for using the github.com/buger/jsonparser. Could the std lib json parser be used instead?
Why? Well, I've looked at the code, and the logger and json parser is only used by the command line tool. Hence, it is unfortunate that users of the mock part must bring in extra (indirect) dependencies that aren't needed for their use case.
PS: You may be interested in the v2 json-experiment that may come in a future Go release.

Hi @meling , thanks for reaching out.

Would you consider removing the github.com/go-kit/log dependency, e.g., just print using fmt or the std lib logger?

The go-kit/log would be the easiest to get rid of. Using either the native logger is probably good enough.

I'm also not sure about the rationale for using the github.com/buger/jsonparser. Could the std lib json parser be used instead?

The case for the jsonparser is different. It's used to provide a dynamic parsing for the Github OpenApiSchema. Without it, although technically possible, one would have to juggle a multi-level map[string]any and given the structure, that would become messy. I'm happy to review a contributiion if you're interested to give it a shot.

PS: You may be interested in the v2 json-experiment that may come in a future Go release.

I'll keep an eye on it :+1: .

Thanks for the quick reply. Yeah, I thought there might be more to it with the jsonparser. I'm okay with keeping that, then.

Closing this for now. It'll come back if any of these issues become pressing again.


Cow: Watch ritvijewels.com

Cow requests the watch of the watch_keyword ritvijewels\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ritvijewels\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Nick in Charcoal HQ


LuaJIT 5.1: Exceptions are not rethrown through sol::error

In my code, I'm trying to force some sandboxing, one part of this is setting the require function to nil.
To test this, I create an "library script":
local M = {}
function M.foo()
    return "foo"
end
return M

I store this into a file in the test dir, m.lua, and then load it via:
local M= require('m')
assert(M.foo() == "foo");

On Lua 5.2, this throws a sol::error claiming the require function is nil. This is the expected behaviour, however, luajit exceptions are never caught and it throws an error unrecognized.

With a bit more testing, it seems to actually segfault the entire thread, which is odd.
What's happening is that it is throwing ... and sol isn't catching that (source). There's a few options here:

Use LuaJIT's C API to catch the exceptions (link)

Since this is a one-time overhead turn on/off, perhaps, if on LuaJIT, create a function to accept the function? Just thinking out loud

Accept that this is how LuaJIT works vs. Lua and ignore it :-1:

We do (1) already in the latest. The only place we actually throw a sol::error is during the atpanic function. If luajit is failing internally and not properly calling it there's really not anything we can do.

It.. seems to work properly, but I'm using Windows x64. I assume it works properly on other x64-platforms. Support for exceptions, as the tables on the page for extensions/exceptions displays, depends on your platform. I have made it so every function bound through sol has an exception-catching trampoline placed under it that will translate things to a lua error. protected_function also goes to great lengths to ensure that exceptions are caught (as best as they can be through the lua code) and then translated into lua errors onto the stack (which also get filtered by a handler function, if there is one).
The exception wrapping can be turned off with SOL_NO_EXCEPTIONS being defined, but I only just added it and I'm still writing the docs.

If you mean that we should be catching ALL errors, including ones that we can't catch with const char*, sol::error or std::exception, then the answer there is going to have to be "we shouldn't", but even in that case for the c-function trampolines we do anyways. If luajit is directly throwing something as it executes the script and not properly calling atpanic, there's very little we can do to reconcile that.

So this was fixed in the latest edition? That's awesome, let me try it out for myself.
I wasn't sure if it was something you were investigating with your recent changes to exceptions.

I'm going to assume this is fixed and closed it now. If something else goes entirely derelict, re-open it ASAP and I'll get on it (maybe even get a few Vagrant files to spin things up).

I actually can't verify that it's working. :-1:
sol::state lua;
                // should error
                // REQUIRE_THROWS_AS(lua.script(script), sol::error);
                try
                {
                    lua.script("local M = require(\"M\")\n"
                               "assert(M.foo() == \"foo\")\n");
                } catch (sol::error& e)
                {
                    std::cout << "caught sol::error: " << e.what() << std::endl;
                }
                catch (std::runtime_error& e)
                {
                    FAIL("caught std::runtime_error: " << e.what());
                }
                catch (std::exception& e)
                {
                    FAIL("caught std::exception: " << e.what());
                }
                catch (const char* e)
                {
                    FAIL("caught char*: " << e);
                }
                catch (...)
                {
                    try {
                        auto eptr = std::current_exception();
                        if (eptr)
                            std::rethrow_exception(eptr);
                        else
                            FAIL("caught(...)");
                    } catch (std::exception& e)
                    {
                        FAIL("caught (...): " << e.what());
                    }
                }

It fails with caught (...) (no extra message).
SOL_LUA_VERSION=501

Yup, needs to be reopened: https://travis-ci.org/Nava2/sol2/jobs/113445378

RIP, and it fails on a regular linux container too. I'm not really running a linux build, just using g++/clang++ on a Windows machine... buh. I'll have to install an actual 64-bit linux at some point.

I've been doing development against all three, but recently, since making this Vagrantfile, I've been using the gcc49 machine more and more.

So, this is a tad troubling, but I'm not sure what I can do about it.
I've wrapped every function call in a trampoline, and even protected_function features additional exception catching guarantees to ensure that luajit doesn't behave like this. The problem is, however, there's no way for me to force luajit to play nice: even after by-default assigning an exception-trapping handler using luajit's API and throwing through the panic function, luajit just doesn't rightly care on certain platforms (like Windows x86 and friends).
This is not something that's really within our control, unfortunately. Unless you can spot out any bright ideas, this seems like one of those "Well, it's your burden for using luajit."

Does the error reproduce on newer version so LuaJIT? I haven't tried, but I will now.
I fear it might come down to, as you said, "known problems." I'm surprised about this since the site does say that it should work as we expect on Linux. It's supposed to have full interoperability with C++ exceptions on Linux.

I'm using 2.0.4, which last I checked is the latest.

Yeah, I just verified as well. I'm not sure what to say about that. It stinks, but it is definitely able to be written around.

Guess we just file bugs in luajit and pray for the best?

I think it's more of a "report to mailing list," but yeah. I think there needs to be a test case created without sol first, though.

Unfortunately, we'll have to close this. :( I'll be sure to throw over a issue on the LuaJit Github, which is apparently the active place of development.

Sounds good to me.. unfortunately. Where is the repository?

One solution is to have exception-throwing C++ functions wrapped in functions that store std::current_exception() in a userdata and throw it as a Lua error, which the Sol API special-cases.

The reason LuaJIT has issues is that, unlike PUC-Lua, LuaJIT is not written in the common subset of C and C++, but rather in C + assembler.  LuaJIT must therefore handle unwinding manually, hence the lack of portability.

There's nothing we can do about this. If we turn off exceptions for LuaJIT and then it hits itsown error and calls atpanic and the atpanic throws, LuaJIT will not respect the thrown error on most platforms. If we simply return the error message or the exception in a userdata, it doesn't matter because LuaJIT -- like vanilla Lua -- will call abort once the atpanic function finishes and the user does not lngjmp or throw their way out of it.

@ThePhD
Thank you for your quick reply. I read both LuaJIT and sol documentation, but I had a hope that I misunderstood.

For what it's worth, you can define SOL_NO_EXCEPTIONS 1 and just let your exceptions tear through the stack without any interference from sol. If it works out, well, have fun.

@ThePhD
Seems to be solved by setting LJ_UNWIND_EXT to 0, but thanks for the help anyway


Add README.md to random_walk package for documentation

Add simple documentation describing all the parameters, how it generates a random walk, etc.

Added description of random walk and parameters to the package README

Awesome!
Can we format it like we've done for trajectory controller? Can format it like a table, and also add services, subscriptions, and publications. Thanks!


Phase exception during MoveEntityEvent.Teleport if closing a player's inventory while holding item in cursor

If during a MoveEntityEvent.Teleport event, a player is holding an item on their cursor and player.closeInventory() is called, a phase exception is thrown.
Sample code to reproduce issue:
@Listener
public void onTest(MessageChannelEvent.Chat event, @Root Player player) {
	// When player writes "!test" in chat
	if (!event.getRawMessage().toPlain().equalsIgnoreCase("!test")) return;

	Inventory inventory = Inventory.builder()
			.of(InventoryArchetypes.CHEST)
			.build(plugin);

	// Put a diamond in the inventory
	inventory.offer(ItemStack.of(ItemTypes.DIAMOND, 1));

	player.openInventory(inventory);

	// Few seconds later, teleport the player to trigger the teleport event
	Task.builder()
			.delayTicks(20 * 3)
			.execute(() -> player.setLocation(player.getLocation().add(0, 2, 0)))
			.submit(plugin);
}

@Listener
public void onPlayerTeleport(MoveEntityEvent.Teleport event, @Getter("getTargetEntity") Player player) {
        // Close the inventory to trigger the phase exception
	player.closeInventory();
}

Reproduce instructions:

Write !test in chat
Quickly click on the diamond to put it on your cursor
After 3 seconds you will be teleported into the air and your inventory will close.
Diamond will disapear and an exception is thrown

Exception log:
https://gist.github.com/VapidLinus/d4a4226444874b1c49a18f6b3e2468d9

spongevanilla-1.12.2-8.0.0-BETA-393

@RedNesto After doing some tests I've come to the conclusion that I was wrong about items not dropping (in other cases; in this issue the item wasn't being dropped but that is now fixed). I was just not considering that the item is dropped before the player gets teleported, so the item gets dropped where the player was before the teleport :)


time_widget: Make the generated time string more readable.

Before: time:2021-07-14T00:00:14:00-07
After: time:2021-07-14|00:14:00|UTC-07:00
Fixes #19205
GIFs or screenshots:

@ganpa3 this looks great!  Do you want to try looking at whether using the browser's native time picker with some CSS could provider a better date picker experience to replace flatpickr as a follow-up?  I think
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/datetime-local
looks a lot nicer, though we'd need a polyfill for Firefox support; https://www.npmjs.com/package/datetime-local-polyfill might work.

@timabbott Sure. I will look into it.

Merged, thanks @ganpa3!

@ganpa3 this looks great! Do you want to try looking at whether using the browser's native time picker with some CSS could provider a better date picker experience to replace flatpickr as a follow-up? I think
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/datetime-local
looks a lot nicer, though we'd need a polyfill for Firefox support; https://www.npmjs.com/package/datetime-local-polyfill might work.

@timabbott So, I tried this and I'm not sure how we could get this.
I tried the naive way of appending the datetime-local input and clicking it programmatically, but that works only on Chromium and its clones on mobile devices. It doesn't work on Firefox (mobile and desktop) and Chromium desktop.
I couldn't test on Safari, but from what I read, it doesn't work well on Safari.


fix(Controls): replace lever to use rotator track

The lever now uses the Rotator_Track as the Track_Object grab mechanic doesn't work as well for moving the lever.

There were the following issues with your Pull Request

Commit: a335c761ce40195f0064d6229e4848e9a90453c8
Your commit message body contains a line that is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com


Ruff configuration

Currently ruff is set-up to replicate the flake8 guidelines, which could be extended.
I'm creating this issue to keep track of the different configuration options and respective branches.
In progress:

[ ] docstring rules #612 and #685
[ ] pytest rules #692
[x] "FIX": Code should not contain FIXME, TODO, etc. #687
[x] "ERA": No commented out code #686
[x] "NPY": Check all numpy related deprecations #688
[x] "COM": trailing comma rules. Can be very helpful to avoid accidental tuple creation and other problems. - COM819?  #689
[x] "A": Avoid builtin function and type shadowing. #690
[x] "I002": missing-required-import  #691
[x] "TID252": Use absolute over relative imports.  #691
[x] "ARG": Remove unused function/method arguments. #697
[ ] "B": flake8-bugbear extension. Contains a number of guidelines (seem to be quite opinionated, but at least some look very reasonable) https://docs.astral.sh/ruff/rules/#flake8-bugbear-b  - without B006, B008 #698
[x] "C4": flake8-comprehensions #699

Some of them are very easy to apply and already mostly fulfilled in pyfar. For example, builtin shadowing was something we tried to avoid (although not 100% successful as it seems :) )
Some may even be auto-fixed., others require more changes and additional care.
Feel free to suggest additional rules or rules (or respective subsets) which should not be enforced.
Update: We agreed on all options except for "B" and will discuss options from @ahms5 at a later point but want to have "C90" and "C4" and "PT011" mentioned in #613 (as a separate pull)

note on in progress:

[ ] docstring rules https://github.com/pyfar/pyfar/pull/612

    "D100", # Missing docstring in public module
    "D101", # Missing docstring in public class
    "D102", # Missing docstring in public method
    "D103", # Missing docstring in public function
    "D104", # Missing docstring in public package
    "D105", # Missing docstring in magic method
    "D106", # Missing docstring in public nested class

[ ] also docstring https://github.com/pyfar/pyfar/pull/614

    "D206", # Docstring should be indented with spaces, not tabs
    "D207", # Docstring is under-indented
    "D208", # Docstring is over-indented
    "D407", # Missing section underline
    "D408", # Section underline should be in the line following the section's name
    "D409", # Section underline should match the length of its name
    "D410", # Missing blank line after section
    "D411", # Missing blank line before section
    "D412", # No blank lines allowed between a section and its content
    "D414", # Section has no content
    "D419", # Docstring is empty

and i would also add the the discussion:

"ARG" check for unused arguments in functions methods, classes, etc
"C4" flake8-comprehensions
"I" sort import, might be good to also uniform our imports and make them properly
"C90"  Checks for functions with a high McCabe complexity.
"N" naming conventions (maybe a subset)

we decided to add the following rules:
In progress:

[ ] docstring rules #614 #612, add D415, D417
[ ] pytest rules #613
[ ] "FIX": Code should not contain FIXME, TODO, etc.
[ ] "ERA": No commented out code
[ ] "NPY": Check all numpy related deprecations
[ ] "COM": trailing comma rules. Can be very helpful to avoid accidental tuple creation and other problems. - COM819?
[ ] "A": Avoid builtin function and type shadowing.
[ ] "I002": missing-required-import
[ ] "TID252": Use absolute over relative imports.
[ ] "ARG": Remove unused function/method arguments.
[ ] "B": flake8-bugbear extension. Contains a number of guidelines (seem to be quite opinionated, but at least some look very reasonable) https://docs.astral.sh/ruff/rules/#flake8-bugbear-b  - without B006, B008
[ ] "C4": flake8-comprehensions

i guess its done


CLI need to support run single DF application

CLI need to support run single DF application meaning both rest and ui are in the same host and same port with different url
after #21

This issue was moved to datafibers-community/df_data_service#1

This issue was moved to datafibers-community/df_data_service#3


unable to run the command gem install cocoapods-playgrounds

I keep having an error when I want to run the command gem install cocoapods-playgrounds. Here is the error that I get :  ERROR:  While executing gem ... (Gem::FilePermissionError)
You don't have write permissions for the /Library/Ruby/Gems/2.0.0 directory.`

I also tried to run sudo gem install cocoapods-playgrounds and here is the error that I get:
ERROR:  While executing gem ... (Errno::EPERM)  Operation not permitted - /usr/bin/carthage-play

I was having the same issue and was able to get the gem installed in a local/bin using this command:
sudo gem install cocoapods-playgrounds -n/usr/local/bin
I found this work around in this StackOverflow question:
http://stackoverflow.com/questions/31972968/cant-install-gems-on-os-x-el-capitan

Yep, this is entirely an OS X 10.11 issue, sorry.


EF Core 1.1 .ToString() translation causes exception

Steps to reproduce
Create a basic asp.net application for one of the controllers just insert the query code below with the model provided. As soon as the query is run the exception takes place
The issue
The following query works in .net core 1.0.1 but doesn't work in 1.1
Model
 public class Supplier
    {
        [Key]
        public Guid Id { get; set; }

        [Required]
        public string Name { get; set; }

        [EmailAddress]
        public string Email { get; set; }

    }

Query
var supplierList = _context.Suppliers.OrderBy(x => x.Name).Select(x => new SelectListItem
{
   Text = x.Name,
   Value = x.Id.ToString()
}).ToList();

If you are seeing an exception, include the full exceptions details (message and stack trace).
Exception message:
NotSupportedException: TryExpression is not supported as a child expression when accessing a member on type 'System.Guid' because it is a value type. Construct the tree so the TryExpression is not nested inside of this expression.

System.NotSupportedException: TryExpression is not supported as a child expression when accessing a member on type 'System.Guid' because it is a value type. Construct the tree so the TryExpression is not nested inside of this expression.

   at System.Linq.Expressions.Compiler.StackSpiller.RequireNotRefInstance(Expression instance)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteMethodCallExpression(Expression expr, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteExpression(Expression node, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.MemberAssignmentRewriter..ctor(MemberAssignment binding, StackSpiller spiller, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.BindingRewriter.Create(MemberBinding binding, StackSpiller spiller, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteMemberInitExpression(Expression expr, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteExpression(Expression node, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteExpressionFreeTemps(Expression expression, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.Rewrite[T](Expression`1 lambda)

   at System.Linq.Expressions.Expression`1.Accept(StackSpiller spiller)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteLambdaExpression(Expression expr, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteExpression(Expression node, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.ChildRewriter.Add(Expression node)

   at System.Linq.Expressions.Compiler.StackSpiller.ChildRewriter.AddArguments(IArgumentProvider expressions)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteMethodCallExpression(Expression expr, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteExpression(Expression node, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.ChildRewriter.Add(Expression node)

   at System.Linq.Expressions.Compiler.StackSpiller.ChildRewriter.AddArguments(IArgumentProvider expressions)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteMethodCallExpression(Expression expr, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteExpression(Expression node, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.RewriteExpressionFreeTemps(Expression expression, Stack stack)

   at System.Linq.Expressions.Compiler.StackSpiller.Rewrite[T](Expression`1 lambda)

   at System.Linq.Expressions.Expression`1.Accept(StackSpiller spiller)

   at System.Linq.Expressions.Compiler.LambdaCompiler.Compile(LambdaExpression lambda)

   at System.Linq.Expressions.Expression`1.Compile(Boolean preferInterpretation)

   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.CreateExecutorLambda[TResults]()

   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.CreateAsyncQueryExecutor[TResult](QueryModel queryModel)

   at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQueryCore[TFunc](Object cacheKey, Func`1 compiler)

   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.ExecuteAsync[TResult](Expression query)

   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1.System.Collections.Generic.IAsyncEnumerable<TResult>.GetEnumerator()

   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.<ToListAsync>d__129`1.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()

   at CarTracker.Controllers.AccountController.<Register>d__10.MoveNext() in D:\Programming\Projects\Personal\CarTracker\src\CarTracker\Controllers\AccountController.cs:line 122

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeActionMethodAsync>d__27.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextActionFilterAsync>d__25.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ActionExecutedContext context)

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextResourceFilter>d__22.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ResourceExecutedContext context)

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)

   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeAsync>d__20.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.VisualStudio.Web.BrowserLink.Runtime.BrowserLinkMiddleware.<ExecuteWithFilter>d__7.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.MigrationsEndPointMiddleware.<Invoke>d__5.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.DatabaseErrorPageMiddleware.<Invoke>d__6.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.DatabaseErrorPageMiddleware.<Invoke>d__6.MoveNext()

--- End of stack trace from previous location where exception was thrown ---

   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.<Invoke>d__7.MoveNext()

Further technical details
EF Core version:
Microsoft.EntityFrameworkCore.SqlServer": "1.1.0
Operating system: Windows 10 x64
Visual Studio version: VS 2015 Ultimate
SQL: LocalDb 2016
Other details about my project setup:
ASP.NET core 1.1 application

@dotnetshadow Thanks for reporting. I'm still investigating, but it looks like the following workaround might work:
var supplierList = _context.Suppliers.OrderBy(x => x.Name).Select(x => new SelectListItem
{
    Value = x.Id.ToString(), // flip the order of these properties
    Text = x.Name
}).ToList();

Can you confirm it this works?

I have a same issue when trying to get ToString() or GetHashCode() just after  updating to 1.1
 return result.Select(p => new SearchResultItem
            {
                Description = p.Description,
                Title = $"{p.Title} ({p.TitleEN})",
                Action = "Product",
                Controller = "Guide",
                SourceType = Models.Enums.SearchResultItemType.Product,
                RouteParameter = new Dictionary<string, string>(){
                    {"productID", p.ProductID.ToString()}},
                HashCode =  p.ProductID.GetHashCode()
            }).ToList<SearchResultItem>();

Possible issue in corefx (#13864)

@anpete I tried your suggestion but it didn't work

I was looking at it and you need to pass the string as a literal or iNumerable perhaps.

@anpete was the fix pushed to the 1.1.1 branch? should this be open waiting approval?

Yes, re-opened.

This patch bug is approved. Please use the normal code review process w/ a PR and make sure the fix is in the correct branch, then close the bug and mark it as done.

@anpete closing again since this is approved and the fix is in. please reopen if necessary.

✅ Verified

I'm still getting error:
Critical part of linq query (T1Score is type: int?):
strT1Score = (s.T1Score != null) ? s.T1Score.ToString() : "?"
From project.json:
"Microsoft.AspNetCore.Mvc.Core": "1.1.1",

I also get the same error:
public IList<SelectListDto> GetMatchSections(Guid matchId)
{
	return Items.Where(x => x.MatchId == matchId).Include(s => s.Section).Select(s => new SelectListDto
	{
		Text = s.Section.Name,
		Value = s.Section.Id.ToString(),
	}).ToList();
}

@toddtsic - You need EF packages to be 1.1.1. MVC patches have been bit ahead in versioning. EF patches are still not released on nuget. They will be there soon.

Yup, within a week or two they'll be on NuGet.org.


upgrade command

Maybe in the future: https://github.com/software-mansion/protostar/issues/54

Codecov Report

Merging #53 (91ff91b) into master (8e1e67e) will decrease coverage by 5.25%.
The diff coverage is 36.95%.

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


A formula for the terminal Space Invaders clone posted today on HN.

HN article: https://news.ycombinator.com/item?id=10078928
Written in Golang, this is a completely ASCII art implementation of
Space Invaders which works in the terminal.  Compatible with any POSIX
compliant system that can run Go.

Not sure if this will pass your automated testing.  There's no test loop in the formula because the application being built/installed does not provide any non-interactive way to invoke it.  It's a shell-based game which can only be invoked interactively at the terminal.

You might be able to get a primitive test going using IO.popen.  Take a look at the datomic formula (https://github.com/Homebrew/homebrew/blob/master/Library/Formula/datomic.rb#L30) or grep around for other examples.

Thanks @dunn, I'll see if I can make that work.  Stand-by for an update to this PR.

I am not a Ruby programmer by any stretch, so maybe I misunderstand the pipe facility for Ruby, but I get the following stacktrace when attempting a test using IO.popen where I do pipe.puts "q", since "q" quits the application.  I have to manually enter "q" after I run the test, as it never pipes it through...
❯ brew test spaceinvaders
Testing homebrew/games/spaceinvaders
Error: homebrew/games/spaceinvaders: failed
not opened for writing
/usr/local/Library/Taps/homebrew/homebrew-games/spaceinvaders.rb:35:in `write'
/usr/local/Library/Taps/homebrew/homebrew-games/spaceinvaders.rb:35:in `puts'
/usr/local/Library/Taps/homebrew/homebrew-games/spaceinvaders.rb:35:in `block (2 levels) in <class:Spaceinvaders>'
/usr/local/Library/Taps/homebrew/homebrew-games/spaceinvaders.rb:34:in `popen'
/usr/local/Library/Taps/homebrew/homebrew-games/spaceinvaders.rb:34:in `block in <class:Spaceinvaders>'
/usr/local/Library/Homebrew/formula.rb:951:in `block in run_test'
/usr/local/Library/Homebrew/extend/fileutils.rb:16:in `mktemp'
/usr/local/Library/Homebrew/formula.rb:947:in `run_test'
/usr/local/Library/Homebrew/test.rb:28:in `block in <main>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/timeout.rb:66:in `timeout'
/usr/local/Library/Homebrew/test.rb:27:in `<main>'

Here's the test:
  test do
    IO.popen("#{bin}/spaceinvaders") do |pipe|
      pipe.puts "q"
      pipe.close_write
      pipe.close
    end
  end

I fixed the pipe where it was previously read-only which is what resulted in the stack trace.  I dug through the Go code of the underlying software and it's just checking for a character event from nsf-termbox (which it imports), which is mapped to a rune.    So the pipe writing a "q" should be enough to trigger the application to quit, but it does not.  Manually hitting "q" results in it quitting without a stacktrace now in the unit test, and reports the test successful, but it does not quit automatically.
I'm at a loss on what could resolve this.  Among other changes, I just updated the formula to release v1.1 from the spaceinvaders Git repo.  Any thoughts @dunn on what I can do going from here?  @asib is the original author of the spaceinvaders game, perhaps he has some ideas as well.
Thanks all for your patience with me, this is my first attempt at writing a formula for Homebrew.

I'm seeing the same behavior, but it's passing on the bot so I think it's OK.

I do wonder, though, if the name of the formula should be tweaked.  It's not THE Space Invaders; maybe space-invaders-go would be more appropriate.  @mistydemeo can override me here, though.

Thanks for the update @dunn.  Sorry I didn't respond sooner.  Can we go ahead and merge this since the bot test passed then?  If you think a rename is necessary that's a quick fix, just let me know.

Also, not sure what the project policy is, but if you'd like I can rebase my changes to a single commit since we have a passing test now.

Yeah, could you rename it, and squash it down to a single commit? Thanks so much!

It's been renamed to spaceinvaders-go and squashed to a single commit.  Please let me know if I need to make any other modifications for acceptance.  Thanks @dunn

Actually, hold on merging.  Looks like a few days ago he released version 1.2.  Let me update the formula to the latest version of upstream and re-verify before we merge.  Might as well get it in one go.  I will layer the v1.2 update on top of the rebase, since those should remain in history as two separate commits.
Thanks.

It's been updated to v1.2.  Had to update it's Go build-deps, including adding a new dependency upstream added for joystick support.  Looks like the testbot might be failing now on this however, although I haven't changed anything about the test parameters and the behavior when manually running "brew test" remains the same.

Looks like a bot hiccup. I queued it up again.

Looks like it worked after you queued the bot again.  Is this now acceptable to merge or do I need to make any other changes?  Thanks.

Looks good, thank you for contributing! Merged in bfa5e172c6bdbcb85a30e860b67e2b0360d77ea0.


Incorrect Command Line Arguments

According to this doc https://docs.microsoft.com/en-us/windows-hardware/manufacture/desktop/windows-setup-command-line-options the correct command line argument for ignoring dismissable warnings is /Compat IgnoreWarning not /compat /ignore warning as specified here in the docs. Also, the same incorrect message is included in the setupdiag.exe, so when the report is generated, it is providing incorrect guidance.

Additionally, line 47 referring to FindMigGatherApplyFailure appears to be incomplete. I don't know what the correct verbiage is to update it.

@AdamGrossTX To our knowledge the entire docset for deployments is being rewritten and thus the PR review might take some time.
@greg-lindsay Would you be able to review the PR that introduces correct command line arguments to Windows Setup Command-Line Options article if appropriate? Thanks.

@greg-lindsay Following up for updates.

@greg-lindsay Any chance with this PR? We'd like to follow-up once more before escalations. Thanks.

FYI this topic is under windows-hardware/manufacture/desktop not windows/deployment and these are quite different content libraries.  I don't write this content.

However I'm 95% sure that this is just a correction of a typo. The space between ignore and warning was probably unintentional.  I'm approving the edit.

Yes, you are correct. It is a typo correction. Then again, knowing that so many readers of the docs.microsoft.com pages use the content literally as it is, without questioning, any typo will seem to be incorrect commands or parameters to them.


Pushed files/folder colour in tree-view

A small feature that doesn't seem to be working:
When I push my files through this built in terminal, my files in Atom's tree-view are still coloured orange as if they haven't been pushed.
When I use my "external" terminal, atom automatically recognises this and reverts the files/folders to original colours.

Please retest again with platformio-atom-ide-terminal 2.9.0 is using node-pty-prebuilt instead unmaintained and outdated pty.js


Erratic NPC vehicle mechanics

if i remember correctly this bug was also part of the original game, wonder if there ever was a fix to this.
https://github.com/user-attachments/assets/a6a5047b-1c37-4974-8bf3-ec2416470b48
bug can be seen at 0:08 mark.

If I remember correctly, this is one of the consequences of the high playtime in the save game.

There are already several mods that fix this, I recommend you look at mixmods.com.br


A few small improvements

3 small improvements :

a wording for frequency term
the default proposed host url to remove what I assume is your configuration value
a default API call timeout to 3s to avoid timeouts at startup

@cnico now I have to enter my IP and password all the time when I want to do some testing 😪😂. But you are right, it is easier for the user's to change it to a meaningful default 👍


SQL Lab Query doesn't get reflected in chart

I go into SQL Lab and create a query. e.g. "SELECT * FROM data_table WHERE id > 10;"
Then I do save the query.
Clicking on 'run'.
On the data view click on 'explore'
Create a char with e.b. Bar Charts
Save chart as "MyChart"

THEN:

Open saved query and change it. e.g. "SELECT * FROM data_table WHERE id >999"
Save changed query
Reopen Chart
Data is same as before
Chart datasource is something like<EMAIL_ADDRESS>--queryname-FYzgGs0CD"

Expected results
I'd expect the chart reflects the changed query and doesn't do an 'in-time-smapshot'
OR
Another way to point to the 'original' query using a kind of pointer logic.
(By the way - changing a Datasource to SQLLab Query seems non supported as well)

@durchgedreht are you using Redis to cache results?

No, I'm running against Clickhouse directly.

To be more precise: I can edit the query in the data source view within the chart and can see the changes are not reflected. I'm wondering why every query should be immutable bound to a chart? This way it's pretty hard to do changes upon the same query. Fixing an error will result in n changes. This seems unwanted as changes in the other data sources are also reflected - so why should SQL queries underneath a chart behave differently?

FYI I identified a bug (or more like an oversight) in the code that I've already fixed locally thanks to this ticket. Will do some more testing and put in PR in the coming days.

Ok, cool! I'll keep an eye to the ticket so I will do some testing when done.

Is this a bug? I thought this was intentional. I agree it's frustrating but I just assumed the "derived" queries created from SqlLab were intentional.


Doesn't report to sauce labs automatically

In the README it says:
4. It reports test result to sauce labs automatically

It doesn't actually report the test status or test name to sauce labs.
When the tests have run sauce labs still marks them with question marks.
I can report myself to sauce labs using a custom command like
https://github.com/saucelabs-sample-test-frameworks/JS-Nightwatch.js/tree/master/custom_commands
Is this intended or am I missing some flag or option somewhere?

@qacwnfq It reports to the saucelabs only if you're using nightwatch-extra. magellan-saucelabs-executor needs test data , especially the selenium sessionid in order to report to the correct saucelabs test run. However such session id only could be retrieved by nightwatch-extra.
thanks for the feedback, I should change the readme to correct statement.

I was using nightwatch-extra. My project is based on the boilerplate. I think my problem is the sessionId.
Using a custom reporter I could extract:
OK. 29 assertions passed. (59.714s)

[ERROR] [Saucelabs Executor] ErrorTypeError: Cannot read property 'sessionId' of undefined
[INFO] [Magellan] (6 / 7) <-- Worker 2 PASS  tests/shop-demo.js @env:safari_7_OS_X_10_9_Desktop|executor:sauce

The only difference between the boilerplate and my project really is drydrock. I removed it since I don't need a mock server. I don't see where drydock helps with preventing this error.
all my settings are here: shop-demo

I think I found the error.
client.end() should not be called by the user in the tests since the example-base-class takes care of ending the tests by calling the super callback.
Still the test results remain a question mark in my sauce lab when I do it like this.

@qacwnfq your shop-demo link is invalid to me.

Yeah, not calling client.end() fixed my issue. Calling it in the nightwatch test, like I used to, ends the selenium test before it can report to saucelabs, which expires the SessionId needed to report to saucelabs. Magellan takes care of these things by itself!
If you're still interested in the code I made a PR it is now in new shop-demo.
Anyways thanks for your work on magellan! It's an awesome project.


ConversationalRetrievalQAChain + Streaming fails with Chat History

From this doc
This fails with streaming
const chain = ConversationalRetrievalQAChain.fromLLM(
        llm,
        vectorStore.asRetriever()
    );

chat_history = [] // This works
chat_history="What is time? time is a precious resource." // This fails
chain.call({ question, chat_history: chat_history });

Gives error
Error in handler Handler, handleLLMNewToken: TypeError: The stream is closing or closed and cannot be written to
Error in handler Handler, handleLLMNewToken: TypeError: The stream is closing or closed and cannot be written to
Error in handler Handler, handleLLMNewToken: TypeError: The stream is closing or closed and cannot be written to
Error in handler Handler, handleLLMNewToken: TypeError: The stream is closing or closed and cannot be written to

Can you provide a bit more context? Seems like the error is happening in your handleLLMNewToken function, could be useful to include your model declaration.
I have ConversationalRetrievalQAChain working with streaming and memory but I had to pass the history in as type ChatMessageHistory() which consist of HumanChatMessage and AIChatMessage

Yeah I figured out that, from
https://github.com/hwchase17/langchainjs/issues/1607#issuecomment-1590971348
but responses are not good now.
On Wed, 28 Jun 2023, 05:16 Emilio, @.***> wrote:

Can you provide a bit more context? Seems like the error is happening in
your handleLLMNewToken function, could be useful to include your model
declaration.
I have ConversationalRetrievalQAChain working with streaming and memory
but I had to pass the history in as type ChatMessageHistory() which consist
of HumanChatMessage and AIChatMessage
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchainjs/issues/1781#issuecomment-1610369794,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJL5U6NIJRZN3VD2FCRGQDLXNNWFVANCNFSM6AAAAAAZVWKB34
.
You are receiving this because you authored the thread.Message ID:
@.***>

If you share your implementation, we can help you better

take a look at this issue (and its PR): https://github.com/hwchase17/langchainjs/issues/1800
I've added an example to the docs which works pretty good.


vim_configurable: Add vi symlink to vim

Motivation for this change
Like our regular vim, this version should have the vi symlink, so that tools starting vi will not fail but start vim.
Moved preInstall before postInstall because of logic and temporal ordering.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @lovek323

Briefly tested. LGTM.

Thanks :)


Bugfix - Docker payload

Made payload map instead of string

@ngmiller @theMagicalKarp any argument to merging this?

looks good +1


One Question

Hi @haiyangwu ,
First, thank you for your support.
I want ask you: will we have IOS swift demo that can connect to https://v3demo.mediasoup.org server?
I saw android demo(https://github.com/haiyangwu/mediasoup-client-android), but can't find ios swift demo anywhere?
This sample (https://github.com/ethand91/mediasoup-ios-client-sample) can't connect because data that send by web socket is not right. It is only fit with sample server (https://github.com/ethand91/mediasoup-sample-server)
If you know, please help.
Thank you very much.

I solved it

I solved it


Infrastructure Manager - Perform load and stress test to verify operator's behaviour under load

Description
We should verify how the operator behaves under load. To increase the stabilisation and reliability of the infrastructure manager, a performance test has to be implemented which verifies common use cases. Goals is to measure regularly our internally defined performance KPIs (benchmarking/load test), verify the limits of the application (stress test) and detect performance critical behaviours before the Infrastructure Manager gets deployed on a productive landscape (no memory leaks etc.).
Acceptance criteria:

[ ] Identify the most relevant use-cases of the Infrastructure Manager

[ ] define input parameters (e.g. execute the test for 100, 1000, and 5000 CRDs)
[ ] specify the execution context/boundaries (how often the use case will be applied in parallel, limits for CPU/RAM consumption, max. execution time per test case etc.)
[ ] share the collected use-cases and the defined boundaries with in the team and collect their feedback

[ ] Learn what is the recommended way of load testing Kubebuilder projects
[ ] Implement the use-cases in a load test using one of the mainstream load testing tools (e.g. Grafana K6). This test has to cover

[ ] the creation of a load test landscape (e.g. by using a local K3d cluster or provisioning a Gardener Cluster) and deployment of a particular Infrastructure Manager version
[ ] ensure metrics of the Infrastructure Manager are recorded during the test execution
[ ] visualisation of the measured metrics in a Dashboard (e.g. Plutono)
[ ] mocks for 3rd party systems to avoid an overload of external systems (e.g. Gardener service)

[ ] Run the load test and increase the amount of (parallel) workers until the application starts to behave unstable/crashing to detect our maximum performance capacities.

[ ] Document test results

[ ] Integrate the load test into the release process to detect critical performance changes between releases

Reasons
Before deploying the operator on production we must know its performance characteristic.

The IM was working on ~420 CRs and we have to still perform tests on ~1000 and 5000~ CRs.

On 24.01.2024 we've faced situation on PROD where it took 141 seconds to Infrastructure-manager to rotate the certificate since Gardener Cluster CR creation. (internal issue reference - no. 5012).

Capturing here a valuable comment from @piotrmiskiewicz, from internal Slack:

In my opinion IM should have something like priorities. If the GardenerCluster is new it should be processed as soon as possible. If the Kubeconfig must be rotated - it can wait few minutes. Please analyze why there is so much GardenerCluster to process, because changing the timeout from 2 to 3 minutes could also be not enough

Yip, we had similar problem of load-peaks in the reconciler. Adding a jitter helped to distribute the load over time (e.g. https://github.com/octo/retry/blob/master/jitter.go)

Decisions taken with @m00g3n and @koala7659
An separate application will be produced that creates given number of Runtime CRs on Kubernetes cluster.
Application details:

application will only create a one fully working Gardener cluster (if shoot name wasn't provided by the user in program parameters),
application will create a RuntimeCRs (user must provide the desired number in parameters),
all of the Runtime CRs will be linked to the same Gardener Shoot (each of Runtime CR will be sharing the same shoot),
application must clean-up the created Runtime CRs after given period of time (value passed in parameters).

Application parameters:

path to Kubeconfig that points to the cluster where we want to create RuntimeCRs (required),
name of the Gardener Shoot (optional),
number of RuntimeCRs to create (required),
time after application will perform auto clean-up.

(COMMENT IN PROGRESS)
The script used to run stress tests is located in PR to internal_team_docu_repo#99.
Following test was performed

Ten gardener clusters were created
Additional 1000 Runtime CRs were created that were pointing to clusters created in previous step (100 Runtime CRs per cluster)
Every 36 minutes 10 Runtime CRs were patched with a new administrators list and newer k8s version. This totals to about 6hours of testing.
All gardener clusters and Runtime CR created in 1) and 2) step are deleted.


Requesting to add Appknox Taxonomy

Requesting to add a new top-level appknox taxonomy

closes #39

CC: @coderpatros @stevespringett
Apologies for mentioning. Wasn't sure if would see notifications :stuck_out_tongue:

Thanks for the ping, I'll check it out


[REVIEW] Add E2E Deep Learning Mortgage Notebook

This notebook, based on the original mortgage xgboost end-to-end example, trains a deep learning
model from scratch to predict 12 month mortgage delinquencies.
Note it requires cudf >= 0.6 and additional python dependencies that are currently not in the RAPIDS demo containers.
Closes #110.

cc @EvenOldridge

@randerzander @robocopnixon how is this notebook review going?  Just want to know if it is backlogged and assistance is needed!

closing.  Added notebook to notebooks extended.

@taureandyernv I'm trying to locate the notebook in the notebooks extended repo but can't seem to find it.  Can you post the link?  I want to link to the correct location in my blog post.


Floating-point arithmetic

A link to the guide "What Every Programmer Should Know About Floating-Point Arithmetic" https://floating-point-gui.de/ seems appropriate, but I am not sure exactly in which episode to put it.

I will take this one since I want to add an optional exercise on comparing floats.


[bug]: 2.4.8 update is still 2.4.7.post1 after update

Is there an existing issue for this problem?

[X] I have searched the existing issues

Operating system
Windows
GPU vendor
Nvidia (CUDA)
GPU model
rtx 3090 ti
GPU VRAM
24
Version number
2.4.7.post1
Browser
firefox 129.0.2
Python dependencies
No response
What happened
after installing 2.4.8 update and restarting invokeai and refreshing the page 2.4.7.post1 is still installed.
What you expected to happen
to have invokeai 2.4.8 installed
How to reproduce the problem
No response
Additional context
No response
Discord username
No response

Same issue happen to me.

im thinking, they just uploaded 2.4.7.post1 and not 2.4.8

good, thanks for sharing

The release workflow hadn't completed fully and the package wasn't published to pypi. It's fixed now.


I follow the guide,but it can't run correctly

It tips like this:
Unable to resolve module react-redux from snowflake/src/snowflake.js:Invalid directiry /Users/node_modules/react-redux

did you
0) rm -rf node_modules

npm install -g npm3
2)npm3 install
3)cp config.example.js config.js
4)edit config to setup your backend
3)react-native run-ios/run-android

Please provide the command line output

Unfortunately it could be relative to https://github.com/facebook/react-native/issues/4968.
Clean installation + copy snowflake source is the "best" solution.

The latest release should resolve: https://github.com/bartonhammond/snowflake/releases/tag/0.1.2-alpha
Please read these notes!


Implement transformation to interchange zero-like constants

#3132
I have implemented the TransformationToggleConstantNull transformation, tests and fuzzer pass, which transforms zero-valued constants of type int, float or bool into null constants of the same type (according to the specification, a null constant of these types is equivalent to a constant with value zero - false for booleans).
Is the code generally ok? Are there other cases that I should have considered?
I chose the upper and lower bounds for the probability of applying the transformation to be 20 and 90 a bit randomly.
I am also not sure if the name is explicative enough.

 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.

In spirv-fuzz there is a "fact manager" that tracks facts about the module being transformed.
One kind of fact is called a "data synonym" fact, and it records that two pieces of data are known to be equal.  In the simplest case - which suffices for this discussion - it tracks that two ids are equal.
There is a transformation called TransformationReplaceIdWithSynonym that takes some use of an id %a, and an id %b that is synonymous with %a, and replaces the use of %a with a use of %b.
For example, there might be an instruction:
%c = OpIAdd %int %a %a ; This is like c = a + a
If the fact manager knows that %a and %b are synonymous (where %b is some other id) then TransformationReplaceIdWithSynonym could say "replace the first use of %a in '%c = OpIAdd %int %a %a' with %b".
That would turn the above instruction into:
%c = OpIAdd %int %b %a ; This is like c = b + a
and this is semantics-preserving if the fact that %a and %b are synonymous really is true.
Now, what does this have to do with interchanging zero-like constants?
Well: we could write a fuzzer pass that does the following:

For each zero-like constant %c

Add an equivalent, but toggled, zero-like constant %d to the module*
Record the fact that %c and %d are synonymous**
For each use of %c
- Probabilistically decide whether to leave %c alone or replace %c with %d***

This will involve using TransformationAddConstantNull, TransformationAddConstantBoolean, or similar

** This will need a new transformation, TransformationRecordSynonymousConstants, which should be applicable if two given constants really are equivalent and should have the effect of adding the fact that they are synonymous to the fact manager - it should not actually change the module
*** This will require applying a TransformationReplaceIdWithSynonym
I suggest you take a look at FuzzerPassApplyIdSynonyms to see some related code.
Let me know if this makes sense - keen to discuss.

I have implemented the fuzzer pass and the transformation as described.
For each zero-like constant %c, the fuzzer pass looks for an existing toggled instruction and adds a new one only if it doesn't find it, otherwise it uses the already-existing one (there were some functions in the fuzzer_pass file that I could reuse to do that).
I was unsure whether the IsApplicable function of the transformation class should return true with two constants that are equal (same opcode and same operands). I opted for also returning true in this case (even though this is not used by the pass) because in my mind it seemed to make more sense this way.
Let me know about any observations or corrections to make.

I have opened a new pull request (#3494 ) that just introduces the transformation class and tests, so I am closing this one.


TimesNet

What happened + What you expected to happen
I have a dataset with 264 time series and weekly data.
I'm trying to use TimesNet to forecast all of them with static and dynamic covariates.
However, I get an error when using the predict function with freq='W' that says **ValueError: futr_df must have one row per id and ds in the forecasting horizon (16).**. I checked and all of them have 16 observations and the ds is also ok, I have 16 consecutive mondays.
Nevertheless, if I change the freq argument to W-MON, the error does not happen but the predictions are NANs...
Any idea how to solve this issue?
Versions / Dependencies
neuralforecast==1.6.4
python 3.10
Reproduction script
just try to forecast weekly data with several time series
Issue Severity
Low: It annoys or frustrates me.

Hey @rjguedes8, thanks for using neuralforecast. Can you provide a reproducible example? The following works fine:
from utilsforecast.data import generate_series
from neuralforecast import NeuralForecast
from neuralforecast.losses.pytorch import DistributionLoss
from neuralforecast.models import TimesNet

data = generate_series(10, freq='W-MON')

model = TimesNet(
    h=4,
    input_size=8,
    hidden_size=16,
    conv_hidden_size = 32,
    loss=DistributionLoss(distribution='Normal', level=[80, 90]),
    learning_rate=1e-3,
    max_steps=5,
    val_check_steps=50,
    early_stop_patience_steps=2
)
nf = NeuralForecast(models=[model], freq='W-MON')
nf.fit(data, val_size=4)
nf.predict()

Hey José, thank you for your quick response!
I tried to run your code and got this error:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[106], line 8
      6 freq = 'W-MON'
      7 h = 4
----> 8 data = generate_series(10, freq=freq, n_static_features=2)
      9 valid = data.groupby('unique_id', observed=True).tail(h)
     10 train = data.drop(valid.index)

File /opt/conda/lib/python3.10/site-packages/utilsforecast/data.py:72, in generate_series(n_series, freq, min_length, max_length, n_static_features, equal_ends, with_trend, static_as_categorical, n_models, level, engine, seed)
     70 available_frequencies = seasonalities.keys()
     71 if freq not in available_frequencies:
---> 72     raise ValueError(
     73         f"Currently soported frequencies are: {available_frequencies}, got {freq}"
     74     )
     76 rng = np.random.RandomState(seed)
     77 series_lengths = rng.randint(min_length, max_length + 1, n_series)

ValueError: Currently soported frequencies are: dict_keys(['D', 'M']), got W-MON

What is the package version that you are using? I'm using 1.6.4

Can you upgrade your utilsforecast version? pip install -U utilsforecast

it's working now, thanks :)


Can't get user credentials/getuid() with frolvlad/alpine-glibc:alpine-3.3_glibc-2.22

Hi, I was using frolvlad/alpine-glibc and it worked OK with my testing script, but it isn't working properly with new frolvlad/alpine-glibc:alpine-3.3_glibc-2.22.
The test script is written in Go and tests C bindings, zlib compress and CGO (by calling getuid() with user.Current() from Go). I am not cross-compiling.
Bellow I show you how to reproduce the error:
main.go (test script)
package main

import (
	"bytes"
	"compress/zlib"
	"fmt"
	"io"
	"os"
	"os/user"
)

/*
char* foo(void) { return "Hello world - from the good old C"; }
*/
import "C"

func main() {
	fmt.Println(C.GoString(C.foo()))

	var b bytes.Buffer
	w := zlib.NewWriter(&b)
	w.Write([]byte("Hello world - compressed\n"))
	w.Close()
	r, err := zlib.NewReader(&b)
	check(err)
	io.Copy(os.Stdout, r)
	r.Close()

	user, err := user.Current()
	check(err)
	fmt.Printf("Olar, I am %v (%v:%v) - from Go app\n", user.Username, user.Uid, user.Gid)
}

func check(err error) {
	if err != nil {
		fmt.Println(err)
		os.Exit(1)
	}
}

Builder container (to compile Go binaries)
$ docker run -it --rm -v $(pwd):/go/src/main golang:1.5.3
root@go-container:/go# cd src/main
root@go-container:/go/src/main# ls
main.go
Build dynamic
root@go-container:/go/src/main# go build
root@go-container:/go/src/main# mv main main.simple
Build static
root@go-container:/go/src/main# go build --ldflags='-extldflags "-static"'
root@go-container:/go/src/main# mv main main.static
Prep for upx
root@go-container:/go/src/main# apt-get update
root@go-container:/go/src/main# apt-get install -y upx-ucl
root@go-container:/go/src/main# go get github.com/pwaller/goupx
Binary upx
root@go-container:/go/src/main# cp main.simple main.simple.upx
root@go-container:/go/src/main# goupx --strip-binary main.simple.upx
root@go-container:/go/src/main# cp main.static main.static.upx
root@go-container:/go/src/main# goupx --strip-binary main.static.upx
root@go-container:/go/src/main# ls -l
-rw-r--r-- 1 root root     603 Feb 14 23:52 main.go
-rwxr-xr-x 1 root root 2885624 Feb 14 23:52 main.simple*
-rwxr-xr-x 1 root root  611192 Feb 14 23:59 main.simple.upx*
-rwxr-xr-x 1 root root 3974304 Feb 14 23:53 main.static*
-rwxr-xr-x 1 root root  935388 Feb 14 23:59 main.static.upx*
Test frolvlad/alpine-glibc (works OK)
$ for x in main.s*; do echo "- $x"; docker run -it --rm -v $(pwd):/go/src/main frolvlad/alpine-glibc /go/src/main/$x; echo; done
main.simple
Hello world - from the good old C
Hello world - compressed
Olar, I am root (0:0) - from Go app
main.simple.upx
Hello world - from the good old C
Hello world - compressed
Olar, I am root (0:0) - from Go app
main.static
Hello world - from the good old C
Hello world - compressed
Olar, I am root (0:0) - from Go app
main.static.upx
Hello world - from the good old C
Hello world - compressed
Olar, I am root (0:0) - from Go app
Test frolvlad/alpine-glibc:alpine-3.3_glibc-2.22 (is failing for some binaries)
$ for x in main.s*; do echo "- $x"; docker run -it --rm -v $(pwd):/go/src/main frolvlad/alpine-glibc:alpine-3.3_glibc-2.22 /go/src/main/$x; echo; done
main.simple
Hello world - from the good old C
Hello world - compressed
Olar, I am root (0:0) - from Go app
main.simple.upx
Hello world - from the good old C
Hello world - compressed
Olar, I am root (0:0) - from Go app
main.static
Hello world - from the good old C
Hello world - compressed
user: lookup userid 0: no such file or directory
main.static.upx
Hello world - from the good old C
Hello world - compressed
user: lookup userid 0: no such file or directory
I would like to still building my code with --ldflags='-extldflags "-static"' to make tottaly static binaries. Why this approach generates erros only in the frolvlad/alpine-glibc:alpine-3.3_glibc-2.22 image?

I don't have a clue how this might happen. I will investigate in more details once I have some free time.
@andyshinn There seems to be a new regression in glibc 2.22 artifacts.

Can we get some strace -ff -vv of the working static one and failing static one? If it is failing to lookup a userid with a no such file or directory error this smells a lot like a passwd file not found. A strace might be helpful here to know what the missing file or directory is.

Also, we might want the output of /usr/glibc-compat/bin/ldd against the binary in question to make sure there isn't some missing lib.

Hi, here are the outputs for:
ldd
http://pastebin.com/RSG3vbip
strace
http://pastebin.com/rB9B8ctN

It looks like the static compiled versions never consult /usr/glibc-compat/etc/ld.so.cache and that it just uses static locations (/etc/ld/so.cache). This is probably why it was working on 2.21 as we put the ld.so.cache at /etc/ instead of just /usr/glibc-compat/etc/.
I'm trying to find a good way to solve this without adding files to /etc/. I'll do some more research on a good solution.
However, a couple things just to throw out there (which may be viable for you):

There is an -alpine variant to the Golang images.
Golang 1.6 will have a fix for statically compiling binaries that run on multiple libc. Try it out:

docker run -it --rm -v $(pwd):/go/src/main golang:1.6-alpine sh
apk --no-cache add musl-dev gcc
go build  --ldflags '-linkmode external -extldflags "-static -s"'

This static binary should work on glibc and musl.

It seems that there is no way around but creating a symlink /etc/ld.so.cache to /usr/glibc-compat/etc/ld.so.cache. /etc/ld.so.cache is hardcoded in the static binary:
# strings /go/src/main/main.static | grep 'ld\.'
/etc/ld.so.nohwcap
/etc/ld.so.cache
ld.so-1.7.0
glibc-ld.so.cache1.1
runtime.osyield.f

Once I had added the symlink, the static executable got fixed.

I pushed a change to add a symlink for /etc/ld.so.cache. It should be in r9 which will be built shortly.

@stockrt I have upgraded my frolvlad/alpine-glibc:alpine-3.3_glibc-2.22, please, give it a try. At least, this issue is resolved for me on your reproduction example.
Feel free to reopen the issue if it still fails for you.


MCM fails to answer requests on the signalling channel

I'm attaching two logs to help debugging the problem.
I was using Cockpit fine with the BlueOS MCM, and when I restarted Cockpit, the stream didn't connect.
I took a look at the logs from cockpit and it was sending the signals requesting the streams correctly, but MCM was not responding to them, neither logging the requests. So I just restarted MCM and everything worked.
The first (oldest) log is when the problem happened, and the more recent one was after the restart.
bug-mcm-no-webrtc-comms.zip

Not reproducible, possibly closed by #384


Add command to delete assets [enhancement]

It would be great to be able to actualy cancel the exectution of an asset running on substra (as I unsderstand substra cancel only serves to cancel compute plans).
Also could we get a substra delete command that allows us to remove unwanted assets from substra (failed traintuples for example)?

Thanks Chris for your suggestion.
We plan to have a filtering based on tag or on metadata to hide some assets but deleting asset is against Substra design.

Thanks Chris for your suggestion.
We plan to have a filtering based on tag or on metadata to hide some assets but deleting asset is against Substra design.

Sounds great!


Embedded languages syntax highlighting stopped working in 1.92.0

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

VS Code Version: 1.92.0
OS Version: Windows 10, macOS 14 Sonoma

I have been using the following two extensions a lot. Since 1.92.0 update both extensions stopped working meaning the embedded code in JS does not get highlighted inside of YAML anymore. Each extension basically does the same job for me, so I tried to switch from one to another before realizing that my VSCode got an update and this functionality stopped working in both extensions. I've also tried the insiders build and the problem is still there. When I downgraded to 1.91.1, the extensions worked perfectly. This happens on both Windows and Mac.
The extensions are:

Extended Embedded Languages - https://marketplace.visualstudio.com/items?itemName=ruschaaf.extended-embedded-languages
YAML Embedded Languages - https://marketplace.visualstudio.com/items?itemName=harrydowning.yaml-embedded-languages

Steps to reproduce:

Install VSCode 1.92.0 or the insiders build.
Install one of the above extensions.
Create a yaml file with the following content:

test: | #js
  console.log("hello world")

Make sure the js code gets highlighted correctly

This is how it looks like in 1.92.0

1.91.1:

Installing this extension https://marketplace.visualstudio.com/items?itemName=redhat.vscode-yaml helps as a workaround. I guess because it takes over the yaml grammar and completes it with what is expected by other extensions.

Duplicate of https://github.com/microsoft/vscode/issues/224862.
Should be fixed by https://github.com/microsoft/vscode/pull/224954 in the very latest insiders (from d4c4e690b9f1e70291da9a46c5c8ac5ebfbda81a) and will be fixed in the upcoming recovery release.

"block-scalar" was taking precedence over injections because it was consuming the whitespace before it while the embedded language extensions were not
if I remove the ([\t ]*+) part
it starts working

but it will still break down when inside a nested block-map

this is because the other extensions use ^ instead of (^|\\G)
they will need to fix that part for themselves
Extended Embedded Languages
YAML Embedded Languages

@alexr00 VSCode needs to pull the latest yaml commits to fix this
https://github.com/RedCMD/YAML-Syntax-Highlighter/commit/71a88711ec184b7dde5240c8f150ad3c2dbbd5f1
sorry, I was under the impression the changes had already gone through
but it seems I missed the pull update by a few hours? https://github.com/microsoft/vscode/pull/224954

Re-using the old candidate here as this is a separate issue: https://github.com/microsoft/vscode/issues/224862

separate issue?
#224862 is already fixed tho?
@harrydowning has already pushed a temporary patch to his extension in the meantime https://github.com/harrydowning/vscode-yaml-embedded-languages/pull/45

I'm seeing https://github.com/microsoft/vscode/issues/224862 still occur. I guess I can't say for certain whether it's a separate issue since https://github.com/microsoft/vscode/issues/224862 is still happening.

Fixed in the latest version of my grammar

Awesome, I'll try adopting https://github.com/RedCMD/YAML-Syntax-Highlighter again for the November milestone.

@RedCMD I'm still seeing the original bug:

@alexr00 that is highlighted correctly

if you think it is an issue
then a new report should be made
as #224862 has been fixed by me and by the mdx extension

Thanks @RedCMD, I forgot to install the MDX extension 🤦♀
I've gone through the issues, and they all no longer repro with your changes 🚀

For using yaml as an embedded language in C++ with Extend Embedded Languages, I see the following:

The yaml scope doesn't escape outside of the string, but it also doesn't get any yaml scopes besides source.yaml.


Custom labels for toggle button

I liked this library because it's so simple and small, that of course meant some features I needed were missing;
I added a backwards compatible system of setting custom labels, through multi-type validation in the props. Either a boolean is passed and it defaults to no-labels / on-off, or an object is passed with a checked & unchecked field, containing the labels to be drawn on the togglebutton.
  labels: {
      type: [Boolean, Object],
      default: false,
      validator(value) {

        if(typeof value === 'boolean')
          return true;
        else if(value !== null && typeof value === 'object')
          return typeof value.checked === 'string' &&
            typeof value.unchecked === 'string' &&
            value.checked !== '' &&
            value.unchecked !== '';

        return false;
      }
    },

This meant some scaling issues though, and I noticed the width prop wasn't used anywhere in the code, so now 2 vars are set in the CSS, based on the width value, so users can scale them however they like.

Hey @BramVer thanks a lot for your pull request, it is the first time i got pull request to my public repo, so i'm kind of VERY excited! 😄
I've got a couple of questions if you dont mind:

Don't you think it will be simpler to use Array type instead of the Object?
Example:
Instead of labels="{checked: 'Foo', unchecked: 'Bar'} it could be labels="['Foo', 'Bar']"

Are use sure using css variables is a good idea? The reason I am asking is that I've just checked on canIUse ( http://caniuse.com/#search=css variable) and the Edge 14 (which was released less than a year ago is not supporting those)

Again, thanks a lot for your pull request! Looking forward for your response 😄

Hi @euvl, I get that, but with pretty and minimal libs like yours, these'll happen quite often :)
To address your concerns;

I was thinking of using an Array, but an Object just seemed safer to me.
This way the user cannot mix up the order of labels (putting the 'checked' text on the 'unchecked' side) and both properties need to be present before the object gets validated.
An array works just as well, but in my personal opinion, I think labels.checked && labels.unchecked is a cleaner approach than labels[0] && labels[1].

No I'm not sure about that at all. I'm horrible when it comes to CSS, and when I read about this new feature, I didn't stop to think it might be an issue to some users.

This feature seemed to me like the most ideal solution to the problem of specifying a unique style for each component (instead of just altering the CSS class, which changes all components).

Thanks for the positive response to the PR, if you know a solution to your 2nd concern, I'd be happy to look into it.

Cool, thanks a lot for detailed response! Merging to master. I will update npm package and examples laters today or tomorrow.
Thanks! 👍


Adjust type conversion in epacems_to_parquet

Be explicit about types, and convert them as the CSV is read in.
Partition on original CEMS data year (as the datapackages do) instead of UTC year.
Now works for zero-row data and state-years with all-missing measurement code variables.

Fixes #451.

This works (pending Travis checks), but it might be best to wait on @roll's response to this question: https://github.com/catalyst-cooperative/pudl/issues/451#issuecomment-544593399

Finally got around to merging this and testing the CEMS ETL and parquet conversion script, and it works great. Thank you @karldw!


it is possible to use LIME in regression problem for images, right?

i see the notebook example, but i dont understand the theory behind. Do i miss something in the essay or what?
The problem is that i want to use LIME in a regression problem for images. Dont know how to use it yet.
By the way, really like your paper.

See #124.
Thanks : )


Custom Video Hotspot floats in panorama when navigating

Can somebody help me? My custom video hotspot is working just fine, but it just floats around slightly when I drag around the panorama... Any tips? What am I doing wrong?
I created my viewer with all my hotspots like this:
viewer = pannellum.viewer('panorama', {
             "default": {
                 "type": "equirectangular",
                 "firstScene": "dome",
                 "sceneFadeDuration": 2000,
                 "autoLoad": true,
                 "showControls": false,
                 "hfov": 90,
                 "hotSpotDebug": false,
             },
             "scenes": {
                 "dome": {
                     "pitch": -5.6,
                     "yaw": 7.1,
                     "panorama": "./dist/images/dome-360.jpg",
                     "hotSpots": [
                         {
                             "pitch": -0.6,
                             "yaw": 42.1,
                             "type": "scene",
                             "text": "Iceland",
                             "targetYaw": -23,
                             "targetPitch": 2,
                             "cssClass": "link-hotspot",
                             "createTooltipFunc": linkHotSpot,
                             "clickHandlerFunc": onHotSpotClicked,
                             "clickHandlerArgs" : {target:"iceland"}
                         },
                         {
                             "pitch": 10,
                             "yaw": 222.6,
                             "type": "info",
                             "text": "Much of the library was constructed from cast-iron to reduce the risk of fire.",
                             "cssClass": "custom-hotspot",
                             "createTooltipFunc": hotSpot,
                             "createTooltipArgs": "Spot 2",
                             "createTooltipArgs": "Our website",
                             "URL": "https://www.syngenta.com/en",
                         },
                         {
                             "pitch": 10,
                             "yaw": 144.4,
                             "type": "info",
                             "text": "Much of the library was constructed from cast-iron to reduce the risk of fire.",
                             "cssClass": "custom-hotspot",
                             "createTooltipFunc": hotSpot,
                             "createTooltipArgs": "Spot 3",
                             "createTooltipArgs": "North Charles Street",
                         },
                         {
                             "pitch": -10,
                             "yaw": 4.5,
                             "createTooltipFunc": hologramVideoHotSpot,
                             "createTooltipArgs": "Spot 4",
                         },
                     ]
                 },

                 "iceland": {
                     "pitch": -0.6,
                     "yaw": 5,
                     "panorama": "./dist/images/iceland.jpg",
                     "hotSpots": [
                         {
                             "pitch": -0.6,
                             "yaw": 37.1,
                             "type": "scene",
                             "text": "Dome",
                             "targetYaw": 7.1,
                             "targetPitch": -5.6,
                             "cssClass": "link-hotspot-return",
                             "createTooltipFunc": linkHotSpot,
                             "clickHandlerFunc": onHotSpotClicked,
                             "clickHandlerArgs" : {target:"dome"}
                         }
                     ]
                 }
             }
         });

I added a custom video and when I navigate in the panorama it floats gently. It isn't in a fixed position as the other hotspots.
function hologramVideoHotSpot(hotSpotDiv) {

            hotSpotDiv.classList.add('pnlm-pointer');
            var video = document.createElement('video');
            video.autoplay = true;
            video.playsInline = true;
            video.loop = true;
            video.muted = true;
            video.className = "custom-hologram-hotspot-video";
            hotSpotDiv.appendChild(video);

            video.play();

            //Remove generated sprite class
            video.parentElement.classList.remove('pnlm-sprite');
            video.parentElement.classList.remove('pnlm-undefined');
            video.parentElement.classList.remove('pnlm-hotspot');
        }

I styled the video with the sass code below:
.custom-hologram-hotspot-video {
        width: 450px;
        position: absolute;
        top: -400px;
        left: -195px;
}

I gave the video element that I created a source in a different js file that checks which browser the user is currently using for video support reasons:
export default {
    init() {
        this.checkBrowser();
    },
    checkBrowser() {
        // HEVC with alpha is supported ONLY on Safari >=13 / ios >=13
        // previous versions also supported HEVC but WITHOUT alpha channel (video will play with black bg)
        // Safari 13 is the first version to support mediaCapabilities

        this.supportsHEVCAlpha();

        this.isIE11();

        if (!this.isIE11()) {
            setTimeout(() => {
                const player = document.querySelector('.custom-hologram-hotspot-video');
                player.src = this.supportsHEVCAlpha() ? './dist/videos/hologram-alpha-safari.mov': './dist/videos/hologram-alpha.webm';
            }, 100);
        }
    },
    supportsHEVCAlpha() {
        const navigator = window.navigator;
        const ua = navigator.userAgent.toLowerCase()
        const hasMediaCapabilities = !!(navigator.mediaCapabilities && navigator.mediaCapabilities.decodingInfo)
        const isSafari = ((ua.indexOf('safari') != -1)&& (!(ua.indexOf('chrome')!= -1) && (ua.indexOf('version/')!= -1)))

        return isSafari && hasMediaCapabilities
    },
    isIE11() {
        return !!window.navigator.userAgent.match(/Trident\/7\./,[]);
    }
}

Does anybody know what I'm doing wrong? The video is working just fine, but it just floats around slightly... Any tips?

Found he solution!
I deleted my custom holoram function and gave the hotspot a type of info to view where the point was in the scene. The infospot was not on the same position as the video so I transformed it with translateY(-43%) so that the point and the video were on the same spot in the scene.
This fixed it.


Updated the one-time-core partial data notebook due to name change in the function

Not one_time_core it is due to name change in lazy_multi_tau to lazy_one_time

@sameera2004 Next time note that this is a companion PR to your scikit-beam PR.
@tacaswell This is not ready to go in yet because the companion scikit-beam code has not been merged. I am speaking of scikit-beam/scikit-beam#391.  Should I revert?

I don't think it is worth it, just get the PR done asap.


Makefile: fix make clean

make clean errors out without cleaning everything. Fixes the make file to use rm -f when appropriate and -cmd otherwise.
Added cleaned message at the end to show it completed successfully (otherwise it ends in what can seem like an error message at first glance).
Signed-off-by: Hugo Leblanc<EMAIL_ADDRESS>

:robot: ok to test


Error 800 with 3 tables imported from CSV (March 2023 Version of PBID)

Describe the problem
Hello,
I just loaded 3 tables from csv files stored in my pc
Then I tried to add the date dimension from bravo and got an error 800

I tried saving the file, closing and opening up again, but the error persists
Regards!
Bravo version
1.0.1
Tabular model
Power BI Desktop, import mode
Power BI
Version: 2.115.663.0 64-bit (march 2023)
Steps to reproduce the behavior

Load files
Try to add Date Table --> Error
Save File
Close File
Open File
Try to add Date Table --> Error

Additional context
No response
Screenshots
No response
Anything else ?
No response

Hi @bernatagulloesbrina, this appears to be an internal error generated by the Analysis Services engine, similar to the one reported in issue 597.
Does the error occurs if you disable the holidays in the Date table?

indeed, turning off holidays does seem to solve the problem
so it's a problem on microsoft's side?
thank you -- for most of my use cases holidays are not that relevant

This is probably the issue @marcosqlbi mentioned here https://github.com/sql-bi/Bravo/issues/597#issuecomment-1440694563

We spot a bug in Power BI Desktop that crashes with certain DAX expressions and we are waiting a response from MS.


Add Rhys Arkins to the README

Closes #49

This PR has been open for a while, I see no objects. I will merge it now.


Listener of one field is activating other fields

I'm submitting a ...  (check one with "x")
[ X] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ X] support request

Current behavior

I have 10 custom input field. In the custom component, I have defined an ngAfterViewChecked and am doing validation inside those.
For the first field, I have a checkDecimal validation. Now, When I interact with any other field, I am able to see the console.log meant just for the first field. It means that listener for first field is always in the active state and is somehow linked to the other fields
Expected behavior

Each field should be independent of each other and interaction on one of the field should not activate other fields.
Minimal reproduction of the problem with instructions

I would not be able to provide a sample implementation due to some issue.

without providing a sample example, it's difficult to figure-out what you're referring.

My question is related to https://github.com/formly-js/angular-formly/issues/739 issue. I would be sharing a example for the same soon.

Hey @aitboudad ,
https://stackblitz.com/edit/ngx-formly-ui-material-1vwjsm?file=app/app.module.ts <-Stackblizt
Open inspect element and just hover in any one of the text area component. You would find that console.log(this) is called for all text area and for any event in any component.
Regards,
Devesh

why you would like to use ngAfterViewChecked ?

I would like to add a default validation checkDecimal and would like to initiate the validation when data loads into the form i.e using the model i would be populating the form with some prefilled data.
Also, Using ngAfterViewChecked, I was aiming to add custome validation which I could call directly using templateOptions.
I would just assign checkDecimal: true in templateOptions and then would use ngAfterViewChecked to check whether the string is decimal or not. This way, i would save the time of user to not use validator:{validation: ['checkDecimal']} i.e I am providing another level of validation for the same.

that's not the better place, in v5 we've introduce some hooks that allow a handling a such case:
prePopulate: before touching the field.
onPopulate: defaultOptions, Model, formControl are set (which is suitable to update the child element).
postPopulate: after populate children, if the field doesn't have children it should be the same as onPopulate event.
Warning:
These events can be called whenever the field model change not matter the field was initialized or not.

@aitboudad But we would need to define these lifecycle in jsonConfig.
I want to define the same in TS file and build default validations same as min, max, minLength, maxLength
If postPopulate, prePopulate and onPopulcate can be used in ts file of extended component then I would be happy to hear the way to use them without facing the same problem as ngAfterViewChecked

But we would need to define these lifecycle in jsonConfig.

not sure what do you means but for advanced usage you can define a custom extension for that as descibed in https://github.com/formly-js/ngx-formly/issues/1109#issuecomment-412295111

If postPopulate, prePopulate and onPopulcate can be used in ts file of extended component then I would be happy to hear the way to use them without facing the same problem as ngAfterViewChecked

sorry but I don't have much time to provide a full example, try it on your side and let me know if you have some questions.
here is a simple example of initializing a field
export class AddressFieldType extends FieldType {
  prePopulate(field: FormlyFieldConfig) {
    if (field.fieldGroup) {
      return;
    }

    field.fieldGroup = [{
      type: 'select',
      key: 'city',
      },
      ....
    }];
  }
}

Hey @aitboudad ,
I tried your example and it is not working prePopulate in TS file is being called even before the form is rendered,
Do you know any other way that I would be able to achieve the same
Regards,
Devesh

validation should be defined before the form is rendered, isn't it ?

Hey @aitboudad ,
Sorry! We figured out how to restrict the listener to one element.
But then when we tried to run validation ( Defined as a function not as a form config ) in TS file under postPopulate, the validation worked, but the formControl.markasdirty did not work as the form had not rendered.

we're relying on touched status instead, can you try with .markAsTouched()


"Old school" style.

Hi, thanks for this good alternative that can fulfill our needs.
I'm working on reproduce this questionary application, and I'm using this package as an alternative to grouped_buttons.
I would like to know if it is possible to get the oldschool style for checkbox(box) and radio buttons whit this package.
is there any way to do it with the current state of the package? Does it require a new feature?
Thank you very much.

Hello @SalahAdDin ✋🏻
Until this moment, I had not thought about such functionality. But, in general, this is worth doing.
Now the buttons are customized. But not all users have enough.

It will be great if you suggest your own version of this functionality.

Ok, let me see what can i do. Thanks.

Hello @SalahAdDin !
This feature is available from 4.6.0 package release
You can use buttonBuilder or buttonIndexedBuilder
See more here


fix: 🐛 treat isActive to case-insensitive

✅ Closes: #3656

Hello @posva. Any suggestion on this PR?


(cherry-picked) Broken link to Fraud Detection tutorial in Dashboard Home (RHOAI 2.11.0)

Closes: RHOAIENG-9175
Description
Change the link in manifest file for Fraud Detection tutorial
https://github.com/opendatahub-io/odh-dashboard/assets/97534722/b15b1853-2701-4837-a28c-591d69e4d3e5
How Has This Been Tested?

Deploy the manifest oc apply -k manifests/overlays/apps/base/rhoai -n opendatahub
Verify that the link redirects to the correct destination.

Test Impact
None, just a URL change.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

We no longer create a new custom branch to downstream, as detailed in our docs: https://spaces.redhat.com/pages/viewpage.action?pageId=494994226


Loss is nan when using default configs

I am using pytorch 1.0. When I was training the model, I got these:

iter 0 || Loss: 3.2635 || timer: 0.0842 sec.
iter 10 || Loss: 14.6995 || timer: 0.0851 sec.
iter 20 || Loss: 12.2860 || timer: 0.0846 sec.
iter 30 || Loss: 14.0080 || timer: 0.0854 sec.
iter 40 || Loss: 20.0848 || timer: 0.0903 sec.
iter 50 || Loss:<PHONE_NUMBER>8.0000 || timer: 0.0833 sec.
iter 60 || Loss: nan || timer: 0.0837 sec.
iter 70 || Loss: nan || timer: 0.0833 sec.

I only modified the code to make the code run in pytorch 1.0. The configs I changed:
batch_size = 1
basenet = '.../vgg16_reducedfc.pth'

# using trained model also got loss = nan
# resume = '.../ssd300_mAP_77.43_v2.pth'

Solved this problem by using a small learning rate.
The following post may help to explain this problem:
https://www.reddit.com/r/MachineLearning/comments/84waz4/d_relation_between_learning_rate_batch_size_and/

@DHZS What learning rate did you use?
Did the model predict correctly after training??
Thanks

@cypherix Sorry, I forgot the result. But small batch size(such as 1) in one-stage detector is not a good choice. Setting batch size >= 16 can get good results.
Anyway, if you want to try a small batch size, you can change the learning rate according to this paper:
Accurate, Large Minibatch SGD: Training ImageNet in 1 Hour
The author said:

Linear Scaling Rule: When the minibatch size is multiplied by k, multiply the learning rate by k.

@DHZS Hi again. How many iterations did you train for?

I set the lr 1e-7 and the batch size 16 to solve the problem finally TAT...