Add KafkaExceptionMapper

Add a common Kafka Exception Mapper which can be be used all rest-projects

The PR build failure looks related to the changes.

The PR build failure looks related to the changes.
test failure is not related to this change. looks like transient test failure,

Merging to master.


JOSS Review #2

This PR addresses comments by @mfroeling and @Kevin-Mattheus-Moerman.
The revised version can be tested at the joss_rev2 branch.

Comments by @Kevin-Mattheus-Moerman
Response

Missing DOIs.
https://github.com/qMRLab/qMRLab/commit/ebf5b11b10fdacbd57de95930e930aa0a819ae2b

@Kevin-Mattheus-Moerman I will ask whedon to perform a DOI check after this PR is reviewed and merged.

Comments by @mfroeling
Response

When the toolbox starts it needs a check if the needed Matlab dependencies are installed. The first few tries I had issues during data processing that were due to missing or badly installed packages. Especially for the missing packages, there should be a strong warning when starting the toolbox that it will not run properly.
The GUI documentation suggests that users run startup.m before starting the GUI. Normally, startup.m checks whether the Image Processing and Optimization toolboxes are installed. However, I agree that is a fundamental requirement and should be controlled for even when the startup is not run in advance. Resolved by https://github.com/qMRLab/qMRLab/commit/c03f67fc91f30b3cfcc0b56186adf243b951593b.

Example data is provided for the T1 relaxometry in the documentation. After some browsing the GUI I found the download example button also for other modalities. This should be better documented. I started with the command line and there I could not find a list of available example data.
qMRLab provides an example dataset per model. I created a new section in the documentation about example datasets for clarity  https://github.com/qMRLab/qMRLab/commit/3c1bd2b7982a50bfb49423e9dec0630c5e9f518e.

Each model needs a specific data setup that is apparently inferred from the files in the folder. One can load each data needed separately or by selecting the containing folder. However, for me, it is not clear how the data structure/naming should be for every type of data supported if one wants to used the predefined data format. For example, when additional data quantitative data is loaded how should this be stored (e.g. B1 can be fraction or percent).
The convention for specific data setups (i.e. data names) are only followed for example datasets to make systematic script generation easier for CI tests and to automatically load inputs after Download Example link in the GUI is clicked. The new documentation makes a note about it.  Information regarding the data structure is fetched from the header comments of the class definitions. Previously, users had to hover the cursor over the input labels, or click the ? button to access this information. Commit https://github.com/qMRLab/qMRLab/commit/6c14248218dc854e431ee665c55dc0a698a3d44a now exposes this information to the input (text) box for user's convenience. Classdef headers were revisited to make input data descriptions more informative. For example, now it is obvious that B1+ maps are expected to be normalized.

I loaded one of my own diffusion sets and the bval/bvec files were not loaded or asked for. without this information the tool still allowed me to click the fit button resulting in errors. Only after some exploration I found the load bval/bvec in the options pannel. Nowhere in the help of the diffusion does it state that I have to load my own bval/bvec files.
The default Protocol parameters are intended for the example datasets. A warning is added to the new documentation, advising users against using default parameters to fit their own data.  I agree that the protocol tables of the diffusion models are more crowded compared to the other models, which makes them less favorable to populate using Add Remove buttons located at the bottom of the Options panel. Despite that alternative ways existed, I noticed that they were not well documented. Thank you so much for bringing this to my attention! Commit https://github.com/qMRLab/qMRLab/commit/3c1bd2b7982a50bfb49423e9dec0630c5e9f518e adds a help button to the protocol panel for the models with tabletip property and gives a way to open a web page that contains information about how to populate respective protocols. This feature is made available for all the diffusion models, please see the Screenshot-1 below.

different models ask for different input files. This is nicely provided in the GUI however I would be nice if mandatory and optional inputs would be clear. For example, with DTI processing I don't need a mask or SigmaNoise to be able to fit a Tensor.
Commit https://github.com/qMRLab/qMRLab/commit/6c14248218dc854e431ee665c55dc0a698a3d44a ensures that optional/required input conditions do not escape user's attention. Please see the Screenshot-2.

When I loaded the example diffusion data and then changed the input to one of my own datasets the mask of the previous dataset remained in the Mask input field only after clicking browse and cancel is was cleared.
The previous Browse button is replaced with two buttons (+ and -) to add/remove inputs https://github.com/qMRLab/qMRLab/commit/8de0641cc9a57606c2869137433ca0beb1fda01c. Please see the Screenshot-3.  In addition, when the Path data is set to a directory, + buttons prompts the browser from that directory, which was previously always pointing to the current folder regardless of what Path data contains.

Thank you so much @mfroeling for your constructive and practical comments, all of which helped me improve user friendliness of the GUI and the documentation.

Screenshot-1

Screenshot-2

Screenshot-3

Coverage decreased (-0.1%) to 41.4% when pulling b65f80ef06ed616d08a3a945cdd46c685c73c905 on joss_rev2 into e51d316fa542620bb7c703846caf586a9481a8f3 on master.

@agahkarakuzu great work! GUIs are always a tough challenge especially with complicated models with many inputs. The changes have made a big difference in user-friendliness.

Thank you @mfroeling! If you are happy with the changes, I will merge this branch following @mathieuboudreau's review.

yes im happy with the changes

I reviewed this PR and resolved some issues, I will merge the branch after CI tests.
Manual GUI tests (general)

[x] Does the qMRLab GUI open?
[x] Can you change models?
[x] Can you load a data folder for a model?
[x] Can you view data?
[x] Can you zoom in the image?
[x] Can you pan out of the image?
[x] Can you view the histogram of the data?
[x] Can you change the color map?
[x] Can you fit dataset (Fit data)?
[x] Can you save/load the results?
[x] Can you open the options panel?
[x] Can you change option parameters?
[x] Can you save/load option paramters?
[x] Can you select a voxel?
[x] Can you fit the data of that voxel ("View data fit")?
[x] Can you simulate and fit a voxel ("Single Voxel Curve")?
[x] Can you run a Sensitivity Analysis?
[x] Can you simulate a Multi Voxel Distribution?


Flesh out more things in the repo.

