Thread – Thread parent message not updated in LHN after editing in group chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.26-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4904603
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Log in
Open group chat
Send a message
Reply in thread and send a comment
Edit parent message
Observe the thread in LHN

Expected Result:
Thread parent message updated in LHN
Actual Result:
Thread parent message not updated in LHN
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/541cb165-a19a-4335-99c8-5a6596e295ec
View all open jobs on GitHub

dupe

Looks like it, thanks @FitseTLT!


Unhandled rejection Error: timeout reached while waiting for rooms response

I'm running a test where I have a socket join a room on a remote node. When the socket gets disconnected I would like to remove it from any rooms it was in. So whenever I get a 'disconnect' event I try to call the 'remoteLeave' for any rooms the socket was in, but I get the above error.
I'm curious whats the best way to handle this?

Hi! Rooms are left automatically upon disconnection, so you shouldn't have to worry about that.
Reference: https://github.com/socketio/socket.io/blob/master/docs/API.md#socketleaveroom-callback

Oh ok awesome I wasnt sure


Enable over PSSession

See https://github.com/PowerShell/PowerShellEditorServices/blob/main/src/PowerShellEditorServices/Services/Workspace/RemoteFileManagerService.cs as how PSEditorSevices does it in VSCode

@SteveL-MSFT Should we rename this module so that it doesn't conflict with the builtin psedit? I didn't realize that existed...

