Can't debug the code because of System.Exception: Frame didn't arrived within 5000. UNITY

| Camera Model                          | D435
| Firmware Version                      | <IP_ADDRESS>
| Operating System & Version     | Win (10)
| Platform                                   | PC
| SDK Version                             | 2.29.0
| Language                                 | unity
I need to align a thermal image to a color image using color and depth. For this I'm using the RsDevice code as a base. I've modified the function WaitForFrames() like this:
    /// <summary>
    /// Worker Thread for multithreaded operations
    /// </summary>
    Align align = new Align(Stream.Color);
    CSRegister.CSRegisterTCD regClass = new CSRegister.CSRegisterTCD(
                    "E:\\_MANTRA\\_imagenes\\patron\\output_imagesx640_80\\spaceing24_extrinsics_640_80.xml", true);
    int counter = 0;
    IntPtr thermo;
    IntPtr thermoOverColor;
    int w, h;
    Texture2D toc_texture;
    public Texture2D tex;

    private void WaitForFrames()
    {
        while (!stopEvent.WaitOne(0))
        {
            using (var frames = m_pipeline.WaitForFrames())
            using (var aligned_frames = align.Process(frames))
                if (aligned_frames.IsComposite)
                {
                    using (var fs = aligned_frames.As<FrameSet>())
                    using (var color = fs.ColorFrame)
                    using (var depth = fs.DepthFrame)
                    {
                        Marshal.FreeHGlobal(thermo);
                        thermo = regClass.readGRAYimage("E:\\path\\t_2019711561978742124.png");

                        if (isRegisterTCD)
                        {
                            Marshal.FreeHGlobal(thermoOverColor);
                            regClass.update(thermo, color.Data, depth.Data, true, true, 0.0f, 1.0f, 0.5f, out w, out h);
                            thermoOverColor = regClass.getThermoOverColor();
                            tex.LoadRawTextureData(thermoOverColor, w * h);
                            tex.Apply();
                            //Debug.Log("REGISTER_" + counter);
                        }

                        counter++;
                        RaiseSampleEvent(aligned_frames);
                    }
                }
        }
    }

The thing is that I'm trying to show the IntPtr thermoOverColor (a RGB image) to a Unity canvas with Texture2D object: tex.LoadRawTextureData(image, w * h). But it shows the following error:
LoadRawTextureData_ImplPointer can only be called from the main thread.
Constructors and field initializers will be executed from the loading thread when loading a scene.
Don't use this function in the constructor or field initializers, instead move initialization code to the Awake or Start function.
UnityEngine.Texture2D:LoadRawTextureData(IntPtr, Int32)
RsDevice:WaitForFrames() (at Assets/RealSenseSDK2.0/Scripts/RsDevice.cs:208)
I've tryed to initialize the Texture2D tex object into an Awake() function but whenever isRegisterTCD is true, the frames stops updating.
When I try to debug the code in Visual Studio 2017, it shows the error:
System.Exception: Frame didn't arrived within 5000 ---> System.Runtime.InteropServices.ExternalException: rs2_pipeline_wait_for_frames(pipe:0000000043923A20)
--- End of inner exception stack trace ---
at Intel.RealSense.ErrorMarshaler.MarshalNativeToManaged (IntPtr pNativeData) [0x00000] in :0
at (wrapper managed-to-native) Intel.RealSense.NativeMethods:rs2_pipeline_wait_for_frames (intptr,uint,object&)
at Intel.RealSense.Pipeline.WaitForFrames (UInt32 timeout_ms) [0x00000] in :0
at RsDevice.WaitForFrames () [0x00012] in C:\Users\path\RsDevice.cs:189 happends
The debug error disappears if I erase the code after the line: using (var frames = m_pipeline.WaitForFrames())
Any Idea why I can't debug the code with the example before? or there is any way to increase the waiting frame time?
Also if anyone know how to show an IntPtr RGB image in Unity I'll really appreciate that!
Thanks!

The error message is pretty clear, you cannot call Texture2D.LoadRawTextureData outside Unity's main thread (like the camera's thread in your case).
You can use the Intel.RealSense.FrameQueue for multithreading, see this file:
https://github.com/IntelRealSense/librealsense/blob/2decb32456fc68396da2bc4de25028b1ee2d735f/wrappers/unity/Assets/RealSenseSDK2.0/Scripts/RsStreamTextureRenderer.cs#L167

Thanks! that was it, I've just moved my code inside the unity main thread in RsDevice code.


SAK-50381 Signup improve ui responsiveness

Fixed the responsiveness of tables for signup tool for both the instructor and student views. Marked as draft as the fix works on a Mac PC and not on a Windows PC.

This has been resubmitted.


100% cpu on logout

Dear Devs,
I got a 100% CPU usage of uwgi on modoboa logout (as soon as I click on the logout link uwsgi gets to 100%):  4729 modoboa   20   0  301960  89708  14144 R  99,7  2,2   0:08.70 uwsgi
Any idea where it could come from ?
I am using the modoboa version: 1.5.3 and ldap authentication backend.
Romain

I forgot to tell that the 100% uwsgi process never ends...

Looks like there is an infinite loop somewhere... Have you tried using a local account ? (ie non LDAP)

I just tested it and it works fine with non-ldap users (admin and a test user).

I have the same problem. Not using LDAP.
SimpleUsers results in 100% cpu usage by a uwsgi process and they never get logged out.
Also DomainAdmins.
SuperAdmins can log in / out just fine without any cpu spikes.
Modoboa version 1.5.3.

The issue reported by @ogg1980 was due to a wrong filesystem configuration. (emails were not stored into the location used by dovecot)
@romainreuillon Could you try to logout with such a user and check mail logs to see if there is anything interesting inside?

Great ! Next time I plan to work on this is at the end of September. I'll give it a try then.

Hi is this bounty still open.
I think I can help you on this

@hardikvala Thank you but the bounty is closed.
@romainreuillon I close the issue for now (required by bountysource). Do not hesitate to reopen it when you'll do your tests.


Documentation: A small bug in guides/Managing Parameters and State

The current documentation page of managing state parameters uses flax.core.pop, which doesn't exist. So the following line, which appears twice in the documentation,
state, params = flax.core.pop(variables, 'params')
should be replaced with
state, params = variables.pop('params')

Hey @konstmish, maybe you are using an older version of Flax? Current code base exposes pop:
https://github.com/google/flax/blob/8efa9975700a10a63fd2dbe8c0ec5e4fa7294541/flax/core/__init__.py#L21

You're right, I was behind the current version.


UsernameInput: Set maximum username length to 20

UsernameInput: Set maximum username length to 20
This Pull Request sets the maximum username input field to 20 characters.
This is to prevent having to scroll very far on mobile devices, so nobody can ruin the game for a team.
Current:

New:

Closes #143

Hey @avimoondra - mind taking a look at this?

Hey @avimoondra - Don't know if you want to update react, however the new versions support this change also.

@BlockArchitech sorry I missed this -- thanks for putting up a PR, LGTM!

@avimoondra - just pushed another commit which fixes formatting and (hopefully) passes checks

okay, maybe it will work this time.

@BlockArchitech trying here -- https://github.com/avimoondra/fishbowl/pull/156; I thin linters need to be installed / run! Since this is a small diff, was easy to port... will tag you in of course!

Closing in favor of #156 ! Thanks so much for adding this 🙏
https://github.com/avimoondra/fishbowl/commit/162cd443afd6b301eaccb6a79b3708ac4142346f


Atulk/11621 sweep tm ops, Increase Embedding input range for tiled conversion

Ticket
#11621
#13593
Problem description

Sweeps for many tm ops, squeeze, unsqueeze, embedding, view, repeat, split
Update embedding op to handle tiled inputs

Changes

Created sweeps for squeeze, unsqueeze, embedding, view, repeat, split
If input to embedding is not row major, convert it to row major. Removed fused tilized
Embedding sweep generality improved from 10->20%

post-commits

Embedding Unit tests:

To your PR description add:
link to the ticket (eg) #11621)
description of the changes you made
before and after for sweep numbers

hey @yugi957 could you link the passing post-commit workflow when you're ready


Wouldn't it be better to run pre-push?

This condition also met for pre-push

Linting makes more sense when running before committing your code. By doing that you can ensure no errors are going into the repository and enforce code style. But running a lint process on a whole project is slow and linting results can be irrelevant. Ultimately you only want to lint files that will be committed.

It runs every time on every commit even when there is no change in css. Would not it be better to run pre-push?

It runs every time on every commit even when there is no change in css.

It should not.

Would not it be better to run pre-push?

I don't think so.

I get this every time even when there are no SCSS in my commit.

It's says [skipped]. This was explicitly added in one of PRs

But it still takes few extra seconds. I make small commits frequently.
Why is pre-commit a better idea than pre-push?

Why is pre-commit a better idea than pre-push?

I wrote about the reasons in the blog post.

But it still takes few extra seconds each time.

You can try pre-commit instead of husky since I believe it's a bit faster.


Fixing import errors when installed as separate module

Signed-off-by: Elena Reshetova<EMAIL_ADDRESS>

@pohly, these changes are needed in order to be able to run isafw without bitbake as a standalone module. I haven't tested it with python3. How do you do it with bitbake?

For bitbake, I check out the ostro-os ci-upstream-monitor-master branch, then build some image. That branch has all the latest bitbake and OE upstream repos and thus uses Python3.

yes, looks like, there is really no way to make it work for everything. I will change this PR.

please retest this

retest this please

Let me try...
test this please

Test pass, reports looks fine, merging.


Add network extension

Several blocks for working with Internet:

I didn't add "fetch DELETE" and "fetch PATCH" blocks because I don't understand how these methods work. If someone needs it, I will add them.
*Thanks to  @TheShovel  for code of ping block.
*Method String() for arguments is need.