Add LICENSE
Add CODE_OF_CONDUCT.md
Add .editorconfig
Add eslint (giant yak shave led to me creating https://github.com/rwjblue/qunit-eslint)
Copy ember-cli's eslint configuration
Fixup any lint failures

Can we remove the dangling comma rule?

We can (I argued against it also when it landed in ember-cli), but I think its valuable to use as close to the exact rules as ember-cli as we can (the only difference in this PR and ember-cli master is WRT mocha/chai stuff).
I actually had half a mind to extract the ember-cli config into eslint-config-ember-cli to remove duplication and make it easier for other github.com/ember-cli projects...

@chadhietala - Removed the comma dangle rule for now, we can discuss separately if we want to bring it back.


In cdt.causality.pairwise , the "RCC" example uses "Jarfo"

Hi,
minor problem, just a confusing example I found through experiments, see below
class RCC(PairwiseModel):
    """Randomized Causation Coefficient model. 2nd approach in the Fast
    Causation challenge.
    **Description:** The Randomized causation coefficient (RCC) relies on the
    projection of the empirical distributions into a RKHS using random cosine
    embeddings, then classfies the pairs using a random forest based on those
    features.
    **Data Type:** Continuous, Categorical, Mixed
    **Assumptions:** This method needs a substantial amount of labelled causal
    pairs to train itself. Its final performance depends on the training set
    used.
    Args:
        rand_coeff (int): number of randomized coefficients
        nb_estimators (int): number of estimators
        nb_min_leaves (int): number of min samples leaves of the estimator
        max_depth (): (optional) max depth of the model
        s (float): scaling
        njobs (int): number of jobs to be run on parallel (defaults to ``cdt.SETTINGS.NJOBS``)
        verbose (bool): verbosity (defaults to ``cdt.SETTINGS.verbose``)
    .. note::
       Ref : Lopez-Paz, David and Muandet, Krikamol and Schölkopf, Bernhard and Tolstikhin, Ilya O,
       "Towards a Learning Theory of Cause-Effect Inference", ICML 2015.
    Example:
        >>> from cdt.causality.pairwise import RCC
        >>> import networkx as nx
        >>> import matplotlib.pyplot as plt
        >>> from cdt.data import load_dataset
        >>> from sklearn.model_selection import train_test_split
        >>> data, labels = load_dataset('tuebingen')
        >>> X_tr, X_te, y_tr, y_te = train_test_split(data, labels, train_size=.5)
        >>>
        >>> obj = Jarfo()
        >>> obj.fit(X_tr, y_tr)
        >>> # This example uses the predict() method
        >>> output = obj.predict(X_te)
        >>>
        >>> # This example uses the orient_graph() method. The dataset used
        >>> # can be loaded using the cdt.data module
        >>> data, graph = load_dataset('sachs')
        >>> output = obj.orient_graph(data, nx.DiGraph(graph))
        >>>
        >>> # To view the directed graph run the following command
        >>> nx.draw_networkx(output, font_size=8)
        >>> plt.show()
    """

Hi,
It should be done,
Thanks for the feedback !


Unable to change LDAP filter for non SASL.

Hello!
I am unable to change the LDAP filter to be other than "(&(?objectClass=PostfixBookMailAccount)(uniqueIdentifier=%u))".
No matter what options I put in my DockerCompose.yml I always end up with that filter being executed on my ldap. I do not have a postfixbookmailaccount and I would prefer it if I didn't have to, unless it is necessary.
I have tried

SASLAUTHD_LDAP_FILTER='(&(objectClass=posixAccount)(uid=%U))'
LDAP_FILTER='(&(objectClass=posixAccount)(uid=%U))'
LDAP_QUERY_FILTER='(&(objectClass=posixAccount)(uid=%U))'
but none of them have any effect.

I am using the docker openldap also, so adding the schema would prove a bit of a mess, unless there's another way to do it?

Thanks and any advice/help is greatly appreciated.

Josh

Hello @Joshua-Ashton. This is a known Bug. Please have a look at https://github.com/tomav/docker-mailserver/issues/491#issuecomment-306296358 for a workaround/solution.

I will fix this today and do a pull request!

@Joshua-Ashton please try with the latest version of this container. My PR #644 has been merged and should fix this.
Note that your issue should be related to the missing DOVECOT ldap part (fixed with the PR stated above).
I placed a sample config within the README. Maybe you can take a look!
Keep in Mind to use the following variables for configuration:

LDAP_QUERY_FILTER_USER
LDAP_QUERY_FILTER_GROUP
LDAP_QUERY_FILTER_ALIAS
SASLAUTHD_LDAP_FILTER
DOVECOT_USER_FILTER
DOVECOT_PASS_FILTER

@Joshua-Ashton ; can we close this issue. Also special character should work now. Merged #648.

Hello!
Sorry for taking an insane amount of time to get back to you, I'm pretty blind.
Decided to give this a go today but I'm still having issues although it's probably my fault - I'm fairly new to this. I just want to use regular "posixAccount"s as people with mail and no matter what I put as the filter it doesn't like it. Maybe I'm missing something...
Thanks for your help!

Josh

I also noticed that the DOVECOT filters appear to be commented out in the start script in the latest build.

@Joshua-Ashton can you post your compose file here please, so we can investigate what's going on.

I also noticed that the DOVECOT filters appear to be commented out in the start script in the latest build.

I am not sure whether your are referencing the following lines or not. If so, these are only default values i was not sure, whether they would breaking things or would be useful:
https://github.com/tomav/docker-mailserver/blob/9e7959fafef48b0da620d0d04d0a547029785689/target/start-mailserver.sh#L553

#491 might be interesting to.


Issues with the analyzer errors for unknown identifier in constant expression

There's one error with the following code:
void main() { }

const Duration test = const Duration(seconds: XXX);

...but the analyzer gets really upset with it, reporting five:
[warning] Undefined name 'XXX'. (test.dart, line 3, col 47)
[error] Const variables must be initialized with a constant value. (test.dart, line 3, col 47)
[error] Arguments of a constant creation must be constant expressions. (test.dart, line 3, col 47)
[error] Evaluation of this constant expression throws an exception. (test.dart, line 4, col 4001)
[error] Invalid constant value. (test.dart, line 3, col 47)

This list has a number of problems:

[ ] Despite there being a problem with the constness in three places, all three have the same line/column information. I would have expected the argument value ("Arguments of a constant creation must...") to be inside the parentheses and the top-level constant's value ("Const variables must...") to be outside the parentheses.
[ ] Because the unknown "XXX" identifier isn't a constant, the lack of constness cascades outwards three times. Only the inner one should be reported, the others should be muted to avoid overloading the analyzer output.
[ ] The unknown name has already been reported, but gets reported again when the evaluation throws. Only one of these should be reported (and then it should mute the two mentioned in the previous item as well).
[ ] The line/column information for the "throws an exception" error is bogus.
[ ] The only actual mistake is the only one that's marked as a warning, rather than an error.

We still have 5 errors here. :( But this can be alleviated with https://github.com/dart-lang/sdk/issues/36697.

I consider https://github.com/dart-lang/sdk/issues/36697 fixed now, but it will not fix this issue per se.
Fixing it will require additional changes to the constants evaluation implementation.
AFAIK @kallentu might be looking into improving this area.


Do something about http:// youtube links

It might make sense to replace http:// with https:// for some links, e.g. to youtube videos.
For instance, in Takeout/My Activity/Video Search/MyActivity.{json,html} might contain http:// links for some old entries
{'header': 'youtube.com', 'title': 'Watched Octobass @ the Musical Instrument Museum - YouTube', 'titleUrl': 'http://www.youtube.com/watch?v=FP1QqtGe8ts', 'time': '2015-06-10T12:24:03.796Z', 'products': ['Video Search']}

In case of youtube, switching to https doesn't really hurt (the http/https are equivalent and both are availabe), and it might make it easier to consume downstream, e.g. might prevent duplicates.
zulip discussion: https://memex.zulipchat.com/#narrow/stream/279601-hpi/topic/google_takeout_parser/near/279605540

Hmm, I guess this should be configurable, right...
Though it does look like all the http URLs (at least in titleUrl) are google ones:

In [3]: for y in res:
   ...:     if x.titleUrl and x.titleUrl.startswith("http://"):
   ...:         up = urllib.parse.urlsplit(x.titleUrl)
   ...:         x.append(up.netloc)

Counter({'maps.google.com': 1225,
         'www.google.com': 1098,
         'm.youtube.com': 18,
         'productforums.google.com': 17,
         'www.youtube.com': 15,
         'scholar.google.com': 2,
         'books.google.com': 2,
         'bp0.blogger.com': 1,
         'google.com': 1,
         'www.google.co.uk': 1})

Similar results with the internal url and sourceUrl from locationInfo
In [8]: Counter(items)
Out[8]: Counter({'www.google.com': 74459, b'': 33304, 'support.google.com': 33245})

dont want just all URLs to be converted (perhaps some websites in your history actually use http....), maybe just the common ones found in exports with a flag to configure this


[test] Update QE CPU checks

I have used the performance data stored in /apps/daint/UES/jenscscs/regression/production/logs/daint/mc starting November 7th 2019, after the last upgrade of Piz Daint:

QECheck_large.log, 145 data points, average 149.7 s, 95% within 52% threshold
QECheck_small.log, 164 data points, average 147.3 s, 95% within 41% threshold

The thresholds required to reach the 95th percentile are huge, since the standard deviation of the samples is high.
However, the thresholds would be even higher if I select only recent data points after the last maintenance (January 29th 2020).

@jenkins-cscs retry none


Random gRPC error: cancelled before receiving half close

Description
Similar to issue #17
When inserting data using python client, random gRPC errors occur
Environment

OS (where Grakn server runs): Mac OS Mojave 10.14.5
Grakn version (and platform):  docker graknlabs/grakn:1.5.3
Grakn client-python version: client-python 1.5.3
Python version: 3.7.2

Reproducible Steps
Error random, but when inserting data using phone call examples, one might run into this error.
Seems to be happening when creating a new transaction.
Expected Output
No error.
Actual Output
erver/network error: <_Rendezvous of RPC that terminated with:
        status = StatusCode.UNKNOWN
        details = "null. Please check server logs for the stack trace."
        debug_error_string = "{"created":"@1559448052.745006000","description":"Error received from peer","file":"src/core/lib/surface/call.cc","file_line":1017,"grpc_message":"null. Please check server logs for the stack trace.","grpc_status":2}"
>

 generated from request: open_req {
  sessionId: "dh31abea9bc-7ba7-47e2-a776-05bf8c9a9499"
  type: WRITE
}

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/grakn/service/Session/TransactionService.py", line 156, in send
    response = next(self._response_iterator)
  File "/usr/local/lib/python3.7/site-packages/grpc/_channel.py", line 364, in __next__
    return self._next()
  File "/usr/local/lib/python3.7/site-packages/grpc/_channel.py", line 358, in _next
    raise self
grpc._channel._Rendezvous: <_Rendezvous of RPC that terminated with:
        status = StatusCode.UNKNOWN
        details = "null. Please check server logs for the stack trace."
        debug_error_string = "{"created":"@1559448052.745006000","description":"Error received from peer","file":"src/core/lib/surface/call.cc","file_line":1017,"grpc_message":"null. Please check server logs for the stack trace.","grpc_status":2}"
>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "migrate.py", line 31, in build_phone_call_graph
    load_data_into_grakn(input, session)
  File "migrate.py", line 40, in load_data_into_grakn
    with session.transaction().write() as transaction:
  File "/usr/local/lib/python3.7/site-packages/grakn/client.py", line 106, in write
    transaction_service = TransactionService(self._session_id, _TxType.WRITE, self._transaction_rpc_constructor)
  File "/usr/local/lib/python3.7/site-packages/grakn/service/Session/TransactionService.py", line 39, in __init__
    self._communicator.send(open_req)
  File "/usr/local/lib/python3.7/site-packages/grakn/service/Session/TransactionService.py", line 160, in send
    raise GraknError("Server/network error: {0}\n\n generated from request: {1}".format(e, request))
grakn.exception.GraknError.GraknError: Server/network error: <_Rendezvous of RPC that terminated with:
        status = StatusCode.UNKNOWN
        details = "null. Please check server logs for the stack trace."
        debug_error_string = "{"created":"@1559448052.745006000","description":"Error received from peer","file":"src/core/lib/surface/call.cc","file_line":1017,"grpc_message":"null. Please check server logs for the stack trace.","grpc_status":2}"
>

 generated from request: open_req {
  sessionId: "dh31abea9bc-7ba7-47e2-a776-05bf8c9a9499"
  type: WRITE
}

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "migrate.py", line 303, in <module>
    build_phone_call_graph(inputs)
  File "migrate.py", line 34, in build_phone_call_graph
    raise
  File "/usr/local/lib/python3.7/site-packages/grakn/client.py", line 88, in __exit__
    self.close()
  File "/usr/local/lib/python3.7/site-packages/grakn/client.py", line 81, in close
    self._stub.close(close_session_req)
  File "/usr/local/lib/python3.7/site-packages/grpc/_channel.py", line 533, in __call__
    return _end_unary_response_blocking(state, call, False, None)
  File "/usr/local/lib/python3.7/site-packages/grpc/_channel.py", line 467, in _end_unary_response_blocking
    raise _Rendezvous(state, None, None, deadline)
grpc._channel._Rendezvous: <_Rendezvous of RPC that terminated with:
        status = StatusCode.UNKNOWN
        details = "null. Please check server logs for the stack trace."
        debug_error_string = "{"created":"@1559448052.748124000","description":"Error received from peer","file":"src/core/lib/surface/call.cc","file_line":1017,"grpc_message":"null. Please check server logs for the stack trace.","grpc_status":2}"
>

Logs from grakn server:
[grpc-default-executor-4] ERROR g.c.s.r.SessionService$TransactionListener - Runtime Exception in RPC TransactionListener:
io.grpc.StatusRuntimeException: CANCELLED: cancelled before receiving half close
	at io.grpc.Status.asRuntimeException(Status.java:517)
	at io.grpc.stub.ServerCalls$StreamingServerCallHandler$StreamingServerCallListener.onCancel(ServerCalls.java:272)
	at io.grpc.internal.ServerCallImpl$ServerStreamListenerImpl.closed(ServerCallImpl.java:293)
	at io.grpc.internal.ServerImpl$JumpToApplicationThreadServerStreamListener$1Closed.runInContext(ServerImpl.java:741)
	at io.grpc.internal.ContextRunnable.run(ContextRunnable.java:37)
	at io.grpc.internal.SerializingExecutor.run(SerializingExecutor.java:123)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

So I found what was causing the issue, but really can't explain it...
I use your migrate.py provided in the phone calls example, which randomly returns the error described above:
def build_phone_call_graph(inputs):
    with GraknClient(uri=URI) as client:
        with client.session(keyspace = KEYSPACE) as session:
            for input in inputs:
                print("Loading from [" + input["data_path"] + "] into Grakn ...")
                load_data_into_grakn(input, session)

def load_data_into_grakn(input, session):
    items = parse_data_to_dictionaries(input)

    for item in items:
        with session.transaction().write() as transaction:
            graql_insert_query = input["template"](item)
            print("Executing Graql Query: " + graql_insert_query)
            transaction.query(graql_insert_query)
            transaction.commit()

    print("\nInserted " + str(len(items)) + " items from [ " + input["data_path"] + "] into Grakn.\n")

However, if you write the same thing this way, by simply not passing the session to the function, the error completely disappear...
def build_phone_call_graph(inputs):
    with GraknClient(uri=URI) as client:
        with client.session(keyspace = KEYSPACE) as session:
            for input in inputs:
                print("Loading from [" + input["data_path"] + "] into Grakn ...")

                # load_data_into_grakn(input, session)
                items = parse_data_to_dictionaries(input)

                for item in items:
                    with session.transaction().write() as transaction:
                        graql_insert_query = input["template"](item)
                        print("Executing Graql Query: " + graql_insert_query)
                        transaction.query(graql_insert_query)
                        transaction.commit()

                print("\nInserted " + str(len(items)) + " items from [ " + input["data_path"] + "] into Grakn.\n")

Can't explain why... Only happens with docker.

Hi! We've improved the software quite significantly between this version (1.5.3) and the current version (2.5.0).
Feel free to try it out with the new version and come back to us if the issue still persists.

Thanks for reporting this issue, @viktorrobin. Given how old this issue is from Grakn 1.5, and that we have rewritten Grakn in the 2.0 release (and renamed it to TypeDB), we do not believe this issue still exist and will close this now.


Total ordering for floating-point values.

Adds a totalOrder function as specified in IEEE 754-2008.
An immediate benefit is the ability to sort arrays of floating-point values and get predictable results even in presence of NaNs (see http://dpaste.dzfl.pl/1f78bc6f7d2b).

Maybe return -1, 0 and 1? Then it can be directly used in opCmp().

I agree, returning -1, 0, and 1 would be significantly more useful than returning a bool.

I tried to follow semantics described in the standard, and it explicitly specifies the return type as boolean. I'm not sure how important that is, but that was the motivation for the current design.
Besides, the three-way comparison would still be unusuable in e.g. sort, because it requires a predicate (and current version is unusable because the order isn't strict).
It seems like it would be nice to have 3 functions:

Something returning -1, 0, and 1 for opCmp.
Strict comparison predicate for sort and friends.
Current function for strict IEEE compliance.

However, having all three is definitely an overkill. So, which one?

I don't see that being overkill in the slightest.

I decided to settle on option 1, because

the problem with sort isn't specific to this particular case — the same problem exists with e.g. std.algorithm.comparison.cmp;
strict IEEE compliance isn't really that important, especially since standard behaviour doesn't seem to fit any of use-cases directly.

And I still don't think it's a good idea to have three functions doing the same thing in slightly different ways.

Code style nit, we generally don't put braces around single statements in Phobos.

Fixed.

@9il you seem like a numerics expert - would you mind reviewing this? Thanks!

I'll try:) Thanks!

@9il I'm not sure what you meant by 'unroll'..?

@ivan-timokhin in release mode loop should be unrolled (eliminated). If it is not unrolled in release mode then we need an manually unrolled version.

Yeah, tuples is fine :)

sort function has note about floating point. Please add reference to the new function into the sort docs.

... and one documented unittest for sort!((x, y) => cmp(x, y) < 0) into the sort docs too.

If this function works faster then naive floating point comparison we can make it default for functions like sort.

If this function works faster then naive floating point comparison we can make it default for functions like sort.

Well, I was unable to construct a benchmark that would measure just that, so I
have no idea.

sort function has note about floating point. Please add reference to the new function into the sort docs.

Hm. I used $(XREF) for cross-module referencing, but the resulting link points
to stable docs on dlang.org, which, of course, contain no such function. What
should I do?

Hm. I used $(XREF) for cross-module referencing, but the resulting link points
to stable docs on dlang.org, which, of course, contain no such function. What
should I do?
Nothing. XREF is fine.

LGTM except  name cmp. fcmp would be more clear, thought.
Ping @DmitryOlshansky

I dunno. The first thing fcmp reminds me of is the whole C  math functions family with names like fabsf and atanl, which exist because of lack of overloading.
However, I'm not attached to any particular name, so I'll happily go with fcmp, provided there's consensus.

OT: As we seem to have most D math experts assembled in this thread:
Could somebody have a look at this issue:
https://issues.dlang.org/show_bug.cgi?id=13032
AFAICS std.math.internal.gammafunction is the last module with only works with X86 80bit reals and has never been ported for standard double / float formats. As this probably affects every non X86 architecture it'd be great if somebody could have a look.

@jpf91 Hi, std.mathspecial was ported from CEPHES library (thought). You may want to ask @donc about license change and port stuff for float and double versions.

Thanks, I had a quick look at gammafunction.d/CEPHES. The constants have been renamed in the D port, the code layout is different and the constant values also don't match the constants in gammal.c. As I don't have any experience with numerical maths porting gammafunction.d for double types is more like random-guessing for me ;-)

FYI see std.math for the license of redistributing CEPHES in another language (I spoke to the original author for clarification a couple years ago).

Porting is pretty simple except constants. Sometimes they are defined in different binary forms (depends on endianness). The steps can be described as:

Port with core.stdc.math or core.stdc.tgmath
Add unittests. Floating hexadecimal approximate constants for unittests can be computed with Wolfram Alpha.
Port with std.math.

@jpf91 See also #3045

Floating hexadecimal approximate constants for unittests can be computed with Wolfram Alpha.

I'm not familiar with Wolfram, I'd be interested to know how you do this.

@ibuclaw Example: hexadecimal log(2^52).

Also you can press More digits.

LGTM except name cmp. fcmp would be more clear, thought.
Ping @DmitryOlshansky

Okay I'm here. Press the button I guess? :)

Yes please =)

Yes please =)

