Throwing a more specific exception instead of a generic RuntimeException

Description:
Just passing the RuntimeException error does not give clarity due to what issue the code implementation will be failed. It also makes it difficult to fix the issue. Besides, when working with a generic type of exception, the only way to distinguish between multiple exceptions is to check their message, which is error-prone and difficult to maintain. Legitimate exceptions may be unintentionally silenced and errors may be hidden.
Location:
src/main/java/de/dennisguse/opentracks/settings/PreferencesOpenHelper.java
Category: Maintainability

issue is solved by using UnsupportedOperationException.

commit reference: https://github.com/Reva-B98/OpenTracks-Winter-SOEN-6431_2024/commit/003e56e9ca7f68a3e90625f24dae7c3cf06b3442


PreProcessing... so slowly

can I make it quickly please ， it‘s so slow，PreProcessing...:  12%|█▏        | 5540/44613 [1:33:41<11:00:46,  1.01s/it]
is it normal？

Have you checked issue #38 ?


Rewrite $ identifiers

In essence I originally hoped to be able to shift all work to the macro, but names of things were the hardest. The worst thing about it is that all the async method names were wrong, so in a stack trace asyncMap would show up as map.
This change fixes that, eliminates the need for additional infrastructure to allow tests to import sync or async variants easily using $, and also is what we need in order to use our system on our typescript definitions.


Cannot create bank account

The ts api allows the following which result in a 500 internal server error.
    const details: bankAccount.IBANDetails = {
      Type: 'IBAN',
      OwnerName: data.ownerName,
      OwnerAddress: {
        AddressLine1: data.address.line1,
        AddressLine2: data.address.line2,
        City: data.address.city,
        Region: data.address.state,
        PostalCode: data.address.zip,
        Country: data.address.country as CountryISO,
      },
      IBAN: data.iban,
    };
    await this._api.Users.createBankAccount(data.userId, details);

The example in /test does not compile
    const details = new models.BankAccount({
      Type: 'IBAN',
      OwnerName: data.ownerName,
      OwnerAddress: {
        AddressLine1: data.address.line1,
        AddressLine2: data.address.line2,
        City: data.address.city,
        Region: data.address.state,
        PostalCode: data.address.zip,
        Country: data.address.country as CountryISO,
      },
      IBAN: data.iban,
    });
    await this._api.Users.createBankAccount(data.userId, details);

The IBAN I was using was incorrect, which returned a 500.


replay: send frames based on encodeIdx packet

This should simplify things a bit. @deanlee what do you think?

lgtm. the only problem is that the missing frames cannot be traced after this.are we going to do some special processing for lost frames in the future? for example, output a blank frame

What do you mean by missing frames? A frame that has an encodeIdx packet but is not in the hevc?

What do you mean by missing frames? A frame that has an encodeIdx packet but is not in the hevc?

yes

Where do we still call getRoadCameraState? I think I made everything driven by encodeIdx now.

Sorry for my bad expression, I mean, do we need to consider the situation where the frame does not has a corresponding encodeid? and do some special processing in the future: #22095

Think it's fine for replay if we only show frames that are actually in the video file for now. We will deal with dropped frames later.

After this change the demo route doesn't perform very well since it doesn't have eof/sof timestaps set correctly. This is no longer a problem on recent routes. Shouldn't be a problem since we will replace the demo route anyway by something that has modelV2 packets.

I suggest to set the ENCODE_IDX's mono_time to SOF to ensure the orders

Do you want to handle this outside of the event class? We still need to check for both eof and sof. C2 only has eof set, and in earlier routes none of them are set.

How about put aside this simplification for now, back later and continue to do it?

This change is necessary for replay to work on qlogs, since they don't contain full frequency cameraState packets.

hmmm...OIC

Let's merge this for now, but the real solution is probably to insert a special event type that only handles sending the frame data over visionIPC. That should allow us to send the encodeIdx packet at the correct logMonoTime, while also maintaining timing accuracy of the video.

@pd0wm I have submitted a new PR #22364 after read your code. insert new custom encode events  in logreader.  the new event's memory will be handled by logreader as before.

@pd0wm It seems that there are duplicate eidx in the dmeo route, each frame has two identical eidx

It shouldn't since it checks for the type to be HEVC before publishing. We should really replace the demo route with something more recent though.


cx_Oracle.DatabaseError: ORA-01455: converting column overflows integer datatype

Old error, but just for reference. Migration of two or three tables (all named waveform_signals) failed with the following error:
Traceback (most recent call last):
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/engine/result.py", line 1131, in fetchall
    l = self.process_rows(self._fetchall_impl())
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/engine/result.py", line 1082, in _fetchall_impl
    return self.cursor.fetchall()
cx_Oracle.DatabaseError: ORA-01455: converting column overflows integer datatype

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "run_migration.py", line 173, in <module>
    logged,trialrun=trialrun)
  File "oracle-to-postgres/migrate.py", line 119, in copy_data
    data = source_session.execute(query).fetchall()
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/engine/result.py", line 1137, in fetchall
    self.cursor, self.context)
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/engine/base.py", line 1413, in _handle_dbapi_exception
    exc_info
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/util/compat.py", line 265, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb, cause=cause)
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/util/compat.py", line 248, in reraise
    raise value.with_traceback(tb)
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/engine/result.py", line 1131, in fetchall
    l = self.process_rows(self._fetchall_impl())
  File "oracle-to-postgres/env/lib64/python3.6/site-packages/sqlalchemy/engine/result.py", line 1082, in _fetchall_impl
    return self.cursor.fetchall()
sqlalchemy.exc.DatabaseError: (cx_Oracle.DatabaseError) ORA-01455: converting column overflows integer datatype (Background on this error at: http://sqlalche.me/e/4xp6)

Fix was to convert the columns in the source table to varchar2 before migration (then convert bask to numerical type in the destination database, if necessary):
ALTER TABLE <schema>.<table> ADD (temp VARCHAR2(100));
UPDATE <schema>.<table> SET temp = <column>;
ALTER TABLE <schema>.<table> DROP COLUMN <column>;
ALTER TABLE <schema>.<table> RENAME COLUMN temp TO <column>;


Ammo in clip & infinite ammo fixes (#70, #71)

Some fixes for #70 and #71.
I have built the repo and tested everything, works like it should now.

Call SetAmmoInClip on spawn, for consistency
Fix infinite ammo refill clip when reloading finishes
Fix infinite ammo refill start reloading
Fix refill clip depending on ammo left when reloading finishes
Set ammo in clip when weapon changes
Set ammo in clip when weapon state changes
Set ammo in clip when weapon skill changes
Set ammo in clip when weapon clip size changes
Set ammo in clip when weapon info (clip size) changes
Don't send bullet and lower ammo counts when setAmmoInClip was not
used after setAmmo:

SetAmmoInClip not used after SetAmmo ==> will reload first
SetAmmo not used before SetAmmoInClip ==> won't shoot

Well, I am fucking moron.  Have already deleted the local repository and data recovery results to nothing. So yeah all fixes are lost.

I'm working on this bugs right now, don't worry.


Fan-in/fan-out of projections

For now it is not possible to set fan-in and fan-out of projections.

For now fan-in will be sufficient. Set at top interface level in builder.py


Regression: CTRL-X completion does not work

CTRL-X completion—CTRL-X_CTRL-F, CTRL-X_CTRL-V, etc—in insert mode does not work unless it is performed promptly after nvim starts. Also, -- INSERT -- may blink.
This bisects to 5d996271561db01f1a173c5b2481b9b410cf0225.
minimal vimrc:
set runtimepath-=~/.config/nvim
set runtimepath-=~/.config/nvim/after
set runtimepath-=~/.local/share/nvim/site
set runtimepath-=~/.local/share/nvim/site/after

set runtimepath+=path/to/denops.vim
set runtimepath+=path/to/ddc.vim
set runtimepath+=path/to/ddc-ui-native

call ddc#custom#patch_global('ui', 'native')

call ddc#enable()

Steps to reproduce:

nvim -u vimrc
Enter insert mode and press CTRL-X; it must show -- ^X mode (^]^D^E^F^I^K^L^N^O^Ps^U^V^Y) but doesn’t
Successively press CTRL-F; it must complete files under the working directory but doesn’t

Environment:

OS: EndeavourOS
deno: 1.36.0
nvim: v0.10.0-dev-766+gef44e5972
ddc.vim: shougo/ddc.vim@0d49052eb38f92c5f1b8c3bb660dd1fffc1a1e1f
denops: vim-denops/denops.vim@22e7ae3d712f190b3f46483e6b9dd50a99a9abbd

Fixed.

Thank you for your quick response, but it still does not complete a non-empty string.
The following is a gif of echo 'set runtim' | nvim -u /tmp/vimrc -c 'setf vim' - under /:

it must complete runtimepath but it doesn’t; -- INSERT -- blinks
it works as expected only when there are no characters before the cursor

Oh.  Too bad.
I have removed call ddc#hide().  Please update.

Thanks, now it works.


Express.Session isn't obvious

Hi @benawad ! Thank you for your great videos.
Today I faced with one problem here
https://github.com/benawad/lireddit/blob/28011261643e466bb9258acf1f5c335582639201/server/src/types.ts#L7
as I can see this type we get from express-session  even though you tried to import it from express and it didn't work.
when I try to write something like  Express.Session I get the error
Namespace 'global.Express' has no exported member 'Session'

i tried to solve it via, but I am not good at typescript ;)
import { Connection, EntityManager, IDatabaseDriver } from "@mikro-orm/core";
import { Request, Response } from "express";

export type MyContext = {
  em: EntityManager<IDatabaseDriver<Connection>>;
  req: Request & { session: Request };
  res: Response;
};

in server/src/types.ts
and
req.session.sessionID = user.id.toString();

in the server/src/resolvers/user.ts
versions:
{
    "express": "^4.17.1",
    "express-session": "^1.17.1",
}

sorry in advance if I just didn't understand  or miss something .

Looks like the session type has been removed, you could probably just do:
Request & { session: { userId: string } };

Thanks so much for asking this. Spent 6+ hrs trying to figure out what's happening.

I found this to be working!
in /server/index.d.ts
import { User } from './src/entities/User';

declare global {
  namespace Express {
    interface Session {
      userid?: number;
    }
  }
}

and add this to "files": [] attribute of tsconfig.json.
Express.Session is available at /server/src/types.ts :tada:
is it okay to play with declaration merging or it's not advisable to do so?