@adamdriscoll I believe the "built-in" psedit only exists in ISE and when using PS remotely (it's created on the remote session).  For the module name, it may make sense to name it more specific like: IronmanSoftware.PSEdit.  For the alias itself, you should probably pick something else or detect if the name is already used and not create the alias.

Enabling this over PSSessions would be amazing. I would use this tool non-stop if I could use it in a similar way to how I use nano in remote ssh sessions.

Hi, any update regarding this ?
I would be so please to use this module because for now the only solution that I have is to use VSCode only which is not really convenient.


Make it easy to make a hardware backed soft signal

Often the soft signals on a device may be derived from the value of real hardware signals. As such, when the soft signal is read it should read the real hardware and provide its value based on this. This can be done by having the device optionally provide a readback function when creating the signal.
See https://github.com/DiamondLightSource/dodal/issues/782 for more detail on the usecase and https://github.com/DiamondLightSource/dodal/pull/789 for a solution.

This sounds quite similar to https://github.com/bluesky/ophyd-async/issues/525 , maybe good to add your use-case to that issue as well?

Thanks, I agree, closing this as a duplicate.


Redistribute PortionOfMatter

closes #87: Everything under PortionOfMatter, except Air and Water, should be deleted and Fuel redristributed to EnergyCarrier
I moved all subclasses of Fuel to EnergyCarrier, except for the following that I classified as PortionOfMatter:

Peat
Coal and subclasses
OilAndPetroleumProducts
SolidBiomass
Biogas

Can some economic expert tell me if I classified the classes right (as PortionOfMatter or Fuel)?

@l-emele I think this requires an energy expert instead. Can you please have a look at the approach @akleinau took?
From my point of view the above mentioned should also be under EnergyCarrier, as they are a fuel that can be burned.

I put things like coal under portionOfMatter because in my understanding (which is very basic so maybe I‘m wrong) to be a Fuel is a property, a disposition. But to be coal is not a property, just coal is coal, coal is an independent material entity.

Like: Coal has the disposition to be a fuel but its at its heart just a sedimentary rock

After reading wiki I‘m not sure anymore why fuel is not just a portionOfMatter like it was at the beginning. https://en.wikipedia.org/wiki/Fuel. Its described as a material and a material is a portion of matter.

Or else what is the difference between water and coal that one is a portionOfMatter (independent entity) and the other a disposition (specifically dependent entity)?

I think what we need to do is think from the perspective of energy system modeling and how coal etc. would be viewed from this point of view. Maybe it helps to remember our discourse on the correct definition of "wave" from a physics point of view in our previous meeting and how in the ontology we may need to be a bit more flexible in how to represent terms, so that the energy system modelling community can relate.
To my understanding, for achieving the latter, coal would be categorised as an EnergyCarrier that can be burnt to generate energy.

To my understanding, for achieving the latter, coal would be categorised as an EnergyCarrier that can be burnt to generate energy.

An EnergyCarrier that can be burnt to generate energy is a Fuel and that is why coal has to stay a fuel.

Btw, for me it is unclear why we need PortionOfMatter at all. Also Water and Air can be described with has_normal_state_of.

To my understanding, for achieving the latter, coal would be categorised as an EnergyCarrier that can be burnt to generate energy.

An EnergyCarrier that can be burnt to generate energy is a Fuel and that is why coal has to stay a fuel.
Yes, I did not intend to say that it should not stay a Fuel, but that it should be moved, as a part of Fuel which is a subclass of EnergyCarrier.

We have EnergyCarrier not as a class of objects right now but as the property of an object. Thats why it is a specifically dependent continuant right now, it is a property that depends on some object to have it.
Coal is not a property, so in the current context it cant be an EnergyCarrier, it can just have the property/ disposition to be an EnergyCarrier.

So either Coal is an object with the disposition to be an EnergyCarrier or we move EnergyCarrier to material entity that its not a property anymore but a class of objects.

@stap-m pushed the idea of EnergyCarrier being a disposition, could you give some input?

Yes, in the "real world"  EnergyCarrier is a disposition.
I think if I were looking from a modeller's perspective however, I would more naturally understand EnergyCarrier as a class of objects (which I use for modelling inputs to some kind of transformation process).
In the end both would work, but the latter would come more naturally for me at least.

I agree with @han-f that from a modelling perspective, it makes more sense to see Coal etc. as instances of EneryCarrier, even if it looks "wrong" from a more physical perspective.
In general, the ontology should reflect the way the energy system is described in these very abstract models, with the terminology that is used in the community.

ok, then I‘ll delete the changes of this request and propose:

Coal etc remain subclasses of fuel
fuel gets subclass of EnergyCarrier
EnergyCarrier gets subclass of one of the material entity subclasses (new issue for that)

because if we make fuel a subclass of energycarrier we have to move energycarrier to respect the bfo definitions

I think if I were looking from a modeller's perspective however, I would more naturally understand EnergyCarrier as a class of objects (which I use for modelling inputs to some kind of transformation process).

No. EnergyCarrier (and Fuel as its subclass) should be bfo: dispositions and 'coal' etc. should not be subclasses of 'fuel'. Instead coal gets the relation 'has_disposition'.
Like that, we can keep monohierarchy. If we hurt the monohierarchy, we probably can find some more 'things' of which e.g. coal might be a subclass of (we had 'solid' before).

@stap-m If we change all fuels to dispositions, how do we then keep the internal structure of Fuel? We definitely need a structure within the fuels.

Btw, for me it is unclear why we need PortionOfMatter at all. Also Water and Air can be described with has_normal_state_of.

In this PR #58 @fabianneuhaus described why PortionOfMatter makes sense. I am very much in favor of keeping it.
E.g. Air and water have the 'is_a' relation with portionOfMatter (i.e. they have to be subclasses of SOMETHING) and additionally have the has_normal_state_of_matter relation with a stateOfMatter. And will have maybe even more relations in future.

@stap-m If we change all fuels to dispositions, how do we then keep the internal structure of Fuel? We definitely need a structure within the fuels.

Yes, then we need to restructure. Subclasses like biofluel and fossil fuel could be useful. Biogas on the other hand should get the relation 'has_disposition' to fuel etc.
And the restructuring of fuel should definitely get an own issue.

After reading wiki I‘m not sure anymore why fuel is not just a portionOfMatter like it was at the beginning. https://en.wikipedia.org/wiki/Fuel. Its described as a material and a material is a portion of matter.

@l-emele  I am suggesting the following definition:
Fuel is an EnergyCarrier that releases energy in form of heat or work, by chemical reaction with other substances.

Fuel is an EnergyCarrier that releases energy in form of heat or work, by chemical reaction with other substances.

I am fine with this definition.

So: the problem is fuel is seen from an energy modelers perspective as a class of objects but because many fuels like coal can also be used or found in non-fuel-contexts they would have to be classified as something else too for every other context, which hurts the monohierarchy.
Our (hopefully soon) new energy expert (whose github name I dont know yet) had a suggestion yesterday: To use defined classes.
Defined classes are a way to give classes more then one superclass without hurting the monohierarchy principle. This way all the subclasses of fuel can be subclasses of fuel and portionOfMatter at the same time. Fuel can stay a class of objects.
The idea:

We make every subclass of fuel a subclass of portionOfMatter instead first
We give them the disposition "EnergyCarrier" in a way that makes clear they are energy carriers that release energy as heat energy (can be burnt)
We create a defined class "Fuel" that is a subclass of PortionOfMatter that is defined by "every subclass of this class has the disposition EnergyCarrier and releases energy as heat energy". In other words we define the class through the relation we gave in 2.

This way objects like coal are portionsOfMatter and are classified because of the way they react as fuel. So they are both.

The consequences on the example of coal:

coal is a portionOfMatter
coal is an EnergyCarrier that releases energy as heat energy
coal is therefore a fuel

The consequences for fuel:

fuel is a class of objects (or object aggregates to be correct) and not a disposition
fuel is a subclass of PortionOfMatter
fuel can keep its inner hierarchy

@akleinau I like the possibility of defined classes.
Anyway in this case, it makes so much sense to make fuel a disposition and subclass of energyCarrier. Please read the end of the discussion. @l-emele and I agrred on a definition for fuel, see comment above.

I read the end of the discussion but didn't got the impression that everyone agreed to make it a disposition. I got the impression that some people wanted it to be a class of objects, others a disposition and there was no agreement yet. I like the definition though

To biogas: can we formulate all those requirements yet?

@han-f would it be a problem for you if we make fuel a disposition?
(We can still if needed create a defined class or just a query/ a view to show all the fuels in one class hierarchy)
Btw we'd have to make Biofuel and SolidFossilFuel then dispositions too

@han-f would it be a problem for you if we make fuel a disposition?
(We can still if needed create a defined class or just a query/ a view to show all the fuels in one class hierarchy)

@l-emele and I would be ok with making fuel a disposition as long as we can show all fuels in one class hierarchy.
We also think that we want to treat all fuels the same, or would there be a reason to treat biogas, coal, peat, solid biomass differently to the other fuels?

can a specific biofuel be more than a fuel? Like coal as a specific fuel is more than a fuel?

All of the fuels can be more than a fuel in real. But is that relevant here?

yes, because I'm wondering. If we define fuel as a disposition, should we define biofuel as a disposition too? What are the differences between biofuel and fuel?

We have to draw somewhere the line through the hierarchy between disposition and object

What are the differences between biofuel and fuel? Compared to e.g. Fuel and Biomass

A biofuel is a fuel that has a biogenic origin. Or it can be described as biomass that is used as a fuel but a class biomass currently does not exist. If you are talking about the class SolidBiomass: this can be renamed to SolidFuel if it solves the issue.

We have to draw somewhere the line through the hierarchy between disposition and object.

What exactly is then your idea how the existing structure within fuels will be represented?

Coal is a portionOfMatter with the disposition fuel.
Coal is subclass of SolidFossilFuels.
SolidFossilFuels is a subclass of Fuel. Fuel is a disposition.
What is SolidFossilFuels? A disposition or an object aggregate?

We have to ask that for everything.

What is SolidFossilFuels? A disposition or a portionOfMatter?

SolidFossilFuels is a fuel.

so, I'm working on implementing this and implemented this structure now for SolidFossilFuels.
I created a defined class (FuelObjects) that will at the end be the fuel hierarchy like it was at the beginning.

which means: coal is a portionOfMatter, but the reasoner will create the old fuel hierarchy again.
I'd continue sorting the other subclasses of fuel too in dispositions and object aggregates. Is this a solution everyone is content with? (The name of the defined classes is open for discussion)

Asserted view (without reasoner)

So then we also need BioFuelObject, NaturalGasFuelObject, OilAndPetroleumProductsFuelObject and WasteFuelObject, right?

yes, if those are dispositions without the "object" suffix

my current idea. What are NaturalGas and its subclasses? All dispositions?

I'm unhappy with the class "OtherLiquidBiofuels" because I read its not good practice to have such classes (link) and I agree. It's not a type itself, its a collective term. An ontology should just contain types. So I'd like to delete it, does anyone disagree?

So, first I renamed "FuelObject" to "FuelPortion" but actually "PortionOfFuel" would be more coherent. I'm just a bit reluctant to do that because than we'd have the way too long name "PortionOfRenewableMunicipalWasteFuel". Actually we have to add Portion or PortionOf to everything in PortionOfMatter (Coal, Water, etc). What do you think? Portion or PortionOf?
Second, I have all the fuels now in their old order in the class FuelPortion, shown in the picture above (inferred). However in the asserted view they are separated now in:

PortionOfMatter: everything that is used as a fuel
disposition fuel: every subtype of the disposition fuel.

Thats why I treated NaturalGas a bit different than SolidFossilFuels. SolidFossilFuels is by its name a subtype of the disposition/property fuel. NaturalGas is used as a fuel and not a disposition/property.

(inferred) I now have exactly the former fuel hierarchy (just with a bit changed names because of PortionOf prefix and with deleted "other" classes because they are not good practice) implemented through defined classes. Fuel is now a disposition. I ordered everything as best as I can in dispositions and objects that have those dispositions. Can you please review this again?

This pull request is just for restructuring, I'll make a second updating the definitions afterwards to not overload this one.

This is a very complex topic. I'd prefer to discuss this at the upcoming Telko before reviewing.

We decided at the telco to approve and merge this request. Can someone please review it so I can do that?

I still have to fix some merging problems, then I'll merge


Always run router in host network namespace

The router container falls back to bridged networking if WEAVE_NO_FASTDP is set as a precaution against any unforeseen issues with running in the host network namespace. No problems have arisen, consequently this complication can be removed.
Related to #1546 and #1589.

Closed by #1930.


it don't install

npm install ->

# npm resolution error report

While resolving: undefined@undefined
Found<EMAIL_ADDRESS>node_modules/typescript
  dev<EMAIL_ADDRESS>from the root project

Could not resolve dependency:
peer<EMAIL_ADDRESS>|| 4.7.x || 4.8.x || 4.9.x || 5.0.x || 5.1.x || 5.2.x || 5.3.x || 5.4.x" from<EMAIL_ADDRESS>node_modules/typedoc
  dev<EMAIL_ADDRESS>from the root project

Fix the upstream dependency conflict, or retry
this command with --force or --legacy-peer-deps
to accept an incorrect (and potentially broken) dependency resolution.

"The error message you're seeing indicates that there's a version conflict between the TypeScript version you have installed (5.5.4) and the versions that<EMAIL_ADDRESS>is compatible with<EMAIL_ADDRESS>only supports TypeScript versions up to 5.4.x, but you've installed 5.5.4.
Here are a few ways you can resolve this issue:
Option 1: Downgrade TypeScript
[..]
Option 2: Update TypeDoc (if possible)
[..]
Option 3: Use --force or --legacy-peer-deps
[..]
Option 4: Use pnpm or yarn
[..]
Choose the option that best fits your project's needs and stability requirements. Let me know if you need further assistance!"
... none of which slap

I've just bumped typedoc, can you try again?

thank you! However now I get
npm install
npm error code EUNSUPPORTEDPROTOCOL
npm error Unsupported URL Type "workspace:": workspace:*

thank you! However now I get
npm install npm error code EUNSUPPORTEDPROTOCOL npm error Unsupported URL Type "workspace:": workspace:*

Use "pnpm" instead?

Oops, didn't catch that. Thanks, it worked <3


KBC-2515 add docs for project purge auditlog event

connection https://github.com/keboola/connection/pull/3659

👁


Upcoming Release Changes

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#226
96f0bb3
Thanks @ardatan! - Workaround for missing type field and other
response types

<EMAIL_ADDRESS>Patch Changes

Updated dependencies
[96f0bb3]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies
[96f0bb3]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies
[96f0bb3]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies
[96f0bb3]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies
[96f0bb3]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies
[96f0bb3]:

<EMAIL_ADDRESS>

🚀 Snapshot Release (rc)
The latest changes of this PR are available as rc on npm (based on the declared changesets):

Package
Version
Info

fets
0.2.4-rc-20230608164315-498e345
npm ↗ unpkg ↗

💻 Website Preview
The latest changes are available as preview in: https://5f894017.fets.pages.dev


where is the pre-trained model?

when run the test code, it seems that we need the pre-trained model. Will you release the pre-trained model?

https://drive.google.com/open?id=1Tmo2GU6KXwKoGlY_Epy5cae9dApk5AdL

Please download our pretrained model at Google Drive and put this model in "./examples/DAF-Net/snapshot/."


Closes #2325.
Screenshots

Before

After

Changelog

:cl:

tweak: Ejected casings now have much more variation and look cooler

which dumbshit coded guns?

which dumbshit coded guns?


Defers checksums and uploads to last step

Before, the exec jar created by spring boot had incorrect MD5s. This was
likely due to interaction with the install plugin, which when deferred
creates correct checksums. This also defers uploads, while we're at it.
Fixes #1819

will verify this fixed here http://oss.jfrog.org/oss-snapshot-local/io/zipkin/java/zipkin-server/2.3.2-SNAPSHOT/ once it publishes

derp.. no checksums on snapshots


bottles: update version + bugfixes

Motivation for this change
Previous state of bottles was not able to run any wine related commands beside installation of a wine prefix.
As it seems staging-next is stuck for some time and I want to get rid of that changes I want to merge this PR into staging.
see #132342
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"

 asbachb@nixo-t14s  ~/dev/src/nix/nixpkgs   update/bottles  nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0
$ git worktree add /home/asbachb/.cache/nixpkgs-review/rev-ecb3c84196017d3ea37c6e7ec9c56a82d237b719-dirty-1/nixpkgs 739f0b22b74a35f65e5b862108db4d4380feecc8
Preparing worktree (detached HEAD 739f0b22b74)
Updating files: 100% (27423/27423), done.
HEAD is now at 739f0b22b74 elixir_ls: 0.7.0 -> 0.8.0
$ nix-env --option system x86_64-linux -f /home/asbachb/.cache/nixpkgs-review/rev-ecb3c84196017d3ea37c6e7ec9c56a82d237b719-dirty-1/nixpkgs -qaP --xml --out-path --show-trace
No diff detected, stopping review...
$ git worktree prune

[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[ ] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_review

/status needs_reviewer

This was already reviewed in https://github.com/NixOS/nixpkgs/pull/132342

@ofborg eval

@SuperSandro2000 Can you have a look? I guess evaluation was successful.


Fix mermaid option

I made a typo in my #195 PR earlier, which prevent the rendering of the Mermaid dependency tree. This PR fix this issue.
Sorry for the noise. 🙇

All tests are passing, this PR is ready to be merged upstream and release.

BTW, this fixes an upstream issue that starts to be noticed by the community:
https://github.com/tox-dev/pipdeptree/issues/197
https://github.com/tox-dev/pipdeptree/issues/198
And thanks @gst for the approval and test run!


set buildHtml root attrs to "display: contents;"

Is your feature request related to a problem? Please describe. 🤔
One kind of undocumented syntax with buildHtml is that the content sit in a div. This makes the built html content different from the literal content of the DSLs body in a couple of significant ways. First, it imposes mandatory line breaks pretty much all the time. Second, it disrupts parent-child relationships between tags.
Describe the solution you'd like 💡
"display: contents;"  is the CSS setting made to get around this and is designed with computer-generated html in mind where divs might be written without any semantic intent. While it may be nice to have the content enclosed in a div automatically in some circumstances, it's much harder to opt-out of being in a div once it is baked in to buildHtml and buildHtmlProcedure. It seems like a div with "display: contents" would change the existing code minimally while making it easier to reason about and use [build/thunk]Html data.

Solved in the next commit via deleting unnecessary div elements.


[Bug] ConvexHullTrickAddMonotone

Hi, there is a bug in ConvexHullTrickAddMonotone.
The check function will return ab * bc >= 0 and it uses division to avoid data overflow. But for integers, it cannot be easy just to use division to compare because it will be rounded down.
There is a small case to hack this test
5 1 9 0 6 2 5 -2 -9 0 -6 8 1 0
The test will return 0. However, the answer is -2.
This is my implementation:
    bool check(const Data &a, const Data &b, const Data &c) {
        // return (b.F-a.F)*(c.S-b.S) >= (b.S-a.S)*(c.F-b.F); return ab * bc >= 0
        if (b.first == a.first || c.first == b.first) {
            return sgn(b.first - a.first) * sgn(c.second - b.second) >= sgn(c.first - b.first) * (b.second - a.second);
        }
        // return (b.second - a.second) / (a.first - b.first) >= (c.second - b.second) / (b.first - c.first);

        Data ab = b - a;
        Data bc = c - b;

        if (sgn(ab.first) * sgn(bc.second) != sgn(ab.second) * sgn(bc.first)) {
            return sgn(ab.first) * sgn(bc.second) >= sgn(ab.second) * sgn(bc.first);
        }

        Data left = {bc.second / bc.first, bc.second % bc.first};
        Data right = {ab.second / ab.first, ab.second % ab.first};

        if (left.first != right.first) {
            return left.first >= right.first;
        }

        return left.second * ab.first >= right.second * bc.first;
    }

I think the check function may not be very common. Because I also use multiply in the last line, and it may be overflows.

Thank you for reporting this. Maybe we need to use the floor function rather than truncation 🙇♂

Thank you for reporting this. Maybe we need to use the floor function rather than truncation 🙇♂ https://noshi91.hatenablog.com/entry/2021/03/23/200810

Thank you. This article is helpful for me.


Small update to fix issue #40

This answer on Gurobi Google Groups (https://groups.google.com/forum/#!topic/gurobi/8JiUTt7CcMg) suggests that Gurobi 'Slack' variables are signed based on the 'Sense' attribute of the constraint. This may have led to a small sign error in getconstrsolution which I attempted to fix. I hope this works :)

Thanks!

@adelarue, thanks!
Would you mind adding a small test case for this to the MathProgBase tests? It could just be your example from #40 (but using the loadproblem! interface instead of JuMP).


Unable to retrieve ID token for Keycloak id_token_hint logout

Hi, I'm encountering difficulties obtaining a valid ID token that I would like to use for Keycloak logout. The ID token is not included in the user object, and attempting to set exposeIdToken: true in the provider configuration results in an infinite loop after login.
Current configuration:
oidc: {
  defaultProvider: 'keycloak',
  providers: {
    keycloak: {
      audience: 'account',
      baseUrl: 'http://localhost:8080/realms/dzg',
      redirectUri: 'http://localhost:3000/auth/keycloak/callback',
      clientId: 'xx',
      clientSecret: 'xx',
      authorizationUrl: 'http://localhost:8080/realms/dzg/protocol/openid-connect/auth',
      tokenUrl: 'http://localhost:8080/realms/dzg/protocol/openid-connect/token',
      logoutUrl: 'http://localhost:8080/realms/dzg/protocol/openid-connect/logout',
      logoutRedirectParameterName: 'post_logout_redirect_uri',
      scope: ['openid', 'profile', 'email'],
      responseType: 'code',
      exposeAccessToken: true,
      grantType: 'authorization_code'
    }
  },
  middleware: {
    globalMiddlewareEnabled: true,
    customLoginPage: false
  }
}

@Degoya-Studio did you manage to find a way to get the idToken value somehow? Im struggling with the same situation. I want to logout the user (end session) without the logout confirmation screen. Ive searched all around and indeed you need to provide id_token_hint with post_logout_redirect_uri parameter to get this behaviour from keycloak. Ive tried a number of things to try and get the idToken but no luck so far.
When I add exposeIdToken: true to my config I dont get infinite redirecting behaviour but I do get:
Set-Cookie header is ignored in response from url: https://localhost:3000/auth/keycloak/callback. The combined size of the name and value must be less than or equal to 4096 characters.

I assume the logic breaks here and no userInfo is passed nuxt-oidc-auth module because the user data is null. Perhaps this could be the issue? Thanks for your time 🍻
My configuration:
oidc: {
    providers: {
      keycloak: {
        baseUrl: process.env.NUXT_OIDC_PROVIDERS_KEYCLOAK_BASE_URL || '',
        clientId: process.env.NUXT_OIDC_PROVIDERS_KEYCLOAK_CLIENT_ID || '',
        clientSecret: process.env.NUXT_OIDC_PROVIDERS_KEYCLOAK_CLIENT_SECRET || '',
        redirectUri: process.env.BASE_URL + '/auth/keycloak/callback',
        logoutUrl: process.env.NUXT_OIDC_PROVIDERS_KEYCLOAK_BASE_URL + '/protocol/openid-connect/logout',
        logoutRedirectParameterName: `client_id=${process.env.NUXT_OIDC_PROVIDERS_KEYCLOAK_CLIENT_ID}&post_logout_redirect_uri`,
        tokenUrl: process.env.NUXT_OIDC_PROVIDERS_KEYCLOAK_BASE_URL +'/protocol/openid-connect/token',
        pkce: true,
        grantType: 'authorization_code',
        exposeAccessToken: true,
        tokenRequestType: 'form-urlencoded',
        responseType: 'code id_token',
        nonce: true,
        scopeInTokenRequest: true,
        scope: ['openid'],
      },
    },
    middleware: {
      globalMiddlewareEnabled: false,
    },
  },

@Mattymoo007 I switched to nuxt-auth-utils and there it worked fine for me. I think you're getting this issue, because the library can't store id_token plus the other tokens you're using in the cookies. You need to somehow only save the id_token without refresh and access token, I reckon.

@Mattymoo007 I switched to nuxt-auth-utils and there it worked fine for me. I think you're getting this issue, because the library can't store id_token plus the other tokens you're using in the cookies. You need to somehow only save the id_token without refresh and access token, I reckon.

This is a general problem of cookies/h3 and would impact nuxt-auth-utils in the same way if they would expose id or access token via the session.
A general problem with Keycloak is that is promotes a lot of bad practices towards JWT usage such as storing non-essential user data in tokens instead of using the tokens for auth only and then requesting additonal user information from a db/service/userendpoint etc..
Try to not use exposeAccessToken or exposeIdToken at all and try to reduce the load of these on Keycloak side. If you absolutely need one of them try to only expose that one.

@Degoya-Studio How did you make that work ? nuxt-auth-utils is very bare bone and only found the clear method which only logout in the Nuxt app

I've got the same issue with Keycloak 25.

@Degoya-Studio How did you make that work with nuxt-auth-utils ? It's very bare bone and only found the clear method which only logout in the Nuxt app

@Degoya-Studio it is not possible with nuxt-auth-utils currently, you can only leave out the post redirect uri completely. That is how they handled it, there is simply no logoutUrl option at all.
With the current version v0.16.0, there is a cognito provider, as well as the capability to add custom fields to auth, token and logout requests.
The provider preset for cognito will handle the two required parameters client_id and logout_uri. Don't forget to specify logoutRedirectUri in the provider options and to add that specific url to your cognito configuration in the AWS portal.

@itpropro I think I am missing something super simple, but I have this module working in my SSR Nuxt3 app against KeyCloak.
I have a logged-in user now and userInfo populated.
Finally, I want to call my API which is protected by KeyCloak. I need my access token to make an API call.
How do I get the access token to attach to my authorization header and make that call?

@itpropro I think I am missing something super simple, but I have this module working in my SSR Nuxt3 app against KeyCloak.
I have a logged-in user now and userInfo populated.
Finally, I want to call my API which is protected by KeyCloak. I need my access token to make an API call.
How do I get the access token to attach to my authorization header and make that call?

Hey,
you can use the exposeAccessToken parameter to expose the access token though the user session. From a OIDC standpoint you should make sure that the audience of the token is the id of your api. To have a correct access token for your API, the aud field has to match the one of the registration of your API, otherwise you will not be able to correctly validate it inside your API handlers.

@itpropro  I added more info to my orig comment. Those flags do nothing in my config.

@VictorioBerra It does work, here is what I have on my side:
export default defineNitroPlugin(() => {
    sessionHooks.hook('fetch', async (session) => {
        console.log('session', session);
        if (!(Object.keys(session).length === 0)) {
            const profileResponse = await $fetch('http://api.campus-eeq.local.vici.io/api/users/me', {
                method: 'GET',
                headers: {
                    Accept: 'application/json',
                    Authorization: `Bearer ${session.accessToken}`,
                },
            });

            session.entrepreneur = profileResponse.data.entrepreneur;
        }
    });
});

@mrleblanc101 Is there anything wrong with my config, or my button click event?

@VictorioBerra Not that I see, maybe the token is only exposed on the server side.
As you can see, my console.log is in the nitro plugin fetch hook, so I can pull the the profile before login in the user.

Same result with:
@mrleblanc101
    const { fetch } = useOidcAuth()
    fetch()
        .then(data => {
            console.log(data)
        })
        .catch(error => {
            console.error('Error:', error)
        })

Which I confirm hits /api/_auth/session server side so I assume tokens would be returned and they are not. Is that supposed to work?

@VictorioBerra I doubt you are using the latest version.
Also, the sessionHooks should be in server/api/auth/session.ts

@mrleblanc101 package.json "nuxt-oidc-auth": "^0.18.0",.
Not sure I follow on the hooks, the above code I showed fetch() hits api/_auth/session

@mrleblanc101 Is there anything wrong with my config, or my button click event?

In addition to what @mrleblanc101 already stated, you should try to avoid using process.env in your Nuxt config.
The first reason is that you should in general only use import.meta.env and the main one is that you should load variables from the environment in the documented way by using NUXT env variables:
NUXT_OIDC_PROVIDERS_KEYCLOAK_CLIENT_SECRET=YOUR_SECRET
NUXT_OIDC_PROVIDERS_KEYCLOAK_CLIENT_ID=YOUR_ID
NUXT_OIDC_PROVIDERS_KEYCLOAK_BASE_URL=KEYCLOAK_HOST/realms/YOUR_REALM

@VictorioBerra Did you logout and login again ?
If you just refresh it work work I think.
@itpropro I've never heard of not using process.env, I use it all the time and that's what they use in the Nuxt doc:
https://nuxt.com/docs/api/composables/use-runtime-config#define-runtime-config
What's the difference with import.meta.env ?

@VictorioBerra Did you logout and login again ? If you just refresh it work work I think.
@itpropro I've never heard of not using process.env, I use it all the time and that's what they use in the Nuxt doc: https://nuxt.com/docs/api/composables/use-runtime-config#define-runtime-config What's the difference with import.meta.env ?

There is no strict don't use, but linters for example have already started to pick this up as recommended. The Nuxt team also recommends it, you can read more about that here (and you will find it in many places across the ecosystem already) https://nuxt.com/docs/api/advanced/import-meta.
Especially for determining, if you are on client or browser and in which node environment you are it is recommended to use import.meta.client or import.meta.dev respectively.
For security reasons, you should not directly load from process.env in you nuxt.config, as that bypasses the mechanisms of config merging that requires you to "allow" a value by defining it in your config as empty string, which will then get merged if it has the NUXT or NITRO prefix if you are only using Nitro.
In addition for better DX, I recommend using T3 Env https://env.t3.gg/docs/nuxt

GOT IT. KeyCloak by default sets the access token expiration to 1 minute. Once I bumped to 1 hour, it all started working. That was frustrating.

GOT IT. KeyCloak by default sets the access token expiration to 1 minute. Once I bumped to 1 hour, it all started working. That was frustrating.

Yeah, Keycloak has a lot of weird defaults :D Glad you found that!

If anyone is still experiencing this issue - could you check if my changes from the linked issue above fixes the issue for you?


device manage

implement #91
1.user can update CR remove device and volumegroup that has no logical volume.
2.discover device daemonset will check the device is Online or Offline
3. discover device daemonset will list all device stored in lvmd configmap instead of available devices.

thanks, I will add a test later.  Remove vg only vg has no logicalvolume. Make sure the operator doesn't touch anything on the data side.


Windows Error Reporting runs on fatal error of TestHost process

Ported from: https://github.com/dotnet/testimpact/issues/1043

Create a test project
Add a stack overflow to the test (i.e. just have the method call itself)

Expected: everything handles it gracefully
Actual: it does handle it gracefully, but I see werfault.exe running in the background dealing with the crash. Not sure if this is some IT policy being applied, but I'm guessing we don't want to be reporting these anywhere.

Closing this; reactivate if still required.


Add Beginners-friendly repositories list

Hi @sindresorhus
As per #380 opening this PR after ~2 weeks of list creation :)
Please let me know if there is anything else I need to change.
Thanks,
MunGell

Updated the name of the list


Better handling of Rational values in #to_s

Added #value_string to parallel #unit_string, which does two things when the value is Rational:

If the denominator is 1, don't show it.
Otherwise, wrap the value in parentheses so that it can be parsed properly.

Also added specs for two cases that would fail previously.
Should fix #22.

Hi, I added you as a collaborator. You can push yourself and just tell we when you think it is ready for a release. Could you also add tests for value_string etc?

Great! Specs on value_string was a great idea.


test: extend block coverage

block.put now returns a rejected promise when no callback is provided and an error occurs; more tests.

Thank you @gavinmcdermott :)
It LGTM, but I defer the final verdict to our in house promise expert, @dignifiedquire which added the promise support to tell if this matches the pattern he designed for js-ipfs-api :)

Thanks :octocat:


Docker: doesn't update db if docker-entrypoint-initdb/001-init.sql is changed

That can't work because schema changes are not possible that way. :) I think you need to create another sql file, e.g. 002-*.sql

Or do something similar to what we have done with vagrant: an command to replace the db. It's annoying currently that you have to go into phpMyAdmin to import the sql file again.

Ok so what I suggested does not work, but with docker it's a breeze to recreate the container, just do docker ps -a && docker rm -f <id> && docker-compose up

I have done yarn down:server && yarn start:server each time I updated the .sql file. It worked fine for me. Please reopen this issue if this is important.


Split IMediator interface in IPublisher and ISender interface

The reason for this change is the quick twitter chat I had with Jimmy about this.

Oh yes, I've removed unused usings as this is a pet peeve of mine. 😃

@KenBonny once this is merged the dependency injection package should be updated as well to register the publisher and sender. https://github.com/jbogard/MediatR.Extensions.Microsoft.DependencyInjection

@lilasquared Thanks for the heads up. I think the only thing that needs to happen is for the MediatR package to be updated. If the IMediator interface is registered, then I do believe most functionality should work. I did see that there is a line that registers all types in an assembly instead of registering everything by hand. I think that takes care of registering every type and interface in the assembly.

I'm pretty sure that only scans and searches for implementations of the handlers.  I'm referring to https://github.com/jbogard/MediatR.Extensions.Microsoft.DependencyInjection/blob/master/src/MediatR.Extensions.Microsoft.DependencyInjection/Registration/ServiceRegistrar.cs#L220
where the IMediator type is registered explicitly.  If you test your branch with that package and you try to inject ISender it fails to resolve.

Is this ok, can it be merged or do other steps need to happen? That the other reviewers approve, for example. Or do I need to press a button to merge the PR?

Don't mistake my question for impatience, but when will this be available in the package?
I'm asking so I can also update the DI package @lilasquared mentioned. I have a leaky memory (I should get that looked after) and if this takes a month or 2 (which is fine 😄 ), then I need to add a calendar notification to remind me to do the DI update then.

Tagged! Should be on NuGet shortly.

@jbogard  If you make such changes, please consider increasing the major version instead of the minor version in the future  ;-)
This interface split is technically a breaking change.
For example if an assembly "A" references IMediator  from 8.0.2, but at runtime 8.2. is supplied (assembly unification at build time), the interface split leads to the following runtime exception in "A" if it has not been recompiled with 8.2.
System.MissingMethodException : Method not found: 'System.Threading.Tasks.Task`1<!!0> MediatR.IMediator.Send(MediatR.IRequest`1<!!0>, System.Threading.CancellationToken)'.