post should have a content-type option that is default text, for stuff like POST`in JSON

@showierdata9978
As I understand it, you are proposing to add menu with options "string" and "json". If "string" is selected, the code that exists now is executed. If "json" is selected, BODY is converted to JSON by method: "BODY = JSON.stringify(BODY);" and then same code. Did I understand you correctly?

it should be arbitrary, as the header in html spec is arbitrary
On Fri, Dec 16, 2022, 1:09 PM True Fantøm @.***> wrote:

@showierdata9978 https://github.com/showierdata9978
As I understand it, you are proposing to add menu with options "string"
and "json". If "string" is selected, the code that exists now is executed.
If "json" is selected, BODY is converted to JSON by method: "BODY =
JSON.stringify(BODY);" and then same code. Did I understand you correctly?
—
Reply to this email directly, view it on GitHub
https://github.com/TurboWarp/extensions/pull/67#issuecomment-1355449495,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQHW4PZK2PEPUJE5VCQYJFLWNS477ANCNFSM6AAAAAATBLTQYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

s I understand it, you are proposing to add menu with options "string" and "json". If "string" is selected, the code that exists now is executed. If "json" is selected, BODY is converted to JSON by method: "BODY = JSON.stringify(BODY);" and then same code. Did I understand you correctly?

I think they mean a content type option that sets the Content-Type header to any string.

This should probably be merged with the fecth extension.

Merge this into the existing fetch extension

And keep the websocket pinging stuff in the websocket pinging extension, we don't need to duplicate it in the fetch extension.
A way to specify the Content-Type of the body will be required. A way to get the status code of the response and other response headers would also be interesting.


Datepicker invalid week numbers

I've set the first day of my datepickers to 1, meaning the weeks start on monday.
I'm encountering a strange bug now though. December of this year (2014) shows a full week of January (5-1 / 11-1). The datepicker also says this is week 54. The week before that is week 53 according to the datepicker. These weeks should be week 1 and 2.

Even stranger behaviour is when I check out December 2015 and January 2016.

The numbering of weeks is wrong. The numbering is correct again in February 2016. I haven't checked out more years, but I'm guessing thesame problem arises...

I confirmed this bug in the following environment:

Angular Bootstrap version 0.12.0
Angular version 1.2.19
Chrome 39 (in addition to IE 11)
Windows 7

An additional problem with this is that even the week numbers in December are wrong. According to multiple sources 2014's week 52 was the week that started with December 22 (or December 21 if you start on Sundays). The week numbers are off by +1.
All this was already covered in https://github.com/angular-ui/bootstrap/issues/2506, it seems a fix for that issue was already committed but the guy never updated his code after review feedback was provided. I will look into this tomorrow and update the pull request.

A fix for the starting day that PR references had already been made in the meantime, I added fixes for the January week numbers  and adjusted the algorithm to remove the wrong offset in all week numbers. New PR here: https://github.com/angular-ui/bootstrap/pull/3158


Revit : crash when running on german Revit

the connector crashes Revit when /language DEU is in use (maybe for other Revit languages, too) when iterating over the buildin categories and creating the list in ConnectorRevitUtils.cs, line 41.
in german Revit it receives 2 times the same key for columns. Maybe a translation issue from Revit?
can be fixed like that:
try
{
var category = Category.GetCategory(doc, bic);
_categories.Add(category.Name, category);
}
catch (Exception ex)
{
}

sorry, I'm not allowed to add labels ?

Hey @thoffeller, thanks for reporting this! We'll have a look at it 👍🏻  I added the tags for you already ;)

Hey @thoffeller I've pushed a hotfix for this and triggered a beta release of the Revit connector, otherwise it'll be included in the next stable release in ~ 1.5 weeks.
Hopefully no more unexpected crashes :)


Making react-server a relative dependency

Issue reproduction:

In repo root npm run bootstrap.
cd packages/react-server
npm start

This results in the example site loading, but assets not being loadable.
I misunderstood this comment from @gigabo earlier. This PR applies the suggestion from the comment.

This change is duplicated in the already-merged #358.


Inconsistency whether percentile iterators should return time

Bug report
System info: InfluxDB 1.1
Additional info:
Unsure whether this is actually a bug or not, however I noticed the inconsistency when I was trying to see how percentile() handles null or missing data.  I thought it better to ask whether an issue than ignore it.  Forgive for being unable to develop a process to confirm if an actual bug or not.

float percentile iterator returning time

/ NewFloatPercentileReduceSliceFunc returns the percentile value within a window.
func NewFloatPercentileReduceSliceFunc(percentile float64) FloatReduceSliceFunc {
	return func(a []FloatPoint) []FloatPoint {
		length := len(a)
		i := int(math.Floor(float64(length)*percentile/100.0+0.5)) - 1

		if i < 0 || i >= length {
			return nil
		}

		sort.Sort(floatPointsByValue(a))
		return []FloatPoint{{Time: a[i].Time, Value: a[i].Value, Aux: a[i].Aux}}
	}
}

integer percentile iterator returning ZeroTime

// NewIntegerPercentileReduceSliceFunc returns the percentile value within a window.
func NewIntegerPercentileReduceSliceFunc(percentile float64) IntegerReduceSliceFunc {
	return func(a []IntegerPoint) []IntegerPoint {
		length := len(a)
		i := int(math.Floor(float64(length)*percentile/100.0+0.5)) - 1

		if i < 0 || i >= length {
			return nil
		}

		sort.Sort(integerPointsByValue(a))
		return []IntegerPoint{{Time: ZeroTime, Value: a[i].Value, Aux: a[i].Aux}}
	}
}

Seems like this relates to #6292 commit hash 66a599825b7afaa031604b1ef5bfca5ef7a4c0e5 as only floats were update. @jsternberg thoughts?

That definitely looks like a bug. Percentile should be a selector for all types.
@jwilder should this fix target 1.1 or 1.2?


Display abnormality on dual screens(both 4K)

Issue Type: Bug
When Visual Code shows on the secondary screen, the right border is drawed on the main screen.
This may happens when Visual Code shows directly on the secondary screen.
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T09:03:20.462Z)
OS version: Windows_NT x64 10.0.19041

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz (4 x 3408)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.87GB (18.67GB free)

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

vscode-JS-CSS-HTML-formatter
lon
0.2.3

cpptools
ms-
0.28.3

Hi, this happens when opening a file with VS Code on display 2.

Ok this sounds more like an electron issue to me.

Looks like #85310 but that one has 1ps that matches theme etc, not a white line

Merging this to #85310 , please follow that issue for updates. Thanks!


Add URL for JDK Temurin Windows

Description
Related issue #471. This PR adds the Temurin url for Windows.  I can't access to a Windows machine right now.
@stratika , when you get the time, give it a try.
Problem description
n/ a.
Backend/s tested
Mark the backends affected by this PR.

[X] OpenCL
[X] PTX
[X] SPIRV

OS tested
Mark the OS where this PR is tested.

[ ] Linux
[ ] OSx
[ ] Windows      >>>> To be tested

Did you check on FPGAs?
If it is applicable, check your changes on FPGAs.

[ ] Yes
[X] No

How to test the new patch?
To be tested on Windows:
python ./bin/tornadovm-installer --jdk temurin-jdk-21 --backend=opencl

I tried it, it works. Just a minor fix to test it, as per our guideline:
python -m venv .venv
.venv\Scripts\activate.bat
.\bin\windowsMicrosoftStudioTools2022.cmd
python .\bin\tornadovm-installer --jdk temurin-jdk-21 --backend=opencl,ptx
tornado-test -V

Great, thanks Thano

==================================================
              Unit tests report
==================================================

{'[PASS]': 663, '[FAILED]': 7, '[UNSUPPORTED]': 43}
Coverage [PASS/(PASS+FAIL)]: 98.96%
Coverage [PASS/(PASS+FAIL+UNSUPPORTED)]: 92.99%

==================================================

on my Dell Precision laptop with Windows 11.


explorer: item label for logged-in users loses collection text when queries are updated by word selection

click on a word in the Comparative word Cloud data card. The updated q field with the word term is properly pushed into the queries store, but when the page refreshes and QueryPickerItems are selected, the collection information (retrieved via server call) as confirmed by isMediaReady is somehow lost

it's the copying of the media field into the collections field (which contains id, tags_id and selected - causing the isMediaReady tag to succeed) when changing selection...

clearing the state on willMount and if/def-ing the return function of the SELECT_QUERY did the trick


Question about webrtc-ips

Hi there,
I came to ur lib trough Google, first  searching searching for HTML5 video player which can play remote video file. So it came to my mind to search something related to WebRTC.
I am not sure what is ur lib doing exactly, but I think it can be use for me in two ways.
Can this be used in case that you are playing video in player using remote video file, from other server. For example, I am hotlinking Tumblr video file, but I don't wanna show my server IP in their logs. So instead it will show exactly users IP.
You said This makes these types of requests available for online tracking if an advertiser sets up a STUN server with a wildcard domain.
Can you give more details how this an be used in tracking and advertising. I am building ad server as well and would like to know if I can have use for it.
Tnx

@shtefcs This is a demonstration of security breach in modern browsers which can be used to track users and can not be blocked by extensions like AdBlock and Ghostery. It's not a library.

As @fuwaneko said, this is a proof of concept of a privacy exploit for WebRTC. I will not assist in making ad servers that use this exploit.

ok, thank you guys.


Unable to change key bindings on keyboard shortcuts

Issue Type: Bug
I have tried to change the mode of creating new line on a file from ctrl+enter to ENTER key only but it is impossible
VS Code version: Code 1.48.2 (a0479759d6e9ea56afa657e454193f72aef85bd0, 2020-08-25T10:56:10.074Z)
OS version: Linux x64 5.4.0-42-generic

/duplicate
#85850


Prompt for ssh key passphrase

It feels very insecure to have to save my ssh key passphrase in plaintext in order to use keys to authenticate with servers. Since vscode-sftp already prompts for ssh password when it's not stored, can we do this for key passphrases as well?

"passphrases": true

Why is this not in the docs? It's also "passphrase": true (single). Took me forever to figure this out.

It seems to require both :
"passphrase": true,
"interactiveAuth": true

Note that "passphrase" is singular - no trailing 's'


Mapping deserialization

Replaces wasm32 triplet with just wasm32 arch
Reset changelog / version
Add mapping deserialization (something cdsl should do)

Mapping deserialization was added, because I need it for reconfix. We can move it to cdsl later, when the playground will be finished.

@resin-ci retest


Files are encrypted

Hey,
I have a wallet from 2013 with all associated files (.key, .wallet, .wallet.cipher)
My problem is that the key is encrypted.
Are any of your scripts useful if the encryption password is unknown?
Thanks!

Unfortunately, all the scripts require the password to function properly. They aren't useful for "cracking" passwords, per se, as they need the password to decrypt the files and output the private keys etc.
You might have better luck using something like hashcat for the password recovery (assuming the password wasn't too complex and/or you have a decent amount of GPU power etc). https://hashcat.net/hashcat/


Update getting started guide

Link to relevant issue
This pull request resolves #74
This pull request resolves #348
Description of changes
Updates the getting started guide to include new content on how to:

Avoid loading the toolkit package directly from node_modules
Register the toolkit components in JS/TS
Add proper content security policies for loading scripts, styles, and images

Misc
There is also a sibling PR in the toolkit samples repo that implements these changes into the toolkit extension samples.

@daviddossett these new docs could probably use another round or two of editing so whenever you have some free time I'd appreciate a pass on the content for any glaring typos, grammar, phrasing issues, etc.

I'm excited for this update but wanted to mention that I reproduced the steps and got an error on Stage 2.
Uncaught ReferenceError: module is not defined is shown when trying to load webview.js when running the extension. Any ideas for fixes @hawkticehurst?

I'm excited for this update but wanted to mention that I reproduced the steps and got an error on Stage 2.
Uncaught ReferenceError: module is not defined is shown when trying to load webview.js when running the extension. Any ideas for fixes @hawkticehurst?

Hey @OliverMKing, thanks for chiming in! Alas I've run into this problem too and it's one of a handful of issues that have been blocking the release of these docs 😪
I thinkkk the issue is caused by how we are bundling the toolkit and that not playing nicely once you start trying to use the components in an ESM context. Sadly, I've been pulled away to do other work in the last handful of weeks so it might take a while to circle back and get into the weeds on this one.
For the time being I'd probably stick with the original docs –– although the writing on content security policies should still be valid/helpful!

@OliverMKing revisiting this PR and I believe an accidental regression caused the issue where the "type": "module" field was removed from package.json, meaning the whole toolkit was using CommonJS instead of ESM 😅
PRs #421 and #436 should have fixed these issues now, so I'd love to know if this addresses your problems.
I've also been able to continue (now successfully) updating the sample extensions to demonstrate registering the components in typescript. They're still a work in progress, but I have completed the hello world samples, so you're welcome to reference them if you'd like:

Hello world sample (esbuild-based)
Hello world sample (webpack-based)


Fix param defaults for Ubuntu > 12.04 (and implement Beaker for acceptance tests)

Module defaults (i.e, include composer) fail on Augeas['whitelist_phar'] for Ubuntu > 12.04.
This PR will default to suhosin_enabled=false for Ubuntu releases > 12.04, since suhosin no longer ships with versions newer than precise.
In order to prove the existence of the bug, and to ease testing multiple distros, I refactored composer_spec.rb to use shared examples, and added an acceptance test for composer::project, using Beaker.
I also updated the README to include information on running beaker tests, including how to provide a GitHub auth token to beaker via hiera.

Sorry for all the extra travis commits -- I didn't get all the excludes configured properly the first time around.
The 1.8.7 builds are failing (an issue with the il8n gem, it appears), but all others pass without issue.

Beaker is something we should definitely get hooked up. Thanks for taking the plunge!
I'm all for this in general. I'll take a look at the current failures and review in the next week or so. Sorry, I'm a little backed up on other paid work at the moment.
In the meantime, can you squash the travis commits? I'll see if we can safely drop Ruby 1.8.x support.

Sorry for lumping beaker into the bugfix PR -- I can split the two if that makes things simpler.

I think the fix and Beaker do make more sense as separate PRs. Beaker is a new way of testing and something I would like to get familiar with myself. I can certainly get the ultimate fix for the issue out much faster if you could do that. I appreciate it!

No problem!  I'll close this PR and split into two.


Xeus-Cling kernel not appearing in VS Code's Jupyter Notebook kernel picker

Version: 1.79.2 (user setup)
Commit: 695af097c7bd098fbf017ce3ac85e09bbc5dda06
Date: 2023-06-14T08:57:04.379Z
Electron: 22.5.7
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Unable to connect to xeus-cling kernel in WSL2 Ubuntu from VS Code jupyter notebook even though it works fine when launched in browser.
Steps to Reproduce:

I've VS Code Jupyter extension v2023.5.1101742258 installed

I'm running Ubuntu 22.04.2 LTS on WSL2 on windows 11 Home ,OS Build 22621.1848

I've installed xeus-cling and jupyter in a conda environment , "XeusCling" using miniconda 23.1.0 following the instructions at https://envyen.com/posts/2021-06-27-C-CPP-Jupyter-Notebook-Windows/

When I try to launch the jupyter server in --no-browser mode and open the url in the browser xeus-cling works fine and I'm able to connect to c++11,c++14 & c++17 as shown below:-

But when I open another '.pynb' file on windows in vs code the XeusCling env doesn't show up:

I tried using VS Code Insiders and got the same issue as shown below:-

Thank you for reporting this and sorry you are running into this.
We are aware of this issue and will look into this
here’s an old(related) issue https://github.com/microsoft/vscode-jupyter/issues/5469

Oops, sorry, the issue was related to xeus-python, not xeus-cling, they are two different issues.
Will look into this again.

Tested and this should work, the kernels are now displayed.
@mrinmoy2developer Let me know if this issue still persists.


getDocumentInfo can get pretty slow

When using exactly the same PSD, the speed of getDocumentInfo really differs. Sometimes it is almost instantly resolved, but sometimes it takes about 5 - 10 secs.
I already tried to bring the required data to a minimum, but still the delay remains.
_generator.getDocumentInfo(null, {
    compInfo:             false,
    imageInfo:            true,
    layerInfo:            false,
    expandSmartObjects:   false,
    getTextStyles:        false,
    getFullTextStyles:    false,
    selectedLayers:       false,
    getCompLayerSettings: false,
    getDefaultLayerFX:    false,
    getPathData:          false
  })

Can we explain this behaviour? What are the possible causes for the slow excecution?

Not a definitive explanation, but getDocumentInfo is competing for time on Photoshop's main thread with everything else that is going on: scripts, commands, idle tasks like font menu previews, other generator plugins, etc.  Without some profiling it would be hard to say where it's spending that extra 5-10 seconds (it may not actually be getting doc info during that time).


Connection Request: scrobble from ccmixter.org

I'd like to be able to scrobble from http://ccmixter.org/
Expected behavior

Just an additional site request
Current behavior

It just doesn't do it right now.
Context

When i got to a page such http://ccmixter.org/people/Reiswerk I'd like to be able to scrobble those tracks.
There are also pages like http://ccmixter.org/view/media/remix/picks that have a slightly different format

The extension now supports ccMixter scrobbling, thanks to @xyauhideto.


Add BLS fairy ring.

Want to add it to World Map Plugin as well?

1295, 3493, 0

Added it to the map as well @MESLewis , good idea.


feat: public "getCroppedBitmap" method

There are cases when we need to use that crop method beyond the use of an instance of a CropController.

Well, if you think... :-)

Hi @deakjahn! Oh it's even a fact: we need it in our app.
The use case is: we crop an image the size of the screen in real time (with the CropController), which is good enough and fast enough in a first approach, but processing the full image takes too long and we do it later in a background task. And in that background task we don't have the CropController anymore, just the full file and the crop parameters.

I did copy the code, temporarily, until you release 1.0.6 ;)
I assumed that making the method public would make sense and couldn't jeopardy your package's integrity.

Nudge, nudge, wink, wink. I know... :-)

Nudge, nudge, wink, wink. I know... :-)

Thank you for the 1.0.6, and thank you for the quote!
Slightly related: https://www.youtube.com/watch?v=ifLqzLEB3E0


Sm slide

Finished

Trying again to fix.

Good pull requests have a body description that tells other contributors about the change you're suggesting, so they understand the context.
Let's edit this pull request to add a body description.
⌨ Activity: Fixing your pull request
1. The first comment on your pull request will have the default text of **No description provided**. Click on the **...** icon located at the top right corner of the comment box, then click on **Edit** to make an edit

2. Add a description of the changes you've made in the comment box. Feel free to add a description of what you’ve accomplished so far. As a reminder, you have: created a branch, created a file and made a commit, and opened a pull request

3. Click the green **Update comment** button at the bottom right of the comment box when done

If you would like assistance troubleshooting the issue you are encountering, create a post on the GitHub Community board. You might also want to search for your issue to see if other people have resolved it in the past.
I'll respond when I detect this pull request's body has been edited.
Edited

Grawl


[BUG] carousel use with layout

Version
1.8.0
Environment
chrome 62.0.3202.89
Reproduction link
https://stackblitz.com/edit/angular-sjo7ja?embed=1&file=src/app/app.component.ts
Steps to reproduce
default :
[nz-carousel-content]
height: 160px;
line-height: 160px;

change to :
[nz-carousel-content]
height: 360px;
line-height: 360px;

What is expected?
when i change the carousel height , the view is don't my want need.
What is actually happening?
The pictures overlap.
Other?

https://stackblitz.com/edit/ng-zorro-antd-start-fr1qs7?embed=1&file=src/app/app.component.ts

https://stackblitz.com/edit/angular-sjo7ja?embed=1&file=src/app/app.component.ts

import { Component } from '@angular/core';

@Component({
  selector: 'nz-demo-carousel-vertical',
  template: `
      <nz-layout>
      <nz-content>
        <nz-carousel nzAutoPlay>

