Log JWT Claims

https://github.com/department-of-veterans-affairs/vets-contrib/issues/4017
https://github.com/department-of-veterans-affairs/vets-contrib/issues/4036
This PR adds some logic to decode an access token and add the claims to the kong logs. At the moment it blindly includes all claims, but we should modify that. I'm looking for feedback about how that should be achieved: should we make that configurable at runtime or just specify it in the code? Or should I only log the cid claim and call it a day?
Also welcome any feedback on the code itself – lua is not very familiar to me!

I'm looking for feedback about how that should be achieved: should we make that configurable at runtime or just specify it in the code? Or should I only log the cid claim and call it a day?

Looks like the blacklist and whitelist filters for headers might be reusable. Maybe have another list in the config with the whitelisted claims to log? (request_claims_whitelist?)

@jasalisbury agreed, we could definitely use those functions here. I'm more asking about whether we would actually take advantage of that configurability.
I think we may need a migration if we add to the schema, and given the fact that this logic is probably going to move to another plugin soon, I'm slightly inclined to hold off, but I could be convinced otherwise!
If I just hardcode them in here, I think I would whitelist cid, exp, iat, iss, jti, and scp. Anyone see any issues with logging those?

@jasalisbury agreed, we could definitely use those functions here. I'm more asking about whether we would actually take advantage of that configurability.
I think we may need a migration if we add to the schema, and given the fact that this logic is probably going to move to another plugin soon, I'm slightly inclined to hold off, but I could be convinced otherwise!
If I just hardcode them in here, I think I would whitelist cid, exp, iat, iss, jti, and scp. Anyone see any issues with logging those?

yeah I don't see anything wrong with those, hard-coding is probably the way to go. The claims we want logged is not going to be changing that much!


Cleanup and version updates

Cleaning the build so that it is easier for users to see what is actually needed and what not.
Type of change

[ ] Bugfix
[ ] Feature

Checklist

[ ] I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
[ ] I have added a description following the guideline.
[ ] The issue is created in the corresponding repository and I have referenced it.
[ ] I have added tests to ensure my change is effective and works as intended.
[ ] New and existing tests are passing locally with my change.
[ ] I have performed self-review and corrected misspellings.

Additional for Feature type of change

[ ] Enhancement / new feature was discussed in a corresponding GitHub issue and Acceptance Criteria were created.

Running production build currently gives me:
Caused by: java.lang.ClassNotFoundException: com.vaadin.base.devserver.startup.DevModeStartupListener
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
at io.quarkus.bootstrap.runner.RunnerClassLoader.loadClass(RunnerClassLoader.java:115)
at io.quarkus.bootstrap.runner.RunnerClassLoader.loadClass(RunnerClassLoader.java:65)
at java.base/java.lang.Class.forName0(Native Method)
at java.base/java.lang.Class.forName(Class.java:496)
at java.base/java.lang.Class.forName(Class.java:475)
at com.vaadin.base.devserver.startup.DevModeStartupListener_Bean.(Unknown Source)
at io.quarkus.arc.setup.Default_ComponentsProvider.addBeans1(Unknown Source)
at io.quarkus.arc.setup.Default_ComponentsProvider.getComponents(Unknown Source)
at io.quarkus.arc.impl.ArcContainerImpl.(ArcContainerImpl.java:117)
at io.quarkus.arc.Arc.initialize(Arc.java:31)
at io.quarkus.arc.runtime.ArcRecorder.initContainer(ArcRecorder.java:45)
at io.quarkus.deployment.steps.ArcProcessor$generateResources844392269.deploy_0(Unknown Source)
at io.quarkus.deployment.steps.ArcProcessor$generateResources844392269.deploy(Unknown Source)

Removed the exclusions (which we have for the spring boot starter) and now it seems to work properly 🤷♂


Fresh installation (installer): problem with auto booting NIS

0.5.18-BETA (installer), Win 7, 64-bit Java 8

All previous versions of NEM software uninstalled and working/user data deleted.
Fresh installation with the 0.5.18 installer -> check the option to run the wallet after closing the installer
NCC web interface opens up -> create new wallet and log in
at this time, the status bar is yellow and says "NIS needs to be booted. Boot a local node blah blah"
dropdown menu (top right) is used -> click on Boot Local Node (which is NOT grayed out) -> nothing happens (no popup windows appears), even if tried repeatedly
Close NCC -> Restart NCC -> Log into the new wallet -> the status bar is yellow and says "NIS is synchronizing. At block XYZ etc." and the "Boot Local Node" is now grayed out.

After some investigation, we assume that the auto-boot procedure (which is enabled by default) does not work, if NIS is not ready yet. This probably leads to a faulty state and the manual boot procedure (via NCC dropdown menu) doesn't work either.
I was not able to reproduce this issue with the standalone (=log into the wallet when NIS is not ready to be booted yet), because I then see a popup where I am supposed to enter the node name and another popup window is displayed with a warning, that NIS is not available. Whenever NIS is ready, I can boot it by clicking "boot" in the popup window.

when nis is not ready, auto-boot will simply return error, and you'll have to boot manually (that is sane and sort of expected).
the bug was actually caused by the fact, that right after creating new wallet UI was getting bad addressbook data.


Added a dateLookBack function

Update looks good, but one last thing, change the variables to a more pythonic scheme and change the line length to loosely conform to PEP8

@mkhaitman When you are back, please test this with Eric on df01 with a fake job.

@oldmantaiter Tested this change on DF01 with Eric. Tested on a fake rule with archive_lookback=2, as well as confirmed working on production rule with same config. Safe to merge into Master!


Layout issues with TextSkittle

Icon-Only skittles work fine but when using one with text there are 2 problems:

a) the shadow is cropped
b) the text is not centered vertically
with this code:
  final SkittleBuilder builder = new SkittleBuilder.Builder(this, fam)
                .mainSkittleColor( R.color.accent)
                .build();

        builder.makeTextSkittle("foo",R.drawable.ic_action_bulb).add();

Noted
On 01-Feb-2016 1:52 AM, "ligi"<EMAIL_ADDRESS>wrote:

Icon-Only skittles work fine but when using one with text there are 2
problems:
[image: skittles_problem]
https://cloud.githubusercontent.com/assets/111600/12704692/80e315d0-c860-11e5-9378-30fa60b00850.png
a) the shadow is cropped
b) the text is not centered vertically
with this code:
final SkittleBuilder builder = new SkittleBuilder.Builder(this, fam)
.mainSkittleColor( R.color.accent)
.build();
    builder.makeTextSkittle("foo",R.drawable.ic_action_bulb).add();

—
Reply to this email directly or view it on GitHub
https://github.com/aashrairavooru/Skittles/issues/18.

Checkout the new version 3.0.0 it fixes everything, NOTE: the new version has breaking changes refer to the readme for the changes or feel free to raise an issue

having a problem using 3.0.0:
* What went wrong:
A problem occurred configuring project ':mobile'.
> Could not resolve all dependencies for configuration ':mobile:_debugCompile'.
   > Could not resolve com.rlj.library:skittles:3.0.0.
     Required by:
         VaporizerControl:mobile:unspecified
      > Could not resolve com.rlj.library:skittles:3.0.0.
         > inconsistent module metadata found. Descriptor: com.rlj.library:skittles:3.0.0-SNAPSHOT Errors: bad version: expected='3.0.0' found='3.0.0-SNAPSHOT'

Hmm seems like the artifact wasn't published correctly I have uploaded a new version 4.0.0, I tested it should work correctly thanks for your patiece

Looks better now - but why did you jump a whole major for this ?-)


☂ Copilot Chat gets "confused" - which context to follow while responding -> Short-term memory loss?

VS Code Version: 1.79.0
OS Version: Win 11 x64
Logs: NA

Summary
Although, we know that Copilot Chat is still under development, there are critical bugs (or design issues of the underlying model) that need to be taken care of, the prominent one being:
"Copilot Chat gets confused - which context to follow while responding -> Short-term memory loss?/Lack of Conversational Skills?"

This is an umbrella issue consisting of all such bugs/issues as follows along with suggestions :)

NOTE: I'll keep updating the below table whenever I encounter any bugs while interacting with Copilot Chat :)

#
Bug/Issue
Description
Suggestion

1.
Model gets confused with which "context" to follow/use while responding
See screenshots 1.x below
Well this looks like more of a design issue but, I'm sure that; more fine-tuning can help improve this critical bug/issue with the underlying model :)

...n.
To be discovered (TBD)
TBA
TBA

Steps to Reproduce/Chat history {Click to see the screenshots}:

Creating and developing the context
1.1 (about Kotlin programming language):

1.2 Continue developing the context with relevant questions

1.3 More context development relevant questions, here as we can see the model has theoretically started to get confused; losing the hold of context. Although it answers the questions generally, it fails to mention anything about "Kotlin" ;) _GPT-4 will mention Kotlin while answering this question, even though it's a general-purpose LLM and not specifically trained/fine-tuned to answer only the questions related to code.

1.4 With the below question, it is clear that the model has failed to consider/completely lost the current context, by responding with a "context unaware" answer -> It has lost the memories :( -> Short-term memory loss :)

1.5 Now I try to provide a hint (ask the same question with a little bit more context, remember not to give it full context but, to see if it is able to recover and use the context that was built) Even though, it fails to provide "context-aware" answer

1.6 Providing the previous "full context", which it should have built in the first place by itself"

1.7 Let's test again if it remembers the context: +1 here

1.8 One more test for checking the context awareness: +1 here as well It seems I helped it to regain its short-term memories :)

## Taskslist for the bugs/issues mentioned above, in the same order:
- [ ] Underlying Model seems to suffer from "Confusion" and "Short-term memory loss".

I'm noticing this as well, it feels like the conversation is reset after each response.