Thanks 😃


Added validation to the date picker in the dataset tab

Have you looked at the callback function "callback_accept_dataset_metadata_input" ? I think maybe it is the cause of some of the issues...

And we have to change callback_accept_dataset_metadata_input in register_callbacks.py


Sonar Vulnerability Issue

Sonar reports an issue about  Microsoft.Office.Interop.Excel.dll that you probably use in this package. Can you please update/check this dependency,
Reference: CVE-2015-1770 | CVSS Score: 9.3 | Category: CWE-19 | Microsoft Office 2013 SP1 and 2013 RT SP1 allows remote attackers to execute arbitrary code via a crafted Office document, aka "Microsoft Office Uninitialized Memory Use Vulnerability."

ExcelDataReader does not have any dependency on Microsoft.Office.Interop.Excel.dll.


feat(HTTP): Missing handlers

Description
I worte getAccountByNumber and getValidatorByNumber handler and also test for the handlers.
Related issue(s)
feat: #528

@b00f I think you can review this PR know!

@kehiy Everything looks good. Please not that HTTP is a just wrapper over gRPC.
So, it is better to just call the gRPC, and let gRPC validates the data

@kehiy Everything looks good. Please not that HTTP is a just wrapper over gRPC. So, it is better to just call the gRPC, and let gRPC validates the data

that't mean I need just remove the checks and test for them?

@kehiy Everything looks good. Please not that HTTP is a just wrapper over gRPC. So, it is better to just call the gRPC, and let gRPC validates the data

that't mean I need just remove the checks and test for them?

Correct. It is OK now


[NAV-220] show pointer when hovering in toast close icon

Description
Using the cursor: pointer when hovering on the toast close icon.
References #1629
Type of change

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

How Has This Been Tested?
Manually tested. All existing tests run successfully.
Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[X] I have checked my code and corrected any misspellings

Looks good at: https://deploy-preview-1636--modus-webcomponents.netlify.app/?path=/story/components-toast--default&args=dismissible:true

@coliff I Am merging the PR


Enabling Cosmos for Unified Pipelines

This PR converts the existing Cosmos PR validation into the full unified pipeline model. Because the Cosmos pipeline was heavily modified to begin with this change is not straight forward.

/azp run java - cosmos - ci

/azp run java - cosmos

/azp run java - cosmos

/azp run java - cosmos

@HarshaNalluru pinging you on this PR because this is an example of how I am going to "normalize" the cosmos pipeline in the JS repo so I wanted to give you a heads up since the JS repo is probably that next one I'll tackle after finishing the Java repo.

/azp run java - cosmos - ci

Good call. Should probably be a more localised variable
Get Outlook for Androidhttps://aka.ms/ghei36

From: Wes Haggard<EMAIL_ADDRESS>Sent: Wednesday, August 28, 2019 1:44:06 AM
To: Azure/azure-sdk-for-java<EMAIL_ADDRESS>Cc: Mitch Denny<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-java] Enabling Cosmos for Unified Pipelines (#5102)
@weshaggard commented on this pull request.

In eng/pipelines/templates/variables/globals.ymlhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-java%2Fpull%2F5102%23discussion_r318156249&data=02|01|midenn%40microsoft.com|4bb13f6d58624d9f49cd08d72b05658c|72f988bf86f141af91ab2d7cd011db47|1|0|637025174485633173&sdata=P3xmcavq1oxhFqhnfBzJSUbwqIbADAI2nfpwt73%2BEwI%3D&reserved=0:

\ No newline at end of file

skipComponentGovernanceDetection: true
EmulatorMsiUrl: 'https://acpedaily1.blob.core.windows.net/emulator/azure-cosmos-emulator.msi'

Why would this go into the shared globals file?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-java%2Fpull%2F5102%3Femail_source%3Dnotifications%26email_token%3DAAD5K5QRF4BC76NAEIKZY3LQGVDUNA5CNFSM4IPKX362YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCC2O7ZQ%23pullrequestreview-280293350&data=02|01|midenn%40microsoft.com|4bb13f6d58624d9f49cd08d72b05658c|72f988bf86f141af91ab2d7cd011db47|1|0|637025174485643168&sdata=XuxSf7GTK0fs96rGDZhAlcLCBzZS9LeKWWZ7WuuJAkk%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAD5K5UL353WI6D7DXAPPFTQGVDUNANCNFSM4IPKX36Q&data=02|01|midenn%40microsoft.com|4bb13f6d58624d9f49cd08d72b05658c|72f988bf86f141af91ab2d7cd011db47|1|0|637025174485643168&sdata=uXX5VcgaDdQ7jwwauCGKypDHKVO61MYF3fUoetRTRfo%3D&reserved=0.


Support for double struck numbers and lower case letters

Description
Add support for all double struck letters and numbers. They can be found in the Unicode block Mathematical Alphanumeric Symbols.
Uppercase letters are available in sym, but numbers and lowercase letters are not.
Use Case
The main usecase would probably be the double struck 1 for the identity matrix. But in general having the possibility to reach more characters would be nice to have.

Was this closed by accident?

Am 20. Juli 2023 14:12:29 MESZ schrieb Enivex @.***>:

Was this closed by accident?


loraserver not starting, appears to complain about an SQL error

I've been running a >really< old version of loraserver and friends, and decided to upgrade it this weekend. I see 2.6.0 is the most recent version, but 2.3.0 is the most recent that apt-get will pull. Postgresql is version 9.4.
Is this a bug or a feature request?
Bug (I think. Could be me...)
What did you expect?
loraserver to start
What happened?
loraserver didn't start (errors below)
What version are your using?
2.3.0 (most recent for apt-get with raspian)
How can your issue be reproduced?
Fresh raspian SD card, install loraserver, configure and start it
Could you share your log output?
Oct 28 22:24:52 rpi3 systemd[1]: Started LoRa Server.
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="starting LoRa Server" band=US_902_928 docs="https://docs.loraserver.io/" net_id=000000 version=2.3.0
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="disabling all channels"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="enabling channels" channels="[0 1 2 3 4 5 6 7]"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="setup redis connection pool" url="redis://localhost:6379"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="connecting to postgresql"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="gateway/mqtt: connecting to mqtt broker" server="tcp://localhost:1883"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="no geolocation-server configured"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="configuring join-server client" ca_cert= server="http://localhost:8003" tls_cert= tls_key=
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="no network-controller configured"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="applying database migrations"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="backend/gateway: connected to mqtt server"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="gateway/mqtt: subscribing to rx topic" qos=0 topic=gateway/+/rx
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="gateway/mqtt: subscribing to stats topic" qos=0 topic=gateway/+/stats
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="backend/gateway: subscribing to ack topic" qos=0 topic=gateway/+/ack
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=info msg="migrations applied" count=0
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=debug msg="sql query executed" args="[]" duration="540.365µs" query="lock table code_migration"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=debug msg="sql query executed" args="[v1_to_v2_flush_profiles_cache 2018-10-28 22:24:52.494088763 +0000 UTC m=+0.058419340]" duration="586.979µs" query="\n\t\t\tinsert into code_migration (\n\t\t\t\tid,\n\t\t\t\tapplied_at\n\t\t\t) values ($1, $2)\n\t\t\ton conflict\n\t\t\t\tdo nothing\n\t\t"
Oct 28 22:24:52 rpi3 loraserver[8478]: time="2018-10-28T22:24:52Z" level=fatal msg="pq: syntax error at or near "on""
Oct 28 22:24:52 rpi3 systemd[1]: loraserver.service: main process exited, code=exited, status=1/FAILURE
Oct 28 22:24:52 rpi3 systemd[1]: Unit loraserver.service entered failed state.
Oct 28 22:24:52 rpi3 systemd[1]: loraserver.service holdoff time over, scheduling restart.
Oct 28 22:24:52 rpi3 systemd[1]: Stopping LoRa Server...
Oct 28 22:24:52 rpi3 systemd[1]: Starting LoRa Server...
Oct 28 22:24:52 rpi3 systemd[1]: loraserver.service start request repeated too quickly, refusing to start.
Oct 28 22:24:52 rpi3 systemd[1]: Failed to start LoRa Server.
Oct 28 22:24:52 rpi3 systemd[1]: Unit loraserver.service entered failed state.

Looking through the closed comments, I found #171. I knew I didn't like postgresql all these years for a good reason...

Please refer to https://www.loraserver.io/loraserver/install/requirements/. PostgreSQL 9.5+ is required.

Alas, https://www.loraserver.io/guides/debian-ubuntu/ says nothing about the required version, which was where I started.

But it does mention the Ubuntu / Debian distribution versions it has been tested on. The tested versions all contain PostgreSQL 9.5+ in the repository :)

Following the guide word-for-word from https://www.loraserver.io/guides/debian-ubuntu/ does not install the correct version of PostgreSQL when using Debian Jessie.  Debian Jessie repository provides PostgreSQL 9.4.18.

FWIW Debian Jessie (Debian 8), provides PostgreSQL 9.4.18, which will not work.  Debian Stretch is required.

Stretch is not required but you need to perform extra steps to install
PostgreSQL 9.5+
On Thu, Nov 15, 2018, 15:12 Michael D Peters<EMAIL_ADDRESS>wrote:

FWIW Debian Jessie (Debian 8), provides PostgreSQL 9.4.18, which will not
work. Debian Stretch is required.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/brocaar/loraserver/issues/358#issuecomment-439117076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKGeVEfhKIbvt5bby-bFhN93i7PZZqrks5uvaCEgaJpZM4X-OOm
.


Yet another segfault with proc variable capture

require "http"

def get(host, &block)
  HTTP::Client.new(host).exec("GET", "/") do
    block.call
  end
end

a = "example.org"
while a
  get(a) do
    a = nil
  end
end

Invalid memory access (signal 11) at address 0xc
[0x55c0dbf2cf76] print_backtrace at crystal/src/exception/call_stack.cr:121:5
[0x55c0dbf1576c] __crystal_sigfault_handler at crystal/src/signal.cr:347:3
[0x55c0dc043a00] sigfault_handler +40
[0x7facdc9170f0] ???
[0x55c0dbf238a6] [] at crystal/src/pointer.cr:117:6
[0x55c0dbf46a17] calc_excess_left at crystal/src/string.cr:1921:11
[0x55c0dbf469a3] strip at rystal/src/string.cr:1705:5
[0x55c0dc02cce5] initialize at crystal/src/uri/uri_parser.cr:17:16
[0x55c0dc02cc95] new at crystal/src/uri/uri_parser.cr:15:5
[0x55c0dbfd2b61] parse at crystal/src/uri.cr:477:5
[0x55c0dc02b69b] check_host_only at crystal/src/http/client.cr:164:11
[0x55c0dc02b5c8] initialize at crystal/src/http/client.cr:132:5
[0x55c0dc02b580] new at crystal/src/http/client.cr:129:3
[0x55c0dbf18fec] get at /tmp/test.cr:4:3
[0x55c0dbf10f3d] __crystal_main at /tmp/test.cr:11:3
[0x55c0dc0439d6] main_user_code at crystal/src/crystal/main.cr:105:5
[0x55c0dc04386c] main at crystal/src/crystal/main.cr:91:7
[0x55c0dbf120c6] main at crystal/src/crystal/main.cr:114:3
[0x7facdc6de152] __libc_start_main +242
[0x55c0dbf0295e] _start +46

This is on master.
Possibly related to #9935

I have a fix for #9935 but unfortunately it doesn't seem to be related.

Reduced a bit more, removing the HTTP::Client stuff:
require "uri"

def get(uri, &block)
  ->(uri : String) do
    URI.parse(uri)
    block.call
  end.call uri
end

url = "example.org"
while url
  get(url) do
    url = nil
  end
end

Invalid memory access (signal 11) at address 0xc
[0x563da561cb76] print_backtrace at /home/george/dev/git/crystal/src/exception/call_stack.cr:121:5
[0x563da560ed1c] __crystal_sigfault_handler at /home/george/dev/git/crystal/src/signal.cr:347:3
[0x563da56905c0] sigfault_handler +40
[0x7f978b0a50f0] ???
[0x563da56138e6] [] at /home/george/dev/git/crystal/src/pointer.cr:117:6
[0x563da562e8d7] calc_excess_left at /home/george/dev/git/crystal/src/string.cr:1921:11
[0x563da562e863] strip at /home/george/dev/git/crystal/src/string.cr:1705:5
[0x563da5685b65] initialize at /home/george/dev/git/crystal/src/uri/uri_parser.cr:17:16
[0x563da5685b15] new at /home/george/dev/git/crystal/src/uri/uri_parser.cr:15:5
[0x563da56852a1] parse at /home/george/dev/git/crystal/src/uri.cr:477:5
[0x563da560f391] -> at /home/george/dev/git/athena-framework/athena/test.cr:5:5
[0x563da560f376] get at /home/george/dev/git/crystal/src/primitives.cr:255:3
[0x563da560ad01] __crystal_main at /home/george/dev/git/athena-framework/athena/test.cr:12:3
[0x563da5690596] main_user_code at /home/george/dev/git/crystal/src/crystal/main.cr:105:5
[0x563da569042c] main at /home/george/dev/git/crystal/src/crystal/main.cr:91:7
[0x563da560be86] main at /home/george/dev/git/crystal/src/crystal/main.cr:114:3
[0x7f978ae6a152] __libc_start_main +242
[0x563da560055e] _start +46
[0x0] ???

It seems the internal URI::Parser is working with a pointer of the input uri.  https://github.com/crystal-lang/crystal/blob/master/src/uri/uri_parser.cr#L17

Thanks!
That lets me reduce more
def get(host, &block)
  p host  # crash
  block.call
end

a = "foo"
while a
  get(a) do
    a = nil
  end
end

Of course, this is not about nil. a = <anything non-string>
If I write a = 5, the crash is at address 0x55a800000011 consistently
So this is definitely about unions and still likely related to #9935

Oh... duplicate of #5609


A visual notification after completing Task

Is your feature request related to a problem? Please describe.
A Visual confirmation when the task is done
Describe the solution you'd like
A small popup below showing the completion of task
Additional context
This is the original video, the feat has been added
screen-capture (1).webm
Kindly mark this for hacktober fest

Hi once you mark this, I will create a pull request

Hi kindly check I have raised PR


Web: Review other web framework

https://gohugo.io/
does it fit our usages ?

I don't thinks we specially need it most things should be done on front end with vue.js and Phaser3.


Fixed Grammar

Description of the change: Made Grammar Corrections
Reason for the change: Improved reading experience
Link to original source: https://sendgrid.com/docs/ui/sending-email/a-b-testing/

Closes #

Hello @shinypenny31,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Mark routes as opaque when added without animation

Description
Prior to this change TransitionRoutes that were added without animation would not properly mark their first OverlayEntry as opaque. This means Flutter was rendering Routes that weren't visible (and those invisible Routes also showed up in the inspector and confused people there). This happened most prominently when the initial routes where added because those always skip their entrance animation.
This change now properly marks the OverlayEntrys of those routes as opaque.
It also modifies OverlayEntry.opaque to allow modifying the opaqueness before the entry is added to an overlay.
Related Issues
Fixes https://github.com/flutter/flutter/issues/38038.
Tests
I added the following tests:

OverlayEntry.opaque can be changed when OverlayEntry is not part of an Overlay (yet)
OverlayEntry of topmost initial route is marked as opaque
OverlayEntry of topmost route is set to opaque after Push
OverlayEntry of topmost route is set to opaque after Replace

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

/cc @HansMuller @dnfield @chunhtai


Remove bech32 string limit

nevent and nprofile includes a list of relays, this makes the size of the string limitless so the encode and decoding functions won't work for the majority of them.

Thanks, Koala!


Enable document links in chat code blocks

We should support document links (such as http(s)) in the code blocks generated by chat

To verify, as something like: write a jsdoc coumment that contains a markdown link
Confirm that the http link in the generated block is detected and clickable


Bug fix SA-203 - AvgStatsLast5mins is not populating correctly as it always display nil

Bug SA-203
The volume average stats for last 5mins always display nil and doesnt poplate it correctly.
Fix :
v1/nimbleos/volume.go , file contains the definition of volume struct with supported fields and json tags. The json tag for the field AvgStatsLast5mins has a typo . It has "avg_stats_last5mins" instead of "avg_stats_last_5mins" which causes to return default value nil for AvgStatsLast5mins. Made the tag name changes to fix the issue .
Unit test:
Ran the unit test to validate it correctly populate the filed values.
Signed-off-by: vidhut-kumar-singh<EMAIL_ADDRESS>

