Handling Nested Computations under Branching Logic

I hit an interesting problem that I'm gathering I'm just attacking wrong but I wanted to see what you think. This goes back to issues around trying to do conditionals with boolean statements that can run too many times without changing their results. Obviously breaking synchronicity is another option. But as you may remember I'd been creating Roots to handle disposal. I came across an issue there recently that I hadn't noticed before, I'm gathering due to manual disposal getting queued up (ie it's frozen like the rest).
Here is the simplest reproduction I have. You can understand why you wouldn't expect data to be re-evaluated again after the ternary evaluates the other way. Thoughts?
https://codesandbox.io/s/objective-tdd-dif57
The issue is when the condition turned from true to false and value is set to undefined it still evaluates the nested child and fails to find a value with length.
EDIT
I was wrong this even happens when you break synchronicity:
import S from "s-js";

S.root(() => {
  const trigger = S.data(false),
    data = S.data(null),
    cache = S.value(S.sample(trigger)),
    child = data => {
      S(() => console.log("nested", data().length));
      return "Hi";
    };
  S(() => cache(trigger()));
  const memo = S(() => (cache() ? child(data) : undefined));
  S(() => console.log("view", memo()));
  S.freeze(() => {
    trigger(true);
    data("name");
  });
  S.freeze(() => {
    trigger(false);
    data(undefined);
  });
});

This produces the same error.
EDIT 2
https://codesandbox.io/s/vigorous-cherry-vo79c shows an even simpler re-production using only a single condition and no freezing.
import S from "s-js";

S.root(() => {
  const data = S.data(null),
    cache = S.value(S.sample(() => !!data())),
    child = data => {
      S(() => console.log("nested", data().length));
      return "Hi";
    };
  S(() => cache(!!data()));
  const memo = S(() => (cache() ? child(data) : undefined));
  S(() => console.log("view", memo()));
  console.log("ON");
  data("name");
  console.log("OFF");
  data(undefined);
});

Is this just by design and we should expect these states to exist at the same time?

I've tested this now using the track method as described in https://github.com/adamhaile/S/issues/26. And using subclocks as they both seem to address this. They do not rerun the title length child computation on re-evaluation when the upstream computation does not follow that branch. Although with the track method the order of setters matters in the freeze method whereas subclocks cleanly handles the situation regardless of the order (as you'd expect).
Is something like subclocks the only way to cleanly handle this case? I have been attempting to update the subclocks implementation with the new features in the main implementation (exposing nodes, and node recycling) but I haven't quite got it working properly. Thanks for any assistance.

Hey Ryan!
Thanks for the detailed bug report and findings! I noticed I had missed this case for my suggested fix with S.track. The issue was that pending nodes were not lifted when a node was pending disposal but not pending in any other way, yet still being stale.
I've pushed a minor fix (and used your code as a test case to validate, hope that's ok?) to my repo, and the order in which each trigger is being set in freeze should no longer be an issue. Does that bring track on par with the current subclocks functionality?
I'm glad for this issue, designing test cases to cover everything has been quite tricky.

Yes, that fixed my issue.
Subclocks may have some benefits, but currently, from an API standpoint, I don't have much need to expose either beyond letting people set a comparer on a Computation. ie, the downstream conditionals I've been looking at. That is probably a limited view though.
I have had to do some breaking of synchronicity a few places that I'm unsure how either approach would work. Implementing a React Suspense-like experience involved making a counter that would increment as downstream code is executed, and decrement as async responses came back. I only want to trigger the parent when entering count 0 or 1. Since the whole thing is in a computation it didn't like me incrementing the value multiple times. I ended up making it boolean signal and kept the count outside the reactivity.

From an API standpoint, I evaluated my thoughts in https://github.com/adamhaile/S/issues/26#issuecomment-504938191. Generally, I've experienced that ´S.track´ more often than subclocks provide an easy way to handle conditional dependencies, which is the major lacking feature I found in S. However, I still think subclocks is a more complete solution and would be more versatile for use cases where track just wouldn't suffice.
I know that your example is what Adam would call a "bug" in your program. I've encountered similar situations myself, and it shouldn't be that hard providing a custom signal type that allows mutating the signal value multiple times for each tick, only using the finally computed value when entering the next. However, I don't know enough about this topic to foresee which consequences this may have.
Regarding async callbacks, I think that is a different area than branching logic, more in line with Adam's thoughts in https://github.com/adamhaile/S/issues/18?
On another note, it feels as if people believing in fine-grained solutions is quite scattered among many small libraries, tackling the same issues over and over. Perhaps it would be interesting to form a Gitter channel or something where people can share their findings to speed up research in this topic?

I think Adam saw Subclocks as an ability to address all the issues which I can respect. They are also the most fundamental change. I love how track's behavior is mostly completely opt in. I was having some issues getting node recycling working with Subclocks and looked over at what you were doing to find the other way to solve the issue. I hacked something together, but your latest update is much cleaner.

On another note, it feels as if people believing in fine-grained solutions is quite scattered among many small libraries, tackling the same issues over and over. Perhaps it would be interesting to form a Gitter channel or something where people can share their findings to speed up research in this topic?

I love it. I used to talk to @adamhaile  pretty frequently via email to just keep the ball rolling on ideas etc.. admittedly I spend much more of my time looking at DOM api part of the equation but I've made some interesting modifications myself, like the ability to use the Computational context to support Dependency Injection very similar React Context API. I've also been playing with pausing and resuming computations, and looking at ways to simulate React Suspense (although again more on the DOM side of things). I think @luwes would also likely be interested in some sort of forum like this. We also end up chatting about these things. We've been both working on the same problem of client side hydration with fine-grained (albeit having to attack it a bit differently because of differences in DOM renderers) but this is a space where there are generally way less resources and a bit out of date. We all have been sort of re-inventing the wheel to try to bring this tech up to speed with where FrontEnd UI development is today.

Sounds good. I'm not very active in social media in general, but I've attempted to create a forum for us to use, I named it reactive-js on Gitter. We can continue these discussions there instead of clogging up S's issue tracker!

Cool, great idea! Definitely interested in a forum like this, @ryansolid helped me a lot with Sinuous and using GH issues to ask (quick) questions doesn't always feel like the right place.

Hey Ryan, slow response, even for me :(.  Start-up ended up not working out, now on some intense contracting work trying to build back.  Such is life.
On your examples up above, if I follow what you're trying to do, the issue is that cache changes one tick after data.  Since the nested computation's lifecycle follows cache, it runs one more tick than it should and sees the undefined value in data, causing the exception.
Your code is probably boiled out of a more complicated actual case that would explain what you're after, so this might not apply, but my thought would have been to do the simple thing:
const data = S.data(null);
S(() => {
    if (data()) console.log("(not) nested (any more)", data().length));
});