I've also had this issue a few times, mostly when working between two different programming languages (C and Common Lisp, in my case), where Copilot Chat responds according to the C language context even though I've explicitly specified in my prompt that I'm asking about Common Lisp.
Occasionally, Copilot Chat also gets confused about context when I'm only working in one language (Common Lisp), but then it starts giving me answers in Python, when at no point in the conversation did I specify Python and have no Python projects open.
In both cases, it's a bit of a struggle to wrangle Copilot Chat back to the right language. Sometimes it will accept a reply such as "I didn't ask about Python, I asked about Common Lisp" and regenerates its reply with the correct language context. Sometimes it won't accept a conversational correction, but accepts a /fix command. And other times it falls into a loop where no matter what you ask it replies with "Sorry, but I can only assist with programming related questions."
But most of the time, Copilot Chat is surprisingly good with maintaining correct context, generating applicable and helpful suggestions for follow-up questions. It does better with contexts about formally specified models, such as algebraic and logical, rather than subjective and qualitative contexts that can only be analyzed statistically. Which is a bit weird, because with GPT-4 it's the other way around.

Here's an alternative which lets you use GPT4 with your own API key

I notice this everytime and it's frustrating. I haven't in my 4 months of copilot chat use(keep in mind that I havent used it much due to this issue) I found it remembering anything that happened in the same chat.
So, I'm just using copilot for code auto suggestions and chatgpt 3.5 for conversational style questions, even though I paid for copilot.

The issue is still present. It completely loses context.

Same issue. I has the memory of a goldfish


[bug]: labelFormatter in ChartTooltip component not working properly

Describe the bug
I'm trying to build a chart like the Line Chart - Interactive on this page: https://ui.shadcn.com/charts#line-chart, with date shown in each tooltip. But it doesn't work and when I go to Edit in V0, it's also showing invalid date on the tooltips.
Affected component/components
ChartTooltip
How to reproduce

Go to https://v0.dev/r/Gv5F7HkGIGo
Hover on the chart and you'll see invalid date on the tooltips.

Codesandbox/StackBlitz link
https://v0.dev/r/Gv5F7HkGIGo
Logs
No response
System Info
Safari

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I fixed it by removing '&& typeof label === "string"' at line 112 in chart.tsx

I fixed it by removing '&& typeof label === "string"' at line 112 in chart.tsx

Ran into the same problem and this fixed it for me thank u so much

I fixed it by removing '&& typeof label === "string"' at line 112 in chart.tsx

Ran into the same problem and this fixed it for me thank u so much

Noticed this too :/ but at least its an easy fix

It is now at line 145


Implement display mathematics for AsciiMath

There are two display options for TeX content:

in-line mathematics - delimiters $...$, \(...\)
display mathematics (centered, clear both) - delimiters $$...$$, \[...\]

However for AsciiMath only the inline version is available - delimiters `...`
It would be nice to have  display mathematics version for AsciiMath also - ``...`` delimiters maybe.
Currently, in order to achieve the display mathematics effects for AsciiMath content requires one to intervene at markup level
<p style="text-align:center">
  `x = (-b +- sqrt(b^2-4ac))/(2a) .`
</p>

When it comes to tools that use MathJax (e.g. Asciidoctor) the user has no access to the markup and the desired effect cannot be achieved.
AsciiMath is arguably more beginner friendly, learning TeX for this reason alone goes hard on the user.

The AsciiMath notation does not have a mechanism for distinguishing in-line from displayed math, like TeX does.  Indeed, its output is a hybrid of the two, as it uses the equivalent of \displaystyle for the math it lays out (at least by default), but as an in-line expression.  The intention (I assume) was that the rest of the page layout would be used to handle things like centering, as you have done in your example above.
There is a discussion in the MathJax user's forum about one possible way to extend AsciiMath to include notation for both in-line and display math formats.

Thank you @dpvc,
You provided all the  information needed to fix the issue downstream.
Closing issue.


refactor(common): pull RCC clock select into the common directory

Overview
We shouldn't have the RCC clock select function be dependent on UART. This PR separates that function, and also error handler functions into their own C files that can be used as needed.
Review Requests
Are we OK with the file locations/naming/functionality as is? Anything you'd like changed?

Is this something that can actually be shared?
As implemented right now this is controlling peripheral clock gating for every peripheral on the system. For it to be shared means either

every project gates on the clocks of the exact same peripherals, no matter whether they need them or not
this only handles some core set of peripherals, or just creates the periphclkinit struct with that data and returns it for a later call or something, or takes a struct with uncommon stuff and fills it in.

Some of this is definitely common, like the can; some of it may not be, like that specific i2c bus driver.
Maybe it's something that we share for now and take back later I guess but this is going to become really painful as soon as these places diverge unless it becomes something that either takes or provides the clock init struct.

What about if we make this function a little more versatile and allow each project to pass in the type of clocks it needs by using a struct?

Is this something that can actually be shared?
As implemented right now this is controlling peripheral clock gating for every peripheral on the system. For it to be shared means either

every project gates on the clocks of the exact same peripherals, no matter whether they need them or not
this only handles some core set of peripherals, or just creates the periphclkinit struct with that data and returns it for a later call or something, or takes a struct with uncommon stuff and fills it in.

Some of this is definitely common, like the can; some of it may not be, like that specific i2c bus driver.
Maybe it's something that we share for now and take back later I guess but this is going to become really painful as soon as these places diverge unless it becomes something that either takes or provides the clock init struct.

What about if we make this function a little more versatile and allow each project to pass in the type of clocks it needs by using a struct?

Sure we definitely could but also... what does this function now do? It's a function that takes a struct and then just immediately passes that struct to another function. What does it gain us?


Do not create workspace if overridden by environment TF_WORKSPACE

Describe the Bug
In our deployment pipeline, we create the TF workspace via HTTP API as need to configuration not possible from backend.tf, e.g. working-directory, global-remote-state, etc.
Once the workspace is created/checked, it sets the TF_WORKSPACE variable.
In setting Atmos for the first time, found what I think is incorrect behaviour.
Terraform has been successfully initialized!

Command info:
  Terraform binary: terraform
  Terraform command: plan
  Arguments and flags: []
  Component: ecr/redacted-app
  Terraform component: ecr
  Stack: redacted-dev
  Working dir: components/terraform/ecr

Executing command:
/usr/bin/terraform workspace select long_complex_workspace_name_redacted
The selected workspace is currently overridden using the TF_WORKSPACE
environment variable.
To select a new workspace, either update this environment variable or unset
it and then run this command again.

Executing command:
/usr/bin/terraform workspace new long_complex_workspace_name_redacted
Workspace "new long_complex_workspace_name_redacted" already exists
exit status 1

Expected Behavior
If the workspace is overridden by TF_WORKSPACE, then atmos should accept that it is managed elsewhere and don't try create it again.
Steps to Reproduce
Run atmos with TF_WORKSPACE environment variable set.
Screenshots
No response
Environment

OS: Windows 11
Atmos: 1.83.1
Terraform: 1.8.2

Additional Context
I think we could change the error handling in ~L394
https://github.com/cloudposse/atmos/blob/8060adb59caa71b5b4ca3a489844070478feb361/internal/exec/terraform.go#L382-L409

Unfortunately, the terraform workspace select doesn't return a nuanced exitCode for this event.
https://github.com/hashicorp/terraform/blob/5868f994139db04a5f6989491f1dbb84486873ac/internal/command/workspace_select.go#L54C2-L58C3
So might need to add a check to see if TF_WORKSPACE is set, e.g. https://github.com/hashicorp/terraform/blob/125cf53470a3e9ada9e1c79cb077c3ca36a06607/internal/command/meta.go#L784C1-L801C2

I've workaround this for now by unsetting the TF_WORKSPACE environment and then using terraform_workspace_template to override the workspace.
Note, terraform_workspace_template is not documented at https://atmos.tools/core-concepts/components/terraform/workspaces#terraform-workspace-override-in-atmos

@aknysh that should work.
As I said, I've worked around this by using the terraform_workspace_template override; this might be a better solution as then we can plan stacks outside of the CICD.
Whilst searching for other issues, I just found a discussion on TF_WORKSPACE that might be relevant/past decision: https://github.com/cloudposse/atmos/issues/192

Fwiw, we snuck the requested support into this PR

#654


feat: Self-adjoint LinearPMaps are densely defined

Define the star on LinearPMap as the adjoint and prove that every self-adjoint operator is automatically densely defined.

bors r+
Thanks!


SSL Certificate Setup

Follow the steps in here:
https://apps.nrs.gov.bc.ca/int/confluence/display/DEVGUILD/Generating+a+CSR#GeneratingaCSR-oc-secret

@dleard as an update, I'm still waiting to hear about about a missing code we need to submit this. But I've escalated it so hopefully we'll get it by EOD so that they can submit this request. cc @hannavovk

@dleard @hannavovk  FYI I got the missing coding so they are moving it along now

@dleard they just sent over the SSL certificate to be installed (via email). cc @hannavovk @andrea-williams

On hold until January - middletier is trying to troubleshoot.

Middletier response on Dec 19 via email:
"The SSL certificate for “industryemissions.gov.bc.ca” has been cross verified again and there is no issue on the attachments forwarded. FYR, please find the below screenshot. ... [in email]
The site https://industryemissions.gov.bc.ca/ has been configured with the wrong certificate. Please reach out to the below mentioned contacts regarding configuring certificates for OpenShift."
...
"the certificate is issued to logontest7.gov.bc.ca and but should be industryemissions.gov.bc.ca"

Update re: domain name. Changing it to industrialemissions.gov.bc.ca rather than industryemissions.gov.bc.ca. This change will be submitted as we also figure out the issue with the SSL cert the way it was setup by the middle tier team.

new domain name has been approved (industrialemissions.gov.bc.ca) and we can proceed with generating the .csr and submitting it to middletier services via jira. cc @dleard

We have received the ssl certs to be installed. @dleard @hannavovk

