The agent doesn’t send its own logs

What happened?
Description
Field set:
logsEngine: otel
Setting the value values.logsCollection.containers.excludeAgentLogs: false doesn’t work.
The agent sends all logs except logs from: /var/log/pods/*/otel-collector/*.log.
After that I tried another option. I copied and created an additional filelog receiver, set include: [/var/log/pods/otel*/otel-collector/*.log]. I noticed that operators[14] doesn’t work in receiver:
- field: resource[“com.splunk.sourcetype”]
  type: add
  value: EXPR(“kube:container:”+resource[“k8s.container.name”])

If you remove this field or replace it with
- from: resource[“k8s.container.name”]
  to: resource[“com.splunk.sourcetype”]
  type: copy

, then the agent logs are sent to splunk.
But I would like to receive logs with correct sourcetype.
Also I noticed that I can’t change the resource[“com.splunk.sourcetype”].
It is not possible to change this field using processors: attributes, resource, transform. When setting a value with a string or expression, logs don’t come.
It also fails to change sourcetype if you specify it in
filelog:
  resource:
    com.splunk.sourcetype: “custom_value”

as described by the link
Expected Result
logs from agent (/var/log/pods/*/otel-collector/*)
Actual Result
No logs
Chart version
0.85.0
Environment information
Environment
Cloud: "EKS"
k8s version: 1.27.0
Chart configuration
No response
Log output
No response
Additional context
No response

I have not ever had issues with setting exclude agent logs to false.
You should not need to create your own filelog receiver..that gets into advanced pipeline building which may be prone to error that can explain the operator and processor not working.
Will test that version to confirm.

I have not ever had issues with setting exclude agent logs to false.
You should not need to create your own filelog receiver..that gets into advanced pipeline building which may be prone to error that can explain the operator and processor not working.
Will test that version to confirm.

A custom receiver and pipeline was created later as a separate option.

I've just tested this with 0.85.0 (but not EKS) and works fine.
@chisuzume at the beginning your values.yaml contained just the default options and values.logsCollection.containers.excludeAgentLogs: false? No custom changes?

@omrozowicz-splunk , no special changes, but clusterReseiver is enabled, splunk_hec/platform_logs exporter is used in the logs pipeline.

@omrozowicz-splunk , no special changes, but clusterReseiver is enabled, splunk_hec/platform_logs exporter is used in the logs pipeline.

splunk_hec/platform_logs is something enabled by default. Can you send your anonymized values.yaml? We've tested it on EKS as well and it also works fine.

@matthewmodestino A custom receiver and pipeline was created later as a separate option

Right, which also was unsuccessful, so lets focus on how you are trying to deploy the helm chart and if your settings are actually making it to the configmap..if not something is wrong with your values.yaml

Hi @matthewmodestino @omrozowicz-splunk , I tried to simplify values.yaml, the usual logs were received, as they were then, but the agent's own logs were not received. I mean the logs that you can see when executing the following command: kubectl logs -n <ns> <otel collector pod name> these are logs from the file /var/log/pods/*/otel-collector/*.log, not /var/log/pods/*/migration-checkpoint/*.log.
values.yaml:
clusterName: <CUSTOM>
logsEngine: otel
splunkPlatform:
  endpoint: <CUSTOM>
  index: <CUSTOM>
secret:
  create: <CUSTOM>
  name: <CUSTOM>
  validateSecret: <CUSTOM>
cloudProvider: "aws"
distribution: "eks"
image:
  otelcol:
    repository: <CUSTOM>
logsCollection:
  containers:
    enabled: true
    excludeAgentLogs: false

Perhaps its how you are searching for them? Testing again..

Nothing in your values.yaml points to misconfiguration. We have tests covering this functionality, showing we can get agent logs when excludeAgentLogs is set to false. See https://github.com/signalfx/splunk-otel-collector-chart/blob/main/test/k8s_agent_pod_tests/test_agent_correctness_tests.py#L67 for the query we use to search for them.

Tested on 0.85.0 with the following config:
values.yaml
clusterName: "mattymo-test"
endpoint: "https://foo.bar.baz:8088/services/collector"
token: "00000000-0000-0000-0000-000000000000"
insecureSkipVerify: true
logsEngine: otel
containerRuntime: "containerd"
excludeAgentLogs: false

helm -n otel install mattymo-test -f values.yaml splunk-otel-collector-chart/splunk-otel-collector --version 0.85.0

kubectl get pods -w -n otel
NAME                                             READY   STATUS            RESTARTS   AGE
mattymo-test-splunk-otel-collector-agent-ctr4g   0/1     PodInitializing   0          33s
mattymo-test-splunk-otel-collector-agent-ctr4g   0/1     Running           0          34s
mattymo-test-splunk-otel-collector-agent-ctr4g   1/1     Running           0          35s

kubectl -n otel logs -f mattymo-test-splunk-otel-collector-agent-ctr4g

2023/10/13 23:20:33 settings.go:399: Set config to [/conf/relay.yaml]
2023/10/13 23:20:33 settings.go:452: Set ballast to 165 MiB
2023/10/13 23:20:33 settings.go:468: Set memory limit to 450 MiB
2023-10-13T23:20:34.017Z	info	service/telemetry.go:84	Setting up own telemetry...
2023-10-13T23:20:34.017Z	info	service/telemetry.go:201	Serving Prometheus metrics	{"address": "<IP_ADDRESS>:8889", "level": "Basic"}
2023-10-13T23:20:34.018Z	info	kube/client.go:107	k8s filtering	{"kind": "processor", "name": "k8sattributes", "pipeline": "logs", "labelSelector": "", "fieldSelector": "spec.nodeName=so1"}
2023-10-13T23:20:34.018Z	info	memorylimiterprocessor@v0.85.0/memorylimiter.go:102	Memory limiter configured	{"kind": "processor", "name": "memory_limiter", "pipeline": "logs", "limit_mib": 450, "spike_limit_mib": 90, "check_interval": 2}
2023-10-13T23:20:34.036Z	info	service/service.go:138	Starting otelcol...	 Begin running and processing data.
2023-10-13T23:20:34.323Z	info	fileconsumer/file.go:194	Started watching file	{"kind": "receiver", "name": "filelog", "data_type": "logs", "component": "fileconsumer", "path": "/var/log/pods/kube-system_calico-kube-controllers-6c99c8747f-2mr29_fc0baf10-376a-438b-a967-70c4ad5d3c39/calico-kube-controllers/0.log"}

And in Splunk:
index=main sourcetype="kube:container:otel-collector" | reverse

working as expected.
Here's the rendered configmap for good measure:
kubectl -n otel get cm mattymo-test-splunk-otel-collector-otel-agent -o yaml

<truncated>
  receivers:
      filelog:
        encoding: utf-8
        exclude: null
        fingerprint_size: 1kb
        force_flush_period: "0"
        include:
        - /var/log/pods/*/*/*.log
        include_file_name: false
        include_file_path: true
        max_concurrent_files: 1024
        max_log_size: 1MiB
        operators:
<truncated>

note it is not excluding the log path for the otel collector.
working as expected. must be how they are being searched or perhaps routed once they hit Splunk by props and transfroms. Keep in mind the collector doesnt say a whole lot when working...so ensure you are searching a long enough span.

Closing this issue as inactive. Please reopen if more work is needed.


Document spy properties (calls)

What would you think of adding properties to the documentation about spies?
As I was first using this library's spies today, it wasn't immediately evident to me that there was a calls property that I could use to count and check individual calls — because it was just part of an example instead of being explicitly documented along with the methods.
We could change that section title to "Spy methods and properties", then add a subsection for calls, and document the property of the call object that calls lists.
I'd be happy to write up a PR for this, but wanted to run it by you first to see what you think.

@davidtheclark That would be great, thank you. Overall the API docs are kind of getting long. Eventually I'd like to find a way to make browsing them a bit easier.

Ok. I'll try to find time to put together a quick PR this weekend.


Swizzle it just a little bit

Fixes #179
This approach amounts to a simplified version of hackishlyHidesInputAccessoryView, but removes all instances of the string UIWebBrowserView (as I believe this string is what was causing the App Store rejections).
This version retains classname checking but only checks for the "UIWeb" prefix in order to avoid referencing the string UIWebBrowserView.
The classname check is needed to avoid swizzling every view in webView.scrollView.subviews (originally I thought I would be able to limit this via respondsToSelector: @selector(inputAccessoryView), but then I realized that every UIView responds to @selector(inputAccessoryView)).
According to this SO post, this classname check will not cause problems with app store.
However, if it is still a concern, the classname check could be removed but this might have a performance impact on some apps that have a lot of subviews (e.g., apps with a lot of images).

Thanks for the PR! I'm wondering if just replacing the inputAccessoryView method on the existing UIWebBrowserView class is less intrusive that replacing the class entirely? Looking at something like https://github.com/cjpearson/cordova-plugin-keyboard/blob/master/src/ios/CDVKeyboard.m#L118-L146, I feel like it's less code and fails pretty gracefully. Also it's potentially less of a red flag to Apple if at runtime the class itself doesn't change, although from what I can tell they are just searching for string matches.
In the interest of transparency it might make sense to pull this out into its own plugin. So if you do want to hide the bar, you are explicitly opting in to potential future rejection, considering there is no way to do this with public APIs at the moment. Any thoughts?

I thought about just replacing the method, but wasn't sure how to do it.  Now that I see the example, it looks pretty straightforward!  Personally, I don't think it will be less of a red flag - I think the rejections are based on string matching, but I like that it is simpler.
I see where you are coming from in wanting to shield users from possible rejection.
Personally, I think avoiding strings that match non-public API and class names will be sufficient to avoid rejections, but isolating this functionality might be reassuring for some users.
Feel free to close this if you want to go with cjpearson's simpler approach and/or want to pull this out into a separate plugin.
Also let me know if you'd like me to update this PR to swizzle in the method only (instead of swapping classes).

Yeah the reason I thought there might be more than just string matching was
because there were no reports (that I know of) of the other plugin, which
had the same string, causing rejections. But I suppose that could be
explained by the difference in usage numbers.
I'll admit to probably overthinking this. If apps were to start getting
rejected again (meaning they are detecting it with more than just string
matching) I think we would be justified in asking our users to open Radars
to prod Apple into exposing, at a minimum, inputAccessoryView for
UI/WKWebView.
If you don't mind updating the PR to only swizzle the method I'll merge it
in and release 2.2.0. Thanks!
On Tue, May 3, 2016 at 12:06 PM Tony Homer<EMAIL_ADDRESS>wrote:

I thought about just replacing the method, but wasn't sure how to do it.
Now that I see the example, it looks pretty straightforward! Personally, I
don't think it will be less of a red flag - I think the rejections are
based on string matching, but I like that it is simpler.
I see where you are coming from in wanting to shield users from possible
rejection.
Personally, I think avoiding strings that match non-public API and class
names will be sufficient to avoid rejections, but isolating this
functionality might be reassuring for some users.
Feel free to close this if you want to go with cjpearson's simpler
approach and/or want to pull this out into a separate plugin.
Also let me know if you'd like me to update this PR to swizzle in the
method only (instead of swapping classes).
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ionic-plugin-keyboard/pull/197#issuecomment-216597220

The one thing I don't like about this approach is the inclusion of the obfuscated class name strings.
I experimented with some other approaches, but they all had side-effects.
In the end I came back to this approach, including the obfuscated class name strings, but I tweaked it a bit to fix some potential issues and/or (in my view) clean it up.
Specifically, I moved the static IMP references to member variables (and added Method member variables in order to avoid looking them up multiple times) and I moved the initialization to pluginInitialize.  I wanted to avoid the possibility of overwriting the original IMP on multiple calls to hideKeyboardAccessoryBar.

Hi @tony-- and @tlancina, really sorry to be such a pain, but I just wanted to check if you can resolve the conflict and merge it with 2.2.0? I am only asking because it seems the fix is imminent, and I would love to get this back in to our app.

@jacquesdev Thanks for the comment!  I merged the recent changes from master into my PR branch to resolve the conflict.

@tony-- looks great! Could you remove the swizzled const? I think it's a relic of the first implementation. Or if it's a hassle I can merge it in right now and can remove it as well.
@jacquesdev for future reference you can install any plugin from github at any time: cordova plugin add https://github.com/tony--/ionic-plugin-keyboard.git#swizzle-it-just-a-little-bit.

@tlancina - right you are, sorry about that!  I removed it; also removed the commented out references to UIWebViewExtension from plugin.xml.

@tony-- thank you for your contribution and patience!

@tlancina glad to have an opportunity to help out a bit!

Thanks to both of you much appreciated :)


Include cdt_name to pickup CDT's matching OS

Ensure cdt_name is included to pull in CDTs matching the OS as discussed in PR ( https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/867 ).

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Update GPS Info App and "Battery Level Widget (with percentage)"

GPS Info App: Resolve one FIFO_FULL case and exit App with button press.
Battery Level Widget (with percentage): Increase screen update rate when charging - as requested by issue https://github.com/espruino/BangleApps/issues/1137

Thanks - I'm interested in the FIFO_FULL issue... Was it this code that you found helped?
  Bangle.setGPSPower(0, "app");
  Bangle.removeListener('GPS-raw', onGPSraw);
  Bangle.removeListener('GPS', onGPS);
  load();

I've have hoped that load() would have effectively done the removeListener already - although we do generally leave GPS on while switching apps so that if the next app needs GPS you don't have to wait for it to get a lock again.

@gfwilliams

I'm interested in the FIFO_FULL issue... Was it this code that you found helped?

This is what had helped, because GPS-raw spills out many characters within a second:
Bangle.removeListener('GPS-raw', onGPSraw);

I've have hoped that load() would have effectively done the removeListener already

Sure, but as I can't prove if a listener is still active, I've done it just for safety. I will remove the removeListener() functions with the next update.

although we do generally leave GPS on while switching apps so that if the next app needs GPS you don't have to wait for it to get a lock again.

Really? It draws a big current while active and I've found (with the help of "GPS Widget (v0.03) Tiny widget to show the power on/off status of the GPS") that the GPS power is switched off (may be in load()?) when you return to the clock - at least in my Bangle.js 2 . I've also done it for safety and will remove it with the next update.

Thanks!

I will remove the removeListener() functions with the next update.

Don't worry about it - it doesn't matter either way. It's just if doing that really helps then it's a sign that I may be able to do something in the firmware to help.

Really? I've found ... that the GPS power is switched off when you return to the clock

GPS is left on, then 0.5 sec after the the next app loads it checks to see if it's still needed or not and changes state accordingly. As far as the next app is concerned, if it didn't ask for it, GPS is off.
But if you do explicitly turn GPS off in an app after that initial 0.5 sec then that state change is immediate :)


Modify packet before it's sent

I need to modify a packet before it's sent (namely, I need to change the UUID of a LoginSuccessPacket). Since I'm using PacketLib, I have no control over what UUID is used during the sending of said packet, and would like to change it after session#send has been called, but before it's actually sent. Is there a way I can do this, or no? Thanks!

There isn't. Why would you need to do this?

I would like that too, people usually don't want to put their login & password to some "random" application or don't even have spare account to do so.. it would be great if it would be possible to implement something like mcleaks api...

I don't quite understand. If you're concerned about logging in through MCProtocolLib you can just disable authentication, although you won't be able to connect to online mode servers obviously.

@Mareckoo01 You can implement it your self. Get the username & password from the api and connect to the server using MCPL.

^

I haven't looked into how auth & login works but alot of people use and trust this library so it should be ok..
@Steveice10 im not, but if people ask me if i could share my bot with them and i decide to share it, they might not want to put their login & pw in it to login..
or maybe they have only 1 minecraft account and then they can't play on the same server as the bot because the bot is on their acc..
@phase this is what i would use, so i need to modify LoginStartPacket and EncryptionResponsePacket i think.
so instead of login & pw they can use mcl alt token.

I've fixed it by making my own class with the same canocial name as the class of the packet I was trying to modify, just tweaking a copypaste of the original code. But there should be a way to do this less hackily

I'm still not clear on what you're trying to do; if you're trying to login without a valid account, why not just disable authentication?

We are asking for a way to modify packets before they are sent(not by our code but by the library, i think @DaMatrix wants to modify server to client packet, i want to modify client to server packets) when i use
client.getSession().connect();
like this
@Override
public void packetSent(PacketSentEvent arg0) {
	if(arg0.getPacket() instanceof LoginStartPacket){ //Sent when client.getSession().connect(); (can't be modified)
		LoginStartPacket lsp = arg0.getPacket();
		lsp.setUsername("Nick"); // I want to change username
	}
	//Packet would be sent modified..
}

but i need to change not just packets but authentication too..
from mojang POST request authentication with username & password
to
mcl redeem - (user gets alt token from website) POST request to mcl with alt token
and
POST request to https://sessionserver.mojang.com/session/minecraft/join
to
POST request to http://auth.mcleaks.net/v1/joinserver
Quick links:
wiki.vg-
Protocol FAQ
Encryption Request
Authentication
mcl-
api
their forge mod(looking at source) (skip to step3)
TL;DR
im asking you to implement a way to edit packets before they are sent when i use client.getSession().connect();
im not asking you to implement mcl api or anything other
why? scroll up..

Yes, but as I keep saying, why do you need to modify packets to accomplish this when you can just disable authentication and give it whatever username you want to begin with?

@Steveice10 change skins and way more

so basically we want to modify packets before they are sent or before mcprotocollib does anything with received packets

...but once again, wouldn't changing the UUID in the login packets invalidate authentication?

No

nope, it doesn't invalidate it. check out https://github.com/DaMatrix/Pork2b2tBot , that's where i'm using it at. i want the client to show up as having a specific UUID and name, BUT for security purposes, we are still using mojang auth.

https://github.com/Steveice10/PacketLib/commit/e8a86dafcb99f4952c423ee9a2bf66de95c7c245
Let me know if you encounter any further issues.

Thanks!

looks great, thanks!

Thanks!
I had to add Mareckoo01/PacketLib@98eabb9 and now it works great!

Thanks for pointing that out, should be fixed now.

If anyone would be looking for mcl login implementation, here it is..
I didn't even need to edit packets.. well i'm not that good at coding because i do it just for fun..


first draft IMEX JinXin

I made a first dirty and "straightforward" draft, but as I anticipated it might have some bugs. I still didn't generalize to keep it simple until I'm sure it will work. I broke it down to the simplest way to make it work, but I couldn't find any major mistakes.

There was a missing sign on my implementation, now the code seems to run well. I'm gonna make further test, clean up everything and improve the implementation where is possible.

Could you please run the code formatter as described in https://trixi-framework.github.io/Trixi.jl/stable/styleguide/?

Could you please run the code formatter as described in https://trixi-framework.github.io/Trixi.jl/stable/styleguide/?

Done!

Looks like you used another version of the JuliaFormatter for formatting resulting in many unrelated changes, see also #1976. Can you rerun the formatter with the other version, please?


Fix: RENT Token

Accidentally submitted incorrect token ID, here's the fix.
Fix:
https://github.com/solana-labs/token-list/pull/24270/files
Original:
https://github.com/solana-labs/token-list/pull/23580

Hello?

Hello?


Icons have different font sizes using sidebar

I notice that when I was using different icons, in same areas, they've different size/ratio.
Do you know if that's normal (for instance user and settings) ?
It happened using sidebar component (left side, not top for instance)

Hi, please provide a test case as per our contributing guidelines; you can fork this JSFiddle.

@Banandrew Apparently it doesn't happen there with the last CSS version.
I noticed this issue with React 15 and the official React-Semantic-UI library
http://jsfiddle.net/1r5wvjr5/1/

@dbrrt Sorry for the delay. React integration is in a separate repository, it’s being maintained by different people, so I think it’d be best to open a thread there. I’ll close it for now, but please mention this issue in Semantic-Org/Semantic-UI-React in case you’ll find that it’s an error in SUI’s CSS, and someone will reopen it.

@Banandrew The guidelines told me to post it there :
✖ CSS ISSUES → Post on https://github.com/Semantic-Org/Semantic-UI
apparently not a CSS issue, more of an issue on integration side in React library.


Incorrect resizing of inline SVG (Material Design icons)

Version
2.6.11
Reproduction link
https://github.com/TokugawaTakesi/Issue-IccorrectSvgResizingInVue
Steps to reproduce
See README.md of repro.
What is expected?
SVG could be resized as in static HTML
What is actually happening?
SVG path resizing is incorrect.

I know about  "vue-material-design-icons" library, but I don't want to use it because:

I want to unify HTML and styles in static HTML version and VUE SPA version.
Resizing "vue-material-design-icons" is not elegant.
Although vue-material-design-icons" allows to resize, there are the problems with align.

Hello, this is way too big for a reproduction. When reporting a bug, only include what is necessary, avoiding uncesseray things like sass, pug and typescript (unless it's a typing error) as well as extra code and project files (IntelliJ).
I tried to find the problem but there are many layers that are unrelated to Vue... I don't think this is related to Vue, but rather to the layers making a difference in the way some of the attributes are being passed to Vue
If you manage to create a boiled down reproduction, please ping me or open a new issue

@posva Hello,
I removed all unnecessary files as simplified my code as you said.
Please check the updated issue repository:
https://github.com/TokugawaTakesi/Issue-IccorrectSvgResizingInVue

@TokugawaTakesi
-<svg viewbox="0 0 24 24" class="CheckmarkIcon">
+<svg viewBox="0 0 24 24" class="CheckmarkIcon">

Please note that viewBox is case sensitive.

@Justineo
It was the cause!
First, thank you for the solution.
So, although HTML attributes names in case-insensitive, inline SVG  is not obeys the rules, right?

Yes SVG has its own rules.

@Justineo
O'K got it.
Thank you for the investigations and sorry for the false issue.


Reactivity / hydratation issue on render with reactive object

Environment
Nuxi 3.3.3

RootDir: /project/home/benjaminrobinet/workspace
Nuxt project info:

------------------------------
- Operating System: `Linux`
- Node Version:     `v16.17.0`
- Nuxt Version:     `3.3.3`
- Nitro Version:    `2.3.2`
- Package Manager<EMAIL_ADDRESS>- Builder:          `vite`
- User Config:      `-`
- Runtime Modules:  `-`
- Build Modules:    `-`
------------------------------

Reproduction
https://codesandbox.io/p/sandbox/reactivity-issue-dlxt09
The page index.vue is mutating the reactive object, and the component StateComponent.vue is displaying the value and using it to apply a CSS class using a computed.
Describe the bug
Here we have a useState method that is mutated to another value on the setup of a page. The template reacts well when using {{ stateValue }}, but not in computed, template binding, etc...
I added a button to mutate the state on a nextTick, and it works as expected. That's probably not a useState issue, as I have the same issue with Pinia.
EDIT: On HMR, it works fine.
The expected behavior is that everything should be correctly reactive to that reactive object.
Additional context
No response
Logs
No response

Same here 👋

The issue here is that on the server, order matters. Components are rendered from the top down, and there is no reactivity. In this case you are setting the data in your page, but by this point it's too late. The component above it has already rendered. So on the client side we have a mismatch, because the data is present now for the header.

@danielroe I see, any workaround for that ? In my specific case, I can just use definePageMeta, but no other way to have it reactive than duplicating it for client only ?

It's definitely reactive on client-side - it's just that on the server you need to have it set to the correct value before the first component depending on it is rendered. So setting it in a plugin or middleware would likely be your best option.

@danielroe oh yes make sense... sorry ! thanks for your time then!


Hound gracefully shutting down at startup

I have deployed Hound in my K8S cluster to index my gitlab instance.
Container is running all all index are built successfully, however hound is immediately shutting down whith no reason.
2023/06/26 13:32:02 All indexes built!
2023/06/26 13:32:02 running server at http://localhost:6080
2023/06/26 13:32:02 Graceful shutdown requested..

These are all the logs I have.

I have the same issue as well. I've not figured out what the issue is yet

Hi,
it is fixed on my side.
I have deployed Hound on K8S and I had to extend the startup/liveness probes to give some time at startup.

That would make sense.
If you don't mind me asking @MohamedTalhaoui - How are you dealing with SSH keys?
We've mounted some private keys in to the /root/.ssh/id_rsa but then we have issues with known_hosts being write only.
Interested to see your approach to this

Relevant part of my deployment yaml file below:
...       containers:           env:           - name: GIT_SSH_COMMAND             value: "ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i /etc/ssh-key/id_rsa " ...           volumeMounts:             - name: ssh-key               mountPath: /etc/ssh-key               readOnly: true ...       volumes:       - name: ssh-key         secret:           secretName: hound-ssh-key                defaultMode: 0600 ...
Hope that helps


Global notification banner

There needs to be a banner that is accessible and viewed globally within the staking app to show messages to the user.
This will support

Vote notifications (telling a user to vote, i.e there's a proposal that needs to be voted on)
Liquidation warning (when a user has gone at or below 200% c-ratio)

Proposed UI for global notification banners:

Designs found here:
https://www.figma.com/file/M58E2cxkOgTPWMHRX7FTna/Staking-User-Flows?node-id=1873%3A0


docs(project): added angular (ngx-linkifyjs) documentation for linkifyjs

As discussed in this thread #241, I have written a detailed documentation for angular ts version (ngx-linkifyjs) of linkifyjs

I am not able to run jekyll on my machine! Im getting errors like

Configuration file: /Users/anthonnahas/git/linkifyjs/_config.yml
  Dependency Error: Yikes! It looks like you don't have jekyll-sitemap or one of its dependencies installed. In order to use Jekyll as currently configured, you'll need to install this gem. The full error message from Ruby is: 'cannot load such file -- jekyll-sitemap' If you run into trouble, you can find helpful resources at https://jekyllrb.com/help/!


Ground plane

Hi there again!
First off not a major issue, so let me start with this:
What a fantastic kit! Super well packaged, labelled and straight forward. My pico-56 is working and I'm amped to get into playing with it. I was lucky enough to to have a mate visiting from Oz that could bring it over for me.
While I was soldering up the board, I had some issues with components connected to the ground plane. I have a good temp controlled iron with a medium chisel tip and good thermal mass, that I usually run at 330C. This wasn't sufficient to create a good joint on certain components e.g. some caps, resistors, a transistor leg, pins on some connectors etc. I ended up having to boost the temp up to about 390-400C with uncomfortably long dwell times.
If I'm the only one with this issue, then tough for me, but I would suggest revising the PCB so that the these components are connected via traces through an island to the ground plane to make assembly easier.
Thanks for the awesome project.

Thanks Peter.
You are the second person to mention this. The other was one of the giveaway winners.
I run a TS-100 with the BC2 tip at 375C. I haven't had the same issues having assembled ~25 of these, but certainly can see that it could be a problem.
If I make another revision of the board, I will certainly look to reduce the ground plane to pad connection thickness.

Thank you, I'll close this.


Pendo

pendo API support
testing key
84e57b87-0c59-486f-6289-2a517d39b078.us

This one has some bad commit history, you should probably squash it after merging https://github.com/SpectralOps/keyscope/pull/31


Add vaadin-router-meta custom element class

Related to #32, as the vaadin-usage-statistics assumes the Custom Element definition and contains a whitelist here.
Related to #146 as a possible option to expose the API globally but not creating any global variables.

This change is

Reviewed 3 of 3 files at r1.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable


Rt 11724 is it really a bug ?

Hello again.
I have the felling it is not really a bug.
In fact $xp->find('/xml/tag') call return an 'XML::XPath::NodeSet' object.
So in order to test it we have to invoke "string_value()" method.
I added "99rt_11724.t" in t directory.
Can you take a look to say me if you agree with my understanding about this RT ?
Regards
sdeseille

Hi,
You are going in the right direction, well done :-)
As the ticket RT# 11724 says the package XML::XPath::NodeSet is not overloading "eq", "ne" etc. So, for example, in order to enable the "eq" overloading, you could just change one line as below:
diff --git a/lib/XML/XPath/NodeSet.pm b/lib/XML/XPath/NodeSet.pm
index 7a86d52..8520e3e 100644
--- a/lib/XML/XPath/NodeSet.pm
+++ b/lib/XML/XPath/NodeSet.pm
@@ -8,6 +8,7 @@ use XML::XPath::Boolean;
use overload
'""' => &to_literal,

           'eq' => \&string_value,
           'bool' => \&to_boolean,
   ;

Here is the test script to verify the above change.
use Test::More;
use XML::XPath;
my $sample = <<EOS;

FOO

EOS
my $xp = XML::XPath->new(xml=>$sample);
my $nodelist = $xp->find('/xml/tag');
ok($nodelist eq 'FOO');
done_testing();
Best Regards,
Mohammad S Anwar

Hello
I added all string and number operators.
I added a new testfile dedicated to number comparison operators.
regards
sdeseille

Thanks.


Plugin doesn't work on fixed mode

When playing in fixed mode the interface is not on screen:

I've successfully fixed it. I swapped it to the other side by adding 630 pixels to the X axis. It's "ugly" but works exactly as expected.
ShatteredRelicsFragmentPresetsSidebarOverlayPanel.java
ShatteredRelicsFragmentPresetsSidebarOverlayPanel.txt
shattered_relics_fragment_presetsb.zip

fixed in #4


Form validation to check that first and last name are not longer than 20 characters each

Needed to support https://github.com/all3dp/printing-engine/issues/408

We also need to check if (user.shippingAddress.street user.shippingAddress.houseNumber).length <= 80 because sculpteo order will fails othewise.


build(PC): move modules back off the classpath so the sandbox works

Changing facade's dependencies so all the modules still get their dependencies without being on the classpath.
Adds a new moduleCache directory that automatically-resolved jars get put in, kept separate from the modules/ source directory.
Fixes #4375.
How to test
Brief description of how to test / confirm this PR before merging
Outstanding before merging

[x] merge parent #4347. :information_source:  Until then, this diff might help: https://github.com/MovingBlocks/Terasology/compare/chore/tidy-dist-tasks...fix/sandboxed-module-path
[ ] re-integrate serverModules
[ ] build module cache as an output of :modules project instead of defining the task in :facades:PC
[ ] Still have to adjust the wiki doc

Current status: The provideModuleDependencies task is a good proof of concept of how to do the jar-gathering thing without the DependencyResolutionListener hooks. Works nicely under gradle. IntelliJ doesn't realize it's supposed to build the modules at all.
It might be worth looking up how IntelliJ does make its dependency graph out of the gradle projects. I'm assuming it's only checking the main runtimeClasspath or something like that, so when we put the module dependencies in a non-main gradle Configuration it doesn't take them into account.
I guess we can probably come at this from two directions: the gradle side or the IntelliJ side.
From the IntelliJ side, we could edit the run configs so they have to do some additional build-the-modules thing before they start.
From the gradle side, we could include the modules in main runtimeClasspath, but then have the exec tasks use some other classpath without the modules?
I spent a while rummaging around in the intellij-gradle sources. I am far from understanding how it works, but this test suite is informative: https://github.com/JetBrains/intellij-community/blob/211.5361/plugins/gradle/testSources/org/jetbrains/plugins/gradle/importing/GradleDependenciesImportingTest.java
The tests include non-default Configurations and SourceSets, so maybe we don't need to put everything in main after all. In which case it's a matter of how we declare the dependency on that moduleCache, or it's our use of platform. (I don't see any test cases using platform there.)

Going to also link this to #4367 for relevance, since I guess if we have a different module directory that might be a good place to direct the in-game module downloader to in some fashion when running from source, especially if it can sort out dependencies (am realizing I never tried using the in-game downloader to grab something with dependencies - no idea what would happen then)
As for testing - ran this branch with gradlew distForLauncher in a workspace without modules, was then able to run the distributed game fine, very smooth - even if that just tests the dists overall. Added all Omega modules as jars and game still ran, although naturally something crashed because JOML ;-)
Maybe we could get TutorialMinimalEngineDemo some quick love in the near term, only two errors or so in there, and it is my quick goto when moduleland is broken. But me trying to bumble through a fix in the narrow little window I found tonight didn't really go anywhere good. So actually testing from source or breaking the sandbox I haven't been able to do just yet.
(Not that TMED would let me test that without hacking in a simple violation in its module - even Sample needs some sort of world and an inventory to be able to retrieve and trigger the crash blocks ...)

I merged #4347, which makes the diff of this vs develop a more manageable size.

There have been some small changes since Cerv's last review, but I'm confident enough in them to move ahead without a re-review. I'll at least wait for CI to finish. It would be way too embarrassing if one of my "small" changes broke that.
Test Plan

groovyw get CoreSampleGameplay Explosives Sample # note this is not recurse
./gradlew clean extractConfig extractNatives
launch game. One of

for testing gradle:

./gradlew game

for testing IntelliJ:

Click the 🔄 Reload All Gradle Projects button in the Gradle Tool Window. If you don't do this (or restart IntelliJ) after checking out new things to modules/, it might not have recognized them.
Run TerasologyPC configuration

assert game starts
assert Advanced Game Setup screen shows CoreSampleGameplay and Sample modules and all their dependencies, and they are not error-color-red.
activate CoreSampleGameplay and Sample, start a new game
get the gun from Explosives (it's in the second chest CoreSampleGameplay puts in the inventory of new players), make sure it has its shoot-and-destroy-block effects. (There's a possible failure condition where the module loads assets but not code, in which case you can have a gun that doesn't shoot.)
give ReallyCrashGameBlock 1 in the terasology console
place the ReallyCrashGameBlock
assert the game really crashes when you Use it.

Troubleshooting
If things don't go as expected, a few things to check are:

did Sample and Explosives actually build their code?

fdfind -Ig '*.class' modules/

did the modules cache get filled?

ls cachedModules  # you should see .jar files for dependencies here, like CoreRendering and Inventory. There should not be jars for modules we have source for in modules/, or for non-terasology-module things.

For IntelliJ: look in Project Structure / Modules / facades/PC/main / Dependencies and see if our source modules are included in that list, e.g. Terasology.modules.Explosives.main.


fix: idp changes

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

GitHub release
v0.95.5

Your semantic-release bot :package::rocket:


Add more docs based on proposed wishlist

Some of the items originally discussed here:
https://groups.google.com/forum/#!topic/druid-development/skWOuYETk7g

:+1:

+1


Describe and write down the CI policy

Describe the CI policy for jobs, like:

having contact information
resources defined

We have a policy for blocking and informing jobs, https://github.com/kubernetes/sig-release/blob/master/release-blocking-jobs.md .
If we compare this policy with what is proposed in https://github.com/kubernetes/test-infra/issues/18599, what would we add? what would we change?
We should evaluate what changes we need to make to help ensure we are acting on useful information and check tht CI jobs are maintained and well.
/area release-eng
/area ci
/kind documentation
/priority important-soon
/milestone v1.20

/sig testing


my contribution

I recently discovered your repository, and it was really helpful for me when starting my competitive programming journey. So, I thought of contributing to this repository.

Done


How can I get the names of the stages of a built stagerunner?

Suppose sr is you stagerunner. Then, you can do the following
stages <- sr$stage_names()
# get the names of all import-related stages
stages[which(grepl("import/", stages))]

names(sr$stages)

names(syberia_project(syberia_root())$resource(paste0("models/", model_string))$value()$stages)
[1] "import" "data"   "model"  "export"

Why not just $value(parse.=FALSE) and then you don't need to reach into the runner


Need to add support for latched/impulse physical events (initial pressure in collisions)

Need a way to capture the initial impulse event only of ie a player stepping on a pressure plate, instead of firing thousands of times
Re: EntityCollideBlock event
Posted per blood's request.
Note, would it make sense to think of a Transducer Input Event to capture things like vanilla plates, levers, tripwires and other on/off devices - capturing the toggle state of those toggleable mechanics (and allowing mods to register such on/off input devices to trigger as well) without having to capture the high-frequency notify-neighbor event and fish out the toggle input from the chain...?

I reduced the amount of CollideEntityEvent calls by ignoring AIR blocks. It seems to perform much better now and should be good enough for what you need. I also looked into how Bukkit handled this and it was the same.

Hey friend,
I've got some interesting  news for  you,  you're gonna be  pleaseantly  surprised. Read more  here   http://eight.eatsantiagos.com/e4fljijo
Yours faithfully, boomod


HOME env var may be 'slashed', so not set to writable dir

Resolves #430
Have actually done a full run through by pushing a tag to this commit.
Actions work off the current commit (rather than main - forgot that I could test that way)

[x] Resolve other ICAV2 env vars that are needed by the cwl-ica-cli docker image - https://github.com/umccr/cwl-ica-cli/pull/205 / https://github.com/umccr/cwl-ica-cli/releases/tag/v1.6.3

Now working with - https://github.com/umccr/cwl-ica/actions/runs/6634123390


Operation validator: add request validation from URL only

Is your feature request related to a problem? Please describe.
I would like to validate a request/response by path rather than a known operation id. To validate requests as they arrive to an end-point the only information we have is the path, headers and method. We don't know what the "operation" is. We can do this with code similar to:
Path path = api.getPath(request.getPath());
Operation op = path.getOperation(request.getMethod());
OperationValidator operationValidator = new OperationValidator(ctx, api, path, op);
ValidationResults vr = new ValidationResults();
operationValidator.validateBody(request, vr);

However this only works for paths with no parameters.  I would like to find paths such as "/pet/{petId}".
Describe the solution you'd like
I believe the "api.getPath" could take an overload which converts all paths with "{xxx}" to regexes similar to: "/pet/.*/" and find a match using this instead.
It would be nice if it also took the 'servers:url' component into account as well. Perhaps an api.findPath(url) that correctly finds the path by skipping over the server part and regexing through the parameters parts.
Describe alternatives you've considered
I can write my own iteration over the map of paths but it would be nice to have this part of the API. I suspect many people would like to validate using the path+method/response code rather than a known operation id.