@dzirg44 This should work.
import { Request, Response } from 'express';
import { Session, SessionData } from 'express-session';

export type MyContext = {
  req: Request & {
    session: Session & Partial<SessionData> & { userId?: number };
  };
  res: Response;
};

I did something like this -
interface SessionData {
  userId: number;
}

export type Context = {
  req: Request & { session: SessionData };
};

@dzirg44 This should work.
import { Request, Response } from 'express';
import { Session, SessionData } from 'express-session';

export type MyContext = {
  req: Request & {
    session: Session & Partial<SessionData> & { userId?: number };
  };
  res: Response;
};

This worked fine. Thanks @dzirg44

This helped me

I did something like this -
interface SessionData {
  userId: number;
}

export type Context = {
  req: Request & { session: SessionData };
};

@dzirg44 This should work.
import { Request, Response } from 'express';
import { Session, SessionData } from 'express-session';

export type MyContext = {
  req: Request & {
    session: Session & Partial<SessionData> & { userId?: number };
  };
  res: Response;
};

Thank you, it worked

Looks like the session type has been removed from express, you could probably just do:
Request & { session: { userId: string } };

Your videos are on the next level, thank you so much for all of this

This solves too
import session from "express-session";
...
Request & { session: session.Session & { userId?: number } };

import session from "express-session";
declare module "express-session" {
interface SessionData {
  userId?: number;
 }
}
export type MyContext = {
  req: Request & { session: session.Session & Partial<session.SessionData> };
res: Response;
redis: Redis;
usersLoader: ReturnType;
updootLoader: ReturnType;
};

import session from "express-session";
declare module "express-session" {
interface SessionData {
  userId?: number;
 }
}
export type MyContext = {
  req: Request & { session: session.Session & Partial<session.SessionData> };
res: Response;
redis: Redis;
usersLoader: ReturnType;
updootLoader: ReturnType;
};

I have tried several methods and for whatever reason, there is not a cookie being saved...

I have tried several methods and for whatever reason, there is not a cookie being saved...

@BookmDan this is all my advice for debugging cookies: https://benawad.com/cookie

@BookmDan this is all my advice for debugging cookies: https://benawad.com/cookie

Thank you @benawad That was helpful. Actually, because we are incorporating with RedisStore would that be considered a 3rd Party Cookie? Asking for a friend.

Thank you @benawad That was helpful. Actually, because we are incorporating with RedisStore would that be considered a 3rd Party Cookie? Asking for a friend.

This is the error message that I am seeing:

This is the error message that I am seeing:

I have tried several methods and for whatever reason, there is not a cookie being saved...

hey, the thing that solved the problem for me was "removing" secure while in development.
like this:
app.use(
    session({
      name: "qid",
      store: new RedisStore({
        client: redisClient,
        disableTouch: true,
      }),
      cookie: {
        maxAge: 1000 * 60 * 60 * 24 * 365 * 1, // 10 years
        httpOnly: true,
        sameSite: "lax", // csrf
        // secure: __prod__, // cookie only works in https
      },
      saveUninitialized: false,
      secret: "qowihho",
      resave: false,
    })
  );

I have tried several methods and for whatever reason, there is not a cookie being saved...

hey, the thing that solved the problem for me was "removing" secure while in development.
like this:
app.use(
    session({
      name: "qid",
      store: new RedisStore({
        client: redisClient,
        disableTouch: true,
      }),
      cookie: {
        maxAge: 1000 * 60 * 60 * 24 * 365 * 1, // 10 years
        httpOnly: true,
        sameSite: "lax", // csrf
        // secure: __prod__, // cookie only works in https
      },
      saveUninitialized: false,
      secret: "qowihho",
      resave: false,
    })
  );

I have tried several methods and for whatever reason, there is not a cookie being saved...

I have solved this by setting cors in apollo server.
apolloServer.applyMiddleware({
app,
cors: {
credentials: true,
},
});

I have tried several methods and for whatever reason, there is not a cookie being saved...

I have solved this by setting cors in apollo server.
apolloServer.applyMiddleware({
app,
cors: {
credentials: true,
},
});

According to express-session, this is what I did:
import { Request } from "express";
import session from "express-session";

declare module "express-session" {
  interface SessionData {
    userId?: number;
  }
}

export type MyContext = {
  req: Request & { session: session.SessionData };
  res: Response;
};

I guess this issue has to be closed! like all solutions here do work fine!

I literally can't get express-session to work lmao, even if it sets a cookie, req.session is undefined and it's in the right order. Even copied your code & package versions. It seems to send the cookie back to express but it just refuses to fetch the set value...
At this point, setting a session cookie manually if it does not exist then looking up the key is redis is actually a lot easier to do than getting express-session to work.

Yes I'm having a similar issue.
On to the next project for now...
On Mon, Feb 22, 2021, 2:36 PM SSouper<EMAIL_ADDRESS>wrote:

I literally can't get express-session to work lmao, even if it sets a
cookie, req.session is undefined and it's in the right order. Even copied
your code & package versions. It seems to send the cookie back to express
but it just refuses to fetch the set value...
At this point, setting a session cookie manually if it does not exist then
looking up the key is redis is actually a lot easier to do than getting
express-session to work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/benawad/lireddit/issues/13#issuecomment-783725687,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEDGR47SSYDDVMOZXG6MSLTALL5HANCNFSM4TRFP64A
.

Yes I'm having a similar issue. On to the next project for now...

It seems like the apollo middleware removes every field from req that "should not be there" (session included). I put a middleware function before apollo but after express-session and req.session was there.

do you get the same result if you clone my code?

@benawad looks like i forgot to destructure the first argument to the context function. Strangely, typescript didn't tell me that i'm a bad boy...

req: Request & {
    session: Session & Partial<SessionData> & { userId?: number };
  };

Trying this method has resulted in this error for me
node_modules/@types/connect-redis/index.d.ts:32:72 - error TS2694: Namespace 'global.Express' has no exported member 'SessionData'.

    ttl?: number | string | ((store: RedisStore, sess: Express.SessionData, sid: string) => number);

Package.json
devDependencies": {
    "@types/connect-redis": "^0.0.14",
    "@types/express": "^4.17.7",
    "@types/express-session": "^1.17.0",
    "@types/node": "^14.0.27",
    "@types/redis": "^2.8.25",
    "nodemon": "^2.0.4",
    "ts-node": "^8.10.2",
    "typescript": "^3.9.7"
  },
  "dependencies": {
    "@mikro-orm/cli": "^4.0.0-alpha.0",
    "@mikro-orm/core": "^4.0.0-alpha.0",
    "@mikro-orm/migrations": "^4.0.0-alpha.0",
    "@mikro-orm/postgresql": "^4.0.0-alpha.0",
    "apollo-server-express": "^2.16.1",
    "argon2": "^0.26.2",
    "class-validator": "^0.13.1",
    "connect-redis": "^5.0.0",
    "express": "^4.17.1",
    "express-session": "^1.17.1",
    "graphql": "^15.3.0",
    "pg": "^8.3.0",
    "redis": "^3.0.2",
    "reflect-metadata": "^0.1.13",
    "type-graphql": "^1.0.0-rc.3"
  },

The solution I found was adding this, but it feels kind of hacky, is there a better solution?
declare global {
  namespace Express {
    interface SessionData {
      cookie: any;
    }
    interface Session {
      userId?: number;
    }
  }
}

Don't forget to ensure that the GraphQL playground settings have credentials to include. This took me an hour to remember that Ben mentioned this very early on in the video:

Looks like the session type has been removed from express, you could probably just do:
Request & { session: { userId: string } };

It should be

Request & { session: { userId?: number } }

because again in UserResolver  me( ) Query, Id is a number.

const user = em.findOne(User, { id: req.session.userId }); // where id is a number.

I have tried several methods and for whatever reason, there is not a cookie being saved...

maybe you should check your constant
it must be
 export const __prod__ = process.env.NODE_ENV === "production";
not
  export const __prod__ = process.env.NODE_ENV !== "production";

Don't forget to ensure that the GraphQL playground settings have credentials to include. This took me an hour to remember that Ben mentioned this very early on in the video:

When I change it from "omit" to "include" I get this error, any solution for this?

I found a weird behaviour and can't really explain why this is:
When setting the req.session.userId to the uid in login or register, the mutation gets executed properly, but the frontend never revieves a response. But when i don't set the 'userID', a response is recieved.
So for example the following code
req.session.userId = user._id;
console.log(req.session) // this gets printed
return { user };

would print the session object with cookie and userId, but the frontend never leaves the fetching state and eventually times out.
The same for the playground btw

This post helps fix the the switch from playground to sandbox if you are unable to test the cookie login
https://stackoverflow.com/questions/69333408/express-session-does-not-set-cookie/70673115#70673115

If you still haven't fix this issue, use this commit as a guide: https://github.com/backendbro/reddit-jnr/commit/766e01462a6a678145bcad043b1190e3d58d64eb
For my apollo sanbox I turned on:

Include cookies

And set:

x-forwarded-proto: https


range can't iterate over

I use helm install --dry-run --debug debugging my chart.
deployment.yaml
...
          volumeMounts:
            - mountPath: /usr/local/apisix/conf/config.yaml
              name: apisix-config
              subPath: config.yaml
            - mountPath: /etc/localtime
              name: localtime
              readOnly: true
              {{- range $customerPlugin := .Values.customerPlugins}}
            - mountPath: /usr/local/apisix/apisix/plugins/{{ $customerPlugin }}
              name: apisix-config
              subPath: {{ $customerPlugin }}
              {{- end }}
...

values.yaml
customerPlugins:
  hmac-auth.lua
  count.lua

[root@localhost gjn]# helm install mychart --dry-run --debug ./apisix
install.go:172: [debug] Original chart version: ""
install.go:189: [debug] CHART PATH: /home/gjn/apisix

Error: template: apisix/templates/deployment.yaml:95:50: executing "apisix/templates/deployment.yaml" at <.Values.customerPlugins>: range can't iterate over hmac-auth.lua
helm.go:94: [debug] template: apisix/templates/deployment.yaml:95:50: executing "apisix/templates/deployment.yaml" at <.Values.customerPlugins>: range can't iterate over hmac-auth.lua

I think I may not use the function well, but I don't know how to use range() to achieve my target.
I hope you can give me some advices.

In your example customerPlugins is not a list. You probably meant this:
customerPlugins:
  - hmac-auth.lua
  - count.lua