I've added milestone but @ivan-timokhin please consider adding a nice changelog entry or the change may go unnoticed.

Auto-merge toggled on


fix(box shadow utility): Add left variants

Adds the missing leftward variants of the Box Shadow utility. Resolves #856.

Deploy preview for pf-next ready!
Built with commit 537b41333641986b6d2b478a9e8b34455b9e4a68
https://deploy-preview-857--pf-next.netlify.com

:tada: This PR is included in version 1.0.64 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


It's german. It means "the dev, the"

Overview
https://www.youtube.com/watch?v=gaXigSu72A4
Covers most of the tasks in #3179 which is the last bit of #3140

Update docs to refer to main, nightly, and stable branches rather than dev in explaining our git workflow and nightly build / data distribution setup.
Change all direct links to Read The Docs to use the newly default nightly docs rather than dev.
Update nightly build scripts and other github actions workflows to run off of main rather than dev.
Update dependabot and pre-commit-ci to run against main instead of dev.
Switch to using gs://builds.catalyst.coop for our build outputs.
Remove obsolete docker/.env file and define those envvars either in the container / workflow / script.

Testing

Manually ran a build off of this branch.

# To-do list
- [x] Update the [release notes](../docs/release_notes.rst): reference the PR and related issues.
- [x] Review the PR yourself and call out any questions or issues you have

Hmm @bendnorman @jdangerx it looks like Google was really hoping for that .env file. Or is this a setting that's associated with the VM somewhere?
Updated [https://www.googleapis.com/compute/v1/projects/***/zones/***/instances/pudl-deployment-tag].
ERROR: (gcloud.compute.instances.update-container) Unable to read file [./docker/.env]: [Errno 2] No such file or directory: './docker/.env'
Error: Process completed with exit code 1.


Create repository to store util packages within NPWG

What issue would you like to bring attention to?
Currently for repositories like SRIOV CNI / SRIOV CNI Operator / Network Resources Injector and Device Plugin there is or will be a set of E2E tests that are using utils from util package. The set of packages inside allows for instance to create / delete K8s objects, read interface details. Those packages are almost similar in implementation between repositories and most probably could be implemented in one place and imported by each component. Beside tests such utils can be used in code if needed.
Some examples:
https://github.com/k8snetworkplumbingwg/network-resources-injector/tree/master/test/util
https://github.com/k8snetworkplumbingwg/sriov-network-operator/tree/master/test/util
https://github.com/k8snetworkplumbingwg/sriov-network-device-plugin/tree/master/pkg/utils
https://github.com/k8snetworkplumbingwg/sriov-cni/tree/master/pkg/utils
What is the impact of this issue?
As a result of storing utils in once place, I see

less effort to maintain code
clear information about available functions inside utils packages
avoid code copy when not imported

Do you have a proposed response or remediation for the issue?
Create repository within NPWG to store utils.

/cc @pliurh @martinkennelly @adrianchiris  @amorenoz
This was raised in resource management meeting (July 5) and we thought it would be better to create an issue for continued discussion.
I recognize this has been an issue since we have multiple sriov related projects with each needs to perform similar function calls.
This issue was not obvious before, I think partially because it is still manageable to do the import of code from another project, but as we are adding more features (e.g. switchdev), there will be more common functions introduced.
I knew we have several other repos implementing functions as a lib that's easy to share across projects, such as the Mellanox/sriovnet and go-vdpa etc. Would it make sense to consider merging them together under a separate npwg project?
I presume this would benefit us in the long term.

Big +1.

+1 for merging go-vdpa into a npwg repo

+1 for reducing code duplication, however we need to be careful about what we put in the newly propsed util repo, its structure
and versioning and guidelines on API changes.

+1 for the proposal.
ovs-cni also have code duplication in w.r.t using caching netConf, we could try to generalise those APIs and make it consumable from both sriov-cni and ovs-cni.

+1 for reducing code duplication, however we need to be careful about what we put in the newly propsed util repo, its structure
and versioning and guidelines on API changes.

@adrianchiris do you have something in mind which we may not want to put in the newly proposed repo?
Re repo structure, will the flat rep structure in netlink library be a good model for sriov library or do we have the need to create separate folders or packages for various sriov features such as switchdev, vdpa, general sriov etc?

if others plan to use the packages/libraries, the convention is to put it into a pkg directory within the project.  see[1]
so i dont think we need to do the same as netlink.
[1] https://github.com/golang-standards/project-layout#go-directories

This has been discussed in yesterday's community meeting.
agreement that having a project for common functionality that is desirable.
next steps would be:

decide on a name (proposed names: sriov-utils, resource-mgmt-utils, common-utils)
present idea to NPWG, get approval and create  the project
decide on project layout (positive approach in having stable/unstable folder to mark APIs as stable/unstable)
move basic common functionality

I have created the first issue in the new project.
we should continue discussion there.
closing this issue so discussion may continue in the new repository

https://github.com/k8snetworkplumbingwg/resource-mgmt-utils/issues/1


ImportError: This platform lacks a functioning sem_open implementation

Hi i get on FreeBSD 10.1 this error message:
[root@www1 /var/log/beaver]# beaver -c /usr/local/etc/beaver/beaver.conf
Traceback (most recent call last):
  File "/usr/local/bin/beaver", line 31, in <module>
    worker_run(args)
  File "/usr/local/lib/python2.7/site-packages/beaver/dispatcher/worker.py", line 24, in run
    queue = multiprocessing.Queue(beaver_config.get('max_queue_size'))
  File "/usr/local/lib/python2.7/multiprocessing/__init__.py", line 217, in Queue
    from multiprocessing.queues import Queue
  File "/usr/local/lib/python2.7/multiprocessing/queues.py", line 48, in <module>
    from multiprocessing.synchronize import Lock, BoundedSemaphore, Semaphore, Condition
  File "/usr/local/lib/python2.7/multiprocessing/synchronize.py", line 59, in <module>
    " function, see issue 3770.")