SSL Cert setup documentation has been recorded here: https://bcgov.sharepoint.com/:w:/r/teams/00608-ScrumTeam/Shared Documents/Scrum Team/OBPS/Domain Setup/SSL_Certificate_Documentation.docx?d=w3140fee140ef4eb2abf7615a53fb391e&csf=1&web=1&e=QQhH8C


fixes build issue with gcc 4.8+

Since stout is part of Mesos I am not sure we'd want to suppress all warnings from it instead of fixing them. What's the issue this patch addresses?

it was complaining about auto_ptr being deprecated

@bbannier this issue was a fixed a while back in Mesos:
https://issues.apache.org/jira/browse/MESOS-3799
So needs to be fixed in the mesos-modules as well since we are directly inheriting the headers.

@asridharan  I earlier did it only for boost but then stout complained about some other stuff forgot what was it about.

What are the actual error messages you're seeing in stout?

It was again complaining about auto_ptr

Seems this PR is in limbo without clear motivation and reviews. It sounds a lot as if the motivation was not explained sufficiently.
Can we have

ticket showing the error/s in full
agree on the most appropriate solution for (1)
have a PR narrow and verbose enough to solve just that

The complete error can be found https://jira.mesosphere.com/browse/DCOS_OSS-1422

The issue was fixed as a part of https://github.com/dcos/dcos-mesos-modules/pull/36, please see https://github.com/dcos/dcos-mesos-modules/pull/36/files#diff-67e997bcfdac55191033d57a16d1408aR224


Parsing the text field

I was hoping someone could help. I am trying to find all the problems associated with Windows 10 versions so for example I run the code: python Exporter.py --querysearch "Windows 10 & 1909 & problems" --maxtweets 100. This returns the CSV output file and everything is in the text column. I would like to seperate them so that Windows 10 is in one column, 1909 (or any other version I query) is in another column and lastly, so the problems are listed in another separate column. Is this possible? The date is the only other thing that I need, I don't need username, geo, ID, etc

I don't think it's a Windows bug. Try opening CSV with LibreOffice Calc for example - Choose coma as separator and you will have your columns


[vmdk api adoption] Switch from shadow-test mode to fallback mode

This PR will start using API inflater as the main inflater.
Shadow test mode: daisy inflater as main inflater; API inflater as shadow test.
Fallback mode: api inflater as main inflater; daisy inflater as fallback inflater (expect to be used by non-vmdk).
Changes:

api_inflater: use a flag "isShadowTest" to determine whether to run shadow-test-only logic (calculate checksum for example).
Split "inflater.go" to "daisy_inflater.go" and "inflater.go"; "inflater_test.go" is split accordingly.
Renamed original "inflaterFacade" to "shadowTestInflaterFacade", which handles shadow-test mode; added a new "inflaterFacade" to handle fallback mode. Added tests accordingly.

New logic is only in api_inflater.go and inflater.go. Others are only code movement or trivial changes.

/retest


Having Error [ERR_PACKAGE_PATH_NOT_EXPORTED]

While testing sample code provided for Neo4j graph ( class Neo4jGraph ), I got:
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './graphs/neo4j_graph' is not defined by "exports" in ***\node_modules\langchain\package.json imported from ***
EDIT:
The import statement:
import { Neo4jGraph } from "langchain/graphs/neo4j_graph";
and I tweak the code (trying to use Neo4j Aura) provided here:
https://js.langchain.com/docs/modules/data_connection/experimental/graph_databases/neo4j
Using:
Windows 10
Node: 18.16.0
Langchain<EMAIL_ADDRESS>| MIT | deps: 22 | versions: 167

@dosu-bot
The import is like:
import { Neo4jGraph } from "langchain/graphs/neo4j_graph";
and I tweak the code provided here:
https://js.langchain.com/docs/modules/data_connection/experimental/graph_databases/neo4j

@dosu-bot
I added your code to package.json, even restarted VSC but still having the error.
I even tried it without "/src" like this:
    "./graphs/neo4j_graph": {
      "import": "./graphs/neo4j_graph.js",
      "require": "./graphs/neo4j_graph.cjs"
    },

but having the same error (again even after VSC restart)

I've just tried it and it seemed to work ok - can you paste your package.json file here?

You'll have to populate your own secrets but here's a Repl showing it<EMAIL_ADDRESS>

@jacoblee93
pls. note since the original error referred to:
***\node_modules\langchain\package.json imported from ***
I made the modifications in THAT package.json
Your one looks like the project's package.json, but I modified the referred one, since dosu-bot didn't referred another.
My one is:
{
  "type": "module",
  "dependencies": {
    "dotenv": "^16.3.1",
    "langchain": "^0.0.152",
    "neo4j-driver": "^5.13.0",
    "typescript": "^5.2.2"
  },
  "devDependencies": {
    "@types/js-yaml": "^4.0.6"
  }
}

You'll have to populate your own secrets but here's a Repl showing it<EMAIL_ADDRESS>@jacoblee93
Basically using the same code, having this at the beginning:

import dotenv from 'dotenv';
dotenv.config();

pls. note my js files in src directrory of the project, which is pretty usual I think

@jacoblee93 pls. note since the original error referred to: ***\node_modules\langchain\package.json imported from *** I made the modifications in THAT package.json Your one looks like the project's package.json, but I modified the referred one, since dosu-bot didn't refer another. My project's one is:
{
  "type": "module",
  "dependencies": {
    "dotenv": "^16.3.1",
    "langchain": "^0.0.152",
    "neo4j-driver": "^5.13.0",
    "typescript": "^5.2.2"
  },
  "devDependencies": {
    "@types/js-yaml": "^4.0.6"
  }
}

It looks like you're using "langchain": "^0.0.152", - can you try updating it to latest (0.0.167)?

@jacoblee93
No, I don't, like I posted in OP it is:
Langchain<EMAIL_ADDRESS>| MIT | deps: 22 | versions: 167
Project's package.json was created when I first installed LCh JS, when 0.0.152 was the latest version, long ago

Is it possible the version being used is the wrong one at runtime?
Can you try from a fresh project?

@jacoblee93
The same code works in a new project!
Other differences:

JS code run from project directory, not from src dir
project's package.json:

{
  "type": "module",
  "dependencies": {
    "dotenv": "^16.3.1",
    "langchain": "^0.0.167",
    "langsmith": "^0.0.42",
    "neo4j-driver": "^5.13.0",
    "openai": "^4.12.1",
    "tiktoken": "^1.0.10"
  }
}

When I first setup the project, I planned to use ts files from src dir and compile them to dist, but later I met some issues and  changed my code file extensions from ts to js in that src dir.
Later I'll close issue, thank you!

If it works from a fresh install, it must be a used version mismatch - going to close this. Thanks again!


test client crashing at rtpsession-rtcp when running at ARM target

Hi,
We are trying to run the test client application with custom gst sources.
When executing, the application is always crashing at the thread "rtpsession-rtcp"?
The log is below.
0:00:14.094600997 10012  0x1845550 INFO                 default owr_utils.c:198:_owr_deep_notify: transport-agent-1/transport_bin_1/send-output-bin-2/dtls_srtp_rtp_encoder_2/funnel/funnelpad5/caps = application/x-srtcp, srtp-key=(buffer)bb5a6dd769c9c367da0a54997b16979e6e4cf938deaa9b13ed334d813333, srtp-cipher=(string)aes-128-icm, srtp-auth=(string)hmac-sha1-80, srtcp-cipher=(string)aes-128-icm, srtcp-auth=(string)hmac-sha1-80
0:00:14.095089573 10012  0x1845550 INFO                 default owr_utils.c:198:_owr_deep_notify: transport-agent-1/transport_bin_1/send-output-bin-2/dtls_srtp_rtp_encoder_2/srtp-encoder/rtcp_sink_2/caps = application/x-rtcp
0:00:14.095565967 10012  0x1845550 INFO                 default owr_utils.c:198:_owr_deep_notify: transport-agent-1/transport_bin_1/send-output-bin-2/dtls_srtp_rtp_encoder_2/rtcp_sink_2/caps = application/x-rtcp
0:00:14.096030437 10012  0x1845550 INFO                 default owr_utils.c:198:_owr_deep_notify: transport-agent-1/transport_bin_1/send-output-bin-2/rtcp_output_selector_2/sink/caps = application/x-rtcp
0:00:14.096487543 10012  0x1845550 INFO                 default owr_utils.c:198:_owr_deep_notify: transport-agent-1/transport_bin_1/send-output-bin-2/rtcp_sink_2/caps = application/x-rtcp
0:00:14.096949073 10012  0x1845550 INFO                 default owr_utils.c:198:_owr_deep_notify: transport-agent-1/transport_bin_1/rtpbin/send_rtcp_src_2/proxypad34/caps = application/x-rtcp
0:00:14.097296876 10012  0x1845550 INFO               GST_EVENT gstevent.c:760:gst_event_new_segment: creating segment event time segment start=0:00:00.000000000, offset=0:00:00.000000000, stop=99:99:99.999999999, rate=1.000000, applied_rate=1.000000, flags=0x00, time=0:00:00.000000000, base=0:00:00.000000000, position 0:00:00.000000000, duration 99:99:99.999999999
Segmentation fault
Have any one faced the issue and fixed?
Thanks.

How the rtpsession-rtcp thread is invoking while running the test-client application? The application is crashing.

All team members are currently on holiday this week and next. We'll get back to you as soon as we can.

Ok.


implement refresh

hi frist of all thanks for your script modifications and Commitments. It's cool
the bot sometime shows errors when when servers overloads. recently it shows repaint error "unauthorized"
please modify script to hit refresh when errors like that appears. it fails to paint and claim but goes to sleep

hi frist of all thanks for your script modifications and Commitments. It's cool
the bot sometime shows errors when when servers overloads. recently it shows repaint error "unauthorized"
please modify script to hit refresh when errors like that appears. it fails to paint and claim but goes to sleep