{{index}}

        </nz-carousel>
      </nz-content>
    </nz-layout>
    `,
  styles: [
    `[nz-carousel-content] {
      text-align: center;
      height: 360px;
      line-height: 360px;
      background: #364d79;
      color: #fff;
      overflow: hidden;
    }

    h3 {
      color: #fff;
    }
    :host {
      text-align: center;
    }

    :host ::ng-deep .ant-layout-content {
      background: rgba(16, 142, 233, 1);
      color: #fff;
    }

    `
  ]
})
export class NzDemoCarouselVerticalComponent {
  array = [1, 2, 3, 4];
}

https://stackblitz.com/edit/angular-sjo7ja-tmhtsp?file=src/app/app.component.ts


Home Page Banner/logo section buttons not looking nice

On the homepage, just below the Djangocon logo there are 2 buttons for now "Submit your CFP📢" and "Buy Tickets🎫".
But these 2 buttons is not same width and the icon need a space in between text.
This happens the same for mobile when stacked.
I have attached 2 screenshots below.
Desktop:

Mobile:

Hello sir, I would get that fix real quick


[android] Fix Bad notification for startForeground error

This is updated https://github.com/react-native-kit/react-native-track-player/pull/579 fix by @puckey based on discussion there, fixes https://github.com/react-native-kit/react-native-track-player/issues/534 (or at least the crash mentioned in comment https://github.com/react-native-kit/react-native-track-player/issues/534#issuecomment-484014486 if it's not the one that issue author has encountered)

I'm also having the issues described in #534 , with the Pixel 3XL AVD, running Android 9.
Playing music works for me, but the media notification buttons all crash the app with the mentioned 'Bad notification for startForeground' error when I interact with them.
With this branch, I'm not getting the entire app crashing, but I'm still getting the below error:
12-09 09:28:51.872  1631  3961 W audio_hw_generic: Not supplying enough data to HAL, expected position 179285918 , only wrote 179285760
12-09 09:28:39.018  1897  1897 I NotificationService: Cannot find enqueued record for key: 0|com.example.app|1|null|10085
12-09 09:28:51.898  1897  1912 V ActivityManager: Attempted to start a foreground service (ComponentInfo{com.example.app/com.guichaguri.trackplayer.service.MusicService}) with a broken notification (no icon: Notification(channel=com.guichaguri.trackplayer pri=0 contentView=null vibrate=null sound=null defaults=0x0 flags=0x40 color=0x00000000 vis=PRIVATE))
12-09 09:28:51.919 11318 11318 D RNTrackPlayer: Releasing service resources...

Is this fixed in v2 branch?

Is this in 2.0.0-rc13?


Bind magit-dispatch instead of magit-dispatch-popup to "C-x M-g"

Hello:
Since magit switched to transient, magit-dispath-popup has been removed and it has no "obsolete function alias".
Thanks.

Thanks!


Check to see if a server is currently running

The interface throws a confusing error if someone is already running either a dev or normal Rill Developer instance. We should probably just throw a clearer error in this case.

similar issue filed by @ericpgreen2 as #873 -- closing this one in favor of that, which has a bit more detail and is more recent


added Instrument.delete_calibration_data() method for testing

This PR adds a Instrument.delete_calibration_data() method, which is most helpful while testing, or on the off chance a user wants to re-calibrate everything

Coverage increased (+0.04%) to 93.831% when pulling 53fb1ec42b16e5aaff4647588b1fd9230ee5a00c on give-instrument-a-delete-calibrations-method into 17939c026323ecfed4996078b62728ee460b115d on master.


I am looking for the document on how to deploy to AWS EKS

Please see the following: https://github.com/awslabs/djl-spring-boot-starter-demo#deploying-to-eks-amazon-elastic-kubernetes-service
The starter documentation will be updated to include this link.


email validation error

GSField.email(
                    tag: 'email',
                    title: getTranslated('login_hint', context)!,
                    weight: 12,
                    required: true,
                    maxLength: 100,
                    prefixWidget:const Icon(FontAwesomeIcons.envelope,size: 15,),
                    errorMessage:  getTranslated('invalid_email', context)!,
                  ),

whatever I type, even if I type my email account, the validation always returns an error.

Hello @Amber916Young  we fix this bug
you can update  version of GSForm and use it

Hi, the pub version is still   ^0.3.5, which is the lasted version? Thank you very much.


[top_englishbreakfast] Add second UART

For penetration testing, we've been using a second UART on the CW310 and the silicon platform such that the host can directly communicate with the target using the uJSON protocol instead of looping this communication through the simpleserial channel with the scope (available on Husky only anyways).
This commit now adds a second UART to the English Breakfast top level such that we can re-align the capture setups again. Ultimately, this will allow removing the simpleserial-based communication which will quite notably simplify code maintenance in the future.
For reference, see also https://github.com/lowRISC/ot-sca/issues/235.

@nasahlpa , @andrea-caforio , I've successfully tested this using simpleserial on the FPGA connected to our shared workstation. To get the uJSON communication working in ot-sca CI, we'll also need to connect an additional FTDI to the IO5 and IO6 pins of the JP3 header. This is because on the CW305, the integrated of UARTs/USARTs of the SAM3U microcontroller are unfortunately not connected to the FPGA.

Thanks for your reviews guys!


Contributor role error message is missing

Package version (if known): 0.14.8
Describe the bug
When filling a contributor data on the deposit page, but forgetting to fill out the role, the contributors section is highlighted in red (good!), but the pop-over for the role field (the one that says "Missing required data...") isn't showing up (bad!).
Steps to Reproduce

Go to /uploads/new
Fill out contributors except for role
Save Draft
See highligted mistake but missing pop-over

Expected behavior
Having the pop-over like there is one for the resource type.

Cannot be reproduced anymore since the role filed always gets a default value.


Build environment

Get Jenkins on the case
special considerations

going to have to disable various pieces of the site to prevent accidental production usage
secret keys and such need to be kept secret

Going to need scripts for deploying to dev and deploying to prod

Build errors forever

06ebc40516e82af7fa1685db8373f4dad03aedae fixed the dev environment, this feature is now a beta milestone


[QUESTION] Please ensure that the container has a non-static position...

Hi @mattgperry
I have the following message in the console :
Please ensure that the container has a non-static position, like 'relative', 'fixed', or 'absolute' to ensure scroll offset is calculated correctly.

The container is  thehtml element, is this also true for this element, or can I ignore the warning?
Thanks!

Could you spin this up in a codesandbox? It helps me investigate and write a test against it

Here is the link: https://codesandbox.io/p/sandbox/95ttr2
Thanks @mattgperry


Crash on right mouse click on the searchbar

Steps to reproduce

Click with your right mouse on the searchbar
It crashes or when you press copy it will crash

System information
Windows 10 with the latest nightly version

Seems to be caused by 23e82969821fbf369650533624aa4eaa57a9e06d.
I'll push a fix right away 👍


Port pull request #1838 to master

After #1838 was integrated to the 6_x branch, I now ported it to master.

Hi. I wrote a tiny plugin, SolrServiceFileInfoPlugin, (implementing SolrServiceIndexPlugin) to make filenames and file descriptions from the ORIGINAL bundle searchable.
The plugin must be added as a bean to discovery.xml.
To see it's effect, you have to reindex the Solr search core by executing:
[dspace]/bin/dspace index-discovery -b
I documented it on https://gist.github.com/MW3000/5771c0488f8952bb888357ce8c655279

@kshepherd wrote:

Just a note that we'll need to port this to master branch as well, should be pretty easy, we just won't want the changes to messages.xml (as xmlui isn't in master)

I just have not been able to test it on master. Is there already a REST endpoint available for this functionality?

I run tests locally by executing:
mvn test -Dmaven.test.skip=false
But this does not seem to trigger all the tests that are run by TravisCI. Especially Travis runs a test named org.dspace.app.rest.DiscoveryRestControllerIT that is failing. How do I trigger the execution of this test?

Integration tests (IT*, *IT) are run in a different step, and controlled by a different property.  Add -DskipITs=false to enable them.

Hi @MW3000 , if it helps, the full output of the test is below. It's probably a case of us updating the integration tests to test for this new filter... it looks as though DiscoveryRestControllerIT.discoverSearchTest looks for the default range of filters using hardcoded SearchFilterMatcher tests:
.andExpect(jsonPath("$.filters", containsInAnyOrder(                        SearchFilterMatcher.titleFilter(),                        SearchFilterMatcher.authorFilter(),                        SearchFilterMatcher.subjectFilter(),                        SearchFilterMatcher.dateIssuedFilter(),                        SearchFilterMatcher.hasContentInOriginalBundleFilter()                    )))
Even though Discovery filters themselves are technically spring beans, and so the integration tests are including your spring config in the test item, which is why it's failing, because there's no corresponding filter to match it. I think that's what's going on. If you'd like a hand updating the integration tests I can chip in, perhaps we could ask for clarification in the #rest-api channel in Slack.
Failed tests:    DiscoveryRestControllerIT.discoverSearchTest:756 JSON path "$.filters" Expected: iterable over [(is json with json path "$['filter']" evaluated to is "title" and (is json with json path "$['operators']" evaluated to iterable over [is json with json path "$['operator']" evaluated to is "equals", is json with json path "$['operator']" evaluated to is "notequals", is json with json path "$['operator']" evaluated to is "authority", is json with json path "$['operator']" evaluated to is "notauthority", is json with json path "$['operator']" evaluated to is "contains", is json with json path "$['operator']" evaluated to is "notcontains"] in any order)), (is json with json path "$['filter']" evaluated to is "author" and (is json with json path "$['operators']" evaluated to iterable over [is json with json path "$['operator']" evaluated to is "equals", is json with json path "$['operator']" evaluated to is "notequals", is json with json path "$['operator']" evaluated to is "authority", is json with json path "$['operator']" evaluated to is "notauthority", is json with json path "$['operator']" evaluated to is "contains", is json with json path "$['operator']" evaluated to is "notcontains"] in any order)), (is json with json path "$['filter']" evaluated to is "subject" and (is json with json path "$['operators']" evaluated to iterable over [is json with json path "$['operator']" evaluated to is "equals", is json with json path "$['operator']" evaluated to is "notequals", is json with json path "$['operator']" evaluated to is "authority", is json with json path "$['operator']" evaluated to is "notauthority", is json with json path "$['operator']" evaluated to is "contains", is json with json path "$['operator']" evaluated to is "notcontains"] in any order)), (is json with json path "$['filter']" evaluated to is "dateIssued" and (is json with json path "$['operators']" evaluated to iterable over [is json with json path "$['operator']" evaluated to is "equals", is json with json path "$['operator']" evaluated to is "notequals", is json with json path "$['operator']" evaluated to is "authority", is json with json path "$['operator']" evaluated to is "notauthority", is json with json path "$['operator']" evaluated to is "contains", is json with json path "$['operator']" evaluated to is "notcontains"] in any order)), (is json with json path "$['filter']" evaluated to is "has_content_in_original_bundle" and (is json with json path "$['operators']" evaluated to iterable over [is json with json path "$['operator']" evaluated to is "equals", is json with json path "$['operator']" evaluated to is "notequals", is json with json path "$['operator']" evaluated to is "authority", is json with json path "$['operator']" evaluated to is "notauthority", is json with json path "$['operator']" evaluated to is "contains", is json with json path "$['operator']" evaluated to is "notcontains"] in any order))] in any order      but: Not matched: <{filter=original_bundle_filenames, operators=[{"operator":"equals"},{"operator":"notequals"},{"operator":"authority"},{"operator":"notauthority"},{"operator":"contains"},{"operator":"notcontains"}]}>

@mwoodiupui @kshepherd Thank you for your help. I adapted the test.

Resolved the merge conflict.  I see that this one is still marked "work in progress".


Fix a link error when async_cuda is enabled, but not cuda_compute

Just replaces a couple of CUDA statements with ASYNC_CUDA in cmake - not 100% tested, but dashboard should reveal if it causes trouble

Does HPX_WITH_CUDA imply HPX_WITH_CUDA_COMPUTE? Or is HPX_WITH_CUDA implicitly defined whenever one of HPX_WITH_CUDA_COMPUTE or HPX_WITH_ASYNC_CUDA is defined?
I think it should be the latter in which case the fix proposed here would break using cuda compute, IIUC.

if HPX_WITH_CUDA is set - then both HPX_WITH_ASYNC_CUDA and HPX_WITH_CUDA_COMPUTE are turned on. If HPX_WITH_CUDA_COMPUTE is enabled, then HPX_WITH_ASYNC_CUDA is also enabled, but if ONLY HPX_WITH_ASYNC_CUDA is enabled, then the other two are not set. I mostly use HPX_WITH_ASYNC_CUDA and do not use the CUDA_COMPUTE - (I wanted to remove HPX_WITH_CUDA completely and only have the other two, but this was previously rejected).

Does HPX_WITH_CUDA imply HPX_WITH_CUDA_COMPUTE? Or is HPX_WITH_CUDA implicitly defined whenever one of HPX_WITH_CUDA_COMPUTE or HPX_WITH_ASYNC_CUDA is defined?
I think it should be the latter in which case the fix proposed here would break using cuda compute, IIUC.

if HPX_WITH_CUDA is set - then both HPX_WITH_ASYNC_CUDA and HPX_WITH_CUDA_COMPUTE are turned on. If HPX_WITH_CUDA_COMPUTE is enabled, then HPX_WITH_ASYNC_CUDA is also enabled, but if ONLY HPX_WITH_ASYNC_CUDA is enabled, then the other two are not set. I mostly use HPX_WITH_ASYNC_CUDA and do not use the CUDA_COMPUTE - (I wanted to remove HPX_WITH_CUDA completely and only have the other two, but this was previously rejected).

When adding HIP support we decided to keep the HPX_WITH_CUDA option to distinguish it from HPX_WITH_HIP. These two options control whether support for either of these is generally enabled. The two other options, HPX_WITH_ASYNC_CUDA and HPX_WITH_COMPUTE_CUDA, are dependent on either of those two being set (HPX_WITH_HIP is set automatically when hipcc is the compiler) and control whether the two individual modules are enabled.
I think what you probably want to do is set HPX_WITH_CUDA=ON and HPX_WITH_COMPUTE_CUDA=OFF (with the exception that we seem to disagree on which flags you should be required to set).
Also, in my opinion, the fact that this is still so confusing tells me that the HPX_WITH_{ASYNC,COMPUTE}_CUDA options are not worth it, or that they really badly need to be renamed (as they now apply to both CUDA and HIP).
Finally, this breaks the HIP configuration.

OK. I obviously missed out of the change to support HIP. I'm happy to have
HPX_WITH_CUDA - implies HPX_WITH_ASYNC_CUDA is enabled and we no longer need a specific HPX_WITH_ASYNC_CUDA cmake flag any more since WITH_XXX will imply ASYNC_XXX - where XXX=CUDA / HIP
As long as we make sure that turning on HPX_WITH_CUDA does not forcefully turn on HPX_WITH_COMPUTE_CUDA - (because that's what it used to do).
I will modify this PR and resubmit later.

As long as we make sure that turning on HPX_WITH_CUDA does not forcefully turn on HPX_WITH_COMPUTE_CUDA - (because that's what it used to do).

Depends what you mean by forcefully. It enables HPX_WITH_COMPUTE_CUDA by default, but one is free to disable HPX_WITH_COMPUTE_CUDA explicitly.

As long as we make sure that turning on HPX_WITH_CUDA does not forcefully turn on HPX_WITH_COMPUTE_CUDA - (because that's what it used to do).

Depends what you mean by forcefully. It enables HPX_WITH_COMPUTE_CUDA by default, but one is free to disable HPX_WITH_COMPUTE_CUDA explicitly.

@msimberg: could you please document here which flags need to be set to what for all of the combinations (CUDA on/off, HIP on/off, async CUDA on/off, etc.)? I lost track of that as well.

@msimberg: could you please document here which flags need to be set to what for all of the combinations (CUDA on/off, HIP on/off, async CUDA on/off, etc.)? I lost track of that as well.

Apparently I can't. It doesn't behave the way I been trying to describe above. I apologize sincerely.
That said, what I've been trying to describe is how I would still like it to behave, given that we don't just remove the module-specific options, which would be my first choice.
Could we first try to agree on how we think the options should behave?

Could we first try to agree on how we think the options should behave?

@msimberg: That's what I was trying to get at by asking you how you think the options should behave. I think that HPX_WITH_CUDA and HPX_WITH_HIP should be the main options and should enable all of the CUDA/HIP support. If we need to have more fine-grain control, those should be separate options that would have to be disabled explicitly (i.e. HPX_WITH_COMPUTE_CUDA=OFF etc.). I believe that is what you were suggesting earlier as well.
Getting rid of the module-specific options would be fine with me as well. @biddisco if you have some additional rationale why those shouldn't/couldn't be removed, please let us know. I might not have the full picture.
This is however not set in stone for me and I'd be happy to change my mind, if there are better options.

@msimberg: That's what I was trying to get at by asking you how you think the options should behave. I think that HPX_WITH_CUDA and HPX_WITH_HIP should be the main options and should enable all of the CUDA/HIP support. If we need to have more fine-grain control, those should be separate options that would have to be disabled explicitly (i.e. HPX_WITH_COMPUTE_CUDA=OFF etc.). I believe that is what you were suggesting earlier as well.

Sorry. Yes, this sounds like how I expected it to behave currently as well. The discrepancy right now seems to be that if e.g. HPX_WITH_CUDA=ON then it forces HPX_WITH_COMPUTE_CUDA=ON even if the user sets HPX_WITH_COMPUTE_CUDA=OFF explicitly. I think this is the only discrepancy...

Closing in favour of #5145.


Fatal error running multiple karma targets in one grunt task

We're using Grunt to build multiple, but similar, applications in one build. Each app has files with the same names like HomeController, MenuController. So we split up the karma targets per application so dependencies are loaded only from the shared and specific application being tested. However, we do want all applications to be testes when deploying so we want to run all these targets on after another.
Fatal error
When using grunt to run the multiple karma targets it only runs the first successful and fails to run the second. Fatal error: listen EADDRINUSE The error is somehow related to a port being used.
Grunt-karma config
var dep = [
  'bower_components/**/*.js',
  'app/shared/**/*.js',
];

module.exports = {
  options: {
    configFile: 'test/karma.conf.js',
    reporters: ['brackets', 'html', 'junit', 'coverage'],
    browsers: ['PhantomJS'],
    port: 9002,
    singleRun: true
  },
  A: {
    options: {
      files: dep.concat([
        'app/A/src/**/*.js'
      ]),
    }
  },
  B: {
    options: {
      files: dep.concat([
        'app/B/src/**/*.js'
      ]),
    }
  }
};

when running: grunt karma this ends with a Fatal error: listen EADDRINUSE
I've also create a question on stack overflow. I hope someone can help me out!

I think this is the same bug as was tried to be solved in #57

"tried to be solved"?
Did it make it to a release?
Can I help?

See my comment https://github.com/karma-runner/grunt-karma/pull/57#issuecomment-54463199 for what is left to do, as the author has been pretty silent, feel free to create a new PR based on that

I believe #mokkabonna is right in stating his solution is not ideal and the karma-server should be stopped after completing a singleRun. Abusing the background option doesn't solve the problem.
Workaround
I've managed to run the karma tasks separately by chaining grunt command on the CML instead of inside a grunt task. that works for us.


Reduce arguments to handle_cleartext_payload in PacketSpaceManager

handle_cleartext_payload currently requires 8 arguments, which is more than Clippy is happy with. We should potentially gather some of the arguments into a Struct to simplify

there are a few more functions that take too_many parameters. search for all instances of #[allow(clippy::too_many_arguments)] and possibly create structs

We should consider ignoring too_many_arguments as there doesn't seem to be a good alternative most times


Finish writing the rendering script ！！！！  Look！！

This is the address of my repository, according to Openlrm to modify the rendering script to conform to the Instantmesh dataset
https://github.com/Mrguanglei/Instantmesh_scriptData?tab=readme-ov-file

谢谢！！可以认识一下您吗？感觉您太厉害了！！

Nice, thankssss a lot!

This is the address of my repository, according to Openlrm to modify the rendering script to conform to the Instantmesh dataset
https://github.com/Mrguanglei/Instantmesh_scriptData?tab=readme-ov-file

Care to explain a little what you've done here?
is this using OpenLRM?

have you tried training the instant mesh? Need some help with training. Can we connect?
mail<EMAIL_ADDRESS>

This is the address of my repository, according to Openlrm to modify the rendering script to conform to the Instantmesh dataset
https://github.com/Mrguanglei/Instantmesh_scriptData?tab=readme-ov-file

Care to explain a little what you've done here?
is this using OpenLRM?

+1


Add alert for DB connection exhaustion

DEVOPS PULL REQUEST
Related Issue

Postmortem item. We decided that we needed better insight into connection pooling issues, and connection exhaustion in particular.

Changes Proposed

Added an alert that triggers on >= 5 instances of connection exhaustion over a 5 minute window.
Moved the various datas in services/alerts/app_service_metrics to a data.tf file for better organization

Testing

Deploying the branch to a dev environment should deploy the new alert (or locally if you have local Terraform set up).

Checklist for Primary Reviewer
Infrastructure

[ ] Consult the results of the terraform-plan job inside the "Terraform Checks" workflow run for this PR. Confirm that there are no unexpected changes!

Cloud

[ ] If there are changes that cannot be tested locally, this has been deployed to our Azure test, dev, or pentest environment for verification

Heads up, it looks like terraform plan is kicking back some errors for this.

Heads up, it looks like terraform plan is kicking back some errors for this.

I wondered when/if we'd run into this. The reason here is that this PR adds a new output from persistent, which gets applied before the rest of the env, but terraform plan has no way of knowing that. I'll have to split this up.

Heads up, it looks like terraform plan is kicking back some errors for this.

I wondered when/if we'd run into this. The reason here is that this PR adds a new output from persistent, which gets applied before the rest of the env, but terraform plan has no way of knowing that. I'll have to split this up.

Actually it looks like I can manually apply the new output (temporarily), since it's just a new output. This mimics what the new state will be once applied, and the plan is now passing.


sink(ticdc): split the updat events all at once

What problem does this PR solve?

Issue Number: close #9689
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`

