Crash when open project [Linux, Flatpak, 2.0.dev3]

Describe the bug
Godot is crashing after I have added the addon and restart godot and try to open my project with Orchestrator addon 2.0.dev3. I have tried it with 1.1.0 and that version is working.
I have tested it with both Arch and Debian system in flatpak with the same result.
handle_crash: Program crashed with signal 11 Engine version: Godot Engine v4.2.1.stable.flathub (b09f793f564a6c95dc76acc654b390e68441bd01) Dumping the backtrace. Please include this when reporting the bug to the project developer. [1] /usr/lib/x86_64-linux-gnu/libc.so.6(+0x3ee80) [0x7f20c9597e80] (??:0) [2] std::__codecvt_utf16_base<char16_t>::do_unshift(__mbstate_t&, char*, char*, char*&) const (??:0) [3] std::ostream& std::ostream::_M_insert<long>(long) (??:0) [4] void Logger::_log<char const (&) [41]>(Logger::Level, char const (&) [41]) (??:0) [5] orchestrator::initialize_extension_module(godot::ModuleInitializationLevel) (??:0) [6] godot::GDExtensionBinding::initialize_level(void*, GDExtensionInitializationLevel) (??:0) [7] /app/bin/godot-bin(+0x440ebdb) [0x561ae98c9bdb] (??:?) [8] /app/bin/godot-bin(+0x3eeea43) [0x561ae93a9a43] (??:?) [9] /app/bin/godot-bin(+0x63d999) [0x561ae5af8999] (??:?) [10] /app/bin/godot-bin(+0x5bd83b) [0x561ae5a7883b] (??:?) [11] /usr/lib/x86_64-linux-gnu/libc.so.6(+0x2808a) [0x7f20c958108a] (??:0) [12] /usr/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0x8b) [0x7f20c958114b] (??:0) [13] /app/bin/godot-bin(+0x5cb3e5) [0x561ae5a863e5] (??:?) -- END OF BACKTRACE --
Expected behavior
Open Godot without crashing
Actual behavior
Crashing godot when starting project
How to Reproduce?

Download latest version of Godot on flatpak.
Create a new empty project.
Download and add orchestrator addon to the project
Rest Godot as requested

Godot full version
v4.2.1.stable.flathub [b09f793f5]
Orchestrator version
2.0.dev3
Additional information
No response

Hi @GhostNr1, this seems to indicate that Godot is unable to write to the user:// path on your system, likely due to some permission or SELinux issue.  You can read about various paths in Godot here.
In short, we write a file to user://orchestrator.log and if this path cannot be accessed by the plug-in then you will experience this failure.  Please check your file permissions and security settings and verify this path can be written to.

Hi @GhostNr1 I'll have a fix attached in PR #145 shortly, when I attach that could you try that new build?

Hi @GhostNr1, here is an arch-linux fix (only contains updated Linux libraries).  Can you please test this change and let me know if it works.  Thanks!
godot-orchestrator-plugin-archlinux-fix.zip

I will only confirm that it's working without any problem. Thanks for the fast fix


Is there a upgrade plant for js-controller >= 2.0.0?

Hi,
some adapters need version >= 2.0.0 of js-controller. Is there a plan to upgrade this in the image?
Thanks and regards
PS: Thanks for providing this image.

As js-controller is part of the ioBroker and located inside the ioBroker folder you have to update it manually inside your container:
https://buanet.de/knowledge-base/steuerung-iob-ueber-kommandozeile/
Regards,
André
btw: if you download the actual image and set up a fresh & empty new container, js-controller will already be > v2.0.0...


Is CIS(D) available?

Dear developers,
First, thank you for the great program!
What I'm trying to do is TD-DFT with double hybrid functionals and found CIS(D) calculation is needed.
Is CIS(D) already coded and available? or is it on the TODO list?
Thank you!

@nesquik91 CISD code is already available, here's an example to get you started.

@jamesETsmith CIS(D) is a perturbative correction to CIS; CISD is a different method.
See https://doi.org/10.1016/0009-2614(94)00070-0 and https://doi.org/10.1016/0009-2614(95)01111-L

It should only be a few-line snippet to add the (D) correction. After CIS,
one would need the ground-state MP2 amplitudes, some ERIs that are not
calculated by CIS, and a couple of einsums. I can try to write this in the
next few days if desired.
On Thu, Jun 3, 2021 at 9:16 AM Susi Lehtola @.***>
wrote:

@jamesETsmith https://github.com/jamesETsmith CIS(D) is a perturbative
correction to CIS; CISD is a different method.
See https://doi.org/10.1016/0009-2614(94)00070-0 and
https://doi.org/10.1016/0009-2614(95)01111-L
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/967#issuecomment-853860853, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC3SXHVA52P2QKXZKVTTQ56DLANCNFSM46AT5JCA
.

Apologies for the mix up, I misunderstood the initial question

@susilehtola Thank you for the explanation!
@tberkel It would be really great if you can code it because CIS(D) is essential for performing TD-DFT with double hybrid approximation!

I noticed this thread some time ago and, without remarking on the effectiveness of CIS(D) as a method, thought to add some comments.
I don't have access to the original paper but found this which outlines the method which I'll reproduce below to make concrete what @tbekel said.

So firstly, it uses (pq||rs) rather than the more usual <pq||rs> but from the Physicist definition of MP(2) given below

You can see that the aabij are the same (denominators have opposite signs) so (pq||rs) = <pq||rs>. So once you've got the physicist's eri's and the orbital energies you can do the following sort of thing to get the  aabij tensor.
eps = np.concatenate((e,e), axis=0)
eps = np.sort(eps)
n = np.newaxis
o = slice(None, nocc)
v = slice(nocc, None)

dd = 1.0 / (eps[v, n, n, n] + eps[n, v, n, n] - eps[n, n, o, n] - eps[n, n, n, o] )
#a tensor
a = -g[v,v,o,o]*dd

The bao matrix comes from solving the CIS equations (RPA Tamm-Dancoff). pyscf has a solver for this which will return the matrix of expansion coefficients in (nocc, nvir) form.
The uabij tensor is I think
#u tensor
u  = np.einsum('abcj,ci->abij', g[v,v,v,o], b, optimize=True)
u -= np.einsum('abci,cj->abij', g[v,v,v,o], b, optimize=True)
u += np.einsum('kaij,bk->abij', g[o,v,o,o], b, optimize=True)
u -= np.einsum('kbij,ak->abij', g[o,v,o,o], b, optimize=True)

The rabij tensors
#r tensors
rab = -np.einsum('jckb,cajk->ab', g[o,v,o,v], a, optimize=True)
rij = -np.einsum('jakb,abik->ij', g[o,v,o,v], a, optimize=True)

and the wai tensor
#w tensor
w = np.einsum('jkbc,acik,bj->ai', g[o,o,v,v], a, b, optimize=True)

Finally ECIS(D) is
#CIS(D) contribution
e_cisd =  np.einsum('ai,bi,ab->', b, b, rab, optimize=True)
e_cisd += np.einsum('ci,cj,ij->', b, b, rij, optimize=True)
e_cisd += np.einsum('ai,ai->', b, w, optimize=True)

d_abij_omega = 1.0 / (eps[v, n, n, n] + eps[n, v, n, n] - eps[n, n, o, n] - eps[n, n, n, o] - eCIS[i])
e_cisd -= 0.25 * np.einsum('abij,abij,abij->', u, u, d_abij_omega, optimize=True)

I think this all that's needed. You will be aware that there are a plethora of related methods CIS(D)-RI, spin-component scaled versions etc of which the resolution of the identity is the most popular. CIS(D) is also the last in a series of EOM-CCSD approximations - EOM-CCSD -> EOM-MBPT2 -> P-EOM-MBPT2 -> CIS(D). There's a nice paper by JJ Goings, M Caricato,  MJ Frisch3, and Xiaosong Li that explains this.
In TDHF full RPA methods have non-symmetric matrices to solve, leading to left and right eigenvectors which need to be bi-orthonormal - which can be a pain. The Tamm-Dancoff Approximation avoids this and as CIS(D) is founded on the TDA so does it.  If anyone has access to a program that does do CIS(D) then it would be good to have some test numbers to check any attempts to program it (say water STO-3G). Hope this helps.
PS What's the best (easiest) way to get Physicist's double-bar integrals from pyscf? They're pretty ubiquitous but seem hard to produce?

Here's the original paper by Martin Head-Gordan et al.
I have an implementation of CIS(D) in python that can be easily scaled to multiple cores.
If anyone's still interested, I can try to find a way to add it the codebase in pyscf.

A PR for CIS(D) would be very welcome! If possible, please try to follow
the PySCF coding style. Because the CIS code (e.g., in pyscf/tdscf/rhf.py)
does not use ERIs, the MP2 or EOM-CCSD codes are probably good examples.
On Mon, Apr 11, 2022 at 12:48 AM Vijay Mocherla @.***>
wrote:

Here's the original paper by Martin Head-Gordan et al
https://www.sciencedirect.com/science/article/pii/0009261494000700.
I have an implementation of CIS(D) in python that can be easily scaled to
multiple cores.
If anyone's still interested, I can try to find a way to add it the
codebase in pyscf.
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/967#issuecomment-1094547536, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC5IOLSMBLUQ7XTI2XTVEOVIZANCNFSM46AT5JCA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I've looked into the cc module to get an idea about the coding style and the PySCF API.
A few questions:

Would ci be a good place for all code on CIS(D)?
Apart from perturbative doubles corrections for energies of CIS states, are there any other features that need to be coded?(At the moment only the theta diagnostic comes to my mind.)
I've seen calls for function lib.einsum() and numpy.einsum() in a few place across the code, are there any major differences between the two, that calls for preferentially using one over the other in certain cases? The former seems to be a more efficient way for tensor contractions. For example, an (https://github.com/pyscf/pyscf/issues/1132)

I might actually suggest the tdscf module, since the correction will always
be applied to a CIS type calculation. In fact, the (D) correction could
possibly be a simple method of the tdscf class.
I agree about theta diagnostic, but it’s not a priority (we don’t even have
it for CIS to my knowledge).
lib.einsum is safer to use.
Thanks again!
On Thu, Apr 21, 2022 at 4:21 PM Vijay Mocherla @.***>
wrote:

I've looked into the cc module to get an idea about the coding style and
the PySCF API.
A few questions:

Would ci be a good place for all code on CIS(D)?
Apart from perturbative doubles corrections for energies of CIS
states, are there any other features that need to be coded?(At the moment
only the theta diagnostic
https://www.sciencedirect.com/science/article/pii/S0009261497010282
comes to my mind.)
I've seen calls for function lib.einsum() and numpy.einsum() in a
few place across the code, are there any major differences between the two,
that calls for preferentially using one over the other in certain cases?
The former seems to be a more efficient way for tensor contractions. For
example, an (#1132 https://github.com/pyscf/pyscf/issues/1132)

—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/967#issuecomment-1105720384, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC2XMCUF3SLKFZN6XPDVGG2GDANCNFSM46AT5JCA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Primordial Jade Winged-Spear must be in standart and weapon banner

I think you forgot about that spear lol.

nice 😅 thanks for your attention, I forgot to add slingshot to the Standard list too.
So, both weapons have added in this commit ba8a37ea208de0f15e6db879c2306243b93bb803 . I wish it works right now


Only one sensor

Since version 1.31 its shown as one sensor of temp with info about humidity inside. In version 1.30 its shown as two different sensor. Maybe you correct this. (iOS 13.2.3)

With version 1.3.1, I changed the behaviour as with iOS 13 it was no longer showing the temperature on the main screen. Current version shows the temperature

On Dec 5, 2019, at 11:09 AM, acoolax<EMAIL_ADDRESS>wrote:

Since version 1.31 its shown as one sensor of temp with info about humidity inside. In version 1.30 its shown as two different sensor. Maybe you correct this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I have iOS13 with version 1.30. Everything showing fine.

Pls update and try again


when fixing usings, return namespaces associated with ambiguous result

the potential using statements are added to the Text of the QuickFix response, allowing the user manually choose which using statement to employ.

Hi Ravi,
I did add a test addressing the scenario in question.
I use omnisharp-vim with omnisharp-roslyn as a dependency. When I run the fix usings  command, sometimes I get a response indicating there is an ambiguous reference. This PR would update the message that comes back to not only indicate an ambiguous reference, but also indicate the ambiguous namespaces. That way, I as the user can decide which reference I want to use.
If you take a look at the commits you should see a test addressing this scenario.
Thanks,
Tolu

From: Ravi Chande<EMAIL_ADDRESS>Sent: Monday, April 30, 2018 9:24:19 AM
To: OmniSharp/omnisharp-roslyn
Cc: Tolu Adesegha; Author
Subject: Re: [OmniSharp/omnisharp-roslyn] when fixing usings, return namespaces associated with ambiguous result (#1169)
@rchande requested changes on this pull request.
Thanks for the PR!
I'm having trouble understanding the scenario. Can you
add applicable test(s) so that we don't regress this, and to help me understand what's being added?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/OmniSharp/omnisharp-roslyn/pull/1169#pullrequestreview-116327287, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIW3CFFoNUVNvc5i95Uj-Mvs4-poZrDzks5ttyyjgaJpZM4Tex8W.

@tadesegha I'm sorry, you did add a test. I missed it since it was at the bottom of the diff. I do think there are some issues with the test you modified. Mind taking a look? I'm happy to help out if you have any questions.

@rchande the last line of the test 'await AssertUnresolvedReferencesAsync(content.Code, expectedUnresolved);' has assertions. If you update the test without updating the code you should get a failing test.


Add documentation on how to use CallStackTree with atos

I like the repo so far, it seems like you've gone the farthest trying to force unwrap (pun) the json data of the MXDiagnosticPayload.
Have you had any luck getting atos to work with the output of the call stack tree? I didn't see any updates to this repo with corresponding documentation.
From Apple's documentation here: MXCallStackTree.jsonRepresentation()
It was very briefly mentioned in the "What's new in MetricKit" from WWDC 2020 that you should use the callStackTree stuff with atos.
I assume that translates to then:
atos -arch  -o /Contents/Resources/DWARF/  -l
Where
 is the diagnostic metadata's .platformArchitecture
 is the 'offsetIntoBinaryTextSegment' from the StackFrame
 is one or more 'address' from the StackFrame
This is what I get, which doesn't seem to be right:
[MACHINE:Products/Applications/MetricKitDemo.app] username% atos -arch arm64e -o ~/Desktop/MetricKit2/archive.xcarchive/dSYMs/MetricKitdemo.app.dSYM/Contents/Resources/DWARF/MetricKitDemo -l<PHONE_NUMBER><PHONE_NUMBER>
atos[23823]: respawning is disabled (because DYLD_ROOT_PATH or DT_NO_RESPAWN is set), but the analysis process does not match the SDK variant of the target process 0.
Analysis of malloc zones may fail.
4377106860
[MACHINE:Products/Applications/MetricKitDemo.app] username%
I have no clue how to resolve the respawning message here.

Hello! First, I'm sorry for the delay responding, just caught me at a bad time.
I'm glad you're finding it of some use. I actually have not used the MetricKit output with atos directly myself, but it should definitely work. I always find it confusing to determine which addresses/offsets to use for symbolication, and MetricKit adds to the complexity by omitting a value that is normally provided by a crash report.
The -l flag for atos is the binary image load address. This is for convenience, as normally you know the load address and the absolute addresses of stack frames. While MetricKit does provide the absolute frame address, it does not provide the binary load address. However, it does provide offsetIntoBinaryTextSegment value.
offsetIntoBinaryTextSegment = absolute frame address - load address

So, you should be able to use atos by just omitting the -l flag, since MetricKit has done this math for you.
One other note. MetricKit does not provide architecture per-binary. A typical application builds with arm64, but all of Apple's binaries are built arm64e. These can be mixed and matched, but when using atos, the architecture name will match exactly. You can verify which architectures are within your binary with dwarfdump -uuid. That will print out both the uuid and architecture name, which can be really handy.
As for the warning output from atos, I'm not sure :( I guess it might be related to the architecture mis-match, but I think that's a long shot.

I thought atos used the offset into the binary, not the offset into the text segment, is that not true? And if it is, would we need to calculate the distance between the start of the binary and the text segment?

Yeah, I always get tripped up by this too. I believe the Mach-O load commands from the binary result in an in-memory layout that is not the same as just putting the binary directly into memory. Symbolication tools like atos take this into account. However, it's been a while since I've looked at this exactly, so I might have the details wrong. But, the take-away is these values can be used to symbolicate successfully without too much thinking.

@mattmassicotte would you by any chance have a combo JSON file + the binary it was for so I can try symbolicating myself? E.g., you posted this JSON before, so just that sort of file + a matching binary/dSYM

@michaeleisel I'm afraid I no longer have the artifacts needed for that. But, if you're starting to experiment with MetricKit, this is pretty straightforward to do yourself, since you'll need to force a crash anyways. If you run into problems, let me know and I'll try to help!

Yeah, any ideas what the issue is with this code? Shouldn't it trigger the diagnostic payload to be delivered?
@implementation AppDelegate

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions {
    // Override point for customization after application launch.
    [[MXMetricManager sharedManager] addSubscriber:self];
    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(5 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
        // Triggers a crash that should(?) result in a diagnostic payload
        abort();
    });
    return YES;
}

- (void)didReceiveMetricPayloads:(NSArray<MXMetricPayload *> *)payloads
{
    NSLog(@"------ METRICS");
}

- (void)didReceiveDiagnosticPayloads:(NSArray<MXDiagnosticPayload *> *)payloads
{
    NSLog(@"------ DIAGNOSTICS");
}

Cool, got it now. If we look at the stack frame starting at line 23 in the JSON, we see an address of<PHONE_NUMBER> and offsetIntoBinaryTextSegment of<PHONE_NUMBER>. If we run atos for either of these addresses with -l omitted, or with -l 0x0, it fails to symbolicate it. This makes sense to me because these values are well past the end of the binary. If instead I use -l <offset> and address for the address, I get a decently close symbol (AppDelegate.m:26, which is 5 lines too much, but maybe I shifted things around a bit). Full command: atos -l 0x100c68000 -o metrico-dsym 100c6dd6c. We can also see that offsetIntoBinaryTextSegment is the same between the two stack frames from metrico. So, it appears that that's the load address of the binary.
.zip with JSON, binary, and dSYM binary

Hang on a sec... for crash diagnostics, it seems that's the case. But for the one CPU exception diagnostic report I'm looking at, the offsets are much smaller, like you could indeed pass them without a load address to atos. So, depending on the exception type, there's different behavior 🤯

I apologize @michaeleisel. I missed the notifications for these messages. I'm really sorry I left you waiting for so long...
I will admit that I have not looked closely at non-crash diagnostics. But, I would be really surprised (and very annoyed) to learn that the address bookkeeping is different. I can definitely confirm that offsetIntoBinaryTextSegment is the binary load address, and must be used as the -l option to atos. This is how the now build-in symbolication system works for Meter.
I would be interested to investigate further, though!

Yeah, CPU exceptions, at least when I tested them when making this issue, don't add the load address in.

Hmm, ok I'm going to pay more attention to this. Without a load address, or a way to derive it, symbolication is impossible.

IIRC symbolication is possible by giving it a load address of 0

Ok, I managed to track down some real diskWriteExceptionDiagnostics data.  In these cases, offsetIntoBinaryTextSegment is very small and cannot be a load address. So, I'm inclined to believe that offsetIntoBinaryTextSegment is probably actually an offset here.
So, no, 0 won't work, but the load address calculation will now actually be address - offsetIntoBinaryTextSegment.

And found some hangDiagnostics too, which also look like they have real offsets. Sounds like this weirdness only affects crash diagnostics.

The library now has a bunch of built-in support for symbolication, including load address calculations. And, making things more complex, Apple fixed the offsetIntoBinaryTextSegment differences in macOS 13/iOS 16. There's now logic there to account for this too.
Hope this helps resolve everything! If there's more trouble, I'd be happy to help.

@mattmassicotte did you get a feedback response from Apple that it was fixed in iOS 16, or your code started to work so you inferred it was fixed?
I ask because I'd like to know if it was explicitly stated fixed in iOS 16 so we can assume it won't be fixed in older versions.

@mattmassicotte got the feedback number handy? I play the game of posting feedback numbers in the developer forums and it might be worth posting about this in that community or here for other devs to see and put it in theirs so apple can link as common issue.

@SwiftNativeDeveloper I like this game! It is FB9160176.

In case this is helpful, I've implemented symbolication of MXDiagnosticReport json files using atos in a Python script: https://github.com/OliveTreeBible/MXSymbolicate


[Feature] falco-ignore

Fastly VCL supports some hidden features such as map and callback that are undocumented. Falco does not support these functions which causes the lint to fail every time. I think it could be beneficial to include some way to tell Falco to ignore the next line, sub-routine, or file.
For example, if you use the prettier linter for JS, you can tell prettier to ignore the next line like such:
// prettier-ignore
let positiveInts = test.map((value) => {
    if (value > 0) {
        return value
    }
})

We could potentially do the same with Falco:
// falco-ignore
map(var.test, test_callback, ",");

This can also act as a bandaid to any unexpected bugs a developer might run into while using Falco. If a developer is experiencing some other linting issues that is causing their lint to fail, they can always tell Falco to skip the next like, sub-routine or file to resolve said issues until the linting bug can be patched.

Yes, long-time I'm thinking about it.
We'll try to implement that is able to ignore or suppress the error reporting mechanism.

huge +1 on this one

I would like to discuss a proposal for implementing this feature with you here and then I can take it to implementation.
Feature's Acceptance Criteria

2 levels for ignoring will be introduced:

On the subroutine level
On the statement Level

This feature shouldn't interfere with the current semantic comments that falco has. To be more specific: Fastly macro comments and Scope comments
Tokens should be parsed even with ignore in place. Ignoring's effect should only be during the linting phase.
(optional) Show what linting errors being ignored

High-level implementation
Introduce 2 new comments:

--falco-ignore: Don't lint the next statement!
--faloc-sub-ignore: Don't lint the next subroutine block

Inside the declaration of the subroutine and the statement, we will add a new field that indicates if any of them have these comments set! - Or we can use the infixComment - but I need to double check the impelementation of this comment.
With these two new comments, we introduce two new Flags to Context:

falcoIgnore: false
falcoSubIgnore: false
setting and unsetting the values for these 2 flags will happen in 2 places:
lintSubroutineDeclaration
lintBlockStatement

Suppressing the errors will happen based on the value of these flags.
So what do we think here? I would love to hear your feedback and your ideas!

Tokens should be parsed even with ignore in place. Ignoring's effect should only be during the linting phase.
(optional) Show what linting errors being ignored

This makes me wonder if we should allow the user to specify which errors / warnings they would like to ignore for a statement or sub-routine? Similar to ESLint or prettier? For example, with ESLint, I can ignore the next statement for a specific rule like such:
// eslint-disable-next-line no-use-before-define
var thing = new Thing();

Thanks for feedback
I am proposing another solution and hope to get some feedback from you:
Acceptance Criteria:

//falco-ignore-subroutine will ignore linting errors in the next subroutine
//falco-ignore-next-line will ignore linting errors in the next line
//falco-ignore-start -> //falco-ignore-end will ignore linting errors between two lines
All the comments above take a space-separated list of FalcoError where users can specify which errors to ignore
//falco-ignore-subroutine invalid-type undefined-variable
This feature shouldn't interfere with the current semantic comments that Falco has. To be more specific: Fastly macro comments and Scope comments.
Without a list of errors to ignore, all errors will be ignored.

API Defination
package context

type FalcoError string

const (
	InvalidNameErr                   FalcoError = "invalid-name"
	InvalidValueErr                    FalcoError = "invalid-value"
	InvalidTypeErr                     FalcoError = "invalid-type"
...
)

// falcoIgnore keeps track of Falco Ignore state during execution.
type falcoIgnore struct {
	ignoreNextSubroutine bool
	ignoreNextLine       bool
	ignoreRange          bool
	errorsToIgnore       map[FalcoError]bool
}

// newFalcoIgnore creates a new instance of Falco ignore.
func newFalcoIgnore() *falcoIgnore {
	return &falcoIgnore{}
}

// isEnabled checks if falco ignore is enabled currently.
func (fi *falcoIgnore) isEnabled() bool {
	return fi.ignoreNextSubroutine || fi.ignoreNextLine || fi.ignoreRange
}

// reset sets falco ignore state to initial state.
func (fi *falcoIgnore) reset() {
	fi.ignoreNextSubroutine = false
	fi.ignoreNextLine = false
	fi.ignoreRange = false
	fi.errorsToIgnore = map[FalcoError]bool{}
}

// addToErrorsToIgnore adds new error for falco ignore for falco to ignore it.
func (fi *falcoIgnore) addToErrorsToIgnore(e FalcoError) {
	fi.errorsToIgnore[e] = true
}

// shouldIgnoreError checks if falco should ignore the current error.
func (fi *falcoIgnore) shouldIgnoreError(e FalcoError) bool {
	_, ok := fi.errorsToIgnore[e]
	return ok
}

Implementation
Context will have an instance of falcoIgnore where its state is gonna change based on the comments above.
LintError will have a new field called FalcoError that defines the current error object and help us to check if we need to ignore it or not!

@shadialtarsha I think this makes a lot of sense! I have not looked at the code very closely for this project, so I don't have a strong opinion about the API definition, but I think this is great! :)

@shadialtarsha
Finally, I've started to think about this, and thank you for your proposals 👍
Almost I agree with you, but just one thing we need to consider, @kevin-fagan says:
// falco-ignore
map(var.test, test_callback, ",");

The map should be ignored on the parser level, not the linter level.
Currently, I have no idea about that. Probably we should implement some kind of functions that come from Varnish.
This is because ignoring parser errors will occur unexpected behavior, for example, a user also could ignore the Syntax Error!
Therefore, we should support to ignore parser error with deep carefully


Review cilium 1.0 datapath implementation

Parent: https://github.com/cilium/cilium/issues/2594
Cilium API policies implemented by BPF datapath in Cilium 1.0-rc6
Actual behaviour as at 2018-03-02
This issue attempts to capture the datapath-level perspective on how policy is enforced in Cilium as of https://github.com/cilium/cilium/tree/v1.0.0-rc6 , so that as we review the network policy model (#2594) we can plan for necessary datapath changes.
Interpretation Notes:

Rule name column is provided for discussion reference.
Each row represents a single rule.
toServices is translated into toCIDR in the datapath, and in the table below.
fromEndpoints / toEndpoints is represented as fromIdentity / toIdentity in the below table.
fromEntities / toEntities is represented as fromIdentity / toIdentity in the below table.
toPorts matches destination port range in UDP/TCP after service translation is applied.

Ingress

Rule
fromIdentity
fromCIDR
toPorts
Supported?

Requires L4 lookup as part of the CIDR map lookup
Ingress toPorts is unsupported, however similar behaviour may be supported via "Label-dependent L4" with a wildcard endpoint selector.
Egress toCIDR is only applied to traffic with destination IP that:

Does not match a local endpoint IP
Does not match a remote node destination prefix
Does not match the cluster prefix

Implications on datapath design

Supporting fromEndpoints AND toPorts requires both of these to be part of
the same lookup. This is the case for ingress and for planned egress
label-based policy.
Similarly, to implement toCIDR AND toPorts, the ports must become part of
the lookup.
If we reject containing fromEndpoints and fromCIDR from the same policy,
then the datapath can retain separate maps for these, which for the
identity(+L4)-based map may be a plain hash lookup, while for fromCIDR it
may be an LPM
If we add another policy stage before the service translation, then every
constraint we intend to enforce at the same time must be part of that lookup.

Follow-up items

[x] Check that rules C, F-K, M are rejected at API layer (#3015)
[x] Update documentation to refer to label-based l4 as such, not layer 3 dependent

Allows us to refer to label-dependent or CIDR-dependent and have that make sense

[x] Document where CIDR policies may be applied

fromCIDR only makes sense at ingress
toCIDR only makes sense at egress

[x] Document and/or validate API for fromEntities  + toPorts

Discussion

Policy constraints that apply at the same time as other policy constraints
must be part of the same lookup. For example, to implement Labels+L4, both
are part of the same BPF map.
There's a proposal to introduce new services-specific policies, so the above
table would add another column on egress for allowing destination services.
Initial implementation would likely only support in combination with L4
ports. This could be implemented by embedding such policy into the existing
service translation lookup.

Why do we support B but not I?

@aanm I depends on label matching, which is not (yet) available on egress.

@joestringer so we do have plans to support it

@aanm not currently. Issue #3005 actually proposes to restrict both B and I. Given that the current behaviour of B can be achieved by just splitting the fromEndpoints and fromCIDR parts of the rule up into separate rules, I think we don't lose anything from this.
In future it may be interesting to implement I, so that for example you could combine toServices with toEndpoints and ensure that traffic to particular labels are only reaching them via a particular service, however this would only be useful in situations where the service translation occurs within Cilium.  In a few k8s cases for instance, the service constructs are implemented outside so we wouldn't be able to enforce such a policy.

I've addressed the "follow-up" items and I think that this issue has served its purpose for discussion so I will close this issue now.
Note that several things have already changed since the issue description, for instance we now reject unsupported combinations of matches in the rules.


How to add a select field as a column in table

I am trying to add a column in material-ui table with selectfield. https://github.com/narendravenkata/React_Form. this is the code link. I am unable to set the values and update in edit mode.

[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Current Behavior

Steps to Reproduce (for bugs)

Context

Your Environment

Tech
Version

Material-UI
"^0.18.6"

React
"^15.6.1"

browser
chrome

etc

We have been porting the component on the v1-beta branch. We have reimplemented it from the ground-up.
We encourage people starting with Material-UI to go with v1-beta. The master branch is in a low maintenance mode. We don't plan any work on it. However, we still do our best to review and merge pull requests that fix bugs on this branch.


Debootstrap Error Still...

I Used Drinkcats Methods To Fix It And Now When I Try Downloading I Get
I Running The Command
chronos@localhost / $ sudo sh ~/Downloads/crouton -r vivid -t xfce-desktop,audio,core,x11,keyboard,extension -n xfce-main
Installing vivid-armhf chroot to /usr/local/chroots/xfce-main
Downloading debootstrap 1.0.75...
2016-01-25 06:55:57 URL:https://anonscm.debian.org/gitweb/?p=d-i/debootstrap.git;a=snapshot;h=1.0.75;sf=tgz [63296] -> "-" [1]
Patching debootstrap...
Downloading bootstrap files...
I: Retrieving Release
I: Retrieving Release.gpg
I: Checking Release signature
E: Error executing gpgv to check Release signature
Please Help ASAP

Do you get an error trying to install trusty? I had last nights error as well, now resolved and i have Trusty installed with XiWi?

same with trusty

i wiped the install cache and had no chroots to delete. Why isnt it working for me

Can you do sudo sh ~/Downloads/crouton -t xfce?

(i think that defaults to precise)

nope still same error

This Is The Error

mayby in trying to fix last nights error you caused another one? I would try deleting the crouton download, re downloading and trying again.
Failing that, powerwash.

Alright, can we not start any centithread here? This is not a chat, nor a random discussion forum. If you want to have us looking at your issues, please stay on the topic, and limit the chatter to the necessary. Thanks!

Your Fix Has Not Fixed My Crouton @drinkcat

In the snapshot, I see that your rootfs is writable, is that still the case after powerwash?

no

wait i mean yes

my boot partition is selected

Do you get the "WARNING: Your rootfs is writable" warning, or not?

yes

alright, do you have any idea what you did to the rootfs? If not I suggest that you follow the advise on the following line.

no, i just enabled it to see what it would do, but it did nothing

Not even this? https://github.com/dnschneid/crouton/issues/2387#issuecomment-174339655

Anyway, just recover from USB. https://support.google.com/chromebook/answer/1080595?hl=en

Recovering from USB will definitely fix this particular problem of yours.

@drinkcat, @dnschneid,
This has me a little concerned, do the recent fixes preclude having your rootfs r-w ?
Just wondering because I put things in /etc/init to mount CROUTON and autostart chroots.

Nope, OP just moved a file (the keyring) into a place that confused debootstrap.

Okay, thanx. Sorry for the unwarranted panicking. ;-)


v7.6.5 Cannot Install on Mac Catalina

Is it possible to get a build working for Mac Catalina?  The latest dist v7.6.5 does not seem to be working after install.
I tried
kextstat | grep intel
and see nothing.
Furthermore, the android emulator gives no indication that HAXM is being utilized.
Thanks for all your work that you have already put into this project!

Since macOS Catalina 10.15, Kernel Extension is no longer supported to be installed by Apple. Instead, it is replaced by System Extensions, which requires to rewrite the driver framework. We are evaluating the workload. Please use HVF instead during this period. Sorry for bringing you the inconvenience, thank you for your attention to HAXM.

Hi @wcwang what's HVF ? Could you provide a link please ?

@Mouradif, thanks for your question. The QEMU HVF feature can be found here. Currently, HVF is the default hypervisor for macOS, you may try to remove the parameter of '-accel hax' from the command line to launch QEMU or add '-accel hvf' instead. Thanks.


Added ColorMaximum and ColorMinus property in NumericUpDown control

In addition to resolving the comments, I have added two new properties: MinimumTextColor and MaximumTextColor to be able to also change the text color of the Minimum and Maximum.

Thanks!

In addition to resolving the comments, I have added two new properties: MinimumTextColor and MaximumTextColor to be able to also change the text color of the Minimum and Maximum.

Thanks!


improve the instruction in qasm simulator

Informations
In the Dependencies section of README (https://github.com/Qiskit/qiskit-terra/blob/master/src/qasm-simulator-cpp/README.md#dependencies), the command to install the dependency should be

make depend

not s at the end.

Qiskit Terra version: the master branch
Python version: 3.6
Operating system: Ubuntu 16.04

thanks this is a bug.

I will push a quick patch to fix the typo.
But we might end up having to update the doc after we change the build process (see #1056) I'm not sure that it will change that depend make target (well really the underlying build_dependencies.sh script), but the script also isn't super complicated just calling a package manager to install blas, etc.


fix: fixed nextjs/react hydration error when using portals

Overview
Currently, while using nextjs, if the page will have an evergreen component which uses portal that is visible (isShown={true}), during initial render (page load) react will complain about failed hydration:
Unhandled Runtime Error
Error: Hydration failed because the initial UI does not match what was rendered on the server.
See more info here: https://nextjs.org/docs/messages/react-hydration-error

This PR is my attempt to change the smallest amount code and following these resources:

https://github.com/vercel/next.js/blob/canary/examples/with-portals/components/ClientOnlyPortal.js
https://github.com/tajo/react-portal/issues/217

Side note:
This doesn't actually render portal content on the server-side. Portal initiation happens on the component mounts on client.
As far as I googled, there is no solution to have portals properly rendering on server-side. Some modules showcasing the idea, like this module. But they are not production ready, as they can leak portals across concurrent requests.
Documentation

[ ] Updated Typescript types and/or component PropTypes
[ ] Added / modified component docs
[ ] Added / modified Storybook stories

Hey @oxala, thanks for taking a look into this. I pulled this down and tried running it with your example repo, but I'm still seeing the same hydration error:

Is that expected, or should it be rendering properly on the client?

Pulled it down w/ your example repo and it seems to solve the issue. Going to do a little more smoke testing with this change before merging it in, but it makes sense based on those discussions linked. Thanks for fixing this up!


sub-project within a git repo cannot have custom .venv file

Due to the way these lines of code are written:
https://github.com/robbyrussell/oh-my-zsh/blob/master/plugins/virtualenvwrapper/virtualenvwrapper.plugin.zsh#L13-L26
It's not possible to have a Git repo which contains a subfolder with a .venv file in that subfolder containing the virtualenv for that sub-project.
For example, I have a Git repo named "backend". It has two subfolders, "analytics" and "crawl". Each has a separate venv associated with it; by convention, I name them backend+analytics and backend+crawl. However, I can't put a separate .venv file in each subfolder because the zsh code will "walk up" to the project root directory and look for the file there.

This is fixed by  #3918, which is now merged, and supersedes #2376. However, #3918 broke normal git repo venv detection, which is fixed by pending PR #4122. Could you test out #4122 and see if it works for you?

This can be closed


Update to fix listener errors

Fix for following error
Calling JS function after bridge has been destroyed

Thank you @waqas19921  I will publish after changing the variable naming


How to use event ?

hello,
I need to recover the event but I can not find it.
I tried :
this.$bus.$on('input', event => { console.log() });

<vue-datepicker-local @input="change"/>


tools: add support board plugin modules to fpgainfo

Dynamically load plugin module
Support MAC, PHY group commands
Fix Invalid Metrics value

Coverage decreased (-0.6%) to 80.186% when pulling ad94625b912916d64a5d5529f12cfef359af4d16 on aravuri/fpga_info into c9fd92cb6f352d623576423e0bef9a6ad9ea0115 on master.

I think your board libraries also need to be added to the rpm spec file.

Will you be adding your libs to the rpm spec here, or in another pull request?

I think your board libraries also need to be added to the rpm spec file.

Will you be adding your libs to the rpm spec here, or in another pull request?

i will fix in another PR


%datetime% still a no-go in 4.2

? : Alternative - Indie Pop Rocks! [SomaFM]
? : Real Estate - Water Underground
? : Alternative - Radio Paradise (128k aac)
? : Mathias Duplessy - Tcheren Deya
Also, still no "last-played first" option.  Example above is from streaming radio logfile.

Oops, forgot a nullptr - https://github.com/foxx1337/foo_nowplaying2/blob/514dd66a00694c6712d925c5ca42b18136a97b7d/src/nowplaying.cpp#L245


Unable to sync the gradle

Hi ,
I'm unable to sync the project because I'm getting Manifest merge error
Kindly check the error log
Manifest merger failed : Attribute application@allowBackup value=(false) from AndroidManifest.xml:15:9-36 	is also present at [com.github.MOLO17:CustomizableCalendar:v0.1.4] AndroidManifest.xml:12:9-35 value=(true). 	Suggestion: add 'tools:replace="android:allowBackup"' to <application> element at AndroidManifest.xml:13:5-160:19 to override.
Additional Info :  I can able to sync the gradle if I used android:allowBackup="true" in my app manifest. but I need to set allowBackup as false
Thanks in advance

Hi @rajasekaranm,
I've removed the allowBackup attribute on the manifest, you can test the v0.1.5 version.

yes @furlan97 , now working fine :-) thanks


[add] fucolor utility: colorize vars in futhark output

As a human, I find it easier to read Futhark output with (generated) variable names in colorz.
Works well in 256-colors terminals, with all kind of futhark outputs (C/py OpenCL, futhark IL, ...)

That's a fun tool.  Maybe it can be useful for debugging.


pip version missing code?

Hi,
I have ran pip install django-cors-headers and it reported that CorsPostCsrfMiddleware doesn't exist.
I opened files that pip installed and in deed CorsPostCsrfMiddleware class is missing. I have checked github repo, and normally there you have CorsPostCsrfMiddleware.
Then I went to
https://pypi.python.org/pypi/django-cors-headers
and saw that on last two versions available up there there is some missing code.
But when I do pip install git+git://github.com/ottoyiu/django-cors-headers.git all goes good
Can you help what is going on?

New release is required ;)

+1 if this is still required.

I think that is still needed. Somebody can verify this, I don't have my python project close?

@popara I checked earlier last week and it seemed like the last release had the middleware class.

Let's close it then! :)


safe silently fails when given a bad target

A user gave safe target the URL of the vault instance (https://<IP_ADDRESS>:8200), instead of the name of the vault as defined in their ~/.saferc. That was accepted, and updated the current: line to be the URL. They were then able to successfully authenticate, but it did not add a token to the file. I verified that if they put in a bad password, they would get an error, so they were definitely hitting the vault server they had targeted. But trying to use safe didn't work, because of the lack of token.
Example ~/.saferc after running the "bad" safe target and a "successful" safe auth ldap:
version: 1
current: https://<IP_ADDRESS>:8200
vaults:
  aws-vault-1:
    url: https://<IP_ADDRESS>:8200
    token: ""
    skip_verify: false

Expected behavior: safe target warns about targeting an undefined vault, and doesn't get into a weird halfway working state.

I think the problem here is that your LDAP auth did not work.
A while back we deliberately made changes to support target by alias or URL, so that automated systems didn't could co-exist with user-managed aliases.  The bots target by URL and the people by name.
I just validated this as follows:
→  safe local -m
Now targeting (temporary) inviolable-castle at http://<IP_ADDRESS>:8201
This Vault is MEMORY-BACKED!
If you want to retain your secrets be sure to safe export.
Ctrl-C to shut down the Vault

Then, in another console:
→  safe targets

Known Vault targets - current target indicated with a (*):
    my--lab              (noverify) https://10.x.x.x
(*) inviolable-castle    (insecure) http://<IP_ADDRESS>:8201

→  safe set secret/proof of=identity
of: identity

→  safe tree
.
└── secret
    ├── handshake
    └── proof

→  safe target http://<IP_ADDRESS>:8201
Now targeting http://<IP_ADDRESS>:8201 at  (skipping TLS certificate verification)

→  safe tree
.
└── secret
    ├── handshake
    └── proof


How can specify a commit I want to use for a plugin?

There is a current bug in zsh-autocomplete that causes a freeze when typing a command, and the current workaround it is to revert back to a previous commit.
I see you are able to specify a branch for the plugin but not a commit hash.
Also, I cant seem to find any documentation as to where the git repos are cloned to to be able to manually switch the commit

DUplicate issue. https://github.com/mattmc3/antidote/issues/1

Here's how you can revert back to a specific commit until this feature is implemented:
# make sure antidote is loaded, and then set these vars to whatever you need
source /path/to/antidote.zsh
GIT_WORKING_SHA=cfc3fd9a75d0577aa9d65e35849f2d8c2719b873
GIT_DIR=$(antidote path marlonrichert/zsh-autocomplete)

# revert Zsh plugin managed by antidote to a prior SHA
git -C "$GIT_DIR" fetch --unshallow
git -C "$GIT_DIR" checkout $GIT_WORKING_SHA


Revert "Fix: update swap to match SQS return (#2991)"

This reverts commit 40c8b961206da5cea5dd30126549505e49bbe534.

What is the purpose of the change
revert https://github.com/osmosis-labs/osmosis-frontend/pull/2992
ClickUp Task
ClickUp Task URL
Brief Changelog

Testing and Verifying

Documentation and Release Note

Summary by CodeRabbit

Refactor

Renamed a key variable for clarity in token conversion calculations.

Chores

Restored tracking of sitemap.xml in version control for better web management.

New Features & Improvements

Enhanced the swap tool by simplifying the asset conversion details display and improving the logic for fetching and handling spot price quotes.
Improved error handling in the swap functionality for a smoother user experience.
Updated the asset selection logic in the swap tool for better performance and usability.

Did some tests to observe that the change is back to the original state. Will make a revert against master and seek more approvals


[question] can containerd be run on OSX?

Hi, All,
I found that in the released 1.0.0 containerd, we can download containerd-1.0.0.darwin-amd64.tar.gz which is for OSX operating system.

At my first glance, it is fun. Just double check if this is true.
Thanks
/cc @stevvooe

containerd itself should work, but no functional runtime plugin

No, the client code compiles but its not functional for OSX

@crosbymichael @AkihiroSuda What is the current status of containerd on OSX 3 years after this was closed?

It's not a blocker on containerd side. In order to support containers natively -  OS X's kernel must provide various container isolation primitives (like cgroups and namespaces on Linux), which we don't have today.
However there are other variations possible like this one: https://github.com/containerd/containerd/pull/4526

Where are we with https://github.com/containerd/containerd/pull/4526? There has been no movement since Jan 22. Is there something blocking this?

Keeping the thread here up to date.  The work for #4526 was split into #5395 and #5396.  #5396 was merged but #5395 appears to be stuck now.
We are really butting up on docker-desktop licensing killing productivity for mac users.  Solutions like rancher-desktop still need to run containerd inside a VM and it would be great to get away from that with native support.

if https://github.com/containerd/containerd/pull/5935 gets merged, is there full support for containerd on OS X or not really?

Wow! Waiting this bery much!

if https://github.com/containerd/containerd/pull/5935 gets merged, is there full support for containerd on OS X or not really?

Depends on what do you mean by "full support".
You can run containerd on MacOS today (same way as any other Go application).
You'd still need proper runtime to launch containers though.


Feature Request --force -f

Please add a --force flag to force task to run even if hash has not changed on a task with a @skip:key
$bake install
 + Executing @skip(key=requirements.txt):
 + Skipping install/python.
 + Executing install:
 + Done.

$bake install --force
 + Executing install/python:
  .
  .
  .
 + Executing install:
 + Done.

--dont-skip

Submitted PR https://github.com/kennethreitz/bake/pull/8


Cast unsuccessful

Atempting to work around problem 1221300 with the
following:
class t4 {
  public String answer = "Yes";

  public void w() {
    String answer = "No!";
  }
}

public class t4ext extends t4 {
  public String answer = "No!";

  public void w() {
    String answer = "Maybe";
    System.out.println("Is Nigel best? > " + ((t4)this).answer);
  }

  public static void main(String[] args) {
    t4 w = new t4ext();
    w.w();
  }
}

Process cat exited with code 0
Works in Java:
> javac t4ext.java
Process javac exited with code 0
> java t4ext
Is Nigel best? > Yes
Process java exited with code 0
BeanShell still hasn't read "How to make friends and
influence people":
/home/nigel/myBsh/Pitfalls/temp/> java -
classpath ../../bsh-2.0b4.jar bsh.Interpreter t4ext.java
Is Nigel best? > No!
Process java exited with code 0
Reported by: nigel_galloway
Original Ticket: "beanshell/bugs/220":https://sourceforge.net/p/beanshell/bugs/220

Can reproduce, hidden  fields not accessible via a cast to a superclass type.
In this example the call to ((t4)this).answer still returns this.answer.

parent #660

And if you change the reference to
    System.out.println("Is Nigel best? > " + super.answer);

it still does not work in BeanShell, so it is not only casting.  Or rather it is lookups not casting that is having the problem.

Some more tests:
--> void
(new t4ext()).answer

--> $2 = "No!" :String
((t4)new t4ext()).answer

--> $3 = "No!" :String

Resolved:
BeanShell 3.0.0-SNAPSHOT.3056
bsh % (new t4ext()).answer;
--> $7 = "No!" :String
bsh % ((t4)new t4ext()).answer;
--> $8 = "Yes" :String
bsh % t4ext.main({""});
Is Nigel best? > Yes
--> void


Add maxResultBuffer property

Implementation of new property - maxResultBuffer. Enables max bytes read during reading result set.
Enable declare in two styles:
• as size of bytes (i.e. 100, 150M, 300K, 400G);
• as percent of max heap memory (i.e. 10p, 15pct, 20percent).
It also validates if defined size isn't bigger than 90% of max heap memory. If so, it limits max bytes read to 90% of memory heap. If maxResultBuffer property isn't provided, driver work isn't changed.

Hi @sawkoj, can you give some points regarding why this feature is necessary? The JDBC Statement APIs provide a similar "memory" control option in setFetchSize and setMaxFieldSize. Although this doesn't directly allow you to control memory, the intentions are the same. Can you give a use case where your proposed feature would be significantly better than the existing JDBC apis?

Thanks for the contribution @sawkoj, sorry for the delay in response. I see that this change allows the user to set a limit to how much data can be read from a resultset, but in what situation will this change be useful? So far, users haven't asked for such feature because they can choose to stream the data instead of storing the entire resultset in memory, which I think is more flexible than the driver throwing an error if the amount of data read goes over the limit specified in this buffer.

Hi @rene-ye @peterbae , a purpose of the added property is to protect user from Out Of Memory Exception. Properties for “memory” control, which you have mentioned, won’t be enough in some cases. The most common case, I can think of, is when we have 2 queries which produce different result sets. Let’s call them A and B. A is large, contains thousands of rows, B’s size doesn’t matter. Upon executing A’s query, only first packet of A is kept in memory. We don’t iterate or do anything with A. Next, we call B query, to get B result set. In this moment, all remaining packets of A’s result set are being read to memory with first packet of B. Here OOM can occur and setting setFetchSize or setMaxFieldSize can prevent that.
Setting these limits can be very inconvenient though. Setting fetchSize changes amount of packets that need to be send to read whole resultSet. It could be a way of controlling of memory usage, but this approach is very error prone (you have to calculate the number of  bytes for single row from a given query). If any column in query/table changes, you have to recompute number of rows to fetch if you want to handle memory using setFetchSize. The similar issue applies for setting maxFieldSize. You depend memory control on size of a single column, on change of query/table, you have to take that in consideration in code.
By using maxResultBuffer you just put amount of memory that you want to be used and don’t worry about having OOM and stopping whole application.

Adding to this.  This work is being done as part of a larger project at Heimdall Data, which provides a proxy for a variety of database types.  We are working to bring consistent controls over memory usage across multiple driver types.  This feature has already been included in the Postgres JDBC driver (https://github.com/pgjdbc/pgjdbc/pull/1657), as well as a fix submitted to the Oracle MySQL jdbc driver (https://bugs.mysql.com/bug.php?id=100415), so our next target was the SQL Server driver.  To completely understand the need, don't think about one connection, but when you have thousands of connections active, and you need to be able to ensure there is enough memory for all of them.  @sawkoj explained one very specific case, but there are other cases where (I believe) adaptive fetching doesn't work, in particular if retrieving large result sets from tables with columnstore indexes.  Cursors are not allowed in this case, and can trigger oom problems.

Hi @sawkoj, I've been testing the PR and there is currently some undesirable behavior with the new feature. There are currently some scenarios where if the maxResultBuffer property wasn't specifically set, the property will default to -1, and when attempting to call getMultiplier while the property is -1, the driver will throw an error. New properties should be introduced with a default value where it won't impact the rest of the driver when either not specified or off. The error message also seems to need to work, currently I'm getting:
com.microsoft.sqlserver.jdbc.SQLServerException: Invalid syntax: [Ljava.lang.Object;@628190ec in maxResultBuffer parameter at com.microsoft.sqlserver.jdbc.MaxResultBufferParser.throwNewInvalidMaxResultBufferParameterException(MaxResultBufferParser.java:120)
The above error is easily reproducible if you run our unit tests provided in the repo, if you have any trouble setting it up, we can help with that. Some of the tests require special setup, and it's ok if you can't run them on your end, but the CIs should pass.

Hi @rene-ye, thank you for your feedback. According to your suggestions, I changed the default value of maxResultBuffer property to "-1", so from now it should be parsed correctly whether value for the property is specified or not. Also, I fixed the error message (I was unnecessarily packaging it as Array). I refactored the parsing logic, so from now it's gonna throw an error upon providing negative values, tests for negative values had been added as well. Of course I didn't forget to add missing break statements in AbstractTest.java.

Hi @ulvii, thank you for the feedback. According to your suggestions, I changed access modifiers in ICounter interface and IOBuffer class methods getCounter(), createCounter() as well. In MaxResultBufferTest I added an another private static field, which is used as local connection String. The original connection string is unaffected. Also I added more test cases to MaxResultBufferParserTest (valid and invalid ones). MaxResultBufferTest class was refactored too, I parameterized existing tests and I added different MaxResultBuffer property values to illustrate the change of behavior.

The PR looks good to me. Once the other team members have reviewed & approved, we will re-run the CI tests and merge the PR.

Hi @peterbae and @ulvii, thanks for feedback. Loggers were adjusted in MaxResultBufferParser and in MaxResultBufferCounter. I created additional resource: R_maxResultBufferPropertyExceeded when MaxResultBuffer is exceeded. Description of R_maxResultBufferPropertyDescription is updated too.


fix delta definition

#157

update delta definitions to match what is actually happening in the code

For reference:
https://github.com/dogfessional/react-swipeable/blob/ff4043f55b10a7472aed053e59a81bca9c7bf030/src/index.js#L78-L79


npx prisma -v on Windows returns: Error: Get config Error: Command failed with exit code<PHONE_NUMBER>

Hello!
When running "prisma -v" (version 2.16.1) on my cmd i get this error:
Environment variables loaded from .env
Environment variables loaded from prisma.env
Prisma schema loaded from prisma\schema.prisma
Error: Get config Error: Command failed with exit code<PHONE_NUMBER><EMAIL_ADDRESS>cli get-config --ignoreEnvVarErrors
error Command failed with exit code 1.
Update:
I also tried reinstalling prisma as a global package, still having the same issue.
When running "prisma -v" (version 2.16.1) on my cmd i get this error:
Error: Command failed with exit code<PHONE_NUMBER><EMAIL_ADDRESS>--version
I think this is something wrong on my WINDOWS machine cause my friend tried in MacOS and worked well.
Does anyone knows how to solve this issue?
System Information:
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.18363 N/A Build 18363
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free
Original Install Date:     1/17/2021, 4:05:55 PM
System Boot Time:          11/10/2020, 7:28:01 AM
System Manufacturer:       LENOVO
System Model:              20A8S05K00
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
[01]: Intel64 Family 6 Model 69 Stepping 1 GenuineIntel ~1895 Mhz
BIOS Version:              LENOVO GRET45WW (1.22 ), 8/31/2015
Windows Directory:         C:\WINDOWS
System Directory:          C:\WINDOWS\system32
Boot Device:               \Device\HarddiskVolume2
System Locale:             en-us;English (United States)
Input Locale:              en-us;English (United States)
Time Zone:                 (UTC+01:00) Belgrade, Bratislava, Budapest, Ljubljana, Prague
Total Physical Memory:     8,072 MB
Available Physical Memory: 1,764 MB
Virtual Memory: Max Size:  15,080 MB
Virtual Memory: Available: 4,796 MB
Virtual Memory: In Use:    10,284 MB
Node Environment:
node version: v14.15.4
npm version: 6.14.10
yarn version: 1.22.5
prisma version: 2.16.1

Originally posted by @dibranhoxha in https://github.com/prisma/prisma/discussions/5595

This exit code is usually when Ctrl+C is used:
https://github.com/clinicjs/node-clinic-doctor/commit/96a79c2c6ea8a15bfe9a7c39ca024eb8197a36ef
https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-erref/596a1078-e883-4972-9bbc-49e60bebca55?redirectedfrom=MSDN
I assume you are not doing that really but this just happens when you execute prisma -v?
For context: I can run the command just fine on my Windows machine.

Note: It seems from the docs you found @janpio that the error code is
0xC0000135   STATUS_DLL_NOT_FOUND | {Unable To Locate Component} This application has failed to start because %hs was not found. Reinstalling the application might fix this problem.
-- | --

@Jolg42 Can you tell me what application should be reinstalling please?  I have some trouble understanding this.
Do you mean to reinstall 'Prisma' or what?

I found this in the docs, I do not know what this means exactly, it's the first time I see it.
Only thing I can suggest at the moment is to uninstall the globally installed @prisma/cli from yarn and retry, so:
yarn global remove @prisma/cli
npx prisma -v

Maybe that helps.

Thanks @Jolg42 and @janpio i solved this issue after  removing @prisma/cli and reinstalling it and i have also installed windows build tools using npm install --global windows-build-tools and restarting my machine. So i think you can close this issue.

Awesome!

Note: It seems from the docs you found @janpio that the error code is
0xC0000135   STATUS_DLL_NOT_FOUND | {Unable To Locate Component} This application has failed to start because %hs was not found. Reinstalling the application might fix this problem.
-- | --

You are right, the code I linked to is actually about another error code. I missed that.
windows-build-tools should totally not be necessary to run the Windows binaries. Let's hope this does not happen more often.

I'm facing this issue.
Error: Get config: Error: Command failed with exit code<PHONE_NUMBER>: C:\path\npm\node_modules\prisma\query-engine-windows.exe cli get-config --ignoreEnvVarErrors

Please open a new issue and provide as much information as possible. Thanks.


Integrating Apollo cacheControl feature into schema not easily possible

I was looking into integrating the automatic caching of Apollo which I use as a server into my schema and there are two approaches. Either declaring a directive or setting it dynamically. Unfortunately there are two issues:

The directive that I tried to add via the directive plugin is not appearing in the schema at all. Thus it has no effect when declaring it for certain fields. I guess I need a transformer but there is none that exists for that as far as I can tell.
Setting the cacheControl dynamically works, but is not part of the info object according to TypeScript.

schemaBuilder.queryField('cachedData', (t) =>
  t.field({
    type: 'String',
    resolve: (_1, _2, _3, info) => {
      // @ts-expect-error
      info.cacheControl.setCacheHint({ maxAge: 30 });
      console.log('heavy computation');
      return 'result';
    }
  })
);

Do you have suggestions on how to support this within Pothos? Either option would be nice, the former slightly more as it clearly documents the behavior for consumers within the schema.

You could do something like this to add the cacheControl types to the info object
declare module 'graphql' {
  export interface GraphQLResolveInfo {
    cacheControl: {
      setCacheHint: (hint: { maxAge: number }) => void;
    };
  }
}

This can either be added to a declaration file that ts automatically imports, or add or import this from the file that defines the builder to ensure that the definition is always imported.
For the directives plugin, to get the directives in the schema, you need to use something like graphQL tools to print the schema with directives.  Unfortunately the printSchema method from graphql does not have a way to print directives

Yeah I was also considering to go for module augmentation as this is one of the very rare occasions where it's actually less detrimental and unexpected. I'll just use that for now even though the DX of explicitly calling it over adding a directive is less convenient. Will research at some point how to print the directives as you proposed.
Thanks once again.


Run server and client bundle in parallel

start server and client in parallel
add HMR for both of them
use betterScript
use node settings file

done


Add files via upload

@AGThrow I am sorry, the domain otc.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


usingRecursiveComparison with usingOverriddenEquals ignores equals method of the compared object

Summary
Version 3.22.0
I expect if I use usingRecursiveComparison with usingOverriddenEquals, field "age" should be ignored.
But as result I get

java.lang.AssertionError:
Expecting actual:
com.ComparatorDevices$Person@63be0d6
to be equal to:
com.ComparatorDevices$Person@63be0b6
when recursively comparing field by field, but found the following difference:
field/property 'age' differ:

actual value  : "22"
expected value: "11"

The recursive comparison was performed with this configuration:

overridden equals methods were used in the comparison
these types were compared with the following comparators:

java.lang.Double -> DoubleComparator[precision=1.0E-15]
java.lang.Float -> FloatComparator[precision=1.0E-6]
java.nio.file.Path -> lexicographic comparator (Path natural order)

actual and expected objects and their fields were compared field by field recursively even if they were not of the same type, this allows for example to compare a Person to a PersonDto (call strictTypeChecking(true) to change that behavior).

Example
class Person {
        String name;
        String age;

        public Person(String name, String age) {
            this.name = name;
            this.age = age;
        }

        @Override
        public boolean equals(Object o) {
            if (this == o) return true;
            if (o == null || getClass() != o.getClass()) return false;
            Person person = (Person) o;
            return Objects.equals(name, person.name);
        }

        @Override
        public int hashCode() {
            return Objects.hash(name, age);
        }
    }

@Test()
    public void personTest() {
        Person expectedPerson = new Person("name", "11");
        Person actualPerson = new Person("name", "22");
        Assertions.assertThat(actualPerson)
                .usingRecursiveComparison()
                .usingOverriddenEquals()
                .isEqualTo(expectedPerson);
    }

Thanks for reporting it, @xbif. We'll look into it.

Hey, is this something I could work one as a newcomer?

@JanUrb thanks for the offer, the recursive comparison is a bit tricky so I would not recommend it as a first issue to work on.
We usually label issues that newcomers can pick with "Good first issue", have a look if there is one you'd like to contribute

Hey @joel-costigliola,
Thanks for the advice. I will take a look at those issues. Maybe my initial analysis is of some use for you. It seems like usingOverriddenEquals ignores an overridden equals method of the root but respects  equals when comparing leaf objects.
I wrote this test to make the behaviour repeatable:
class RecursiveComparisonAssert_isEqualTo_usingOverriddenEquals_Test {

  public static class PersonWithOverriddenEquals {
    String name;
    String color;
    Pet pet;

    public PersonWithOverriddenEquals(String name, String color, Pet pet) {
      this.name = name; // only name is used in equals
      this.color = color;
      this.pet = pet;
    }

    @Override
    public boolean equals(Object o) {
      PersonWithOverriddenEquals person = (PersonWithOverriddenEquals) o;
      return Objects.equals(name, person.name);
    }

    @Override
    public int hashCode() {
      return Objects.hash(name, color);
    }

    @Override
    public String toString() {
      return String.format("Person [name=%s, color=%s]", name, color);
    }
  }

  public static class Pet {
    String name;
    String type; // only type is used in equals

    public Pet(String name, String type) {
      this.name = name;
      this.type = type;
    }

    @Override
    public boolean equals(Object o) {
      Pet pet = (Pet) o;
      return type.equals(pet.type);
    }

    @Override
    public int hashCode() {
      return Objects.hash(type);
    }
  }

  @ParameterizedTest(name = "{2}: actual={0} / expected={1}")
  @MethodSource("comparison_uses_overriden_equals")
  void should_pass_when_comparison_using_overriden_equals(Object actual, Object expected) {
    assertThat(actual).usingRecursiveComparison()
                      .usingOverriddenEquals()
                      .isEqualTo(expected);
  }

  private static Stream<Arguments> comparison_uses_overriden_equals() {
    // expected behaviour: PersonWithOverriddenEquals has an overridden equals method that only compares the field name. Therefore the following objects
    // should be the same.
    // actual behaviour: All fields of PersonWithOverriddenEquals are compared. However, the overridden equals method of Pet is respected.
    PersonWithOverriddenEquals person1 = new PersonWithOverriddenEquals("John", "green", new Pet("Ducky", "Duck"));
    PersonWithOverriddenEquals person2 = new PersonWithOverriddenEquals("John", "blue", new Pet("Mia", "Duck"));
    return Stream.of(arguments(person1, person2));
  }
}

I'm sorry.
I was also starting to work.
I made it in the draft version, but I had some concerns, so let me ask you a question.
@joel-costigliola
I have confirmed that the commit "1f8a43e6fa" has been changed so that the equals method is not used in the case of root object.
Is there anything wrong with my fix?
@JanUrb
Once the above problem is resolved, you can use my pull request to change it.
In that case I will close my pull request.
thank you.

thanks for the analysis and the test case @JanUrb! :)

So it looks like the issue only occurs when comparing root object and usingOverriddenEquals, although this is a bug there is no point of using the recursive comparison combined with usingOverriddenEquals on the root object, better in this case to simply use isEqualTo which will perform the same assertion but will give a better error message (one that IDE can show visual diff):
assertThat(actual).isEqualTo(expected);

Thank you for letting me know.
You're right.
I will close the pull request.

Leave it open @uc4w6c, I'm looking at it and it might be that we don't need isRootObject anymore (which you have removed in your PR). I'll see if I can confirm that and if I do I'll integrate your PR and you'll get the credit for fixing this one :) (usually I get the PR code, play with it, refactor it if needed, squash all commits but keep the original PR author)

@joel-costigliola
Thank you :)


README - Update to new domain

http://thejewsarecoming.tv/

Thanks for the help. and a good point.
I prefer the http://www.thejewsarecoming.tv

I prefer the http://www.thejewsarecoming.tv
Why?

as websites, usually have www. as a prefix. but I less care about it.
if you change it please also change it in the gatsby-config where we set the URL for the site (used for generating absolute links for social etc.

as websites, usually have www. as a prefix.
Not any more. https://dropwww.com/why

Updated gatsby-config.js as well.

I'm ok with that
I think that we need to change the config/.env.production
I also have GA on www.* site.
I will merge that after I check GA

No response from owner... 😢
Closing.


Move out container runtime configuration

Move container runtime configuration out from configure.yml to different file or even a role

Assignee is None

Created by bissquit at 2023-04-27T12:36:05Z


Styling with Orgmode support

Styling with Orgmode support

https://advance-esthetic.us/rf-facial-machines


fix: ServiceMonitor PodMonitor scrape config null values

Description
When using liqo's ServiceMonitors and PodMonitors with ArgoCD we encounter unresolvable diff with null values.
ArgoCD failed to sync this resources.
How Has This Been Tested?
Deployed new version -> ArgoCD does not show wrong diff.

/rebase test=true

/merge


[Communication] - SDK - Added MicrosoftTeamsUserIdentifier model

Let's update the changelog as well

Let's update the changelog as well


[Feature Request] Specify mobs immune to neutral aggro mechanic

A blacklist would be very much appreciated since some mods add things that the curse renders aggro that shouldn't be. Not to mention even the mod this project derives from has an option to excuse bees specifically, but I would personally prefer tweaking the list to fit the needs of my pack

My mod doesn't work on Animals, so bees and wolves are excluded for example.

Also, there's already a blacklist. Been in the mod since the beginning, so am closing this.

Where? I'm not seeing it anywhere and this is all the config has

It's done via a datapack using the vanilla tag system: "curse/tags/entity_types/no_angry.json"

that's not quite the same, could you please add it to the main config file?

That's... Literally the exact same thing. I'm just using the vanilla system to give better control for modpack creators.


Where is the stitches.js

It says you need this to install.

I can't find stitches.js.
Help? Thanks!

Looking at the sauce of http://draeton.github.io/stitches/ stitches.js (minified) is here: http://draeton.github.io/stitches/stitches/build/stitches/js/stitches-1.3.5.min.js


Create dataset loader for VnDT: A Vietnamese dependency treebank

Dataloader name: vndt/vndt.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?vndt

Dataset
vndt

Description
VnDT is a Vietnamese dependency treebank, consisting of 10K+ sentences. We conducted experiments of Vietnamese dependency parsing on our VnDT Treebank consisting of 10200 trees (219k words). The VnDT Treebank is automatically converted from the input Vietnamese Treebank based on our new conversion approach.

Subsets
gold_standard, predicted

Languages
vie

Tasks
Dependency Parsing

License
Unknown (unknown)

Homepage
https://github.com/datquocnguyen/VnDT

HF URL
-

Paper URL
https://github.com/datquocnguyen/VnDT/blob/master/VnDT-paper-CameraReadyVersion.pdf

#self-assign

Yep working on this issues got busy with some things but will try to wrap this issues by next week.

Thanks for letting us know, @bp-high, I'm removing the stale tag for now. Please add a tag pr-ready whenever you have finished on your dataloader so that the bot won't tag this issue as stale or let us know if you need more time for this issue.

Sorry couldn't work on this last weekend due to christmas holidays and celebration will try to conclude this, this weekend.

Thanks for the update, @bp-high! no rush on this; please take your time to enjoy ur holiday!

#self-assign

Hi @holylovenia and @SamuelCahyawijaya, for this task, I plan to use the load_ud_data_as_seacrowd_kb function from utils/common_parser.py. Unfortunately, this function assumes that a CoNLL file provided must contain the sent_id, text, and other fields from a fully annotated CoNLL file.
Unfortunately, the data from this VnDT dataset does not have these fields, as these fields are technically optional for CoNLL files. Therefore, using the function throws an error. How would you recommend to proceed with this? Thanks!

Hi, I implemented a function to impute the missing sent_id and text fields. For sent_id, it just generates a unique id on the fly. For text, the function naively concatenates the tokens with spaces in between, regardless if the .conll file contains SpaceAfter field or otherwise.
Please let me know if this is a valid approach. Thanks!


Add responsive design

Add response design for differents mobiles sizes:

[ ]  Add responsive Hero for large mobiles
[ ]  Add responsive Ticket for large mobiles
[ ]  Add responsive Hero for small mobiles
[ ]  Add responsive Ticket for small mobiles

Large mobiles (720x1280):

Small mobiles (375x667):

@LUISEDOCCOR  Open a PR  on #35 just for the Hero section, I think that we should do all the task over that pr

Hero responsive implemented on #35 and Ticket responsive on #41


Remember project-related settings

Is your feature request related to a problem?
No, it is a feature request.
Describe the solution you'd like
The plugin is extremely useful as it is, but it would be really neat if it could remember the previously used encryption algorithm in the project. So it would be automatically selected when the enryption/decryption popup is displayed.
As an extra, it would be nice if it would be able to recognized if the selected text is already wrapped in an "ENC(...)" block so users don`t need to tick that option on the popup.
Describe alternatives you've considered
Additional context
If someone uses the plugin frequently in many projects these little features could be very useful.

Hi, thanks for creating the issue. I'm glad you found the plugin useful :)
For the algorithm , the plugin should already remember what was used previously, and also for the "ENC(...)" checkbox since the #1
Please check you have the last version of the plugin, if it's not working, try to see if the file jasyptplugin.xml exists in the .idea folder of your project. If not, maybe there is an issue :/
For the extra recognized if the selected text is already wrapped in an "ENC(...)" I will do it, because it's simple and seems right to check that.
Also I'm currently updating the plugin, so I'll ship this in the 1.2.x

Thank you, that was superquick!
I have updated to the latest version and it does remember the algorithm. The jasyptplugin.xml has the saved value as well. Thanks!
I am wondering if it would be possible to remember the already used master password to a project. That way the plugin could check if the encrypted passwords are correct in each environment according to the provided jasypt master password. This could make the checking and testing a bit easier, especially when someone working with multiple projects/subprojects in the same repository.
This is just an idea and I know it has a security-side as well which you need to consider.

I have a similar idea in mind.
I think the first version of that could be to remembrer the last used password and try to reuse it to decrypt. If it fail, the plugin will ask again.
For the encrypt part, it's not that simple since if the password is not the good one, the plugin could not know so I have to think about it.
But before doing that, I have to learn about the CredentialAttributes feature from jetbrains.
But feel free to open a new feature request :)

Thank you! I have created an other feature request.


Use base_class rather than superclass

I've had some issues with this gem after updating to rails 5.
Upon installation, the object.follow(thing) works fine the first time, but every time thereafter, it fails with:
TypeError: no implicit conversion of nil into String

It's because ApplicationRecord is the class that parent_class_name is returning. You can see by this query:
INSERT INTO "follows" ("followable_id", "followable_type", "follower_id", "follower_type", "created_at", "updated_at") VALUES (14829, 'ApplicationRecord', 1, 'User', '2017-01-15 09:49:16.098371', '2017-01-15 09:49:16.098371') RETURNING "id"

I've done all the recommended steps with the config and I was about to give up until I saw that you're using superclass, which doesn't always provide the right class in a rails environment:
class A < ActiveRecord::Base
end

class B < A
end

> A.superclass
=> ActiveRecord::Base
> B.superclass
=> A

> A.base_class
=> A
> B.base_class
=> A

I added this commit to my own forked branch and it now works as expected: https://github.com/bufordtaylor/acts_as_follower/commit/43ad8ae9903f37cedf2b47a2ae0271926d3c5429

This is a duplicate of #56

@bufordtaylor Now that #56 has been merged, are you still experiencing this issue?


fix bug in firewall_rules_edit.php reported by tillsense in the forum…

… at https://forum.opnsense.org/index.php?topic=3951.0;topicseen

@fabianfrz thank you


feat: LLVM backend: implement the equivalent of -fstack-clash-protection

Fixes: #2373

I suggest merging this only after #2340 is merged. In theory, this PR should not interfere. In practice, it feels best to not tempt fate.

#2340 is now complete and pushed (in small pieces) to master. I rebased this PR on master to see if CI still goes through.

I fixed the review comments and squashed @tobiasgrosser commit in so we can have a bisectable history just in case.

Thank you @hargoniX


How to use variables in algebra problems using a string?

Hi again.
Really love this package. Now I can solve equations and simplify algebra. This is great.
I'm coding in Visual Basic.
I would like to create an algebra problem, where a string is saying eg "2*a+a+a" and the question is, what it is if "a=2"!? Can anyone help me with this?
Str="2*a+a+a"
Kind regards.

Imports MathNet.Symbolics
Imports expr = MathNet.Symbolics.SymbolicExpression
Module Program
    Sub Main(args As String())
        Dim e1 = expr.Parse("2*a+a+a")
        Dim dict As New Dictionary(Of String, FloatingPoint)
        dict.Add("a", 2)
        Console.WriteLine(e1.Evaluate(dict)) 'Real 8
    End Sub
End Module

Thank you, @FoggyFinder. Again it worked.👌💪 Really helpfull.👌
Sent with GitHawk


BugFix - update filter status with measurement

Currently, the filter result does not display the visit/measurement on adding the filters, but the filtered patient count is updated with the change in visit/measurement field. This needs to be updated by adding measurement to the filter_status and do necessary changes wherever required in the Filter Service.

Visit: baseline

Visit: 1 year followup

New Changes:
Visit: None

Visit: baseline

Visit: 1 year followup


let GenericExchangeExecutor2 poll all the sources at the same instead of polling them sequentially

Currently, the executor would poll each upstream sources sequentially.
    loop {
        if self.source_idx >= self.sources.len() {
            break;
        }
        if self.current_source.is_none() {
            let proto_source = &self.sources[self.source_idx];
            let source =
                CS::create_source(self.env.clone(), proto_source, self.task_id.clone()).await?;
            self.current_source = Some(source);
        }
        let mut source = self.current_source.take().unwrap();
        match source.take_data().await? {
            None => {
                self.current_source = None;
                self.source_idx += 1;
            }
            Some(res) => {
                if res.cardinality() == 0 {
                    debug!("Exchange source {:?} output empty chunk.", source);
                    assert_ne!(res.cardinality(), 0);
                }
                self.current_source = Some(source);
                yield res;
            }
        }
    }

This is less efficient than polling all the upstream sources at the same time,
as no matter which upstream source has data chunk being ready to be fetched, the downstream can immediately process the data chunk instead of waiting for a specific upstream source.
We may make CS::create_source return a stream for each upstream source, and then select_all them.

I wanna try this

I wanna try this

Thanks for your interest and contribution! Please feel free to ping me on Slack (@Martin) if there is any question.


docs(hybrid) documentation for 2.0 Hybrid Mode deployment

Hi @dndx , are you planning to add this to the navigation as well?

This doc can be previewed at https://deploy-preview-1670--kongdocs.netlify.com/2.0.x/hybrid-mode/. As discussed with @h3xar0n, this will be rebased and merged once the 2.0 doc branch comes up.


OnebotV11 QQ频道机器人，登录通过短信验证码获取 login_ticket: 网络请求失败

按照私聊频道机器人登录-输入手机号-扫码-获取验证码-发送给bot的流程。
以下是日志信息
02-26 14:18:24 [INFO] nonebot | Event will be handled by Matcher(type='message', module=nonebot_plugin_mystool.command.login, lineno=23)
02-26 14:18:40 [ERROR] nonebot_plugin_mystool | 通过短信验证码获取 login_ticket: 网络请求失败
Traceback (most recent call last):
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot_plugin_mystool/api/common.py", line 939, in get_login_ticket_by_captcha
    res = await request()
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot_plugin_mystool/api/common.py", line 929, in request
    return await client.post(URL_LOGIN_TICKET_BY_CAPTCHA,
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/httpx/_client.py", line 1877, in post
    return await self.request(
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/httpx/_client.py", line 1559, in request
    return await self.send(request, auth=auth, follow_redirects=follow_redirects)
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/httpx/_client.py", line 1635, in send
    raise RuntimeError("Cannot send a request, as the client has been closed.")
RuntimeError: Cannot send a request, as the client has been closed.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/root/bot/mihoyobot/bot.py", line 46, in <module>
    nonebot.run()
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot/__init__.py", line 334, in run
    get_driver().run(*args, **kwargs)
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot/drivers/fastapi.py", line 186, in run
    uvicorn.run(
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/uvicorn/main.py", line 587, in run
    server.run()
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/uvicorn/server.py", line 62, in run
    return asyncio.run(self.serve(sockets=sockets))
  File "/root/.pyenv/versions/3.11.8/lib/python3.11/asyncio/runners.py", line 190, in run
    return runner.run(main)
  File "/root/.pyenv/versions/3.11.8/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot/message.py", line 476, in check_and_run_matcher
    await _run_matcher(
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot/message.py", line 428, in _run_matcher
    await matcher.run(bot, event, state, stack, dependency_cache)
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot/internal/matcher/matcher.py", line 842, in run
    await self.simple_run(bot, event, state, stack, dependency_cache)
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot/internal/matcher/matcher.py", line 817, in simple_run
    await handler(
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot/dependencies/__init__.py", line 106, in __call__
    return await cast(Callable[..., Awaitable[R]], self.call)(**values)
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot_plugin_mystool/command/login.py", line 137, in _
    login_status, cookies = await get_login_ticket_by_captcha(phone_number, int(captcha), device_id)
> File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/nonebot_plugin_mystool/api/common.py", line 936, in get_login_ticket_by_captcha
    async for attempt in get_async_retry(retry):
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/tenacity/_asyncio.py", line 71, in __anext__
    do = self.iter(retry_state=self._retry_state)
  File "/root/bot/mihoyobot/.venv/lib/python3.11/site-packages/tenacity/__init__.py", line 326, in iter
    raise retry_exc from fut.exception()
tenacity.RetryError: RetryError[<Future at 0x7f0db8867b50 state=finished raised RuntimeError>]
02-26 14:18:41 [INFO] nonebot | Matcher(type='message', module=nonebot_plugin_mystool.command.login, lineno=23) running complete

RuntimeError: Cannot send a request, as the client has been closed.

看上去确实是网络问题

切换到了大陆服务器，正常了）小心香港服务器骗局


zip is not available in the image used by task ibm-pak.

Request: to add zip to the image used by tas ibm-pak.
As part of the work done around CP4I, in order to be able to deploy Configurations used by App Connect Integration runtimes it is required to zip a folder that was previously cloned to use the source code, but when trying to use the zip command I got the following error:
/tekton/scripts/script-0-jpwzt: line 15: zip: command not found
As a reference, this is the script I used in my task:
            echo "Entering test script."
            cd workspace
            mkdir mydir
            touch mydir/test.txt
            ls mydir
            zip -r mydir.zip mydir
            echo "Test has been completed."

See https://github.com/cloud-native-toolkit/deployer-tekton-tasks/issues/3#issuecomment-2039021811


400 Bad Request Exception for Unlike post

Steps to produce issue:
calling unlike feature as per official php api guide
example:
$client->unlike(1234567890, 'abcd1234');
Then it will throw uncaught Uncaught Tumblr\API\RequestException: [400]: Bad Request.
Really appreciate if I could be enlightened if there's any parameter name change for Unlike request, thanks!

@angstdeprived Has the configured user previously liked the post? If not then you will get an error as seen.

@oligriffiths yes, the configured user has already liked the post previously

I finally found why. I'm trying to unlike a reblog using post ID of my blog. It turns out I can't like/unlike any of my reblogs/original post. It works fine if I like/unlike other blog's original posts. I guess this is a new tumblr policy,

Yup, you shouldn't be able to like your own posts. Glad you solved it.

Yeah I'll keep note in the future, thanks for the response @oligriffiths ! Also I found that I also can't unlike posts from blogs that has blocked me (I got 403 forbidden response).
I understand that liking posts from blog that has blocked you must be disabled to prevent it to show on their notifications. But unliking their post should be permittable logically since unliking won't show by default in notifications. Do you have any idea to unlike posts from blogs that has blocked you? Or is it possible to consider to change that policy in the future?
I'm sorry if this is slight OOT but I don't know where else to ask since I've submitted this to Tumblr team since years ago and I don't think it's in the Tumblr developer's to do list since it's not an error issue. Really appreciate it if tumblr team could consider changing that policy in the future.
Thanks in advance!


SharedConfigState (work in progress)

Issue #, if available:
#74 use of a credential profile
Description of changes:
I'm loading the AWS Region based on the region set in the local profile.  The summary of key changes are:

getDefaultSession now enables the SharedConfigState option.
if noMetadata is set to true, we load the region from Session.Config.Region

I've gone through a few iterations attempting to find the best place to put this check and I'm not sure I have it where it should go.
It does work for me locally but I'm not 100% sure it won't interfere with other use cases.  I haven't been able to completely follow the EC2 metadata flows.
While I have confirmed that my traces are being published as expected (using my profile and that profile's region), I do get a context canceled error like:
2020-12-12T10:20:11-08:00 [Debug] Received request on HTTP Proxy server : /GetSamplingRules
2020/12/12 10:20:18 http: proxy error: context canceled

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mousedownmike Kindly ask, this PR still work in progress?

Hi @wangzlei I marked it as a "work in progress" to indicate it works for my use case but I'm not sure what the other use cases are.  If it works where you expect it to, it should be good.


Windows local time zone should be mapped into WSL properly

As touched on in #856, #1392, and #3109 (without resolution), WSL doesn't correctly map the local time zone of Windows into Linux. Instead, LsxxManager creates a pseudo time zone in /usr/share/zoneinfo/Msft/localtime, and hard links it to /etc/localtime.
This approach is problematic. Though it gets the current time correct, it removes a ton of detail about the time zone, such as its identifier, abbreviations, historical and future transitions (if they deviate from those in the current year), etc.  It also removes any ability for downstream APIs (such as ICU) to generate a correct display name.
This is seen in many environments, but perhaps the easiest is in Node JS.  Install the latest Node LTS (I'm on 10.15.0) into a regular (non-WSL) Windows environment and also into WSL.  Then in the Node command line:
console.log(new Date().toString())
console.log(Intl.DateTimeFormat().resolvedOptions().timeZone)

In a regular environment, with my computer's time zone set for US Pacific Time, I get:
Mon Dec 31 2018 15:45:41 GMT-0800 (Pacific Standard Time)
America/Los_Angeles

In a WSL environment (for me, Ubuntu 18.04.1 LTS), I get:
Mon Dec 31 2018 15:45:41 GMT-0800 (GMT-08:00)
undefined

It can also be seen just on the Linux command line:
date +%Z

For me running this in December, it should return PST, but in WSL it returns STD because that is the abbreviation assigned in the pseudo zone.
Additionally - the pseudo zone is not necessary.  Windows already has the ability to generate the Linux (IANA) time zone identifier, via either of two mechanisms:

Call the WinRT/UWP function: Windows::Globalization::Calendar().GetTimeZone() (docs here)

The above function uses the data file at %WINDIR%\Globalization\Time Zone\timezoneMapping.xml.  It can be used directly if WinRT/UWP is not available.  The entry to use is the one with Region="001" and a WinID corresponding to the output from GetDynamicTimeZoneInformation->TimeZoneKeyName (or from the registry at HKLM\SYSTEM\CurrentControlSet\Control\TimeZoneInformation\TimeZoneKeyName).  The Linux time zone ID is in the TZID field.

Once the Linux time zone ID is known, then /etc/localtime should simply be symlinked to the corresponding file in /usr/share/zoneinfo.   If it doesn't exist (perhaps on certain versions of Linux without a tzdata distribution), then and only then should it fall back to the pseudo Msft time zone.
Please revisit this asap.  Thanks.
(I can be reached internally at ms alias mattjoh if you have any additional questions)

Note, #3109 describes how the MSFT zone shows up on .NET Core:
TimeZoneInfo.Local.DisplayName:                          GMT
TimeZoneInfo.Local.Id:                                   Msft/localtime

Again, if mapped correctly, these APIs would just work.

Thanks for the detailed post, @mj1856.  Agreed, this is something that should be revisited.

Thanks for filing this issue!
I just wanted to add that another potential option, if you can't call any WinRT/UWP functions for some reason, is to call the flat C API ucal_getDefaultTimeZone in order to get the mapped IANA time zone identifier. (Note that this API is only available if you are on RS2+ though).

@jefgen - Thanks!  I didn't know about that one.

The new implementation is in 18344, I hope it ticks all the boxes for you :)

Thanks! I'll take a look at some point. ;)

Hi, it seems like the fix for this breaks things for people using utc on windows (dual booters), which can be done with a registry key like so
reg add "HKEY_LOCAL_MACHINE\System\CurrentControlSet\Control\TimeZoneInformation" /v RealTimeIsUniversal /d 1 /t REG_QWORD /f
I'm not even sure if the above is something officially supported, but with windows set to utc wsl2 still runs the conversion from localtime, meaning my clock ends up 7 hours fast.

@chetgurevitch, could you please open a separate issue for that problem so it doesn't get lost?  Thanks.

RealTimeIsUniversal refers only to the RTC setting being UTC rather than local, and should have no influence on Windows timekeeping after boot, and should make it easier as WSL RTC virtualization should not have to backconvert to UTC at Unix boot.
I've been running with that setting for over a decade to make life easier for Windows time sync utilities, mainly NTP (Meinberg build mostly) with my Garmin 18x LVC GPS, after the old NTP "compatible" Windows Time Sync Service was downgraded to the current implementation.
[Syncing system time since DataStorm Procomm-Plus to NIST ACTS dialup.]


fix: Allow package repository urls to be http.

Allow package repository urls to be http as discussed in https://github.com/bagetter/BaGetter/issues/151.

@Regenhardt What about a new release? Look at Discord :)


[BUG] nvcomp namespace required in target_link_libraries (CMake 2.23.2)

Describe the bug
An "nvcomp::" namespace seems to be required in target_link_libraries with newer versions of CMake (I'm using 2.23.2 under Linux).  Otherwise the include path doesn't include nvcomp/include:
low_level_quickstart_example.cpp:5:10: fatal error: nvcomp/lz4.h: No such file or directory
Steps/Code to reproduce bug
Build with CMake 2.23.2 under Linux
Additional context
I will submit a pull request with a fix.

Hi Mark. Thanks for the report / PR. I'll approve it.
FYI, I think the issue isn't with cmake version but rather in our local testing nvcomp is installed in a location that was otherwise added to the include path.