Add a unit test?

It was a typo in the field name which caused this issue. I think we should not add unit test case of field name typo however i am open to suggestion

Add a unit test?

It was a typo in the field name which caused this issue. I think we should not add unit test case of field name typo however i am open to suggestion

Add a unit test?

It was a typo in the field name which caused this issue. I think we should not add unit test case of field name typo however i am open to suggestion

Was it actually a typo, as in a person mistyping it?  I might have guessed it came out wrong from the generator, which could indicate a potential class of problems that might recur.
Previously we discussed the absence of unit tests in this project, and George indicated they are needed.  The SDK behavior should be verified, and regressions identified, without requiring a connection to another network device.
For this issue, you were unable to test a proposed fix because you lacked access to an array that returned non-zero data.  So you called on me to verify it.  I wrote a unit test to demonstrate that access to an array was not required (and because I also lack access to such an array).  I do not understand why it would be considered preferable not to have unit tests.

Add a unit test?

It was a typo in the field name which caused this issue. I think we should not add unit test case of field name typo however i am open to suggestion

Was it actually a typo, as in a person mistyping it?  I might have guessed it came out wrong from the generator, which could indicate a potential class of problems that might recur.
Previously we discussed the absence of unit tests in this project, and George indicated they are needed.  The SDK behavior should be verified, and regressions identified, without requiring a connection to another network device.
For this issue, you were unable to test a proposed fix because you lacked access to an array that returned non-zero data.  So you called on me to verify it.  I wrote a unit test to demonstrate that access to an array was not required (and because I also lack access to such an array).  I do not understand why it would be considered preferable not to have unit tests.

Add a unit test?

It was a typo in the field name which caused this issue. I think we should not add unit test case of field name typo however i am open to suggestion

Was it actually a typo, as in a person mistyping it? I might have guessed it came out wrong from the generator, which could indicate a potential class of problems that might recur.
Previously we discussed the absence of unit tests in this project, and George indicated they are needed. The SDK behavior should be verified, and regressions identified, without requiring a connection to another network device.
For this issue, you were unable to test a proposed fix because you lacked access to an array that returned non-zero data. So you called on me to verify it. I wrote a unit test to demonstrate that access to an array was not required (and because I also lack access to such an array). I do not understand why it would be considered preferable not to have unit tests.

Does it really matter who does it? typographical mistake remains the same. There was a bug in the Generator which caused this typo and instead of generating avg_stats_last_5mins it generated avg_stats_last5mins.
Ideally, the unit test for this should go to the generator not in the SDK. since it is a Generator bug.
There is no absence of unit test, regression test, and integration test in this project. you are assuming that you should be able to run the unit test without Array that was never a requirement from start, that is nice to have a suggestion not mandatory. We are already running a regression test with every build. please don't assume.

volume_service_test.go - contains the unit tests .
pkg/test - contains functional and integration tests .

Appreciate your help, I didn't know even you don't have the actual Array setup, if you could have said so I would figure it out.

Add a unit test?

It was a typo in the field name which caused this issue. I think we should not add unit test case of field name typo however i am open to suggestion

Was it actually a typo, as in a person mistyping it? I might have guessed it came out wrong from the generator, which could indicate a potential class of problems that might recur.
Previously we discussed the absence of unit tests in this project, and George indicated they are needed. The SDK behavior should be verified, and regressions identified, without requiring a connection to another network device.
For this issue, you were unable to test a proposed fix because you lacked access to an array that returned non-zero data. So you called on me to verify it. I wrote a unit test to demonstrate that access to an array was not required (and because I also lack access to such an array). I do not understand why it would be considered preferable not to have unit tests.

Does it really matter who does it? typographical mistake remains the same. There was a bug in the Generator which caused this typo and instead of generating avg_stats_last_5mins it generated avg_stats_last5mins.
Ideally, the unit test for this should go to the generator not in the SDK. since it is a Generator bug.
There is no absence of unit test, regression test, and integration test in this project. you are assuming that you should be able to run the unit test without Array that was never a requirement from start, that is nice to have a suggestion not mandatory. We are already running a regression test with every build. please don't assume.

volume_service_test.go - contains the unit tests .
pkg/test - contains functional and integration tests .

Appreciate your help, I didn't know even you don't have the actual Array setup, if you could have said so I would figure it out.


Profile API endpoints

Adds the /api/v1/profiles/:uid endpoint for retrieving the details of one particular profile
Adds the api/v1/profiles endpoint for retrieving the details of many profiles, with an option uids parameter used for filtering

TBC: Need confirmation from @ansonK that the latter endpoint will also require filtering by type with another optional types parameter

@ansonK You may also wanna glance over this as I removed your respond_as_json method, at least temporarily. It didn't allow me to easily update the status code of the response so if you have a way to make that a bit DRYer I'd like to hear it.

Also, I realise that I've re-privatised the serialize methods on the classes in this PR which ties in with comments on #41. This will need looking over.

Oh, and another thing - currently the API only returns one organisation link, so I've just taken the first one for that profile. However, the underlying data structure has a many-many relationship, so a profile could easily have more than one organisation. How do we want to handle this?

Oh, and another thing - currently the API only returns one organisation link, so I've just taken the first one for that profile. However, the underlying data structure has a many-many relationship, so a profile could easily have more than one organisation. How do we want to handle this?

We currently do not have a use case for a profile belonging to more than 1 organisation. When / if we do then we can change the API. Until then returning the first organisation is fine

You may also wanna glance over this as I removed your respond_as_json method, at least temporarily. It didn't allow me to easily update the status code of the response so if you have a way to make that a bit DRYer I'd like to hear it

That's fine. Just keep using respond_to block in each controller action until patterns emerge that we can dry up

@ansonK the respond_with method was extracted to the responders gem because it was felt that it was too restrictive and 'magical' - I'd stick to respond_to blocks as that's the expected style going forward.

@robmckinnon By consumer of the API, I do not strictly mean another app. Other objects are consumers as well, and since the public API exposes a method to retrieve the serialized hash, surprises may occur.
We have discussed this offline with @ansonK and @chriscarter90 will be implementing the necessary changes and making the change to symbols. If anyone feels strongly about further changes to this, submitting a PR might be best way to proceed with the discussion.

Finally, I think that all the comments have been addressed.
One last thing, @ansonK - should we be able to filter by type at this point as the story requires? Or can we leave that out until we're sure we need it?

respond_with method was extracted to the responders gem because it was felt that it was too restrictive and 'magical' - I'd stick to respond_to blocks as that's the expected style going forward.

@pixeltrix I meant dry up using respond_to blocks

@chriscarter90 no filtering is needed right now. Can add as a later iteration

@ansonK Just to clarify, there is already filtering on UIDs just not on types.

Pushed an example of how a collections serialiser can consume an object serialiser so we don't have duplication.
2nd commit to introduce a BaseSerializer. We should also have a CollectionSerializer

@ansonK - I've ensured there is are BaseSerializer and CollectionSerializer classes now, fully tested. I have rebased the branch so you'll need to reset your local branch if you want to run it locally.

@ansonK Please merge at your earliest convenience! Will be good to get this damn PR out the way!

Just one last comment on strings vs. symbols for as_json - symbols are about 5% slower. :smile:
require "benchmark/ips"
require "active_support/json"

STRING_KEYS = (1..100).inject({}){ |h, i| h["key#{i}"] = i; h }
SYMBOL_KEYS = (1..100).inject({}){ |h, i| h[:"key#{i}"] = i; h }

Benchmark.ips do |x|
  x.report("String Keys:  ") { ActiveSupport::JSON.encode(STRING_KEYS) }
  x.report("Symbol Keys:  ") { ActiveSupport::JSON.encode(SYMBOL_KEYS) }
  x.compare!
end

$ ruby test.rb
Calculating -------------------------------------
      String Keys:     435.000  i/100ms
      Symbol Keys:     401.000  i/100ms
-------------------------------------------------
      String Keys:        4.313k (± 2.9%) i/s -     21.750k
      Symbol Keys:        4.089k (± 4.4%) i/s -     20.451k

Comparison:
      String Keys:  :     4312.8 i/s
      Symbol Keys:  :     4089.4 i/s - 1.05x slower


Multiple vaccination entries on UK NHS certificate - but one QR code only