/check-issue-triage-complete

/test dm-integration-test

/retest

/retest

/test dm-integration-test


parser: implement named array type declaration

Fixes neugram/ng#149.

Codecov Report

Merging #150 into master will decrease coverage by 0.12%.
The diff coverage is 68.75%.

@@            Coverage Diff             @@
##           master     #150      +/-   ##
==========================================
- Coverage   50.18%   50.05%   -0.13%
==========================================
  Files          22       22
  Lines        9337     9347      +10
==========================================
- Hits         4686     4679       -7
- Misses       4184     4199      +15
- Partials      467      469       +2

Impacted Files
Coverage Δ

parser/parser.go
67.76% <68.75%> (-0.21%)
:arrow_down:

parser/scanner.go
64.3% <0%> (-2.24%)
: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 bf80c73...0af2d79. Read the comment docs.


Wrong wheels for python-ldap

Hi!
This is a followup to #97 (it will be more visible than a closed PR :smile: )
The python-ldap wheels that were uploaded to wheels.galaxyproject.org are not the good ones.They come from https://depot.galaxyproject.org/starforge/wheels/build-132/ while the good ones are in https://depot.galaxyproject.org/starforge/wheels/build-133/
The only difference between the 2 builds is a49f517 that fix a linking problem.

Resolved on gitter.