ImportError: This platform lacks a functioning sem_open implementation, therefore, the required synchronization primitives needed will not function, see issue 3770.

any helpe here ?

On a real operating system like FreeBSD, to avoid this error you should rebuild lang/python27 with SEM.


HDFS: Need to handle non-ASCII characters in filename

Azure Data Studio Version:

Steps to Reproduce:

Connect to a SQL Server big data cluster
Right-click on the HDFDS node and choose New directory
Enter test_mkdir$%^

Expected:

Directory is created
Actual:
Get Error uploading files: Unknown error

@YurongHe  and @geleems this is a good candidate as we work on unified connection finish up items. Similar to fixing preview to not fail on > 30mb files


UX research and wireframes: User details

As a ...
I want to view/read details about users and their roles
So that I know what they are allowed to see in the UI
Clarification

[ ] What information should be presented?
[ ] As a user would I need to perform actions?

AC

[ ] Clarify with stakeholders what requirements exist, in order to be able to see details
[ ] Clarify whether you want to perform actions on user details
[ ] Build wireframes based on these requirements
[ ] Demonstrate flows from start to finish
[ ] Verify with stakeholders


Fix stories code execution

Description

Supersedes #2559. Decouples stories workspace logic in WorkspaceSaga from the workspace location, thus fixing the regressions introduced by the initial decoupling of stories state from workspace location in #2424.
Also introduced code quality improvements.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Code quality improvements

Checklist

[x] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 14 (50.0%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 35.815%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/features/stories/storiesComponents/SourceBlock.tsx
0
1
0.0%

src/commons/sagas/WorkspaceSaga.ts
3
9
33.33%

Files with Coverage Reduction
New Missed Lines
%

src/commons/sagas/WorkspaceSaga.ts
1
65.85%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
5478

Relevant Lines:
14327

💛  - Coveralls


Actually add some sort of volume control for song player

the button is purley cosmetic atm:

done :D


completed-docs false positive when entire doc comment consists of only tags

Bug Report

TSLint version: 4.2.0
TypeScript version: 2.1.4
Running TSLint via: CLI

TypeScript code being linted
/**
 * @ignore
 * @see foo
 */
private static bar () : number {
	return 4;
}

with tslint.json configuration:
{
	"rules": {
		"completed-docs": true
	}
}

Actual behavior
Methods must have documentation.
Expected behavior
No error, as with TS 2.0.10 and tslint 4.1.1.
Rationale: we currently have /** @ignore */ on all private properties and methods (which will be the case until we have some extra bandwidth to spend on documenting all that stuff), and in some cases where there isn't anything more to add on top of a class description we have public properties whose docs are just /** @see $ClassName */.

A good solution to this could be changing the rule config to allow for conditions that mark a construct as not needing to be documented. Some conditions:

Not on the whitelist of types (what's currently implemented)
Has tags but no textual description (project may or may not consider that acceptable)
Has a tag that indicates it shouldn't be documented, such as @ignore or @inheritdoc

Proposal: extend the rule options to have global and per-construct settings.

If settings is the string "all", enable it for every possible construct.
If settings is an object, apply the IConstructDocsSettings logic to every possible constructs.
If settings is an array, for each member:

If it's a string "all": enable all constructs
If it's a string: enable it for that construct
If it's an object, for each [key, value]:

The key is either "all" for all constructors or the name of an individual construct
The value is either false to disable, true to enable completely, or an IConstructDocsSettings

interface IConstructDocsSettings {
    /**
     * Tags that indicate a block is ok.
     */
    "ok-tags"?: boolean;

    /**
     * Having tags without text is ok.
     */
    "tags-without-text"?: boolean;
}

Thoughts?

Just to add another usage example. I tend to use @inheritdoc when implementing an interface or abstract class. However, I'm currently getting the "Documentation must exist for public methods" warning when doing so.
/**
 * An abstract class to illustrate my point.
 */
abstract class Foo {
  /**
   * A demo method to illustrate my point.
   * @param message - Do something with it.
   */
  public abstract bar(message: string): void;
}

/**
 * A concrete class that extends the Foo base class.
 */
class ConcreteFoo extends Foo {

  /**
   * @inheritdoc
   */
  public bar(message: string): void {  // Documentation must exist for public methods
    console.log(message);
  }
}

:anchor: :+1:

While it would be nice to have the ability to fine tune which tags are whitelisted a suggested above, I would think that "@inheritdoc" in particular should automatically and always bypass this rule, due to the inherent meaning of that tag. Without this, this rule is pretty much unusable for me right now, which is a shame since I really need it.


Update manage-creation-of-groups.md

spelling

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@MikePlumleyMSFT - PR has been copy edited and is ready for final review.
Thank you.
cc: @kenwith


Unable to install netlify-cli using Yarn

Describe the bug
I have tried yarn global add netlify-cli with several different versions of Node with no improvement.
npm install netlify-cli -g brew install netlify-cli These two succeed.
But I want to manage it with Yarn.
How can I install it?
Steps to reproduce
% yarn global add netlify-cli

yarn global v1.22.19
[1/4] 🔍  Resolving packages...
warning netlify-cli ><EMAIL_ADDRESS>Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
warning "redoc-cli ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "redoc-cli ><EMAIL_ADDRESS>has unmet peer dependency "react-is@>= 16.8.0".
warning "redoc-cli > redoc ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "typescript@*".
warning "ts-node-dev ><EMAIL_ADDRESS>has unmet peer dependency "@types/node@*".
warning "ts-node-dev ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[4/4] 🔨  Building fresh packages...
[6/9] ⠠ sqlite3
[-/9] ⠠ waiting...
[-/9] ⠠ waiting...
[-/9] ⠠ waiting...
error /Users/sdr/.config/yarn/global/node_modules/netlify-cli: Command failed.
Exit code: 1
Command: node ./scripts/postinstall.mjs
Arguments:
Directory: /Users/sdr/.config/yarn/global/node_modules/netlify-cli
Output:
file:///Users/sdr/.config/yarn/global/node_modules/gh-release-fetch/dist/index.js:5
import { gt } from 'semver';
         ^^
SyntaxError: Named export 'gt' not found. The requested module 'semver' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'semver';
const { gt } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:123:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:189:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)

Configuration
No response
Environment
System:
OS: macOS 13.4
CPU: (8) arm64 Apple M2
Memory: 2.46 GB / 24.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 16.20.0 - ~/.nvm/versions/node/v16.20.0/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 8.19.4 - ~/.nvm/versions/node/v16.20.0/bin/npm

Thanks for reporting, I think the problem is we are requiring semver ^7.0.0 in some packages, but before 7.3.8 it would not work with ESM. I'm on it fixing this now.

Just ran into this, will a new version with the fix be released soon?

Could you please try with 15.3.2 which was just released. I think I found all places where the dependencies were wrong.

@danez It worked for me! thank you!

Possibly this has regressed:
# This file contains the result of Yarn building a package (netlify-cli@npm:17.15.4)
# Script name: postinstall

file:///Users/.../node_modules/@netlify/build-info/lib/workspaces/get-workspace-packages.js:1
import { minimatch, Minimatch } from 'minimatch';
                    ^^^^^^^^^
SyntaxError: Named export 'Minimatch' not found. The requested module 'minimatch' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'minimatch';
const { minimatch, Minimatch } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:190:5)

Node.js v18.16.0

Bumping this because it does appear to have regressed at multiple releases. I'm seeing this issue: https://github.com/netlify/cli/issues/6371
error {REDACTED}/node_modules/netlify-cli: Command failed.
Exit code: 1
Command: node ./scripts/postinstall.mjs
Arguments:
Directory: {REDACTED}/node_modules/netlify-cli
Output:
{REDACTED}/node_modules/@netlify/build/lib/plugins_core/functions/index.js:2
import { RUNTIME, zipFunctions } from '@netlify/zip-it-and-ship-it';
         ^^^^^^^
SyntaxError: Named export 'RUNTIME' not found. The requested module '@netlify/zip-it-and-ship-it' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@netlify/zip-it-and-ship-it';
const { RUNTIME, zipFunctions } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:123:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:191:5)
    at async ModuleLoader.import (node:internal/modules/esm/loader:336:24)
    at async loadESM (node:internal/process/esm_loader:34:7)

Appearing in 15.8.1 too.


feat: angular integration

This PR adds neodrag integration with Angular

NeoDraggable directive
Documentations updated with Angular

Would love some help to set up changesets for publishing.
Closes #106

Here are the steps to run the test locally and the final result:

