configfile does not completely overwrite previous config files

Snakemake version
5.27.4
Describe the bug
When there is a config file loaded and another one is loaded through --configfile, the config dict is updated and not overwritten.
Minimal example
With a snakefile:
import json
configfile: "config.yaml"
print(json.dumps(config, indent="\t"))

and a config file:
ref:
  n_chromosomes: 17
  species: saccharomyces_cerevisiae
  release: 100
  build: R64-1-1
  url: ensembl.org

you get (with some errors that are not relevant for here):
$ snakemake -j 1
{
        "ref": {
                "n_chromosomes": 17,
                "species": "saccharomyces_cerevisiae",
                "release": 100,
                "build": "R64-1-1",
                "url": "ensembl.org"
        }
}

but, if you also load this config through the command line:
ref:
  n_chromosomes: 17
  species: saccharomyces_cerevisiae
  release: 101
  build: R64-1-1
  name: NEW

you get:
$ snakemake -j 1 --configfile config_NEW.yaml
{
        "ref": {
                "n_chromosomes": 17,
                "species": "saccharomyces_cerevisiae",
                "release": 101,
                "build": "R64-1-1",
                "name": "NEW",
                "url": "ensembl.org"
        }
}

Shouldn't we expect the config to be fully overwritten (and the key url should not be there)?
If not and snakemake actually merges config files (instead of overwriting them), is there a way to enforce the old one to be overwritten?
thanks,

Shouldn't we expect the config to be fully overwritten (and the key url should not be there)?

No, as the documentation states:

Finally, you can use the –configfile command line argument to overwrite values from the configfile statement. Note that any values parsed into the config dictionary with any of above mechanisms are merged, i.e., all keys defined via a configfile statement, or the –configfile and –config command line arguments will end up in the final config dictionary, but if two methods define the same key, command line overwrites the configfile statement.

But I think that an option to overwrite a config file completely could be useful in some cases.

Exactly. PRs for an additional flag to modify the behavior in this regard are welcome.


[WIP] Implement everything for 0.5

I am working off of the list of issues under 0.5 milestone.
This should address: #158, #75, #139, #196, #74
Changelog

Renames the Buf::take method to Buf::limit to better describe what that method does. The rename was necessary to prevent ambiguity with BytesMut::take. I renamed Take to Limit to match the constructor method name.

Removed Buf impl for Cursor and replaced it with impl Buf for Bytes

Removed Buf impl for Cursor<BytesMut> and replaced it with impl Buf for BytesMut

Renamed Iter to IntoIter to match naming convention of rust where iterators that move the collection are named IntoIter

Adds impl<T: Buf> IntoIterator for T that constructs a bytes::buf::IntoIter struct.

Removs iter method from Buf trait. The existing behavior with respect to ownership matched the convention of into_iter instead of iter. The builtin IntoIterator trait already provides for into_iter, so I decided to avoid ambiguity by just removing Buf::iter instead of a rename. It is up to each container implementing Buf to implement iter efficiently if possible.

Adds Bytes::iter which returns slice::Iter.

Adds BytesMut::iter which returns slice::Iter.