There are working examples in the documentation:
https://helm.sh/docs/chart_template_guide/control_structures/#looping-with-the-range-action

In your example customerPlugins is not a list. You probably meant this:
customerPlugins:
  - hmac-auth.lua
  - count.lua

There are examples in the documentation:
https://helm.sh/docs/chart_template_guide/control_structures/#looping-with-the-range-action

Thank you, I will learn more ...


Have the possibility to not use certmanager

Is your feature request related to a problem? Please describe.
In my entreprise k3s cluster we don't have let's encrypt connection and we just have a wildcard cert so we don't need cert manager but it's a requirement for epinio
Describe the solution you'd like
Have the possibility to install Epinio without cert manager and support tsl cert file

Basic check: Install E without a cert-manager on the cluster.
Result:
Error: unable to build kubernetes objects from release manifest: [resource mapping not found for name: "epinio" namespace: "epinio" from "": no matches for kind "Certificate" in version "cert-manager.io/v1"
ensure CRDs are installed first, resource mapping not found for name: "minio-cert" namespace: "epinio" from "": no matches for kind "Certificate" in version "cert-manager.io/v1"
ensure CRDs are installed first, resource mapping not found for name: "epinio-ca" namespace: "cert-manager" from "": no matches for kind "Certificate" in version "cert-manager.io/v1"
ensure CRDs are installed first, resource mapping not found for name: "epinio-registry" namespace: "epinio" from "": no matches for kind "Certificate" in version "cert-manager.io/v1"
ensure CRDs are installed first, resource mapping not found for name: "selfsigned-issuer" namespace: "" from "": no matches for kind "ClusterIssuer" in version "cert-manager.io/v1"
ensure CRDs are installed first, resource mapping not found for name: "epinio-ca" namespace: "" from "": no matches for kind "ClusterIssuer" in version "cert-manager.io/v1"
ensure CRDs are installed first]

E chart uploads a number of cert-manager.io/v1 resources. Without the cert-manager (CRD) present that fails.
Step I: Exclude these resources.

With the cert-manager resources disabled a number of secrets are missing (IOW those which are generated from the cert resources by cer-manager):

dex-tls
epinio-registry-tls
epinio-tls
epinio-ui-tls
minio-tls

These secrets have to be created as regular resources when the cert-manager is not present.

Beware: The standard application chart also make use of cert-manager Certificate resources.
Namely to set up the tls secret for the app's ingress.

Beware: The standard application chart also make use of cert-manager Certificate resources. Namely to set up the tls secret for the app's ingress.

Having slept over this we actually have a way for this already. Our Routing secrets.
See https://docs.epinio.io/references/customization/routing_secrets
The operator has to provide one more secrets for the app domains
Epinio will automatically use them for matching domains.
And the cert-manager certificates in the app chart are disabled in favor of that secret.

As a summary, and possible specification:

Enable the disabling of cert manager by setting the .Values.certManagerNamespace to the empty string (not leaving it out, that would be null). Change templating to react to this by excluding all the Certificate resources.
Extend the E chart with kube Secret resources for all the excluded certificates. Activate them in templating when the cert manager is disabled.
Extend the values.yaml with variables for the content of the Secret keys (tls.crt, tls.key, and ca.crt) and change the chart to template them in. This enables the operator to insert their cert(s) on installation.
Question: Expose each Secret separately ? It would be easier to expose only one set of variables for this, and simply use them in all the new secrets.
Document that the operator is forced to use routing secrets in this mode to bypass the ingress Certificate in the standard app chart.
Actually the operator could create a non-standard appchart instead, which uses a Secret instead of a Certificate and making that the default in the E installation. Basically having the necessary routing secret(s) baked in. This would obviate the need for the operator to put the routing secrets into all possible namespaces. The appchart would do it for the developer.

@enrichman Your thoughts and opinions ?
@yodatak Ditto.

As a summary, and possible specification:

Enable the disabling of cert manager by setting the .Values.certManagerNamespace to the empty string (not leaving it out, that would be null). Change templating to react to this by excluding all the Certificate resources.

The certManagerNamespace in the Rancher installation question.yaml is not required:
- variable: certManagerNamespace
  label: Cert-manager namespace
  description: "Namespace where cert-manager is installed in"
  type: string
  required: false
  group: "Advanced settings"

so I would prefer having a more explicit and consistent flag, like the other that we have for Minio, Dex, or
minio:
  enabled: true

epinioUI:
  enabled: true

kubed:
  enabled: true

s3gw:
  enabled: false

containerregistry:
  enabled: true

global:
  dex:
    enabled: true

Extend the E chart with kube Secret resources for all the excluded certificates. Activate them in templating when the cert manager is disabled.
Extend the values.yaml with variables for the content of the Secret keys (tls.crt, tls.key, and ca.crt) and change the chart to template them in. This enables the operator to insert their cert(s) on installation.

Usually the tls.crt and tls.key are enough, the ca.crt is needed only with custom CA, and it needs some work on the application side, so maybe we don't need that. We can also add a more generic "tlsSecretName" to provide the secret with the cert, I find it more ergonomic. Having a look at other charts they have sometimes both of the options.

Question: Expose each Secret separately ? It would be easier to expose only one set of variables for this, and simply use them in all the new secrets.

It's probably possible with some of the secret, not sure if it's doable for all of them. I.e. the Dex secret is the one for the auth.* endpoint, the registry is an internal one, and epinio is on the epinio.*. So for those you can probably expose the wildcard cert, but the registry needs a different one. Also if you are using an external registry you can skip this part, and you need a different secret.
Be aware for the subchart that we should stick to their implementation, when possible. I.e. for minio there is already a way to provide the secret, we just need to drop the cert-manager, and expecting the secret to have been already created by the user.
minio:
  tls:
    enabled: true
    certSecret: minio-tls

We just need to document that if you are using Minio without cert-manager the minio-tls secret won't be there, and you need to create it upfront. Or provide the right secret name. Or drop Minio and use your S3.

Document that the operator is forced to use routing secrets in this mode to bypass the ingress Certificate in the standard app chart.
This is the way. :+1:  If we already provide a way, just continue with this.

Actually the operator could create a non-standard appchart instead, which uses a Secret instead of a Certificate and making that the default in the E installation. Basically having the necessary routing secret(s) baked in. This would obviate the need for the operator to put the routing secrets into all possible namespaces. The appchart would do it for the developer.

@enrichman Your thoughts and opinions ? @yodatak Ditto.

To finalize a bit regarding the secrets:

minio-tls (it should be "already done")
epinio-registry-tls (this is internal and different from the other host.. :/ )

This can be probably handled with the same, but this means that it has to be a wildcard, or not?

dex-tls
epinio-tls
epinio-ui-tls

One thing that we can probably do upfront without any problem is "merging" the two latest secret. I guess that they were separated because the two were two different charts, but they can probably be merged into one secret with two dnsNames. Basically dropping the epinio-ui-tls, in favor of a single epinio-tls with this spec:
spec:
  dnsNames:
  - {{ .Values.global.domain }}
  - epinio.{{ .Values.global.domain }}

We can try this as a first iteration. One secret less that we should care about.


Default Date/Time 12/29/1899 4:00 PM

Here is the feature service - date is set as a required field - not sure why the Default Date is set to 12/29/1899 4:00 PM when I use preview mode.
http://disasterresponse.maps.arcgis.com/home/item.html?id=03e7ac020c964d5fb5af8a43b0ff01f4
Expected behavior would default to today's date?

Matt -
This popped up again...

I will see if I can fix the feature service, but why wouldn't the date / time picker just default to today's date instead of 1899. When would 1899 ever be expected behavior?
I know it may not be a bug - but perhaps this could be 'enhanced'?

We have encountered this also in the latest version of the Geoform as well when a date field is set to required (NULL not allowed in the Geodatabase). When making an on premise feature service where the date field is required the default value is 00:00:00 in the feature's default edit template (in ArcMap).
I think I got around this last time (at least with on premise feature services) by managing the feature's edit template in the AGOL web map after publishing the service and putting it in the web map. There you should be able to set the default value of the date field on new features to "empty" and then it starts empty in the Geoform.
I've forgotten to do that a couple of times and users constantly complain about it :)

Here is a potential workaround using ArcGIS Pro. Design the Field - change the default to January 1 of the current year. This returns expected behavior on the GeoForm. I am not sure how you would do this without ArcGIS Pro other than creating a new field via the attribute table in the web map.
Please consider and enhancement for this feature.

@driskull if you drop the last three zeros from -2209161600000 > the UTC equivalent is Sat, 30 Dec 1899 00:00:00 GMT.
@nixta where else were you seeing this behavior the other day?

@jgravois I was trying to get the max date out of a Feature Service table based off a date field, for which I had to use statistics. Once I'd figured out the JSON parameters, it was returning the max date including milliseconds (and also setting the output statistic field to a numeric type, as it happens).

There should be an option in the GeoForm to set the current date. There's a gear options icon you can see more checkboxes when configuring a date field. That should do it.
Otherwise, I could look into automatically handling it when the date is matched to "00:00:00" or whatever it is that is causing the issue.
-Matt

It is worth testing on your end - but it did not appear that the current date setting was able to override the 1899 issue.
Sent from my iPhone

On Jan 19, 2016, at 09:19, Matt Driscoll<EMAIL_ADDRESS>wrote:
There should be an option in the GeoForm to set the current date. There's a gear options icon you can see more checkboxes when configuring a date field. That should do it.
Otherwise, I could look into automatically handling it when the date is matched to "00:00:00" or whatever it is that is causing the issue.
-Matt
—
Reply to this email directly or view it on GitHub.

Should be fixed in next update. You could download and test or test via the github URL.
http://esri.github.io/geoform-template-js/
http://esri.github.io/geoform-template-js/?appid=[my apps id]

Excellent!
Sent from my iPhone

On Jan 19, 2016, at 16:52, Matt Driscoll<EMAIL_ADDRESS>wrote:
Should be fixed in next update. You could download and test or test via the github URL.
http://esri.github.io/geoform-template-js/
http://esri.github.io/geoform-template-js/?appid=[my apps id]
—
Reply to this email directly or view it on GitHub.


MacOS Catalina and Xcode 11.7 not able to build using ReadME

I ran through the setup instructions and have been scouring the internet for ways to solve the issue but my build continues to fail on react-native-mapbox-gl in various ways. I've also tried to update to the new repo to no avail.

#7


LayoutDOM

