Formatting of message box

I have the following confirmation format for BulkServiceAction. Inserting HTML tags or crlf did not help formatting it properly. How can I format it as a list with each item on a separate line?
public class RecalcBudgetAction : BulkServiceAction
{
    protected override string GetConfirmationFormat()
    {
        StringBuilder buffer = new StringBuilder("This action will automatically perform the following for selected department(s):");
        buffer.Append(" ");
        buffer.Append(" (1). Calculate default appropriation requested budget (last year $ + user defined %");
        buffer.Append(" (2). Calculate default position requested budget (last year $ + user defined %)");
        buffer.Append(" (3). Calculate default insurance amount (full time employee count * user defined $)");
        buffer.Append(" (4). Calculate default FICA amount (total salary $ * user defined %)");
        buffer.Append(" (5). Calculate default PERF amount (total salary $ * user defined %)");
        buffer.Append(" ");
        buffer.Append("Perform this operation on {0} selected record(s)?");

        return buffer.ToString();
    }

Try using
 and
 tags then set HtmlEncode false in third param to confirm

Ah you need to override Confirm method in BulkServiceAction to set these options

That worked beautifully - thanks again. Here is the code I used:
public class RecalcBudgetAction : BulkServiceAction
{
    protected override void Confirm(int targetCount, Action action)
    {
        ConfirmOptions co = new ConfirmOptions();
        co.HtmlEncode = false;
        Q.Confirm(GetConfirmationMessage(targetCount), action, co);
    }