Interop stress tests

@dgquintas - I have made the changes and pushed the latest changes. Please review them at your convenience.

LGTM. It only needs a merge with HEAD and a jenkins green

Merged with HEAD and pushed the new changes.

There are some issues, see Jenkins for details.

btw, The interop tests failures look unrelated to my change.


Phone Number - Update to Scala 2.12.1 - refs #235. Add topics to conf…

Update to Scala 2.12.1 - refs #235.
Add topics to config.json - refs #125.
Rename specs to test for consistency with other exercises.

thanks @ricemery


Handle arrays where the item type is an object define inline.

Previously, the generated code would have a name conflict, such as:
struct Foo {
    // fields
}

type Foo = Vec<Foo>;

Now it'll append Item to objects that are defined inline:
struct FooItem {
    // fields
}

type Foo = Vec<FooItem>;

I was working on this at the same time as #14 - I think if you accept that PR, I can rebase this one and the tests will pass.

Rebased and passing now.

Thanks, released as 0.4.4


Add subcommands to modify env vars using Job API

This is the "client half" of #1943. It adds a client for the socket-based current-job API, and three new subcommands that exercise the Get, Update, and Delete methods:
# Get the current environment variables and values from the job executor
buildkite-agent env get
# Add or update one or more environment variable values in the job executor
buildkite-agent env set ...
# Remove one or more environment variable values in the job executor
buildkite-agent env delete ...