ValueError: expected an instance of type LayoutDOM, got...
I keep getting this error each time I try to make a gridplot and I have run out ideas to tackle

Hi @Nkimo ,
can you please provide a minimal example such that I can reproduce your problem?
Best Patrik


Add new from_xml structure

I notice that you have reorganised the way from_xml works in isobib, ietfbib and gbbib. The reorganisation needs to take place for iecbib as well. Fortunately, iecbib should be identical to isobib in this regard: the only difference between the two is the addition of subpart for iecbib references.

Note that I'm going to delay releasing the relaton gems until this is done; the switch to reading references from XML means that there will be major releases for all gems.

Exciting change. Hope it works :wink:

@opoudjis It's not so much reorganization for iecbib since it uses iso-bib-item's to_xml function.
Done it.


EIP WTF

EIP Wiki: Will be gradually updated over time.
Where to find ACDC/ACDE meeting schedule: https://github.com/ethereum/pm/issues
The calendar of (almost) every Ethereum meeting: https://www.ethereumcatherders.com/calendar

EIP Meme
https://twitter.com/AnettRolikova/status/1511622959575842817

PEEPanEIP #40: A Brief History of Ethereum's Future with Ben Edgington


Disable flaky test.

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
397f3384e5d2b5efaf318ef8e2d7a9afd4fb0f68
link
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
397f3384e5d2b5efaf318ef8e2d7a9afd4fb0f68
link
/test lint

lintmodern_istio
397f3384e5d2b5efaf318ef8e2d7a9afd4fb0f68
link
/test lintmodern

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Al/telemetry events for pipeline

\cc @binaryseed @deadtrickster @arkgil @josevalim, pinging all of you since you gave great feedback on the last PR. This PR adds tracing to commands sent to Redis.
I'm looking for feedback on:

the separation between the [:redix, :pipeline] and [:redix, :error] events. I did this because if we have a single event with a success? meta field and an elapsed time measurements, then errors like :disconnected are gonna screw up measurements since their elapsed time will be very short as there's no talking with the database.

putting the whole list of commands in the metadata. I'm doing this so users can figure out how to group them, by count or type or whatever.

Thoughts?

errors like :disconnected are gonna screw up measurements since their elapsed time will be very short as there's no talking with the database.

I wouldn't be worried about skewed measurements, since error events could be always filtered out, although I agree that it's not perfect. Additionally, having a single event simplifies e.g. counting how many pipelines were executed.
Maybe alternative, valid approach would be to have an event before the pipeline is executed (e.g. [:redix, :pipeline, :start]) which allows for easier counting, another event when the response is received successfully for measuring elapsed time, and the last one for errors.


Check remote git URL if it is accessible

fixes: https://github.com/psss/tmt/issues/720

/packit test

Hm, copr build failed because of no network access. More changes will be necessary.

Yep, I've fixed the tests by fixture and simple mocking and also rebased

Would be better to add new commits during the review and squash changes at the end. It's hard to see what actually has changed after three force pushes. Plus it seems some of my changes in 360837d have been lost (the tmt test show -v regression is back).

Would be better to add new commits during the review and squash changes at the end. It's hard to see what actually has changed after three force pushes. Plus it seems some of my changes in 360837d have been lost (the tmt test show -v regression is back).

Yep, I'll use this way instead of implicit squashing when I found that it is not necessarry to see older commit. Understand that in process of review it is hard when I change it by force pushes.
But please let me show what I've exactly forgot when applying your commit. or better to say. I've called also pre-commit manually. so possible that precommit did something else, than you, but similar, but at lest test messages and comment were changed exactly to your suggestions.
and I also need to know what tmt test show -v regression is back means? I do not know what exactly you are talking about and what is bad there.

But please let me show what I've exactly forgot when applying your commit.

The changes in the check_git_url() method itself.

I also need to know what tmt test show -v regression is back means?

Try to run it ;-) The output from git ls-remoteis not captured and is shown to the user.

But please let me show what I've exactly forgot when applying your commit.

The changes in the check_git_url() method itself.

I also need to know what tmt test show -v regression is back means?

Try to run it ;-) The output from git ls-remote is not captured and is shown to the user.

@psss thank you very much after calling pre-commit I did not see this improvement and also as I run the mentioned command I did not see the output of the command, because output were too big that blind to see it :-)

Sorry, for late response. We've shortly discussed this today on the hacking session and there are concerns that after this change it's not possible to list tests in verbose mode without a working internet connection. This does not play nicely with the expectation that a user should be able to clone a git repository and then be able to both implement and test offline without any external dependencies.
Furthermore, the current implementation fails if there is no git repository at all (probably not frequent but possible use case):
tmt init -t base
tmt test show -v
/tests/example
     summary Concise summary describing what the test does
        test ./test.sh
        path /tests/example
   framework beakerlib
      manual false
     require beakerlib
    duration 5m
     enabled true
      result respect
     sources /tmp/base/tests/example/main.fmf
Unable to contact remote git via ''.

But the current implementation is not much better as it produces a bunch of warnings:
/tests/example
     summary Concise summary describing what the test does
        test ./test.sh
        path /tests/example
   framework beakerlib
      manual false
     require beakerlib
    duration 5m
     enabled true
      result respect
     sources /tmp/base/tests/example/main.fmf
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
      fmf-id name: /tests/example
             url:
             ref:
             path: /.

Need to discuss this more and find the best way forward.

I think it would make sense to include the url check during update of an existing test case as well. What do you think? Could you please add it?

Yes please - Right now I'm fixing rogue update of all cases with ref pointing to PR. This should have been prevented by this check.

Yes please - Right now I'm fixing rogue update of all cases with ref pointing to PR. This should have been prevented by this check.

@jscotka, are you going to extend the check for update as well?

Hi @lukaszachy @psss sure, I can also add it for update of existing testcases.

Great, thanks! Shall we try to squeeze it into today's release?


Icon shapes broken with rendering artifacts

VSCode Version: insiders build 1.12.0
OS Version: macOS Sierra

Steps to Reproduce:

Launch code-insiders
Observe how icon shapes are way off

This only started to manifest itself when my insiders build switched to the 1.12 branch. It looks like vector icon rasterization goes wrong or maybe the alpha channel bitmap gets corrupted. Restarting or upgrading sometimes fixes the issue but not always.

@patrys that is quite crazy! can you try to reproduce by picking a standard theme (e.g. dark+) and no icons? This might be related to the electron update but I cannot explain it currently.
Also, can you check if there is anything output?
Please see if you can find any logged output that could help us find the source of the issue:

run code --verbose from the command line and attach the output
in Code select Help | Toggle Developer Tools and select the Console
attach any output you see in the Console you can find

Thanks!

I don't see anything particularly interesting in the output. Not attaching the console output as it's included in the verbose output below.

❯ code-insiders . --verbose
[main 15:38:41] Starting VS Code in verbose mode
[main 15:38:41] from: /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app
[main 15:38:41] args: { _: [ '/Users/patrys/GitHub/mirumee/saleor' ],
  help: false,
  h: false,
  version: false,
  v: false,
  wait: false,
  w: false,
  diff: false,
  d: false,
  goto: false,
  g: false,
  'new-window': false,
  n: false,
  'unity-launch': false,
  'reuse-window': false,
  r: false,
  performance: false,
  p: false,
  'prof-startup': false,
  verbose: true,
  logExtensionHostCommunication: false,
  'disable-extensions': false,
  disableExtensions: false,
  'list-extensions': false,
  'show-versions': false,
  nolazy: false }
[main 15:38:43] IPC#vscode-machineId
[73369:0418/153844.169299:INFO:CONSOLE(29)] "[/Users/patrys/.vscode-insiders/extensions/samverschueren.final-newline-0.3.2]: Cannot register 'files.insertFinalNewline'.  What I've found is even more puzzling: icons are fine if I launch it using the application icon and corrupted when launched from the command line (which is how I usually launch it). I'm using zsh, iTerm2 (3.1.20170416-nightly) and usually work on projects using virtualenv and direnv.

@patrys good findings so it may well be terminal related. are you using some kind of terminal emulator like tmux?

No, not really.
After looking very closely at the glitched icons I think they are rendered correctly but with improperly wide stroke widths. Any idea what could affect the vector rasterizer in such an uncommon way?

@patrys no really not, if I had to guess I would assume that some environment variable you are setting from the terminal has an impact on that. Maybe some unrelated environment variable is actually being picked up by Chrome now that we updated to Electron 1.6.x? Can you try to run VS Code from command line with an empty environment?

Found the culprit. My env has the following locale settings:
LANG=pl_PL.UTF-8
LC_CTYPE=pl_PL.UTF-8

If I unset LANG the icons are rendered correctly 😱

@patrys thanks, I can reproduce. reported to Electron as https://github.com/electron/electron/issues/9225

To add to @patrys LC_NUMERIC discovery: pl_PL uses , instead of . as decimal separator.

I pushed a workaround by explicitly setting LC_NUMERIC = 'C' early on startup of Code.

Same issue can happen when LC_ALL is specified, so I pushed another change to set LC_ALL to C if it is defined.

The problem seems to be back even with a workaround in place with the most recent insiders build. Here's the output of locale:
LANG="pl_PL.UTF-8"
LC_COLLATE="pl_PL.UTF-8"
LC_CTYPE="pl_PL.UTF-8"
LC_MESSAGES="pl_PL.UTF-8"
LC_MONETARY="pl_PL.UTF-8"
LC_NUMERIC="pl_PL.UTF-8"
LC_TIME="pl_PL.UTF-8"
LC_ALL=

@patrys I do not see blurry icons with that environment using VS Code Insiders.