Start by running pnpm install on the root directory. Make sure you are using pnpm version 8. ℹ Note: Initially, I encountered an issue because I was using pnpm version 7. Here is a screenshot of the error:

After installing the dependencies, still on the root directory, run npx nx compile @neodrag/angular. If everything is successful, you should see a similar output as shown in the screenshot.

Run pnpm install again to ensure that @neodrag/angular is installed in the demo project.

Now navigate to the packages/angular/demo directory and run npm start. There are two possible scenarios that can occur, and both @nartc and I are investigating the cause.
4.1 ✅ Everything works fine, and you should see the expected output as shown in my screenshot.
4.2 ❌ The Angular dev server runs successfully, but you encounter an error in the browser console. To resolve this issue, go back to the root folder, delete the node_modules and pnpm-lock files, and then run pnpm install again. For some reason, when running pnpm install<EMAIL_ADDRESS>gets installed, but if we remove the pnpm-lock file<EMAIL_ADDRESS>is installed instead.

I hope this helps! Let me know if you have any further questions.

@PuruVJ is there a strategy you have in mind for including the angular integration now in terms of release version/changeset?

It's a bit tricky. it should go out as 0.x release first. Then as it gains more attractions within a month, releasing 2.0.0 feels fine. I know skipping over 1.0.0 is weird, but I dont think angular users would mind, considering the whole thing with skipping angular 3 and updating other deps directly v4 as well.
I haven't given this much time yet. It seems the docs site is also breaking due to old version of something and that'll need fixing before this can be merged. I'm on a bit of a limited bandwidth for next few days due to some urgent bugs in svelte REPL and sites. Once that's done, I'll give this PR a full review along with updated docs site.

@PuruVJ can you please look into this PR and merge it. I'm waiting for this support

Hi! I am able to work on this finally!!
Thanks a lot for this @nartc. Ill be going through the changes and asking questions to understand it. However, I dont really understand angular deeply, so I'll be going forward with the assumption that you took care of the edge cases. I have full confidence in you, considering your background and experience 🔥😁
Once again, thanks a lot for this!

I am getting this error:

Even though I installed 5.2.2 in workspace and as Devdependency in angular package.json as well
Any idea how to fix this one?

I'll check it out later today. Thanks @PuruVJ for taking a look.

@PuruVJ I've updated the PR. There are some changes that I want to point out:

Can we go with TypeScript 5.2 at the moment? The Angular Compiler has yet to support 5.3 (it should be soon). To answer your question about ng-packagr, it is required because Angular decorators aren't just TypeScript decorators. Those decorators will be compiled away using the Angular Compiler into Angular Instructions (template instructions)
Angular Library is not minified as the Angular Team encourages folks to only minify at the application level (where the library is consumed). This makes the Angular size look huge on the docs

Re: minification - Other versions also ship unminified version, but the sizes are calculated from dist/min/index.js, which is minified, but isn't used directly, unless someone explicitly uses its path.
Is it possible to have that for Angular integration too? Just for reporting the size?

@PuruVJ, It is possible to run a script to minify Angular's output, but it wouldn't seem fair either since the other min/index.js seem to have @neodrag/core bundled while the Angular one does not bundle its dependencies.

Wait what?! That doesn't seem right. @neodrag/core's functionality should be bundled inside the package. Is that possible?

Aha!
Is it possible to make a separate minified file with decorators intact, but entirety of @neodrag/core included, and then minified? I'd still like the users to get an idea of how big this is gonna be

I can figure something out


fix: missing space between experission and bracket

Coverage remained the same at 52.943% when pulling ca73d69f2bc93fb5bc3d024dfc24001b52418a56 on mintao/pai-0.12.y-fix into 198e130f454f91c122dacfed0368b9381e892797 on pai-0.12.y.

Coverage remained the same at 52.943% when pulling ca73d69f2bc93fb5bc3d024dfc24001b52418a56 on mintao/pai-0.12.y-fix into 198e130f454f91c122dacfed0368b9381e892797 on pai-0.12.y.

Coverage remained the same at 52.943% when pulling ca73d69f2bc93fb5bc3d024dfc24001b52418a56 on mintao/pai-0.12.y-fix into 198e130f454f91c122dacfed0368b9381e892797 on pai-0.12.y.


Connection Primary Server (with Console) and Reporting Server

Hello,
"6.Verify that status message ID 1015 is listed, which indicates that the reporting services point was successfully installed. Alternatively, you can open the Srsrp.log file, located in <ConfigMgr Installation Path>\Logs, and look for Installation was successful"
I do not see the ID 1015 but I see
Severity	Type	Site code	Date / Time	System	Component	Message ID	Description
Information	Milestone	UCP	8/3/2018 3:38:11 PM	VRPSCCMRS01.AD.MEDCTR.UCLA.EDU	SMS_SRS_REPORTING_POINT	1019	Site Component Manager successfully reinstalled this component on this site system.
I can run reports from anywhere  but when I am trying to create/edit report from the Primary Server it refused so far ...
"Before you can create or edit reports, your computer must run a version o SQL Server Report Builder that matches the version of SQL Server that you use for your report server."
When running on the Reporting Server everything is working Run/Create/Edit.
Do I need Reporting Server or Report Builder on all machines used to create reports or they connect to the Reporting Server and run locally ...!!!?
Thanks,
Dom
Thanks Dom

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ed3637a6-0616-a14a-2864-9c7d6bbc70e0
Version Independent ID: 7b0ddfd8-2c0f-9518-39e6-e49130ae2f58
Content: Configure reporting - Configuration Manager
Content Source: sccm/core/servers/manage/configuring-reporting.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

I found in srsrp.log
Successfully checked that the SRS web service is healthy on server VRPSCCMRS01.AD	SMS_SRS_REPORTING_POINT	8/30/2018 8:30:57 AM	5092 (0x13E4)
Waiting for changes for 1 minutes	SMS_SRS_REPORTING_POINT	8/30/2018 10:24:13 AM	5092 (0x13E4)

Thanks @felymich for taking the time to share this with the Configuration Manager documentation team. I’m sorry that you’re having problems, and I want to make sure it gets to the right people that can help. There are a couple of options available to provide the best level of support for this:

Post on Configuration Manager forums. These are moderated by our community MVPs who are very experienced, knowledgeable, and helpful.
Use Microsoft Support to search for the issue, or open a case with professional support.
Contact your Microsoft Premier team, such as a Premier Field Engineer or Technical Account Manager.

I'm closing this issue as there's nothing actionable for docs at this time. If later you find there's something specific missing from this article, please reopen this issue or file a new one.


Added elo card

This pull request added elo card

Thanks @isacjunior , soon i will release this version on NPM


Streamline component creation via updating React-USWDS library

Problem statement
We make components a lot on the front end. We need the components to do things that are not standard React-USWDS functions/features. Therefore, we need to enhance the primary library so that we don't have to make bespoke components every single time. This saves front-end devs time and requires less code/testing in the RS repo/website.
What you need to know

In the process of making changes to a library we depend on (front-end) for the RS environment to streamline using that library for our developers. (~75% completion as of 1/30)
Propose those changes to the react library (outside of RS). Pull request needs to be accepted to React-uswds.
In the mean time, we should move those changes to a centrally hosted location in the CDC repo, and needs to be moved off of Joseph's public repo.

Acceptance criteria

[ ] Have a stable CDC fork of React-USWDS that is technically capable of being accepted into the official library but can be used on our RS project before then

To do

[ ] ...

There is roughly (estimated) 2 sprints worth of work left on this.


Change ibeacon ranging interval to be more than one sceond

Unfortunately, there is a restriction by iOS Core Location APIs to do so.

https://stackoverflow.com/questions/29694841/how-to-set-time-for-scanning-interval-for-ibeacon-on-ios
https://stackoverflow.com/questions/29694841/how-to-set-time-for-scanning-interval-for-ibeacon-on-ios


Add batch progress section to gift entry batch mode

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch: attach W-037478

Tracking W-037478

Replaced by this PR, which includes both the batch progress section and the functionality to enforce require total match: https://github.com/SalesforceFoundation/NPSP/pull/5494

**lurch: attach W-038938

Tracking W-038938


p2p: use async bipipe

Closes #841
I had to clone the original crate to introduce 4 additional methods:

async_pipe
async_pipe_buffered
async_bipipe
async_bipipe_buffered

The difference is they all use unbounded crossbeam channel rather then
bounded(0) which makes them async.
And we need pipe to be async because of how the handshake is written
currently.

[x] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[ ] Updated CHANGELOG.md

I will see if this could be upstreamed (I doubt that, but I will try nonetheless)

I will see if this could be upstreamed (I doubt that, but I will try nonetheless) arcnmx/pipe-rs#10

If we have doubts that it will be upstream-ed should we just maintain a pipe implementation in the p2p package for our testing purposes?

If we have doubts that it will be upstream-ed should we just maintain a pipe implementation in the p2p package for our testing purposes?

Yeah this makes sense to me :+1:

I will see if this could be upstreamed (I doubt that, but I will try nonetheless) arcnmx/pipe-rs#10

If we have doubts that it will be upstream-ed should we just maintain a pipe implementation in the p2p package for our testing purposes?

done 👍


Update Dockerfile

Status
Ready
Related Content Pull Request
Related PR:https://github.com/demisto/content/pull/14508
Related Issues
Related: https://github.com/demisto/etc/issues/40210

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-pdf-reports:<IP_ADDRESS>25
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-pdf-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-pdf-reports:<IP_ADDRESS>25

Docker Metadata

Image Size: 661.68 MB

Image ID: sha256:345bb580a50333985ef18528fed5e5d4aecafc0fdd8fcd4a13f3d6e795142661

Created: 2021-08-30T15:02:25.834128893Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=E3FF2839C048B25C084DEBE9B26995E310250568
PYTHON_VERSION=3.9.6
PYTHON_PIP_VERSION=21.1.3
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/a1675ab6c2bd898ed82b1f58c486097f763c74a9/public/get-pip.py
PYTHON_GET_PIP_SHA256=6665659241292b2147b58922b9ffe11dda66b39d52d8a6f3aa310bc1d60ea6f7
DOCKER_IMAGE=devdemisto/sane-pdf-reports:<IP_ADDRESS>25

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:510ca1813841e41bfb6e7f7f467ad874d9a746a0
org.opencontainers.image.version:<IP_ADDRESS>25


Delete gitsync's Child Resources In the Cluster If Gitsync Gets Deleted