I’ve added a new feature for selecting a less popular template, which significantly increases the chances of getting triple rewards.
Regarding the errors, particularly the "unauthorized" issue that occurs when the servers are overloaded: the automatic sleep after errors has been disabled. This approach worked well when errors were rare, but now, with sessions frequently ending before the timer is up, it causes more problems. Now, if two request attempts fail, the current function terminates before the next cycle. This has proven to be the most effective approach from all that I've tested.
Looking forward to your feedback!

thank you very much for explaining the issues and updating the bot!  I've just tested and it seems way better than before!  I'll attach all my session and leave it working now to see how it goes. I'm glad to hear from you! thanks again


Firstdraft for shopify M1

@FrankAuwers I added comments that are important to fix before we release the app.
@grobyns @AndersLarsenMicrosoft - feel free to add if something is missing.

Code processed in other PRs. Closing this PR as we don't need to merge it.


NO-ISSUE: Sample DMN not compatible with classic DMN Editor

When opening the sample.dmn file using the classic DMN Editor, it's now incompatible due to the updated dmn:definitions. This PR Reverts them back to the 2018 versions, so that users can interact with the sample.dmn in both the new and classic DMN Editor

Ubuntu-latest 2 failing due to one test. Said test is failing in multiple PRs, and passes when run locally

@kbowers-ibm great catch. It looks like the branch you used wasn't from main, as it has many commits from other PRs.

It was originally, but when I did the 3rd commit I messed up the versioning, and couldn't figure out how to revert it. Best I could do was make it so that it only shows the files I changed!

What I suggest is to rename this branch in your local to another name git branch -m <new_name>, get your commits SHA with git log, create a new branch from main with the same name as the one used in this PR no-issue-rever-sample-dmn-definitions. Apply the commits to this new branch using git cherry-pick <commit_SHA>, and make a push force. With the push force you will fix the commit history, but double/triple check if the commits are correct before you do it.

It worked, thank you very much @ljmotta! I've noted that process down for future reference :)


how do I join the team "Jigsaw Operation LLC"?

I have developer account, but I don't know how to do the following?
does some give me a picture demo?
Click "Outline" in the left navigation bar:
Under "Signing" select "Jigsaw Operations LLC"

Hi @woodymoo, I recommend you create an App ID (with the appropriate entitlements) for the project on your Apple developer account and modify the Outline and VpnExtension targets in the generated Xcode project to match the ID. You will have to register the device(s) you want to use for development. You may also need to change the code signing options to match your development team.
Let me know if that works, and feel free to re-open this issue with further questions.


[v1.3] As a user, I want to be able to make remarks on my contacts

..so that I can make temporary/permanent descriptions for my contacts about things to take note of.

[x] Implement remark field
[x] JUnit Tests

https://github.com/CS2103AUG2017-W09-B2/main/pull/74


Update forum references

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

[secure]
GitHub release

Your semantic-release bot :package::rocket:


CommandHandler notifications called out of order

In some cases, event notifications for reconnection CommandHandler are called out-of-order.
Adding a command handler id makes visible what's going on:

The channel is deactivated
The channel is reconnected
The channel receives an activation
The channel is unregistered (notification from the previous channel I/O)

2016-07-23 20:58:54 [DEBUG] [lettuce-nioEventLoop-6-1] [CommandHandlerId 0x31e, /<IP_ADDRESS>:36367 -> localhost/<IP_ADDRESS>:6479] deactivating channel handler (CommandHandler:708)
2016-07-23 20:58:54 [DEBUG] [lettuce-nioEventLoop-6-2] [CommandHandlerId 0x31e, (inactive)] channelRegistered() (CommandHandler:113)
2016-07-23 20:58:54 [DEBUG] [lettuce-nioEventLoop-6-2] [CommandHandlerId 0x31e, /<IP_ADDRESS>:36371 -> localhost/<IP_ADDRESS>:6479] channelActive() (CommandHandler:583)
2016-07-23 20:58:54 [DEBUG] [lettuce-nioEventLoop-6-1] [CommandHandlerId 0x31e, /<IP_ADDRESS>:36371 -> localhost/<IP_ADDRESS>:6479] channelInactive() done (CommandHandler:733)
2016-07-23 20:58:54 [DEBUG] [lettuce-nioEventLoop-6-1] [CommandHandlerId 0x31e, /<IP_ADDRESS>:36371 -> localhost/<IP_ADDRESS>:6479] channelUnregistered()

Fixed.


[BEAM-11936] Remove suppression in ReadFromKafkaDoFn

I got carried away and merged #15890 before we addressed this.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @benWize


cmd/compile: invalid typeparameters code can be compiled

What version of Go are you using (go version)?

$ go version
go version devel go1.18-4d550727f8 Mon Oct 18 16:33:42 2021 +0000 darwin/arm64

Does this issue reproduce with the latest release?
Yes, with the tip version, not G1.17
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/yoshiki.shibata/Library/Caches/go-build"
GOENV="/Users/yoshiki.shibata/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/yoshiki.shibata/gocode/pkg/mod"
GONOPROXY="github.com/kouzoh/*"
GONOSUMDB="github.com/kouzoh/*"
GOOS="darwin"
GOPATH="/Users/yoshiki.shibata/gocode"
GOPRIVATE="github.com/kouzoh/*"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/yoshiki.shibata/tools/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/yoshiki.shibata/tools/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="devel go1.18-4d550727f8 Mon Oct 18 16:33:42 2021 +0000"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/5y/6k1nfy3n0sn9h4g61bp36wv40000gp/T/go-build1538714687=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version devel go1.18-4d550727f8 Mon Oct 18 16:33:42 2021 +0000 darwin/arm64
GOROOT/bin/go tool compile -V: compile version devel go1.18-4d550727f8 Mon Oct 18 16:33:42 2021 +0000
uname -v: Darwin Kernel Version 20.6.0: Mon Aug 30 06:12:20 PDT 2021; root:xnu-7195.141.6~3/RELEASE_ARM64_T8101
ProductName:	macOS
ProductVersion:	11.6
BuildVersion:	20G165
lldb --version: lldb-13<IP_ADDRESS>
Swift version 5.5-dev

What did you do?
Tried to compile the following code:
// This type is INVALID.
type P[T1, T2 any] struct {
	F *P[T2, T1] // INVALID; must be [T1, T2]
}

What did you expect to see?
Compile error
What did you see instead?
No compile error
Type Parameters Proposal says:

A generic type can refer to itself in cases where a type can ordinarily refer to itself, but when it does so the type arguments must be the type parameters, listed in the same order. This restriction prevents infinite recursion of type instantiation.

// List is a linked list of values of type T.
type List[T any] struct {
	next *List[T] // this reference to List[T] is OK
	val  T
}

// This type is INVALID.
type P[T1, T2 any] struct {
	F *P[T2, T1] // INVALID; must be [T1, T2]
}

This seems only matters when type parameters have different constraints:
package p

type I interface {
	int
}

// This type is INVALID.
type P[T1 I, T2 any] struct {
	F *P[T2, T1] // INVALID; must be [T1, T2]
}

fails with:
rpgo:9:7: T2 has no constraints

Thank you for the report.
We're tracking this in #48098, so closing as a dupe.
CC @griesemer @mdempsky


OIDC Login fails with express-session Error

OIDC Login fails
Summary
I set up scicat in a kubernetes deployment and
want to use Keycloak as OIDC auth provider.
I turned off the login fields and configured the paths in the
frontend and the provider in providers.json
but when I click on the sign in with Keycloak button I get a 500 Internal Server Error:
Unhandled error for request GET /auth/keycloak: Error: OpenID Connect requires session support. Did you forget to use `express-session` middleware?
    at SessionStore.store (/home/node/app/node_modules/passport-openidconnect/lib/state/session.js:39:33)
    at Strategy.authenticate (/home/node/app/node_modules/passport-openidconnect/lib/strategy.js:358:24)
    at attempt (/home/node/app/node_modules/passport/lib/middleware/authenticate.js:366:16)
    at authenticate (/home/node/app/node_modules/passport/lib/middleware/authenticate.js:367:7)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/node/app/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/home/node/app/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at /home/node/app/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/home/node/app/node_modules/express/lib/router/index.js:335:12)
    at next (/home/node/app/node_modules/express/lib/router/index.js:275:10)
    at urlencodedParser (/home/node/app/node_modules/body-parser/lib/types/urlencoded.js:91:7)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/home/node/app/node_modules/express/lib/router/index.js:317:13)
    at /home/node/app/node_modules/express/lib/router/index.js:284:7
    at Function.process_params (/home/node/app/node_modules/express/lib/router/index.js:335:12)

Steps to Reproduce
my providers.json  snippet is:
        "keycloak": {
            "session": false,
            "provider": "oidc",
            "authScheme": "openid connect",
            "module": "/home/node/app/server/boot/oidcCompatibleStrategy",
            "authPath": "/auth/keycloak",
            "issuer": "https://XXXX/auth/realms/testing",
            "clientID": "XXXXX,
            "clientSecret": "XXXXX",
            "callbackURL": "/auth/keycloak/callback",
            "authorizationURL": "https://XXXX/auth/realms/testing/protocol/openid-connect/auth",
            "tokenURL": "https://XXXX/auth/realms/testing/protocol/openid-connect/token",
            "userInfoURL": "https://XXXX/auth/realms/testing/protocol/openid-connect/userinfo",
            "scope":   ["openid","profile","email"],
            "callbackPath": "/auth/keycloak/callback",
            "successRedirect": "/user",
            "failureRedirect": "/login",
            "failureFlash": true
      }

Current Behaviour
no redirect to Keycloak, but internal server error.
Expected Behaviour
redirect to OIDC Provider (Keycloak) and after successful authentication redirect back to scicat.
Extra Details
Here you should include details about the system (if it is unique) and possible information about a fix (feel free to link to code where relevant). Screenshots/GIFs are also fine here.

