Schedule Disbursements

Create a new interface to schedule disbursements in an automated fashion, similar to schedule payments and/or manage allocations

[ ] Determine Rollups Needed
[ ] What field(s) need to account for new Rollups
[ ] What changes to interface code need to be made to account for pending disbursements
[ ] Commit Code to Outbound Funds Core Package

Code has been committed to the branch.
Work Completed

Created New Rollup Field for Unpaid Disbursements
Updated Total Disbursements formula field =  Awarded - Paid Disbursements - Unpaid Disbursements
Updated field labels and descriptions on Total Remaining and Total Disbursed fields.

Can close issue as create disbursement branch has been merged into master.


get-started video doesn't add value

Problem description
Under https://www.pulumi.com/docs/get-started/ the first link is “infrastructure as code” which contains a 7 minute video that doesn’t seem to add much to the text summary on the page. This seems to be adding more friction to getting started quickly.
Suggestions for a fix
Remove video
Thanks @aaijazi

btw I recently saw some other dev tool (can't remember which one), but the docs there explicitly called out that the video and text were just two different ways of getting the same information. It said something like "if you would prefer to have this information in video form instead, here a video that explains the same concepts". I really appreciated that, and I think that could be a good compromise to just removing the video.


Add support for EnvoyRatelimit service for HTTP

Mixer V1 supports memquota / redisQuota rate limits. When MixerV1 is deprecated, this function should be provided by Istio thru other means.
Istio is adding this support for thrift

This potentially involves API design
Documentation
Equivalence or otherwise between Mixer ratelimits and Envoy.

I noticed that the milestone for this work used to be "1.6", but got then de-prioritized to "backlog". Are you able to share any information on potential release schedule for this feature?

We are adding it to 1.7 .

@mandarjog as this topic got closed due to inactivity, I was wondering if it is still scheduled to be released as part of 1.7?

@mandarjog  – just reaching out to get an update here. Is support for HTTP still on track to be in 1.8?
If not, I will use one of the suggested workarounds.
Thanks in advance.

@jgreens we do not have an Istio API for it (yet?), however we have tests and examples of how to do it.
@gargnupur can you point to the PRs that you have done to document how to do it?

Thank you @mandarjog! I really appreciate it.
I'm guessing that the version I need to upgrade to will be apparent in the examples. I am on 1.6.4 now.

We have a pr for tests currently: https://github.com/istio/istio/pull/28888
Will add the PR for documentation soon and will post it here too...
@jgreens : You should be able to use global rate limiting on 1.6.4

Thank you @gargnupur!

I have designed an API, appreciate if anyone can leave your comments
https://docs.google.com/document/d/1628LHcwuCvTRFhk8rsQKQUmgCxnY6loPFSxliADQDIc/edit#

Documentation PR: https://github.com/istio/istio.io/pull/8537

I'm trying to understand the rate limiting strategy for Istio moving forward: We used to leverage Istio rate limits in Istio <1.4. Since it got deprecated, we don't use rate limits anymore, but we would really like to/need them.
Is the decision being made that Istio will not provide an API for it, but instead Istio solutions should use Envoy rate limits as described in Enabling Rate Limits using Envoy?
Or are there still ongoing efforts to design/implement an Istio API (wrapping Envoy rate limits) to make it easier to configure for Istio users?

We do intend to provide an Istio api to wrap Envoy rate limit.
@hzxuzhonghu has the api doc linked above. Please take a look and comment there.

@mandarjog would it be possible to keep this item "open" to track progress and expected ETA? I understand there is the design doc @hzxuzhonghu shared. However, it would be nice to track it also in GitHub.

not stale


esult::unwrap()on anErr` value: "Unsupported OS: android"

How to build it for android?
I use it as a dependency and it does not work, while in README you mention that android is supported but just not tested.

Sorry, that error message should say "Microsoft does not provide prebuilt binaries for this platform. You'll need to build ONNX Runtime from source." I fixed this in v2 but forgot to backport the change.
Android is supported, you just need to compile ONNX Runtime from source. See ONNX Runtime docs and ort's docs to point it to your binaries.


Remove index.html detail

No Longer required to update the index.html because of the front page app

Looks good, I just also removed references to index.html and added a sentence reminding to update the front page app.


cannot find name=Public

pls  set headless flase and upload a screenrecording to help me debug

https://user-images.githubusercontent.com/43066683/156889316-5de290a2-1f8c-4198-892c-4e6febf93f3a.mp4
the title is wrong it doesnt delete that what is in the right,and for the thumbnail this channel is not verified thats why it doesnt go there,and it doesnt go to make account public but i might think the problem is at the title it gets longer because it doesnt erase the rest

i just removed len(text_fiel) and just putted 120 and now it works .it finds and the name=PUBLIC
for _ in range(120):
# more backspaces than needed just to be sure
title_field.send_keys(Keys.BACKSPACE)
sleep(0.1)

thanks for your input, you always can manually see what happens when headless is false and fix that
you definitely pre-process meta-field to meet youtube all kinds of limits
especially title,tags,description


Bundle analyzer Error: listen EADDRINUSE :::8888

Node version: v6.5.0
npm version: 3.10.3
Operating system: macos 10.12.1
Command line used: npm start
Steps to reproduce:
1, change some code under src.
2, save
3, Got the errmsg:
webpack: bundle is now VALID.

Unable to find bundle asset "/Users/jifeng/gitlab/cmbc/dist/bundle.js".
Analyzer will use module sizes from stats file.

events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE :::8888
    at Object.exports._errnoException (util.js:1026:11)
    at exports._exceptionWithHostPort (util.js:1049:20)
    at Server._listen2 (net.js:1253:14)
    at listen (net.js:1289:10)
    at Server.listen (net.js:1385:5)
    at EventEmitter.listen (/Users/jifeng/gitlab/cmbc/node_modules/webpack-bundle-analyzer/node_modules/express/lib/application.js:617:24)
    at Object.startServer (/Users/jifeng/gitlab/cmbc/node_modules/webpack-bundle-analyzer/lib/viewer.js:57:14)
    at BundleAnalyzerPlugin.startAnalyzerServer (/Users/jifeng/gitlab/cmbc/node_modules/webpack-bundle-analyzer/lib/BundleAnalyzerPlugin.js:78:12)
    at analyzeFn (/Users/jifeng/gitlab/cmbc/node_modules/webpack-bundle-analyzer/lib/BundleAnalyzerPlugin.js:57:24)
    at Timeout._onTimeout (/Users/jifeng/gitlab/cmbc/node_modules/webpack-bundle-analyzer/lib/BundleAnalyzerPlugin.js:69:11)
    at tryOnTimeout (timers.js:232:11)
    at Timer.listOnTimeout (timers.js:202:5)

This just means you already have something running on port 3000 (likely the same project, running in another window) :)

Same problem, nothing is running on port 8888 (analyzer uses this port actually)

Hrmm. I've reopened this to see if we can figure out a root cause. @iyel, if you specify a different port, does it work for you?
Here's how:
Change this line to specify a port:
new BundleAnalyzerPlugin({
  // Port that will be used by in `server` mode to start HTTP server.
  analyzerPort: 4000,
})

More info

@coryhouse I've specified the different port, error persists
Error: listen EADDRINUSE :::4000

@iyel Have you check to confirm that there is not an outstanding process in the background that is hung? Here is a SO that may help. The first command will list all processes that have node in them and if needed the second command will kill them off. I have had to start using this recently after updating my MBP to Sierra

@iyel, I checked this as well and even after doing edits and hot reloading kicks in there is no error. The only time this happens is if something is running on the port that it's complaining about like Kyle and Cory mention, e.g. running react-slingshot's npm start when an npm start is still running.

looks like new BundleAnalyzerPlugin() causes the error. By commenting the line https://github.com/coryhouse/react-slingshot/blob/master/webpack.config.dev.js#L41 there will be no crash and no analyzer

Interesting. I can't seem to reproduce. @cpietsch, can you share the following:
npm version
node version
OS
Command line that you're using (built into an editor, or OS' native)

npm 3.10.8
node v6.9.1
osx terminal

Same issue, I believe the BundleAnalyzerPlugin isn't closing the port.

OK, so @falconmick and co., the issue is you're running npm start and while it's still running, you're running another npm start for react-slingshot and at that point the port is in use like I mentioned above in a previous comment?

@falconmick, @cpietsch, @iyel , I definitely get the error then, but I wonder if this is a non-issue as you're running npm start while another is still running. @coryhouse, @kwelch thoughts?

Another option could be to just run the bundle analyzer when running a different npm script.

I'd prefer the bundle analyzer run as a separate script and would welcome a pr.

@coryhouse, I'll get a PR for this-this weekend probably unless there are any OSS peeps that would like to tackle this before then.

I wasn't running another instance. I would be running npm start -s and as soon as I got an error in my code it would try to rebind the bundle analyser (I did nothing) which then caused it to fail due to port not available

On 10/11/2016, at 4:41 PM, Nick Taylor<EMAIL_ADDRESS>wrote:
@coryhouse, I'll get a PR for this-this weekend probably unless there are any OSS peeps that would like to tackle this before then.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

same issue @coryhouse @nickytonline
changed the port , still issue exists.
and No i am not running another terminal instance or any other npm start
just like @falconmick

I a wrapping up a project tonight. I should be available after or this weekend to look into this a bit.

I've removed bundleAnalyzerPlugin for now since it's apparently buggy on some machines. Needs to be implemented as a separate npm script anyway since displaying every time the app opens is overkill. https://github.com/coryhouse/react-slingshot/commit/47f020c0c416f6d20bbbb5c54f92d4b45c42af28

Cheers
Sent from my iPhone

On 11/11/2016, at 4:20 PM, Cory House<EMAIL_ADDRESS>wrote:
I've removed bundleAnalyzerPlugin for now since it's apparently buggy on some machines. Needs to be implemented as a separate npm script anyway since displaying every time the app opens is overkill. 47f020c
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@coryhouse Is this something we want to support, if we are able to find a fix for this?

Yeah, I'm a fan of the feature. But I think a dedicated bundle-analyzer script makes sense.

I got this to work by modifying the config options passed into the plugin:
new BundleAnalyzerPlugin({analyzerMode: 'static'})
This launches the analyzer page against the generated data file rather than the bundle itself, and works well to get past the issue when using hot reloading.
Good luck!

Good stuff @mlwigdahl! Thanks for sharing. I still feel a separate npm script makes sense to avoid always opening a report that is only worth looking at occasionally. Anyone strongly disagree?

@coryhouse, I'm still on board with a separate script as it really is something you don't need to run all the time.

good job!! @nickytonline @mlwigdahl

i'm a soft enigner of chinese, i got the same error when my project run。 tks the answer of this list。

I got this to work by modifying the config options passed into the plugin:
new BundleAnalyzerPlugin({analyzerMode: 'static'})
This launches the analyzer page against the generated data file rather than the bundle itself, and works well to get past the issue when using hot reloading.
Good luck!

well done。

I'm going to leave this here for anyone using the Vue's modern mode build with webpack's BundleAnalyzerPlugin.
Problem
I received this error when using the vue ui build task for production in --modern mode. Because it generates a legacy build and a modern build, the webpack bundle analyzer plugin fails on the second build.
Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:8888
    at Server.setupListenHandle [as _listen2] (net.js:1256:14)
    at listenInCluster (net.js:1304:12)
    at doListen (net.js:1443:7)
    at processTicksAndRejections (internal/process/next_tick.js:76:17)
Emitted 'error' event at:
    at emitErrorNT (net.js:1283:8)
    at processTicksAndRejections (internal/process/next_tick.js:76:17)
 ERROR  Error: Command failed<EMAIL_ADDRESS>build --mode production --dest dist --target app --modern --dashboard

Solution
// vue.config.js

const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin;

module.exports = {
  configureWebpack: {
    plugins: [
      new BundleAnalyzerPlugin({
        analyzerPort: process.env.VUE_CLI_MODERN_BUILD ? 8888 : 9999 // Prevents build errors when running --modern
      })
    ]
  }
}

Docs for modern mode
Vue CLI v3 Modern Mode Link

Since I'm falling into this old issue, I'll also add my word:
webpack-bundle-analyzer now seems to be included into vue-cli-service
So I explicitely uninstalled it
npm uninstall webpack-bundle-analyzer

And generated a report with
vue-cli-service build --report # Or include it as a script in package-json

The report should now be generated in src/build/report.html


Workers not paused when switching screens

Following the recent update to textual that disallowed instances in MODES, workers are not paused when switching screens resulting in unnecessary increased queries to the API.

Partially resolved in https://github.com/freqtrade/ftui/commit/5859126d1dc1b01a5c4a9edb1247b52d952d105d, but pausing of timers still seems problematic. Will continue investigating.


[BUG] Wrong ui size sorting

Describe the bug
A clear and concise description of what the bug is.
A picture say's more that thousands words 😉

To Reproduce
Steps to reproduce the behavior:

Create volumes of diffentent sizes
Click on volume size order
See wrong ordering

Expected behavior
As the json values from server are all in same magnitude, the uui should use that values for size sorting
Log
If applicable, add the Longhorn managers' log when the issue happens.
You can also attach a Support Bundle here. You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment:

Longhorn version: v1.1.0 (Helm)
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: k3s (not relevant)

Number of management node in the cluster: 3
Number of worker node in the cluster 1

Node config (not relevant)

OS type and version: ubuntu 20.04
CPU per node: 4-8
Memory per node: 4 manager / 8 worker
Disk type(e.g. SSD/NVMe): HDD / SSD
Network bandwidth between the nodes: 2x10GBit

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal

Additional context
I can send a pr

Looks like size is returned as string, so fo the browser will sort it as string here
Also actualSize sorting is wrong [here](https://github.com/longhorn/longhorn-ui/blob/f61d662f3623bc990c36c4afb39f412ad137c868/src/routes/volume/VolumeList.js#L198]. It's sorted by size and has wrong order too.

Looks like size is returned as string, so fo the browser will sort it as string here
Also actualSize sorting is wrong [here](https://github.com/longhorn/longhorn-ui/blob/f61d662f3623bc990c36c4afb39f412ad137c868/src/routes/volume/VolumeList.js#L198]. It's sorted by size and has wrong order too.

State should sort by robustness, if volume is attached here

State should sort by robustness, if volume is attached here

Validate with longhorn-ui:master 01/25/2021
Validation : Pass
The sorting based on the size is working fine.

Validate with longhorn-ui:master 01/25/2021
Validation : Pass
The sorting based on the size is working fine.


Show attending users after booking and payment

Now this information is left out.

Should be fixed as of current master


[Feature] Draw Crosshairs programmatically

Feature Description
I have multiple charts in different timeframes, and I'd like to draw the crosshairs on one chart's mouseover to the other charts.
To Do

setCrossHair(x,y)

yeah, this is useful for sync crosshair across different chart. upvote this as well


rework update cashflow

Updates the submodule ID for the CashFlow submodule.
Also pulls in some updates from devel.
Tests are under way and will be reported soon.

**** Test plugins/CashFlow.CashFlow_NPV | has changed outcome from <FAILED> to <OK> !!!!
**** Test plugins/CashFlow.CashFlow_NPVsearch | has changed outcome from <FAILED> to <OK> !!!!
**** Test plugins/CashFlow.CashFlow_PI | has changed outcome from <FAILED> to <OK> !!!!
*****************
Number of success first file:
472
Number of success second file:
476
NUMBER OF SUCCESSES INCREASED: +4
*********************************

So, all of these changes came from those merged into "devel".  Do we want to modify those changes in this PR or make a PR to change them on devel?

The changes are from devel, basically we need to create a new PR for devel to fix these comments. I think we can merge this PR as it is. @alfoa What do you think?

Good to me.

ok, good to go.


Form submitted state not correct in different situtations

Reproduction
https://formkit.link/9fc88976fc72d4d09f5deb2b1891abce
Describe the bug
In the Multi-Step form from the documentation examples, I saw that the submitted state on the main form is not updated when I click on the submit button at the end (I mean not the "next Step"-Button).
Environment
• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

I could very well be misunderstanding your issue or completely missing something, but I'm unable to reproduce any issues given your reproduction link. Could you please provide a little more information regarding the expected behavior and what you're seeing on your end?

I'm talking about the submitted flag inside the state object in the node context. For example when you take a look in the Vue Dev-Console or when you want to use the node to check if the form was submitted in one place.

Ok — this is resolved in beta 16


Any plan for 0.17.0?

Hi @ZieIony Good morning sir! Hope everything goes well for you.
Sorry to bother you but I just want to check is there any plan for 0.17.0 release? I saw you had updated the version to 0.17.0 in build.gradle in March, but haven't released it yet.
I am working on a project which depends heavily on Carbon, and I know I can checkout the code and build from my local, yes I did like this (because I need features in the latest version) and this has been working perfect for me for a long time in my application project, now I'll need to convert my project into a library, but in this way I am running into a trouble which is I cannot have the carbon.aar inside my aar file. Hence I am wondering if there is any plan for the 0.17.0 release? Thank you!

Well, I guess I could finally release it to Maven. I'll try to do that this weekend.

Sounds great! Thanks!

done

Awesome!! Thanks!!


General Question

Are this library can rotate when full screen?
Are this library can play video list in recycleview?
when scrolling and auto play

Full screen? I believe it doesn't support rotation, but I think you can help us add this feature :)
Yes, I use it this way
Not sure.

hi this features how long you will implement it? simple like facebook video which are list video that has enlarge icon on bottom right corner (landscape) and shrink back (potrait). If i click one of those video goes to list scroll video that can auto play when visible to view

I am not planning to implement it at all.


Upload time

In the site interface, it only shows the upload date not upload time. Can it be changed?

Yes, it can.


Use Java 17 for 1.20.1

Java 17 is the standard for 1.20.1.

I didn't notice this PR initially, already converted to java 17.


Htmx + safe attribute doesn't work

Have simple example of Elysia/html plugin, kita and htmx

Elysia endpoint

app.post('/clicked', ({ html }) => html(<script>alert("hacked!")</script>))

Component with htmx tags

const Component = () => (
	<button
		hx-post="/devtools/clicked"
		hx-trigger="click"
		hx-swap="innerHTML"
		safe
	>
		Click Me!
	</button>
)

results injecting and running js code...
maaaaaybe I need to sanitize it on server then?

Sanitizing on server works as expected. I just wanted to make sure it is how htmx works there or if it can be fixed

This is intended behavior. the safe attribute only has effects when the inner createElement call is made. No extra JS code is sent to the client, so there's no way to do that. You should use safe attribute where you would interpolate with user generated content, not as a html response sent from a server.

Thanks for the clarification ❤


Sanitize AC Names

Make sure we pass valid strings to NewACName
Fixes #12

Actually, could you please just export this in the upstream repository?

Should I also move sanitizeACName to upstream?

Sure, seems like a reasonable helper function. And a test table, please ;-)

appc/spec#179

Now that appc/spec#179 is merged we can merge this


App registration is not working

callback url is not saved it looks like both App Name and callback url inputs mapped to same model as both their IDs are "appName" can you please fix this as I keep getting 'Bad Request' with errordescription: Invalid Redirect URL

I first faced this issue like two months ago, I thought it would've been fixed by now...

Hi, thank you for your comment. Would you mind providing a bit more detail regarding your inquiry?

@UPSRahulBhatnagar The issue is when you try to set or update the Callback URL of an App and save nothing happens. The ajax request fails with this error:
You don't have permission to access "http://developer.ups.com/dpui/api/UpdateApplication" on this server.

Hi, thank you for the follow-up. This issue is being investigated and we will update the thread once we have a resolution.

Hi, we have resolved the issue. For similar inquiries, please feel free to contact the UPS Help Desk (https://developer.ups.com/contactus) for real-time debugging.


Add a simple date parser to editable datepicker

Proposal:

takes the format used by pickadate as its configuration
simple parsing of the current date entered, match format with value entered
maybe just throw an error if the value entered does not fit the configured format
pluggable(?) to enable users to bring their own parser - how would this work?
maybe add a placeholder showing the current format

#196

The pluggable parser could easily work in the same style as the equality matcher https://github.com/aurelia/binding/issues/94

So that would be something like...
.plugin('aurelia-materialize-bridge', bridge => { bridge.useAll().useDatePickerFormatter(myFormatter); });

And we would somehow get DatePicker to use myFormatter? :smile:

I was thinking more along the lines of simply specifying a function as a bindable that accepts a string and outputs a Date or null so something like:
md-datepicker="container: body; value.two-way: selectedValue; options.bind: advancedOptions"
 this.advancedOptions = {                     closeOnSelect: true,                     closeOnClear: true,                     selectYears: 50,                     editable: {},                     showIcon: false,                     parser: this.specialDateParser                 };
public specialDateParser( text: string ): Date {         if ( text === "Incredibly complex date" )             return new Date();//we aint got time fo'dat     }

But maybe a mix of both so that you're able to set a default parser globally and then allow yourself to override it with an inline bindable :)

I like the thought. :smile:


std.traits: Unqual strips qualifiers on structs with references that would break const/immutable.

lat7h reported this on 2009-12-02T12:35:58Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=3567
CC List

default_357-line (@FeepingCreature)
dsimcha
issues.dlang (@jmdavis)
schveiguy (@schveiguy)
timosesu

Description
Phobos' traits.d's Unqual! template (needed for several of its other templates) does not handle arrays well:

writeln(Unqual!(const(int[])).stringof); // writes const(int)[] not int[]

Adding the following near line 1188 of traits.d solves this problem:

template Unqual(T : U[], U) { alias U[] Unqual; }

schveiguy (@schveiguy) commented on 2018-06-18T14:37:57Z
This is likely going to be disruptive to fix, so many things use Unqual. But nonetheless, it's an important bug.

timosesu commented on 2018-07-12T14:36:46Z
(In reply to Steven Schveighoffer from comment #6)
> I'm changing the title accordingly. Perhaps we should actually close this
> bug and open another, as the whole issue has really been flipped around?

Perhaps you're looking for this? https://issues.dlang.org/show_bug.cgi?id=8338

lat7h commented on 2009-12-02T12:46:35Z
Then I propose this design decision ought to be made explicit in the documentation; "removes all qualifiers" seemed, to me, to imply removing *all* qualifiers, not just the top level.

timosesu commented on 2018-07-12T08:56:15Z
The fact the Unqual only "shallowly" unqualifies a type can be pretty annoying when dealing with templates and inout functions.

struct S
{
    int[] vals;
}

template Temp(T)
{
    class Temp
    {
        T t;

        this(T t)
        { this.t = t; }

        auto get() inout
        {
            static if (is(T == S))
            {
                import std.traits : Unqual;
                alias tType = Unqual!(typeof(t.vals));
                pragma(msg, tType); // inout(int)[]
                return new inout Temp!tType(t.vals); // ERROR: Can't create class Temp!(inout(int)[])
            }
        }
    }
}

unittest
{
    auto t = Temp!S;
    t.get();
}

Error: variable `onlineapp.Temp!(inout(int)[]).Temp.t` only parameters or stack based variables can be inout
Error: template instance `onlineapp.Temp!(inout(int)[])` error instantiating

In this case a complete stripping of qualifiers is required!

Related discussion<EMAIL_ADDRESS>

schveiguy (@schveiguy) commented on 2018-07-12T12:28:57Z
Note, the bug here in FeepingCreature's case is that it strips more than it should. It is supposed to be safe to use Unqual.

So the expectation that Unqual should remove all mutability modifiers is incorrect.

I'm changing the title accordingly. Perhaps we should actually close this bug and open another, as the whole issue has really been flipped around?

dsimcha commented on 2009-12-02T12:39:37Z
This behavior is by design.  Unqual is supposed to shallowly strip const, etc, not deeply strip it.

default_357-line (@FeepingCreature) commented on 2018-06-18T07:25:35Z
This bug needs a second look.

Consider this code:

struct S
{
  int* ipointer;
}

pragma(msg, (Unqual!(const S)).stringof);

As can be seen, Unqual has just deeply stripped const, not shallowly.

Either Unqual must be made to error when passed a const struct with reference, or Unqual must leave const(int[]) as int[].

The current state is broken.

bugzilla (@WalterBright) commented on 2019-12-12T11:45:34Z
*** Issue 8338 has been marked as a duplicate of this issue. ***

schveiguy (@schveiguy) commented on 2018-07-12T15:11:44Z
I think a trait "ForceUnqual" may be needed in some cases, but "Unqual" I have always expected to behave the way it does (except in the case of structs containing references, for which it should not strip anything).

Man, it would be nice to have a tail-modifier in D...

timosesu commented on 2018-07-12T15:02:58Z
(In reply to Steven Schveighoffer from comment #6)
> Note, the bug here in FeepingCreature's case is that it strips more than it
> should. It is supposed to be safe to use Unqual.
>
> So the expectation that Unqual should remove all mutability modifiers is
> incorrect.

Tried to find examples where a complete stripping of qualifiers would break something. I guess here it is really required that it only strips the "head" qualifier:

https://github.com/dlang/phobos/blob/90a8fc387f25f9bdfc2c6ad4508da63c523be670/std/array.d#L1465

Though this is not the subject of this issue.. Should I perhaps create a new issue for https://issues.dlang.org/show_bug.cgi?id=3567#c5 ? Perhaps asking for a new trait?


Add dockerfile and docker compose file

Adds a Dockerfile to easily compile and run the server on any platform.
Adds docker-compose to make starting the server as easy as: docker-compose up -d

You can also add a reference to docker hub to make starting the server even more convenient
Reference issues: #66

Docker container would be great, but I think it'd better to:

make it from local files instead of installing git,
use a production server.

Here's what I came up with:
FROM frolvlad/alpine-python3:latest

WORKDIR /app

ADD . /app
RUN pip3 install gunicorn
RUN pip3 install -e .

EXPOSE 8080

ENTRYPOINT ["gunicorn", "--bind", "<IP_ADDRESS>:8080", "youtube_dl_server.app:app"]

Though, at this point I doubt owner of this repo will merge anything...


Custom properties support

Prerequisites:

Is the functionality available in the GitHub UI? If so, please provide a link to information about the feature.

Organization owners can define custom properties in organization settings:
https://github.com/organizations/ORGANIZATION/settings/custom-properties
Custom properties can be added to individual repositories in their settings page:
https://github.com/ORGANIZATION/REPOSITORY/settings/custom-properties
Custom properties can also be edited in bulk in organization's settings:
https://github.com/organizations/ORGANIZATION/settings/custom-properties?tab=set-values
Custom Properties can be used as search filters and to categorize repositories for custom automations etc.

Is the functionality available through the GitHub API? If the functionality is available, please provide links to the
API documentation (https://developer.github.com/v3/) as well as the Octokit documentation (https://octokit.github.io/).

https://docs.github.com/en/rest/repos/custom-properties?apiVersion=2022-11-28
New Feature
Add new custom_properties section to .github/settings.yml:
repository:
...
    custom_properties:
        my_custom_property: some_value
        my_other_custom_property: some_other_value
...

App should sync repository's custom properties, removing those that are set in repository but not in settings.yml, adding those that are not yet set in repository & modifying the rest as needed.

Our use case for this: Have repository templates for different organization units that among other things set repository settings and certain custom properties conforming to unit's dev practises when repository is created or template is merged to existing repository.
The aim is to minimize things to remember when creating new repositories and to enable further automations for groups of repositories identified by their custom properties.
This app would by our initial assessment work very well for our intended purposes but lacks the support for custom properties.


[Request]: calendarView type

What feature would you like to add?
Change dynamically the calendar view type using a select input
Notes
Instead of setting the calendarView property on the widget class

Hi, thanks for the idea. I'll see what I can do. :)

Great ! thanks :)))))


NestedText.NET

Hello again. I just finished a .NET NestedText library.
In addition to dumping/loading (serializing/deserializing as it is usually called in .NET), the library includes a customizable formatter.
It passes all the official tests, except for two (inline_dict_20, inline_dict_22) for which I am explicitly skipping the validation of the error messages as I believe the expectation is inconsistent/incorrect. For reasons I don't understand, the expectations are just "expected value", instead of the usual form "expected ':', found '}'", which my library reports. This is all the more strange that the inline_dict_21 case differs only by whitespace (which is insignificant unless inside a string value), yet there, the expectation is "expected ':', found '}'".

Don't feel that you need to match the error messages given in the load_err.json files.  They are meant to be suggestive rather than prescriptive.
As for why 20 and 22 differ from 21, in 20 and 22 there is nothing after the comma, so the python implementation takes that as no value being given.  In 21 something does follow the comma, a space, so the python implementation interprets that as user starting to give a value, but not completing it.  That is why the error messages differ.  But your interpretation is equally valid.

They are meant to be suggestive rather than prescriptive.

Cool. I opened a PR for linking the library in the docs.


Switch to concourse-release-scripts image

The jar file is no longer publicly available.

Thanks again @Albertoimpl!


Model don't consider database table, only json

I noticed that when I migrate the table, and change eg. the id from Afghanistan from 4 to 5, in the rendered list it is still 4, meaning the contents are taken from the json and not from the table... so why do we need a table at all then?

For foreign keys in tables. The ID's are the official ISO ID's so changing those is maybe not the best idea.


Upgrading all testing libraries to be the same as cats.

@imarios is this PR still relevant?

@kanterov this is not relevant any more. All the logic here was used in #96


Angular-snap doesn't work together with ngSticky

Duplicate my question from SO here http://stackoverflow.com/questions/37527823/angular-snap-doesnt-work-together-with-ngsticky

trying to use together Angular-Snap with ngSticky but it doesn't work, panel which should stay sticky is being scrolled.
I've prepared two examples:

Example with Angular-Snap which doesn't work: https://plnkr.co/edit/ARhUti3jJCEjB8sLL5q0?p=preview
Example without Angular-Snap which works:
https://plnkr.co/edit/QJSQelcdUP2AamFBuNiH?p=preview

You can expand in both examples accordion panel and try to scroll to see the difference.
If I remove in 1st, non-working example angular-snap.css file it begins to work, but obviously I need it. I couldn't debug this issue further, so any help would be appreciated.

Just curious - have you tried using:

angular-snap-only.css - from this repo
snap.css - from the Snap.js repo

Rather than the combined angular-snap.css? IIRC the combined css file also includes some extra things (e.g. smooth scrolling) that may be getting in the way here. In general I'd recommend using the dedicated CSS files for each library independently. We're just providing a bundled version here for convenience and backward compatibility.
If that still doesn't work it'll at least tell us if the issue lies in the Snap.js css or ours.

Yes, it has solved this issue. Thank you very much!

Only now seen that angular-snap-only.css and snap.css not enough, right menu now shown (left menu I haven't checked), you need to add manually:
.snap-drawer-left {
  left: 0;
}
.snap-drawer-right {
  right: 0;
}

It looks like there rules are in place: https://github.com/jakiestfu/Snap.js/blob/develop/snap.css#L55-L63


error while using the script (custom firmware)

Hey everyone I've got this error while using the script
./tuya-cloudcutter.sh
Checking UDP port 53... [sudo] contraseña para javier:
Occupied by systemd-resolve with PID 498.
Port 53 is needed to resolve DNS queries
Do you wish to terminate systemd-resolve? [y/N] y
Attempting to stop systemd-resolved.service
Checking UDP port 67... Available.
Checking TCP port 80... Available.
Checking TCP port 443... Available.
Checking TCP port 1883... Available.
Checking TCP port 8886... Available.
[!] WARNING: Selected wifi AP support: sí
AP support is manditory for tuya-cloudcutter to work.  If this is blank or 'no' your adapter doesn't support this feature.
Press any key to continue, or CTRL+C to exitBuilding cloudcutter docker image
[+] Building 0.0s (3/3) FINISHED
 => [internal] load .dockerignore                                          0.0s
 => => transferring context: 2B                                            0.0s
 => [internal] load build definition from Dockerfile                       0.0s
 => => transferring dockerfile: 457B                                       0.0s
 => ERROR [internal] load metadata for docker.io/library/python:3.9-slim-  0.0s
------
 > [internal] load metadata for docker.io/library/python:3.9-slim-buster:
------
WARNING: buildx: failed to read current commit information with git rev-parse --is-inside-work-tree
Dockerfile:1
--------------------
   1 | >>> FROM python:3.9-slim-buster AS base
   2 |
   3 |     RUN apt-get -qq update && apt-get install -qy --no-install-recommends git hostapd rfkill dnsmasq build-essential libssl-dev iproute2 mosquitto
--------------------
ERROR: failed to solve: python:3.9-slim-buster: failed to do request: Head "https://registry-1.docker.io/v2/library/python/manifests/3.9-slim-buster": dial tcp: lookup registry-1.docker.io: Temporary failure in name resolution
Failed to build Docker image, stopping script

I've tried on sudo mode but I get the same error all time
what can I do?
ty in advance

Your system likely runs a local dns loopback, and cloudcutter kills dnsd daemons because it needs to run them for it's own process.
You likely need to add external dns servers to your resolve.conf file in order to resolve dns names while your local dns server is turned off.

Thank you!!!


Modify legacy schemaRef in on-tasks repo

Background
We find there are legacy "schemaRef" in our code. This "schemaRef" should be "optionsSchema" now in our code.
Details:

Remove schemaRef in lib/task-data/tasks/set-boot-disk.js and lib/task-data/tasks/dell-wsman-update-firmware.js.
Related unit test is also added to avoid such errors.

@iceiilin @anhou

BUILD image-service-backup #31 : FAILURE

BUILD image-service-backup #32 : FAILURE

BUILD image-service-backup #34 : FAILURE

@rahmanmuhamad , as @pengz1 found here, the task schema added in  lib/task-data/tasks/dell-wsman-update-firmware.js is incorrect, you may need to create a new schema for this task which corresponds to your options definition.


Introduction of EventProcessor in Nether Analytics

Issue: #267

[x] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:
A new WebJob (EventHubProcessor) has been added to provide the required services to route incomming messages to correct targets. Job monitors ingest event hub, parses and enriches events and routs them to the right target. Currently two targets have been implemented: Blob Storage and an intermediate Event Hub.
Test:
PreRequisits
2 storage accounts and 2 event hubs need to be setup and configured using the following EnvironmentVariables:
NETHER_WEBJOB_DASHBOARD_AND_STORAGE_CONNECTIONSTRING
NETHER_INGEST_EVENTHUB_CONNECTIONSTRING
NETHER_INGEST_EVENTHUB_NAME
NETHER_ANALYTICS_STORAGE_CONNECTIONSTRING
NETHER_ANALYTICS_STORAGE_CONTAINER
NETHER_INTERMEDIATE_EVENTHUB_CONNECTIONSTRING
NETHER_INTERMEDIATE_EVENTHUB_NAME
Due to a small "bug" in the WebJobs SDK, the ingest EventHub is hardcoded in code to have the name: "ingest", hence the EnvironmentVariable NETHER_INGEST_EVENTHUB_NAME currently need to be set to the same.

Idealy, launch both AnalyticsTestClient and the Nether.Analytics.EventProcessor at the same time after making sure the above Environment Variables are set (sometimes a restart of VS is required for VS-debugging to catch the new Environment Variables).

Use the AnalyticsTestClient to send TypedGameEvents to the ingest EventHub (option 2 in main menu)

Optionally you can use (option 3) to manually specify the json to be sent, even unknown JSON-format or even incorrect JSON should be handled and will be visible as console output in the WebJobs-console window.

WebJob should take different game event types, parse them, and put them in different folders in the Storage Account specified by NETHER_ANALYTICS_STORAGE_CONTAINER environment variable. Several events of the same type should stack up and be placed in the same blob until blob gets too big or time passes so a new folder structure become in use.

Test and look inside blobs to see multiple messages ending up in same bloc, separated by a \n character (Notice that \n is not visible using Notepad and some other more basic editors)

Some event types should also be passed on to the Event Hub represented by Environment Variables NETHER_INTERMEDIATE_EVENTHUB_CONNECTIONSTRING & NETHER_INTERMEDIATE_EVENTHUB_NAME. Currently that is events of type Count & GameHeartbeat. Use ServiceBus Explorer or similar program to make sure events end up there.

Left to do in later PR

Documentation of messages and the flow of them need to be documented
ARM Templates need to be updated to automatically deploy this infrastructure
Configuration need to be looked at to see if we are doing configuration in the same way across all Nether
Optimization - Current implementation is not optimized for speed. For example: We only pick one message at a time from EventHub, we don't re-use connections to services like EventHub and/or Storage Accounts

Work still to do: #275 and #276


[GH-722] - Fix content wider than screen on mobile

Summary

prevented content from overflowing the screen on small-sized mobile phone devices, such as iPhone 5/SE
deleted unused code block with css class .table-striped and no specified styles

Note: I have moved all the css media queries to the end of the file to prevent styles overwrite which is why there seem to be so many changes in the diff.
Ticket Link
Fixes https://github.com/mattermost/mattermost-developer-documentation/issues/722
Screenshots
Here is an example of the proposed changes, shown on iPhone 5:
Proposed fix (left) vs. Current state (right)

Hello @lucievr,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Awesome work, thanks @lucievr!


Change in docker run parameters not refreshed properly on CentOS/RHEL

The /etc/rc.d/init.d/docker- script that gets created with the docker-run.erb template on CentOS/RHEL6 doesn't do a "docker rm" - so if you change any value, during the refresh you get a puppet agent error.  The container is stopped, and can't be restarted because the name is already allocated.  A "docker rm" added to the stop function in the init script would allow that to work properly.
So, something along the lines of:
--- a/docker/templates/etc/init.d/docker-run.erb
+++ b/docker/templates/etc/init.d/docker-run.erb
@@ -94,6 +94,9 @@ stop() {
             echo -n $"Stopping $prog: "
             docker stop $(cat $cidfile)
             retval=$?
+            docker rm $(cat $cidfile)
+            retval2=$?
+            retval=$((retval + retval2))
             [ $retval -eq 0 ] && rm -f $lockfile $cidfile
             return $retval
         else

(that's the patch we're working with in our module, anyway)

This should have been fixed with the current version
stop() {
    echo -n "Stopping $prog: "
    $docker stop <%= @sanitised_title %>
    $docker rm <%= @sanitised_title %>
    return $?
}

Thanks


Update project and fix deprecations

Changes

Update lowest supported version to iOS 12 and fix more deprecations (https://github.com/Wootric/WootricSDK-iOS/issues/59)
Update build phases to fix cycle in dependencies
Remove incorrect tests. These tests were basically testing Apple's implementation of NSOperationQueue
Update social sharing to fix SLServiceTypeTwitter & SLServiceTypeFacebook deprecation (https://github.com/Wootric/WootricSDK-iOS/issues/55)
Fix SPM support (https://github.com/Wootric/WootricSDK-iOS/pull/96 & https://github.com/Wootric/WootricSDK-iOS/issues/86)

Tested SPM integration - Getting error that the package contains mixed sources.

Apparently at some point, some swift files were added. SPM cannot contain both Objcective-C and Swift source files at the same time.
Found the following swift files in the project:

UIApplication+Extension.swift
UIButton+Extension.swift

I think, these can easily be converted to Objective-C and everything should be fine

Aaah yes, these are new ones. Let me check this and update the PR. Thanks for testing @KoCMoHaBTa !

Thanks for the update. One more issue - the package does not build. You should use
#ifdef SWIFTPM_MODULE_BUNDLE instead of #if SWIFTPM_MODULE_BUNDLE

Now it works. Thanks @diegoserranoa 🥳

@KoCMoHaBTa thanks for the review! I pushed a last commit to also fix it for Cocoapods 😅

Thanks for the help with this! I need someone from my team to also review and approve this and then I can merge and release the new version.


feat: Spanish translations

Summary
Spanish translations
Testing
Validate syntax and grammatic

Hi, thank you for contribution! But the change looks not correct. There should not be bundle.l10n.es.json file under root directory. The l10n/bundle.l10n.es.json file contains garbage characters. The key of the json should remain in English and the translation should be in the value part of the json. Please refer to other locales. Thanks.

es-update-feb2024.csv
@elypke If it's helpful, you can update this spreadsheet and re-attach it to the PR instead of worrying about json files. You'll just need to update everything in column C

@scnwwu @scottdover Hi guys, thank you for your comments,
I have added the file bundle.l10n.es
@scottdover I have just read your comment recently,  just in case I will update your file and i will send you

Hi @elypke, there're some updates about the messages in main branch. Would you please update?
And please make sure to have LF line ending to pass the build.
If any problems, @scottdover might be able to provide you an updated csv file to update. Thanks.

Hi @scnwwu @scottdover
I'm sending you the updated file
Regards
es-update-feb2024.csv

Thank you for your contribution, @elypke 😊. It's appreciated. I realize I asked you to update the contributors table then merged this one in before that was done. Do you mind providing your first/last name and I'll get that updated?

Ok,
Elyana Mastache and Raquel Nunez

Thank you!


Different configuration levels to support profiles.

The same way that there are "User" and "Workspace" settings, now that we have "profiles" it would be really useful to include a new level of settings for profiles as well, so that we can have overrides common to all profiles and overrides specific for certain profiles.
My particular use case is that I have multiple profiles, 10 approximately, and there are some customizations/overrides that I'd like to have in all of them and some that I'd like to have in specific profile. For example, for keyboard shortcuts, there are keyboard shortcuts like one for expanding the terminal that I'd like to have in all of them, and other keyboard shortcuts like "search in unreal documentation", that I'd like to have in the "Unreal" profile only. At the moment, every time that I figure out a new "global"-ish shortcut that I'd like to have in all profiles, I need to switch to all of them and manually insert/copy it, which is incredibly inconvenient, specially since sometimes I'm not fully settled on that shortcut yet and some time later I realise that another one would suit it better.
Another possible alternative would be to be able to export just the shortcuts from a given profile (which can be done currently), and import that exported data into another profile with some degree of diffing or merging capabilities

/duplicate
https://github.com/microsoft/vscode/issues/156144

Apologies, I can see in your issue that it's mentioned a few times. Thank you for the reference


Usemin doesn't change when references assets are in the views

How would you handle updating references to assets in the views when using usemin?

Thanks @zxbodya. Works fine


Audio stopped with app on background when using setUrl

Hi, I said this on #204 but didn't notice it was closed, apologies.
When the audio is started using AudioPlayer's setUrl and then resume it stops on iOS when the app is in the background or the device is blocked.

hi there is a new feature 'stayAwake' inside the play method that keeps the media player awake when in background, but if you use this feature you need to add 1 more permission to your manifest - WAKE_LOCK (for more info just read in the plugin readme).

Closing due inactivity.


Nested Tabs

5x`tabs
tab: DARK ROGUE
3x`tabs
tab: ARMOR
tab: WEAPONS
tab: CONSUMABLES
tab: SKILLS
3 backticks
5 backticks
Why doesn't this work?

Try v1.1.0

Works like a charm.  Thank you much.


graph client^2.2.8 failing to run graphql server

When I run graphclient serve-dev, it starts playground and immediately returning error
TypeError: Failed to fetch\n    at HTe (https://unpkg.com/<EMAIL_ADDRESS>
in console,
D:\ethdev279\uniswap-graph-client\node_modules\@whatwg-node\server\index.js:126
async function sendNodeResponse({ headers, status, statusText, body }, serverResponse) {
                                  ^
TypeError: Cannot destructure property 'headers' of 'undefined' as it is undefined.

Thank you for reporting this issue. It's been fixed in the latest release!


Having nonzero Infused Reagents on the crafting tab should contribute 1 light bulk to your inventory tab

In GitLab by @danielagardeazabal on Jan 29, 2022, 04:54
That way, we no longer have to add a separate, permenant Infused Reagents item to our inventory.

Closing due to changes introduced in PC2


feat: documentation for download func.

Need
Even though the upload-plugin supports download functionality too, there is no documentation regarding this so I added
a brief documentation about the download functionality along with a small example similar to the upload function
Changes Implemented

In the console.log message, it was written download even though it is an upload function, same with the associated comment

Code example for download function

Thanks for contributing!
First time i really look at it tbh, but shouldn't it be Content-Type instead of ContentType? 🤔


[AMBARI-24356] Add TIMELINE_READER into Ambari BlueprintConfigurationP…

…rocessor.
What changes were proposed in this pull request?
Adding TIMELINE_READER web address into blueprint configurations.
How was this patch tested?
Ran all corresponding tests cases to validate it.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3300/
Test PASSed.

Updated the patch adding tests for verifying newly added yarn properties.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3311/
Test PASSed.

LGTM. One remark though: I think you only need to add a SingleTopologyUpdater if you want to replace localhost in the property. %HOSTGROUP_N% tokens are replaced for all properties now.


Use single quotes in Craft "switch" snippet

This is a really minor one but: the "switch" snippet for Craft uses double quotes around the case statements ({% case "value1" %}). Would be nice to have those as single quotes ({% case 'value1' %}) to make it consistent with the other snippets. (I've not tried all the snippets – but that's the only single-quote one I've seen so far.) Great work on this extension though!

Great, thank you!


Add Chains of War showcase.js

Name: Chains of War
Description: Fantasy saga that carves a new world on the Cardano blockchain by means of storytelling, artworks, 3D assets and gamification.
Link: https://chainsofwar.io/
Tags:

game
nft

Hi @fill-the-fill
Thanks for adding the showcase label.
Are there any further instructions?
Best regards,
The Chains of War team

Hi @fill-the-fill
Thanks for adding the showcase label. Are there any further instructions?
Best regards, The Chains of War team

@aarondunnington hi, at the moment the next step is to receive 3 approvals from the community. Sometimes it takes time to review the projects, but you're on a right track. :)

Hi @fill-the-fill is there anything we can do to get this merge proposal approved?

@

Hi @fill-the-fill is there anything we can do to get this merge proposal approved?

I've just noticed you are missing preview image. Please add one to the showcase folder where all of the showcase images are being stored 🙂

@fill-the-fill I missed that one too.  Somehow between the inline preview image & the committed JS file my brain registered the submission as complete 🤪

@katomm is this showcase image file within the size constraint suggested in https://github.com/cardano-foundation/developer-portal/pull/720#issuecomment-1201718156?  It would be the largest image in the showcase by a good margin.

Personally, I'm struggling to see how this is a showcase for the developer portal. (but this is just my opinion)
What is more important: I don't think it's a good idea to accept more NFT projects at this time without further discussion of how we handle NFT projects in general. Anyone, please feel free to join the discussion here: #243 (We don't seem to be making any progress here because many people only seem interested in advertising.)

Thank you for your contribution and for building on Cardano. This PR has been open for quite a while and I would like to give you a little feedback. Projects need 3 reviewer approvals and it seems the project didn't get enough "traction" among the reviewers.
Please note that showcase projects need to provide a unique value from existing showcase items - which is especially difficult with many tokens and FT/NFT or FT/NFT platforms. Feel free to join the discussion here: #243 (or in case you prefer to discuss this not in GitHub please join https://discord.gg/Exe6XmqKDx)


🐞 Prevent pre-commit hook from adding un-staged changes

Since the pre-commit hook runs git add -A it will add any un-staged files. Instead this should only add previously staged files.

Closed by #452


code for redirect

allows you to redirect a user to a URL

how does this behave when used in a ui_worldmap node ? (ie in dashboard) - does it redirect whole page - or just the iframe ?

Good call. I never use the ui_worldmap node. It is a mess. Hold off on merge. I need to decided if I'm going to fix it or disable the functionality when using inside of ui_node

Working on issue today.   Took me awhile to wrap my head around the
https://github.com/dceejay/RedMap/blob/master/worldmap.js
as ive spent most my time in the land of the
https://github.com/dceejay/RedMap/blob/master/worldmap/worldmap.js
I see how function HTML works with ui_worldmap
going to use something like
if (msg.hasOwnProperty("payload") && msg.payload.hasOwnProperty("command") && msg.payload.command.hasOwnProperty("redirect"))

then I can cause the same effect on the ui_worldmap dashboard  ..... I think.
This project is not as dead as I thought.

Working on issue today.
if (msg.hasOwnProperty("payload") && msg.payload.hasOwnProperty("command") && msg.payload.command.hasOwnProperty("clear"))

Need some help when you find time.
So I got to thinking it would be real nice if I only had to change one area of the code to get this to work.
https://github.com/dceejay/RedMap/blob/master/worldmap.js line 148
I am trying this but I just cant get it to work
var html = `

`;
        return html;

Ive stuck in just:

and that works fine.
Care to point me in a direction ?


[Release-1.26] - cli: k3s fails to execute for users lacking read perms for /etc/rancher/k3s/config.yaml

Backport fix for cli: k3s fails to execute for users lacking read perms for /etc/rancher/k3s/config.yaml

#4321

Validated on branch <branch_name> with commit <commit_id> / version
Environment Details

Infrastructure

[X] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

Linux ip-172-31-30-17 5.19.0-1025-aws #26~22.04.1-Ubuntu SMP Mon Apr 24 01:58:15 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
PRETTY_NAME="Ubuntu 22.04.2 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.2 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

Cluster Configuration:

single node

Config.yaml:

cluster-init: true
write-kubeconfig-mode: 644

Additional files

N/A

Testing Steps

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Run the following commands:

touch /etc/rancher/k3s/config.yaml
sudo chmod 700 /etc/rancher/k3s/config.yaml

Install k3s
Run the following commands:

k3s -v
k3s --version
k3s --help
k3s server -h

Replication Results:

k3s version used for replication:

FATA[0000] open /etc/rancher/k3s/config.yaml: permission denied

ubuntu@ip-172-31-30-17:~$ k3s --version
FATA[0000] open /etc/rancher/k3s/config.yaml: permission denied
ubuntu@ip-172-31-30-17:~$ k3s -v
FATA[0000] open /etc/rancher/k3s/config.yaml: permission denied
ubuntu@ip-172-31-30-17:~$ k3s --help
FATA[0000] open /etc/rancher/k3s/config.yaml: permission denied
ubuntu@ip-172-31-30-17:~$ k3s server -h
FATA[0000] open /etc/rancher/k3s/config.yaml: permission denied
ubuntu@ip-172-31-30-17:~$

Validation Results:

k3s version used for validation:

k3s version v1.26.6-rc1+k3s1 (3b1919b0)
go version go1.19.10

ubuntu@ip-172-31-30-17:~$ k3s -v
k3s version v1.26.6-rc1+k3s1 (3b1919b0)
go version go1.19.10
ubuntu@ip-172-31-30-17:~$ k3s --version
k3s version v1.26.6-rc1+k3s1 (3b1919b0)
go version go1.19.10
ubuntu@ip-172-31-30-17:~$ k3s --help
NAME:
   k3s - Kubernetes, but small and simple

USAGE:
   k3s [global options] command [command options] [arguments...]

VERSION:
   v1.26.6-rc1+k3s1 (3b1919b0)

COMMANDS:
   server           Run management server
   agent            Run node agent
   kubectl          Run kubectl
   crictl           Run crictl
   ctr              Run ctr
   check-config     Run config check
   token            Manage bootstrap tokens
   etcd-snapshot    Trigger an immediate etcd snapshot
   secrets-encrypt  Control secrets encryption and keys rotation
   certificate      Manage K3s certificates
   completion       Install shell completion script
   help, h          Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --debug                     (logging) Turn on debug logs [$K3S_DEBUG]
   --data-dir value, -d value  (data) Folder to hold state (default: /var/lib/rancher/k3s or ${HOME}/.rancher/k3s if not root)
   --help, -h                  show help
   --version, -v               print the version
ubuntu@ip-172-31-30-17:~$ k3s server -h
NAME:
   k3s server - Run management server

USAGE:
   k3s server [OPTIONS] ...

Additional context / logs:
N/A


offsetTop is the same for all table rows

There has been a change since HtmlUnit 2.33 which causes the Y location (or offsetTop) of all table rows to be the same. My guess was this was related to this old sourceforge bug #1986. The change was such that ComputedCSSStyleDeclaration.getTop(...) only considers siblings with "display: block". However, for table rows, it is "table-row". I feel like most display values are block-like.
Simple ComputedCSSStyleDeclarationTest test:
    @Test
    @Alerts("true")
    public void offsetTopTableRows() throws Exception {
        final String html = "\n"
            + "
\n"
            + "
row1
\n"
            + "
row2
\n"
            + "
\n"

            + "\n"
            + "";
        loadPageWithAlerts2(html);
    }

A similar issue is with the calculation of offsetLeft, where it takes all previous siblings until the first "display: block" sibling, which should also instead search for the first block-like sibling.

Sounds reasonable, will have a more detailed look at this at the end of next week. You can also check on Twitter for information about the HtmlUnit progress.

Have changed the code to handle most display values as block.
Many thanks for the report an the test case.
Will make a new snapshot build available soon (check https://twitter.com/HtmlUnit for news)

Seems to work. Thanks!


How to use it to stream system audio,

How to use it to stream system audio, or use it to receive system audio streams directly？

I'm not sure what are you trying to achieve, this library is meant to be used with either a remote URL or a local file.
What does "stream system audio", mean?


We cannot track users through their journey without searching through logs

There is no way for us to follow a user through a journey without us just looking through MSA logs. The VSP sends the request ID together with the AuthnRequest SAML but to corrolate this to an MSA request, all we can do is check the logs. This makes it difficult to provide any metrics on things like matching, journey times or success rates. Consider adding the request ID to the JSON requests the MSA makes to the local matching service in some way.

This repo is only temporarily unarchived in order to clean up some Terraform; this issue is no longer pertinent.


Added EmailService interface

Added EmailService interface
Added EmailMessage model
Added SMTPEmailService implementation
Added ClientEmailService

Looks good. Add PM's in a follow-up PR.


"Add to cart" should be beside "Buy now" button

The "Add to cart" button should be placed next to the "Buy now" button for a cleaner look and easier access.
This makes it simpler for users to quickly choose between buying immediately or adding the item to their cart, improving the shopping experience.
Please assign this issue to me, I will happy to work on that.

@swaraj-das could you please add label hacktoberfest to this issue


[CI] SharedClusterSnapshotRestoreIT.testSnapshotCanceledOnRemovedShard fails sporadically

This one has failed a few times. Most recent on #1390.
Needs triage and further investigation
REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.snapshots.SharedClusterSnapshotRestoreIT.testSnapshotCanceledOnRemovedShard" -Dtests.seed=9350EB7A8335C1F9 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1" -Dtests.locale=sl -Dtests.timezone=SystemV/PST8 -Druntime.java=17

org.opensearch.snapshots.SharedClusterSnapshotRestoreIT > testSnapshotCanceledOnRemovedShard FAILED
    SnapshotMissingException[[test-repo:test-snap] is missing]
        at org.opensearch.action.admin.cluster.snapshots.get.TransportGetSnapshotsAction.masterOperation(TransportGetSnapshotsAction.java:160)
        at org.opensearch.action.admin.cluster.snapshots.get.TransportGetSnapshotsAction.masterOperation(TransportGetSnapshotsAction.java:80)
        at org.opensearch.action.support.master.TransportMasterNodeAction.masterOperation(TransportMasterNodeAction.java:124)
        at org.opensearch.action.support.master.TransportMasterNodeAction$AsyncSingleAction.lambda$doStart$3(TransportMasterNodeAction.java:197)
        at org.opensearch.action.ActionRunnable$2.doRun(ActionRunnable.java:86)
        at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:792)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
        at java.lang.Thread.run(Thread.java:833)

🙋♀

I am not able to reproduce this. need some advice.😵💫

Update: We will attempt to dig into the test to find a hypothesis for why the snapshot is intermittently missing.
@nknize can you link to other instances where this failure has occurred, so we can get more data to try and identify a pattern?

java.lang.IllegalStateException*: Cannot cleanup [test-repo] - a snapshot is currently running in [SnapshotsInProgress[test-snap]]
From my point of view, this failure is because when creating a snapshot, setWaitForCompletion(false)
https://github.com/opensearch-project/OpenSearch/blob/996d33adb22ac6a523962166f3677f8dc6ac9c1f/server/src/internalClusterTest/java/org/opensearch/snapshots/SharedClusterSnapshotRestoreIT.java#L1918
will lead to SnapshotsInProgress(as the cluster state) exist for times and fail the test cleanup before it removed from cluster state.

I am closing this issue because the failure is not reproducible and not be seen for a while. If the failure shows again or someone can reproduce it reliably, feel free to reopen it😄


Update testing docs

def success_response(self):
... # won't be found by test runner
def test_success_response(self):
... # will be found by test runner

https://stackoverflow.com/questions/2037364/django-test-runner-not-finding-tests


Missing event after ha restart of VM

ISSUE TYPE

Improvement Request

COMPONENT NAME

UI

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

When using a ha enabled offering and a VM was restarted because of this, no entry is showing in events.
STEPS TO REPRODUCE

1) Create a VM with ha enabled offering
2) Destroy the VM on the hypervisor (in KVM via virsh destroy <id>)
3) Wait until CS is restarting the VM
4) Take a look at Events tab of VM and/or Events Tab of main menu

EXPECTED RESULTS

A new entry should be added to the events history because of ha restart

ACTUAL RESULTS

No entry is made to the events history

Fixed in #9202


Relationship with GoogleTest in the long run

The build instruction (either Bazel or CMake) said at Abseil unit tests need GoogleTest. However, GoogleTest said in README.md that it will (though not yet) depend on Abseil.
So my question is.. in the long run, who is depending on whom? Thx.

Cyclic dependencies at the project level aren't inherently a problem. As
you said: only the Abseil tests need GoogleTest, and GoogleTest isn't
going to depend upon Abseil's tests.
On Wed, Aug 12, 2020 at 11:59 AM Leedehai<EMAIL_ADDRESS>wrote:

Abseil unit tests depend GoogleTest. The build instruction (either Bazel
or CMake) said at Abseil unit tests need GoogleTest. However, GoogleTest said
in README.md https://github.com/google/googletest#coming-soon that it
will (though not yet) depend on Abseil.
So my question is.. in the long run, who is depending on whom? Thx.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/abseil/abseil-cpp/issues/765, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7CRX42FZCC5CZZPPTBQMDSAK34PANCNFSM4P45EQXA
.

--
*If you get an email from me outside of the 9-5 it is not because I'm
always on or expect an immediate response from you; it is because of work
flexibility
http://www.inc.com/john-boitnott/how-flexible-hours-can-create-a-better-work-life-balance.html
.  Evening and weekend emails are a sign I allocated some regular working
hours for other things (such as family, gym, friends,...).  And I encourage
you to feel free to do the same.

I see, thanks.


gitops run: ctrl+c doesn't work during dashboard installation

76d7151d7 tried to make it so that hitting ctrl+c while part of the way through the dashboard installation would still clean up. However, if the dashboard installation gets stuck - for example if the port forward fails - then we're stuck with a process that is ignoring ctrl+c.

ctrl+c should terminate the process - no cleanup is performed, as we haven't finished installing onto the cluster yet.

closed by #3113


Do we change the DOMDocument instance that get passed in, and is this an issue?

See microformats/mf2py#104. For backwards compatibility parsing, the Python parser changes the DOM on the fly. I believe the PHP parser does a similar thing. It turns out that – in the case of the Python parser – the same DOM object can’t be parsed successfully a second time. The microformats in the base document have been “damaged”.
How can we best test if this is the case with our parser too? Maybe also add a test case where we check that a second parse gives the same result?
Needs investigating. Thanks @kartikprabhu for bringing this up!
(This is basically a todo for myself, therefore also assigning myself.)

Confirmed in php-mf2 if you pass in a DOMDocument, it's modified during parsing:
Input HTML:

This is a summary

This is mytag inside content.

$doc = new DOMDocument();
$doc->loadHTML($html);
echo $doc->saveHTML();
$parse = Mf2\parse($doc);
echo $doc->saveHTML();

Output (trimmed doctype and html, body elements):

This is a summary

This is mytag inside content.

This is a summary

This is mytag inside content.

<data class="category p-category" value="mytag"></data>

Appears to be a simple fix: $doc = clone $input; at this line. Only tested locally with the above HTML.


Update 2.0.3 npm build failure

I'm posting a new issue as I did not receive any feedback on the comment I made in this closed issue: https://github.com/hakimel/Ladda/issues/109
Is there a way to support both node.js and Dart Sass? When building our application we get the following error after the above change was made in v2.0.3:
Module build failed: ModuleBuildError: Module build failed:
        right: math.div($spinnerSize, -2) + 10;
                  ^
      Invalid CSS after "...    right: math": expected expression (e.g. 1px, bold), was ".div($spinnerSize, "
      in <redacted>node_modules/ladda/css/ladda.scss (line 118, column 20)

Per https://github.com/sass/node-sass#readme:

Warning: LibSass and Node Sass are deprecated. While they will continue to receive maintenance releases indefinitely, there are no plans to add additional features or compatibility with any new CSS or Sass features. Projects that still use it should move onto Dart Sass.

So I recommend that you switch to Dart Sass in order to support modern Sass features and use the latest version of Ladda.

Sure, we will eventually, I just think it isn't great to break backwards compatibility in a minor version update.

The change was necessary to avoid deprecation warnings when using Dart Sass (see #109). Since this was the only change in Ladda v2.0.3, feel free to continue using v2.0.2 until you are able to switch to Dart Sass.


NewRepoCluster overview

I want to test it

This is only a simple check if all Spark nodes are running. The branch is part of the bigger task where we plan to monitor the cluster as a whole (together with running application). It is not prepared yet and requires additional updates in Ansible.
However, you can try the web page using the following steps:

Switch to the branch and download Stream4Flow directory
Replace the original version of the web interface on your server (/var/www/web2py/applications)
Set cluster IP addresses in the apconfig.ini (Stream4Flow/private/appconfig.ini)
Restart the Apache service
Go to the web interface and LogIn
Visit the page http:///system/cluster_control

The current version is not complete and merge is rejected


can't get Enable-Migrations to work

Can't get Enable-Migrations to work.
I know this is not a complete bug report. It isn't meant to be. Please reply with a guide how Migrations should work using EF 6.3 (targeting .NET FrameWork). EF 6.3 works in the sense that I succeeded to insert and query some data from a fresh database (CreateIfNotExists).
Kind regards, Jan.

@jancg Tooling is still being worked on, so it likely doesn't work because it hasn't been implemented yet in the build you are using.
/cc @bricelam


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/235 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 613cc9a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-11-21-08-59-37 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Archive_master_2019-11-21-08-59-37

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


NetworkServer.Destroy() calls before Rpc call

I have GrenadeEntity (thrown grenade with detonation timer) class in my game. At the end of Explode() method I'm activating explosion effect on all clients (RPC), and then destroying the GrenadeEntity object. But... Destroy() calls before RPC call (only on host), and this cause some problems.
Can you fix this?
public class GrenadeEntity : NetworkBehaviour
{
/* ... */
[SerializeField] private ExplosionEffect _explosionEffects;
/* ... */

private void Explode()
{
    /* ... */
    RpcActivateEffect(); // This doesn't call on host, but calls on other clients
    NetworkServer.Destroy(gameObject); // This calls first
}

[ClientRpc]
private void RpcActivateEffect()
{
    _explosionEffects.Show();
}

This is not a bug, and issue tracker is not for support.  Please join our Discord for support.
Replace
NetworkServer.Destroy(gameObject);
with
Destroy(gameObject) 0.1f);


clever way to turn connect/disconnect the websockets during specific times

Hey there,
Is it possible to connect the websockets from (say) M-F 9am-5pm, and then disconnect when outside of that time range?
It sounds like you would need to jump through hoops with callbacks?
Thanks!

Hello, as far as websockets is concerned, assuming you're talking about a client, run client = await websockets.connect(...) at 9 am and await client.close() at 5pm. Probably you need to handle reconnecting in case the connection drops during the business day.

Another way to look at this -- try to do this with a TCP connection managed by asyncio, and then you should be able to do something similar with websockets. If you can't do it with asyncio, well, I'm going to consider it a Python / asyncio issue rather than a websockets issue.


feat(XY): :sparkles: Add GridXY & CellXY to get more clear semantic

[x] 您的代码是否能够编译通过？
[x] 您是否检查了目前在 pull requests 中是否有与你的贡献功能相同的更改？
[x] 您的贡献是否符合贡献者代码协议？
[x] 您的贡献是否符合开发规则？

Descriptions of this pull request:

GridXY & CellXY only makes code more readable. It will not change the implementation.

All XYs are Grids. There is no need to divide XYs into GridXYs and CellXYs. Moreover, there are some functions to convert Grids to Cells in GameData.cs such as PosGridToCellXY().
The division may also result in bugs in functions like ApproachToInteract() in GameData.cs.
It is not suggested to make such modifications.

@Panxuc
First, it's not  a division because it's non-invasive and non-compulsory, which means it may not be used (just as the inherited PosGridToCellXY which was firstly introduced in THUAI6 commit 257d7ede41663e84f7d07b64e20d826d4b2cd3a4 but never used)
Second, if you review the commit, you will know it's actually not the repeat of PosGridToCellXY because there's no difference among XY, GridXY and CellXY except the names; and as a consequence it will not result in bugs, especially in ApproachToInteract (in fact, to avoid bugs for ApproachToInteract is the motivation of this PR). My local code experiment also testify this point.

@Panxuc First, it's not a division because it's non-invasive and non-compulsory, which means it may not be used (just as the inherited PosGridToCellXY which was firstly introduced in THUAI6 commit 257d7ede41663e84f7d07b64e20d826d4b2cd3a4 but never used)
Second, if you review the commit, you will know it's actually not the repeat of PosGridToCellXY because there's no difference among XY, GridXY and CellXY except the names; and as a consequence it will not result in bugs, especially in ApproachToInteract (in fact, to avoid bugs for ApproachToInteract is the motivation of this PR). My local code experiment also testify this point.

It seems that I misunderstood your motivation. As you said, there's no difference among XY, GridXY and CellXY except the names, so I am not sure what you attempted to do. I thought more commits would be pushed to this PR modifying those stuffs referred.
As XY, GridXY and CellXY are actually the same, is there necessity to distinguish them? I am not quite sure. It depends on you.


Can't override proc dumpHook*[T](s: var string, v: seq[T])

I can override 5 other dumpHooks, but secifically this one gives me nim ambiguous call.
It might be a nim error and not the fault of this library, and then it might make sense to at least document this hook with a NOTE about it not being overridable.
proc dumpHook*[T](s: var string, v: seq[T])

This is some what of a Nim issue. Because I already defined proc dumpHook*[T](s: var string, v: seq[T]) you can't defined it again. You need to use some thing else seq of a spesific T?

.. boah.. now I start seeing the same issue for other dumpHooks ...
it worked before, now I changed my internal code structure a bit... I am fully confused now.
How to properly override a hook?

... are you saying, it always has to be a more specific type when overriding (or say, a hook that is not defined internally in jsony yet)?
I guess not ... but sometimes yes.. boy I am confused.

thank you @ee7, that link is useful.
I am still very confused though, because I copied 5 dumpHooks from the jsony source code, edited each one slightly (without changing the signature), and then converted a ref object that contained all the types of these dumpHooks (and more). it compiled and I got the desired output (my code was run for the hooks I defined). now I reordered my code a bit, and I get the disambiguity message.
how could it have worked before?
also, that makes the whole hook idea much less useful, to be honest. there should be a way to overload the general hooks with the exact same types as are in the library. I am sure that is somehow possible. I imagine, to rename all the *Hook procs, removing the Hook postfix. then inside each of these procs, the first lines check if a function with the same name + Hook postfix exists, and if yes, it is called, else the standard code is executed. I imagine this would be a compile-time check, so it would not make the library slower.
then we could use jsony the way.. I imagined it would work, when reading the README, and with all the power of hooks.
Alternatively, maybe method overloading could be used, which I guess would allow for even more flexibility, like different parts of my code using different sets of hooks to (de-)serialize something.

wouldnt be an issue if import except syntax allowed specifying overloads. crazy that its not a feature.

but, running this
import pkg/jsony

proc dumpHook*[T](s: var string, v: seq[T]) =
  echo T
  jsony.dumpHook(s, v)

echo toJson @[1, 2, 3]

gives expected result
int
[1,2,3]

do you have failing examples?

thank you @SolitudeSF , that is important info.. which I did not see in the overload resolution docu (might be there anyway though).
... but .. how is this possible then?
/home/user/myTool/src/main.nim(238, 20) template/generic instantiation of `toJsonStr` from here
/home/user/myTool/src/config_cmd_check.nim(434, 26) template/generic instantiation of `toJson` from here
/home/user/.nimble/pkgs2/jsony-1.1.5-6aeb83e7481ca8686396a568096054bc668294df/jsony.nim(881, 11) template/generic instantiation of `dumpHook` from here
/home/user/myTool/src/config_cmd_check.nim(362, 10) Error: ambiguous call; both config_cmd_check.dumpHook(s: var string, v: object) [proc declared in
/home/user/myTool/src/config_cmd_check.nim(289, 6)] and jsony.dumpHook(s: var string, v: object) [proc declared in /home/user/.nimble/pkgs2/jsony-1.1.5-6aeb83e7481ca8686396a568096054bc668294df/jsony.nim(784, 6)] match for: (string, OrderedTable[system.string, check_config.CheckConfig])
       Tip: 72 messages have been suppressed, use --verbose to show them.

toJsonStr is template/generic, meaning its body looks for symbols not where its defined, but where it is expanded/called. so effectively you are still importing same dumpHook from two different modules. if it was a regular proc it would work.

its because toJsonStr is generic. if you can have a non-generic function in config_cmd_check.nim that invokes toJsonStr, which you will then use in main.nim, then it should work, i think.

Indeed that was the solution.. thank you @SolitudeSF !
ok learned some things here.. thanks to everyone else too!


Add way of using map-to-image

I might be wrong, but it looks like there is no cairocffi way of using cairo-map-to-image, (unless this is the same as passing data to ImageSurface constructor?) - map-to-image and set-dirty and flush seem to be a way to get cairo to act on some existing data.

Hi. I don’t know what cairo-map-to-image is, and Google Search is not very helpful. Can you explain?

The first mention I can find is here:
http://cairographics.org/news/cairo-1.11.4/
Think I found some other info the other day, will need to keep looking .. think it's for making a cairo surface that just uses existing memory as opposed to just copying existing data in.
Here is a RUBY info here, though not quite sure whats going on..
http://ruby-doc.org//gems/docs/c/cairo-1.12.9/Cairo/Surface.html#method-i-map_to_image
A patch to add it to some LUA bindings...
https://github.com/awesomeWM/oocairo/commit/8ea86484e3a1d52d9e9fa818d2876401a7139ff8
Can't actually find any examples that use it so far + haven't been able to test it..

Ok, I see. This is the cairo_surface_map_to_image function. It does not have bindings in cairocffi yet. I’d review a pull request to add it. It would not be a lot of code, but it looks a bit tricky to get right as the returned surface needs to clean itself up with cairo_surface_unmap_image instead of cairo_surface_destroy.
I’m not sure when it is useful, but it does not look like it creates a surface from an existing memory buffer (that’s ImageSurface.create_for_data or passing data to the ImageSurface constructor) since it works with an existing surface.


NTR: quercetin 4ʼ-O-β-D-glucopyranoside hydrolase activity

Please provide as much information as you can:

Suggested term label:
Quercetin 4ʼ-O-β-D-glucopyranoside hydrolase activity

Definition (free text)
Catalysis of the reaction: quercetin 4'-O-beta-D-glucoside + H2O = beta-D-glucose + quercetin.

Reference, in format PMID:#######
PubMed=12594539

Gene product name and ID to be annotated to this term
LCT Uniprot/P09848

Parent term(s)
GO:0004553  hydrolase activity, hydrolyzing O-glycosyl compounds

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

RHEA:69647

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

RXN-13785

Any other information

On hold until we have a solution to import Rhea reactions as GO terms.

Will add the RHEA as NARROW xref of GO:0008422 beta-glucosidase activity


Update frontend index page

Update the home page (index.html), especially for the new OVH server:
As discussed with @glemaitre and @agramfort;

remove CDS/Paris Saclay specific things. If ramp board is intended to be a package used by others to setup their own server, the home page should be more general instead of looking like it is only for CDS
make it clear that ramp board is both a bundle of packages that allow you to setup your own server and a platform for ML challenges

Is this an accurate summary of our discussion? What do you think @kegl?

let's have section at the bottom that can be configured and that says that
RAMP is developed by CDS
and than hosted and sponsored by OVH

I agree: we should keep CDS as the originator of the package.


it's possible to sign documents using certificates from windows store?

hello,
I'm looking for a library to sign PDF files with an instance of the certificate directly from the windows store ( certificates with token usb or others), not from an exported file on the local version. Your library can do this?
I'm using wincertstore.CertSystemStore to get certificates available in python.
thank you and congrats on your work.

Hi @Fulea-Andrei,
Yes and no. PyHanko certainly doesn't do that out of the box, but you should be able to relatively easily extend pyHanko to support this use case by defining your own Signer implementation that uses the Windows Crypto APIs. I'm not sure if there exist Python bindings for that, but if not, I'm sure it can be done by slotting in a Cython/ctypes/... module to handle the native parts.
 I'd also be a little hesitant to merge platform-dependent code into pyHanko for similar reasons, but if you find a way to get this working in a more or less compact manner, I'd be happy to accept a documentation PR with some sample code (see e.g. this example).


chore: release main

:robot: I have created a release beep boop
redlog: 1.4.2
1.4.2 (2024-05-11)
Bug Fixes

include internal modules in release (1688b6e)

logtheme: 1.0.0
1.0.0 (2024-05-11)
Features

catppuccin: bump version (1ea6b99)
catppuccin: bump version (ad1a9fb)
catppuccin: bump version (db94821)
release 1.2 (c2ec968)

Bug Fixes

add gallery section (4da21ab)
new version with license (5809ad9)
require slog (40e9556)

themes: 1.0.0
1.0.0 (2024-05-11)
Features

catppuccin: bump version (1ea6b99)
catppuccin: bump version (ad1a9fb)
catppuccin: bump version (db94821)
release 1.2 (c2ec968)

Bug Fixes

add gallery section (4da21ab)
new version with license (5809ad9)
require slog (40e9556)
versions (4a2abfb)

This PR was generated with Release Please. See documentation.

:rocket: This description was created by Ellipsis for commit 5442fcd7b502f0cb34b1701f1fb3622ac24b6d2b

Summary:
This PR releases the main branch with updated versions for redlog, logtheme, and themes, including detailed changelogs and the inclusion of internal modules.
Key points:

Release main branch with updates to redlog, logtheme, and themes modules.
Update .release-please-manifest.json with new versions.
Include detailed changelogs for each module, documenting bug fixes and feature enhancements.
Ensure inclusion of internal modules in the release.

Generated with :heart: by ellipsis.dev

:robot: Release is at https://github.com/op/redlog/releases/tag/redlog-v1.4.2 :sunflower:

:robot: Release is at https://github.com/op/redlog/releases/tag/logtheme-v1.0.0 :sunflower:

:robot: Release is at https://github.com/op/redlog/releases/tag/themes-v1.0.0 :sunflower:


Crash on accessing secure note (Keychain Plugin)

QS crashes reproducibly when selecting a secure note from the login keychain in the results list:
20/11/13 17:29:05,652 Quicksilver[27464]: Error getting contents of Keychain item <SecKeychainItem 0x7fb9f200e5c0 [0x7fff72094eb0]>: The user name or passphrase you entered is not correct. -25293
20/11/13 17:29:05,653 Quicksilver[27464]: Exception detected while handling key input.
20/11/13 17:29:05,653 Quicksilver[27464]: *** -[__NSPlaceholderArray initWithObjects:count:]: attempt to insert nil object from objects[0]
20/11/13 17:29:05,654 Quicksilver[27464]: (
    0   CoreFoundation                      0x00007fff83e1041c __exceptionPreprocess + 172
    1   libobjc.A.dylib                     0x00007fff84269e75 objc_exception_throw + 43
    2   CoreFoundation                      0x00007fff83cc8dd6 -[__NSPlaceholderArray initWithObjects:count:] + 358
    3   CoreFoundation                      0x00007fff83d46282 +[NSArray arrayWithObject:] + 50
    4   Keychain Plugin                     0x00000001085e3006 objc_unretainedPointer + 4918
)
20/11/13 17:29:05,767 Quicksilver[27464]: Error getting contents of Keychain item <SecKeychainItem 0x7fb9f200e5c0 [0x7fff72094eb0]>: The user name or passphrase you entered is not correct. -25293
20/11/13 17:29:05,768 Quicksilver[27464]: An uncaught exception was raised
20/11/13 17:29:05,768 Quicksilver[27464]: *** -[__NSPlaceholderArray initWithObjects:count:]: attempt to insert nil object from objects[0]
20/11/13 17:29:05,768 Quicksilver[27464]: (
    0   CoreFoundation                      0x00007fff83e1041c __exceptionPreprocess + 172
    1   libobjc.A.dylib                     0x00007fff84269e75 objc_exception_throw + 43
    2   CoreFoundation                      0x00007fff83cc8dd6 -[__NSPlaceholderArray initWithObjects:count:] + 358
    3   CoreFoundation                      0x00007fff83d46282 +[NSArray arrayWithObject:] + 50
    4   Keychain Plugin                     0x00000001085e3006 objc_unretainedPointer + 4918
)
20/11/13 17:29:05,769 Quicksilver[27464]: *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[__NSPlaceholderArray initWithObjects:count:]: attempt to insert nil object from objects[0]'
*** First throw call stack:
(
    0   CoreFoundation                      0x00007fff83e1041c __exceptionPreprocess + 172
    1   libobjc.A.dylib                     0x00007fff84269e75 objc_exception_throw + 43
    2   CoreFoundation                      0x00007fff83cc8dd6 -[__NSPlaceholderArray initWithObjects:count:] + 358
    3   CoreFoundation                      0x00007fff83d46282 +[NSArray arrayWithObject:] + 50
    4   Keychain Plugin                     0x00000001085e3006 objc_unretainedPointer + 4918
)

I am unable to reproduce this issue. It may be to do with the contents of the secure note, or an updated version of the plugin. If you're still facing an issue with the latest version of Quicksilver, please create a new issue.
Thanks!


solved bug in contact & faq section

Made the contact us and faq section responsive by adding media queries
closes issue-57
email-<EMAIL_ADDRESS>

@Mehak-Mattoo add "# " before issue number next time, so that The PR will get Linked to the issue

LGTM!


checkout local action in all jobs and shellcheck fixes

The first runs of the workflows failed to some extent:

the combine job missed the checkout that is needed to run ./ this is fixed in this PR ~.. but we won't see the effect in the PR .. but only when its merged~
the setup as in PR jobs failed because the tools and workflows have been added in the same commit .. which violates some assumption of the test, i.e. which tools/workflows changed .. this does not happen normally
[x] the shellcheck test shall also be fixed in this PR .. I hope that I used all those arrays correctly (the syntax appears still a bit odd to me)
[ ] what do you think about renaming master to main in this repo? We would (I hope) only need to change https://github.com/galaxyproject/planemo-ci-action/blob/717a72fc3d823c15a089ae0625af448a67c7b3ca/.github/workflows/tools.yaml#L120 and https://github.com/galaxyproject/planemo-ci-action/blob/717a72fc3d823c15a089ae0625af448a67c7b3ca/.github/workflows/workflows.yaml#L122

I was also thinking

if we should install python in the action (only for the setup and flake8 jobs we would need to have an [additional] python setup step)
if we should have releases for this action?

what do you think about renaming master to main in this repo? We would (I hope) only need to change

Yes, we should. Can we just checkout the default branch, is ref actually required ? I will make you admin of the repo if you are not already.

if we should install python in the action (only for the setup and flake8 jobs we would need to have an [additional] python setup step)

Is that possible ? I was under the impression we can't do this (yet?) in composite actions.

if we should have releases for this action?

Absolutely, and I think deployers should those releases so we can tinker with this without breaking anyone's setup.

what do you think about renaming master to main in this repo? We would (I hope) only need to change

Yes, we should. Can we just checkout the default branch, is ref actually required ? I will make you admin of the repo if you are not already.

if we should install python in the action (only for the setup and flake8 jobs we would need to have an [additional] python setup step)

Is that possible ? I was under the impression we can't do this (yet?) in composite actions.

if we should have releases for this action?

Absolutely, and I think deployers should those releases so we can tinker with this without breaking anyone's setup.

Alright, I changed the default branch to main.

Alright, I changed the default branch to main.

Can we just checkout the default branch, is ref actually required ?

Thinking about it the two lines are nonsense anyway, since:

in PRs we need the pull request branch
when merging we need the default branch

The checkout action should do this automatically. So I will remove them.

Is that possible ? I was under the impression we can't do this (yet?) in composite actions.

Didn't know. Does that mean we can not call other actions? Then we could still install it with apt-get ...

Can we just checkout the default branch, is ref actually required ?

Thinking about it the two lines are nonsense anyway, since:

in PRs we need the pull request branch
when merging we need the default branch

The checkout action should do this automatically. So I will remove them.

Is that possible ? I was under the impression we can't do this (yet?) in composite actions.

Didn't know. Does that mean we can not call other actions? Then we could still install it with apt-get ...

Then we could still install it with apt-get ...

But I don't think we have access to the matrix definition where the python version is being selected. I think it's ok if this needs to be set up in the workflow. Maybe we can try this in the next iteration ?

Then we could still install it with apt-get ...

But I don't think we have access to the matrix definition where the python version is being selected. I think it's ok if this needs to be set up in the workflow. Maybe we can try this in the next iteration ?

Absolutely, and I think deployers should those releases so we can tinker with this without breaking anyone's setup.

Will we go with the Vx.y.z versioning scheme, used for most actions that I have seen so far? Then I will update the IUC action accordingly.

Absolutely, and I think deployers should those releases so we can tinker with this without breaking anyone's setup.

Will we go with the Vx.y.z versioning scheme, used for most actions that I have seen so far? Then I will update the IUC action accordingly.

Sounds good!

Sounds good!

I created v1.0.0. In the IUC PR I get now
Failed to resolve action download info. Error: Unable to resolve action `galaxyproject/planemo-ci-action@v1`, unable to find version `v1`

Did I do something wrong?

I created v1.0.0. In the IUC PR I get now
Failed to resolve action download info. Error: Unable to resolve action `galaxyproject/planemo-ci-action@v1`, unable to find version `v1`

Did I do something wrong?

These are tags, so I think they need to match exactly ... even though I think I have seen inexact matching.

These are tags, so I think they need to match exactly ... even though I think I have seen inexact matching.

Yeah, we need exact tags. I can push a v1 tag that moves while we don't make breaking changes.

Yeah, we need exact tags. I can push a v1 tag that moves while we don't make breaking changes.

According to https://michaelheap.com/semantic-versioning-for-github-actions/ we could use the actions-tagger action to auto forward the v1 tag to the latest v1.x.y version.
I will have a look.

According to https://michaelheap.com/semantic-versioning-for-github-actions/ we could use the actions-tagger action to auto forward the v1 tag to the latest v1.x.y version.
I will have a look.


Add compatibility with StaticPage

I've added compatibility for RainLab.StaticPages.
The StaticPages plugin doesn't append the default locale to the content blocks filename. In other words, it only appends locales for translated files.
Ex:
// English: en (default)
file.htm

// French: fr
file.fr.htm

I've also taken the opportunity to refactor some code in the onRender() function to make it cleaner and more readable.
Let me know if everything looks good or you need me to change a few things.

Wow! thanks, i test it and merge it

Can you please provide some example how to setup static pages with content editor?

The steps are the same except you drop the component to the CMS Layout you use for the Static Page. (instead of the CMS Page)
Then Content Editor will detect if the current page is a StaticPage or a CMS Page. If the current page is a StaticPage it will look for Content Blocks created from the StaticPage plugin, which has a different behavior, stated in my pull request description, for translated content block than CMS content block.
It is not possible to use component inside the StaticPage Page content, so this only enables Content Block created from StaticPage plugin to work with the CMS Layout, and/or Snippets.
Let me know if you require more explanation.

@haleksandre
{% component 'contenteditor' file='static-pages/' ~ page.fileName %} instead of {% page %} in CMS layout for static pages?

{% component 'contenteditor' file='static-pages/' ~ page.fileName %}

That works for me if anyone was wondering


Cancelable

Pull Request Test Coverage Report for Build 35

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.716%

Totals

Change from base Build 32:
0.0%

Covered Lines:
70

Relevant Lines:
122

💛  - Coveralls


crypto/x509: offer a useful error when ParsePKCS8PrivateKey/ParseECPrivateKey/ParsePKCS1PrivateKey or ParsePKIXPublicKey/ParsePKCS1PrivateKey are mixed up

When parsing fails in one of these functions, we could check if one of the others would have worked, and offer a helpful error message.

I can work on this.
So if I understand correctly when unmarshalling to one structure if it fails try the counterpart to see if it succeeds and return a descriptive error.

Correct. Please add me to the reviewers for the change. Thank you!


DockerMachine boot2docker issue

I have installed DOCKER HOST with docker-machine virutalbox and then when i use docker-py client to create a container getting the below error
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', BadStatusLine('\x15\x03\x01\x00\x02\x02\n',)

You need to make sure you configure it for TLS.  See here https://docker-py.readthedocs.org/en/latest/tls/