Yes, it fixes Issue #137.
Modifications
DeleteManagedObjectsGitSync has been added, which is called at the places mentioned in the aforementioned issue. This method is designed to get all the resources from Syncer Cache  and subsequently delete them.
Verification
Unit tests have been written for the DeleteManagedObjectsGitSync method, as documented here. Furthermore, the deletion functionality has been tested by running the controller and Gitsync on a local Kubernetes cluster.
If gitsync is deleted ,then the related resources are deleted as well having the same annotation key value pair

@shubhamdixit863 Other than the comments I left, can you add an option in ConfigMap to enable or disable this cascade deletion feature. By default, let's put it as disabled. Thanks!

You mean add in the config.yaml here   then I need to add a global config variable too here

@shubhamdixit863 Other than the comments I left, can you add an option in ConfigMap to enable or disable this cascade deletion feature. By default, let's put it as disabled. Thanks!

You mean add in the config.yaml here then I need to add a global config variable too here . As until its not available in GlobalConfig how will I get it as a variable .

Yes, you need to do that


Any way to do a HTTP request in Transform / Otto

Hey guys,
Is there any way to do a http request in a transform file / otto ?
Trying to import/require https module but get an error...
Thanks
Checks

[X ] I have mentioned my abc version in the issue, if applicable.
[ X] This issue is not a duplicate.
[ X] This issue has a description.

This isn't possible afaict.


29.97 Compound Clip fails in 23.98 Project

Reported by Dustin Svehlak via email.
Let's say we have a 29.97fps GoPro Clip. We create a 29.97fps Compound Clip, and apply Gyroflow Toolbox - everything works great! If we apply it to a 29.97fps Project/Timeline, it works as expected. However, if we put this 29.97fps Compound Clip in a 23.98 Project/Timeline, things break.
I believe this is due to how Final Cut Pro handles Rate Conform.

As a possible workaround, applying an Automatic Speed Effect to the Compound Clip will fix it - but may not be visually what the user wants, as it'll now be Slow 80%.

Or you could render out the Compound Clip and import it back in (which is less than ideal - and goes against the whole reason Gyroflow Toolbox exists in the first place).
This MIGHT be possible to fix - although I'm not sure that FxPlug gives us the information we need to do the maths.
FWIW - I've tried using a Synchronised Clip instead, creating another Compound Clip with the Automatic Speed Effect applied, etc but none of these workarounds work.
For reference, here's our current method of detecting the current frame:
    //---------------------------------------------------------
    // Get the frame to render:
    //---------------------------------------------------------
    CMTime timelineFrameDuration = kCMTimeZero;
    timelineFrameDuration = CMTimeMake( [timingAPI timelineFpsDenominatorForEffect:self],
                                       (int)[timingAPI timelineFpsNumeratorForEffect:self] );

    CMTime timelineTime = kCMTimeZero;
    [timingAPI timelineTime:&timelineTime fromInputTime:renderTime];

    CMTime startTimeOfInputToFilter = kCMTimeZero;
    [timingAPI startTimeForEffect:&startTimeOfInputToFilter];

    CMTime startTimeOfInputToFilterInTimelineTime = kCMTimeZero;
    [timingAPI timelineTime:&startTimeOfInputToFilterInTimelineTime fromInputTime:startTimeOfInputToFilter];

    Float64 timelineTimeMinusStartTimeOfInputToFilterNumerator = (Float64)timelineTime.value * (Float64)startTimeOfInputToFilterInTimelineTime.timescale - (Float64)startTimeOfInputToFilterInTimelineTime.value * (Float64)timelineTime.timescale;
    Float64 timelineTimeMinusStartTimeOfInputToFilterDenominator = (Float64)timelineTime.timescale * (Float64)startTimeOfInputToFilterInTimelineTime.timescale;

    Float64 frame = ( ((Float64)timelineTimeMinusStartTimeOfInputToFilterNumerator / (Float64)timelineTimeMinusStartTimeOfInputToFilterDenominator) / ((Float64)timelineFrameDuration.value / (Float64)timelineFrameDuration.timescale) );

    //---------------------------------------------------------
    // Calculate the Timestamp:
    //---------------------------------------------------------
    Float64 timelineFpsNumerator    = [timingAPI timelineFpsNumeratorForEffect:self];
    Float64 timelineFpsDenominator  = [timingAPI timelineFpsDenominatorForEffect:self];
    Float64 frameRate               = timelineFpsNumerator / timelineFpsDenominator;
    Float64 timestamp               = (frame / frameRate) * 1000000.0;
    params.timestamp                = [[[NSNumber alloc] initWithFloat:timestamp] autorelease];

See: Understanding time in FxPlug
@AdrianEddy - Any ideas?

When adding this debugging code:
NSLog(@"---------------------------------");
NSLog(@"timelineFrameDuration: %.2f seconds", CMTimeGetSeconds(timelineFrameDuration));
NSLog(@"timelineTime: %.2f seconds", CMTimeGetSeconds(timelineTime));
NSLog(@"startTimeOfInputToFilter: %.2f seconds", CMTimeGetSeconds(startTimeOfInputToFilter));
NSLog(@"startTimeOfInputToFilterInTimelineTime: %.2f seconds", CMTimeGetSeconds(startTimeOfInputToFilterInTimelineTime));
NSLog(@"timelineTimeMinusStartTimeOfInputToFilterNumerator: %f", timelineTimeMinusStartTimeOfInputToFilterNumerator);
NSLog(@"timelineTimeMinusStartTimeOfInputToFilterDenominator: %f", timelineTimeMinusStartTimeOfInputToFilterDenominator);
NSLog(@"frame: %f", frame);
NSLog(@"timelineFpsNumerator: %f", timelineFpsNumerator);
NSLog(@"timelineFpsDenominator: %f", timelineFpsDenominator);
NSLog(@"frameRate: %f", frameRate);
NSLog(@"timestamp: %f", timestamp);
NSLog(@"---------------------------------");

For the first frame of the 29.97fps Compound Clip I get:
timelineFrameDuration: 0.03 seconds
timelineTime: 0.00 seconds
startTimeOfInputToFilter: 37960.22 seconds
startTimeOfInputToFilterInTimelineTime: 0.00 seconds
timelineTimeMinusStartTimeOfInputToFilterNumerator: 0.000000
timelineTimeMinusStartTimeOfInputToFilterDenominator:<PHONE_NUMBER>.000000
frame: 0.000000
timelineFpsNumerator: 30000.000000
timelineFpsDenominator: 1001.000000
frameRate: 29.970030
timestamp: 0.000000

For the first frame of the 23.98fps Project/Timeline I get:
timelineFrameDuration: 0.03 seconds
timelineTime: 0.00 seconds
startTimeOfInputToFilter: 37960.22 seconds
startTimeOfInputToFilterInTimelineTime: 0.00 seconds
timelineTimeMinusStartTimeOfInputToFilterNumerator: 0.000000
timelineTimeMinusStartTimeOfInputToFilterDenominator:<PHONE_NUMBER>.000000
frame: 0.000000
timelineFpsNumerator: 30000.000000
timelineFpsDenominator: 1001.000000
frameRate: 29.970030
timestamp: 0.000000

So maybe what is needed is a way to enter the "Project/Timeline Frame Rate" in the Effect, and then we can scale the frame number accordingly?

The Final Cut Pro team confirm:

We don’t currently have a method to do what you’re looking for.

Haha, so helpful.


Small Fixes

Added missing semicolons, use [] for new array and compare 0 with ===.

have integrated these changes in the latest commits


No changes but still seems like it has changes

Testing #226665

This was a old notebook. Not really sure what the changes are, but it might be changes to the JSON itself, rather than the content. But, it is not clear from this view why it says there is a difference.

@karthiknadig Please can you open the plain text diff viewer (icon on the trop right in editor toolbar area)

This is what it shows with plain text diff viewer

Duplicate of https://github.com/microsoft/vscode/issues/208280


[RFR] Allow listActions on ReferencedList

Closes #269

Note that the next major version will probably refactor that, because it doesn't make sense to replicate the listView API on the ReferencedList. The right solution is to let the ReferencedList take a listView as parameter.
But that's a deeper refactoring, and it doesn't justify delaying this simple change.


bluetooth stack change affects timer behavior

Commit 8a92818c0a0ceea30a9540e3e1557dec2cdf6fbf broke my bluetooth mesh application, in that a work item submitted to the system work queue is never executed.
Prior to the commit the backtrace when idle was:
#0  k_cpu_idle () at /mnt/devel/external/zp/zephyr/arch/arm/core/cpu_idle.S:139
#1  0x0001bf94 in sys_power_save_idle () at /mnt/devel/external/zp/zephyr/kernel/idle.c:100
#2  idle (unused1=<optimized out>, unused2=<optimized out>, unused3=<optimized out>)
    at /mnt/devel/external/zp/zephyr/kernel/idle.c:155
#3  0x0001d5a2 in z_thread_entry (entry=0x30 <z_get_fd_obj+48>, p1=<optimized out>, p2=<optimized out>,
    p3=<optimized out>) at /mnt/devel/external/zp/zephyr/lib/os/thread_entry.c:29
#4  0x0001d5a2 in z_thread_entry (entry=0x30 <z_get_fd_obj+48>, p1=<optimized out>, p2=<optimized out>,
    p3=<optimized out>) at /mnt/devel/external/zp/zephyr/lib/os/thread_entry.c:29