TODOs:

[x] Tests

buildkite-agent env delete ...

I'd lean towards unset rather than delete, since you asked for opinions 😁
The former seems more unixy envish.


Update sbt to 1.6.2

Updates org.scala-sbt:sbt from 1.6.1 to 1.6.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #366 (b5f842e) into main (2e916ff) will decrease coverage by 0.08%.
The diff coverage is n/a.

:exclamation: Current head b5f842e differs from pull request most recent head 7416fb6. Consider uploading reports for the commit 7416fb6 to get more accurate results

@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
- Coverage   96.63%   96.54%   -0.09%
==========================================
  Files           9        9
  Lines        1130     1130
  Branches      111      111
==========================================
- Hits         1092     1091       -1
- Misses         38       39       +1

Impacted Files
Coverage Δ

core/shared/src/main/scala/cats/parse/Parser.scala
96.48% <0.00%> (-0.11%)
: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 2e916ff...7416fb6. Read the comment docs.


PrettyName on EmailTemplates

Hi,
I think there's an error in email templates, on @field.PrettyName, the message shows the field type (eg, "Textbox") and not the pretty name (eg "Telephone").
I don't know if there is any other way to get the "Name" attribute or even the "Placeholder" attribute.
Thanks!

Oh! Heh... funny. I'll fix that asap.
You'll want to use @field.Name instead. It should work with the current version.

Thanks! @field.Name works!
If it isn't difficult, it could be interesting be able to access to the "Label" attribute too (eg @field.Label).

It's not too difficult - it's all in the API. If you've based your work on the sample email template, you'll just need to do a bit of casting.
The sample email template loops through the form value fields returned by Model.AllValueFields(). These are of type FieldWithValue, and do not necessarily have a label because the label is defined on the type FieldWithLabel (alongside the help text). However, FieldWithLabel inherits from FieldWithValue, so if you can cast the field to FieldWithLabel, you can access its label (and help text).
Try something along these lines:
@foreach (var field in Model.AllValueFields())
{
  var fieldWithLabel = field as FormEditor.Fields.FieldWithLabel;

<EMAIL_ADDRESS>    @(fieldWithLabel != null ? fieldWithLabel.Label : null)
    @(fieldWithLabel != null ? fieldWithLabel.HelpText : null)
    @field.SubmittedValueForEmail()

}

Word of warning: I haven't tested if that code actually works, but I think it will - if nothing else it'll give you an idea of what you can do.

The email template has been fixed in the latest release (<IP_ADDRESS>).


After reading the readme

I finish reading the readme and go to the website in Heroku
I almost understand the visa-application-platform.
What is the next you want to do for visa-application-platform?
For example:
new function 、 new action 、new design  ......

I write this application to polish up my Rails skills, the next thing I want to do is improving design patterns, some code might smells bad since I don't have enough experience. What's your opinion on that ?

Keep coding for your applicaton and git commit to your github
One Step:
List the new functions and actions you plan to do. For example: register module for users and so on
To use a lot of awesome gems for your application. You know the gems is the core part of Rails
Don't forget "Rails guide". When you forget something, you can find the things you want in "Rails guide".
When you have the time, repeart the "Rails guide"
Understand how to use ajax
Two Step:
《Effective Ruby》
《The Ruby Way》
Metaprogramming Ruby
Believe me it will tell you that Ruby is amazing.
I help you review the codes when I have time, and give my advice.
In fact, I know the logic form your visa application, but I dont't know the visa application platform in the real world. Would you like to email me that introduce the visa platform program in the real world ?
After that I will know what we could do for visa-application-platform more

You are in winter vacation  :beer::tea::pizza::hamburger::fries::poultry_leg::meat_on_bone::spaghetti::curry::fried_shrimp::sushi::ramen::stew::rice_ball::icecream::cookie::cookie::tangerine::lemon::cherries::grapes::watermelon:

Yes ! How about you ?

I have no winter vacation, but I am not busy recently.  I want to learn Japanese after Spring Day

So you're working, right ? I can speak Japanese, maybe I can teach you.
2017年1月22日 下午3:17，"pathbox"<EMAIL_ADDRESS>
I have no winter vacation, but I am not busy recently. I want to learn
Japanese after Spring Day
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Adagio-cantabile/visa-application-platform/issues/2#issuecomment-274314061,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANGHhTb5o6XV33-onlruEjTGu1YVTQSWks5rUwKEgaJpZM4LhaAi
.

OK, I send a email to your<EMAIL_ADDRESS>Please check it out


State does not get edited in long lasting (background) tasks