I think the intent of cache and its setter was to be more performant (correct me if I have that wrong), but I think it likely went the other way.  Fundamentally, something still needs to run every time value changes.  Adding the plumbing of a value signal and a computation that sets it doesn't change that, it just adds more plumbing.  So maybe tell me more about what you're after and I can comment.
I kept parity between subclocks and the main implementation for a while, but the stuff around lazy node construction broke that :(.  I'm mixed on whether lazy node construction is worth the complexity it added or not.  At one point, it occurred to me that the initial case we were looking at -- avoiding constructing unneeded computations in DOM components -- would be largely eradicated just by making a single computation per component rather than per node, the way Surplus had been.

I've been trying to do a branchSample. We've talked about this before in other threads. Basically a helper function of the form:
function branchSample<T>(condition: () => Boolean, pass: () => T, fail: () => T): T

The goal is to handle branching in views. Basically only desiring to run the conditions when the state changes otherwise you recreate a bunch of DOM nodes. Of course people are free to write conditions like () => count > 5.
I've attempted to do this a couple of ways. My original approach was using S.root with early returns and handling disposal whenever the branch changed(https://codesandbox.io/s/objective-tdd-dif57). I'd also tried breaking synchronicity and using an intermediate S.value(https://codesandbox.io/s/vigorous-cherry-vo79c). We've also talked about hoisting but that is less generalizable because untravelled paths need to be executed and the data may be invalid.
In any case, a user of Solid noticed that if signal would be changed in a way that caused the branch to change resolution (true to false for example) that any computations created downstream would execute again even though their assumption would be due to the condition failing the resolution wouldn't run again. Why it fails is different in each case. In the nested root version the disposal is suspended until execution is done so it ends up running all the way through. And when synchronicity is broken as you mentioned it takes to the next tick to catch up.
The problem is that while you could just null check everything it isn't obvious since people are thinking of it like an if statement. Like calling the helper should be the null check and I tend to agree.  It just isn't intuitive. So I was trying to think of ways to get this to work. Incidentally subclocks and VSJolund's track both work. It's a decent overhead for something so simple, but to me essential. I'm not sure if there are any other options.
Part of it is the actual purpose of subclocks isn't completely clear. Or even how they work in cases where signal tracks in computations in and outside of subclocks where the value is updated inside. There are several edge cases but I haven't had that ah-ha moment. The chat that was setup also included a big chunk of us discussing trying to figure it out. Ultimately the only case I was really looking at solving was the notifyOnlyOnChange version of computations.
I actually like node recycling now. My favourite example is that component scenario where passed in values may or may not be reactive. See I've done something with Solid where most contexts are actually sampled since all my control flow helper functions are sampled. I didn't like having things tracking in most zones since it could easily lead to unintentional re-renders. I suppose changing the granularity to a per template level would solve it since likely atleast something is reactive. I've kept a per expression basis so far (not even per node). I suppose it's worth experimenting with.

Shoot, I had started writing you a longer reply yesterday, but I must have closed that tab or something because it's gone :(.  So this may be a little shorter.
Thanks for writing at length, because this is the first argument (or maybe just the first time I've gotten it!) that seems to be on to something.  Most of the discussion around something like track has been about efficiency.  Assuming all data signals are properly declared as S.data() or S.value(), the only scenario where track increases efficiency is when you have intermediate computations that are less-selective than their upstream value signals.  So a graph like:
let v = S.value(1),
    p = S(() => v() < 5),
    c = S(() => p() ? "less than 5" : "more than 5");

p() is less-selective than v(), because its value doesn't always change when v()'s does.
The problem with the efficiency argument is that to avoid extra updates of cases like the above, it requires a more complicated (hence more expensive) bookkeeping algorithm.  Specifically, we no longer know exactly which nodes are stale or not, as there's a new state, maybe-stale, with a new backtracking process to resolve the maybe-stale node into stale or current before we know whether it needs updating.  That's a global slowdown to solve a minority of cases.
Furthermore, since we need to run p() for each change in v(), it's not a big-O efficiency gain, just a scalar improvement: the cost of running p() alone vs p() and c().
So in terms of efficiency, it's trading off two scalar factors: universally slower bookkeeping for sometimes faster updates.  It's not clear to me whether that would result in a slower or faster system overall.
What I liked about subclocks is, as the README gets into, they do have the possibility of changing big-O complexity.  Fundamentally, the difference is that track can only decide whether one signal (its own) does or doesn't change value.  Subclocks can decide that for N signals (all the data signals created in the subclock), so they can change the overall big-O cost.  That seems compelling enough to pursue, even though it also requires a higher bookkeeping cost.
But here, you're not talking about efficiency, you're talking about correctness and the "principle of least surprise."  The case you describe is a subcategory of what I think of as the general problem of "dependent state": when a computation creates a piece of state, like a DOM node or a data signal, that state has a transitive dependency on the computation's sources, because if they change, it gets destroyed and re-created.  That changes the less-selective scenario into one that's not just inefficient but semantically surprising.
I may try adding something like track behavior.  Subclocks also introduce a maybe-stale state and an additional backtracking pass.  It might be that this is an intermediate step between the two, or that both can use the same backtracking pass to resolve.

Ok, that's interesting. I look forward to seeing what can be done there. Your understanding in this area goes a lot farther than mine, so I only really get my head into it when I hit a practical scenario. It will be interesting to see which way makes sense.
On an aside, I wanted to mention I have come around on a few things. I ended up going back to pre-recycling version of S.js for the basis of Solid. As you said I think that wider than per binding or node computations are correct. I've changed to per template (JSX block) where all attributes (non-inserts) share the same computations. What I"m looking at now is actually joining them at runtime into a single computation where the Compiler can't get past component boundaries. Basically having a computation node be able to add more tracking functions at initial execution time. Obviously this doesn't apply to things that are dynamic structurally like loops or conditionals. But basically we could reconstruct the template even if broken into many components and in so wrapping them all in a single computation node. At some point simple diffing probably makes sense but it's very much like getting the best benefits of something like stage0/domc without retriggering complete top-down reconciliation all the time. Basically I think Components are not the correct boundaries. The conceptual template (structurally static piece) are. I cannot completely rely on at compile-time, but this run time approach would do the trick I think.
Secondly, I got rid of the explicit binding syntax that you always felt wasn't quite right. I'm using a heuristic like Surplus. It just happens mine has to beyond function calls include property access like state.value or state[key] to ensure that proxies or getters are tracked. I think it's fine though since literals and simple vars are never wrapped. And as you know it is pretty easy to hoist and assign to a variable (assuming the context is sampled, something I generally strive for everywhere). It's actually consistent with how computeds work in general. You access the value outside if you don't want it to be reactive. Since I've taken the approach of wrapping helpers in Components for Control Flow in Solid (the conditional being the one that started this thread) to ensure the insert points are sampled this is all around pretty safe since people generally won't be wrapping ternaries. Although I'm looking at ways to be smarter about those and boolean expressions. I don't think people always realize that when they do:
<div>{ someCondition() ? <ComponentA /> : <ComponentB />  }</div>

That now everything top level in ComponentA is being tracked. I could always just wrap every Component in a sample call but I've been considering other ways around. I think it is very weird as Component creator to be wary of whether someone might have wrapped you in a computation. You kind of want to design your component without worrying about what weird stuff someone who would use it might do.
Anyway I'm super interested in any progress you make in this subclock/track scenario.

let v = S.value(1),
    p = S(() => v() < 5),
    c = S(() => p() ? "less than 5" : "more than 5");

Furthermore, since we need to run p() for each change in v(), it's not a big-O efficiency gain, just a scalar improvement: the cost of running p() alone vs p() and c().
So in terms of efficiency, it's trading off two scalar factors: universally slower bookkeeping for sometimes faster updates. It's not clear to me whether that would result in a slower or faster system overall.

Despite being scalar O improvement, the O's operate in different O time spaces. A more illustrative would be like this:
let v = S.value(1),
  p = S(() => v() < 5),
  c = S(() => p() ? costlyOperation() : evenCostlierOperation());
S.on(c, () => { applyCostlyDomUpdates(); } );

Only evaluating efficiency in terms of how the internal graph operations are affected misses the bigger picture of triggering unnecessary operations that might be long-running. A practical example of an operation running in a different time space could for instance be fetching remote data from the server.
My vs-bind attempt is far from optimal. Nodes should ideally only be queued a maximum of 2 times per tick: once if they are triggered by a tracker, and once if they are marked stale in a downstream dependency. This could be achievable by linking both STALE and PENDING to the node's age.
In order for them to run in topological order the update queue could be implemented as a double-ended queue, such as Deque.
Essentially you would push every node to the queue when applying data changes, and whenever you encounter a tracking node that is marked changed, you unshift each dependent node into the queue so they run in the correct order.
I have not evaluated the performance implications of switching the Queue implementation to a double-ended queue, but as long as that works the track should incur no performance overhead compared to the current implementation.

What I liked about subclocks is, as the README gets into, they do have the possibility of changing big-O complexity. Fundamentally, the difference is that track can only decide whether one signal (its own) does or doesn't change value. Subclocks can decide that for N signals (all the data signals created in the subclock), so they can change the overall big-O cost. That seems compelling enough to pursue, even though it also requires a higher bookkeeping cost.

This is the essential point: track (probably) incurs lower bookkeeping penalty than a subclock implementation, but only provides a solution for 1-N bindings, whereas subclocks solve N-N bindings.

I created some benchmarks between double ended queues and the current, simpler Queue version, and preliminary it looks as if it does hit performance quite a bit.
If track cannot be implemented without causing too much slowdown I agree with @adamhaile that Subclocks probably is a better way forward as it solves more cases than track.


Update script.json

@ssveter1
If this doesn't give you the API scripter tag on your Roll20 profile let me know.
Thanks,
Cassie

When would I expect to see the tag? Because I do not at this time.
From: Cassie Levett<EMAIL_ADDRESS>Sent: Thursday, April 11, 2019 9:33 AM
To: Roll20/roll20-api-scripts<EMAIL_ADDRESS>Cc: Savage GM<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Roll20/roll20-api-scripts] Update script.json (#791)
@ssveter1 https://github.com/ssveter1
If this doesn't give you the API scripter tag on your Roll20 profile let me know.
Thanks,
Cassie
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Roll20/roll20-api-scripts/pull/791#issuecomment-482115773 , or mute the thread https://github.com/notifications/unsubscribe-auth/AKVFxl2WfxC5q8Qm3LWRHfdBMmpozexfks5vfzmhgaJpZM4ccrPD .  https://github.com/notifications/beacon/AKVFxnchuGsU_ZCQxBHXPkLKbOijvUEQks5vfzmhgaJpZM4ccrPD.gif

@ssveter1
Can you email me a link to your Roll20 Profile<EMAIL_ADDRESS>I'll take a look and see if I can find out why it may not be adding it.
Thanks,
Cassie


Iframe content height

I need to get height value of generated iframe content.
I'm on the same domain (no cross-site scripting)
How I can do this with magnific popup callbacks? I've tried, unfortunally without successful

XMLHttpRequest cannot load http://somedomain.com/2014/12/22/20141222111037-e88ebfcd.jpg. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://<IP_ADDRESS>:49504' is therefore not allowed access.

Any idea to get this cross domain issue resolved?


Fix error while building on iOS

Issue Description
Fixes the following error that occurs while building on iOS:
declaration of 'MLKTextRecognizedLanguage' must be imported from module 'MLKitTextRecognitionCommon.MLKTextRecognizedLanguage' before it is required

This issue was reproducible with the following environment:

Flutter Version 3.10.3
Xcode Version 14.3.1
google_ml_vision Version 0.0.8

Related issue: #30
Closes #30

Ho @brianmtully! Thanks for the amazing library and merging this PR.


Failed to get connector for display 2

[√] I have read the FAQ.
[√] I have searched in existing issues.

Environment

OS: Windows
scrcpy version: 1.24
installation method: releases
device model: SUNMI D2S
Android version: 7.1.2

Describe the bug
Sorry I don't speak English, the following is from Google Translate:
When I use scrcpy to connect to the device, the log viewed by the adb logcat command will repeatedly output such statements:

06-30 14:36:20.229 215 215 E hwcomposer-drm: Failed to get connector for display 2 line=1564

It doesn't stop appearing until I close the window.
Command run:

scrcpy -d --window-title "SUNMI D2S" --always-on-top --stay-awake

Remove -d Flag

06-30 14:36:20.229 215 215 E hwcomposer-drm: Failed to get connector for display 2 line=1564

It looks like a device ROM issue.
Does mirroring work, even with these logs?


ability to automatically connect to a replica server in a mongo cluster

it would be great to configure a cluster url like this and have the app automatically connect to the replica:
mongodb://:@foo-a0.mongolab.com:4444,foo-a1.mongolab.com:444/?replicaSet=rs-foo

+1

Multiple server config will be added in 3.8.1


Add working Dockerfile to example

I tried creating a Dockerfile but failed.
Maybe we can add a working example to the examples dir.

Maybe the platform has to be build with CGO_ENABLED=0 so that the result is not dynamicly linked to a librabry that does not necessarily exists in an context of usage (here usage of a docker image  FROM scratch).

This is a problem, but can not be solved with CGO_ENABLED=0  cgo is needed for the go - roc combination.
We have to find another way  for a static build.
If we find a way to create a static build, then I don't think we need a Dockerfile. But if it is not possible to create a static build, we should create a docker file that contains the required dependencies


WIP: Fix for CPU and Memory issues fetching resources

Fixes:
Fixes RHODS-3191
Analysis / Root cause:
Fetching all CSVs is extremely expensive. On the cluster noted, there are 90k CSVs. This is the bulk of the issue.
Fetching all Services (2k) is also expensive, we only need a specific service when an ISV is installed and then just one specific service by name.
There is a bug in the build retrievals that can cause an infinite loop fetching builds.
Solution Description:
Don't retrieve all CSVs or Services.
Fetch subscriptions and use the matching subscription to then find the specific CSV.
Only retrieve the service by name (across all namespaces) when we need to.
Return the current inactive timeout interval for builds so that the fetch routine doesn't think it needs to update the timer and re-fetch.

Dependent on https://github.com/opendatahub-io/odh-manifests/pull/555


update azure installation guide

Update azure installation guide to point KFDef manifest for v1.1 for Azure
[https://github.com/kubeflow/manifests/pull/1487] (url)

CC- @aronchick

/assign @aronchick

@8bitmp3  if you see the netflify preview formatting under "Unpack the tarball:" is broken post-deployment , any pointers what's missing here? However local rendering is perfect

Thank you for your efforts @sudivate
I've just submitted a PR that creates a new OWNERS file to help with Azure docs review process https://github.com/kubeflow/website/pull/2199 - it turns out KF Azure hasn't had official OWNERS assigned yet.
@aronchick Let us know what you think about the changes here and in https://github.com/kubeflow/website/pull/2199
Cheers

@8bitmp3  while we get the approvers added for Azure who can be the provisional approver for this PR?

/assign @eedorenko

/lgtm
/approve

/assign @animeshsingh

@animeshsingh  can I please get your approval?

/lgtm
/approve


Update a message by adding a PDF file to it

I am trying to update a message by adding a PDF file to it, but I cannot achieve it.
Is this even possible to do?
If yes, please tell what I am doing wrong.
My code:
methodsClient.chatUpdate(ChatUpdateRequest.builder()
                .token(apiToken)
                .channel(originalMessage.getChannel())
                .ts(originalMessage.getTs())
                .text(message)
                .build()));
methodsClient.filesUpload(FilesUploadRequest.builder()
                .token(apiToken)
                .filetype("pdf")
                .filename("test.pdf")
                .fileData(bytes)
                .channels(List.of(originalMessage.getChannel()))
                .build();

The result of this code is that the file is uploaded in separate message instead of being added to the original message.
Best I can do is to provide threadTs to FileUploadRequest, then the file is added as separate message but in a thread reply to the main message.
I've also tried to first upload a file without specyifying a channel and then updating the message by adding a file attachment, but this did nothing -> file didn't show as an attachment or a message:
var uploadResponse = methodsClient.filesUpload(FilesUploadRequest.builder()
                .token(apiToken)
                .filetype("pdf")
                .filename("test.pdf")
                .fileData(bytes)
                .build();
methodsClient.chatUpdate(ChatUpdateRequest.builder()
                .token(apiToken)
                .channel(originalMessage.getChannel())
                .ts(originalMessage.getTs())
                .attachments(List.of(Attachment.builder().files(List..of(uploadResponse.getFile)))
                .text(message)
                .build()));

The Slack SDK version
1.20.0
Java Runtime version
Java 17
OS info
Windows 11

I'm removing the "stale" status of this ticket, and noting that our internal team responsible for this particular backend API is aware of the issue and it has been triaged, but no progress has been made thus far. For other Slack folks dealing with this issue in the future, the internal ticket identifier is MSG-27596.
In the mean time, my last comment stands: if you create a message with a file attached to it, you can update its file(s) using chat.update to your heart's content. The bug is that a message created with no file originally attached will not be updated with any new files attached to it via the chat.update API.

Hey @filmaj any update on this?
You said

if you create a message with a file attached to it, you can update its file(s) using chat.update to your heart's content

But is there a way to send a message w/ a file attached to it?
It seems like messages and files are always separate entities. I can upload a file by itself into a channel and then update that file w/ a new file ID by using chat.update but I can't seem to send a message with blocks + a file.
Any ideas?


Exception encountered: ApiTypeError("SystemInfo is missing 1 required argument: ['buildTag']")

$ nitropy nethsm --host nethsmdemo.nitrokey.com --username admin --password passphrase system-info
Command line tool to interact with Nitrokey devices 0.4.37
Critical error:
An unhandled exception occurred
Exception encountered: ApiTypeError("SystemInfo is missing 1 required argument: ['buildTag']")

Critical error occurred, exiting now
Unexpected? Is this a bug? Would you like to get support/help?

You can report issues at: https://support.nitrokey.com/
Writing an e-mail to<EMAIL_ADDRESS>is also possible
Please attach the log: '/tmp/nitropy.log.phtwgywr' with any support/help request!
Please check if you have udev rules installed: https://docs.nitrokey.com/nitrokey3/linux/firmware-update.html#troubleshooting

Caused by an old version of pynitrokey.


[wip] ci: add github actions workflow to test self hosted runner

[ ] need to use a cloud storage for the backend, to be able to remove the instance after the job
[ ] Add mapt destroy job to remove provisioned instance

Hey can we test  selfhosted runner running crc on it. I am curious on the user running the agent.
I mean when I set up ssh I had to trick the user starting the service, as with default setup it is a system user (privileged) as so crc is not working. How is working with the gh agent?

Hey can we test selfhosted runner running crc on it. I am curious on the user running the agent.
I mean when I set up ssh I had to trick the user starting the service, as with default setup it is a system user (privileged) as so crc is not working. How is working with the gh agent?

the runner agent running as service using the nt authority\network service account, i did a test run here: https://github.com/anjannath/mapt/actions/runs/11459558776/job/31884314010
will have to try running crc start to see it it works with this user, the NetworkService is also a privileged user account i think

@adrianriobo the workflow_call strategy will pick the commit from the PR if we add a step for checkout code in the workflow job
so currently this PR adds to re-usable workflows, provision_hosted_runner and destroy_hosted_runner and a build_on_hosted_runner workflow that makes use of the reusable workflows, but currently it seems the secrets are not accessible to the workflows as they are not part of the repo main branch yet
the build-on-hosted-runner workflow additionally has to fetch a runner registration token which is needed to add a self hosted runner and is an input for the provision_hosted_runner workflow

@adrianriobo this is finally ready for review again, got some successful runs on my fork, https://github.com/anjannath/mapt/actions/runs/11698979004

^^ @anjannath About your comment you can add a filter on PRs were any file of the util/ghactions is modified i.e. https://github.com/crc-org/ci-definitions/blob/main/.github/workflows/crc-builder-builder.yaml#L6C3-L8C79
Beyond that LGTM

actually this cannot be used, since we trigger the build-on-hosted-runner workflow based on if oci-build workflow is finished, and then on the job check if the event was a pull_request or not


json_encode() on handle token method. use array key exists

This is just the implementation based on @arcanedev-maroc referenced here: https://github.com/JoeDawson/youtube/issues/66#issuecomment-321379318

hey @JoeDawson / @arcanedev-maroc would you guys mind taking a look at this pr?

Thanks Daniel, I just want to do a little testing as soon as possible then I'll merge if all good.

Any chance that this will get merged soon?

Taking a risk this is all good, don't want people to wait any longer. Just too busy to sit down and manually test this atm.
A new release is now available, thank you @danielpuent3.


diskEncryptionSetId=$(az disk-encryption-set show -n mydiskEncryptionSetName -g myResourceGroup --query "[id]" -o tsv) Is wrong

[Enter feedback here]
You forgot to put the $ in the front of diskencytpionsetid/  for example $diskEncryptionSetId=(az disk-encryption-set show -n mydiskEncryptionSetName -g myResourceGroup --query "[id]" -o tsv) Is wrong

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

ID: cb36087b-eaa6-ae3b-b3bb-6cb58268c3b9
Version Independent ID: 2567f115-766c-f94f-7696-eb3bb94ba480
Content: Use a customer-managed key to encrypt Azure disks in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/azure-disk-customer-managed-keys.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@jedelblu
Thanks for the feedback. We'll update the document with the necessary changes.


Can I get your open source Simplified Chinese documents?

Thanks for your dayjs project.
I want get simplified chinese documents,but I could't find it in this repository.
Can you give me some help?Thank you very much for your efforts into dayjs.

https://github.com/dayjs/dayjs-website/tree/gh-pages


Update GraalVM Native Image Dockerfile to use slimmed down container image

Issue description
GraalVM Community edition now provides a set of images with different components pre-installed, including Native Image.   The Dockerfile generated by dockerfileNative should:

Change FROM to ghcr.io/graalvm/native-image:<version>
remove RUN gu install native-image since the native-image utility is pre-installed in the FROM image

The size of the ghcr.io/graalvm/native-image image is ~830MB vs the 1.2GB size of the currently used ghcr.io/graalvm/graalvm-ce image.
Native Image container images: https://github.com/graalvm/container/pkgs/container/native-image
All GraalVM container images: https://github.com/orgs/graalvm/packages

Added corresponding Maven issue for this: https://github.com/micronaut-projects/micronaut-maven-plugin/issues/222

+1


Update prepare.sh

Changed the wrong filenames when shuffling the train cuts.

Changed special word "[UNK]" into <UNK> in order to match with the special word used in local/prepare_lang_bpe.sh

Thanks for your contribution!


Remove uses of Microsoft.UI.Dispatching.DispatcherQueue.GetForCurrentThread()

Summary of the pull request
Microsoft.UI.Dispatching.DispatcherQueue.GetForCurrentThread() will return null if the calling code isn't running on the UI thread. This can cause crashes, since running on the UI thread is not guaranteed. Instead, we should use the WindowEx.DispatcherQueue, which will always return a valid DispatcherQueue.
ExtensionAdaptiveCardPanel still uses GetForCurrentThread() because I think that is guaranteed to have a DispatcherQueue (please tell me if anyone thinks that's wrong).
References and relevant issues
Detailed description of the pull request / Additional comments
Validation steps performed
PR checklist

[x] Closes #2504
[ ] Tests added/passed
[ ] Documentation updated

Signing off on the dev drive insights changes. LGTM.

What are you using to always catch these spelling mistakes/whitespace issues? 👀

@bbonaby For spelling, I use the built-in VS spell checker. It could be better (frustratingly, only runs on open files, but if the file is open there's an indicator to the right of the line, as well as a suggestion in the Error List).

For whitespace, I'm just neurotic 😆


Browser Stucks While Updating Chart

Hi,
I am Facing an issue while updating Charts "Low Option". By Default the low is set to 0
When I trigger the update to around 6000 low it works fine.But when i change it back to 0, the browser stuck up.
var updateOptions = {
low: 6000 //works fine
};
chart.update(data, updateOptions, true);
var updateOptions2 = {
low: 0 // from 6000 to 0 low it stuck up
};
chart.update(data, updateOptions2, true);
Please help me out on this.
Thanks

There was a common problem with the options and overriding mechanism. Can you check out the dev branch and do a build grunt build and use this version to test if the issue is still open?

0.7.3 is released, can you check if the problem persists?

Yeah It has been Fixed, Thanks For Fixing It!

Great! Glad to hear that :-)


[Documentation] Add contribution file with guidelines

Part of reviewing process for JOSS acceptance (openjournals/joss-reviews/issues/4681)
I couldn't find clear instructions on how to participate in this project. Ideally you should provide a CONTRIBUTE file with some informations about:

How to contribute to the software (e.g. how to create a pull request or minimum requirements for accept a PR )
How to report issues: your procedure of choice for handle bug reports (is there a template to follow? is required to provide a minimum reproducible example? is ok just a simple issue opened on GitHub?)
How to reach support: alternate channels where to get help (email, forum, ecc..)

Thank you for the comment!
Closing because the following commit adds a CONTRIBUTING.md with extensive documentation on how to contribute and references it in the README.
https://github.com/siavashk/pycpd/commit/08e8cf0b86018948e670cdf1d5e10b450c4547b1


Is it possible to sync old wandb local runs?

Hi, I just managed to setup the local wandb version (had some issues with not being able to login, fixed thanks to Issue #64 ) and I was wondering if it is possible to sync old runs that saved also wandb logs locally.
The problem that I seem to be having is that my previous runs all were using a different entity than what I have now and I couldn't find any information on whether it is possible to somehow change the logs of the old runs to make it compatible. Basically when I try to sync old offline runs I get into the "Permission denied issue", and I checked the code and the entity was different than what I am using now.
Is there any way to visualize old wandb runs that have not been synced so far?

Sure!  You can either override the entity by passing the -e flag to wandb sync or create a team with the same name as the old runs used and add yourself to it.

Hi @AndreeaRadoescu , to add to the above with an expanded example, you can use the following command:
wandb sync --project "<project-name>" --entity "<entity>" <path-to-run-folder>
Test this with the following basic run, extract the content and add the run folder to your working directory
run-20220829_191828-3cvxvrbj.zip
More on wandb sync can be found in our documentation, see here. Please let us know if you have any questions.

Thanks so much guys, it works. Before your answers I thought about just creating a username with the same name as the entity used in the experiments and that seemed to work but being able to change the entity name is a much better solution.
One more question though: in my attempt to try and get it to work, I might have greped for where that entity name was used in the wandb files and I saw it was in one of the binaries, so I kinda changed the binary (using nano which apparently parses it nicely) to use a different entity but I think that broke the validation of the binary that happens at a later time. When I tried to sync it with a changed name, I got an error message that checksum failed.
So I changed it back to what it was before but I still get the same error message, probably the hash of the binary changed or something....
So now comes the question, is it possible to still recover that experiment somehow and sync the files with the wandb backend?

If you change the file back to use the original entity name the checksum should succeed.

Ok, I will try that then. Thanks so much for all your help!


Added showBackground() by default into getBrowsershot() function.

Issue: Background colours aren't supported by default.
This can be overcome by setting
withBrowsershot(function (Browsershot $browsershot) {             $browsershot->showBackground();         })
however it should be enabled by default for the use case of this package.

Thanks!


Add image classification tutorial in AutoMM

Issue #, if available:
NA
Description of changes:
A tutorial on how to use AutoMMPredictor to do image classification.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-1869-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1869/1/index.html

Job PR-1869-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1869/2/index.html

Cool, thank you. Let me rebase and trigger the CI.

Job PR-1869-5 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1869/5/index.html


ANPEEE: BGP Prefix not required on Address Prefix CLI

Adding the same info from Portal doc to CLI doc, stating that is not required to use the AddressPrefix field if BGP is configured.

WI: https://msazure.visualstudio.com/One/_workitems/edit/28874567

#sign-off

@cherylmc I just added the same sentence from the Portal doc to the CLI doc:

Portal Doc:  https://learn.microsoft.com/en-us/azure/vpn-gateway/bgp-howto#important-configuration-considerations

So the sentence should already have been validated on the Portal doc.

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

#sign-off


ASCII image doesn't appear on OS X El Captain.

I installed the script as per the instructions on the web page, including this instruction:

El Captain: PREFIX=/usr/local make install

When I run the command such as fetch, fetch --image ascii, fetch --image ascii --ascii distro, etc. it prints the system information without the fancy logo.
I'm no expert, but looking at the script's code it appears to get the ascii text from either /usr/share/fetch/ascii, or from the script's physical path. However, the files are actually located at /usr/local/share/fetch/ascii. Is this a bug, or did I do something wrong?

Fixed, thanks for reporting this.
We'll need to eventually find a way for the script to use "$PREFIX"
from the makefile but for now I've added support for /usr/local.
You'll need to pull the latest master and rerun the makefile to get
the fix. Also you can just launch fetch with fetch --ascii to use
your distros ascii art.

That was fast - thanks! I updated to the latest version and can confirm that it works. :)

Awesome! :)


Add service_model property to operation_model

This makes it so that given a reference to an operation model, you can access its parent object.  Otherwise, existing code needs to track the service model when needed, despite the fact that it's accessible via an internal attribute on the operation model.
This is also needed as part of the client switchover work in the CLI.
cc @kyleknap @danielgtaylor

Coverage increased (+0.0%) to 94.55% when pulling ff072076c5c659f85d59a5907b9e00b591667f70 on jamesls:add-service-model into 2eee0f1a2a459c2ab285d49975cd2621b0bb5789 on boto:develop.

Looks good. Had one question/comment. Otherwise :ship:


Create an initial project to work with it as out of the box

Create an initial project to work with it as out of the box

[ ] Initial project by composer structure
[ ] Setup project by docker images
[ ] First tests to pass green scene

Has been added github actions to be able continuous integrations as an essential workflow

[X] Added Github Actions CI


Update pg, cockroach and all go dependencies

Codecov Report

Merging #70 (cf0c2c9) into master (6dd93fd) will decrease coverage by 0.31%.
The diff coverage is n/a.

:exclamation: Current head cf0c2c9 differs from pull request most recent head 01e9626. Consider uploading reports for the commit 01e9626 to get more accurate results

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   78.50%   78.19%   -0.32%
==========================================
  Files           7        7
  Lines         642      642
==========================================
- Hits          504      502       -2
- Misses         74       75       +1
- Partials       64       65       +1

Impacted Files
Coverage Δ

memory.go
92.45% <0.00%> (-3.78%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6dd93fd...01e9626. Read the comment docs.

superseede by #72


day6 not working

day6 not working，only tarBar

day6 is Spotify,you should set  'react-native-video' first. May be there are any warning?

Some compatibility issue with react-native-video & RN0.34. To be fixed.


Variables for Non-Power Sectors in GENeSYS-MOD

This pull-requests adds the variables needed for sending non-power sector related data from GENeSYS-MOD to the openEntrance-Database. The added variables suggested in this pull-requests are based on Deliverable 5.2 "Definition of the interface between the models in the suite and the Common Database" of openEntrance.
It is also related to #7.

Thanks @tburandt. This is a pretty large PR, so the review could become quite messy. It might be easier to separate this branch into several branches with more specific changes (one for transport modes, one for transport-services, one for other changes to fuel types and energy variables), and then work through them one by one.
Also, please add a test for each type of exploding-tag variable.

Thanks, @danielhuppmann for the detailed review! For now, I would rather stick to this large pull-request as it is right now because it makes it much easier for me to simultaneously update the script that actually converts the GENeSYS-MOD results. But I agree: if reviewing the pull request becomes too messy, I will split it up.
However, I will definitely reimplement the exploding variable and add a corresponding test

@danielhuppmann Just a small reminder, to re-re-review this pull request


Fix & Add: Japanese Translation (3)

Continued of https://github.com/bluesky-social/social-app/pull/2209
This PR is a continuation of the PR above.
A message appeared stating that the branch conflicts, and in an attempt to address this, I tried to synchronize and merge it with the PR above, but it did not work, so I created a new PR.

Mention those who were associated with the previous PR.
@nostr-bluesky @tkusano @dolciss @ansh @oboenikui @mimonelu @noritada

Great. Will try to merge today.
On Mon, Dec 18 2023 at 13:39, Takayuki KUSANO < @.*** > wrote:

@.**** commented on this pull request.
checked
—
Reply to this email directly, view it on GitHub (
https://github.com/bluesky-social/social-app/pull/2226#pullrequestreview-1786075498
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AB6T25JU2GV43JQSDACNKCLYJ724ZAVCNFSM6AAAAABAY7OJX2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOOBWGA3TKNBZHA
).
You are receiving this because you were mentioned. Message ID: <bluesky-social/social-app/pull/2226/review/1786075498
@ github. com>


Can't download data folder

The data folder download website can't be opened

Hi, it seems to be working from our side. The direct link also seems to be pointing to the right dropbox page. Could you please share the error information you got from running the wget command or clicking the link?

Thanks for your reply, I solve it by using a VPN.

Hi, it seems to be working from our side. The direct link also seems to be pointing to the right dropbox page. Could you please share the error information you got from running the wget command or clicking the link?

could you please share data.zip by other driver, cause dropbox cannot be accessed in China, :(
or  just a sample of data showing format of those data files

Hi, it seems to be working from our side. The direct link also seems to be pointing to the right dropbox page. Could you please share the error information you got from running the wget command or clicking the link?

could you please share data.zip by other driver, cause dropbox cannot be accessed in China, :( or just a sample of data showing format of those data files

Fully understand... Unfortunately I also suffer from Baidu wangpan... Is there a good alternative?
I have uploaded the devdata folder for now, which should be sufficient for an example pass through the pipeline. Please let me know if it works!

Hi, it seems to be working from our side. The direct link also seems to be pointing to the right dropbox page. Could you please share the error information you got from running the wget command or clicking the link?

could you please share data.zip by other driver, cause dropbox cannot be accessed in China, :( or just a sample of data showing format of those data files

Fully understand... Unfortunately I also suffer from Baidu wangpan... Is there a good alternative?
I have uploaded the devdata folder for now, which should be sufficient for an example pass through the pipeline. Please let me know if it works!

It works, it really saves me a lot of time.  THANK YOU SO MUCH


Possible replacement for slugify

I recently saw this fella:
http://pid.github.io/speakingurl/
It claims to do very good slugging in only 12kb minified before compression.
The current solution requires XRegExp, which is a pretty big sized script to load, and if I'm not mistaken - it is used exclusively for the slugify both back and front end.
Would this be an area of interest for a PR?

We're using sluggo:
https://www.npmjs.com/package/sluggo
XRegExp is still a server-side dependency because of our truncatePlaintext
method, but that's not slugging.
On Sun, Apr 26, 2015 at 10:14 AM, Danny Povolotski<EMAIL_ADDRESS>
wrote:

I recently saw this fella:
http://pid.github.io/speakingurl/
It claims to do very good slugging in only 12kb minified before
compression.
The current solution requires XRegExp, which is a pretty big sized script
to load, and if I'm not mistaken - it is used exclusively for the slugify
both back and front end.
Would this be an area of interest for a PR?
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/241.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com


Removes alias method :use_framework=

This bit me today and I yak shaved for a couple of hours. In my config, I had the following:
config.suite do |suite|
  suite.use_framework = :jasmine, "1.3.1"
end

Teaspoon accepted the configuration except every time I ran the rake task, it would raise an exception:
Teaspoon::UnknownFramework: Teaspoon::UnknownFramework
/home/v/.gem/ruby/2.2.2/gems/teaspoon-1.0.2/lib/teaspoon/registry.rb:24:in `fetch'
/home/v/.gem/ruby/2.2.2/gems/teaspoon-1.0.2/lib/teaspoon/configuration.rb:88:in `use_framework'
/home/v/code/spec/javascripts/config.rb:6:in `block (2 levels) in <top (required)>'
/home/v/.gem/ruby/2.2.2/gems/teaspoon-1.0.2/lib/teaspoon/configuration.rb:82:in `initialize'
/home/v/.gem/ruby/2.2.2/gems/teaspoon-1.0.2/lib/teaspoon/configuration.rb:59:in `new'
/home/v/.gem/ruby/2.2.2/gems/teaspoon-1.0.2/lib/teaspoon/configuration.rb:59:in `suite'
/home/v/code/spec/javascripts/config.rb:5:in `block in <top (required)>'
/home/v/.gem/ruby/2.2.2/gems/teaspoon-1.0.2/lib/teaspoon/configuration.rb:167:in `configure'
/home/v/code/spec/javascripts/config.rb:1:in `<top (required)>'

A bit of debugging revealed this:
[1] pry(main)> class Test
[1] pry(main)*   def use(one, two = nil)
[1] pry(main)*     puts "one #{one}"
[1] pry(main)*     puts "two #{two}"
[1] pry(main)*   end
[1] pry(main)*   alias_method :use=, :use
[1] pry(main)* end
=> Test
[2] pry(main)> test = Test.new
=> #<Test:0x007f92b6ee8ea0>
[3] pry(main)> test.use = :jasmine, '1.3.1'
one [:jasmine, "1.3.1"]
two
=> [:jasmine, "1.3.1"]
[4] pry(main)> test.use = :jasmine
one jasmine
two
=> :jasmine
[5] pry(main)> test.use(:jasmine, '1.3.1')
one jasmine
two 1.3.1
=> nil

The only time the use_framework= method works is when you pass in a single argument (framework name). Is this method worth keeping around?

Good catch, and sorry about the yak shaving. Programming, right?
I don't think it would be wise to straight remove the alias, as this would introduce a breaking change. We should define use_framework=, provide the correct functionality, and deprecate the method.

Good point. I can work on explicitly defining use_framework= to handle my case and deprecate the method.

Thanks. We try to isolate deprecations here via monkey patching. It seems like you could remove the alias, and define the method there.

Sounds good. Shall I close this request and open a new one when ready?

Seems reasonable to keep this open for now and just push changes to this PR.

I think aruba 0.8.x introduced some breaking changes that is causing the CI builds to fail.

Damn, that Aruba upgrade was painful.
Thanks so much for this high quality PR :heart:

:+1:

Thanks for merging! :+1:


[fix] gws schedule : add_todo url

Coverage decreased (-10.8%) to 71.021% when pulling 174a2b9f51b96a220b5bd344d5ba337034001a80 on itowtips:fix-gws-todo into 7ac78ae212d9946f53a8ccf26cf5bb9eb26fbfdc on shirasagi:master.


enable the wiki...

can you add me to the authorisation to the wiki...
I want to start using the wiki to provide instructions rather than the readme...
you can navidage the readme, and then you can find you get lost in the information.
The wiki means we can focus on specific instructions and drill into the detail.
https://help.github.com/en/github/building-a-strong-community/changing-access-permissions-for-wikis

added you as collaborator to edit wiki

ive started to populate the basic stuff, will do a custom index later and add some graphics.

thanks I will edit as needed

I need to build the side bar
then want to start adding the sections so we explain the content in the devices.yml and also in the smartapp.
then want to explain how to use or build a device handler.
the bridge bit should be the easiest thing to sort... its ensuring people understand the device side and how to connect to smartthings.
ultimately the more device handlers people can build - so we have a library - the better.
that way we can try to change the way smartthings have adopted MQTT and think about a generic bridge rather than just connecting to their own MQTT device.

Frankly I don’t really think most folks need a custom device handler for basic operation.
For example if u have a 3 channel switch then u would need to create 3 virtual switches in ST (which u would anyway assuming u want to control them independently) and do the corresponding mapping in the confit file.
So if the device u r using corresponds to a ST simulated device or for that matter any other device handler that can be mapped to any of the field maps in the SmartApp - u don’t need to implement custom device handlers.
It’s for composite devices or building dashboards for multiple devices is when u need to get there but 80/20 rule should need custom DTH

If u check my new repository I am going to move all these under MQTT namespace.
I just built a presence sensor today that basically Pings a d1 or any IP or host and sends an MQTT msg that can now be sent to  ST but in ST I am using the basic simulated ST presence sensor - did not have to create a custom DTH


ImportError: cannot import TableModelView from superset.connectors.sqla.views in superset.views.core.py

Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if any
[ ] I have reproduced the issue with at least the latest released version of superset
[ ] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.19.1
Content
I am going to import TableModelView from superset.connectors.sqla.views in superset.views.core.py. However, I get a ImportError: can not import name 'TableModelView'. How can I fix it, Thanks a lot!
Expected results
Should import TableModelView.
Actual results
Getting ImportError: can not import name TableModelView.

What are you trying to do?

@xrmx
Thank you for your reply!
I'm trying to cteate a MultipleView class with TableModelView and DashboardModelView.The MultipleView can render them on the same page.

Notice: this issue has been closed because it has been inactive for 216 days. Feel free to comment and request for this issue to be reopened.


Enabling translation

More strings enabled for translation in components
/cc @hummingbird-me/staff

I added some comments, although I may have missed some as it was only a quick look atm.
I think a lot of these translations should include the full text (including any dynamic properties) as some languages may require an order change.
I'll let others who are putting time into the recent translation effort weigh in.

Added some dynamic properties.
After this get aproved will continue with more strings, now I understand how to put them in the strings. Thanks!


[Feature Request]: Playground emulated data sources require predefined data sets

What would you like to happen?
Playground emulated data sources require predefined data sets
Depends on: #22494
Issue Priority
Priority: 3
Issue Component
Component: beam-playground

Tracked in https://github.com/apache/beam/issues/23850


fuzz: reset the exit code global variable after every test

Fix #5777

thanks. let's see if it sticks


EMI doesnt work with gamephases

the main issue with it is emi doesnt really update and hide/unhide mods and items when unlocking gamephases

EMI does not currently support live editing of registered recipes and stacks or the visibility of them, however gamephases could have integration by using vanilla datapack reloads. Either way, EMI can't provide this integration, only adding features to make it more convenient.

yeah doing a datapack reload is far more inconvenient then not having the mod installed. so sadly guess it will have to be


use to_str before gsubing on a string

gsub on safe strings poses problems because $x variables are local, therefore they're not available in the safe strings gsubs.
See https://github.com/rails/rails/issues/1555
Therefore, gsub on safe strings is going to be disable on rails.
As mail uses it, it breaks tests on rails. This PR fixes the problem.

Thanks :)


Make repository as template

I'm not sure if this is already done but would be nice to set the repo as a "template" https://docs.github.com/en/repositories/creating-and-managing-repositories/creating-a-template-repository

So nope that's not the case but what we have (already ready to flight) is the module_factory command in sepal_ui lib. It does pretty much the same + some initialization on your local machine (pre-commit hooks, create the venv, fill all the placeholders, and set-up a release branch). I've documented here.
we used it 2 days ago with @t-elisee. (https://github.com/t-elisee/ofgt_kmean).
Happy to have a look on what's possible to do directly from github

The repository is now a template and can be used as such. Be aware that only the master branch should be used and that all the placeholdres/readme/about section will not be adapted (links in the ui.ipynb as well).
I started a documentation section here: https://github.com/12rambau/sepal_ui/commit/bc5640b3b85247478dc9bf1b857377c09eb02adf
I'll close this issue for now, if you see any typo or want to improve the section help yourself !


Buttons, links, validation, etc.

This PR addresses #296 and #297.
I've only done work for event-related pages. If they're okay, I'll complete other pages, too.
Screenshots
"New object" button was changed (green color, no "Add a" text):

Event details page now contains "Edit" and "Delete event" buttons:

Events without URL show "Error" button with explanation:

Events with URL show "Validate" button that opens a modal window with validation results inside:

Known issues
Validation on fake URL triggers 500 Server Error.

:+1: please add similar buttons for sites, persons, tasks, airports, and merge in as soon as you're done - thanks!

I'll merge when I fix the bug (I forgot about it) tomorrow.


Corrected spelling mistake - fixed issue 67

Pull Request
Title

Description

Corrected the spelling of follow
Related Issues

Fixes #67
Changes Made
bug fix
Checklist

[x] I have tested the changes locally
[x] Documentation has been updated (if necessary)
[x] Changes are backward-compatible

Screenshots (if applicable)

Additional Notes

Footer

@subinoybiswas
Kindly assign me this issue and add labels
Review and merge


FIPS Self-test

This commit implements a MB FIPS command which perform the required FIPS Self-Tests.

https://github.com/chipsalliance/caliptra-sw/issues/321

@jhand2 , @korran  : Added a commit to remove LMS and SHA-1 until we confirm with John there is no issue with adding firmware based crypto algorithms in RT KAT.


Adjust "Map Data" options on the right menu

Reasoning:

it is still connected to osm-options
we can use it in a better way

Things to do:

[ ] check these fancy buttons when done
[x] remove section "Data Layers"
[x] set option "Full Fill" as default value in section "Fill Areas"
[x] change possible options in section "Map Features" to "Plant" (all single points), "Connection" (all lines), "Region" (all hub-areas)
[ ] add section "Energy Sector" with the possible option "Heat" (everything with tag energy_sector == heat") and "Electricity" (everything with tag energy_sector == electricity
[ ] make Map Features selection and Energy Sector selection work correctly together (together they form relevant subsets to displayed being the intersection of type (point, line, polygon) and sector (electricity, heat)

one should be able to decide if polygons/lines should be colored at all or not. and if so, which of the two and for what tag value.

Issue moved to energiekollektiv/openmod.sh #21 via ZenHub


404 at /config/containers/bridges/

URL: /config/containers/bridges/

One more, thanks! Fixing. In the meantime: https://docs.docker.com/network/bridge/


Error when switching audio language on safari

What version of Shaka Player are you using?
The latest shaka player version : v4.10.13
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Both
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Safari on MacOS (not reproduced on Chrome)
What are the manifest and license server URIs?
I'll send you the manifest URI by mail.
What configuration are you using?  What is the output of player.getConfiguration()?
I reproduce the error with the demo app.
What did you do?

Start the stream (ok)
Change the audio language

What did you expect to happen?
Video continue with the other audio language.
What actually happened?
The video stopped and there is no way to continue.
Are you planning send a PR to fix it?
No

Duplicate of https://github.com/shaka-project/shaka-player/issues/3243

It's not exactly a duplicate of #3243
I can reproduce with a HLS stream too. I'll send you an other manifest URI.

See: https://github.com/WebKit/WebKit/pull/32938

It can actually happen with DASH, HLS, MS, anyone using MSE


Delete unused frames

These frames are either completely unused or only used in reporting a status message. Let's delete them for now. I know we might need to reimplement some of them to handle different tool transforms later.
Merge with (https://github.com/SchillingRobotics/schilling_manipulators/pull/287) to avoid build errors.

Added as part of #30


Rename a wishlist

Add the rename wishlist function.

Codecov Report

Merging #72 (8f77773) into main (f7227ab) will decrease coverage by 0.11%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
- Coverage   97.13%   97.01%   -0.12%
==========================================
  Files           7        7
  Lines         489      503      +14
==========================================
+ Hits          475      488      +13
- Misses         14       15       +1

Impacted Files
Coverage Δ

service/routes.py
97.93% <92.85%> (-0.55%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7227ab...8f77773. Read the comment docs.

Fixed the Test coverage.

IBM Cloud toolchain: Delivery Pipeline deployed wishlists to prod, including commit 9573027a887106dedbeb3946fc37b217965111dc


(macOS) All Text appears to be shifted a few pixels up

Comparisons between macOS and Windows in mat1jaczyyy/apollo-studio:

The difference is pretty obvious and I'd say it gives my app a completely different, unnatural feel on macOS.

We can clearly see that that you don't use the same font for your App on all platforms. To make sure you use the same font you have to use an embedded font that is shared on all platforms. Rendering quality can differ because on Window text is rendered by DirectWrite and on Mac FreeType is used. Pls check if an embedded font works for you.
https://fonts.google.com/

I don't use a font at all in fact - I use whatever Avalonia throws at me by default. In my opinion, while the font doesn't necessarily need to match, the measures and positioning should.

If you don't specify a font you will use the one your system uses by default. Some fonts have different sizes for whatever reason. Now I am using NotoSans because it has the same size as Segoe UI (Default from Windows) and looks almost the same.

I've worked around this with Open Sans, but yeah, free to use any font we want. Close if you don't consider the inconsistent vertical positioning an issue.

We can keep this open until the new text layout has been merged. You can also try it out via myget pr feed if you want.

@Gillibald I believe this can be closed. I also can see this issue on mac.


cp: debug with --update=none should show 'skipped'

Should fix tests/cp/debug.sh

Congrats! The gnu test tests/cp/debug is no longer failing!

Great :)


Add the pip package testresources for testing

Eliminate the "ERROR: launchpadlib 1.10.13 requires testresources, which is not installed", in Ubuntu 20.04
Seems like the error won't affect the succeeding installation, but it's always better to make errors disappear :)

Eliminate the "ERROR: launchpadlib 1.10.13 requires testresources, which is not installed", in Ubuntu 20.04

Hm, weird.  As far as I know, we don't use either testresources nor launchpadlib inside of a ROS 2 installation.  Could you provide more information as to where this error is coming from?

I tried to install ROS2 by following the tutorial Building ROS2 on Linux on a fresh Ubuntu 20.04(not 20.04.1) Desktop.
Btw, I'm out of office these days and can't access my own desktop computer, so I also try it with a Ubuntu Server 20.04 cloud server, it also complains the same error.
I'll take a look into this, pretty weird.

@crystaldust Friendly ping, any luck in reproducing this problem?  If so, can you provide steps so I can look into it?  If not, maybe we should close this PR.

I'm going to go ahead and close this one out.  If you see this issue again, please feel free to open a new PR and provide detailed instructions on how you got there.  I'd like to eventually try to figure out where the dependency came from.  Thanks!

I was actually able to eventually reproduce this on a fresh install with the instructions from source.  It turns out that the problem is because of the upgrade of setuptools:
$ python3 -m pip install -U setuptools
Collecting setuptools
  Using cached setuptools-57.0.0-py3-none-any.whl (821 kB)
ERROR: launchpadlib 1.10.13 requires testresources, which is not installed.
Installing collected packages: setuptools
Successfully installed setuptools-57.0.0

This definitely seems like a problem with setuptools itself, rather than in ROS 2.
However, it does beg the question of whether we should really be instructing people to upgrade their setuptools version.  Given that things largely work in the from-debian install case, we probably don't need that.  But we would need to do additional testing to see if that is a feasible option.  I'm going to open a separate issue to investigate whether we need to update setuptools.


Previous and Next Pager Not Showing Up

I am unable to add Prev and Next Pager. I did npm update @reflexjs/gatsby-theme-post and followed this guide:
https://reflexjs.org/docs/blog-recipes/#previous-and-next-pager

Can you paste in the whole content of post-footer.mdx?
There's an issue in MDX where leaving spaces between lines causes an error. Make sure you don't have spaces between your JSX.

export const PostFooter = ({ previous, next }) => (
  <Container maxW="null|null|null|900px">
    <Flexbox justifyContent="space-between">
      {previous && (
        <PagerLink>

            <Icon name="arrow-left" mr="2" /> Previous

          <PagerTitle title={previous.title} />
        </PagerLink>
      )}
      {next && (
        <PagerLink ml="auto" textAlign="right" alignItems="flex-end">

            Next <Icon name="arrow-right" ml="2" />

          <PagerTitle title={next.title} />
        </PagerLink>
      )}
    </Flexbox>
  </Container>
)

export const PagerLink = ({ link, children, ...props }) => (
  <ButtonLink
    href={link}
    d="flex"
    flexDirection="column"
    alignItems="flex-start"
    bg="transparent"
    rounded="md"
    border="0"
    p="4"
    w="40%"
    hoverBg="muted"
    {...props}
  >
    {children}
  </ButtonLink>
)

export const PagerTitle = ({ title }) => (

    {title}

)

Can you try the following:
export const PostFooter = ({ previous, next }) => (
  <Container maxW="null|null|null|900px">
    <Flexbox justifyContent="space-between">
      {previous && (
        <PagerLink href={previous.slug}>

            <Icon name="arrow-left" mr="2" /> Previous

          <PagerTitle title={previous.title} />
        </PagerLink>
      )}
      {next && (
        <PagerLink
          href={next.slug}
          ml="auto"
          textAlign="right"
          alignItems="flex-end"
        >

            Next <Icon name="arrow-right" ml="2" />

          <PagerTitle title={next.title} />
        </PagerLink>
      )}
    </Flexbox>
  </Container>
)

export const PagerLink = ({ children, ...props }) => (
  <ButtonLink
    d="flex"
    flexDirection="column"
    alignItems="flex-start"
    bg="transparent"
    rounded="md"
    border="0"
    p="4"
    w="40%"
    hoverBg="muted"
    {...props}
  >
    {children}
  </ButtonLink>
)

export const PagerTitle = ({ title }) => (

    {title}

)

Its not showing up yet.

I tried to add tags counter following this issue: https://github.com/reflexjs/reflex/issues/29 but the posts-header.mdx was not visible as well.

Can you try removing npm remove @reflexjs/gatsby-theme-post and install it again?

I did that and updated npm update @reflexjs/gatsby-theme-post, however, I got this error while using gatsby develop.

Error:

Couldn't find the "@reflexjs/gatsby-theme-post" plugin declared in "/home/amanbhattarai/mysite/gatsby-config.js".
Tried looking for a local plugin in /home/amanbhattarai/mysite/plugins/@reflexjs/gatsby-theme-post.
Tried looking for an installed package in the following paths:

/home/amanbhattarai/mysite/node_modules/gatsby/dist/bootstrap/load-themes/node_modules/@reflexjs/gatsby-theme-post
/home/amanbhattarai/mysite/node_modules/gatsby/dist/bootstrap/node_modules/@reflexjs/gatsby-theme-post
/home/amanbhattarai/mysite/node_modules/gatsby/dist/node_modules/@reflexjs/gatsby-theme-post
/home/amanbhattarai/mysite/node_modules/gatsby/node_modules/@reflexjs/gatsby-theme-post
/home/amanbhattarai/mysite/node_modules/@reflexjs/gatsby-theme-post
/home/amanbhattarai/node_modules/@reflexjs/gatsby-theme-post
/home/node_modules/@reflexjs/gatsby-theme-post
/node_modules/@reflexjs/gatsby-theme-post

not finished open and validate gatsby-configs - 0.194s

I'm getting this error :

ERROR #98123  WEBPACK
Generating SSR bundle failed
Unexpected token (17:6)
File: node_modules/@reflexjs/gatsby-plugin-image/src/image.js:17:6
not finished Generating image thumbnails - 107.193s

@amanbhattarai4400 Did you figure this out? Did this start happening after the gatsby-theme-post update?

I tried every possible options, but none of them seemed to work. Then I created a new Base Starter and shifted everything there, and installed previously installed plugins.
Now, everything runs smoothly, no errors and prev-next pager is showing up as well. Maybe that project got corrupted.
Thanks :)


Drop NodeJS <9 support

NodeJS 9 reach EOL at 2018-06-30 . See https://github.com/nodejs/Release .

Ups,NodeJS 8 reach EOL at December 2019 . Too early.

Again, thanks for the contribution, but I don't see why this one is a good idea. If it becomes a burden to support old versions then sure, but I wouldn't drop support just for the sake of it.


Updated Big-O Notation.markdown

Checklist

[x] I've looked at the contribution guidelines.
[x] This pull request is complete and ready for review.

Description
Fixes #632. It improves the Big-O Notation.markdown by providing code snippets to better understand the topic.

Just noticed this is in your repo, hehe. Don't forget to make a PR to our main repo.

Done. Thanks a lot!


ADD: Data Structure for generate ManifestParser to take in

Add a struct to store all the variables, not that familiar with cpp, partially implemented it in a Java way ; )

data structures already created.


Fix resource group name for the cluster

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

/kind bug
What this PR does / why we need it:
The prefix from DNS is not always same with resource group name. We need
to get it from providerID.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Release note:
NONE

/assign @ritazh @andyzhangx


Bitly gem vs bare metal get

In #5, @philnash points out wisely that the code base now:

has an unnecessary dependency on the Bitly gem,
does not adequately deal with the resulting network exceptions that Net::HTTP could generate.

I think a short-term solution is to remove the dependency, and comment in the code on the above disadvantage of this choice, so that students understand the pros and cons.
Long-term, we could use the gem and also rescue from its own exceptions/properly check for errors. That would also be something we should illustrate with all API calls.
Probably best to handle after the 11/4 hackathon.

@philnash thanks :) @siruguri I'm okay with the short term solution

PR coming up
On Thu, Nov 3, 2016 at 7:09 PM, Sanderfer<EMAIL_ADDRESS>wrote:

@philnash https://github.com/philnash thanks :) @siruguri
https://github.com/siruguri I'm okay with the short term solution
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/akanshmurthy/railsbridge-hackathon/issues/18#issuecomment-258327551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACezRQ2il1kO2Gx-XjCDF1Gu18M2gyKks5q6pPsgaJpZM4KpDvG
.

ok ill keep an eye out

@siruguri  ok PR merged, I'll keep this issue open so we can work on the long-term solution in the future


Segment and track all objects in the video

It is possible to segment and track all objects automatically in the video?

+1

+1


Submodule links not recognized by web ui when SSH_DOMAIN is used

Gitea version (or commit ref): 1.12.4

Description
When using a different domain for ssh access, submodules links are not correct within the gitea web interface.
Having a .gitmodules set up like this:
...
url =<EMAIL_ADDRESS>
and a gitea instance set up with https://git.martb.eu/ with its SSH_DOMAIN set to ssh.martb.eu leads to ssh. being used as a http href instead of gitea translating it to the "known" git. domain.
Affected lines
https://github.com/go-gitea/gitea/blob/332dbe73935ba960d8769bfc20b8d275922e4137/templates/repo/view_list.tmpl#L50
Screenshots

I just noticed that this is a duplicate of https://github.com/go-gitea/gitea/issues/9756, any news regarding this?

@MartB I think it is similar or related ... throu I think we can keep both open

there was done some work on submodule recognizion in the master branch some time ago , so it has to be checked if this is still present on master ...

@MartB could you confirm that this is present on try?

@zeripath I don't think you can check this on try because it doesn't use a different domain name for SSH_HOST and DOMAIN
The issue is that we use AppUrl as the urlPrefix value here:
https://github.com/go-gitea/gitea/blob/ad2bf376dfd934394cad46c3ff3e022ca232958f/modules/git/submodule.go#L42
So this will never match:
https://github.com/go-gitea/gitea/blob/ad2bf376dfd934394cad46c3ff3e022ca232958f/modules/git/submodule.go#L79
Because they use a different hostname in the submodule file that matches the separate hostname they have used for SSH_DOMAIN. So our logic thinks this is hosted elsewhere but really it is still the same copy of Gitea just with a different hostname for SSH_DOMAIN.
I think the issue here is that we need to also check if the hostname found in the submodule file matches the SSH_DOMAIN value and then create the link with the AppUrl instead.

Thanks for looking into that!


Can multiple directories be configured with the storage.tsdb.path option

I use docker swarm to create Prometheus service. I create multiple copies for it on the same node, but a directory can only be mounted by one. Therefore, can multiple data directories be configured

Thanks for your report. It looks as if this is actually a question about usage and not development.
It makes more sense to ask questions like this on the prometheus-users mailing list or our forums rather than in a GitHub issue. In our community channels, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Overriding highlight groups set by integrations

Is your feature request related to a problem? Please describe.
Hi, I'm trying to override some highlight groups set by the telescope and leap integrations, but it seems like the highlight groups created by integrations are set after the user-defined groups.
Describe the solution you'd like
I would like to override custom highlight groups in by providing a table to the custom_highlights config option and have them replace the ones set by integrations.
custom_highlights = function(colors)
  return {
    LspInlayHint = { style = { "italic" } },
    -- lightspeed-style highlighting for leap
    LeapMatch = { fg = colors.text, bold = true },
    LeapLabelPrimary = { fg = colors.pink, bold = true },
    LeapLabelSecondary = { fg = colors.blue, bold = true },
    -- hide telescope results title
    TelescopeResultsTitle = { fg = colors.mantle },
  }
end,

Describe alternatives you've considered
I've considered having an autocmd to set the highlight groups after catppuccin loads, or disabling the integration and manually setting all of the telescope highlight groups myself, but I would prefer being able to just override a few specific groups via custom_highlights.
Thanks!

This should already work if you're using highlight_overrides :thinking:
highlight_overrides = {
  all = function(colors)
    return {
      LspInlayHint = { style = { "italic" } },
      -- lightspeed-style highlighting for leap
      LeapMatch = { fg = colors.text, bold = true },
      LeapLabelPrimary = { fg = colors.pink, bold = true },
      LeapLabelSecondary = { fg = colors.blue, bold = true },
      -- hide telescope results title
      TelescopeResultsTitle = { fg = colors.mantle },
    }
  end,
},

custom_highlights is alias of highlight_overrides.all so there shouldn't be any difference

Can you provide more context, i.e. your telescope config

It works when I set telescope integration to true:

But not when I enable the "nvchad" style:
opts = {
  integrations = {
    telescope = { enabled = true, style = "nvchad"}
  }
}

As for my telescope config, I don't think it makes a difference, but the issue persists with a fairly minimal config:
return {
  "nvim-telescope/telescope.nvim",
  dependencies = {
    { "nvim-telescope/telescope-fzf-native.nvim", enabled = vim.fn.executable "make" == 1, build = "make" },
  },
  cmd = "telescope",
  opts = {},
  config = function(_, opts)
    require("telescope").load_extension("fzf")
  end
}

custom_highlights works similar to :highlight which extends the current highlight group. You should override the bg value too
TelescopeResultsTitle = { fg = colors.mantle, bg = colors.none }

Thank you so much!


Highlighted text in search bar is unreadable

As of title

FYI you can override this in settings.json with:
"workbench.colorCustomizations": {
  "selection.background": "#007acc"
}

Thank you!!


Update sandwich hand detector so it can also check for the black outline

Updated the detector to reduce false positives on lettuce. It checks that the matched waterfill object contains the black outline.

for waterfill objects that matched the hand template, apply a mask filter to retain all black components
if this is a true positive, waterfill_sub_binary_image should be a black outline of the hand.
if false positive, it will be a pure white image
then apply the waterfill+template match algorithm on the resulting waterfill_sub_binary_image
this helps to remove false positives (e.g. lettuce)

SandwichHandLocator::detect will still use the old method of detection, for performance reasons (I didn't test it, but I imagine that adding an extra round of waterfill matching will worsen the lag in an already laggy program).
So, I think that detecting the black outline should only be done for recovery routines.
I recommend merging #428 first.

Closing for now, since this doesn't seem to be an issue with the new sandwich hand template.


Include NWS Near-term Forecasts with production runs

Brian pointed out this news item regarding the WI DNR doing a runoff risk model based on NWS RFC data.  Can something similar be done with DEP?
The primary issue is that we don't have a good source of high-temporal resolution forecast precipitation data, so getting the detachment/soil loss numbers right would be tough.  But, if we only cared about runoff, having hourly or three-hourly precip input would likely be good enough.  From some old sensitivity work I did, it seemed that WEPP was mostly sensitive to daily precip totals and not sub-hourly intensities when it came to runoff values.
Implementation
Currently, DEP runs for dates into the 'future' by simply repeating a previous years data to fill out the current year. So, in theory, we could have a process that merges a NWS forecast to replace the upcoming few days within the CLI files.  The realtime runs ignore any dates into the future, so we could have a secondary post-processor that extracts the runoff data.
The actual computation of a 'risk' and/or visualization of this data would need further discussion.

Links to nearby state sites for forecast runoff risk
MN: https://mnag.maps.arcgis.com/apps/MapSeries/index.html?appid=7a6538ffc8994715ba668f0579fe53a2
WI: http://www.manureadvisorysystem.wi.gov/runoffrisk/index


Explicitly set https required

Purpose
Fixes #580
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Raised the false-positive report here https://github.com/Azure/PSRule.Rules.Azure/issues/2414


WIP: Add 'deployment metadata' to cdappconfig

So that the ClowdApp is able to know version info about its deployments via cdappconfig.json
This will also allow us to share the configs of all cdapps within the "/cdenv/cdenvconfig.json" on the IQE ClowdJobInvocation -- which will allow IQE to fetch deployment information about each ClowdApp (such as image/image tag being used by the deployments) without having to query the k8s API itself.

We have discussed using a metadata field into cdappconfig
metadata:
  deployments:
    name:
      image: some image ref

Example cdappconfigs presented to the IQE pod now with 2 clowdapps running in the environment (host-inventory and rbac):
bash-4.4$ cat cdenvconfig.json | jq
{
  "cdappconfigs": {
    "host-inventory": {
      "metadata": {
        "deployments": [
          {
            "name": "service",
            "image": "quay.io/cloudservices/insights-inventory:0d0343e"
          },
          {
            "name": "mq-pmin",
            "image": "quay.io/cloudservices/insights-inventory:0d0343e"
          },
          {
            "name": "mq-p1",
            "image": "quay.io/cloudservices/insights-inventory:0d0343e"
          },
          {
            "name": "mq-sp",
            "image": "quay.io/cloudservices/insights-inventory:0d0343e"
          }
        ]
      },
<...snipped...>
    "rbac": {
      "metadata": {
        "deployments": [
          {
            "name": "service",
            "image": "quay.io/cloudservices/rbac:2fd2815"
          },
          {
            "name": "worker-service",
            "image": "quay.io/cloudservices/rbac:2fd2815"
          },
          {
            "name": "scheduler-service",
            "image": "quay.io/cloudservices/rbac:2fd2815"
          }
        ]
<...snipped...>


Update Jackson to latest version

Expected Behaviour
Need latest version for stability (2.9.9+)
Actual Behaviour
Currently have 2.9.8
Reproduce Scenario (including but not limited to)
Examine Pom
Steps to Reproduce
Examine Pom
Platform and Version
All
Sample Code that illustrates the problem
N/A
Logs taken while reproducing problem
N/A

Complete


Cannot find Ruff

Describe the bug
Ruff plugin "Cannot find Ruff" (error window with that message)
To Reproduce

Configure plugin to "Run ruff when the python file is saved"
Save a file

Expected behavior
Ruff being run
Environments (please complete the following information):

IDE: PyCharm 2022.3.2 (Community Edition)
OS: Windows 10
Ruff Version: 0.0.239 or 0.0.244
Plugin version: 0.0.6

Additional context
I am using conda environments and ruff is installed in my development environment for that project (and works fine on the command-line when that environment is activated). Would it be possible that your plugin does not support conda environments?
I do not think this is relevant, but since ruff is not available in anaconda yet, I install it via pip in my conda environment.

@gdementen
Thank you for creating the issue.
Have you tried 0.0.7 which is support Windows and user sites?
Unfotually, I have not used conda environment yet.
Can you share the path of your ruff path?

Sorry I didn't see the closed issue. I haven't not tried yet the new version, I don't think it was available when I tried. I will try that hopefully next week.

The path of ruff on my installation is IIRC: c:\[somewhere]\Miniconda3\envs\dev310\Scripts\ruff.exe

I just tried here at home and the new version does not seem to make a difference for me.
I also tried to install ruff in the base conda environment but it does not seem to work either.
I think the easiest fix would be to let the user configure the path explicitly (ideally would still autodetect when left blank for those users where that works).

I will check how to get conda path in PyCharm.

I think the easiest fix would be to let the user configure the path explicitly (ideally would still autodetect when left blank for those users where that works).

Great idea. I will think the way.
Thank you very much.

FWIW, I just tested the version 0.9. It still does not autodetect ruff in my project environment, but the global path settings works very nicely. Thanks for the nice plugin and quick reaction !


fix: Cannot read properties of undefined (reading 'run')

Fixes the error "Cannot read properties of undefined (reading 'run')" when using an incorrect command.

Nice! Thanks for your contribution!


Add tests

Changes Unknown when pulling 5ce8e38d858411d7464c995d2ea7bd06bd025b3a on tests into ** on master**.

Changes Unknown when pulling 3edaf73ce8b9eddb926be3ca50c57097a587410e on tests into ** on master**.

Changes Unknown when pulling a97bdacf63de87489c173dae0889280486a41631 on tests into ** on master**.

Changes Unknown when pulling 7a791004c8a3933dcb218e4a69ec45020ee59e66 on tests into ** on master**.

Changes Unknown when pulling f1dd6369046abe9ea15a21f1f7ec77c924957655 on tests into ** on master**.

Changes Unknown when pulling f4452bf0031a19f66282a977a261f79e990dae08 on tests into ** on master**.

Changes Unknown when pulling 6677d3b8344f4302b2e36dab07ad990ab51989c0 on tests into ** on master**.

Changes Unknown when pulling 6aa8ad3327ff0f8bb2cdbd5f7fb9080097cd344c on tests into ** on master**.

Changes Unknown when pulling 6aa8ad3327ff0f8bb2cdbd5f7fb9080097cd344c on tests into ** on master**.

Changes Unknown when pulling 02983ab181f4306e6c14900e97183678e6f808fb on tests into ** on master**.

Changes Unknown when pulling 4cf9b75e23e50245f11fae091f5b5a2b3888f718 on tests into ** on master**.

Changes Unknown when pulling 4e25f3cb3665d99867f569a79294306e73342de3 on tests into ** on master**.

Changes Unknown when pulling 29fc7f3d14b7f97e93730c9f331dedbfc03c5990 on tests into ** on master**.

Changes Unknown when pulling 7fe78d1e2fa1baf0e6332f673e6c56a2952b101f on tests into ** on master**.

Changes Unknown when pulling c00270e9016b272c2433581cae6a139af3df6879 on tests into ** on master**.

Changes Unknown when pulling ea50a0f846cf0b5506ab3d9fcd4d1e0d1e4e556a on tests into ** on master**.

Changes Unknown when pulling ea50a0f846cf0b5506ab3d9fcd4d1e0d1e4e556a on tests into ** on master**.

Changes Unknown when pulling af9238a062f5010988f575f36cc40a297de9acc9 on tests into ** on master**.

Changes Unknown when pulling e6c7e22a62806334334aeb810e524f1d119ac430 on tests into ** on master**.

Changes Unknown when pulling e6c7e22a62806334334aeb810e524f1d119ac430 on tests into ** on master**.

Changes Unknown when pulling c21567ee1ff457dc56a9b98b395b70ce372de38e on tests into ** on master**.

Changes Unknown when pulling f6110fdb1119c9756bfd320f00c941f525576635 on tests into ** on master**.

Changes Unknown when pulling 20029c6e5908db6e57be103211e7de8b066dac34 on tests into ** on master**.

Changes Unknown when pulling 20029c6e5908db6e57be103211e7de8b066dac34 on tests into ** on master**.

Changes Unknown when pulling c9b7902fa221a2b8a748770065497260bfe31eb4 on tests into ** on master**.

@u9520107 @embbnux  Please review, I would like to merge.

Changes Unknown when pulling c38decbc84f77d4524e5509480da6e25c311e158 on tests into ** on master**.

Changes Unknown when pulling 401a19706cd5a40c937b7f3b580ec92de3953c15 on tests into ** on master**.

Changes Unknown when pulling 2dfdd16ca0f2cc6212098b0fbdc05ddaafaef40d on tests into ** on master**.

Changes Unknown when pulling e4a65e36a547dad311c56a7c0eb7dc57e5b08f61 on tests into ** on master**.

Changes Unknown when pulling fc62d0d33f7592085685dcea1978181dcfd0e945 on tests into ** on master**.

Changes Unknown when pulling 5af7fb973b99e9be0352180686bfd1519b2a56cb on tests into ** on master**.

Changes Unknown when pulling c1e950eef9f752b50cf79fc9c8fb0d7e99ff5199 on tests into ** on master**.

Changes Unknown when pulling fa283d29edca52ba82df9ad2a393cd6084bc6d18 on tests into ** on master**.

Changes Unknown when pulling b8eb9a5fc99a57891366e624a17745cfdb61a158 on tests into ** on master**.

Changes Unknown when pulling c3cc374024dfceae273919f5a15e6568266ac514 on tests into ** on master**.

Changes Unknown when pulling 3707cd46958deeeb9b2ebb19f1dcb2d92ec84722 on tests into ** on master**.

Changes Unknown when pulling a75e9cdbb49c89163de93223e231fa0e2faba0e5 on tests into ** on master**.

Changes Unknown when pulling e198424acffb0216ee8804f080e37be7dbc640d1 on tests into ** on master**.