At the commit the idle backtrace is:
#0  cpu_sleep () at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/ll_sw/nordic/hal/cpu.h:11
#1  0x000184ca in role_disable (ticker_id_primary=ticker_id_primary@entry=5 '\005', ticker_id_stop=ticker_id_stop@entry=3 '\003') at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/ll_sw/ctrl.c:10566
#2  0x0001a052 in radio_adv_disable () at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/ll_sw/ctrl.c:10997
#3  0x0001a4d2 in ll_adv_enable (enable=<optimized out>) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/ll_sw/ll_adv.c:404
#4  0x00017990 in le_set_adv_enable (buf=0x2000523c <net_buf_hci_cmd_pool+28>, evt=0x20002724 <tx_thread_stack+516>) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/hci/hci.c:1650
#5  controller_cmd_handle (node_rx=0x2000273c <tx_thread_stack+540>, evt=0x20002724 <tx_thread_stack+516>, cmd=0x2000523c <net_buf_hci_cmd_pool+28>, ocf=10) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/hci/hci.c:1650
#6  hci_cmd_handle (cmd=0x2000523c <net_buf_hci_cmd_pool+28>, node_rx=node_rx@entry=0x20002754 <tx_thread_stack+564>) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/hci/hci.c:2241
#7  0x00017094 in cmd_handle (buf=0x2000523c <net_buf_hci_cmd_pool+28>) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/hci/hci_driver.c:384
#8  hci_driver_send (buf=0x2000523c <net_buf_hci_cmd_pool+28>) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/controller/hci/hci_driver.c:434
#9  0x00007a98 in bt_send (buf=0x2000523c <net_buf_hci_cmd_pool+28>) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/host/hci_core.c:4561
#10 send_cmd () at /mnt/devel/external/zp/zephyr/subsys/bluetooth/host/hci_core.c:3565
#11 process_events (ev=0x20005cac <events>, count=2) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/host/hci_core.c:3589
#12 hci_tx_thread (p1=<optimized out>, p2=<optimized out>, p3=<optimized out>) at /mnt/devel/external/zp/zephyr/subsys/bluetooth/host/hci_core.c:3649
#13 0x0001d452 in z_thread_entry (entry=0x79c9 <hci_tx_thread>, p1=<optimized out>, p2=<optimized out>, p3=<optimized out>) at /mnt/devel/external/zp/zephyr/lib/os/thread_entry.c:29
#14 0x0001d452 in z_thread_entry (entry=0x79c9 <hci_tx_thread>, p1=<optimized out>, p2=<optimized out>, p3=<optimized out>) at /mnt/devel/external/zp/zephyr/lib/os/thread_entry.c:29

In short, it's waiting here:
		if (ret == TICKER_STATUS_BUSY) {
			/* wait for ticker to be stopped */
			while (ret_cb == TICKER_STATUS_BUSY) {
				cpu_sleep();
			}
		}

but whatever's supposed to change the ticker state doesn't happen.
Commenting out the sleep has no effect.
Changing the loop body to invoke ticker_job_sched like similar loops also has no effect.

The behavior is non-deterministic, i.e. the application may run for a few seconds before the conditions that introduce the failure occur.
Reverting 8a92818c0a0ceea30a9540e3e1557dec2cdf6fbf works.  The cause of the failure may be related to something like the diagnostic below (before reverting I'd just see nothing after the first or second periodic.
0:00:12.078: periodic
[00:00:11.887,634] <wrn> bt_mesh_net: Duplicate found in Network Message Cache
0:00:17.078: periodic

That commit has now been reverted from master.
Therefore I guess this issue can be closed?


Coin balancer shouldn't use reference inputs from set of additional UTxOs

Completed in https://github.com/geniusyield/atlas/pull/59


Running Cost Analysis

Create running cost analysis

Cost model is functioning properly. Line items need to be added to finish project. But is a workable model for now.
https://docs.google.com/spreadsheets/d/1Qe-xZRGo1yf2owT5izp9FuysLm5nl5NTM4XeVc4r-KM/edit?pli=1#gid=1115838130


ch(UI): updates the CSS of the landing page

What does this PR do?
It updates the CSS of the landing page
Description of Task to be completed?
Developer should be able to update CSS of the landing page
Any background context you want to provide?
I updates the CSS of the landing page
What are the relevant pivotal tracker stories?
Story Link
Screenshots

@lionelsenyana Can you just add the UI's screenshot for tasks related to the frontend?


Code Cleanup: Use GeneratedRegexAttribute for All Regular Expressions (.NET 7+)

Description
This small update improves trimmability of MudBlazor as well as improves performance of regular expressions by converting them to Source Generated Regular Expressions via the GeneratedRegexAttribute.  These changes were done by fixing all Roslyn suggestions solution-wide for SYSLIB1045.
How Has This Been Tested?
These changes were tested by running all unit tests and going through all of the "Masking" examples.
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[X] I've added relevant tests.

Notes for Reviewers
NOTE: The GeneratedRegexAttribute class is a .NET 7 feature.
Each code change made by Roslyn for SYSLIB1045 made these changes:

Added a static partial method for the regular expression.
Changed the code to use the new partial method.
Added the partial keyword to classes containing regular expressions.

Hi.
Thanks for the PR. I wanted to bring this optimization soon or later, glad someone contributed.

Hi.
Thanks for the PR. I wanted to bring this optimization soon or later, glad someone contributed.

I'm glad to hear it!  I recall people talking about wanting to enable trimming so I figured this could help in that direction.  Also, it's really neat how the generated methods have really nice documentation.

Sorry, not related dotnet format errors

No need to deal in this PR with the dotnet format, @danielchalmers will make separate PR for the fix, since he broke it here https://github.com/MudBlazor/MudBlazor/pull/8421
but the commit didn't have the dotnet format yet

No need to deal in this PR with the dotnet format, @danielchalmers will make separate PR for the fix, since he broke it here #8421 but the commit didn't have the dotnet format yet

Ohh ok, whoops, I just saw this after doing a commit.  I'll undo

Thanks for being so proactive though @jperson2000

@jperson2000 you can now merge your branch with the latest changes for it to build.

@jperson2000 you can now merge your branch with the latest changes for it to build.

Nice work, @ScarletKuro !  The build now succeeds :)

@jperson2000 you can now merge your branch with the latest changes for it to build.

Nice work, @ScarletKuro ! The build now succeeds :)

Also @henon the merge worked way better than a rebase this time haha

Just a few naming issues left mentioned in my review

@jperson2000 thanks!

I noticed at least one error you introduced during refactoring - you have changed the program logic by mistake. Could you check all the changes in the code to make sure there are no other errors?

These changes were tested by running all unit tests and going through all of the "Masking" examples.

The code you changed is also heavily used in API documentation - https://mudblazor.com/api/alert etc. So please also check these pages. You can check them automatically using the https://github.com/MudBlazor/MudBlazor/discussions/3595 program.

I noticed at least one error you introduced during refactoring - you have changed the program logic by mistake. Could you check all the changes in the code to make sure there are no other errors?

These changes were tested by running all unit tests and going through all of the "Masking" examples.

The code you changed is also heavily used in the API documentation - https://mudblazor.com/api/alert etc. So please also check these pages. It is best to use also the https://github.com/MudBlazor/MudBlazor/discussions/3595 program to check all the pages automatically because some code you changed is used only in some of them.
@ScarletKuro, please don't make a release until it is fixed.

I noticed at least one error you introduced during refactoring - you have changed the program logic by mistake. Could you check all the changes in the code to make sure there are no other errors?

Hi.
Can you be more specific? What is the mistake and at what line, also what is the difference in the behavior.
Currently your message is too vague stating that during refactoring the logic was changed by mistake.

The code behind the documentation of MudBlazor API is not straightforward. Thus, any changes should be checked twice in the code, and/or it should be tested using https://github.com/MudBlazor/MudBlazor/discussions/3595. Otherwise, it is very easy to introduce bugs. I believe none of them has been done. I could be more specific but then only this one bug will be fixed, and we will not know if there are others.
The bug I have found can be noticed easily by just comparing the previous and the current code, thus I believe it wasn't checked twice.

@iwis Thanks for checking our PRs so thoroughly! It whould be really nice of you to report the mistakes you found.
As for testing with Selenium, we know that that would be awesome if we had a selenium test suite. However we weren't able to figure out how to integrate it with our CI server.


How to use getSource with Client Application

Hi,
I am new to JavaScript.
I want to get all audio & video source in client application (means without any web server e.g. apache).
I used getSources API to get all devices but it doesn,t work without web-server (apache server).
Could you please guide me how to get all local devices in client application (without any web-server)?
Thanks in Advance.

Hi – you need to run getUserMedia() examples from a server.
It's easy enough to run a server locally with SimpleHTTPServer, or stacks
like MAMP or XAMPP.
Alternatively, run Chrome with flags to enable calling getUserMedia() from
a file:// URL:
stackoverflow.com/questions/13723699/chrome-getusermedia-not-requesting-permission-locally
.
Sam
On Tue, Jan 20, 2015 at 7:53 AM, KP Singh<EMAIL_ADDRESS>wrote:

Hi,
I am new to JavaScript.
I want to get all audio & video source in client application (means
without any web server e.g. apache).
I used getSources API to get all devices but it doesn,t work without
web-server (apache server).
Could you please guide me how to get all local devices in client
application (without any web-server)?
Thanks in Advance.
—
Reply to this email directly or view it on GitHub
https://github.com/samdutton/simpl/issues/61.

Thanks Sam


add conda install doc to end-to-end tutorial

I had some trouble with conda installation because the tutorial was incomplete. I added some instructions that would have helped me.

Very helpful, thank you! :)


Uninitialized RigidBody When Created While in Sandbox Paused Mode

Summary
When starting the sandbox and pausing it, cloning or creating new RigidBodies will leave them uninitialized, so they won't participate in physical simulation.
How to reproduce

Set up a simple scene with a single RigidBody.
Enter sandbox mode.
Pause the sandbox.
Clone the RigidBody.
Resume the sandbox.

Analysis

The culprit is likely this line that will leave the RigidBody disabled while in the editor.
Since RigidBody doesn't listen to execution context changes, a body that is created in a paused sandbox (editor mode) will remain disabled indefinitely.
The easiest fix is probably adding an execution context check in OnUpdate that will enable the internal body when appropriate.

Done.


It is taking too much tym

The website is taking too much time to load

@tushargupta1504 This is a very good issue but I will like to suggest you to please use proper issue templates to raise your issue

The issue is still open just update your issue properly so that I can assign you this

bro it happens because of css file and lots of Optimize Images in this project
there are may css stm which are not used any where or some  same css stm are present in many files

@tushargupta1504 are you working on this issue or I will close this issue

Working

Working

In that Case, I am closing this issue , you must raise a new issue with proper issue template describing your issue


How to use libraries?

Sending the class across is really cool, but I'm trying to figure out how it would reference libraries for using in that class. For example I need to do import * as moment from 'moment' over there somehow?

Ohai @esprehn 👋
yes, the API currently is a bit deceptive. Any extended class or something probably won’t work. Eventually (when modules land in workers), I‘d like to change the API to Clooney.spawn(modulePath, exportName).
But for now, I thought I’d let people play with this.
If accessing globals becomes a problem, this can be solved via a custom strategy. But I haven’t done that yet.