I am getting the same error when following the tutorial for google OIDC: https://scicatproject.github.io/documentation/Development/OIDC.html
This is with docker.

So. I looked into the code and:
server/server.js lines 20+
// express sessions are required for passport ocid. If your
// config.Local has a session for the expressionSecret, configure
// express-session
if (configLocal.expressSessionSecret){
  var session = require("express-session");
  app.use(session({
    secret: configLocal.expressSessionSecret,
    resave: false,
    saveUninitialized: true
  }));
}

So it seems like expressSessionSecret needs to be set in config.local.js.
A first test confirms that and now gives different errors
Unhandled error for request GET /api/v3/Users/62fb4944304d14f82fc002cb: Error: Authorization Required
but the Keycloak part seems to be working.
Cheers

That worked for me as well. @belfhi maybe we should keep this issue open so that we can request for better documentation in the tutorial about setting the express session secret?

Never mind. It looks like it is in the documentation.

Yes, my PR was merged after I found out about the issue 😅

Do you know what solved this: /api/v3/Users/62fb4944304d14f82fc002cb: Error: Authorization Required?

Try this callback function
"use strict";

exports.accessGroupsToProfile =
  function (req, done) {
    return function (err, user, identity, token) {
      identity.updateAttributes({
        "profile": {
          accessGroups: identity.profile._json.groups,
          email: identity.profile._json.email,
          ...identity.profile
        },
        "credentials": null });
      var authInfo = {
        identity: identity,
      };
      if (token) {
        authInfo.accessToken = token;
      }
      done(err, user, authInfo);
    };
  };

In keycloak you need to map the users' groups to the groups claim
and then login should work. Don't forget to reference the function in the providers.json
file


Renamed 'Commit duration' into 'Build duration' in Charts

In the Charts page it makes more sense to rename "Commit duration in minutes for last 30 commits" to "Build duration in minutes for last 30 commits", since we are building commits.

We can have N builds per commit. And on charts we show sum of all builds. So it cant be Build duration because single build can take 20-30% of total all builds in one commit

I was not aware of that, since I am using only one build per commit. Either way, "Commit duration..." still does not sound right to me. Maybe something in the direction of "Cumulative build duration"?

I have feeling Cumulative build duration will be even more confusing. Maybe something like Summary builds duration or Builds duration?

Builds duration also sounds nice. Or maybe even Jobs duration to be consistent with items in menu?

@nejctrdin we removed Jobs in 7.12.0.pre :)  Maybe Builds duration will be a good choice so far

Gitlab CI has been merged with Gitlab CE. We don't use this project to merge new code anymore. If you still think this code needs to be in Gitlab CI, I urge you to re-open a PR with this code on the Gitlab CE project.


fix: fix clippy

comment fix-ci

clippy emits a false-positive error on a mutating code in datadriven.
it's known issue: https://github.com/rust-lang/rust-clippy/issues/13185

Thank you!


Add ReplicationOptions to SessionConfig for excluding keyspaces from replica precompute

We add ReplicationOptions which includes a vector of keyspace names to SessionConfig. When instantiating ClusterData, we exclude keyspaces from provided by ReplicationOptions when computing PrecomputedReplicas to conserve memory/compute resources.
Fixes: #671
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[ ] I have provided docstrings for the public items that I want to introduce.
[ ] I have adjusted the documentation in ./docs/source/.
[ ] I added appropriate Fixes: annotations to PR description.

@havaker would you mind taking a quick look? Happy to write tests and finish up docs if this seems correct.

@Lorak-mmk please review

You can already choose which keyspaces to precompute (see keyspaces_to_fetch) in SessionConfig, but:

it is an allowlist, while your approach is denylist. I'm wondering if maybe the best approach is to allow both (make enum Allowlist(Vec) and Denylist(Vec) instead of just Vec).
It prevents info about other keyspaces to be fetched at all (so it's not available in ClusterData), while your solution only avoids precomputing replication strategies.

So I think this change may be valuable. I'll try to post initial review soon.

@michaelhly Sorry for so delayed response.
I really like the idea. The taken approach is correct: ReplicationConfig (a small name change to be in line with other Configs) should be passed down to ClusterData::new().
As @Lorak-mmk noted, there should be both an Allowlist and Denylist variant for specyfing keyspaces to precompute.

Thank you for contributing the idea @michaelhly!


Webhook for determining user's group membership

Decouple Group Population from Authentication

One-line feature description (can be used as a release note): User group membership can be determined by a webhook seperately from authentication.
Primary contact (assignee): @ericchiang
Responsible SIGs: @kubernetes/sig-auth-feature-requests
Design proposal link (community repo): https://github.com/kubernetes/kubernetes/issues/23720
Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred:

@liggitt
@deads2k
@cjcullen

Approver (likely from SIG/area to which feature belongs):
@liggitt
Feature target (which target equals to which milestone):

Alpha release target (1.7)
Beta release target (x.y)
Stable release target (x.y)

After internal debate I don't think I'm going to be pursuing this for 1.7. Much of this can be achieved through token access reviews or bulk syncing to RBAC roles. Closing this out.


[MemoryLeak] DCMTKImageReader and INRImageReader -- master

Same as https://github.com/medInria/medInria-public/pull/1119
"Solve some memory leaks found with valgrind when a user import a data in medInria. In DCMTK reader, fix "new" deletion, and in INR reader fix private class leak."
:m:

This PR has been switched from master branch to dev


Import dowhy.causal_estimators.econml locally

This imports it within the function where it's used. It removes the hard dependency on EconML and only requires it when using sensitivity_non_parametric_partial_r2.
Addresses:

https://github.com/py-why/dowhy/issues/712

thanks for the quick turnaround, @petergtz This is a good idea, but now I see that isort is now throwing an error. Would adding "#noqa" help here, or there another preferred way?

@amit-sharma Ran:
poetry run poe format

Should be fixed now. Let's see.


[Feature] Allow for the secret config to point to forwarding.secret file

I like to keep my config files in source control. Currently I have to keep FabricProxy-Lite.toml in my .gitignore file because it contains the private key. If I could configure FabricProxy-Lite to read the secret directly from a file, I could then point it to the forwarding.secret so the config file doesn't contain any secret data.
Thanks! :)

Currently supports setting secrets through environment variables FABRIC_PROXY_SECRET.

I use velocity in docker and I'd love to be able to add that one file to both containers without fussing with ansible.

@ryleu If you're using itzg/minecraft-server for the server image and itzg/mc-proxy for Velocity, it's already possible.
Both of these images allow for interpolation in the config files in order to replace env-vars with values.
I use this in conjunction with the optional config mount-point, and the following env-vars in the docker-compose.yml and .env files. Note! the forwarding secret requires the double quotes around the value.:
environment:
  - REPLACE_ENV_DURING_SYNC=true
  - REPLACE_ENV_VARIABLES=true
  - CFG_FABRIC_PROXY_FORWARDING_SECRET="$VELOCITY_FORWARDING_SECRET"
  - CFG_FABRIC_PROXY_HACK_EARLY_SEND=true
  - CFG_FABRIC_PROXY_HACK_MESSAGE_CHAIN=true

Then in the FabricProxy-Lite.toml file which is inside a persistent volume to the containers /config dir, you'd set the variables like so:
hackOnlineMode = true
hackEarlySend = ${CFG_FABRIC_PROXY_HACK_EARLY_SEND}
hackMessageChain = ${CFG_FABRIC_PROXY_HACK_MESSAGE_CHAIN}
disconnectMessage = "This server requires you to connect with Velocity."
secret = ${CFG_FABRIC_PROXY_FORWARDING_SECRET}

These images are pretty neat, because they allow for automation like this, and e.g. installing mods/plugins automatically from CurseForge, Modrinth, or any site with a direct .jar download link.
Documentation:

itzg/minecraft-server - Modifying config files
itzg/minecraft-server - Optional plugins, mods, and config attach points
itzg/mc-proxy - Volumes
itzg/mc-proxy - Replacing variables inside configs


Add estimateGas method

Context / issue
Users want to know if a transaction will fail before it is executed.
if (!safeSdk.estimateGas(transaction)) {
  showWarning()
}

Proposed solution
Add a estimateGas(transaction) method to the Safe class that calls this method.

This one would be very useful for us too!

This was replace with #279


[BEAM-4028] Adding NameContext to Python SDK.

Steps can have different names depending on the runner (stage, step, user, system name...). Depending on the needs of different components (operations, logging, metrics, statesampling) these step names are passed around without a specific order.
This change starts by creating name context classes and passing them into operations. Next steps are:

Migrating operations away from using their operation names, and removing them from the object definition in operations.pxd, and clean all the operation creating logic.


feat: Removed "above" from label-commenter-config.yml

Notes for Reviewers
This PR fixes #8293
Signed commits

[ ] Yes, I signed my commits.

Hii @abhijeetgauravm
Let's discuss it on the websites call.
Please add this as an agenda item in the meeting minutes, if you would :)
https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.lohhtewfwima

@Ghat0tkach Sure!


Invisible element block tapping the 2nd row

Describe the bug
When opening the popup, there is an invisible element that blocks the tapping of buttons on the 2nd row. To tap those buttons, the user has to tap the bottom of them. Using the browser developer tool, I can see that there is an invisible element that blocks the action.

As you can see from the screenshot, that bubble-card::after blocks almost all the area of the 2nd row, making tapping any buttons on it difficult.

Hi and thank you for your feedback! Indeed this element is for the top gradient, I have a separator in the top of all my pop-ups so I've never had this issue, but I will fix that in the next release.
As a temporary workaround you can add this to your pop-up YAML:
styles: |
  bubble-card::after {
    pointer-events: none;
  }

Thanks, I was having the same issue and this fixed it!

This is fixed in the v2 beta!
Here is the link to the full changelog:
https://github.com/Clooos/Bubble-Card/releases/tag/v2.0.0-beta.1


