Divi Theme Compatibility (Shortcodes)

Thanks for this plugin, working great. We're using Divi by ElegantThemes.com and when viewing our pages via the Divi Builder, AMP pages are outputting shortcodes used at the top and bottom.
Live here: https://heelthatpain.com/heel-pain/heel-pain-exercises/?amp
and here's a screenshot:

Any ideas on how these can be filtered from the content output?

Hey Jim @noinput
I really love the idea of Divi support, and we want to do this, but honestly, this is not a simple task. It is complex and a big one.
We're two developers, we dedicate 2 days out of our week to the AMP plugin development. I will try my best to look into adding Divi support next week and If it is easy, we will even start the implementation immediately.
If we cannot implement the Divi support right away then we will at least find an alternative solution to this problem.
I hope it helps,
Regards,
Ahmed

This was added after 0.8 update


feat: checkbox_group re-export

Resolves #2211
Needed for changes in plugins PR: https://github.com/deephaven/deephaven-plugins/pull/813

Cannot verify in styleguide as labels are currently blank due to latest core change @bmingles , will update and add examples as soon as that is fixed

Cannot verify in styleguide as labels are currently blank due to latest core change @bmingles , will update and add examples as soon as that is fixed

@AkshatJawne I don't think those are related. The labels that are missing are in the ListView right?

@bmingles Getting the above when I am running the following snippet, which I believe is related, since there is no reason for the label's to disappear.
from deephaven import ui
my_radio_group_basic_with_value = ui.radio_group(
    ui.radio("Dogs", value="dogs"),
    ui.radio("Cats", value="cats"),
    ui.radio("Lizard", value="lizard"),
    ui.radio("Fish", value="fish"),
    label="What is the best type of pet",
    on_change=handle_on_change
)

Ironed out comments, not sure why the String(child) is resulting in booleans resulting in null showing up as the text, @bmingles any ideas?

Should be good now, I still had to clone element, or else when we passed in Checkbox children, then the value would not be assigned properly, and thus selecting one would select all of them. @bmingles

I think the unit test failures are at least in part to needing a rebase / merge on latest main.

@AkshatJawne Code looks good. I think e2e tests may need to be updated.

Attempting to update snapshots right now, running into some difficulty, will update them soon hopefully.


Fix README for Storage Blob Cryptography

Add some examples on how to use this module.
Update the comments and artifact name.

@gapra-msft I believe you've done this, so this issue can be closed.


Export and import a project with workflow details

Add ability to export a project and associated models to json, including a project, project contents, workflow (tasks) and workflow contents. It will not copy the subject sets / subjects, these will need to be re-setup.
Add the abilty to import an exported json doc to recreate the set project and associated models.

Why? This seems like it should be a separate utility that downloads resources and can re-upload them.

Primarily to allow users to share project and workflows without our intervention. It would also enable us to move a project from staging to prod.


Read me 2

Release 2.0.0 version
This release includes spring boot upgradation from spring boot 1.x.x to 2.x.x
Details
Spring Boot Upgradation to 2.x.x.
Tracer fix for better traceability with new sleuth
pom updates for release
Read Me update for release
Context
Why is this change required?  This release support new spring boot versions (2.x.x)
Checklist:

[x] I've added tests for my code.
[x] Documentation has been updated accordingly to this PR

Related issue :
https://github.com/societe-generale/rabbitmq-advanced-spring-boot-starter/issues/11

Pull Request Test Coverage Report for Build 78

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.542%

Totals

Change from base Build 74:
0.0%

Covered Lines:
338

Relevant Lines:
343

💛  - Coveralls


Custom name of package.py

Excuse me if this is a recently added feature that I haven't noticed (my facility is not on the bleeding edge), but to the best of my knowledge, the packaging specification MUST be named package.py.
It would be helpful if the various rez commands (rez build, etc.) had a command line argument that allowed you to specify the name and/or path to this file. (Similar to how you can use command line overrides to change the package repository path, or the build system.)
Motivation
There are times when it would be helpful for a single repository to be the source of multiple packages -- somewhat similar to "super-variants", where even the name of the package can change. If you could have multiple package.py with arbitrary names and specify on the rez build or rez release line which one you want, this would be super easy. But currently, I don't believe you can do this, so you have to resort to tricks like putting a second package.py in a subdirectory, building from there, and having to change a lots of its path references to ../blah, if you know what I mean. It would all be simpler if you could have package1.py and package2.py in the same top level directory.

I will leave the actual request up for other people as I am not sure about the implications this may or may not have on other parts of rez.
Just a quick note, you can also create package.py files programatically so if you have a case as specific as this that might be worth doing (an example where i use this is for some 3rd party packages. For VRay for 3dsmax on Windows for example i have a custom command-line tool that you can drop an installer on and it will install to a temporary location, assemble the payload from the various folders, detect version and variant from the installer files and then create the package.py programatically)

Actually you can have a package family as one single python script, not really obvious but it's called "combined-package". Referencing from this discussion https://github.com/AcademySoftwareFoundation/rez/discussions/1187 :)

Would they share a build folder? I could see some potential issues arise from such a situation, where mutual contamination of the build folder has some unexpected results, depending on how the build functions. Then again, I can also see it being a wildly efficient way to potentially force an interesting scenario of a large multi-lib codebase (think USD) being componetized into smaller pieces, that nonetheless can use their neighboring build components directly, to speed up the build, ultimately releasing them as separately packaged components.

@maxnbk As I use it, I would blow away the build folder before moving on to generate the second package.

While doing some triage, I found https://github.com/AcademySoftwareFoundation/rez/issues/299 which is kind of related.

@lgritz - I've been turning this over in my mind a bit (in terms of consequences) and have been considering having a tinker to see what happens when we do this, ...
However, we had a good discussion in the TSC today and we wanted to learn a little more about the use-case that is attempting to be supported. The best phrase/summary was, I think, "Tell us about the problem, not the solution", ..
So if you have a little time, clarifying what you're attempting to achieve would be helpful towards understanding where such a feature might need to head, or addressing concerns. I have my own concept of what it might be, but I'm also guessing.
For example, are you wanting to build multiple versions of the same thing, which, for practical reasons, it makes sense to have different package-definitions for? Or building multiple components of an ecosystem together?

I want to make two different software packages, with different names (and maybe other things set differently), from the same source repo. In my mind, I could have a foopackage.py and barpackage.py, and
rez release --pkg foopackage.py

rez release --pkg barpackage.py

The only option now is to have it named package.py, there is no choice. So making two software packages out of one repo necessitates some trickery. There are a variety of approaches -- construct package.py on the fly from one of the options, use different subdirs, whatever. I have ways I'm using now to achieve this. But all are more steps and conceptually more complicated and brittle than just doing the above and simply having the ability to override which filename is used as the "package instructions" for any rez command.


RestGridModel requires enableExport to avoid console errors, export button always visible

We are currently always showing an exportButton in the RestGrid toolbar, but we are not passing enableExport: true to the GridModel owned by RestGridModel. This means by default you will see console errors about having an export button attached to a grid which does not have export enabled.
We should be enabling export by default (if that is what we want), and also omitting the exportButton if export is not enabled on the GridModel.

Another minor export related ticket for you Peter.  Thanks!


repo webpage

create webpage using github's pages
helpful links:
https://help.github.com/articles/user-organization-and-project-pages/

created page: https://keymic.github.io/km-qrs/
still needs some content


NO-JIRA Adding Release notes about removing rest

@clebertsuconic I created #4230 to add a release not for the removed web content.


something about "VisibilityScoreField::traverse"

Hi, very appreciate your wonderful work and i'm learning your code .But could you tell me the the intention of "VisibilityScoreField::traverse".   I can not understand it clearly.
Thanks very much.