Nova Flexible Content

Hello,
Is it compatible with Nova Flexible Content ?
I tried it and had a JS error..
Thanks !

Currently it doesn't work with flexible content

Yep, that's why i tried Nova Dependency Container but I encountered 2
issues with that package too :

The conditional field doesn't show on the detail view

Some values are overwritten by the last item of the layout

I've seen you used that package too, do these issues ring a bell to you ?
Thanks.
Le sam. 14 déc. 2019 à 02:34, Rafael Milewski<EMAIL_ADDRESS>a
écrit :

Currently it doesn't work with flexible content
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dcasia/conditional-container/issues/17?email_source=notifications&email_token=AEV5PSOYN3CPZ25FQU3FBOTQYQZ23A5CNFSM4J2O6QCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG3WQSQ#issuecomment-565667914,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEV5PSJZRTECFPWA5OZNMJTQYQZ23ANCNFSM4J2O6QCA
.

Yeah I used that package before too but had too many issues for my use case so I built this one instead, I have seen how the Nova Dependency Container implemented support for the flexible content on this commit: https://github.com/epartment/nova-dependency-container/commit/d4081499c45b86111d5d95ef3c4457db4dd4bbdf perhaps it may be very simple to port it over... I will give it a shoot, and about the 2 issues, I don't see how conditional-container would fail in there

@dividy I got it almost working and I had pushed my proof of concept to a branch called flexible field, it works on create, on update and on details view as expected, however validation is still broken :(

Wow.. This sounds like a great step forward to something functional ;-)
I'll try to have a look at this too!

Just did a composer require digital-creative/conditional-container:dev-flexible-content
Tried the package, and I'm still having an issue inside Flexible Content...
Here's the code, can you try it on your side and tell me if it works ?
``
Flexible::make('Produits', 'products')
->addLayout('Product', 'product', [
Select::make('Type de contrat', 'type')->options([
'L' => 'Location',
'VL' => 'Vente & Location',
'R' => 'Relevé',
'V' => 'Vente',
'VU' => 'Vente unique',
])->displayUsingLabels()->rules('required'),
            ConditionalContainer::make([ Text::make('Prix en Location (htva)', 'price_rent') ])
            ->if('type = L'),

            ConditionalContainer::make([ Text::make('Prix de Vente (htva)', 'price_sale') ])
            ->if('type = V')
            ->if('type = VU'),

            ConditionalContainer::make([
                Text::make('Prix en Location (htva)', 'price_rent'),
                Text::make('Prix de Vente (htva)', 'price_sale')
            ])->if('type = VL'),

            ConditionalContainer::make([
                Text::make('Prix du Relevé (htva)', 'price_reading'),
            ])->if('type = R'),
        ]),

On my side, it never works, and displays all the fields when I select 'R - Relevé'.. Strange ;)

@milewski  I just used the branch as well and tried the code of @dividy but i get
TypeError: Cannot read property 'join' of undefined
    at _loop (conditional-container:175)

And I even dont see the fields.
I also tried my own, simpler example with the same result:
Flexible::make("Data")
    ->addLayout("Simple Layout", "html", [
        Select::make("Type", "type")->options([
            "html" => "HTML",
            "plain" => "Plaintext"
        ]),
        ConditionalContainer::make([
            Trix::make('Content', 'html_content')->gridClass("w-full")
        ])->if('type = html'),
        ConditionalContainer::make([
            Textarea::make('Content', 'html_content')->gridClass("w-full")
        ])->if('type = plain"')
    ])

@bernhardh you probably forgot to include the HasConditionalContainer trait your example works for me:

The join issue is because the HasConditionalContainer wasn't included ... but yeah indeed there are several issues, it did work for the example I had when I was developing it but using your samples has some bugs...

@bernhardh I just fixed the issues with your example you can try again it should be working perfectly now (but without validation of course)
Form View

Detail View

@dividy your example also works now for me... I think the issue were you forgot the trait

Oh.  Thank you!
Now I get
Method Whitecube\NovaFlexibleContent\Flexible::resolveConditionalContainer does not exist.

The
if ($flexibleContent->isNotEmpty()) {
    $this->registerFlexibleMacros($request, $flexibleContent);
}

is false, so this macro is never added?

whats the endpoint nova is trying to hit that throws that error Method Whitecube\NovaFlexibleContent\Flexible::resolveConditionalContainer does not exist. ?

It occurs on edit and detail page. And the endpoint throwing it is /nova-api/MODELNAME/ID

Btw. I am on dev-flexible-content 7e5e620 atm.

Oh I see when you use the nova-tabs the "flexible" content is not found, so the macros are not added to it, that's why it crashes, I think it might be possible to detect, I think it is possible to add a check and see if its an instance of panel and recursively try to find flexible contents within its children

Ok, at least I can fix the issue with the tabs package (and all other packages based on Panel items).
In HasConditionalContainer.php:460 just add
``php
if($field instanceof Panel && $field->data) {
return $this->findAllFlexibleContentFields($field->data);
}

Ah ok let me add

Pushed

Ok, now it only worked on empty data. After save i get
Cannot bind an instance to a static closure

To fix this change HasConditionalContainer:178
$field::macro('generateFieldName', static function (array $fields) {

to
$field::macro('generateFieldName', function (array $fields = []) {

Not sure why it is decleared static in the first place? Or does this have any sideeffects?

That was a mistake, I didn't know macros wouldnt run in static context, I have removed it and pushed again

Thank you very much, for your help, this package (and nova-mega-filter and collapsible-resource-manager). Do you have a "buy me a cup of coffee" link or something else?

Good job guys, I watched all the action from far away, getting spammed by notifications.
I have to deliver the application to my customer, so I finally chose another working option, I used different presets instead of conditional fields. However, great work!

@dividy Can I know what you use in order to fix this issue?

@dividy I got it almost working and I had pushed my proof of concept to a branch called flexible field, it works on create, on update and on details view as expected, however validation is still broken :(

if ($controller instanceof CreationFieldController ||
            $controller instanceof UpdateFieldController) {

this is the part where you check the fields for creation?


Unable to install

I can't install this module because of the following error:
ENOENT: no such file or directory, chmod 'C:\Users\leurba\AppData\Roaming\npm\node_modules\ngx-i18nsupport\dist\xliffmerge\xliffmerge'

It looks like maybe the latest npm was published without the dist folder?

I just tried npm i<EMAIL_ADDRESS>and it worked so I'm guessing it's just 0.3.0 that is broken.

Oops, sorry about that.
There was a compilation problem and I was a litle bit in hurry and ignored the travis output.
Should be fixed in 0.3.1 now.


Make sidebar position configurable & other styling issues

[x] Make sidebar's position (left or right) configurable from chaiseConfig
[x] Remove ellipsis overflow in search results table

Wrap the text but give each cell an appropriate max-height.

[x] When sidebar is on left, fix the bug where the navbar's dropdowns render behind the sidebar.
[x] Leave a hard buffer between the sidebar and the search results as the browser's width decreases.
[x] Add a scrollbar to the left sidebar
[x] #td.results-title should contract like the other cells
[x] Increase width of #main-content to take advantage of more surrounding whitespace
[x] Make the "Choose Attributes" text in the Selected Attributes sidebar configurable via chaiseConfig
[x] Prevent set of.view-toggle btns from individually breaking into next line when screen size decreases.
[x] Document chaiseConfig.sidebarPosition, chaiseConfig.attributesSidebarHeading

These fixes and features have been pushed to master. Closing the issue.


SUGADD, SUGGET; added Interfaces for the contract to the consumer of the API. A Build…

…er for the Client and ClusterClient probably would be good next too. Allow the consumers of this api not to be concrete connected to the implementations.
I also commented out testPayloads() line 203 it was broken when i ran the test suite to begin with.

I see that some new classes some how missed on the add will have them added by 16:00 CST

Generally looks ok:

I'd rather that the client remain a concrete class (I'd be ok with Client and AbstractClient, etc. though). Are you intending for there to be another provider of this API? If and when such a time arrives, we can consider using interfaces.
Please consolidate all the suggestion methods. I don't like expanding the API surface area like that (but I'm not a Java guy). I'd rather use an option-builder like thingy. Less API surface area to maintain.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: davidparry:x: mutualinkdavidparry seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

(1)
It was not my intent to remove the Concrete class by Introducing the initial 2 Interfaces or to have other providers, but instead to allow easier maintainability without headaches to the api consumer. I am addressing qualifying factors that java teams and companies, current one included, that must have these standard set of quality features before a project can adopt a new library dependency. The Interface introduction is only a step towards a handful of Design Principles and other factors that increase the chances of a Java library to be adopted in a new project vs (another library, writing from scratch, etc..). Interface Segregation Principle, Dependency Inversion Principle, Inversion of Control, Dependency Injection, Code Quality and Metrics. I plan on introducing some more of these Principles to this project, so I can get it accepted as a dependency into our Solution.
(2)
I was torn on which way to go, seeing the style you were using.  My thoughts where that the caller knows if they want a payload, score, both or none, so let them make the choice and tell us this in the form of a more specific method. Hence removing the case statements within the method to find out what the caller desired. In Java and the Unit Test encourages this to have smaller methods than one larger more complex method which we need to perform logic to see if the caller wanted a payload, score, both or none. I will go ahead and make it one and add the complexity in the one method.

Yeah, if we keep the API surface area small, then the duplication of interfaces vs classes is not horrible- but if we offer a ton of overloads then we'd end up with a sticky mess.

I'm looking at the current interface, and perhaps Replace needs to go as well, anyway, that's enough for a single iteration :)

Ok will get it tomorrow I hope
On Fri, Aug 17, 2018, 17:01 Mark Nunberg<EMAIL_ADDRESS>wrote:

I'm looking at the current interface, and perhaps Replace needs to go as
well, anyway, that's enough for a single iteration :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RedisLabs/JRediSearch/pull/31#issuecomment-413998977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVRjG-xx5Suf-hQ7zCYgc3vmCTi0qGfks5uRz06gaJpZM4V7q-4
.


When I updated the master record, the child record on all related tables is soft deleted.

It's a strange behavior or bug? When I updated the master record,  all the related records automatically soft deleted.

This does happen when the children aren't presented on update. ie:- if we've removed the childrens form from the frontend.
loadAll and saveAll expect the children to be present, and if they're not... they'll be soft-deleted.
Hope this helps others ... as it had me going for a few hours :)