Update switch.sh to provide more details.

Retooled the switch.sh script to provide more details about what the PAXs see in the view of incorporating this into the nodediag infrastructure.
Additionally:

updated script for the power testers to use to create and exercise the Rabbit.
updated the rabbit-s.sh script to add the specific parameters needed for logging when investigating the HPECRAY-23 issue.

Tony, you should run shellcheck on these scripts. Either clean up all of the warnings, or at least knock off a few each time you come back to these scripts.
Eventually, we'd like shellcheck to run clean. Then we can run it in the github workflow.
brew install shellcheck


Documentation about palworld settings

Hey there,
I've found an article on shockbyte that explains what the palworld settings do and what they can be set to.
Maybe add a reference to it in the Editing Server Settings section? (If I find time later, I could put a PR)
Also this might come in handy when parameterising the settings in itself.

@MSpreckels Nice find, I don't have mucht time right now, but please feel free to create a PR :)

@thijsvanloef Is a note enough? Should i insert a md table?

Attribute
Default
Values
Notes

Difficulty
None

DayTimeSpeedRate
1.000000

NightTimeSpeedRate
1.000000

ExpRate
1.000000

PalCaptureRate
1.000000

PalSpawnNumRate
1.000000

PalDamageRateAttack
1.000000

PalDamageRateDefense
1.000000

PlayerDamageRateAttack
1.000000

...

@MSpreckels a note with a link should be enough, that way I would not need to update it every time it gets changed :)


SummedOp updates & optimization converters to use Opflow

Summary
Update the converters in the optimization module to accept and return OperatorBase objects (WeightedPauliOperator is supported as before). Also adds an equals implementation to SummedOp which was necessary for the tests to work and moves simplify into reduce as suggested in https://github.com/Qiskit/qiskit-aqua/pull/1000#issuecomment-633844384.
Marked as on hold to give @t-imamichi time for some performance checks.
Details and comments
equals has been updated such that expressions such as X + X == 2*X and X + Z == Z + X evaluate to True, unlike before. This calls reduce on the operators and is  is slower than the previous equals used in ListOp I removed two lines of add which checked if other is equal to self, as this adds a lot of overhead e.g. if many Pauli terms are summed. Since the opflow focuses on ease-of-use and intuitive syntax close to math, I think it is very important to have X + X = 2X and XZ = ZX equivalencies.
simplify has been renamed to collapse_summands to be more descriptive and such that users have one distinct method they should call to reduce/simplify the operator, namely reduce.

I checked the performance with some Hamiltonians of molecules and confirmed that it works well.
I add a short comment for simplicity.

Thanks for checking @t-imamichi!

@Cryoris seems like this PR causes troubles in the build pipeline.

Indeed there seems the word summands which is marked as error by the spellcheck. The test failures are coming from Qiskit/qiskit-terra#4622, where the type of Instruction.definition has been changed from a list to a circuit. Thus expressions such as if gate.definition == list are failing now.
From the CI log:
if isinstance(gate, IGate) or (type(gate) == Instruction and gate.definition == []):
      File "/home/travis/miniconda/lib/python3.7/site-packages/qiskit/circuit/quantumcircuit.py", line 211, in __eq__
    return circuit_to_dag(self) == circuit_to_dag(other)

Thanks for the info!

Haven't looked into details but some other PRs(including my own) are failing at
qiskit/aqua/utils/controlled_circuit.py:125:12: E1120: No value for argument 'c' in function call (no-value-for-parameter)

Not sure why that happens, but in #1100 @manoelmarques also had to change the input to controlled_circuit. Looks like the linter wants to have the arguments unrolled 🤔

Just curious, something in this PR caused this behavior or the problem somewhere else?

Pretty sure this is from somewhere else, as this PR didn't touch controlled_circuit and the failures of the tests in the CI are coming from Qiskit/qiskit-terra#4622. This PR was able to merge because it was built and tested before that terra PR merged and has not been rebuilt since then.

Thanks for the update, a pretty large PR in terra as I see.


Sky Span: commander.attack() documentation suggests commander can move

The documentation for commander.attack() states that the commander will move toward its target if the target is out of range. It does qualify it with "and the commander is smart enough" but it still seems like people might get confused.

Removed the confusing part about movement entirely. Now it just says:

The attack method makes the #{spriteName} attack the target unit.


UD - Brouzouf

It should be better to specify UD with currency index like :
UD-meta brouzouf-
or
UD -mb-
or any other way to know which currency it's about.

In which dialog or window is it missing ?

It's missing in "wallet", in "transactions" and also in "transfer money" window. Like in Physics, it is always necessary to specify the unit of money you use. Imagine we have metabrouzouf (mb) and zetabrouzouf (zb), we always need to know exactly which unit is used when we watch it :
110 mb or 1,1 UD mb
110 zb or 1,1 UD zb

@c-geek , we need a way to have the currency name in a "short" name, like two or three letters....

Added in https://github.com/ucoin-io/cutecoin/commit/b566c4cdc027da9bfa3d800b86ccd224cdc30c03


Debug Adapter immediately killed after starting on Windows

I've got a Debug Adapter for LLDB (lldb.llvm.org) which works fine on Linux.  I'm trying to get this working on Windows and running into some problems.  The second I make a call into the CRT to read from stdin the adapter process just exits.  I have tried attaching to the debug adapter to see if I can figure out what's going on, but even this doesn't seem to work.  It seems that after some period of time (very short), VS Code just kills the adapter for no apparent reason.  My main function looks like this:
int main(int argc, char *argv[]) {
  ::OutputDebugStringW(L"Checkpoint 1!");
  while (true);
  ::OutputDebugStringW(L"Checkpoint 2!");
  ...
}

And I can see using DbgView that Checkpoint 1 is printed and Checkpoint 2 is not printed.  Instead, the process just starts up and immediately dies.
Are debug adapters supported on Windows, because AFAICT I'm not doing anything special or interesting at all, and it just straight up doesn't work.

@zjturner of course, DAs are supported on Windows otherwise VS Code debugging would not work in Windows at all.
But I'm not sure that your "main" is a good test test program...
VS Code expects that responses are returned by the DAs promptly and it will kill the DA after a timeout.

@weinand I only resorted to this test after trying many other thing.  Of course, my real adapter calls fgets on stdin, which also terminates.  And the amount of time before it kills my process is on the order of 1 second or less, so there is not enough time to really do anything.
Is there any way to adjust this timeout so I can confirm whether this is actually the problem?

How can I re-open this, since it's not yet clear to me that this isn't a bug in VS Code?

Can you run your DA in server mode? (so that VS Code can connect to via a socket instead of stdin/stdout?

Running in server mode is indeed something I have tried, and it works (In the sense that VS Code does not kill it).  But I'd like to be able to debug it in non-server mode if possible.  Reading from a socket and reading from stdin have some differences, and it would be nice to be able to debug the file codepath.

how does the package.json of your debugger extension looks like?
At least this part:

If you have LiveShare installed, uninstall it and see whether it makes a difference.

I have a lot of stuff under there, so I guess I'll post it all since I'm not sure which part matters.  I wonder if it has to do with the fact that I don't specify .exe?  Maybe I'll try that.
		"debuggers": [
			{
				"type": "lldb-vscode",
				"label": "Native LLDB Debugger",
				"enableBreakpointsFor": {
					"languageIds": [
						"ada",
						"arm",
						"asm",
						"c",
						"cpp",
						"crystal",
						"d",
						"fortan",
						"fortran-modern",
						"nim",
						"objective-c",
						"objectpascal",
						"pascal",
						"rust",
						"swift"
					]
				},
				"program": "./bin/lldb-vscode",
				"configurationAttributes": {
					"launch": {
						"required": [
							"program"
						],
						"properties": {
							"program": {
								"type": "string",
								"description": "Path to the program to debug."
							},
							"args": {
								"type": [ "array", "string" ],
								"description": "Program arguments.",
								"default": []
							},
							"cwd": {
								"type": "string",
								"description": "Program working directory.",
								"default": "${workspaceRoot}"
							},
							"env": {
								"type": "array",
								"description": "Additional environment variables.",
								"default": []
							},
							"stopOnEntry": {
								"type": "boolean",
								"description": "Automatically stop after launch.",
								"default": false
							},
							"disableASLR": {
								"type": "boolean",
								"description": "Enable or disable Address space layout randomization if the debugger supports it.",
								"default": true
							},
							"disableSTDIO": {
								"type": "boolean",
								"description": "Don't retrieve STDIN, STDOUT and STDERR as the program is running.",
								"default": false
							},
							"shellExpandArguments": {
								"type": "boolean",
								"description": "Expand program arguments as a shell would without actually launching the program in a shell.",
								"default": false
							},
							"detachOnError": {
								"type": "boolean",
								"description": "Detach from the program.",
								"default": false
							},
							"trace": {
								"type": "boolean",
								"description": "Enable logging of the Debug Adapter Protocol.",
								"default": true
							},
							"sourcePath": {
								"type": "string",
								"description": "Specify a source path to remap \"./\" to allow full paths to be used when setting breakpoints in binaries that have relative source paths."
							},
							"sourceMap": {
								"type": "array",
								"description": "Specify an array of path remappings; each element must itself be a two element array containing a source and desination pathname. Overrides sourcePath.",
								"default": []
							},
							"debuggerRoot": {
								"type": "string",
								"description": "Specify a working directory to set the debug adaptor to so relative object files can be located."
							},
							"initCommands": {
									"type": "array",
									"description": "Initialization commands executed upon debugger startup.",
									"default": []
							},
							"preRunCommands": {
									"type": "array",
									"description": "Commands executed just before the program is launched.",
									"default": []
							},
							"stopCommands": {
									"type": "array",
									"description": "Commands executed each time the program stops.",
									"default": []
							},
							"exitCommands": {
									"type": "array",
									"description": "Commands executed at the end of debugging session.",
									"default": []
							}
						}
					},
					"attach": {
						"properties": {
							"program": {
								"type": "string",
									"description": "Path to the program to attach to."
							},
							"pid": {
								"type": [
										"number",
										"string"
								],
								"description": "System process ID to attach to."
							},
							"waitFor": {
								"type": "boolean",
								"description": "If set to true, then wait for the process to launch by looking for a process with a basename that matches `program`. No process ID needs to be specified when using this flag.",
								"default": true
							},
							"trace": {
								"type": "boolean",
								"description": "Enable logging of the Debug Adapter Protocol.",
								"default": true
							},
							"sourcePath": {
								"type": "string",
								"description": "Specify a source path to remap \"./\" to allow full paths to be used when setting breakpoints in binaries that have relative source paths."
							},
							"sourceMap": {
								"type": "array",
								"description": "Specify an array of path remappings; each element must itself be a two element array containing a source and desination pathname. Overrides sourcePath.",
								"default": []
							},
							"debuggerRoot": {
								"type": "string",
								"description": "Specify a working directory to set the debug adaptor to so relative object files can be located."
							},
							"attachCommands": {
								"type": "array",
								"description": "Custom commands that are executed instead of attaching to a process ID or to a process by name. These commands may optionally create a new target and must perform an attach. A valid process must exist after these commands complete or the \"attach\" will fail.",
								"default": []
							},
							"initCommands": {
								"type": "array",
								"description": "Initialization commands executed upon debugger startup.",
								"default": []
							},
							"preRunCommands": {
								"type": "array",
								"description": "Commands executed just before the program is attached to.",
								"default": []
							},
							"stopCommands": {
								"type": "array",
								"description": "Commands executed each time the program stops.",
								"default": []
							},
							"exitCommands": {
								"type": "array",
									"description": "Commands executed at the end of debugging session.",
									"default": []
							}
						}
					}
				},
				"initialConfigurations": [
					{
						"type": "lldb-vscode",
						"request": "launch",
						"name": "Debug",
						"program": "${workspaceRoot}/<your program>",
						"args": [],
						"env": [],
						"cwd": "${workspaceRoot}"
					}
				],
				"configurationSnippets": [
					{
						"label": "LLDB: Launch",
						"description": "",
						"body": {
							"type": "lldb-vscode",
							"request": "launch",
							"name": "${2:Launch}",
							"program": "^\"\\${workspaceRoot}/${1:<your program>}\"",
							"args": [],
							"env": [],
							"cwd": "^\"\\${workspaceRoot}\""
						}
					}
				]
			}

and your "program" attribute doesn't require a ".exe" on Windows?

Interestingly, adding the .exe seems to have made a difference.  I don't have time to investigate this more as I'm leaving for the day, but I will say that VS Code was definitely calling my extension even when I don't have .exe, but adding it seems to make it wait longer before killing it, which is extremely strange.  So maybe there actually is some kind of bug here when you forget to specify .exe in your program name.  It should either report an error or behave identically.
I'll dig in more tomorrow.


a prior distribution which has two parameter

I have a function ：Y = beta_1+beta_2 * X
and I want to construct a prior beta = (beta_1 , beta_2) iid Normal (mu, covariance matrix）
how to deal with this problem
https://www.sciencedirect.com/science/article/pii/S0142112320300426
https://doi.org/10.1016/j.ijfatigue.2020.105511

Hi,
Thanks for your question. As this is more a modeling one than an issue with the code in this repo, can you please post it to our Discourse?
Thanks and have a nice day 🖖


Enhabce oVirt provider create, view/edit credentials help text fields

Reference: https://github.com/kubev2v/forklift-console-plugin/issues/646
Reference: https://github.com/kubev2v/forklift-console-plugin/issues/541
Rephrase all oVirt fields help text messages and add hints for the following dialogs:

Create oVirt provider,
oVirt Credentials view/edit modes.

The main changes include:

Rephrase help text messages for user, password, insecureSkipVerify, cacert fields (name and url fields were already rephrased in separate PRs)
Support 'warning' validation state for the 'user' field.
Add hint sections for insecureSkipVerify, cacert fields.
In Create/edit dialogs, hide the 'cacert' field help text message and the hint section if insecureSkipVerify=true.

This fix assumes that leaving the 'cacert' field empty, will trigger verifying the the system CA certificates.
Screenshots
Create

View Credentials

Edit Credentials

can you add a little space above the blue hint box ?

can you add a little space above the blue hint box ?

@yaacov Done, please check the new screenshot added to the description comment

cc:// @RichardHoch @anarnold97
Please review text changes

This fix assumes that leaving the 'cacert' field empty, will trigger verifying the the system CA certificates.

@ahadas
Do we still support the combination of unchecking the 'skip cert validation' and  leaving the oVirt provider's CA field empty for retrieving the system engine CA certificate by the webhook?
Based on https://redhat-internal.slack.com/archives/C049WBVTRD2/p1694592350447549?thread_ts=1694517736.539569&cid=C049WBVTRD2, there was an issue with that. It's currently not working for me as well. What is the status of this?
@RichardHoch
If above issue is still supported and should work then we need to document it as well since currently this option is not mentioned for the oVirt provider.

This fix assumes that leaving the 'cacert' field empty, will trigger verifying the the system CA certificates.

@ahadas Do we still support the combination of unchecking the 'skip cert validation' and leaving the oVirt provider's CA field empty for retrieving the system engine CA certificate by the webhook?
Based on https://redhat-internal.slack.com/archives/C049WBVTRD2/p1694592350447549?thread_ts=1694517736.539569&cid=C049WBVTRD2, there was an issue with that. It's currently not working for me as well. What is the status of this?

it is supposed to work once the certificate is added to the system certification pool but we didn't test it

I see that mixing #646 for the help msgs and #541for the hints is not easy, lets separate the PR to one that do help msgs for the tasks in #646 and another for adding hints as required by #541, this way way we can focus better on each task

After having an offline discussion, it was decided to convert both hints into  tooltips that will include relevant fields help text for now.
Reminder: It seems that that help text included relevant info for explaining how to use the fields, but since it's an extra info then tooltips seems preferable and less space expending.
We can discuss using hints in the separate #541 issue.

This current fix addressed all comments mentioned above.

I also updated the main comment and screenshots with the changes


Is this project still using simplelogin?

I've noticed that this boilerplate creates the users as "simplelogin:xx" when using the Email & Password auth provider. Is this normal? Can we change this id's?
Sorry for the newbie questions!
Edit: Ok, I've just learnt in a SO thread that this is the default naming... but can it be changed?

Ok, I've just learnt in a SO thread that this is the default naming... but can it be changed?

I don't see simpleLogin anywhere in the main branch. https://github.com/firebase/angularfire-seed/tree/master/app/login

Ah, the uids are generated by Firebase and use simplelogin for historical purposes. This is unrelated to the seed or AngularFire.

Ok! Thanks for the clarification!


Some chairs should be able to be picked up

I want my immersive rage cage experience.

just add folding chairs that can be used as weapons

at this rate just add dedicated wrestling rings to the maps

I support the increase of the power of wrestle mania. So @metalgearsloth would you kindly assign me to this issue ^^


Option "Bulk insert missing fields" adds fields in reverse

Hello. I've just installed this plugin, and I've been liking it so far.
When I try to "Bulk insert missing fields" either by hotkey or modal, the items are inserted in a different order than they are defined in the class file. Here is a screen recording of what I mean.

My expectation is that the fields would appear in the same order as I defined them.
Please let me know if what I expect is not what the intended behaviour is!
Thank you.

I experience the same issue and would prefer they insert in the order they are defined.

fixed in 0.7.1


Oops! Something went wrong. Please try again later

Issue Description

The error "Oops! Something went wrong. Please try again later" appears on screen when completing a challenge.

Same issue here.

@dainamisk thanks for the issue. This issue should be resolved. If you're still having issues. Please comment back. Otherwise, I'll be closing this issue. Happy coding!


unit-conversions SIGSEGV on armv7hl

Hello,
when trying to build and run the test suite on armv7hl architecture, I get the following results:
make[1]: Leaving directory '/builddir/build/BUILD/json-2.0.3/test'
test/json_unit "*"
json_unit is a Catch v1.5.6 host application.

Run with -? for options

value conversion
  get an object (explicit)

src/unit-conversions.cpp:41
src/unit-conversions.cpp:43: FAILED:
due to a fatal error condition:
  SIGSEGV - Segmentation violation signal

test cases:   14 |   13 passed | 1 failed
assertions: 4004 | 4003 passed | 1 failed
Makefile:28: recipe for target 'check' failed
RPM build errors:
make: *** [check] Error 245

Thanks for reporting. Unfortunately, I do not have access to an armv7hl architecture, so I cannot debug into the problem. Can you please give me details on the compiler you are using?

The compiler is gcc-c++ 6.2.1. Here is the full build.log: https://kojipkgs.fedoraproject.org//work/tasks/7056/15497056/build.log

I've managed to run the conversions unit test via valgrind on the armv7hl machine, this is the output:
+ valgrind ./test/json_unit '*conversion*'
==25420== Memcheck, a memory error detector
==25420== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==25420== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==25420== Command: ./test/json_unit *conversion*
==25420==
==25420== Invalid read of size 4
==25420==    at 0x4975034: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/libstdc++.so.6.0.22)
==25420==    by 0x92BEF: std::pair<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const, nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long long, unsigned long long, double, std::allocator> >::pair(std::pair<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const, nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long long, unsigned long long, double, std::allocator> > const&) (in /builddir/build/BUILD/json-2.0.5/test/json_unit)
==25420==  Address 0x6 is not stack'd, malloc'd or (recently) free'd
==25420==

Meanwhile, the build log at https://kojipkgs.fedoraproject.org//work/tasks/7056/15497056/build.log is no longer available. Do you have a copy?

It may not be 100% comparable, but the tests (current develop version) succeed on a Raspberry Pi:

gcc (Raspbian 4.9.2-10) 4.9.2
Linux raspberrypi 4.1.19+ #858 Tue Mar 15 15:52:03 GMT 2016 armv6l GNU/Linux

I shall try to run the tests with Valgrind as well.

Thanks @navybk for helping debugging!

@nlohmann Here's the build log as an attachment.
json-2.0.5-armv7-build.txt

I fear I have to mark this issue as "won't fix"...

@nlohmann ok, I'll have to exclude amrv7 builds in Fedora then.

Thanks for the quick response. Sorry I could not fix this, but I lack the hardware to debug into it.

I was running into the same problem on armv7l. Using 32 bit data types instead of the default bsaic_json<> template fixed the problem. There seems to be a problem with memory alignment of structures on this platform.

@mwittgen Thanks for the hint! Can you provide the concrete instantiation of the templates so I can update the README?

basic_json<std::map, std::vector, std::string, bool, std::int32_t, std::uint32_t, float>
I have only run our application code and switching to 32 bit values fixed it. This would need some detailed investigation to confirm what is actually going wrong. I might get to it later.

Thanks a lot! This is really strange...

I'll see if I can provide a simple test case with QEMU.

Forgot to mention gcc 5.2 on ZYNQ (armv7l) running ArchLinux. Compiled natively or cross compiled with crosstool-ng. Both executables crashed on invalid memory pointers in different places. Switching off optimization made the crash go away.


prevent error when using relativedate on field without arguments

This prevents the error I get when using  $item->published()->relative() because the merge_array function in the relativeDate function expects an array and receives NULL.

2.0.0 moved to be just a wrapper for the Carbon library. Still thank you for your effort even though I could not live up to them 😞  👍


Null string throws Object reference not set to an instance of an object

hi,
in previous version, when i pass a null string to the parser, it will render and output as if it were an empty string. but in version 3 of razorengine, it throws an object reference not set to an instance of an object.
here's the sample code:
		using (var service = new TemplateService())
		{
			const string template = "
Hello @Model.Person.Forename
";
			const string expected = "
Hello
";

			var model = new {Person = new Person {Forename = null}};
			string result = service.Parse(template, model);

			Assert.That(result == expected, "Result does not match expected: " + result);
		}

currently, my workaround is to not use anonymous objects and just define a class as a container for the objects i want to pass along to the parser. is there a better workaround?
thanks for the really neat engine.
jake

I'm looking to introduce a change to support cascading null properties in expressions, but that would (and potentially should) only work for dynamic objects whereby we can intercept the property get at runtime (e.g., with the new ViewBag property).
In terms of null values for statically typed models (including anonymous models), we can only handle the last property get in the expression (in your example, the Forename property). If for instance, the Person property of the model was null, I believe the expected behaviour to be a NullReferenceException, if we tried to handle that, the template wouldn't run at all.


Fully enable project creation stepper

Description
A stepper already existed for the project creation, but it was incomplete and broken. This PR intends to propose the available relevant features in the project creation, and to constraint the user paths to the working ones.
The stepper has now two steps:

Create the project with the name and an optional description
Add users

This PR adds the following behaviors:

preventing navigation between steps other than with the “Create project” button
Introduce a “Finish” button that redirects to the project details.
Now the active project in the stepper is defined in the project service, allowing to keep it when redirected to the details
Adding and listing users of a project is repeared.

This PR also introduce wrappers and nested routers in order to centralise the initialization and the destruction of the active project, models list and model based on the router.
Checklist

[ ] My code follows the guidelines of this project: CONTRIBUTING.md
[ ] I updated the documentation with the new functionality
[ ] I went through the code and removed all print statements, breakpoints and unused code
[ ] Error handling for all possible scenarios has been implemented
[ ] I've add logging for all relevant operations
[ ] I added the appropriate labels to this PR (enhancement/bug/documentation, effort, priority)

I just tried to create a new project as user without admin rights. I see the Add new project field, however, when I click "Create project", I just get the message: "An error occurred" and the backend returns 403 Forbidden.
I know that the implementation in the backend was not part of this PR, but as the title of this PR is named "Fully enable project creation stepper", I'd expect functionality for non-root users as well.

When the user tries to create a project with a name that already exists, the message is "Backend not reachable" and the backend logs a SQL validation error (because the key does already exist). We should catch this in the backend and return a error message (which then automatically gets printed in the frontend by the interceptor).

In https://github.com/DSD-DBS/capella-collab-manager/commit/51ebb65d7629a7363949d8063674302a2dbeca90, the reference to the capella-dockerimages submodule was modified and points to a way older version now. Was this change intended? If not, please revert the change.


Scaling argument for exp and retract, docs, tangent vector tests

I've made a few new changes:

isapprox for tangent vectors,
scaling argument for exp and retract,
some docs,
new tests for tangent vectors,
log, exp and isapprox are not explicitly exported (they don't have to be).

Thanks, sorry for merging the previous one too fast!

No problem, opening new PRs is easy 😃 .

isn't there an infinite loop when calling log(M,x,x)? it calls zero_tangent(M,x)which calls log(M,x,x)again...?

I do not yet see the need for isapprox – could you elaborate on that?

isapprox is very useful for correctness checks of various operations. Usually you will make specialised tests for specific manifolds but isapprox allows us to have test_manifold(M::Manifold, pts::AbstractVector) which is supposed to check that given manifold conforms to the API.

And yes, there is by default an infinite loop, I wasn't sure whether one should have to implement zero_tangent_vector and log! or log and log!, this way both variants work. One or the other should probably be left unimplemented though.

And yes, there is by default an infinite loop, [...]

Can we find a way to avoid that loop? I just don't have a good idea yet.

And yes, there is by default an infinite loop, [...]

Can we find a way to avoid that loop? I just don't have a good idea yet.

Support for AD will require changing the default implementation of log anyway, I'd wait a bit with this until we have some tests for AD.


Cryptical error message when referer not provided

When referer is not provided and Puppet is configured to use WebSocket protocol, the error message is too cryptical.
See the screenshot:

Splitting the discussion from #58 (which is about the acutal JavaScript error).
This issue (#49) is about How a Starcounter app developer should get notified by Puppet that he forgot to create a session.
By now we have contradicting expectations - we want to leave it up to the developer, but the default option allows to shoot yourself in the foot with unclear error.
Puppet protocol does not actually require server to set Location / X-Location / Cookie: Location header. It is server's privilege of doing it, but in case the server does not do it, the client will just use the page URL for subsequent requests.
After #58 is fixed, the developer will see no error in such case. I think the correct error behavior should be that the server rejects application/json-patch+json request or WebSocket upgrade if there is no session.
That should solve it. I will investigate what is the current behavior and assign the tasks.

After recent fix in #58, now we have a more meaningful cavalcade of errors:

These errors come because client tries to upgrate to WebSocket when there is no session:
WebSocket connection to 'ws://localhost:8080/invoicedemo/new-invoice' failed: Error during WebSocket handshake: Unexpected response code: 200
Uncaught Error: WebSocket connection could not be made.
  Could not connect to: ws://localhost:8080/invoicedemo/new-invoicePuppetNetworkChannel.webSocketUpgrade.that._ws.onerror
WebSocket connection closed. Status code: 1006.

These errors come because client falls back to HTTP PATCH when there is no session:
PATCH http://localhost:8080/invoicedemo/new-invoice 404 (Not Found)
Uncaught Error: PuppetJs JSON response error. Server responded with error 404 Not Found
  URI not found: PATCH /invoicedemo/new-invoice

This is better than before, but still not perfect. Why WebSocket upgrade responds with 200 OK and HTTP PATCH responds with 404 Not Found? It needs to be discussed and fixed on the server side.

The issue was reported today again by @niklasbjorkman.
When you forget to set page.Session in your C#, this is the error that you get:

This is super confusing. I will think of a solution.

@alshakero how is this presented now, since remoteUrl is required?

@warpech I've never seen this issue. It's very important.
I'm still not happy with the way Palindrom deals with URLs. The current scenario is:

Remote URL is given and is only used for HTTP initial request. Even if it was the correct session URL, WebSocket will not upgrade until the HTTP response is received.
The response comes with a Location header that Palindrom uses to start a WebSocket connection. If the response didn't have that header, the WebSocket connection will be to the same URL (with ws scheme).

And this makes remoteURL useless in most cases, the needed information is within Location header anyways. And if the information was not there, we are still waiting for HTTP request to finish.
I think a video conversation would do better. My thoughts aren't clear on the issue.

And this makes remoteURL useless in most cases, the needed information is within Location header anyways. And if the information was not there, we are still waiting for HTTP request to finish.

Remote URL is needed to (1) obtain the state of the session application/json as well as to send (2) HTTP Patch application/json-patch+json changes to it. WebSocket is just an upgrade to the part (2).
So I would not call it "mostly useless", I would call it "fundamental". Let's have a chat on HO if you please.

@warpech, Of course, Remote URL is important. I just mean that in effect we are only using the one in Location header.

@warpech I am ready for HO anytime today.

We decided on HO that this issue is no longer relevant since now ~referer~ removeUrl parameter is required.


JSON: added root option to mapping, and root argument overload to…

… Object.from_json. Fixes #2642
A tricky case is root + nilable + emit_null, I think I got it right (but in any case, I don't think it matters much as I'm not sure there are some cases like this in the wild)

:heart:


Add course selection journey for courses with no vacancies

This PR adds a new step to the course selection wizard that allows candidates to select a course that is full, but redirects them to a page that explains that they can't apply to this course and what actions they can take next.
Shows the full courses on the radio buttons as well:

And adds the full corse page:

And if the provider has no more available courses:

@MaeveRoseveare I did mention this on Trello, should we keep the wording consistent to the duplicates page.

 CA: Handle courses with no vacancies

@MaeveRoseveare I implemented the changes and updated the screenshots.

@MaeveRoseveare I implemented the changes and updated the screenshots.

Thank you! This looks good. I originally thought we could remove the 'Unfortunately' but... I've changed my mind (again) and I think I prefer it. So this is good to go for my side!


update URL; revise description

@meltsufin

:ship:

I guess you wanted to do more than just update the description. :)
Let me know when it's ready for review.

PTAL @meltsufin

LGTM


⚠ Bofrost.fr has degraded performance

In 6efbcbe, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1230 ms

Resolved: Bofrost.fr performance has improved in 5d76d4c after 6 minutes.


launching a newly added project doesnt show version selector

should display it, since it doesnt have any version yet, instead of opening in latest unity version

it did show dialog


MudDrawer Exception when rendering

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudDrawer
What happened?
Getting an exception when there is rendering :
Expected behavior
No exception.
Reproduction link
No reproduction link
Reproduction steps
I do not know how to reproduce it. It happens from time to time.
Relevant log output
[14:42:34 WRN] Unhandled exception rendering component: Cannot access a disposed object.
Object name: 'Microsoft.JSInterop.DotNetObjectReference`1[[MudBlazor.BrowserViewportService, MudBlazor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]'.
System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'Microsoft.JSInterop.DotNetObjectReference`1[[MudBlazor.BrowserViewportService, MudBlazor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]'.
   at Microsoft.JSInterop.JSRuntime.TrackObjectReference[TValue](DotNetObjectReference`1 dotNetObjectReference)
   at Microsoft.JSInterop.Infrastructure.DotNetObjectReferenceJsonConverter`1.Write(Utf8JsonWriter writer, DotNetObjectReference`1 value, JsonSerializerOptions options)
   at System.Text.Json.Serialization.JsonConverter`1.TryWrite(Utf8JsonWriter writer, T& value, JsonSerializerOptions options, WriteStack& state)
   at System.Text.Json.Serialization.JsonConverter`1.TryWriteAsObject(Utf8JsonWriter writer, Object value, JsonSerializerOptions options, WriteStack& state)
   at System.Text.Json.Serialization.JsonConverter`1.TryWrite(Utf8JsonWriter writer, T& value, JsonSerializerOptions options, WriteStack& state)
   at System.Text.Json.Serialization.JsonCollectionConverter`2.OnTryWrite(Utf8JsonWriter writer, TCollection value, JsonSerializerOptions options, WriteStack& state)
   at System.Text.Json.Serialization.JsonConverter`1.TryWrite(Utf8JsonWriter writer, T& value, JsonSerializerOptions options, WriteStack& state)
   at System.Text.Json.Serialization.JsonConverter`1.WriteCore(Utf8JsonWriter writer, T& value, JsonSerializerOptions options, WriteStack& state)
   at System.Text.Json.Serialization.Metadata.JsonTypeInfo`1.Serialize(Utf8JsonWriter writer, T& rootValue, Object rootValueBoxed)
   at System.Text.Json.JsonSerializer.WriteString[TValue](TValue& value, JsonTypeInfo`1 jsonTypeInfo)
   at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, CancellationToken cancellationToken, Object[] args)
   at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
   at Microsoft.JSInterop.JSRuntimeExtensions.InvokeVoidAsync(IJSRuntime jsRuntime, String identifier, Object[] args)
   at MudBlazor.IIJSRuntimeExtentions.InvokeVoidAsyncWithErrorHandling(IJSRuntime jsRuntime, String identifier, Object[] args)
   at MudBlazor.BrowserViewportService.CreateJavaScriptListener(ResizeOptions clonedOptions, Guid observerId)
   at MudBlazor.BrowserViewportService.SubscribeAsync(IBrowserViewportObserver observer, Boolean fireImmediately)
   at MudBlazor.MudDrawer.OnAfterRenderAsync(Boolean firstRender)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)

Version (bug)
<IP_ADDRESS>
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I do not know how to reproduce it. It happens from time to time.

You'd need to post the project then in a zip as example. If it has anything sensitive, then remove it.
We need to see how the component is used and what is your project configuration overall, otherwise we can't help much.

I will try to provide one next week. Thanks for the quick feedback.

otherwise we can't help much

I made this improvement for v8 https://github.com/MudBlazor/MudBlazor/pull/9990 tho I'm not yet sure we will backport this to v7.

otherwise we can't help much

I made this improvement for v8 #9990 tho I'm not yet sure we will backport this to v7.

I pushed v7.11.0 with mentioned improvement. Let me know if it helps.

The project has been upgraded to MudBlazor 7.11.0. I am getting the following error :
2024-10-14 09:10:04.305 +02:00 [WRN] Unhandled exception rendering component: The CancellationTokenSource has been disposed.
System.ObjectDisposedException: The CancellationTokenSource has been disposed.
   at System.Threading.CancellationTokenSource.get_Token()
   at MudBlazor.BrowserViewportService.CreateJavaScriptListener(ResizeOptions clonedOptions, Guid observerId)
   at MudBlazor.BrowserViewportService.SubscribeAsync(IBrowserViewportObserver observer, Boolean fireImmediately)
   at MudBlazor.MudDrawer.OnAfterRenderAsync(Boolean firstRender)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)
2024-10-14 09:10:04.499 +02:00 [ERR] Unhandled exception in circuit 'jQD-oamrrU_2kGyVhx3CL-EI2FiA1q_HfhSnr8YqPLU'.
System.ObjectDisposedException: The CancellationTokenSource has been disposed.
   at System.Threading.CancellationTokenSource.get_Token()
   at MudBlazor.BrowserViewportService.CreateJavaScriptListener(ResizeOptions clonedOptions, Guid observerId)
   at MudBlazor.BrowserViewportService.SubscribeAsync(IBrowserViewportObserver observer, Boolean fireImmediately)
   at MudBlazor.MudDrawer.OnAfterRenderAsync(Boolean firstRender)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)

Okay, I didn't think that CancellationTokenSource.Token itself will throw when it's disposed. I thought that the underlying InvokeVoidAsync will that has the try-catch.
Blazor is very "interesting" at this point that it calls OnAfterRenderAsync just right after it called Dispose for the component. Idk why would they do that.

Will soon post new version, hope this time it will be correct.

https://www.nuget.org/packages/MudBlazor/7.12.0

Thank you @ScarletKuro ! It seems the problem is gone.


PP-5605: include charge's refunds when verifying ledger's parity

instead of only checking the charge, there will be additional calls to check whether refunds exist as well

Merging - possibly with follow up PR if we decide on logging the missing ids.


Upgrade BCDevExchange App Dependencies

This is a paid opportunity through the BCDevExchange.To apply, visit the opportunity page and submit a proposal before Thursday, July 12, 2018

This was a duplicate of #402 ...


Address follow up to https://github.com/flutter/flutter/pull/20354

This also removes the assertions failures from animation_controller_test.dart

One issue with this approach is that we have no way to notify tickers when the value of disableAnimations updates, nor can we update absorbed tickers.
In theory we could use the media query data, but that would probably be a breaking change - at least it breaks the gallery. Otherwise we could use a value listener and have every provider subscribe to the binding. that seems like the best way for now.

I've gone with the latter approach

(Modulo comments above)

Not really sure why, but this seems to have worsened our memory usage. Maybe all the listeners? (In which case maybe switching to a MediaQuery would actually be better?)

I think we should just go back to reading off of the binding and having a test specific override. We're just adding lifecycle issues (didChangeDependencies) and extra event listeners just to make what is effectively a global setting more testable.

This isn't just about making it testable. It's about reducing global state. It's sort of our philosophy in general. :-) It's the same reason for example we don't just directly access rootBundle or window.devicePixelRatio, but instead go through inherited widgets. It allows subtrees to be independently configured.
For example, suppose someone wanted to write an app that let you configure if animations were enabled or not, and wanted to show two side-by-side sections one of which showed how it works when animations are enabled and one that showed how it works when disabled.

I agree, however there is a significant fan-out problem (Ticker -> Animation Controller -> Simulation) from the Ticker which might require ... a more creative solution that what I implemented here.
I have https://github.com/flutter/flutter/pull/21296 to move back to bindings and I can go back to the drawing board when I have time (perf)

FWIW, this PR is implicated in a small performance regression on flutter_gallery_ios32__transition_perf missed_frame_build_budget_count (only on 32bit iOS). I've rebaselined the benchmark.

Oh actually it did have a slight effect on flutter_gallery__transition_perf average_frame_build_time_millis.

@Hixie could you take a look at https://github.com/flutter/flutter/pull/21296? It doesn't solve the issue of making the setting non global but it should improve performance while I work on that


Ruler labels

Given the possibility of defining multiple rulers (vertical lines), I think it might be nice to give those rulers a label/name/description, that is displayed e.g. at the top of the ruler. This can be useful for new project members to quickly understand what ruler to follow.
For example PEP 8 specifies:

Limit all lines to a maximum of 79 characters.
For flowing long blocks of text with fewer structural restrictions (docstrings or comments), the line length should be limited to 72 characters.

Thus, I have at least two rulers for working on projects that follow PEP 8. New project members might be confused why there are two different rulers defining the maximum code with. It would be helpful to have a small label at the top of the ruler like docstring and code.
Furthermore, for me personally, I have rulers activated by default for 72, 79, 80,and  88 if the project itself doesn't specify otherwise. And more often then not I might get confused. Even just putting the actual width next to the ruler would help.

I was just looking for this.
I understand it might be a bit niche, but it would indeed be helpful.
Like a sticky label either on top or the bottom of the ruler line.


Cast Array to Array<?string>

I wanna cast an Array<string> to an Array<?string>. Why isn't this possible?
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAodBjOA7AzgFzAICcBLbAc1wC4wBBY4gQwE8AeE8igPjAF4wAbQDkACQCmMeMIA0YYQHU4xGABNhAXQDcmHATABbVgCNxAZXxlKNeo1ZsA-J0q8BzqgDpcMUhnEAKAEotIA

tldr: cast to $ReadOnlyArray<?string>.
If you try to cast it to Array<?string>, writing null or undefined would be possible, which would violate the original constraint. $ReadOnlyArray<?string> is read-only, so it's safe to use (you cannot write null or undefined to it). Obviously, when you use slice, there is no reference to that original Array<string>, so you can safely use an any cast to bypass this restriction (ie. (((string.slice()): any): Array<string>))

@madbence Thanks your comment.
My goal here was though to effectively push null to it, but with a clone (with .slice()).
So I think I'll stick with ((strings.slice()): any): Array<string>).
But this seems a bit hacky, do you think so?
Shouldn't it be allowed to do
const strings: Array<string> = ['Hello', 'World'];
const maybeStrings: Array<?string> = strings.slice();
maybeStrings.push(null);

As maybeStrings is a clone of strings, strings isn't being mutated. Though this code throws an error.

/* @flow */

const strings: Array<string> = ['Hello', 'World'];
const maybeStrings: Array<?string> = Array.from(strings);
maybeStrings.push(null);

This works fine
Not sure if slice(0) or slice() should allow this behaviour or forcing upcasting to be done using Array.from is fine (if the latter this can be closed)
@goodmind


Print a backtrace for crates which fail to build

Closes #821
r? @pietroalbini

I had trouble testing this because building from the queue requires running the daemon, and the daemon will try to add every crate ever published to the queue.


Replace Github Actions for Travis

Description
Replace Github Actions for an equivalent CI with Travis
Files

cast-sh/.github/workflows/pythonpackage.yml

Tasks

[ ] Remove Githbub Actions
[ ] Set Travis CI

Add Travis badge

Also, I would like to work on this.

Okay @Benardi, I'll assign you. :rocket:


Enter Choice - not working

When i run the file, it asks for enter choice - but "yes" or "no" or any value for that matter is simply spits "You entered an incorrect value."

i guess you didn't add APIKEY,you can try it.

You could remove the API parts of the script. You just won't be able to download nsfw wallpapers 😂


Effective java.math.BigDecimal to BigDecimal converter

Under JVM, this is an often case in my projects. Converting every time to string or binary representations seems to be ineffective. Can you please provide platform-dependent effective converter?

Hi @sergeych
This issue is a duplicate of issue #250, and was also discussed in #99
This sounds like a great feature to add, unfortunately I currently don't have time to add new features, so a pull request would be very welcome. Since this is a duplicate we can close it and continue discussion in original issue #250
Cheers!


worker_threads dependency breaks on Node 10

https://nodejs.org/docs/latest-v10.x/api/worker_threads.html#worker_threads_worker_threads
We need to run Node with an extra experimental flag for this. Are there any good alternatives to keep Node 10 compat?
See-also: https://github.com/ssb-js/chloride/pull/21

All that is required is some way of detaching the buffer

Right, but are there ways to detach the buffer cleanly without any postMessage() API?

MesaageChannels was the only way I could find, but I didn’t consider node 10 at the time

@emilbayes could just make it a noop if that dep doesn't exist

I'm seeing something similar on a build using node v14.15.0, npm 6.14.8 and webpack 4.43.0. It happens after installing dat-sdk:
WARNING in ./node_modules/sodium-javascript/randombytes.js 23:6-13
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
 @ ./node_modules/sodium-javascript/index.js
 @ ./node_modules/sodium-universal/index.js
 @ ./node_modules/hypercore-crypto/index.js
 @ ./node_modules/dat-sdk/index.js
 @ ./src/containers/pages/settings.tsx
 @ ./src/containers/pages/index.ts
 @ ./src/components/content.tsx
 @ ./src/components/pages/main.tsx
 @ ./src/components/root.tsx
 @ ./src/components/index.tsx
 @ multi @babel/polyfill ./src/components/index.tsx

ERROR in ./node_modules/sodium-javascript/memory.js
Module not found: Error: Can't resolve 'worker_threads' in '/Users/sh601/gitRepos/Storefront-minima/node_modules/sodium-javascript'
 @ ./node_modules/sodium-javascript/memory.js 3:50-80
 @ ./node_modules/sodium-javascript/index.js
 @ ./node_modules/sodium-universal/index.js
 @ ./node_modules/hypercore-crypto/index.js
 @ ./node_modules/dat-sdk/index.js
 @ ./src/containers/pages/settings.tsx
 @ ./src/containers/pages/index.ts
 @ ./src/components/content.tsx
 @ ./src/components/pages/main.tsx
 @ ./src/components/root.tsx
 @ ./src/components/index.tsx
 @ multi @babel/polyfill ./src/components/index.tsx

I've managed to get my project building by hacking a couple of your files:
In memory.js, I commented out if (MessageChannel == null) ({ MessageChannel } = require('worker' + '_threads')) on line 3.
And in randombytes.js, I commented out if (require != null) { (and its associated curly brace) on line 23.
But I know 'me bad' :( Is this an easy fix?


Fix indentation of closing parentheses

This PR will fix the issue describe in https://github.com/yannickcr/eslint-plugin-react/issues/618 where the indentation of the closing parentheses on return statements was not working.
I know some tests are failing, because the existing fixtures doesn't fit with the new behaviour. Therefore I think this should be enabled with an option. Before I fix that, I would like to get some feedback if the implementation is good so far.
Fixes #618.

This is a bug fix, I don't think it should be an option - I think if the test fixtures are wrong, they are what should be fixed. I'll let @yannickcr comment on the implementation.

I see your point, but I'm a bit worried that too many tests would fail because of this change. Just my two cents. The remaining failing tests are fixed with c65bfd7b74f96044d2b778e1e4c2287891efdc93

Coverage increased (+0.01%) to 96.779% when pulling c65bfd7b74f96044d2b778e1e4c2287891efdc93 on stefanbuck:fix-618 into 82b3aa9101aa2124b934add61734cec026b4c278 on yannickcr:master.

That just means that too many tests were wrong before. When the changes are just "how many errors" and not "it wasn't an error and now it is", that's not consequential.

@yannickcr what do you think about the implementation?

I'm still interested in finishing this PR @yannickcr @ljharb

@stefanbuck sorry for the delay :(

@stefanbuck I've rebased your PR branch, but there's still some test failures, and I'm unable to push to it. Can you check the "allow edits" checkbox on the right hand side of the PR? After that, you can check out my changes and iterate from there.

@ljharb try it again, please.

@stefanbuck thanks; updated. please delete your local branch, fetch, and re-check out the latest changes, and let's see if we can get this ready to land :-)

@stefanbuck i've rebased this, but tests are failing. can you take another look?

@ljharb I'm unable to rebase because of https://github.com/yannickcr/eslint-plugin-react/issues/2525 which is super weird.

Nothing is incorrect here; commit dates are irrelevant. I already rebased the branch for you - if you delete it and check it out fresh, it should work fine.

@ljharb failing tests are fixed

@stefanbuck I've rebased this again adding output to the invalid test cases, to test the autofixer, and the tests are failing.

Rebased, and added the fixer changes needed to make the tests pass.


Establish mirror for all bazel dependencies

This should be fixing https://github.com/grpc/grpc/issues/22392 (and also internal b/151773507).
Both are caused by http_archive failing to download from github from time to time.
Also including a script that makes it easy to keep the mirror up-to-date after we update our dependencies.

Need to make a few fixes:
https://source.cloud.google.com/results/invocations/53a63db6-4858-4623-afb1-91f8780af225/targets/github%2Fgrpc%2Frun_tests%2Fsanity_linux_dbg_native%2Ftools%2Frun_tests%2Fsanity%2Fcheck_bazel_workspace.py/tests

Known failures:
https://github.com/grpc/grpc/issues/18892

Nice job! I wish we had something similar for Gradle in grpc-java. I constantly see failures to download artifacts there.

AFAIK that could be possible.
I'm not an expert on java bazel build, but the jvm_maven_import_external seems to allow multiple server_urls:
https://github.com/grpc/grpc-java/blob/d605faafc39d1bf6568a1016364fee5b77c7c02c/repositories.bzl#L297
So we could perhaps use the same approach to provide mirrors for the  maven packages we use?
CC @ejona86

Nice job! I wish we had something similar for Gradle in grpc-java. I constantly see failures to download artifacts there.

AFAIK that could be possible.
I'm not an expert on java bazel build, but the jvm_maven_import_external seems to allow multiple server_urls:
https://github.com/grpc/grpc-java/blob/d605faafc39d1bf6568a1016364fee5b77c7c02c/repositories.bzl#L297
So we could perhaps use the same approach to provide mirrors for the maven artifacts we use?
CC @ejona86

Also https://stackoverflow.com/questions/5233610/what-are-the-official-mirrors-of-the-maven-central-repository
Also
https://www.deps.co/guides/public-maven-repositories/#google-maven-central-mirror

We are using the Google mirror:
https://github.com/grpc/grpc-java/blob/d605faafc39d1bf6568a1016364fee5b77c7c02c/build.gradle#L25-L26
Plugins aren't loaded from the Google mirror, so that can still be an external source of failure, but there aren't many of those. Most failures we see are when downloading from the googleapis Maven Central mirror. https://github.com/grpc/grpc-java/issues/3284 tracks, and this is what I was complaining to Kokoro team about and saying was the need for caching.
From what I can tell, we are simply downloading so many things that the chances of one of them failing is high. In grpc repo you download on the order of 10 (22 in this PR) things. We are more like 1000. In Travis we cache those and have limited trouble.
Do also be aware that one thing that having two mirrors for Bazel might do is act as a poor-man's retry.

We are using the Google mirror:
https://github.com/grpc/grpc-java/blob/d605faafc39d1bf6568a1016364fee5b77c7c02c/build.gradle#L25-L26
Plugins aren't loaded from the Google mirror, so that can still be an external source of failure, but there aren't many of those. Most failures we see are when downloading from the googleapis Maven Central mirror. grpc/grpc-java#3284 tracks, and this is what I was complaining to Kokoro team about and saying was the need for caching.
From what I can tell, we are simply downloading so many things that the chances of one of them failing is high. In grpc repo you download on the order of 10 (22 in this PR) things. We are more like 1000. In Travis we cache those and have limited trouble.
Do also be aware that one thing that having two mirrors for Bazel might do is act as a poor-man's retry.

Well, what I meant is that by specifying two server_urls here:
https://github.com/grpc/grpc-java/blob/d605faafc39d1bf6568a1016364fee5b77c7c02c/repositories.bzl#L297 (one from the mirror, one from elsewhere) would basically be a simple way of retrying (just as you mentioned), and should reduce the rate of download failures by a lot (the chances of two different URLs both failing is pretty low)


Embedding the form in a Modal

Hi, I'm trying to embed the form in a modal window in my website so that my users can fill the form out on demand during the use of my application.  But it never allows me to see the form in the modal, just providing a link to launch it in a new window.
Can you please provide documentation on how to show this form in a bootstrap modal or via other method to popup?  the current article for this https://docs.microsoft.com/en-us/forms-pro/embed-web-page does not accurately represent the "Embed" feature in the website anymore.  See screenshot below for the view that is provided to me in the embed feature.

@sbmjais

Hi @akirby - The document you are referring to is for Forms Pro. However, the image is form Microsoft Forms. The document for Forms is: https://support.microsoft.com/en-us/office/send-a-form-and-collect-responses-2eaf3294-0cff-492d-884d-a1dee909e845

Thanks for contacting us. As we haven’t heard back from you, I am closing this issue. If you need further help, please reopen the issue or open a new issue.


Add fp16 support for various ops used by RNNs

This is a joint effort from @pooyadavoodi, @slayton58 and me.

@Yangqing It'd be good to get this looked at quickly - the size of the PR means rebasing is going to be painful..

needs rebase

Rebased (but now I need to go convert the RNNApplyLinkOp to fp16 as well).

Can you run at least the following tests

Will do. Just so you know, this code (minus rebasing) is already integrated in our internal branches, which has pretty comprehensive automated GPU testing.
@akyrola I'm thinking about trying to break this down into smaller PRs so it's easier to review and so simple things can get merged quickly without getting bogged down by more complicated parts (e.g. tests for RecurrentNetworkOps). Any concern with that?

Splitting to smaller PRs is always a good idea :)

Great. I'll start doing that. I'll note when I've addressed feedback in this PR inside new PRs.

Ok, I finally broke this all out into separate PRs. Sorry - I didn't realize how many there would be!


[springer]

Updated springer harvester to ignore biomed central
Updated springer to log when a source is not biomed central or springer

Looks good! :+1:


fix fix add broken tablet core dump

Proposed changes
Issue Number: close #17105
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall


[SPARK-20954][SQL] Simple DESCRIBE result should be compatible with previous Spark

What changes were proposed in this pull request?
After SPARK-20067, a simple DESCRIBE (without EXTENDED/FORMATTED) shows the following result. This is incompatible with Spark 2.1.1. This PR removes the column header line in case of the simple DESCRIBE command.
MASTER and BRANCH-2.2
scala> sql("desc t").show(false)
+----------+---------+-------+
|col_name  |data_type|comment|
+----------+---------+-------+
|# col_name|data_type|comment|
|a         |int      |null   |
+----------+---------+-------+

SPARK 2.1.1 and this PR
scala> sql("desc t").show(false)
+--------+---------+-------+
|col_name|data_type|comment|
+--------+---------+-------+
|a       |int      |null   |
+--------+---------+-------+

How was this patch tested?
Pass the Jenkins with the updated test suites.

Test build #77734 has started for PR 18203 at commit a0b8b1d.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77734/
Test FAILed.

Retest this please

Test build #77735 has started for PR 18203 at commit a0b8b1d.

Test build #77735 has finished for PR 18203 at commit a0b8b1d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77735/
Test FAILed.

Test build #77737 has started for PR 18203 at commit 02913a5.

Test build #77737 has finished for PR 18203 at commit 02913a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77737/
Test PASSed.

Hi, @gatorsmile .
Could you review this PR?

Hi, @gatorsmile and @cloud-fan .
Could you review this PR when you have sometime?
This will recover the incompatible changes at Spark 2.2.0.

LGTM, cc @gatorsmile for final sign-off

Test build #77812 has started for PR 18203 at commit 48329a4.

Test build #77813 has started for PR 18203 at commit 744d166.

Test build #77812 has finished for PR 18203 at commit 48329a4.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77812/
Test FAILed.

Test build #77813 has finished for PR 18203 at commit 744d166.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77813/
Test PASSed.

Merged build finished. Test PASSed.

Hi, @cloud-fan and @gatorsmile .
I updated the PR according the comment.

Oops. I'll ping again. Sorry. I didn't push it.

Test build #77819 has started for PR 18203 at commit 1eb712d.

Test build #77819 has finished for PR 18203 at commit 1eb712d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77819/
Test PASSed.

LGTM

Thank you. Please wait for a run. The last style fix is not tested by Jenkins.

Test build #77827 has started for PR 18203 at commit 5d27020.

thanks, merging to master! can you send a new PR for 2.2?

oops, I though it has passed tests... Let's see the test result then

Sorry for that. I'll start to prepare a new PR for 2.2.

Test build #77827 has finished for PR 18203 at commit 5d27020.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77827/
Test PASSed.

It passed. Also, a PR for branch-2.2, https://github.com/apache/spark/pull/18245, passed, too.

Thank you for review and merge both PRs, @gatorsmile and @cloud-fan .


Export landset image...

Hello, professor,
I want to use your GEEMAP library to export pictures. There are 300 examples in your tutorial. I am a bit confused and I don’t know how to use Google Earth.
Can you give some suggestions based on the picture I gave and which examples should I use?
There are two key points:

Add label of localtion, like A in the pic.
Display the image acquisition time.

Thank you very much.

Another question:
I use this script in jupyterlab, there are some error..
image = ee.Image('LANDSAT/LC08/C01/T1_SR/LC08_038029_20180810')

landsat_vis = {
    'bands': ['B4', 'B3', 'B2'],
    'gamma': 1.4
}
Map.addLayer(image, landsat_vis, "LE7_TOA_5YEAR/1999_2003", True, 0.7)
# Draw any shapes on the map using the Drawing tools before executing this code block
feature = Map.draw_last_feature

if feature is None:
    geom = ee.Geometry.Polygon([[[-115.413031, 35.889467],
    [-115.413031, 36.543157],
    [-114.034328, 36.543157],
    [-114.034328, 35.889467],
    [-115.413031, 35.889467]]])
    feature = ee.Feature(geom, {})

roi = feature.geometry()
import os
out_dir = os.path.join('/root/Downloads')
filename = os.path.join(out_dir, 'landsat.tif')
geemap.ee_export_image(image, filename=filename, scale=90, region=roi, file_per_band=True)

Error like this:
Generating URL ...
An error occurred while downloading.
Total request size (42441000 bytes) must be less than or equal to 33554432 bytes.

As the error message indicates, the file size exceeds the limit enforced by GEE. You need to export large images to Google Drive. https://geemap.org/notebooks/11_export_image/#export-an-image-to-google-drive


Skip over private image collection for Azure on failure

Because private image collection can be a brittle process (including outages on the Azure side, which is what inspired this PR), this modifies the Azure refresh parser to just log and skip over errors raised in the get_images method.


transport: conn, add QueryAsync

BenchmarkConnQueryIntegration-8             2233           2654839 ns/op            1251 B/op         34 allocs/op
BenchmarkConnAsyncQueryIntegration-8       10000           2088146 ns/op            1289 B/op         11 allocs/op

QueryAsync runs Query in a separated goroutine
Benchmarks moved to separate file
connTestHelper added applyFixture
requestChanSize increased to maxStreamID/2, with 1k goroutines block on putting to the channel
newQueryForStatement changed to makeQueryForStatement returning Query this removes the gc pressure

Probably the results would be even better but on some requests we face t/o due to semaphore in Scylla https://github.com/mmatczuk/scylla-go-driver/issues/116.


5.2. issues

After updating to 5.2 I noticed a few changes which seem to be bugs, but I'm not quite sure. First, the gutter is too much on the left resulting in a big distance between lapels and headings. Second, the edit buttons of code blocks are also in the gutter now. Is this expected behavior? And last, the backlinks section in the document is max width – I'm not sure if this is desired, but personally, I think it looks off.
I'm on insider 0.14.19 and newest minimal theme, as well as the corresponding plugins

Debug info (required):
SYSTEM INFO:
Obsidian version: v0.14.9
Installer version: v0.14.6
Operating system: Darwin Kernel Version 21.4.0: Fri Mar 18 00:47:26 PDT 2022; root:xnu-8020.101.4~15/RELEASE_ARM64_T8101 21.4.0
Login status: logged in
Catalyst license: insider
Insider build toggle: off
Live preview: on
Legacy editor: off
Base theme: light
Community theme: Minimal
Snippets enabled: 27
Safe mode: off
Plugins installed: 57
Plugins enabled: 57
1: Divide & Conquer
2: Advanced Mobile Toolbar
3: Better footnote
4: BibNotes Formatter
5: Calendar
6: Citation Style Translation
7: Typewriter Scroll
8: Code block from selection
9: Convert url to preview (iframe)
10: Creases
11: Quick Switcher++
12: Dataview
13: Garble Text
14: Hotkey Helper
15: Hotkeys++
16: Lapel
17: Metadata Extractor
18: Natural Language Dates
19: Advanced Slides
20: Advanced URI
21: Apply Patterns
22: Contextual Typography
23: Copy Block Link
24: Dictionary
25: Dynamic Highlights
26: Dynamic Table of Contents
27: Electron Window Tweaker
28: Focus Mode
29: Obsidian Git
30: Hider
31: Hotkeys for templates
32: Hover Editor
33: Obsidian Link Converter
34: Minimal Theme Settings
35: Pandoc Plugin
36: Quiet Outline
37: Regex Find/Replace
38: Smart Typography
39: Sortable
40: Style Settings
41: System Dark Mode
42: Tabout
43: Obsidian Tabs
44: Version History Diff
45: Zotero Desktop Connector
46: Obsidian42 - BRAT
47: Omnisearch
48: Pane Relief
49: Periodic Notes
50: Plugin Changelogs
51: Advanced Tables
52: Templater
53: Workspaces Plus
54: Block Reference Counter
55: Pandoc Reference List
56: Admonition
57: Markdown Table Editor
RECOMMENDATIONS:
Custom theme: for cosmetic issues, please first try updating your theme to latest. If still not fixed, please try to make the issue happen in the help vault or disable community theme and snippets.
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the help vault or disable community plugins.

Backlinks pane should be fixed now. The gutter is now left aligned intentionally, see release notes.


Small convenience updates

Few files added

.gitignore to avoid accidental pushing any binary into the repo
.clang-format - For now, this is only (preliminary) placeholder, but we may add automatic formatting later
update_dependencies.sh  - convenient way to install/update dependencies locally

Would it make sense to remove binary .deb files from fog_sw/tools? They could be automatically downloaded and installed in the script. In this way we can also control package versions much easier, than by replacing deb file.

@jnippula   we may also put list of ubuntu dependencies into separate file and use some tricks to install them.
https://askubuntu.com/questions/252734/apt-get-mass-install-packages-from-a-file
In this way we would only need 1 single file with such list, Dockerfile and local script may both use it

@sergey-unikie
We can also have a debian meta package(s) which do not contain any files. They just point to needed dependencies.

@jnippula we may also put list of ubuntu dependencies into separate file and use some tricks to install them.
https://askubuntu.com/questions/252734/apt-get-mass-install-packages-from-a-file
In this way we would only need 1 single file with such list, Dockerfile and local script may both use it

Yes, this is good. The dependencies could also contain the version numbers of modules, to verify that we have a consistent env.
And we should have separate build and run dep lists. Also, as Jari R. has mentioned we should consider having separate run deps for each vm running on hypervisor.


Training Data Download issue

Dear author,
I'm trying to download the Rladder and OpAmp pretraining data. Then I meet some problem here.
Rladder Data
In README, it said "You can download the tarball here and extract them in rdiv folder (size ~300MB)", which is a 404 page
OpAmp pretraining data
After environment configuration, I run the code in Jupyter notebook:
`from cgl.data.graph_data import CircuitInMemDataset
from torch_geometric.data import DataLoader
dset = CircuitInMemDataset(root='/tmp/dataset/opamp_pt', mode='train')
sample_data = dset[0]
splits = dset.splits
train_dset = dset[splits['train']]
valid_dset = dset[splits['valid']]
test_dset = dset[splits['test']]
train_dloader = DataLoader(train_dset, batch_size=16)`
Then it gives the error: BadGzipFile: Not a gzipped file (b'<!')

I figure out the problem for OpAmp dataset. When it try to download the file, google drive return a notification webpage, and said file is too large to scan virus. Then this web page will be downloaded, not the real file. So that's why python show the error:Not a gzipped file (b'<!').

I figure out the problem for OpAmp dataset. When it try to download the file, google drive return a notification webpage, and said file is too large to scan virus. Then this web page will be downloaded, not the real file. So that's why python show the error:Not a gzipped file (b'<!').

Dear,
I have the same issue. How did you solve it?

Also, the tarball Rladder Data is not working.

I figure out the problem for OpAmp dataset. When it try to download the file, google drive return a notification webpage, and said file is too large to scan virus. Then this web page will be downloaded, not the real file. So that's why python show the error:Not a gzipped file (b'<!').

Dear,
I have the same issue. How did you solve it?

You can check his code, there are google drive file ID for datasets in  graph-data.py.
Rladder datasets address was corrupted. It won't available, unless author fix it.

Hey @MohsenMokrian and @yihangyuan. I have the updated links to the file. Do you guys have a suggestion on how to share the dataset for free more permanently that would not need maintenance?

I added the links to the readme. The rladder one was anonymized before the submission. But for the other two datasets the IDs match with what I have, so I am not sure where the error is coming from.

Hey @MohsenMokrian and @yihangyuan. I have the updated links to the file. Do you guys have a suggestion on how to share the dataset for free more permanently that would not need maintenance?

Thanks a lot for adding the links.

I added the links to the readme. The rladder one was anonymized before the submission. But for the other two datasets the IDs match with what I have, so I am not sure where the error is coming from.

Thanks!


Convert 2D simulation to working 1D version

HIGH PRIORITY!
This allows us to efficiently simulate 1D scenarios for comparison with Dr Gs data.

Convert the 2D math into 1D math
Convert 2D boundary conditions and initial conditions to 1D
Plan out simulation for 1D

Created repo for this. Done


rename track -> follow

I think I got most of it....

r? @oconnor663

@maxtaco for coordination: #3643 is the GUI side of this.  Since the GUI mentions the commands to use, I think the ordering for merging down is:
#3646, then #3643, then the website.

🚢


[WIP] proposed fix for adding timing support to config policies

The goal is to address https://github.com/open-cluster-management/backlog/issues/13313 by adding an optional annotation to the ConfigurationPolicy which limits the current ConfigurationPolicy to only proceed when the one specified is Compliant.
Signed-off-by: Gus Parvin<EMAIL_ADDRESS>

/hold


whats new

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/whats-new.md
89
39
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 6bda036:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Closing the PR.

Close


[tests]: set the serial label for vmwatch test since it involves migration

Signed-off-by: Igor Bezukh<EMAIL_ADDRESS>What this PR does / why we need it:
Migration e2e tests run in serial manner. The serial label was missing in the vm watch tests.
Release note:
NONE

/cc @zcahana

Why do tests that involve migration have to be run serially?

Migration e2e tests run in serial manner. The serial label was missing in the vm watch tests.

I don't think we have a strict rule regarding to that. Since I do most of the changes which make tests run in parallel, I can say that I just did not look into most migration tests yes because there were easier targets. If the test does not cause any issues, it can definitely run in parallel. Does it cause issues?

Btw. here the readme: https://github.com/kubevirt/kubevirt/tree/main/tests#writing-e2e-tests. It is not entirely up-to-date, because the line
Tests which use PVCs or DataVolumes (parallelizing these is on the way).

is not accurate anymore.
The only by-definition excluded tests are the ones which make use of BeforeAll.

@rmohr @zcahana Thanks the heads up! my  apologies for the noise
/close


Missing bin/redis_exporter

Steps to reproduce

juju deploy redis-k8s --channel edge

Expected behavior
No error in the Juju debug logs.
Actual behavior

The charm shows an error related to not being able to start bin/redis_exporter.
Versions

Operating system: Ubuntu 24.04 LTS

Juju CLI: 3.5.1

Juju agent: 3.5.1

Charm revision: 29

microk8s: v1.30.0
Log output

Juju debug log:

unit-redis-k8s-0: 14:28:40 ERROR unit.redis-k8s/0.juju-log Uncaught exception while in charm code:
Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/./src/charm.py", line 723, in <module>
    main(RedisK8sCharm)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/main.py", line 441, in main
    _emit_charm_event(charm, dispatcher.event_name)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/main.py", line 149, in _emit_charm_event
    event_to_emit.emit(*args, **kwargs)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/framework.py", line 354, in emit
    framework._emit(event)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/framework.py", line 830, in _emit
    self._reemit(event_path)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/framework.py", line 919, in _reemit
    custom_handler(event)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/./src/charm.py", line 106, in _redis_pebble_ready
    self._update_layer()
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/./src/charm.py", line 352, in _update_layer
    container.restart("redis", "redis_exporter")
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/model.py", line 1893, in restart
    self._pebble.restart_services(service_names)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/pebble.py", line 1638, in restart_services
    return self._services_action('restart', services, timeout, delay)
  File "/var/lib/juju/agents/unit-redis-k8s-0/charm/venv/ops/pebble.py", line 1659, in _services_action
    raise ChangeError(change.err, change)
ops.pebble.ChangeError: cannot perform the following tasks:
- Start service "redis_exporter" (cannot start service: fork/exec bin/redis_exporter: no such file or directory)
----- Logs from task 0 -----
2024-06-12T14:28:39Z INFO Service "redis" has never been started.
----- Logs from task 1 -----
2024-06-12T14:28:39Z INFO Service "redis_exporter" has never been started.
----- Logs from task 3 -----
2024-06-12T14:28:40Z ERROR cannot start service: fork/exec bin/redis_exporter: no such file or directory
-----
unit-redis-k8s-0: 14:28:41 ERROR juju.worker.uniter.operation hook "redis-pebble-ready" (via hook dispatching script: dispatch) failed: exit status 1
unit-redis-k8s-0: 14:28:41 ERROR juju.worker.uniter pebble poll failed for container "redis": failed to send pebble-ready event: hook failed

Additional context

Revision 28 is working correctly.

Hi, also ran into this issue.
In the release workflow the redis-image is pinned to revision 3. This means that the updated image is not uploaded, and theredis_exporter is not found.

https://github.com/canonical/redis-k8s-operator/pull/88

It has been fixed with revision 31: juju deploy redis-k8s --channel edge will give you the required image now.

Thanks, @reneradoi!


Change the star timing units to seconds

Seconds seem to be more intuitive than what we have currently: milliseconds (see https://discord.com/channels/526188679006388254/680197233337172092/785193162246586368 and the uploads above the Discord message).

I would be willing to get this implemented for you, but I've tried to find where the time it takes to complete a level is even displayed, let alone calculated.
Could you possibly point me in the right direction?

Gladly!
This is the file for the star timings interface that appears in the top-right side of the editor: https://github.com/getkey/ble/blob/e16180aadae2a3e1c7af0450719990edff42cdb4/src/components/LevelParamsBox.tsx#L67-L80
https://github.com/getkey/ble/blob/e16180aadae2a3e1c7af0450719990edff42cdb4/src/components/LevelParamsBox.tsx#L31

And this is where the logic actually happens. Times must be stored in ms because that's how they are defined in the level file format, so a conversion has to happen before.
https://github.com/getkey/ble/blob/e16180aadae2a3e1c7af0450719990edff42cdb4/src/models/Level.ts#L13-L28

Finding time to do this now, however, I'm having some issues installing the dependencies with yarn

You might be using this command which has the same name but is unrelated to the Yarn Package Manager. Try yarn --help to get more info, if you have the right one, it will display Visit https://yarnpkg.com/en/docs/cli/ to learn more about Yarn. at the bottom.
Since you are on Ubuntu, you can install yarnpkg from apt or install it directly from the website.
Also I should mention that though yarn 2.x exists, this project uses yarn 1.x. Both links above are for this version. 🙂 (you can use yarn --version to check which version you have)

Ok thanks for the info :) I definitely had the wrong package set up. Installing now

While it looks like I got a successful compilation, proceeding to localhost:8080 leaves me with a blank page. Should I be expecting some kind of output?

That's weird. Do you have error messages in your console?
The page has the proper title so at least the HTML is downloaded by your browser but the JS might not.

Oh I see. Yes, I am getting some browser errors.

Currently using Firefox v 83.0 (it's up to date)
Should I be trying a different browser?

Looks like Firefox isn't able to use your GPU. Check about:support, it will tell you what GPU driver it's using (or not).
You can also try the page out on Chromium and if it doesn't work get similar debug information in chrome://gpu.
Btw, can you play desktop games? If you can with normal FPS and the page works fine on Chrome, then the culprit is Firefox. Otherwise, I'd place my bets on a driver issue. :slightly_smiling_face:
If Firefox is the problem, set layers.acceleration.force-enabled to true in about:config, it forces Firefox to use the GPU even on configurations that have not been tested by the Mozilla team.
If it's a driver issue, mess around Ubuntu's GUI for listing and changing your GPU drivers.

Thanks for pointing me in the right direction! Firefox was needing some adjustments, so I'm ready to start this fix now :)

While I've been able to adjust the values for the timing, I have yet to find a way to "hijack" the timings in the text boxes as they are saved to the JSON file. Tracking down where the values are extracted is proving difficult.

There is no need to hijack the timings when the JSON is saved.
The timings in the internal state of the program can stay in milliseconds, like in the JSON. But in the interface, they must be converted to seconds. When timings are changed in the interface (for example to 4s), what must be stored is 4000.
You can do the * 1000 here https://github.com/getkey/ble/blob/e16180aadae2a3e1c7af0450719990edff42cdb4/src/components/LevelParamsBox.tsx#L46-L51
And of course, https://github.com/getkey/ble/blob/e16180aadae2a3e1c7af0450719990edff42cdb4/src/components/LevelParamsBox.tsx#L72 and https://github.com/getkey/ble/blob/e16180aadae2a3e1c7af0450719990edff42cdb4/src/components/LevelParamsBox.tsx#L77, you will need to do a / 1000.

Interesting. So far I've tried to multiply the values by 1000 in LevelParamsBox, but it hasn't seemed to have an effect on the value being saved within the JSON. Testing some hard coding to see if it's the method that I'm going about doing.

Deal!


Add Greek translations

Added Greek translations. Let me know if there is anything that needs changes.

Thanks a lot! :)


All looks the same...

Have imported theme using:
include ~/.config/i3/themes/catppuccin-mocha
added color and bar colors to i3 config but it looks terrible.
Do I need to add color theme into i3 config as well? as I am unsure theme is being loaded.

Never mind added theme into config and it is working now, odd that i3 does include file....


Make keyboard scrolling optional?

I have a scrollable content area with content in it. I listen to the key up/down event to jump to the next/prev. items.
The problem is that smooth-scrollbar catch this events in (https://github.com/idiotWu/smooth-scrollbar/blob/master/src/events/keyboard.js) and handle it itself w/o propagation.
This event's are nice but I need an exception for the up/down keys.
Have you an idea how to solve that? Maybe some modification in this lib?

Hmm...it seems that more configurable options are need.

@aight8 I think it over and I'd like not to extend scrollbar options, instead scrollbar will ignore events which are preventing default behavior through event.preventDefault() method like what browsers do.
To do this you should add keydown event handler in scrollable content and call event.preventDefault() or event.stopPropagation(). Will this be OK?
See commit 69d9f3a.

Cool I bring it to work, thanks. The following thing I had to complement:
When you want prevent the scrolling behaviour on key down/up you must add a tabIndex to the scrollable content, too. Since the keydown event throws to the element only when it's active, and that's only possible if the scrollable content have a tabIndex for divs.
I tried to attach the keydown event to the scrollbar and scrollable content. Both worked (so the event is bubbling up). But if so, I wondering why the scrollbar's event will not be handled (I don't use preventDefault or preventDefault in any case).
Anyway, I got it to work how I wanted it but wondering about that behavior.

Take a look at keyboard.js L29~L37, keydown event will only be handled while scrollbar is focused. Attaching tabindex attribute to an element makes it focusable, and there will be only one element to be focused on at a time. That means, when you focus on your divs, the scrollbar will lose focus so keydown event is ignored.

@aight8 Hey I came up with a new idea! I can resolve the keypress event asynchronously, so you can simply call event.preventDefault() on window's keydown event handler:
window.onkeydown = e => e.preventDefault()

I tested it and everything goes as expected :). For more details you can take a look at commit 0026a0b.
On this way you can code like what you'll do on the real scrollbars.

OMG I can't handle keydown event on the window object, any input area in your page will crash coz I called event.preventDefault() in keydown handle :(

Oh buddy, the resolution brings another problem: I can not prevent whole document from scrolling :(

Well finally I add ignoreEvents option, see wiki here.

Good work! Works very well

Hey buddy, I'm planning to remove the ignoreEvents option due to its bad performance, instead I will provide a unregisterEvents method likes this:
instance.unregisterEvents( regexRule [, regexRule...] )

What do you think?

And I also want to know whether users like you will keep the ignoreEvents value stay unchanged during the lifecycle? If so, perhaps I can perform optimization in the other way :)

@aight8  So...ignoreEvents options is replaced by #unregisterEvents method since 5.6.0, if you updated to above v5.6.0 unconsciously and got your code broken, please change your code or use scrollbar#~5.5.0.
Sorry for the rudeness, but I don't want to bump major version :(

Closed as no response.

Library looks very promising but I couldn't find a way to remove the keyboard events, rendering my application's custom keydown handler unusable. The methods you discussed above no longer seem to exist, and removeListener doesn't seem to work for that. A simple option to disable keyboard support would be nice.

You can use the plugin system in v8.
FYI: https://github.com/idiotWu/smooth-scrollbar/blob/develop/docs/migration.md#scrollbarregisterevents-scrollbarunregisterevents

Does this mean it can only work in v8 based browsers, for instance not in Firefox?

@madprops literally it means smooth-scrollbar: ^8.0.0

Oh :)
That explain why I wasn't finding much information about it.

I can't seem to get the plugin to work. If I use the code in the example it fails because apparently can't have static properties. So I'm trying something like "static defaultOptions(){return {blacklist: []}}" which non surprisingly throws an error. I'm not using any bundler this is pure Chrome and javascript. Any idea?

@madprops FYI: https://github.com/idiotWu/smooth-scrollbar/issues/104#issuecomment-335286699


Support for Jakarta 2

Hi all,
is anyone already working on that?
I'll start looking into upgrading our client this week. in particular, from the upgrade notes

consumed_gas is deprecated and consumed_milligas should be used (parsed and exposed) instead
the deprecated big_map_diff is removed

The rest seems to concern votes, which might not be in the scope of tzgo (have not checked)
If nobody has started, I'll see if I'm able to do it

Hi,
the current master branch contains most Jakarta changes already (not version tagged, but soon).
What's already there

new hash and address types
protocol parameters
new Michelson opcodes
bigmap -> lazy diff (see func (r OperationResult) BigmapEvents())

What's coming

rollup operation parameter parsing in the RPC lib

The consumed_gas deprecation is an announcement, will go away in K or L.
Is there anything else that you need?

ah ok, I had not tried to browse the commits, just used the tags and was looking for consumed_milligas
Nope, nothing else. I wanted to do the consumed_gas thing in advance to be ready for when the field actually goes away. Not a high priority anyway. I'll put that on the side and if I come to it, I'll try to send a PR your way
(what I'm doing on my side is keeping both fields ConsumedGas and ConsumedMilliGas for backward compatibility but I want to derive the old ConsumedGas from the new ConsumedMilliGas so that when it disappears, it'll be a non-event)

Then let me add two function wrappers that return gas and milligas no matter which fields are populated. It's not a big deal.

Ok, its in the recent master branch. LMK if this is useful to you?

awesome. that looks like what I was doing on my side. thanks for the quick turnaround


Do we still need SurfaceTopography.Tools?

It seems there is only FFT stuff in there which has been superseeded by muFFT

I don't care. It is only used in Generation.


⚠ Bofrost.hr has degraded performance

In 75a44ae, Bofrost.hr (https://www.bofrost.hr) experienced degraded performance:

HTTP code: 200
Response time: 2696 ms

Resolved: Bofrost.hr performance has improved in 9df2274 after 6 minutes.


Invisible characters break ruby code

Description
I love Atom interface and functionality, but sometimes it creates an invisible character that breaks my code. Allow me to explain. When I write this, in my tests (for example):
subject(:square) { Square.new }

# ... rest of my test

When running the test, I get this error:
syntax error, unexpected tIDENTIFIER, expecting '}' (SyntaxError)
  subject(:square) { Square.new(7) }
                                    ^

What I do in this cases is to delete and rewrite the line without using autocomplete. Sometimes it works. If that does not solve the problem, I directly rewrite the line in vim.
However, that does happens with this specific character -> }.
Any ideas?
Versions
OS X 10.11.5
Atom 1.8.0
ruby 2.3.1

I've never run into this problem with Atom. Can you reproduce the problem in Safe Mode? To launch Safe Mode:

Completely exit all instances of Atom
Launch Atom with the command atom --safe

Also, do you use a non-US English keyboard layout?

I'm sorry for this odd problem. Neither I have been able to reproduce at will; it happens about once a week. I'll be using atom in safe mode for the next two weeks in order to be able to detect any kind of anomalies.
I'm using a spanish keyboard layout, it is called "Spanish ISO" in my preferences panel.

I suspect it might be something to do with your keyboard layout and atom/atom-keymap#35. But definitely let us know about the results on the Safe Mode tests. I'll leave this open for now and check back with you.

@h-salazaro Any new information?

Hi everybody,
I have also this kind if issue with invisible character.
it appear with I copy/paste, some colored text from a console, or from an atom tab which output some command runned in console (think every plugin for running tasks (grunt, rake, rspec, ...)
in console colors are rendered using some "unprintable" character. I have to edit the line in 'gedit' or any text editor ()  which support these characters.
I think Atom should handle them with a special rendering (often a square) allowing users to cop/paste/delete them. pretty much as number of texteditor are doing.

Thank you for your issue!
We haven't gotten a response to our questions in our comment https://github.com/atom/atom/issues/12041#issuecomment-232403930. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.


Broken diff editor layout

have a diff
reload vscode (have all editors closed)
go to the SCM viewlet and click a modified file
Observe the diff editor has a broken layout (look to the right overview ruler)

@rebornix I can only reproduce this when having the PR extension installed & enabled. If I disable the PR extension, then I can no longer reproduce. It might be related to the commenting support. Can you please take a look?

The gutter width was adjusted when we have comments available and somehow the overview width was affected by this. I'm looking into it now.

The width of the modified editor was not kept when we adjust the lineDecoration width.


Collaboration with Biojs

Hi genomejs, hi @contra
I am one of the dudes of BioJS.
Basically we try to write reusable JavaScript for life sciences with a special focus on visualization - so I do think that some of our goals overlap and that both of us could profit beneficial from each other. Here is a raw list of ideas that come to mind just after discovering your project:

Use existing visualizations <-> or share yours with the community
List your parsers on the BioJS registry  (we have built a registry where people can share and discover bio* related components)
Work on an easy-to-embed genome browser
Or maybe you have even an amazing idea for a GSoC project.

I do believe there are more even ways to work together - so we are very open for your ideas :)
Cheers,
Seb
Other BioJS dudes that are interested and would be happy to help include at least @daviddao, @manuelcorpas, @selewis, @nathundunn, @ljgarcia, @saketc.

Nice - I would totally be happy to merge our efforts.

Any interest in this still?


scrollable-areas: weird scrollbar position

Description

How to replicate the issue + URL
https://github.com/hydro-dev/hydro-dev.github.io/blob/docs/docs/install/compiler.md
Extension version
24.11.21
Browser(s) used
chrome 131.0
Token ⚠

[X] I set a token in the options page. I understand that some features require a token to work.

Thanks for the report. It has been fixed by #8100.

I pushed a CSS hotfix for this so you should get it soon. Can you check the Hotfix section in the options page? It should contain this https://github.com/refined-github/yolo/commit/4b1e3018391dcd1c61c4c7dde0558e5e78f09dce

My Hotfixes section is empty.

Is the name of the style fix (21.11.21.css) correct? The current extension version is 24.11.21.

Indeed. Should work now. Refresh your GHE a few times
https://www.youtube.com/watch?v=QIk_yUkisPU