I started seeing this issue with the latest non-insiders update :(

VSCode 1.13.0
MacOS 10.12.5

LANGand LC_CTYPE env vars are set to de_DE.UTF-8
The proposed workaround of setting LC_NUMERIC does not work for me.

Also upgraded to 1.13.0 (376c52) today and now getting blurry icons. Mac OS X 10.10.5. My LC_ALL is empty, LANG="fi_FI.UTF-8".
Bizarrely, opening Extensions panel fixes this for me:

Seeing this on OS X 10.11.6 after upgrading to 1.13.0. LC_ALL is empty and LANG="fi_FI.UTF-8"

Same issue.

VS Code 1.13.0
OS X 10.12.5

LC_ALL is empty
LANG=ru_RU.UTF-8
LC_CTYPE=ru_RU.UTF-8
The problem appears only if launched from terminal

Same here:
LANG="de_DE.UTF-8"
LC_COLLATE="de_DE.UTF-8"
LC_CTYPE="de_DE.UTF-8"
LC_MESSAGES="de_DE.UTF-8"
LC_MONETARY="de_DE.UTF-8"
LC_NUMERIC="de_DE.UTF-8"
LC_TIME="de_DE.UTF-8"
LC_ALL=

Exactly same issue, macOS Sierra with LANG=es_ES.UTF-8, when opened with code . from Terminal it goes blurry. If opened from Spotlight search for example, everything's fine.

Having the same issue, my workaround is to open code in terminal with LC_NUMERIC=C code .

@bpasero Is your fix included in the latest release?

@fuzzy76 will be in the 1.13.1 release later today. Workaround still: https://code.visualstudio.com/updates/v1_12#_macos-blurry-icons

Fixed with our recent Electron update.

Verify: see https://github.com/Microsoft/vscode/issues/24820#issuecomment-294916658


Code in the commented lines #1

What is the issue?
There are commented lines of code which does not give a detailed description and are not relevant to the code.
How is this issue relevant?
In general, comments are used to increase readability and understandability but in this case, there are lines of code that have been commented which decreases the readability of the code which is exactly the opposite of what comments should be used for.
How can we resolve this issue?
To resolve this issue, we have removed the unused code and if necessary that can be retrieved from the control history

Why is only half the comment deleted? Now the commented text is incomplete and contributes to more confusion for people reading the code


Dummy PR t#124

WARNING: Could not create a deploy configuration. Task ID not found in the title of the PR, supported formats:
{ task type }#{ number }
{ task type }{ number }

Available task types: task, t, vx, issue, i, us, hu (case insensitive)
Examples:

task#123

Issue123

US#123

A new configuration was created for openacademy80. You can deploy it here.

A new configuration was created for libreacademia80. You can deploy it here.

The instance created for openacademy80 with task id t124 has been deleted.

A new configuration was created for openacademy80. You can deploy it here.

The instance created for libreacademia80 with task id t124 has been deleted.

A new configuration was created for libreacademia80. You can deploy it here.

WARNING: Could not create a deploy configuration. Task ID not found in the title of the PR, supported formats:
{ task type }#{ number }
{ task type }{ number }

Available task types: task, t, vx, issue, i, us, hu (case insensitive)
Examples:

task#123

Issue123

US#123

The instance created for openacademy80 with task id t124 has been deleted.

The instance created for libreacademia80 with task id t124 has been deleted.

The instance created for openacademy80 with task id t125 has been deleted.

The instance created for libreacademia80 with task id t125 has been deleted.

The instance created for libreacademia80 with task id t124 has been deleted.

The instance created for openacademy80 with task id t124 has been deleted.

The instance created for openacademy80 with task id t125 has been deleted.

The instance created for libreacademia80 with task id t125 has been deleted.


Config does not seem to work.

Changing values in the config file does not seem to affect the game in any way.
Changing the health, availability, or potion effect intensity does not do anything.
MC 1.20.1 Mod 2.0.1

Thanks for the report! Should be fixed in the latest version
https://legacy.curseforge.com/minecraft/mc-mods/rings-of-ascension/files/5166182


use a logger instead of prints

Thank you very much for your nice work 👍!
I can now remove this item from my todo list 😉.

make this todo list public 😉


remove python2.6 and python3.3 support

@Turbo87: i think this is all that needs to be removed. any places i am missing?
after this i want to merge the updates from dependabot and when everything is up to date release a new version


Add quap for Turing

This takes a Turing model and performs quadratic approximation, which means finding the maximum of the loglikelihood (or the minimum of the NLL) and taking the hessian to find the variance and covariance of the parameters.
The goal is to make it just work™ which is why I put some tricks in there like sampling from the prior to find a starting point and chaining optimizers to make sure it finds a good optimum. I tried to put some checks in there, still, there are probably many ways to make this break.
On the simple model in the file it takes ~5ms on my computer, so I didn't take much time to make it any faster.
Since I consider this very experimental, there is still a list of things that need to be done:

Importing the file
Add dependencies
Tests and testing this with more than one or two models
Docstrings
Formatting? For example, in the existing code I have found some 2 spaces, 4 spaces and tabs. I think it would be good to unify this.
Add more features like calling precis

@karajan9 I definitely like this approach.
Do you think (know if) it is possible to get hold of the mu & sigma in the prior and simply use those as start values for quap? E.g.:
quap(m, [178.0,20.0])
(coef = [154.60702994440686, 7.731331556591009], vcov = [0.16973954614799505 0.00021817160619773412; 0.00021817160619773412 0.08490575274440608], converged = true)

It would be nice if we could have just a single Turing model definition instead of having the Turing model version augmented with the return statement just for quap.
I think Turing.VarInfo(m) holds the mu & sigma.
I remember Richard somewhere in the book mentions that quap might sometimes need some coaching.
Rob

Yes, this would be nicer, right now with the model in the file (I guess that's m4.1 or something?) return μ, σ is okay, while all of
return σ, μ
return σ
return μ
nothing

won't work correctly (I think, haven't tested).
It's pretty easy to get the distributions needed:
vi = Turing.VarInfo(m)
d = vi.metadata.μ.dists  ----->
1-element Array{Normal{Float64},1}:
 Normal{Float64}(μ=178.0, σ=20.0)
median(d) == 178.0

which means that
function quap(model::DynamicPPL.Model; method = SimulatedAnnealing())
    vi = Turing.VarInfo(m)
    meta = vi.metadata

    start = [median(var.dists[1]) for var in meta]
    @show start

    quap(model, start; method = method)
end

should work pretty well. I'm a little weary because of the hardcoded [1]. I don't even know what else is supposed to go into that array, so as soon as anything does, it means trouble. Maybe I can ask the Turing devs.
If I change the model to
@model height(heights) = begin
    μ ~ Normal(178, 20)
    σ ~ Uniform(0, 50)
    tmp ~ Normal(μ, 1)
    heights .~ Normal(tmp, σ)
end

then tmps distribution is hardcoded to something like Normal(165..., 1), presumably some random number drawn from μ. I'm not sure how to circumvent that except maybe doing a few samples; it seems to get redrawn by running m().
In spite of those few open questions this seems to me like a good idea and a way better solution than what I cooked up.

Hmm, not sure if I didn't like your initial approach better then adding the tmp ~ Normal(μ, 1) line.
So in m4.1t, your solution would be:
@model line(x, y) = begin
    #priors
    alpha ~ Normal(178.0, 100.0)
    beta ~ Normal(0.0, 10.0)
    s ~ Uniform(0, 50)

    #model
    mu = alpha .+ beta*x
    y .~ Normal.(mu, s)

    return alpha, beta, s
end

which can then be used as in your example:
ml=line(x, y);
quap(ml, [160.0, 0.0, 0.0]) ==>
(coef = [154.5972710428122, 0.9050152762513541, 5.071883205900706], vcov = [0.07307900945890079 -1.4023086590478741e-11 2.570950382816481e-6; -1.4023086590478741e-11 0.0017579294049063208 -1.9930588768800978e-7; 2.570950382816481e-6 -1.9930588768800978e-7 0.036540143453361824], converged = true)

quap(ml, [160.0, 0.0, 25.0]) ==>
(coef = [154.59725198909285, 0.90501297192992, 5.071871422255509], vcov = [0.07307866985227743 -1.4612147142026734e-11 2.296387802369754e-6; -1.4612147142026734e-11 0.001757921236960678 -2.3250589352437402e-7; 2.296387802369754e-6 -2.3250589352437402e-7 0.03653971897865233], converged = true)

MvNormal(res.coef, res.vcov)
FullNormal(
dim: 3
μ: [154.59725198909285, 0.90501297192992, 5.071871422255509]
Σ: [0.07307866985227743 -1.4612147142026734e-11 2.296387802369754e-6; -1.4612147142026734e-11 0.001757921236960678 -2.3250589352437402e-7; 2.296387802369754e-6 -2.3250589352437402e-7 0.03653971897865233]
)

while script m4.1t.jl works as before.
I actually don't mind at all if we would ask our customers to always provide start values. They do that anyway in the priors.

I'm not sure if we aren't talking past each other. The tmp ~ Normal(μ, 1) was just to show that there might be problems if the models get bigger/hierarchical. But this is easily resolved by taking multiple samples, just like I did in the first version by sampling from the prior.
So basically, there are three possible versions given this model:
@model line(x, y) = begin
    #priors
    alpha ~ Normal(178.0, 100.0)
    beta ~ Normal(0.0, 10.0)
    s ~ Uniform(0, 50)

    #model
    mu = alpha .+ beta*x
    y .~ Normal.(mu, s)

    return alpha, beta, s
end

Specifying a start vector. In this case it could be [178.0, 0.0, 25.0] (the mean values).
The version in the PR that samples from the prior. Requires the return statement. Since it takes the median of a bunch of samples the vector will be approximately [178.0, 0.0, 25.0] as well.
The quap version in this comment. It looks at the distributions and from there calculates (not averages) the identical start vector [178.0, 0.0, 25.0] without the need for a return statement. In fact, it will work when you remove the return statement from the model above, while version 2) will fail.
(Using NUTS to compare results.)

With
d = CSV.read(datadir("exp_raw/Howell_1.csv"), copycols = true)
d2 = filter(row -> row.age >= 18, d)
d2 = d[d.age .>= 18, :]
m = line(d2.weight, d2.height)

all 3 optimizations result in (approximately)
(coef = [113.90338516901606, 0.9045062614138886, 5.071867147465665], vcov = [3.6300435001413702 -0.07906095794838724 0.00034555641375722055; -0.07906095794838724 0.0017572945349224637 -7.530697042737666e-6; 0.00034555641375722055 -7.530697042737666e-6 0.036539573181395675], converged = true)

and NUTS
  parameters      mean     std  naive_se    mcse       ess   r_hat
  ──────────  ────────  ──────  ────────  ──────  ────────  ──────
       alpha  113.7695  1.7799    0.0796  0.1339  184.8117  0.9997
        beta    0.9075  0.0393    0.0018  0.0029  183.2906  0.9990
           s    5.1037  0.2186    0.0098  0.0161  179.0444  0.9985

It seems like this PR might need a rework soon anyways :smile:
https://github.com/TuringLang/Turing.jl/pull/1230

Yeah, after https://github.com/TuringLang/Turing.jl/pull/1230 goes through you should be able to just do
using Turing
using Optim