Hi all,
It is really great to see that the NHS vaccination certificate is now supported by the CWA. There is, however, a problem with people who are fully vaccinated: instead of being issued one QR code for each jab, the NHS issues a single QR code which contains information for both jabs. Upon scanning of the UK NHS QR code of a fully vaccinated person, the following error message appears:
`This vaccination certificate is not supported in your app version yet. Please update your app or contact the technical hotline under "App Information". (VC_MULTIPLE_VACCINATION_ENTRIES)'
For a single-jabbed person, the certificate is properly processed and loaded onto the CWA.
Cheers,
David-Benjamin

We have to wait until UK public key is added to the EU gateway and they align to one certificate in a QR code

@thomasaugsten

We have to wait until UK public key is added to the EU gateway and they align to one certificate in a QR code

Is the UK participating? They are not part of the EU any more (Brexit) and they are not listed on
https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_en in the section for non-EU countries. Maybe there is some exception for UK?

They are not participating in the e-health group but there are discussion with non eu countries to participate

Any updates on this?

CH is participating and is accepting DCC and you can convert DCC in swiss certificates
https://bag-coronavirus.ch/zertifikat/wo-kommt-das-covid-zertifikat-zum-einsatz/

@thomasaugsten
And NHS QR codes?

No update yet

@coronawarnapp tweeted:

So far, the UK certificates are not accepted by and in the EU.
The talks are almost finished and should be accepted in the next few days.
If that is the case you could scan them immediately in the CWA without error.

@Ein-Tim Thanks!

@dsarkar

If that is the case you could scan them immediately in the CWA without error.

Does the dev team know this? Unless neither the NHS DCC format has been changed to allow only one single entry, nor CWA and business rules have been adapted to accept DCCs with multiple entries, this is not true, unfortunately.

UK aligned to the EU standard to integrate only one cert see demo qr codes
https://github.com/eu-digital-green-certificates/dcc-quality-assurance/tree/main/GB/1.3.0

Some users on Twitter are reporting that this works now, they had to download a new certificate from some portal. Could you check @chopin1810?

@Ein-Tim Yeah, there was a BBC article about this recently. Let me download some fresh certificates and try. I will report here.

The United Kingdom is now officially listed on https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_en under the heading:
"Recognition of COVID certificates from third (non-EU) countries
So far, 24 non-EU countries (and territories) joined the EU Digital COVID Certificate system. This includes:
...
United Kingdom
...
The COVID Certificates issued in those 24 countries (and territories) are accepted in the EU under the same conditions as the EU Digital COVID Certificate. Likewise, the EU Digital COVID Certificate is accepted by those 24 countries."

So, I downloaded two newly generated QR codes (expiring 24/12/2021). Scanning the first one works fine, checking validity fails though (CWA 2.14.1).
However, after scanning the second code, I see an error message on the screen which reads 'The certificate is already registered in your app. (VC_ALREADY_REGISTERED)'. Same happens after fully reinstalling CWA.

@chopin1810
I'm assuming from your original wishlist description that you now have two certificates and one is for your first jab with a second one for your second jab. Can you confirm?
What were the details of the failed validity check?
Were you checking the validity of a 2/2 vaccination certificate i.e. 2nd in a series of 2 jabs? or were you checking a 1/2 vaccination certificate? A 1/2 certificate should fail the validity check. If you check against rules for Germany you should get the error "The vaccination schedule must be complete (e.g., 1/1, 2/2)"
Are you using an iOS or Android device?

PS
@chopin1810
If you need to remove a certificate after you have scanned it in, there are instructions for this on https://www.coronawarn.app/en/faq/#eu_dcc_remove. You should not need to resort to uninstalling the app and reinstalling it.

So, I downloaded two newly generated QR codes (expiring 24/12/2021). Scanning the first one works fine, checking validity fails though (CWA 2.14.1). However, after scanning the second code, I see an error message on the screen which reads 'The certificate is already registered in your app. (VC_ALREADY_REGISTERED)'. Same happens after fully reinstalling CWA.

@chopin1810
Thanks. We received further reports from users that weren't able to scan their NHS certificate. How does it work in the UK to get a new certificate? Is there a way for everybody to obtain a new certificate in the correct format? Can you tell us which portal you've used to download the new qr codes?

@ucmichan
I don't have first-hand experience of UK NHS certificates, but there is some information on https://www.nhsx.nhs.uk/covid-19-response/eu-gateway/ which may help you.

@ucmichan
When I'm not mistaken you are a member of the CWA-Team, right? If so, could you maybe make your membership of the CWA organization public?
This helps new/one time contributes to differentiate between the community and members of the development team.
Thank you!


tsp, generate PollingStrategy in emitter

fix https://github.com/Azure/autorest.java/issues/2385
typescript code https://github.com/Azure/autorest.java/pull/2757/commits/721c1bfeacfa9ccda3c19341137a18597fcabe4e
java helper class https://github.com/Azure/autorest.java/pull/2757/commits/13896da03dcc1d38fd96b165a9913f361ad7417a
java code https://github.com/Azure/autorest.java/pull/2757/commits/5b1ecfb66dcdf3bf776277f05bcfacec55eca0f0
The hope is that this PR can also handle case https://github.com/Azure/autorest.java/issues/2756, as long as LroMetadata from typespec-azure-core is correct.

Could we list what polling strategy will we generate in emitter, and what will we use those in azure-core?

Could we list what polling strategy will we generate in emitter, and what will we use those in azure-core?

Basically, this line (and func) const useNewPollStrategy = isLroNewPollingStrategy(httpOperation, lroMetadata);


Proxy help..

My work computer internet is behind an http proxy.
I want to connect to a Watson websocket, but it fails because of the proxy.
It works fine  if I switch to an internet that is not behind a proxy...
How do I fix this?
Thanks

This issue is not related with uWS.

maybe your http proxy doesn't support websocket...

does the uWebSocket lib support http proxies? I cant find any option to specify an http proxy to use?


[react/prop-types] False positive when destructuring

Hey
I've run into a false positive on react/prop-types when doing destructuring.
I've created a branch with the failing test-case here.
The test-case is as close to what we had when it triggered.
}, {
  code: [
    'class Hello extends React.Component {',
    '  props: {',
    '    name: {',
    '      firstname: string;',
    '    }',
    '  };',
    '  arbitrary({one, two}: {one: string, two: string}) {}', // @flow annotation
    '  render () {',
    '    const { lastname } = this.props.name;',
    '    return
Hello {lastname}
;',
    '  }',
    '}'
  ].join('\n'),
  parser: 'babel-eslint',
  errors: [
    {message: '\'name\' is missing in props validation'}
  ]
}

I also ran into this today. The code that triggered the false positive was like so:
function displaySomething({ startDate, endDate }) { //offending line
  // ...
}

const MyComponent = React.createClass({
  propTypes: {
    startDate: React.PropTypes.string.isRequired,
    endDate: React.PropTypes.string.isRequired,
  },

  render() {
    return
{ displaySomething(this.props) }
;
  }
});

@kurtzur does displaySomething return JSX? if so, our component detection probably things it's an SFC (and it probably should be one)

@ljharb,
In his specific case it seems like it should be a SFC as he's providing this.props to the function, but what if the function returns JSX because it's a functions that's gonna be used in a map?
render () {
  return (
{ this.props.items.map(displaySomething) }
);
}

@zimme return (
{this.props.items.map(item => (<SFC item={item} />))}
);

I got false positive connected to destruction assignment.
Code preview
export default class NavigationButton extends React.Component {
  static propTypes = {
    route: PropTypes.shape({
      getBarTintColor: PropTypes.func.isRequired,
    }).isRequired,
  };

  renderTitle() {
    const { route } = this.props;
    return <Title tintColor={route.getBarTintColor()}>TITLE</Title>;
  }
}

I get this false report, despite getBarTintColor is clearly used.
'route.getBarTintColor'PropType is defined but prop is never used

@zimme Thanks for your test case, but when I run it with eslint-plugin-react 6.5.0 I get 'name.lastname' is missing in props validation, which see correct to me (in your component you declare name.firstname but not name.lastname). Did I miss something ?
@kurtzur Like @ljharb said, if displaySomething is returning JSX then we considere it as an SFC and you must declare propTypes on it.
@Andreyco Seems your talking about the react/no-unused-prop-types rule, this issue is about the react/prop-types. react/no-unused-prop-types has currently some issues and a lot of false positive :cry:

Either this has been fixed since I ran into the problem or I just messed something up with the test case. Anyways I can't really seem to replicate the problem and I've since changed my code so I don't have this problem at the moment.
Maybe it was related to @kurtzur problem which I understand why it would seem like a false positive but isn't.
I'll just open another issue if I run into this again.

I see this is recently closed, but wanted to chime in that I am seeing this problem as well. I'm going to go the route of very small SFC even though I'm not entirely convinced that is a solution to all problems.


Knowledge transfer of CSS FAQ chefs prototype to Bonnie

Describe the issue
Emily has created a proof of concept CSS FAQ prototype using chefs.  We wish to enable Bonnie to do the same and iterate further on this work.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context

links to both the airtable and chefs implementations were demoed to Bonnie and Don (with thanks to Emily)
Bonnie chose the chefs implementation to go forward with

Definition of done

chefs access for Bonnie and Don
an hour session with Bonnie to get her up to speed on using chefs
definition of a ticket to link or embed the chefs CSS FAQ form into the front of the cloud login page

Access given to Bonnie and Don, had session with Bonnie to walk through CHEFs and give overview of approach.


Changed scope of db context factory to no longer be hard-coded to Scoped

Closes #135

@koenbeuk I see some tests are failing, in particular I noticed:
EntityFrameworkCore.Triggered.Tests.Infrastructure.ServiceCollectionExtensionsTests.AddTriggeredDbContextFactory_WithCustomFactory_ReusesScopedServiceProvider
Any advise on how to proceed?

Updated as per your request @koenbeuk


Multiple Containers? Scope of Toasts? Dynamic Container Selector?

I have attempted to create an AngularJS Service that handles the interaction with Toastr. This service would allow the change of target containers for different scenarios. Different scenarios include Global Messaging (happens when on any page based on a service outage/etc) and per page (where the messages in the "page" scope are removed when route changes or on a "clear changes" event).
I have attempted to inject the toasterConfig into the service and change the target before I initiate the toast (toasterConfig.target = "#newSelector"). However I found that if this selector is a dynamic element (ui-view) then the toasts work for the first event but not for the next toast (it still fires as you can see console logs happen, but nothing appears.)
So I guess the issue is -
Can you have multiple containers? (I cannot seem to make it work)
Can you have scoped containers? or can the selector be Dynamically created such at  which is created on each route with ui-router.

You can't have multiple containers right now, sadly. That is something I want to fix on the future.
The .target functionality would match anything with .querySelector so see if you can select what you need using that function. It is personally a functionality I never used, added it because someone wanted it.
I am repeating a lot my TODO lately, but I maintain lots of open source projects and that is a hard work.
The scoped containers, I am not entirely sure if I follow the idea tho.

Scoped container would be the idea that each message being toasted would have a scope applied to them (ie, tag) so that if you initiated a clear it would only clear the tag specified allowing you to have a single container but manage the messages that are being placed in/removed with human readable logic instead of loops+filters on the open toasts.


support for test-unit 2.x

is there any chance to support test-unit gem 2.x ? currently :
NoMethodError: undefined method `test_suites' for Test::Unit::TestCase:Class
   suites at /opt/local/rvm/gems/jruby-1.7.21@jdbc/gems/m-1.3.4/lib/m/runners/test_unit.rb:5
   suites at /opt/local/rvm/gems/jruby-1.7.21@jdbc/gems/m-1.3.4/lib/m/executor.rb:85
    tests at /opt/local/rvm/gems/jruby-1.7.21@jdbc/gems/m-1.3.4/lib/m/executor.rb:61
  execute at /opt/local/rvm/gems/jruby-1.7.21@jdbc/gems/m-1.3.4/lib/m/executor.rb:15
      run at /opt/local/rvm/gems/jruby-1.7.21@jdbc/gems/m-1.3.4/lib/m/runner.rb:17
      run at /opt/local/rvm/gems/jruby-1.7.21@jdbc/gems/m-1.3.4/lib/m.rb:13
   (root) at /opt/local/rvm/gems/jruby-1.7.21@jdbc/gems/m-1.3.4/bin/m:4
     load at org/jruby/RubyKernel.java:1059
   (root) at /opt/local/rvm/gems/jruby-1.7.21@jdbc/bin/m:1

using test-unit (2.5.5)

To be honest I've never used test-unit, the gem, since most of my work is on MRI. I'll try to test it with jruby and see how hard it is to put it all together.

@kares Do you still have this issue? I have given it a try, but it was proving a bit to hard, and I'm not how many people really want/need support for this. What I could do is here, feel free to fiddle around and try to make it work.
Sorry for not being of more assistance.