Hi and welcome,
You're right, this feature was on my roadmap before 1.0.
I thought about making this lookup in RequestValidator via a new validate method:
RequestParameters validate(final Request request) throws ValidationException.
This method would look at the URL of the request (to add) to make the successive matches:

server path parameters variables if any (we can't arbitrarily skip this) to detect the "prefixing" fragments
the path of the operation from the remaining path (after removal of server "prefixing" fragments).

In this way, I expect to keep only one OperationValidator per Operation, the laziness as behaviour and avoid performance downgrade.
Also, we must honor server variable parameter which can be open or close with enum.
By the way, is the sample code written for illustration or are you really using directly OperationValidator instead of RequestValidator?

Hi, and thanks for your project.
I noticed there is a 'PathResolver' class that may be useful for doing this matching.
I am using org.restlet and have an object per route. This object knows the templated path and method, thus I create and store an Operation per end-point. I am yet to deal with validating responses from the requests.
I am using OperationValidator because;
a. I know the path up front, so don't need to create a new one per request.
b. RequestValidator has no way to return the validation results if there is no error.
I need the validation results for 'info' messages. I need this because I am using a custom 'format' validator to collect all the crumbs to the nodes with "format: datetime".  I then use this collection to perform string to date conversion on the data. This allows generic date conversion of Json payloads by simply having a openapi spec defined.
Along these lines I do have a couple other feature requests;

Register a custom 'converter' that converts the data in addition to validating it. This is particularly important for types with a 'format'. But may also be needed for converting values to 'reals' when the Json data only has an integer value for example.
Allow processing Map<String,Object> data without first converting it to JsonNode.

I have workarounds for the above, but it might be nice to see it in-built.

Hi,
I merged all this in master branch, now we can lookup to operation from request URL only.
This is done from RequestValidator not OperationValidator. This last needs all the information to perform the subsequent validations.
Now to talk about your needs, the naming could be misleading but you should use a shared RequestValidator. It stores a OperationValidator collection and caches them for reusability.
Before reading further, it would be nice if you can validate the changes with latest 0.8-SNAPSHOT.
Let me know...
That said, if I understand well what you're trying to achieve, you should use directly openapi-schema-validator to validate the body saving you of re-contructing all this.
This would be like the following:
    // Setup this block once
    ////////////////////////////////
    // coming from parser or not, you're choice.
    JsonNode schemaNode = operation.getRequestBody().getContentMediaType("application/json").getSchema();
    // you can also setup directly the schema from URL
    JsonNode schemaNode = TreeUtil.load(getResource("my-schema.yaml"));

    OAI3Context apiContext = new OAI3Context(URI.create("/"), schemaNode);
    ValidationContext<OAI3> validationContext = new ValidationContext<>(apiContext);
    SchemaValidator schemaValidator = new SchemaValidator(validationContext, "schema", schemaNode);
    ////////////////////////////////

    // On each request do the following
    ////////////////////////////////
    ValidationResults results = new ValidationResults();
    schemaValidator.validate(YOUR_BODY_JsonNode, results);
    ////////////////////////////////

About registering a custom validator, which will be available for openapi-schema-validator resolution proposal, you can do this already with business (or override) validator: https://github.com/openapi4j/openapi4j/tree/master/openapi-schema-validator#business-validator
You can do every you need with, I just trigger it.
I hope these lines are helpful to you.

I tried out the changes and it seemed to work sometimes, but was matching paths inconsistently. I had paths of:
path: /v1/abc
{
   "post"
}
path: /v1/abc/{Id}/{Age}
{
  "get"
}

And a GET request of "/v1/abc/123/123" would occasionally match the path of "/v1/abc" instead, throwing an error regarding no "get" operation.
Perhaps you don't wish to support it, but it may also be useful to match templated strings to templated strings where the named params differ. e.g
"/v1/abc/{p1}/{p2}" would match "/v1/abc/{Id}/{Age}"
This is useful for routers that are trying to match end points to the spec up front prior to receiving a real request. I'm unclear how this might work if the types of the params change, e.g. integers and string params.
Thanks for your tips on using SchemaValidator.  I create a OperationValidator per end point and re-use them for each request. But unfortunately I am working with a Map<String,Object> rather than JsonNode, so the conversion needs to occur anyway. If openapi4j could work with native Map<String,Object> that would be a great feature, for me at least.
I am using a validator to override the "format" handling. However there doesn't appear to be anything I can do other than record a validation error. I am using this to record all the points "format" occurs then going back through the data and performing conversion.
I was suggesting openapi4j could allow conversion of the data via a validator/converter. However for my case that would require supporting validating the original Map<String,Object> rather than a JsonNode copy of the data.

Thanks for the report. I can confirm the issue you noticed. I'll make further internal changes to be fully accurate here.
Matching templated strings to templated strings will never be possible here. I need named parameters fit to the path or/and operation parameters.
Also, you can already wrap your body with public static Body from(Map<String, Object> body). The conversion will occur internally.
Finally, for your feature validator/converter, are you available to contribute to it? At least as an experiment?

PR #63 should make this lookup accurate now.
To make this, the last changes handle now all servers definitions (absolute, relative and none) to match exact path template.
Can you confirm the issue is fixed in latest 0.8-SNAPSHOT?

I will try find time to confirm the changes soon and will get back to you.

Matching templated strings to templated strings will never be possible here. I need named parameters fit to the path or/and operation parameters

This is fine. One could always just create a URL with dummy values to do the match if needed.

Also, you can already wrap your body with public static Body from(Map<String, Object> body). The conversion will occur internally.

Yes this is what I am doing. However there is a performance/memory cost associated with doing this. It also makes a copy of the data, so mutation during validation is not possible (but perhaps that's not a good idea anyway).
I am also unsure the conversion is very accurate. It seems to use the schema to do the conversion but has no support for discriminators/anyof etc. So will not convert correctly. And if it did read the schema correctly it would essentially be doing that schema processing twice (see https://github.com/openapi4j/openapi4j/blob/bc7d6134a501f36a28e51e0a0b95ee49eff0a798/openapi-operation-validator/src/main/java/org/openapi4j/operation/validator/validation/BodyValidator.java#L51).
Is there any reason you cannot simply use JsonGenerator/JsonFactory/Objectmapper etc? Java objects already encode the type information needed to convert to Json, there's no need to use the schema.
If openapi4j were to support Map<String,Object> natively, I think it might require using generics. The conversion helper I was suggesting would probably best be left as a post validation step, to avoid mutation during validation. In hindsight the way I am doing it is probably close to optimal - I just wish I could avoid the unnecessary copies of the data.

It also makes a copy of the data so mutation during validation is not possible

Half incorrect. Object reference is passed by value, there's no deep copy around there.

(but perhaps that's not a good idea anyway).

True :)
It does not convert the content twice as you suggest, getContentAsNode will return JsonNode directly if this is what is wrapped. Schema parameter is mostly needed for un-structured content types (i.e. form, ...) to construct the common form of readable content.
Let me know if the issue still persist on your side.
Thanks.

ok thanks for the confirmation. However I can see there is significant difference between passing a Map<String,Object> vs a JsonNode. If a value does not match the schema it gets converted incorrectly, thus validated incorrectly.
e.g

numbers that should be strings get converted to strings by 'getContentAsNode' and it validates fine.
passing strings that should be numbers get converted to 'null' and you get "Null value is not allowed." errors instead of "Type expected 'number', found 'string'. errors.

However converting to JsonNode manually prior to validation works as expected:
            ObjectMapper OBJECT_MAPPER = new ObjectMapper();
            JsonNode jsonNodeMap = OBJECT_MAPPER.convertValue(objMap, JsonNode.class);
            builder.body(Body.from(jsonNodeMap));

I will raise a separate issue for this.

I tested the changes out and they seem to work ok.
It might be nice to expose the "buildPathPatterns" and "findPath" methods publically as they are useful utility methods. It could perhaps live on the "OpenApi3" class instead of RequestValidator.
But otherwise the patch matching seems to be working ok for simple test cases.

Thanks for the feedback.
Both methods are already available from PathResolver.


K8SPXC-898 - Fix scheduled-backup test by decreasing number of backups

Test name
Status

upgrade-haproxy
passed

scheduled-backup
passed

storage
passed

recreate
passed

scaling
passed

big-data
passed

cross-site
passed

haproxy
passed

scaling-proxysql
passed

self-healing
passed

self-healing-chaos
failed

restore-to-encrypted-cluster
passed

security-context
passed

upgrade-proxysql
passed

self-healing-advanced
passed

self-healing-advanced-chaos
failed

init-deploy
failed

demand-backup
passed

smart-update
passed

operator-self-healing
passed

operator-self-healing-chaos
failed

limits
passed

monitoring-2-0
passed

pitr
failed

affinity
passed

one-pod
passed

upgrade-consistency
failed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

demand-backup-encrypted-with-tls
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1203/commits/14aa5bba110938b0061bdf4ea81cee1ee827883c
image: perconalab/percona-xtradb-cluster-operator:PR-1203-14aa5bba

Test name
Status

storage
passed

big-data
passed

self-healing
passed

scheduled-backup
passed

self-healing-chaos
failed

scaling
passed

cross-site
passed

upgrade-haproxy
passed

recreate
passed

haproxy
passed

scaling-proxysql
passed

restore-to-encrypted-cluster
passed

init-deploy
failed

security-context
passed

demand-backup
passed

pitr
failed

upgrade-proxysql
passed

smart-update
passed

upgrade-consistency
passed

self-healing-advanced
passed

self-healing-advanced-chaos
failed

limits
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

operator-self-healing
passed

operator-self-healing-chaos
failed

demand-backup-encrypted-with-tls
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1203/commits/14aa5bba110938b0061bdf4ea81cee1ee827883c
image: perconalab/percona-xtradb-cluster-operator:PR-1203-14aa5bba

Test name
Status

scaling
passed

scheduled-backup
passed

recreate
passed

big-data
passed

storage
passed

cross-site
passed

upgrade-haproxy
passed

haproxy
passed

scaling-proxysql
passed

init-deploy
passed

restore-to-encrypted-cluster
passed

security-context
passed

self-healing
passed

self-healing-chaos
passed

limits
passed

upgrade-proxysql
passed

self-healing-advanced
passed

self-healing-advanced-chaos
passed

demand-backup
passed

monitoring-2-0
passed

affinity
passed

operator-self-healing
passed

smart-update
passed

operator-self-healing-chaos
passed

one-pod
passed

auto-tuning
passed

pitr
failed

proxysql-sidecar-res-limits
passed

users
failed

upgrade-consistency
passed

demand-backup-encrypted-with-tls
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1203/commits/7107f16435b2e357026763ed5913169546391a6b
image: perconalab/percona-xtradb-cluster-operator:PR-1203-7107f164

Test name
Status

recreate
passed

restore-to-encrypted-cluster
passed

demand-backup
passed

storage
passed

big-data
passed

pitr
passed

self-healing
passed

scaling
passed

scheduled-backup
passed

upgrade-haproxy
passed

haproxy
passed

cross-site
passed

self-healing-chaos
passed

scaling-proxysql
passed

init-deploy
passed

self-healing-advanced
passed

security-context
passed

limits
passed

self-healing-advanced-chaos
passed

monitoring-2-0
passed

operator-self-healing
passed

affinity
passed

operator-self-healing-chaos
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

users
passed

upgrade-proxysql
passed

smart-update
passed

upgrade-consistency
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

demand-backup-encrypted-with-tls
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1203/commits/7107f16435b2e357026763ed5913169546391a6b
image: perconalab/percona-xtradb-cluster-operator:PR-1203-7107f164


Add selectable line numbers

Registry component looks for "#LX-LY" or "#LX" in location.hash,
selects code from line X to line Y and then scrolls to line X's position.
Selected code has lighter or darker background-color
(custom hljs-selection class added to each theme) according to
atom-one-dark-ui[1] or atom-one-light-ui[2] style declarations. The themes'
@base-background-color is based on the one in react-syntax-highlighter styles[3][4].
For example in the dark theme CodeBlock's background-color is #282c34
so lighten(@base-background-color, 8%) = lighten(#282c34, 8%) => #3a404b
Added line numbers for increased readability.
Edited CodeBlock component to use react-syntax-highlighter's
SyntaxHighlighterProps instead of custom Props object.
Fixes #43.
[1] https://github.com/atom/one-dark-ui/blob/master/styles/ui-variables.less#L32
[2] https://github.com/atom/one-light-ui/blob/master/styles/ui-variables.less#L32
[3] https://github.com/conorhastings/react-syntax-highlighter/blob/master/src/styles/hljs/atom-one-dark.js#L7
[4] https://github.com/conorhastings/react-syntax-highlighter/blob/master/src/styles/hljs/atom-one-light.js#L7

Examples:

no line specified - https://deploy-preview-126--denoland.netlify.com/std/fs/copy.ts
one line specified - https://deploy-preview-126--denoland.netlify.com/std/fs/copy.ts#L128
range specified - https://deploy-preview-126--denoland.netlify.com/std/fs/copy.ts#L159-L185


How to switch automatically when server is down

Hi team,
How wiremock automatically switch from main server to proxy when main server down?
I have created test case on Rest APi.  Created Rest api on my local server.  Normally run the test case And ran the test senarioies. Now I am using service virtulazation using wiremock. It's good experience but i have some concerns

I have created test case for rest api.
2.then use stand alone server
3.then start stand alone server using  -- proxy with default server
4.it created mapping files in __file and mapping folder
5.then I stop my main server and then use that proxy and then run properly
But my concerns are below please check
1.how can i create mapping files automatically using wirmock open source library with java (note : find solution to create mapping automatically using wiremock server and config port and then record and save in mapping and file folder )
2.i want cotinus recording and switch automatically when main server is down how can i use point #1 to compare old recording with new,  if new recording is dmiss match with old one then I want to update mapping file automatically.

Please assist me and I am very thank full if giving me proper direction.  If possible if any one have small demo on this automatic switch please share
Thank in advance
Swapnil

Please post questions to the mailing list

Thanks,
but I am not getting any answer from mailing list,That why I have created.I will do that. Thanks for your suggestion.

Reposting on here won't get you an answer any faster. It just clogs up my inbox.


[Alignment] attribute valign for inky not working

Example style="vertical-align: middle;"  - works well, but valign="middle"  not working

I also stumbled across this. This example from the documentation does not work: https://get.foundation/emails/docs/alignment.html#vertical-alignment

I can confirm this bug on my end as well. Hopefully this project gets some attention soon!

I confirm, bug reproduced. Was downgrade to version 2.2.1 of foundation-emails package and it's working again

It's because in 2.3.0 version th is also set to top https://github.com/foundation/foundation-emails/blob/v2.3.0/scss/components/_normalize.scss#L80
I fixed it for myself by
[valign="top"] {
  vertical-align: top;
}

[valign="center"] {
  vertical-align: center;
}

[valign="bottom"] {
  vertical-align: bottom;
}


Does the library guarantee that the coords will be picked from the input points and not cloned?

Hi,
From knowing the theoretical algorithm, I guess that the returned values are "picked" from the original array of points (each is a "Javascript Object" - an array of two floats). I don't see why a future implementation would want to clone these arrays.
Is this guaranteed to be the same way in the future? If so, please mention this in the README.
If not, will it be guaranteed that the floating-point values will be copied and not "calculated"? i.e. that the convex result will not have points that are "very close" to some of the original points, but not equal (due to floating-point errors).
Thanks

the returned values are "picked" from the original array of points
Is this guaranteed to be the same way in the future?

Yes.


Peltool: Enhance error logging

Ensuring that error messages are directed appropriately.
Error logs are sent to standard error, improving visibility and
separation from standard output.
Signed-off-by: Harsh Agarwal<EMAIL_ADDRESS>

Minor request - the commit message one liner is a bit vague, can make it more exact by saying that it's writing errors to standard error.  Otherwise, looks good to me and it works well with your other PR.


Cursor ignoring zindex

Environment
react-native 0.64.0:
npm ls react-native-macos: 0.64.13
node: 17.0.1
npm: 8.1.0
yarn: N/A
xcodebuild: 13A1030d

Steps to reproduce the bug
If you have a popup covering a TextInput, if you hover over the overlayed TextInput it will show the text cursor.
Expected Behavior
Cursor should be based on top layer.
Actual Behavior
Text cursor shows for when hovering over a TextInput underneath the popup.
Reproducible Demo
https://github.com/jbcullis/test
Additional context
Doesn't matter about opacity, this also occurs when full opacity: 1 views are covering the TextInput.
Further, clicking a button that sits over the top of a TextInput will focus the TextInput.
Temporary workaround is to set editable to false on the TextInput when the UI container the TextInput is in the background.

Could you please attach some sample code we can repro this with?

@HeyImChris  I've created a demo with an absolute opaque view overlaying a TextInput and you'll see the cursor will change when you hover over the TextInput, and you can select and type into it.
It seems like that the TextInput control is always on top in terms of zIndex.

@jbcullis How are you displaying the popover in your test app? I cloned it and right clicked to get a contextual menu to show over the TextInput control and the cursor is the correct type in that scenario even when the menu is over a TextInput. Contextual Menu != Popover though so want to make sure I'm testing this with the same controls as you.

@HeyImChris the issue I’m having is that I can still select a text input even when there is a view overlayed over the top of it.
Same issue with a scroll view, if you create a full screen view over the top of a scroll view, you can still scroll.

Thanks, I'm able to repro your issue. Could you try testing it on an iOS app just with a simple text input and View over top of it? I'm curious if this is a macOS bug or just a generic RCTView issue.

@HeyImChris issue seems limited to macOS. It's fine on iOS and Android. We are still working on the Windows port but looks like it's fine their too.

@HeyImChris I don't have any experience with core RN but I'm hoping to learn so I started poking around over the weekend to try an identify the cause. Could it be an issue with catalyst cause I've seen some weird cursor behavior in the iOS simulator before.
This issue is the most pressing for us as it's causing users to think they can't press buttons if there is a textinput below the overlayed view.

Any update on the issue? Tested on latest macOs version it is still coming. Any workaround can be done for short period ?

Workaround is to set an active window flag and on textbox set editable to the active window status.
It's a pretty dirty hack and adds cruft to the codebase but it works.

@HeyImChris We updated to .68 and the issue remains.

@Saadnajmi it's a global variable we set for active window, then any screen that doesn't match active window disables all input text elements.
It's not part of the element, it's something we came up with as a work around for macos. You still get the wrong cursor if you're hovering over a text input at a lower z index but it at least prevents those elements grabbing focus.

@jbcullis thanks for the info. We have some fairly substantial changes to both Text and Textinput on our main branch, but they haven't made it to our next stable release yet (planned to be 0.71, no eta just yet when we can get it out yet though). We can revisit this at our next release and see if the bug still persists. I'm glad you found a workaround =) albeit a hacky one

@Saadnajmi  It seems that zIndex is not valid for all mouseover events.  Any mouseover event on a lower level window element can be triggered on a higher level window.  This causes a lot of problems in most single-window applications.  This problem still exists in version 0.73.0.

@Saadnajmi Yep, still a problem. Mac is only 1.9% of our install base on the main app so not the biggest issue but if we went web with our latest app because of this issue.

https://github.com/microsoft/react-native-macos/assets/61616262/ef701f5e-c8a7-4927-bfb3-700643fcbe7b
I was wondering if there has been any progress on this issue. Or does anyone know what the cause of this problem is? What should be done to fix it. I am currently using version 0.72.* and still have this problem. If there are no plans to fix it in the near future, I may have to make some compromises to my project interactions.

@bear-ei 看看你的视频，是否有一个视图延伸到整个窗口的宽度，或者只是抽屉出来的地方？

I can confirm that there is a top-level View covering the entire width and height of the window, and the drawer pops up within this View. I have set the zIndex to the maximum, and only when encountering onHoverIn and onHoverOut events do they penetrate to the underlying elements. This is one of the reasons causing the incorrect cursor display. However, events like onPress and onPressOut do not penetrate to the underlying elements, indicating that the zIndex is taking effect. I lack knowledge of MacOS native UI components, so I cannot further investigate the issue.

@bear-ei Yes, onHoverIn and onHoverOut (implemented separately from everything else since they are desktop specific) are implemented differently. I don't think we have any plans to fix this bug, at least in paper. In the case of your example, I think even in a native macOS app the highlights would happen. To block highlights, you would probably want to present a modal like an NSSheet. But you would also need a native module for that, so I am not sure I've got a good workaround for you 🙁. Knowing that this issue keeps coming up is good though, thank you for reporting it. I'll update if anything changes.

@bear-ei Yes, onHoverIn and onHoverOut (implemented separately from everything else since they are desktop specific) are implemented differently. I don't think we have any plans to fix this bug, at least in paper. In the case of your example, I think even in a native macOS app the highlights would happen. To block highlights, you would probably want to present a modal like an NSSheet. But you would also need a native module for that, so I am not sure I've got a good workaround for you 🙁. Knowing that this issue keeps coming up is good though, thank you for reporting it. I'll update if anything changes.

Would it be possible to create a native component, something like OverlayView, that would block all interaction with views beneath it, but would otherwise work like a normal View? By the way, react-native-web behaves differently than react-native-macos here. I understand that it might not be easy to fix, but still it's a bit unfortunate for modal (or modal-like) views.

For completeness sake, here is the code on Paper (not Fabric) that send the onMouseEnter / onMouseExit events. which turn into onHoverIn / onHoverOut. As you can see, it's a very simple implementation of

Possibly a simple solution? https://stackoverflow.com/a/35344315

Mouse enters tracking rect -> sends event. To account for zIndex, maybe we just need to make sure the view isn't obscured somehow? I'll look around to see if there's a simple way to do this in Appkit

At least in my case I don't think the issue itself was related to z-index in any way. It also happened with other obscuring views (based on element order).
So yeah that kind of a hit test might work.

There's definitely a bigger issue where it gets in the way of long press on higher level buttons.


Error spam in console. Prolly PLOTSQUARED V5 uncompatibility

Describe the bug
EXCEPTION SPAMMING IN CONSOLE.
Versions (please complete the following information):

Minecraft Server Version: 1.15.2
Plugin Version: 1.8 dev 8.
Involved plugin: Plotsquared v5

Pastebin any errors in console
Link: https://pastebin.com/V2SDQsKQ (This but spammed)
Issue
It's looks like a problem with some enchants on boots (prolly fly onplotsquared)

I've just tested and its the "Wings" enchantment.

This is not happening anymore with lastest build.


FERC to EIA Entity Matching Refactor

Overview
I'm not sure how to pin the issue since it's in a different repo, but it closes https://github.com/catalyst-cooperative/ccai-entity-matching/issues/108 in the CCAI repo.
This PR incorporates the new vectorization infrastructure for entity matching problems into the FERC to EIA matching model.
Major Changes

Removed the recordlinkage dependency that was previously being used to vectorize the model input dataframes and replaced that vectorization process with the infrastructure that newly went in with the FERC to FERC model update.
Added cleaning steps to the columns to improve results.
Had to rework the dataframe embedding graph and ops because I couldn't get it to cooperate with a factory function (see below inline comment)

Still need to do:

Update the doc string in the new vectorization functions that I added to the embed_dataframe module to be more descriptive.

Out of scope

Experiment tracking
Integration of blocking with faiss clustering
Splink model
More advanced imputations of nulls in columns.

Testing
How did you make sure this worked? How can a reviewer verify this?
Accuracy is now at .74 on the test set (previously .73). Consistency of matches is still .74. I'm going to compare the output matches from the model in dev and this model and see if there are any changes in what got matched.
# To-do list
- [ ] Make sure full ETL runs & `make pytest-integration-full` passes locally
- [ ] For major data coverage & analysis changes, [run data validation tests](https://catalystcoop-pudl.readthedocs.io/en/latest/dev/testing.html#data-validation)
- [ ] If updating analyses or data processing functions: make sure to update or write data validation tests
- [ ] Update the [release notes](../docs/release_notes.rst): reference the PR and related issues.
- [ ] Review the PR yourself and call out any questions or issues you have

Hey @katie-lamb if we know that the integration tests can't pass, I'm going to switch this back to a draft so that every new push doesn't cost us a few dollars in GitHub runner time.

@zschira The CI is passing locally for me, so I'm not sure why it's not passing here. Investigating!

After comparing results of the current vs incoming matching model, there are 519 out of the 24430 matches (2% of matches) that change from the old to the new model. After spot checking these changed matches and looking at the average change in each of the "matching columns" between the new and old results, it doesn't seem like the 519 new matches are decidedly better or worse. The change likely comes from the name cleaning and the null filling that happens in this PR.

Merging this manually as an admin since it's passed CI 3 times and keeps getting scooped by other PRs.


Deadlocking?

I'm not sure what the problem is, but it seems that siad becomes unresponsive after a few minutes when it's being run by the UI.
It doesn't seem to be a problem if I run my own siad in the background first.

It seems that the stderr buffer is filling up and that's causing things to freeze. We got rid of the log file, which means that all logging messages from the api were just going to stderr.  The UI triggers so many log messages though that the buffer was filling up and dying in only like 30 seconds.
We fixed this by sending all logging messages to the cross-platform version of /dev/null.
My guess is that there's still a problem here, and that all Sia-UI windows are destined for death after running long enough. This is no longer an immediate concern though.

Looks like we can tell node to pipe stderr to /dev/null: https://nodejs.org/api/child_process.html#child_process_options_stdio
Alternatively we could pipe it to stdout. It doesn't matter much at this point since we aren't doing anything with the output.


RFE: swarming: please support conditional includes in isolates

Currently isolates' includes can not be contained within a conditional block. Attempting to do so causes an exception in isolate.py when uploading the isolates.
It would be helpful if this feature could be supported. This would allow a popular isolate included from many places to have significantly different behavior on different platforms.

With the move to GN, this shouldn't become an issue anymore as GN targets can properly have conditional dependencies.

Cool. I hope Windows and Mac will move to GN soon.


quick town setup command?

When run from the control channel, it would infer as much as possible via naming/regexes:

assume the control channel is in the Day category
assume the night category is "{day} - Night"
assume the roles are "{day} Storyteller" and "{day} Villager"
assume the first voice channel in the Day category is Town Square

The problem is that creation requires giving broader permissions to the bot - I don’t know if people would be hesitant to give that level of perms? Probably not

When you invite a bot, you are supposed to be able to uncheck permissions you don't want to grant. The bot should then be able to gracefully handle a lack of permissions (for example, by telling you it can't do an operation because it doesn't have permission, please reinvite the bot with that permission).

I like how your readme details precisely what it uses each permission for, that seems really nice!

I think this is no longer necessary with !createTown and !addTown available


Remove duplicate logic -wmagent version

WMAgent branch version of #7726

 Fix #7761 - Remove duplicate logic -wmagent version

Merging it since it's for the wmagent branch


Switches Closure Compiler to advanced (5178 to 4336 bytes = 16.3pct savings) fixes #44

hm, seems to break require()
wth
no merge


"Killed" when imagining

(myenv) chris@chris-desktop:~$ imagine "the most beautiful image in the world"
Starting up...
Imagining "the_most_beautiful_image_in_the_world" from the depths of my weights...
Killed

I'm seeing this message when I try to imagine.  I am running this on an Nvidia Jetson Nano (the 4gb version).
These are the versions I am running
(myenv) chris@chris-desktop:~$ python --version
Python 3.6.9
(myenv) chris@chris-desktop:~$ python
Python 3.6.9 (default, Jan 26 2021, 15:33:00)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
>>> print(torch.__version__)
1.8.0
>>> import torchvision
>>> print(torchvision.__version__)
0.9.0

I was able to get beyond this error by increasing the swap memory on the Jetson following these directions: https://github.com/JetsonHacksNano/resizeSwapMemory
However, now when I begin to imagine I get cuda error: the launch timed out and was terminated

I would assume that the process was killed due to a lack in RAM. What you can try is to reduce the image_width and batch_sizeto an acceptable level.
I have no clue about the cuda launch time-out error.

Where can we learn about the minimum RAM requirements?


[Bug] #formSaveTip not visible on REDCap v14.x

Add z-index.

This was not an issue if PDFI, but and interception of RHD.


Adjustments in GA4 events in the Wallet ( only web )

Description
We changed the events from GA universal to GA4 in July.
Everything worked until December, when something broke with the web events.
We did a fix in December and this started to send the "standard" GA4 events ( Page view). This event is triggered by the gtag
gtag("event", "page_view", {
  page_location: "https://wallet.gooddollar.org/AppNavigation/Dashboa" +
                 "rd/Send/Amount?route=Home",
  page_referrer: "https://wallet.gooddollar.org/AppNavigation/Dashboa" +
                 "rd/Home?gtm_debug=1706185703594",
  send_to: "G-94ZZEM10YJ"
})

But the other events (claims, transactions, etc) are sending using datalayer:
dataLayer.push({
  event: "SEND_AMOUNT",
  chainId: 42220,
  token: "4f2b83cfbd3457fb6933c3f3c186540b",
  gtm.uniqueEventId: 8
})

As we are not using Google Tag Manager in the wallet, we must to send the events directly:
gtag("event", "SEND_AMOUNT", {
  "chainId": 42220,
  "token": "4f2b83cfbd3457fb6933c3f3c186540b"
});

Supporting data
This is only about the web events, the events from Android still working properly.
Related:
https://github.com/GoodDollar/GoodDAPP/issues/4013
https://github.com/GoodDollar/GoodDAPP/pull/4014

@patpedrosa this in sprint?

@drptiago this can be verified and tested on prod

Fixed on production


NPM package

Hi
First of all, good job!
I noticed someone already asked this (https://github.com/cferdinandi/gumshoe/issues/72) but do you mind publishing this in NPM as gumshoejs (or another name since gumshoe is a package already)?
NPM has a few problems in installing packages from git (https://github.com/npm/npm/issues/17379) so it would be great if your package was living there. My colleagues thought in forking it and publishing it but it would be better you do it.
Thanks in advance.

Not advertised anywhere, but it's already there under gumshoejs: https://www.npmjs.com/package/gumshoejs
That said... I won't be maintaining that going forward. So it won't get any future updates.

3.5.1 is not yet there. Can you publish it on npmjs?

3.5.1 is not there. Can you publish it on npmjs?

See above

I maintain several dozen OSS projects. What's just one extra CLI command to you is a maintenance nightmare for me.

Sorry, I was not aware of this. Could you need additional maintainers, should we install cferdinandi/gumshoe instead of using the package name or create a fork?
Please let me know what we can do here.

Honestly, this is just not a thing I'm interested in supporting.
Additional install options include using NPM to pull directly from GitHub (supports versions using GitHub tags) or via CDN from JSDelivr: https://www.jsdelivr.com/package/gh/cferdinandi/gumshoe


Token Box UX when defining variants

During offer create, when defining the variants, the token box gets cut off.

if the window is big enough it looks good

when it's like an ipad, then it gets cut off but these pages are meant to be used in a laptop/desktop so Id say we can close this, up to you @sukrita3

Even on full screen mode on my desktop, BOSON gets cut off.

can you check again in https://github.com/bosonprotocol/interface/pull/572? @sukrita3

@albertfolch-redeemeum the token box seems to work but I caught another UX issue ... the qty box becomes misaligned if the token amount is empty.

Looks good, thanks


A chart to install karpenter resources

Tell us about your request
Hi!
We've developed a solution to automatically generate Helm Charts using Kubernetes Resources JSON Schemas. Using such charts, you can define all required resources in values.yaml file. There are also some features supported like templating support for string values. Our best product is Simple Helm Chart
We have a separate chart for prometheus-operator resources only: meta-karpenter
It's maintenance is pretty simple - you only have to update chart config and re-generate the chart using updated karpenter resources schema.
If you interested in it, we can move the chart to the karpenter repository and you can offer users to use it instead of raw yaml.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
How to install Karpenter resources to Kubernetes using Helm Chart
Are you currently working around this issue?
In Terraform, man cat use kubectl_manifest resource as alternative.
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

We'd rather have the chart stay in your github org. We are trying to scope down what is house in the karpenter repos to the bare minimum so that the community can build and extend installation solutions on-top of what we're offering through things like helm, flux, argo, terraform, etc.


Implement KTO into OpenRLHF

Referenced the implementation of HALOs, the KTO algorithm has been integrated into this branch. It supports both balanced (referred to as the vanilla version) and unbalanced (referred to as the non-vanilla version) scenarios for handling positive and negative samples in a batch. The vanilla version ensures that the number of positive and negative samples is consistent within each batch, while the non-vanilla version does not require this consistency.
A lightweight dataset was selected for algorithm validation, where the effects of DPO, vanilla KTO, non-vanilla KTO, and the baseline were compared. The dataset and the results are as follows:

dataset

--dataset Anthropic/hh-rlhf,tasksource/oasst1_pairwise_rlhf_reward,openai/webgpt_comparisons
--dataset_probs 0.72,0.14,0.14

performance

model
Writing
Roleplay
Reasoning
Math
Coding
Extraction
STEM
Humanities
Average

baseline
7.125
7.425
4.05
2.6
2.85
4.475
7.475
8.475
5.559

DPO
7.4
7.39
3.9
3.05
2.475
4.875
7.2
9.075
5.670

KTO_with_vanilla_loss
7.225
7.325
4.025
2.3
3.475
5.525
7.184
9.075
5.715

KTO
7.145
7.273
4.112
2.666
2.790
5.212
8.315
8.479
5.799

* baseline model is "OpenLLMAI/Llama-2-7b-sft-model-ocra-500k"

Thank you for your contribution and we will review it as soon as possible


[Feature] Add support for changing search file name

Based on the FAQs, people who have installed rbenv or something similar may have followed the FAQs and renamed the tilt binary to tlt. However, this extension seems to only look for binaries under the name Tilt, which causes the following error:

Hi, thanks for the feedback. The extension has a setting where you can specify the full path to the actual Tilt executable.

It goes in the settings.json preference file.

It should work to also just put tlt there too, without a full path. Does that work for your problem?

Oh yeah that fixes it thank you!


HKG: unable to engage with pause/resume button with stock cruise control

With the opendbc refactor this logic is now in openpilot, so we need to either:

detect if car has pause/resume button and don't add cancel button
or allow cancel press to enable for Hyundai like we used to

bump


docker hub "https://hub.docker.com/r/gableroux/unity3d/" is 404?

why?

I think I saw a discussion in discord about this. mainly, it was an issue with docker itself on first login. When logged out, the url worked. 👋

Thank you,It may be that the DockerHub account did not confirm the email.


Can we add typescript support for this?

I found a d.ts file over at Definitely Typed: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/request/request.d.ts

Probably the best approach would be to use some wrapper around request. Adding anything TS related here, even tests, probably won't happen.

@simov There's no need to add any tests at all. It's just a d.ts file with the declaration (signatures and type data) of the functions.

You can solve this by just doing:
npm i @types/request

Unless I don't understand what the issue is.


Add debouncing when writing output prop

I find that if upsertBlockProperty is called too rapidly in succession, it won't write correctly, so I added a debouncing behavior.

Thanks! Any reason why you like this particular library for debouncing?

No particular reason, it's a library I often use.
BTW, there is still issues with output writing, I think it's related to the event listeners. I'll investigate further.

It's ready now.

I prefer rxjs, but as long as it works! Thanks!

@hkgnp You may want to release a version since this PR contains a bug fix for assets bundling that was introduced by the last release. 😅


Enable Code Coverage in Live Tests

Description
Fixes #33150
Enables capturing code coverage in the Live tests pipelines.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - azure-communication-identity - tests

@alzimmermsft, I'm going to approve this, but I do have a question. Do we want to turn this on all the time, or do we want to make this a parameter that individual services can opt into for their pipelines?

The change in the PR makes it appear that test coverage will run all the time but coverage is only gathered in one job based on this logic:
condition: and(eq(variables['Agent.OS'], 'Windows_NT'), eq(variables['JavaTestVersion'], '1.17'), eq('${{ parameters.SDKType }}', 'client'), eq('${{ parameters.SkipAggregateReports }}', 'false'))
https://github.com/Azure/azure-sdk-for-java/blob/main/eng/pipelines/templates/steps/build-and-test.yml#L76
@benbp and I were discussing that this should follow what is done for the FromSource runs where this is something explicitly passed by the test matrix but that may happen later

/azp run java - cosmos

/azp run java - storage - tests

/azp run java - azure-communication-identity - tests

/azp run java - azure-communication-identity - tests

/azp run java - azure-communication-identity - tests

/azp run java - azure-communication-identity - tests


Update screen tracking page with appContainer

See #378
While I'm editing this page, I believe all the redux content on this page should be removed.

thanks!

I believe all the redux content on this page should be removed.

I agree, could you update the PR accordingly?

thanks!


Cut new release / tag ?

Looks like Hex & Github disagree:

https://hex.pm/packages/inflex/1.10.0
https://github.com/nurugger07/inflex/releases/tag/v1.4.1

Is this unmaintained?

No, it's still maintained but just haven't had time as of late. I'll be looking at issues and cutting a release this week


feat(ci): Improve needs condition for new-e2e tests

What does this PR do?
Add precise needs condition for all new-e2e tests

Motivation
Prevent faliures for test not having the correct definition (npm) and accelerate execution for others

Additional Notes
I initially wanted to put a default condition on the template but it induces issues.
CWS e2e tests seems not to require the qa_agent job

test fargate
test AgentSuite

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable DEPLOY_AGENT=true --variable RUN_UNIT_TESTS=on

/merge


scenario.write() output doesn't appear in the Before and After sections

Expected behavior
render cucumber Scenario.write() output to @Before and @After sections as well
Actual behavior
current it's not rendered there, not sure if a bug :) or its an improvement.
Reproduction steps
run some test with scenario.write() in the @Before and @After methods and generate report
Additional details (Maven version, Java version, etc.)
maven 3.5.2
java 1.8

Thanks @rvarum I will add that.

Started developing for release 0.8.0...

This is done and will be released in 0.8.0


Tile-join patchwork

Hi,
I have a feature request.
I'd like to use tile-join to join multiple datasets of the same types of polygons but with overlapping areas. For example I have an mbtiles of polygons covering Europe, but I want to merge an updated Italian region, or append a Turkish region that overlaps. See an example:

I understand currently they would be merged such that they overlapped each other, but this is not ideal for updating a shared area with the new file.
Ideally the new polygons would be authoritative and could be joined like-alike with the polygons in the existing file, or truncated at the edge of the new data.

I think the best way to do this currently is with an intermediate tile-join run with a --feature-filter to remove the out-of-date features. You can then do a second tile-join to merge the two tilesets of new features and of old-but-not-removed features into a final tileset.

Thanks for looking at this so fast. I think that approach won't work though with unlabelled/non-geographic data (in this case weather data), where the only indication of a border is the absence of surrounding polygons.
Attaching two GeoJSON examples.
sfctemp.1800.2.json.gz
sfctemp.1800.json.gz
Perhaps I could work around it by manually inserting a tagged feature of both region shapes in both GeoJSON files, then selecting features contained in the overlap with a feature filter based on which region was authoritative?

Pretty close, although I only have two tilesets, covering adjacent but overlapping areas.
I would like to be able to:

Join them together seamlessly - currently I can join them by unpacking the mbtiles and syncing the new tiles from tileset 2 into tileset 1 which works fine for the bulk of the area, but along the edges I will end up with half blank tiles where the boundary of tileset 2 falls on each border tile.
Have control over whether tileset 1 or tileset 2 is authoritative - to define which one to 'overwrite' the area of the other based on which tileset has newer information for the overlapping areas. Just the ordering of the files on the command line would be sufficient for that.

I also tried 'cropping' my tilesets so that they fall exactly on a tile boundary to avoid the half-blank tile issue with my unpacking and syncing, but this is impractical at 0-4 zoom levels.


v4.0.5 contains pure ES2015 modules

$ grep version packages/postcss-tone-vars/node_modules/type-detect/package.json
  "version": "4.0.5"
$ grep export packages/postcss-tone-vars/node_modules/type-detect/index.js
export default function typeDetect(obj) {

Feels like it requires rebuild ;-(

Looks like it was already fixed in 7900eaf5c8568edf2557810355544f49a064dd65
My bad, I'll reinstall all the things. Thank you!

Btw yes index.js contains export default... but if you run the main of package.json is not index.js
Look at print: