Type hints

I'm in the middle of writing the LSP server I mentioned in #268, and in order to provide more useful information (such as field completions), some kind of type system is essential. Right now I'm planning HM-style type inference and external type definitions for modules just like it is in TypeScript's .d.ts files, but allowing users to define types for function signatures and let/const bindings inline would be very useful.
Would it be possible (acceptable) to add optional type hint syntax in Rhai? I am thinking along the lines of Python's type hints, or a much simpler version of TypeScript's type system.
Type hints would only serve the users and static analyzers, and they could be completely stripped when a script is compiled.
I don't have exact fleshed-out proposal for this yet as I will have to experiment more in the LSP project first, but I would like to see something like this supported in Rhai in the future.

I think something like TS's .d.ts will be great for this purpose.  The user simply loads a .d.whatever within the search path (e.g. same folder as the Rhai scripts) which includes signatures for all functions and modules registered into the Engine (possibly with documentation).  Possibly definition of external types also.
Beware that any function call is likely to come up with multiple hits due to overloading; unless your server is very smart, it is probably not feasible to determine statically what the types of the arguments are, and so it may hit more than one version of the same function.
Type hints are possible but do you really want to go there?  Implementing an entire type system in an LSP is not a trivial task...  You'd probably do well enough just by providing standard features like "go to definition" for variables and functions, detection of dead code and/or unused variables, etc.

I think something like TS's .d.ts will be great for this purpose. The user simply loads a .d.whatever within the search path (e.g. same folder as the Rhai scripts) which includes signatures for all functions and modules registered into the Engine (possibly with documentation). Possibly definition of external types also.

Yes, this is something I'm definitely planning to do, inline type hint proposal would be a follow up based on it.

Beware that any function call is likely to come up with multiple hits due to overloading; unless your server is very smart, it is probably not feasible to determine statically what the types of the arguments are, and so it may hit more than one version of the same function.

I'm aware of this, I'm currently looking into ways to handle it in HM properly. Determining the types of this are quite interesting as well.

Type hints are possible but do you really want to go there? Implementing an entire type system in an LSP is not a trivial task... You'd probably do well enough just by providing standard features like "go to definition" for variables and functions, detection of dead code and/or unused variables, etc.

I would definitely like to have some kind of type system because I'm so used to it as a developer and I hate it when an IDE goes "welp, you have this ... something that come from ... somewhere... good luck!". As for the LSP, I think this is the next major step as implementing function signature help or field completion without types while possible, feels really hackish and fragile.
So my motivation is definitely there, whether I'll have the time and dedication to implement all of this is a good question though.

My suggestion is to first have an LSP with standard functionalities.  Have it work nice and well.
Then start adding new things to it, such as type hints...
  Better nibble at it bite by bite...

I think you're going to want type hints eventually anyway, just because it makes programming so much more tractable. Basically every dynamically typed language that gains any kind of popularity has to add them eventually.
You might not want to bother implementing them yet but I think it would be worth sketching out the syntax so you can add them later without issue. E.g. Typescript has some awkward syntax with function types due to the => syntax, and Python has issues with forward declaring types.

Implementing an entire type system in an LSP is not a trivial task

I agree. It should arguably be part of the compiler. You can't really avoid the work somewhere though.

Beware that any function call is likely to come up with multiple hits due to overloading; unless your server is very smart, it is probably not feasible to determine statically what the types of the arguments are, and so it may hit more than one version of the same function.

Typescript has the same problem and it's not really an issue because the type annotations say what type the arguments are.
One final thing - you'll need to support type hints in the file (not just in .d.ts) because a) it's a much better experience and b) you can't annotate stuff in function bodies (local variables) otherwise.
Ps: great project. I've used it in a little prototype tool as a programmable config file and so far it works well!

I am thinking along the lines of Python's type hints

On that note:

https://threeofwands.com/python-is-two-languages-now-and-thats-actually-great/
https://cs.brown.edu/~sk/Publications/Papers/Published/lgmvpk-static-python/

That second link is quite interesting. I don't think I've seen a language with gradual soundness before. Though a 3.7% performance increase in Python is about as bad a result as I could imagine.
I think Dart is probably the most interesting language to take inspiration from when it comes to gradual typing and soundness, since Dart 1 was gradually typed and unsound, and Dart 2 is now fully statically typed and sound. The soundness does actually cause some big inconveniences compared to e.g. Typescript which is not sound.
Those were caused by language choices that they can't change now so if Rhai wants to do sound static typing then it's definitely worth doing as soon as possible.

Well most of these languages do not assume to work closely with another, compiled language, so their needs are more pronounced...

I also think that this idea of using static types in Rhai is very good.
On the one hand, you dont really need it, because of rust types, but on the other hand, Rhai can be used as standalone scripting/modding language for projects written in rust.
For example for game engines. Real-life example, is Godot. You can code with gdscript, c++ etc. You could use gdscript for only dynamic typed scripts and c++ for statically, but would you?
In most scenarios you dont really need the speed of c++ for game logic, but you still need you code to be typed, it makes everything more readable and much more safe. Thats why gdscript implements typing. Thats what i think

Well, it is actually quite easy to put typing into Rhai...  typing removes the possible number of states the program can be in, therefore it is a restriction and thus easy to implement.
The hairy thing is not to restrict too much.  For a dynamic language, one would expect to inter-mix different types (for example, returning () for not-found is a good example...  the value and () would be two distinct types).
Therefore, for useful typing, you need some form of union typing like TypeScript, and possibly some form of algebra to express the allowed set of types.  Thus this typing system, which is distinct from Rhai by the way, is the problematic part.

Therefore, for useful typing, you need some form of union typing like TypeScript

Yeeeahh... Thats the good idea. For example:
fn foo(a: bool) -> () | bool{
   if a {
      return true;
   } else {
      return ();
   }
}

Or:
fn foo(a: bool) -> bool? {
   if a {
      return true;
   } else {
      return ();
   }
}

As for this keyword, Rhai could use this:
fn foo(this: type1 | type2){
   // todo
}

Yeah I think basically copying Typescript is probably the way to go, including any, unknown, as hoc sum types, interfaces, generics, etc. You can probably stop short of some of the more advanced features.
The only thing I would really change from Typescript is that discriminated unions are very tedious to define, but fixing that properly requires it to be an actual language feature rather than just type annotations.

The TS type system is an absolute mess and turing complete.
https://github.com/microsoft/TypeScript/issues/14833
And I absolutely despise Typescript's as operator, because type casting in dynamic languages is horrible.
function always_returns_number(): number | string {
  return 2;
}

// Cast is necessary, because it's a `string | number`.
// It requires more typing and is still no better than JS because you can always do `as unknown as T`.
// And the worst part for me, it's hard to see in a noisy code environment.
const x: number = always_returns_number() as number;

Most types for Rust interfaces should be known from the .d.* files and a lot of bottom-up interference might be able to help with types from there.
I prefer an approach similar to the below, because it's much less difficult to miss the reason why x is a number when
parsing this code visually.
fn always_returns_number(): number | string {
  return 2;
}

let x = always_returns_number();
if typeof(x) == "number" {
  // x must be a number
}

There probably is an even better approach.

turing complete

It's a very common misconception but Turing completeness is irrelevant in most contexts, including this one. There's no real benefit to making a type system not Turing complete.


__esModule true in build/d3.js?

In the current version of d3 (4.1.1) installed via npm the built d3.js (in node_modules/d3/build/d3.js) includes the line:
Object.defineProperty(exports, '__esModule', { value: true });

Should it?
I'm finding d3 is undefined  when I import d3 from 'd3'
Babel is generating this:
var _d = require('d3');
var _d2 = _interopRequireDefault(_d);
function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }

...and later in the same file my transpiled code does this:
const layout = _d2.default.tree().size([height, width]);

However d3.js seems to be assigning directly to the module object and so _d2.default is undefined.
Can anyone help?

There’s no default export from D3, so import d3 from 'd3' isn’t supposed to work. Did you mean to say import * as d3 from 'd3'?
As for __esModule, that is something inserted by Rollup to indicate that the UMD bundle was transpiled from ES2015 modules, which is the case with all the D3 modules; see rollup/rollup#496.

:thumbsup: thank you!


feature: complete modular starter module

Overview/Summary
The PR adds an end to end example of using the new avm modules for ALZ.
This PR fixes/adds/changes/removes

#29

Breaking Changes
None
Testing Evidence
Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
As part of this Pull Request I have

[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant issues, for tracking and closure.
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.

Nice work here Jared.
I'd be tempted to remove the template file variables though, it layers in more complexity. I think being more opinionated would be better, the idea behind the complete was to somewhat replicate the scalability of caf-enterprise-scale.
Do you think we could just hard code certain variables directly in the modules in main.tf, like log_analytics_workspace_id etc. I don't think they necessarily need exposing in the config.yaml as 9 times out of 10 it will be the same, i.e. the output of the alz-management module.

Nice work here Jared.
I'd be tempted to remove the template file variables though, it layers in more complexity. I think being more opinionated would be better, the idea behind the complete was to somewhat replicate the scalability of caf-enterprise-scale.
Do you think we could just hard code certain variables directly in the modules in main.tf, like log_analytics_workspace_id etc. I don't think they necessarily need exposing in the config.yaml as 9 times out of 10 it will be the same, i.e. the output of the alz-management module.

It is quite possible I am over-engineering it. I was just thinking ahead to VBD usage and keeping it DRY. But yes, the templating could be confusing if not well documented. I will have a think about how to simplify it. I think the log_analytics_workspace_id can be removed for sure. The subscription ids might be problematic, but sure there is a way to handle that.

Nice work here Jared.
I'd be tempted to remove the template file variables though, it layers in more complexity. I think being more opinionated would be better, the idea behind the complete was to somewhat replicate the scalability of caf-enterprise-scale.
Do you think we could just hard code certain variables directly in the modules in main.tf, like log_analytics_workspace_id etc. I don't think they necessarily need exposing in the config.yaml as 9 times out of 10 it will be the same, i.e. the output of the alz-management module.

It is quite possible I am over-engineering it. I was just thinking ahead to VBD usage and keeping it DRY. But yes, the templating could be confusing if not well documented. I will have a think about how to simplify it. I think the log_analytics_workspace_id can be removed for sure. The subscription ids might be problematic, but sure there is a way to handle that.

Yeah for sure, and the folks delivering the VBDs may not necessarily be fully clued up on advanced features of terraform. Why may the subscription Ids be problematic?

@jaredfholgate see this branch for some progress:
https://github.com/Azure/terraform-azurerm-avm-ptn-alz/tree/experiment/yamlinput/examples/dynamic-input


Added strokeWidth property for line chart datasets.

Great work! thank you

This is now available in<EMAIL_ADDRESS>thanks again.


fix(api): fix run cancellation race condition bug

Closes RSS-305
Overview
Prior to this PR, run cancellations would result in protocol engine cleanup errors. This was mainly due to a bad sequence of things happening in the engine-

engine.stop was dispatching a StopAction and cancelling the queue worker, and then sending a hardware.halt command, in that order. This would result in the engine finish/cleanup method being called as soon as the queue worker was cancelled and hardware.stop() would be called as part of the cleanup.
Some background info:

hardware.halt() sets the execution manager to a cancelled state, which disallows any further movements until the execution manager is reset.
hardware.stop() resets the execution manager and attempts to home the robot.
both execution_manager methods- cancel & reset- wait on asyncio.Condition to execute

Now, as mentioned in (1), the engine cleanup begins as soon as queue worker cancellation is processed. Which means that hardware.halt() begins execution at the same time as engine.finish, and around the same time as the call to hardware.stop(). Depending on the time it takes to execute other cleanup tasks before getting to hardware stop, sometimes, the sequence would be as follows:
i. engine.finish() calls hardware.stop(), which makes execution_manager.reset gain control of asyncio.Condition lock
ii. immediately after, hardware.halt is called, which calls execution_manager.cancel but needs to wait on the condition lock.
iii. the condition lock is released after reset, execution_manager.cancel gets control of the condition and sets the state to Cancelled.
iv. right around this time, hardware.stop() calls hardware.home(), but now the state has been set to Cancelled and hence results in an ExecutionCancelledError which makes the engine abandon the cleanup and raise a RunAndFinishFailed error.

SO, this PR gets rid of this race condition by doing two things-

moving hardware.halt from inside engine.stop, and into engine finish method, before hardware.stop is called
making sure hardware.halt is fully executed before moving onto hardware.stop to avoid getting into the race condition again. We achieve this by waiting until execution manager cancellation is completed instead of starting the cancellation coroutine and moving onto the next thing before making sure it's done and the state has been updated correctly.

Test Plan

Since this PR changes the exit sequence of a run, we should run a protocol on both Flex & OT2 and confirm that:

if you cancel the protocol during a move, then the protocol cancels before the move completes
the cancellation completes cleanly- ie, the gantry homes in the end and you see a ‘RunCancelledError’ instead of protocol failed errors
if an error is raised in the protocol during runtime, the protocol still cancels correctly, drops tip and homes

Review requests

make sure this is not going to cause any change in behavior for run cancellations or error handling from 6.3.1

Risk assessment
High. Touches a pretty important and a bit hairy part of protocol execution & cleanup.

Tested the changes on OT2 and verified:

[x] if you cancel the protocol during a move, then the protocol cancels before the move completes
[x] the cancellation completes cleanly- ie, the gantry homes in the end and you see a ‘RunCancelledError’ instead of protocol failed errors
[x] if an error is raised in the protocol during runtime, the protocol still cancels correctly, drops tip and homes


Correcting the Position of Marquee tag for attracting the user on site

The marquee tag of "Get 30% OFF on your first order!" should be present on the top as it will attract the users and also they will be more likely to place order .
I can correct this issue , so please assign me this issue.
Looking for a positive response from your side.
Thank you

Please assign this issue to me @Riyasharma28 under GSSoC'24.


Added new pupil shape hexagon leaf

@aghaev Hey mate!
Thank you so much for the addition to the library! So great to be able to add new features and styles.
I've taken your code, fixed a few small missing APIs (adding the new pupil generator to the pupil factory) plus added some tests, updated the inline comments to add you in the licensing here and here, and updated the test mechanism to generate samples for the new pupil.  You can see your changes added in this commit.
You can see your new hexagonLeaf pupil type in the pupil examples.
This change was added on top of some pending additions I've made to address some Xcode issues, so your code has been added to the new tag 18.0.0.
As a result, I'll close this pull request (as the code has been integrated directly into 18.0.0).
Once again, thank you so much for making this library better!

First of all, I would like to be grateful to you because you created such a wonderful tool that will make life easier for us developers, and apologize for the fact that you had to supplement my commit, could not figure out how to make a PNG and add it to the project and tests, I did not notice them at all

Hey @aghaev, you did all the hard work by getting the drawing to work flawlessly.  For the most part, the rest of the work you probably wouldn't have known about, mostly because I haven't documented it!  And it's quite quick for me to do because I know how to do it (maybe 20 mins of work or so).
Again, many thanks for your work.  Please feel free to add more!


[How to save a model on disk?]

Question
[During the training process, how to save current model params in a file?]
Additional Context
[Add any additional information here]

Here is an example how we do it https://github.com/facebookresearch/flashlight/blob/master/flashlight/app/asr/Train.cpp#L471


segmentation fault using deflate_decompress

I've got a segmentation fault using the function deflate_decompress. Here is the test used:
(.env) @:~/wkpy/test$ python
Python 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import base64
>>> import deflate
>>> tt1 = "Sy1LzNFQt7dT10uvKs1Lzs8tKEotLtZIr8rMS8tJLEnVSEosTjUziU9JT\x635PSdUoLikqSi3TU\x43kuKTHQ\x42\x41Fr\x41\x41\x3d\x3d"
>>> tt1_decoded = base64.b64decode( tt1 )
>>> deflate.gzip_decompress( tt1_decoded )
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
deflate.DeflateError: Invalid gzip data.
>>> deflate.deflate_decompress( tt1_decoded )
Segmentation fault (core dumped)

I was trying to decompress a file found in a php file, apparently compressed with the gzdeflate() function.

I have the same problem with zlib_decompress. Using an older option also is not an option because deflate_decompress and zlib_decompress seem to be newly added in 0.5.0, the most recent version.
Can be reproduced very easily with:
python3 -c 'import zlib, deflate; deflate.zlib_decompress(zlib.compress(b"a"))'

Backtrace:
#0  deflate_zlib_decompress (self=<optimized out>, args=<optimized out>) at deflate.c:283
#1  0x00000000005127a0 in ?? ()
#2  0x00000000004e0bab in _PyObject_MakeTpCall ()
#3  0x00000000004f627a in _PyEval_EvalFrameDefault ()
#4  0x00000000005d99bf in PyEval_EvalCode ()
#5  0x00000000005f7fc7 in ?? ()
#6  0x00000000005f49c3 in ?? ()
#7  0x00000000005e9371 in PyRun_StringFlags ()
#8  0x00000000005e924a in PyRun_SimpleStringFlags ()
#9  0x0000000000606a05 in Py_RunMain ()
#10 0x00000000005cb9db in Py_BytesMain ()
#11 0x00007ffff7c28150 in __libc_start_call_main (main=main@entry=0x5cb940, argc=argc@entry=3, argv=argv@entry=0x7fffffffda18) at ../sysdeps/nptl/libc_start_call_main.h:58
#12 0x00007ffff7c28209 in __libc_start_main_impl (main=0x5cb940, argc=3, argv=0x7fffffffda18, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7fffffffda08)
    at ../csu/libc-start.c:360
#13 0x00000000005cb875 in _start ()

Line 283 is the Py_DECREF here:
    if (result != LIBDEFLATE_SUCCESS) {
        Py_DECREF(output);
        PyErr_SetString(DeflateError, "Decompression failed.");
        return NULL;
    }

Note that Py_DECREF has this warning: The object must not be NULL; if you aren’t sure that it isn’t NULL, use [Py_XDECREF()](https://docs.python.org/3/c-api/refcounting.html#c.Py_XDECREF). although I'm not sure whether this is what happens here.
It is already surprising that this is inside the non-success path. Looking further at the source code, it seems that two arguments are expected, a bytes object and a size. No idea why it can't get the size from the bytes object, but ok. Nevertheless, it shouldn't segfault on wrong usage.
This works:
python3 -c 'import zlib, deflate; compressed=zlib.compress(b"a"); print(deflate.zlib_decompress(compressed, len(compressed)))'

Which version of Python are you using, and are you using the bundled libdeflate, or a system-installed version?
dcwatson@alektra deflate % python3.11 -c 'import zlib, deflate; deflate.zlib_decompress(zlib.compress(b"a"))'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
deflate.DeflateError: Decompression failed.

I agree that the documentation and API could use improvement. As for the Py_DECREF warning, there is a NULL check above, but perhaps _PyBytes_Resize is setting it to NULL after that? I'd be happy to accept a PR with a failing test so I can reproduce/fix this. I'm also due to bump the bundled libdeflate version, so I'll probably do that soon as well.

Python 3.11.6

are you using the bundled libdeflate, or a system-installed version?

No idea. Is there a way to query that? I simply installed it via pip install deflate, so I'd assume it is the bundled one? But, I also have libdeflate 1.18 installed.

I just pushed out 0.6.0 if you want to give it a try.

Ok, very interesting. I was trying to reproduce it on my other system and couldn't. I also only got the same non-segfault error you showed.
System with segfault:

Ubuntu 23.10
Can reproduce it with Python 3.11.6 and Python 3.12.0
Pip installs manylinux2014 deflate-0.5.0-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl

System that only shows the error message:

Ubuntu 22.04.4 LTS
Can reproduce ith with Python 3.10.12, 3.11.8, 3.12.2
Pip installs manylinux2014 deflate-0.5.0-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl

I don't know what else could be different and could have an influence. I also tried listing all loaded shared libraries after importing deflate but it doesn't look as if it uses the system libdeflate.so.
import os, deflate
mappedFilesFolder = f"/proc/{os.getpid()}/map_files"
if os.path.isdir(mappedFilesFolder):
    libraries = set(
        os.readlink(os.path.join(mappedFilesFolder, link)) for link in os.listdir(mappedFilesFolder)
    )
    print(sorted(list(libraries)))

Output:
/home/user/.local/lib/python3.11/site-packages/deflate.cpython-311-x86_64-linux-gnu.so
/usr/bin/python3.11
/usr/lib/locale/locale-archive
/usr/lib/python3.11/lib-dynload/readline.cpython-311-x86_64-linux-gnu.so
/usr/lib/x86_64-linux-gnu/gconv/gconv-modules.cache
/usr/lib/x86_64-linux-gnu/ld-linux-x86-64.so.2
/usr/lib/x86_64-linux-gnu/libc.so.6
/usr/lib/x86_64-linux-gnu/libexpat.so.1.8.10
/usr/lib/x86_64-linux-gnu/libm.so.6
/usr/lib/x86_64-linux-gnu/libpthread.so.0
/usr/lib/x86_64-linux-gnu/libreadline.so.8.2
/usr/lib/x86_64-linux-gnu/libtinfo.so.6.4
/usr/lib/x86_64-linux-gnu/libz.so.1.2.13

Well, on the system, on which I can still reproduce the segfault, updating to 0.6.0 fixes it. I now get:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
TypeError: function missing required argument 'originalsize' (pos 2)

Great! This was most likely caused by PyBytes_FromStringAndSize returning an immortal object for size 0, then using that as the decompression buffer and calling _PyBytes_Resize on it.


"anyTouched" meta info doesn't change when "FieldArray" item is removed

Subject of the issue

Your environment

Software
Version(s)

redux-form
8.2.3

redux
4.0.1

react-redux
7.0.3

react
16.8.0

react-dom
16.8.0

node
13.12.0

npm if using
6.14.5

yarn if using

flow if using

typescript if using
3.8.3

Browser
87.0.4280.88 (Official Build) (64-bit)

Operating System
Win 10

Steps to reproduce

Repro: https://codesandbox.io/s/redux-form-field-arrays-example-forked-3x4nk?file=/FieldArraysForm.js
Try to remove hobby/member (or add a new one without filling in the name) and see that "anyTouched" value doesn't get changed. Though, it's changed when an existing item is modified.
Expected behaviour

"anyTouched" meta info is changed
Actual behaviour

"anyTouched" meta info isn't changed
Other information
May be linked to https://github.com/redux-form/redux-form/issues/1014

"remove" / "add new" do not trigger change

Is issue bumping allowed?

Would be nice to have an additional configuration prop to touch the form on items push/delete:
reduxForm({
  form: FormNames.PatientInformation,
  touchOnFieldPush: true,
  touchOnFieldRemove: true,
})(PatientInformationComponent)


Problems due to $HOME/.cardano-wallet being default state-dir for jormungandr and http-bridge

Release
Operating System
Cause

next
Windows & OSX & Linux
Code  v Configuration

Context
State dir has currently a default value $HOME/.cardano-wallet and it is the same for cardano-wallet-http-bridge and cardano-wallet-jormungandr. If you are a tester and play around with both backends interchangeably you may stumble upon some unexpected behaviour and I suppose that's due to the fact that that wallet.db is located in the same path for both backends, i.e. $HOME/.cardano-wallet/wallet.db
Perhaps, to avoid this behavior one might get, default state-dir could/should be different for bridge and jormungandr, e.g.:
bridge -> $HOME/.cardano-wallet-http-bridge
jormungandr -> $HOME/.cardano-wallet-jormungandr
?
Steps to Reproduce

Start cardano-wallet-http-bridge launch
Create a wallet
Start cardano-wallet-jormungandr launch
List wallets -> You can see the wallet created using cardano-wallet-http-bridge

Expected behavior
You shouldn't see the wallet in Jormungandr
Actual behavior
You do see the wallet in Jormungandr.
(and some errors in the log, I suppose, related?)
[iohk.cardano-wallet.serve.worker.61796365:Error:ThreadId 35] [2019-07-18 12:05:42.39 UTC] Failed to get next blocks: ErrGetBlockNotFound (Hash {getHash = "q@\246\184>v\b#z\SO\216t%\155\&8\239FYP\ESC-\252\176/4\"\227\231{\SYN\SOp"}).

Resolution Plan

Provide sensible default values which put state in different namespaces depending on the backend and network. This way, we have different state directory for each network, and can go from testnet to mainnet without problems.

PR

Number
Base

#566
master

#575
master

QA

Launcher tests have been updated, but only cover the case where a --state-dir is explicitly given.

When not explicitly passed in the CLI, the state directory will be in a system-specific location, properly namespaces, see also:

$ tree ~/.local/share/cardano-wallet/
/home/ktorz/.local/share/cardano-wallet/
├── http-bridge
│   ├── mainnet
│   │   ├── mainnet
│   │   │   ├── blob
│   │   │   ├── chainstate
│   │   │   ├── config.yml
│   │   │   ├── epoch
│   │   │   ├── index
│   │   │   ├── pack
│   │   │   ├── refpack
│   │   │   └── tag
│   │   ├── wallet.db
│   │   ├── wallet.db-shm
│   │   └── wallet.db-wal
│   └── testnet
│       ├── testnet
│       │   ├── blob
│       │   ├── chainstate
│       │   ├── config.yml
│       │   ├── epoch
│       │   ├── index
│       │   ├── pack
│       │   ├── refpack
│       │   └── tag
│       ├── wallet.db
│       ├── wallet.db-shm
│       └── wallet.db-wal
└── jormungandr
    └── testnet
        ├── chain
        │   ├── blocks.sqlite
        │   ├── blocks.sqlite-shm
        │   └── blocks.sqlite-wal
        ├── jormungandr-config.json
        ├── wallet.db
        ├── wallet.db-shm
        └── wallet.db-wal

Will close after #575.


#9587 exposes another error in the solver?

I tried the code example in #9587 in hope it will work thanks to injectivity. I declared Arr as injective:
type family Arr (repr :: * -> *) (a :: *) (b :: *) = (r :: *) | r -> repr a b

But alas compilation ends with an error:
Couldn't match type ‘Arr repr (Arr repr a a) (Arr repr a a)’
               with ‘Arr repr (Arr repr a0 a0) (Arr repr a0 a0)’
NB: ‘Arr’ is a type function, and may not be injective
The type variable ‘a0’ is ambiguous
Expected type: repr (Arr repr (Arr repr a a) (Arr repr a a))
  Actual type: repr (Arr repr (Arr repr a0 a0) (Arr repr a0 a0))
When checking that ‘c3’ has the inferred type
  c3 :: forall (repr :: * -> *) a.
        ESymantics repr =>
        repr (Arr repr (Arr repr a a) (Arr repr a a))
Probable cause: the inferred type is ambiguous
In the expression:
  let c3 = lam (\ f -> lam (\ x -> ...))
  in (c3 `app` (lam (\ x -> x `add` int 14))) `app` (int 0)
In an equation for ‘te4’:
    te4
      = let c3 = lam (\ f -> ...)
        in (c3 `app` (lam (\ x -> x `add` int 14))) `app` (int 0)

This error message is better than the previous one (better = some type ambiguities seem to have been resolved), but why can't we do better?

This ticket also is another example of #107

I would want input from @simonpj before continuing here.  The problem is that GHC doesn't recognize the s_aoT ~ s_aoX equation can actually lead to further improvement via injectivity annotations. This is seen as a straightforward CTyVarEq, and no type function magic happens. Note that s_aoT is the fuv for Arr aoK aoO aoO
and s_aoX is the fuv for Arr aoK aoL aoL. It's not hard to see that GHC should generated aoO ~ aoL, but this never happens. I wonder if a CTyVarEq between two fuvs is better represented as two CFunEqs. But I would imagine this would have already cropped up elsewhere -- hence my desire for Simon's input.

Aside from the original problem there might be one more issue in here:
Emitting [D] [D] _ :: s_aoT[fuv:0] ~ s_aoX[fuv:0]
Emitting [D] [D] _ :: s_aoT[fuv:0] ~ s_aoX[fuv:0]

Why is that emitted twice? I've already had a bug like this in my implementation of injectivity. I fixed it but there might be another one.

That's not problematic. It comes from the fact that we're dealing with Arr xxx s_aoT s_aoT ~ Arr xxx s_aoX s_aoX. So a naive algorithm would emit two Deriveds. A more advanced algorithm is possible, but I don't think it's worth the trouble, frankly. There is some caching that helps to reduce these, but only inerts are looked up. Right after a Derived is emitted, it's not yet inert and so the cache lookup fails. Fixing this is something to worry about later.

See a comment from Simon on #142, which addresses this bug.

(Replying to Simon's comment on #142)

In short, in TcSMonad.kickOutRewritable (assuming you are up to date with HEAD), in the line
(feqs_out, feqs_in) = partitionFunEqs kick_out_ct funeqmap

you want to have a more specialised kick_out function, one that kicks it out if the RHS of the CFunEqCan is the same as the new_tv.

Sounds like a rebase would be needed. I tend to do upstream rebases. Richard, would this interfere with anything you are currently doing on my branch? Or perhaps we don't need a rebase? I suspect that might take a day or so and we don't have too much time before the deadline.

Sounds like a rebase would be needed
I’m not sure.  But yes, the more up to date you are with HEAD the easier it is.
I’d like to be able to build this etc.  How do I do that? Is it in the main GHC repo, or just on Github?  If the latter, how do I pull from it? (I’m sure it’s easy, just have not done it recently.)  Which branch?  Etc
Simon
From: Jan Stolarek<EMAIL_ADDRESS>Sent: 14 July 2015 08:51
To: jstolarek/ghc
Cc: Simon Peyton Jones
Subject: Re: [ghc] #9587 exposes another error in the solver? (#119)
(Replying to Simon's comment on #142https://github.com/jstolarek/ghc/issues/142)
In short, in TcSMonad.kickOutRewritable (assuming you are up to date with HEAD), in the line
(feqs_out, feqs_in) = partitionFunEqs kick_out_ct funeqmap
you want to have a more specialised kick_out function, one that kicks it out if the RHS of the CFunEqCan is the same as the new_tv.
Sounds like a rebase would be needed. I tend to do upstream rebases. Richard, would this interfere with anything you are currently doing on my branch? Or perhaps we don't need a rebase? I suspect that might take a day or so and we don't have too much time before the deadline.
—
Reply to this email directly or view it on GitHubhttps://github.com/jstolarek/ghc/issues/119#issuecomment-121156157.

My branch is only on github: https://github.com/jstolarek/ghc/tree/T6018-injective-type-families
You should be able to get it like this:
git remote add jstolarek https://github.com/jstolarek/ghc
git fetch jstolarek
cd utils/haddock
git remote add jstolarek https://github.com/jstolarek/haddock
git fetch jstolarek
cd ../..
git checkout -b T6018-injective-type-families jstolarek/T6018-injective-type-families
git submodule update

My patch changes format of interface files, so you will need a full rebuild.

Rebase all you like. I do the same and know how to deal with the sequelae.

In that case I'll do a rebase this evening or tomorrow.

I applied the patch Simon mailed earlier:
--- a/compiler/typecheck/TcInteract.hs
+++ b/compiler/typecheck/TcInteract.hs
@@ -1115,8 +1115,8 @@ kickOutRewritable :: CtFlavour    -- Flavour of the equality that is
                   -> TcTyVar      -- The new equality is tv ~ ty
                   -> TcS Int
kickOutRewritable new_flavour new_eq_rel new_tv
-  | not ((new_flavour, new_eq_rel) `eqCanRewriteFR` (new_flavour, new_eq_rel))
-  = return 0  -- If new_flavour can't rewrite itself, it can't rewrite
+--  | not ((new_flavour, new_eq_rel) `eqCanRewriteFR` (new_flavour, new_eq_rel))
+--  = return 0  -- If new_flavour can't rewrite itself, it can't rewrite
               -- anything else, so no need to kick out anything
               -- This is a common case: wanteds can't rewrite wanteds
@@ -1159,7 +1159,7 @@ kick_out new_flavour new_eq_rel new_tv (IC { inert_eqs      = tv_eqs
                     , wl_implics = emptyBag }
     (tv_eqs_out, tv_eqs_in) = foldVarEnv kick_out_eqs ([], emptyVarEnv) tv_eqs
-    (feqs_out,   feqs_in)   = partitionFunEqs  kick_out_ct funeqmap
+    (feqs_out,   feqs_in)   = partitionFunEqs  kick_out_fe funeqmap
     (dicts_out,  dicts_in)  = partitionDicts   kick_out_ct dictmap
     (irs_out,    irs_in)    = partitionBag     kick_out_irred irreds
     (insols_out, insols_in) = partitionBag     kick_out_ct    insols
@@ -1168,6 +1168,11 @@ kick_out new_flavour new_eq_rel new_tv (IC { inert_eqs      = tv_eqs
     can_rewrite :: CtEvidence -> Bool
     can_rewrite = ((new_flavour, new_eq_rel) `eqCanRewriteFR`) . ctEvFlavourRole
+    kick_out_fe :: Ct -> Bool
+    kick_out_fe (CFunEqCan { cc_ev = ev, cc_fsk = fsk })
+      =  kick_out_ctev ev || fsk == new_tv
+    kick_out_fe _ = False  -- Can't happen
+
     kick_out_ct :: Ct -> Bool
     kick_out_ct ct = kick_out_ctev (ctEvidence ct)

The only difference is that now this is in TcSMonad and not TcInteract. To my surprise it does not fix this ticket -- ie. 9587 still does not work -- but it fixes #112. I'm puzzled.

Why comment out the call to eqCanRewriteFR? I don't think that's changed here.
I don't have much insight as to why this doesn't solve this bug, but I haven't yet rebuilt and such.

Why comment out the call to eqCanRewriteFR? I don't think that's changed here.

I have no idea what it does, but from experimenting I can tell that restoring call to eqCanRewriteFR:

still does not fix this ticket
restores issues with numeric literals (#112)

Fixed by Simon in 5d8a71bc514f3128d18a2d6e4ec5cf29acb68e6c


Augment warning on active plugin endpoints

Fixes #2320
Example message:
$ weave stop-plugin
WARNING: the following containers are still attached to network "weave":
  9e396c66e0d2
Docker operations involving those containers may pause or fail
while Weave is not running

I changed the text of the error message after comments.


validation: add linux default devices validation

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>

I'd like to test the major/minor eventually, but eb147670 looks like a
step in the right direction to me.

ping @mrunalp

LGTM


Tag or Branch

Hello,
It would be awesome to have a tag or a dedicated branch for old versions of gnome.
Currently I'm using Gnome 3.10 and to make this extension work perfectly I should checkout the commit 4fb64092fe1075df831ad156525a709f1ee61b6e
Anyways, good job, I love your extension ;)

You got it !


npm support

Hi, great can please add npm support? thanks!

Do you have an example or pull request for me? I have never done this before.

maybe read here, http://www.mattpalmerlee.com/2013/04/23/create-your-first-node-module-and-publish-it-to-the-npm-registry/ also is like same to bower :)

This would be nice

See latest release, let me know if this works.

I'm sorry but this doesn't worked for npm. The problem is that someone has published a fork of this package with the same name before you and now npm links the name "angular-bootstrap-checkbox" to his repository (which is also outdated).
You can see the problem in this page: the github link on the right is not your repo and the version is wrong.
This link about disputes on npm should help you to solve the problem.
If someone has the same problem it is temporary fixable by using this value in package.json:
 "angular-bootstrap-checkbox": "git://github.com/sebastianha/angular-bootstrap-checkbox.git#0.5.0"

Any news on this?

Hi,
I am very sorry for not answering for such a long time. I took the chance and just emailed the package owner of the fork if he would be so nice and would free the package name so I can publish the original code under the name "angular-bootstrap-checkbox".
I think this should solve your problem, right?

I have transferred ownership of the package. Sorry everyone!
Also, for future reference, you can install github packages like this:
"angular-bootstrap-checkbox": "sebastianha/angular-bootstrap-checkbox"

There's no need for the full git url if it's from Github. In fact, there's no need to publish at all if the repo has a package.json file (and even if it doesn't you can get around that too)

Thank you very much, I just published the package.

Thank you for the tip!


Version 1.6.0

Back-ports changes from #61, #62, #65 and #66 into V1, to be released as v1.6.0. Thanks @xjunior!

Diff looks good so I'm just going to expedite this - merging now.


Add ability to choose an existing managed environment for deployWorkspaceProject

Some of this logic can probably be eventually re-purposed into a ManagedEnvironmentListStep

Could you show me how it's used? There's been so many PRs, I'm not even sure if that part has been merged in yet.

Yeah, come to think of it, I think some of the related sections were merged while you were out, so you may not have the full context with what you already reviewed.  Let me message you offline to discuss more.


Implement shared custom data (re-worked) based on the previous work from kitgxrl

Greetings, just wanted to say this is a great library and you've done fantastic work.
Just wanted to throw my hat in the ring to solve the shared data issue.
I read the discussion on kitgxrl's PR and tried to solve the comments you mentioned.
I decided to go with the naming convention of 'transmitter' as that's really what's going on.
I stayed with Arc, as a Fat pointer caused way too many lifetime issues, although I did give it the good ol' college try, in the end an Arc isn't that much overhead.
I also fixed all the old clippy warnings that were causing the make clippy command to complain. This included removing the two client/client.rs modules as its a module naming repetition warning. I just put the code in the corresponding client/mod.rs module instead.
I wrote two comprehensive example clients for both async and raw clients and added proper documentation.
I also added an error, and went with try_transmitter which returns a Result, to ensure the library can't panic.
If there is anything else you would like done, please let me know.

@thebashpotato I'm a bit confused about the naming convention of Transmitter, could you explain a bit more? Sorry if this is a bit of a silly question. Seems like this is assuming that this is some sort of channel which isn't necessarily true. For my use case for example, this is used more as a way of shared state than just having a channel, it just happen to be one of the issues some had that implementing custom data would fix.

@1c3t3a Sure! Although, I'm still a bit confused with transmitter as the naming here, honestly.

@thebashpotato Sure, however you're able to store arbitrary data here, which was my original intention. transmitter I feel is a bit misleading of how it can actually be used. custom_data or something similar still feels a bit better as it's made clear arbitrary data is able to be stored, not just a channel sender.

@kitgxrl Fair enough, I leave the decision up to @1c3t3a as the repo owner. I'll name it whatever, in the mean time I'll continue fixing the other issues.

HI!
I like the idea of this PR also
at the same time, I'm trying to implement Custom parser for socketio Packet
so, there may be some part of code need to be changed later for supporting parser


[iOS]Crash when hot reload JS with safari JSContext console open

Issue Description
On both simulator or real iOS devices, when having the JSContext console opening from Safari,
if use cmd+R to reload javascript, the app will crash.
From the stack trace, it looks like 2 javascript environment instances have been created.
This wasn't happening on xcode7 with react-native 0.33.
Tested on a fresh installed react-native project with react-native init command.
There's a chance where first time when you reload it doesn't crash but on a second refresh, it will 100% crash.
Steps to Reproduce / Code Snippets
Start a project, open the console from Safari, then reload.
Expected Results
Shouldn't crash.
Additional Information

React Native version:0.34.1/0.34.0
Platform(s) (iOS, Android, or both?): iOS
Operating System (macOS, Linux, or Windows?): macOS 10.11.6, Xcode 8.0(8A218a), safari 10.0 (116<IP_ADDRESS>.10)

@seph14 have you found a work around or a solution to this yet? I experience the same issue with the exact same specs

@zzimbler not really. just don't forget closing the console window before you reload...

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


Having a dead mouse in inventory slots causes issues.

 Steps to repro, put live mouse on head, let it die. Enjoy cosplaying as an old man.
If it dies in pockets, that jumpsuit pocket becomes unusable semi-permanently. Steps to repro, put live mouse in pocket, let it die.
In both cases, the player is unable to remove the item from their inventory slot themselves and they need to be stripped in order to properly remove it/gain access to that slot again. May just be worth making it so you can't put mice into inventory slots and limiting them to containers only, such as backpacks or toolboxes.

Oh, this is why I was slowed as the habitual janitorial knight.


Improve explanation about IndexSet's complexity.

This PR fixes this comment in #187.
In addition to just adding an explanation only about get_index_of() method, I added Complexity section to explain IndexSet<T, S> is internally just an IndexMap<T, (), S>.

Thanks!


Minor wording/spelling/grammar and Biocontainers advice

Some minor spelling and Grammar
Anthony's suggestion RE Biocontainers

This doesn't look like a minor change given the diff. Is this mis-targetted, are you intending to reach another branch? :)
(But I'm excited! Can't wait to see this completed + merged!)

Oh jeez! Yep did not mean to merge against the upstream sorry!


[WIP] Consider the singleton's methods exported manually deprecated

Coverage decreased (-5.9%) to 94.059% when pulling aaa734893730b065f4700027d652b9e5a650ff4c on deprecate/named-methods into 23f69b6a41dae9320fa88312899700996a7f1064 on master.


Angular-cli - Remote authorization bypass vulnerability

Hi,
I would like to report authorization bypass vulnerability found in Angular-cli version 1.3.2 (latest).
In your documentation you write that you can run the Angular-cli with the following command:
ng serve --host <IP_ADDRESS> --port 4201
By doing so, it is possible to access Angular-cli server that runs on localhost and bypass the Host Header Protection.
Please contact me via email for the full report -<EMAIL_ADDRESS>What we do: SSD receives information from security researchers about yet-undisclosed
security holes in various products. Once this information is received and confirmed, we will
notify the appropriate product vendor as well as selected customers who are part of the SSD
program of the security flaw in their product or service.
What we ask: When notifying the vendor, we will provide full technical details and work
with the vendor to fix the issue, and coordinate a public advisory. In return, SSD is asking to
acknowledge the researcher for finding the vulnerability and SSD for reporting the
vulnerabilities to the vendor in official vendor advisories about this issue.

Hi,
This is a known "problem", and people are using that feature quite extensively. Please note that we write a large warning message when users are running serve in production mode, and it is not a supported use case.
The assumption that we are making (and maybe we could be clearer about it) is that you always run your development server (which is what ng serve is) in a local development environment, on a computer that's firewalled properly from the internet. We do not support serving your website to the public as a production environment.
As such, the Host header protection is of little use for a development server use case like this one.
Closing this as answered, but if you feel there are more points to make, you can either open a new issue or answer this one directly and ping me.


fix os.execute

fix utils.execute to return proper results when LuaJIT is being used with 52 compatibility enabled.
OpenResty just switched in having this enabled by default in their latest release, hence causing troubles...

@Tieske lgtm, just fix the merge conflict please.
@stevedonovan is there any reason why compat.execute doesn't get rid of exit code multiplier of 256 when under Lua 5.1 and not on Windows? I think that should be fixed.

@mpeterv Done!

Coverage increased (+0.009%) to 83.318% when pulling b9b7d85fcfe74eb3eae03df30fbe6e9be4db3967 on Tieske:fix/execute into ca74299f3c5490d15222f27b6160234166c92453 on stevedonovan:master.

Thanks!

@mpeterv  - ah, that's a good point. Fine on Windows/5.1 and Linux/5.2.  We should be consistent, so that there is indeed a guarantee that the caller gets the program exit code, not the shell-mangled form. Thanks @Tieske, I think we are covering most of the variations now.


Rename static methods for constants to const_<name> and add Board.size instance method

I did not know github anyway formats the commit name in this way.
I will rename this commit to "Rename constants to const_<name> and add Board.size instance method".


.Net: "Use for function calling" attribute for functions

There are some functions that should only be used in prompt templates. For example, all of the new history reducer plugins (top N, summarize previous conversation, retrieve relevant messages, etc.) should not be used by the AI.
To control this logic, there should be an attribute that can be added to a function to prevent the AI from ever calling this functions.

Closing as stale


AO-8850: reset transaction names map in every metrics reporting cycle

Jira: https://swicloud.atlassian.net/browse/AO-8850

Refactored the transaction names map and reset it after reporting the metrics
Some minor fixes.

Codecov Report

Merging #36 into master will increase coverage by 0.27%.
The diff coverage is 67.44%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   78.02%   78.29%   +0.27%
==========================================
  Files          25       25
  Lines        2466     2483      +17
==========================================
+ Hits         1924     1944      +20
+ Misses        491      486       -5
- Partials       51       53       +2

Impacted Files
Coverage Δ

v1/ao/internal/reporter/oboe.go
88.33% <0%> (-0.5%)
:arrow_down:

v1/ao/trace.go
80.62% <0%> (-1.27%)
:arrow_down:

v1/ao/internal/reporter/reporter_grpc.go
42.97% <0%> (+0.16%)
:arrow_up:

v1/ao/internal/reporter/context.go
92.76% <0%> (-0.85%)
:arrow_down:

v1/ao/internal/reporter/metrics.go
77.06% <85.29%> (+2.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix Cannot read property 'text' of undefined crash

This is a first time for me to send this repository.
I tried to fix the crash bug.
Fixes #32358

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

Sorry, I fixed my code.

Thanks!


Be able to configure replication network

Export REPLICATION_SERVERS=true to enable. When enabled:

[1-4].conf.d will only have configs for the data-network servers.
[5-8].conf.d will have the replication-network servers bound on ports 605x-608x, as well as configs for all background daemons.

Note that some upstream changes are required to get probe tests passing; at the very least,

account-reaper needs to be willing to look at replication_port to determine whether a device is local
container-sync needs to do the same
probe test brain probably needs to include replication servers when starting/stopping halves

So, I could get probe tests passing with https://review.opendev.org/#/c/741723/ (and its parent patch), and I could even revert the parent to see the 405s during SSYNC that led to https://launchpad.net/bugs/1446873. I think the (mostly) probe tests patch could get cleaned up a bunch, though... as it is, it's quite ugly, downright brutal.

Think I addressed all the comments. Thanks for the second set of eyes!


jquerymx download builder is out of date

The downloaded scripts are not the same as the complete download.
For example, shipped jquery version is 1.6.4

I'll try to take a look at this today.

Is there any jupiterjs repository that I can observe to see when this issue is resolved?

These changes would go into jupiterjs/javascriptmvc-site.

I'm not seeing this. I just downloaded the complete JavascriptMVC and a custom bundle from the jQueryMX download builder. Both are showing jQuery 1.7.1.

Now is fine.
Seems someone updated the site this weekend.


Use appropriate base64 flags

Signed-off-by: Simon Bein<EMAIL_ADDRESS>What this PR does / why we need it:
Adds detection whether gnu or bsd base64 package is available and
selects the correct flag. This is necessary as MacOs (or BSD) uses the
-b flag while gnu uses the -w flag. Unfortunately there is no flag for
line-wrapping that both of them support
None

/lgtm

/retest

/override pull-machine-controller-e2e-vsphere
vCenter is currently down.

/retest

/override pull-machine-controller-e2e-vsphere


prod between Wishart and WishartFast is not defined

Apparently we did not define an analytical product between Wishart and WishartFast. WishartFast is just slightly more optimized version of Wishart , which we use for messages. I propose for such a prod we simply convert Wishart to WishartFast and just call the product between two WishartFast objects.
BayesBase.default_prod_rule(::Type{<:Wishart}, ::Type{<:WishartFast}) = PreserveTypeProd(Distribution)

function BayesBase.prod(::PreserveTypeProd{Distribution}, left::Wishart, right::WishartFast)
    return prod(PreserveTypeProd(Distribution), convert(WishartFast, left), right)
end

We can put a new product right after this one. And the convert methods are already implemented here, so no extra work required for them.
@Nimrais can you add those and also implement similar for InverseWishartFast?

also the conversion does not work, e.g. convert(ExponentialFamilyDistribution, Wishart(2.0, Matrix(30.0I(2))))

https://github.com/ReactiveBayes/ExponentialFamily.jl/pull/217


Works on my macbook, but not on amazon ec2

Provide environment information
Amazon EC2 instance (Ubuntu)
/home/ubuntu/mydir/venv/bin/python
Python 3.10.12
gdown              4.7.1

My Macbook ()
/Users/me/Documents/VS Code/mydir/venv/bin/python
Python 3.10.6
gdown              4.7.1

What OS are you using?
Ubuntu
Describe the Bug
Downloading google docs file
https://docs.google.com/document/u/1/d/1xCz_3YHDp1-Cj0fWxdQHHGI1Vq2e5GUgkyjSrD9qV_Y/edit
Using URL
/home/ubuntu/mydir/venv/lib/python3.10/site-packages/gdown/parse_url.py:44:
UserWarning: You specified a Google Drive link that is not the correct link to download a file.
You might want to try `--fuzzy` option or the following url:
https://drive.google.com/uc?id=1xCz_3YHDp1-Cj0fWxdQHHGI1Vq2e5GUgkyjSrD9qV_Y
  warnings.warn(

Using ID
Access denied with the following error:

        Cannot retrieve the public link of the file. You may need to change
        the permission to 'Anyone with the link', or have had many accesses.

You may still be able to access the file from the browser:

         https://drive.google.com/uc?id=1xCz_3YHDp1-Cj0fWxdQHHGI1Vq2e5GUgkyjSrD9qV_Y

Shows this warning on Amazon EC2 when using gdown.download() And, the downloaded file is not properly downloaded.
This works well with the same parameters on my macbook.
Expected Behavior
File should be downloaded properly without warnings
To Reproduce
No response

Dont amazon blocks outbound requests by default somewhere in the config/firewall and you need to unblock it/allow it?


Added symbol for mils.

Added missing unit symbol for mils which is "th". "mil" is also used, mainly in the US.
https://simple.wikipedia.org/wiki/Thou_(unit)

Codecov Report
Base: 99.77% // Head: 99.77% // No change to project coverage :thumbsup:

Coverage data is based on head (ef839f5) compared to base (46020b1).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #34   +/-   ##
=======================================
  Coverage   99.77%   99.77%
=======================================
  Files          31       31
  Lines         905      905
=======================================
  Hits          903      903
  Misses          2        2

Impacted Files
Coverage Δ

lib/properties/length.dart
100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Thank you!


Crash after collecting metrics

Hi @all,
I justed started testing this nice software and I am experiencing some problems.
                                GaugeMetricFamily object at 0x7f6b5c323510>, <prometheus_client.core.GaugeMetricFamily object at 0x7f6b5c323990>, <prometheus_client.core.GaugeMetricFamily o                                bject at 0x7f6b5c323210>, <prometheus_client.core.GaugeMetricFamily object at 0x7f6b5c323590>, <prometheus_client.core.GaugeMetricFamily object at 0x7f6b5c32                                3d90>, <prometheus_client.core.GaugeMetricFamily object at 0x7f6b5c389690>, <prometheus_client.core.GaugeMetricFamily object at 0x7f6b5c389790>, <prometheus                                client.core.GaugeMetricFamily object at 0x7f6b5c389b10>, <prometheus_client.core.GaugeMetricFamily object at 0x7f6b5c389d50>]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/vmware_exporter/vmware_exporter.py", line 868, in _async_render_GET
yield self.generate_latest_metrics(request)
File "/usr/local/lib/python3.7/site-packages/twisted/internet/defer.py", line 1418, in _inlineCallbacks
result = g.send(result)
File "/usr/local/lib/python3.7/site-packages/vmware_exporter/vmware_exporter.py", line 916, in generate_latest_metrics
request.finish()
File "/usr/local/lib/python3.7/site-packages/twisted/web/server.py", line 268, in finish
return http.Request.finish(self)
File "/usr/local/lib/python3.7/site-packages/twisted/web/http.py", line 1071, in finish
"Request.finish called on a request after its connection was lost; "
RuntimeError: Request.finish called on a request after its connection was lost; use Request.notifyFinish to keep track of this.
^CUnhandled error in Deferred:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/twisted/internet/defer.py", line 501, in errback
self._startRunCallbacks(fail)
File "/usr/local/lib/python3.7/site-packages/twisted/internet/defer.py", line 568, in _startRunCallbacks
self._runCallbacks()
File "/usr/local/lib/python3.7/site-packages/twisted/internet/defer.py", line 654, in _runCallbacks
current.result = callback(current.result, *args, **kw)
File "/usr/local/lib/python3.7/site-packages/twisted/internet/defer.py", line 1475, in gotResult
_inlineCallbacks(r, g, status)
---  ---
File "/usr/local/lib/python3.7/site-packages/twisted/internet/defer.py", line 1416, in _inlineCallbacks
result = result.throwExceptionIntoGenerator(g)
File "/usr/local/lib/python3.7/site-packages/twisted/python/failure.py", line 512, in throwExceptionIntoGenerator
return g.throw(self.type, self.value, self.tb)
File "/usr/local/lib/python3.7/site-packages/vmware_exporter/vmware_exporter.py", line 873, in _async_render_GET
request.finish()
File "/usr/local/lib/python3.7/site-packages/twisted/web/server.py", line 268, in finish
return http.Request.finish(self)
File "/usr/local/lib/python3.7/site-packages/twisted/web/http.py", line 1071, in finish
"Request.finish called on a request after its connection was lost; "
builtins.RuntimeError: Request.finish called on a request after its connection was lost; use Request.notifyFinish to keep track of this.
If I try this with a different, much smaller Vsphere Cluster all works fine.
Can anyone give me a hint how I can fix this ?

That error means Prometheus timed out while waiting for the metrics. The exporter carried on working but when it was done there was no one there to admire its hard work :( :( you can increase the timeout on the Prometheus side and it should work. But the VMware api can be very slow

Wow ! That was fast :-)
Just tried your hint and it did the job perfectly :-)
Thanks a lot !


Refresh / Scan All folders

It would be nice to scan all but to achieve this, the users would have to edit a .cfg file or something stating :

Folders (because of different emulators)
Creating images folders in each folder

Then the script would automatically fetch the informations based on the current folder the scraper is on.
I don't know if it's doable.
Thanks for everything !

You mean reading something like es_systems.cfg to get all the roms folders and then using that to scan all the rom folders and writing the gamelists and images to the correct folders?

Yes something like. One command line and all the data would be updated without cd to each system everytime.

I added a -scrape_all flag. If you run the script on a system with ES installed it should attempt to scrape every rom folder on the system. It still places then gamelist and images in the rom folder itself.


Bump to Submariner 0.12.1

Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

/lgtm

/cc @sridhargaddam @tpantelis

/cherrypick release-2.5


Safari Browser : Cannot select track from ENCODE Modal Table

This is Safari browser specific. The ENCODE modal is does not allow any rows to be selected, as if clicking has been disabled.

@turner I think we are exceeding some max size for the table,  Encode for genome "dm6" does work on Safari.  Actually hg19 works as well, which has ~50,000 rows for signals vs ~90,000 for hg38.
Are we datatables optimally for large tables?   I hoped that filter some rows with the search box would bring the number down and allow selection but that doesn't have any effect.
I see this in the console

@turner I think the best fix for this to to further divide the Encode table,  to "Encode Signals (ChIP-Seq)" and "Encode Signals (other)".    This will yield sizes of ~50,000 and 40,000 rows respectively.    Hopefully no coding changes will be needed for this.

Transferring this to "widgets" as that is where it needs to be fixed.


.TableOfContents does not work anymore

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.60.0-F2DEA9B0/extended darwin/amd64 BuildDate: 2019-11-27T10:16:12Z

Does this issue reproduce with the latest release?
Yes
How to reproduce

Create a new Hugo site
$ hugo new site mysite
Congratulations! Your new Hugo site is created in /tmp/mysite.
...
$ cd mysite

Create a simple homepage document with just three captions:
$ cat > content/_index.md << EOF
heredoc> # Section 1
heredoc>
heredoc> Foo
heredoc>
heredoc> # Section 2
heredoc>
heredoc> Foo
heredoc>
heredoc> # Section 3
heredoc>
heredoc> Foo
heredoc> EOF

Create a home page layout which prints the table of contents:
$ echo '{{ .TableOfContents }}' > layouts/index.html

Build the web site
$ hugo

Expected behaviour
I expect an index.html document with a table of contents providing links to three headers. This is what Hugo v0.59.1 produces:
$ cat public/index.html

Section 1

Section 2

Section 3

Observed behaviour
No list items are generated in the table of contents. This is what Hugo v0.60.0 produces:
$ cat public/index.html

This issue appears to be specific to the goldmark format. Specifying
---
markup: blackfriday
---

At the top of _index.md yields a correct table of contents.

I think you will find that we have improved the TableOfContents in Hugo. The default config starts the ToC at h2, which is the most common case. But you can change it in your config.toml:
[markup]
  [markup.tableOfContents]
    endLevel = 3
    startLevel = 2

https://gohugo.io/getting-started/configuration-markup/#table-of-contents

I see. Now I have to decide whether I'd like to adjust my site configuration to restore the previous behaviour (i.e. starting the ToC at level 1) or whether it would be more sensible to go through all Markdown documents and bump the header level from n to n+1.

I think a common (and probably also recommended for SEO etc.) pattern is to have 1 H1 only per page, and that fits naturally in the template, e.g. single.html, using the .Title on the page.

A quick search-and-replace also came to my mind - alas, our web site contains a few hundred code examples with C++ source code, which happens to use a leading # character for preprocessor directives :-/).


Restore Data Monitoring menu on lnb

Is your feature request related to a problem? Please describe.
Restore Data Monitoring menu on lnb. (exclude data lineage)
Describe the solution you'd like
Restore Data Monitoring menu on lnb. (exclude data lineage)
Describe alternatives you've considered
NA
Additional context
NA

Duplicated issue created, cause of Github's incident.
I close it.


bug: wrong margin in slotted ion-label in RTL and ios and Ionic angular 5.0

Bug Report
Ionic version:

[x] 5.x
Current behavior:
ion-label has left margin=8px and right margin=0
Expected behavior:
ion-label should have right margin=8px to seperate it from the ion-icon next to it
Steps to reproduce:
1- generate a new project using the default 'list' template
2- add this to index.html <html lang="ar" dir="rtl">
3- run project in ios mode
Related code:
<ion-list lines="none">
    <ion-list-header>
      <ion-label>Resources</ion-label>
    </ion-list-header>
    <ion-item>
      <ion-icon slot="start" color="medium" name="book"></ion-icon>
      <ion-label>Ionic Documentation</ion-label>
    </ion-item>
</ion-list>

Other information:

Ionic info:
Ionic:
   Ionic CLI                     : 6.1.0 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.0.1
   @angular-devkit/build-angular : 0.900.3
   @angular-devkit/schematics    : 9.0.3
   @angular/cli                  : 9.0.3
   @ionic/angular-toolkit        : 2.1.2

Cordova:
   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : android 8.1.0, ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 12 other plugins)

Utility:
   cordova-res : 0.9.0
   native-run  : 0.3.0

System:
   ios-deploy : 1.10.0
   ios-sim    : 8.0.2
   NodeJS     : v12.14.1 (/usr/local/bin/node)
   npm        : 6.13.4
   OS         : macOS Mojave
   Xcode      : Xcode 11.3 Build version 11C29

I tried to override the margin as a workaround using:
::slotted(ion-label){
    -webkit-margin-start: 8px !important;
    margin-inline-start: 8px !important;
    -webkit-margin-end: 0 !important;
    margin-inline-end: 0 !important;
}

but it had no effect, please guide me how to fix the margins manually till you release a proper solution.

Thanks for the issue. I was able to confirm this behavior -- looks like it is only happening in Safari.

I was able to reproduce this in Chrome, it seems like it only happens in iOS. The issue is with the margin on the icon:
LTR

RTL

For some reason the CSS for margin-inline-end is not working unless I add !important, but there's nothing overriding it that I can see:

@abennouna Have you seen anything like this?

More digging has shown me that this is due to the way we set the defaults:
/// @prop - Margin top for an icon in the start/end slot
$item-ios-icon-slot-margin-top:              7px !default;

/// @prop - Margin end for an icon in the start/end slot
$item-ios-icon-slot-margin-end:              null !default;

/// @prop - Margin bottom for an icon in the start/end slot
$item-ios-icon-slot-margin-bottom:           7px !default;

/// @prop - Margin start for an icon in the start/end slot
$item-ios-icon-slot-margin-start:            0 !default;

The margin-left being set to 0 is the reason it is breaking in RTL.


Feature: Filter component

Hey, I just made a Pull Request!
Closes #2218
This is still a work in progress because I need to do some adjustments (code + style), refactoring and add tests. (There is still a lot of messy code)
This PR introduces two new types of components: the Select and the CheckboxTree.

Select

Default

Multiple

CheckboxTree

It also introduce the Filter component inside the Table.

Here is how it works:

Notes and remarks
Things left to do:

[ ] Add missing tests
[ ] Add search bar to the multiple select
[ ] Clear all: resets the data without filter but I need to reset the filters initial states
[ ] Add missing styles
[ ] Global refactoring

I have some questions about some UX/UI behaviors:

What is the behavior of the filter tab when the number of filters exceeds the size (height) of the table ? Maybe a scroll ?
I don't know in which case the CheckboxTree should have 2 levels ? So I can reproduce a working example.
Do the filters have to add up?
What I mean is if we have checked value1 for the column1 and value2 for the column2 do we have to display the data with the column1 = value1 and column2 = value2 ?
(For the moment the logic cover this type of behavior)

Moreover I find my implementation of the CheckboxTree component a bit complex.
For the items props we have something like:
[
   {
      "label":"Genereic subcategory name 1",
      "options":[
         {
            "label":"Option 1",
            "value":1
         },
         {
            "label":"Option 2",
            "value":2
         }
      ]
   }
]

And then in the component, for performance purpose we need to index each item to obtain something like this:
{
   "Genereic subcategory name 1":{
      "label":"Genereic subcategory name 1",
      "isOpen":false,
      "isChecked":false,
      "options":{
         "Option 1":{
            "label":"Option 1",
            "isChecked":false,
            "value":1
         },
         "Option 2":{
            "label":"Option 2",
            "isChecked":false,
            "value":2
         }
      }
   }
}

And we have a reducer with 3 types of actions: checkOption, checkCategory and openCategory (that should be named toggleCategory) that mutate our state which is our only source of truth.
Maybe we should split the responsibilities (a stage managing the checked options, another managing the opening categories...) to have a cleaner codebase.
But I like having an object that contains all of our properties. (This way we keep the notion of hierarchy between categories and options).
The problem with this method is that for every change I have to go deep through my object to change a specific value.
Can I have your point of view on this implementation ?
:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

😍 😍 😍

I definitely expected the groups of filters to be having an AND between them (which they do not seem to have in your gif), and OR within each group (which seems to match your gif). E.g. in the gif when you select "More data 1", I expect the result set to be empty because there are no entries that match both that, and either of 141 / 249. This is how I feel that filters generally work on e-commerce sites and similar.

Hey, @ayshiff is this good to be promoted from a draft -> PR?
I went through the UI storybook tests in chromatic and in some cases, the tables width have become smaller - can you take a look

Hey @soapraj !
I was looking to add the search bar to the multiple select and cover some edge cases (UI/UX behaviors) that I mentionned in the description:
- What is the behavior of the filter tab when the number of filters exceeds the size (height) of the table ? Maybe a scroll ?
- I don't know in which case the CheckboxTree should have 2 levels ? So I can reproduce a working example.
But I think I will promote it to a PR as we can cover these edge cases in a followup PR (As this one starts to get a little big)
Yes you are right, I'll fix that 👍

Hey @ayshiff, thanks for this contribution, much appreciated!
I think it's better to merge it sooner than later and iterate on it! Code looks pretty good to me, found a small bug, but again - maybe fix it in a separate PR: in "multiple" select story there is possibility to click the "All results", which is just a placeholder and it gets you to this weird state -

I fixed the problem that @shmidt-i reported in the multiple select.
If you are happy with the code, this PR should be ready to merge 👍


Add Ballerina to the conector list

This pull request adds the Ballerina MSSQL connector to the Connection Libraries for Microsoft SQL Database documentation. The Ballerina MSSQL connector provides seamless integration between Ballerina and Microsoft SQL Server, enabling developers to efficiently interact with MSSQL databases in their Ballerina projects.

@David-Engel
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close


Aimbot and general not working properly

Aimbot is bind in m1, and u press, kill someone, in the same time, he flicks to another head in other side of the map
after the update, the cheat stops works , all stops
sorry for the english

@sirhT https://github.com/Blaumaus/le_chiffre/releases/tag/v1.4.15


fixed the config checking for use_iam_profile

The config validation fails if you try to use use_iam_profile and don't include the access key and secret

Hey did you check out the branch use_iam_profile?

ahh. My bad feel free to dump this

I can update master from upstream and merge this in if you'd find it helpful


Not all images in Geometry3K are in name_pairs.txt

Hi,
I am currently trying to perform inference on the Geometry3K dataset to generate logic forms. The test set in Geometry3K ranges from 2401 to 3001. However, I noticed that some images are missing in the name_pairs.txt file, such as 2401, 2403, 2408, 2409, and many others. Did I make a mistake somewhere?
Thank you!

We remove all duplicated samples in Geometry3K. If you want to get all samples from 2401 to 3001, you could obtain from  other sample annotation according to images in PGDP5K.

Thank you for your reply. But how can I find the actual corresponding samples for Geometry3K test set?

according to image, the same images have the same annotaion

Thank you! I counted the number of duplicate images in the Geometry3K, and extended the name_pairs.txt file so that everyone can find the direct pair relations for each image in the Geometry3K dataset.
new_name_pairs.txt

Thanks a lot!


Incorrect syntax highlighting in heex case expression

VSCode version: 1.85.1
vscode-phoenix version: 0.1.2
The following snippet is incorrectly highlighted

    Some:
    <%= case some do %>
      <% map when is_map(map) -> %>

          Map

      <% other -> %>
        Other
    <% end %>

The tokens look OK, most likely it's unmatched bracket

it appears the -> isn't being marked correctly as non-bracket

This line is the culprit here https://github.com/phoenixframework/vscode-phoenix/blob/f98531327c04210d6d04affd35cac4cfa00fa805/language-configuration.json#L6 as it defines < > as bracket pair.
This may not be easy to fix without removing it. AFAIK VSCode does not support negative matches nor regex on brackets https://code.visualstudio.com/api/language-extensions/language-configuration-guide#brackets-definition
I had similar issues in https://github.com/elixir-lsp/vscode-elixir-ls/issues/353

you can disable symbols being used as brackets with "unbalancedBracketScopes"
for example:
"unbalancedBracketScopes": [
	"invalid.illegal.characters-not-allowed-here.html",
	"entity.other.attribute-name.html"
]


Fixing decks resource

listing decks with flashcards count is working correctly now
own decks are now included when searching by name
there are no duplicate decks when includeOwn=true
some names were changed to more suitable

Looks legit. Rebase onto new master please.

Done.

Thanks!


five services

Hello, I have a problem, that is, your compose files use different commands to start five different services. Although these five service division functions. But what I don't understand is how do you find the other four services between them?

The system uses the db to register the components to enable service-discovery since they all connect to the same DB. For docker-compose they all listen on different internal ports and use those to communicate directly. The existing docker-compose is not expected to support swarm or distributed execution. For that, the model we use in Kubernetes is more applicable and involves usign the service entries in the config.yaml of each container to specify the hostname other services should use to contact the service and uses k8s service endpoints instead of pod or container-level names. A similar approach can be taken with Swarm IIRC.

thank you very much , I understand

系统使用db注册组件以启用服务发现，因为它们都连接到同一DB。对于docker-compose，他们都在不同的内部端口上监听，并使用它们直接进行通信。现有的docker-compose不应支持群集或分布式执行。为此，我们在Kubernetes中使用的模型更适用，并且涉及在每个容器的config.yaml中使用服务条目来指定其他服务应用于联系服务的主机名，并使用k8s服务端点而不是pod或容器级别名称。Swarm IIRC可以采用类似的方法。

The system uses the db to register the components to enable service-discovery since they all connect to the same DB. For docker-compose they all listen on different internal ports and use those to communicate directly. The existing docker-compose is not expected to support swarm or distributed execution. For that, the model we use in Kubernetes is more applicable and involves usign the service entries in the config.yaml of each container to specify the hostname other services should use to contact the service and uses k8s service endpoints instead of pod or container-level names. A similar approach can be taken with Swarm IIRC.

I understand, think you very much !


(refactor) enhance chaostoolkit experiment to lend itself to orchestration by  chaos operator

Signed-off-by: ksatchit<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR consists of changes to the experiment artefacts to ensure it can be run by the chaos-operator. The changes in each component are explained below:
Chaostoolkit Dockerfile/Image

Removes the steps to start kubectl proxy in the install script as it no longer serves as entrypoint. Uses the CHAOSTOOLKIT_IN_POD env as per chaostoolkit documentation.
Adds package libcairo2-dev to aid generation of chaostoolkit reports
Ignores errors on apt autoremove/autoclean steps to account for broken network connectivity/other failures
Fixes some typos/syntax issues with package install steps

Python Script to execute chaostoolkit experiment

Runs the chaos command with --journal-path arg to output a json summary which is used to derive stage-wise success (pre-chaos steady-state check/ chaos-injection / post-chaos s.s check)
Adds functions to derive and generate/patch a chaosresult resource before and after experiment execution with appropriate verdict. Uses jinja templating & kubectl commands to achieve this.
Modularizes the code a bit for reuse & adds additional logs with same structure as the chaostoolkit

Chaostoolkit chaos manifest (experiment json)

Makes minor changes to names/keys and uses minimum toleration to make it more meaningful.

Special notes for your reviewer:

Also makes changes to the CI scripts to build the chaostoolkit image
Adds an example/reference json output of the test
Log snippets indicating successful run via the chaos-operator are provided here for reference:

Every 1.0s: kubectl get pods                                                                                Fri Feb 28 18:57:11 2020
NAME                         READY   STATUS    RESTARTS   AGE
k8-pod-delete-kyvfnr-6h9wp   1/1     Running   0          21s
nginx-65f88748fd-lkwjb       1/1     Running   0          109m
nginx-65f88748fd-q22hp       1/1     Running   0          14s
nginx-chaos-runner           1/1     Running   0          23s

Status:
  Experiments:
    Last Update Time:  2020-02-28T19:23:42Z
    Name:              k8-pod-delete-omks8z
    Status:            Execution Successful
    Verdict:           Pass
Events:
  Type    Reason               Age   From            Message
  ----    ------               ----  ----            -------
  Normal  Started ChaosEngine  62s   chaos-operator  Creating the experiment resources

Will Sync the PR and make change for naming in python wrapper


[Bug]: Lose Ultimine after death

Describe the Bug
Using Miners Certificate -> unlocks Ultimine
Player died -> Ultimine locked
On singleplayer and multiplayer game
To Reproduce

Use Miners Certificate
Player died

Catch the bug In Action!
No response
Crash Report or Log
No response
OS
Windows 11
Minecraft Version
1.19.2
ModLoader
Forge
Mod Version
1.0.2a

Checkout the new update v1.1.0 🤍


removed eureka's dependency on SECURED env var

This allows the secured flag to be set for eureka's healthcheck with the SECURED_HEALTH flag. This allows the rest of the modules to be secure will letting eureka stay open so I can handle a large amount of requests without being dependent on outside tokens.

Coverage increased (+0.002%) to 86.865% when pulling e1131157e70a31f575f57793fe93d686889dff44 on ArchanPatel890:eureka_dep_security into d53877e978f690204abc305b1a2040339c57b5a3 on cloudfoundry-incubator:master.


WIP: Add SQL array type support in createEntity

Closes: #275

[x] tests and linter show no problems (npm t)
[ ] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md
[ ] update .d.ts typings

Regarding refactoring, I only changed
if (ignoreField(name, field, def)) continue;

to
if (ignoreField(name, field, def) || !def.type) continue;

to avoid unnecessary nesting inside a for loop with if (def.type) condition that included the rest of the code.
That's why diff shows deletions and insertions of the same code snippets.
Also, I did
const maxLength = def.length?.max;

because we need that maxLength not only when adding length to a generic type but also when particularly dealing with an array type.

As of now we have unacceptable long diff for such a change. I will refactor initial code then you can rebase on master branch and update solution. Renaming to WIP (work in progress) and I'll inform you to continue. Thanks for patience.

Also, in future, it would be way to add types without changing js codebase. See metaschema, we need to extract all metadata from types and use custom types described in schemas.

Got it.


Missing authorization header in nightly version. Tag: swift-DEVELOPMENT-SNAPSHOT-2023-01-08-a

Description
Resolve package is not working. Followed after requests and found missing 'authorization' header on nightly version.
compared between swift version 5.7.2 which is latest version to Swift version 5.8-dev (LLVM d0e30b7f831ad7b, Swift f72816e547e3a9f)
Tag: swift-DEVELOPMENT-SNAPSHOT-2023-01-08-a
Latest release resolve the package successfully and the request headers are:
accept : application/vnd.swift.registry.v1+json
accept-encoding : deflate, gzip, br, zstd
accept-language : en
authorization : Basic xxxxxxxxxxx
connection : keep-alive
host : <IP_ADDRESS>:55019
user-agent : SwiftPackageManager/5.7.1

This is the curl that the client is sending:
curl '<my-regisrty-url>/test/swifttest'  -H 'accept-encoding: deflate, gzip, br, zstd' -H 'user-agent: SwiftPackageManager/5.7.1' -H 'authorization: Basic xxxxxxxxxxx' -H 'accept: application/vnd.swift.registry.v1+json' -H 'connection: keep-alive' -H 'accept-language: en' --compressed --insecure

in nightly version the Request headers are:
accept : application/vnd.swift.registry.v1+json
accept-encoding : deflate, gzip, br, zstd
accept-language : en
connection : keep-alive
host : <IP_ADDRESS>:55019
user-agent : SwiftPackageManager/5.8.0-dev

This is the curl that the client is sending:
curl '<my-regisrty-url>/test/swifttest' -H 'accept-encoding: deflate, gzip, br, zstd' -H 'accept: application/vnd.swift.registry.v1+json' -H 'user-agent: SwiftPackageManager/5.8.0-dev' -H 'connection: keep-alive' -H 'accept-language: en' --compressed --insecure

You can see that the different is that in nightly version the authorization header is missing.
Also the command is stuck and doesn't return error
Steps to reproduce

Create swift package
Set package configuration to point on registry.
Set Package.swift to resolve some package from registry.
Run 'swift package resolve' to resolve packages.

Expected behavior
I expect the resolve package to work.
Also if it doesn't work, to return some error and to not stack.

@dorsJfrog Please file dependency resolution related issue in https://github.com/apple/swift-package-manager
The issue you are experiencing is likely due to SE-0378 and the recently merged implementation.
Please try updating the user-level registries.json by adding the authentication block. For example:
{
  "registries": {
    "[default]": {
      "url": "https://example-registry.com"
    }
  },
  "authentication": {
    "example-registry.com": {
      "type": "basic"
    }
  },
  "version": 1
}

cc @tomerd

@yim-lee
Thanks for replying!

Duplicate of https://github.com/apple/swift-package-manager/issues/4358
@yim-lee Can you transfer this issue to the package manager repo?

I do not have write access to this repo. @shahmishal @tomerd Can either of you do the transfer please?


ACN_CD_SymantecEndpointProtection_Workbook_01

Proposed Changes

Symantec Endpoint Protection Workbook
2x screenshots
WorkbookMetadata updates

@Amitbergman, @liatlishams  - Can you please review the PR? Thanks.

@Amitbergman, @liatlishams  - Can you please review the PR? Thanks.

Can you please change the value of fromTemplateId to "sentinel-SymantecEndpointProtection" as it should be sentinel-workbookname. Please refer the documentation here https://github.com/Azure/Azure-Sentinel/blob/45310fe6be3cb30cbd8f073443a0c2f9144115fa/Workbooks/README.md

Can you please change the value of fromTemplateId to "sentinel-SymantecEndpointProtection" as it should be sentinel-workbookname. Please refer the documentation here https://github.com/Azure/Azure-Sentinel/blob/45310fe6be3cb30cbd8f073443a0c2f9144115fa/Workbooks/README.md

updates made

updates made

@chicduong - Can you resolve the conflicts? Thanks.

@chicduong - Can you resolve the conflicts? Thanks.

@chicduong - There are conflicts in WorkbookMetadata.json, Can you please resolve them for approval and merging? Thanks.

@chicduong - There are conflicts in WorkbookMetadata.json, Can you please resolve them for approval and merging? Thanks.

@chicduong - There are conflicts in WorkbookMetadata.json, Can you please resolve them for approval and merging? Thanks.

I believe I have made the correct updates. Let me know if theyre incorrect.

@chicduong - There are conflicts in WorkbookMetadata.json, Can you please resolve them for approval and merging? Thanks.

I believe I have made the correct updates. Let me know if theyre incorrect.


using manufacturer specific clusters / homeassistant.js update

@Koenkk, I have just submitted a pull request for some additional Smartthings devices, updated homeassistant.js entries below:
'3310-S': [
    configurations.sensor_temperature,
    configurations.sensor_battery,
],
'3315-S': [
    configurations.sensor_temperature, configurations.binary_sensor_water_leak,
    configurations.sensor_battery,
],
'F-MLT-US-2': [
    configurations.sensor_temperature, configurations.binary_sensor_contact,
    configurations.sensor_battery, configurations.binary_sensor_battery_low,
],

Some of the devices use manufacturer defined clusters. Any idea how to implement them (I have some info on what the clusters are), so I can realise the full device functionality?
Thanks.

Thanks again Rene,
These are the required changes: https://github.com/Koenkk/zcl-id/commit/77ba3058c6c4b09a9958761ae5e3046765c12872 and https://github.com/Koenkk/zcl-packet/commit/39786f72ccc5ade1d4141d9d85d9e9249ce7080f

Thanks, will have a go at it.


{Compute} az snapshot create: incremental snapshot should use CreateOption Copy instead of CopyStart

Related command

Should not use CreateOption 'CopyStart' for full snapshot creation https://github.com/Azure/azure-cli/issues/22370
For full snapshot use CreateOption Copy
For incremental snapshot use CreateOption Copy if same region and use CreateOption CopyStart if different region.
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


Resource scheduling fix

PR includes minor refactoring and more accurate resource calculation for worker slot assignment

Looks like you need to rebase this against master. Otherwise, it looks good.
Have you tested this on a cluster?

Yeah, these changes worked for us resulting in more accurate resource allocation.
I did rebase

Thanks for the patch!


withProps type name === MapProps :|

HOC
withProps(({ isFormValid }) => ({
  checkedForm: Object.values(isFormValid).every(item => item),
}))

TEST
Don't pay attention to getNestedTree. It just helps to get deep nested tree.
const component = getNestedTree(
  nestingLevel,
  EnhancedCustomForm,
  { children: mockedChildren }
);

expect(component.type.name).toBe('WithProps');

RESULT
expect(received).toBe(expected)

    Expected value to be (using Object.is):
      "WithProps"
    Received:
      "MapProps"

P.S.
nodeType: 'component',
        type:
         { [Function: MapProps] // name
           displayName: 'withProps(CustomForm)' }, // displayName
           ...

Hello.
Wanna ask whats wrong? why you check internals? why we should care about react internal details, as I understand node structures are not documented and can be changed.
BTW withProps is just a tiny wrapper over mapProps.

Hello, just noticed a small flaw.

BTW withProps is just a tiny wrapper over mapProps.
True, I have seen at the implementation :)

P.S. nothing serious, nothing serious, just can be better :)

I am going to close this ticket because there is nothing we can do here. Please feel free to reopen it if you have further thought.


Update Roadmap and Community Meeting

Description
This pr updates the Liqo roadmap and the community meeting snippet in the repo README.
It also adds some other small fixes.

/test

/merge


More fixes and stuff

...And an embracing of CSS attribution selectors because that's the direction our craft is going.


Angular Material doesn't work on Ionic 3...

Hello Ionic team.
Would you guys please make Angular Material compatible with Ionic?
I would love to use Angular Material components on Ionic but it doesn't seem to work at all on Ionic.
Please unlock this feature for us!
material.angular.io

It works :)

Hello! While we appreciate you opening an issue with us I am going to close this issue and ask that you open a new one that has the issue template filled out, please.
When filling that out, please provide further clarification on what is not working and what you are trying to accomplish. Given that both libraries essentially operate in the same space (they are both essentially libraries used to create and layout the UI) it is not clear to me why you want to use both at once. It is also not clear to me why you would not necessarily be able to, though it does seem to me, given the overlap in functionality of the two projects, that there would be a large potential for confusion or perhaps conflicting paradigms for how things get done, etc.  Basically, I think more information is required here.
You may also want to wait until version 4 is released. It may be easier to accomplish what you are trying to do with the new @ionic/core components.
Thanks for using Ionic!

@kensodemann  Tabs is one of the reason why to use material.

I am facing same issue with data tables. Material theme not applied. I am using
 import { MatTableModule } from '@angular/material/table'

Well, I tried to use it for Angular Material Tabs..
Yes, please make a tab like Angular Material Tabs for Ionic 4.

Anyone know why it doesn't work? I imported the components and add them to the app.module.ts and they not showing up

@francovp it's hard to guess. Create question on stackoverflow and provide details.

@francovp does yours work with a custom theme?


Questions about Table 7 in the paper

What losses are used for pSp training for the car inversion in Table 7, and are there moco loss used?
Thank you!

Hi @wty-ustc!
We used the L2 + LPIPS losses for the pSp comparison.
As the MOCO based loss was introduced in our paper (and not in pSp), we have not used it in the comparison.
However, config A in table 6 is equivalent to pSp with the MOCO loss (for the cars domain).
Best,
Omer


Update Scrollbar widget to use CupertinoScrollbar on macOS

Update Scrollbar widget to use CupertinoScrollbar on macOS

Fix #138447

If we are using the CupertinoScrollbar by default on iOS then it's a good idea to use it also on macOS since both use the Cupertino theme system designed by Apple
Previous PR
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

I will update the scrollbar_test.dart soon
I just need more info

Hi @Piinks
We have four falling tests
It's all because the tests expect the Scrollbar widget on macOS but after this change, it's now CupertinoScrollbar
Example:

  testWidgets('Scrollbar showTrackOnHover', (WidgetTester tester) async {
    expect(
      find.byType(Scrollbar),
      paints..rrect(
        rrect: RRect.fromRectAndRadius(
          getStartingThumbRect(isAndroid: false),
          _kDefaultThumbRadius,
        ),
        color: _kDefaultIdleThumbColor,
      ),
    );

  },
    variant: const TargetPlatformVariant(<TargetPlatform>{
      TargetPlatform.linux,
      TargetPlatform.macOS,
      TargetPlatform.windows,
    }),
  );

Sure, I noticed that, too. I will wait until the CupertinoScrollbar is updated for macOS, too, or we can close this PR
It's up to you

I will wait until the CupertinoScrollbar is updated for macOS,

I don't know that anyone is working on that right now, so I do not know when that will be updated. If you would like to work on that you are more than welcome to! Thank you!

I am going to close this for now since there have not been any updates. If you would like to continue working on this, you are welcome to return to it anytime. Thank you!


Updated deprecated df.ix[] to df.loc[] in '03.11-working-with-time-se…

Thanks – see https://www.oreilly.com/library/view/python-data-science/9781098121211/; I'll update the notebooks here soon.

Thanks but you did not yet merged the pull request. is there anything wrong with the updation work I have done?

No, it looks fine, I'm just not merging PRs because everything is about to be overwritten anyway.


Add extension methods for each LRO type to OperationExtensions

We currently have a single extension method for retrieving the result directly from an LRO, without having to use more complicated LRO patterns:
public static ValueTask<Response<IReadOnlyList<RecognizedReceipt>>> WaitForCompletionAsync(this Task<RecognizeReceiptsOperation> operation, CancellationToken cancellationToken = default);
See this thread for discussion.
We will add one such extension method for each of the LRO methods in both FormRecognizerClient and FormTrainingClient.

See file https://github.com/annelo-msft/azure-sdk-for-net/blob/db94c229fd9da844a674ee2b69633454fbf0ed56/sdk/formrecognizer/Azure.AI.FormRecognizer/src/OperationExtensions.cs in PR https://github.com/Azure/azure-sdk-for-net/pull/11048

Can we extract a base interface, implement it in every operation, and have only one extension method?

Something like:
public static async ValueTask<Response<T>> WaitForCompletionAsync<T>(this Task<Operation<T>> operation, CancellationToken cancellationToken = default)
{
    Operation<T> o = await operation.ConfigureAwait(false);
    return await o.WaitForCompletionAsync(cancellationToken).ConfigureAwait(false);
}

public static async ValueTask<Response<T>> WaitForCompletionAsync<T>(this Task<Operation<T>> operation, TimeSpan pollingInterval, CancellationToken cancellationToken = default)
{
    Operation<T> o = await operation.ConfigureAwait(false);
    return await o.WaitForCompletionAsync(pollingInterval, cancellationToken).ConfigureAwait(false);
}

Per discussion with @mayurid, addressed for Preview 1 with https://github.com/Azure/azure-sdk-for-net/pull/11444 and moving to Preview 3 to do final implementation.

@AlexanderSher, I was unable to make this work when I tried it recently, because the compiler doesn't use the where clause to use type resolution, and so it could not figure out the return type from the LRO -- the T in Response<T> in your example.  We can reopen this is you can provide a working implementation.

@annelo-msft , what code have you tried to make work? Is it this one: https://github.com/Azure/azure-sdk-for-net/pull/11359 ?