Automated cherry pick of #3355: fix: shoule purge snat table and dnat table when sync remove local natgateway; nat rule's 'real_name'.

Cherry pick of #3355 on release/2.11.
#3355: fix: shoule purge snat table and dnat table when sync remove local natgateway; nat rule's 'real_name'.

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@087365c). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #3356   +/-   ##
==============================================
  Coverage                ?   7.14%
==============================================
  Files                   ?     476
  Lines                   ?   77234
  Branches                ?       0
==============================================
  Hits                    ?    5516
  Misses                  ?   71255
  Partials                ?     463

Flag
Coverage Δ

#aFlag
7.14% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/purge.go
0% <0%> (ø)

pkg/compute/models/natgateways.go
2.99% <0%> (ø)

pkg/compute/models/natstable.go
3.96% <0%> (ø)

pkg/compute/models/natdtable.go
4.91% <0%> (ø)

Continue to review full report at Codecov.

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

/approve
/lgtm


Replace incorrect test cmdlet for proxy service install

It does not make sense to attempt the DC Agent health test (Test-AzureADPasswordProtectionDCAgentHealth) during the proxy service installation. Instead, the proxy health test (Test-AzureADPasswordProtectionProxyHealth) should be completed.

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

#sign-off


Fix argument name

Previously, the -n flag sent its data to the "max-queries" variable, which results in an unknown variable name when running the script because the python variable used to generate the run script is 'limit' (see line 163). "Max-queries" is only applicable as a flag for the tsbs_run_queries script, i.e., "--max-queries=###".

Codecov Report

Merging #83 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #83   +/-   ##
=======================================
  Coverage   64.96%   64.96%
=======================================
  Files          84       84
  Lines        4282     4282
=======================================
  Hits         2782     2782
  Misses       1463     1463
  Partials       37       37

Continue to review full report at Codecov.

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


SPARK-5408: Use -XX:MaxPermSize specified by user instead of default in ...

...ExecutorRunner and DriverRunner

Test build #26092 has started for   PR 4202 at commit 6062321.

This patch merges cleanly.

That looks reasonable to me.

Test build #26092 has finished for   PR 4202 at commit 6062321.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26092/
Test PASSed.

If the user-defined java options are appended to the Spark defined ones, shouldn't they override the latter? That way you'd have both -XX:MaxPermSize=128m in the command line and also the user-defined value, and the user-defined value would "win" by being later in the command line.
Also, you have a typo in the PR title.

@vanzin I agree, I'd imagine this actually works as expected here since the last one wins. This still feels like a small good thing just so anyone looking at the command line is less likely to be confused.

I'm just trying to understand whether there's a larger bug here that applies to more than just that option. Because the bug seems to imply the user setting is not used at all.

I guess I should be overridden, but I couldn't find any specification which says that it actually works in this way.

@jacek-lewandowski in practice have you found they aren't overwritten?

@pwendell for JVM which I use it is overwritten. But this is only one JVM. As I said - if such behaviour is in JVM specification - that's good. However if it is not, this should be considered as non-deterministic. I couldn't find that information.

I merged PR https://github.com/apache/spark/pull/4203 but don't think we need to back port it, so this can be closed.

Mind closing this PR?


Investigate how to trigger OS notification

We could generate a notification every time the wallet service send an event requiring the user to review something.
https://github.com/gen2brain/beeep

This library it not a suitable solution on macOS as it uses osascript that doesn't support customization, and opens AppleScript software when the notifications are clicked instead of the emitter.
We can still use it for Linux and Windows, if it doesn't have the same kind of problem.


[SPARK-47507][BUILD][3.5] Upgrade ORC to 1.9.3

What changes were proposed in this pull request?
This PR aims to upgrade ORC to 1.9.3 for Apache Spark 3.5.2.
Why are the changes needed?
Apache ORC 1.9.3 is the latest maintenance release. To bring the latest bug fixes, we had better upgrade.

https://orc.apache.org/news/2024/03/20/ORC-1.9.3/

Does this PR introduce any user-facing change?

How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

Thank you, @yaooqinn .

Merged to branch-3.5 for Apache Spark 3.5.2.

Thank you @dongjoon-hyun!

Thank YOU. :)


Adding getSupportedPlatformInfoFunctions

Description
re #1085

Added a helper/convenience function, getSupportedPlatformInfoFunctions(...) that gets rid of some of the repetitive logic around supported platforms and memoization
Added helper functions, getSupportedPlatformInfoSync and getSupportedPlatformInfoAsync , that can be used to explicitly get info sync or async if one desires to do so

Compatibility

OS
Implemented

iOS
✅

Android
✅

Windows
✅

Checklist

[X] I have tested this on a device/simulator for each compatible OS

unable to verify on Windows, but did verify for iOS and Android

[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[X] I updated the typings files (privateTypes.ts, types.ts)
[ ] I added a sample use of the API (example/App.js)

Screen Shots
Below are screen shots of iOS and Android and the updated getDeviceName* functions (no difference)

The usage of this function can probably be used with a lookup table to make the index file more terse, but that's a future thing if desired — me thinks.

Totally fine with me.
I'll knock out a few conversions later today.


Support styles and stylesUrl

Because you cannot use NativeScript’s “page-specific” CSS approach with Angular 2, I’ve been trying to come up with some way to scope styling to individual components. If we could find some way to support Angular’s styles and stylesUrl that would be great.

:+1: Yes please. I tried part of this earlier in the week (i think)
Is this what you mean by Angular's styles?
<label (tap)="tapIcon" #item           class="ion-icon"          [style.color]="color"                 [text]="GetIcon()"></label>
ORIGINAL EXCEPTION: Error: Not implemented: setStyleProperty

Support for component specific css (both styles and stylesUrls are supported) is added with commit #68 and is available with nativescript-angular-0.30 npm version.


Install-time vulnerability in chownr introduced through tar

There's a TOCTOU vuln in chownr that snyk is just reporting now (even though disclosure happened 31 Jul, 2018).
Given that this is part of the installation step for sharp, I don't think this is terribly exploitable, it's more an FYI.
Thanks for sharp :1st_place_medal: !

Thanks for reporting this. As you say, chownr is an install-time (transient) dependency of sharp and an attacker would already need to have write access to the filesystem to be able to exploit this.
https://github.com/isaacs/chownr/issues/14
The truly paranoid could run npm install && rm -rf node_modules/chownr.


bootgrid glue: make it more robust to errors; handle disabled entries…

… correctly and make it more flexible to changes

this is nr. 4 of #1774

@fabianfrz thanks!


Housekeeping

Codecov Report

Merging #10 into master will increase coverage by 3.63%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #10      +/-   ##
============================================
+ Coverage     78.18%   81.81%   +3.63%
- Complexity       55       56       +1
============================================
  Files             8        8
  Lines           220      220
  Branches          5        5
============================================
+ Hits            172      180       +8
+ Misses           44       36       -8
  Partials          4        4

Impacted Files
Coverage Δ
Complexity Δ

.../postgres/serializer/JdkSerializationStrategy.java
100% <0%> (+13.33%)
3% <0%> (ø)
:arrow_down:

...tgres/serializer/JacksonSerializationStrategy.java
100% <0%> (+23.52%)
5% <0%> (ø)
:arrow_down:

.../postgres/serializer/DeserializationException.java
100% <0%> (+100%)
1% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


[TST] Idempotence test needed

Referencing #197, there's value in ensuring that every function is idempotent. This means ensuring that func(func(x)) == func(x), as described in the original issue.
The steps suggested for this PR would be as follows:

Create a generic testing utility function, that might look like:

def idempotence_test(func, df, *args, **kwargs):
    assert func(df, *args, **kwargs) == func(func(df, *args, **kwargs), *args, **kwargs)

Clearly, a few tests would be required for the idempotence_test function, to ensure that it works as expected.

Identify the functions that should behave idempotently, and add this test to the test suite for that function.

I would like to work on this.

A few initial ideas come to mind:

With add_column I wouldn't want to add the same column multiple times (however, I believe the current implementation will throw an error if the column already exists).

filtering functions probably won't be affected (since the data is already filtered for a given range)

convert_*_date functions might be problematic since it might assume that the input is not a date

I briefly looked at the list of main functions on readthedocs (looks amazing btw!) and thought that most of them fall in the "filtering" bucket and shouldn't be affected (i.e. change_type won't hurt the values if they get changed from a str to a str).

the function ´´´convert_unix_date(df, column)´´´ does not have the idempotence property. It is converting the values into int which throws an error when the value is datetime formart.
I believe many of the convert_"some date format" functions does not have the property.

@SorenFrohlich @szuckerman I'll leave this in both your hands, and trust your judgment here.
I've set up more checks on the CI system, so it's harder to break - feel free to go crazy!


Merging sites produces a MultipleObjectsReturned error

When merging data (using manage.py loaddata) from one site on top of another, and in both locations there is a default site (which happens even if you're starting from a "blank canvas", or a freshly migrated Wagtail database), then you get an error in the admin console trying to edit one of the default sites:
python3.7/site-packages/wagtail/core/models.py in clean_fields
147. default = Site.objects.get(is_default_site=True)

This looks a bit like #1745 - just for the Site object.
Steps to Reproduce

Export data from a Wagtail site using dumpdata
Create a fresh Wagtail site and loaddata that you exported
Log into the admin, go to sites, you should see multiple sites with "default_sites" enabled
Try to edit and save one of those sites, you get an error.

Notes
This is fixable by editing the database directly, or doing a quick search and replace just before you loaddata of the JSON dump, e.g.:
sed -i 's/\"is_default_site\": true/\"is_default_site\": false/g' my-site-export.json
It's a minor issue, but could save people some headache in trying to figure out why they can't import their site properly.

As per #6043, merging the data of multiple sites through loaddata isn't something we can feasibly support. One improvement that might be possible is to add a database constraint on the is_default_site field so that this is caught during the import - however, it looks like the recipe for that kind of constraint is a bit esoteric and I suspect it doesn't have an equivalent in the Django ORM.


fixes enter bug for issue #26

🛠 Fixes #26
👨💻 What's being changed:
Tweaks the onClick code for making a new post so that it only calls the function if were not already posting ( isPosting )

Hey @kezdev, this solution is not working. I can still post it multiple times


Explain better in the DOCS advantages of cache (CDN)

https://403page.com/scaling-for-volume-with-frontity-cloudflare/

with that setup, he is pretty much testing how well cloudflare CDN works
because 99.89% of the requests when to the CF CDN
and that obviously scales to the infinitum
but that’s one of the points I always try to make: Frontity is as fast and as scalable as a static site as soon as you add a CDN
and you should add a CDN
(by the way, all the modern static site hostings like Now, Netlify, Surge… have CDNs)
For the rest of 0.11% of the requests that went into Frontity:
If you use a Node server: much more scalable than WordPress, because it’s Node (which by default is more scalable for its I/O) and it’s stateless so you can easily scale horizontally.
If you use Serverless: then this part is also infinite scalable, like a CDN.
and finally the last thing that needs to be taken into consideration to scale Frontity is how well you cache the REST API

I think creating a "cache" subcategory under "Architecture" can be a good place to document this.
We should document either cache at a Frontend level and cache at a WP level


⚠ Contemporanea Group has degraded performance

In 275cd1d, Contemporanea Group ($SITE_CNTMG) experienced degraded performance:

HTTP code: 200
Response time: 23103 ms

Resolved: Contemporanea Group performance has improved in 7671db7.


Adds the OWNERS and OWNERS_ALIASES files

Issue aws-controllers-k8s/community#778
Adds the OWNERS and OWNERS_ALIASES files
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/verify-owners


Either wrong comment or BUG

While I was searching the code base for reasons why my tests would break under Azure.Identity 1.13 I came across the linked piece of code ...
I do not believe this is the reason for the change in behavior I'm experiencing but the discrepancy between the comment and the code jumped at me.
https://github.com/Azure/azure-sdk-for-net/blob/d0aadf70031d8bb14d62a2c81e9ce1de8e967e53/sdk/identity/Azure.Identity/src/AppServiceManagedIdentitySource.cs#L25C1-L30C14

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

This code is actually deprecated in 1.13.0, so it shouldn't be relevant.


Sometime MV shows empty alias instead of worldname for newly created worlds

https://paste.gg/35c55eae656c4d6b83599445b77329e9
If anyone can provide steps to reproduce, that will be great!

Looks like setAlias isn't working...
Code:
        Logging.info("'" + props.getAlias() + "'");
        if (props.getAlias().length() == 0) {
            Logging.info("No alias");
            props.setAlias(this.getName());
            Logging.info("'" + this.getName() + "'");
            Logging.info("'" + props.getAlias() + "'");
        }

Logs:
[21:02:15 INFO]: [Multiverse-Core] ''
[21:02:15 INFO]: [Multiverse-Core] No alias
[21:02:15 INFO]: [Multiverse-Core] 'aaa'
[21:02:15 INFO]: [Multiverse-Core] ''

Looks like setAlias isn't working...
Code:
        Logging.info("'" + props.getAlias() + "'");
        if (props.getAlias().length() == 0) {
            Logging.info("No alias");
            props.setAlias(this.getName());
            Logging.info("'" + this.getName() + "'");
            Logging.info("'" + props.getAlias() + "'");
        }

Logs:
[21:02:15 INFO]: [Multiverse-Core] ''
[21:02:15 INFO]: [Multiverse-Core] No alias
[21:02:15 INFO]: [Multiverse-Core] 'aaa'
[21:02:15 INFO]: [Multiverse-Core] ''


[analyzer] Discrepancy with UP(X & B1, X & B2)

Consider the following program:
bool condition = true;

num f<X>(X x1, X x2) {
  if (x1 is int && x2 is num) {
    var y = condition ? x1 : x2;
    num n = y; // OK.
    X x = y; // OK.
    int i = y; // Compile-time error claims that `y` has type `X & num`.
    return y; // OK.
  }
  return 0;
}

void main() {} // Just so we can use `dart` to test the CFE.

This program is accepted by the analyzer, which is consistent with the assumption that y has an inferred, declared type which is X, and it is immediately promoted to X & num (based on the static type of the initializing expression).
However, the CFE reports errors consistent with the assumption that the static type of the initializing expression is X.
The relevant rule is the following:

UP(X1 & B1, T2) =

T2 if X1 <: T2
otherwise X1 if T2 <: X1
otherwise UP(B1a, T2)
where B1a is the greatest closure of B1 with respect to X1,
as defined in [inference.md].

Instantiating this for the given example we get the following: UP(X & int, X & num) is X, because it is not true that X <: X & num, but it is true that X & num <: X.
My interpretation of the situation is that the analyzer should be adjusted such that it yields the result X as well.
Note that these considerations are independent of https://github.com/dart-lang/language/pull/3435, they are valid both before and after the change which is performed by that PR.

I've suggested a generalization of UP here: https://github.com/dart-lang/language/issues/3462. This is a proposal which will change UP such that we get the best possible result: UP(X & int, X & num) == X & num. However, this is something we can consider later, right now I think the most important consideration is that all tools should agree.
I believe the change proposed in this issue is non-breaking because it will only cause the analyzer to report an error for a source code construct in the case where that same location in code would already be reported as an error by the CFE.

Note that dart-lang/language#3462 does not block this issue: It will surely take some time to develop and accept dart-lang/language#3462, if it is indeed accepted, and this issue should not wait for that.


Is this proposal still active?

cc @mathiasbynens

IIRC when this was last discussed, it was decided that we’d only add this feature if codePointAt turned out to be useful. Revising this today, it’s not clear to me that it was, given that it accepts code unit indices rather than code point indices. '💩'.codePointAt(0) is useful, but '💩'.codePointAt(1) for example probably isn’t as it still returns half of a surrogate pair.
For String#at specifically, spreading a string to get an array of symbols (one for each code point) and then accessing indexes in that array is a possible approach:
const string = 'foo💩bar';
const result = [...string][3]; // 💩

To avoid turning the entire string into an array if you only need, say, the fourth code point, string iteration is the way to go:
const getSymbol = (string, position) => {
  let index = 0;
  for (const symbol of string) {
    if (index++ == position) return symbol;
  }
  return '';
};

getSymbol('foo💩bar', 3);
// → '💩'

@mathiasbynens  Yes, codePointAt seems not very useful on its own.
I always think JS UTF16-only string API may too compromised today, introduce UTF8/UTF16 conversion burden, and very clumsy when you really need to deal with characters beyond BMP. Do we have chance to introduce some new APIs?

One and a half year after the same question - is where any plans about promotion it or it should be moved to inactive proposals and I should remove it from the next version of core-js? -)


Weird behaviour when refresh/load another data on RecyclerView

I use your code to my app, and add spinned menu on top of recyclerview. Each menu have different value/playlist id, I successfully load the data from that API playlist. But the weird thing, when I select second/third option from spinned menu, the result of recyclerview is new data placed on bottom layout, I have to scroll it up to view the result. Any solution for this? Or why this is happen? Much appreciate for your help and thanks for the code.

@indrac can you provide some code for me to look at?  Prefereably if you could link to your project so I can see all the code that is in question would be best.
Also, see which branch of the YouTubePlaylist are you using ?  I have a couple of branches with various changes.  I would recommend you to try the data-binding branch or the rx-java branch as those are a bit burther along than the master branch.

I posted my code on gist github of my 3 files, the YoutubeViewFragment.java is the main view of youtube playlist, I hardcoded it and set playlist ID value on number 1 and 2 on that spinner menu. PlaylistCardAdapter.java is same like your code, I changed a lil bit image loader to Universal Image Loader. And the last file fragment_youtubeview.xml is the layout for my YoutubeViewFragment class.
YoutubeViewFragment.java https://gist.github.com/indrac/22fecce7119c741645d661ed105e2642
PlaylistCardAdapter.java https://gist.github.com/indrac/07faa0fcb53d83eb8a8937a747224dbf
fragment_youtubeview.xml https://gist.github.com/indrac/0518e69555000e20e3a803320e3d2aba
So thanksful for your reply on this issue.

@indrac  Not sure what exactly is wrong with your code.  It might have something to do with the StatefulLayout class that you are using.   Take a look at the recent changes I just made in the master branch.  I added support for selecting a playlist using a spinner from a list of playlist id's.  As an added bonus I also added the capability of fetching the playlist titles and updating the spinner with the playlist title data.

Thanks for answering my question, and finally I know what is wrong with my code. I compile and debug with my Nexus tablet, and I just have to declare this :
mLayoutManager = new StaggeredGridLayoutManager(resources.getInteger(R.integer.columns), StaggeredGridLayoutManager.VERTICAL);
to my OnItemSelected inside spinner.setOnItemSelectedListener.

Glad to hear you were able to figure out the issue you were facing!


Transform strings to normalize URLs

Problem
Users typically do not include the protocol when hand-typing links. As such, applying z.string().url() leads to unintuitive use experience when validating user input. Further a custom transformation (ex: z.string().transform(normalizeUrlLikeString).url()) is not supported by Zod.
Proposed solution
A .toUrl() transformation function that will--

Add https:// if the string does not include a protocol
Validate the string as a URL
Return the new/valid string.

Is this what you are looking for?
const schema = z.union( [
    z.string().url(),
    z.string().transform( x => x.startsWith( 'https://' ) ? x : `https://${ x }` )
] )

console.log( schema.parse( 'foo.com/bar/baz' ) ) // https://foo.com/bar/baz

If you found my answer satisfactory, please consider supporting me. Even a small amount is greatly appreciated. Thanks friend! 🙏
https://github.com/sponsors/JacobWeisenburger

I understand the desire for something like this and your use case makes sense.
Hopefully it's clear why you can't call .url() after .transform(), since your transform could return a value of any data type, not just strings.
Zod needs to support some way to transform input in a non-type-transforming way inside refinements. I have a few ideas in mind for how this could work. You could return the new value from .superRefine, for instance. Or .superRefine could provide a method like resolve on the ctx object (similar to the notion of Promise's resolve.
z.string().superRefine((val, ctx) => {
   ctx.val;  // access input data
   ctx.addIssue(..);   // add issues (similar to super refine)
   ctx.resolve("new value here");
});

I'm weighing these and other options for Zod 4. Leaving open till this gets settled.

Maybe this could just be solved with an option passed to the .url()? Like z.string().url({ protocolOption: true })? 🤷♂


Include Presence API from JS client.

Ref: https://github.com/supabase/realtime-js#presence

Is the new broadcast API already implemented?

Not at the moment, my focus has been on postgrest lately! We’re always open to PRs - otherwise I’ll start looking at this issue this week!

Nice! I'm thinking of using this to make a multiplayer game.
I'm not sure I have the knowledge/time to have a go doing it myself but thanks :)


Demon Gates should notice when a demon is removed from the game

Successor to #32.
Each Demon Gate can only sustain 12 demons at a time. When a demon dies, space is freed up.
When a demon is removed instead of killed, it should also be freed up. This may require the implementation of a custom system to detect unit removals, probably using the Defend bug, which works as follows.
Defend bug
When a unit with the Defend ability is removed from the game, it fires the undefend order even if the unit did not have Defend active. This can be used to make a robust unit removal detection system.

Not worth it. Quite hard to do and this will never matter in a real game.


Server Difficulty

Hi, I'm trying to increase the difficulty on my server.
To do this, ?OverrideOfficialDifficulty=5.0 need to go in the command line.
In the cfg file which of these would be the correct setting ?....
arkOverrideOfficialDifficulty=5.0
arkopt_OverrideOfficialDifficulty=5.0
arkflag_OverrideOfficialDifficulty=5.0
Thanx in advance.

Ignore that.  Just read documentation fully.
ark_=""


Decide on a canonical date format: xsd:date vs xsd:dateTime

I'm trying to put together a template for creating mappings. Is there a recommended format for the date?
Also is there any additional information you would consider essential for a template other than what I already have in this?
https://docs.google.com/spreadsheets/d/1e0hm_WNlzP-kUJHC54LU1W_gwUMVXvDv1kZfGAaMcGI/edit?usp=sharing

Thank you @sbello
These are the two datatype we are considering:
https://www.w3schools.com/xml/schema_dtypes_date.asp

👍 xsd:date is very simple and probably sufficient for mappings, e.g. 2002-09-24
🚀 xsd:dateTime is more granular, and probably more popular in the semantic web 2002-05-30T09:00:00

I personally for now will vote for xsd:date because I like its simplicity; I trust more @cthoyt, @graybeal, @cmungall and @jamesaoverton though and may change my mind. In any case, you can go aheave an use either one for now and convert it later! Thank you for the ticket :)
For the minimum metadata see #69 - we will continue the convo there.

I would suggest using xsd:date.
In the Open PHACTS dataset descriptions we recommended the use of xsd:datetime because that was the type specified in the Provenance, Versioning, and Authoring (PAV) ontology for the properties that we were using. However, most data providers only provided xsd:date.

Thank you @AlasdairGray you can cast your vote in the issue above but adding the thumbs up!

The text is ambiguous as there are two options outlined, which is why I wrote my comment.
Also the comment gives further insight and evidence to support the position.

Yes, we use this a lot now across obo. We assign emoticons to options, then we vote. You can even vote for multiple options - and your additional comments will be read by anyone looking to cast a vote to influence their opinion, so that's great!

An yes, thumbs up mean you prefer dad date!

Thanks Nico! xsd:date makes sense, xsd:dateTime is way to granular.

As always, depends on the applications. A main purpose of this information is to determine (after the fact) order of inputs/operations. If you have a group of 10 people developing mappings in a day, and you are trying to determine which mappings preceded/followed which other mappings (example use cases: which mappings got included in version X? when mapping Y was created could the user already see mapping Z? was the key person there when this mapping was created? when was the system or process most busy?), you have to have the timestamps to do that. That's why any computer event logging application will use datetime.  (Using datetime also make more interesting meta-analyses possible, but I'm sure that's not of keen interest to the average end user!)
That said, I know it's a huge pain to manage that information manually, so if you're going to make a human being provide lots of their own timestamps (can't there be an Excel function to auto-fill this somehow?), I can't expect meaningful datetimes.
But call this a place to thumbs-up the date-time option.

Hey @graybeal:
Just because I think this voting mechanism should be more prevalent everywhere: You should cast your vote directly on the VOTE comment above. Thank you for your insights!

Oops, sorry, got it!

I'm not super opinionated about this. I think date might be easier to deal with just because it's also more human-friendly. I'm not sure if there are any really specific scenarios where the time of day or specific ordering is important. However, if these will all be added automatically, there's no reason not to use the more granular one.

xsd:date it is!


TypeError: setup_keycloak_middleware() got an unexpected keyword argument 'authorization_method'

i was coding with docs but dont work ,how to config authorization thanks

Hi, thanks for raising this issue! It does seem to be an error in the documentation, the authorization related args have to be added to the KeycloakConfiguration. Check the API references for details, I’ll try to work on the docs over the weekend if time permits.


Feature - Basic auth for prometheus logger

Hello, thank you very match for your work. Can you please add functionaity - basic auth for prometheus logger, similarry restapi logger?

i'm sorry for my persistence, it's possible to implement?

Thanks

beta release available, any feedback will be appreciated

Thanks a lot!!! Metrics run.

@dmachard Can we make this optional? This is breaking on my end since my end since scraping expects an endpoint without authentication. I tried setting login/pwd to "" but that didn't work. Maybe a new basic-auth-enable boolean would help here?

Yes I can with a new setting basic-auth-enable
But basic auth will still be enabled by default.

Beta5 available with basic-auth-enable setting

Works a charm! Thanks for the ultra-quick turnaround! 🙏🏻

Implemented in release v0.30.0.

all works fine in our production, large uptime


Docs improvements

Hello
I fixed some minor typos in the code comments, docs,
Hope it helps.
Br,
Tudor.

Hey there, some of the fixes are invalid, e.g. process composes files. Process compose is a framework we are utilizing in our stack.

hello, that s my fault, i reverted it


[3.9] [ES-1727] Fix Smart- and EnterpriseGraph modifications with ignoreRevs: false

Backport of https://github.com/arangodb/arangodb/pull/19683.
Enterprise companion PR: https://github.com/arangodb/enterprise/pull/1368

3.9 is EoL.


Can no longer access to server with basic HTTP authentication

My Setup

Operating System: macOS 10.14.6
App Version: 3.0.4
Installation type: dmg
&
App Version: 3.0.3
Installation type: app Store

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description
After the update (first via App Store), both servers are not connected anymore. Server list lost?
Entering the url using<EMAIL_ADDRESS>results in "No valid server found at the URL"
Entering the url with https://xxx.org results in "Validating..." being stuck.
-> No subfolder used!
Using username and password of the HTTP authentication works with version 2.17.11
I also tried to add the server to the config.json file as recommended in #1796 without success.
Current Behavior
Does not connect
Expected Behavior
Does connect.

I am also encountering this issue. Was trying to find a way to downgrade my MacOS clients to 2.17.9 to get them working again.

The issue persists after upgrade to macOS 10.15 and macOS 11.
Also using the new release 3.1.0, nothing changed...

Unfortunately still the same with version 3.2 (.1/.2).

Since nothing changed on this topic, this comment is just to bump the issue 😉
At least entering the URL without username and password does not result in the process being stuck.
But entering the URL with credentials still results in "No valid server found at the URL"

Once again, bumping.
Still the same situation in version 3.5.

This problem still persists in client version 3.7.1 (tested on Windows).
To elaborate further, the following error seems to be the cause (from DevTools -> Console, credentials and server address redacted):
6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1337 Uncaught (in promise) TypeError: Failed to execute 'fetch' on 'Window': Request cannot be constructed from a URL that includes credentials<EMAIL_ADDRESS>   at i (6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1337)     at 6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1337     at 6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1 i @ 6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1337 (anonymous) @ 6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1337 (anonymous) @ 6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1 await in (anonymous) (async) a @ 6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1 u @ 6823d82bbf984f80d3bbcb33799f26c97653a8df.js?meteor_js_resource=true:1 [server.url-1640080058484.log](https://github.com/RocketChat/Rocket.Chat.Electron/files/7751326/server.url-1640080058484.log)
The full log is attached.

Same problem here

Same problem here

Even though I did not expect any changes, version 3.8.16 is still not capable of connection through basic HTTP authentication. :(

Same issue here (on Linux)

I believe it's for security concerns that it's disabled.
@jeanfbrito should we allow it?

I believe it's for security concerns that it's disabled.
@jeanfbrito should we allow it?

I just found this info from Mozilla^1, where it is stated that the feature is deprecated and removed in Chromium. I know that this authentication is maybe not preferred, but it prevents web crawlers to index the page.

This is not supported on Desktop App.


fix: test 5

Description
fix: test 5

💔 All backports failed

Status
Branch
Result

❌
release/v0.54
Backport failed because of merge conflictsYou might need to backport the following PRs to release/v0.54: - fix: test 4 (#76)

Manual backport
To create the backport manually run:
backport --pr 77

Questions ?
Please refer to the Backport tool documentation and see the Github Action logs for details


Improve visual balance of the output when updating

I would like to propose to move an empty line in doom up's output. It currently looks like this:

As you can see, the lines I've drawn mark logical separation points, i.e. where an update of a package begins and where it ends. However, the included empty lines create a totally different balance and confuse at the first glance, e.g. they suggest that lsp-ivy has been updated with the following commits (which actually belong to haskell-mode). If the empty line coming before $package updated would move to after $package updated and before Updating $package, the visual balance would be less confusing.

As of c841270 the commit log is now displayed below the "updated" message, e.g.

I'll consider this resolved. Let me know if that isn't the case and I'll reopen it. Thanks for bringing it to my attention!


Add usage guide for using RevisionMixin, DraftStateMixin, and PreviewableMixin with snippets

Needs #9021 to be merged.
This PR adds a usage guide on how to use the mixins documented in #9000 with snippets.
I haven't added a management command or migration code example to populate the latest_revision and live_revision fields for existing snippets, and I'm not sure if that's necessary. I've already added hints on how to do it programmatically, though.

Cherry picked to ff38eb2e03f1a883ce307fa59c1a49d92fbb4f95 & parents


Rename eventDef correlation properties

What would you like to be added:
Rename the eventDef's correlation object properties to have something shorter, as explicit, and cleaner IMHO.
Currently, we have:
...
events:
- name: MyEvent
  kind: consumed
  source: ...
  type: ...
  correlation:
  - contextAttributeName: ...
    contextAttributeValue: ...
...

What I propose is to have:
...
events:
- name: MyEvent
  kind: consumed
  source: ...
  type: ...
  correlation:
  - name: ...
    value: ...
...

Why is this needed:
Shorter, cleaner, easier to write without typo/casing mistake, self-explainatory

/remove-stale

This has been addressed in 0.10, and is closed as part of https://github.com/serverlessworkflow/specification/issues/843


[BUG] Sensor values don't update in Screensaver

The following behaviour started with Homeassistant 2023.8:
I show a card with time and some sensor values when the screensaver starts, with following code:
  cards:
    - type: custom:digital-clock
      style: |
        :host {
          font-size: 40px;
        }
    - type: entities
      entities:
        - entity: sensor.easyweatherv1_6_5_indoor_temperature
          name: LR
          icon: hass:none
        - entity: sensor.carport_motion_s3_air_temperature
          name: Out
          icon: hass:none
        - entity: sensor.scd30_co2_2
          name: BR
          icon: hass:none
        - entity: sensor.scd30_lr_co2
          name: LR
          icon: hass:none
      show_header_toggle: false
      card_mod:
        style: |
          ha-card {
            font-size: 60px;
          }

The sensor values display correct when the screensaver starts. But they don't get updated unless I stop the screensaver and start it again. The date/time still updates properly. This has worked in the past and I haven't changed my Wallpanel code.
Device info:

Device: Fire Tablet
OS: Fire OS <IP_ADDRESS>
Fully Kiosk Browser 1.49.1
Homeassistant 2023.8.1

Update: I just rebooted the Fire tablet, and the issue was gone.


problem with theme inheritance

Hi Guys
I love this theme but wanted to extend it using inheritance as described in this document. https://learn.getgrav.org/themes/customization
I believe that mytheme.php would need to be:
`<?php
namespace Grav\Theme;
class Mytheme extends antimatter-open-publishing
{
// Some new methods, properties etc.
}
?>`
However, when I try this I get the error
"unexpected character -" (referring to the hyphens in the theme name)
Is this a bug or am I approaching it the wrong way?

Hi @popebear , glad you are fining the theme useful! As you have discovered, hyphens are not supported for class names. Try this in your mytheme.php:
<?php
namespace Grav\Theme;

use Grav\Common\Theme;

class myTheme extends Theme
{
}
?>

And then this in your mytheme.yaml file try this:
streams:
 schemes:
   theme:
     type: ReadOnlyStream
     prefixes:
       '':
       - user/themes/mytheme
       - user/themes/antimatter-open-publishing
       - user/themes/antimatter

You could also download an earlier release of the Open Publishing Skeleton (it now has Quark as the default theme) to see a working example of this: https://github.com/hibbitts-design/grav-skeleton-open-publishing-space/releases/download/v1.1.6/grav-skeleton-open-publishing-space-site.zip
Let me know if the above works for you.

brilliant - thanks for that!

Hey Paul
It's taken me a while to get to this - sorry.
re your questions;

Let me know if the above works for you?

It's working in that it's now using the files in mytheme, however now when I use admin to change site configurations (for example publish or unpublish a section), that doesn't work
Does that ring any bells with you?
Cheers
Mark.
On 05/05/2018 17:13:30, Paul Hibbitts<EMAIL_ADDRESS>wrote:
Hi @popebear [https://github.com/popebear] , glad you are fining the theme useful! As you have discovered, hyphens are not supported for class names. Try this in your mytheme.php:

And then this in your mytheme.yaml file try this:
streams: schemes: theme: type: ReadOnlyStream prefixes: '': - user/themes/mytheme - user/themes/antimatter-open-publishing - user/themes/antimatter
You could also download an earlier release of the Open Publishing Skeleton (it now has Quark as the default theme) to see a working example of this: https://github.com/hibbitts-design/grav-skeleton-open-publishing-space/releases/download/v1.1.6/grav-skeleton-open-publishing-space-site.zip [https://github.com/hibbitts-design/grav-skeleton-open-publishing-space/releases/download/v1.1.6/grav-skeleton-open-publishing-space-site.zip]
Let me know if the above works for you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [https://github.com/hibbitts-design/grav-theme-antimatter-open-publishing/issues/5#issuecomment-386816394], or mute the thread [https://github.com/notifications/unsubscribe-auth/AiA_F5ZSzq2vjUzLSzAXnEVnu5hfDR3tks5tvc-ngaJpZM4TzmqF].

Hi @popebear , I am not sure why you would be seeing that - have you turned off the cache(s) while you work on your site?


mismatched closing tag

 ->

Interesting. I think this is auto-generated. @tabatkins @szager-chromium right? Is this a bug in bikeshed?

the .bs file is the input file bikeshed uses to generate its html.


Better README.md

Hey @mergesort, your library is really interesting.
The only problem I found was the README.md, which lacks a little bit of information
I created this iOS Open source Readme Template so you can take a look on how to better organize.
If you want, I can help you to organize the lib.
What are your thoughts?

Hey @lfarah! Thanks for the compliment. The readme currently covers the entire API layer, and the rest is relatively tongue in cheek. Do you have any suggestions for how you'd like to see it improved, other than just making it into a template?
Suggestions welcome!

Definitely!
I'd be cool if you had the following sections:
Installation
Meta
Requirements

And please leave the Gifs.... I loved them!

Good feedback, will take care of that! What do you consider to be meta, and requirements?

So Meta would be the part where you talk about yourself, your twitter and the library's license

Requirements is where you say the minimum xcode and iOS version u need to run the library

Sounds good! I'll take this feedback and incorporate it into the README later tonight. 😄

Installation
CocoaPods
You can use CocoaPods to install Anchorman by adding it to your Podfile:
platform :ios, '8.0'
use_frameworks!
pod 'Anchorman'

To get the full benefits import Anchorman wherever you import UIKit
import UIKit
import Anchorman

Manually

Download and drop Anchorman.swift in your project.
Congratulations!

@lfarah It's updated. Feel free to leave me some more feedback if there's anything you'd like to see, or feel free to file a PR.

hahahahha the Readme is awesome. Congrats!


bpf: nodeport cleanups

Some easy cleanups in the nodeport code, to shrink down my feature branch.

/test

@borkmann does the last patch ring any bells?

@borkmann does the last patch ring any bells?

It's been a while, but did you manually test that vxlan and geneve works fine pod to remote pod via tunnel? Did you check each drivers if there's special logic around L2 handling?

@borkmann does the last patch ring any bells?

It's been a while, but did you manually test that vxlan and geneve works fine pod to remote pod via tunnel? Did you check each driver if there's special logic around L2 handling?

CI has pod-to-pod coverage for both vxlan and geneve, so no worries about the good path.
Poked around a bit in geneve.ko and vxlan.ko, but nothing caught my eye. Think that if there's any fallout, it most likely would be in GRO ...?
There's even a few optional things that might work "better" with proper inner L2 addresses (eg. VXLAN_F_LEARN).


[FLINK-20304] Introduce the BroadcastStateTransformation.

What is the purpose of the change
So far the Broadcast State pattern was a TwoInputTransformation.
This did not allow for special handling when it comes to
translating it for Batch or Streaming execution. This commit
introduces a special transformation just for this.
Brief change log
We introduce the BroadcastStateTransformation and the corresponding BroadcastStateTransformationTranslator that throws an UnsupportedOperationException when called for execution in BATCH mode.
Verifying this change
The STREAMING translation is covered by existing tests while I added the BroadcastStateITCase.testBroadcastBatchTranslationThrowsException() for the BATCHcase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@dawidwys I integrated your comments, let me know what you think!

Thanks a lot! I will merge as soon as AZP gives green.


Describe why this will benefit the LibrePhotos
It'll make it easier to correct mistakes made when identifying faces without prompting a full rescan.
Additional context
I'm training faces and occasionally you add a face to a person that it doesn't belong to. No big deal, unfortunaely, there's no way to just remove the face label. My options are move it to another person (which I don't want to do, since it's a random face that got mixed in) or delete the face altogether, which makes the system no longer recognize it as a face (this is not super helpful if I then want to go search for faces or if I find out the name of the individual later and then want to add them). To get that face back I have to rescan the entire database for faces which is inredibly annoying as I have tens of thousands of photos.

Actually I see this a lot. Faces are very often attributed to the wrong people in my test so far with 4000 pictures, while it found around 250 faces.
@derneuere
How about restructuring this two pages below face recognition:

Persons
Unknown

Persons
Contains a short list of some pictures of the person and all non-approved recommendations for this person.
Unknown
Contains a list of faces grouped together like in the face cloud view (but as a normal picture list), to make it easier to select all pictures of the same person.

Additionally a threshold for "auto confirm" could be set, something like 98% or similar.

This will be in the next release :)


Finding contents with CTRL + F

I have a number of nested accordions which is working great, thanks!
I would like to be able to search the contents of the accordions using the "Find on page" features of various browsers (CTRL + F), but I am a bit stuck. I was thinking of intercepting the find call and manually expanding accordions but I'm not sure that will work based on the item in the readme, do you have any advice on how to acomplish this?

Gidday @Meberem, thanks for your issue. Sorry, but at this time we don't support programmatic control of the expanded state of accordion items, as we feel that if fights against the principals of an accessible accordion. Besides that, the native browser search feature does not have a Javascript API with which to bind events to, for example. Hidden content is, by definition, hidden to search.


Enable DEBUG_CLR_GRAPH_PACKET_CAPTURE, the packet capture function can't work with kernel node modified by hipGraphExecKernelNodeSetParams

precondition: enable DEBUG_CLR_GRAPH_PACKET_CAPTURE
steps:

hipGraphCreate
hipGraphAddKernelNode
hipGraphInstantiate
hipGraphExecKernelNodeSetParams
hipGraphLaunch
hipStreamSynchronize

After function hipGraphInstantiate, the AQL packet is generated by CaptureAQLPackets which will be used in hipGraphLaunch step.
But when with hipGraphExecKernelNodeSetParams, the AQL will be modified, and the modified AQL will not executed in hipGraphLaunch.
I think this is an issue.

Thank you @kevin-gjm for raising. I can see the issue from the steps above as the kernel argument offset in particular may become outdated after hipGraphExecKernelNodeSetParams. Would it be possible to share a small reproducer program if you have one already?

This has been fixed in an internal PR.

The fix has been added to develop branch https://github.com/ROCm/clr/commit/a99b163eba57947e77e583494537717a52bc3803 and will be included in a future rocm release.


Demonstrate git stash pop

These changes shouldn't be accepted, this is a demo.

This was a demo, we're not going to merge it.


Regenerated cql4browsers using the newer cake build-cql4browsers command

Latest cql4browsers was not generated properly. This PR is to update cql4browsers.js

Let me run regression tests then we'll call it good.

So... shall I merge?

@cmoesel Yes, go ahead and merge.


setup.py: exclude test.* subpackages from find_packages too

Otherwise, test.test_package will be picked up when installing via PEP517.
Bug: https://bugs.gentoo.org/899628

Thank you

Thanks!


refactor: refactor class SecretKey & PublicKey

by refactor class SecretKey & PublicKey for reliability to match real situation between 2 individual programs and also add an example into README file.

by refactor class SecretKey & PublicKey for reliability to match real situation between 2 individual programs and also add an example into README file.

According to this commit, the testing scripts should also be modified through.
Please take a look and keep and glad to help !!


Update Readme.md

Thanks!

@kamisamamizu would it be possible for you to get me info about your Windows build, hyper version, npm, hyper config, etc?

You’re the first person I’ve seen that it’s worked for

*worked perfectly

Well, not perfectly, like I said in the post, the network info doesn't work, but everything else does, like you can see in the screenshot.

Windows 10 v. 1703 (build 15063.540)
Hyper 1.3.3
Using Bash on Windows, but it automatically starts fish. Doesn't matter, it also works with cmd.
Result of npm version:
{ npm: '3.10.10', ares: '1.10.1-DEV', http_parser: '2.7.0', icu: '58.2', modules: '48', node: '6.11.2', openssl: '1.0.2l', uv: '1.11.0', v8: '5.1.281.103', zlib: '1.2.11' }
Hyper config: https://pastebin.com/D7MtsNYM

If you need anything else, tell me.
Also, why is this conversation in here? Shouldn't it be in https://github.com/Hyperline/hyperline/issues/57#issuecomment-324336456 ?

Uhhhh that's a good question 🤔
When I replied to you I was looking at Hyperline/hyperline#57
I'm so confused right now...


WINAPI NetStatisticsGet will RaiseException 0x000006BA: The RPC server is unavailable

if windows services "LanmanServer" or "LanmanWorkstation" not exist,WINAPI will RaiseException 0x000006BA.
call trace:
KERNELBASE.dll!_RaiseException@16()  + 0x58 字节
rpcrt4.dll!_RpcpRaiseException@4()  + 0x2f 字节
rpcrt4.dll!_NdrGetBuffer@12()  + 0xfe5b 字节
rpcrt4.dll!_NdrClientCall2()  + 0x118 字节
srvcli.dll!_NetrServerStatisticsGet@20()  + 0x19 字节
srvcli.dll!_NetServerStatisticsGet@16()  + 0x37 字节
netapi32.dll!_NetStatisticsGet@20()  + 0x53 字节

test.exe!RAND_poll()  行 272 + 0x12 字节	C
test.exe!ssleay_rand_status()  行 603	C

rpcrt4.dll patched by KB4100480(https://support.microsoft.com/en-us/help/4100480/windows-kernel-update-for-cve-2018-1038)
openssl code:
https://github.com/openssl/openssl/blob/OpenSSL_1_0_2-stable/crypto/rand/rand_win.c
line 268/272
test on windows 7 sp1 /server 2008 r2

I can't confirm this. In sense that if I call NetStatisticsGet with name of a non-existing service I get no exception, NetStatisticsGet simply returns error code. Nor do I get any exceptions if I stop either of the services, call returns different value though. Putting aside the fact that it shouldn't have raised exception (MSDN says NetStatisticsGet should return value), 0x6ba means RPC_S_SERVER_UNAVAILABLE, and this asks for question which server, as it's obviously not any of the services in question. I mean since I don't get any exception if I stop them. Question also is how come services in question don't exist? Did you delete them? Or do you get this error in your service, which starts early enough? If former, then we would probably would have to say "you're on your own". If latter, then it might be appropriate to make your service dependent on at least RPC service...

i have delete this services(for anti WannaCry).
exception maybe ignored by system.
100% catch in VEH(Vectored Exception Handling)

ok,please close this issue.


better message/error formatting

kelp has a solid message facility. It can display multiple levels of priority and change outputs. Yet it is missing fancy formatting. All the necessary data  should be already included in Error

As of today, this is fixed, kelp uses MessageDispatcher facility that can be set and pretty prints errors and other messages.


FreeProxyException

fp.errors.FreeProxyException: There are no working proxies at this time.

Can't reproduce. Is it something frequent? or just one time issue?
This is a free-proxy library issue, not from this package.

It happens a lot these days
srtranslator.translators.base.TimeOutException: Translation timed out
I'll try again. Thank you.

Report.
No problem if you change to TranslatePy(), DeeplTranslator() happens srtranslator.translators.base.TimeOutException: Translation timed out

By the way, thanks for developing this tool.

Can't reproduce, reopen if issue keeps happening


Many2Many Update can not update Middle table

type User struct {
	ID        uint `gorm:"primary_key" json:"id"`
        ......
	Roles []*Role `gorm:"many2many:role_users;" json:"roles"`
}

type Role struct {
	ID        uint `gorm:"primary_key" json:"id"`
        ......
	Menus []*Menu `gorm:"many2many:role_menus;" json:"menus"`
}

func xxx(){
database.DB.Debug().Model(&form).Updates(form)
}

For example, There is a User does not belong to any role.
First time I update a User( user_id = 2 ), And update two Role ( role_id = 4,5 )
First time Debug:
UPDATE `xxx_users` SET `id` = 2, `updated_at` = '2020-03-20 13:56:49', `username` = 'yyadmin2'  WHERE `xxx_users`.`deleted_at` IS NULL AND `xxx_users`.`id` = 2

UPDATE `xxx_roles` SET `created_at` = '2020-03-19 16:28:41', `updated_at` = '2020-03-20 13:56:49', `deleted_at` = NULL, `name` = 'aaa', `description` = 'aa'  WHERE `xxx_roles`.`deleted_at` IS NULL AND `xxx_roles`.`id` = 5

INSERT INTO `xxx_role_users` (`user_id`,`role_id`) SELECT 2,5 FROM DUAL WHERE NOT EXISTS (SELECT * FROM `xxx_role_users` WHERE `user_id` = 2 AND `role_id` = 5)

UPDATE `xxx_roles` SET `created_at` = '2020-03-19 15:58:08', `updated_at` = '2020-03-20 13:56:49', `deleted_at` = NULL, `name` = 'test2', `description` = 'test2'  WHERE `xxx_roles`.`deleted_at` IS NULL AND `xxx_roles`.`id` = 4

INSERT INTO `xxx_role_users` (`user_id`,`role_id`) SELECT 2,4 FROM DUAL WHERE NOT EXISTS (SELECT * FROM `xxx_role_users` WHERE `user_id` = 2 AND `role_id` = 4)

First time Database(Middle table xxx_role_users):

user_id
role_id

2
4

2
5

Second time I update a User( user_id = 2 ), And update two Role ( role_id = 5,6 )
Second time Debug:
 UPDATE `xxx_users` SET `id` = 2, `updated_at` = '2020-03-20 14:02:50', `username` = 'yyadmin2'  WHERE `xxx_users`.`deleted_at` IS NULL AND `xxx_users`.`id` = 2

UPDATE `xxx_roles` SET `created_at` = '2020-03-19 16:28:41', `updated_at` = '2020-03-20 14:02:50', `deleted_at` = NULL, `name` = 'aaa', `description` = 'aa'  WHERE `xxx_roles`.`deleted_at` IS NULL AND `xxx_roles`.`id` = 5

INSERT INTO `xxx_role_users` (`role_id`,`user_id`) SELECT 5,2 FROM DUAL WHERE NOT EXISTS (SELECT * FROM `xxx_role_users` WHERE `role_id` = 5 AND `user_id` = 2)

UPDATE `xxx_roles` SET `created_at` = '2020-03-19 16:46:47', `updated_at` = '2020-03-20 14:02:50', `deleted_at` = NULL, `name` = 'aaaaaa', `description` = ''  WHERE `xxx_roles`.`deleted_at` IS NULL AND `xxx_roles`.`id` = 6

 INSERT INTO `xxx_role_users` (`user_id`,`role_id`) SELECT 2,6 FROM DUAL WHERE NOT EXISTS (SELECT * FROM `xxx_role_users` WHERE `user_id` = 2 AND `role_id` = 6)

Second time Database(Middle table xxx_role_users):

user_id
role_id

2
4

2
5

2
6

##Question:
second time I want to update Middle table. And want to delete role_id = 4. Why GROM can not auto update  Middle table??????
I want to update middle table in second time Like:

user_id
role_id

2
5

2
6

You can try：
db.Model(&user).Association("Roles").Replace(roles)


Fix tests on arm64 & GH Actions

Also on amd64, as a bonus. make test wasn't passing on a fresh instance of Codespaces before the changes.
I took the freedom to fix the Github Actions configuration that was programmed to run builds on pushes to master, but this repository uses main as default branch.

@bwplotka due to issues with the GH Actions being used to setup Docker in the macOS runner, I moved to a simpler alternative solution using Colima and Lima-VM. Lima-VM is using QEMU with some sort of acceleration called HVF that makes it suitable for this task. It's the same mechanism that Rancher Desktop uses under the hood.


Feat:Navbar looks very big

Describe the feature
navbar height is lil lengthy    please add the level also
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@jitendra-ky plz add the lvel also

@jitendra-ky plz add the lvel also

@keerthana2-005 , I am not the maintainer you need to tag the maintainer for that.

@andoriyaprashant  plz assign the level I can work on it

i love to work on that can you assign that to me 👍 here from gssoc

you can assign this task to me...


Minor UI tweak in module list for gameplay modules.

The UI now checks for dependencies while selecting the color to use to display gameplay modules.

Can one of the admins please verify this patch?

@GooeyHub ok to test

Hooray Jenkins reported success with all tests good!

Thanks for the report @oniatus, merged it!


Update JCK_CMD_TEMPLATE to reduce duplicate

Close #3639
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

Grinder temurin-compliance/job/Grinder/1311/

@Mesbah-Alam could you please take a review? Thanks.

Thanks for this work, @sophia-guo - the world is a bit cleaner now!
Tested : Grinder/23742.


Add typings to package

When the typings aren't packaged, then typescript cannot find the
typings when the package is used as dependency.
Fixes #6

Without this, one has to manually copy-and-paste the typings to their project in order to use this package. Please merge if possible. 🙇

Thanks for the PR. Will publish this later.


Processing fails because of wrong binning size

🐛 Description
L1b-processing works now, at least in the operational sense: QA needs to be performed, but the pipeline works.
A few files are unprocessable at the moment, for the following reason:
the size of the image and the binning size are incompatible

This is raised here:
https://github.com/innosat-mats/MATS-L1-processing/blob/84d06517eab86ab84c1bcbe8b66d217793560641/src/mats_l1_processing/L1_calibration_functions.py#L625-L633
First, two comments on this:

it would be nice if the error messages were different, so that you can tell whether rows or cols are the problem
consider adding the sizes to the error-message.

Now the problem: The way processing is implemented, if an uncaught error occurs, the whole file is failed and no calibrated images will be stored. This is intentional, since we want to make sure that we don't store garbage, and we want to have the opportunity to catch and fix bugs. Failed files can be re-run once the issue is fixed. (Each file contains just 1 hour of data now, so hopefully this delay is acceptable.)
We (@innosat-mats/molflow) could make an exception here, and continue processing the file in the hope that some images are processable, but to me this looks like something that should be caught in the processing step, before it reaches our code, and an appropriate error-flag returned. My suggestion is therefore that you (@innosat-mats/scientific) implement error handling for this in the primary code-base.
🔢 To reproduce
Process any of these files:
ops-payload-level1a-v0.4/2022/12/22/19/MATS_OPS_Level0_VC1_APID100_20221222-133247_20221223-134152.parquet
ops-payload-level1a-v0.4/2022/12/23/1/MATS_OPS_Level0_VC1_APID100_20221222-133247_20221223-134152.parquet
ops-payload-level1a-v0.4/2022/12/23/2/MATS_OPS_Level0_VC1_APID100_20221222-133247_20221223-134152.parquet
ops-payload-level1a-v0.4/2023/1/3/4/MATS_OPS_Level0_VC1_APID100_20230102-150007_20230103-151028.parquet
ops-payload-level1a-v0.4/2023/1/10/23/MATS_OPS_Level0_VC1_APID100_20230110-142705_20230111-081424.parquet
ops-payload-level1a-v0.4/2023/1/11/23/MATS_OPS_Level0_VC1_APID100_20230111-151822_20230112-082136.parquet
ops-payload-level1a-v0.4/2023/1/12/22/MATS_OPS_Level0_VC1_APID100_20230112-143446_20230113-145046.parquet
ops-payload-level1a-v0.4/2023/1/14/2/MATS_OPS_Level0_VC1_APID100_20230113-145046_20230114-132321.parquet
ops-payload-level1a-v0.4/2023/2/2/0/MATS_OPS_Level0_VC1_APID100_20230201-135801_20230202-141026.parquet

✌🏽 Expected behaviour
Error should be caught before reaching the batch-processor and appropriate action taken (e.g. return error-flag)

@OleMartinChristensen Do you agree this is something that should be fixed in your end of the code?

Yes, this should only happen if the files are corrupt and I dont think they are (as they would not pass through the l0 processing). So its probably an index error somewhere. Let me check

the problem is that temperature = NAN is not handled. I will fix this.

the problem is that temperature = NAN is not handled. I will fix this.

Interesting thing to cause a binning problem! Not where I would have looked first – good catch!

The problem persists for a number of files, see appended file:
binfail.txt
The problem, from our point of view, is that an Exception is explicitly thrown, but there is no code that catches it. Maybe you want to have a look at the files above, to see if you can fix the root cause, but I think the code should also handle this error.
If you like, I can refactor it a bit so that we can handle it (that is to say: skip offending images, but not the entire file) in the Lambda-code. Let me know what you think is appropriate.

Jag undrar om inte denna test är felaktig formulerad : nchunks_r=int(image.shape[0])/nbin_r
nchunks_c=int(image.shape[1])/nbin_c
I binnade och beskurna bilder detta behöver inte vara sant
Output exceeds the size limit. Open the full output data in a text editor
{'EXP Date': Timestamp('2023-02-06 00:43:13.498855591+0000', tz='UTC'),
'File': 'MATS_OPS_Level0_VC1_APID100_20230205-125841_20230206-130603.rac',
'ProcessingTime': Timestamp('2023-02-06 13:59:31.928491590+0000', tz='UTC'),
'RamsesTime': Timestamp('2023-02-06 13:02:58.725000+0000', tz='UTC'),
'QualityIndicator': 0,
'LossFlag': 0,
'VCFrameCounter': 124,
'SPSequenceCount': 14913,
'TMHeaderTime': Timestamp('2023-02-06 00:43:18.979125977+0000', tz='UTC'),
'TMHeaderNanoseconds':<PHONE_NUMBER>979125977,
'SID': '',
'RID': 'CCD5',
'CCDSEL': 5,
'EXP Nanoseconds':<PHONE_NUMBER>498855591,
'WDW Mode': 'Automatic',
'WDW InputDataWindow': '11..0',
'WDWOV': 0,
'JPEGQ': 90,
'FRAME': 915,
'NROW': 162,
'NRBIN': 2,
'NRSKIP': 5,
'NCOL': 43,
'NCBIN FPGAColumns': 1,
'NCBIN CCDColumns': 40,
Image size is (162, 44)
För rows då är det ok 162 är delbar med 2 men för cols 44 är inte delbar med 40 som jag antar är en nbin_c
Ncol är den binnad image size (egentligen ncol+1) altså 44 cols där 40 pixlar har binnats innan utläsning.
Det finns inget att säga att det finns ett exakt förhållande  mellan dem
NCSKIP + (NCOL+1)*NCBIN CCDColumns bör vara < 2048 där resten är det som skippas upptill
201 + 44 * 40 = 1961


Rename classes from Covid* to Cord19*

With the publication of the paper CORD-19: The Covid-19 Open Research Dataset, I think it's important to accurately capture the actual name of the dataset.

The Solr stuff needs to be renamed from covid to cord19 also:
https://github.com/castorini/anserini/blob/master/docs/experiments-covid.md


Debug mode

Thanks for writing this one. I'm using it on a new project and it seems to work pretty well. Anyways, one item that I had a form that was missing an input box for a required field in the schema. Debugging it was very frustrating b/c it wasn't writing out any input. SimpleSchemas has a debug mode that will give you some output on what is going wrong.
If I get some time this week, I'll work on a pull request.
Thanks!

Hey, yeah, I've been planning on adding this but I'm a bit busy at the moment. A PR would be great, although the code isn't really commented yet so you might have a hard time figuring some stuff out (I'll comment everything this week). Let me know if/when you start and if you have any issues.

I've added a debug setting which for now just prints out validation errors to the console (which includes fields that are missing/failed validation and fields that are not supposed to be there (i.e. are not in the schema)). I'll improve it to be more helpful soon. Let me know if you want something else to be printed out as well.
To enable it, add FormHandler.debug = true; inside a Meteor.startup call.


bug: panic: runtime error: invalid memory address or nil pointer dereference

Observed testing out docs in https://github.com/rollkit/docs/pull/186
Full logs of rollkit node: https://app.warp.dev/block/Umjjh5DfFI44GGN3WoTgYJ
Logs from light node: https://app.warp.dev/block/nbsQ7lYKCWIisGm9t5PUaB
2:11PM INF indexed block height=4 module=txindex
2:11PM INF successfully submitted Rollkit block to DA layer daHeight=42606 module=BlockManager rollkitHeight=4
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x100f1da58]

goroutine 233 [running]:
github.com/cosmos/cosmos-sdk/baseapp.(*BaseApp).Commit(0x14001494380)
	github.com/cosmos/cosmos-sdk@v0.45.4/baseapp/abci.go:310 +0x88
github.com/tendermint/tendermint/abci/client.(*localClient).CommitSync(0x140011faa80)
	github.com/tendermint/tendermint@v0.35.9/abci/client/local_client.go:297 +0xe0
github.com/tendermint/tendermint/proxy.(*appConnConsensus).CommitSync(0x140006dc0a0?)
	github.com/tendermint/tendermint@v0.35.9/proxy/app_conn.go:96 +0x28
github.com/rollkit/rollkit/state.(*BlockExecutor).commit(_, {_, _}, {{{0xb, 0x0}, {0x101c04947, 0x7}}, {0x14001473730, 0x9}, 0x1, ...}, ...)
	github.com/rollkit/rollkit@v0.9.0/state/executor.go:271 +0xc4
github.com/rollkit/rollkit/state.(*BlockExecutor).Commit(_, {_, _}, {{{0xb, 0x0}, {0x101c04947, 0x7}}, {0x14001473730, 0x9}, 0x1, ...}, ...)
	github.com/rollkit/rollkit@v0.9.0/state/executor.go:179 +0x64
github.com/rollkit/rollkit/block.(*Manager).publishBlock(0x14000128000, {0x102c4bc70, 0x140012478b0})
	github.com/rollkit/rollkit@v0.9.0/block/manager.go:586 +0x730
github.com/rollkit/rollkit/block.(*Manager).AggregationLoop(0x14000128000, {0x102c4bc70, 0x140012478b0}, 0x0)
	github.com/rollkit/rollkit@v0.9.0/block/manager.go:212 +0x1d8
created by github.com/rollkit/rollkit/node.(*FullNode).OnStart
	github.com/rollkit/rollkit@v0.9.0/node/full.go:312 +0x56c

I'm suffering from the same bug, it only throws an error under very specific conditions, and it's asynchronous.
version<EMAIL_ADDRESS>Most likely<EMAIL_ADDRESS>is customizing the syntax incorrectly.

Is this still an issue? @nodebreaker0-0 we removed the cometbft (previously tendermint) dependency, and have new rollkit version (v10.2) and corresponding docs.

Closing this stale issue, please re-open if it persists.


Real estate landing page

Describe the project you want to add with tech stack

Reacts js
Expected behavior

A real estate landing page for anyone to get started with building a mern stack project
Screenshots (optional)

Additional context (optional )

I would like to work on this under gssoc'23

Hey there! I noticed the open issue in the repository. I would like to work on this frontend project and describe the tech stack and expectations for a real estate landing page. I will be using React.js for the frontend development. Please assign this issue to me under Gssoc'23.
Thanks!

@pranjay-poddar I HAVE MY PROJECT READY, DO I NEED TO HOST IT AS WELL?

No hosting not required

Submitting project today @pranjay-poddar


Extra steps I had to take to get running under Ubuntu 16.04

I am very much expecting for someone to tell me I did something stupid, but I had to do the following to get this working in Ubuntu 16.04.
So I setup codeclimate cli as per documentation. This was working great via command line. I altered the path in settings to point to correct location so no issues with not finding executable.
I then got this: usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require': cannot load such file -- cc/yaml (LoadError)
To resolve this I need to manually install each dependency using gem install.
Once I had done this I then got this error: Error: error: (Errno::EACCES) Permission denied @ rb_sysopen - /tmp/cc/892aeb9d-d516-4e01-81dd-3b2936387ca2
The cc directory is being mounted as root, I guess due to the daemon running as such. So I changed the owner:group to my user on the cc directory and the plugin worked great. Note I was running the codeclimate cli without sudo via terminal and it was working.

Hi @jakeBarwell!
It sounds like maybe you installed via gem install codeclimate?
We recommend installing the CLI as a docker image - that way you don't need to be manually installing the dependent gems. The steps are here: https://github.com/codeclimate/codeclimate#anywhere

Hey @maxjacobson thanks for reply.
No didn't use gem install. Installed as docker image using the curl method you linked to.

Hm interesting.
I just tried setting this up on a linux machine and found that I had to explicitly set the path in the package settings to /usr/local/bin/codeclimate, and then things worked for me. It definitely seems like it's picking up a different codeclimate binary if it's trying to load missing gems. Another thing you can check is which codeclimate in your terminal, and make sure that it's point to the CLI wrapper script that runs the docker container

Ok thanks I have it working at the moment but will need to reinstall from scratch soon as new Laptop coming. So will test out fully when I reinstall, will close this as it does seem like I messed up somewhere, if I have issues again on fresh install will re raise issue with further detail.

Ok @maxjacobson . I reinstalled today and yea the extra step of setting full path worked flawlessly. So the default does not work (I went wrong on my first install by pointing to the full path of the bin in the Codeclimate repo directory rather than the /usr/local/bin/codeclimate). Could be a useful tip on documentation.

Glad it's working for you now!
Agreed the docs (or maybe the in-app configuration screen?) could be more clear


Add user group

Description:
This add a "user" group. User group has access to all entities, but is not an admin, hence cannot administrate things.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

The admin group is different from the user group not based on the policy, but based on the group. All admin actions should check this property: https://github.com/home-assistant/home-assistant/blob/113db9afd47e5db905940a82715033a0be24018b/homeassistant/auth/models.py#L77-L84
Once we can launch that, we can slowly start breaking policies out of the admin group while making sure the admin system group remains keeping those policies. That way we can improve it backwards compatible.

I'd call it operator instead, too many users with this naming

@Kane610 I don't understand. Which one would you rename ? I think that having a dropdown with system groups, being able to pick "Administrator", "Users", "Read Only" seems like it makes sense ?

Sorry, I'm talking about all things user management. Which is a separate concept from these roles, something like administrator, operator and viewer/read only is better as explanatory in a role purpose, everyone is still a user. But I guess it might mainly affect us people affected by a specific type of detail obsession :)

Yes 😛


Create test_mixin.md

/testcheck

Here is a result of fact-checking:
Mixin Network lost at least $141 million due to a database attack. :white_check_mark:
https://twitter.com/peckshieldalert/status/1706199059705598406
The attacker compromised the cloud and recovered the private keys of deposit addresses. :white_check_mark:
https://twitter.com/BlockSecTeam/status/1706319766544155068
The attacker transferred funds involving 10,000+ transactions. :white_check_mark:
https://twitter.com/MixinKernel/status/1706139175018529139
The North Korean Lazarus Group is suspected to be behind the hack. :white_check_mark:
https://rekt.news/mixin-rekt
Mixin Network lost $141,328,868.21 in total, including $94,48M in Ethereum, $23,55M in DAI, and $23,30M in Bitcoin. :white_check_mark:
https://twitter.com/peckshieldalert/status/1706199059705598406

/testcheck

Here is a result of fact-checking:
Mixin Network lost at least $141 million due to a database attack. :white_check_mark:
https://twitter.com/peckshieldalert/status/1706199059705598406
The attacker compromised the cloud and recovered the private keys of deposit addresses. :white_check_mark:
https://twitter.com/BlockSecTeam/status/1706319766544155068
The North Korean Lazarus Group is suspected to be behind the hack. :white_check_mark:
https://rekt.news/mixin-rekt
Mixin Network lost $141,328,868.21, including $94,48M in Ethereum, $23,55M in DAI, and $23,30M in Bitcoin. :white_check_mark:
https://twitter.com/peckshieldalert/status/1706199059705598406
The funds were transferred from Mixin Network starting from September 22, 2023. :white_check_mark:
https://etherscan.io/tx/0xd5e2209c988b8d5a92617bac2ea24ca3e411b011787a9837aedb1e6ee7bbc68d
SlowMist, a blockchain security company, reported the attack on September 24, 2023. :white_check_mark:
https://twitter.com/SlowMist_Team/status/1706133260869468503
Mixin Network offered a reward of 10% ($20M) to the hacker for the funds return. :white_check_mark:
https://etherscan.io/tx/0x63b2433505098c584c09f70d5309ae2a6762883b8b9c83ad29f997c657f2593a
Feng Xiaodong, founder of Mixin Network, stated that users will only get access to half of their assets for now. :white_check_mark:
https://www.theblock.co/post/252716/mixin-network-founder-says-just-half-users-assets-are-safe-after-200-million-hack
The hacker's historical relationship with Mixin Network was revealed. :white_check_mark:
https://twitter.com/ScopeProtocol/status/1706264439882944999
Mixin Network is working with Google (Mandiant) and SlowMist Team to assist with the investigation. :white_check_mark:
https://twitter.com/MixinKernel/status/1706948541850235274

/testcheck

Here is a result of fact-checking:
Mixin Network lost at least $141 million due to a database attack :white_check_mark:
https://twitter.com/peckshieldalert/status/1706199059705598406
The database of Mixin Network's cloud service provider was hacked :white_check_mark:
https://twitter.com/MixinKernel/status/1706139175018529139
The attacker compromised the cloud and recovered the private keys of deposit addresses and hot wallet addresses :white_check_mark:
https://twitter.com/BlockSecTeam/status/1706319766544155068
The North Korean Lazarus Group is suspected to be behind the hack :white_check_mark:
https://rekt.news/mixin-rekt
Mixin Network lost $141,328,868.21 in the hack :white_check_mark:
https://twitter.com/peckshieldalert/status/1706199059705598406
The hack involved losses in Ethereum, DAI, and Bitcoin :white_check_mark:
https://twitter.com/peckshieldalert/status/1706199059705598406
The hack resulted in 10,000+ transactions lasting several hours :white_check_mark:
https://twitter.com/MixinKernel/status/1706139175018529139
Mixin Network offered the hacker a reward of 10% ($20M) for funds return :white_check_mark:
https://etherscan.io/tx/0x63b2433505098c584c09f70d5309ae2a6762883b8b9c83ad29f997c657f2593a
Mixin Network founder stated that only half of the user assets are safe :white_check_mark:
https://www.theblock.co/post/252716/mixin-network-founder-says-just-half-users-assets-are-safe-after-200-million-hack
Mixin Network is working with Google (Mandiant) and SlowMist Team for the investigation :white_check_mark:
https://twitter.com/MixinKernel/status/1706948541850235274


Fix compilation for GCC-12 and make some refactoring

Scope & Purpose

Update function2 to f569a63cfe369df867a1a4d17aaa12269156536c
For noexcept and some other fixes

Add absl::strings for working StrCat

Fix gcc-12 compilation

Fix missing includes on gcc-12

Fix initialization order warning on gcc-12

Make CallbackGuard better
Make noexcept check in compile time and use move only function

Make RebootTracker better
https://github.com/arangodb/arangodb/pull/16328#discussion_r888285559

Make RestHandler better
Don't need to use Future<T> here, enough Future<Unit>

Make UpgradeTasks better
Minor improvements

Fix RebootTracker usage

Fix RestHandler usage

[ ] :hankey: Bugfix
[ ] :pizza: New feature
[x] :fire: Performance improvement
[x] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)
[ ] Backport for 3.7: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

https://jenkins.arangodb.biz/job/arangodb-matrix-pr/21923/

https://jenkins01.arangodb.biz/job/arangodb-matrix-pr/21969/


Suspense tags

I do not find Caniuse page for Suspense tags

@DerekDcq do you have a link to what you mean by any chance?
do you mean https://vuejs.org/guide/built-ins/suspense.html…?

Yeah I'm guessing it's that one or this: https://17.reactjs.org/docs/concurrent-mode-suspense.html
As these are constructs used in their respective frameworks and not native DOM elements they will not be tracked on caniuse.


File size? Why so large?

Can someone please explain why the executable is so much larger than Electrum proper and the LTC variant? Trusting this exe is difficult without fully interrogating the source given this size differential.
Any info you can share would be appreciated...

Any chance you can comment on this @csulmone as it would go far to help boost trust in this wallet?

Closed? No comment!? Seriously? @csulmone

actually we are looking for a straight answer, something like, because this
and that... I am not a developer and I want an answer too.
El mié., 11 jul. 2018 a las 17:25, J62<EMAIL_ADDRESS>escribió:

Source code is there.. You can see everything... If you don't trust it,
read the code and build it on your machine. It will come out the same size,
then do you have any idea why it is larger?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BTCPrivate/electrum-btcp/issues/80#issuecomment-404329771,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-NXmN3fFKTw8UqcS-8EBxrdktA8AVgks5uFntIgaJpZM4UCLEt
.

@nolteman You probably won't get an answer like that, since it was forked from ZClassic's repo and there have been a multitude of specialized modifications for Z-based coins and then more modifications and improvements specific to Bitcoin Private but if you wanted to know, you can compare the repo with the main spesmilo repo. https://github.com/spesmilo/electrum/compare/master...BTCPrivate:master

                                                                                That's a perfect answer to understand thanks                                                                                Sent from my BlackBerry 10 smartphone.                                                                                From: michaelotisSent: Wednesday, July 11, 2018 10:53 PMTo: BTCPrivate/electrum-btcpReply To: BTCPrivate/electrum-btcpCc: nolteman; MentionSubject: Re: [BTCPrivate/electrum-btcp] File size? Why so large? (#80)@nolteman You probably won't get an answer like that, since it was forked from ZClassic's repo and there have been a multitude of specialized modifications for Z-based coins and then more modifications and improvements specific to Bitcoin Private but if you wanted to know, you can compare the repo with the main spesmilo repo. spesmilo/electrum@master...BTCPrivate:master

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/BTCPrivate/electrum-btcp","title":"BTCPrivate/electrum-btcp","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/BTCPrivate/electrum-btcp"}},"updates":{"snippets":[{"icon":"PERSON","message":"@michaelotis in #80: @nolteman You probably won't get an answer like that, since it was forked from ZClassic's repo and there have been a multitude of specialized modifications for Z-based coins and then more modifications and improvements specific to Bitcoin Private but if you wanted to know, you can compare the repo with the main spesmilo repo. https://github.com/spesmilo/electrum/compare/master...BTCPrivate:master"}],"action":{"name":"View Issue","url":"https://github.com/BTCPrivate/electrum-btcp/issues/80#issuecomment-404381154"}}}
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/BTCPrivate/electrum-btcp/issues/80#issuecomment-404381154",
"url": "https://github.com/BTCPrivate/electrum-btcp/issues/80#issuecomment-404381154",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [BTCPrivate/electrum-btcp] File size? Why so large? (#80)",
"sections": [
{
"text": "",
"activityTitle": "michaelotis",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@michaelotis",
"facts": [
]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "BTCPrivate/electrum-btcp",\n"issueId": 80,\n"IssueComment": "{{IssueComment.value}}"\n}"
}
]
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/BTCPrivate/electrum-btcp/issues/80#issuecomment-404381154"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "MuteNotification",\n"threadId": 336113965\n}"
}
],
"themeColor": "26292E"
}
]


TypeError when logging exceptions

Some change introduced in 5.4.3 causes TypeErrors in our app every time an exception is logged.
This is a typical traceback with app code until logger invocation removed:
 File "[our_app].py", line 1126, in [some_function]
    logger.exception("Foo")
  File "/usr/local/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1193, in exception
    self.error(msg, *args, **kwargs)
  File "/usr/local/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1186, in error
    self._log(ERROR, msg, args, **kwargs)
  File "/usr/local/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1279, in _log
    self.handle(record)
  File "/usr/local/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1289, in handle
    self.callHandlers(record)
  File "/usr/local/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1329, in callHandlers
    hdlr.handle(record)
  File "/usr/local/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 757, in handle
    self.emit(record)
  File "/Users/me/.virtualenvs/our_app/lib/python2.7/site-packages/raven/handlers/logging.py", line 70, in emit
    if self.client.raise_send_errors:
  File "/Users/me/.virtualenvs/our_app/lib/python2.7/site-packages/raven/contrib/django/models.py", line 54, in <lambda>
    __getattr__ = lambda x, o: getattr(get_client(), o)
  File "/Users/me/.virtualenvs/our_app/lib/python2.7/site-packages/raven/contrib/django/models.py", line 150, in get_client
    Client = import_string(client)
  File "/Users/me/.virtualenvs/our_app/lib/python2.7/site-packages/raven/utils/imports.py", line 9, in import_string
    module = __import__(module_name, {}, {}, [class_name])
TypeError: Item in ``from list'' not a string

The relevant settings.LOGGING['handlers'] snippet:
        'sentry': {
            'level': 'ERROR',
            'filters': ['require_debug_false'],
            'class': 'raven.contrib.django.raven_compat.handlers.SentryHandler',
        },

Please let me know if you need any other details.

Are you using unicode literals future import? This would suggest the 'class' here is a unicode string, and we dont coerce it to a bytestring, which is required for imports.

I found exactly one unicode_literals instance in the application code, but removing it doesn't fix the TypeErrors.
Besides that, all of our migrations which are generated by Django use unicode_literals import, but that code should not be imported during normal operation.

I'm not entirely sure why something wouldn't be a string here, but we'll put in a bit to force statements to a string in Python 2.

Also seeing this issue. Not using unicode literals.

I've pushed the given fix to 5.4.4.


getUserMedia method example will be TypeError(API doc bug)

https://api.dartlang.org/stable/2.4.0/dart-html/Navigator/getUserMedia.html
Uncaught (in promise) TypeError: Failed to execute
'createObjectURL' on 'URL': No function was found that matched the signature provided.

window.navigator.getUserMedia(audio: true, video: true).then((stream) {
  var video = new VideoElement()
    ..autoplay = true
    ..src = Url.createObjectUrlFromStream(stream);
  document.body.append(video);
});

Removed URL.createObjectURL from MediaStream(https://developers.google.com/web/updates/2018/10/chrome-71-deps-rems).
We should use srcObject.

cc @terrylucas


sscan is not implemented

sscan is not implemented.
I can fix it if you want but I wanted to report it here before I intrude.

I think you're in the wrong repository. You linked to a different package.

@Anut-py omg lol yes, haha sorry


PARQUET-835: Read Arrow columns in parallel with thread pool

Also implements PARQUET-836, but need to add a unit test for that

In general this seems fine but once we would add S3 support to Arrow IO, this would definitely need to be reworked. The I/O is hidden inside of this code and would need to be more explicit to be tunable for the underlying storage access cost model.
Not saying that parallel I/O doesn't work with object stores (it actually can be beneficial) but it behaves differently than to an SSD where you can seek randomly.

We should look at Impala's S3 support for inspiration (they spent a long time designing / building it as I recall).

The only thing I found in Impala's source code was that they actually redirect to HDFS (read the s3a implementation in Hadoop) in be/src/runtime/hdfs-fs-cache.{h,cc} and don't actually implement it in native code.

Thanks. I'll add a unit test for PARQUET-836 in a follow up patch


[BUG] Exception when there are many combinations

Describe the bug
When selecting the "Combinations" option in the Job data pool section, the combinations are generated instantly, opposed to dynamically as they are supposed to.
To Reproduce
Steps to reproduce the behavior:

Create a Job
Go to the Data Pool Section
Select "Combinations"
Set a combination that has more than euler's number (1.32131....E +) combinaitons
Save the Job

Expected behavior
The combinations are created dynamically.
However, the conbinations are all generated at once, causing OB2 to crash.

Can you please describe what you mean by "crash"? Do you get the red bar at the bottom of the page telling you to reload? If that's the case, can you please press F12 and send me the content of the exception you see in red in the javascript console? Thanks!

is it the blazor-error-ui?

[2021-03-16T15:14:13.846Z] Error: System.NullReferenceException: Object reference not set to an instance of an object.
at OpenBullet2.Shared.DataPoolSelector.b__0_28(RenderTreeBuilder __builder3) in D:\a\OpenBullet2\OpenBullet2\OpenBullet2\Shared\DataPoolSelector.razor:line 85
at Microsoft.AspNetCore.Components.Forms.InputSelect`1.BuildRenderTree(RenderTreeBuilder builder)
at Microsoft.AspNetCore.Components.Rendering.ComponentState.RenderIntoBatch(RenderBatchBuilder batchBuilder, RenderFragment renderFragment)
at Microsoft.AspNetCore.Components.RenderTree.Renderer.RenderInExistingBatch(RenderQueueEntry renderQueueEntry)
at Microsoft.AspNetCore.Components.RenderTree.Renderer.ProcessRenderQueue()

@openbullet

What you sent is related to wordlist types, not this bug.
Anyways, I fixed the error with huge combinations in https://github.com/openbullet/OpenBullet2/commit/105a4d96e0a96738ad74721c158b420ca9b0e702 (tested and works)


Change Site Region Field 'Name' to Dropdown

Describe the solution you'd like
Change the 'name' and 'short name' field into a dropdown since there are only 17 regions in the Philippines.
Drop Database Column 'short_name' from table 'regions'.
Describe alternatives you've considered
String input from users is considered but prone to a lot of validation errors.
Is your feature request related to a problem? Please describe.
String input from users are prone to error.

Check ChangeRegionNameToDropdown branch at https://github.com/radical-doubting/anikultura-backend/tree/ChangeRegionNameToDropdown where I implemented

Orchid Listener
Apply Scope

Issues Found:

Solution worked but the dropdown field refreshes once item is selected.
When listening to a specific field, public $target only accepts field in 'fieldname' format, but not 'fieldname.fieldid' format.

The short_name field in the regions table is used for tags metrics because they are shorter and easier to read. They should not be removed.
A proposed simple fix is to focus on municipality and city edit/create screen instead. Do not allow the user to pick a region separately for a municipality and city. The region value should be derived from the province dropdown instead. For example, this will prevent a city from Bulacan province belonging to Mindanao region.


docker host network mode is not working on MAC with docker version 1.12.0

BUG REPORT INFORMATION
Output of docker version:
Client:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 21:04:48 2016
OS/Arch:      darwin/amd64
Experimental: true
Server:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 21:04:48 2016
OS/Arch:      linux/amd64
Experimental: true
Output of docker info:
Containers: 22
Running: 4
Paused: 0
Stopped: 18
Images: 128
Server Version: 1.12.0
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 189
Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: host overlay null bridge
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.15-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 1.954 GiB
Name: moby
ID: NP7Q:5LKP:5IVQ:GIBS:PSP5:E2P6:HMYT:RKYE:W4LX:3K6B:AFM5:QPCJ
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
File Descriptors: 38
Goroutines: 76
System Time: 2016-08-04T09:33:39.027956419Z
EventsListeners: 2
Username: darwink
Registry: https://index.docker.io/v1/
Experimental: true
Steps to reproduce the issue:
Use docker-compose to run below docker-compose.yml:
zookeeper:
image: jplock/zookeeper:3.4.8
net: host
Describe the results you received:
MAC WIFI IP: <IP_ADDRESS>, but I cannot access default zookeeper port 2181 from <IP_ADDRESS>:2181 by zookeeper client.
Describe the results you expected:
It should be accessible from <IP_ADDRESS>:2181 since it's in host network mode.
I login the container and run ifconfig -a, but the result is totally different from MAC result with same command.

Thanks for reporting; since this issue is specific for docker for mac, could you open it in the docker for mac issue tracker? https://github.com/docker/for-mac/issues. Possibly this is the same issue as https://github.com/docker/for-mac/issues/68?


Excalidraw package is difficult to embed in a scrollable page

I wanted to use the embedded @excalidraw/excalidraw package but ran into a number of difficulties. Its built-in CSS seems to assume that it's the only thing on the page — for example, it uses position: fixed which means I can't embed it in a scrolling page.
With code like this:
export default function Example() {
  return (
    <>

Editor 1

        <Excalidraw width={500} height={500} />

Editor 2

        <Excalidraw width={500} height={500} />

    </>
  );
}

I would expect to see a header, a 500x500 editor, another header, and another 500x500 editor. Instead I'm seeing this:

As you can see, containment isn't respected.
I can try to override .excalidraw to have { position: static !important } and give position: relative to the wrapper.
This provides a decent rendering (which doesn't respect the width, though):

However, there is another problem — all of the hit testing is wrong because it uses global coordinates. So as soon as I scroll the page, none of the interactions work correctly.
My next thought was to try rendering it inside an  using something like react-frame-component to fix the hit testing. However, that caused other problems. For example, it appears that CSS is injected at runtime as


Fix prometheus_rule_group_last_evaluation_timestamp_seconds

It should be a unix timestamp, not the seconds in the minute.
Signed-off-by: Brian Brazil<EMAIL_ADDRESS>

I have created #5186 to fix.


Update the documentation for build process.

Describe the bug
Creating this issue to track the documentation changes w.r.t build process.
Old Docs:
https://argocd-operator.readthedocs.io/en/latest/contribute/development/#build
https://github.com/argoproj-labs/argocd-operator/blob/master/docs/contribute/development.md
Expected behavior
Update the new documentation where build process is made easy by the operator-sdk upgrade.

This issue is partially fixed by #451.
we need to update the below doc.
https://argocd-operator.readthedocs.io/en/latest/contribute/development/#build

@iam-veeramalla It seems to me like all of the docs have been updated. Is there anything else that you think is still missing?

@iam-veeramalla It seems to me like all of the docs have been updated. Is there anything else that you think is still missing?

This can be closed. Thanks @tylerauerbeck


When updating the logo from the channel settings, the change is not reflected on the shop as the new logo.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
When updating the logo from the channel settings, the change is not reflected on the shop as the new logo.
Preconditions

Admin should be logged in.

Steps to reproduce

Go to 'Admin panel'
Click on 'Edit channel' inside the Settings/Channel
Add logo and save it
Go to the Shop
See the error

Expected Result
It should be shown the newly added logo.
Actual Result
Still shows the default logo. Admin panel Shop
Screenshots
No response
Additional context
No response

try this @amit-webkul   it fixed a similar bug for me caused by me changing the app name in .env
cd public
unlink storage

then
cd ..
php artisan storage:link

credit:

Hi @zackAJ, I appreciate your suggestion, but in this case, the issue doesn't seem to be related to the storage link. It appears to be a different matter.

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


SPARKC-707: Remove references to DataStax Academy

Description
Removes two references to DS Academy courses and replaces them with links to the same courses on YouTube.
How did the Spark Cassandra Connector Work or Not Work Before this Patch
Two markdown files contained references to DataStax Academy courses that no longer exist at the referenced URLs.
General Design of the patch
I replaced the DS Academy course URLs with links to the same courses on YouTube
Fixes: SPARKC-707
How Has This Been Tested?
Links in rendered markdown successfully open the correct video playlists in YouTube.
Checklist:

[x] I have a ticket in the OSS JIRA
[x] I have performed a self-review of my own code
[x] Locally all tests pass (make sure tests fail without your patch)

Merging as is. Thank you @eric-schneider !

@jtgrabowski, sorry I didn't have time to go back and follow the correct merge path. I will make sure to do that next time!

Also, @jtgrabowski , do you have permissions to close https://datastax-oss.atlassian.net/browse/SPARKC-707?

Sure, closed.