@model function something()
   some kinda model
end

model = something()

# To account for the prior probability
optimize(model, MAP())

# If you just want maximum likelihood
optimize(model, MLE())

Cool!

@karajan9 Is your decision to hold off further work on quap-turing for now until #1230 is released? If so, I hope it was still worthwhile the effort you put in there! I certainly learned again some stuff and it was good to refresh my Turing memory a bit!

I think so, yes. It sounds like it's going to be merged relatively soon, and in the meantime I hope the current version will do.
When it's here my plan is to use it to provide an interface similar to the book (just a thin layer, should be pretty easy then with all the hard work already done by the Turing team) and integrate it a little more (tests, docstrings, etc.).
Since most of the current code won't be used any more then I don't think working on it is a good investment on time, but I certainly enjoyed making it work (although I'm equally glad we'll get a well thought out version soon!).


Take into account softbreaks

Fixes https://github.com/vieiro/xmark/issues/2
Now the output within the article element is:

Title

This is a line, this is a line, this is a line.

Hi @enrmarc , thanks for your pull request!
What about adding a line break inside the text, like so?
<xsl:template match="cm:softbreak">
    <xsl:text>
</xsl:text>
</xsl:template>

That way the generated HTML has a line break as well (I know this makes no difference in the rendered HTML, but in the source only).

@vieiro that's indeed a better solution.


Cant export gltf object with animations in corrupted blender project

Describe the bug
So the problem appeared when i exported once object at 60 sampling rate and since that whenever i want to export object with animations its giving me error-logs and file isnt exported.
I tried to reproduce this problem from a scratch but it didnt work out exactly like in my corrupted project. Heres how im trying to export gltf https://streamable.com/75mn2 with failure and visible logs.
To Reproduce
Steps to reproduce the behavior:

File -> Export -> GLTF...
Choose:

General (Selected objects, +Y Up),
Meshes (UVs, Normals, Materials),
Animation ( Use Current Frame, Animations, Limit to Playback..., Sampling Rate:1, Always Sample Animations )

Expected behavior
Properly exported gltf file
Screenshots
Video: https://streamable.com/75mn2

.blend file/ .gltf
ObjOnTexture.zip
Version

OS: [Windows10 Home]
Blender Version [2.80 (sub 75), branch: master, commit date: 2019-07-29 14:47, hash: f6cb5f54494e, type:  build date: 2019-07-29, 09:44 AM platform: Windows]

Additional context
Add any other context about the problem here.

You have a strip in NLA without any action on it.
Adding a check on that

Closed by 6ab5c9b


Adding docstrings for PyMaterialXGen*.

This PR adds docstrings in markdown format to the following modules:

PyMaterialXGenGlsl
PyMaterialXGenMdl
PyMaterialXGenMsl
PyMaterialXGenOsl
PyMaterialXGenShader

The docstrings are defined in macros named PyMaterialXGen*_DOCSTRING that use the new PYMATERIALX_DOCSTRING macro that is introduced in #2038 to allow us to place the first and last lines in lines by themselves, rather than starting the docstring contents immediately after the R"docstring( marker.
The PyMaterialXGen*_DOCSTRING macros are placed in separate header files named __doc__.md.h which live side-by-side with their corresponding PyModule.cpp files in which they are included.
Note that the docstrings for individual classes, methods, and functions are to be added in separate PRs.

Before
After

Split from #1567.
Depends on #2038.
Update #342.

Abandoning this PR, following discussion in https://github.com/AcademySoftwareFoundation/MaterialX/pull/2039


Various bugs on export

On firefox (before enabling dev mode)

I tried it and got an error when I clicked on the Share tab: Uncaught TypeError: i is undefined. After that, it seems like the PNG and SVG buttons don't do anything.

On Chrome

It works on Chrome, but also still gives an error: Uncaught TypeError: Cannot read properties of undefined (reading '0'). ... and the downloaded PNG won't load in Windows photo viewer or Paint. ... because its contents are an SVG, despite the .png extension.

Unknown

After I do [export] the first time, it's stuck as the same file

Firefox 99ish?

So if I export as png, update it, and export again I get the first png
Uncaught TypeError: colorvec is undefined

Apparently the cause was that I didn't set up the colorvec for white and black pixels. Fixed now.


ARTEMIS-1541 Make the JDBC Node Manager more resilient on failures

In order to make the JDBC Node Manager more resilient has been implemented:

lowered from SERIALIZABLE to READ_COMMITTED Isolation level on NodeId setup to avoid
ad-hoc (ie per DBMS) handling of failures
recovering with fixed number of retries during the NodeId setup + unrecoverable failure otherwise
unrecoverable fail on exceptions while renewing a lease lock

In addition, in different parts of these critical processes are added more log informations to help diagnose.

@mtaylor Please do not merge it yet: I need to run the CI tests first
@mtaylor @jmesnil How it seems?

@mtaylor The CI tests hasn't shown any regression due to this PR :+1:

@franz1981 nice one.  Did you test this against all our current supported databases?  Our CI currently doesn't cover DB tests.

@mtaylor Thanks! And good point: I need to do further tests with the different DBMS first :+1:

@mtaylor I've performed the failover tests on other DBMS and it is working as expected, but disabling the ThreadLeakCheckRule due to a "known" leak on the Oracle thin driver - the uninterruptible/unstoppable/unbreakable oracle.jdbc.driver.BlockSource.ThreadedCachingBlockSource.BlockReleaser thread:
public void run() {
                while(true) {
                    try {
                        BlockSource.ThreadedCachingBlockSource.BlockReleaser var1 = SOLE_INSTANCE;
                        synchronized(SOLE_INSTANCE) {
                            SOLE_INSTANCE.wait(300000L);
                        }

                        BlockSource.ThreadedCachingBlockSource.releaseFromAllSources();
                    } catch (Throwable var4) {
                        ;
                    }
                }
            }

@franz1981 If you really can't kill it and the thread does not cause interference with other tests you can add an exception to the ThreadCheckerRule.  Which systems did you run the tests against?  It would be good to have this set up in the CI env.

@mtaylor

and the thread does not cause interference with other tests

That's the part I'm sure of it: it seems that only a SIGKILL can close a process that has any of these threads running-

Which systems did you run the tests against?

Do you mean which versions of which DBMS?

It would be good to have this set up in the CI env

Considering that I'm not sure that is possible to kill gently the Oracle one, how do you propose to do that?
eg The Oracle thin driver is not on maven AFAIK

Do you mean which versions of which DBMS?
Yes.

how do you propose to do that?
Just add the driver to the class path and pass in the driver class parameter via system properties.  We do this for other JDBC tests

@mtaylor
The DBMS I've tested are MySQL 5.7 and Oracle 12 c (and obviously Derby!).
What I think it worths to do is improve the tests to make them more configurations with ease but I'm thinking to address it as a separate PR in order to not charge this one with other changes, wdyt?


Not able to initialize the autoware agent

Hello! Thanks for actively maintaining this repo.
I followed all the steps in the documentation, but when the autoware was launched, I got "no-image" on the default "/perception/traffic_light_recognition/camera6/debug/rois" topic. And it seems to be same empty topic for others.
On the side of terminal output, I kept getting this error: [ERROR] [1722718763.870537177] [system.service_log_checker]: /localization/initialize: status code 1 'The vehicle is not stopped.' (/localization/util/pose_initializer_node) while I am trying to set the initial pose for the vehicle.
Here is the full output of one round of "pose initialization":
[service_log_checker-4] [ERROR] [1722718796.722090621] [system.service_log_checker]: /localization/initialize: status code 1 'The vehicle is not stopped.' (/default_ad_api/node/localization)
[service_log_checker-4] [ERROR] [1722718796.722133374] [system.service_log_checker]: /api/localization/initialize: status code 1 'The vehicle is not stopped.' (/default_ad_api/node/localization)
[component_container-64] [INFO] [1722718797.529683160] [planning.scenario_planning.parking.costmap_generator]: waiting for initial pose...
[initial_pose_adaptor-78] [INFO] [1722718797.721587826] [default_ad_api.helpers.initial_pose_adaptor]: waiting response
[initial_pose_adaptor-78] [INFO] [1722718797.927563001] [default_ad_api.helpers.initial_pose_adaptor]: Loaded partial pcd map from map_loader (grid size: 1)
[initial_pose_adaptor-78] [INFO] [1722718798.158808398] [default_ad_api.helpers.initial_pose_adaptor]: modified point: 10.000 20.000 -3.144
[initial_pose_adaptor-78] [INFO] [1722718798.158886730] [default_ad_api.helpers.initial_pose_adaptor]: client call: /api/localization/initialize
[initial_pose_adaptor-78] [INFO] [1722718798.159030022] [default_ad_api.helpers.initial_pose_adaptor]: original point: 10.000 20.000 0.000
[initial_pose_adaptor-78] [INFO] [1722718798.159429069] [default_ad_api.helpers.initial_pose_adaptor]: Send request to map_loader
[component_container_mt-76] [INFO] [1722718798.159105321] [default_ad_api.node.localization]: server call: /api/localization/initialize
[component_container_mt-76] [INFO] [1722718798.159703233] [default_ad_api.node.localization]: client call: /localization/initialize
[pose_initializer_node-31] [INFO] [1722718798.159830340] [localization.util.pose_initializer_node]: server call: /localization/initialize
[pose_initializer_node-31] [INFO] [1722718798.159901283] [localization.util.pose_initializer_node]: server exit: /localization/initialize
Any help is appreciated. Thank you in advance.

Hi @PeterMal0ne
Could you please provide the information on what branch you are using?
Also providing the logfile will be helpful. It should be under the bridge_log and autoware_log folder.