    protected override string GetConfirmationFormat()
    {
        StringBuilder buffer = new StringBuilder("This action will automatically perform the following for selected department(s):");
        buffer.Append("
");
        buffer.Append("
Calculate default appropriation requested budget (last year $ + user defined %
");
        buffer.Append("
Calculate default position requested budget (last year $ + user defined %)
");
        buffer.Append("
Calculate default insurance amount (full time employee count * user defined $)
");
        buffer.Append("
Calculate default FICA amount (total salary $ * user defined %)
");
        buffer.Append("
Calculate default PERF amount (total salary $ * user defined %)
");
        buffer.Append("
Roll up salaries by account category into matching appropriation line items
");
        buffer.Append("
");
        buffer.Append("Perform this operation on {0} selected record(s)?");

        return buffer.ToString();
    }


Use factorized matrices

A first try to tackle #107. The SummaryCallback gives the following after an initial run to exclude compilation time:
Current main:
──────────────────────────────────────────────────────────────────────────────
        DispersiveSWE                 Time                    Allocations
                             ───────────────────────   ────────────────────────
      Tot / % measured:           1.38s /  98.7%            762MiB /  99.9%

 Section             ncalls     time    %tot     avg     alloc    %tot      avg
 ──────────────────────────────────────────────────────────────────────────────
 rhs!                 16.8k    1.33s   97.8%  79.5μs    745MiB   98.0%  45.5KiB
   deta elliptic      16.8k    535ms   39.2%  31.9μs   67.5MiB    8.9%  4.12KiB
   dv elliptic        16.8k    517ms   38.0%  30.9μs   67.5MiB    8.9%  4.12KiB
   dv hyperbolic      16.8k    141ms   10.3%  8.42μs    338MiB   44.4%  20.6KiB
   deta hyperbolic    16.8k    128ms    9.4%  7.66μs    270MiB   35.5%  16.5KiB
   ~rhs!~             16.8k   10.7ms    0.8%   639ns   2.56MiB    0.3%     160B
   source terms       16.8k    640μs    0.0%  38.2ns     0.00B    0.0%    0.00B
 analyze solution       280   30.2ms    2.2%   108μs   15.4MiB    2.0%  56.5KiB
 ──────────────────────────────────────────────────────────────────────────────

This branch
  ──────────────────────────────────────────────────────────────────────────────
        DispersiveSWE                 Time                    Allocations
                             ───────────────────────   ────────────────────────
      Tot / % measured:           680ms /  98.0%           1.66GiB /  99.9%

 Section             ncalls     time    %tot     avg     alloc    %tot      avg
 ──────────────────────────────────────────────────────────────────────────────
 rhs!                 16.8k    635ms   95.2%  37.9μs   1.64GiB   99.1%   103KiB
   dv elliptic        16.8k    175ms   26.2%  10.4μs    534MiB   31.5%  32.6KiB
   deta elliptic      16.8k    174ms   26.1%  10.4μs    534MiB   31.5%  32.6KiB
   dv hyperbolic      16.8k    147ms   22.0%  8.77μs    338MiB   19.9%  20.6KiB
   deta hyperbolic    16.8k    127ms   19.0%  7.55μs    270MiB   15.9%  16.5KiB
   ~rhs!~             16.8k   11.6ms    1.7%   690ns   2.64MiB    0.2%     165B
   source terms       16.8k    579μs    0.1%  34.6ns     0.00B    0.0%    0.00B
 analyze solution       280   32.1ms    4.8%   115μs   15.4MiB    0.9%  56.5KiB
 ──────────────────────────────────────────────────────────────────────────────

So the new version is almost twice as fast as the old one, which is good. But I feel like we can avoid the many allocations in the new version. I tried using ldiv!(invImD2, deta), but this gave the following error message:
ERROR: LoadError: MethodError: no method matching ldiv!(::SparseArrays.CHOLMOD.Factor{Float64, Int64}, ::SubArray{Float64, 1, Matrix{Float64}, Tuple{Int64, Base.Slice{Base.OneTo{Int64}}}, true})

Any ideas to improve this further, @ranocha? After we settled on an efficient way, I can apply it to the other equations, too.

Using lu instead of factorize gives:
──────────────────────────────────────────────────────────────────────────────
       DispersiveSWE                 Time                    Allocations
                            ───────────────────────   ────────────────────────
     Tot / % measured:           586ms /  96.4%            904MiB /  99.1%

Section             ncalls     time    %tot     avg     alloc    %tot      avg
──────────────────────────────────────────────────────────────────────────────
rhs!                 16.8k    536ms   94.8%  31.9μs    880MiB   98.3%  53.8KiB
  deta elliptic      16.8k    137ms   24.3%  8.18μs    135MiB   15.1%  8.25KiB
  dv elliptic        16.8k    135ms   23.9%  8.07μs    135MiB   15.1%  8.25KiB
  dv hyperbolic      16.8k    129ms   22.9%  7.72μs    338MiB   37.7%  20.6KiB
  deta hyperbolic    16.8k    126ms   22.3%  7.52μs    270MiB   30.2%  16.5KiB
  ~rhs!~             16.8k   7.18ms    1.3%   428ns   2.56MiB    0.3%     160B
  source terms       16.8k    538μs    0.1%  32.1ns     0.00B    0.0%    0.00B
analyze solution       280   29.6ms    5.2%   106μs   15.4MiB    1.7%  56.5KiB
──────────────────────────────────────────────────────────────────────────────

Again, ldiv! doesn't work. This time because of ERROR: LoadError: ArgumentError: in and output vectors must have unit strides.

Using your first approach (i.e. invImD2 = inv(I - 1 / 6 * D^2 * solver.D2)) gives ERROR: LoadError: ArgumentError: FFTW plan applied to wrong-strides array while deta[:]=invImD2 * deta.

Trying your second approach gives me ERROR: LoadError: matrix does not have contiguous columns while ldiv!(deta, invImD2, deta) (or ldiv!(invImD2, deta))

Yes, I do. Just so that I understand: The problem is that u_ode, i.e. the data OrdinaryDiffEq.jl is working with, is just a vector and not the matrix of size nvariables(equations), nnodes(mesh)? So we could also consider just using the matrix as the data for OrdinaryDiffEq.jl and would also spare us some wrap_arrays? IIRC, in Trixi.jl we need to store the data as a vector to be able to resize! it for AMR, right? We don't need to do this here. What do you think? Or am I misunderstanding something?

I tried your suggestion to use temporary arrays in 885c329. This works with no allocations, but is much slower. Did you do something wrong? (Ignore the reflecting boundary conditions for now. The result below is for examples/bbm_bbm_1d/bbm_bbm_1d_basic.jl, so periodic bc.
 ──────────────────────────────────────────────────────────────────────────────
        DispersiveSWE                 Time                    Allocations
                             ───────────────────────   ────────────────────────
      Tot / % measured:           3.62s /  99.5%            627MiB /  99.8%

 Section             ncalls     time    %tot     avg     alloc    %tot      avg
 ──────────────────────────────────────────────────────────────────────────────
 rhs!                 16.8k    3.56s   99.0%   212μs    610MiB   97.5%  37.3KiB
   deta elliptic      16.8k    1.66s   46.1%  98.9μs     0.00B    0.0%    0.00B
   dv elliptic        16.8k    1.62s   45.0%  96.5μs     0.00B    0.0%    0.00B
   dv hyperbolic      16.8k    133ms    3.7%  7.92μs    338MiB   54.0%  20.6KiB
   deta hyperbolic    16.8k    130ms    3.6%  7.75μs    270MiB   43.2%  16.5KiB
   ~rhs!~             16.8k   23.3ms    0.6%  1.39μs   2.56MiB    0.4%     160B
   source terms       16.8k    709μs    0.0%  42.3ns     0.00B    0.0%    0.00B
 analyze solution       280   36.1ms    1.0%   129μs   15.4MiB    2.5%  56.5KiB
 ──────────────────────────────────────────────────────────────────────────────

Yes, I do. Just so that I understand: The problem is that u_ode, i.e. the data OrdinaryDiffEq.jl is working with, is just a vector and not the matrix of size nvariables(equations), nnodes(mesh)? So we could also consider just using the matrix as the data for OrdinaryDiffEq.jl and would also spare us some wrap_arrays? IIRC, in Trixi.jl we need to store the data as a vector to be able to resize! it for AMR, right? We don't need to do this here. What do you think? Or am I misunderstanding something?

It would have to be a matrix of size nnodes(mesh), nvariables(equations) so that eta and v are contiguous in memory. Alternatively, one could use some types from RecursiveArrayTools.jl, e.g., ArrayPartition

It would have to be a matrix of size nnodes(mesh), nvariables(equations) so that eta and v are contiguous in memory.

Ah yes. Julia stores column major.

Another question for my understanding: Do the two error messages ERROR: LoadError: ArgumentError: in and output vectors must have unit strides and ERROR: LoadError: matrix does not have contiguous columns have the same root cause, namely that we store deta and dv as a vector instead of a matrix or is the former caused by something else?

Yes, the same root cause

Yes, the same root cause

Ok, thanks!


Convert Result<impl IntoVal<>, impl Into> into Status

A Status is sort of like a Rust Result. It has an ok state, as well as many error states.
We should consider translating a Result that contains a Ok(IntoVal<>) into the contained value, and a Result that contains a Err(Into<u32>) into a Status with the type ContractError (ref: https://github.com/stellar/stellar-xdr-next/issues/28).
cc @graydon

Instead I'm doing https://github.com/stellar/rs-soroban-sdk/pull/601.


Metadata API: Metadata "factory" constructors misbehave with inheritance

class DerivedMetadata(Metadata):
    def ok(self):
        print("ok")

md = DerivedMetadata.from_file("root.json")
md.ok() # fails because md type is Metadata, not DerivedMetadata

This seems to happen because the construction path goes

DerivedMetadata.from_file()
DerivedMetadata.from_bytes()
JsonDeserializer.deserialize() here deserializer doesn't know the correct class anymore and calls Metadata.from_dict()
Metadata.from_dict()

Well spotted! The easiest fix would probably be to pass the class to JsonDeserializer.deserialize(), like so:
diff --git a/tuf/api/serialization/json.py b/tuf/api/serialization/json.py
index b043310f..050ef0dc 100644
--- a/tuf/api/serialization/json.py
+++ b/tuf/api/serialization/json.py
@@ -25,15 +25,22 @@ from tuf.api.serialization import (
     SignedSerializer,
 )

+# TODO: Adopt in MetadataDeserializer and define below generic types there
+from typing import Type, TypeVar
+
+# Generic type for **instances** of subclasses of Metadata
+TMetadata = TypeVar("TMetadata", bound=Metadata)
+
+# Generic type for **subclasses** of Metadata
+CMetadata = Type[TMetadata]

 class JSONDeserializer(MetadataDeserializer):
     """Provides JSON to Metadata deserialize method."""

-    def deserialize(self, raw_data: bytes) -> Metadata:
+    def deserialize(self, cls: CMetadata, raw_data: bytes) -> TMetadata:
         """Deserialize utf-8 encoded JSON bytes into Metadata object."""
         try:
             json_dict = json.loads(raw_data.decode("utf-8"))
-            metadata_obj = Metadata.from_dict(json_dict)
+            metadata_obj = cls.from_dict(json_dict)

         except Exception as e:
             raise DeserializationError from e

An alternative would be to require subclasses of Metadata use their custom MetadataDeserializer implementation. This seems cleaner, but maybe unnecessarily complex as long as the subclass can use the base Metadata from_dict factory.

@jku, was there a specific use case for a DerivedMetadata class? Do you think we can move forward with the patch from above?

@jku, was there a specific use case for a DerivedMetadata class?

Nothing urgent at all: I was just experimenting with providing some editing helpers for Metadata. All of that is possible via other designs.
The fix seems right, although we could maybe improve the existing TypeVar in metadata.py instead of adding a new one -- I used T as the type name there thinking it would never be used out of context but your TMetadata might be better...

Speaking of T...
https://github.com/theupdateframework/python-tuf/blob/8ae944ccb256ba0aa13a62f691fd79d4e16c11ad/tuf/api/metadata.py#L77
... is this  recursive definition intentional? That is, Metadata being a subclass of either of "Root", "Timestamp", "Snapshot", "Targets", but also containing them?

At least from my understanding class Metadata(Generic[T]): means  "Metadata is a Generic[T]", or can it also indicate that "Metadata has a Generic[T]"? (typing is still fairly new to me)

so it should mean Metadata is a Generic, and it's generic over the type T -- this is often used to mean that the Generic is container for type T, as it is here (even if our container is a bit unique in that there is always one and only one T in Metadata).

@jku, unless you need this resolved quickly, I think it would be a good self-contained task for someone to get started with python-tuf development. AFAICS what's missing is:

apply/revise patch from above
adopt MetadataDeserializer accordingly (beware of type check case handling and cyclic-imports!)
add a test

Sounds good to me.
Maybe I should point out that e.g. the return value annotation change for deserialize() is not really required: annotating as Metadata is likely just fine (as long as the correct constructor gets called).

Hi, I would like to take up this issue.

I'm sure you're right... I'm not very familiar with serialisation ideas: happy to discuss (maybe after OSS EU) if you want comments


Not sharing session across all nodes

Hey @jondubois,
I can't seem to get the sessions to share across nodes. I'm using Nginx with two "applications servers" running socketcluster with 8 workers each. I also have a state and a broker server connecting them. In the emit middleware, I'm storing an object to the authToken and when a user gets routed to the other server the authToken is empty so their "session" variables are reset and they have to update their filters again. Is there a way to share authToken's between application server?
scServerEmit({ data,event,socket },next) {
        if (event == 'logout') return next()
        let authToken = socket.authToken || {}

       // Set some data to the authToken
       ....

       if (authToken && Object.keys(authToken).length) socket.setAuthToken(authToken)
        return next()
})

I'm not sure if it matters but I'm using express-session-middleware for logins and API requests. Would this interfere? If user happens to be routed to the same server they were on before by nginx then their session works just fine.

I dirty fixed this by adding ip_hash; to my nginx config but all that does is force users to stay on the same server. I would like to have the ability to move them to whatever server I want without having to deal with any issues. @jondubois Is there a way to save to express sessions from inside the emit middleware instead of using SC session? That would also solve it for at least me. Thanks!

@happilymarrieddad The express session plugin doesn't seem to scale out of the box because it stores the data on the server's memory by default. Maybe there is an option to make it store the session data in an external database instead?
Unless you need to store sensitive session data, then JWT is usually a much better, simpler and more scalable solution.

@jondubois
Ok, I stored the data in the DB using the express-session-middleware. Using JWT is great and I would love to do so but I wrote this app a long time ago and switching our login page would be quite complicated... Is there a way to access the express session from the middleware?

... I just answered my own question... I can store and update the authtoken inside the db store... wow... I'll just do that.

In case anyone else encounters this, all you have to do is update the data object in the db and it works across all systems. They can even login multiple browsers at different IP's and it works. I've been keying off the user_id that I store in the req.session object. Probably not the most performant thing to do but I need accuracy more than performance at this point. I'll do it a better way if I find a better way soon. Thanks!


[IOTDB-4117] fix compact bug ArithmeticException: / by zero

https://issues.apache.org/jira/browse/IOTDB-4117

@THUMarkLau @choubenson   Could you help review this PR about compact?  ;-)


Fix schema gen test errors

Just a fix

Might be good to document this process or something like that?


Update support to most recent Python 3.x versions.

This includes:

Updating test suite
Updating tags in setup.py
Updating py.test version to work with Python 3.5

Closing in favor of one on the main fork.


Problem when using Docker image

I'd like to use the Docker image to generate the Ignition Config with Gitlab CI.
Currently my configuration (common.fcc) looks like this:
variant: fcos
version: 1.0.0
passwd:
  users:
    - name: core
      password_hash: "<removed>"

And my .gitlab-ci.yml looks like this:
stages:
  - build

generate_config:
  stage: build
  image: quay.io/coreos/fcct:v0.2.0
  script:
    - mkdir config
    - /usr/local/bin/fcct --strict --pretty --input common.fcc --output config/common.ign
  artifacts:
    name: "config-${CI_JOB_STAGE}_${CI_COMMIT_REF_NAME}"
    paths:
      - config/*

Every push in my repo triggers a CI build. The build always fails with the following error message:
...
Skipping Git submodules setup

Error translating config: Error unmarshaling yaml: yaml: unmarshal errors:
  line 3: cannot unmarshal !!str `set -eo...` into common.Common
Error translating config: Error unmarshaling yaml: yaml: unmarshal errors:
  line 3: cannot unmarshal !!str `set -eo...` into common.Common

ERROR: Job failed: exit code 1

But on a local VM I am able to successfully generate the Ignition Config with the same command and config file. How is this possible?

line 3: cannot unmarshal !!str set -eo... into common.Common

This sounds like the input given to fcct is a shell script. I'd suggest to double-check your pipeline to make sure common.fcc has the right content first, before plugging in fcct.

The Dockerfile only contains the fcct executable. But Gitlab CI seems to need shell to execute the commands. So I've built my own Debian based fcct image and it worked. :)


Since Update on 4th Match, 1.22 Unable to Add Consumer Key

After recent update, I see a generate new token banner on the inspector.
I had previously saved my consumer key in the "Data export" option
However It looks to have been lost on Update and I can't see the Data Export option now to Re-enter it.
It's now un-usable.

Hi @tprouvot,
Started testing these steps in sandbox and early results are positive.
Noticed your comments about default connected app. I'm trying to weigh up whether we want to go down this route since I suspect it would mean uninstall of current one, generate a new Oauth entry and install app from that entry.
Can you please advise whether  users will only need to click 'Generate new token' to gain access or will they still need to know/ enter the consumer key. I'm thinking of what happens when we refresh a sandbox or give access to new users.
Thanks
@PHOUSTON90

@tprouvot our testing has shown that users who had not previously used the extension in an org get the error message below:
OAUTH_APP_BLOCKED This app is blocked by Admin
We can only overcome this by granting 'Use Any API Client' to user, getting user to connect and then add Consumer key. After that 'Use Any API Client' can be removed and on next login they come in via our Connected App

Hi @LCh07,
The users that did not saved a consumer key don't need to do it, they can use the default one.
Based on the error "OAUTH_APP_BLOCKED This app is blocked by Admin" I think you missed a part which is the (default) connected app installation and configuration.
https://tprouvot.github.io/Salesforce-Inspector-reloaded/troubleshooting/#unauthorized-or-network-error
To do once by the admin:

Generate new token

Go to "Connected Apps OAuth Usage" to install the connected app

Configure the app (add profile or pset)

Check on  "Connected Apps OAuth Usage" that the app is not blocked, if it is then unblock it

Generate new token

Then the other users should have no error when clicking on "Generate new token" banner
Let me know if it solved your issue

I am seeing this as well for users that did not have the app install prior to the update. The app is not listed as blocked in "Connected Apps OAuth Usage"

I've done that. I've followed all instructions. The app works for all users who have not recently installed it. All users are on v1.22.

Hi @scox-workato , I'm seeing the same as you. I reinstalled the app and it doesn't work, but others in my org who haven't reinstalled can use it fine (after we updated the callback url).

@scox-workato if my understanding is correct you had previously (before 1.22) created a custom connected app and saved its consumer key in data export.
If you did not uninstall the extension, this consumer key should still be saved in your local storage. Have you updated the callback url of this custom connected app?
If you want to use the default connected app introduced in 1.22, you can delete property ending with "_clientId" in your local storage (see https://tprouvot.github.io/Salesforce-Inspector-reloaded/how-to/#migrate-saved-queries-from-legacy-extension-to-salesforce-inspector-reloaded for accessing local storage) and then generate new token.

I have reinstalled the extension. There is no property ending in _clientId in my local storage. The concern is that the connected app is working for users that had connected via earlier versions of the extension, but in 1.22 the extension is only usable by users that have not reinstalled or newly installed the update.

@tprouvot
Is installing the default connected app the only way to get new users to use the extension? What are the other benefits of the default connected app in relation to future updates you may make (want to understand whether default will avoid this type of problems in the future or do we have other ways to do this)?
In a sandbox I've tried deleting our custom app and then attempting to create a new Oauth Entry from which to create the default connected app but this gets stuck (see attached), and can't find property ending in _clientId.
Thanks
SF Inspector reloaded - Agdev30 - Install default.docx

@LCh07 No it is not the only way, but it is the easiest since the users won't have to save a consumer key in the extension.
We should not make any updates on the connected app so the only benefit is to make the configuration easier (no connected app creation and configuration and no need to save the consumer key on each user computer).
Deleting the custom connected app won't have any effect on your configuration.
Make sure to inspect the popup page (right click in the popup), then you should see all the localStorage used for the extension and search for _clientId

Hi all!
Let me add a few additional explanations on top of what Thomas already said:
Before the latest release of the extension, the only way to use it with API Access Control was to create a connected app, add its Consumer Key to the extension settings, and then not forget about refreshing the access token any time it expired.
That was really cumbersome, which is why we introduced a default (and optional) connected app.
With the new version, once the app is approved (which you do just once per org), users don't have anything to do other than clicking the link that shows in the extension.
For this default connected app to work, we had to make a change to the redirect URL used in the OAuth flow, which is why old users of the extensions see a breaking change in orgs with API Access Control enabled. And I'm sincerely sorry for that impact! (But it was the only way to significantly improve the user experience, so overall I think it's worth it).
Now, with the new version, you have 2 ways to use the extensions in orgs with API Access Control:

Legacy way, with your own Connected App.
For this to work, you need to:
1.1) Make sure the callback URL allowed in the connected app is
chrome-extension://hpijlohoihegkfehhibggnkbjhoemldh/data-export.html (which is a change compared to the previous release of the extension)
1.2) And, make sure each user of the extension have registered the Consumer Key of your own Connected App in their Inspector Reloaded install (under Options | API Consumer Key). This step is no different that in the previous version of the extension, and is still required if you want to use your own custom app.

New way, much simpler for new users: install and use the new Connected App.
Pre-requisite: if you used an older version of Inspector reloaded + your connected app, make sure to clean the entry in local storage. Alternatively, you can register manually the key of the new app under Options | API Consumer Key: 3MVG9HB6vm3GZZR9qrol39RJW_sZZjYV5CZXSWbkdi6dd74gTIUaEcanh7arx9BHhl35WhHW4AlNUY8HtG2hs
You also need to install and authorize for all relevant users the new "Salesforce Inspector reloaded" Connected App (as with any app you want to use in an org with API Access Control enabled).

@scox-workato If your new users got the "OAUTH_APP_BLOCKED This app is blocked by Admin" error, it most likely means that then haven't entered the Consumer Key of your own Connected App in their Inspector Reloaded install (under Options | API Consumer Key). In that case, the extension defaults to the new default connected app (that you may not have pre-authorized at that point for the impacted users).

@mehdisfdc , @tprouvot , thanks for all of your work and documentation on this app! I truly appreciate it. And it'll be nice to not have to refresh the access token with the latest version.
@scox-workato , I was seeing the same issue as you. I followed tprouvot's instructions from above more carefully and it worked. I missed adding the default Salesforce Inspector reloaded connected app to the permission set, and also didn't unblock the connected app when it was blocked.

Thank @pyao-bwc !
And just to make sure I've set the right expectations: the new version does not eliminate the need to refresh the token after it expired. But it makes the process faster and more user friendly.

@mehdisfdc @tprouvot I do not have any more time to talk through this issue so I'm going to just move on. I too was able to get it working with the default app. However, with a custom app, the clientId was not being set under the correct name no matter how many different things we tried. We set the consumer key, changed the callback url, made sure the app was unblocked, and made sure the users were on v1.22. I read through the docs all FOUR times I was referred to them on this thread. The consumer key for new users was being saved in local storage as null_clientid because with the connected app, the sfdc url was set to NULL. I have to manually update the local storage to have the correct _clientId prefix.
To reproduce, you could setup a custom connected app with a newly installed salesforce connector reloaded extension and attempt to set the consumer key. @pyao-bwc the default app works for us, but this still is not a resolution to the custom connected app issue. If any of the developers would test a custom app with a newly installed extension (or cleared cache, empty extension local storage) instead of repeatedly referring to the docs, they would see the same issue several of us our seeing, that storing the consumer key and trying to generate a new token does not currently work.

@scox-workato Thank you for insisting!
So, taking a closer look, there is indeed a bug in this version the way the clientId is stored, when entered from the "Data Export" tab. The new "API Consumer Key" field (in the new section tab) works as expected, but this one is not accessible before successful authentication.
@tprouvot  I will log a separate issue (with more details), as it's different than than the issue reported by @PHOUSTON90

@mehdisfdc Thank you! I appreciate your help. This extension is incredible, we are glad we can continue using it.


Huge performance issues (using redux as store)

Sometimes I get 1000 new objects within 1-10 seconds that I would like to display inside tables, if I don't use react-bootstrap-table then it's added to store normally... but if I do, it pretty much makes browser unresponsive for a ~ 2 minutes.
I tried with the most basic table(1 column, no fancy settings) and it doesn't seem to help.
<BootstrapTable
          data={this.props.treeData}
        >
          <TableHeaderColumn
            dataField="keyword"
            isKey
          >Branch
          </TableHeaderColumn>

        </BootstrapTable>

What options do I have ?

There's a known performance issue particular enabling selectRow and or cellEdit. But in your case, you dont. so maybe you can try to add new row by exposed API, but it's not a very redux way(practice)...

handleAddingNewRow(newRow) {
    this.refs.table.handleAddRow(newRow);
}

render(){
    return(
        <BootstrapTable ref='table' ...>
    );
}

After v3.0.0 released, I'll try to fix this issue, if I still have time. Thanks

If somebody else runs into this problem, solution was to buffer data (I did it with RxJS) before updating redux store, so it doesn't re-render 1000 times in a second :)


Sonoff DW2-Wi-Fi-L not supported

Please explain your feature request in a one or two sentences.

Sonoff Model DW2-Wi-Fi-L is currently not supported by the plugin

Is your feature request related to a problem? Please describe.

After adding the new device to ewelink app and having Homebridge restarted, I found the following error...
7/12/2022, 12:22:11 PM] [eWeLink] [Bedroom Door Sensor] is not supported, do create a GitHub issue with the following info:
[154] {"bindInfos":{"gaction":[]},"version":8,"init":1,"fwVersion":"1000.2.1122","mac":"d02702b91420","chipID":"2","type":"3","switch":"off","rssi":"-47","battery":"100","defense":0,"actionTime":1617825378000}.

Any particular eWeLink devices that this relates to?

Anything else?

Hi @thetatae
Please install the beta version of the plugin to see if this device now starts working?
Thanks!

Thanks a lot for your help. I’ll let you know once I test it.


[#690]Implement the ability to read total_private_repos for GHPerson

Description
I took a crack at implementing what is mentioned in #690
For test writing, I wasn't sure whether to add to GHPersonTest or GHUserTest. Went with GHUserTest since that's a resource under which you can create a private repo
Prerequisites:

[x] Push your changes to a branch other than master. Create your PR from that branch.
[x] Add JavaDocs and other comments
[x] Write tests that run and pass in CI. See CONTRIBUTING.md for details on how to capture snapshot data.
[x] Run mvn formatter:format. This may reformat your code, commit those changes. If the command mvn -D enable-ci install site locally command doesn't succeed, your change will not pass CI.

I fixed the instructions: you just need to do mvn clean to make sure the formatter does all the needed formatting.

@bitwiseman Gotcha
Could you provide an estimate on when there will be a new release containing this change?
Thanks!


500: INTERNAL_SERVER_ERROR On Vercel

Hey! Thanks for making this! Although I will deploy it on my own, but just wanted to let you know that the link to the website is down with the following error message:

ah! thanks for the report!  does it work when deployed to your own deployment?


Release Google.Cloud.AlloyDb.V1Beta version 1.0.0-beta02

Changes in this release:
Bug fixes

Deprecated SSL modes SSL_MODE_ALLOW, SSL_MODE_REQUIRE, SSL_MODE_VERIFY_CA (commit a839416)

New features

Added ClusterView supporting more granular view of continuous backups (commit a839416)
Added new SSL modes ALLOW_UNENCRYPTED_AND_ENCRYPTED, ENCRYPTED_ONLY (commit a839416)
Added users API (commit a839416)
Added fault injection API (commit a839416)
Added instance update policy (commit a839416)
Added cluster network config (commit a839416)

Documentation improvements

Minor formatting in description of AvailabilityType (commit 460b883)

Created release for Google.Cloud.AlloyDb.V1Beta-1.0.0-beta02


Tweak tasklist AB logic

We want to display headers if the page is in the AB test whether or not the
sidebar is shown.
Also, we only want to show the sidebar if the page is in the test cohort.
Also, added some rudimentary tests to make sure we're calling the ab test component when we're supposed to.

Thanks @sihugh 👍


dev:  remove panic in starknet pallet

Current Behavior
We are panicking if the transactions and transaction_hashes mappings don't have the same length.
Expected Behavior
We should investigate a cleaner way of handling this potential mistake rather than just panicking which is a very bad practice especially production-wise.
see here

So if this one panics then there's definitely something wrong in the logic and our block commitment is highly likely going to be incorrect. So if we let the chain continue then there's a good chance we would need to rollback later on as we won't be able to prove it. On the other hand, if we halt, then the UX will be bad for sure but we won't have to go through rollbacks. Besides, I think Starknet would have some agreed on method to analyze this decision => should we halt or continue. I am sure we would need to use this framework to analyse things at other places as well. wdyt?

So if this one panics then there's definitely something wrong in the logic and our block commitment is highly likely going to be incorrect. So if we let the chain continue then there's a good chance we would need to rollback later on as we won't be able to prove it. On the other hand, if we halt, then the UX will be bad for sure but we won't have to go through rollbacks. So I think we should halt (via panic or some other way). Besides, I think Starknet would have some agreed on method to analyze this decision => should we halt or continue. I am sure we would need to use this framework to analyse things at other places as well. wdyt?

Starknet doesn't really have a clear framework for this. Agreed with @tdelabro that it's fine to stop block production in such cases. We might just want to test that clearly to make sure the chain can be restarted properly.
Also do we still want to work on importing genesis state from pathfinder snapshot ? If so I can create the issue cc @tdelabro @apoorvsadana

Also do we still want to work on importing genesis state from pathfinder snapshot ?

That would be a really really good feature. It would make a lot of testing so much easier.


Proposal - Allow auth "type" to be sent in request body.

It's impossible to accurately document in Swagger different request schemas when mixing query parameters, ?type=local for instance. I also personally find it awkward because you aren't querying a resource it's the same resource you are just creating a different type.
{
  "type": "LocalAuth",
  "email": "xxx",
  "password": "xxx"
}

Note that the type needs to be well named as you are specifying a schema name in Swagger. See https://github.com/swagger-api/swagger-spec/blob/master/versions/2.0.md#models-with-polymorphism-support.
Another alternative would be to expose separate routes per auth provider which I'd be okay with too.

We can expose separate routes per provider. I'm guessing this would be the
cleaner approach - correct me if I'm wrong.
Then ?type=xxx could be optional for backwards compat
Would that be acceptable?
On Thu, 8 Oct 2015 at 21:07, Chris LeBlanc<EMAIL_ADDRESS>wrote:

It's impossible to accurately document in Swagger difference schemas when
mixing query parameters (?type=local) for instance. I also personally find
it awkward because you aren't querying a resource it's the same resource
you are just creating a different type.
{
"type": "LocalAuth",
"email": "xxx",
"password": "xxx"
}
Note that the type needs to be well named as you are specifying a schema
name in Swagger. See
https://github.com/swagger-api/swagger-spec/blob/master/versions/2.0.md#models-with-polymorphism-support
.
Another alternative would be to expose separate routes per auth provider
which I'd be okay with too.
—
Reply to this email directly or view it on GitHub
https://github.com/waterlock/waterlock/issues/108.

Yep, works for me.

Actually let me think about this further. I feel like /auth/login should accept user/password or auth_token and in both instances return the jwt. Currently it awkwardly is returning a url which isn't even relevant to mobile apps and honestly not even all that useful in web because, at least with facebook, I think it can just be generated clientside the extra request.

It needs to know what the type is so that it can validate the auth code
against the provider.
What do you mean it's returning a URL? You should be receiving a token as a
result of logging in?
On Thu, 8 Oct 2015 at 21:22, Chris LeBlanc<EMAIL_ADDRESS>wrote:

Actually let me think about this further. I feel like /auth/login should
accept user/password or auth_token and in both instances return the jwt.
Currently it awkwardly is returning a url which isn't even relevant to
mobile apps and honestly not even all that useful in web because, at least
with facebook, I think it can just be generated clientside the extra
request.
—
Reply to this email directly or view it on GitHub
https://github.com/waterlock/waterlock/issues/108#issuecomment-146675154
.


Upgrade to 1.4.3

It looks so trivial: let me know if I missed something.

LGTM

@hmalphettes This should maybe be documented better, but when this is merged someone must create a PR to the official docker image registry for io.js in order for this to be available on the Docker Hub.

LGTM, thanks @hmalphettes. I held off on this, because the changes in 1.4.2 and 1.4.3 are Windows-specific (don't matter in Docker), plus an NPM upgrade (which only matters in interactive usage).

@Starefossen it is documented here, in the wiki: https://github.com/iojs/docker-iojs/wiki/Updating-versions
@hmalphettes can you please also send the PR wich @Starefossen mentioned (step 5 from the instructions in the wiki)

Maybe hold of that PR to the official docker image until 1.5 has shipped?!
Ref. iojs/io.js#1060

@pesho @Starefossen thanks for kindly pointing to the documentation.
I'll try this with 1.5 that has just shipped.

@hmalphettes already done - https://github.com/iojs/docker-iojs/pull/35 :) (but I don't think the Docker folks will merge the Hub PR before Monday)

@pesho cheers!


Xamarin Studio version

Could you please take a look at this https://github.com/MvvmCross/MvvmCross/issues/1088#issuecomment-185421939

I replied on the MvvmCross issue. Any problems let me know.


Crashes if _input or _unhandled_input is present

On my machine playing scene crashes if i have a js class with _input or _unhandled_input.
Reproduction: create empty project, create some root node, attach new ecma script to it and add _input(event) {} or _unhandled_input(even) {}. Play the scene and move a mouse few seconds.
Without those methods everything is working, as soon as i add them, crashes  appear. On crashes i can see messages about malloc in terminal with godot editor process.
Is this reproducible on your machine?

i can reproduce this using
export default class Example extends godot.KinematicBody {
  _input() {}
}

Im using

Godot e707b712e8
ECMAScript ec69d88
on mac os 10.15.4 (19E266)

I can't reproduce this problem
Test case1
_input() {
    console.log(arguments);
}

Output
[object Arguments]

Test case2
_input(event) {
    console.log(event);
}

_unhandled_input(event) {
    console.log(event);
}

Output
[InputEventKey:1620]
[InputEventKey:1620]

I am on mac as domis is. Maybe it is a platform specific issue?

Works now.

The issue is happening to me on v0.0.17. The same steps to reproduce as  before:
export default class Example extends godot.Node{
  _input() {}
}

Tested on both windows and linux builds. On windows 11 it crashes with no logs (even in verbose mode), on latest fedora - the IDE just freeze.


Oscar Ching - Sprint 6: Engineering Portfolio/Notebook + Mounting + Movement Coding

2024/12/9

Engineering Portfolio
Attended 15m Team Meeting
Assisted Ella with the Autonomous Section

2024/12/10

Assisted with movement coding
Worked on the presentation script

2024/12/11

Filed intake
Recorded Dremeling the intake
Worked on the Notebook

2024/12/12

Worked on Portfolio (Drivetrain)
Helped shave the 3D-printed intake

2024/12/13

Began writing the YorkU trip section of the engineering portfolio


Merge pull request #2 from mybatis/master

update...

11111

Coverage decreased (-0.009%) to 87.273% when pulling 5da8e88de0d29e75693b15386422336111e08b29 on peterchenhdu:master into e7c67ce0b449587a536897d8b4d6aad9f3b824a2 on mybatis:master.


Contributors Section

On the homepage, it would be nice to have a contributors section that includes an array of all contributors with a reusable component for contributors.
To add to this thought, it might be best if we eliminate the 'admins' section so there is no feel for any type of 'ranking system'. Then we will have one accumulative group section that is seen as together as a team and organization, and not dictated by any one person or select group.

I think it's a good idea to keep some sort of admin section or otherwise indicate admins within a contributor list. It doesn't have to be flashy or call a lot of attention to admins but I think making them easy to find is important in case anyone needs to find someone to contact directly. Our email address is useful for that of course, but I can think of scenarios where someone might want to speak to an admin personally or through a direct message.


NPM angular2

Please update npm package https://www.npmjs.com/package/ui-router-ng2
At  this moment i can't install a latest version of ui-router for using with angular2 release

beta.3 should be ready in the next couple of days

beta.3 should be ready in the next couple of days

Many thanks for your quick reply. We use ui-router in production and it is very important to us.


Godot Asset Library

Hey, I'm sorry, but still can't figure out why I can't approve your latest changes to the Godot Asset Lib. I'll try to get in touch with other members of the team to see if we can fix the issue.

Ok, thanks!

It's not that sophisticated as far as I know 😅
We are still trying to figure out why it fails.

We were able to approve it, but there wan an empty field (Type) on the new video submitted.

Are you using an automated system to publish the updates or our website?

No problem! Closing this now, hope we don't have the issue again in the future. Cheers!


Modify code to call Microsoft's Azure OpenAI chat completion instead of OpenAI?

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [X] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Mention any other details that might be useful
Can someone shed some light on this for me.  On Microsoft's page: https://learn.microsoft.com/en-us/legal/cognitive-services/openai/data-privacy:

Your fine-tuned Azure OpenAI models are available exclusively for your use. The Azure OpenAI Service is fully controlled by Microsoft; Microsoft hosts the OpenAI models in Microsoft’s Azure environment and the Service does NOT interact with any services operated by OpenAI (e.g. ChatGPT, or the OpenAI API).

Is the above what's occurring in the Azure OPenAI ChatPlayground?  What does Microsoft mean by "Service does NOT interact with any services operated ....   (or the OpenAI API)"?
Can we be certain that Microsoft's Azure OpenAI ChatPlayground does not call OpenAI's url (https://api.openai.com/v1)?  I wasn't able to capture the traffic from Microsoft Azure Open AI Chatplayground.  Still researching; if anyone knows, that would help also.
In this demo's codebase, it is referencing OpenAI library and calls are made to OpenAI url .  If the Azure OpenAI Playground is calling it's own implementation of ChatCompletion, can I call Microsoft's version instead?  I could easily port this code to an Azure Function, if I need to be in the Azure environment.
In summary,  can I call Microsoft's ChatCompletion endpoint instead of OpenAI's.
Thanks in advance.

Thanks! We'll be in touch soon.

@aymiee This sample uses Azure's OpenAI. You can see it configured here, where it sets api_type:
https://github.com/Azure-Samples/azure-search-openai-demo/blob/0f5fb2668c52c56d9961f5dde1f2bb831c6f62ef/app/backend/app.py#L43
This whole sample is designed to be deployed entirely on Azure resources. It uses App Service as the host, for both the frontend and backend.

Thank you for the clarification. I was reviewing the backend code at app/backend/approaches/chatreadretrieveread.py :

class ChatReadRetrieveReadApproach(Approach):
"""
Simple retrieve-then-read implementation, using the Cognitive Search and OpenAI APIs directly. It first retrieves
top documents from search, then constructs a prompt with them, and then uses OpenAI to generate an completion
(answer) with that prompt.
"""

and

#STEP 3: Generate a contextual and content specific answer using the search results and chat history
completion = openai.Completion.create(
engine=self.chatgpt_deployment,
prompt=prompt,
temperature=overrides.get("temperature") or 0.7,
max_tokens=1024,
n=1,
stop=["<|im_end|>", "<|im_start|>"])

The comments: "..using the Cognitive Search and OpenAI APIs directly..."  and the use "openai.Completion.create"  startled me a bit.

Sorry for the confusion! It would actually be possible to use the non-Azure OpenAI API with this sample with a little configuration change, and there's a PR for adding that as an option. The important aspect is openapi.api_type - always check that if you're not sure. Plus, the keys/credentials aren't shared across the two, so you wouldnt be able to use OpenAI's version with an Azure key.


docs: clarify detach_on_error

Change description
What problem does this solve?
Issue number: (if applicable)
Example usage
Additional details and screenshots
Checklist

[ ] I have added unit tests to cover my changes.
[ ] I have added documentation to cover my changes.
[ ] My changes have passed unit tests and have been tested E2E in an example project.

In my opinion the wording in the previous documentation makes you think that true will let the MFA to continue to be invoked after a fail.

coverage: 76.931% (-0.2%) from 77.131%
when pulling 15a39bbb5822cf6467c7a956a9bd6a0c6f644620 on kivra-mikgra:patch-1
into 6f979435d1645d65e2a480e07fefb06328355357 on akoutmos:master.


Gillespie started major rebase

A lot of things have changed, including some API.
It all started with implementing a proper Gillespie wrapper around KineticMonteCarlo, which included developing some nice event handler. After talking with George, I remove all modular substructure and cleaned up API.
@fmikulasch please check if the current API works for you and can be integrated into your workflow.

The last commit seems to lack some of the routines. Can you fix this? Then I will go over it.

Which routines are you missing.  I rather cleaned the package and changed
the API, so routines may have a new name ...
For the inhomogeneous pp for example you should call next(Inhomogeneous
Poisson(), rng, rates,...)
On Tue, Mar 3, 2020, 13:25 fmikulasch<EMAIL_ADDRESS>wrote:

The last commit seems to lack some of the routines. Can you fix this? Then
I will go over it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zierenberg/MonteCarloX.jl/pull/33?email_source=notifications&email_token=AGBMN5ZNHQL6KTALX3Q62JTRFTZK3A5CNFSM4K766MJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENTJEWA#issuecomment-593924696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGBMN52ADKHHQ3MXKZC4AR3RFTZK3ANCNFSM4K766MJQ
.

Sorry I wasn't clear. It seems the last commit removed almost all files in src/ or am I seeing this wrongly?

Sorry, I renamed files to match with typical package convention in julia and ... forgot to add the new files to git. Hence local tests passed and I did not notice. Now, you should be able to see them.

Codecov Report

Merging #33 into master will decrease coverage by 10.92%.
The diff coverage is 71.29%.

@@             Coverage Diff             @@
##           master      #33       +/-   ##
===========================================
- Coverage   80.21%   69.29%   -10.93%
===========================================
  Files           9        9
  Lines         187      254       +67
===========================================
+ Hits          150      176       +26
- Misses         37       78       +41

Impacted Files
Coverage Δ

src/MonteCarloX.jl
100% <ø> (+50%)
:arrow_up:

src/gillespie.jl
0% <0%> (ø)

src/reweighting.jl
100% <100%> (ø)

src/cluster_wolff.jl
100% <100%> (ø)

src/kinetic_monte_carlo.jl
39.21% <39.21%> (ø)

src/importance_sampling.jl
72% <72%> (ø)

src/poisson_process.jl
45.16% <80%> (ø)

src/event_handler.jl
81.66% <81.66%> (ø)

src/utils.jl
93.75% <93.75%> (ø)

... and 5 more

Continue to review full report at Codecov.

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

Hi, sorry took me a while to get to it. I tried to get the Gillespie algorithm running but I am unsure how it is intended to be used. Is the evolution to be stored through the event handler? If so I find it a bit inconvenient to have to define my own event handler before I can use it. I think the user should have more of a "plug and play" experience. Could you give me an example of how you think it should be used?

Hi,
there is a set of event handlers implemented. I have a running program to
show how these can be used. This will be added to examples at some point.
If you do not want to use this feature, you can always go back to kineticMC
(which is the proper algorithm where you can pass a list of weights). The
Gillespie wrapper was (naming convention wise) an artifact from me not
knowing which me to give the child. This we can still debate.
However, right now the question was if all the changes I did are still okay
with YOUR usage of the framework ;). Namely the Inhomogeneous poisson
process.
Cheers,
On Mon, Mar 9, 2020, 22:43 fmikulasch<EMAIL_ADDRESS>wrote:

Hi, sorry took me a while to get to it. I tried to get the Gillespie
algorithm running but I am unsure how it is intended to be used. Is the
evolution to be stored through the event handler? If so I find it a bit
inconvenient to have to define my own event handler before I can use it. I
think the user should have more of a "plug and play" experience. Could you
give me an example of how you think it should be used?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zierenberg/MonteCarloX.jl/pull/33?email_source=notifications&email_token=AGBMN56DJ5LFA2R6N3D6MHTRGVPG5A5CNFSM4K766MJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOJFV7A#issuecomment-596794108,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGBMN55ASYL5W63KDUINEGDRGVPG5ANCNFSM4K766MJQ
.

Suggestion:
I move the current Gillespie advance!() function to kinetic_monte_carlo.jl (because it belongs there) and we write an additional wrapper for Gillespie, which essentially implements the original context of Gillespie (namely for chemical reactions) to be consistent with the wikipedia entry stating:

Gillespie developed two different, but equivalent formulations; the direct method and the first reaction method. Below is a summary of the steps to run the algorithm (mathematics omitted):

Initialization: Initialize the number of molecules in the system, reaction constants, and random number generators.
Monte Carlo step: Generate random numbers to determine the next reaction to occur as well as the time interval. The probability of a given reaction to be chosen is proportional to the number of substrate molecules, the time interval is exponentially distributed with mean 1 / R T O T {\displaystyle 1/R_{\mathrm {TOT} }} {\displaystyle 1/R_{\mathrm {TOT} }}.

Update: Increase the time by the randomly generated time in Step 2. Update the molecule count based on the reaction that occurred.
Iterate: Go back to Step 2 unless the number of reactants is zero or the simulation time has been exceeded.
see more here

We could then start with

initialize (which returns an event handler)

advance (which essentially does 2-4)

step (which is 2)

I found that it is difficult to debug the kineticMC when you mistakenly hand it negative or zero rates - it doesn't produce an error and the next sampled timestep is infinty or negative. We could think about sacrificing performance and warn the user?
Otherwise I approve :)

I found that it is difficult to debug the kineticMC when you mistakenly hand it negative or zero rates - it doesn't produce an error and the next sampled timestep is infinty or negative. We could think about sacrificing performance and warn the user?
Otherwise I approve :)
Edit: I think we could also merge some of the search functions used to select events...

Good point. I would argue that negative timestep would be warning enough :). Maybe this can be checked without much effort via
if not(sum(rates)>0)
  ...
end

The problem is rather: "where would we set the bound?". This could already fail for sum(rates) about 1e-10 if all rates are zero but the sum has cumulated floating point errors (actually thinking of which ... is this a thing we should look out for? In terms that every so and so often the rates.sum should be recomputed....)

@fmikulasch : I think for now, we could merge this into master to keep on working on details in different (smaller) development steps. What do you think?
I would be curious to see if you can merge the pull request.


I messed up, rip

Your OS:
App version:

Note:

If you're reporting a bug, please describe how to reproduce it.
If you're suggesting a new feature, please detailed spec of it.

I was putting in my account info and didn't click the README first. so I didn't know I was supposed to put anything in default_page. and I exited out and of course the app didn't work after that, so I tried deleting and redownloading to reset it, but the file I edited is deep down in my computer and I cannot find it.

You should watch your language before anything else


1.0.0 to dev

Coverage increased (+18.1%) to 87.586% when pulling 2003dbc0e6b2cc6dc25f1d8c3eb8b3d3d8055423 on master into c74c719f67d0c2d5b3a8867718960aea249516e2 on dev.


Corsair and Asus Support via RGB.NET

Experimental Corsair and Asus Support using RGB.NET. Using RGB.NET from now on will be as easy to add new devices as adding 3 lines of codes. Technically, all devices supported by RGB.NET can be supported by Aurora now.
Please test this PR for different corsair and asus peripherals such as Light Strips, RAM, Cooler etc.

How do I test this?

You can go to Appveyor link: https://ci.appveyor.com/project/Gurjot95/aurora/builds/24973101/artifacts

Works for me with my asus rog maximus x hero, corsair vengence rgb ram, corsair h100i pro watercooler, asus 2080ti strix.
It takes the color from the mouse right? I assume for more ways to configure the rewrite of aurora is needed.

cant get into share modus // or am i just blind ? can u tell me where i can controll my MM800 via Aurora and my LL140 via iCue? sorry for that dumb question :)

please can you please help me? i got a MM800 mousepad and 2x LL140 fans. i want to control the mousepad via aurora constantly and the fans by icue. is that possible?

Hey, I need to add shared mode option in device manager but it should be possible in future builds.

Experimental Corsair and Asus Support using RGB.NET. Using RGB.NET from now on will be as easy to add new devices as adding 3 lines of codes. Technically, all devices supported by RGB.NET can be supported by Aurora now.
Please test this PR for different corsair and asus peripherals such as Light Strips, RAM, Cooler etc.

Where  do you find the keyboard layout files?

Works for me except for the new rgb pump and res combo (XD5) It is hooked up to a command pro hub and doesn't seem to work :C

Yay, 124 made all the lights work again!


proposal: add an optional to load edge-ids (foreign-keys) in query builder

Maybe this is a uniqueness of relational DB's. But edge ID's are a part of a row (Assuming the foreign key is on the side of the rows table).
However this has big benefits:

You can generally check if a relation exists
I can use the relation ID for various things (where the ID only is enough, links/actions/other ID only deterministic things like loading an image, and the ID is a hash)

Is it possible somehow?
When not, maybe it would be nice to make this accessible. If graph DB's don't support this, maybe behind a function. Mostly those are 5 int/varchar fields to also select in a simple query. I really missing this one.

Hey @aight8 and thanks for proposing this issue.
It was actually raised before in the issue tracker a few times, so I think we need to address this.
I have a few ideas how such API should look like (and I'll write them later), but you're welcome to write here your suggestions as well.

huck, sry I posted this issue by mistake... I wrote it as part of my other "Feedback" issue.

I've run into this while moving an existing codebase to ent. Here are some ideas for how this could be solved.
1. Exposing the foreign keys
I believe being able to access the foreign key values is important so I would suggest exporting the foreign key values
type Pet struct {
	config `json:"-"`
	// ID of the ent.
	ID int `json:"id,omitempty"`
	// Name holds the value of the "name" field.
	Name string `json:"name,omitempty"`
	// Edges holds the relations/edges for other nodes in the graph.
	// The values are being populated by the PetQuery when eager-loading is set.
	Edges     PetEdges `json:"edges"`
	UserPets *int
	UserTeam *int
}

2. Match nilability in struct
As far as I understand, all FKs in the generated struct are pointers as they are nil until the edges are eager-loaded. To make 1. actually usable I believe it should match the edge'se required state.
3. Expose predicates for querying foreign keys

and 2. are an approach to solve the read path I believe. The only way I've found to do the equivalent of:

SELECT * FROM `pets` WHERE user_pets = ?

is as follows:
client.Pet.Query().Where(func(selector *sql.Selector) {
		selector.Where(sql.EQ(selector.C(pet.OwnerColumn), ownerID))
}).All(ctx)

I think there is value in adding generated predicates so you can write the following:
client.Pet.Query().Where(pet.HasOwnerWithID(ownerID)).All(ctx)

As with @aight8 I haven't looked into how ent handles Gremlin so I'm not sure if this suggestion has any issues with it.

I've run into this while moving an existing codebase to ent. Here are some ideas for how this could be solved.
1. Exposing the foreign keys
I believe being able to access the foreign key values is important so I would suggest exporting the foreign key values
type Pet struct {
	config `json:"-"`
	// ID of the ent.
	ID int `json:"id,omitempty"`
	// Name holds the value of the "name" field.
	Name string `json:"name,omitempty"`
	// Edges holds the relations/edges for other nodes in the graph.
	// The values are being populated by the PetQuery when eager-loading is set.
	Edges     PetEdges `json:"edges"`
	UserPets *int
	UserTeam *int
}

2. Match nilability in struct
As far as I understand, all FKs in the generated struct are pointers as they are nil until the edges are eager-loaded. To make 1. actually usable I believe it should match the edge's required state.
3. Expose predicates for querying foreign keys

and 2. are an approach to solve the read path I believe. The only way I've found to do the equivalent of:

SELECT * FROM `pets` WHERE user_pets = ?

is as follows:
client.Pet.Query().Where(func(selector *sql.Selector) {
		selector.Where(sql.EQ(selector.C(pet.OwnerColumn), ownerID))
}).All(ctx)

I think there is value in adding generated predicates so you can write the following:
client.Pet.Query().Where(pet.HasOwnerWithID(ownerID)).All(ctx)

As with @aight8 I haven't looked into how ent handles Gremlin so I'm not sure if this suggestion has any issues with it.

Hey @errorhandler and thanks for writing this - I missed it somehow.

As with @aight8 I haven't looked into how ent handles Gremlin so I'm not sure if this suggestion has any issues with it.

This shouldn't be a problem, because ent generates codegen based on the storage-type.
These options can be added using feature flags. I can start pushing this in the upcoming days, but if you want to contribute these changes, let me know and I can guide you.

Great to hear @a8m! I've done 3 with a template for my use case so I'm happy to expand this work and contribute. I'll have a go making what I believe are the required changes and follow up with any questions I may have.

Closing with #1213
See https://entgo.io/docs/schema-edges/#edge-field for full docs.
Thanks @errorhandler and @aight8 for pushing this.


Bootstrap documentation website

Scope
The scope of this PR is to bootstrap a documentation website.

[x] Readme how to run/use the Jekyll website
[x] GitHub action to deploy automatically upon merge to master
[x] Has a homepage (identical to README.md from GitHub)

Follow-ups
We exclude auto-generating the docs, which will be done in a follow-up PR.

[ ] Lists all BugPattern checks

[ ] Displays summary, description, severity, and tag (currently dummy values)
[ ] Displays source code link
[ ] In-line replacement examples

[ ] Lists all Refaster templates (grouped per collection)

[ ] In-line replacement examples

[ ] Support additional Markdown docs

Preview

I added two things, which I believe were important to do for the website

Auto-generate a sitemap.xml / robots.txt for SEO and scaping
Introduce html-proofer to validate all our URLs (also used in https://github.com/PicnicSupermarket/picnicsupermarket.github.io)

The latter actually caught quite some invalid urls in our readme, as relative paths won't work when hosting our site.
FYI @rickie as this was added after your review

Open question @Stephan202 (?)
Should we build on all branches, and only deploy on the default branch? Would allow verifying things work (e.g. html proofer and page generation).

Open question @Stephan202 (?)
Should we build on all branches, and only deploy on the default branch? Would allow verifying things work (e.g. html proofer and page generation).

I took the liberty of adding this, by setting an if on the deploy step, and running this also on PR builds.

Open question @Stephan202 (?)
Should we build on all branches, and only deploy on the default branch? Would allow verifying things work (e.g. html proofer and page generation).

I took the liberty of adding this, by setting an if on the deploy step, and running this also on PR builds.

I guess this answers my question above ;). Yes, more feedback is better, and any issues we can catch before merging are a plus :)

It seems that github-pages / jekyll-build-pages automatically ships with jekyll-remote-theme, which was introduce to use themes on GitHub that did not have a nice gem provided https://www.siteleaf.com/blog/remote-themes/
Whilst specifying the gem works locally, the build - however - doesn't actually run bundle install so it fails the build when we're using a local theme.
That leads two options:

Setup ruby, bundler, and run the install manually
Pin the remote theme for a reproducable build

I've now done the latter, as that drastically simplifies the build setup. As we're pinning the version and the latest is an RC, we should keep an eye out for the final v4 release.

Discussed offline with @Stephan202.
Pinning the version is okay for now, but we should investigate a 100% reproducible build s.t. the setup in GHA and locally are 1:1 the same. This is not just for the theme, but for everything (jekyll, html-proofer, etc.)

Added a commit.
Caught up on all the discussions, changes LGTM.
@maxsumrall Do you want to finish your review still 😄 ?

Nope, just had the comment, but seems like a lot of effort to get that link to work.

Yep, favicons are a mess 🙄 Just look at the list of "Why so many files?" at https://realfavicongenerator.net/faq.
Thanks for the last tweaks @Stephan202 🙏

ps. before merging I verified that generate-docs.sh also runs as expected on a MacOS machine (as it was authored on a Linux machine).


feat: single ec-code annotation per reaction

Description of the issue:
Many reactions do not have any ec-code, or are annotated with multiple ec-codes. Instead, each reaction should be annotated with one ec-code, and if not full ec-code can be defined (with 4 sets of digits), then wild-cards can be given.
This might involve manual curation, although parsing gene associations through Uniprot might be helpful.
These single ec-codes can then be used when constructing GECKO models.
Expected feature/value/output:
model.eccodes(:) should give a single ec-code entry for each reaction.
Current feature/value/output:
>> model.eccodes([1,3,6,17,22]) % Some random example reactions)

ans =

  5×1 cell array

    {'<IP_ADDRESS>;1.1.99.-'                              }
    {'<IP_ADDRESS>'                                       }
    {0×0 char                                        }
    {'1.14.13.-;<IP_ADDRESS>;<IP_ADDRESS>;<IP_ADDRESS>;2.7.-.-'}
    {0×0 char                                        }

I hereby confirm that I have:

[x] Tested my code with all requirements for running the model
[x] Done this analysis in the main branch of the repository
[x] Checked that a similar issue does not exist already
[ ] If needed, asked first in the Gitter chat room about the issue

This is good idea. It should be also noted that the reaction is mainly found based on sequence, not by EC number as one EC number could be mapped mutiple reactions(in some cases). Currently, EC number for each protein can be found from UniProt or SGD. Previously I found the EC number annotation from different database is not the same. We can still make this step automatic by refererring some standard rxn database like Rhea, Metnetx or ModelSeed.

But by definition, there should really be only one EC number per reaction, although perhaps with some wildcards if the exact substrate or cofactor has not been given a specific EC number (in that case, something like EC1.4.2.-). But indeed, we can automatically pull these from databases, probably MetaNetX will be useful. Manual curation will be required to resolve when we find multiple EC numbers for the same reaction.

I fully support/encourage automatic mapping of EC numbers. A good way to map to MNX would be via Rhea IDs; however, these are missing from the model annotation, so then I would suggest using the BridgeDB API.

Yes.  I just checked EC number for each reaction in yeast-GEM. Most reactions have one unique EC number. About 490 reactions have multiple EC number. It will be wonderful to have some automatic way to curate these EC number.

Indeed, there are many that are already fine.
I'm not 100% certain how automated curation would work, because many gene/protein based approaches (e.g. BridgeDB, Uniprot) are not ideal as it is not uncommon for enzymes to have been assigned multiple ec-numbers (due to them being multifunctional enzymes), or because they are part of a complex where subunits perform dedicated functions. GECKO is however doing this based on Uniprot and KEGG.
But probably a mixture of using GECKO's function, supplemented with looking at other annotations. And most likely we'd manually have to define EC numbers with wild-cards for those reactions that have not specific EC number (we want the EC-numbers to exactly match the reaction: if it doesn't match, use a wild-card, until all reactions (except exchange and non-active transport) have assigned (partial) EC-number).
This all does not have very high priority, but would be good to address.


Sync time in Docker container with host

Resolves #177

@dr0i Deployed to q1, please have a look.


Why two subgraphs instead of one whole graph

In the README, it says "Here we extract two pure nn models from the whole computation graph---pfe and rpn, this is to make it easier for trt to optimize its inference engines, and we use cuda to connect these nn engines."
Is there any repo/docu/link/tutorial that supports this argument? i.e., why is it easier for trt to optimize its inference engines? (one onnx vs two onnxs)

TensorRT is developed to optimize nn inference, the connection part between pfe & rpn (voxel assigning) involves no nn computing, I don't think TRT would optimize that part of computation.
Since TensorRT is a like black box, I'd rather believe you should take over the non-nn computation part as much as possible, because you can control something like memory allocation or threads allocation and so on.

That is why I spilled  into two part of nn graphs, and connected them using cuda.

But 2 onnx will generate more mem cost, like pfe output ,intermediates and rpn input  cost more resources.


Question: How to use Emotiv on Max/MSP with NODE RED ?

@halybang @nguoithichkhampha @patdotme @jdpigeon @NguyenDuyToan @pstechfreak @prabalv @bchaperon
Hi Everybody !
It has been several weeks since using Emotiv with Max / MSP using the OSC protocol.
That's why I use NODE RED, but today Max/MSP just gets a "bang", instead of receiving digital data from the Emotiv headset!
I would like to receive digital data !
Do you have any idea how to solve this problem?
Have a nice day
Ed

up

Hey @Edvoy, you don't need to wire profile name with Performance Metric node since it doesn't associate with a training profile. Connect it directly to EMOTIV node, and add debug node to the output of Performance Metric node to debug, see any data was coming out from it?


Punjabi translation

See #173 for details

It is not actual no longer, the translations will be needed for v2.0.0.


Vulnerability roundup 39 (master)

Scanned nixos/release-combined.nix @ 911a6da. Filtered out previously reported CVEs. May contain false positives.
This is a fresh start - CVEs filtered out for 18.03 are included here. Please look especially for those packages that had no upstream fix a while ago.
bazaar-2.7.0 (search, files)

[x] CVE-2017-14176

binutils-2.30 (search, files)

[ ] CVE-2018-6543
[ ] CVE-2018-6759
[ ] CVE-2018-6872
[ ] CVE-2018-7208
[ ] CVE-2018-7568
[ ] CVE-2018-7569
[ ] CVE-2018-7570
[ ] CVE-2018-7642
[ ] CVE-2018-7643

@Ericson2314
coreutils-8.29 (search, files)

[x] CVE-2017-18018

@edolstra
cyrus-sasl-2.1.26 (search, files)

[x] CVE-2013-4122

exiv2-0.26 (search, files)

[ ] CVE-2017-1000126
[ ] CVE-2017-1000127
[ ] CVE-2017-1000128
[ ] CVE-2017-11336
[ ] CVE-2017-11337
[ ] CVE-2017-11338
[ ] CVE-2017-11339
[ ] CVE-2017-11340
[ ] CVE-2017-11553
[ ] CVE-2017-11591
[ ] CVE-2017-11592
[ ] CVE-2017-11683
[ ] CVE-2017-12955
[ ] CVE-2017-12956
[ ] CVE-2017-12957
[ ] CVE-2017-17669
[ ] CVE-2017-18005
[ ] CVE-2018-4868
[ ] CVE-2018-5772

ffmpeg-3.4.2 (search, files)

[x] CVE-2018-6912
[x] CVE-2018-7557

@codyopel @Fuuzetsu
gstreamer-0.10.36 (search, files)

[ ] CVE-2016-9447

jasper-2.0.14 (search, files)

[x] CVE-2018-9055

@pSub
jquery-ui-1.11.4 (search, files)

[ ] CVE-2016-7103

libarchive-3.3.2 (search, files)

[ ] CVE-2017-14501
[ ] CVE-2017-14503

@jamescun
libcroco-0.6.12 (search, files)

[ ] CVE-2017-8834
[ ] CVE-2017-8871

libgcrypt-1.8.2 (search, files)

[ ] CVE-2018-6829

@wkennington @vrthra
libid3tag-0.15.1b (search, files)

[ ] CVE-2017-11550
[ ] CVE-2017-11551

@Fuuzetsu
libimobiledevice-1.2.0 (search, files)

[x] CVE-2016-5104

libmad-0.15.1b (search, files)

[ ] CVE-2018-7263

@lovek323
libsndfile-1.0.28 (search, files)

[ ] CVE-2017-14245
[ ] CVE-2017-14246

@lovek323
libtiff-4.0.9 (search, files)

[ ] CVE-2017-17095
[ ] CVE-2018-5784
[ ] CVE-2018-7456
[ ] CVE-2018-8905

libusbmuxd-1.0.10 (search, files)

[x] CVE-2016-5104

net-snmp-5.7.3 (search, files)

[ ] CVE-2014-2285

@wkennington
openjpeg-2.3.0 (search, files)

[ ] CVE-2017-17479
[x] CVE-2017-17480
[ ] CVE-2018-5727
[ ] CVE-2018-5785
[ ] CVE-2018-6616
[x] CVE-2018-7648

@codyopel
openldap-2.4.45 (search, files)

[ ] CVE-2017-14159
[ ] CVE-2017-17740

@lovek323
openvpn-2.4.5 (search, files)

[ ] CVE-2018-7544

@viric
patch-2.7.6 (search, files)

[x] CVE-2018-6951
[x] CVE-2018-6952

pcre-8.41 (search, files)

[ ] CVE-2017-11164

python-2.7.14 (search, files)

[x] CVE-2018-1000030

@FRidh
rsync-3.1.3 (search, files)

[x] CVE-2017-16548

shadow-4.5 (search, files)

[ ] CVE-2018-7169

sharutils-4.15.2 (search, files)

[ ] CVE-2018-1000097

@ndowens
sqlite-3.22.0 (search, files)

[x] CVE-2018-8740

@np
util-linux-2.31.1 (search, files)

[x] CVE-2018-7738

wildmidi-0.4.2 (search, files)

[x] CVE-2017-11661
[x] CVE-2017-11662
[x] CVE-2017-11663
[x] CVE-2017-11664

@bjornfor
wpa_supplicant-2.6 (search, files)

[x] CVE-2017-13077
[x] CVE-2017-13078
[x] CVE-2017-13079
[x] CVE-2017-13080
[x] CVE-2017-13081
[x] CVE-2017-13082
[x] CVE-2017-13084
[x] CVE-2017-13086
[x] CVE-2017-13087
[x] CVE-2017-13088

@MarcWeber @wkennington
Cc: @joepie91, @phanimahesh, @the-kenny, @7c6f434c, @k0001, @peterhoeg, @nh2, @LnL7, @grahamc, @adisbladis, @fpletz
Contact @ckauhaus for any questions.

CVE-2017-16548 (rsync) is fixed in our rsync release (latest one).

@ckauhaus I think you've mistakenly tagged me for libarchive. I do not own this package nor use NixOS 😄.

@jamescun oops sorry

CVE-2018-6952 (gnupatch) no patch are available (https://savannah.gnu.org/bugs/index.php?53133)

The wildmidi CVEs seem to have been fixed in the 0.4.2 release (https://github.com/Mindwerks/wildmidi/releases/tag/wildmidi-0.4.2), which is only missing in 17.09.

@dotlambda That's interesting - the CVEs list 0.4.2 as vulnerable. Probably they've got the version number wrong. Marking wildmidi as fixed on master. A backport to 17.09 would be sensible.

bazaar (CVE-2017-14176): 19bc90f91111f9d02e5e68fcdb4135913b9569eb

libgcrypt: no fix upstream yet; one proposed way is documentation-only fix (1).

cyrus_sasl vulnerability CVE-2013-4122 was fixed in 5d41dda02db3f538fb253056575c151192d4ce41. I've included the CVE in the patch name for vulnix auto-detection.

I've also included the CVE tag in the patch for CVE-2016-5104, libimobiledevice-1.2.0. (29e5da8e2fcb745826fdb33c9354413e0b46408f)

coreutils: my understanding of upstream ML messages and their commits is that they decided the risk is not really fixable and updated the documentation.  We could rush applying the doc updates like e.g. Fedora, but I don't think it's really worth the mass rebuilds now.

util-linux: fixed in staging by #37814

openjpeg: fixed in Openjpeg Patching CVE-2018-7648 #49262

This issue needs review. Many of the affected packages reported here are still present on master or 18.09.

CVE-2017-17480 fixed in a4700aa526c2de1e4dc234215fdee5bc3c8ea4e5 on master and b3aff3a0400a609e5c1415dc6e9b945948175454 for 18.09

18.09 is EOL


Illegal instruction (core dumped)

I am unable to use llama-node on a xeon E5-2690 v2, I get the error:
Illegal instruction (core dumped)

I am assuming it is because it doesn't support AVX2, as llama-node works on my i7 12700F. Is there any way to get it to work?

I am unable to use llama-node on a xeon E5-2690 v2, I get the error:
Illegal instruction (core dumped)

I am assuming it is because it doesn't support AVX2, as llama-node works on my i7 12700F. Is there any way to get it to work?

which nodejs version and backend you are using?

@itz-coffee I m not going to support disable AVX2 by default (since theres are only very old cpus dont support AVX2)
You can change the build.rs in llama-sys here
just follow the CMakeList.txt https://github.com/ggerganov/llama.cpp/blob/master/CMakeLists.txt and OFF the AVX2 support.
Also I would assume that without AVX2, you inference probably very slow.

This issue is definitely caused by AVX2 instructions. I may provider self build features on rust side in the future for easier compilation.


ZaupShop SDG.Unturned.Asset.Get_id Error need help!!

I can use /i  /v  all zaup plugin  except ZaupShop I cannot use /buy /cost /sell but can /shop add
It said    :   An error occured while executing buy [] System.MissingMethodException Method not found 'SDG.Unturned.Asset.get_id'.
at Rocket.Core.Commands.RocketCommandManager+RegisteredRocketCommand.Execute (IRocketPlayer caller, System.String[] command) [0x00000] in filename unknown0
at Rocket.Core.Commands.RocketCommandManager.Execute (IRocketPlayer player, System.String command) [0x00000] in filename unknown0

My rocket version is <IP_ADDRESS> (Lastest Version on website) and lastest version of ZaupShop plugin. My Uconomy plugin work fine,ZaupHomeCommand too. But i have only problem on ZaupShop plugin.please help me (Sorry for my bad english)

You are using an outdated version of ZaupShop.
Issue is not related to Rocket


proposal: round-out complex number operator support by adding ordering.

when i try to drop-in support for another built-in type, in this case from floatXX to complexXX, it fails due to lack of support for ordering complex numbers.
from reading around i see some mathematicians saying they have, and some say they don't have, this property, I’m not sure, but it seems to me that the real issue is that there are several, so its mathematically undefined, but that practically you could make a decision and just define what you were using. (if you need something else then use a library.)
and for me, practically and simply, you could compare their absolute squares;
ie   (x+yi) [<|>] (a+bi)   ===   xx+yy [<|>] aa+bb
(so ordered by the distance, in the complex plane, from origin.)
i think because complex numbers are built-in makes them having uniform operator support really desirable, so much so that having something is better than having a perfect answer, which, in this case, i think really just doesn't exist anyway.

Note that this was rejected in the past; see #9681 ("spec: Complex number types should support " > < <= >=" operators").

didn't see that, but yes its EXACTLY what i'm saying, seems to of been dismissed very quickly though.

The complex numbers are not ordered. True, one can define orderings on them, but that doesn't mean any one of those ordering should be built into the language.

@josharian said is all. There's nothing dismissive about that, it's simply a mathematical fact (that they don't have a natural ordering).
Any ordering imposed will depend on the specific use case and thus needs to be implemented explicitly. We will not add a default ordering for complex numbers to the language.

"complex numbers are not ordered. True, one can define orderings on them"
sure about that?
"ordering imposed will depend on the specific use case and thus needs to be implemented explicitly"
so if complex numbers were library(s) then this wouldn't be an issue.

https://en.wikipedia.org/wiki/Complex_number#Ordering
Because there is no natural linear ordering for complex numbers, Go doesn't define some (essentially arbitrary) ordering.
I don't know that there's anything else to say about this here. If you want a specific ordering, you will have to define it yourself.

OK, never-mind, complex numbers aren't really worth significant effort to support for me. i'll just drop them.


incorrect name mappings: last name index is used for unrelated entries

IntelliJ IDEA supports name mappings and currently typescript compiler produces incorrect name mappings (https://youtrack.jetbrains.com/issue/WEB-17717).
Typescript code:
class Foo {
    public b = 3;
    bar(param1){
        var a = 5;
        return a;
    }
}

new Foo().bar(1);

Debug output: https://gist.github.com/develar/598a984f27bb92526094
Project to get debug output: https://dl.dropboxusercontent.com/u/43511007/s/ts-name-mappings-bug.zip
IDEA sourcemap visualiser:

Is it known issue? Or was not yet investigated and pull request is welcome?

We have decided that we wont emit the names since the sourcemap spec is unclear of its use. Browser tools don't use it either. We would be waiting to hear about symbol translation and then get to this.

@sheetalkamat Answered in the linked ticket. It is a wrong decision to drop name mappings generation.


Add permissive option to image registry initializer

This PR adds the --permissive flag to the image registry docker build to ignore errors when loading and initializing the OLM bundles restoring the behavior the image registry initializer had before this PR operator-framework/operator-registry#74
This option should be removed once operator-framework/operator-registry#76 is merged which fixes the initial issue where the initalizer attempts to parse any file as JSON or YAML.
This should be considered temporary to fix our Jenkins builds until the above is complete.

/lgtm


GA Universla : extract UserID parameter and value

Verifying proper implementation of UserID can be central in Google Analytics audit.
Could you include it in the base panel data parsed from the GA hit, instead of requiring to look at the full parameter list (through the plus sign expanding values)

Added in 0.10.13


Command w/o args displaying usage should return non-zero

Try running git, and you will see its usage. You will also receive a non-zero return code, indicating failure. This can be vital in case arguments got truncated.
Current behavior of showing usage instructions (help_text) is great!
But it should also error out with a code 1.
Example:
➜  git
usage: git [--version] [--help] [-C <path>] [-c <name>=<value>]
           [--exec-path[=<path>]] [--html-path] [--man-path] [--info-path]
           [-p | --paginate | --no-pager] [--no-replace-objects] [--bare]
           [--git-dir=<path>] [--work-tree=<path>] [--namespace=<name>]
           <command> [<args>]

(snip)

'git help -a' and 'git help -g' list available subcommands and some
concept guides. See 'git help <command>' or 'git help <concept>'
to read about a specific subcommand or concept.
➜  echo $?
1

Reference: https://github.com/learningequality/kolibri/pull/5937

I'm not completely understanding, can you clarify what you are asking for?

If my wonderful-command is run without arguments, it should be understood as an error. But it should also output usage instructions.

Git actually returns a 0 exit code if you call it with --help.

That's because git --help is understood as a deliberate command. git is understood as a wrong command but will output usage instructions for convenience.

I looked at the linked issue, what specifically would you change within click.
The following seems fairly straightforward to add if your Click app doesn't throw a missing argument expection, but you still want it to return a non zero exit code.
    if ctx.invoked_subcommand is None:
        click.echo(ctx.get_help())
        ctx.exit(1)

Yes, it's simple to implement, as you can see that's what we did -- but perhaps Click can add it as default behavior? I intended to open the issue to flag that it seems like a deviation from conventional behavior.

You've only described it differing from git's behavior, not "conventional" behavior. Is there any convention or standard for this that is demonstrated by a majority of applications?
I don't see "no args is error" as any more valid than "no args printing help is expected, so not an error", so I don't currently see a compelling reason to change this.

@davidism

so I don't currently see a compelling reason to change this

mycommand $filename <= if $filename is empty by mistake, then it will return 0 as if execution was successful.

Is there any convention or standard for this that is demonstrated by a majority of applications?

I tested git, rsync, ssh, cp, mv, rm, ip. They all return non-zero.

This only happens with groups - invoking a (sub-)command requiring arguments, without arguments, will raise and exit with a non-zero status code.  The default behaviour of a group is not to call the callback (i.e. the wrapped function) if the argument is not supplied, which presupposes that it is required.  Click should exit with a non-zero status code if invoke_without_command is false and defer to the callback if it's true.

If anyone else has trouble parsing @benjaoming's comment above, the "<=" is not meant to be "less than or equal". It's meant more like "so" or just full stop. In other words:

mycommand $filename. If $filename is empty by mistake, then it will return 0 as if execution was successful.

I do see this as one valid reason that Click's current default behavior might lead to unexpected results.
I also have seen, more often than not, widely-used command line tools print usage to stderr and exit nonzero when invoked without required arg(s).
But while changing this default in Click may help for this use case, it's not immediately obvious that it wouldn't hurt other use cases, given the wide variety of different CLIs this could affect. It would be helpful if someone could show how this change would affect other use cases.

Thanks @jab , here is the example again without confusing characters:
# if `$filename` is empty by mistake, then it will return 0 as if execution was successful.
mycommand $filename


fix(es/minifier): Don't track usage assign

Description:
Don't know if it's right, but it generate correct output.
BREAKING CHANGE:

Related issue (if exists):

Oh, sorry

Yeah, if isObject is a normal function it would work fine.

The difference can be run down to https://github.com/terser/terser/blob/master/lib/compress/tighten-body.js#L887. It checks if it's a AST_SymbolDefun(definition of function) or defined in current scope, which I believe is just a convenient implementation.

The difference can be run down to https://github.com/terser/terser/blob/master/lib/compress/tighten-body.js#L887. It checks if it's a AST_SymbolDefun(definition of function) or defined in current scope, which I believe is just a convenient implementation.

So basically function isObject is fine, but const isObject = function is fine only if it's defined in same scope of newValue.


Routes not defined

Have just upgraded an app from laravel 5.1 to laravel 6.0
Modules from 2.1 to 6.1
Worked through the bulk of issues already and now stuck on module routes being undeclared.
local.ERROR: Route [admin.event.alert.index] not defined. {"userId":1,"exception":"[object] (Symfony\\Component\\Routing\\Exception\\RouteNotFoundException(code: 0): Route [admin.event.alert.index] not defined. at /core/vendor/laravel/framework/src/Illuminate/Routing/UrlGenerator.php:420)
[stacktrace]
#0 /src/core/app/Http/helpers.php(41): Illuminate\\Routing\\UrlGenerator->route('admin.event.ale...', Array, true, NULL)
#1 /src/modules/Events/Events.php(133): route('admin.event.ale...')
#2 /src/core/app/Http/Controllers/BaseController.php(191):

This was also being triggered with artisan commands but resolved that.
Had to remove some route() links that were being set during controller constructs
Some could stay in, some caused the error - guessing this is due to order of the module loading?
Another instance is in a Modules main file, again removing route gets it a stage further.
Is there anything I can add to the service provider or similar to ensure that all route, for all modules are registered, before it starts to execute main code?
Was working fine in the old version so some logic or function has changed in the later versions.

Ignore this.
For reference incase anyone experiences similar was due to core laravel changes in how to name routes between old Controller:: and newer Resource:: formats.


Ignores not being applied for Gradle

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
gradle
Package manager version
Gradle 7.6
Language version
Java 17
Manifest location and content before the Dependabot update
Reposistory at relevant commit: https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1/
I have a few separate independent projects inside a single repo: plugin, internal plugin, demo for AGP 7 and java demo.
Repository: https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1/

https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1/settings.gradle.kts
https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1/internal/settings.gradle.kts
https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1//demo/java/settings.gradle.kts
https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1//demo/android/agp7/settings.gradle.kts

plugin contains a few modules, depending on libraries with conflicting API by design, and it working well:

https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1//plugin-gradle/compat/agp34/build.gradle.kts
https://github.com/jsonschema2dataclass/js2d-gradle/blob/69cf5b5c1b3290301508e98d581f70764771a3f1//plugin-gradle/compat/agp7/build.gradle.kts

dependabot.yml content
# To get started with Dependabot version updates, you'll need to specify which
# package ecosystems to update and where the package manifests are located.
# Please see the documentation for all configuration options:
# https://help.github.com/github/administering-a-repository/configuration-options-for-dependency-updates

version: 2
updates:
  - package-ecosystem: gradle
    directory: "plugin-gradle/compat/agp7"
    schedule:
      interval: "daily"
    ignore:
      - dependency-name: com.android.tools.build:gradle
        update-types: [ "version-update:semver-major" ]
  - package-ecosystem: gradle
    directory: "/plugin-gradle/compat/agp34"
    schedule:
      interval: "daily"
    ignore:
      - dependency-name: com.android.tools.build:gradle
        update-types: [ "version-update:semver-major" ]
  - package-ecosystem: gradle
    directory: "demo/java"
    schedule:
      interval: "daily"
  - package-ecosystem: gradle
    directory: "demo/android/agp7"
    schedule:
      interval: "daily"
    ignore:
      - dependency-name: com.android.tools.build:gradle
        update-types: [ "version-update:semver-major" ]
  - package-ecosystem: "github-actions"
    directory: ""
    schedule:
      interval: "daily"
  - package-ecosystem: gradle
    directory: ""
    schedule:
      interval: "daily"

Updated dependency
No response
What you expected to see, versus what you actually saw

Despite on rules created to skip major updates for certain modules, dependabot stubbornly updates the PR for it. example: https://github.com/jsonschema2dataclass/js2d-gradle/pull/620 and there's serveral such PR
Despite I remove includeBuild from the settings.build.kts for a subproject, it still has an attachment to the root project which is incorrect as well
Changes in settings.gradle.kts are ignored while this file defines a gradle project, not build.gradle.kts

Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
https://github.com/jsonschema2dataclass/js2d-gradle/pull/620
and few recently closed from dependabot. I ususally merge, not close as in most cases it makes more sence
Smallest manifest that reproduces the issue
No response

Sorry for the slow response. I keep thinking I'll have time to dive into this further, but reality is that may not happen for a bit due to some of the other larger improvements we've got in flight.
In the meantime you are welcome to use the dry-run script to investigate further. It should be relatively straightforward, just run it within the docker container (see the linked docs), and sprinkle debugger / puts statements through the code as needed to try to track down what's happening.
If you are able to zero in on the problem a bit further, that'd certainly be helpful for whenever we do have time to pick this up...

Updated the issue to include relevant commit

I don't know how to run properly docker x86-64 machines on M2 CPU


Changed the sentForgotPasswordEmail to /auth

[x] Changed the location of the endpoint to /auth
[x] Refactored tests to properly use the new initializers

closes #332


Add custom tag directive on vddl-draggable component

This allow the user to change the rendered HTML of the vddl-draggable component.

@ChucKN0risK
👍Thank you for your PR, Could you please add this attribute to other components? 😄

Will do ;)

Done ;)


Feat: store pokemons info into context api

Prevent too many api calls

TODO: add abilities description into context api


Adds pyasn1 0.1.9 to setup.py

Oauth2client requires 0.1.7 and something
else requires 0.1.8. If pip resolves Oauth2client
first the brkt cli installs incorrectly. This
ensures we have a sufficiently up to date version
of pyasn1

Ship it.


The search bar looks very basic and does not have a valid design wrt to the aesthetic website.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The search bar looks very basic and does not have a valid design wrt to the aesthetic website.
Add Screenshots / Video Clip

What browsers are you seeing the problem on?
No response
Agreements

[X] I agree to follow this project's Code of Conduct
[ ] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

Assign this task to me

pls assign this issue to me.


Tutorial on how to add the bot to telegram

**Is your feature request related to a problem? No
Describe the solution you'd like
Uhmm I want to learn how to add the bot on telegram.
Describe alternatives you've considered
None
Additional context

Check out the bot section on the Readme.  Link


[4pt]  - Display Reservations pop up with selected item's details - Student C

Create a reservations popup only with the top section (displaying details of the selected item)

When the popup loads, the webapp retrieves data from the selected API and shows details about the selected item.
Add event to button prepared by your teammate that will open your popup. You need to communicate with your team member to make sure that you do not block each other (most likely the easiest solution is to merge the list of items feature to the dev branch before you try to add that event).

We only had 2 group members.


Add Delete state

Description
Please explain the changes you've made
Added Delete State to the DaprClient.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #74
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[ ] Extended the documentation

@chinzhiweiblank Thanks for this contribution. can you please address the comments that I left in https://github.com/dapr/python-sdk/pull/100 ? or please create the separate pr to resolve the comments in #100

@youngbupark Actually I was wondering are there no integration tests for the python-sdk? Or is the fake dapr server sufficient for testing?

@youngbupark Actually I was wondering are there no integration tests for the python-sdk? Or is the fake dapr server sufficient for testing?

No integration test for python-sdk. fake server may not sufficient, but having integration tests is not easy in terms of maintenance. so that I didn't add integration test in the first place. If you can work on this, please create an issue.

@chinzhiweiblank Thanks for your contribution!


problem with date rendering

the class PiwikDate renders the date in a 12-hours format without the AM/PM indication, as a result all actions are logged in the morning!
Changing the following line in both constructors for the PiwikDate class fixes the issue
format = new SimpleDateFormat("yyyy-MM-dd hh:mm:ss");
into
format = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
Thank you!
Andrea

Good find @rivoduck - and kuddos for quick fix @bcsorba :

Released in v1.1