Added beginners section

Here is the live link: https://codedude1.github.io/juniorninjas.github.io/

Still, have to make changes to the course section of the home page.

TODOs:

Remove "Reviews" section
Replace "Trainer’s Name ..." with Course duration.
Add some cool image for beginners main-image

Only this section remains to be updated! Will it be Okay to add it to the courses page or shall I add it to the home?

Only this section remains to be updated! Will it be Okay to add it to the courses page or shall I add it to the home?

Yes, please add this in courses page, might be as footer.

Can we make "Course Outline" items as look like a list with numbers or bullets?


mm_ubsan: add implement for dynamic_type_cache_miss

Summary
When try to use local ubsan to bypass the ubsan in host native, find dynamic_type_cache_miss was not catched by default
Impact
can re-direct dynamic_type_cache_miss from native to local after merge.
Testing
CI-test, local ubuntu sim.

CI failing by
Failed to connect to github.com port 443 after 133600 ms: Connection timed out


Ae setup command

review @jsingle  command to setup user initially

Would be nice to be able to automatically set up some of these config keys. In our install script, we'll want to set the dusty-specs repo to our gc dusty specs repo.  Can't see how to automate that with the current setup...

:fireworks:


The MIT license is not present on your repository

On your "readme", you say it is dual licensed, but the MIT license is absent from your repository.
Thanks,
Guillaume

Thanks for noting. I believe Github has problems detecting dual licenses. It is however clearly stated in the README file.


[examples] improve core_input_virtual_controls example

Improved core_input_virtual_controls example a lot:

Reviewed code to follow code conventions and match other examples (e.g. use Vector2 instead of float[2])
Move buttons to enum for easier extension
Fixed desktop input (buttons activating even when you don't press them)
Added colored labels for buttons
Fixed button label disappearing when you press it
Replaced player rectangle with ball (to match other input examples)

@danilwhale thanks for the review, I'm merging it but notice it adds extra complexity that can be avoided, no need to get the stri ngs as separate objects or even all the buttons positions (just the middel point). I'll try to review it at some point.


"Hărman" commune is listed in the drop-down menu

Description:
The commune is listed in the drop-down menu according to the law
Preconditions: the website (https://peviitor-ro.github.io/ui.orase/) is up and runing.
After every step delete the text you typed from the search bar.

Step 1
Type "Hărman" in the search bar.
Expected results
The location is listed in the drop-down menu as "Hărman, Brașov"


+2 resetting to false after inspection.

When the inspection time goes down to -1 or -2, +2 (as expected) changes to true. When inspection is left to enter the "timing mode," though, +2 resets back to false. I'm fairly certain this isn't due to a misconfiguration on my part, but I'll test it in some other instances too.

I'll add some more information as to my specific situation (not sure how this is going to format, I'll update it if it turns out odd).
Settings:
const settings = {
timerInput: 'timer',
inspection: 'always',
timerUpdate: 'deciseconds',
timeToRelease: 'stackmat',
};
Callback:
const keyboardTimerCallback = (time, penalties) => {
setTimes([...times, {time, penalties}])
}
Hook:
const { time, inspectionTime, state, isTiming, dnf, plusTwo } = useKeyboardTimer(settings, keyboardTimerCallback);

I opened this a long time ago but it I realized I was not using the library properly. Of course it should reset back to false. There's no issue.


Adding unit tests

This PR does the following:

add Petri -> ODEProblem conversion
refactor Petri.jl into multiple files

types.jl, the type definitions
stochastic.jl the conversion of Model -> stochastic simulation
ode.jl, the conversion of Model -> ODEProblem
metaprogramming, tools for working with Expr trees

add example
add tests
add CT tests that aren't included because of dependencies

An open question is what to do about testing the interaction between SemanticModels.jl and Petri.jl. I'm inclined to put all the code for the integration into SemanticModels.jl in a submodule. Pkg.jl will support multiple packages in one repository, which will make this easier.

It looks good other than it breaking the testing, could you fix the errors with travis? I believe its just missing some packages ("LabelledArrays") at least. Once we get it passing testing I think it can go to master.

Yeah, how do I do "test time" dependencies again?

I copied it from SemanticModels.

I think this is ready to go. Probably a good idea to squash and merge


5.4.0 did not find new devices and find old devices double

Hello,
5.4.0 did not find new devices - so I renumberd them and restarted the service and delete all the devices.
That was a mistake, because it still dont find new devices, but i find now all devices doube and none of them are working.
I did shutdown my pi and tested a rediscovery - alexa still shows all devices double. Normaly it should find nothing, because the pi was offline, there must be a cache or something.
Any suggestions how to fix this?
Best regards, Thorsten

I'm having the same problem. I have had mine setup for years. I fixed the port 80 problem with a proxy awhile back. I wanted to added a new device and Alexa could not find it. I upgraded to 5.4.0, deleted all me devices(big mistake), and asked for a check of new devices. No matter how I do it from an echo or from the Alexa app,  it  can't find any of my 59 devices. It can find the thermostat, my two ROKUS and one of my harmony hubs(even thought both seem to work.) I tried shutting down Apache2 and moving the HA-Bridge to port 80. No joy. Although I found for some reason Firefox won't bring up the bridge page when moved to port 80(thank God google chrome did work.) I turned on debug logging on the UPNP and UPNP listen and saw traffic between my echo's and the bridge. I never say my device names in the traffic but they weren't found by Alexa. I use the bridge every day! Please can anyone help.

Update!!! I found that changing the number seed of the devices on the bridge control tab to a number bigger than my last previous number, saving the bridge settings, renumbering the devices on bridge devices page with the renumber button at top of that page, deleting all devices with the Alexa app, and then performing a discovery from the Alexa app will then allow all my devices(even new ones) to be found. Be careful once a device number is used it can't be reused. Don't renumber with the same seed after Alex finds devices using the that seed  or you have to increase the seed above the last used device number and restart the procedure above all over.  I don't know how high the seed can be set and have this work around still work. I discovered this by wondering why there was a renumber button and googling its use. Some bug in original numbering scheme requires renumbering. This was purely emperically discovered, others with a better understanding of the API might better explain this behavior.
Tom

@tvonderhaar
Thanks for the Info. With update my number from 100 to 200 under
Bridge control -> ID Seed (start number...) -> 100 change to 200
alexa found all my devices again
And it works again

Hi,
I did renumber already, with 200, before my first post.
But same here, since today everything is runing like it should. I think Amazon changed something.
Best regards
Thorsten


Submit workflow step params not updated when loading a workflow

Just saw this on branch load_speedup, need to confirm whether this is happening on main as well.

Blocked on #56


Read a Grammar directly from a BNF File

I have found a BNF grammar for BNF itself.  This can be used to hand code a BNFNode tree for that grammar to bootstrap and verify that it can parse the BNF for BNF.
The grammar for BNF does not provide syntax for adding commentary to the grammar.  We can express the commentary as  text in a Markdown file and the grammar itself in fenced code blocks.  Markdown.parse_file can be used to separate the code from the commentary.
How would we include Constructor nodes in the resulting BNFNode tree?
Markdown's fenced code blocks can take a language identifier immediately after the opening three backquotes.  We can use "bnf" for the BNF grammar itself and "julia" for additional code.  The language is available in the language field of a Markdown.Code object.
In trhe Markdown file that defines a grammar, we can thusly specify which constructor code to wrap around which non-terminal's DerivationRule.  Maybe something like
constructor_for("<non-terminal identifier>", constructor_function)

to register the constructor.  The Julia code blocks could be processed before the BNF code blocks so that the constructor functions are known when the grammar is built.

I don't know if there is a way to get "source" locations using Markdown.jl.
Maybe insteaqd we should wrap the BNF code in a BNF_str string macro.  I've verified that __source__ is bound within _str macros.


ClassCastException (not always) for KeyValue async lookup

We are seeing random classcastexception, from the looks of it the reactor callback is return invalid tool. Its not only invalid (or null) but a complete different type of object (here, Integer)
This is a possible issue with Boon or QBit. We will try to reproduce and at a minimum come up with better error reporting.
Exception:
io.advantageous.boon.core.Exceptions$SoftenedException: unable to invoke method public void com.mycompany.push.PushService.process()  on object  com.mycompay.calypso.push.PushService@5b444398 with arguments []
parameter types []
argument types are []

CAUSE java.lang.ClassCastException :: java.lang.Integer cannot be cast to com.mycompany.dashboard.storage.Tool
	at com.mycompany.calypso.push.PushService.lambda$loadToolThenDoOperation$15(PushService.java:859) ~[push-service-1.1-SNAPSHOT.jar:na]
	at io.advantageous.qbit.reactive.async.AsyncFutureCallbackImpl.run(AsyncFutureCallbackImpl.java:121) ~[qbit-core-0.9.2-RC6.jar:na]
	at io.advantageous.qbit.reactive.Reactor.monitorCallBacks(Reactor.java:509) ~[qbit-core-0.9.2-RC6.jar:na]
	at io.advantageous.qbit.reactive.Reactor.process(Reactor.java:137) ~[qbit-core-0.9.2-RC6.jar:na]
	at com.mycompany.calypso.push.PushService.process(PushService.java:580) ~[push-service-1.1-SNAPSHOT.jar:na]
	at sun.reflect.GeneratedMethodAccessor32.invoke(Unknown Source) ~[na:na]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:1.8.0_72-cedar14]
	at java.lang.reflect.Method.invoke(Method.java:498) ~[na:1.8.0_72-cedar14]
	at io.advantageous.boon.core.reflection.impl.MethodAccessImpl.invoke(MethodAccessImpl.java:268) ~[boon-reflekt-0.5.6-RC2.jar:na]
	at

I have not been able to reproduce this yet.


plot_period_transactions with low "s" parameter

Hello,
I have problem with plot_period_transactions for estimated Pareto/NBD parameters.
Parameters:
OrderedDict([('r', 1.2264068998969124),
             ('alpha', 12.16294151917949),
             ('s', 4.6377925306795664e-10),
             ('beta', 32.52434379061591)])

I get this error message:
C:\ProgramData\Anaconda3\lib\site-packages\lifetimes\generate_data.py:105: RuntimeWarning: divide by zero encountered in double_scalars
  time_of_death = stats.expon.rvs(scale=1. / mu)

I think this issue comes with "s" parameter, which is extreamly low. Could you confirm it? Is it bug or I do something wrong?

I've used UCI online retail
I aplied it to cohort - customer which buy first time durring first 3 months.
I tried also other range of date but always s is low.
The same date was used in here and in R
The author also get low s. (2.29118710^{-5})

Are you truncating the transactions at 3 months? Or summarizing the customers, and then truncating?

I choose customers which buy first time durring first 3 months.
Next step is filtring dataset by customerID of them.
Methodology like in CDNOW dataset

"dataset contains the entire purchase history up to the end of June 1998 of the cohort of 23,570 individuals who made their first-ever purchase at CDNOW in the first quarter of 1997."

I tried also other cohort or just all data but always s is very low
I upload notebook:
https://github.com/JJDS0/training/blob/master/lifetimes.ipynb

@CamDavidsonPilon  any ideas?

the s parameter controls how fast people die, so I would do some summary statistics on your training data. Perhaps there is something wrong there.

Hi, @CamDavidsonPilon
Do you have any conclusions why CLV is wrongly calculated for this dataset. Have you made any statistics that can explain the low value of s?
Kind regards,
Leszek


fix(summeruniversity): loosen equality for SU locations

:tada: This PR is included in version 1.33.24 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Push Light the Bridge color into open data

It would be cool to make open data our platform of choice for IFTTT. We could trigger primarily on open data if we can get all of our data in there.

Turns out this data is already there.


Update maven dependencies (README)

Changed from 2.1.15 to 2.1.16

I would skip 2.1.16 for this and merge with 2.1.17 in development.


Implement CSV/sheet view for data

It would be good if we can get a CSV dump or sheet view of all the filtered data in clomonitor
This is related to:
https://github.com/cncf/landscape2/issues/659