I have a slow task that I need to run. I can approach it as a background task or trigger it as a normal event. The problem is that I cannot use async because it is Selenium. Maybe it is worth mentioning that the task uses multiprocessing. After running it for a minute, I cannot modify the state anymore eg.
State(rx.State):
  ...

  def on_click(self):
     return State.slow_task

  @rx.background
  async def slow_task(self):
     async with self:
       print("10% done")
       self.var = "10% done"

     sleep(60) # slow code, I use sleep and not async sleep to simulate Selenium

     async with self:
       print("40% done"
       self.var = "40% done"
     # usually by this point var is not longer updates the UI.
     # var does not get updated in the UI but I can see in the terminal "40% done"

    ...

I have played with setting the following variables (see below) but sometimes it works and sometimes it does not work, sometimes even the on_click event does not get executed. I have also tried making slow_task a not background task and running it.
config.Expiration.PING = 10000
config.Expiration.TOKEN = 10000
config.Expiration.LOCK = 1000 * 60 * 10
config.Ping.INTERVAL = 250_000

What is the best way of running these type of very slow functions?

You really do not want to block in a background task.
Because reflex is an async-based framework, any non-async functions that are long running will block the whole server from processing other requests, sending deltas, etc.
If you need to execute blocking code in reflex, then you can farm it out to a separate process or thread and monitor the result in the background task.
In your code example above, you'll notice that if you open a second tab and try to click the button it actually wont do anything while the first tab is crunching along.
For simple stuff, you can use await asyncio.get_event_loop().run_in_executor(my_func) to run the function in a thread and await the result.
For heavier or CPU-bound tasks, then you can run with a process pool. Here's some example
from __future__ import annotations

import asyncio
import multiprocessing
import multiprocessing.pool
import time

import reflex as rx

_global_pool: multiprocessing.Pool | None = None

class State(rx.State):
    var: str = "Not Started"

    def _pool(self) -> multiprocessing.Pool:
        global _global_pool
        if _global_pool is None:
            _global_pool = multiprocessing.Pool()
        return _global_pool

    @rx.background
    async def slow_task(self):
        start_time = time.time()
        async_result = self._pool().apply_async(time.sleep, (60,))
        async with self:
            self.var = "Working..."
        print("Dispatched task to process pool")

        # wait for the task to complete
        while not async_result.ready():
            await asyncio.sleep(1)

        print(f"elapsed time: {time.time() - start_time}")
        print(f"result: {async_result.get()}")
        async with self:
            self.var = "Done!"

def index() -> rx.Component:
    return rx.container(
        rx.vstack(
            rx.button("Run slow task", on_click=State.slow_task),
            rx.text(f"Status: {State.var}"),
            align="center",
        ),
    )

app = rx.App()
app.add_page(index)

Thank you for your answer, I will give it a try and get back to you!

Your idea worked great. thanks for suggesting it!


Improve order of transforms

This PR moves the transforms around a little bit to have the stylistic ones at the end, which will hopefully catch and standardize any additional nodes that were added in subsequent transforms.
cc/ @bouk @GoodForOneFare @fandy

Maybe we should apply all the transforms in a loop until we reach a stable position :trollface:
lgtm


Allow loading images with crossorigin="use-credentials"

Currently there aren't any ways to customize how images are loaded via CORS aside from making a new image class. I've adjusted BrowserImage to keep whatever CORS mode is already on the passed in HTMLImageElement, so to load an image with "use-credentials", I can do
const img = document.createElement('img')
img.crossOrigin = 'use-credentials'
img.src = 'https://mydomain.com/a.png'
return Vibrant.from(img).getPalette()

In the future I think there should be a better way to customize this behavior, but this suits my needs for now.

@kherock I don't know the status of upstream, but I've been doing my best to get this library updated and fully working (and will try to make PRs to bring them back to upstream so they can be merged when ready). Feel free to checkout my branch, I've added the build info in master so you can use npm's git@ feature to npm i the fork if you need the merge fixes applied out of box


Support on default templates of IntelliJ annotations

https://www.jetbrains.com/help/idea/template-data-languages.html#special-comments

Comment
Description

@implicitly included
Any  FreeMarker file that starts with this comment is included into all  files that are in the same module or its dependencies. Use such a  dedicated comments file to define IDE-specific comments, so that they do  not pollute your template files.

@ftlvariable
Defines a variable. For example, the following comment declares a variable named foo of type String, visible only in the specified file: <#-- @ftlvariable name="foo" type="java.lang.String" file="path/to/file" -->

@ftlroot
Defines the path relative to which import and include  directives are resolved. The path is specified relative to any of the  directories in the current FTL file's package. Paths inside JARs are  also supported, for example: <#-- @ftlroot "path/to.jar!/path/inside/jar" -->

Issue #9 it's required to handle this.


add feature inherit_auto_import

to support DRY: add a flag to allow sub_pools to inherit parent auto_import

Isn't that top_auto_import ?

No, that just sets auto_import at the top level pool. currently you have to set auto_import at every level. pools, by design dont inherit any attributes from the parent pools


fix(logger-plugin): fix next.js url logging (baseUrl not supported)

Description
fixes
[HPM] GET undefinedundefined -> http://jsonplaceholder.typicode.com/users [304]

Motivation and Context
Next.js doesn't seem to expose the baseUrl (/api) ...
How has this been tested?
Manual
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Coverage remained the same at 97.872% when pulling 339b7e323b67b801848d798686c87797af563821 on logger-plugin-nextjs-fix into 2d28db7ff063403079d090f000673c3f1995667d on master.


Fails to concatenate the whole last file.

I'm using gulp-concat version 2.4.0 and gulp 3.8.7.
This plugin isn't concatenating the whole file in my list of js files. Specifically, it's ignoring a portion of the last JS file in the array.

Here is my task code which does not consistently concatenate all specified files from the modules array:
// Process project scripts.
gulp.task('scripts-project', ['transform-jsx', 'jshint'], function() {
    var modules = ['model', 'view', 'controller', 'main'];
    var jsSrcDir = './public/src/scripts/compiled-jsx/';
    var jsDst = './public/build/scripts';
    var jsSrc = modules.map(function(module) { return jsSrcDir + module + '.js'; });

    gulp.src(jsSrc)
        .pipe(sourcemaps.init())
        .pipe(to5())
        .pipe(concat('app.js'))
        .pipe(uglify())
        .pipe(sourcemaps.write('.'))
        .pipe(gulp.dest(jsDst))
});

If I understand correctly, this issue seems to occur when concatenating a list of specific files.

A couple more notes:

Sometimes changing the order of the files in the modules variable helps but there's only a certain extent to which that could be done (otherwise an ordered list would be unnecessary).
The issue has occured on both Windows and Linux (Ubuntu).

My repo is here. Since sourcemaps are made in the task, it is apparent in the screenshot below that the main.js module specified in the task is missing.

@jthuraisamy Your gulpfile is totally messed up in that repo, you might want to make a new repo that isolates the issues a little better.

You're right, @contra. After more testing with a reduced gulp task, the issue does not seem to be with gulp-concat after all. For example, the following code also intermittently produces an empty main.js file.
gulp.task('scripts-project', ['transform-jsx', 'jshint'], function() {
    var modules = ['model', 'view', 'controller', 'main'];
    var jsSrcDir = './public/src/scripts/compiled-jsx/';
    var jsDst = './public/build/scripts';
    var jsSrc = modules.map(function(module) { return jsSrcDir + module + '.js'; });

    gulp.src(jsSrc)
        .pipe(gulp.dest(jsDst));

Perhaps @beardedinbinary can try his task code a few times without the .pipe(concat('all.js')) line, and see if the resulting feedback.js and docCookies.js are randomly blank.

Does it still occur in v2.5.0?

I can confirm it still occur in the v2.5.2
Here, I would say that ./frontend/tests/integrations/*.js get ignored about 30% of the time
gulp.task('concat-test', function() {
    return gulp.src([
        "./frontend/tests/app.js",
        "./frontend/src/router.js",
        "./frontend/src/helpers.js",
        "./frontend/src/inputs.js",
        "./frontend/src/injections/*.js",
        "./frontend/src/mixins/**/*.js",
        "./frontend/src/components/**/*.js",
        "./frontend/src/models/**/*.js",
        "./frontend/src/controllers/**/*.js",
        "./frontend/src/routes/**/*.js",
        "./frontend/src/views/**/*.js",
        "./frontend/src/components.js",
        "./frontend/tests/helpers/*.js",
        "./frontend/tests/fixtures/*.js",
        "./frontend/tests/integrations/*.js"
    ])
    .pipe(concat('test.js'))
    .pipe(gulp.dest('public/js/'));
});

Anybody want to make a repo so I can reproduce?

I will try to put something together for you @contra , I will keep you tuned.

With the same file structure and the isolated gulpfile, I'm not even able to reproduce the bug... that's very strange... I'm lost on that one..

@pboutin Which version of Node are you using?

@shinnn node --version : v0.10.25

@pboutin
I ran into Buffer-related problem when using older Node, but the problem no longer occurs after I update Node.
I recommend using the latest stable version (v0.12.0) or io.js v1.3.0 whether it fixes gulp-concat problem or not. v0.10.25 is too older version.

Did it with no success... unfortunately

Still bugging, sometimes the last "file selector" is missing, sometimes the first "file selector" is at the end... for no reason...

Same here. Node 0.12.0, gulp-concat 2.5.2.

I can't reproduce it and if none of you can either then I don't know how I'm going to debug this one

After updating my Node, the problem persisted. The problem went away after a full reboot (don't know how it could be related but it's what I observed on my side).
Everything runs smoothly now !

@contra @pboutin New to Gulp -- I was having this exact problem, but it turned out to be that I was not doing the async plumbing properly from task to task, i.e returning gulp.src.. and specifying task dependencies per task. Once I did that, the inconsistency went away.

I still have this issue in Gulp 3.9.1 when using gulp-concat. It's not the last file - it's one of the files. I have set dependencies etc. It works for a while after restarting my PC - then the output of that file is empty again.


Why is there such a big difference in the results between the models trained with nerfacto and neus-facto?

Describe the bug
ns-train neus-facto  --pipeline.model.sdf-field.inside-outside False   --pipeline.model.sdf-field.bias 0.3  --pipeline.model.background-model grid --pipeline.model.eikonal-loss-mult 0.0001 nerfstudio-data    --data data/nerfstudio/tangtousimple

But the result generated through the under command is rendered almost identical to a photograph.
ns-train nerfacto --data data/sdfstudio/tangtousimple

Hi, could you check how the near and far plane are set? Could you try to manually set the near and far from CLI with --pipeline.model.near-plane 0.01 --pipeline.model.far-plane 1000 --pipeline.model.overwrite-near-far-plane True --pipeline.model.background-model grid?
We have just updated the neus-facto to also use proposal network for background modeling which will likely generate better results.

@niujinshuchong  It got worse after adding these parameters.
ns-train neus-facto  --pipeline.model.sdf-field.inside-outside False  --pipeline.model.sdf-field.use-appearance-embedding True  --pipeline.model.sdf-field.bias 0.3  --pipeline.model.near-plane 0.01 --pipeline.model.far-plane 1000 --pipeline.model.overwrite-near-far-plane True --pipeline.model.background-model grid --pipeline.model.eikonal-loss-mult 0.0001  nerfstudio-data    --data data/nerfstudio/tangtousimple

I will try the updated version again, as well as the bakedangelo model.  Thanks for your work.

@niujinshuchong The latest code throw error: AttributeError: 'NeuSFactoModel' object has no attribute 'curvature_loss_multi_factor'

@niujinshuchong What is the minimum VRAM requirement for running BakedAngelo? I'm experiencing a shortage of graphics memory when running the BakedAngelo model with 130 images. My VRAM is 12GB.

@xiemeilong The default setting of bakedangelo runs on 24 GB GPUs. You could try to reduce the number of training rays. Btw, why do you use a very small eikonal loss?

@niujinshuchong Because the training effect was very poor using the default configuration, I found this parameter 0.0001 from your example and it improved the result. Moreover, changing it to 0.001 made it worse.

@xiemeilong Do you mind sharing your data? I can check when I have time.

@xiemeilong Even when I set my pipeline.datamanager.train_num_rays_per_batch to 256, I still receive a CUDA out of memory error.

@niujinshuchong I am sorry，as it is an electrical facility, the images is not allowed to be made public.

OK, you can also reduce the learnable parameters of bakedangelo such as log2_hashmap_size and hash_features_per_level.

--pipeline.model.near-plane 0.01 --pipeline.model.far-plane 1000 --pipeline.model.overwrite-near-far-plane True --pipeline.model.background-model grid

I have the same question. Use nerfstudio 's nerfacto, I didnt adjust so many parameters, actually nothing,
however, the result was quite good.

Hi, currently the sdf-based methods are not very robust. As many people found that nerfacto can get good result, we will try to add a new option to use nerfacto for pre-training and then switch to SDF. Will make an update if it works.

@niujinshuchong The upper part of the object was not captured. Could this be the reason why the SDF model failed to reconstruct it well?

Today  I took new photos and once again achieved excellent results using Nerfacto, while Neus-Facto didn't work well. I can share this data with you.
ns-train neus-facto  --pipeline.model.sdf-field.inside-outside False  --pipeline.model.sdf-field.use-appearance-embedding True  --pipeline.model.sdf-field.bias 0.3  --pipeline.model.near-plane 0.01 --pipeline.model.far-plane 1000 --pipeline.model.overwrite-near-far-plane True --pipeline.model.background-model grid nerfstudio-data    --data data/nerfstudio/shizi

https://drive.google.com/file/d/1CMbkWRTDJB6DIod3B2tPsIitpJGWYyPn/view

@xiemeilong I don't have access permission for your data.

@flow-specter Looks the model overfits to the background color. Maybe you could try to set the level_init to a smaller value.

@niujinshuchong https://drive.google.com/file/d/1Yv4cI9OebVoWIxVlYQc_YYafiVnv7jni/view?usp=sharing

@xiemeilong I just downloaded your data but it only has images and camera information is missing? Also the images is different as the one you shared above?

@meneldil12555 I think I met the same problem with yours: I found that when I use neus-facto(bakedsdf won't) to train my outdoor dataset and it will breakdown in every first eval(I checked the nvidia-smi and the VRAM consumption will continously increasing until OOM, then the running stoped). So what is the "output " you mentioned, color/density...? How to check it or log it? Thanks.

At that time I found the losses are NaN from wandb logs, and during the first eval assertions of the output image will stop the program. If this is your problem you can try to add some more assertions at the model or field part of code.


Make tx JSON structure consistent

Right now the JSON structure just follows what's in the bitcoin node (as well as WoC). Unfortunately that isn't the most consistent and clean structure (value/satoshis, scriptsig/scriptpubkey, etc.). I don't think we should constrain the library to fit that. go-bt should be as consistent and clean as possible and we can just add a tx.ToNodeJSON() method or something like that to compensate for that in the short term and then at some point we'll probably just deprecate that.

another reason to do this is that i just realised this index field that was added to the output struct - don't think it should be there, it's up to the tx to set the index of the output, the output itself shouldn't be aware of higher level logic. feels like a big code smell...

Still need to sort out entire lib (not just the output)


Fixes post formatting

Click merge if the button is green :)