Were you able to run the code? You issued that the bag file could not run :(
The code is an implementation of FIGURE 8 of my paper. !

Refer my paper:
https://ieeexplore.ieee.org/abstract/document/9557293/

Thanks so much ! i will read it carefully.

I am sorry, but the repo is not runnable. Closing this issue.


1.31 metrics reference is out of date and doesn't match the code

This is a Bug Report

Problem:
1.31 metrics reference is out of date and doesn't match the code.
e.g. scheduler_event_handling_duration_seconds is a metrics that exist in 1.31, but the doc doesn't have it.
Proposed Solution:
IIUC metrics reference  is generated. We should regenerate it to make it up to date.
Page to Update:
https://kubernetes.io/docs/reference/instrumentation/metrics/

latest: 1.31

@kubernetes/sig-docs-en-owners

It looks like the Kubernetes metrics reference page at https://kubernetes.io/docs/reference/instrumentation/metrics/ says that it is for Kubernetes v1.30 [Metrics (v1.30)], while the current release is already v1.31. This means the metrics page is outdated.  It would be good to update the content to reflect the latest version.

/sig instrumentation
/language en
/triage accepted
/priority important-soon
(Please feel free to adjust the priority as needed, if the SIG consensus leans toward a different priority)

Refer the guidance at https://kubernetes.io/docs/contribute/generate-ref-docs/metrics-reference/ on how to generate the metrics reference documentation.

/assign

I have submitted the PR https://github.com/kubernetes/kubernetes/pull/128376 in K8s upstream, and when it is merged, I will sync the v1.31 latest metrics reference to the website.


[doc][flexible-ipam] Update document for Multiple-VLAN support

Signed-off-by: gran<EMAIL_ADDRESS>

Codecov Report

Merging #3507 (cda4145) into main (2c7d486) will decrease coverage by 27.36%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3507       +/-   ##
===========================================
- Coverage   65.58%   38.22%   -27.37%
===========================================
  Files         277      114      -163
  Lines       27264    14999    -12265
===========================================
- Hits        17881     5733    -12148
- Misses       7483     8795     +1312
+ Partials     1900      471     -1429

Flag
Coverage Δ

integration-tests
38.22% <ø> (?)

kind-e2e-tests
?

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/cniserver/ipam/antrea_ipam.go

pkg/controller/querier/querier.go

pkg/controller/networkpolicy/status_controller.go

pkg/agent/nodeportlocal/portcache/port_table.go

pkg/agent/nodeportlocal/npl_agent_init.go

pkg/agent/proxy/types/groupcounter.go

pkg/agent/interfacestore/types.go

pkg/util/sets/string.go

pkg/apiserver/certificate/cacert_controller.go

pkg/querier/querier.go

... and 381 more

/skip-all


Docker stack fails

Hi,
I have tried to run the Chirpstack using the supplied docker-compose.yml and I am getting the following errors
ERROR: for mosquitto  Cannot start service mosquitto: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/compose/configuration/mosquitto/mosquitto.conf" to rootfs at "/mosquitto/config/mosquitto.conf": mount /compose/configuration/mosquitto/mosquitto.conf:/mosquitto/config/mosquitto.conf (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

This will happen if the repo isnt cloned first

i have the same problem

@remyar please see the comment from @timmills, does this solve your issue? if not, please provide step by step instructions to reproduce the issue.

Apparently one needs to run make import-lorawan-devices in current directory to populate one of the required directories for mqtt container setup.

The above is not true, there is no need to run make import-lorawan-devices.
Please see this documentation: https://www.chirpstack.io/docs/getting-started/docker.html. With the documented steps, I'm not able to reproduce this issue. As there hasn't been any feedback with reproducible steps I'm closing this issue.


Add code coverage checking to builds

We need to add automatic code coverage capability to our builds.

Completed in #1


Wrong type inference from optional generic types

Bug Report
Type inference from optional generics type become wrong in all major IDEs when descructuring.
I accidently discovered this behavior in a project im working on that's using a third party library: I used to have an excellent type inference when using expressions like this:
const {value: {nested}} = constructObject(src);

Let's say constructObject<T, E = Value<T>> is a function that operates on type T while returning a portion of it as E, that when omitted, means that we are using Value.
I did not have any issues before 4.7 and all my IDE's (intellij, vscode and codesandbox) were reporting it fine.
Now, and only when desctructuring, rather than using the optional Value<T> as type, it tries to infer it from the descructuring I am making and try to invest a new type for that.
🔎 Search Terms
Wrong type inference
type inference on optional generic types
...
🕗 Version & Regression Information
I did not pay attention to the first time this occurred, I used to have exact and correct type inference in 4.6.x and I did notice this behavior unless 4.7+
⏯ Playground Link
This codesandbox.
💻 Code
type Value<T> = { t: T }
type Test<T, E = Value<T>> = { value: E }

function constructValue<T>(value: T): Value<T> {
  return {t: value}
}

function constructTest<T, E = Value<T>>(
  value: T, sl?: (e: T) => E): Test<T, E> {

  return {
    // ts warning: 'E' could be instantiated with an arbitrary type which could be unrelated to 'Value<T> | E'.
    // IDE says value is of type 'E = value<T>'
    value: typeof sl === "function" ? sl(value) : constructValue(value)
  }
}

// if we assign then destructure later, it s fine
const result = constructTest(5)
const {value: {t: t1}} = result; // t: number
// --> destructuring directly make it try to create the optional parameter
// rather than using the default one
const {value: {t: t2}} = constructTest(5); // t: any

// if we assign then destructure later, it s fine
const {value} = constructTest({hello: "world"}); // value: Value<{hello: string}>
const {t: {hello}} = value; // t: {hello: string}
// --> destructuring directly make it try to create the optional parameter
// rather than using the default one
const {value: {t: t3}} = constructTest({hello: "world"}); // t: any

// adding the selector that syncs the optional generic type seems to work as expected
const {value: {override: o1}} = constructTest(5, e => ({override: e})); // override: number
const {value: {override: o2}} = constructTest(5, e => ({override: e.toString()})); // override: string

🙁 Actual behavior
It tries to invent a type from what i'm destructuring rather than using the optional generic type.
🙂 Expected behavior
If omitting the optional type, this means: use the default one, not invent one.

I also created few days ago this stackoverflow question, but without luck.

The error is correct. When sl is undefined, you use constructValue(value) and to create a value typed T, and try to return this with the return type being E. But T and E can be completely unrelated types.
Here's an example:
const res = constructTest<string, number>('abc')
console.log(res.value)

The type of res.value is number, but the value is actually the string "abc".

One solution is to use overloads to prevent the situation from sl being undefined and having an incompatible type for E, and also return two distinct object literals to prevent the type of the object literal to be incompatible.
function constructTest<T>(value: T): Test<T, Value<T>>;
function constructTest<T, E = Value<T>>(value: T, sl: (e: T) => E): Test<T, E>;
function constructTest<T, E = Value<T>>(value: T, sl?: (e: T) => E): Test<T, Value<T>> | Test<T, E> {
  return typeof sl === 'function'
    ? { value: sl(value) }
    : { value: constructValue(value) };
}

Thanks @MartinJohns for your answers.
I guess I understand the danger with T and E being unrelated, and this should be (in our case) a developer concern and the IDE will still help in it.
But, I cannot really understand this: which is actually my issue:
// if we assign then destructure later, it s fine
const result = constructTest(5)
const {value: {t: t1}} = result; // t: number
// --> destructuring directly make it try to create and invent the optional parameter's type
// rather than using the default one
const {value: {t: t2}} = constructTest(5); // t: any

They are basically equivalent, but do not act the same.
The overload solution may work in our case, but my real world scenario is a complex configuration object with 10+ properties.

That's a duplicate of #45074 and is fixed in TypeScript 4.8.0. It already works in the nightly version: Playground link

Thanks a lot Martin, So all we have to do is wait for the 4.8.

wait for the 4.8.

If you're not aware of it, according to #49074 the RC is scheduled for the 9th August, and the final release is scheduled for the 23th August. So a little more than a month left.


Go Online Error

I've tried to go online for awhile and I get this error. I believe I've followed the process to connect wallet properly and generated keys.  Any suggestions welcome to make the final step.

Were you able to get it running? On Twitter it looked like somehow your node skipped to the dashboard before syncing. Did restarting the app work?

Restarting the app did indeed work.


Remove travis-ci build

We're not using Travis in Sleuth

It needs to be removed from Travis

It's inactive https://travis-ci.org/spring-cloud/spring-cloud-sleuth


Class 'Module' not found L5.3

Failed after I create first module. I made second module and I got the error

[solve] I am wrong placing Facade ...


question about reward guidance in Mujoco tasks

Hello, i cannot understand the reward guidance synthetic generation for continuous reward function. Could you give more information or related codes?

Our implementation is similar to this file of Diffuser. And we will release our code of Mujoco later.

Thanks. After reading the implementation of Diffuser, i have known the generation process and how $\mu+\Sigma*g$ works. But i still have a question about the binary random variable indicating the optimality. What does this mean and how it influence the generation process?

It's a misunderstanding. We just means using a random variable obeying Bernoulli distribution.


Upd packages

fix cover
fix lint errors
move @types/hapi__joi to devDependencies
sync package-lock.json
sync yarn.lock

@marshallswain Hi, can you please take a look on this PR ? Thx

Hey @marshallswain, can you please merge it? Thnx


Workaround issue detected by JET.@test_call

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.51%. Comparing base (2822a31) to head (2bb1629).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   94.51%   94.51%
=======================================
  Files          13       13
  Lines         912      912
=======================================
  Hits          862      862
  Misses         50       50

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Build Docker image in parallel

Devnet docker image could be built in parallel with tests, then in the publish job it could just be pushed - right now the building needs to wait for tests to finish
Not sure if this can actually be done, i.e. if the built image will be preserved for other circleci steps/jobs

This can be done by combining with #160:

an image with an appropriate tag (candidates: next, beta, master) could be built on each push to master (regardless of a new version being released) - this could be done in a separate job running in parallel with tests
the current image build and push would just do the pushing (and only on the master branch when:

first it would fetch the image tagged with next, then it give it the official tag, then it would push


sfizz.lv2 needs to respect CMAKE_INSTALL_PREFIX

I was trying to build and install development version of sfizz into non-standard location, apart from the some stable-ish version at /usr/local. But sfizz.lv2 cannot be installed because it seems always trying to install it to /usr/local.
/sources/sfizz/build$ make install
Install the project...
-- Install configuration: "RelWithDebInfo"
-- Installing: /sources/sfizz/dist/lib/libsfizz.a
-- Up-to-date: /sources/sfizz/dist/include/sfizz.h
-- Up-to-date: /sources/sfizz/dist/include/sfizz.hpp
-- Up-to-date: /sources/sfizz/dist/lib/pkgconfig/sfizz.pc
-- Installing: /sources/sfizz/dist/lib/libsfizz.so.0
-- Up-to-date: /sources/sfizz/dist/lib/libsfizz.so
-- Installing: /sources/sfizz/dist/bin/sfizz_jack
-- Installing: /sources/sfizz/dist/bin/sfizz_render
-- Up-to-date: /usr/local/lib/lv2/sfizz.lv2
CMake Error at lv2/cmake_install.cmake:49 (file):
  file INSTALL cannot set permissions on "/usr/local/lib/lv2/sfizz.lv2":
  Operation not permitted.
Call Stack (most recent call first):
  cmake_install.cmake:44 (include)

Hi, you would have to also override the cmake variable LV2PLUGIN_INSTALL_DIR after the first run.
It's cached and defined at cmake/LV2Config.cmake, and it will be defined according to the initial value of the prefix.
(same applies to VST and VSTPLUGIN_INSTALL_DIR)

Ahh, I guess I only had a look at the topdir and src, not lv2. Thanks, it just worked. I'm not sure if there is any reason to have them independent of CMAKE_INSTALL_PREFIX (i.e. by design) and not sure if I had better close this issue or not. I leave it to you folks.

if (MSVC)
    set (LV2PLUGIN_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/lv2" CACHE STRING
    "Install destination for LV2 bundle [default: ${CMAKE_INSTALL_PREFIX}/lv2}]")
else()
    set (LV2PLUGIN_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/lib/lv2" CACHE STRING
    "Install destination for LV2 bundle [default: ${CMAKE_INSTALL_PREFIX}/lib/lv2}]")
endif()

We could FORCE these, or make them normal cmake variables rather than cached values? I don't remember if we had a good reason to cache these.

Ok so there are still reasons to cache these, name that you could want them decoupled from the cmake install prefix.
The current best solution to this problem is thus a clean configure (with only the CMAKE_INSTALL_PREFIX that will propagate throughout the cache) or explicitely mention the LV2_PLUGIN_INSTALL_DIR on reconfigure. I don't think it is mandatory to add complexity to the build scripts at this point.
Thanks for reporting! :slightly_smiling_face:


Design changes to some files

Current dependencies on/for this PR:

main

PR #88   👈

This comment was auto-generated by Graphite.


Why not use React & Typescript ?

I think you can have a lot of power if you can implement only a subset of the DOM drawing and have a typescript/js interpreter in RUST that can leverage the xilem primitives.

I may not be the most qualified to answer your question, but given the nature of Xilem, I'm quite shocked that you would even ask such a question.
Have you read the article on Xilem? https://raphlinus.github.io/rust/gui/2022/05/07/ui-architecture.html
Adding a TS/JS interpreter or likewise bundling any kind of web technology would be adding accidental complexity for no additional benefit compared to other GUI frameworks, particularly when it comes to performance.
Besides, the whole point is to be able to make performant, native GUIs in Rust without having to rely on third parties, especially the web.

A new UI system is born every time there is a new language. GTK, QT, wxPython, the list goes infinite here. I (surprisingly) see the only one that resists the times is HTML + CSS. Learn from thousands of now semi-abandoned projects and don't reinvent the wheel. What sounds interesting to me is a faster GL/HTML-based UI system. Where I can apply #tailwindcss styling and don't have to learn yet another API. just my unsolicited 2 cents, with the only hope that is helpful :)

I understand that html/css devs feel scared of learning a real programming language like rust. Especially with GitHub copilot probably producing good enough results that the least competent 10-25% of programmers can be replaced soon.

Learn from thousands of now semi-abandoned projects and don't reinvent the wheel. What sounds interesting to me is a faster GL/HTML-based UI system. Where I can apply #tailwindcss styling and don't have to learn yet another API. just my unsolicited 2 cents, with the only hope that is helpful :)