Hi @caniszczyk
We already have something similar to this in CLOMonitor 🙂
https://clomonitor.io/data/repositories.csv
(this file can be downloaded from the stats page in CLOMonitor -screenshot attached below-)
The main difference is that, even though all entries specify the corresponding project, the data is classified per repository. The reason it works this way is that this data includes the results of all checks, and they can vary from repository to repository within the same project (i.e. one of the project's repos may pass the DCO check, but other may not).
If this is a problem and you prefer to have a dump with an entry per project, we'd just need to agree on the criteria to summarize the checks across multiple repositories in a single entry for a project.
Some would be straightforward, like the website or roadmap checks, but many others are repository specific and depending on the criteria used the result may be misleading. Some examples would be the license approved or the dangerous workflow checks (there are quite a few more). Should we consider the check passed across the project if it passes in all the project's repositories? In any of them?


Unable to use Error as a base type

I've been trying to get FluentResults to work with Orleans, and I'm super grateful I was able to find your converters. However, I'm still unable to use the Error type from FluentResults as a base type for my own custom errors.
I'm getting an error inside the ErrorSurrogateConverter when it calls Populate. The error is caused by the Reasons field on the supplied Error being null, which happens because when Orleans creates the type to populate it does not use the constructor, thereby avoiding the initialization of the Reasons and Metadata properties on the Error.
Have you been able to successfully use your converters to convert custom error classes that extend Error?

I have used custom errors only with message, without any other properties (i.e. reasons, metadata)
[GenerateSerializer]
internal class QueryReturnedOldDataError : Error
{
    public QueryReturnedOldDataError()
        : base("Query response contains data older than persisted one.")
    {
    }
}

return Result.Fail(new QueryReturnedOldDataError());


[python-package] ValueError when using scikit-learn API for multiclass binary classification

Description
The scikit-learn interface throws an error when used with objective="multiclass" for a binary classification task.
Reproducible example
import numpy as np
import lightgbm as lgb

num_examples, num_features = 20, 3
X = np.random.uniform(size=[num_examples, num_features])
y = np.random.choice([0, 1], size=num_examples)
model = lgb.LGBMClassifier(objective="multiclass", num_classes=2).fit(X, y)
model.predict(X)  # Raises ValueError

Traceback:

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[19], line 1
----> 1 model.predict(X)

File ~/miniforge3/envs/shap/lib/python3.11/site-packages/lightgbm/sklearn.py:1237, in LGBMClassifier.predict(self, X, raw_score, start_iteration, num_iteration, pred_leaf, pred_contrib, validate_features, **kwargs)
   1235 else:
   1236     class_index = np.argmax(result, axis=1)
-> 1237     return self._le.inverse_transform(class_index)

File ~/miniforge3/envs/shap/lib/python3.11/site-packages/sklearn/preprocessing/_label.py:160, in LabelEncoder.inverse_transform(self, y)
    158 diff = np.setdiff1d(y, np.arange(len(self.classes_)))
    159 if len(diff):
--> 160     raise ValueError("y contains previously unseen labels: %s" % str(diff))
    161 y = np.asarray(y)
    162 return self.classes_[y]

ValueError: y contains previously unseen labels: [20]

Environment info
LightGBM version or commit hash: 4.3.0
Command(s) you used to install LightGBM: pip install lightgbm
OS: Linux
Additional Comments
I recognise that using the multi-class objective is unusual in the context of binary classification.
The usual lightGBM interface works as expected:
model = lgb.train(dict(objective="multiclass", num_classes=2), lgb.Dataset(X, label=y))
model.predict(X)  # Succeeds

As a bit of context. I am investigating some issues in the shap project which relate to how the package handles multi-class predictions. There is some inconsistency between the various ML libraries in the shape of returned predictions for binary classification: num_samples x num_classes, or just num_samples.

Thanks for taking the time to report this, and for the excellent minimal, reproducible example!
This is very similar to #3636. But totally fine to keep it open, as it has some other details that that report doesn't have.
Using lightgbm==4.3.0 and numpy==2.0.0, I was able to reproduce this tonight. Adding the text of the ValueError here, so others experiencing this can find it from search engines.
ValueError: y contains previously unseen labels: [20]

I think @RektPunk's PR (#6524) will resolve this, so I recommend subscribing for notifications there to see when that makes it into a release.

I recognise that using the multi-class objective is unusual in the context of binary classification.

It is. And in the case of LightGBM, it'll roughly double the training time and the model's physical size (in memory and on disk), compared to using a binary classification objective, because LightGBM will train 2 trees per boosting round.
But still... the experience here should be better than that hard-to-understand ValueError, so we appreciate the report!

It seems that the reason for the ValueError is related to a shape issue. The value of self.n_classes_ is 2, and suppose that
>>> result
array([[0.45, 0.55],
       [0.45, 0.55],
        ...
       [0.45, 0.55]])

is returned. Depending on the condition, the following value is returned:
>>> np.vstack((1. - result, result)).transpose()
array([[0.55, 0.55, 0.55, ..., 0.45, 0.45, 0.45],
       [0.45, 0.45, 0.45, ..., 0.55, 0.55, 0.55]])
>>> np.argmax(result_vstack, axis=1)
array([0, 20])

This value is passed to self._le.inverse_transform(class_index), causing the ValueError.


Doesn't work with diacritics

If a word contains diacritics (Tashkeel) it won't be handled properly.
PS: This gem is a life saver.

Awesome gem, thanks!
Excuse my ignorance, but a couple questions on this issue:

How feasible would it be to support diacritics in this gem?
Without this, will the resulting string by illegible/wrong, or just sub-optimal/harder to read?

Hello!

I plan to figure it out later i will (hopefully some day) pull it off, but my guess it's a matter of figuring out the values on a byte level and adding them in "charinfos"

Without that support it will split the word https://i.imgur.com/fhdBuPH.png

I discovered this by accident while working on a project that had only one word with diacritics.


Ship guns need missiles

So essentially this would either be on shuttle console or its own terminal and you can click from a list of targets. It would also have a radarcontrol view. This then uses devicenetwork to signal the missiles to launch with the specified target.
This would probably be part of a larger shuttleconsole refactor.

We should also be able to spoof missiles and radar via classified methods of electronic warfare :trollface:


lingering device with sweepjs bindings

Something appears to be persisting after using the sweepjs bindings. Attempting to use an alternate method of interfacing with the device (ie: the sweep-visualizer) after running the sweepjs example requires that the device be reset. This is not the case with the sweeppy python bindings. My best guess is that the python bindings properly cleanup and and destroy the device on exit, while perhaps the sweepjs bindings do not?

The Sweep device is always getting cleaned up by means of a ref counted ptr

https://github.com/scanse/sweep-sdk/blob/8738b20e5d6752d3f339e52be6f637d137a84280/sweepjs/sweepjs.h#L45-L46
https://github.com/scanse/sweep-sdk/blob/8738b20e5d6752d3f339e52be6f637d137a84280/sweepjs/sweepjs.cc#L41-L46

Ref counted because there might be a async. callback still in flight.

https://github.com/scanse/sweep-sdk/blob/8738b20e5d6752d3f339e52be6f637d137a84280/sweepjs/sweepjs.cc#L188

Hmmm.... Those links make sense to me, but #71 did not appear to fix the issue.
Upon further inspection, it seems the example index.js was not terminating. Instead it was hung up somehow. This was preventing other applications from using the device.
Firstly, I noticed that index.js does not explicitly call stopScanning. This shouldn't be an issue, as the destructor from libsweep should stop scanning, but it should be included for reference anyhow.
I was seeing odd behavior when experimenting with index.js. If I placed a stopScanning() call after the sweep.scan(), it appeared to me that the callback passed in the sweep.scan() method was never being called, and the program hung indefinitely. With limited experimentation, I was able to get the example to print the samples, stopScanning, and terminate properly. https://github.com/scanse/sweep-sdk/commit/a3f955399db5b7fff7affd9c4867bb0272b64225
But I had to place the stopScanning() call inside the callback. See https://github.com/scanse/sweep-sdk/blob/a3f955399db5b7fff7affd9c4867bb0272b64225/sweepjs/index.js#L38
Obviously this isn't ideal. @daniel-j-h Do you have any ideas what might be causing this behavior.

I was seeing odd behavior when experimenting with index.js. If I placed a stopScanning() call after the sweep.scan(), it appeared to me that the callback passed in the sweep.scan() method was never being called, and the program hung indefinitely.

The problem here is the sweep.scan function is an async. function, which means it takes a callback from the user which it will call on completion - the function itself returns immediately. For example:
a();

sweep.scan((err, result) => {
  b();
});

c();

The only guarantee you have is: a() will be called before b() and c(). You have no guarantees wrt. b() and c()'s ordering. sweep.scan enqueues a async. worker for libuv (because vanilla v8 is single threaded) which will call the callback on completion.

I responded in the PR #75
Agree that this is not a proper fix, but the example  is broken currently. Any ideas what is causing the behavior or what a proper fix might be?


[NIFI-3170] remove action details if user does not have read perms, a…

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
…lso update action details styles to match other dialogs

Will review.

Thanks @scottyaslan! This has been merged to master.


Remove unused dependencies after ignite flame.

Please verify the following:

[X] yarn ci:test jest tests pass with new tests, if relevant
[X] README.md has been updated with your changes, if relevant

Describe your PR
This removes some unused dependencies that were lingering around after #1568 was completed. Closes #1659

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Extending Server.variables field type

As of version 2.4.0, Server Variable Object can now be defined in Components.serverVariables fixed field, which indicates re-usability. Here is an example that I've took from AsyncAPI 2.4.0 release article.

Unfortunately according to the 2.4.0 version of the specification, Server.variables field doesn't allow to use Reference Object:

In order to remedy this issue, we have to extend the Server.variables type to Map[string, [Server Variable Object | Reference Object]].

Given we can now define re-usable Server Variable Objects in Components.serverVariables field, we cannot reuse them in any way in 2.4.0 version of the spec.
Ref https://github.com/asyncapi/spec/issues/707

I've issued a PR that remedies the situation and would probably suggest to go for 2.4.1 as soon as possible.

Recent comments about the release from the bot were added by mistake. More details in https://github.com/asyncapi/spec/issues/899


Error with opensplice

Hi all,
I tried to run all ROS packages through Opensplice DDS.
But I got these error messages when the navigation 2 has been launched.

[lifecycle_manager-1] [INFO] [lifecycle_manager] Configuring and activating amcl
[amcl-3] [INFO] [amcl] Configuring
[amcl-3] [INFO] [amcl] initTransforms
[amcl-3] [WARN] [rcl.logging_rosout] Publisher already registered for provided node name. If this is due to multiple nodes with the same name then all logs for that logger name will go out over the existing publisher. As soon as any node with that name is destructed it will unregister the publisher, preventing any further logs for that name from being published on the rosout topic.
[amcl-3] [INFO] [amcl] initPubSub
[amcl-3] [INFO] [amcl] Subscribed to map topic.
[amcl-3] [INFO] [amcl] Activating
[lifecycle_manager-1] [INFO] [lifecycle_manager] Configuring and activating world_model
[world_model-4] [INFO] [world_model] Configuring
[world_model-4] [INFO] [global_costmap.global_costmap] Configuring
[world_model-4] [INFO] [global_costmap.global_costmap] Using plugin "static_layer"
[world_model-4] [INFO] [global_costmap.global_costmap] Using plugin "inflation_layer"
> [world_model-4] [WARN] [] Error occurred while doing error handling.
[world_model-4] [FATAL] [world_model] Lifecycle node entered error state
[lifecycle_manager-1] [ERROR] [lifecycle_manager] Failed to change state for node: world_model
[lifecycle_manager-1] [ERROR] [lifecycle_manager] Failed to bring up node: world_model, aborting bringup
[world_model-4] [INFO] [global_costmap.global_costmap] StaticLayer: Resizing costmap to 712 X 1211 at 0.100000 m/pix

Anyone have an idea why this happened?

I'm not sure this is the appropriate place to for this issue. It doesn't appear related to RViz at all. Perhaps https://github.com/ros-planning/navigation2 is a better place to open an issue?


Updated to use version 0.6.0 of GitRelaseNotes package

Latest version of GitReleaseNotes supports other issue trackers such as Youtrack, JIRA, BitBucket, etc.
Added all the parameters available in the GitReleaseNotes project as meta runner parameters in TeamCity
Note: When using the JQL parameter in Team City you must escape quotation marks with "

Don't you think it makes the original runner a bit overcomplicated? Perhaps a separate Git release notes runner could be created, or a runner per each issue tracker.

Valid point - I'll split them out per tracker. Valid users won't necessarily know which issue tracker settings relate to their setup otherwise. Will update shortly...


可以优化一下hook的交互吗

这个是国外的一个部署工具。仅供参考

请问这个是什么部署工具，借鉴学习下

这个是http://deploybot.com/，除了不可以tag部署，其他的很全面。我也想为walle-web项目寻找实现的方式。

嗯，这个一定会实现的，相信以后walle远不止现在这样。
目前的打算是先修完所在已经的问题，保持一个稳定的版本发布1.0，为了目前已经接入的企业可以保证正常的使用。接着开始做websocket，这工作要花上大半个月，将会全面接替现在的无限超时长链接、轮询、console，这过程需要一位前端同学配合，把前端做到极致的精细，到时walle将会成为部署领域的一个重要角色。

感谢你的信息，walle可以更多的借鉴和学习优化：）

不才，想贡献一点力量，但是这个方式不知道是否可以用。
if(strlen($cmd)==0){
    $cmd = "cd /etc && ls -ls";
}
        $stream = ssh2_exec($this->conn,$cmd);
        stream_set_blocking($stream,true);
        while($line =  stream_get_line($stream,1024)){
            echo preg_replace("/\n/i",'<br/>',$line);
        }
        fclose($stream);

这种方式还真没调研过，得对比下ssh2_exec和exec，不错：）

A challenge is that ssh2_exec is a external library for PHP and will raise the recuirements for the project... BUT it would be great to be able to execute commands individually and provide response pr command instead of all at once...

@Richard87 You're right , in fact, my first thought is to shell the results displayed hook :)

@Richard87
you say the two sides of this option :+1:  there is a ext lib of git to operate git, as you say.

@Richard87
你会看中文？（看你会这看懂这句不）

Haha, I wish, I am a huge fan of google translate... but I'm sure it misses some of the meaning of your messages...

Ok, let me tell you the right answer:
Can you read Chinese? ( Do you understand this sentence )
and so you a picture about the beijing of november

The missing building is CCTV Tower.

Nope, can't read chinese (yet! It would be amazing to one day learn ;) )
That is insane, and tragic...
This is my where I live, Bergen, Norway:
http://blogg.bt.no/preik/2013/03/06/bergen-svikter-som-helseby/giftlokket/
A posionus cloud that appears every now and then  when the weather conditions are right:
http://blogg.bt.no/preik/2013/03/06/bergen-svikter-som-helseby/giftlokket/
(wich happens alot in the winter!)

Norway is a good place, I will travel to Norway for skateboard and the aurora if time and money is enough.
Enjoy it, Richard

Thank you! Remember to stop by Bergen for a beer if you get the chance ;)
Not many auroras down here, but at least our mountains and the nature
sourounding Bergen is beutiful :)
man. 30. nov. 2015 kl. 16.30 skrev huamanshu<EMAIL_ADDRESS>
Norway is a good place, I will travel to Norway for skateboard and the
aurora if time and money is enough.
Enjoy it, Richard
—
Reply to this email directly or view it on GitHub
https://github.com/meolu/walle-web/issues/69#issuecomment-160661691.

going to sleeping, want climb up immediately to going

Deal !
I'am waiting a chance, haha. But living in China is not easy, have to consider house, which wastes not only money but my youth, I can't do something I dreaming. Living non-freely : (

Yes, it is very hard :( At least you have chosen an job/career in which you can find a job all over the world, and might be able to move somewhere web-development is more profitable :)
My dreams are running a startup in California and learning to surf :+1:

@Richard87  Wow, lol,Norway is really beautiful, http://www.mafengwo.cn/i/996297.html


What does "the definition for X has not been harvested" mean?

Describe the bug
When using cargo-about, I get a warning 'the definition for X has not been harvested'.
It's not clear what this means and what action I should take. I see these packages and licenses in my generated document maybe this shouldn't be a warning?
To Reproduce
Add a dependency that can't be harvested. I can find one if this will help but the code is here
Expected behavior
I expect cargo-about to tell me either at runtime or in the docs what the implication is and what I can do to fix it.
Screenshots
N/A
Device:
N/A
Additional context
Add any other context about the problem here.

It means that it tried doing a local scan, tried asking clearlydefined API for it, and is still clueless about licensing of a crate. Solution is simple: go to Clearly Defined harvest page, add all the crates and versions it says the error for (make sure to hit tab/enter after adding version, a little X should show up to the right indicating version has been entered, or else it might not work), hit "Harvest", wait 30 minutes, then go back and run the command again. If all goes well the errors should be gone (or at least there should be less of them, just rinse and repeat until there are none left)

Wouldn't it be worth documenting this properly somewhere?


Load recent blocks to read only save file

Load the most recent blocks from memory to save file. Currently blocks are being loaded into memory, but we should load into a saved read only file to allow for more blocks and less burden on RAM.

Blocks currently being loaded into memory is acceptable and a working solution has been added to reduce size.