I don't know about the image, I think you need to move it into an images folder. Then change the path in the post.

Bah! I don't know how to make an images folder :S whereabouts do you do that? Also, when you click on the "Read More" link on the site (which actually comes at the end of the blog post rather than only showing a bit of the post), it goes to a page that has no styling.

@jessgurr don't worry! It's all good, I'll click the button and show you. Luckily I have permission to change things too :smile:


Write title by default

Add an option to focus title instead of text on note creation

Great idea! Added for the next release. Thanks.


3.5 regression concerning type inference for non-inferable types

TypeScript Version<EMAIL_ADDRESS>
Search Terms:
3.5, infer
Code
// mocked CssProperties type
type CssProperties = { position: "absolute" | "relative" };

type StylesObject<ClassKey extends string> = Record<ClassKey, CssProperties>;
type PropsBasedstyles<ClassKey extends string, Props extends object> = (
  props: Props
) => StylesObject<ClassKey>;
type Styles<ClassKey extends string, Props extends object> =
  | StylesObject<ClassKey>
  | PropsBasedstyles<ClassKey, Props>;

function createStyles<ClassKey extends string, Props extends object>(
  styles: Styles<ClassKey, Props>
): Styles<ClassKey, Props> {
  return styles;
}

const styles = createStyles({
  root: {
    position: "absolute"
  }
});

Expected behavior:
Behavior in 3.4.5
typeof styles = Styles<'root', {}>
Actual behavior:
Behavior in >=3.5.0
typeof styles = Styles<'root', object>
Playground Link:
playground
Related Issues: Did not find any
Notes:
Seems like the playground is still using 3.4.
Usage of this behavior is showcased in https://github.com/eps1lon/ts-inference-3.5/blob/master/index.ts with a fix for 3.5.

This is an effect of #30637, a known breaking change. In your example we make no inferences for the Props type parameter in the call to createStyles. This means we default to unknown, but since that doesn't satisfy the constraint object, we pick object. Previously we'd default to {} which did satisfy the constraint.

Suspected as much though I wasn't aware how this interacts with non-inferrable types. Thanks for the explanation.


Remove local redirect policy entry and potentially restore original clusterIP service entry sooner upon last local backend delete

Proposal / RFE
Is your feature request related to a problem?
Currently, nodelocaldns deployment with Cilium utilizes LRP to redirect DNS traffic to the local dns-cache pods. When the local dns-cache pods are down (e.g., due to rolling upgrade), there might be a disruption in DNS queries because cilium-agent would only restore the original DNS service upon the local pod's delete event. The delay from when local dns-cache is gone to when cilium-agent restores the DNS service seems to be the root problem here.
Describe the solution you'd like
Since LRP reacts on "local" pods, maybe we could hook into local CNI pod events (e.g., delete) and try to restore the original service (from svcCache) sooner.
Ofc I'm open to other possible solutions too :)

Had discussion with @aditighag @aanm offline over slack, creating an issue to track the discussion more formally here.

re @aanm , we would still treat the controlplan's svc info as the ground truth and always restore to that, so even if the original clusterIP is invalid (and some new clusterIP is allocated), we should be able to pick up that too? I'm a bit confused why is that an issue?

Thanks for filing the issue.
Title is confusing. LRP is a control-plane abstraction for users to configure node-local redirection. The LRP manager behind the scenes monitors relevant services/pods to set up the networking/datapath. Also, I'm looking for more details in the proposal to understand whether it's feasible or not. :)

I'm also wondering if you have any data around latency incurred in different LRP orchestration stages? It'll be good to attribute delays to watcher related events vs local-redirect service removal and original service restore window, etc.

I'm also wondering if you have any data around latency incurred in different LRP orchestration stages? It'll be good to attribute delays to watcher related events vs local-redirect service removal and original service restore window, etc.

good point, I'll do some profiling and post back here.
Also changed the title to be a bit more precise.
My thought is that we don't change how LRP manager currently handles restoration, but we try to invoke the restoration earlier upon last local backend delete, then when agent gets the event from control plane, it'll try to restore it again and fail (but it's okay since it is already restored). We can control this early restoration with a flag or something so that it can be switched on in scenarios where it is required to not have an outdated LRP policy in place (like in the DNS case).

I'm also wondering if you have any data around latency incurred in different LRP orchestration stages? It'll be good to attribute delays to watcher related events vs local-redirect service removal and original service restore window, etc.

good point, I'll do some profiling and post back here.
Also changed the title to be a bit more precise.
My thought is that we don't change how LRP manager currently handles restoration, but we try to invoke the restoration earlier upon last local backend delete, then when agent gets the event from control plane, it'll try to restore it again and fail (but it's okay since it is already restored). We can control this early restoration with a flag or something so that it can be switched on in scenarios where it is required to not have an outdated LRP policy in place (like in the DNS case).

Removing LRP out of the picture, isn't this a problem that could happen with any application backend that is behind a service?

Removing LRP out of the picture, isn't this a problem that could happen with any application backend that is behind a service?

Hmm...good point...maybe the proper solution here is to add a controller that monitors dns-cache and manipulates the svc entry directly as a special case?

Will send out PR soon.


Update Setup-SCCMOfficeUpdates.ps1

Renamed all functions not using correct verb. Set 2016 as the default ProductVerison and added ProductVersion and Branch parameters. A little code cleanup where variables were not being used. Those were commented out. Included setting Branch and SourcePath to New-DownloadXmlFile function. Probably a few other changes that I can't remember.

Hi @zakgodwin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Added note to indicate that this is not yet supported in Azure gov cl…

…oud.
This feature is not supported in Government clouds.

@ramakoni1 : Thanks for your contribution! The author, @MashaMSFT, has been notified to review your proposed change.

@ramakoni1 : Thanks for your contribution! The author, @MashaMSFT, has been notified to review your proposed change.

#sign-off


validation pdf from markdown file

generates a pdf from the markdown file @giblet22 created and puts it where it is available for download

great! thanks @danbjoseph


Firebase TODOS within commit 'redux-observable ftw'

I was reading through the diffs of the commit 'redux-observable ftw' and saw a couple of TODOS on Firebase.
Since the implementation is really straightforward I won't submit a pull request.
1- Firebase.bolt : 69
 // TODO: Add isViewer and isFirebaseServer somehow.
Checking to see if isFirebaseServer is quite simple. You'd check if the 'auth.uid' field is the same as some predefined value, like so
".read": "auth.uid === 'my-service-worker'",
Check https://firebase.google.com/docs/database/server/start#authenticate-with-limited-privileges for more.
2- configureDeps.js : 10 / 11
// Ensure only one Firebase instance. I don't know how costly new instance is
// and how to dispose of it. Yes, firebase.initializeApp is weird API.

While lurking on how to work with Firebase and AWS Lambda I came across this:
 if(firebase.apps.length === 0) {
    firebase.initializeApp(config);
 }

It's working as expected on Lambda. I'll test this on este setup and edit here.
--I gave a look into the docs and firebase.apps always returns an array of initialized apps. More than one app is only intended if you have two distincts Firebase projects and would need both on the same app. This is not the case on este.

As for isFirebaseServer, it's much more complicated. The problem is, it's not mine worker, it's Firebase Cloud engine itself, which detects lost connections. It must be set to true, because I can not detect isFirebaseServer.
As for initializeApp, if documented, let's use it :-)

It's undocummented.
https://groups.google.com/d/msg/firebase-talk/aBonTOiQJWA/3hCgzDydAQAJ is a response from some Firebase member about this particular technique. I guess multiple Firebase instances are a problem only on development because of HMR.
Perhaps this under some if(isDevelopment) will do it.
All the other feasible (for us) solutions on the mentioned thread did not aim to stop double initialization but only to allow Firebase SDK to run on Lambda.

The new Firebase API is really shitty. Nevermind. I think the current workaround is good enough for Este. I don't see a reason why I should change what works. Anyway, thank you for an effort.

They're now serving the Firebase SDK split by services.
var firebase = require("firebase/app");
require("firebase/auth");
require("firebase/database");

// Leave out Storage
//require("firebase/storage");

var config = {
  // ...
};
firebase.initializeApp(config);

check here: https://firebase.google.com/docs/web/setup
By not using storage and messaging sdks we can get rid of about 90kb of minified code.

Unfortunately, I can't.
https://github.com/este/este/blob/master/src/common/configureDeps.js#L2
It's actually a nasty bug. It just timeouts in Node.js.  So, it's the trade-off as always. 90kb is just one little image.


[DOC-3747] Update install delegate tutorial

Updates to the  Install delegate tutorial

Remove left nav from screenshot
Fix typos
Update instructions

For reviewers, a preview is available.
Install Harness Delegate on Kubernetes or Docker
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


fix(dht routing): Re-enabling DHT routing

:tada: This PR is included in version 6.8.6 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Add DataTransferManger interop APIs

Add DataTransferManger interop APIs
Update some interop api supported version(Because some of the minimum versions required for WinRT interface interoperability apis do not correspond to the official documentation)

@Gaoyifei1011 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree


Full Folder Decryption file path having incorrect property name

On line 81, there is reference to a property on the File Name mistyped as '$File.FuleName', should be '$File.FullName'
https://github.com/EvotecIT/PSPGP/blob/1741478a441c748ee42c1c8cce6bf7b8432a58a1/Public/Unprotect-PGP.ps1#L81

This and a few other typos should be fixed. Published as well.


Add payment processing options to QRs and deep links

Description
Adds additional optional parameters to the QR code scheme in the app and formalizes encoding/decoding:

adds currencyCode for locale specific PoS systems
adds amount for prefilling transaction value in SendConfirmation component
adds comment for use with transferWithComment

Other changes

Adopts io-ts for parsing qr data contents
Sets preferredCurrencyCode state to qr contents to minimize code churn

Tested
celo://wallet/pay?address=0xf7f551752A78Ce650385B58364225e5ec18D96cB&displayName=Super+8&currencyCode=PHP&amount=500&comment=92a53156-c0f2-11ea-b3de-0242ac13000
corresponding QR generated from deeplink:

Currently app does not gracefully handle insufficient user balances on the SendConfirmation screen
Related issues

Fixes #[issue number here]

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

@yorhodes you'll need to update the yarn.lock by running yarn