Thanks for the replying!
I am using the main branch, and here is my log file. In addition to what I posted before, I noticed the topics are all up in the autoware container, but there is nothing as I showed in the bridge container.
Here is the bridge.log:
INFO  zenoh_carla_bridge > Running Carla Autoware Zenoh bridge...
INFO  zenoh::net::runtime > Using ZID: c542051a5fe7c408f8b0c09f9e275323
INFO  zenoh::net::runtime::orchestrator > Zenoh can be reached at: tcp/<IP_ADDRESS>:7447
INFO  zenoh::net::runtime::orchestrator > Zenoh can be reached at: tcp/<IP_ADDRESS>:7447
INFO  zenoh::net::runtime::orchestrator > zenohd listening scout messages on <IP_ADDRESS>:7446
INFO  zenoh_carla_bridge::bridge::sensor_bridge > Detected a sensor 'godview' on 'v1'
INFO  zenoh_carla_bridge                        > Actor 204 created
INFO  zenoh_carla_bridge::bridge::sensor_bridge > Detected a sensor 'tamagawa' on 'v1'
INFO  zenoh_carla_bridge                        > Actor 201 created
INFO  zenoh_carla_bridge::bridge::sensor_bridge > Detected a sensor 'traffic_light' on 'v1'
INFO  zenoh_carla_bridge                        > Actor 203 created
INFO  zenoh_carla_bridge::bridge::sensor_bridge > Detected a sensor 'top' on 'v1'
INFO  zenoh_carla_bridge                        > Actor 202 created
INFO  zenoh_carla_bridge::bridge::vehicle_bridge > Detect a vehicle v1
INFO  zenoh_carla_bridge                         > Actor 197 created
INFO  zenoh_carla_bridge::bridge::sensor_bridge  > Detected a sensor 'collision' on 'v1'
WARN  zenoh_carla_bridge::bridge::sensor_bridge  > Collision sensor is not supported yet
INFO  zenoh_carla_bridge                         > Actor 198 created
INFO  zenoh_carla_bridge::bridge::sensor_bridge  > Detected a sensor 'ublox' on 'v1'
INFO  zenoh_carla_bridge                         > Actor 200 created
INFO  zenoh_carla_bridge::bridge::sensor_bridge  > Detected a sensor 'lane_invasion' on 'v1'
WARN  zenoh_carla_bridge::bridge::sensor_bridge  > Unsupported sensor type 'sensor.other.lane_invasion'
INFO  zenoh_carla_bridge                         > Actor 199 created
and vehicle.log:
INFO: listening to server <IP_ADDRESS>:2000
ALSA lib confmisc.c:855:(parse_card) cannot find card '0'
ALSA lib conf.c:5178:(_snd_config_evaluate) function snd_func_card_inum returned error: No such file or directory
ALSA lib confmisc.c:422:(snd_func_concat) error evaluating strings
ALSA lib conf.c:5178:(_snd_config_evaluate) function snd_func_concat returned error: No such file or directory
ALSA lib confmisc.c:1334:(snd_func_refer) error evaluating name
ALSA lib conf.c:5178:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:5701:(snd_config_expand) Evaluate error: No such file or directory
ALSA lib pcm.c:2664:(snd_pcm_open_noupdate) Unknown PCM default
Where I think the ALSA lib should not cause the issue. I also attached the autoware log.
launch.log
(I also have tried the humble branch as it is mentioned more stable in README, but seems autoware is not providing the elder docker version, specifically, this one:  ghcr.io/autowarefoundation/autoware-universe:galactic-20221201-prebuilt-cuda-amd64)
Please let me know if you need more information!

Hi, @PeterMal0ne!,
The information shown in your bridge.log from the bridge container looks correct and sufficient. However, you mentioned that the topics are all up in the Autoware container. Are they successfully displayed in RViz, such as the camera view, localization, and so on?

Hi, @PeterMal0ne!, The information shown in your bridge.log from the bridge container looks correct and sufficient. However, you mentioned that the topics are all up in the Autoware container. Are they successfully displayed in RViz, such as the camera view, localization, and so on?

@habby1012 Thanks for the reply. No, they are not displayed in RViz, and that is the part that confused me. I saw the car spawned in the CARLA but none of the information is displayed in the RViz. And I got this in my Auotware container:
1723950013.3359039 [component_container-64] [INFO] [1723950013.335673204] [planning.scenario_planning.parking.costmap_generator]: waiting for initial pose...
When I am trying to set the initial pose using RViz, it did not set the position of the vehicle.

Hi @PeterMal0ne
Maybe you can try the main branch now. I fixed the docker image issue there.
Also, I didn't see the log of zenoh_bridge_ros2dds.log. Does it run successfully?

I will close the issue. If you have any further questions, feel free to reopen it.


Devided docker.sh script to smaller onse

@partikus @dominikpgs  ping

@jhryniuk rebase please

Is it still relevant? Is it necessary to split docker.sh into few smaller one?
I think refactoring of docker.sh script is necessary but I have no ideas what we could improve.
Let's discuss.

I think that we can divide this file on basic sections.
Form example:

file with configuration of variables,
script with basic functions,
main script which use functions from script with functions and variables from configuration file.

It is just proposition.
I admit that Id divide this scrip on to many files.


HBASE-23564 RegionStates may has some expired serverinfo and make reg…

…ions do not balance.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

+1 :green_heart:
mvninstall
6m  6s
master passed

+1 :green_heart:
compile
1m  0s
master passed

+1 :green_heart:
checkstyle
1m 32s
master passed

+1 :green_heart:
shadedjars
4m 56s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
master passed

+0 :ok:
spotbugs
4m 42s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m 39s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
6m  1s
the patch passed

+1 :green_heart:
compile
1m  5s
the patch passed

+1 :green_heart:
javac
1m  5s
the patch passed

+1 :green_heart:
checkstyle
1m 37s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
5m 14s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
17m 17s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 35s
the patch passed

+1 :green_heart:
findbugs
4m 38s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
233m  7s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 29s
The patch does not generate ASF License warnings.

297m 19s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-930/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/930

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux e39b95b44b68 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-930/out/precommit/personality/provided.sh

git revision
master / 85a081925b

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-930/1/testReport/

Max. process+thread count
5132 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-930/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

I have run into this issue. Thanks for digging in.  One concern I have is why the deviation between serverManager list of online servers and those seen down in RegionStates? The RegionStates list gets updated eventually? Or not? Do you know @binlijin ?  Thanks.

@saintstack
The RegionStates list do not gets updated eventually, and has some expired serverinfo.
https://issues.apache.org/jira/browse/HBASE-23564
TransitRegionStateProcedure (open a region on a server) may run after the server have done ServerCrashProcedure. The ServerCrashProcedure will remove the ServerName entry from serverMap in RegionStates, but later the TransitRegionStateProcedure open a region on this server will add a entry in serverMap in RegionStates for this ServerName, the open will fail and retry, but serverMap in RegionStates leaves some expired ServerNames and have no chance to clear and it will make balance not correct.

@saintstack  any other concern?

I am just trying to follow accounting of Regions by Server and vice versa. This patch bypasses any possible deviation by pushing the current list of online servers. +1.


Flip function has been removed in 0.19.0

Perhaps text should be updated appropriately

I will be updating the text for 0.19 now that I'm done traveling.


add option followAllRedirects to follow post/put redirects

This should close #55.

this looks good but it's a big change that i want to spend some time reviewing.
i will get to it, i promise, just be patient :)

Sure you've gotta lot of stuff on your plate @mikeal, any chance you have some time to revisit this pull request?  This would be very helpful to have.

@kitek I've fixed this typo in my branch but somehow github didn't automatically add the commits to the request. bc057aff
I'll try to get back to this next week and wrap up the remaining loose ends.

Ok, took care of the comments raised in this pull request.

@kitek's comment has now been addressed.
2 months of changes merged in
line added to docs to describe option

Not sure why the maxRedirects test is required for acceptance, it wasn't added in this request. Should we create a new pull request for that issue?


Implement ERC20 Total Supply Interaction for Native Transaction Calculation

Overview
This pull request introduces a new feature in the ERC20 contract class to utilize the total supply of ERC20 tokens for calculating native transaction amounts within our ABCI learning agents. By deriving transaction values from token economics, this enhancement aims to link the agent's transactional behavior with the underlying ERC20 token metrics, ensuring that the native transactions are informed by real-time token supply data.
Changes
Detailed changes include updates to the ERC20 contract interface and agent behaviors to facilitate the calculation and execution of native transactions based on ERC20 token supply metrics:
Modified Files

ERC20 Contracts:

Added totalSupply function to retrieve the total supply of tokens.

behaviours.py:

Introduced get_erc20_totalSupply to fetch the total supply from the blockchain.
Added get_erc20_balance to determine the balance of tokens held by the safe.
Implemented reduce_to_single_digit and sum_of_digits functions to compute the transaction amount.

ERC20 Token Metrics Integration

Data Retrieval: The total supply and balance of ERC20 tokens are retrieved to compute the base transaction value.
Calculation Logic: The quotient of the total supply divided by the safe's balance is processed to extract a single-digit value through digit summation and reduction techniques.
Transaction Execution: This single-digit value is then used as the amount for executing native transactions, directly linking the transactional activity to the ERC20 token's overall distribution and availability.

Impact
This integration enhances the transactional intelligence of our agents by:

Aligning Transactions with Token Economics: Ensures that native transactions reflect the current state of token distribution and supply dynamics.
Increasing Transaction Relevance: Utilizes up-to-date token metrics to inform transaction values, making each transaction more meaningful and contextually appropriate.
Improving Decision-Making Processes: By embedding token supply data into transaction logic, agents can make more informed decisions that are consistent with broader economic indicators.

I've pushed new changes to this pull request that include the implementation of the ERC20_NEW contract package and related updates across various components. Please review the latest commits. Thanks!


Update index.html

The source for this site is hosted on master in the website/ directory. Please make a PR there.


fix: copy extended remarks from sdn_comments.csv

Codecov Report

Merging #529 (12d02bd) into master (3fc5e3f) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

:exclamation: Current head 12d02bd differs from pull request most recent head fa39540. Consider uploading reports for the commit fa39540 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##           master    #529      +/-   ##
=========================================
- Coverage    9.69%   9.67%   -0.03%
=========================================
  Files          39      39
  Lines        2867    2874       +7
=========================================
  Hits          278     278
- Misses       2566    2573       +7
  Partials       23      23


⚠ Bofrost.lu has degraded performance

In 1970a1a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1252 ms

Resolved: Bofrost.lu performance has improved in 27ce4aa.


flexget: 3.9.3 -> 3.9.4

Diff: https://github.com/Flexget/Flexget/compare/refs/tags/v3.9.3...v3.9.4
Changelog: https://github.com/Flexget/Flexget/releases/tag/v3.9.4
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 250714 run on x86_64-linux 1

2 packages built:

flexget
flexget.dist


x86 support

Give support for x86 since the android tv emulator does not work and there is only the x86 version

Sorry, I'm not able to do that. Please use a tablet emulator or real device instead


Binding In whatsapp

Can I bind this to another whatsapp like fm Whastapp, yo whatsapp? Can I do that?

