defer_js loading javascript twice

Catchpoint is reporting that defer_js is double-loading javascript files:
http://blog.catchpoint.com/2015/03/18/pagespeed-third-party-issue/

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 19 Mar 2015 at 4:41

I encountered this while using IISpeed and was able to prevent some third party resources (jscache.com & tripadvisor.com) from being loaded twice by adding the "Disallow" directive.
https://developers.google.com/speed/pagespeed/module/restricting_urls
pagespeed Disallow "*jscache.com*";
pagespeed Disallow "*tripadvisor.com*";

We use a CMS, so it would be difficult to identify and add all third-party rules as directives to the config file.  Previously adding pagespeed_no_defer="" to scripts and resources was enough, but this seems to be ignored.  (As a result, we've disabled "defer" features altogether.)
I use the options to combine JS & CSS.  I had to disable "InPlaceResourceOptimization".  It would correctly combine original files into a single file and then additionally call the original file separately.

On Chrome standard 45.0.2454.93 pagespeed defer is still double loading my js-file. On Firefox it only loads once.

Testing defer_js now I see:
example1.js
  document.write("hello ");

example2.js
  document.write("world");

djs.html
  <script src="example1.js"></script>
  <script src="example2.js"></script>

When run normally I see "hello world", as expected.  When run with defer_javascript I see a blank screen.
Test it here: http://www.jefftk.com/djs?PageSpeedFilters=defer_javascript
Here are WPT runs for Chrome, FF, and Safari showing the same thing:

Chrome: http://www.webpagetest.org/result/160623_DD_115V/
Firefox: http://www.webpagetest.org/result/160623_0N_1163/
Safari: http://www.webpagetest.org/result/160623_R8_1169/

Reproduced with mps trunk.

Moved blank js issue to #1325, since it turns out not to be related.

Took a packet capture of chrome talking to a "frozen" copy of this (pagespeed not running, just pagespeed's output) with tcpdump.  I see:
c: request for html
s: html text
c: request for example1
s: example1 text
c: request for example2
c: request for js_defer
s: example2 text
s: js_defer text
c: request for example1
c: request for example2
s: example1 text
s: example2 text
c: request for example2
c: example2 text

(This matches what chrome reports in the network tab in developer tools.)
Note that the first requests for example1 and example2 go out before js_defer has even been requested, which suggests that labeling with type=psajs doesn't keep the preload scanner from fetching it.
Here's the dump:
current3.txt

To test this, I made a simple page with no defer javascript at all:
<body>
<script src="example1.js" type="text/psajs" orig_index="0"></script>
<script src="example2.js" type="text/psajs" orig_index="1"></script>

Loading this with chrome, it still gets fetched, presumably by the preload scanner since they don't get run.
packet capture: current4.txt

My current hypothesis is that we're getting triple loads because:

one load from the preload scanner
another load from defer-js's image-based prefetch
another load from defer-js, for real

To test this, I modified defer_js:
deferJsNs.DeferJs.prototype.attemptPrefetchOrQueue = function(url) {
//  if (this.isWebKit()) {
//    new Image().src = url;
//  }
};

Now I see:
c: request html
s: html text
c: request example1
c: request example2
c: request js_defer
s: example1 text
s: example2 text
s: js_defer text
c: request example1
c: request example2
s: example1 text
s: example2 text

This is consistent with the hypothesis, since with image-based preloading available only did one load of each js file after running defer-js instead of two.
packet capture: no-image-prefetch.txt

What's not clear to me is why Chrome would make three requests for the same url, instead of re-using what's in its cache.  Why would you want a preload scanner to throw away its results?

I'm also not sure whether we want to thwart the preload scanner here.  The reason to thwart it is that because we're deferring the execution of the js the browser is wrong to heavily prioritize loading it since it won't actually be blocking things.  The reason not to thwart it is that simply loading the js isn't that bad, we want to preload it somehow anyway, and for page loading the important thing is keeping the incoming pipe full.

I had thought that maybe the preload scanner results getting re-fetched was because we were using different headers, but diffing them they are both exactly:
Accept:*/*
Accept-Encoding:gzip, deflate, sdch
Accept-Language:en-US,en;q=0.8,es;q=0.6
Cache-Control:no-cache
Connection:keep-alive
Host:www.trycontra.com
Pragma:no-cache
Referer:http://www.trycontra.com/djs-frozen.html
User-Agent:Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.112 Safari/537.36

I want to get a simple repro case for the preload scanner so I can talk to chromium and figure out if it's a bug, but my first try failed to trigger a duplicate load:
<script id=a type=invalid src=example.js></script>
<script id=b>
initial_script = document.getElementById("a");
s = document.createElement("script");
s.setAttribute("src", "example.js");
s.setAttribute("type", "text/javascript");

this_script = document.getElementById("b");
this_script.parentNode.insertBefore(s, this_script);
</script>

capture: repro1.txt

(@crowell just checked, and firefox doesn't see either kind of duplicate load)

The preload scanner duplicate load doesn't happen in safari.
packet capture: safari4.txt

It looks like two things are going on:

Our image-based preloading doesn't work any more, and we should remove it
Even with (1) fixed, we have a duplicate load on Chrome (but not Safari or Firefox) where it looks like the preload scanner is ignoring what it already has when the type of the script tag is different.

We can easily fix (1) ourselves.  I've posted on net-dev@chromium to figure out if (2) is something we can get fixed in Chrome:  https://groups.google.com/a/chromium.org/forum/#!topic/net-dev/PpITTXI7VVs

As requested on net-dev, filed a crbug: https://bugs.chromium.org/p/chromium/issues/detail?id=623109

In debugging this with the Chromium folks they pointed out that these scripts should be loaded from cache, and the extra network loads only happen when you disable the cache.  The cache is disabled if you use Ctrl+Shift+R or if you have devtools open with Disable Cache checked, and I had both of these.
As long as you're running normally, without disabling the cache, this shouldn't cause any additional network loads in practice.  My guess at this point is that all the results that did show network loads were results run with Ctrl+Shift+R.

We could "fix" this by using <pagespeed-script ...> instead of <script type=text/psajs ...> but I'm pretty sure this would actually be slower in practice because it thwarts the preload scanner.

Planning to start adding <link rel=preload> to replace the image-based preload hack.  We don't need this yet, because currently the webkit preload scanner does find and load <script type=text/psajs>, but following the chromium bug it looks like they're going to make the preload scanner stop handling these.


client connect support directly use ipaddress

fix https://github.com/burakoner/TcpSharp/issues/3
sometime the ipaddress can not lookup by the dns

thx, yu pr good!
on stock package i get error in docker container


Provide wav data directly?

Is there a way to provide the wav data directly, say from a microphone-like peripheral instead of providing a wav file or having to record to one first?

Same, I would like to pass data externally from js or another module. Data can be passed down as string base64 encoded or number[] assuming it is in the correct pcm format (16kHz / 16-bit PCM). It would also be great if we could directly pass mel spectrogram data and skip the WAV conversion.
For context, I am working on a separate package @siteed/expo-audio-stream to do the audio streaming and want to pass down the data to whisper.rn. It could also directly pass down mel spectogram datan and skip the conversion. I have done n audio playground demo with live transcription on web but now want to integrate the same in native (try it at https://deeeed.github.io/expo-audio-stream/playground/
I forked the repository and started experimenting with the code to see if I could integrate this feature, but it doesn't seem straightforward.
My initial thougths were to:

Add new method to JS Interface

  startRealtimeTranscribeWithAudioInput(
    contextId: number,
    jobId: number,
    options: TranscribeOptions,
  ): Promise<void>;
  receiveAudioDataChunk(jobId: number, audioData: number[]): Promise<void>;

Implement the methods in cpp:

namespace rnwhisper {

std::mutex job_mutex;
std::unordered_map<int, job*> job_map;

void start_realtime_transcribe_with_audio_input(int job_id, const whisper_full_params& params) {
    RNWHISPER_LOG_INFO("Started real-time transcription with audio input for job_id: %d\n", job_id);

    std::lock_guard<std::mutex> lock(job_mutex);
    job* new_job = new job{job_id, false, params};
    job_map[job_id] = new_job;
}

void receive_audio_data(int job_id, const std::vector<short>& audio_data) {
    std::lock_guard<std::mutex> lock(job_mutex);
    auto it = job_map.find(job_id);
    if (it != job_map.end()) {
        job* current_job = it->second;

        // Log the size of the received audio data
        RNWHISPER_LOG_INFO("Received audio data for job_id: %d, data size: %zu\n", job_id, audio_data.size());

        // Convert std::vector<int> to std::vector<short>
        std::vector<short> audio_data_short(audio_data.begin(), audio_data.end());
        current_job->put_pcm_data(audio_data_short.data(), 0, audio_data_short.size(), audio_data_short.size());

         // Log after processing the audio data
        RNWHISPER_LOG_INFO("Processed audio data for job_id: %d\n", job_id);
    } else {
        RNWHISPER_LOG_WARN("No job found for job_id: %d\n", job_id);
    }
}

Integrate Amdroid and IOS

Android:
Add native method declarations in the RNWhisper class.
Implement methods to call the native functions, handle audio data, and resolve promises
iOS:
Add corresponding method declarations in the Objective-C/Swift code.
Implement methods to interface with the C++ functions, handle audio data, and communicate with JavaScript.

Unfortunately, it would take me a while to figure out the full implementation as I haven't done much C++ integration in RN before.
@jhen0409 does this feature makes sense to you? Could you highlight the steps you would take and things to watch for? I can then take a proper pass at implementing it with much less struggles :)
Cheers

After #267, we provided base64 wav decode in context.transcribe.

@jhen0409 does this feature makes sense to you? Could you highlight the steps you would take and things to watch for? I can then take a proper pass at implementing it with much less struggles :)

Sorry for the long delay in responding.
Our goal will be move the realtime transcription logic, allows us to process audio buffers more easily, and I added context.transcribeData first for transcribe PCM data (base64). You can see TranscribeData example. (Note that currently it's not a realtime transcription example)
#52 will be next we need to implement for non-bridge data send.


CI for Node

Contains the compiled JavaScript (4 ms)
Initializes with two players (2 ms)
Starts the game with a random player (1 ms)
Sets the current player to be whoever it is not (1 ms)


Form submission doesn't happen if formaction button is disabled after click in Chrome/Safari

Describe the bug
WARNING: Can Reproduced only using Chrome/Safari, works good in Firefox
Basically, you need two files: +page.server.js and +page.svelte.
In +page.server.js you can have any action you like:
const checkout = () => {
	console.log('checkout successful')
	return { success: true }
};
export const actions = { checkout };

And in +page.svelte you have <form> and <button>. Button has formaction attribute and also logic to disable itself on submit:
<script>
	import { enhance } from '$app/forms';
	let disabled = false;
</script>

<form method="POST" use:enhance>
	<button
		on:click={() => (disabled = true)}
		{disabled}
		formaction="?/checkout"
	>Send me</button>
</form>

Expected behaviour: form is submitted, "checkout successful" logged to console, button is disabled.
Actual behaviour: button is disabled, but no request to form sent.
Reproduction
StackBlitz link: https://stackblitz.com/edit/form-submission-bug
Sveltelab link: https://www.sveltelab.dev/hoh6scx1j2wzqk5
Logs
No interesting logs.
See for yourself inside reproduction.

System Info
System:
    OS: macOS 14.1.2
    CPU: (8) arm64 Apple M1
    Memory: 127.39 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.8.1 - ~/Library/Caches/fnm_multishells/51879_1673605975417/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 10.1.0 - ~/Library/Caches/fnm_multishells/51879_1673605975417/bin/npm
    pnpm: 8.11.0 - ~/Library/pnpm/pnpm
    bun: 1.0.0 - /opt/homebrew/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 119.0.6045.199
    Safari: 17.1.2
    Firefox: 120.0.1
  npmPackages:
    @sveltejs/adapter-auto: ^2.1.1 => 2.1.1
    @sveltejs/adapter-node: ^1.3.1 => 1.3.1
    @sveltejs/kit: ^1.27.7 => 1.27.7
    svelte: ^4.2.8 => 4.2.8
    vite: ^4.5.1 => 4.5.1

Severity
serious, but I can work around it
Additional Information
No response

I would generally not recommend adding click events on buttons in a form, but instead just use the submit event.
When using enhance, you can just add a handler there:
<form method="POST" use:enhance={() => { disabled = true }}>

This is not a Svelte problem either and I looked at the spec and found it to be inconclusive. So again: Just don't do it.


Arrow icons

It seems like the project is missing some icons, as it shows the Godot icon most of the time

Assets are fine!
That was me, forgetting to take care of a workaround required to changing executable icon for windows version.


Script behaves differently when run as Platypus app

I'm using the command line platypus (current branch c836575d3fc6931945696c6b9ed0e45f4a798fed) on OSX 10.11.3 (El Capitan)
My script works fine when run by itself (toggles Internet Sharing) but stops prematurely when run as a platypus app.
You can download the development directory from here: http://xmlizer.net/internetSharing.tgz. To build the app, call make in the top level directory. To run the app, call make run.
Thank you

What's the output? Are you getting any error messages?

when I run ./internetSharing.pl in the development directory, I don't get any output. When I run the platypus app with make run the output is open InternetSharing.app, which obviously comes from the Makefile. If I inhibit output by preceding the Makefile-command like so @open $(TARGET), there's no output.
The app stops always at the same moment when running as a platypus app: When the Sharing dialog has just opened.

When I run your app (macOS Sierra 10.12), I get the following:
348:888: execution error: System Events got an error: osascript is not allowed assistive access. (-1728)
failed to close osascript: at /Users/sveinbjorn/Desktop/internetSharing/InternetSharing.app/Contents/Resources/script line 46.

I don't know what exactly is going on there, but I'm pretty sure it has nothing to do with Platypus.

How exactly do you run it to get that output?

cd InternetSharing.app/Contents/MacOS
./InternetSharing

Interesting... that works just fine in my case (10.11.6 (El Capitan)) with no output.
Do you get that output in the terminal?
It's when I run the app from the Finder by selecting it and then typing Cmd-O (or double-clicking it) that the app halts: When the Sharing dialog has just opened.

Don't know if this is still relevant, the error: osascript is not allowed assistive access. is because you need to give accessibility permissions to the script editor.


Tweet action failing with Unexpected input(s) warning

I suspect this is outside the scope of your excellent documentation, but after setting up the bot and working in the creds, I find that I'm getting exit 1 failures on the "Tweet" action. I see that your actions are getting the same warnings (so I think that's probably a red herring), but are succeeding.
Yours

Mine

Is this an issue with the app credentials? Do you have any idea where I could get more descriptive errors for that?
Thanks again for putting this bot in the open!

Can you click the [x tweet    15s] button and see the actual error by chance?

@JasonSanford Massive 🤦 - I appear ot have misspelled my email address. Will update and check, but please assume that this was the dumbest of possible errors. Many apologies.

Ah nope - I spoke too soon. Still failing semi-quietly:

It looks like it's still failing. It appears the tweet_a_mile script isn't actually doing anything? There's nothing to commit which indicates the JSON for the tweeted mile hasn't been changed. I don't see any valuable output from the tweet_a_mile script though. In fact I don't see anything.

Comparing the logs from my "Config it" section to yours, it looks like yours is actually writing something to .env, but mine is writing empty strings.

Ahh, here we go. Let's try changing the following in your .workflows/lt.yml
node dist/scripts/tweet_a_mile.js at -> node dist/scripts/tweet_a_mile.js lt
(l -> t)

Ok, looks like it's getting further. I see some key errors from the Twitter API now. Here are how my secrets are declared (double what you'll have)

@JasonSanford I've added those, but they're not registering. Hard to tell if it's a.) a slow cache, or b.) I put them in the wrong env/workflow.

Mine are added as repository secrets, not environment secrets. I have no idea why or if that's the correct-est way to do it. 🤷🏼♂

@JasonSanford That worked! That's got it writing the creds to the .env! Thanks for all your help!
Now it's a 403, but that's something I can work with 😁

Great! Yeah I struggled a bit getting Twitter creds and permissions just right. You'll want to make sure whatever Access Token and Secret you created have "Read and Write" permissions.

R/W was it! Off to the races! https://twitter.com/every_mile_lt/status/1439963619257556996

@JasonSanford I am sincerely sorry to come back to this, but I've run out of investigatory options. My bot has only succeeded twice out of 53 runs under circumstances I can't seem to reproduce. The vast majority of actions fail, with the suggestion of a credentials error, specifically a 401.
My question to you is: have you experienced any sort of silent revocation of Twitter creds? And if so, what's the best channel to inquire with them?
Things I've tried:

Rolling credentials ("Regenerate" on all available tokens).
Deleting tweets, then rolling credentials.
Reverting to the stock settings of your app.
Confirming media upload by logging IDs.
The usual googling through StackOverflow, though in most cases there it seems like rolling credentials worked for the OPs.
Confirming secret names in the github config:

I know this is not your bug to fix, but it's hard to tell exactly who to ask at this point. Thanks again for all the help thus far!

The only thing I see that is different from my working code is that you're posting with two images. The docs for the twitter client say to pass an array of string ids (media_ids) but I don't see anything in the source code that joins those into the format twitter expects, 1234,9876. What happens if you make the following change (you might need to ts-ignore the new line since it might not match the expected type)
Current
const statusResponse = await client.v1.tweet(status, { media_ids: mediaIds });

New
const statusResponse = await client.v1.tweet(status, { media_ids: mediaIds.join(',') });


do not consider _msgid when merging objects

Hello. This one is fixing functionality of join node when merging complete objects. Currently it does not work as one might expect, because of _msgid property, which is not obvious.
But I'm not 100% sure this is a good solution.

Coverage remained the same at 86.429% when pulling 32a7f2d4ee523b0850b9b21122d46e17fb35aac9 on tiger-seo:patch-1 into cb4f46decceaf39caf608838f1af01dcbbf5f51f on node-red:master.

Hi,
please can you expand on why you think this change is necessary? Can you provide an example that shows what you believe is not desirable behaviour? And who this PR will address it?
Any change will also need to include appropriate unit tests in the corresponding file under the test directory before we can consider it
Thanks!

Hello. I created an example to show the behaviour which i think is not obvious.
[{"id":"29002bb3.173684","type":"tab","label":"Flow 1","disabled":false},{"id":"cbb7a7a9.f8bbd8","type":"inject","z":"29002bb3.173684","name":"","topic":"","payload":"{\"title\":\"title\",\"body\":\"some message\"}","payloadType":"json","repeat":"","crontab":"","once":false,"x":190,"y":120,"wires":[["6bbb685e.479e88","c5410ff1.fdafc"]]},{"id":"71665771.099b48","type":"debug","z":"29002bb3.173684","name":"","active":true,"console":"false","complete":"false","x":670,"y":120,"wires":[]},{"id":"b175f620.665a38","type":"join","z":"29002bb3.173684","name":"","mode":"custom","build":"merged","property":"","propertyType":"full","key":"topic","joiner":"\\n","joinerType":"str","accumulate":false,"timeout":"","count":"2","x":530,"y":120,"wires":[["71665771.099b48"]]},{"id":"6bbb685e.479e88","type":"function","z":"29002bb3.173684","name":"","func":"return {\n    tokens: [\"a\", \"b\", \"c\"]\n}","outputs":1,"noerr":0,"x":370,"y":80,"wires":[["b175f620.665a38"]]},{"id":"c5410ff1.fdafc","type":"function","z":"29002bb3.173684","name":"","func":"return {\n    notification: msg.payload\n}","outputs":1,"noerr":0,"x":370,"y":160,"wires":[["b175f620.665a38"]]}]

In this example join node has parameter After a number of message parts set to 2, because there is an expectation to receive 2 messages. This results in 2 messages from join node:
{"tokens":["a","b","c"],"_msgid":"7663267.b76cfd8"} and {"notification":{"title":"title","body":"some message"},"_msgid":"7663267.b76cfd8"}, which is not desired output.
When parameter After a number of message parts set to 3, the output corresponds with expectations, i.e. {"tokens":["a","b","c"],"_msgid":"67557c27.dedbd4","notification":{"title":"title","body":"some message"}}. I believe that property _msgid is the reason. It is not obvious that i have to consider _msgid when setting parameter After a number of message parts.
Thank you.

Ah right - the clue was "when merging complete objects" - as we always add a _msgid it will count... So yes it probably should be discounted when joining complete messages.

closing as no cla - so hand coded instead


Adding Dynamic Form Rendering Section

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This is a feature update to incorporate the Open Source JSON form renderer @ https://github.com/formio/ng2-formio

What is the current behavior? (You can also link to an open issue here)
The current behavior shows static forms. This illustrates how a dynamic form renderer can also be used within this theme.

What is the new behavior (if this is a feature change)?
This shows off how a dynamic form renderer can be used within this theme. Form.io provides an Open Source dynamic JSON powered form renderer which offers a lot of capability to this template.

Other information:
The Form.io renderer is Open Source so I hope it gets included in this amazing template.

Hey @travist, thanks for your PR, but I don't think we can merge this now.


Bump time dependency to avoid build failure.

Added commit:

Command::kill() doesn’t need to take &mut self, just &self.

It feels like kill() should require a mutable self, but strictly speaking it doesn’t change the object.


Not getting hit part #2

At the way far right were the robots spawn, if you put a top hat frog at the line then a cannon frog behind the top hat frog the  robot doesn't ever take damage.

Well, that's just protection.


[Debian] Add release 9.0

See: https://www.debian.org/News/2017/20170506

See: https://app.vagrantup.com/nlamirault/boxes/debian-9


How to cusstomize icon ?

Hello, thanks for your gem. But I have some problem with it.
How I can change glyphicon icons to font awesome icons ?
I saw closed issues when you propose used glyphicon but in my project I use font awesome and glyphicon isn't needed.

Hi, thanks for using Bootsy!
You can do so on master, since I recently moved to Bootstrap3-wysiwyg, which supports Font-Awesome.
An approach would be to override Bootstrap3-wysiwyg's default options so you would have Font-Awesome enabled to all Bootsy areas:
$(function(){
  $.fn.wysihtml5.defaultOptions.toolbar.fa = true;
});

Well, that line gives me an undefined error on .toolbar. Is there another i have it like this:
  $(document).on('ready page:load', function(){
    console.log($.fn.wysihtml5.defaultOptions.toolbar);
    $.fn.wysihtml5.defaultOptions.toolbar.fa = true;
  });

The console.log() prints undefined.
Is there something i'm doing wrong, or was there a change?

@joaoribeirost Sorry, on the latest 2.2.1 the above snippet will no longer work since I rolled back Bootstrap3-wysihtml5 as it is no longer maintained. If you definitely need Font Awesome you can lock Bootsy 2.2.0 on your Gemfile.

Oh, okay then. Yeah, i need because i'm not installing all of bootstrap. Thanks! :)

Is there any way to customize the icons with your current version?

Unfortunately not at this point.

cool. i just did an @extend for each of the missing icons to extend the font awesome

On Mar 29, 2016, at 6:02 PM, Volmer<EMAIL_ADDRESS>wrote:
Unfortunately not at this point.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/volmer/bootsy/issues/158#issuecomment-203176966


Add includedSoftware field to Package spec

Allows users to specify an array of included software within a package
=> it's name, version and description

Signed-off-by: Neil Hickey<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #212
Does this PR introduce a user-facing change?

Additional Notes for your reviewer:
Review Checklist:

[ ] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[ ] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

I am not so sure showing the package contents as part of kubectl get is a great idea.
--field-selector= as suggested in the issue is very limited to the things it can filter, and custom types are not part of that.
The only real way to see this is with some -o jsonpath filtering or using kubectl describe. OR finally relying on the new kctrl to show the info?

WDYT @cppforlife ?

Looking for opinions on field naming:

@seemiller do you all have any concrete use cases for these fields? Would really help us narrow down the naming conventions.
Does accessing through kubectl describe and eventually kctrl seem sufficient to you?

@neil-hickey App-toolkit was just added to TCE as the first package that bundles other packages, so that's a  pretty concrete use case. This change will greatly enhance the ability of package authors to identify what software is included.
I agree that including a potentially large amount of information in a standard kubectl get pkg/... call would not be helpful. Using describe, or adding a -o wide option, would suffice for a first cut of this in my opinion.

Given that "a package of packages" is typically implemented as a package that contains PackageInstall CRDs, would it be possible for kapp-controller to introspect the package for this and display the include software name/version/description?

this is really valuable! I'm glad we are doing it. I know @jorgemoralespou will like it. thanks @seemiller for letting me know about this PR.

Looking for opinions on field naming:

Could we just includedPackages given this is mostly going to be applicable for package of packages? Or is the use-case also for packages that contain multiple pieces of software? Not too strongly opinionated here. includedSoftware is fine too IMO.

Does accessing through kubectl describe and eventually kctrl seem sufficient to you?

I would definitely want it to be enabled by tanzu package eventually, but I understand as a starting point kubectl and kctrl are good places to start.

Given that "a package of packages" is typically implemented as a package that contains PackageInstall CRDs, would it be possible for kapp-controller to introspect the package for this and display the include software name/version/description?

I agree with this. One question:

Do we plan to have a more concrete/first-class name for the concept of package of packages?

This should be for any package

This should be for any package

i can confirm that this is just plain metadatat -- applied to any package.

Do we plan to have a more concrete/first-class name for the concept of package of packages?

no, we do not at this point.

Thanks for the input everyone!

Given that "a package of packages" is typically implemented as a package that contains PackageInstall CRDs, would it be possible for kapp-controller to introspect the package for this and display the include software name/version/description?

Not currently, and I don't expect this is something we could support. The only way for KC to "tell" that a Package contains Packages is to look at the output and then parse those yaml files to get info.

Could we just includedPackages given this is mostly going to be applicable for package of packages? Or is the use-case also for packages that contain multiple pieces of software? Not too strongly opinionated here. includedSoftware is fine too IMO.

includedPackages makes sense to me. /cc @cppforlife

hi @seemiller / @kartiklunkad26 Do you all have any sense of urgency on this request? We are not planning on cutting a new KC immediately with this change, so if it is urgent we could do that.

When would a new KC be available with this change? If it is a few weeks/a month, then I think that's fine.
If it's much more than that, then that would feel very late.


Bypass MAX_ITERATIONS and MAX_BUDGET_PER_TASK

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Closes #1493
Give a brief summary of what the PR does, explaining any non-trivial design decisions
Introduced TRAFFIC_CONTROL_STATE to allow OpenDevin to switch between normal traffic limiting mode and temporarily disabled mode.
Demo
I set MAX_ITERATIONS=1 so I have to hit "RESUME" button multiple times to get the result step by step. I think it's also useful when you'd like the agent to solve the task step-by-step under your surveillance.

This feature is only available on web GUI. If you are using main.py, then the control flow would be paused and there's nothing you could do but quit the program:

I realize this could make evaluation never finish. May need to do some trick in main.py

I realize this could make evaluation never finish. May need to do some trick in main.py
UPDATE: fixed. Root cause was that root agent's state was RUNNING even thought the delegate agent's state was PAUSED. The fix is to always return the current "active" agent's state. Per current design, if there's a delegate, then the "active" agent is delegate.

Just curious, why would it affect evaluation, I thought we aren't using delegates in evals, or are we?
Another thought, I haven't tested this PR with cli sessions, I wonder if it still works to restore that session when the app dies to an error of this kind. If it doesn't work, we may want to consider simply clearing this status upon restore from cli, at least for now. FWIW currently it works to restore from budget error (it's cleared), not sure from max_iterations.
I don't think that needs to hold this PR, this is a very helpful improvement for the user experience with UI.

Just curious, why would it affect evaluation, I thought we aren't using delegates in evals, or are we?

You are right, it won't affect evaluation because no evaluation uses delegates at the moment.

FWIW currently it works to restore from budget error (it's cleared)

That surprises me. Shouldn't the accumulated cost be serialized as part of metrics in State?

FWIW currently it works to restore from budget error (it's cleared)

That surprises me. Shouldn't the accumulated cost be serialized as part of metrics in State?

Sorry I didn't see the notification. Yes, I thought so too. Without re-testing just now, I think the explanation is or was in the line:

self.state.metrics = self.agent.llm.metrics

I think the explanation is or was in the line:
self.state.metrics = self.agent.llm.metrics

That makes sense, and I think that's desired behavior
One thing that doesn't make much sense anymore is max_iterations. When a session is restored, we probably should update max_iterations. Restore likely doesn't work with delegates either. But that's separate issue...


Terminal is not always focused when opening panel

Focus the terminal, close the panel and reload the window
Press cmd+J to show the panel, with the terminal persisted to be the active view
The terminal should be created and focused. Sometimes, it's not

I think this is new in 1.54. Seems like it usually reproes the first time the panel is opened. But, if I kill all terminals, causing the panel to close, then press cmd+J again, it focuses the new terminal correctly.

repro:

create a terminal
kill it (panel closes)
reload window
open panel -> terminal isn't focused

https://github.com/microsoft/vscode/blob/9093f265a64f640c3fb385195f6b1883deea2b93/src/vs/workbench/contrib/terminal/browser/terminalService.ts#L[…]8
isn't getting hit when I put a breakpoint there and open the panel


No longer able to discover any icon when using @iconify/json

Since https://github.com/egoist/tailwindcss-icons/releases/tag/v1.4.0 release and #29 or maybe https://github.com/egoist/tailwindcss-icons/commit/b53dde99c57d8f0735a851c6065d4313a13cbdc5, I'm not able to discover any icon when using the @iconify/json package.
It was done previously by passing all to getIconCollections.
I've also tried to use the dynamicIconsPlugin, but I don't really want to use such syntax  as mentioned here as it would cause a breaking change.

Sorry, I forgot to mark this as a breaking change.
Maybe i should revert and still support all options? But this also seems to have an impact on the auto-completion of the tailwindcss vscode plugin. Or there is no problem when using it with UIcon?


Fehler beim Import: B-Tocs  abap_btocs_core (SDK-Package) über ABAPGIT ( S/4HANA 2022  Trial - https://hub.docker.com/r/sapse/abap-cloud-developer-trial))

Hi,
there will probably be more errors. I can't import the solution into the customer system.
I also noticed that some domains and/or data elements are not available there (S/4HANA FI, MM; SD ... SAP_BASIS 757 0002) as well, e.g. Tel, Uri (customer S/4HANA FI, MM; SD ... SAP_BASIS 757 0002, current system).
Your solution is really good for developments in your own development system (e.g. ABAPTRIAL2022).
I would want to run the solution on a powerful laptop because of the use of larger LLMs with Ollama (B-Tocs ABAP Plugin - OLLAMA Connector).
Error messages when importing https://github.com/b-tocs/abap_btocs_core.git Package ZBTOCS_CORE:
Output length (22) is less than the calculated output length (23)
Value table ZTC_BTOCS_PRF is not active
**Field EMAIL: Component type or domain used not active or does not exist
Field FAX: Component type or domain used not active or does not exist
Field TEL: Component type or domain used not active or does not exist
Field URI: Component type or domain used not active or does not exist
Could not determine structure width of ZBTOCS_BOD_S_DOC_BUPA_ADR
Initialization of Extensibility Contract Check failed.
Execution of the consistency checks failed.
Data type does not exist
Nametab for table ZBTOCS_BOD_S_DOC_BUPA_ADR cannot be generated
Field EMAIL: Component type or domain used not active or does not exist
Field FAX: Component type or domain used not active or does not exist
Field TEL: Component type or domain used not active or does not exist
Field URI: Component type or domain used not active or does not exist
Could not determine structure width of ZBTOCS_BOD_S_DOC_BUPA_CEN
Initialization of Extensibility Contract Check failed.
Execution of the consistency checks failed.
Data type does not exist
Nametab for table ZBTOCS_BOD_S_DOC_BUPA_CEN cannot be generated
Field name KEY is reserved (Do not use structure as include in DB table)
Field name POSITION is reserved (Do not use structure as include in DB table)
Field name OFFSET is reserved (Do not use structure as include in DB table)
Type ZBTOCS_BOD_S_DOC_BUPA_CEN for component CENTRAL is not active
Type ZBTOCS_BOD_S_DOC_BUPA_ADR for component ADDRESS is not active
Routine EXPAND (Table ZBTOCS_BOD_S_DOC_BUPA could not be expanded)
Field name KEY is reserved (Do not use structure as include in DB table)
Table ZBTOCS_BOD_S_DOC_BUPA could not be activated
Table ZBTOCS_BOD_S_DOC_BUPA_ADR could not be activated
Table ZBTOCS_BOD_S_DOC_BUPA_CEN could not be activated
Table Type ZBTOCS_BOD_T_DOC_BUPA_CON was activated (error in the dependencies)
Table Type ZBTOCS_BOD_T_DOC_BUPA_REL was activated (error in the dependencies)
Domain ZBTOCS_LLM_EMBEDDING_VALUE was activated (warning for the dependent tables)
Domain ZBTOCS_RWS_PROFILE was activated (warning for the dependent tables)
Data Element ZBTOCS_DOC_KEY_CHAR was activated (warning for the dependent tables)
Data Element ZBTOCS_DOC_VALUE_CHAR was activated (warning for the dependent tables)
Table ZBTOCS_S_DOC_KEY_VALUE was activated with warnings
Table ZBTOCS_S_KEY_VALUE was activated (warning for the dependent tables)
Table ZBTOCS_S_TABLE_METAINFO was activated (warning for the dependent tables)
Many thanks for your help in advance
BG Heinzpeter

Now:
Field EMAIL: Component type or domain used not active or does not exist
Field FAX: Component type or domain used not active or does not exist
Field TEL: Component type or domain used not active or does not exist
Field URI: Component type or domain used not active or does not exist
Could not determine structure width of ZBTOCS_BOD_S_DOC_BUPA_ADR
Initialization of Extensibility Contract Check failed.
Execution of the consistency checks failed.
Data type does not exist
Nametab for table ZBTOCS_BOD_S_DOC_BUPA_ADR cannot be generated
Field EMAIL: Component type or domain used not active or does not exist
Field FAX: Component type or domain used not active or does not exist
Field TEL: Component type or domain used not active or does not exist
Field URI: Component type or domain used not active or does not exist
Could not determine structure width of ZBTOCS_BOD_S_DOC_BUPA_CEN
Initialization of Extensibility Contract Check failed.
Execution of the consistency checks failed.
Data type does not exist
Nametab for table ZBTOCS_BOD_S_DOC_BUPA_CEN cannot be generated
Type ZBTOCS_BOD_S_DOC_BUPA_CEN for component CENTRAL is not active
Type ZBTOCS_BOD_S_DOC_BUPA_ADR for component ADDRESS is not active
Routine EXPAND (Table ZBTOCS_BOD_S_DOC_BUPA could not be expanded)
Table ZBTOCS_BOD_S_DOC_BUPA could not be activated
Table ZBTOCS_BOD_S_DOC_BUPA_ADR could not be activated
Table ZBTOCS_BOD_S_DOC_BUPA_CEN could not be activated
Table Type ZBTOCS_BOD_T_DOC_BUPA_CON was activated (error in the dependencies)
Table Type ZBTOCS_BOD_T_DOC_BUPA_REL was activated (error in the dependencies)


PyQt program compiles and threads run but all i can see is the frame of the window not the contents...

PyQt program compiles and threads run but all i can see is the frame of the window not the contents... I use QThreads if it's relevant and i also contact a lot of servers in these threads.
[mainfraeme test]$ python -m nuitka --version
0.6.5
Python: 3.7.4 (default, Jul 16 2019, 07:12:58)
Executable: /usr/bin/python
OS: Linux
Arch: x86_64
python -m nuitka --version
python installed through pacman, nuitka installed through pip since mirrors only have 0.6.4.
Can't seem to reproduce with short example program. My code is 7000 lines and id rather not share.
Any ideas to what it could be? Also how to troubleshoot. Note i don't know C nor the first thing about compilers. Any help appreciated!

This is normal with PyQt5, only PySide2 and PySide6 are well supported. For PyQt6 timers may not be working with threads.


[Bug]: Side navigation bar is not showing login button in mobile view

Description

The side navbar is not rendering properly resulting blocked display of login button
In desktop view and it is working properly, But in mobile view it's not showing.
Tried with brave and chrome browsers in mobile and faced the same issue

Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
Chrome

@Nishitbaria Can you assign this issue to me under GSSOC'24? I can update this bug

@Nishitbaria Can you assign this issue to me under GSSOC'24? I can update this bug

Hey @01Sriharsha  I am assigning this issue to you
you can start working on this issue
Happy hacking ✨

@Nishitbaria I tried to create a PR. But it failed! What should I do now?

@Nishitbaria I tried to create a PR. But it failed! What should I do now?

Show me ss what the problem


Update comments in EuTxManager.java, GETTING_STARTED.md and GETTING_STARTED_KR.md

I request to update comments about process() method in EuTxManager.java and document descriptions in GETTING_STARTED.md, GETTING_STARTED_KR.md.

@yeonns @designe Thanks for letting me know these! I applied your feedback!!

Great! I appreciate your fast response :)
Thanks for your contribution!
Here is your contributor certification!
https://promise.is/verified/euphony/certificates/SeonJK

@designe Thanks for missing part😂!! I applied your feedback!


Questions about config

I don't see any reference to ranger or xfce terminal but I can't assign the keybind to anything else

im a,fucking dumbass i had been doing i3 restart and I thought it was updating idk I rebooted twice and now it's working


enableGlobalCutMode could't work

Hello! I like the geoman library very much.
When I was doing my own demo, I found that the "draw" method in asynchrony could not trigger the "cut".
Here is the demo
https://jsfiddle.net/4okvL2qb/

Wait for enableDraw('Polygon') in setTimeout
draw a polygon
draw another polygon and cut the first polygon
the first polygon can't be cut

Wait for your reply!

Your code:
map.pm.enableDraw('Polygon', {
  continueDrawing: true
})
setTimeout(() => {
  map.pm.enableDraw('Polygon')
}, 5000);
map.on('pm:create', e => {
  map.pm.enableGlobalCutMode({
    continueDrawing: true
  })
})

There is a problem with enabling the cut mode in pm:create. You enable Poylgon drawing with continueDrawing: true this means that it automaticly re-enable for drawing. BUT you also want to enable the cut mode. Now we need to know in which order the functions are executed:

enable poylgon drawing
create polygon -> fire pm:create
enable cut mode
because of continueDrawing: true cut mode is disabled because polygon drawing is enabled again
So this means to us, that the cut mode is not enabled and you draw a polygon again

You need to remove the cut mode function in pm:create or remove continueDrawing: true

@Falke-Design
Thanks for your reply.
I probably understand what you mean. Here is my real application scenario.
https://jsfiddle.net/yazuL9c0/

click "draw multipolygon" button
click "draw hole polygon"

I just want to be able to switch the “add” mode and "cut" mode when I click these two buttons. At same time, these two mode need continueDrawing: true .
I try to use map.off('pm:create'), but it doesn't work.
So what other solutions can solve this problem?
Wait for your reply!
Thanks!

Try to add continueDrawing: false to the hole polygon:
map.pm.enableDraw('Polygon',{
        continueDrawing: false
      })

This is needed because the options are saved, so when you call multipolygon with continueDrawing: true and call map.pm.enableDraw('Polygon') it has continueDrawing: true too.

Thanks a lot.
Problem has been resolved.
But I personally recommend that this problem be noted in the API.
Thanks to the geoman team for providing us with such a great plugin.
Keep going!

Nice to hear!
We already have it documented:

The options will persist, even when the mode is enabled/disabled via the toolbar.

But maybe not on the perfect place


Touching Polyline hard to trigger

Issue Type
[x] Question
Description and/or steps/code to reproduce the problem
I have a general touch handling overlay + quite few overlays on my layer which are circle polylines (thin for there are many of them). The polylines have infowindows attached but they are annoyingly hard to trigger with touch on my high resolution tablets.  I need to touch many times to hit the thin lines.  Is there a way to improve sensitivity?
I looked briefly at Polyline code: it depends screen density but the sensitivity seem closed for modifications.
If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
6.1.0

@pasniak You're right, we should be able to change the touch sensitivity for Polylines (cf. https://github.com/osmdroid/osmdroid/blob/master/osmdroid-android/src/main/java/org/osmdroid/views/overlay/Polyline.java#L220 I guess)
Feel free to send a pull request.

PR created


feedback service problem

I sent some message to a list of 10 push token. Only 1 of them was valid (belonging to a device that's still alive). I received the notification on the device and received no feedback for hours.
After a few hours i got a response from the feedback service invalidating all the tokens including the one that worked just fine.
This is the thing i am writing https://github.com/davibe/push-notification-service
And this is the specific apn usage https://github.com/davibe/push-notification-service/blob/master/src/services/apn.coffee

How you fix it? I am facing similar issues.

That is probably valid behaviour the timestamp is the crucial bit here. Please read the documentation from Apple that explains how to handle the feedback service.


Add why

It would be cool to have a "How to use" section in the ReadMe.md for usage as docker image only.
Or an example of a docker-compose file to immediately start testing / using it.

Created #10 for example.


Flickering pins on annotation updates

ClusterKit is causing a flickering issue on custom annotation views using MapKit.
I have tried solving this issue by using: - addAnnotations(...) and - removeAnnotations(...),
but that doesn't seam to solve the problem.
I'm calling - updateClustersIfNeeded in -mapView(_:, regionDidChangeAnimated:) as described in the documentation.
Tested with:

iOS Version 11.0 GM Seed

Thanks for the report @roccx , I've also noticed that flickering on iOS11
I havent found a solution yet.

I'm using custom annotations views with an animation during the update and running iOS11.
Everything works perfectly but the animation occurs also for clusters/annotations that don't change during the update.
For me it seems like that during mapView(_:, regionDidChangeAnimated:) all the annotations are rebuilt even the ones that should not be updated. Could the flickering related to this?

I think the issue is that every annotation is replaced by new ones as @JacoGasp says. Isn't there a way to filter annotations by -isEqual(_:) or -hash?

I did some debugging and I can say, that the flickering just happens when zooming not when moving the map to west/east/north/south. mapView(_:, regionDidChangeAnimated:) just rebuilt the annotations when zooming and moving the map. When moving without zooming it doesn't happen. Maybe there is some issue with the zooming?

It seems like MKMapView is delaying the rendering process of new annotations in some way. Yes I think the annotations are all rebuild because the algorithm instance is getting replaced on zoom.
I think a solution for that problem would be to manage the annotations in a more smarter way or just delay the removal of the old annotations by 2 or 3 frames.

@frgmt the refresh occurs to me also panning the map without zooming.

The issue appears everytime the clusters are updated, on zoom or significant pan. As @roccx suggested, adding an annotation to MKMapView seems to be delayed. The idea would be to update the clusters (replace, expand, collapse) more efficiely. The update is made in the private method updateMapRect:animated:, the implementation should be rethink.

@maxep is anyone working on the solution for this issue? Please let us know when this is fixed.
In the time being is there any good alternative to fix this issue?

I had the similar problem, and after some research, it happens that ClusterKit reloads each annotations on each pan, rotation etc.
I began to debug the lib, but didn't found any problem in the algorithm which could cause this bug.
As some said above, the underlying problem is that annotation adding are delayed in MKMapView compared to annotation removal (even if they are added before other get removed.)
In my case, the issue was mainly visible because I have several type of annotationViews, on which I update the image depending on type of annotation to display.
As annotationViews got reused, same pin could be reconfigured from a previous recycled pin (with a different image), causing a glitch on each reload.
My workaround consists in having 1 reuseIdentifier per annotation type (including CKCluster), thus configuring my annotationViews only after instantiating them.
As a result, my annotationViews are 100% stable, without any glitch.
Hope that helps !

@thibauddavid Thank you for sharing your solution for this issue.

I've released a new version : 0.3.0 including breaking changes answering this issue.
The clusters' updates and animations are now more efficient and as @thibauddavid described, making a good use of the reusable annotation view is a good practice.
Please check the CHANGELOG

Just read new algorithm, seems much more efficient 🔥


Create a CHANGELOG.md

Track high level changes to the source code in a single file

I'm impressed! I figured we'd just start at 0.3.8 but you went retroactive to 0.3!
Good job! :+1:

Well done!


[Idea] Move check to monitoring-checks repo

We recently created a new repository to hold all our monitoring-checks.
Do you want to put your check_springboot_actuator check into there, too?
It would basically mean moving the whole contents to a new folder here: https://github.com/telekom-mms/monitoring-checks/tree/main/check-plugins

@rndmh3ro we currently have the check registered in exchange.icinga.com. As far as I know they support only a single item per repository. Merging the check with the other checks would break this.

I took a look at other checks at the exchange.
The Linuxfabrik-Monitoring Plugins can be found here: https://exchange.icinga.com/Linuxfabrik/Linuxfabrik Monitoring Plugins Collection
There are dozens of checks in there. So in theory it should be possible to register the monitoring-checks repo to the exchange. Do you use any other special features I'm not aware of that would make this impossible?

I looked into their GitHub repository, but could not find an icingaexchange.yml. That makes me wonder how they created/update that Icinga Exchange entry. Maybe manually or somehow combined with their release process. Needs investigation.


[magic-enum] Bump to 0.9.3

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Tested usage successfully by magic-enum:x64-windows:
magic-enum provides CMake targets:

    find_package(magic_enum CONFIG REQUIRED)
    target_link_libraries(main PRIVATE magic_enum::magic_enum)


:sparkles: suggest values when filling-up form fields

solve #5

Coverage: 64.193% (-0.4%) from 64.611% when pulling 4cf20f5e736098db38d4308e9ccf249ce027d41a on issue-5 into 76286ecc8ccd3073a14e445290a7fa2ab1474293 on devel.


Document agent version for TFS 2017 Update 3

TFS 2017 Update 3 is now available.
Please update the minimum agent versions table to include information about this update.

Completed by PR referenced above. Thanks!


updated snana sources with IAU names

and downcased the whalen source names in the registry loader.
See also the pull request in sncosmo.github.io to update the urls.json file accordingly:

Looks good to me. I'll wait until we merge sncosmo.github.io#2 first though.

BTW, sorry I've been shifting things around in builtins.py so much recently. Glad to see you could still find your way around.
I realized recently that with how I've set it up now, the urls.json file is fetched once per session, regardless of whether or now the data has already been downloaded. This is undesirable for cases where you want to work offline. I'm working on a fix but it will take a bit more time.


isolate pattern tests with --language flag

@RunDevelopment all yours :)

Thank you for contributing @JaKXz!


Doing Backoff on Individual Call

Hello,
When doing large calls (like nb.dcim.devices.all()), it could happen that one call fails (for example when we you have 50000 devices with paging set at 500, there is 100 calls). If one call is failing, the whole function is in error when send back to the client.
This issue did bother me by doing retry in my own code when this is sub efficient especially on large calls.
I did patch pynetbox by adding a retry decorator (standard behaviour when interacting with an API). This allow a retry in seamless way from the client point of view and allow to handle random errors that could occurs due to load on Netbox side.
In my case I used backoff with:
import backoff
def backoff_hdlr(details):
    """
    Helper function used to print details when an exception is caught by backoff

    Arguments:
        details {dict} -- Backoff dict containing info to print
    """
    print(
        "Backing off {wait:0.1f} seconds afters {tries} tries "
        "calling function {target} with args {args} and kwargs "
        "{kwargs}".format(**details)
    )
[...]
class Request(object):
[...]
    @backoff.on_exception(
        backoff.expo, (RequestError), max_tries=8, on_backoff=backoff_hdlr
    )
    def _make_call(
        self, verb="get", url_override=None, add_params=None, data=None
    ):
[...]

Can we consider to implement this in pynetbox ?

FYI, for those reading this and wanting to monkey patch this behaviour without modifying pynetbox, the following snippet is working:
import backoff
def backoff_hdlr(details):
    """
    Helper function used to print details when an exception is caught by backoff

    Arguments:
        details {dict} -- Backoff dict containing info to print
    """
    print(
        "Backing off {wait:0.1f} seconds afters {tries} tries "
        "calling function {target} with args {args} and kwargs "
        "{kwargs}".format(**details)
    )
retry_timeout = backoff.on_exception(backoff.expo, (pynetbox.RequestError), max_tries=8, on_backoff=backoff_hdlr)
# Decorate Netbox call to handle individual requests error in large batch
pynetbox.core.query.Request._make_call = retry_timeout(pynetbox.core.query.Request._make_call)


Optimize MoveGenerator::generate_captures

Only generate captures in the first place instead of generating all
moves and then retaining the captures

Codecov Report

Merging #124 (d61ea15) into main (919cfb2) will decrease coverage by 0.08%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
- Coverage   94.09%   94.01%   -0.09%
==========================================
  Files          68       68
  Lines        7860     7921      +61
==========================================
+ Hits         7396     7447      +51
- Misses        464      474      +10

Impacted Files
Coverage Δ

...egen/src/move_generator/move_generator_template.rs
76.55% <81.63%> (+0.89%)
:arrow_up:

movegen/src/move_generator.rs
99.62% <93.75%> (-0.13%)
:arrow_down:

Continue to review full report at Codecov.

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


nix traefik test is flaky

Describe the bug
The traefik nix test is flaky, as pointed out in https://github.com/NixOS/nixpkgs/pull/113878#issuecomment-782817274.
To Reproduce
Run nix-build nixpkgs/nixos/tests/traefik.nix several times. In personal experience about 1 in 3 test runs fail.
Expected behavior
The test always succeeds.
Screenshots
Relevant excerpts from the referenced PR:
Failing test:
[nix-shell:~/.cache/nixpkgs-review/pr-113878-1]$ nix-build nixpkgs/nixos/tests/traefik.nix
...
traefik # [   16.605144] dhcpcd[726]: veth3aa29b9: soliciting a DHCP lease
Check that a container can be reached via Traefik
client: must succeed: curl -sSf -H Host:nginx.traefik.test http://traefik/
client # curl: (22) The requested URL returned error: 404 Not Found
client: output:
Test "Check that a container can be reached via Traefik" failed with error: "command `curl -sSf -H Host:nginx.traefik.test http://traefik/` failed (exit code 22)"
error:
Traceback (most recent call last):
  File "/nix/store/c0zxi1j4ch5c0v446k4m31crx4npkb82-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 897, in run_tests
    exec(tests, globals())
  File "<string>", line 1, in <module>
  File "<string>", line 13, in <module>
  File "/nix/store/c0zxi1j4ch5c0v446k4m31crx4npkb82-nixos-test-driver/bin/.nixos-test-driver-wrapped", line 422, in succeed
    raise Exception(
Exception: command `curl -sSf -H Host:nginx.traefik.test http://traefik/` failed (exit code 22)

Succeeding test:
Check that a container can be reached via Traefik
client: must succeed: curl -sSf -H Host:nginx.traefik.test http://traefik/
traefik # [   16.630064] 9d3fb07985a7[529]: <IP_ADDRESS> - - [21/Feb/2021:07:53:40 +0000] "GET / HTTP/1.1" 200 52 "-" "curl/7.74.0

Test succeeding by waiting for the success:
client: waiting for success: curl -sSf -H Host:nginx.traefik.test http://traefik
client # curl: (22) The requested URL returned error: 404 Not Found
traefik # [   26.299894] c564e47bdead[619]: <IP_ADDRESS> - - [07/Mar/2021:15:25:13 +0000] "GET / HTTP/1.1" 200 52 "-" "curl/7.74.0"
(1.54 seconds)
Check that a container can be reached via Traefik
client: must succeed: curl -sSf -H Host:nginx.traefik.test http://traefik/
traefik # [   26.342905] c564e47bdead[619]: <IP_ADDRESS> - - [07/Mar/2021:15:25:13 +0000] "GET / HTTP/1.1" 200 52 "-" "curl/7.74.0"

Additional context
It seems to be purely a timing issue, see further notes https://github.com/NixOS/nixpkgs/pull/113878#issuecomment-792690659.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.11.1, NixOS, 21.05pre-git (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.10`
 - channels(root): `"nixpkgs-maxine, home-manager"`
 - channels(maxine): `"home-manager"`
 - nixpkgs: `/var/run/current-system/systems/nixpkgs`

Maintainer information:
attribute: traefik
# a list of nixos modules affected by the problem
module:

I've seen this on traefik too -- it takes some time from serving 404s to being fully operational. Maybe there's some service endpoints that could be used instead? (Just spewing some ideas)


Request minor improvement to QSG guide

Description:
The json payload should be listed under pine valley hospital service endpoint which accepts a json payload. As shown in the attached image, in the QSG intro section at [1] this json payload is listed under grand oak hospital service endpoint.

This is documented correctly at [2] as shown below.

[1]. https://ei.docs.wso2.com/en/7.2.0/micro-integrator/overview/quick-start-guide/
[2]. https://ei.docs.wso2.com/en/7.2.0/micro-integrator/develop/integration-development-kickstart/
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

We are revamping the quick start guide with https://github.com/wso2/docs-mi/issues/791. Hence resolving this.


Need process number in afterTextChanged

Hi, u need process number in afterTextChanged, i have a bug with fast typing numbers because of it

Hello, fixed in with release 2.0.0. Thank you.


WASD controls to move the character, and find the trophy!
Code

[ x] The game was made using the Sprig editor.
[ x] The game is placed in the in the /games directory.
[ x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[ x] The game runs without errors.
[ x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[ x] The game name is not the same as the others from gallery

Image (If an image is used)

[ N/A] The image is in the /games/img directory.
[ N/A] The name of the image matches the name of your file.

I added running physics, better more detailed maps and bitmaps, and a chest-key-lock mechanism to differentiate it

@grymmy lmk what you think

Nice work on improving this game- I really enjoyed the walking animation. You should see it in the Sprig gallery tomorrow!
If you're in high school, you can request a free Sprig console at this link and join our online community to talk to other teenagers making Sprig games.


Fix usage with Minitest

Was raising since an upgrade of Minitest.
gems/pry-rescue-1.5.2/lib/pry-rescue/minitest.rb:7:in `<main>': uninitialized constant Minitest::Unit (NameError)

if defined?(Minitest::Test) && Minitest::Unit::VERSION.split('.').first.to_i >= 5

👋
Had the same issue and this fixes it (fresh rails app using Minitest)
Thanks!


fix(amplify-appsync-simulator): allow token.iss ending with slash

Issue #, if available:
https://github.com/aws-amplify/amplify-cli/issues/5372#issuecomment-758247839
Description of changes:

Improved isValidOIDCToken(): Added support for token.iss ending with slash. (Auth0)
Added unit tests

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

@yuth can you please review this PR?
Previous change to this file was: https://github.com/aws-amplify/amplify-cli/pull/5434

@yuth can you please review this PR?
Previous change to this file was: https://github.com/aws-amplify/amplify-cli/pull/5434

@attilah thank you for reviewing this PR!
Can you give estimate when this fix will be merged and released? I'm not sure about next steps...
Thank you!

@attilah thank you for reviewing this PR!
Can you give estimate when this fix will be merged and released? I'm not sure about next steps...
Thank you!

@safonovklim will be in the next release.

@safonovklim will be in the next release.

@attilah awesome!! thank you!

@attilah awesome!! thank you!


[Feature Request] Skype calling bot voice support for Bing voices

Bot Info

SDK Platform: .NET
SDK Version: 3.15
Active Channels: Skype
Deployment Environment: Azure Bot Service

Issue Description
I am opening a new issue that is the same as #2886
The answers provided require a call to the Bing API (or alternatively using a local TTS) however my explicit request was that for a Skype CALLING bot that it natively supports the Bing voices. I do not want to have to use an external TTS, the Skype bot API on the Prompt should allow you to configure the voice to use, you do not have any option other than the single voice Skype provides.
As mentioned, if you use an external TTS that takes time, you then need to send back an WAV file to the client. All i want to do is send back the text and the voice and let Skype sort it out, this seems by far the most efficient way.
Code Example
None
Reproduction Steps
Try and change the voice Skype using in its calling bot!
Expected Behavior
As per description
Actual Results
You cannot change the voice and for that matter even the gender - it is always female!

Thank you for your input.  This is not something that is currently on our radar but is something we could look at adding in the future.

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


rtl support

hi,
Can you add rtl support (not only css issue but all tree layout incuding arrows)?

Hi,
checkout https://angular2-tree.readme.io/v6.1.0/docs/rtl


npm audit found vulnerabilities

# npm audit report

semver  <5.7.2 || >=6.0.0 <6.3.1
Severity: moderate
semver vulnerable to Regular Expression Denial of Service - https://github.com/advisories/GHSA-c2qf-rxjj-qqgw
semver vulnerable to Regular Expression Denial of Service - https://github.com/advisories/GHSA-c2qf-rxjj-qqgw
fix available via `npm audit fix`
node_modules/@babel/core/node_modules/semver
node_modules/@babel/helper-compilation-targets/node_modules/semver
node_modules/conventional-changelog-writer/node_modules/semver
node_modules/istanbul-lib-instrument/node_modules/semver
node_modules/read-pkg/node_modules/semver

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix

# npm audit report

semver  <5.7.2 || >=6.0.0 <6.3.1
Severity: moderate
semver vulnerable to Regular Expression Denial of Service - https://github.com/advisories/GHSA-c2qf-rxjj-qqgw
semver vulnerable to Regular Expression Denial of Service - https://github.com/advisories/GHSA-c2qf-rxjj-qqgw
fix available via `npm audit fix`
node_modules/@babel/core/node_modules/semver
node_modules/@babel/helper-compilation-targets/node_modules/semver
node_modules/conventional-changelog-writer/node_modules/semver
node_modules/istanbul-lib-instrument/node_modules/semver
node_modules/read-pkg/node_modules/semver

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix


influxdb: add missing statds/dogstastd metrics

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area outputs
What this PR does / why we need it:
influxdb: add missing statds/dogstastd metrics
Which issue(s) this PR fixes:

Fixes partially https://github.com/falcosecurity/falcosidekick/issues/144
Special notes for your reviewer:
/assign @Issif

@cpanato: The label(s) area/outputs cannot be applied, because the repository doesn't have them

In response to this:

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area outputs
What this PR does / why we need it:
influxdb: add missing statds/dogstastd metrics
Which issue(s) this PR fixes:
Fixes partially https://github.com/falcosecurity/falcosidekick/issues/144
Special notes for your reviewer:
/assign @Issif

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: c2c259c9cb9420f51be2db706a0bdb9411ee904c


add comment note for node-label-selector filter logic

The parameter manageNodesWithLabelSelector is used in client-go opt filter logic , while the parameter manageNodesWithAnnotationSelector is in NodeController's filter nodeSelectorFunc logic, that confused me..

good catch
/ok-to-test

@wzshiming thanks , just now i saw the code ..
it is used in opt, so it is not  so this is not necessary

/approve
/lgtm

/kind cleanup


Obtain LE Certs Tasks Failing (Timeout errors)

The playbook is failing for me on the Obtain Let's Encrypt Certificate task. It mentions a timeout and it is likely a firewall issue to reference /opt/postal/.acme.sh/acme.sh.log for more info. See below link with my output of that log. I thought maybe it had to do with Port 80 not being open on Vultr but I added a Port 80 allow rule to my firewall on Vultr for the VM and still no dice. So for some reason, LE can't seem to verify for the cert because it can't access the verification file on the VM via port 80. This is my guess, but open to any input and how this can be fixed.
https://pastebin.com/wLxQm0Ut

Disregard the above. I setup a new Ubuntu 18.04 VM to just try everything from scratch and it worked. Not sure what was wrong with other VM, but something was messing up comms on port 80 I guess. UFW and Vultr both were allowing 80 so I'm not sure what it was.

Disregard the above. I setup a new Ubuntu 18.04 VM to just try everything from scratch and it worked. Not sure what was wrong with other VM, but something was messing up comms on port 80 I guess. UFW and Vultr both were allowing 80 so I'm not sure what it was.

Thanks for the follow up! No sweat about it. Stuff like this happens to me all of the time 🤣
Glad you were able to get this resolved!

Thanks for the follow up! No sweat about it. Stuff like this happens to me all of the time 🤣
Glad you were able to get this resolved!


flutter build ios-framework lack FlutterPluginRegistrant.framework

I want to add flutter in my ios app as frameworks, then when I run  flutter build ios-framework --output=/path/, the Release dir does not include FlutterPluginRegistrant.framework, just has App.framework and Flutter.framework. But my ios app need  FlutterPluginRegistrant.framework to do something.
Can someone help me? thanks ~🙏
here is my doctor info:
[✓] Flutter (Channel unknown, v1.15.3, on Mac OS X 10.15.3 19D76, locale zh-Hans-CN)
    • Flutter version 1.15.3 at /Users/aboo/flutter
    • Framework revision 67826bdce5 (10 days ago), 2020-02-10 14:59:32 -0800
    • Engine revision 6158f03ef5
    • Dart version 2.8.0 (build 2.8.0-dev.8.0 514a8d4c84)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/aboo/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    ✗ Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C504
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 43.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.42.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.8.1

Do you have Flutter plugins in your module?

@jmagman I have not plugins, here is my pubspec.yaml
dependencies:
  flutter:
    sdk: flutter
  cupertino_icons: ^0.1.2
  intl: ^0.16.0
  circular_check_box: ^1.0.1

dev_dependencies:
  flutter_test:
    sdk: flutter

at the flutter docs the simple code use GeneratedPluginRegistrant:
- (BOOL)application:(UIApplication*)application
didFinishLaunchingWithOptions:(NSDictionary<UIApplicationLaunchOptionsKey, id>*))launchOptions {
    self.flutterEngine = [[FlutterEngine alloc]<EMAIL_ADDRESS>project:nil];
    [self.flutterEngine runWithEntrypoint:nil];
    [GeneratedPluginRegistrant registerWithRegistry:self.flutterEngine];
    return [_lifeCycleDelegate application:application didFinishLaunchingWithOptions:launchOptions];
}

so in my ios project need import GeneratedPluginRegistrant.framework , I am not clear the framework's function.

You don't need to include GeneratedPluginRegistrant if you don't have plugins.
That detail should be included in the docs.

Reopening to track the documentation issue.

I'm going to claim that this is fixed by https://github.com/flutter/website/pull/3754


Maker to connect to should not be a commandline argument

Currently, which maker to connect to is a command-line argument.
Eventually, we want taker's to connect to one or more makers.
As an in-between step, we want to re-arrange the internals so that the maker to connect to is dynamic. We can optionally expose this through the UI as well.

Outdated for now, deal with it once we approach this topic.


Feature: add pre-checks to avoid failing at a later stage

remove OpenShift 4.12 / IPv6 installation blocker (issue didn't show up during tests)
add 00-provision-hetzner.yml to setup.yml as it doesn't do anything unless requested.
rename import_playbook to ansible.builtin.import_playbook.

Description
Checklist/ToDo's

[X] Added documentation?
[X] Added release note entry? (  docs/release-notes.md )
[X] Tested?

/ok-to-test


Prevented blink spells from triggering after player dies

The blink spell ball may not touch the surface while the player is respawning. This will lead to unwanted teleportation. I suggest removing this entity as soon as the player spawns.

Thanks! 👍

Thanks! 👍


Add a --dry-run to pipeline upload that dumps json

This is a follow on of https://github.com/buildkite/agent/pull/765.
It adds a buildkite-agent pipeline upload --dry-run command that outputs the pretty-printed pipeline json rather than uploading it. This works locally with no configuration required.

@toolmantim sure! Would be good to verify it works with bksrsly.

I've just done some standalone tests using various pipeline.yml files, and the env interpolation and JSON output seems to work really nicely! 🎉
I jumped into bksrsly to update it to so it works with it, but it's gonna take a bit more work than I thought, because sub-processes etc. I'll let you know when I've got something working 👌🏼

@lox Does this validate the schema of the pipeline too? Will it fail on invalid pipelines?


assigning w-id via invoked renderer

Per https://github.com/raptorjs/marko/issues/94
It would be very useful to be able to assign w-id(s) to renderers that are directly invoked, such as:
<for each="renderer in data.renderers">
    <invoke function="renderer(myInputObject, out)" w-id="widget[]" />
</for>

Fixed. New version published<EMAIL_ADDRESS>


Split CLI developer documentation

Following the CLI restructuring carried out in https://github.com/openmicroscopy/ome-documentation/pull/1151, this PR now splits the CLI developer documentation into separate pages.

Otherwise a useful split. It also raises the question of where to put pages about delete and chgrp: here with object creation or over is the sysadmin docs?

Agreed delete, chgrp but also obj plugins belong to the complex "power user" category. I do not have any strong opinion but it might be worth cross-referencing pages? My personal preference would go either to the developers section or to the users section (like login and import) /cc @hflynn

Maybe we need a separate section for cli power users? It would make some sense as there are commands that cross all boundaries even if some have limits for non-admin users.

Good to merge

Assuming RFEs are captured elsewhere.


Fix several issues with the code and make it work nicely on large set of videos

Hi @wanmeihuali
I would like to start a PR effort on a few recent fixes we developed for Taichi Gaussian Splatting (Taichi-GS). I had branched out from this commit ( https://github.com/wanmeihuali/taichi_3d_gaussian_splatting/commit/f7631e327d3e6e995324f1755bc3da25d603a584 ) and made a few fixes to make the current code work robustly on a wide range of examples (100+ scenes).
I tested the latest commit on main branch ( https://github.com/wanmeihuali/taichi_3d_gaussian_splatting/commit/2cea5deb3940079efe917ad32a07c8390907507a ), which crashes on many videos, I don't really have clues.
Particularly, I made a set of changes fixes several issues of the current project ( https://github.com/jb-ye/taichi_3d_gaussian_splatting/tree/eames ):
(1) It automatically scale the training image to no more than 1600 (if needed). Training higher resolution requires more fine-tuning of existing parameters, otherwise, you may observe undesirable outcomes like I mentioned here https://github.com/wanmeihuali/taichi_3d_gaussian_splatting/issues/144.
(2) It addresses the root cause of https://github.com/wanmeihuali/taichi_3d_gaussian_splatting/issues/119 and the issue was also reported by many other people on this forum who tried this repo.
The key idea to avoid numerical overflow is to always do a 2D convolution in pixel space after splatting 3D Gaussians. In official GS code here and here: the 2D Gaussian is convolved with an isotropic 2D Gaussian of σ=3⋅I to simulate pixel integration (this is not in the paper). However, even official code is not done this in proper way. See also discussion here.
Let us say you have a 2D Gaussian with an opacity of 1 at the center. When doing a convolution with another 2D Gaussian, if the opacity is currently left unchanged the Gaussian will become larger while remaining opaque and may obscure Gaussians that are behind (we observed that on grid patterns).
When we implemented this strategy for Taichi GS (was not part of original implementation), we noticed in one example this issue becomes quite obvious:
https://github.com/wanmeihuali/taichi_3d_gaussian_splatting/assets/132313008/076f2c3a-47f8-4c50-8b18-54ef8bdab068
When render camera moves from far to close (near the captured distance), we observe the color on the grid pattern of acoustic amplifier changes and creates aliasing like effect (though it is not aliasing). This issue is not usually reflected in standard metrics of validation images if those validation images are captured at similar distance as training images.
Instead, the opacity should be reduced so that the (2D) integrated opacity of the resulting Gaussian is the same as the original one. Thus in the 3DGS code the opacity should be multiplied by the factor  as sqrt(∣Σ+0.3I∣/∣Σ∣) . We implemented this rescaling factor into Taichi GS, and observed that the aliasing like effect is gone:
https://github.com/wanmeihuali/taichi_3d_gaussian_splatting/assets/132313008/0f88ed9b-cc29-431d-aefa-2d70ab393b1d
In our internal evaluation, I found this change also improves standard metrics (PSNR, SSIM, LPIPS).
(3) We also reduce the parameter densification-view-space-position-gradients-threshold: 3e-6 by half, and observe nice improvements in standard metrics albeit not being too slow.
Besides those fixes, I also implemented a simple offline renderer for rendering images from a custom trajectory.
I noticed a few more recent interesting changes (e.g. camera optimization, and depth cov loss), but they are not of our primary interest at the moment. Is there a way to move them to a separate experimental branch before they becomes mature and tested.

@jb-ye Nice work done! You can go ahead and create the PR. I still need some time to read your code. I'll try to roll back the master branch because many people reports crash(I guess the two new features: camera pose optimization and depth covariance loss causes some bug).

Okay, once you roll back the main branch. I will create the PR.

Hi @jb-ye , I've already rolled back the current main branch. You can go ahead and create the PR. Thanks!

Code merged. close it.

Hi, may I ask why the normalization factor is sqrt(∣Σ∣ / ∣Σ+0.3I∣) instead of 1 / sqrt(∣Σ+0.3I∣)? What do sqrt(∣Σ∣) compensate for? I mean, to normalized the convolution result such that the integral of the result gaussian is 1, we only exactly need 1 / (2π * sqrt(∣Σ+0.3I∣)), where 1 / 2π can be omitted since it's a constant.

Hi, may I ask why the normalization factor is sqrt(∣Σ∣ / ∣Σ+0.3I∣) instead of 1 / sqrt(∣Σ+0.3I∣)? What do sqrt(∣Σ∣) compensate for? I mean, to normalized the convolution result such that the integral of the result gaussian is 1, we only exactly need 1 / (2π * sqrt(∣Σ+0.3I∣)), where 1 / 2π can be omitted since it's a constant.
@jb-ye


Cloudflare doesn't verify and just keep loading, and when its not loading it just ask me to check the box... And then it starts to load again

Your issue may already be reported!
Please search on the issue tracker before creating one.
 I have searched a bunch about it, but nearly all tutorial I saw was for desktop and the few I saw for mobile seemed to have buttons that didn't show up for me

Open nclientv2 apk => setting => search Change User-Agen => open =>delete your user agent =>paste Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MRA58N) => save=> relog apk
I hope this work for you

change your user agent to this
Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MRA58N)
Its work for me with same problem

@Norikazen Sorry if this is a stupid question, but how do you change your user agent on mobile? I have searched a bunch about it, but nearly all tutorial I saw was for desktop and the few I saw for mobile seemed to have buttons that didn't show up for me

@Slasly
Open nclientv2 apk => setting => search Change User-Agen => open =>delete your user agent =>paste Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MRA58N) => save=> relog apk
I hope this work for you

Ah I thought I had the change something with the browser itself.
That explains why I couldn't figure it out.
Thanks for the help!

first you can try it, just reply to my comment if it works for you


Use any commit or version of Dataverse in container and Kubernetes

Currently, the upstream project does not contain up-to-date Docker images nor does IQSS provide officiall support these installation adventures (which is why this project came to live in the first place).
With the current rise of attention for salvation of IQSS/dataverse#4172, a demand to try things on K8s is rising, too. In addendum, it should be possible to develop, run and test any commit or code change on containers (enhanced by K8s), locally or within CI.
A current solution to this is usage of IQSS/dataverse-ansible in combination with ec2-create-instance.sh. This is fine, but requires an Amazon EC2 account and payment. Also, DataverseEU is going to run things on K8s and the word goes it might be a bad idea to have too much differences between development and production (coming from the principles of CD), especially when talking about technology stack.
Thus, a workflow is needed, which preferably is adaptable to different scenarios and use cases.
To all readers: any feedback is welcome on this. Please join the discussion.
Pinging @pdurbin, @4tikhonov, @donsizemore, @pameyer here.

Let me post a first idea about a possible workflow.
First, let us make some assumptions for this:

People interested in developing things on K8s most certainly will be ok with installing a tool or two.
As far as I understood it, interest in running Dataverse on K8s is relatively low at IQSS. Thus I will try to keep upstream code as much as possible out of this for now - the less code changes the better. (There will be more than enough for getting things like Config API in place...)
People don't like waiting. Try to cache as much as possible.
Anybody hacking on Dataverse will want to build and execute unit tests locally. It should be safe to assume Maven is installed.

A possible flow:

git clone dataverse-kubernetes
git submodule update --init will download upstream code in a subfolder.
This submodule can be versioned in the branch of the kubernetes project. It can point to any fork of upstream, etc. Thus people can develop in their forks/branches and follow their guidelines. Obviously, using submodules has its pitfalls. Everything comes at a price.
skaffold dev triggers the build and more (see https://skaffold.dev):

Build source code in multistage (so keep Maven and Java version in sync with app server container)
Build application image with build artifacts
Test with container-structure-test
Tag with commit id or branch. Maybe overrideable via variable?
Push to registry
Deploy to K8s
Watch for changes :-)

I just pushed commits with initial groundwork on this.
When you are feeling lucky, you should already have a usable image building manually.
Checkout the branch and do a docker build -t iqss/dataverse-k8s:4.14-test -f docker/dataverse-k8s/dev-payara/Dockerfile . from the source folder of this project. The initial build will take about 4 minutes. Succeding builds should be MUCH faster due to caching in Docker layers :smile:

Blockers:

GoogleContainerTools/skaffold#2432
GoogleContainerTools/skaffold#2547


Improve repo workflow

What do you need?
A better model for managing content changes.
Which entry/section is this related to?
https://github.com/AusDTO/gov-au-content-guide/
Good example
GitFlow.

Having a chat with Mick and @klepas about this on Thursday.

Would be good to do a template definition of done for issues/PRs.

Relates to https://github.com/AusDTO/gov-au-ui-kit/issues/400


[CELEBORN-882][WORKER][METRICS] Add Pause Push Data Time Count Metrics & Dashboard Panel

What changes were proposed in this pull request?
Add PausePushDataTime  Metrics
Why are the changes needed?
Count each celeborn worker pause time.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Cluster Test

Marked as draft, should do some logic optimize before this.

CI fail related to #1844, will rebase and test agine.

According to the snapshot, the metrics will monotonously increase. Is it better to define the metric as pause time in last xxx minutes?

@waitinfuture
Sorry for late reply, current add logic to force append pause spent time after N TRIM action.
As your comment to show pause time in last xxx minutes, IMO, users may care about how long the cluster in back-pressure state over this time.  pause time in last xxx minutes can tell us state in specifies the time frame, but not tell us macroscopically about the pause for the whole duration.
Current PR implementation can do both of the things mentioned above.
If you have any other ideas, please let me know.

@waitinfuture Thanks for your review, made correction, PTAL


Need to edit classpath in bin\spark-notebook.bat to run on Windows

To get spark-notebook to work on Windows I had to edit the file bin\spark-notebook.bat.
There is a line that sets the application classpath (line 132 in my build) and I had to add the conf directory to it:
set "APP_CLASSPATH=%CLASSPATH_OVERRIDES%;%YARN_CONF_DIR%;%HADOOP_CONF_DIR%;%EXTRA_CLASSPATH%;%APP_LIB_DIR%..\conf;%APP_CLASSPATH%"
I figured this out by comparing with the bin\spark-notebook bash script where the conf directory is already added to the classpath.
Is there a better way to do this or should this change be added to the build?

Getting org.slf4j.LoggerFactory file error when compiling the spark-notebook project
spark-notebook/modules/subprocess/src/main/scala/notebook/kernel/pfork/BetterFork.scala:12: object LoggerFactory is not a member of package org.slf4j
[error] import org.slf4j.LoggerFactory
spark-notebook/modules/core/src/main/scala/notebook/util/Logging.scala:3: object slf4j is not a member of package org
[error] import org.slf4j.LoggerFactory
These 2 errors are coming so pls help me bec i am new to this

@markpavey sounds legit, feel free to make a PR ;)

I have created a pull request: https://github.com/spark-notebook/spark-notebook/pull/917


🚧 feat: build on schedule every Monday and Thursday DO NOT MERGE

What this PR does / why we need it:
This PR will choose to automatically compile the RC version on Monday afternoon at 3:30 pm, or automatically compile the official version on Thursday afternoon at 3:30 pm, depending on whether the previous version was released or not;
At the same time, some PR(auto build PR) will be automatically processed, greatly improving the efficiency of developers
Will this PR make the community happier?
Yes
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
How is this PR tested

[ ] unit test
[ ] e2e test
[ ] other (please specify)

Special notes for your reviewer:
Release note:

automatically build on schedule every Monday and Thursday

now, this job has been assigned to David by #937


Documentation Update

Adding settings section for setting up application, more in-depth Usage cases. Implemented a Table of contents for ease of access of all sections

Closes #ISSUE_NUMBER

Thanks for the PR, but I don't think it provides much value.


Release Google.Cloud.ConfidentialComputing.V1 version 1.0.0-beta03

Changes in this release:
New features

Mark all fields Optional for ContainerImageSignagure proto (commit 99f3e64)

Created release for Google.Cloud.ConfidentialComputing.V1-1.0.0-beta03


Spectator keeps falling out of bounds resulting in massive console dump

If you stay dead for a while, this happens

IIRC that should be fixed with v1.4.0

Yup fixed.


BUG-11412 - datalake clusters in detailed env response

@attilapalfi92 integration tests failed

@doktoric I know


fix(docker): Updating docker image

:tada: This PR is included in version 3.4.4 :tada:
Your semantic-release bot :package::rocket:


Inspiration demo controls

This is an attempt at adding controls to demo components based on things set in the demoProperties field.
Currently I've only added support for:

Boolean values
Text Input
Calendar Input
Select Input (dropdown of options)

Somehow the changes I made in another branch for status indications for pulled into this, so disregard those 😅


groups 'kdc' and 'ldap' are used interchangeably

Hi everyone 👋  and thank you for your amazing work.
In deploy-ldap-kerberos.yml from tdp-getting-started, OpenLDAP and Kerberos servers are both installed on the kdc group :
# tdp-getting-started : deploy-ldap-kerberos.yml
- hosts: kdc
  tasks:
    - import_role:
        name: ansible_roles/collections/ansible_collections/tosit/tdp_extra/roles/ansible-ldap-kerberos

but you'll find direct reference to the first item of the ldap group or kdc group in certain situations like this one :
# tdp-collection-extras : roles/ansible-ldap-kerberos/defaults/main.yml
ldap_server: "{{ groups['ldap'][0] }}.{{ domain }}"
...
kerberos_server: "{{ groups['ldap'][0] }}.{{ domain }}"  # <-- kerberos_server from ldap group

In other words, nothing is installed using the ldap group, but this group is referenced afterwards.
Everything works fine since by default the host in the ldap group is the same as the one in the kdc group, but I feel like these groups aren't really useful as long as we can't separate openldap from kdc server.
What are your thoughts on this ? I might submit a PR to better separate ldap from kdc if you think it's worth it.

This should be fixed with https://github.com/TOSIT-IO/tdp-collection-prerequisites/pull/10, ansible-ldap-kerberos will be removed (see #53).
The ldap Ansible group has been deleted, kdc group is used.
Do not hesitate to open an issue in https://github.com/TOSIT-IO/tdp-collection-prerequisites if this is not fixed.


add step name as tag to step histogram

This should allow the  use of "avg by". e.g.

@JoshNavi I totally buy that. I really need to ask an analyst to tell me what's useful.


Fix logic to generate manifest in webpack config

Else manifest-[chunkhash].js will never be generated outside dev mode.

No need to change anything here, as per #4639


Exchange Rate: Include Stack Size

The Exchange Rate should also show the total amount of my stack.

0.5.18 (2020-02-19)

add stack size amount to exchange rate (#227)


RFP - gmail-desktop

Checklist

[x] The package I am requesting does not already exist on https://chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts with 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL : https://github.com/timche/gmail-desktop/releases/tag/v2.25.3
Direct download URL for the software / installer : https://github.com/timche/gmail-desktop/releases/tag/v2.25.3
Software summary / short description: Gmail Desktop
Nifty Gmail desktop app for macOS, Linux & Windows

I pick up this package :slightly_smiling_face:
I actually already have created a Chocolatey package which is currently waiting for the review of Moderators :wink:

This package has been reviewed and approved by @TheCakeIsNaOH :slightly_smiling_face:


feat(TFD-1666): Use JsonSchemaInferrer & JsonGenericRecordConverter

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TFD-1666
What is the new behavior?
Map the PythonRow output schema into an avro schema.
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[x] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information:
https://jira.talendforge.org/browse/TFD-1666

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:


Add DockerContainersEnumerationTask analysis task

sorry for procrastinating the PR for so long :(

I added a couple comments, but I'll do a more thorough review tomorrow (pending oncall load :).

@rgayon Can we drop this PR in favor of #465 then?


Tasks crud working

Added an api to get a single task using useparams.
Models adjusted for Task and Report Table.
Add,Update and Delete working.

Done.


BLD: meson: fix version string related code

Works now, version string is correct:

Compared to before:


Consumer config

Is line 110-111 here redundant? It's not clear to me why we set self.name but then don't use it.
https://github.com/aineko-dev/aineko/blob/9eb368352d637ca01ca147b6759895a1d3d89335/aineko/core/dataset.py#L103-L115

I'm doing a deep dive of the Kafka setup right now, came across this. Obviously not high priority or blocking but context here would be helpful to know if I'm missing something around how we're using this class / setting config.

We're using it to describe the consumer in:
https://github.com/aineko-dev/aineko/blob/9eb368352d637ca01ca147b6759895a1d3d89335/aineko/core/dataset.py#L164-L166 and
https://github.com/aineko-dev/aineko/blob/9eb368352d637ca01ca147b6759895a1d3d89335/aineko/core/dataset.py#L210-L214


ImmutableDB: use the ChainDB's registry instead of an internal one

This fixes master.
See the separate commits for more details

bors r+


Documentation: Added example to DateTime opBinary and reference to core.time.Duration

There is currently no usage example and no way for the reader to know how to specify a duration.

The example is taken from this forum post.

There's also SysTime.opBinary

I've made these changes. Hopefully the SysTime.opBinary example makes sense, as I haven't used that before.

LGTM

BTW, when you update something, a notification is NOT sent out to reviewers unless you make a comment. I happened to notice your change this time.

Okay. I assumed you would get a notification.

Auto-merge toggled on


OSX monochrome menu bar icon option

OS:
Mac OS X
Issue Description:
It would be nice to have an option for the menu bar icon to be monochromatic.  The current menu bar icon style doesn't match several other menu bar icons I have:

FYI it looks like this has been implemented on the master branch, so you'll probably see it in the next release

:-)

@ajacobson What @chrismou said.  It's in master and will be in the next release

I'm not seeing this on 3.0.1.