If you want something like that, go look at Dioxus. Xilem is not what you're looking for.
I would also not put HTML/CSS/JS on a high pedestal just because it resists the times, the reason for that is what @CryZe said above and additionally because it's relatively easy to make. Computers are so fast these days that people tend to not care enough about performance. In my experience, it creates this sub-par experience with software that is really slow, takes a huge amount of memory, with FPS stutters on simple tasks.
Meanwhile any native software I've used that was written in C++ felt immediately less resource and power-hungry with much less input latency.

@Speykious Dioxus seems exactly what I imagined this project would be trying to do. 7k stars on Dioxus mean that the idea was pretty good. Thank you a lot for letting me discover Dioxus. The post will be helpful for others that would land to this page with the same idea asking the same question.

I understand that HTML/CSS devs are scared of learning a programming language like rust. Especially with GitHub copilot probably producing good enough results, the least competent 10-25% of programmers can be replaced soon.
 The entire internet saas world is about to collapse. I need an HTML-like UI because I can prompt it way easier with an LLM than with a new, very low example code. Then ask yourself what happens when a model can generate entire apps in 30 seconds with one framework vs the other.  Let's get back to this thread in 3 years. Right now is just words on a github issue. talk is cheap. show me the code.

Thank you again for pointing me at Dioxus. Compiling it right now

It seems like we've come to the resolution that xilem will not match your vision. I hope you can find a framework which allows you to build what you want to build :)
Another couple of suggestions which match what you are describing:

Tauri, which I believe supports React and TypeScript
Electron, which also enables creating desktop apps using the DOM

Learn from thousands of now semi-abandoned projects and don't reinvent the wheel.

Learning from history is certainly what we are doing. To go with your wheel metaphor, we're not building yet another wooden wheel, we're going for a hollow-spoke carbon fibre wheel. Unprecedented performance.

What sounds interesting to me is a faster GL/HTML-based UI system. Where I can apply #tailwindcss styling and don't have to learn yet another API.

That's fine, but you're quite bold to think that this community that has spent half a decade building custom Rust UI will completely change its direction based on your unwillingness to learn yet another API. It will not happen.

To quote Casey Muratori:

Saying "reinvent the wheel" is a very misleading statement. [...] The wheel is an amazing invention. It is almost perfect for what it does. When you look at a wheel, it is the thing that perfectly turns dynamic friction into static friction for moving objects. It's something that is so elegant and beautiful, that if you ask me why you would want to reinvent it I would say "of course you don't want to reinvent the wheel, the wheel is amazing". It has worked for thousands of years, it has been unchanged for thousands of years (the concept is the same today as it was when they first made a wheel probably, it makes the exact same stuff physically). Nothing we have developed in the past 30 years of game development is a wheel. If you think Unity is a wheel or Unreal is a wheel, it's just plain shortsided.

Likewise, I would say nothing we have developed in the past 40 years of UI development is a wheel. If you think React is a wheel or HTML/CSS/JS is a wheel, it's just plain shortsided.


Add tasker-types to allowedPackageJsonDependencies.txt

Hello! I'm new with Typescript and new in defining types for it, so please tell me if I'm doing something wrong.
I'm going to create the definition types for the following Javascript library: https://github.com/amoshydra/tasker-js-runner, but for that I need the types of another library: https://github.com/Rayquaza01/tasker-types. So, it would be really helpful if "tasker-types"'s library is added to the repository.
Thank you in advance ;)

Sounds good @ivansoriarab—once you open a DefinitelyTyped PR, post a link here and we’ll merge this.

Sounds good @ivansoriarab—once you open a DefinitelyTyped PR, post a link here and we’ll merge this.

Hi @andrewbranch. I've just made a PR in DefinitelyTyped. Here you have it in case you want to take a look on it: https://github.com/DefinitelyTyped/DefinitelyTyped/pull/50304
Thank you!


Amend var_name documentation

Following https://github.com/SciTools/iris/pull/2874 this PR makes all docstrings/comments/exception messages refer to var_name as netCDF variable name.

Thanks @tv3141!


WS notifications for @mentions

This PR adds the initial setup for having real-time and persistent @mentions notifications.
It's a very simplistic approach right now (not the slack flow, fo sho!), but it's a good starting point to iterate upon later, once the logic around notifications becomes more complex (muted channels, user preferences, etc.)
This adds support for both user-specific mentions (i.e. @prego) but also broadcast-type (i.e. @here or @everyone)

related to https://github.com/NewShadesDAO/api/issues/61


tags & categories not working in github pages

Hey,
after uploading to github pages the tags & categories are not working.
both return a "404: Page not found" error

I'm getting the same error. It seems that for the Categories page it is inserting 'localhost:4000' in the href link (see below).
Academia vs. Industry
You can see the behavior here: http://www.bstankie.com/categories/career/
Tags are also having issues that I haven't been able to diagnose.
I've read through the manual and couldn't find any information other than: https://chirpy.cotes.info/posts/write-a-new-post/
I've configured those pages as described.

@bstankie There's a mistake in docs https://chirpy.cotes.info/posts/write-a-new-post/:

the content of a tag type page is:
---
layout: tag
title: TAG_NAME
category: TAG_NAME
---

The correct way is to change the category above to tag, my bad; I'll fix the docs online.

OK. I'm still having trouble. Here are the steps that I'm going through. See https://bstankie.com
Here are the steps, output and the behavior.

Create a post with a category and a tag.

---
layout: post
title:  "Academia vs. Industry"
date:   2019-08-01 12:21:24 -0600
categories: [career]
tags: [academia]
---