@zamith thanks for the inspiration ... actually got it working with test-unit 2.5 (which I need to use with JRuby 1.7) and was able to pass tests using latest test-unit gem with JRuby 9K as well ... yay!

@kares This is now in version 1.4.0.


Sending custom command

Please add this information on all Readmes:
lettuce 3.x works with Java 6, 7 and 8, lettuce 4.x requires Java 8
I tried version 4 with Java 7 :)
Is there an easy way how to send a custom command to the Redis server?
I need something like the Low level Redis client of redisson.

Hi @WhiteTrashLord,
I'll add the Java versions to the readme.
I'm not sure I understand your requirement. Lettuce is a low-level driver that allows to send Redis commands to Redis (HMGET, 'BLPOP', ...). Checkout the Javadocs at http://redis.paluch.biz/docs/api/releases/latest/com/lambdaworks/redis/RedisConnection.html
If these commands are not sufficient, you still can send commands by building your own command with args and an output-parser: https://github.com/mp911de/lettuce/wiki/Custom-commands%2C-outputs-and-command-mechanics
HTH, Mark

Sorry I have no idea why I didn't see that.

Np, questions are always welcome.


Allow a minimum number of fonts to always be cache

I feel like we know we want "at least 1 font". (which is already the case), but I think that users want to load many other fonts (italics and maybe a few more at least).
We use the PyGFX Stats, so that creates "at least 1 font more than our application's font" and the continuous eviction creates random lags.
I also have some experience creating caches, so I spent a bit of time refactoring things. I hope that is "appreciated".
I feel like "20MB of fonts" for a graphics intensive python application is "fine".
Before this, if I would wait 10 seconds, I would get the following evictions:
Removing /usr/share/fonts/truetype/noto/NotoSans-Regular.ttf from cache
Removing /usr/share/fonts/truetype/noto/NotoSans-Regular.ttf from cache

With the following changes, I no longer get it.
I would say that an LRU cache is likely enough, but the LRU cache might not work very well if the user is flipping through many many fonts quickly

maybe i should have not done the refactor, now i have to "fix" the tests ...

On main
https://github.com/user-attachments/assets/a340c50a-ee01-4113-951f-b0f4a8e02770
 print statements
diff --git a/pygfx/utils/_dirs.py b/pygfx/utils/_dirs.py
index 73e201f..d61c592 100644
--- a/pygfx/utils/_dirs.py
+++ b/pygfx/utils/_dirs.py
@@ -22,7 +22,9 @@ def get_resources_dir():
     with context as path:
         pass
     # Return the dir. We assume that the data files are on a normal dir on the fs.
-    return str(path.parent)
+    directory = str(path.parent)
+    print(directory)
+    return directory

 def get_cache_dir():
diff --git a/pygfx/utils/text/_shaper.py b/pygfx/utils/text/_shaper.py
index 418ac74..f5e99a3 100644
--- a/pygfx/utils/text/_shaper.py
+++ b/pygfx/utils/text/_shaper.py
@@ -84,6 +84,7 @@ class TemporalCache:
             reverse=True,
         )
         for key, lt in to_remove[self._minimum_font_files :]:
+            print(f"Evicting {key}")
             self._lifetimes.pop(key, None)
             self._cache.pop(key, None)

https://github.com/user-attachments/assets/8dba591b-4be5-4a18-a4f2-663fdd79a44b

Thank you!


Plasma 2040 UART RX TX Possible?

Does anyone know if it's possible to connect to the UART pins on the pico with this board?

I'm getting an error on this:
uart = UART(0, baudrate=9600, tx=Pin(20), rx=Pin(21), timeout=1)

%Run -c $EDITOR_CONTENT
MPY: soft reboot
Traceback (most recent call last):
File "", line 11, in
ValueError: bad TX pin


API

Hello, is it convenient to disclose which external api to use?

I didn't quite understand your question. As you can see from the code, I'm using the API from https://aitohumanconverter.com. Have I answered your question?

I didn't quite understand your question. As you can see from the code, I'm using the API from https://aitohumanconverter.com. Have I answered your question?

Thank you for your reply. Did you develop this target website? I have been looking for a specific approach to ai humanize because I lack experience with transformer models.

Nope, I was using primarily because it's free


In RC doc, add deployment & RS as alternatives, and link to kubectl rolling-update tutorial

@kubernetes/docs

@bgrant0607 all comments addressed; PTAL

LGTM, thanks.


OSGi runtime artifact does not have complete sources artifact

It seems, OSGi runtime sources artifact rdf4j-runtime-osgi-sources is almost empty. Class repackaging is working, but source repackaging is not.

Which version are you using @amivanoff ?


Unfolding of subject header removes consecutive spaces

s = "Your Order  with ABC"
Mail::Field.new('subject', s, 'UTF-8')
f.to_s == s # => false
f.to_s # => "Your Order with ABC"

I've narrowed it down to line 236 in field.rb:
s.gsub(/[\r\n \t]+/m, ' ') # => "Your Order with ABC"

Is there any reason to strip turn multiple spaces in a subject field. If nothing else, this violates the principle of least surprise.
I was writing a spec to test a mailer and the subject I sent in wasn't what I got back out.

Agree it's odd. What does git history indicate, @aselder?

@jeremy This was last changed 2/1/13 in 72befdc4dab3e6e288ce226a7da2aa474cf5be83, with the comment "This has little effect on the specs, but on my header reading example it
makes about a 10x performance difference"
So it looks like this was a performance enhancement that led to a regression. The only reason I'm discovering it is we're upgrade from Rails 3 (mail 2.5.x) to Rails 4 (mail 2.6.x). All of a sudden a bunch of specs started failing.

The comment says "Unfolding is accomplished by simply removing any CRLF that is immediately followed by WSP."
How ever this is not what the regex does. Based on the comment, I can probably correct the regex.

This issue has major consequences when it comes to S/MIME signing the messages as the headers contents are not preserved and the signature is made invalid.

Fixed by #982


mvn cmd in documentation does not run correctly

https://quarkiverse.github.io/quarkiverse-docs/quarkus-github-action/dev/create-github-action.html
mvn io.quarkus.platform:quarkus-maven-plugin:2.15.1.Final:create
-DprojectGroupId=org.acme \
-DprojectArtifactId=my-github-action \
-DprojectVersion=999-SNAPSHOT \
-DprojectName="My Action" \
-DprojectDescription="Description of my action" \
-Ddata=github-action-codestart.github-repository=my/action-github-repository \
-Dextensions="io.quarkiverse.githubaction:quarkus-github-action:1.0.1"
For me the -Ddata is patricia-hartnett/gitHibActions_testQuarkus, but apart from that I have not edited anything else in the mvn command.
When I run this the values are not being passed in.
mvn prompts me to fill in the values, but then it falls over with the error message:
[ERROR] Failed to execute goal io.quarkus.platform:quarkus-maven-plugin:2.15.1.Final:create (default-cli) on project artifactid: Failed to generate Quarkus project: Error while rendering template: /codestarts/quarkus/github-action-codestart/base/action.docker.tpl.qute.yml: Missing required data: {project.name} -> [Help 1]
Please see the screengrab attached.
I am using MacOS 12.6.1
I have 3.6.3 Maven

Hi!
This rings a bell as we have this warning in the Quarkus main documentation:
For Windows users

    If using cmd , (don’t use backward slash \ and put everything on the same line)
    If using Powershell , wrap -D parameters in double quotes e.g. "-DprojectArtifactId=getting-started"

If it fixes the issue for you, I'll add it to the documentation.

Thank you that worked perfectly ! I am on a Mac.
Really appreciate your help.

Thanks for confirming. I created https://github.com/quarkiverse/quarkus-github-action/pull/104/files to improve things a bit.


Kev/102_change_key_frontend

Pull Request Details
What issue does this PR address

implemented change key widget in keypod app (frontend)

Link to associated issue: #102

Checklist
Complete the check-list below to ensure your branch is ready for PR.
Flutter Style Guide: https://survivor.togaware.com/gnulinux/flutter-style.html

[x] Screenshots included in linked issue
[x] Changes adhere to the team style and coding guideline
[x] No confidential information
[x] No duplicated content
[x] No lint check errors related to your changes (make prep or flutter analyze lib)
[ ] Pre-exisiting lint errors noted: [HERE]
[x] Tested on at least one device

[ ] Android Phone
[ ] Android Emulator
[ ] Chrome on Android
[x] Chrome
[ ] iOS
[ ] Linux
[ ] MacOS
[ ] Windows

[x] Added 2 reviewers (or 1 for private repositories then they add another)

Finalising
Once PR discussion is complete and 2 reviewers have approved:

[ ] Merge dev into the branch
[ ] Resolve any conflicts
[ ] Add one line summary into CHANGELOG.md
[ ] Bump appropriate version number in pubspec.yaml
[ ] Push to git repository and review
[ ] Merge PR into dev

On the "back" button.

Thanks @junhaow1. Functioanlity looks great.
Could we have ChangeSolidKeyDialog() exported from solidpod and in keypod I create a button in the test tab to call this ChangeSolidKeyDiaolog().

sounds good, on it

Thanks @junhaow1. Functioanlity looks great.
Could we have ChangeSolidKeyDialog() exported from solidpod and in keypod I create a button in the test tab to call this ChangeSolidKeyDiaolog().

ready to be reviewed :)

Am I doing something wrong?
Do I need to be logged in first, before I can change my key? Or is this all local?
Currently I start the app, click CONTINUE, click CHANGE KEY and it goes straight into the change key popup.
I then type any rubbish and it seems to accept it, irrespective of my old key (but then how does it know who I am), and seems to return without complaint?
I think on the popup page it also needs to show me the current WebID.
I suspect I need to be logged in to change the key.

Hi @gjwgit  we have another backend ticket to deal with  what you mentioned. This PR is just for the frontend part.

Also, I logged in, then changed my security key but it did not seem to get changed? I had to use my old one again.

Also, As an App developer, how do I do this with an icon rather than a button?

Not fully tested but let's get it in  to dev asap.


Keep Tiki SDK state in method channel

As a developer, I need to reuse the same TIKI SDK node between calls.
If the developer calls the method channel from native code, every call should use the same SDK instance, otherwise, the Dart SDK creates a new node in each build.
To achieve that the Flutter SDK needs to
~a. keep the same TIKI SDK instance built and reuse it~
b. accept an address as parameter to use in the build method to build an existing node
~c. return the node address in the build, so the native SDK can reuse it in subsequent calls, if necessary~
Acceptance criteria

Flutter SDK method channel accomplish the three points above (a,b,c)