Adds Bytes::copy_from_slice(src: &'a [u8]) which constructs a new

Removed impl<'a> From<&'a [u8]> for Bytes and replaced with impl From<&'static [u8]> for Bytes which does not copy the data.

Adds IntoBuf::len which returns the size of the buffer that would be created by calling into_buf() on self.

Thanks for taking this on!

As a suggestion, I think changing to Buf::limit(n) may be clearer. I'm not a fan of Iterator::take(n), since it sounds like an immediate action, but is actually lazy. Buf::prefix(n) might imply that n bytes are being prefixed before this buffer.
There was a reason for Buf::iter, which was to provide an Iterator as long as you implement Buf. This could be helpful for composite buffer types. However, I agree with your decision. There isn't a need to provide an Iter implementation over Buf, it is more efficient for types to implement their own iterators if they can.

@seanmonstar, you mentioned in
https://github.com/carllerche/bytes/issues/75#issuecomment-372767509 that Bytes and BytesMut should implement IntoIterator. My current patch completely removed IntoIterator impl on these types in favor of a blanket impl for T: Buf.
Is this design acceptable?

For reference, this PR fixes #158, #75

@YetAnotherMinion I think Bytes and BytesMut probably should implement IntoIterator, since it's needed to allow a_bytes_mut.extend(some_other_bytes).

@seanmonstar They do implement IntoIterator through their implementation of Buf.
Apply this patch to this PR branch to see it working.
diff --git a/tests/test_chain.rs b/tests/test_chain.rs
index 8877258..42a16eb 100644
--- a/tests/test_chain.rs
+++ b/tests/test_chain.rs
@@ -48,6 +48,15 @@ fn iterating_two_bufs() {
 }

 #[test]
+fn dirty_hack() {
+    let mut a = BytesMut::from(&b"hello"[..]);
+    let b = Bytes::from(&b"world"[..]);
+    a.extend(b);
+
+    assert_eq!(a.bytes(), &b"helloworld"[..]);
+}
+
+#[test]
 fn vectored_read() {
     let a = Cursor::new(Bytes::from(&b"hello"[..]));
     let b = Cursor::new(Bytes::from(&b"world"[..]));

Oh OK, I get it! That's probably good then.

now also addresses #139

Removed impl<'a> From<&'a [u8]> for Bytes and replaced with impl From<&'static [u8]> for Bytes which does not copy the data.

Is this really a good idea?

While I understand the reasoning this severely limits Byte as a Vec replacement, like backing a https://crates.io/crates/string

@YetAnotherMinion Sorry for the radio silence, it was not cool on my part to drop the ball. This PR is great work and very helpful.
The 0.5 release delay is mostly due to an attempt to coordinate a release with Tokio 0.2 release. Tokio includes Bytes as a public dependency in core crates like tokio-io, which means that users of Tokio would have to stay on 0.4.
0.5 represents a cleanup pass and, as such, doesn't really have a reason to ship on its own. Once Tokio 0.2 gets closer (which I hope to start tracking better), we can revive this PR.

Sorry for it taking so long, we're now actively working on 0.5 things. I've started splitting this PR into specific features, such as #260. I'll close once all have been merged.

Hey, thanks for picking this stuff up again. I ended up using a private fork at work and moved on. It will be nice to switch back over coming soon. If there are still some other issues for 0.5 that need help, I am happy to pitch in.

We've released 0.5!


Weird Android Path Rendering

"react": "16.0.0-alpha.6",
"react-native": "0.44.0",
"react-native-svg": "https://github.com/react-native-community/react-native-svg"

I have path that is supposed to be a 270 degree arc. It looks great on iOS but is not round on Android...
    let pathCommands = "M" + M_X + "," + M_Y +                         "A" + RADIUS + "," + RADIUS + " 270 1 1 " + E_X + "," + E_Y         <Path d={pathCommands}               fill="transparent"               stroke="url(#linear-gradient)"               strokeWidth={STROKE_WIDTH}               strokeLinecap="round"             />
ANDROID:

iOS:

Thanks for reporting this.
Can you print the d prop?

@magicismight The size of the arc is dynamic, so the radius, beginning and end points are also dynamic. d={pathCommands}
const M_X = (CENTER_X) + (RADIUSMath.cos(ANGLE_RADIANS_START))
const M_Y = (CENTER_Y) + (RADIUSMath.sin(ANGLE_RADIANS_START))
const E_X = (CENTER_X) + (RADIUSMath.cos(ANGLE_RADIANS_END))
const E_Y = (CENTER_Y) + (RADIUSMath.sin(ANGLE_RADIANS_END))
let pathCommands = "M" + M_X + "," + M_Y + "A" + RADIUS + "," + RADIUS + " 270 1 1 " + E_X + "," + E_Y

@magicismight, any idea what is causing this?

Are you still having this issue? If you are, then it's probably the path parser in the native code part.

I have the same issue. The arcs are a bit angular, however the browser shows well.
<EMAIL_ADDRESS><EMAIL_ADDRESS>

I'm seeing this as well. Here is the code that reproduces the issue:
html version - makes a nice oval:
<div style="width: 270px; height: 480px;">
    <svg viewbox="0 0 270 480">
        <path d="
          M 0 0 H270 V480 H0z
          M 135 240 m -105, 0
          a 2,3.8 0 1,0 210,0
          a 2,3.8 0 1,0 -210,0z"/>
    </svg>
</div>

react-native + react-native-svg on android - a strange egg-shaped oval:
<View style={{
                position: 'absolute',
                left: 0,
                right: 0,
                top: 0,
                right: 0,
                alignItems: 'center',
                justifyContent: 'center'  }}>
    <Svg viewbox={`0 0 270 480`} width={270} height={480}>
        <Path d={`
                    M 0 0 H270 V480 H0z
                    M 135 240 m -105, 0
                    a 2,3.8 0 1,0 210,0
                    a 2,3.8 0 1,0 -210,0z`} />
    </Svg>
</View>

I can post this as a repository that reproduces the issue too if that would help.
Here is what it looks like on android:

I've had a look through the ios vs android sides for the arc drawing and they look pretty similar - up to the point where they hand off to the native bezier/path code. However, when I looked at what is happening at runtime - there is quite a difference.
The first thing that is noticeable is that this generates 5 calls to arcToBezier on the android side, but only 4 on the ios side. The last 3 calls on each platform seem to be very similar and have the same inputs into the arcToBezier function, so I'll assume they should be the same.
The second thing is the n value in arcToBezier seems to be the crux of the difference. On the second call in ios vs the 3rd call in android, ios has the n value as 2 and android has it at 3. In digging into what it should be I have this:
Relevant line of code from java:
int n = (int) Math.ceil(Math.abs(arc / (Math.PI / 2)));
Correct steps (what I can do on paper):

Math.PI / 2 = 1.57 or PI/2
-PI / (PI/2) = -2
Math.abs(-2) = 2
int n = (int) Math.ceil(2) = 2

Android case (what happens in the code):

Math.PI / 2 = 1.57 or PI/2
-PI / (PI/2) = -2.0000000556550703
Math.abs(-2.0000000556550703) = 2.0000000556550703
int n = (int) Math.ceil(2.0000000556550703) = 3

It looks like this is a classic case of floating point math. One solution would be to round the inner result:
- int n = (int) Math.ceil(Math.abs(arc / (Math.PI / 2)));
+ double arcOverHalfPi = arc / (Math.PI / 2);
+ // no round-to-decimal-places function in java so we have to multiply and divide by 10^n to get n decimal places
+ double roundedArcOverHalfPi = Math.round(arcOverHalfPi * 1000) / 1000D;
+ int n = (int) Math.ceil(Math.abs(roundedArcOverHalfPi));

That fixes the egg shape. The peaks are still too sharp on android compared to iOS though.

Hi @administerspice, you have a really cool screenshot in your initial post. I'm looking to do something similar. Don't you mind sharing the snippet of your code showing how you did it?

Fixed in #704


Missing documentation for Zend\Validator\DateStep

The validator "DateStep" needs some explanations and code examples.
https://github.com/zendframework/zend-validator/blob/master/src/DateStep.php

This repository has been closed and moved to laminas/laminas-validator; a new issue has been opened at https://github.com/laminas/laminas-validator/issues/18.


[cdkDragDrop] override scale porperty so can't use with scale

Please describe the feature you would like to request.
Please support somehow the scale of dragDropContainer.
What is the use-case or motivation for this proposal?
I have too many content on a page, and i have a scaler function with drag and drop and the dragDrop kill my scaler :(
Is there anything else we should know?
Stackblitz link for example: https://stackblitz.com/edit/angular-na8k5p?file=app%2Fcdk-drag-drop-custom-placeholder-example.css

It doesn't work, because the preview element isn't under the drop list in the DOM hierarchy which means that the transform doesn't apply to it. You can use the cdk-drag-preview class to style the preview while it's being dragged. That being said, styling the preview itself won't work for your case, because we use transform to move the preview around on the page. You might be able to work around it by using a custom preview.
This is definitely a valid issue, however I have to close it, because there's no performant way to solve it from the CDK.


Remove heroku free tier

Heroku announce they are removing their free tier offerings.
source: https://help.heroku.com/RSBRUH58/removal-of-heroku-free-product-plans-faq

Indeed 😢
Done


Possible bug in Cinch's hooks on random plugins

Hi,
I'm using Cinch's hooks in yossarian-bot to drop messages from ignored users. The blacklisting code is as follows:
module Cinch
	module Plugin
		module ClassMethods
			def use_blacklist
				hook :pre, :for => [:match], :method => :not_blacklisted?
			end
		end

		def not_blacklisted?(m)
			!@bot.blacklist.include?(m.user.nick) && !@bot.blacklist.include?(m.user.host)
		end
	end
end

Every plugin calls use_blacklist after include Cinch::Plugin, but this seems to fail deterministically for a random group of plugins (meaning that the plugins that fail to hook stay the same between runs, but aren't related to each other in any way I can distinguish).
For example, the HAL plugin fails to hook the blacklist while the Magic8Ball plugin hooks it perfectly.
Could this be a bug in Cinch's hooking system? I'm having trouble finding a cause in my own code (use_blacklist gets invoked, execution passes cleanly through it for both working and non-working plugins).
I'm using Cinch 2.2.7 on Ruby 2.2.3 (MRI) for reference.
Best,
William

Is not_blacklisted? being called?
I cannot reproduce the issue. You would have to provide a minimal example, that is remove as much code as possible while still encountering the bug. Once the minimal example has been built, I can check if the error can be reproduced locally. yossarian-bot, as it is, is too large for me to test.

not_blacklisted? is not being called.
I'll try to set up a minimal example later today.

Sorry for the late response. I've been testing a few minimal examples, and I haven't been able to reproduce the bug.
I'll close this unless I can reproduce it in the future.

@woodruffw c8ed88bec15f0afa09c8198dd5b9d560f2094137 should fix your issue.

Great, thank you so much!


Fix proof direction and key bit order

Fixes #20 and conforms to https://github.com/lazyledger/lazyledger-specs/pull/129.

Looks like everything works as of c5a394b except for deletions. Looking into it now.


HA Energy Dashboard No Longer Displaying Duke Electric Usage

Version of the custom_component
When I look at a device, the about says it is v1.0.0, but HACS shows the version as 0.1.2.
Configuration
It is connected to my account and has been working fine for quite a while.
Describe the bug
I noticed a couple of days ago that my energy usage was no longer being reported in the HA tab.
I think that the timing may have correlated with my upgrade to HA 2022.11.0.
It appears to me from the debug logging (see below) that it is receiving data from the Duke service. So I'm wondering if something about the interface to HA was changed in 2022.11.0 which is causing this integration to not properly pass the data to HA.

I'm currently running Home Assistant 2022.11.1 and version 0.1.2 of the integration and I'm not seeing this issue.
There are 2 different API interaction methods in this integration. One is the realtime push which is the Current Usage entity, which definitely is getting updated as you can see that in the logs. The other is a polling call to the API which shows in the log as a request but for some reason isn't updating

Requesting usage between 2022-11-05T04:00 UTC and 2022-11-06T04:00 UTC

There's no error message so it seems like the API is returning 0 maybe.
I'm not sure if they have made some change to the API that is on some sort of A/B mode so I'm not able to see the same issue

I think it is some problem with Duke's server(s). When I go into their app on my phone, it is showing the smart meter gateway is online. But when I try to display Energy Usage for the current day, I get an error saying "Sorry, but your data is unavailable at this times. Please select a different time frame or check back later." If I go and look at yesterday there, it is showing me data.
I guess I'll just wait a little while longer and see if it starts working again.

Strange, I think people have had issues similar in the past and they've had to re-add their gateway.
If you want, you can try using this web app I made that calls the APIs directly and might provide more insight: https://pyduke-energy-test-app.up.railway.app/
However not sure if it will provide any good insight

Thanks for helping try to troubleshoot.
I tried you test app. It ends up just displaying “ERROR: “ ☹
I did also log into my account on the Duke website. It is only showing usage data through yesterday, too.
When you said “they’ve had to re-add their gateway” did you mean re-add the gateway in their Duke account or re-add the integration and re-enter credentials?
-Brett
From: Michael Meli @.>
Sent: Monday, November 7, 2022 3:37 PM
To: mjmeli/ha-duke-energy-gateway @.>
Cc: Brett Bowman @.>; Author @.>
Subject: Re: [mjmeli/ha-duke-energy-gateway] HA Energy Dashboard No Longer Displaying Duke Electric Usage (Issue #148)
Strange, I think people have had issues similar in the past and they've had to re-add their gateway.
If you want, you can try using this web app I made that calls the APIs directly and might provide more insight: https://pyduke-energy-test-app.up.railway.app/
However not sure if it will provide any good insight
—
Reply to this email directly, view it on GitHubhttps://github.com/mjmeli/ha-duke-energy-gateway/issues/148#issuecomment-1306155690, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEEJW7ORWCTULI5APPTDIN3WHFR5XANCNFSM6AAAAAARYCE4KY.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Usage data through yesterday is the default data that comes from a regular meter without the gateway. It sounds like the realtime data isn't getting through (which is weird how the realtime stream is working).
Ultimately I just reverse engineered the API that the app is making, so if the app isn't working then the integration isn't going to work
For the re-adding, I mean they are re-adding it to their Duke account, so basically running the setup again

Well, I ended up trying rebooting the smart gateway. And now I’m getting data again! Don’t know why it wasn’t working before, but glad it’s back now.
Thanks again for your help!
-Brett
From: Michael Meli @.>
Sent: Monday, November 7, 2022 4:23 PM
To: mjmeli/ha-duke-energy-gateway @.>
Cc: Brett Bowman @.>; Author @.>
Subject: Re: [mjmeli/ha-duke-energy-gateway] HA Energy Dashboard No Longer Displaying Duke Electric Usage (Issue #148)
Usage data through yesterday is the default data that comes from a regular meter without the gateway. It sounds like the realtime data isn't getting through (which is weird how the realtime stream is working).
Ultimately I just reverse engineered the API that the app is making, so if the app isn't working then the integration isn't going to work
For the re-adding, I mean they are re-adding it to their Duke account, so basically running the setup again
—
Reply to this email directly, view it on GitHubhttps://github.com/mjmeli/ha-duke-energy-gateway/issues/148#issuecomment-1306211665, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEEJW7IUTMO5KK7ZMMXV4G3WHFXMBANCNFSM6AAAAAARYCE4KY.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Good to hear! I'll have to remember that in case someone else has that issue


Fixing search query being lost after page change

When I made a query filtering the forms and there were more than one page, the query was lost when I changed the page.

Looks good. Thanks!


Add email validator option

The default email address validator does not support all valid email addresses.
This commit supports the usage of a email validator argument.

I'm sorry that my editor struggled with all the whitespace.

I can see two problems with this pull request:

The new validator argument for the envelope_email function isn't being used
I'm not sure this part should really be configurable. Address validation with regexes is known not to be perfect in any case. We can provide the one that handles 99% cases or think about validating some other way.

Ideas?

@00dingens Any thoughts?

"I'm not sure this part should really be configurable. Address validation with regexes is known not to be perfect in any case. We can provide the one that handles 99% cases or think about validating some other way."
Exactly.
And the one in use right now (see https://github.com/kamilc/mailman/blob/master/lib/mailman/external_smtp_adapter.ex#L31 ) is not sufficient in "our" cases (eg, it does not allow a TLD such a .agency, .blackfriday or .guitars. since each project has their own needs, a configurable email-validator is something nice to have.

@mgumz @00dingens I've just removed the validation we have at the smtp level and pushed. We'll figure out some configurable way of adding that back in. Thanks for your suggestions and time!


LUCENE-9943 DOC: Fix spelling (camelCase it like GitHub)

Description
Please provide a short description of the changes you're making with this pull request.

[x] Docs have been updated

Solution
Please provide a short description of the approach taken to implement your solution.
docs update => spelling: github
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Lucene maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.

Other information:
Signed-off-by: Ayushman Singh Chauhan<EMAIL_ADDRESS>

Thanks!


Cramped style not respected

Issue Summary
I find this surprising enough to think that maybe I'm doing something wrong (and I didn't find an issue about this)...
TeX has a notion of "cramped" which causes superscripts to be lower in a number of cases such as in accents, overlines, fractions (denominators), and subscripts.  Here's a latex rendering:

The position of the '2' in x^2 is lowered in first instance except for the fraction, where it is the second instance (the denominator).
However, MathJax doesn't reduce the superscript position. Here's a codepen that shows that. Here's a screen shot of that:

[To be honest, my sons happened to be looking at what I was doing and liked the MathJax rendering better...]
Steps to Reproduce:
This is the header
<script src="https://polyfill.io/v3/polyfill.min.js?features=es6"></script>
  <script type="text/javascript" id="MathJax-script" async
<EMAIL_ADDRESS></script>

and this is the input
\[ \hat{x^2} x^2\]
\[ \overline{x^2} x^2 \]
\[ \frac{x^2}{x^2} \]
\[ a_{x^2}a^{x^2} \]

Thanks for the report.  MathJax is supposed to handle the cramped styles (what is listed as "prime" styles in appendix G of the TeXBook).  In version 2, properties like "primestyle" where stored in the same place as the MathML attributes, and that caused difficulty in distinguishing the two, so in version 3, they are stored in two separate places.  Unfortunately, when translating the code that determines the superscript offsets to version 3, I neglected to switch the location, so was looking for the prime style in the wrong place.  So it never appeared to be set.
I will make a pull request to fix the problem.  It also turns out that there was a problem handling the prime style in display mode, which I will also fix.


Update linkml-quickstart.md

Add section on how to determine granularity of mappings.

@krausf Here is a set of guidelines on how to determine the granularity of mappings.


Bug in Autocomplete When certain characters are inserted (try  ] or [)

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: v1.5.2.

Platform and Target: Android and IOS.

If you try to input only a single character such as ] in the input field of autocomplete, an error will occur (console shows the error below).
Uncaught SyntaxError: Invalid regular expression: /([)/: Unterminated character class at new RegExp (<anonymous>)     at framework7.js:8987     at Object.source (autocomplete.html:152)     at HTMLInputElement.a (framework7.js:8982)
This issue is probably related to https://github.com/nolimits4web/Framework7/issues/1258
Framework7 attempts to highlight the inputted characters in all the items of the autocomplete dropdown menu items. Certain characters cause an issue with regex expression used for highlighting.
It would be nice to give the users the option of disabling the highlighting altogether as it is really useless in certain languages where characters take different shapes when they are inside words than when they are outside.

looks like it needs to be escaped

True. However, giving the option of avoiding regex expressions and disabling characters highlighting would be a nice option, too.

i believe such a feature was merged in yesterday

That's great. However, the regex line causing the error reported in this issue is still being executed even when highlightMatches is set to false. It would be nice to have Framework7 escape the string or avoids the declaration of the following line when highlightMatches is set to false.
var regExp = new RegExp('('+query+')', 'i');


Git stage selected ranges doesn't work without when condition in a specific case

VSCode Version: 1.52.1
OS Version: Win 10

It's hard to describe, please watch the reproduce gif.
Steps to Reproduce:

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

I'm not sure I'm understanding the exact steps here. Can you please list out the steps you are performing, and what the issue is you are running into and what you expect?

@eamodio Hi, sorry if the gif confused you, I thought it would describe better than words.
The issue is if the Git stage selected ranges's When condition is unbound (which is isInDiffEditor by default), still I can only stage selected changes through diff editor, in the normal editor, it wouldn't work.
This is understandable, but given isInDiffEditor is already unbound, this limitation should be removed as well.

I'm experiencing the same issue. The staging works, but it's not reflected in the diff editor, same as shown in the GIF above.


Error 4: No Clients are active

Error: 4
s.update_client
No clients are active - exiting
Followed the readme correctly so i'm not sure what's going wrong
Is it possible that it doenst work because of the new update to the game?

I forgot to change game version


Fix nil dereference error found during sanity test runs

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build 864650923

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

Totals

Change from base Build 858985996:
0.0%

Covered Lines:
1103

Relevant Lines:
2997

💛  - Coveralls


Invert image?

Hi! Any plans to support invert-image?
Smth like turning this to white bg would be super-useful

Hi!
I can add this feature to the extension.
Thanks for feedback!

Update will be available today or tomorrow

Great news!

v3 is available 🎉

Hm. In some reason it hangs, but if I leave checkbox only on image - it does nothing.

Could you provide additional information and screenshots or screen recordings


fix(helm): fix search bugs

This fixes three things:

#741 output is wrong
#743 nil pointer error because of unchecked error
Printing "Chart:" string in output

Closes #741

Closes #743

Took off lgtm since this failed but I did check it out and test it. It worked nicely.

GOOGLEBOT!!!!!


Fixes for scrolling

This PR introduces a number of fixes for scrolling, primarily for support of inline inlays, such as a parameter hints.

[x] Fixes VIM-2104 - scrolloff instead of sidescrolloff is used for horizontal offset
[x] Fix scrolljump not working
[x] Improve handling of scrolljump to match Vim's fairly unintuitive rules
[x] Fix VIM-1080 - handling of scrolling with virtual space
[x] Add internal action to add test inline inlays, using the parameter hint renderer
[x] Improve handling of inline inlays, especially around positioning after deletion, and also handling inlays associated with preceding text (e.g. Kotlin type annotations, as opposed to inlays associated with following text, such as Kotlin parameter hints)
[x] Fix issues with side scrolling and inline inlays, including sidescrolloff and sidescroll(jump). Leading and trailing inlays are correctly positioned when scrolling offscreen.

Fixes VIM-1556 - Editor window is not correctly scrolled horizontally
Fixes VIM-1770 - Go to end of line conflicts with parameter name hints

[x] Fix VIM-2110 - support for line scrolling with proportional fonts
[x] Implement scroll screen half width actions (zL/zH)

Added tests:

[x] Add tests for vertical scrolling with scrolloff and scrolljump
[x] Add tests for scrolling up/down by line (<C-Y>/<C-E>)
[x] Add tests for scrolling up/down by page (<C-B>/<C-F>)
[x] Add tests and fixes for scrolling page (z+/z^)
[x] Add tests for scrolling caret line to top/bottom/middle of page (zt/zb/zz + z<CR>/z-/z.)
[x] Add tests for scrolling half page up/down (<C-U>/<C-D>)
[x] Add tests for scrolling caret column to first/last column of page (zs/ze)
[x] Add tests for scrolling screen one column to left/right (zl/zh)
[x] Add tests for delete character left/right (+inlays) (X/x)

Known issues:

Virtual space handling doesn't exactly match Vim (e.g. <C-F> at the end of file doesn't scroll up), but it at least behaves sensibly
Side scroll offset is calculated on visual column to correctly handle closed inline folds, but both inline inlays and folds are implemented as visual columns, which means the offset isn't technically correct - it is not an offset of characters. The alternative is to iterate over visual columns and skip columns that are folds or inlays. I think this minor inaccuracy in a feature designed to show additional context is acceptable.

Questions:

Now that virtual space is working, shall we enable it for all Vim-enabled editors? This would match Vim behaviour, but might be a little intrusive, so we could also add an option to disable it (it would be on by default to match Vim)

Yeah, apologies for the size of the PR, it just kept on growing :smile: Horizontal scrolling touches a lot of places and it took a lot of new tests to make sure it was done.
As for virtual space, yes, I mean that we can now enable IJ's virtual space - both the "Show virtual space at the bottom of the file" option, and the "Allow caret placement after the end of line". This would give us behaviour in line with Vim. It would also allow us to implement the other virtualedit options.
My only concern is if it's too big a change for the user - we'd be overriding the settings, so there's no way to disable this. Perhaps we should add an ideavirtualspace setting, or a virtualedit=noideaspace option (which is ignored if all, block, insert or onemore is also set)? Or just implement it and see if anyone complains.
Whatever the answer, it should go in a separate PR :grin:

No, it's absolutely fine. And I like the test coverage, sometimes contributors skip it :)
Okay, I get what you mean about virtual space. Yes, let's start with this PR and deal with virtual space later.


Norconex - Class GenericHttpClientFactory

Hi,
I am trying to crawl a site which require 2 step authentication.
Firstly, a small window appears which requires filling out with login and password, then I am redirected to a site which also requires providing login and password. I am able to run the crawler which requires only one step authentication but when I have to crawl the site with 2 authentications I am not sure which extension / options from this site I should use: https://opensource.norconex.com/collectors/http/v2/apidocs/com/norconex/collector/http/client/impl/GenericHttpClientFactory.html
Any hints ?

What type of authentication is it? BASIC?  To be clear, are you encountering two different sites asking to authenticate, or the same site required two-factor authentication just to get in?
If the first, the best approach would be to create two different crawlers in your collector config.  One for each site.  Then you can have different authentication for each.
If the second, I would have to see it.  If really two-factor authentication, they usually involve a random piece, like a code sent to you for confirmation that you have to provide back.  Those are to ensure unique access, which excludes crawlers.
Can you share your site maybe?  Or can you provide more details on the authentication mechanisms?

Hi,
I would like to crawl only one site: https://sites.google.com/roche.com/biissearchsquad
This site asks for my email and if the given email is correct I am directed to the other site which also require authorization: https://wamqa.roche.com/
If there is only one step authorization - https://wamqa.roche.com/ I am able to run the crawler but if there is two steps authorization - first email, then wamq - I don't know how to do that.

If I understand you well, each site can be accessed via one step authorization?  It is only when the crawler navigates from one site to the other that it becomes an issue, right?  If so, I would try my earlier advice and have a different crawler for each site, each having their own authentication mechanisms.  Make sure they do not crawl each other by using reference filters, or stayOnSite being true.
If my understanding is wrong, please clarify.  Without being able to experience it for myself, it is difficult to give you sound advice.

https://sites.google.com/roche.com/biissearchsquad - site can be accessed via two step authorization.
First you need to give correct email then you are directed to other site which also requires from you credentials.

There is currently no built-in support for multiple authentications for the same crawl.  Unless the content of the site you are interested in can be accessed some other way, I am afraid you will have to use a custom solution. You may have to create your own  IHttpClientFactory implementation or extend GenericHttpClientFactory.  Basically, you will have to code the flow yourself.  Not knowing the inner workings of your authentication mechanisms, I am not sure how hard/feasible you will find it.
There is also the possibility of simulating user interaction with a browser-based crawl, but I am not sure it would be any simpler.  Version 2 supports PhantomJS for this, but it is a bit outdated.  Version 3 (only available as snapshot) allows you to use any popular browser.

Hi,
So, Norconex can not crawl a site that has two steps authentication unless I extend  GenericHttpClientFactory?
Have you extented GenericHttpClientFactory to crawl such a site ? If yes, do you have an example how to do that ?

Sorry, I do not have a code example specific to your use case.
If you want to consider alternatives to extending GenericHttpClientFactory, you can investigate using browser-based crawling, mimicking user interaction.  Version 2 offers an HTTP client based on PhantomJs, but PhantomJs is no longer supported so I am not sure how far you will get.  HTTP Collector Version 3 supports popular browsers if you are comfortable dealing with code still in development (snapshot version).
Can the site you are crawling be adapted to accept requests from a specific "user-agent" or IP?  This could be a way to facilitate crawling.


Workshop version is stale

We should update the workshop version of the mod.
We should also consider adding continuous delivery.

Release pipeline written, need to write the github action now

Pending because we don't need to publish newer versions yet


normalize grads option instead of trust region, defaults

overall changes:

switch default flat start for update prob schedule in utils.py
normalize_grads replaces trust region clipping (let it default to identity in PSGDBase)
change some defaults (max size triangular -> 8192, stochastic=False for krons with momentum)

The main thing is moving away from trust region clipping on outgoing updates and opting for optionally controlling incoming grads instead. This also allows the user more control over this as they can clip grads as they see fit before the optimizer.

Made a couple other changes: add normalize_grads to test_psgd.py, and change norm_clip_ in utils.py from frobenius norm to RMS norm since outgoing updates should be brought to near 1 instead of unit norm.
This makes normalize_grads normalizing incoming grads to unit norm, and clip_fn=norm_clip_ normalizing outgoing updates to around 1.

I think the old way of trust region clipping using log could be a foot gun because it changes the directions of the updates. It should probably be phased out in favor of either normalizing incoming grads to unit norm with normalize_grads=True, or normalizing outgoing updates with clip_fn=norm_clip_ (similar to SOAP) which would normalize to near 1.0 using RMS norm. Neither of these would change the direction of grads or updates.


[loc][Query][VSCode] Please clarify string

Hi, we have a question from our translator. Can you help please? Thanks!
Source:
{0} Errors and Warnings
File Name:
src\vs\workbench\parts\markers\browser\markersWorkbenchContributions.nls.resx
Resource ID:
".resx", 0; "errorsAndWarnings"
Query:
Please clarify placeholder<EMAIL_ADDRESS>10/5/2016 9:01:08 PM

@sandy081 you need to add comments in the nls.localize call so it is clearer for the translators what does 'errors and warnings' mean in this context.
One example where I do this is here

@mkyjakova Placeholder here represents the count (number) of errors and warnings.
@isidorn Thanks for showing me providing the comments in the code for translations. Actually, this string was not introduced by me :).
Anyways added the comment to the string too.
Thanks.


Make Derivation & BrainLocation blank nodes

Both Derivation and BrainLocation shape validation requires a class @type to pass validation:
https://bbpgitlab.epfl.ch/dke/apps/brain-modeling-ontology/-/blob/develop/shapes/commons/entitywithoutannotationsubject.json?ref_type=heads#L131
https://bbpgitlab.epfl.ch/dke/apps/brain-modeling-ontology/-/blob/develop/shapes/commons/entitywithoutannotationsubject.json?ref_type=heads#L206
This change is backwards compatible with older resources that are still Frozen, using the class type as @type where missing.

Codecov Report
Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 81.25%. Comparing base (b3161b8) to head (2192208).

Files
Patch %
Lines

entity_management/base.py
66.66%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   81.35%   81.25%   -0.11%
==========================================
  Files          22       22
  Lines        1427     1430       +3
==========================================
+ Hits         1161     1162       +1
- Misses        266      268       +2

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


Setup initial PyJWT 1.7.1 support

Supporting older PyJWT support due to many old open source packages.
Fixes #464

Tests will need to be... repetitive.

@2ykwang may you please review this? I've disabled the locale updater.

yea not working. I think it doesn't have perms to create a branch here for some reason. I think it would maybe have perms for a fork?


Remove unused status.state from blockdevices

Related: harvester/node-disk-manager#32
Signed-off-by: Weihang Lo<EMAIL_ADDRESS>

Wondering if we use this field in the v1.0.0 NDM CRD resources, if yes probably it will encounter a backward capability issue.

Sorry I didn't consider the compatibility issue. I'll keep the field as-is. Thanks.


Add encodedAsBase64 to AbstractByteArrayAssert

Check List:

Related to #1805
Unit tests : YES
Javadoc with a code example (on API only) : YES

@scordio that looks good to me, I let you merge the PR in case there's something you still want to add.


xiu: 0.12.7 -> 0.13.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/harlanc/xiu/releases.
meta.description for xiu is: Simple, high performance and secure live media server in pure Rust (RTMP[cluster]/RTSP/WebRTC[whip/whep]/HTTP-FLV/HLS
meta.homepage for xiu is: https://github.com/harlanc/xiu
meta.changelog for xiu is: https://github.com/harlanc/xiu/releases/tag/v0.13.0
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.13.0 with grep in /nix/store/h67ni7arg6zppw25v9ald5g2ivbii24h-xiu-0.13.0
found 0.13.0 in filename of file in /nix/store/h67ni7arg6zppw25v9ald5g2ivbii24h-xiu-0.13.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

xiu

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/h67ni7arg6zppw25v9ald5g2ivbii24h-xiu-0.13.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xiu https://github.com/r-ryantm/nixpkgs/archive/5811b36ef4ef75091c98d787f09e7fae64ad52a4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5811b36ef4ef75091c98d787f09e7fae64ad52a4#xiu

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/h67ni7arg6zppw25v9ald5g2ivbii24h-xiu-0.13.0
ls -la /nix/store/h67ni7arg6zppw25v9ald5g2ivbii24h-xiu-0.13.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

xiu

Maintainer pings
cc @cafkafk for testing.

Add a :+1: reaction to pull requests you find important.

should solve #332957 btw, linking so it gets updated in the list


adds flushes to the output stream of teamcity reporter,…

… making the test output more responsive.
Description
There is the issue with the teamcity reporter described here:
https://github.com/philsquared/Catch/blob/master/docs/build-systems.md#teamcity-reporter
Tests are only reported after the whole suite is completed.
This can be fixed by explicitly flushing the output stream of the reporter.
I added flush commands to the output functions of the reporter.
Now (long) running tests are reported in Teamcity more quickly, almost as soon as the tests are completed.
If these flushes would create a performance issue, additional code could be added to only flush every some seconds. But in my setup it did not seem necessary.

This is a potential issue with streaming reporters in general and we have had to fix it for the XML reporter earlier in the year.
Sadly without NVI-based rework of the reporters, it has to be fixed in each reporter individually.


Introduce TokenGenerator interface and randomTokenGenerator type

What this PR does:
This PR introduces:

TokenGenerator interface, which exposes the GenerateTokens() function for generating unique and sorted tokens.
randomTokenGenerator type implementing the TokenGenerator interface, and which generates unique, random and sorted tokens.

Implementation of randomTokenGenerator.GenerateTokens() replaces util.GenerateTokens().
In the following PRs related to this issue, an additional implementation of TokenGenerator interface will be provided.
Which issue(s) this PR fixes:
Part of https://github.com/grafana/mimir-squad/issues/1042
Checklist

[x] Tests updated
[ ] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

n the following PRs related to this issue, an additional implementation of TokenGenerator interface will be provided.
Which issue(s) this PR fixes: Part of grafana/mimir-squad#1042

Note that dskit repository is public, while mimir-squad repository is our internal private repository, and users outside of Grafana Labs org cannot access it. Your description should explain the need for this change without referring to internal repositories.

n the following PRs related to this issue, an additional implementation of TokenGenerator interface will be provided.
Which issue(s) this PR fixes: Part of grafana/mimir-squad#1042

Note that dskit repository is public, while mimir-squad repository is our internal private repository, and users outside of Grafana Labs org cannot access it. Your description should explain the need for this change without referring to internal repositories.

I have removed the issue, because the present PR actually does not do anything related to it, but just prepares the environment for it.


Certifier OSV: fixed emit func when polling

Description of the PR
Fixes #1387
New Goroutine to "flush" the documents collected in the totalDocs based upon one condition being true:

a 30 seconds ticker
1000 documents appended to totalDocs

This also fixes the other bug, i.e. not exiting with CTRL-C
PR Checklist

[X] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[X] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

2efd048 :

refactored totalDocs slice management that now happens only inside ingestion func
ingestionStop chan to properly stop ingestion func
removed last call to ingestor.MergedIngest(ctx, totalDocs, opts.graphqlEndpoint, csubClient) because now all the ingestor invocations happen only within the  ingestion func
ingestion func deals with wg WaitGroup


docs: Added md file that describes the tags for our commits

COMMIT.md contains the structure for how we should add tags to our commits to make them easier to understand.
Solves Issue #40

LGTM


How to use the waitForFunction() method?

My code:
try {
  const InputWhatsOnYourMind = await page.$("#mbasic-composer-form > table > tbody > tr > td._4g34 > div > textarea");
  InputWhatsOnYourMind.type(text)
  await page.waitForTimeout(1000)

  const buttonSubmit = await page.$("#mbasic-composer-form > table > tbody > tr > td:nth-child(3) > div > input");
  buttonSubmit.click()
  await page.waitForNavigation()
} catch (error) {
  console.error(error);
}

The waitForTimeout is not recommended, but what else to use then? If I don't wait, then the text won't be sent in full. I suspect waitForEvent() and waitForFunction() are my choices, but I don't know how to implement them. I take a guess, suggested from this same question on Puppeteer, and modify to match the number of arguments of it:
await page.waitForFunction(() => InputWhatsOnYourMind.value === text);

But it doesn't work. Do you have any idea?

waitForFunction runs the code inside the page. It doesn't have access to InputWhatsOnYourMind or text unless you pass it in as an argument.

How to pass it as an argument? Either:
await page.waitForFunction((InputWhatsOnYourMind, text) => InputWhatsOnYourMind.value === text);

or
await page.waitForFunction(({InputWhatsOnYourMind, text}) => InputWhatsOnYourMind.value === text);

doesn't work.

I believe you should write it like
await page.waitForFunction((InputWhatsOnYourMind, text) => InputWhatsOnYourMind.value === text, {
    args: [InputWhatsOnYourMind, text]
});

I'm not sure if passing selectors into the waitForFunction works. If not, then you should do
await page.waitForFunction((text) => document.querySelector("#mbasic-composer-form > table > tbody > tr > td:nth-child(3) > div > input").value = text, {
    args: [text]
});

The first one returns:
DeadlineError: Deadline
    at https://jsr.io/@std/async/0.223.0/deadline.ts:60:32
    at eventLoopTick (ext:core/01_core.js:214:9)
    at async Page.waitForFunction (https://jsr.io/@astral/astral/0.4.8/src/page.ts:661:5)
    at async file:///D:/QC supplements/Code/Apps/Xây nhân hiệu tự động/astral.ts:37:3 {
  name: "DeadlineError"
}

The second one seems to have no effect. That is, it immediately clicks the submit button and doesn't wait for the text to be finished.

I'm not understanding how it's possible for the result to repeat the first work. BTW, the correct way of awaiting should be
const inputSelector = "#mbasic-composer-form > table > tbody > tr > td._4g34 > div > textarea"
const inputElement = await page.$(inputSelector);
await Promise.all([
  page.waitForFunction((InputSelector, text) => document.querySelector(InputSelector)!.value = text, {
    args: [inputSelector, text]
  }),
  inputElement.type(text)
]);

Hopefully selectors will make this more ergonomic.

Thanks. I forgot that element methods need to be await as well. This solves the problem. I'm still curios with why it happens though.
Anyway, do you think that open a discussion tab on github will help separate discussions on the usage and discussions on the code?

Sure, I just opened them.


[BULK] - DocuTune remediation - Sensitive terms with GUIDs

Applying sensitive terms with GUID changes as part of Content SFI and outlined in Overview - Writing content securely - Platform Manual. Changes are part of the Microsoft-wide SFI effort. Point of contact: @CelesteG
DocuTune v<IP_ADDRESS>
CorrelationId: ac15aa43-4e2b-437f-ab1c-fdd7e79cd4db
#docutune

#docutune-review

@gewarren - Looks like Chris reverted the changes. Should we just go ahead and close this PR? I've added the GUID 22222222-2222-2222-2222-222222222222 to the allow list since it wasn't nominated and discovered until now.

@gewarren - Looks like Chris reverted the changes. Should we just go ahead and close this PR? I've added the GUID 22222222-2222-2222-2222-222222222222 to the allow list since it wasn't nominated and discovered until now.

Those changes still need to be committed and merged.

PR needs to be closed. I've added the GUID to the allow list.
#please-close
CC @gewarren @meganbradley
FYI @cilwerner @OwenRichards1


Hack together using toltecmk for the build process

[x] Rework makefile to use virtual environment
[x] Hack in toltecmk to the build system and remove all unused old python code
[ ] Clean up implementation to simplify hacked in logic
[ ] Update github actions to cache the virtual environment pip install

Currently waiting on https://github.com/toltec-dev/build/pull/39 and toltecmk 0.3.0 to be release.

That should be cleaned up enough for now.

Blocked waiting on https://github.com/toltec-dev/build/pull/49 and a new release.

Opened #867 to resolve mmc-utils no longer building.

netsurf fails to build.
webinterface-upload-button's sha256sum no longer matches


i18n - Fix challenge format for Crowdin

Within the Nest an Anchor Element challenge the text which reads:

The text, link to freecodecamp.org, within the a element is called anchor text, and will display the link to click:

is incorrectly formatted - "anchor text" is within backticks and is thus untranslatable on Crowdin.
The text in the English file should be updated to use <dfn> tags instead:

The text, link to freecodecamp.org, within the a element is called anchor text, and will display the link to click:

This issue is open to first time contributors for this repository. If you are interested in contributing, we recommend you start by reading our contributing documentation. And if you run in to any issues or have any questions, you can reach us in our contributors chat room.

Hi @nhcarrigan . I would like to work on this.

Hey there @pandacoder7,
Thank you for your interest in contributing. Note that we do not typically assign issues except to long-term contributors. Instead, we generally review and merge the first pull request which comprehensively resolves the issue.
Feel free to let us know if you have any questions. 🙂


change name of variable "type"

The variable "type" is a reserved keyword in Python and should not be used elsewhere. Changing the name of this variable to "feed_type" or whatever is appropriate everywhere in the code is a good starting issue for a new contributor

I will do that sir, kindly assign it to me.

go for it.

I believe this is no longer an issue since #164


Support for Alpine on aarch64

On Alpine (3.19.1) on an aarch64 machine, I get:
# curl -LsSf https://astral.sh/uv/install.sh | sh
ERROR: there isn't a package for aarch64-unknown-linux-musl-dynamic

It is necessary (or even supported) to compile uv from source against musl on aarch?
On x86_64 (thanks to fixing https://github.com/astral-sh/uv/issues/1427) I have no problems on Alpine.

This is an issue with our installer, see https://github.com/astral-sh/uv/issues/1392. I think you can work around it by downloading the binary from GitHub releases in the meantime.

Thank you for the quick response!


How to exclude a volume that is configured in de VOLUME parameter in Docker file

Currently I depend on a docker image that I directly use from docker hub. In the Docker file there is a VOLUME parameter that lists 2 volumes. But in our scenario we would like to mount only one particular volume via rex-ray to an external volume (in this case an EC2 volume on AWS).
When trying to launch the docker image I always get issue when I only map one volume that is listed in the Docker file VOLUME parameter. When I map both of them there isn't an issue.
So my question: does rex-ray require all the volumes that are listed in the Docker file VOLUME parameter to be mapped to an external volume path? Can this be overruled during launching of the docker image.
The only other option that I have is to fork the git repo and remove the specific volume from the VOLUME parameter in the Docker file.
While reading the rex-ray documentation I couldn't find a clear answer on this.
Kind regards,
Daniel

@Krieke this is not a problem with REX-Ray but how Docker handles the docker run command with a --volume-driver parameter. Today, if you are using the docker run command and the --volume-driver flagged is used, it will create an anonymous volume on the storage backend if a mapping is not listed as -v myvol:/path/in/container. There is no way to specify using only 1 of the VOLUMEs in the Dockerfile.
However, this is solved if you are using the new Docker Swarm Mode with SwarmKit in 1.12.1. Docker has implemented the docker service feature which does volume drivers on a per-volume basis.
docker service create --replicas 1 --name pg -e POSTGRES_PASSWORD=mysecretpassword --mount type=volume,target=/var/lib/postgresql/data,source=pg_data,volume-driver=rexray postgres

@Krieke i may stand corrected. @clintonskitson pointed me to something that says the volume-driver only has impact if the name specified is not known by docker. So you should pre-create a volume with local driver and then specify that volume by name.


fill_raw not working for type: column

I have a column graph like this
type: custom:apexcharts-card
header:
  show: true
  title: ApexCharts-Card
  show_states: true
  colorize_states: true
graph_span: 24h
span:
  start: day
series:
  - entity: sensor.total_dc_power
    fill_raw: last
    type: column

Since last HA update 2023.2, the state db seems to save storage space and not saving anymore the sensor value if the value is not changing. That happens for my sensor over night. Therefore, I added fill_raw: last to my chart but its not working for column chart but inly for line chart. See picture below - top graph is type: column and button graph is type: line :

Side note: it gets worse if you use stacked: true and grouping by average. Thats all not working anymore since last HA update :-(

Seeing similar for area chart
Noting I'm using  color threshold stuff, if that's relevant. Not to distract of OP's first issue.
experimental:
          color_threshold: true
          brush: true


Loading github:rabbitmq/cluster-operator causes 'unsupported protocol: github'

Description of the bug:

Loading RabbitMQ CRD causes 'unsupported protocol: github' error.
Reproduction Steps:

$ cdk8s import<EMAIL_ADDRESS>
Error Log:

------------------------------------------------------------------------------------------------
A new version 2.118.0 of cdk8s-cli is available (current 2.116.0).
Run "npm install -g cdk8s-cli" to install the latest version on your system.
For additional installation methods, see https://cdk8s.io/docs/latest/getting-started
------------------------------------------------------------------------------------------------
Error: unsupported protocol github:
    at download (/usr/local/Cellar/cdk8s/2.116.0/libexec/lib/node_modules/cdk8s-cli/lib/util.js:177:19)
    at ImportCustomResourceDefinition.fromSpec (/usr/local/Cellar/cdk8s/2.116.0/libexec/lib/node_modules/cdk8s-cli/lib/import/crd.js:108:52)
    at matchImporter (/usr/local/Cellar/cdk8s/2.116.0/libexec/lib/node_modules/cdk8s-cli/lib/import/dispatch.js:40:49)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async importDispatch (/usr/local/Cellar/cdk8s/2.116.0/libexec/lib/node_modules/cdk8s-cli/lib/import/dispatch.js:12:26)
    at async Object.handler (/usr/local/Cellar/cdk8s/2.116.0/libexec/lib/node_modules/cdk8s-cli/lib/cli/cmds/import.js:32:9)

Environment:

Framework Version:
OS: MacOS, using 'brew' build version of cdk8s

This is :bug: Bug Report

@dragonpaw you need to remove the v from the version:
❯ cdk8s import<EMAIL_ADDRESS>                                                                                [02:48:15]
Importing resources, this may take a few moments...
rabbitmq.com
  rabbitmq.com/rabbitmqcluster

Otherwise it thinks its a regular URL, instead of a doc.crds.dev one.


Add a way to make a column unique in the .spy.yaml

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
class: User
table: x_user
fields:
  email: String (unique=true)
  nickname: String
  fullName: String?

Describe alternatives you've considered
Do a manual migration.

This is supported already, checkout: https://docs.serverpod.dev/concepts/database/indexing#making-fields-unique
There is an issue for making the syntax simpler here: https://github.com/serverpod/serverpod/issues/1391 and here https://github.com/serverpod/serverpod/issues/1141


Removed occasional prompt to not confuse inexperienced users.

Then again, I suggest adding the .rst extension to the README file and using code block directives to mark up commands (including the common prompt prefix).

+1 for moving to .rst... it's been on my todo list for a while, I just haven't gotten to it. Would you be willing to submit a PR tackling it?

@jeffwidman, I converted the README to reStructuredText in #20.

:+1: merged!


Cannot get frontend to load

Trying to access front end throws 500 internal server error.
I am unsure if my frontend.cfg is set correctly - I have it set for passwordless ssh access to the server that the appropriate files reside on. The passwordless ssh is confirmed to be working, but i am unsure if it is interpreting the command the way I want it to - does it HAVE to be a url in the .cfg file?

how did you fixed that issue?


intial pass of first class package

initial work

commit #2 output
$ zopen build -vv
Checking environment configuration
Using bare environment
VERBOSE: Searching for dependency cmake
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/cmake/cmake dependency environment
VERBOSE: Searching for dependency comp_clang
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/comp_clang/comp_clang dependency environment
clang not found. Please download and install from https://epwt-www.mybluemix.net/software/support/trial/cst/programwebsite.wss?siteId=1803.
VERBOSE: Searching for dependency comp_xlclang
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/comp_xlclang/comp_xlclang dependency environment
VERBOSE: Searching for dependency curl
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/curl/curl dependency environment
VERBOSE: Searching for dependency depot_tools
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools dependency environment
VERBOSE: Searching for dependency git
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/git/git dependency environment
VERBOSE: Searching for dependency gn
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/gn/gn dependency environment
VERBOSE: Searching for dependency jq
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/jq/jq dependency environment
VERBOSE: Searching for dependency make
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/make/make dependency environment
VERBOSE: Searching for dependency ninja
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/ninja/ninja dependency environment
VERBOSE: Searching for dependency python
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/python/python dependency environment
VERBOSE: Searching for dependency sed
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/sed/sed dependency environment
VERBOSE: Searching for dependency zoslib
VERBOSE: Setting up /z/jd895801/zopen/usr/local/zopen/zoslib/zoslib dependency environment
Building /z/jd895801/zopen/dev/v8port
VERBOSE: Running zopen_pre_patch
VERBOSE: Running zopen_init
Path to depot_tools missing.  Adding it to path..
activating the virtual env - venv
updating the virtual env pip
Requirement already satisfied: pip in /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools-main/venv/lib/python3.11/site-packages (24.0)
Requirement already satisfied: httplib2 in /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools-main/venv/lib/python3.11/site-packages (from -r /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools-main/requirements.txt (line 1)) (0.22.0)
Requirement already satisfied: six in /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools-main/venv/lib/python3.11/site-packages (from -r /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools-main/requirements.txt (line 2)) (1.16.0)
Requirement already satisfied: pyparsing!=3.0.0,!=3.0.1,!=3.0.2,!=3.0.3,<4,>=2.4.2 in /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools-main/venv/lib/python3.11/site-packages (from httplib2->-r /z/jd895801/zopen/usr/local/zopen/depot_tools/depot_tools-main/requirements.txt (line 1)) (3.1.1)
You're not a Googler. Metrics collection is disabled for you.
Running Bootstrap
VERBOSE: "python3" bootstrap/bootstrap.py --bootstrap-name /usr/lpp/IBM/cyp/v3r11/pyz/bin/python3 >/tmp/JD895801.pipe 2>&1
python3: can't open file '/z/jd895801/zopen/dev/v8port/bootstrap/bootstrap.py': [Errno 129] EDC5129I No such file or directory.
***ERROR: Bootstrap failed. Log: /z/jd895801/zopen/dev/v8port/log/20240221_133545_bootstrap.log
zopen-build completed in 80 seconds.

These changes are the first steps to changing from manual workflow to zopen build workflow.

Hi John @netskink. With https://github.com/ZOSOpenTools/depot_toolsport/pull/13 and https://github.com/ZOSOpenTools/v8port/pull/10, it should now be possible to fully automate running fetch v8, gclient sync and applying the patches as well as running the sed command as described in https://github.com/ZOSOpenTools/v8port/pull/10, without the need to run any other commands on another platform.
Note, however, until the required updates to v8 are upstreamed, your script should handle a failure from the initial fetch v8, which internally runs gclient sync that will fail when it attempts to download missing zos-s390x packages as described in https://github.com/ZOSOpenTools/v8port/pull/10. But the gclient sync that buildenv runs should work after the described sed command had run.

This has the first part of what we want.  It sets up the vevn in install dir and checks v8 out into v8base also in that dir.

This latest commit has problems.   We need to activate a venv, install libs in the venv and update the path.  Also we need to cd to a subdir and perform commands.  I wonder if we need a two stage post install setup.   ie. do all the copies to install dir and setup env and then in a second step do the change dir to v8base/v8 and run the depot tools commands.
However for debug purposes, here is the output given the latest patch.
$ rm -rf ~/.local v8base install ${HOME}/zopen/usr/local/zopen/v8; zopen build
Checking environment configuration
Using bare environment
Building /z/jd895801/zopen/dev/v8port
### zopen_init ###
Skip Bootstrap
Skip Configure
Running Build
### zopen_build ###
Running Check
make: *** No rule to make target 'check'.  Stop.
Running Install
### zopen_install ###
Replacing hardcoded /z/jd895801/zopen/usr/local/zopen path
Creating /z/jd895801/zopen/usr/local/zopen/v8/v8-DEV/.env
Install dependencies: zoslib python
Creating /z/jd895801/zopen/usr/local/zopen/v8/v8-DEV/README.md
Creating Metadata JSON
:DEBUG:: 'Calculating the offset path to store from root'
:DEBUG:: 'Temporary processing dir evaluated to: /z/jd895801/zopen/tmp/zopen.20240227104421'
:DEBUG:: 'Creating virtual store at: /z/jd895801/zopen/tmp/zopen.20240227104421/usr/local/zopen/v8'
:DEBUG:: 'Moving from main store location (unpax-dir) to processing store'
:DEBUG:: 'Creating main linked directory in store'
:DEBUG:: 'Creating virtual root directory structure'
:DEBUG:: 'Generating relative symlinks in processing location'
:DEBUG:: 'Generating symlink tree'
:DEBUG:: 'Creating directory structure'
:DEBUG:: 'Processing dir: venv/lib/python3.11/site-packages/setuptools/extern/__pycache__'
<copying venv snipped>
:DEBUG:: 'Processing dir: patches/build'
:DEBUG:: 'Processing dir: patches'
:DEBUG:: 'Processing dir: log'
:DEBUG:: 'Processing dir: install'
:DEBUG:: 'Processing dir: .'
:DEBUG:: 'Moving unpaxed processing-directory back to main rootfs store.'
:DEBUG:: 'Merge symlinks into main filesystem using tmp tar file v8.usr.tar'
:DEBUG:: 'Generating intermediary tar file'
:DEBUG:: 'Generating listing for remove processing (including main symlink).'
:DEBUG:: 'Extracting tar to rootfs.'
:DEBUG:: 'Cleaning temp resources.'
:DEBUG:: 'Switching to previous cwd - current work dir was purged.'
- Integration complete.
- Sourcing environment to run any setup
Setting up v8...
append_to_setup()
Good. depot_tools is in path.
ensure path is correct
PATH:  /z/jd895801/zopen/usr/local/zopen/zoslib/zoslib-main.20240205_123952.zos/bin:/z/jd895801/zopen/usr/local/zopen/sed/sed-4.9.20231121_234356.zos/bin:/usr/lpp/IBM/cyp/v3r11/pyz/bin:/z/jd895801/zopen/usr/local/zopen/ninja/ninja-master.20231109_002036.zos/bin:/z/jd895801/zopen/usr/local/zopen/make/make-<IP_ADDRESS>31113_010159.zos/bin:/z/jd895801/zopen/usr/local/zopen/jq/jq-1.6.20240104_213906.zos/bin:/z/jd895801/zopen/usr/local/zopen/bash/bash-<IP_ADDRESS>31123_190008.zos/bin:/z/jd895801/zopen/usr/local/zopen/perl/perl5-heads.v<IP_ADDRESS>31122_174302.zos/bin:/z/jd895801/zopen/usr/local/zopen/ncurses/ncurses-6.4.20231114_180107.zos/bin:/z/jd895801/zopen/usr/local/zopen/less/less-643.20240104_132250.zos/bin:/z/jd895801/zopen/usr/local/zopen/git/git-<IP_ADDRESS>31127_145951.zos/bin:/z/jd895801/zopen/usr/local/zopen/curl/curl-<IP_ADDRESS>40110_040723.zos/bin:.:/C/CCplus/LangV2GA/usr/lpp/IBM/oelcpp/v2r0/bin:/z/jd895801/zopen/usr/local/zopen/comp_clang/comp_clang-DEV.20240131_172255.zos/bin:/z/jd895801/zopen/usr/local/zopen/cmake/CMake-heads.v<IP_ADDRESS>31124_184234.zos/bin:/bin
    Creating a python venv...
activating the virtual env - venv
updating the virtual env pip
Requirement already satisfied: pip in ./venv/lib/python3.11/site-packages (23.2.1)
Collecting pip
  Obtaining dependency information for pip from https://files.pythonhosted.org/packages/8a/6a/19e9fe04fca059ccf770861c7d5721ab4c2aebc539889e97c7977528a53b/pip-24.0-py3-none-any.whl.metadata
  Using cached pip-24.0-py3-none-any.whl.metadata (3.6 kB)
Using cached pip-24.0-py3-none-any.whl (2.1 MB)
Installing collected packages: pip
  Attempting uninstall: pip
    Found existing installation: pip 23.2.1
    Uninstalling pip-23.2.1:
      Successfully uninstalled pip-23.2.1
Successfully installed pip-24.0
Collecting httplib2 (from -r /z/jd895801/zopen/usr/local/zopen/v8/v8-DEV/requirements.txt (line 1))
  Using cached httplib2-0.22.0-py3-none-any.whl.metadata (2.6 kB)
Collecting six (from -r /z/jd895801/zopen/usr/local/zopen/v8/v8-DEV/requirements.txt (line 2))
  Using cached six-1.16.0-py2.py3-none-any.whl.metadata (1.8 kB)
Collecting pyparsing!=3.0.0,!=3.0.1,!=3.0.2,!=3.0.3,<4,>=2.4.2 (from httplib2->-r /z/jd895801/zopen/usr/local/zopen/v8/v8-DEV/requirements.txt (line 1))
  Using cached pyparsing-3.1.1-py3-none-any.whl.metadata (5.1 kB)
Using cached httplib2-0.22.0-py3-none-any.whl (96 kB)
Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Using cached pyparsing-3.1.1-py3-none-any.whl (103 kB)
Installing collected packages: six, pyparsing, httplib2
Successfully installed httplib2-0.22.0 pyparsing-3.1.1 six-1.16.0
gclient: ./setup.sh 59: FSUM7351 not found
fetch: ./setup.sh 62: FSUM7351 not found
fatal: not a git repository (or any parent up to mount point /z/jd895801)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
fatal: not a git repository (or any parent up to mount point /z/jd895801)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
fatal: not a git repository (or any parent up to mount point /z/jd895801)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
gclient: ./setup.sh 66: FSUM7351 not found
Setup completed.
zopen-build completed in 188 seconds.


Softmax p bblas version

The implementation of SoftMax by using PBblas library
The implementation is done based on the Stanford Deep Learning tutorial
The implementation is done on Classify.SoftMax

@johnholt are you able to review the pblas part please?

I am implementing Stacked Sparse Autoencoder for which I need all my previous implementations. So I have merged all my implementations by now in a new branch that includes NeuralNetwork, DeepLearning and SoftMax implementations. I will make a pull request for that branch when my implementation of Stacked Autoencoder is completed.


Docs update for dapr/components-contrib#1104

What content needs to be created or modified?

Note: this issue depends on #1755. That needs to be addressed first.

When https://github.com/dapr/components-contrib/issues/1104 is merged, we will need to update the following pages adding information on how to authenticate using the "common Azure auth layer" (which supports Azure AD auth), in alternative to the current master key authentication.
Pages impacted:

https://docs.dapr.io/reference/components-reference/supported-state-stores/setup-azure-cosmosdb/
https://docs.dapr.io/reference/components-reference/supported-bindings/cosmosdb/

Describe the solution you'd like
Those docs need to be updated to point out that authentication can happen in one of two ways:

Using masterKey (current option)
Using Azure AD -> this should link back to the docs on using the "common Azure auth layer"

The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos
https://github.com/dapr/components-contrib/issues/1104

@ItalyPaleAle - Please verify whether this can be closed?

Yes thanks for checking.


[Map] MH-HiSpeed

Your description here
Duplicate of https://unrealarchive.org/maps/unreal-tournament/monster-hunt/H/mh-hispeed_a7dd5b1e.html
URL: https://unrealarchive.org/maps/unreal-tournament/monster-hunt/H/mh-hispeed_721c98a0.html
Hash: 721c98a0bef3e0f2f218f6de19c192492a934660
Current name: MH-HiSpeed

updated


Reformatting protobuf files adds extra spaces

What steps will reproduce the issue?

Create a protobuf file.
Add an RPC to a service, such as rpc Add(AddRequest) returns (AddResponse);.
Choose Reformat Code.

What is the expected result?
No change to the line as it appears above.
What happens instead?
A space is added before the request parameter: rpc Add (AddRequest) returns (AddResponse);
This appears to be in contradiction to the style guide: https://developers.google.com/protocol-buffers/docs/style
Original issue at: https://youtrack.jetbrains.com/issue/GO-6156

This issue is closed as plugin is not supported anymore.
Please switch to https://github.com/jvolkman/intellij-protobuf-editor.


KeyError raised for pending funds in wallets with single assets

There is a situation where KeyError is raised: if a wallet has a single (non-ADA) asset, then if all UTXOs containing that asset are consumed in a transaction, calling wallet.assets() while the transaction is pending (and change has not arrived yet) raises the following KeyError:
    aid: Balance(bal["total"], bal["available"], None)
KeyError: 'available'

To illustrate, there are two situations where the error can be replicated:
First:
Send 10 ADA and 1000 asset A to a fresh wallet X with address Y (so wallet X contains a single UTXO at address Y, and nothing else.)
Submit a transaction on wallet X sending 500 units of asset A to address Y (internal transaction).
Call walletX.assets() before the transaction is confirmed - the above KeyError should be raised.
Second:
Assume wallet X has two UTXOs, each having 5 ADA and 500 asset A.
If you submit a transaction on wallet X sending 500 units of asset A to address Y, there is no issue calling walletX.assets(), it will display 'available' asset A as 500 units as expected, as one of the UTXOs is not consumed. But any transaction that requires the consumption of all inputs of asset A (ie, sending self 600 ADA) will cause KeyError to be raised if walletX.assets() is called whilst pending confirmation.

I would guess that this is the case for any transaction made from a wallet that consumes all UTXOs for all the non-ada assets in the wallet. So I would assume even if there are two assets in the wallet, if a single transaction is called that consumes all the UTXOs containing both assets, you would get this issue.
I haven't tested it though, worth taking a look.

I've done some more testing - it turns out that even if there are many distinct AssetIDs held in a wallet, if any single one of them have all their UTXOs exhausted by pending transactions, then any call to wallet.assets() will result in a KeyError.

Hi,
I was unable to reproduce the error but I guess I have found the place in the code where it crashes. Perhaps it was already fixed in the cardano-wallet backend.
Anyway, the release 0.8.2 should be free of that problem. Could you please make a test run to confirm?


Concurrently stat shares in ocs service

@butonic can we merge this?

@ishank011 Awesome! Let us merge that.


[Feature Request]: Hover effect on button

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
at different places of button  we want to add hover effect
Use Case
at different places of button  we want to add hover effect
Benefits
it become more attractive
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

I want to work on this issue #746 .please assign me this issue under hactoberfest also

page??

@mansiruhil13 please assign me this


xmake create -lc -Phello will not replace FAQ and TARGETNAME

Xmake Version
v2.9.1+20240422,
Operating System Version and Architecture
Linux 6.9.1
Describe Bug
$ xmake create -lc -Phello
create Desktop ...
error: cannot open file: xmake.lua, Not access because it is busy
$ cat hello/xmake.lua
add_rules("mode.debug", "mode.release")

target("${TARGETNAME}")
    set_kind("binary")
    add_files("src/*.c")

${FAQ}

Expected Behavior
$ xmake create -lc -Phello
create Desktop ...
error: cannot open file: xmake.lua, Not access because it is busy
$ cat hello/xmake.lua
add_rules("mode.debug", "mode.release")

target("hello")
    set_kind("binary")
    add_files("src/*.c")

Project Configuration
/
Additional Information and Error Logs
And xmake.lua generated by xmake create has permission r--r--r--. Why not use umask?

ruki:dddd ruki$ xmake create -lc -Phello
create dddd ...
  [+]: xmake.lua
  [+]: src/main.c
  [+]: .gitignore
create ok!
ruki:dddd ruki$ cat ./hello/xmake.lua
add_rules("mode.debug", "mode.release")

target("dddd")
    set_kind("binary")
    add_files("src/*.c")

it works for me.

I have tested in other OSs and it can work. I think it is the bug about NixOS.

We should have os.chmod(), then run os.chmod('file', '+x') after os.cp().

why?

See https://github.com/NixOS/nixpkgs/issues/314313

os.cp(srcfile, dstfile)
it will add +w to open and create dstfile.
https://github.com/tboox/tbox/blob/553c32406a09e3e0a0a8e8dfed741b4888ba07ca/src/tbox/platform/posix/file.c#L80
and srcfile is only for readonly.

No. It doesn't happened. If I chmod -w in other OS, the same error will be throw.

Does it work now?

Yes, thanks!

https://github.com/NixOS/nixpkgs/issues/314313#issuecomment-2134239967
unset LD cannot work. Is it a bug?

Oh, I must rm .xmake -rf.

Can we allow use ld as $LD? I think it is nature.


Don't swap the speakers service

This PR removes all swapping of the application.speakers service.
Related to #618.

@derrabus Could you explain to me how the tests know to use the MockableIdentityProvider? It's not immediately obvious to me.

@chartjes This works the same way as with the MockableAuthenticator.
In the testing environment, the IdentityProvider service points to MockableIdentityProvider which decorates the Sentinel implementation. If we feed a user object into that class, it will always serve that object. If not, it will fall back to Sentinel.

@derrabus Again, sorry for my not understanding this. To me it looks like magic because I cannot see the mechanism where it actually happens. If it happens because of the annotation, that's fine. I am just trying to understand it! 👍

Thanks @derrabus

@chartjes No, it's not the annotation that does this. It's this piece of code that we only load in testing environment:
https://github.com/opencfp/opencfp/blob/d58e15ac757bad16bdbf48ddcfaaf73dfd26ed4f/classes/Provider/TestingServiceProvider.php#L30-L32


Feature/forget password

This PR completes issues 73 & 74

@OluwasegunA fix the branch conflicts and codacy checks

@Honeyloveet Done

@OluwasegunA Codacy checks have failed please fix it.

@Honeyloveet Fixed now


Change Reward function to give points to number of unique pokemon

Hi,
Came across the tutorial on youtube and got the program running. How can I change the reward function to give points to number of unique pokemon? The idea is to try to train the agent to Catch them all.
I also have access to cloud resources of several GPUs. I want to try to run run_baseline_parallel_fast.py to simulate hundreds of games at once to see how fast I can get it to capture 10 unique pokemon.

Hi, You can access the reward function in the the get_game_state_reward function in red_gym_env.py
The memory addresses you need to access are 0xD2F7 - 0xD309 with each bit representing a boolean whether you own that specific pokemon. You can find more memory addresses here.
https://datacrystal.romhacking.net/wiki/Pokémon_Red_and_Blue/RAM_map#Pokedex
you could write something along the lines of
def get_unique_pokemon(self):
for i in range(0,18):
pokemon_caught += self.bit_count(self.read_m(0xD2F7 + i))
return pokemon_caught
then add to the state_score dictionary something like
'unique_pokemon': self.reward_scale*self.get_unique_pokemon(),
i would not recommend removing the other rewards as the agent probably wont leave pallet town unless it has some exploration reward but you are welcome to experiment.
good luck!

If it can help you, i went for the same reward.
'seen_pokemons': self.reward_scale * self.seen_pokemons
RedGym
    def update_seen_pokemons(self):
        initial_seen_pokemon = 3
        self.seen_pokemons = sum(self.reader.read_seen_pokemons()) - initial_seen_pokemon

Reader (i extracted everything related to memory in a different class)
    def read_seen_pokemons(self):
        return [self.bit_count(self.read_m(a)) for a in SEEN_POKEMONS_ADDRESSES]

Memory addresses
    SEEN_POKEMONS_ADDRESSES = [0xD30A, 0xD30B, 0xD30C, 0xD30D, 0xD30E, 0xD30F, 0xD310, 0xD311, 0xD312, 0xD313, 0xD314, 0xD315, 0xD316, 0xD317, 0xD318, 0xD319, 0xD31A, 0xD31B, 0xD31C]


Export command : -t param is defined twice

Once for the template option : https://github.com/ddollar/foreman/blob/master/lib/foreman/cli.rb#L53
Once for the timeout option : https://github.com/ddollar/foreman/blob/master/lib/foreman/cli.rb#L55

Get caught by this one...tried to specify timeout when exporting to systemd with -t doesn't works, however, it works with --timeout.

Hi there! Any news?

Hi there! Any news?


accordion opens then immediately hides content

This is happening on your docs page on all the examples and even on the Show Code buttons.  Subsequently it's also happening when I try to implement the accordion in my project.

related
#1082
#1081

Been a while between major code updates, so hopefully the project is still alive, but this workaround has worked for me - I just add the following CSS to the page:
.accordion-collapse.show { visibility: visible !important;
This seems to fix the issue for accordions anyway. As others have observed, the issue is that expanded/non-collapsed "collapsible" elements have the "collapse" class applied, which applies the style "visibility: collapse", which basically makes content invisible without changing its shape/bounding box.
The CSS rule which causes this seems to be:
.collapse {   visibility: collapse; }
whereas in some older code I have (still 1.0.0-alpha11) there is:
.collapse:not(.show) {     display: none }
I haven't dug through the modules to work out where this class is being applied though.

This might actually be caused by a bug in tailwind css between version v3.1.8 (working) and v3.2.1 (broken).

Yes all the library is broken for now ... we can't copy nothing in the website and the library himself is broken ^^
We need to wait :)

related #1082 #1081

also #1089, #1086, and probably #1083
Broke here with Tailwind 3.2.0

I quick fixed it by:
.collapse.show {
    visibility: unset;
}

But waiting for official fix.

I quick fixed it by:
.collapse.show {
    visibility: unset;
}

But waiting for official fix.
tailwindcss: v3.2.2 tw-elements: 1.0.0-alpha12

thanks i added !important
.collapse.show {     visibility: unset !important; }

The problem should be fixed with the latest release.

Nope, issue still occurred.
tailwindcss: v3.3.1
tw-elements: 1.0.0-beta2


Implement inline editing of sign language video URLs for admins

Provide the ability for admins to edit the Sign Language video URLs directly from the page the will display the video. This is an alternative and convenience on top of what is described in #1257.
When an admin user visits a page that includes content that has been identified as sign language interpretable, a button with a pencil icon would appear near where the sign language video is or would be shown. Clicking that button would show some input fields allowing the sign language video referencers (URLs/IDs) to be added/edited.

@cherylhjli, this is something that @greatislander and I talked about the other day. Do you have any thoughts or suggestions about this?

@jobara that sounds good to me!

I'm not sure how feasible this feature is. There are many ASL/LSQ videos on portions of the site that a site admin does not have access too. Such as setting up a public page for a community connector.


Missing author credits for wallpapers and notification of new added wallpaper

Description
Many wallpapers are included in the Hyprland assets, but there seems to be no mention of credits or authors for these works. Acknowledging the artists work is crucial, especially to respect and appreciate the time and creativity invested in the creations.
This oversight not only disrespects the creative efforts of these individuals but also undermines the ethical standards expected within the open-source community. I'm particularly interested in knowing who created these wallpapers for potential future collaborations or commissions.
New wallpapers are added to the project without any announcement or notification to the users, this approach can lead to unexpected surprises for users in the professional context or environment, such as the appearance of random or potentially borderline content.
Here is the todo list:

[ ] Add artist credits
[ ] Content guidelines - guidelines for what constitutes acceptable content for wallpapers included and standards for appropriateness
[ ] User notifications and consent - before introducing new wallpapers, it should be better implementing a notification system or an option to review and accept these changes could enhance the user experience

Who are you? Ethics police?
https://hyprland.org/news/contestWinners/
Authors are mentioned there.

This oversight not only disrespects the creative efforts of these individuals but also undermines the ethical standards expected within the open-source community. I'm particularly interested in knowing who created these wallpapers for potential future collaborations or commissions.

 Rules of the competition were laid out and followed.

New wallpapers are added to the project without any announcement or notification to the users, this approach can lead to unexpected surprises for users in the professional context or environment, such as the appearance of random or potentially borderline content.

I don't care about your surprises, I am not obliged in any way, shape, or form to inform you about anything.

Who are you? Ethics police?
https://hyprland.org/news/contestWinners/
Authors are mentioned there.

This oversight not only disrespects the creative efforts of these individuals but also undermines the ethical standards expected within the open-source community. I'm particularly interested in knowing who created these wallpapers for potential future collaborations or commissions.

 Rules of the competition were laid out and followed.

New wallpapers are added to the project without any announcement or notification to the users, this approach can lead to unexpected surprises for users in the professional context or environment, such as the appearance of random or potentially borderline content.

I don't care about your surprises, I am not obliged in any way, shape, or form to inform you about anything.

I don't care about your surprises, I am not obliged in any way, shape, or form to inform you about anything.

Rude bruh

said the one attacking in the original post after creating an account today probably to avoid backlash.


Tooltips

First thank you for open sourcing your wonderful jstree.
Node text may be long, or additional information may be necessary to help users. I would suggest adding tooltips so we can truncate long node text, or present some additional information when displaying jstree..
Best regards.

Hi,
Thank you for the idea, but custom tooltips are out of the scope of jstree. As for built-in tooltips - you can use a title attribute on the anchor nodes (by using a_attr in your data), which will render the default browser tooltips. If you need custom tooltips - you can attach any library you like, just do not forget to update its bindings (if necessary) when jstree repaints.
Best regards,
Ivan

Thank you very much.

From: Ivan Bozhanov<EMAIL_ADDRESS>Sent: December 7, 2015 6:08 PM
To: vakata/jstree
Cc: changbaishan
Subject: Re: [jstree] Tooltips (#1305)
Hi,
Thank you for the idea, but custom tooltips are out of the scope of jstree. As for built-in tooltips - you can use a title attribute on the anchor nodes (by using a_attr in your data), which will render the default browser tooltips. If you need custom tooltips - you can attach any library you like, just do not forget to update its bindings (if necessary) when jstree repaints.
Best regards,
Ivan

Reply to this email directly or view it on GitHubhttps://github.com/vakata/jstree/issues/1305#issuecomment-162681804.
[https://avatars0.githubusercontent.com/u/12687044?v=3&s=400]https://github.com/vakata/jstree/issues/1305#issuecomment-162681804
Tooltips · Issue #1305 · vakata/jstree
First thank you for open sourcing your wonderful jstree. Node text may be long, or additional information may be necessary to help users. I would suggest adding tooltips so we can truncate long no...
Read more...https://github.com/vakata/jstree/issues/1305#issuecomment-162681804


package size issue

Hello Dear Friend,
this package is over 2mb is very heavy for production

Please decrease package size
BR,

How should I reduce it, am I suppose to remove all the icons you don't need? :wink:
The package size doesn't matter at all. What matters is the resulting size of your app and that size will be as small as possible because you can import only the icons you need.


Listed Clients

While helping a user in #pidgin today setup SASL, I was looking for the guide when I stumbled across the clients page. I was disappointed to see that Pidgin was not listed there. Pidgin has a long and rich history which includes using Libera.chat and it's previous network as it's primary support channel since at least 2003 which is when I got involved in the project.
I brought this up in #libera and was told to create an issue here and to come up with a way to add a list of additional clients. I think an additional list is great as it could give exposure to many clients, but I think Pidgin deserves to be on the first page. I'm of course biased, but the Google Trends results seem to agree with me.
I was unable to add Konversations or Limechat as Google Trends some how breaks when they're added, but as you can see pidgin is searched for more than everything except mIRC. Also note that due to name collisions, we have to explicitly ask Google Trends for Pidgin Software where all the others are Topic searches.
At any rate, this issue is to hopefully open some discussion here but at the end of the day it's your website and you can recommend who you like and we'll just have to deal with that.

Sorry about that, definitely not intentional! The clients on there now are just the ones that I thought of when writing it.
The clients page is meant for first-time Libera users, so an exhaustive list of clients will end up overwhelming them and i'd rather not do that. However, I wouldn't mind Pidgin being added to the list.

No worries, I didn't take it personally, just shows that we have more work to do to get back into peoples thoughts ;)
At any rate, should I create a pull request or is that something you all would rather handle?

feel free to make a PR :+1:


怎么上传、下载压缩文件

Notice
If you need support or clarification regarding the usage of Drogon in your project, visit the official Drogon support channel at gitter
Please create a new issue only if you think you have found a bug or if have a feature request/enhancement.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

在工程中搜索MultiPartParser就能看到例子


gnubiff

I would like to "upgrade" gnubiff to oauth2 using this tool (on ubuntu)
but first it has problems installing webview.
and can someone tell me how to configure this for gnubiff?
(gnubiff checks for new mail evety x minutes, and has a jumping linux if mail arrives..
Thanks!

I can't help with specific gnubiff configuration, but you should be able to configure it to use the proxy in the same way as any other email server.
Given that the GUI-related requirements are at the root of many similar questions, I've added a requirements-no-gui branch – you should be able to install and set the proxy up using this version with no dependency issues, then switch to the GUI later if needed

On Fri, 19 Aug 2022, Simon Robinson wrote:
Thanks a lot, but somehow the instructions confuse me...
So in gnubiff I say that that the address is localhost:1993
when I turn on emailproxy.py I see that its listening for authentication
request.
( I put my email in the config file.) Then I see in debug mode that
gnubiff tries to connect (every 5 seconds for test)
I get every 5 seconds
2022-08-19 14:51:14: IMAP (localhost:1993;
<IP_ADDRESS>:57624->outlook.office365.com:993) <-- b'* OK The Microsoft
Exchange IMAP4 service is ready.
[WgBSADAAUAAyADcAOABDAEEAMAAxADAAOAAuAEMASABFAFAAMgA3ADgALgBQAFIATwBEAC4ATwBVAFQATABPAE8ASwAuAEMATwBNAA==]\r\n'
How do I proceed? Do I need to give my credit card to azure?
Thanks for help
JP
->I can't help with specific gnubiff configuration, but you should be able to configure it to use the proxy in the same way as any other email server.
->
->
->
->Given that the GUI-related requirements are at the root of many similar questions, I've added a requirements-no-gui branch – you should be able to install and set the proxy up using this version with no dependency issues, then switch to the GUI later if needed
->
->
->
->--
->
->Reply to this email directly or view it on GitHub:
->
->https://github.com/simonrob/email-oauth2-proxy/issues/49#issuecomment-1220394488
->
->You are receiving this because you authored the thread.
->
->
->
->Message ID: @.***>

If you haven't already done so you'll need to add values for client_id and client_secret in your account's configuration file entry – see the proxy's readme for more information about how to do this.
The log message you posted is a normal greeting from the Outlook IMAP server, but you should also see a login attempt in the log from your client application once you've set it up to use the proxy.

I'm assuming this is now resolved so I'm going to close this issue.


Pattern TANKS frames while looking at it, or having it in inventory

Describe the bug
I made a pattern with a platinum hammer and a an iron ore to make iron dust.
I also have a platinum hammer pattern, and other patterns.
Suddenly, my game TANKED in frames upon making the pattern. I tossed the pattern using Q, and looked away from it instinctively. My frames returned to normal.
I cannot trash the item (I don't know what mod provides it, but it's a trash icon/slot at the bottom right of inventory), so I had to put it in a chest with a sign saying "DO NOT OPEN", I could probably delete it with lava, but for now that seems to work.
https://github.com/refinedmods/refinedstorage2/assets/54318714/0c110f93-d0e5-4128-886a-5e9c73c33a9d
It's not a big deal once you know what the issue is, but I think that some people might not be able to figure out what the issue is here.
How can we reproduce this bug or crash?

Create necessary RS2 setup to create patterns
Create a hammer from All The Ores (I was using platinum, I'm not sure if this is an issue with the other hammers)
Use the hammer and an ore with a pattern grid to create a pattern that uses the hammer (non-exact, non-processing) to create dust.
You should now have the pattern of-subject which tanks frames

What Minecraft version is this happening on?
Minecraft 1.19.2
What Forge or Fabric version is this happening on?
CurseForge, ATM8 v1.0.19b
What Refined Storage version is this happening on?
1.11.6
Relevant log output
No response

As the issue submitter, it would be great to hear why this was closed.
I took the time to create reproduceable steps, create a video better showing the issue, and deliver the report.
I feel that, as the ticket closer, taking the time to explain why an issue is being closed is warranted. "closed this as not planned" isn't a great explanation either, IMO.

You posted this on the wrong repository

What repository should this have been posted to? Thank you for the response.

You're supposed to post it on the refined storage repo, since you're currently using refined storage and not refined storage 2
https://github.com/refinedmods/refinedstorage

Thank you @jonatjano! I've posted it to the correct location now. I appreciate you taking the time to help resolve this issue by informing me of the right place to submit it. I hope you have a nice day!


Update Spanish Translation correctly - [merged]

In GitLab by @mickypardo on Oct 4, 2022, 20:46
Merges master -> master
Hi.
This change completes the Spanish translations of the es.json file.
Regards.

In GitLab by @Asacolips on Oct 5, 2022, 08:04
approved this merge request

In GitLab by @Asacolips on Oct 5, 2022, 08:04
Thanks @mickypardo! I need to test this on my local environment before merging it, but it looks good at an initial review.

In GitLab by @Asacolips on Oct 9, 2022, 17:13
mentioned in commit 2c46e8d20fb948cbf1f908ab9c9ef968c708e4f5


xml files separated to a dedicated folder

I will slowly add more xmls and tests

let's leave the files there for now, since otherwise the scripts break. We can organize this later, but before it gets out of hand.