Please letest version WhatsApp

Yes


Support turning groups on/off

Request to add support for toggling an entire group at one time.

Do you mean a switch on card header? Or it is about a switch for group.all_automations, for example?

Well my use case is as follows.
I have a ceiling fan which takes two e12 bulbs. I only ever turn them on at the exact same time. I just recently moved from Wink to native Home Assistant ZWave/ZigBee. In Wink I didn't have a problem because they allow grouping lights which then show in Home Assistant as 1 light bulb. However now I get two light bulbs. I grouped those two light bulbs via Home Assistant groups and I am able to toggle them on/off via the web.
What the group looks like in web (its called Fan Light)

And via HA Client

I see now. Thanks


refactor: rewrite proxy <=> client internals

Description
As mentioned in #770, this is the smallest subset of changes required for the Proxy<=>Client refactoring.

Client methods don't trigger the fetch() anymore, but instead hands out all call parameters to the Proxy
only list and get operations are supported
Got rid of the weird & confusing splitArgs() things.
Our actions sets now contain the action "name"

Tested all the screens migrated to the new API so far, every one of them still work without changing a single char.

Coverage increased (+0.3%) to 43.282% when pulling 911b5618b93fab5c3b4f730f9b63d2dcd7d3d88c on machour:refactor/proxy-client into 4b32a92b03be406777da2a6183dd5514fc65c99c on gitpoint:master.


Add English json-file

There are errors in the JSON and values are not from the original server, I added my own version in 950b403.


satisfies support in JSDoc

Suggestion

🔍 Search Terms
satisfies, jsdoc
✅ Viability Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
It would be fantastic to have satisfies support in jsdoc. This was previously discussed in the satisfies PR (with the suggestion to open a separate tracking issue) and in the 8/17/2022 Design Meeting.
There was a discussion about using /** @type {satisfies T} */, but I would love to see TS be more opinionated on this and go for /** @satisfies {T} */.
Two main reasons:

more straightforward in intent and keeps already-lengthy jsdoc lines from getting even longer
allows the same functionality as ts files if (for some reason) you really want to include a type annotation and a satisfies clause. There aren't great reasons to do this, but the fewer functionality differences between ts and jsdoc the better.

The design meeting mentioned:

We try not to add new tags that aren't doc'd in the usejsdoc site.
Ship has sailed with TS-specific syntax in JSDoc, right?

But: unknown tags are common in the wild and jsdoc (the utility) has allowUnknownTags set by default, so @satisfies information isn't included in the generated docs, but it also doesn't cause any errors. Parser-wise, it should also be relatively easy to support in the future, if there's a desire.
The syntax ship has definitely sailed, in contrast. TS hasn't added tags before AFAIK, but common TS-specific syntax is already often not parsed and causes errors (e.g. /** @type {{name: string}[]} */ const arr = [];).
📃 Motivating Example
/** @typedef {"red" | "green" | "blue"} Colors */

/**
 * Ensure that we have exactly the keys from 'Colors'.
 * @satisfies {Record<Colors, unknown>}
 */
const favoriteColors = {
    red: 'yes',
    green: false,
    blue: 'kinda',
    platypus: false,
//  ~~~~~~~~ error - "platypus" was never listed in 'Colors'.
}

💻 Use Cases
satisfies in jsdoc!

I think /** @satisfies {Type} */ is my preference as well.

@DanielRosenwasser Are you planning to include this feature in the 5.0 milestone?


Update case-insensitive to 1.3.0

Updates org.typelevel:case-insensitive from 1.2.0 to 1.3.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "case-insensitive" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.typelevel", artifactId = "case-insensitive" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #870.


[Question] Pressing backspace moves the cursor of another user

I integrated slate-yjs with @udecode/slate-plugins and the problem I've encountered is that when one user presses backspace or replaces some selected text the cursor of another user is moved to the position of the first user's cursor. What could be the problem?

Here is the short video of this issue:
https://devgeni.github.io/slate-yjs-udecode.mp4

@devgeni see the fix linked in the PR above


Uncaught Exception: Command failed:<PHONE_NUMBER>

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.0
Time: Mon Mar 28 2016 08:06:53 GMT-0600 (Mountain Daylight Time)
Error:
Command failed:<PHONE_NUMBER>
Error: Command failed:<PHONE_NUMBER>
System.AggregateException: One or more errors occurred. ---> System.Net.WebException: The remote name could not be resolved: 'update.googleplaymusicdesktopplayer.com'
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Net.WebClient.GetWebResponse(WebRequest request, IAsyncResult result)
   at System.Net.WebClient.DownloadBitsResponseCallback(IAsyncResult result)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<LogIfThrows>d__50`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.FileDownloader.<DownloadUrl>d__14.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerS
...Some of the stack is missing

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #213


better explain project files and the tools used

Thanks for this nice starting point for creating Electron/React desktop apps! I guess a number of the many users starting from this boilerplate may be electron and/or react (or even JS) beginners and are using it, because it provides a valuable preselection of tools and configurations. It would be great for such users to read about the choices made, e.g. why the project structure is like it is now, what it's individual files are used for, and why this very technology stack has been selected.

Lets merge this into https://github.com/chentsulin/electron-react-boilerplate/issues/530

But #530 is this one?!

My bad. I meant #518


ubuntu image with tensorflow preinstalled

Hi every one
I tried install tensorflow-gpu on my pc I failed i tried all possible ways I failed to build it from source I failed to run it from docker image
can on you share ubuntu image with tensorflow-gpu installed on it i iam despaired really need to run tensorlow-gpu
and I will be very thankful

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

Did you try using out docker containers?
https://www.tensorflow.org/install/install_linux#InstallingDocker
Unfortunately, outside docker we cannot provide full ubuntu system images with tensorflow, due to licensing restrictions.


Tricky clipmenu behavior with google slides

I'm sorry for rather a fuzzy title, but here is what is happening in Google slides.
If I select any item on the slide which contains text and have some other than default formatting and do C-C, C-P its formatting is reset to the default one. In order to preserve it I have to press C-C couple of times with some delays in between. Usually it take me two to four tries,
When I disable clipmenu, it works as expected.
Is there some workaround?

Or I wonder if we can detect fancy clips and not persist those... hmm.

Should be fixed in clipmenu 6.0.0, thanks!


Add note on compiling in idea instructions

[Current coverage][cc-pull] is 28.58%

Merging [#51][cc-pull] into [master][cc-base-branch] will increase coverage by 9.11%

@@             master        #51   diff @@
==========================================
  Files           575        575
  Lines         15440      15440
  Methods           0          0
  Messages          0          0
  Branches       1621       1621
==========================================
+ Hits           3007       4413   +1406
+ Misses        12156      11027   -1129
+ Partials        277          0    -277

Powered by Codecov. Last updated by [1576203...a621ee5][cc-compare]
[cc-base-branch]: https://codecov.io/gh/spotify/heroic/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/spotify/heroic/compare/157620339aa5c1ed911d42b03423b056985af307...a621ee5944a69c81daec6fd31c9744da2c6a564f
[cc-pull]: https://codecov.io/gh/spotify/heroic/pull/51?src=pr


[Bug] Use stable clippy instead of nightly

Component
Other (please provide more details)
What version of Alloy are you on?
No response
Operating System
None
Describe the bug
Nightly clippy includes unstable and changing lints and results in random CI breakage

Sorry I disagree, if CI doesn't break now it will break when a stable release happens, it's not really that much different.

happening once at known intervals is way, way better than happening randomly


File tree display

ACs

Show folders from db
Show notes from db
UI

Folder with type icon, click to expand/collapse
Folder with add button, ignore click for now
Notes with type icon, click to show
Scrollable list

[ ] Show folders from db
[ ] Show notes from db
[ ] Folder with type icon, click to expand/collapse
[ ] Folder with add button, ignore click for now
[ ] Notes with type icon, click to show
[ ] Scrollable list


Fix/Update tx_utxos

Describe the bug
Currently, the spec for tx_utxos re-uses fields from tx_info , but that's incorrect - since tx_info provides much more information.
This went through unnoticed until recently, this caused an issue downstream (cardano-community/koios-java-client#120).
Depending on reply on the linked issue, we'd need to:

[x] - fix the specs at minimum, and ask users relying on output fields datum/inline_datum/reference_script to use tx_info instead (#188)
[ ] - Since these were shipped as release, start providing those fields in input/output by adapting the tx_utxos endpoint to add those fields, and redirect actual query to a new endpoint utxo_info
[ ] - Add/align endpoints for address_utxos, credential_utxos and account_utxos

Adding asset_list mapping creates a 1-to-many array adds a bit of cost (tho it may not be such a big loss to sub-array since the largest mapping would be ~1000). However - this is the only field that adds a join that's not based on single-value primary key lookup on table external to tx_out.
To have a good middle-way that checks both the boxes, an alternative would be to add asset_list to the schema, but also provide a parameter _assetscheck to allow disabling this field in the response. This would mean we could align the UTxO lookup using various objects to return same UTxO map and filter asset out using a boolean parameter if needed - allowing for quicker response times for those who simply need list of UTxO hashes, as against the information on them (eg: to specify inputs while building a tx)

@edridudi - This is now 'applied' in #227 , can we confirm the description in issue would resolve the conflict/requirements?

Closed via #238


Suffix return NaN

Hi, for some reason if I add prefix "km" I get a NAN value displayed. Others suffix work correctly

Prefix "THB " returns NaN too

This issue is still not receiving attention


cookie in bot framework

Hi,
I want to use webchannel for out bot. this we'll use on our sharepoint.com homepage.
We have certain microservices, token based authentication.
I want to access a cookie created by our sharepoint online website, and use username in that to find token and use that token for webservices.
Is it possible for webchannel bot?
I am thinking to store token in BotData userdata.
I dont know how will I access that cookie. If i get hold on it, i can find token.
Pls suggest.
Regards,
Ashay

Take a look at the WebChat repo, you may be interested in using the backChannel feature to send additional information: https://github.com/Microsoft/BotFramework-WebChat#using-the-backchannel

I'm closing this issue, since the question has been answered.


InstallToVSCode: remove System.Diagnostics.StackTrace dependency

InstallToVSCode declared a dependency on System.Diagnostics.StackTrace. As
far as I can tell, this is not needed, and having it breaks Dev15 versions
of clrdbg as it pulls down an older version of System.Reflection.Metadata.
This removes the dependency.

LGTM

@rajkumar42 can you take a quick look?

:+1: