Fix: Add missing presentation style to AlertProcedure initializer

The initializer did not take the presentation style as a parameter as hinted in the deprecation message, but instead always used alert. I've added it and made sure naming matched the deprecation message for the old initializer.

Hi @enhorn - we did have a discussion about this last year... See: https://github.com/ProcedureKit/ProcedureKit/pull/882#pullrequestreview-149505728
The issue, is that if you use .action on an iPad, iOS will attempt to present the alert in a pop-over, and it ultimately causes a crash. I appreciate that it might work for you in some cases.
I need to have a think about how we can resolve this issue. Thanks so much for contributing though - I'm very much aware that UIProcedure needs some TLC...


1.0 binding syntax (latest reference)

This is a reference issue that holds the latest binding syntax proposal. The syntax may still be in discussion and not implemented in any released version. This issue is locked to prevent unrelated noise - for discussion/feedback please use this thread.

Quick Reference
If this is your first time here, read the Detailed Changes below first.

<button v-on:click="onClick"></button>

<input v-on:keyup.enter="handleEnter">

<input
  @click="onClick"
  @keyup.enter="handleEnter">

<my-comp
  v-bind:prop="defaultOneWay"
  v-bind:prop.sync="twoWay"
  v-bind:prop.once="oneTime">
</my-comp>

<item-list
  :items="items"
  :mode="active"
  @ready="onItemsReady"
  @update="onItemsUpdate">
</item-list>

<comp v-ref:child></comp>

Detailed Changes from 0.12

Directive Arguments

The concept of multiple clauses (multiple directives separated by comma in the same attribute) is deprecated, and directive arguments are moved into the attribute name:

Using real directives as example:

Literal Directives

There is no longer "literal directives" from the implementation perspective. All directives are reactive by default, which makes it easy to know whether an attribute value is an expression or a literal string. If you wish to pass the directive a literal string, use the following syntax:

The ending .literal is called a Binding Modifier, which forces the directive to be bound in literal mode. We will see this concept used below for prop binding types as well. In literal mode, the directive's update function will be called once, with the literal string as the argument.

Attribute Bindings

Mustache tags will no longer appear inside attribute values. They will only be used for text/HTML interpolation in 1.0. To dynamically bind an attribute, use the v-bind directive (which replaces v-attr):

This also applies to special attributes like is for dynamic component and name for dynamic partials:
<component v-bind:is="view"></component>
<partial v-bind:name="partialName"></partial>

In particular, bindings for class and style have some dedicated enhancements. You can pass in JavaScript Object or Array literals:

Props

Previously props use mustaches to indicate reactivity. Now they also use v-bind:

<my-comp
  prop="a literal string"
  prop="{{expression}}">
<my-comp>

<my-comp
  prop="a literal string"
  v-bind:prop="expression">
</my-comp>

Binding type indicators (@ and *) are now replaced by more explicit binding modifiers:

<my-comp
  prop="{{defaultOneWay}}"
  prop="{{@twoWay}}"
  prop="{{*oneTime}}">
</my-comp>

<my-comp
  v-bind:prop="defaultOneWay"
  v-bind:prop.sync="twoWay"
  v-bind:prop.once="oneTime">
</my-comp>

Shorthands

You may have noticed we will be using v-bind and v-on quite a lot. 1.0 will provide optional shorthand syntax for these two directives. v-bind: can be shortened to a single colon :, while v-on: can be shortened to a single @ symbol:

<input @click="handleClick" @keyup="handleKeyup">

<my-comp
  :prop="expression"
  :prop.sync="twoWay"
  :prop.once="oneTime">
</my-comp>

<component :is="view"></component>
<partial :name="partialName"></partial>

If you are only using Vue as an enhancement on existing HTML pages, you may want to stick with the v- prefixed versions. The shorthand is designed to make the template more succinct when you are building large SPAs where Vue manages everything. Don't worry about it not looking like valid HTML - all browsers can parse it just fine, and Vue removes all the special stuff in the rendered HTML anyway.

v-on on Child Component

v-on will also listen to custom Vue events (emitted via vm.$emit) when it is used on a child component. The idea is that props and events should constitute the public API of a component:

props for passing data down;
events for handling actions up.

A typical component would look like this:
<item-list
  v-bind:items="items"
  v-bind:mode="active"
  v-on:ready="onItemsReady"
  v-on:update="onItemsUpdate">
</item-list>

With shorthand:
<item-list
  :items="items"
  :mode="active"
  @ready="onItemsReady"
  @update="onItemsUpdate">
</item-list>

v-on Key Modifier

v-on can now leverage a Key Modifier that replaces the old key filter (you can use all the old aliases usable in the key filter, or use a direct keyCode):

<input v-on="
  keyup: handleEnter | key 'enter',
  keyup: handleEsc | key 'esc'">

<input
  @keyup.enter="handleEnter"
  @keyup.esc="handleEsc">

New Syntax for v-el and v-ref

v-el and v-ref were previously "literal" directives, but you just need to give them an argument. Also, to make things more explicit, vm.$ is now vm.$refs, and vm.$$ is now vm.$els.

<comp v-ref:child></comp>

{{$refs.child.msg}}

Caveat: camelCase names are converted to all lowercase when the HTML is parsed:
<comp v-ref:someThing></comp>

Gets rendered as:
<comp v-ref:something></comp>

Thus, it is necessary to use the dash-case <-> camelCase mapping for refs too (similar to props):
<comp v-ref:some-thing></comp>
{{ $refs.someThing.msg }}

Update Sep.19th:

v-ref and v-el syntax updated according to this comment

Update Sep.19th:

Now implemented in 1.0.0-alpha.5.

Update Sep. 21st:

Now implemented in 1.0.0-beta.1.


Update Readme to align with website

We remove references to the Radicle language from the readme to avoid confusing users. For now, we’ll user the tag line from the website. If the website is live we can copy part of the guide, expand on the description, and link to the website.

added the value prop from the website straight into your branch!


[DROOLS-1059] race condition when retrieving a query node trying to create a segment from a prototype

DO NOT MERGE (for now)
@baldimir I tried to reproduce the issue reported here https://issues.jboss.org/browse/DROOLS-1059 by creating a more complex version of the reproducer attached to that jira, but with no luck. I also tried to find any possible concurrency issue in that area by statically reading the code, but I couldn't find anything. The small change I made in SegmentUtilities is just intended to simplify a piece of code that was unnecessary complex, but the old version and mine are functionally equivalent. Please take a look at my reproducer and check if you're able to modify it in order to reproduce this issue.
@mdproctor I really don't see how that exception could be possibly thrown: there's no dynamic  KieBase change involved here, so there shouldn't be any problem retrieving that QueryLiaNode regardless of the concurrency. Am I overlooking something?

Merged by https://github.com/droolsjbpm/drools/commit/b815ca37e6749bf4c028060552dea10504670951
@baldimir I investigated this issue a bit more and I'm pretty sure that I found what was causing this issue (even if I haven't been able to reproduce it). I discussed my findings with @mdproctor and he also agreed that the problem had to be there.


A bit slow

First off want to thank you for the great work,
I followed your instructions and worked like a charm.
A problem a find is that if I am using a 10 million password list, it seems a bit too slow for processing.
I have been at it for the whole day and manged to get only 0.5% of the password list.
Any ideas to speed it up?

Having the same issue. What are you running it on? (Operating System, Ram, etc.)

#09135227322


The nuance of working with the var keyword.

Why is there no mention of the nuance of working with the var keyword in any document on this topic?
https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/var
"When var is used with nullable reference types enabled, it always implies a nullable reference type even if the expression type isn't nullable."
In my opinion, this outweighs absolutely all the advantages of using nullable reference types and makes reading and refactoring a real hell.
Madness..

Сведения о документе
⚠ Не вносите правки в этот раздел. Это необходимо для связывания страницы сайта docs.microsoft.com с вопросом на GitHub.

ID: a690704d-da32-dca7-ea3e-66b84fb1d00d
Version Independent ID: 11f314de-dd3a-362e-b2bb-d0a2133cfc7b
Content: Ссылочные типы, допускающие значение null
Content Source: docs/csharp/nullable-references.md
Product: dotnet-csharp
Technology: csharp-null-safety
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

While I think it's reasonable to add this note to the article, I just want to note that the flow analysis is still smart enough and the state of the variable will be "not null" even if the type is deduced as nullable.

So in most situations, you don't need to give much attention.

I agree we should add a note on this page. Adding this to our backlog, and adding the "up for grabs" label.


support rocketMQ-3.x-plugin (from 3.0.4-open to 3.4.6.Final)

New feature or improvement

Describe the details and related test reports.
add plugin from rocketMQ-3.x , because Alibaba and many other company run online tradding with it .
I had test rocketmq-client version from 3.2.6 to 3.4.6.Final , the version below 3.2.6 is supported in theory;

add trace image : producer & consumer

@carlvine500 Thanks for your continuous contributions.
@ascrutae Please review the codes first, then guide @carlvine500 to provide our auto integration test cases.
FYI， @carlvine500 here is our test report: https://github.com/SkywalkingTest/agent-integration-test-report , all the test cases are from https://github.com/SkywalkingTest/skywalking-agent-testcases

@carlvine500 I suggest you should learn https://github.com/SkywalkingTest/skywalking-agent-testcases/tree/master/rocketMQ-scenario first, and

Rename rocketMQ-scenario to apache-rocketMQ-scenario or rocketMQ-4.x-scenario
Add rocketMQ-3.x-scenario
Learn from our 4.x test cases and finish your 3.x test cases.
@ascrutae will help you run the cases on our infra, and provide the test results and related logs in GitHub.

CC @ascrutae please provide helps.

@carlvine500 Please follow this cn document to build the test cases: https://github.com/SkywalkingTest/skywalking-agent-testcases/blob/master/docs/how-to-write-a-plugin-testcase-cn.md
After we have confirmed test reports, I will merge this pull request.

@carlvine500 Do you merge the master in right way? Please be advised my pr #822 . SkyWalking is using git submodule to link protocol and UI now.
I just check you ci logs, only see the UI links.

This is the master branch ci log: https://travis-ci.org/apache/incubator-skywalking/builds/345124146
You can find these:
1.30s$ git submodule update --init --recursive
Submodule 'apm-protocol/apm-network/src/main/proto' (https://github.com/apache/incubator-skywalking-data-collect-protocol.git) registered for path 'apm-protocol/apm-network/src/main/proto'
Submodule 'skywalking-ui' (https://github.com/apache/incubator-skywalking-ui) registered for path 'skywalking-ui'
Cloning into '/home/travis/build/apache/incubator-skywalking/apm-protocol/apm-network/src/main/proto'...
Cloning into '/home/travis/build/apache/incubator-skywalking/skywalking-ui'...
Submodule path 'apm-protocol/apm-network/src/main/proto': checked out '86bfe4f6635a8f71abca2383f54565b621ed3f94'
Submodule path 'skywalking-ui': checked out 'd4e49aa0581e510884e85cc145e647dcb179f5a1'

@carlvine500 I found a similar issue in another committer's pc. I prefer the easiest way to delete/fork, and send a new pull request.

The conflict triggers your ci fail. @carlvine500

@wu-sheng  sorry , I have revert to version<PHONE_NUMBER>2068cea613875ec87aba611f220b9e

add rocketMQ-3.x-plugin again

add rocketMQ-3.x-plugin again

all testCases passed , please review the code
(test result link)

@carlvine500 Can you leave your company info here: https://github.com/apache/incubator-skywalking/issues/443 . I remember you said you are using SkyWalking.

FYI, I will submit a high priority pr today, about javadoc, which have a chance to trigger your ci failure.
After I merge that pr, please be advised your ci status.

@wu-sheng  : Can you leave your company info here: #443
done!

@carlvine500 I sync master branch codes, and change the version number in your codes. Trying to fix ci.

@wu-sheng  the test case project about rocketMQ-3.x plugin has been merged. and this PR LGTM.

@carlvine500 After the final fix, I think this pr is ready to merge. Thank and congrats.

Merged and pr done.


after deleted by send2trash there is no "put back" or "restore" in the right click menu in trash on OSX

after deleted by send2trash command there is no "put back" or "restore" menu  in the right click menu in trash  can on OSX (10.10.x), but the files which were deleted by Command + Backspace i has that menu.
By the way it works on win7. Thanks for your works. :+1:

This used to work in older OS X versions. I think that the API changed and that the old API, used in send2trash doesn't work very well anymore.

See this Sublime Text issue for reference: https://github.com/SublimeTextIssues/Core/issues/2226
Possible solution may be implementing something like https://github.com/ali-rantakari/trash/blob/master/trash.m#L265-L340 in send2trash using ctypes.
According to https://github.com/ali-rantakari/trash#the-put-back-feature the only way to enable "Put Back" is to instead tell finder to put the file in the trash.

Going through finder is horribly slow. I'm not convinced that it's the only way. The "Put Back" feature used to work with send2trash and I'm sure that other native Cocoa/Swift apps are able to send files to trash without needing to ask Finder. We just need to call the API in the same way.

Hum, now that I look a bit more at https://github.com/ali-rantakari/trash, maybe I'm wrong... Maybe that going through finder is the only way.

My knowledge of python's ctypes isn't the best, but IIUC they don't provide a nice interface into Objective-C, so replicating the method in https://github.com/ali-rantakari/trash/blob/master/trash.m#L265-L340 might be rather difficult (at least, this SO answer makes it look cumbersome).
What do you think are our potential options for adding this functionality in send2trash?
Everything that comes to my mind (leveraging things like pyobjc, using a precompiled binary, etc) seems a little overkill just to have this feature...

Yes, that would be complicated. One alternative is to add an optional compiled module. If you look at git history (v1.x tags), you'll see that the OS X code used to be straight objc code. I went to ctypes so that the library could be installed without needing a compiler. But by adding an optional compiled module, we could provide the feature to users who bother to compile the module.

I'm gonna try this using ctypes anyway, just because it's a huge benefit to not have to compile in order have an extra feature. If it ends up being too complicated/too much of a mess, then I'll give it a crack using obj-c instead.

@hsoft in your code you have:
class FSRef(Structure):
    _fields_ = [('hidden', c_char * 80)]

I need to do the same for AppleEvent and AEDesc. Any pointers on how I can make structures for those? I don't know where you got those values from, and I'm unsure of the actual process to do it. 😅
For example, I need to make an AppleEvent variable for this:
AppleEvent replyEvent;
OSStatus sendErr = AESendMessage([descriptor aeDesc], &replyEvent, kAEWaitReply, kAEDefaultTimeout);

@acheronfail sorry, I don't have much of a clue. The only time I ever used ctypes was for this library and I kept it simple. And it was a long time ago.

AppleEvent seems to be an alias of alias of AEDesc.
By quickly reading the doc of the init() method, something more elaborate of this could be a starting point:
class AppleEvent(Structure):
    _fields_ = [('descriptorType', DescType), ('dataHandle', AEDataStorage)]

Will need to find out and declare DescType and AEDataStorage.

I think our options are:

Include manually compiled module

✅ easy to maintain
❌ require compilation to use the module

Implement using ctypes

✅ no need to compile to use feature
❌ messy, difficult to debug/maintain

Optionally depend on PyObjC

✅ easier to use than ctypes
❌ requires a dependency

Something similar happens on Ubuntu 16.04, when trying to restore I get:
Could not determine original location of “01.jpg
The item cannot be restored from trash
Also, the "restore" option is missing from the context menu.

I am wondering if this bug ever got fixed? The issue still persists today

I had looked into alternative ways to see what might be possible on Mac OS, none of the options I tried allowed the "put back" when in Finder.

I actually found a way to do it outside of the library which works quite well. It requires the user to click ok for security preferences otherwise it works great :)
cmd = ['osascript', '-e', 'tell app "Finder" to move {' + ', '.join(files) + '} to trash']
subprocess.run(cmd, stdout=open(os.devnull, 'w+'))


Change getting started with azure blob service in java from track 1 to track2

Change getting started with azure blob service in java from track 1 to track2
Changed file:
1). pom.xml
2). src/BlobAdvanced.java
3). src/BlobBasics.java
4). src/BlobClientProvider.java
5). src/PrintHelper.java

@jongio for notification.

@joshfree

@jaschrep-msft , @gapra-msft We have updated by your comments, please re-review this PR.


Dynamic Programming: Given a rows x cols binary matrix filled with 0's and 1's, find the largest rectangle containing only 1's and return its area in C++

Hi,
Please assign this to me.


Split solvers into 2 classes

Refactored GOSolver -> BNBSolver and EnumerateSolver with no additional changes
Reason is to prepare for the algorithm to spend most of its time in SplitWorker rather than ComputeWorker, which may warrant changes to message structure.

Not doing this


Move Catalog and Information to package object

Let's move Catalog and Information to package object.

Closed by 972b36b.


Filter on skills on homepage

This commit allows the filtering of skills by pressing the skill name, either on the buttons at the bottom or on the named skill on the card.
Signed-off-by: Alistair Hey<EMAIL_ADDRESS>

Also, removed the duplicate links (the one called livePreview? as they both go to the same place

Just added a new commit as the link at the top left of the header needs to be a full url or it appends it as a path (takes you to https://devsecopstutorials.com/devsecopstutorials.com/ at the moment)

A third commit to use Title Case for the technologies, this should avoid any duplication if someone defines "Kubernetes" and "kubernetes"


Redirect to homepage when unauthenticated (Captive Portal)

Hi.
I've configured authentication to expire after 24 hours for security reasons. I though this will make the Wireguard tunnel disconnect when the session expires, but it doesn't and then the users forget that they need to reauthenticate on the website (which also doesn't logoff the user for some reason).
I understand that all this can be solved using clients, but that's not something offered at the moment and I'm not asking for it.
My question is, is it theoretically possible to detect that the session is expired, and redirect the user to the firezone web UI to reauthenticate? Of course I'm talking about use cases where the initial request is HTTP/s based.

Hi @ilyatovbin-pp -- thanks for the suggestion. We have a backlog issue for captive portal functionality, but haven't explored exactly how we'd implement that for Firezone. It likely involves adding a DNS server (like dnsmasq) to Firezone in order to issue the DNS redirects.
I'll keep this open to track work on that.
refs firezone/product#296
refs firezone/product#297

@ilyatovbin-pp Spent some time investigating how feasible this is, and whether we can trigger the OS captive portal detection dialog when a WireGuard tunnel is established.
tl;dr: We can redirect plain HTTP requests to the Firezone portal when a user's session expires, but looks like it won't be possible to trigger the OS-native captive portal dialog for reasons explained below.

The primary way the OS (chrome, mac, ios, windows, etc) knows its in a captive portal is by sending an HTTP probe to a known URL (differs by OS, hardcoded by the vendor) and expecting to receive a certain response. The captive portal system is expected to intercept this probe and redirect to a login screen. When the OS detects this redirected response from their OS probe, it displays the captive portal dialog, prompting the user to sign in (it simply loads the redirect in a web view).
Now, this won't work for WireGuard tunnels because this probe is not sent when the WireGuard client connects. It seems to only happen (at least on macOS) when the WiFi network is changed. In short, there doesn't seem to be a way for the HTTP probe to be sent through the WireGuard tunnel for Firezone to intercept and redirect. The HTTP probe is always sent outside the tunnel.
The other way captive portals work is by detecting DNS hijacking. The thought here is we would override the WireGuard config's DNS to point to a local DNS server bundled with Firezone. When the user's session expires, we override all DNS lookups to be the Firezone's tunnel portal IP. Apparently ChromeOS detects this and enters the captive state, displaying a dialog. Unsure if macOS or other vendors do.
There is a new RFC to standardize this that vendors are now recommending, but it involves DHCP replies which doesn't apply to WireGuard networks (tunnel IPs are set by WireGuard, not configured by DHCP).

I see. Looks like it would be much easier to implement a probe sent from a Firezone client/daemon/service (when one is available), and send some sort of notification to the user, prompting them to re-authenticate. I'd put this on ice for now since it's not a terribly important feature. Thanks for checking this.


download as files (or problem with binaries)

Ran into a problem uploading a tar.gz, in that it downloads as f.txt this will fix that, but creates the problem that everything is then downloaded instead. Not sure if there's a better way to do it.
 Resource resource = resourceLoader.getResource( root.resolve( filename ).toUri().toString() );
            return ResponseEntity.ok()
                    .header( HttpHeaders.CONTENT_DISPOSITION, root.resolve( filename ).toFile().getName() )
                    .body( resource );

Fixed with #30


Set Salamander Max Pop to 80

About the PR
Sets Salamander's max population to 80.
(Or rather, just removes it's set population config so it defaults to the Wizard's Den default like how the other servers are set up.)
Why / Balance
There was a consensus at some point that the game just doesn't function at higher populations than 80, alongside admin sanity being heavily degraded.
All the other servers were adjusted to fit this consensus, but Salamander was left out due to a brief oversight I believe and no one ever bothered to set it properly since it wasn't reaching 80 for the longest while.
I don't think the majority of the playerbase genuinely enjoy this either, but I suppose I haven't asked about.
There's also a comment to be made about how higher population generally leads to lower roleplay, but that's not the focus here.
Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] This PR does not require an ingame showcase.

Changelog
Doesn't fit here, don't think.

Mander is hitting high pop numbers, sometimes even triple digits easily. This is a step in the right direction for sure but I think this just shows the market for a second MRP server.

Mander is hitting high pop numbers, sometimes even triple digits easily. This is a step in the right direction for sure but I think this just shows the market for a second MRP server.

Possibly make miros or vulture MRP?

Possibly make miros or vulture MRP?

I mean, Vulture has a pretty dedicated community and vibe I'd hate to trample on. Also Miros is Miros. A few server were killed previously, we should resurrect one or make a fully new East Coast or EU (pls east coast) MRP server. I'd even volunteer for admining sometimes if it meant getting it (hopefully do not regret saying THAT)

I'd love EU MRP

MRP definitely does not function above 80. While this does touch some problems on Salamander, it won't solve all of it, although it's a first step

bring back spider :trollface:

im waiting until centipede gets revived

im waiting until centipede gets revived

there is no centipede in ba sing se

I'd love EU MRP

Or at least US East MRP.

I'd love EU MRP

Or at least US East MRP.

I say this as a us-easterner I think eu deserves the 2nd MRP server

I say this as a us-easterner I think eu deserves the 2nd MRP server

I mean, it makes me wonder how much of mander is oceania, west coast usa, east coast usa and europa. Because its the ONLY MRP server so everyone is funneled there. Eitherway I just want a second MRP server man, even with a cap if mander is constantly at that cap it just makes me :(

Chat, Merge this please.

Chat, Merge this please.

I think the most important thing to think about in relation to this image is that I can't help but think that Sal only has this many people because all the other servers are max pop as well.
It's nice to know there is a server that will almost always have room for more players while Levi, Liz, Vulture, and Grasshopper are max pop.  This is truly a SS14 suffering from success moment

Chat, Merge this please.

I think the most important thing to think about in relation to this image is that I can't help but think that Sal only has this many people because all the other servers are max pop as well.
It's nice to know there is a server that will almost always have room for more players while Levi, Liz, Vulture, and Grasshopper are max pop. This is truly a SS14 suffering from success moment

The server was super laggy and chugging hard. Stop motion.

Chat, Merge this please.

I think the most important thing to think about in relation to this image is that I can't help but think that Sal only has this many people because all the other servers are max pop as well.
It's nice to know there is a server that will almost always have room for more players while Levi, Liz, Vulture, and Grasshopper are max pop. This is truly a SS14 suffering from success moment

The server was super laggy and chugging hard. Stop motion.

The server went from running at 30 tick rate to around 10-15. Moving around even as a ghost was incredibly incredibly bad, just imagine how the players felt...


Change plugins "reload notification" description

There is no "Reload Now" menu item. Changed to the "Update All Now"

https://github.com/zeit/hyperterm/pull/111

Oops, my bad


2024-10-08 agenda

shielded client side validation: https://github.com/ShieldedCSV/ShieldedCSV, repo has link to paper

https://x.com/2ndbtc/status/1837530708560302522 - first mainnet ark transaction

v28 rpc breakage: https://github.com/bitcoin/bitcoin/issues/31039
btcd disclousure: https://delvingbitcoin.org/t/non-disclosure-of-a-consensus-bug-in-btcd/1177

first "0-fee" lighting commitment transaction: https://mempool.space/testnet4/tx/85f2256c8d6d61498c074d53912d1f0ef907ee508bb06f5701f3826432ba53b8

neutrino vs SPV privacy comparison: https://eprint.iacr.org/2024/1415.pdf

headers sync disclosure: https://bitcoincore.org/en/2024/09/18/disclose-headers-oom/
https://delvingbitcoin.org/t/hybrid-jamming-mitigation-results-and-updates/1147 - channel jamming
https://delvingbitcoin.org/t/superscalar-laddered-timeout-tree-structured-decker-wattenhofer-factories/1143/11
https://github.com/bitcoin/bitcoin/pull/28358/files

FROST roundtable: https://developer.blockchaincommons.com/frost/meeting2/

https://github.com/bitcoin/bitcoin/pull/30807


Error: Member not found: 'UnicodeChar'.

After Upgrading i keep getting this error
: Error: Member not found: 'UnicodeChar'.
  int get UnicodeChar => Char.UnicodeChar;
                              ^^^^^^^^^^^
: Error: Setter not found: 'UnicodeChar'.
  set UnicodeChar(int value) => Char.UnicodeChar = value;

                                     ^^^^^^^^^^^
: Error: Member not found: 'AsciiChar'.
  int get AsciiChar => Char.AsciiChar;
                            ^^^^^^^^^
: Error: Setter not found: 'AsciiChar'.
  set AsciiChar(int value) => Char.AsciiChar = value;

                                   ^^^^^^^^^

Flutter doctor
[✓] Flutter (Channel stable, 3.0.0, on macOS 12.1 21C52 darwin-arm, locale
    en-NG)
[✓] Android toolchain - develop for Android devices (Android SDK version
    32.1.0-rc1)
[✓] Xcode - develop for iOS and macOS (Xcode 13.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.66.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

• No issues found!

error posted on stackoverflow
https://stackoverflow.com/questions/72233476/flutter-error-member-not-found-unicodechar

Duplicate of https://github.com/flutter/flutter/issues/103674


Share type definitions between beacon node and validator api client

PR Description
First stab at sharing a JSON type definition and supporting data class between the beacon REST API and the validator api client.
Put it in the :ethereum:json-types module which seems pretty reasonable, though it could be a module level conceptually between ethereum and beacon/validator but apart from calling it "shared" or "common" I'm not really sure how to name it which suggests it may not need to exist.
The main complication here is that we don't have an internal data structure that can hold the data required for the get genesis API response so a custom wrapper class has to be defined. In this version the wrapper class is also in :ethereum:json-types and the type definition is a static field on that class. It works quite neatly for this case but there's a risk that the :ethereum:json-types module will get used as a dumping ground for types that are actually used internally and not just in the edges of the REST API and validator API client. There is some resistance to that problem though as :ethereum:json-types isn't a widely used dependency so when I initially tried to use the new class more widely I found a lot of compiler errors because it wasn't available. The temptation for overreach is strong however...
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

Looks pretty good... It'd potentially be useful for anything in validatorRequired to maybe add SSZ type while we're going... just a thought... could pretty easily add them to beacon-apis too if we wanted...

I'm pretty strongly against adding SSZ for non-spec stuff without a really good reason for it. It adds a heap of overhead in terms of code and winds up generating a lot more garbage if we use the current remerkalisable stuff for such short lived objects.

I moved the wrapper class into a separate package with some docs to try and dissuade people from using it as a place for actual business logic classes. Also added Api into the name as further discouragement of general use.
Went back to the more normal patter of the type definition being in a *Types class rather than in the object itself. Otherwise it encourages people to put type definitions on our internal model classes which we don't want either.
I think this is a reasonable approach at this point and we'll see how it goes.

To the best of my knowledge ValidatorRequired is pretty much all ssz objects which was what I was thinking...
Some exceptions would be syncing, config/spec, and the event steram... but Ionly really was thinking about the SSZ objects we have that could support it..

hmm, possibly could review SSZ though I think it increases the scope of work when we already have quite a bit of JsonProvider to try and get rid of.


post card padding  update; closes #31

Needs to be merged to the master, so after main merge from the dev, should this be merged

wrong destination


Fix for JEA session leaking functions

PR Summary
This PR corrects an issue with multiple modules loaded in a session and having functions with the same name. Previously all functions with that name become public even if only one module makes the function public.
PR Context
PowerShell's function list does not distinguish between session functions and module functions, and also does not allow duplicate function names.  This caused some private module functions to be made publicly accessible.  The fix is to update module function accessibility code to account for potential duplicate function names.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Test failure unrelated to change:
https://dev.azure.com/powershell/PowerShell/_build/results?buildId=120420&view=ms.vss-test-web.build-test-results-tab&runId=2043857&resultId=100001&paneView=debug

Test failure unrelated to change:
https://dev.azure.com/powershell/PowerShell/_build/results?buildId=120420&view=ms.vss-test-web.build-test-results-tab&runId=2043857&resultId=100001&paneView=debug


Remove useAccountBalance from CLI implementation.

Description
Mirrors the changes present in: PR #788
Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Your ENS/address: JacobHomanics.eth

Hey @Hotmanics
we create ~weekly back merge PRs to the CLI branch (check any of the merged ones from past weeks). This way we can batch-merge all the changes made to main and we don't have to manually do it...which can lead to some manual errors + need to review again, e.g. you missed the newline at the end of a couple of files here.

More info about this workflow here: https://github.com/scaffold-eth/scaffold-eth-2/blob/cli/contributors/DEVELOPER-GUIDE.md#back-merging-main-branch--publishing-to-npm
Thanks!


Queue Line error from #55 still occurring

It looks like while you created v1.0.2 you never published it, so the error from #55 still occurs on the most recent version that is possible to download.

I found what fixed this in the latest version of the repo, but I am not smart enough to know how to push this to releases.
To fix it yourself, open benchwarmer.js with the text editor of your choice (VS Code, Notepad++, etc.) and head down to line 333. You should see the following on lines 333–337:
var _b = args, x = _b.x, y = _b.y, z = _b.z, slope = _b.slope;
var addition = settings.bin;
if (args.object == 11) {
    addition = settings.queuetv;
}

Replace lines 333–337 with the following:
var _b = args, x = _b.x, y = _b.y, z = _b.z, slope = _b.slope, constructFlags = _b.constructFlags;
var addition = settings.bin;
if (constructFlags === 1) {
    addition = settings.queuetv;
}

If you haven't already, follow the steps in issue https://github.com/tubbo/openrct2-benchwarmer/issues/76#issuecomment-1793735455 to stop lamp posts being placed instead of bins/benches.

v1.1.0 should release properly today, some of the CI was broken so it never made it to the release stage.

https://github.com/tubbo/openrct2-benchwarmer/releases/tag/v1.1.0


Site Showcase Request

Minecraft server website made with MDL.
http://randomcraft.ru/

Even though you are using components of mdl, this website doesnt have a feel of Material Design.
Caught a bug at first look too :
Tab border misaligned

Updated.

@wasder  Good job, and thanks for your efforts
But I think, it does not look like Material design. You can take a look at https://material.google.com/


generalize traversing message types for REST query parameters

For Requests that have query parameters nested in messages, we need to synthesize qualified names for those parameters and include them in the URL.
for this example, the URL should be something like:
"https://bigquery.googleapis.com/bigquery/v2/projects/<project id>/queries/<job_id>?maxResults=10&formatOptions.useInt6stamp=false"

// Options for data format adjustments.
message DataFormatOptions {
  // Optional. Output timestamp as usec int64. Default is false.
  bool use_int64_timestamp = 1 [(google.api.field_behavior) = OPTIONAL];
}

// Request object of GetQueryResults.
message GetQueryResultsRequest {
  // Required. Project ID of the query job.
  string project_id = 1 [(google.api.field_behavior) = REQUIRED];

  // Required. Job ID of the query job.
  string job_id = 2 [(google.api.field_behavior) = REQUIRED];

  // Zero-based index of the starting row.
  google.protobuf.UInt64Value start_index = 3;

  // Page token, returned by a previous call, to request the next page of
  // results.
  string page_token = 4;

  // Maximum number of results to read.
  google.protobuf.UInt32Value max_results = 5;

  // Optional: Specifies the maximum amount of time, in milliseconds, that the
  // client is willing to wait for the query to complete. By default, this limit
  // is 10 seconds (10,000 milliseconds). If the query is complete, the
  // jobComplete field in the response is true. If the query has not yet
  // completed, jobComplete is false.
  //
  // You can request a longer timeout period in the timeoutMs field.  However,
  // the call is not guaranteed to wait for the specified timeout; it typically
  // returns after around 200 seconds (200,000 milliseconds), even if the query
  // is not complete.
  //
  // If jobComplete is false, you can continue to wait for the query to complete
  // by calling the getQueryResults method until the jobComplete field in the
  // getQueryResults response is true.
  google.protobuf.UInt32Value timeout_ms = 6;

  // The geographic location of the job. You must specify the location to run
  // the job for the following scenarios:
  //
  // * If the location to run a job is not in the `us` or
  //   the `eu` multi-regional location
  // * If the job's location is in a single region (for example,
  // `us-central1`)
  //
  // For more information, see
  // https://cloud.google.com/bigquery/docs/locations#specifying_your_location.
  string location = 7;

  // Optional. Output format adjustments.
  DataFormatOptions format_options = 8 [(google.api.field_behavior) = OPTIONAL];
}

Just to checkpoint progress before I go on a mini-vacation...
I have generator changes in a branch: https://github.com/dbolduc/google-cloud-cpp/tree/generator-rest-query-params-dev. It also addresses #10176.
The code looks right to my eye, but I haven't written tests. Golden unit tests are where we need to do the bulk of the testing. Existing integration tests should at least give us confidence things are no worse than before. They won't verify the new functionality though.
The PR is too big already. I will want to break it down:

[ ] No need for query parameters when body = "*"
[ ] Factor out the params argument to the rest_internal::Verb(...). This will be the majority of the line changes, although the PR should be trivial.
[ ] Factor out the accessor value adapter
[ ] Probably refactor to set up the algorithm witqh messages, repeated disabled. I will need to remember to leave in the Protobuf value processing, so there is not a regression.
[ ] Turn on messages and repeated fields


[DOCS] Update docs pages

This PR updates the documentation pages and structure.
There are also verified examples for Panel, Bokeh, and Gradio.

These look great @pavithraes! One thing I might suggest (up to you) is to maybe add the framework logo to the top of each doc page? There isn't a lot of color on these pages and that might liven it up and also provide some brand recognition to readers.

One thing I might suggest (up to you) is to maybe add the framework logo to the top of each doc page? There isn't a lot of color on these pages and that might liven it up and also provide some brand recognition to readers.

Regarding my suggestion above, I'm happy to leave this off for a future iteration so we can go ahead and get this merged. It will greatly help users to get these docs published!

@pavithraes I went ahead and pushed my changes since they were minor and I didn't think you'd mind. Do you want to take one final look?

@kcpevey Thanks for the updates, this looks good tome :)


Contract Byte Code Error

I create a contract first. Then I write the byte code that I want to create a contract for ico. Mew doesn't charge me gas. In order to send tokens, an ico contract must be created, right? I do not understand the code I say I have a problem I am doing a due diligence. ico contract doesn't occur so I can send tokens there.
If you look at the photo I sent, I put your current code without changing, but it did not give me the gas fee. I created a normal contract. But I can't create a contract for sale. I'il send a token here if I can.
image hosting
Originally posted by @iegestesi in https://github.com/halls7588/ERC20-Token-and-ICO-Contract/issues/2#issuecomment-537882136

This is an error with MEW of whatever compiler you are using. Please ensure that you are using the correct version of Solidity and that you are using the EVM code and Byte Code appropriately.

I'm doing the same version of what the pragma version says. I'm pasting that byte code. Mew also don't know how it could be a mistake.


PR: [1pt] Dev fim3 collate catchment attributes, adding better code for write csv.

Made changes to collate_catchment_attributes.py to improve performance.
To run code use command:
python collate_catchment_attributes.py -d "/data/previous_fim/fim_3_0_24_14_ms/"  -s "/data/tools/sierra_test/official_versions/fim_3_0_24_14_ms/usgs_gages_stats.gpkg" -o "/data/temp/caleb/collate_401_test.csv" -on "/data/temp/caleb/collate_401_nlcd_test.csv" -l "/data/tools/sierra_test/official_versions/fim_3_0_24_14_ms/agg_usgs_elev_table.csv" -lc "/data/temp/nlcd_pixel_counts_fim_3_0_26_0/"
Additions
-Added a sleep(60) command to line 157 to ensure csv write.
Removals
-Removed hard file reference
##Notes
-Let's not close the branch with this pull request. There is another file pending upload that will use this branch.

@CalebOliven-NOAA Please add the "PR: [1pt]" to the beginning of the title, per the project standard. You can reference other PRs for an example.

@CalebOliven-NOAA It looks like this feature branch is behind dev-fim3 by a couple commits. Also, the Changelog has two entries of <IP_ADDRESS>


Addition/Deletion Injection Fixes

Hi @brendt, this is a continuation of fixes presented in #104 where addition/deletion is not able to get the line endings correctly in my use case scenario, similar to #115 on the gutters it cannot read the current line thus applying it always on the first line.
What I did was standardize the line endings for EOL so it can be more flexible when handling line endings.
$content = preg_replace("/(\r\n|\n|\r)/", PHP_EOL, $content);
While I was debugging this I've notice that the InlineTheme is not getting the IgnoreTokenType for Addition/Deletion endings, being unable to replace them with '{-' and '-}'. So I've fixed it by returning the class when it's not found in the theme file instead of returning an empty span.

Also if you could explain the use of IgnoreTokenType, disabling it even solves this InlineTheme problem and in all my testing I could not find any scenario that it's helping in any way, results being the same with or without. Probably it has a meaning, I was just wondering if it's really necessary or if we could find another solution.
Additional to the last PR, I've removed the spacing after <span class="hl-gutter "> if it does not have another class.
P.S. Sorry for the unit-tests I always fucked them 😳

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

4 of 5 (80.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.009%) to 95.294%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Themes/InlineTheme.php
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
1458

Relevant Lines:
1530

💛  - Coveralls

Looks good! To answer your question:

Also if you could explain the use of IgnoreTokenType

They used to be so that there aren't any collisions between blade comments {{-- and ignore tags {{-.
However, it's more than possible that this now also works without these tokens, because of other changes that have been made. I'll look into this separately, let's keep this PR focussed on one thing :)
I'm fine merging this in, but could you please explain that one question about the regex differences?

Indeed, using '/\R/' is more comprehensive and handles all cases in one go including unicode. Also being inlined with #115

Tagged: https://github.com/tempestphp/highlight/releases/tag/2.4.5


Add support for configurable email template

Lipstick::EmailMessage now takes an optional template parameter to the constructor. The passed argument should be an ERB template in the same manner as the provided default. (That is, the argument contains the actual template, not a file path – this will help if we ever need to source email templates from a database etc.)

Worth documenting extra param in README under Email Template header?

Damn, GitHub properly shows that commit landing after your comment, so I can't even pretend I already did it.


array with non-writable length

When I use the setTranslation method in my project, I get the following error:
ERROR Error: Uncaught (in promise): TypeError: can't define array index property past the end of an array with non-writable length
addLangs/<@http://localhost:4200/vendor.js:145411:20
addLangs@http://localhost:4200/vendor.js:145409:11

pull request: #1371


ERROR Please fix your errors before loading the bot.

$ node .
[21:51:01:150]: LOG =-=-=-=-=-=-=- Config file Verification -=-=-=-=-=-=-=
[21:51:01:150]: LOG Verifying config..
[21:51:02:353]: LOG ✗ Twitch API key is missing.
Error: Invalid authentication credentials
    at Client.user (C:\Discord-Bot\node_modules\fortnite\src\Client.js:40:70)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.module.exports.run (C:\Discord-Bot\src\scripts\verify-config.js:71:3)
    at async C:\Discord-Bot\src\index.js:7:24
[21:51:02:697]: LOG ✗ Fortnite API key is incorrect.
[21:51:03:697]: LOG ✗ Ksoft API key is incorrect.
[21:51:03:697]: ERROR Please fix your errors before loading the bot.

All API's are optional so if you don't want to use a particular API just leave the value as an empty string. ('')

i already leave the string empty sir, but the bot still doesn't want to run :( ( sorry for my bad english sir )

I'm talking about the ksoft & fortnight as it says incorrect API instead of missing API.

when i remove all api string to empty, its stuck on that proccess

fixed. thankyou sir :D


Using 'system' or 'user' reserved names as param names breaks execution

It looks like even in 1.4, if you used a param name "system", it will blow up execution API because of our jinja resolution code.
Now that we allow jinja expression like {{user.}} too with https://github.com/StackStorm/st2/pull/2763, 'user' becomes a reserved param name.
Our chatops CI exposed this issue after we merged https://github.com/StackStorm/st2/pull/2763 (thanks @humblearner and @manasdk). We are using param 'user' in chatops.post_message https://github.com/StackStorm/st2/blob/master/contrib/chatops/actions/post_message.yaml#L13.
I amended the unit tests fixtures to include params 'system' and 'user' so they fail. See travis failure.
Solutions

We have to live with the fact that we want UX in terms of  {{system.}} and {{user.}} and disallow param names 'system' and 'user'. In this case, we would fix https://github.com/StackStorm/st2/blob/master/contrib/chatops/actions/post_message.yaml#L13.

If we want to allow those param names, we have to introduce alternate jinja expressions like {{system_kv.}} and {{user_kv.}}. This only reduces the chance of collision.

I'm ok with 2 (just name st2system and st2user) - but this comes with deprecation of system and migration to anyone who is already using system and user.

First question - can we delay #2763 for post v1.5 (if that's indeed the only change affecting that)?
This way we don't need to rush and have a bit more time to decide how to proceed.
I'm personally OK with 2) - changing the prefix used for accessing the datastore. In addition to that, we should also make it more user friendly and throw when registering actions in case a reserved parameter name is used. I think second part is also important, otherwise users will only get notified during run-time and that's bad UX...

I'm ok with 2, better to use a prefix or even a param (kv.user and kv.system). We can still allow using system for compatibility, but deprecate it eventually.
user. and system. aren't really descriptive, too. I wouldn't tell where the data from user. comes from. A clear prefix would solve this problem.

We decided that we'll revert #2763 until after v1.5 which gives us extra time. We are going to leave {{user.}} as such for now. We'll migrate {{system.}} and {{user.}} in next release.


[Instance Segmentaion]  Cannot import name 'MasksRecordComponent' from 'icevision.core.record_components'

🐛 Bug
To Reproduce
Follow this link which redirects to the example in the doc Instance Segmentaion
Steps to reproduce the behavior:

Import the libraries
Run
Import error occurs

Code sample
from flash.image import InstanceSegmentation, InstanceSegmentationData
Error occurs while executing the above code.
Expected behavior
Still not clear of the potential output.

Environment
-PyTorch Version (e.g., 1.0): 1.10.0
-OS (e.g., Linux): Windows
-How you installed PyTorch (conda, pip, source): pip
-Python version: 3.9
-CUDA/cuDNN version:  Non
-GPU models and configuration: Geforce 640M
-Any other relevant information:
* pytorch-lightning==1.5.5
* lightning-bolts==0.4.0
* lightning-flash[image]==0.5.2

Closing this as it should be fixed in Flash master. Please feel free to re-open if you still get the error :smiley:


Get 3d bbox ground truth

How can i get 3d bbox ground truth of each pose of camera? It seems like benchbot api doesn't support to do so

We currently do not support getting ground_truth bounding boxes for each object when receiving observations after each action. The ground-truth bounding boxes for the objects of interest can be found (for development environments) in the ground-truth-bear-develop add-on folder which should have been installed as part of your standard benchbot install


bunch of FN in our current rules

[x] free user process memory
[x] read data from Internet
[x] copy file
[x] crypto
[x] allocate user process RWX memory https://github.com/vivisect/vivisect/issues/296
[x] modify service
[x] get MAC address
[x] reference processor manufacturer constants
[x] set global application hook
[x] enumerate services
[x] start TCP server
[x] get Program Files directory https://github.com/vivisect/vivisect/issues/297
[x] delay execution
[x] get session integrity level
[x] reference anti-VM strings
[x] check for unmoving mouse cursor
[x] get Program Files directory

@mike-hunhoff "free user process memory"  493167E85E45363D09495D0841C30648:0x404B00 missing API ZwFreeVirtualMemory

fixed "read data from Internet"

fixed "copy file"

delete "crypto" with preference to match: data-manipulation/encryption

"allocate user process RWX memory" works in IDA, not in viv.

"get mac address" viv doesn't recognize the function
also this is apparently an installer, so capa is hesitant to run.

"reference processor manufacturer constants" uses example thats shellcode that doesn't start at offset 0x0

"set global application hook" viv doesn't recognize function

"get Program Files directory" i'm not sure that viv recognizes delayed imports? https://github.com/vivisect/vivisect/issues/297

I've double checked the above and fixed one example address in #815.


Plase add backup all expect some package...

how to select not only one package or select all expect some package,such as container manager....because some package need too long time.
if i can run the script in the plan? i want not run the script by hands.
thank you for your development！

I could add an --auto option
Then in syno_app_mover.conf I could add an auto setting that could be set like:

auto="all"
auto="AudioStation,Calendar,Chat,NoteStation"

As well as an exclude setting for when auto="all" is used:

exclude=
exclude="ContainerManager"
exclude="DownloadStation,ContainerManager,HyperBackup"

I could add an --auto option
Then in syno_app_mover.conf I could add an auto setting that could be set like:

auto="all"
auto="AudioStation,Calendar,Chat,NoteStation"

As well as an exclude setting for when auto="all" is used:

exclude=
exclude="ContainerManager"
exclude="DownloadStation,ContainerManager,HyperBackup"

how about the process? i think can add --auto backup ,--auto move and so on

I thought of that as I was typing my previous reply.
I'll probably need a --automode option:
--automode=backup
--automode=restore

v4.0.73

Added exclude option in syno_app_mover.conf so you can exclude specified apps when "all" mode is selected.


Push Notifications

Come up with an idea how to demonstrate the implementation of push notifications using this app.

Not necessary here


Update example to use more idiomatic Ruby

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@dustyburwell

Please sign the CLA here.

This falls under the obvious fix clause.  That code is certainly simpler. Thanks @dustyburwell


Is the pre-trained facial landmark model included?

Hello
When I run python head_pose_estimation.py I get the following error:
OSError: Unable to open file (file read failed: time = Sun Dec 27 21:55:55 2020 , filename = 'models/pose_model', file descriptor = 3, errno = 21, error message = 'Is a directory', buf = 0x7fff051ef380, total read size = 8, bytes this sub-read = 8, bytes actually read =<PHONE_NUMBER>3709551615, offset = 0)
Is the pre-trained facial landmark model included in the folder? Or we need to train from the repo(https://github.com/yinguobing/facial-landmark-detection-hrnet )you have offered.

There is a pose_model folder present in the models directory. You can use that or download the original one one the link you mentioned.


fix: org name with 2 parts in identifier Ex. com.example

Description
org-name is not accepted when given a single identifier
Steps To Reproduce
For creating app with org name com.example we use very_good create --org-name com.example example but this creates an app with applicationId com.example.example which actually should have been com.example
If very_good create --org-name com example is used, it is not accepted as the identifier required 2 parts between .
flutter create --org com example creates the valid required applicationId com.example
Expected Behavior
very_good create --org-name com example should create an app with applicationId com.example

hi @toufeeqahamedns 👋  thanks for the heads up! For now, our CLI requires an organization domain, which is formatted as <domain>.<organization>.<project_name>, I don't see a case where creating a project with a <domain>.<project_name> format is required.
Android, for example, uses the Java package naming conventions. So I don't know if there's an error when uploading to Play Store or App Store if the package name doesn't follow these conventions.
Maybe you can provide more context about these aspects?

Hey @marcossevilla, thanks for highlighting on Java Package naming conventions, the guidlines defined accept identifiers with 2 parts.
The <domain>.<project_name> would be very much helpful when you have the project say google.com and want to use com.google as the identifier
Also as the flutter create and both native android and ios project setup wizards allow to create the projects with  <domain>.<project_name> it would be good if very_good create support it as well

I think if you have google.com as org_name it would miss the project_name, like com.google.gmail for example.
What we try to achieve with very_good_cli it's to have the standardized formatting of <domain>.<organization>.<project_name>.
Tagging @felangel and @jorgecoca to have some second thoughts on supporting this alternative.

I think rather than passing an organization name in, you should just be able to pass whatever bundle id you want in. I've been so confused when generating new projects and the cli tool appends the project name to the organization name I passed in. I'd much rather just pass in the bundle id instead of VG CLI trying to be smart enough to compose it from the organization name and project name (because sometimes the project folder name is different from the name i'd use in the bundle id).

I think if you have google.com as org_name it would miss the project_name, like com.google.gmail for example.

What if the org_name is your project name, as in google.com being the project and also the org name itself.

I'd much rather just pass in the bundle id instead of VG CLI trying to be smart enough to compose it from the organization name and project name

This would be a much better option

I agree with @definitelyokay, using the bundle ID would be better than having the org_name and project_name composing it. I'm just doubting about the usage of org_name, if it's used solely on another part and if this change would break that usage. Labeling as investigating as this may require to map where org_name is used. 👍

I'm not sure if this would be considered a separate issue, but 4 part bundle identifiers also have issues.
very_good create --template flutter_plugin  --org-name com.example.subdomain.foo places the android class in the wrong dir (not sure if that is the only issue, but the only one I have encounter so far).
It results in an error:
The plugin `foo` doesn't have a main class defined in
/foo/foo_android/android/src/main/java/com/example/subdomain/foo/Foo.java or
/foo/foo_android/android/src/main/kotlin/com/example/subdomain/foo/Foo.kt. This is likely to due to an
incorrect `androidPackage: com.example.subdomain.foo` or `mainClass` entry in the plugin's pubspec.yaml.

If you are the author of this plugin, fix the `androidPackage` entry or move the main class to any of locations used
above. Otherwise, please contact the author of this plugin and consider using a different plugin in the meanwhile.

I believe com.example.subdomain.foo follows the Java package naming conventions shared above.

a package name begins with the top level domain name of the organization and then the organization's domain and then any subdomains, listed in reverse order.

hey all! did a small POC of this change to see what we need to change in order to deprecate org_name + project_name to form a bundle ID/package name and just use project_id, here's what I found:

org_name let's us format the project_name correctly to form a compliant bundle ID and package name:

Android: com.example.veryGoodCore 👉 {org_name}.{project_name in camelCase}
iOS: com.example.very-good-core 👉 {org_name}.{project_name in paramCase}

creating a project ID property that has all the bundle ID/package name can be difficult to parse for both types of identifiers.
there shouldn't be a problem if we use project_id as path to our MainActivity.kt file on Android.

what we can do:

allow org_name to be just 1 part so we can concatenate the project_name and have 2-part identifiers, e.g: very_good create gmail --org-name google
create bundle_id and package_name properties so we can use them as needed in their respective projects

leaning towards allowing 1 part for org_name as it would involve less impact to the existing implementation and there's some redundancy if we maintain a project_id and org_name at the same time for specific use cases. let me know what y'all think!

Hey @marcossevilla , thanks for working on POC, I strongly agree with going ahead with the first approach defined as this is also the case with flutter create
Would like to hear any suggestions from others.

hey all! did a small POC of this change to see what we need to change in order to deprecate org_name + project_name to form a bundle ID/package name and just use project_id, here's what I found:

org_name let's us format the project_name correctly to form a compliant bundle ID and package name:

I'm not sure I fully understand what you mean. Part of the issue is that bundle id's and the folder name where you want to store the project (i.e., the name you give very_good_cli) are not necessarily correlated. In my opinion, Very Good CLI should let you specify an exact bundle id, pretty project name, and folder name all independently. If you don't specify the bundle id or pretty project name, it should infer those from the folder name.

allow org_name to be just 1 part so we can concatenate the project_name and have 2-part identifiers, e.g: very_good create gmail --org-name google

What do you mean by just 1 part exactly?

there's some redundancy if we maintain a project_id and org_name at the same time for specific use cases.

Also, not sure what you mean by this either.

@marcossevilla Thanks for taking the time to explain!

If we have a project_id variable that represents the bundle ID and package name for the project's platforms, we'd still need to maintain org_name for specific cases like the path to this file.

Wouldn't the project name be the right-most section of the bundle id? i.e., for a bundle id com.domain.subdomain2.subdomain1.myAppId, the project name is machine-inferable as everything after the last dot ., myAppId.

org_name currently supports two parts: com.google for example. One part would mean com without .google is supported.

If I understand correct, that's one of the current shortcomings with very_good_cli. Org names should be able to support more than 2 parts, because AFAIK org id's like the aforementioned com.domain.subdomain2.subdomain1 should be perfectly valid. That is, unless I'm wrong about the Java package naming convention. (Interestingly enough, Java doesn't allow hyphens in package names. Also, Oracle has an example package name on this page  showing a package name with several subdomains.)

The formatting for each platform is different as they have different casing and handling these can involve (1) having two separate variables to ask for depending on casing or (2) having one variables and parse it, but this leads to bad formatting because of collision of two different casings (com.example.app uses dot.case and com.example.myApp uses dot.case and camelCase on the current project_name part

Can you explain a bit on the collisions? I think I almost see what you're saying, but you've dug into this far more than me. I am hoping there is a deterministic solution to this with minimal effort on the end-user's part.

Revisiting this and we should be able to support org names of variable length. Adding this to triage 👍

We've discussed potentially adding a --applicationId to provide developers the ability to override the value.

I am trying to wrap my head around all the discussion here still, so please, let me know if I am missing something.
I don't think that creating a --applicationId parameter that overrides org_name.project_name will work as expected as that will generate an invalid package/folder name for android.
For example, we use org_name to generate the folder for the kotlin source:

And then org_name.project_name to generate the package for the MainActivity.kt.

So, if the value on applicationId differs too much from that pattern, the generated folder structure and package name will be invalid.
If I understood correctly, and please correct if I am wrong @definitelyokay, the biggest issue is that org_name.project_name may generate an unwanted result for the iOS bundle id.
Since this seems to be specifically related to iOS, what if instead we created a customBundleId argument that would override this value on iOS, since the current standard seems to make sense for Android?


wm: Added WM version.

Description
The goal of this feature is to be as generic as possible. We call $wm with --version and -v and use whatever exits with 0.
This will stay off by default.
Closes #918
TODO

[ ] Test this on as many WMs as possible. (neofetch --wm_version on)

[x] Openbox
[x] i3
[x] bspwm
[x] dwm
[x] KDE
[x] aewm
[x] blackbox
[x] fluxbox
[x] evilwm

On KDE it does not work because the commands to display the KWin version arekwin_x11 -v or kwin_wayland -v.
Adding "${wm,,}_x11" -v works for me.
            wmv="$wm $("${wm,,}" --version || "${wm,,}" -v || "${wm,,}_x11" -v)"

I don't use wayland, but on my system kwin_wayland -v also works.
To be save maybe "${wm,,}_wayland -v" should also be added?

Done.

It worked for KDE, but with splitting it in two wmv= lines in 2bb8a30a4918dc64e6f73345d196b631c0cf8325 it does not:
WM: KWin ./neofetch: line 796: kwin: command not found

Hi Dylan, I just noticed you merged this into master and wondered if you missed my previous comment?

This was an accident, oops.
I'll revert it and reopen a new PR.

#971
The problem with having all of the commands on one line is that some programs exit with a status code of 1 regardless of whether or not the version flag is correct.
Some commands also print to stderr which is weird.


snapshot testing best practices

Hi,
I don't know where to ask, so I am asking it here.
I read all about snashot testing via websites and issues presented here.
There are many combinations how we can test our apps using this tool.
I've just started using this tool and it would be nice to here how are you using it and your best practices.
e.g.

Are you testing app for all popular iOS versions (10, 11 and 12).
On which resolutions are you testing or which simulator devices are you using.
Are you running tests via Xcode or xcodebuild.
Are you using additional tools like FBSnapshotsViewer  https://github.com/Antondomashnev/FBSnapshotsViewer

Let me know your way of doing it, thank you.

@DavidTomic I use this library in our my company and I wrote and article about the topic and could help you because I answered some of this questions with the solution that worked better for us. This is the link:
https://badootech.badoo.com/implementing-ui-in-ios-better-faster-and-it-scales-9788659e482
Let me know if you have questions about it! I hope it helps!

@alexissan thank you for this article, it is very impressive and helpful.
It is a great idea to make snaphots only for the most popular device.
I suppose you have a lot of components but your snapshot folder is only ~11MB
I am wondering is there a way to make snaphots smaller. Because If I take a snapshot using iPhone X for view which has 375w x 180h in points, resulting snapshot than has 1125 x 540 pixels which is 60KB. So If I have e.g. 1000 snaphots that would be 60MB.
What is your process of resolving failure snaphots, do you use any tool for this.
Do you have issues after Xcode and iOS upgrades, e.g. you were testing on simulator with iOS 10, and after Xcode upgrade you now run tests on iOS 11 and here there is big posibility that many of test will fail. How do you manage this.
@alexissan thank you for your time.

Hey there @DavidTomic

I am wondering is there a way to make snaphots smaller. Because If I take a snapshot using iPhone X for view which has 375w x 180h in points, resulting snapshot than has 1125 x 540 pixels which is 60KB. So If I have e.g. 1000 snaphots that would be 60MB.

Unfortunately this is a problem that always comes up with snapshot testing because Retina images are large by nature. Companies like Facebook and Uber get to the point where our snapshots take up hundreds of MBs of space. One solution is you could look into Git Large File Storage.

What is your process of resolving failure snaphots, do you use any tool for this.

It is pretty much the same as debugging any issue with your code. Putting in a breakpoint and using Xcode's view debugger or Facebook's Chisel to inspect your view hierarchy.
Also if you look at the console for a failed test you'll find something like
If you have [Kaleidoscope](https://www.kaleidoscopeapp.com/) installed you can run this command to see an image diff:
ksdiff<EMAIL_ADDRESS><EMAIL_ADDRESS>
If you don't use Kaledioscope you can still use the absolute file paths printed to the console to open the images in other tools like Preview.

Do you have issues after Xcode and iOS upgrades, e.g. you were testing on simulator with iOS 10, and after Xcode upgrade you now run tests on iOS 11 and here there is big posibility that many of test will fail. How do you manage this.

Yes all of the time. You can guarantee that Apple will always break something involving the UI when you jump major iOS version. You can manage this one of two ways:

When you jump to a new iOS version you regenerate all of the snapshots and manually check to see everything looks good.
Use @property (readwrite, nonatomic, assign) FBSnapshotTestCaseFileNameIncludeOption fileNameOptions; and ensure FBSnapshotTestCaseFileNameIncludeOptionDevice and FBSnapshotTestCaseFileNameIncludeOptionOS are turned on so you don't regenerate any snapshots but you create new ones that have the simulator and OS version embedded in the filenames.

Hopefully this answered your questions David but feel free to reopen the issue.


Create pt.json

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Enhancement (changes that improvement of current feature or performance)
[ ] Refactoring (changes that neither fixes a bug nor adds a feature)
[ ] Code style optimization (changes that do not affect the meaning of the code)
[ ] Docs (changes that only update documentation)
[ ] Chore (changes that don't modify src or test files)

Self Check before Merge

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Screenshots (if appropriate):

Thank you for this translation!


TypeError during first sync

I've downloaded the latest pip version and started a sync from Withings to Garmin (with option -f 2016-01-01)
I'm getting the following error:
2022-07-07 10:47:32,697 - withings - INFO - Get Measurements
Traceback (most recent call last):
  File "/Users/robertvh/Library/Python/3.9/bin/./withings-sync", line 8, in <module>
    sys.exit(main())
  File "/Users/robertvh/Library/Python/3.9/lib/python/site-packages/withings_sync/sync.py", line 396, in main
    sync()
  File "/Users/robertvh/Library/Python/3.9/lib/python/site-packages/withings_sync/sync.py", line 349, in sync
    last_weight, last_date_time, syncdata = prepare_syncdata(height, groups)
  File "/Users/robertvh/Library/Python/3.9/lib/python/site-packages/withings_sync/sync.py", line 248, in prepare_syncdata
    groupdata["weight"] / pow(groupdata["height"], 2), 1
TypeError: unsupported operand type(s) for /: 'NoneType' and 'float'

It looks like the Withings API is returning entries with weight = null.

I also get the same error message with version 3.4.2.
This can be fixed by changing line 246 if height and "weight" in groupdata: to if height and groupdata["weight"]: in withings-sync/sync.py:
        if height and groupdata["weight"]:
            groupdata["bmi"] = round(
                groupdata["weight"] / pow(groupdata["height"], 2), 1
            )

I'll have a look.

@longstone : I tested that PR #76 and it fixes the problem.

When I installed using pip I still have not got this fix and getting TypeError.

When I installed using pip I still have not got this fix and getting TypeError.

This might be caused by setup.py not reflecting the latest version. It is still on *.2?


How do you run tests for a single module/ run test for all the modules

see https://github.com/nWidart/laravel-modules/discussions/1274


Failes to run with kotlin 1.9.20 and Java 21

Expected Behavior
Should be able to run detekt with Kotlin 1.9.20 and Java 21.
Observed Behavior
Receive an error

What went wrong:
Execution failed for task ':project:project-impl:detekt'.

Invalid value (21) passed to --jvm-target, must be one of [1.6, 1.8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20]

Steps to Reproduce
i am using gradle-8.5-rc-3.
My Detekt version is 1.23.3.
When compiling with Kotlin 1.9.20 it works with JDK 17. But when i upgrade to JDK 21 it fails with the error:
Execution failed for task ':project:project-impl:detekt'.

Invalid value (21) passed to --jvm-target, must be one of [1.6, 1.8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20]

Context
I am trying to upgrade our spring boot project to Java 21. To accomplish this i am upgrading: Spring boot to 3.2 with Kotlin to 1.9.20 and Java to 21.
Your Environment

Version of detekt used: 1.23.3
Version of Gradle used (if applicable): gradle-8.5-rc-3
Gradle scan link (add --scan option when running the gradle task):
Operating System and version: Windows
Link to your project (if it's a public repository): nope.

May be related to issue https://github.com/detekt/detekt/issues/6591

This was fixed by https://github.com/detekt/detekt/pull/6572 and will be included in the next detekt release. You can try using a snapshot version of detekt in the meantime.


Fix broken links

Links to applications and api getting started are no longer working.  Update to intended location.

Thx


Appstudio update hacbs-test-project-w7xs

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-w7xs-on-pull-request-
creating pipelinerun hacbs-test-project-w7xs-on-pull-request- in namespace rhtap-demo-pgat-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-w7xs-on-pull-request-
creating pipelinerun hacbs-test-project-w7xs-on-pull-request- in namespace rhtap-demo-hfag-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


[SPARK-31650][SQL] Fix wrong UI in case of AdaptiveSparkPlanExec has unmanaged subqueries

What changes were proposed in this pull request?

Make the non-subquery AdaptiveSparkPlanExec update UI again after execute/executeCollect/executeTake/executeTail if the AdaptiveSparkPlanExec has subqueries which do not belong to any query stages.
Why are the changes needed?

If there're subqueries do not belong to any query stages of the main query, the main query could get final physical plan and update UI before those subqueries finished. As a result, the UI can not reflect the change from the subqueries, e.g. new nodes generated from subqueries.
Before:

After:

Does this PR introduce any user-facing change?

No(AQE feature hasn't been released).
How was this patch tested?

Tested manually.

cc @cloud-fan @maryannxue

Test build #122350 has started for PR 28460 at commit 5e69f87.

Merged build finished. Test PASSed.

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

Test build #122350 has finished for PR 28460 at commit 5e69f87.

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

Merged build finished. Test PASSed.

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

thanks, merging to master/3.0!

thanks!


Migrate Hugo Docs into main repo

Does it make sense to merge the docs and docs-website folders? It looks like the only usage of the current docs folder is to hold the vectordb ADR, a leapfrogai png, and the customer pngs. I feel like the pngs could be moved to the assets subfolder and the ADRs remain its own folder within the combined docs or moved to a wiki, however we want to track those going forward.


Animations

In this Project I did the following

update style.css
update index.html
use animation property.

@leonardodiasb do I have to create a new branch and name it Deployment

https://github.com/sonyco-4u/Portfolio-microverse2/pull/10

@leonardodiasb please can I submit this
https://github.com/sonyco-4u/Portfolio-microverse2/pull/10


chore: update CLI to the latest version

Summary
Bumps CLI to the latest version, needed by https://github.com/facebook/react-native/pull/28572 to work.
Changelog
[INTERNAL] - Bump CLI to latest

Note: I won't be able to land this until ~Friday or so.


Update syntax

I would like to write the result of 'java-jar plantuml.jar -language' and others separately, what do you think?

Thanks a lot, good to have an update!

Thank you.


[HUDI-627] Aggregate code coverage and publish to codecov.io during CI

What is the purpose of the pull request

This PR aggregates code coverage across modules to get a holistic coverage number (since we have tests that gets run across modules. e.g. hudi-client tests classes in hudi-common)
This also published the report to codecov.io (e.g. report from the fork)
Above step automatically adds coverage report to every PR (e.g. PR from fork)

Brief change log

Modified pom files to include coverage in missing modules (hudi-hadoop-mr, hudi-spark & hudi-timeline-service)
Added a wrapper module for aggregation (due to limitations in jacoco). This can be remove in future once this PR gets merged to a release.

Verify this pull request

Ran travis in private fork and verified the published report & PR.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

we have a code coverage report already in codecov.io through this PR

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@078d482). Click here to learn what that means.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             master   #1347   +/-   ##
========================================
  Coverage          ?   0.56%
  Complexity        ?       2
========================================
  Files             ?     333
  Lines             ?   16234
  Branches          ?    1658
========================================
  Hits              ?      92
  Misses            ?   16139
  Partials          ?       3

Impacted Files
Coverage Δ
Complexity Δ

...main/scala/org/apache/hudi/DataSourceOptions.scala
67.04% <0%> (ø)
0 <0> (?)

...in/scala/org/apache/hudi/IncrementalRelation.scala
0% <0%> (ø)
0 <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 078d482...f3c3492. Read the comment docs.

@vinothchandar codecov lets us just upload reports without ownership. here is a screenshot of the admin page. it says that anyone with github permissions will be able to modify the codecov settings for that repo.

@ramachandranms guess that coz its free for OSS? I am good with that part

@ramachandranms could you rebase again and push.. for some reason, its showing 72 files being changed

@vinothchandar will do. also the recent change was not showing coverage for hudi-hive module. will ping you once I get this back to working state.

@vinothchandar cleaned up the PR and everything looks good in my own fork build + codecov. please let me know if there are any other concerns. thanks


Server: API

This will be in general last phase of 0.5.0 - create an API through which a game could be played. Quite roughly, but it will be prepared for Admin and also for some client (for the beginning some simple interface, maybe targeting iOS).

This is basically done.


Set up GitHub Actions to run the normalize repos script every day

Problem Description
The normalize repos script is only run manually currently. We want it to automatically run so that our GitHub repos are always up to date with the latest labels and settings.
Solution Description
Set up a GitHub Actions workflow in this repo to run the script every day.
Additional context

GitHub Actions documentation
Please include instructions on how to set up the appropriate GitHub Token. Your GitHub token will not have permissions to update any repo settings so you'll need to test using repos on your own account.

My PRs are completed and have their reviews pending. So can I take this while I await the reviews?

Yes, go ahead.


LowLevelMPC implementation

Instead of stating a state reference trajectory for every node, follow the same philosophy as with the trajectory generator.

This is already done and more or less working. However, both MPC should follow the same structure.


CI: automatically report ci-datapath configuration

Currently, the ci-datapath Github Action configuration matrix is manually constructed - https://github.com/cilium/cilium/issues/20606.
To prevent the table from becoming stale or false, we could automatically derive the configuration table after each run. The idea is to store a result of cilium status - https://github.com/cilium/cilium/blob/4ff7c8934c485b810fbe50acce2b8d0f7e7b5eac/.github/workflows/conformance-datapath.yaml#L386 for each job, and then parse it the end once all jobs have completed.

I want to work on this issue, can you guide me a little @brb ?

@Kanika637 Thanks, sure!

Actually I am in a middle of something, will surely get back to this issue
ASAP.
On Thu, 23 Mar, 2023, 6:53 pm Martynas Pumputis, @.***>
wrote:

Hey @Kanika637 https://github.com/Kanika637 , do you need any help with
the issue?
—
Reply to this email directly, view it on GitHub
https://github.com/cilium/cilium/issues/24298#issuecomment-1481187575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUDRPL3SEPZMXTHVNCZRGZTW5RFD5ANCNFSM6AAAAAAVWRZRXA
.
You are receiving this because you were mentioned.Message ID:
@.***>

can i work on this issue

@Kanika637 Are you still on the issue?

No, I will make it unassigned to me.

@Saimurugeshwari Are you still interested in working on this issue?

Hey @brb I am trying to understand this issue. I took a quick look and didn't find any yaml file called conformance-datapath.yaml in workflows. Am I missing something?

Hi @SD-13, the file got split and renamed to conformance-e2e.yaml and conformance-ipsec-e2e.yaml.


aio app run for asset compute error

Describe the bug
when running aio app run, for an asset compute worker, the error was thrown
% aio app run
Using local project's @adobe/aio-cli-plugin-asset-compute version 2.0.1
›   Error: You can only run one implementation at the time, please filter with
›   the '-e' flag.
I am only running it in 1 terminal, there are no other terminals running app run.

this means that you are attempting to run an application which implements multiple extensions, you need to use the -e flag to specify which extension to run.
Without knowing more about your project, I suspect you need to run aio app run -e dx/asset-compute/worker/1


Implement CharNGram Based HashVectorizer

Receive ngram_range.
Calculate n_grams within a Doc object's lowerized tokens.
Send (n_gram, str) pair iterator to FeatureHasher.
Implement tests with tokenizer and ngram_range configurations.

Please do add a model which got improved  by the use of this Transformer

I improved telco_sentiment with Model test accuracy (accuracy): 0.7040786809372288. Moving on to other prebuilts that use hash.


Community Committee Chairperson Self Nominations

It's nomination time for the Chairperson of Community Committee.
Please submit your self-nominations to this form by Monday. August 26 at 11:59pm PT. Nominations will be published after the date passes so that candidates can be considered for election.  Please do not publicly share you are running prior to this date. This allows folks to consider contributing as chair without the added distraction of competition.

The CC will elect from amongst voting CC members a CC Chairperson to work on building an agenda for CC meetings [...] The CC shall hold annual elections to select a CC Chairperson [...]; there are no limits on the number of terms a CC Chairperson [...] may serve.

Expected responsibilities include but are not limited to:

Create each meeting's agenda.
Create the event and make sure it is live-streamed in YouTube.
Chair and facilitate each meeting. It is the chairperson's responsibility to ensure that the meeting stays on track, sticks to the agenda, and finishes within the allotted time.
Collaborate with the CPC Reps
Collaborate with the TSC Chair
Coordinate efforts in shared repos, as listed in Admin

All Community Committee active members are eligible to self-nominate. If you are having trouble with voting, please email<EMAIL_ADDRESS>Nominated candidates:

Your Name and @-handle will be added here after nomination closes

@amiller-gh @bnb Would it be possible for you to share a bit more about the time commitment you have found useful/necessary to be effective in the role? Esp things like standing weekly/monthly meetings and the # of ad hoc ones you found yourself in? I think it's helpful for folks to understand how to be helpful and to manage expectations with employers.

I personally found it ended up being about 20/hours a week on a normal week, which was largely during work hours since that was something that my time as a Developer Advocate at NodeSource enabled me to do.
I was attempting to attend as many new Initiative meetings as possible to help them get off the ground and running, in addition to consistently answering 1:1 pings on various platforms (Slack, Twitter DM were the primary ones, I don't recall the others but there were a few people who preferred other platforms. and having discussions about concerns / answering questions that were sometimes rather in-depth.
Outside of that, there were normal responsibilities including attending every single CommComm meeting and following through on leadership tasks such as working with other project leadership (at the time, TSC Chairperson, TSC & CommComm Directors) on certain massively impactful decisions (one of the biggest that I worked on was the beginnings of the merger of the Node.js Foundation and JS Foundation, into OpenJS Foundation).
I believe the number of regular meetings I quoted previously was something like 6-8, with an additional 4-6 meetings ad hoc meetings weekly.
Additionally, it's worth noting that you will have to care about the things nobody else does. You will have to deal with all the things that are seemingly trivial but must be completed – charter/governance processes, updating and maintaining certain aspects of how the group works, maintaining repos that nobody else works on, closing stale/old issues periodically, maintaining Org membership for individuals under the CommComm and its initiatives, responding to random one-off requests like the recent one to unarchive nodejs/evangelisim and nodejs/website-redesign so issues could be unlabeled, and so on.
I would highly recommend having at least 20 hours a week – including at least 10 during working hours – that you can 100% dedicate for the next year to being chairperson if you're considering applying.

@amiller-gh also, quick question: how are you storing the information in the form? We had a discussion about this last time around and were very intentional in our approach – I believe we opted out of using a form for very specific reasons.

@bnb – its in a private sheet in google drive. I duplicated the form used last year.
At its peak, I found it to be a similar time commitment. I attended fewer individual initiative meetings because they were largely self-sufficient (good job @bnb!), so in my experience it was a similar 14-18 hours a week.
Agreed that it is very important to have time at work to dedicate to this. Personally, I had this flexibility for the majority of my tenure, but the past couple months I haven't had as many cycles during work to dedicate, and have struggled to keep up.

@nodejs/community-committee Link to vote was just sent out. You should have it in your inboxes at the address listed in the CommComm README. You have until Saturday EOD to cast your vote for Tierney, or “No Confidence”. Please let me know if you did not receive it!

@nodejs/community-committee Polling is closed as of Saturday night and I'm happy to announce that...
(drum roll 🥁🥖)
@bnb is now Community Committee chairperson! Excited to see what this next year brings ❤

Closing this issue as we have a new a CommComm chair


Cut v0.0.1 release tag

We have the mechanics in place now to publish binary releases for zcertificate. We should take advantage of this and push a v0.0.1 release and tag.
It makes sense to block this on https://github.com/zmap/zcertificate/issues/19 to avoid cutting a release with a RC ZLint.

I'll do this after https://github.com/zmap/zcertificate/pull/22 lands.

https://github.com/zmap/zcertificate/releases/tag/v0.0.1 :tada:
ZCertificate is all grown up now :mortar_board:


Delta Lake diff: Add documentation for the Delta Lake diff plugin

Linked Issue
Closes #5128

Change Description
Add instructions on how to configure the Delta Lake diff plugin.

@arielshaqed what do you mean by:

How was this tested?

?
Do you refer to the contents of the docs?
If so, it's the content of all of the previous PRs which were tested.

@arielshaqed what do you mean by:

How was this tested?

?
Do you refer to the contents of the docs? If so, it's the content of all of the previous PRs which were tested.

How did we test that what the docs say works?  "Copied all the steps from my shell history" is a good answer, but there might be others.

@arielshaqed got ya.
I tested each type of configuration and made sure it worked as described


Reproducing results from Table 1

Hi @ahmed-agiza,
Based on Issues #4 and #5, I ran the code with the mtlora_tiny_448_r64_scale4_pertask config. I got the following results:
semseg: 68.38
humans: 59.58
sal: 63.85
normals: 16.76
delta_m: 1.77%

The validation was run every five epoch. The above-mentioned results are based on the best $\Delta m$ across all validation scores. The saliency performance seems to be the most off compared to the numbers in the paper, followed by normals. Any suggestions on what may be causing the discrepancy?
The task weights are the following:
{
    'depth': 1.0,
    'semseg': 1.0,
    'human_parts': 2.0,
    'sal': 5.0,
    'edge': 50.0,
    'normals': 10.0,
}

Could you also check if you used the same weights for your runs? Also, could you please share a trained checkpoint when you get a chance, as discussed previously in #4.
Thanks!

Hi @neerajgangwar, it is a bit hard to debug an external run, but I can confirm that this is the same configuration yes; assuming you are running everything right, then the only reason I can think of is if we introduced some bug during the cleanup for releasing the public code (this the is only one we didn't get the chance to test thoroughly), so I can try to re-train using the public code after the conference to investigate (since my GPUs are busy for a while).
The exact command was main.py --cfg configs/mtlora/tiny_448/mtlora_tiny_448_r64_scale4_pertask.yaml --pascal <dataset pat> --tasks semseg,normals,sal,human_parts --batch-size 32 --ckpt-freq=20 --epoch=300 --eval-freq 25 --resume-backbone <path>/swin_tiny_patch4_window7_224.pth
And those are the results on wandb:

I have the log file and the wandb run; for the checkpoint it is just in my co-author's home directory, and she is on a break till the conference, so hopefully, I can get it afterward (feel free to re-ping me!).

@ahmed-agiza Thanks for sharing the details. It would help identify the issue if you try running the publicly released code and compare the results.  I will keep checking on my end. I am keeping this issue open for now. Thanks again!

Hi @neerajgangwar, yeah, those numbers are much closer; I will still try to retrain when I get the chance to validate and post the weights to the repository, too.
I didn't try gradient accumulation in this project, but I do remember having a very similar problem before in a different project that was using Swin, too, but I didn't look into it back then.
Thanks for taking the time to investigate, and I will try to ping you when I post the weights.

Sure, thanks for the clarifications and help!


Switch of and by?

Intuitively, you would expect the opposite behavior. E.g. 1 of 2 = 1/2, while 2 by 4 = 8. Both are sometimes used in English, so I think it would make sense to reverse the order of these two keywords.

"One half of six slices of pizza is three slices of pizza."
"Two (groups) of four flowers is eight flowers."
I don't have strong opinions on 'by', but I agree with the current specification when it comes to the meaning of 'of'.

Yeah, agreed - by is potentially confusing. I think of works just fine by by as an alias for over is (a) misleading, and (b) maybe a bit unnecessary given over is already pretty colloquial.

by removed in https://github.com/dylanbeattie/rockstar/commit/bc43323992450850692af8ca10772a1c349822e3


create 0252-meeting-rooms-csharp

File(s) Modified: 0252-meeting-rooms.cs
Language(s) Used: c#
Submission URL: _https://leetcode.com/problems/meeting-rooms/submissions/877051013/_

This solution is similar to the existing solution in the code


[Enhancement] Method builds a list from one element using Arrays.asList [HandleMsgContext]

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

located at:
eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/http/consumer/HandleMsgContext.java line 204
analysis and explanation:
This method builds a list using Arrays.asList(foo), passing in a single element. Arrays.asList needs to first create an array from this one element, and then build a List that wraps this array. It is simpler to use Collections.singletonList(foo), which does not create the array, and produces a far simpler instance of List. Since both of these arrays are immutable (from the List's point of view) they are equivalent from a usage standpoint.
Describe the solution you'd like
It is simpler to use Collections.singletonList(foo).
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

I want to fix it.


Use RegExp in Express use function

If these lines are not in the express.d.ts
, Regexp are not accepted in
app.use( regexp, function(req,res,nex){
//dostuff
});

@Keepertje Thanks mate!


build-sys: drop go1.20 support

to limit it to three Go minors

Summary by CodeRabbit

New Features

Updated supported Go versions to 1.21, 1.22, and 1.23 for enhanced compatibility.

Bug Fixes

Adjusted tool versions for linting and code analysis, potentially improving code quality and performance.

Documentation

Updated Go version in project documentation to reflect the latest changes.

Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)

Makefile (1)> 18-18: LGTM: Updated REVIVE_VERSION to support Go 1.21+

This change is consistent with the update made to GOLANGCI_LINT_VERSION and aligns with the PR objective of dropping Go 1.20 support. The update ensures that the Revive tool version is compatible with Go 1.21 and newer.
Consider extracting the Go version (1.21) into a variable at the top of the Makefile to make future updates easier. For example:
GO_VERSION ?= 1.21

GOLANGCI_LINT_VERSION ?= $(shell $(TOOLSDIR)/get_version.sh $(GO_VERSION) v1.59 v1.61)
REVIVE_VERSION ?= $(shell $(TOOLSDIR)/get_version.sh $(GO_VERSION) v1.4)

This way, you only need to update the GO_VERSION variable when upgrading to a new Go version in the future.

📜 Review details

I think this obscures the link between the base go version and the chosen tool versions