see (webpage)[http://www.bstankie.com/posts/Academia-Industry/]

Create a file academia.html in the tags directory with the following information:

academia.html
---
layout: tag
title: academia             # e.g. bee
tag: academia               # e.g. bee
---

Create a file career.html in the categories directory with the following information:

career.html
---
layout: category
title: career        # e.g. Insect
category: career     # e.g. Insect
---

run tools/init.sh

bstankie@DESKTOP-134K253:/mnt/d/git/BJS-Blog-chirpy$ tools/init.sh
[INFO] Succeed! 0 category-pages created.
[INFO] Succeed! 0 tag-pages created.
[master 6c7f058] [Automation] Updated the Categories, Tags for post(s).
 2 files changed, 10 deletions(-)
 delete mode 100644 categories/career.html
 delete mode 100644 tags/academia.html
Updated the Categories, Tags for post(s).

It seems to say Succeed! but there are no category nor tag pages created.

run tools/build.sh

bstankie@DESKTOP-134K253:/mnt/d/git/BJS-Blog-chirpy$ tools/build.sh
The dependency tzinfo-data (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for ruby but the dependency is only for x86-mingw32, x86-mswin32, x64-mingw32, java. To add those platforms to the bundle, run `bundle lock --add-platform x86-mingw32 x86-mswin32 x64-mingw32 java`.
Configuration file: /mnt/d/git/BJS-Blog-chirpy/_config.yml
           Cleaner: Removing /mnt/d/git/BJS-Blog-chirpy/_site...
           Cleaner: Nothing to do for /mnt/d/git/BJS-Blog-chirpy/.jekyll-metadata.
           Cleaner: Nothing to do for /mnt/d/git/BJS-Blog-chirpy/.jekyll-cache.
           Cleaner: Nothing to do for .sass-cache.
$ cd /mnt/d/git/BJS-Blog-chirpy/.container
$ JEKYLL_ENV=production bundle exec jekyll b -d /mnt/d/git/BJS-Blog-chirpy/_site
Configuration file: /mnt/d/git/BJS-Blog-chirpy/.container/_config.yml
            Source: /mnt/d/git/BJS-Blog-chirpy/.container
       Destination: /mnt/d/git/BJS-Blog-chirpy/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
                    done in 1.255 seconds.
 Auto-regeneration: disabled. Use --watch to enable.

Build success, the site files have been placed in '/mnt/d/git/BJS-Blog-chirpy/_site'.

Upload to AWS S3
See behavior:

bstankie.com
bstankie tags: Click on academia tag and page not found
bstankie categories: Click on career tag and page not found

@bstankie  it seems that you don't understand what the docs saying.

The script tools/init.sh  is used to automatically create pages we need so that you can completely save the time of manual creation.

I just check the post source file (https://raw.githubusercontent.com/bstankie/BJS-Blog-chirpy/master/_posts/2019-08-01-Academia-Industry.markdown) of your repo and found the cause of the problems:

the extend of post file name should be .md.
the value  list of categoires in the fromatter block should be arounded by [ and ]----
categoires: [career]   # or category: career
---

The .md seemed to be the main issue. Tried with and without the [ ] before and ran into the same issues.
Now running into a strange problem. The URL when linking from the category:career to the post is:
http://www.bstankie.com/categories/career/www.bstankie.com/posts/Academia-Industry/
The tags:academia has a similar issue of linking to:
http://www.bstankie.com/tags/academia/www.bstankie.com/posts/Academia-Industry/
I do appreciate your help on this. Wish I were able to figure it out

Check the variable url of _config.yml, I sure that you're missing the protocol part 'https://'

That did it.
Thanks @cotes2020 ! Would you like me to document any of this in the readme to pay it forward?

Of course! I can't wait for somebody join in the doc improvement stuff. 😄

A tip for your website:  Enforcing HTTPS for your GitHub Pages site

Thanks for the HTTPS tip. Working on that next. Right now it is being deployed through AWS.
Coffee sent as a small token of gratitude.

I got the coffee, thanks !

I'm having the same issue of having 404 in categories. I've got the proper format of categories and tags, ran tools/init.sh script. But there's still a 404 error in GitHub Pages. Could you take a look cotes in my blog?
https://cloufish.github.io/tabs/categories/

@Cloufish I am having the same problem. how did you solve the problem can you help

From your _config.yml file I can see that you haven't even filled it! :P. I'm my case it was because I've filled baseurl: parameter which in my makes shouldn't be filled.

@Cloufish Thanks my friend for your help, I couldn't find the problem but I copied your project :)

Brother, I think I can't see it, can you help me change what should I change? :/

I don't think copying is the best solution, because you will eventually come into another issue :p However, if it's less complicated for you then go-ahead

See the 'url:' parameter inside _config.yml? Change it to ' https://onurbolatogluu.github.io ' and you'll be ready to go :100:

@Cloufish Brother, I changed it as you said, I am sending it in the attachment. The only difference is I use my own domain name. I organized it accordingly. but the problem still persists. May I be very unlucky :)


No output to list-scenes

list-scenes and load-scenes are fantastic. I noticed while playing with list-scenes that the --quiet global option suppresses output of  the scene list CSV file, whereas --verbosity none does not. Minor issue but spent some time figuring out what was happening.

Are you not seeing any output files get created when you use -q/--quiet?  I tried to reproduce this just now but am unable to, things seem to be working okay on my end.
Could you provide the full command line arguments and the version of PySceneDetect you are using?

I struggled to reproduce it now myself. Then I found an older csv file in
my home directory. I suspect that, when no --output directory was
specified, my macOS home directory (~) was used instead of the current
directory (.). Possibly an environment variable but not sure.
On Thu, Mar 21, 2024 at 3:26 AM Brandon Castellano @.***>
wrote:

Are you not seeing any output files get created when you use -q/--quiet? I
tried to reproduce this just now but am unable to, things seem to be
working okay on my end.
Could you provide the full command line arguments and the version of
PySceneDetect you are using?
—
Reply to this email directly, view it on GitHub
https://github.com/Breakthrough/PySceneDetect/issues/384#issuecomment-2011018638,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHDD2R4OZYTXE54ZVKRI6X3YZIZN5AVCNFSM6AAAAABFAIEWZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJRGAYTQNRTHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Closing as not reproducible for now, but feel free to re-open if you encounter the issue again or file a new issue.  Thank you!


Android version

Can xhunter rat be run on android version 6 not payload?

It works fine above android 8. you may face signing payload error

Yes when i click on build payloads and give my herokuapp server link and it starts building but when it comes to ziping resourses app crashed any solution?

Try on other device or wait for next update I will make it compatible.


Use decl_error in the treasure module

A slightly different approach than in #4369: ModuleDispatchError::as_str is used to convert errors.

Let's wait until a small update to decl_error! make this a little bit better to merge this PR @thiolliere


Add alternative parametrization for inverse gamma distribution

Codecov Report

Merging #103 (60cb3a5) into main (3fe6b31) will increase coverage by 0.22%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##             main     #103      +/-   ##
==========================================
+ Coverage   72.97%   73.19%   +0.22%
==========================================
  Files          15       15
  Lines        1728     1746      +18
==========================================
+ Hits         1261     1278      +17
- Misses        467      468       +1

Impacted Files
Coverage Δ

preliz/tests/test_distributions.py
100.00% <ø> (ø)

preliz/tests/test_mle.py
100.00% <ø> (ø)

preliz/distributions/continuous.py
98.25% <96.29%> (-0.10%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


A typo in docs

Hi, this looks like a typo in the ip address.
Just added .

Good catch, thank you!!


chore(image): update base image

Automated changes by create-pull-request GitHub action

Codecov Report
Patch and project coverage have no change.

Comparison is base (b41a1a0) 72.94% compared to head (915b5d9) 72.94%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #339   +/-   ##
=======================================
  Coverage   72.94%   72.94%
=======================================
  Files          25       25
  Lines         743      743
=======================================
  Hits          542      542
  Misses        201      201

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fixed #31474 -- Made QuerySet.delete() not return the number of deleted objects if it's zero.

Fixes ticket 31474.

@hramezani Thanks :+1:


fix(eventindexer): Revise the names of the db env vars

Revise the names of the MySQL env vars read from the CLI to match the documentation and the .default.env file.

PRs should not be opened to main right now, they should be opened to alpha-6 branch.


Record INVALID state in deposit.

The validity of the bag in the deposit is checked in the current implementation. However, if the bag is found to be invalid, this is not recorded. Should this be done as a git tag? The git-repository is initialized only after the validity check. Futhermore, the INVALID state should also be available in the deployment with git disabled. What's the best way forward here?

Done (in state.properties)


Proposal: unlink network names from instance IDs

When an instance of a build is started, there are two types of Docker resources created:

an overlay network
containers of any images associated with the build

Currently cmgr explicitly tracks the containers associated with an instance in a containers table, which looks like:

instance
id

instance id (fk)
Docker container id

however, the network is only implicitly associated with the instance, by naming it cmgr-{instance_id}.
This is usually fine, but can lead to issues when cmgr's state becomes out of sync with Docker's.
For example, say a user "forcibly resets" their cmgr state by deleting cmgr.db. Afterwards, attempts to start new instances will fail (as the generated network names will conflict with ones left over from before), unless the user manually runs docker network prune / rm.
However, even that may not work, as Docker will sometimes fail to delete a network even when it does not actually have any remaining containers attached (this can also prevent stopping an instance in typical usage).
I have two proposed changes to make cmgr more resilient to mismatches with Docker engine state and networking bugs:

Allow Docker to generate network IDs, and explicitly store them in a networks table:

instance
id

instance id (fk)
Docker network id

which would then be used when calling Manager.stopNetwork.

Downgrade network removal errors - primarily as a workaround for the libnetwork bug linked previously, upon any network removal error, print a warning and proceed to deleting instance metadata, rather than early-returning an error value from Manager.stopInstance. I think this change would be justifiable, as failure to remove a network would no longer lead to name conflicts.

As a note, a similar unlinking-of-IDs might be useful to avoid state-mismatch issues at the build level as well (using database-stored hashes for image tags and artifact directory names). In the worst case, this might lead to orphaned images and artifact files on disk, but would prevent the possibility of serving outdated content from prior cmgr installations.

I'd like to rethink this a bit, based on recent experiences with Docker network behavior.
Will possibly re-propose something later on in PR form.


rda5807fp

hi dear Ricardo and thanks for your great lib
as far as you also know  RDA5807FP has I2S support
if possible please support this chip in both master and slave mode

It is implemented. See  https://youtu.be/07017sfMYdY

thank you man
god be with you

rda5807fp is better than rda5807m because it can be used without mcu!
for this purpose just connect pin6 (GND3) to vcc!
details:
https://www.icstation.com/rda5807-radio-receiver-87mhz-108mhz-p-15859.html

True. You can do that with rda5807 and rda7088. Both are almost identical.
But if you need a visual interface, RDS, LNA control etc…
Thanks.
Regard.
Ricardo.
On Wed, 24 May 2023 at 02:16 clear-sky @.***> wrote:

rda5807fp is better than rda5807m because it can be used without mcu!
for this purpose just connect pin6 (GND3) to vcc!
details:
https://www.icstation.com/rda5807-radio-receiver-87mhz-108mhz-p-15859.html
—
Reply to this email directly, view it on GitHub
https://github.com/pu2clr/RDA5807/issues/7#issuecomment-1560470070, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVSJNCSHMV7K7FNP6IQ2JLXHWKRZANCNFSM6AAAAAAVW6T3UY
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

--
QRZ: http://www.qrz.com/db/PU2CLR
Skype: ricardo_caratti
Mobile: +55 61 981442296


not supported mysql quoted string concat syntax

https://dev.mysql.com/doc/refman/5.7/en/string-literals.html
Quoted strings placed next to each other are concatenated to a single string. The following lines are equivalent:
'a string'
'a' ' ' 'string'
ps: used in mybatis like this:  select * from t where col like '%' ? '%'

I did not know, this is possible. To be honest, I don't like this syntactic shortcut. What is the problem in writing '%' + ? + '%', which is more characters, but definitely better to read.

number cannot connect string in MySQL.

Sorry. Use a double pipe.

At this time this shortcut will not be implemented. Sorry.


Set demo route to root

After this is merged we should rerelease the demo to gh-pages :P

Yasss! Thank you!


feat: add tests for examples in docs and updated docs

Pull Request Test Coverage Report for Build 445294192

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.137%

Totals

Change from base Build 445272133:
0.0%

Covered Lines:
522

Relevant Lines:
530

💛  - Coveralls


Unused translations are in final bundle

Hey, recently i was investigating my final bundle size and noticed that translations.js size is huge and it doesnt depend on which bundle i use (i created 2 bundles with different components using i18n - they used different translations),  so i have a question, is there any way to decrease translations.js size by spliting it depending on namespaces, files etc. or some kind of lazy loading?

I have config/i18n-js.yml with:
translations:
  - only:
    - "*.js.*"

    # Since there are only a few entries in some already used scopes
    # Allowing these to be output
    - "*.actions.defaults.*"
    - "*.formats.datetime.*"
    - "*.formats.sentence.*"
    # Too time consuming to these entries into specific namespace
    - "*.app_resources.some_resources.*"
    - "*.website_pages.*.js.*"
    - "*.components.main_navbar.universal_search_bar.*"

This way only the useful translations would be exported.
But I don't know your setup so this might not work for you. Try it first :)

@PikachuEXE seems this can help me, it turned out that the wrong architecture was chosen before (all translations were placed in 1 file), now i can split it, but is there any way to lazy load it? If we have several components and their translations are in different files and on current page i use only 5 out of 10 components? I want to load only 5 files with translations, but i require all 10 in my bundle

There is no out of box solution for lazy loading.
Lazy loading itself is complicated and I don't think this project aims to support it out of box.

thanks for help


Crash on startup on Mohist server

Describe the bug
Crash on startup on Mohist server

Remember that a great explanation leads to a greater understanding

Reproducibility
Every depencies correctly installed on server, but when it starts just crash inmediately.

[x] I reproduced this issue with as few other mods as possible installed.
[ ] I am unable to reproduce this issue consistently.

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots or/and videos
If applicable, add screenshots and/or videos to help explain your problem.
Logs or/and crash reports
https://mclo.gs/KbZ4cvs
Versions

Mohist-1.12.2-342
Forge: 1.12.2 - <IP_ADDRESS>60
Java: JDK 8 - 312
Java JVM: OpenJDK

Specification:

OS: Debian 11
CPU: Intel Core i9 9900K
GPU: Not present
RAM Allocated: 8GB

All of these info below are not necessary but may become necessary depending on the issue

CPU Speed: [e.g. 3.80GHz Boost around 4.25Ghz]
CPU Core: [e.g. 8 (Attention Core not logical core)]
CPU Threads: [e.g. 16 (Named Logical processors in task manager)]
RAM Size: [e.g. 32GB]
RAM Speed: [e.g. 2666MHz]
Storage Type: [HDD or SSD]
GPU VRAM: [e.g. 2GB]

Most of these information can be found in the task manager

Additional context
Add any other context about the problem here.

Somewhat expected, will make Alfheim disable itself when Mohist is detected next update if no easy fix is possible

Somewhat expected, will make Alfheim disable itself when Mohist is detected next update if no easy fix is possible

Nevermind that should be fixed in 1.0.2:
CurseForge
Modrinth

It still crashes on server loading: https://mclo.gs/BklC3ZV

It still crashes on server loading: https://mclo.gs/BklC3ZV

I fixed that can you try with the new dev build?
https://www.curseforge.com/minecraft/mc-mods/alfheim-lighting-engine/files/4838765

It still crashes on server loading: https://mclo.gs/BklC3ZV

I fixed that can you try with the new dev build? https://www.curseforge.com/minecraft/mc-mods/alfheim-lighting-engine/files/4838765

Yeah, it launches now.

Yeah, it launches now.

Awesome


Update onboard-your-data.md

@mrbullwinkle
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Update Plugin Artifacts Links for v0.180.8

Automated changes by create-pull-request GitHub action

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Metrics didn't scrape from new targets

What did you do?
I use forked version of prometheus 1.3.0 with support discovery from etcd: https://github.com/THE108/prometheus
I update keys in etcd for new targets. Old targets is removed and new target appears in prometheus targets list. But it stay with status "Unknown" in about 10 minutes. This time may vary. After status is changed to "Up".
Usually after that it works normally.
But sometimes it has last scrape like this "11m12.642s ago" and it's grow for a while (maybe several minutes) without getting metrics from that targets. After it works ok.
What did you expect to see?
That target has status "UP"
What did you see instead? Under which circumstances?
See status Unknown and status "UP" with big last scrape
Environment

System information:
Linux 3.10.0-327.10.1.el7.x86_64 x86_64

Prometheus version:

prometheus, version 1.3.0 (branch: HEAD, revision: c887ed97b0f8f8d748de3aa6a11b20b7ad522e2b)
build user:       root@runner-b1ef57c1-project-858-concurrent-0
build date:       20161102-09:53:10
go version:       go1.7.1

Prometheus configuration file:

global:
  scrape_interval: "10s"
  scrape_timeout: "9s"
  evaluation_interval: "15s"
rule_files:
  - /etc/prometheus/generated-rules.rule
scrape_configs:
  - job_name: prometheus
    static_configs:
      - targets:
          - "localhost:9090"
  - job_name: goapi
    scrape_interval: "20s"
    etcd_sd_configs:
      - endpoints:
          - "http://etcd1:2379"
          - "http://etcd2:2379"
          - "http://etcd3:2379"
          - "http://etcd4:2379"
          - "http://etcd5:2379"
        key_prefix: /live
…

I'd presume by default the issue is with the forked code. If you can reproduce this on a vanilla Prometheus please let us know.


Git viewAllChanges command not working with shortcut

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.86.1 arm64
OS Version: macOS Sonoma 14.2.1

Git viewAllChanges is working with the UI but not with a shortcut.

When launching the command with a shortcut, is not working.
Steps to Reproduce:

Create a shortcut with the viewAllChanges command
Example of shortcut

[
    {
      "comment": "Open view all changes",
      "key": "ctrl+shift+alt+g d",
      "command": "git.viewAllChanges"
   }
]

Press shortcut key combination and the result is the command not being launch.

PD:
I the las release version 1.85 this command was working, but the name has been changed in version 1.86. So I guess probably has broken in the new release 1.86
Old command:
[
    {
      "comment": "Open view all changes",
      "key": "ctrl+shift+alt+g d",
      "command": "git.viewChanges"
   }
]

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.86.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.86.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

My mistake, I am using version 1.86.2

@carseven, please try the latest VS Code Insiders release and we have added couple of new commands that work with keybindings:

git.viewChanges
git.viewStagedChanges
git.viewUntrackedChanges

Tested the keybinding for git.viewChanges and it works.


Remove kubernetes as vendor. Replace with client-go

Now that our control-plane is managed with static pods - we might be able to remove kubernetes as a vendored lib and use client-go instead.
As part of this process we might also consider switching to an actual vendoring tool rather than manual steps in a makefile.

Partially updated via #919. We still import k8s.io/kubernetes for CRI.


Slow performance of "Set Variable If" with many cases

We have a test suite that has a 28-case Set Variable If, viz.
${action} The ${element} At Screen XYZ
    ${locator}    Set Variable If    "${element}"=="Element 1"                       ${ELEMENT_1_LOCATOR}
    ...                              "${element}"=="Element 2"                       ${ELEMENT_2_LOCATOR}
    ...                              "${element}"=="Element 3"                       ${ELEMENT_3_LOCATOR}
etc

The performance of this is fine if the ${element} matches a condition high up in the list, but then starts to become very slow for those lower down. Specifically, according to the logs the time to run the few comparisons is around 1ms each, but after a few it starts approximately doubling for each subsequent comparison. In the logs below the time taken for a single comparison reached 9.1 seconds by the time it got to the successful condition, which in this case was the 23rd case of the Set Variable If. Some calls in the actual testcase that's using this took even longer as they referenced elements further down the list, with the result that a test suite with fewer than 20 keyword calls overall was nowhere close to finishing even after ten minutes.
I did some profiling by hacking time.time() printouts into the Robot Framework source code in my venv. The delay seems to be in the logging code, specifically the output.listeners.Listeners and output.listeners.LibraryLIsteners classes:
                        before LOGGER.start_keyword:<PHONE_NUMBER>.7360394
                            <class 'robot.output.logger.LoggerProxy'> took 3.814697265625e-06
                            <class 'robot.output.logger.LoggerProxy'> took 4.9114227294921875e-05
                            <class 'robot.output.listeners.Listeners'> took 4.5487611293792725
                            <class 'robot.output.listeners.LibraryListeners'> took 4.605533123016357

(values above are in seconds) - you can see the two classes took 9.1 seconds between them to process the start_keyword call.
The first line of the above output comes from a line I added to the very top of the calling function, output.output.Output.start_keyword, while the rest of the debug output was generated by changing the start_keyword function of the output.logger.Logger class to look like this:
    def start_keyword(self, keyword):
        # TODO: Could _prev_log_message_handlers be used also here?
        self._started_keywords += 1
        self.log_message = self._log_message
        for logger in self.start_loggers:
            s = time.time()
            logger.start_keyword(keyword)
            e = time.time()
            print(f"                            {type(logger)} took {e-s}")

where my changes were to add the s = , e =  and print lines. Unfortunately I was unable to drill down further as I got lost in the maze of runtime-defined functions. My suspicion was that perhaps the XML/HTML loggers might take a while if they have to reshuffle large amounts of XML/HTML in order to insert a nested log about a keyword call - when you view the HTML log for the Set Variable If, it is nested 23 layers deep.
Robot Framework version is 6.1.1 (according to version.py), running in Python 3.10.12 on Ubuntu 22.04. The test suite is invoked using the following command pabot --pabotlib --processes 1 --variablefile local_variable.py --outputdir Log --include <tag> <testcase root directory>.
Happy to supply any other info, screenshots etc to help. The problem is readily reproducible on my end and for now I'm going to work around it by splitting up the keyword into several keywords each with a smaller Set Variable If.

I tried to reproduce this with a simple example using just False as the condition, but it run fairly fast on my machine. After adding over 100 conditions, the keyword failed with

Maximum limit of started keywords and control structures exceeded.

Only at this point I realized that Set Variable If is implemented so that each condition is evaluated using a separate keyword call. That makes no sense whatsoever, but luckily changing the implementation is easy. After the fix, Set Variable If with 200 conditions in format like ${0} > 1 run in 43 milliseconds on my machine.
The whole `Set Variable If´ being executed as a single keyword can be considered a behavior change. The old approach was so braindead that it's hard to believe anyone would depend on it. I don't consider this a backwards incompatible change.


Add a mkt logs command similar to fig logs.

This should tail logs in the project dir if they exist and fallback to fig logs otherwise.

I think we shouldn't be doing anything in mkt anymore and should be going the docker-utils route.

I think we shouldn't be doing anything in mkt anymore and should be going the docker-utils route.

I concur.


Re-worked TreeMap comparators, considering null-keys

Fixes #1549
Map<K, V> map = TreeMap.empty();

// ordinary Maps are expected to accept null values:
map.put(null, "ok");

Current coverage is 97.06% (diff: 89.07%)

Merging #1695 into master will decrease coverage by <.01%

@@             master      #1695   diff @@
==========================================
  Files            89         89
  Lines         11247      11277    +30
  Methods           0          0
  Messages          0          0
  Branches       1888       1888
==========================================
+ Hits          10917      10946    +29
- Misses          187        192     +5
+ Partials        143        139     -4

Powered by Codecov. Last update ee2f094...69bacc7

This was the last fix for 2.0.5 - now starting to cherry-pick.


Unpermitted parameter :session when signing in using javascript fetch

When posting issues, please include the following information to speed up the troubleshooting process:

Version: 1.1.3
I'm using a javascript fetch method to accomplish this (see code below)

Request and response headers:
Response Headers:

Access-Control-Allow-Methods: GET, POST, OPTIONS, DELETE, PUT
Access-Control-Allow-Origin: http://localhost:3000
Access-Control-Expose-Headers: access-token, expiry, token-type, uid, client
Access-Control-Max-Age: 7200
access-token: uB4qS-ZLdHIWn169_OMu3A
Cache-Control: max-age=0, private, must-revalidate
client: uzZh5YeNpNPrYFQZMffnrg
Content-Type: application/json; charset=utf-8
ETag: W/"80dbc878e17bf9c64918b01f9be06a09"
expiry:<PHONE_NUMBER>
Referrer-Policy: strict-origin-when-cross-origin
token-type: Bearer
Transfer-Encoding: chunked
uid<EMAIL_ADDRESS>Vary: Origin
X-Content-Type-Options: nosniff
X-Download-Options: noopen
X-Frame-Options: SAMEORIGIN
X-Permitted-Cross-Domain-Policies: none
X-Request-Id: bc0975fc-0dc9-4068-a407-6de2446859a4
X-Runtime: 0.577201
X-XSS-Protection: 1; mode=block

Request Headers:
Accept: */*
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9
Connection: keep-alive
Content-Length: 70
Content-Type: application/json
Host: localhost:3001
Origin: http://localhost:3000
Referer: http://localhost:3000/sign-in
Sec-Fetch-Mode: cors
Sec-Fetch-Site: same-site
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.87 Safari/537.36

Request Payload (it's relevant I think):
{email<EMAIL_ADDRESS>password: "123456", foo_bar: "FOO_TEST"}

Rails Stacktrace:

Started POST "/auth/sign_in" for ::1 at 2019-11-23 13:39:29 -0800
Processing by DeviseTokenAuth::SessionsController#create as */*
  Parameters<EMAIL_ADDRESS>"password"=>"[FILTERED]", "foo_bar"=>"FOO_TEST"<EMAIL_ADDRESS>"password"=>"[FILTERED]", "foo_bar"=>"FOO_TEST"}}
Unpermitted parameter: :session
Unpermitted parameter: :session
  User Load (1.8ms)  SELECT "users".* FROM "users" WHERE "users"."email" = $1 AND "users"."provider" = $2 LIMIT $3  [["email"<EMAIL_ADDRESS>["provider", "email"], ["LIMIT", 1]]
Unpermitted parameter: :session
Unpermitted parameter: :session
   (0.3ms)  BEGIN
  User Update ... (removed for brevity because its not relevant)
   (0.5ms)  COMMIT
  User Load ...
   (0.2ms)  BEGIN
  User Load ...
  User Update ...
   (1.9ms)  COMMIT
Completed 200 OK in 538ms (Views: 0.3ms | ActiveRecord: 10.2ms | Allocations: 12814)

Environmental Info:

Routes:
mount_devise_token_auth_for 'User', at: 'auth'
Gems:
Other than the default gems, this is what's being used:

gem 'devise'
gem 'devise_token_auth'
gem 'rack-cors'

Custom Overrides:
application_controller.rb:

class ApplicationController < ActionController::API
  include DeviseTokenAuth::Concerns::SetUserByToken

  before_action :configure_permitted_parameters, if: :devise_controller?

  protected

  def configure_permitted_parameters
    # I originally was seeing this problem with registering users.
    # I get the same problem as signing in a user except I get
    # "registration" instead of "session".
    devise_parameter_sanitizer.permit(:sign_up, keys: [:name])

    # :foo_bar was a sanity check to see what was causing this problem.
    # Signing in a user creates an extra "session" parameter hash.
    # (see the logs).
    devise_parameter_sanitizer.permit(:sign_in, keys: [:foo_bar])
  end
end

Custom Frontend:
Using a vanilla javascript fetch method:

const onSubmit = (e: React.FormEvent<HTMLFormElement>) => {
    e.preventDefault()
    console.log("Form submitted");

    fetch("http://localhost:3001/auth/sign_in", {
      method: 'POST',
      headers: {
        'Content-Type': 'application/json'
      },
      body: JSON.stringify({
        email,
        password,
        foo_bar: "FOO_TEST"
      })
    })
    .then((response) => {
      for (var pair of response.headers.entries()) {
        console.log(pair[0] + ': ' + pair[1])
      }
      console.log('RESPONSE HEADERS: ', response.headers)
      return response.json()
    })
    .then((data) => {
      console.log(data);
    })
    .catch((error) => {
      console.error('ERROR: ', error);
    })
  }

If you look at the log, you'll see that my payload is sent the form data correctly but I think DTA is adding another set of my parameters under "session". I'm getting the same issue when I go to register a user but the parameters are under "registration". I'm probably doing something wrong here (maybe in the application_controller.rb?), so any help would be much appreciated.

Same issue?
https://github.com/lynndylanhurley/devise_token_auth/issues/676

Thanks! Yeah that fixed it. I'll close this.


Metric argo_pod_missing documentation/usage

Summary
Hi, we want to use  argo_pod_missing metric in our infrastructure but we don't understand it and we think that the documentation is not clear enough "Pods were not seen. E.g. by being deleted by Kubernetes. You should only see this under high load."
Use Cases
We use this in our Datadog infrastructure to set some monitors:

As you can see, the value is above 10k.
What's the real meaning/definition of this metric?
Thanks.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Hey Argo team!
I'd love to work on this quick fix and help you guys out.
Please let me know if you're okay with me working on this and I'll send you a PR for review soon.

Nobody on this?


Move Block internals code to Cython

I'm not sure if there's any way to not break legacy pickles during this process. The Blocks and BlockManager should be extension classes

Closing as Won't Fix


ux-button

Implement the ux-button component for Material and iOS design.

Issue moved to aurelia/ux-components #17 via ZenHub


For 8-ball, allow all the regular colors for solids and stripes as a seperate gamemode.

Within testing and replacing element 0 with the colors needed, it seems to work just fine like it worked straight out of the box. I may take a jab at it once the next release is out.


Can ResultArrayAsInt return 0?

"Produces a positive integer", but surely if results is all zeros, it should return 0?

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

ID: 9db98377-dbb2-24bb-79c3-607d827a1fc0
Version Independent ID: d76a017a-0950-640c-ceda-e7156bd8db55
Content: ResultArrayAsInt function - Q# reference - Microsoft Quantum
Content Source: api/qsharp/microsoft.quantum.convert.resultarrayasint.md
Product: quantum
GitHub Login: @ricardo-espinoza

Good catch, 0 is returned if all results are Zero.

Yeah, I was reading https://docs.microsoft.com/en-us/learn/modules/qsharp-create-first-quantum-development-kit/5-random-number-generator and got slightly confused by this mistake.


feature/script-cache

Provides a script caching class to load scripts included as files in the resources path.
Currently scripts are evaluated lazily, and loaded if they haven't already been.  This is a performance problem for larger scripts and for files, since they must be read and passed into RedisScript every time they are called.  It's a much easier and more common pattern to load Redis scripts in an initializer so they are available on-demand without having to first read and load them into Redis at runtime.
RedisScriptCache holds a simple mutable.Map to store loaded scripts with their names as keys.  It can read and register all scripts in a provided directory and return a Future response when they have all been loaded for initialization purposes.  To then run a script which has been cached, only the name of the script is needed.

Coverage decreased (-0.46%) to 91.25% when pulling 62b0cfda26a5a3c61f5386c8ea8b9c39bd76fb57 on alexanderscott:feature/script-cache into 790d76411acaca20ebde49e9a6369b1c6ecfceae on etaty:master.

Hi,
I think it is a good idea to put the lua scripts in the resource folder.

This is a performance problem for larger scripts and for files, since they must be read and passed into RedisScript every time they are called.

I don't agree
https://github.com/etaty/rediscala/blob/master/README.md#scripting
val redisScript = RedisScript("return 'rediscala'")
val r = redis.evalshaOrEval(redisScript)

I don't see any problem with this code. redisScript can be kept outside of a function if you want to limit instantiation.

However I was thinking that adding the following api might be helpful
RedisScript.fromFile(file:File)
RedisScript.fromResource(path:String)

For "script loading", I think it should depends of the project architecture.
Does it really takes time to load script in Redis? (I don't think so)

I see what you are saying, and understand why it shouldn't be a problem to evalshaOrEval using a small block of inlined code.  However, when using lua scripts from a file with this function, the file must be opened and read every time (thus a performance hit for file operation).  I don't think it takes a very long time to load a script into Redis, but for larger string blocks could mean a considerable amount of I/O since the whole string is passed within the evalshaOrEval command (for reference I have a couple of lua scripts over 100 lines).
And yes, I agree script loading can depend on the project architecture.  Though I do think it's a common pattern to load all scripts inside of an initializer rather than lazily, and having a helper class for doing this is a nice feature so it doesn't need to be re-implemented all the time.
Yes, adding that fromFile command could be helpful.  The purpose of this pull was largely just convenience, so it's totally up to you as the package maintainer on what you think is most helpful without imposing opinion or restriction on how the package is used.  Let me know how/if you would like to proceed and I'd be happy to make the changes in order to give back to the project.

Closing this pull after adding RedisScript.fromFile and RedisScript.fromResource helpers in #80


Question about runtime setup please

Hi Mihai,
I seem to really suck at guessing the sbt configuration for safely running the sample code at https://github.com/sistanlp/processors#annotating-entire-documents within my own application. I made my way through adding the necessary imports, after having added the maven dependency in my build.sbt, and hoped for good. I get the below exception for the line val doc = proc.annotate("John Smith went to China. He visited Beijing, on January 10th, 2013.")
00:11:17.670 [run-main-0] DEBUG e.a.s.discourse.rstparser.RSTParser - Loading RST parsing model from: edu/arizona/sista/discourse/rstparser/model.const.rst.gz
[error] (run-main-0) java.lang.NullPointerException
java.lang.NullPointerException
	at java.util.zip.InflaterInputStream.<init>(InflaterInputStream.java:83)
	at java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:76)
	at java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:90)
	at edu.arizona.sista.discourse.rstparser.RSTParser$.loadFrom(RSTParser.scala:202)
	at edu.arizona.sista.processors.corenlp.CoreNLPProcessor$.fetchParser(CoreNLPProcessor.scala:465)
	at edu.arizona.sista.processors.corenlp.CoreNLPProcessor.rstConstituentParser$lzycompute(CoreNLPProcessor.scala:36)
	at edu.arizona.sista.processors.corenlp.CoreNLPProcessor.rstConstituentParser(CoreNLPProcessor.scala:36)
	at edu.arizona.sista.processors.corenlp.CoreNLPProcessor.discourse(CoreNLPProcessor.scala:453)
	at edu.arizona.sista.processors.Processor$class.annotate(Processor.scala:75)
	at edu.arizona.sista.processors.corenlp.CoreNLPProcessor.annotate(CoreNLPProcessor.scala:25)
	at edu.arizona.sista.processors.Processor$class.annotate(Processor.scala:54)
	at edu.arizona.sista.processors.corenlp.CoreNLPProcessor.annotate(CoreNLPProcessor.scala:25)
	at my scala line of code copied above .....
	at Boot$delayedInit$body.apply(Boot.scala:14)
	at scala.Function0$class.apply$mcV$sp(Function0.scala:40)
	at scala.runtime.AbstractFunction0.apply$mcV$sp(AbstractFunction0.scala:12)
	at scala.App$$anonfun$main$1.apply(App.scala:71)
	at scala.App$$anonfun$main$1.apply(App.scala:71)
	at scala.collection.immutable.List.foreach(List.scala:318)
	at scala.collection.generic.TraversableForwarder$class.foreach(TraversableForwarder.scala:32)
	at scala.App$class.main(App.scala:71)
	at Boot$.main(Boot.scala:4)
	at Boot.main(Boot.scala)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)

Can you please advise what should I include in my build configuration, env variables or classpath, in order to include everything that's needed for run time?
Should I really avoid Java 7 with version 3.3?
I added the following in my build.sbt to that effect:
scalaVersion  := "2.10.4"
scalacOptions += "-target:jvm-1.6"
javacOptions ++= Seq("-source", "1.6", "-target", "1.6")

I have specifically installed Oracle Java 6 now for this (I had only Java 7 before on the current machine), but I think that all should not be required as I assume my Java 7 code should be able to use Java 6 class files. So some advice would come in very much appreciated here....

Hi,
Most of the code works with Java 7 now, with the exception of some concurrency unit tests. If multi-threading is important for your application, I suggest staying with Java 6 byte code. This can be enforced even when the underlying JVM is Java 7, with these sbt options:
scalacOptions += "-target:jvm-1.6"
javacOptions ++= Seq("-source", "1.6", "-target", "1.6")
All our unit tests pass with Java 7, if the above options are present.
Wrt the above Exception, that happens when the runtime doesn't find the processors-*-models.jar. To make sure this is include in the class path, add these dependencies to the sbt of your project:
"edu.arizona.sista" %% "processors" % "4.0-SNAPSHOT",
"edu.arizona.sista" %% "processors" % "4.0-SNAPSHOT" classifier "models",
(If you make changes to the processors source code, make sure to run "sbt publish-local" in the processors project to install the new jars in the correct locations for sbt to see them)

This is fixed?
We can open it again, if not.


Update volumes.md

I'm including a new solution to be referenced on Volumes Section
It's called Veritas InfoScale for Kubernetes / Alta Enterprise Resiliency for Kubernetes is a Software defined solution deployed over containers to provide High performance and DR for persistent storage

I'm including a new solution to be referenced on Volumes page
It's called Veritas InfoScale for Kubernetes / Alta Enterprise Resiliency for Kubernetes is a Software defined solution deployed over containers to provide High performance and DR for persistent storage

Veritas InfoScale is not part of Kubernetes and this content does not align with our policy on 3rd party content.
/close


[BUG] hyperswitch-server panics on startup when running docker compose locally

Bug Description
When running docker compose up on a freshly cloned repo, the server container immediately throws the error "Unable to deserialize application configuration: pm_filters.zsl.local_bank_transfer: data did not match any variant of untagged enum PaymentMethodFilterKey" and exits with code 101.
Expected Behavior
No error
Actual Behavior
Full error with RUST_BACKTRACE=full
2024-04-15 11:02:29 Unable to deserialize application configuration: pm_filters.zsl.local_bank_transfer: data did not match any variant of untagged enum PaymentMethodFilterKey
2024-04-15 11:02:29 thread 'main' panicked at crates/router/src/bin/router.rs:14:10:
2024-04-15 11:02:29 Unable to construct application configuration: ConfigurationError(data did not match any variant of untagged enum PaymentMethodFilterKey)
2024-04-15 11:02:29 stack backtrace:
2024-04-15 11:02:29    0:     0x55555a1b68e9 - <unknown>
2024-04-15 11:02:29    1:     0x55555671e950 - <unknown>
2024-04-15 11:02:29    2:     0x55555a17f8e2 - <unknown>
2024-04-15 11:02:29    3:     0x55555a1b843e - <unknown>
2024-04-15 11:02:29    4:     0x55555a1b7be0 - <unknown>
2024-04-15 11:02:29    5:     0x55555a1b8d6b - <unknown>
2024-04-15 11:02:29    6:     0x55555a1b8790 - <unknown>
2024-04-15 11:02:29    7:     0x55555a1b86e6 - <unknown>
2024-04-15 11:02:29    8:     0x55555a1b86df - <unknown>
2024-04-15 11:02:29    9:     0x555555c5ec34 - <unknown>
2024-04-15 11:02:29   10:     0x555555c5f1a2 - <unknown>
2024-04-15 11:02:29   11:     0x5555560af0e4 - <unknown>
2024-04-15 11:02:29   12:     0x5555560a887f - <unknown>
2024-04-15 11:02:29   13:     0x555555e912d3 - <unknown>
2024-04-15 11:02:29   14:     0x5555560b1184 - <unknown>
2024-04-15 11:02:29   15:     0x7fffff76d24a - <unknown>
2024-04-15 11:02:29   16:     0x7fffff76d305 - __libc_start_main
2024-04-15 11:02:29   17:     0x555555ce22c1 - <unknown>
2024-04-15 11:02:29   18:                0x0 - <unknown>

Steps To Reproduce

Clone hyperswitch repo
Run docker compose up
hyperswitch-server immediately exits

Context For The Bug
No response
Environment
Are you using hyperswitch hosted version? No
If not (or if building/running locally), please provide the following details:

Operating System or Linux distribution: macOS Ventura 13.6.6
Rust version (output of rustc --version): Docker
App version (output of cargo r --features vergen -- --version): Docker

Have you spent some time checking if this bug has been raised before?

[X] I checked and didn't find a similar issue

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to submit a PR?
None

This issue occurs when checking out current main, but does when checking out v1.107.0

Docker Compose setup pulls the latest stable image, v1.107.0. However, the payment method was added after the release of v1.107.0, resulting in an error. Therefore, I kindly request you to checkout to the stable release tag v1.107.0 and then proceed to run Docker Compose.


No module named 'mmdet3d.models'

The folder tree in hrmapnet is following:
├── mmdet3d
│   ├── init.py
│   ├── pycache
│   ├── apis
│   ├── core
│   ├── datasets
│   ├── ops
│   ├── utils
│   └── version.py
but in official https://github.com/open-mmlab/mmdetection3d the folder tree is

The error is not found models.

Thanks for pointing out this. Some folders were not uploaded by mistake...


Coreg.apply replace nodata with max value

When calling the Coreg.apply method using a np.nd_array rather than a Raster, the output contains a finite value (I believe the array's max) instead of NaNs. See figure below.
To reproduce, run the Nuth & Kaab example and run the following commands:
# Add large gaps on the right of the DEM
dem_to_be_aligned.data[:,:,1000:] = dem_to_be_aligned.nodata
dem_to_be_aligned.data[:,:,1000:].mask = True

# Fit coregistration again
nuth_kaab.fit(reference_dem.data, dem_to_be_aligned.data, inlier_mask, transform=reference_dem.transform)

# Apply coreg using a Raster => this works
aligned_dem = nuth_kaab.apply(dem_to_be_aligned)
plt.imshow(aligned_dem.data.squeeze())
plt.show()

# Apply coreg using an array => this turns NaNs into finite values
aligned_dem = nuth_kaab.apply(dem_to_be_aligned.data, dem_to_be_aligned.transform)
plt.imshow(aligned_dem.data.squeeze())
plt.show()

This causes problems in a CoregPipeline, since coreg.apply is called using the np.nd_arrays and so the second coregistration step fails.


Fix (and auto validate) 404ing repository_url fields in info.yml

Rename repository_url -> definition_url, referencing the scheme def URL in the repo
We have to be wary of the 100req/hr rate limit for validation, best to get this right by auto populating field at build time


TT-173 - Change method for reading pending transactions to sign

Task linked: TT-173 [KMS] Change method for reading pending transactions to sign

@isra67 WE are missing README information what is happening and limitations which are being introduced with this change.


Amon Agent Stops Sending Data

I believe this might be similar the issue/request brought up on #9.
We have 2 servers that randomly stop sending data (ironically I restart them around the same time and they seem to stop sending data around the same time). When I check the amonagent.log I can see that no new entries are logged starting around the time they stop sending data.
I am still in the process of trying to debug/troubleshoot what might be causing this. None of our other servers have this issue, which is leading me to think that a) some other process on these servers are interrupting amonagent or b) something about how these servers are configured (at the OS level) is causing the problem.
Some facts:

both servers are Ubuntu 14.04 server
we have not enabled any health checks or plugins on these two servers
a manual restart of the amonagent service resolves the issue
a status check on the amonagent service before restarting the service indicates that the amonagent is running.
both are running amonagent 7.2

I am going to try updating them to the latest amonagent today to see if that helps resolve the issue.
I will gladly update this issue with any additional findings or potential patches if I am able to track this down.

@jyksnw Can you check the log file, I think the default is set to INFO and logs every request made + a timestamp. Maybe it will be easier to debug if we know when the agent stopped sending data.

@martinrusev
Here are the lines of the log from one of the servers (I have obfuscated the URI and API key).
time="2017-03-21T17:44:05-04:00" level=info msg="Metrics collected (Interval:1m0s)\n"
time="2017-03-21T17:44:05-04:00" level=info msg="Sending data to https://server1.example.com/api/system/v2/?api_key=XXXXXXXXXX\n"
time="2017-03-22T06:25:11-04:00" level=info msg="Starting Amon Agent (Version: 0.7.2)\n"
time="2017-03-22T06:25:11-04:00" level=info msg="Agent Config: Interval:1m0s\n"
time="2017-03-22T06:25:19-04:00" level=info msg="Metrics collected (Interval:1m0s)\n"
time="2017-03-22T06:25:20-04:00" level=info msg="Sending data to  https://server1.example.com/api/system/v2/?api_key=XXXXXXXXXX\n"

I think I see the issue. It appears to be a two part issue:

After searching the logs I found that named had a number of logged errors indicating that it couldn't resolve to our amon server.
Go's http.Client defaults to a timeout of 0 which is no timeout. It appears that in the scenario above the call to SendData never returns as the client is stuck looking to complete the connection to a hostname that it can't resolve or reach.

Luckily this is easy to fix by creating the http.Client with a specified timeout. I can add this in without any issue but wanted to know if the timeout should be a configuration option or a statically set value (say 10 seconds).
I am going to create a local build to test this theory out, but after reading through our logs and looking into the http.Client request handling I am highly confident this was the cause of this issue.

Sorry I didn't look into how the transport was constructed before commenting. Looks like a 10 second timeout is already being utilized via the transport.

@jyksnw It could be a goroutine leak somewhere, although I do check for data races before releasing. One way to determine if that is the case is to monitor the memory usage.
What makes this one difficult to catch I think is that it has some parts of this bug which are hardware / distro related. I personally have 5 agents that have been running since last August

We have 3 other servers running with similar hardware/distro configuration that we haven't seen any issues on.
CloudFlare has en excellent writeup and graph outlining Go's client connection sequence and where each of the various timeout settings come into play.

Source - CloudFlare: The complete guide to Go net/http timeouts
So though a timeout is being set for ResponseHeaderTimeout, the request might not have reached that point and still stuck. There is a suggested Transport structure setup in the write-up that could be implemented. I will create a build for just these two servers with the suggested Transport setup and see if the issue presents itself again.

@jyksnw Thanks for sharing the guide. Yes, this could be the issue - the amonagent does not have a cancel request policy, just timeout

I have a local branch that implements a more fine grained timeout along with adding a cancel request policy that currently cancels the request after a 10 second delay. I will test this out a bit against the two servers we have been having issues with to see if it solves the problem as well as see if it introduces any other potential issues.

@jyksnw Cool. If it works - you can submit as a pull request and I will merge / push a new release for the agent with the fix


Strange things with IRC mod

Strange things happenned with the IRC mod when the server boot :
17:13:32: ACTION[main]:         .__               __                   __
17:13:32: ACTION[main]:   _____ |__| ____   _____/  |_  ____   _______/  |_
17:13:32: ACTION[main]:  /     \|  |/    \_/ __ \   __\/ __ \ /  ___/\   __\
17:13:32: ACTION[main]: |  Y Y  \  |   |  \  ___/|  | \  ___/ \___ \  |  |
17:13:32: ACTION[main]: |__|_|  /__|___|  /\___  >__|  \___  >____  > |__|
17:13:32: ACTION[main]:       \/        \/     \/          \/     \/
17:13:32: ACTION[main]: World at [/home/quentinbd/minetest/worlds/minetestforfun/]
17:13:32: ACTION[main]: Server for gameid="minetestforfun_game" listening on <IP_ADDRESS>:30001.
17:13:45: ACTION[ServerThread]: Updated online player file
17:13:45: ACTION[ServerThread]: There are 12467 registered nodes, items and tools.
== This is a debug line, please check for it ==
nil
=== DEBUG FINISHED ===
17:13:56: ACTION[ServerThread]: IRC: Connected!
.../quentinbd/minetest/bin/../mods/irc/irc/messages.lua:106: bad argument #1 to 'sub' (number expected, got nil)
stack traceback:
        [C]: in function 'sub'
        .../quentinbd/minetest/bin/../mods/irc/irc/messages.lua:106: in function 'fromRFC1459'
        .../quentinbd/minetest/bin/../mods/irc/irc/messages.lua:13: in function 'Message'
        ...home/quentinbd/minetest/bin/../mods/irc/irc/main.lua:165: in function 'think'
        /home/quentinbd/minetest/bin/../mods/irc/init.lua:71: in function </home/quentinbd/minetest/bin/../mods/irc/init.lua:71>
        [C]: in function 'xpcall'
        /home/quentinbd/minetest/bin/../mods/irc/init.lua:71: in function </home/quentinbd/minetest/bin/../mods/irc/init.lua:60>
        (tail call): ?
        .../quentinbd/minetest/bin/../builtin/game/register.lua:341: in function <.../quentinbd/minetest/bin/../builtin/game/register.lua:329>

You can see it also here : http://zerobin.qbuissondebon.info/?82d5718a6d807855#UgLpJ43xJh/PdmwLRixccWH84TzBZ7oo4+dQiaygJo8=
What's happening ?

Incompatibility between IRCd and user. Apparently, this function is supposed to read packets from IRCds following the RFC 1459 and not updated since then. A line was apparently misread and caused a crash when trying to use a not found expression in it.
I added an error message providing a value for this, on both actual and old irc versions.
https://github.com/Ombridride/minetest-minetestforfun-server/commit/582f7a363509f6c6684c5f607cc3ecc64f1027a0

Ok, i will see the return for next reboot

The next reboot was OK, i closed the issue


OS-8 | Better test coverage

Pull Request Test Coverage Report for Build 899726992

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 81.651%

Files with Coverage Reduction
New Missed Lines
%

cmd/unseal.go
4
45.45%

Totals

Change from base Build 888242453:
0.0%

Covered Lines:
89

Relevant Lines:
109

💛  - Coveralls


integration-docs: Update Giphy for new doc format.

Fixes: Part of zulip#29592.
Separated out from #32354.

Screenshots and screen captures:
Doc link

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

No updates since last review, none requested.
@zulipbot add "integration review"


Adding devops and acs modules to cli.

Hi @sanar-microsoft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (sanar). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Merging PR.


Async Daemon fails with ConcurrencyException when processing async projections with documents that use Optimistic Concurrency

It works fine when the projected document does not have OptimisticConcurrency turned on. As soon as you add Optimistic Concurrency, the daemon throws this exception:
Exception:
 ---> Marten.Exceptions.ConcurrencyException: Optimistic concurrency check failed for Marten.AsyncDaemon.Testing.TestingSupport.Trip #c9b286ac-2f67-4e98-99d1-2f379c7cd15d
   --- End of inner exception stack trace ---
   at Marten.Events.Daemon.ProjectionUpdateBatch.Marten.Internal.IUpdateBatch.ApplyChangesAsync(IMartenSession session, CancellationToken token) in /Users/ericgreen/OSS/marten/src/Marten/Events/Daemon/ProjectionUpdateBatch.cs:line 110
   at Marten.Internal.Sessions.DocumentSessionBase.ExecuteBatchAsync(IUpdateBatch batch, CancellationToken token) in /Users/ericgreen/OSS/marten/src/Marten/Internal/Sessions/DocumentSessionBase.SaveChanges.cs:line 182
   at Marten.Internal.Sessions.DocumentSessionBase.ExecuteBatchAsync(IUpdateBatch batch, CancellationToken token) in /Users/ericgreen/OSS/marten/src/Marten/Internal/Sessions/DocumentSessionBase.SaveChanges.cs:line 204
   at Marten.Events.Daemon.ShardAgent.ExecuteBatch(ProjectionUpdateBatch batch) in /Users/ericgreen/OSS/marten/src/Marten/Events/Daemon/ShardAgent.cs:line 431
   at Marten.Events.Daemon.ShardAgent.ExecuteBatch(ProjectionUpdateBatch batch) in /Users/ericgreen/OSS/marten/src/Marten/Events/Daemon/ShardAgent.cs:line 449
   at Marten.Events.Daemon.ShardAgent.<>c__DisplayClass44_0.<<processRange>b__2>d.MoveNext() in /Users/ericgreen/OSS/marten/src/Marten/Events/Daemon/ShardAgent.cs:line 228
--- End of stack trace from previous location ---
   at Marten.Events.Daemon.ProjectionDaemon.TryAction(ActionParameters parameters) in /Users/ericgreen/OSS/marten/src/Marten/Events/Daemon/ProjectionDaemon.cs:line 364

Failing test PR coming shortly added

@jeremydmiller That makes sense for Async projections, however we have inline projections that we need optimistic concurrency on.  That seems like a scenario that should work or ignore the OC setting.

I almost buy that, but there's existing optimistic concurrency checks at the event stream level too.
If this is really something you have to have, I'd say the answer is that in the aggregate projection codegen, it opts into using the Overwrite() operations instead of Store() whenever the document has optimistic concurrency turned on.


Oculus Quest app crashes when loading demanding scene with RenderDoc attached

Description
I'm trying to diagnose a scene in an Oculus Quest title (in development) that is hitting the device's GPU hard. I am able to attach RenderDoc to the Quest, launch/attach to the game, and capture frames from other parts of the game.
However, as soon as I try to load into the scene in question, the app crashes out and RenderDoc disconnects. Unfortunately I'm not seeing particularly useful messages in the relevant log files. While the scene is demanding on the device (hence wanting to capture it) it doesn't crash unless RenderDoc is attached.
Including log files from RenderDoc and adb. Any help would be much appreciated! Thanks.
bugreport-256550.6810.0-2019-07-02-12-22-36.txt
CrashLog.log
Repro steps
Attach RenderDoc to Oculus Quest replay context
Launch app in question
Load into scene in question
Crash
Environment

RenderDoc build: 6/30/2019 nightly build
Operating System: profiling Oculus Quest/Android on Windows 10
API: OpenGLES 3.1

Unfortunately I don't have any immediate ideas what this might be. The repro steps are not too useful if I don't have your app to be able to follow them - are you able to share your app with me privately?
Normally if there's an actual crash then the android system dumps out a callstack which would appear in the log, but there's nothing like that - it maybe suggests you're hitting out-of-memory and the crash is actually a forced process kill? I don't know what the bugreport.txt is from and without being familiar with it I can't find much useful in it. There are some mentions of out of memory errors but I can't tell whether or not they happened at the same time as the crash.
It could be that the scene you're interested in is skating very close to the memory limit, and the extra overhead that RenderDoc brings in GLES mode pushes it over the top. At the moment there's not much that you can do about that apart from maybe loading lower resolution textures or smaller asset LODs - if that still reproduces your bug you might then be able to get a capture.

@jkant, can you contact also Oculus Store Ops passing the App ID, and the app title? If your app is still under development, and you can't share it here, Oculus developers should be able to test it, and probably check specific scene issue. Also, does this crash happens while your app is running in background in Renderdoc, or on Active capture?

Turns out this was an out of memory issue. Sorry for the false alarm!!

@jkant, thanks for confirming. Yes, we're investigating OOM issue right now (https://github.com/baldurk/renderdoc/pull/1434 is for Active capturing, and we'll need to look into background capturing issues as well).


Add hint to use Docker Hub access tokens

Adding a hint to use Docker Hub access tokens instead of the Docker Hub user account password.
I specified my Docker Hub user account password in the .env file and mrsk failed with the following message:
ERROR (SSHKit::Command::Failed): docker exit status: 256
docker stdout: Nothing written
docker stderr: WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error response from daemon: Get "https://registry-1.docker.io/v2/": unauthorized: incorrect username or password

I resolved the issue by creating a Docker Hub access token and specifying the access token in the .env file:
MRSK_REGISTRY_PASSWORD=my-docker-hub-access-token
Adding the comment might save any newbies like me to run into the same error.

Added as a general point in f73c526890430f1653e79a973119789bcb2f94d7 👍. Applies to all registries that use tokens.