Strip unknown fields from settings.info during validation

And use the cleaned up value parameter instead of settings.info.
I'm using the popular node-config module to manage all sorts of hapi configuration option (including parameters for hapi-swagger). Unfortunately it seems to inject additional functions to help merging parts of the configuration options, thus the validation of settings.info  will fail. This can easily be fixed by asking Joi to drop all unknown additional keys.

Thanks for your 2 PR, they are now in v0.7.2 on npm

Great, thank you!


How to increase the length of the output ?

I am using this code to document some code files:
import os
from llama_index import SimpleDirectoryReader
from llama_index import GPTSimpleVectorIndex

os.environ['OPENAI_API_KEY'] = "sk-..."
documents = SimpleDirectoryReader('', ['c:/harbour/src/vm/arrays.c']).load_data()
index = GPTSimpleVectorIndex( documents )
response = index.query("list, explain and document all the functions and parameters for each function provided in such file")
print( response )

the printed response stops before finishing. How to increase the output length ?

It shouldn't stop. If the error persists you can open an issue on llamaindex repo.
https://github.com/jerryjliu/llama_index


Frequent and very distracting stuttering

At high fps game hickups massively every 10 seconds or so, no matter what I'm doing. Tried every fix I could possibly think of and I'm running on a high end system. Any possible fixes or thoughts? I think it's just a Unity 2017 issue but I don't see anyone else saying they have this issue. It's very distracting during gameplay and really takes me out of the immersion... (P.S. in rivatuner you can see big spikes in the frametime graph with 20 fps drops whenever it happens)

I came here to see if anyone else had noticed this. I swear it feels worse now than it's ever been.  I've been playing since launch and while it's always had an issue with these hitches, I don't remember it being frequent and regular like this.

So I've been trying every single solution in the book for over 2 days now. This game has a periodic stutter that seems to be tied to framerate. Essentially at 120 fps you will get this periodic stutter at twice the speed of 60 fps. But regardless, the game has stutter no matter what you do at set intervals.
From my testing, at least idle at Mondstadt fountain, I've found the rate was roughly 17.5-20 seconds @ 120fps, and 35-40 seconds @ 60fps. This timing changes depending on what your doing but I'm pretty sure relativity is consistent. So essentially the higher the framerate, the faster this stutter occurs. I flipped between 60 and 120 cap on RTSS for my testing, so it isn't just a case of the mod causing the stutter. There's probably nothing the mod author can do since this seems to be a Genshin problem. I've also tested on Ps5, which also seems to have this periodic stutter. However, the ps5 seems to handle stutter-recovery a lot better so it doesn't feel as jarring as PC stutters.
The game also has stutters not related to timing that happens whenever you finish talking to npc, scrolling through the big map, opening the map, etc.
I can't say for certain if this periodic stutter was in previous versions of Genshin. Both my old gpu and my new gpu were able to play Genshin at a locked 60 but maybe because I've been benchmarking my games lately I only just now notice the stuttering to this degree. But personally, I do kind of feel like stuttering is worse this patch than it ever has been. It could have started last patch though, I barely played 2.3.
P.S just read Ryrin45's post, yeah that pretty much lines up with how I feel about it. I don't think the stutter was this frequent in previous patches either.

I'm experiencing the exact same thing as you two, it's periodic and seems to happen more frequently at higher framerates (and more jarring but that might be from the smoother frames normally). Must not be a mod issue but if anyone figures out anything please comment here, otherwise we should complain to Mihoyo.

Hard to know for sure, but the leading theory is that it's related to how Unity engine does garbage collection. This was apparently addressed in later version of Unity, but Mihoyo is either not using a new enough version or isn't using the fix. If this is what's causing it then there's nothing we can do but hope for a fix from Mihoyo.

I can't say for certain if this periodic stutter was in previous versions of Genshin.

It's been a problem since the game's release, and made worse in the 1.1 update which forced a lot of texture re-streaming because of low-end devices being hampered by their lack of RAM.  Just google, search reddit, etc., and people have been complaining about this issue for well over a year.
I haven't tried this tool to see if it affects the framerate in a different way, nor have I tried it after 2.4 due to the anti-cheat update (not risking my OG account and I'm done testing anything on this game), but it must definitely be more frequent at higher framerates, as the times I've played at its locked 60, the issue is less severe, but still quite common.
The devs don't care about anything but money.  Updating to a newer build of Unity, or doing anything to adjust the GC, will most likely never be done, as it doesn't make them money.  Hell, they haven't even improved the camera control when aiming; it's still a juddery (and mouse accelerated) mess because of how they programmed it
I've not actually playtested Honkai Impact (their previous game, also on PC, with 90/120FPS options), but it should also be running on Unity if anyone wants to see if it suffers the GC stutter.

So today they released an update to greatly lower the amount of stutters that happen. I did some minor testing for now but it at the Fountain I went from the previous 35 second intervals to a whole Minute and 40 seconds. Thats about a 150% decrease in stutters which is fantastic. Stutters after talking to npcs, and panning map are also greatly lessened. Im not too sure about the recovery and intensity of the stutters themselves though. Might be about the same. But just having vastly less amounts of stutter is a great step. I don't think ima test the High Framerate stuff for awhile so if anyone else wants to follow up that'd be great. Mainly see if the stutter intervals are more frequent with higher framerates or if it's still the same regardless of framerate now.

So today they released an update to greatly lower the amount of stutters that happen. I did some minor testing for now but it at the Fountain I went from the previous 35 second intervals to a whole Minute and 40 seconds. Thats about a 150% decrease in stutters which is fantastic. Stutters after talking to npcs, and panning map are also greatly lessened. Im not too sure about the recovery and intensity of the stutters themselves though. Might be about the same. But just having vastly less amounts of stutter is a great step. I don't think ima test the High Framerate stuff for awhile so if anyone else wants to follow up that'd be great. Mainly see if the stutter intervals are more frequent with higher framerates or if it's still the same regardless of framerate now.

It seems they increased the time between the stutters. So now at 60fps it's anywhere from 2 to 3 minutes between stutters. The problem is the issue is still tied to framerate. So the rate of stutters still basically doubles at 120fps. While I appreciate that they did anything at all, it seems more like a band-aid rather than a real fix. I hope they have a more complete fix planned for later.

The stutters have improved for sure. It's nice to know the Genshin team are at least aware of it and trying to fix it, but I hope there's more coming. I have to admit however, the stutters have become far less noticable in my gameplay, and I'm enjoying the game just a bit more. The stutters that still occur also seem to be a lot less jarring and they don't always distract me anymore.

So today they released an update to greatly lower the amount of stutters that happen. I did some minor testing for now but it at the Fountain I went from the previous 35 second intervals to a whole Minute and 40 seconds. Thats about a 150% decrease in stutters which is fantastic. Stutters after talking to npcs, and panning map are also greatly lessened. Im not too sure about the recovery and intensity of the stutters themselves though. Might be about the same. But just having vastly less amounts of stutter is a great step. I don't think ima test the High Framerate stuff for awhile so if anyone else wants to follow up that'd be great. Mainly see if the stutter intervals are more frequent with higher framerates or if it's still the same regardless of framerate now.

It seems they increased the time between the stutters. So now at 60fps it's anywhere from 2 to 3 minutes between stutters. The problem is the issue is still tied to framerate. So the rate of stutters still basically doubles at 120fps. While I appreciate that they did anything at all, it seems more like a band-aid rather than a real fix. I hope they have a more complete fix planned for later.
Though I can't comment on the severity of the stutters themselves yet. I'll have to play more and see if they feel as bad.

Damn, yeah i figured it would still be tied to framerate. I guess maybe though it would atleast be playable status now even with high framerate. Maybe compromise @ 100fps or something wouldn't be too bad too instead of 120. Il give it a try AFTER I claim my Lantern Rite char. Not that I expect anyone to get banned for using this mod but... just in case.

Same here,but i noticed that decreasing Crowd Density and GPU load helps a lot. Clearly game optimization issue. It is also overloading the USB devices somehow because only playing genshin cause wdf100sys to spike up and audio stutters or disappears.


Error: rpcUrl could not be found for network: arbitrum

hello.  I want to test on kovan testnet. but it throw the error. Error: rpcUrl could not be found for network: arbitrum.
i just run REACT_APP_NETWORK=kovab yarn run dev

and when i tested online https://kovan.hop.exchange/send?token=USDC&sourceNetwork=ethereum&destNetwork=xdai, i had some usdc. but it did not show.
thx for your help.

The only supported network on kovan is gnosis chain (xDai) so you need to update the env var REACT_APP_ENABLED_CHAINS=ethereum,gnosis


<EMAIL_ADDRESS>

[Enter feedback here]

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3369738b-f096-6c2e-22d4-ee79057997c7
Version Independent ID: d3fce773-a1e8-193a-b4fa-7f71b9ac3ea4
Content: Office 365 Security including Microsoft Defender for Office 365 and Exchange Online Protection - Office 365
Content Source: microsoft-365/security/office-365-security/overview.md
Product: m365-security
Technology: mdo
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

Pls open the Test is two day imstriklyng

@16015299 Thank you for your feedback.  Could you please elaborate on the issue that you are facing or what changes would like to see in the document?
Thanks
Sri

@16015299 We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Added relevant tags to several cakes, and changed chem in Spacemans cake

About the PR
Tweaked numerous Tags in "Cake.yml" for consistency
Why / Balance
Several cakes were missing relevant tags, and Spaceman's cake, which previously had omnizine, has had its relevant chem Polypyrylium Oligomers added.
Technical details
Added tags to numerous Cakes, and changed the Chem in Spacemans Cake.
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl:

tweak: Spacemans cake now contains Polypyrylium Oligomers instead of Omnizine.
fix: Several cakes now have relevant tags.

The vegetable tag is only used for the "get 14 vegetables" cargo bounty, but since cake is blacklisted it does not make any difference for gameplay.

Burgers, tacos and skewers (or any food sequence) should probably be blacklisted here as well.
The meat tag for brain food makes sense and it allows lizards to eat them.
Slime should not count as meat in my opinion.
Is the christmas cake a fruit cake?
The PolypyryliumOligomers replacement for Omnizine is a good idea.

Is the christmas cake a fruit cake?

https://en.wikipedia.org/wiki/Christmas_cake - yup, yes

Slime should not count as meat in my opinion.

Slime balls do have a tag Meat

Slime is tagged as meat when slimeballs and can be consumed by Lizards already, so I think it's reasonable to assume that lizards would also be able to eat it. Will remove Vegetables tag from carrot cake if it's useless.

Will remove Vegetables tag from carrot cake if it's useless.

I guess you can leave it there if it is consistent with other items tagged as vegetables. Maybe someone will use that tag at some point.

Lizards can drink slime as well, which is why I added the tag. If there's a more appropriate tag to add that lets lizards eat it I would add that, but from what I've seen I can't seem to find one is the issue. Can cut it if you don't want it, just making a case for it.

You convinced me 😄

Updated CL. Haven't mentioned Carrot Cake being changed since it will (currently) have no In game Effect

Thank you for your first contribution! I edited the changelog for clarity, I hope you don't mind
Tested it in-game and it all seems fine


drop npm v1.x.x support

Currently we support npm v1.4.0 and greater.  This will allow us to upgrade production dependencies (or dependencies thereof) which use carets in semver ranges.  It should also allow scoped packages.
I'm not sure exactly in what version of npm v2.x.x scoped packages were introduced.  I've asked @seldo if any data is available for npm v1.x.x usage.

See #2621

It looks like the latest Node.js v0.10 installs npm v2.x.x by default.
I think we're OK to drop npm 1.x support.  Probably still necessitates a major release.

this will arrive as part of #3016.


About the number of fields inserted

When I want to insert 3 or more fields, an error is reported.
expect
createTable({
	tableName: 'people',
	tableFields: [
		{
			columnName: 'id',
			dataType: 'INTEGER PRIMARY KEY AUTOINCREMENT',
		},
		{
			columnName: 'name',
			dataType: 'varchar',
		}, {
			columnName: 'age',
			dataType: 'int',
		}, {
			columnName: 'sex',
 			dataType: 'varchar',
		},
	],
});

recurrent

create table

const created = await createTable("changjing", {
	id: "varchar(50)",
	htmlUrl: "varchar(100)",
	typeName: "varchar(100)"
});

want insert 3 col or more

const likeData = {
	id: '1-1',
	htmlUrl: 'https://xx.com',
	typeName: 'music'
}

insert data

const inserted = (insertData: any) => {
	insert('changjing', [insertData])
		.then(({ row, rowAffected, insertID, lastQuery }) => {
			console.log('success', row, rowAffected, insertID, lastQuery);
			setLikeState(true)
			setVisible(true)
			setTiptext('success！')
	})
	.catch((e) => console.log('error：', e));
}

inserted(likeData)

error

@ltsharma
Deleting the data table has no effect, it must change to another data table to work.

Hi,
there is no such limitations for the number of columns for the table, it should work. Can you make sure the column name you have given while creating & the column names while inserting are same?
Because your error shows different column names than you created...

@ltsharma
It needs to reinstall the app or modify the table name to work.

Will consider as a feature request, thank you for your time 😃


update save_location.md to use Kotlin and reflect current API

use Kotlin instead of Java in the docs
reflect current API
align with existing sample

Thanks for the PR @wiryadev


TypeError: get_request() got an unexpected keyword argument

Encountered TypeError: get_request() got an unexpected keyword argument when url has a query parameter using the following environment:
$ pybot --version
Robot Framework 3.0 (Python 2.7.11 on linux2)

But not encountered in the following enviroment:
>pybot --version
Robot Framework 2.9.2 (Python 2.7.10 on win32)

Code:
*** Settings ***
Test Setup		Create Gateway Session

*** Keywords ***
Create Gateway Session
	Create Session			gateway			${GATEWAY}

*** Test Cases ***
Get Categories
    ${resp}=						Get Request			    gateway		/category
    Should Be Equal As Strings		${resp.status_code}		200

Get Categories with pageSize
    ${resp}=						Get Request			    gateway		/category?pageSize=1
    Should Be Equal As Strings		${resp.status_code}		200

Log:
$ pybot test.robot
==============================================================================
Test :: Test
==============================================================================
Get Categories                                                        | PASS |
------------------------------------------------------------------------------
Get Categories with pageSize                                          | FAIL |
TypeError: get_request() got an unexpected keyword argument '/category?pageSize'
------------------------------------------------------------------------------
Test :: Test                                                          | FAIL |
2 critical tests, 1 passed, 1 failed
2 tests total, 1 passed, 1 failed
==============================================================================

Is there a workaround for this?

you are trying to use both url and params instead use
Get Request    / category   params='pazeSize=1'

@vkosuri Thanks! the workaround works.
The version used for windows and linux are different:
windows: requests-2.8.1 , linux: requests-2.9.1

@momacoy thank you, More information passing params, http://docs.python-requests.org/en/latest/user/quickstart/#passing-parameters-in-urls

You often want to send some sort of data in the URL’s query string. If you were constructing the URL by hand, this data would be given as key/value pairs in the URL after a question mark, e.g. httpbin.org/get?key=val. Requests allows you to provide these arguments as a dictionary, using the params keyword argument. As an example, if you wanted to pass key1=value1 and key2=value2 to httpbin.org/get, you would use the following code:

@vkosuri Thanks a lot!


limit my mods to 1.3.1

I took a look at SigmaBinary. In the download for 1.6.11 there's a .version file, but it's missing in 1.7.0. That's probably why there's no game version limitation. Can you add a .version file to that download?
Removing the $kref would prevent any further releases from being detected.

No I removed the KSP-AVC version file because it was yet another thing to update and I don't have the time or patience to update every single thing anymore.
This is also why I would like ckan to only provide that one version for those players that are still on 1.3.1 but not showing the newer versions

Any ETA on when this will be merged?
Not to rush or anything, it’s just that GPP has an update in the wings and it depends on this PR getting merged :)

I would like to help resolve whatever the specific issue is here, but I don't (personally, just my opinion) see this as merge-able as-is due to the potential harm in hard coding the versions and downloads. Users will complain that new releases aren't being indexed. Indexing them will require hand crafted pull requests for each mod each time such a release comes out. This is the reason why the $kref property exists, to automate the things that can be automated.
Is the only motivation here that specific versions are showing up for KSP 1.4 and shouldn't be? If so there are ways to achieve that without causing other problems. We can even set up version-specific overrides to set KSP compatibility based on some simple comparison logic (see #6528 for an example).
I'll be happy to start a successor pull request once I'm sure I understand the requirements.

So no automatic indexing of future updates, ever?
If that's the case it would be simpler to freeze these netkans so they aren't processed at all anymore.

if you prefere freezing them it's fine for me

OK, I will start a pull request for that.
May I inquire as to why you wish to block further updates?

Further follow-up question:
This PR has KKtoSD being hard coded to version v1.0.4, but the download is for v0.1.4, and there's a later version already indexed at v0.1.5. Do you need v0.1.5 to be deleted?
Also @Galileo88, why do you need this for GPP? It's just disabling future updates, not adding or removing any current versions. How does that help GPP?

I also see that there are some versions indexed now with an extra ".ckan" in the version number, e.g. SigmaDimensions-0.10.0.ckan.ckan. Is that related to this somehow?

I don't like the attitude of squad and I have lost all interest in making mods for KSP
I'm only updating my mods for galileo at this point

OK, I believe the latest versions of your mods should now be as per your preference. Try a refresh in CKAN to make sure.

yes, it seems they are correct now, thanks


ConnectionLine Disappearing

Describe the Bug
i can tell what is the problem and i have tried all possible solution.
i have added the edge connecting my two custom nodes. now there are two issue which i am encountering.

ConnectionLine doesnt appear when i start dragging from source until i leave the click when reached on the target.

I have two custom nodes with default edge connecting both. now that too not appearing.

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Seriously dont know why it is happening.
Expected behavior
ConnectionLine should be visible when there is any short of intention to connect two nodes.
Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

can you provide a codesandbox for this?

I also have this problem, have you found the reason?

@LIJUN-XU please provide an example: https://new.reactflow.dev


Victor VosMottor: Watch anon.fm

Victor VosMottor requests the watch of the watch_keyword anon\.fm. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

anon\.fm has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Mast in Charcoal HQ. Reason: 'Insufficient justification, known false positives. Example: https://ru.stackoverflow.com/q/1072766/'


Ubio Tool Development - February 2024

February 2024 Git Commits
https://github.com/ubiodee/SingularityNet/commits/main/?since=2024-02-01&until=2024-02-29

Setup Issue closed - https://github.com/SingularityNET-Archive/SingularityNET-Archive/issues/96


ESP32-S3/C3 Fatal Exception output not sent when using USB Serial (ACM) Mode

Describe the bug
If there is a fatal exception (e.g. stack overflow) when using the USB serial module instead of the UART serial output, then the panic message is not displayed so the system appears to just halt.  Debugging is made a lot more difficult since a debugger must be attached to debug the system and a lot of the context information must be manually extracted.
For example, failure stackcheckfailure here calls log_panic(); StackCheckHandler(); and with the shell being used over USB serial, the system appears to just freeze and quit responding:
uart:~$ failure stackcheckfailure
# The system appears to have locked up at this point

However, if the same command is run over UART0, then everything works correctly:
uart:~$ failure stackcheckfailure
[00:10:07.585,000] <err> os:  ** FATAL EXCEPTION
[00:10:07.585,000] <err> os:  ** CPU 0 EXCCAUSE 63 (zephyr exception)
[00:10:07.588,000] <err> os:  **  PC 0x40379213 VADDR (nil)
[00:10:07.595,000] <err> os:  **  PS 0x60720
[00:10:07.600,000] <err> os:  **    (INTLEVEL:0 EXCM: 0 UM:1 RING:0 WOE:1 OWB:7 CALLINC:2)
[00:10:07.610,000] <wrn> cbprintf_package: (unsigned) char * used for %p argument. It's recommended to cast it to void * because it may cause misbehavior in certain configurations. String:" **  A0 %p  SP %p  A2 %p  A3 %p" argument:1
[00:10:07.632,000] <err> os:  **  A0 0x8037d7c4  SP 0x3fc9aca0  A2 0x2  A3 0x8b524d00
[00:10:07.641,000] <err> os:  **  A4 0x1  A5 0x8b524d00  A6 (nil)  A7 0x8b524d00
[00:10:07.650,000] <err> os:  **  A8 0x8b524d00  A9 0x8b524d00 A10 (nil) A11 0x8b524d00
[00:10:07.659,000] <err> os:  ** A12 0x8b524d00 A13 0x3fc965cc A14 0x3c001eb8 A15 0x60b20
[00:10:07.669,000] <err> os:  ** LBEG (nil) LEND (nil) LCOUNT (nil)
[00:10:07.677,000] <err> os:  ** SAR 0x15
[00:10:07.682,000] <err> os: >>> ZEPHYR FATAL ERROR 2: Stack overflow on CPU 0
[00:10:07.691,000] <err> os: Current thread: 0x3fc93a80 (shell_uart)
[00:10:07.698,000] <err> os: Halting system

To Reproduce
Using an ESP32-S3 board with USB connected to the USB Serial/JTAG connector, change the console to usb_serial.
        chosen {
                zephyr,sram = &sram0;
-               zephyr,console = &uart0;
-               zephyr,shell-uart = &uart0;
+               zephyr,console = &usb_serial;
+               zephyr,shell-uart = &usb_serial;
                zephyr,flash = &flash0;
        };

+&usb_serial {
+       status = "okay";
+};

And then trigger an exception.  I called log_panic(); StackCheckHandler(); for the examples.
Expected behavior
Exception reports should be printed to the USB serial port.
Impact
Causes difficulties in identifying the root cause of failures.

@EricNRS, can you provide your shell sample code so I can reproduce the exactly same behavior?

Thank you for looking into this Sylvio.
Here are patches to reproduce the issue using the samples/net/wifi application.  These should apply cleanly to the v3.4.0 tag using git am *.patch.
0002-DEBUG-Switch-console-to-the-on-chip-USB-Serial-JTAG-.patch
0001-DEBUG-add-forced-panic-for-testing-panic-output.patch
Alternately, you can pull my branch here:  https://github.com/EricNRS/zephyr/tree/2023-09-27-usb-error-output

I might have some insight on this if nobody else already figured it out. In drivers/serial/serial_esp32_usb.c, function "serial_esp32_usb_poll_out", there is a k_usleep call that gets blocked by an __ASSERT when this function gets called from an interrupt.
/* Wait for space in FIFO */
while (!usb_serial_jtag_ll_txfifo_writable() &&
		s_usbserial_timeout < (USBSERIAL_TIMEOUT_MAX_US / 100)) {
        k_usleep(100);
	s_usbserial_timeout++;
}

My workaround is this for now, which is ugly, because more power hungry obviously, but we're trying to push out everything after a crash:
/* Wait for space in FIFO */
const int64_t timeout =
        k_uptime_ticks() +
        (int64_t)k_us_to_ticks_ceil64(USBSERIAL_TIMEOUT_MAX_US);
while (!usb_serial_jtag_ll_txfifo_writable() &&
                k_uptime_ticks() < timeout) {
        if (!k_is_in_isr()) {
            k_usleep(100);
        }
}

Seems to work for me at least.

For convenience, here's my patch for this change:
From 2e894e7d3f9ee4ec923ed6f23e13468f16b0fa6f Mon Sep 17 00:00:00 2001
From: "havoc-dlt"<EMAIL_ADDRESS>Date: Wed, 16 Aug 2023 09:49:12 +0200
Subject: [PATCH] esp32: fix fatal exception output in usb serial mode

---
 drivers/serial/serial_esp32_usb.c | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/drivers/serial/serial_esp32_usb.c b/drivers/serial/serial_esp32_usb.c
index 4a2fe24324..714034f1d6 100644
--- a/drivers/serial/serial_esp32_usb.c
+++ b/drivers/serial/serial_esp32_usb.c
@@ -68,10 +68,14 @@ static void serial_esp32_usb_poll_out(const struct device *dev, unsigned char c)
 	ARG_UNUSED(dev);

 	/* Wait for space in FIFO */
+	const int64_t timeout =
+	k_uptime_ticks() +
+	(int64_t)k_us_to_ticks_ceil64(USBSERIAL_TIMEOUT_MAX_US);
 	while (!usb_serial_jtag_ll_txfifo_writable() &&
-		s_usbserial_timeout < (USBSERIAL_TIMEOUT_MAX_US / 100)) {
-		k_usleep(100);
-		s_usbserial_timeout++;
+		k_uptime_ticks() < timeout) {
+		if (!k_is_in_isr()) {
+			k_usleep(100);
+		}
 	}

 	if (usb_serial_jtag_ll_txfifo_writable()) {
--
2.41.0

I tried both versions. @havoc-dlt's change works.

I tried both versions. @havoc-dlt's change works. For some reason @sylvioalves's version is printing garbage -- looks like some characters are getting dropped:
uart:~$ debug stack_panic
[3> [r>V 0L[Id ia3i[4> xe9[>aa
24 )[>0[>C>500m07>0>00c

This is easily explainable by @sylvioalves's change effectively eliminating the 'wait for FIFO writable' delay which needs to be polled for.

So, I managed to not only mess up the patch formatting, but also my commit message was badly written. How do I correct that on my pull request? Do I open another one, or is it editable and I'm just too dense to find it?

You just have to do a forced push. If you are using the git command line, you do: git push YOUR_GITHUB_REPO_NAME -f
The YOUR_GITHUB_REPO_NAME is from git remote -vv
This is assuming you've already logged in to your Github account using the github command-line client.
It can be complicated. Maybe open up a thread on the Zephry -> Vendor -> Espressif Discord chat: https://discord.com/channels/720317445772017664/883444902971727882

Thanks a lot! Unfortunately I lost patience and decided to retry with another completely new attempt. Maybe I'll need your suggestion later.

Dang, I didn't know that all my other commits from this branch would be added to the pull request automatically too.

Sorry to everyone for all the trouble I'm causing. As previously mentioned, this pull request stuff on github is new to me.

To avoid all changes, you will need to create a new branch off of main and then cherry-pick your fix on top and push that.  At the command line, it will look like this:
# here are the remotes.  "mygithub" is your personal github branch of the Zephyr repo.  "upstream" is the actual Zephyr repo.
git remote -vv show
mygithub	https://github.com/YOUR_GITHUB_ACCOUNT/zephyr.git (fetch)
mygithub	https://github.com/YOUR_GITHUB_ACCOUNT/zephyr.git (push)
upstream	https://github.com/zephyrproject-rtos/zephyr (fetch)
upstream	https://github.com/zephyrproject-rtos/zephyr (push)

# create a new branch based upon the latest zephyr main
git fetch upstream
git checkout main
git rebase upstream/main
git checkout -b fixes-60825-usb-fatal-exception-output

# cherry-pick your development change onto this branch
git cherry-pick HASH_OF_YOU_CHANGE_FROM_YOUR_DEV_BRANCH

# push the change to your github fork
git push mygithub

# if you make edits, then do a forced push
git push mygithub -f

There are different ways you can do this that are a little less verbose, but this one is easy to follow.
You will eventually also want to run checkpatch on more complex changes (see https://docs.zephyrproject.org/latest/contribute/guidelines.html#coding-style).  You can probably skip that for this one until you get the hang of the rest of the process.  I have mine setup to run it before pushing FWIW.

Thanks for being so helpful!

My PR is on hold because it's getting no reviews. It passed all automated tests. Which wait time is considered 'normal' for reviews? Should I be worried it got sweeped under the rug?

It all depends upon the schedules of the reviewers and most of them are doing this as part of their weekly jobs, so they don't do work on the weekends.  I wouldn't be concerned until it gets over 2 weeks.

Ok, fine by me


LaTeXRenderer: Escape special characters in URLs of generated links

The LaTeX renderer doesn't currently do any URL escaping in links:
def render_link(self, token):
    self.packages['hyperref'] = []
    template = '\\href{{{target}}}{{{inner}}}'
    inner = self.render_inner(token)
    return template.format(target=token.target, inner=inner)

This may lead to unexpected results / errors. The escaping should be done similarly to what can be seen for example at #102.

HELP WANTED: I'm no expert on LaTeX and I cannot find a useful documentation on this topic for it quickly. Maybe all that is needed is to URL-encode characters as described at Encoding of umlaute and special characters in \href links, i. e. as the other renderers in mistletoe do now?


[FEAT]:  Specific steps of reproducing SOTOPIA (ICLR 2024) paper

Description
Here I am again , submitting another issue. I have read through part of the code, as well as the contents in the examples and script folders. It seems that run_interaction.sh is the script closest to reproducing SOTOPIA (ICLR2024). However, I am still not entirely sure, so I hope you could provide the specific script or command to reproduce the data in the GPT-4 column of Table 2 (ICLR 2024) (namely results of GPT4 with GPT4, GPT3.5, etc ). Or, perhaps you could first provide an example of implementing the interaction between GPT-4 and GPT-3.5 on SOTOPIA (ICLR 2024), and I will modify it for other models myself. Thanks for your help.
Additional Information
No response

@XuhuiZhou will help with this.

@Kong-Aobo we didn't maintain a bash script for Sotopia (ICLR 2024)
All the bash scripts are for https://arxiv.org/abs/2403.05020
To initiate an interaction, please follow the https://docs.sotopia.world/

@XuhuiZhou First, thanks for your response. Following your guide, to reproduce the ICLR 2024 paper, I found /cli/benchmark/benchmark.py and ran it successfully. I discovered that the Python file only supports running on the hard subset, so I modified the file to support the complete SOTOPIA dataset (which includes 450 tasks). I have a question to confirm with you:

If I want to test a model's capability on the full dataset, this model should play the role of each agent in each of the 450 tasks (each task includes a pair of agents) once. In other words, evaluating this model should generate 900 episodes. Is my understanding correct?

Yes, I think that's correct. Yeah, for the benchmark.py script, it only supports hard set for now. Good to see that you modified it to the complete set. If you want, feel free to make a pull request to include an argument to switch between hard and full sets.


SP Week of May 3rd

[ ] Create design plan for TEMPEST pipeline
[ ] Take inventory of where data processing is / lives for TEROS, Licor, sapflow, wells, and tree inventory

I think you said you'll also tackle integrating 2018-2020 data into the meta-analysis repo and script? 🙏

Yes though that's on the agenda for this week :-)

Monday 5/3

Week plans? Please create a new issue
Specific next steps for data processing? Progress?
Meta-analysis - incorporating 2018-2020 data into script
Interns - BBL update
IRGA checking and downloading - how to handle
Other?


GameScreen data wrongly persists.

After exiting the game screen and re-entering it through the main menu, some data persists. This includes:

Chef locations,
Chef positions,
Chef inventories
Station statuses

Issue created in wrong repository.


RFC: add support for computing the derivative of cardinal sine (cosc)

Resolves # .

Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines, including the relevant style guides.
[x] Read and understand the Code of Conduct.
[x] Read and understood the licensing terms.
[x] Searched for existing issues and pull requests before submitting this pull request.
[x] Filed an issue (or an issue already existed) prior to submitting this pull request.
[x] Rebased onto latest develop.
[x] Submitted against develop branch.

Description

What is the purpose of this pull request?

This pull request:

Add a function to calculate the derivative of the cardinal sine in accordance with the following request:
https://github.com/stdlib-js/stdlib/issues/224

Related Issues

Does this pull request have any related issues?
No
This pull request:

resolves #224

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

The Julia implementation of cosc(x) went bananas for approximately every tenth value of x in the tiny negative and tiny positive test range To make the tests pass I decided to just skip these fixture values (often in the range of 1e+300 when in fact they should be ~0).

@stdlib-js/reviewers

@Rinbo Thanks for working on this! Will review later this evening!


Latest nvim-dap-ui 1.0.0 and 1.0.1 contains breaking changes

the latest nvim-dap-ui release 1.0.0 upwards contains breaking changes, which cause go.nvim to error out when start debugging.
see nvim-dap-ui changes
temporary fix, revert back to version 0.34.4
e.g. with packer.nvim
...
use {"rcarriga/nvim-dap-ui", commit="85b266e20e45004a86b51f13293129b01e2dcf3b"}
...

Pushed a fix should work now


Make detaching null-safety

Motivation
https://github.com/kmagiera/react-native-reanimated/issues/112
Changes
on iOS it's not allowed to invoke callback with NULL, so I wrapped it into NULL-object

@kmagiera
Callback accepts arrays as an argument. It doesn't work with ur changes


Docker Images too large, example cog-resnet is 2.72GB

maybe should add option to support python alpine docker images as base?
the model weights are 99MB yet the image is 2.72GB

after using python 3.9-slim in cog.yaml i get 1.94GB, still needs improvement in my opinion should be about 500MB when in alpine mode

I suspect a lot of the extra is coming from libtorch itself.  Pre-PyTorch 2.0, libtorch is 1.2 GB with CUDA support (~250MB without).  This problem might solve itself if LTO is enabled for the newer versions of PyTorch.  🤔
But yes, a CPU-only Alpine build would probably only need numpy + torch/ONNX, which puts the lower bounds at 500MB like yairyairyair said.


Added best book for programming beginner to advanced!!

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description..
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

@shrn01,plzz check above

Hey @inbuilt-aura we have specific files for each kind of resource already available in the repo. So please use the proper file to add the links to the resources that are free and useful

@shrn01, so you want me to first upload each file in Google drive and then share their link there, correct?


Add velocity

I needed to track velocity in order to decide whether to advance a horizontal slider on mouse up

@zachasme Nice! If you have more ideas, or need faster access, you can share authorship if you like.

Thanks, I'll contact you if that becomes relevant :+1:


Error while copying data disk from classic storage account to ARM storage account

Error Details
If you cannot resolve this issue on your own, you may open an issue on the Storage Explorer GitHub site. Please include the details below to help us diagnose your issue. Make sure to redact any personal information you do not wish to make public before posting.
{
"message": ""Copy failed.""
}

I assume by "data disk" you meant a VHD ?  which version are you using?  Are you able to do anything else with Storage Explorer?

Thanks for the respond. I guess, I got the answer, I think it was failing because I was copying dynamic vhd.
Fixed by copying same vhd to same SA and from there I started copying to another storage account.
From: jinglouMSFT<EMAIL_ADDRESS>Sent: Monday, December 9, 2019 12:06 PM
To: microsoft/AzureStorageExplorer<EMAIL_ADDRESS>Cc: Ram Borude (Tata Consultancy Services Ltd<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/AzureStorageExplorer] Error while copying data disk from classic storage account to ARM storage account (#2404)
I assume by "data disk" you meant a VHD ? which version are you using? Are you able to do anything else with Storage Explorer?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAzureStorageExplorer%2Fissues%2F2404%3Femail_source%3Dnotifications%26email_token%3DANPRCGYYPT2A6WXHNJLQNQTQX2QLVA5CNFSM4JYPP3P2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGKP5TQ%23issuecomment-563412686&data=02|01|v-raboru%40microsoft.com|00207a51f2f4496384a408d77ce340e9|72f988bf86f141af91ab2d7cd011db47|1|0|637115187797426340&sdata=4ayr2MSMD1eMW5KgedcAcrwpYJhxNlSbt5tUysfDlCk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANPRCG4LJEFNRGNNQTQQTT3QX2QLVANCNFSM4JYPP3PQ&data=02|01|v-raboru%40microsoft.com|00207a51f2f4496384a408d77ce340e9|72f988bf86f141af91ab2d7cd011db47|1|0|637115187797436332&sdata=%2Fi6n3InGIE2E3VOSRd20kQtadnQ1KxMrOGaebPaHq4U%3D&reserved=0.

If you are unblocked, I am closing the issue.  Please feel free to reopen the issue if this is not the case. Thank.


Virus test run

@kkkoech, @alpiazza13, @samueljaval, @jeremykattan, could you point me to an example file that your antivirus would flag as a virus? I tried the Single File Check option with some of your testfiles (e.g., eicar.com), but none of the files were flagged as virus. I also tried a few from theZoo, but no luck there either. Probably the ones I tested happen to be not yet in your list of viruses.

@SebastianZimmeck , thanks for the pointer! It could have been an issue with the version. Could you please download the newer version?

Thanks, @kkkoech, for the newer version I am getting an error message "The application “main.app” can’t be opened. -47" when I am clicking on main.app.

@SebastianZimmeck , I think you're getting that because the app is from an "unidentified develeoper." On your mac, please go to "system preferences", then "security and privacy", and allow "main.app" to run on your computer. If this doesn't solve the problem, could you please attach a screenshot of the error?

Please see the screenshot attached (there is nothing additional under the question mark). The app is not asking for any permissions. I do not see anything in my "security and privacy" preferences. In the previous version I had indeed to give such confirmation, but not with this version.

Is the error possibly due to the app being packaged under a different OS version that I am currently using (macOS Catalina 10.15.4)?

@SebastianZimmeck, I'm the one who packaged all the versions and I'm running macOS Mojave. But if one version worked for you at some point, then this can't be the problem because all the version have been packaged with the same OS.

You could also run the app by cloning the repo and running : python main.py

But it is important to us that the standalone app works, so it would be great if we could fix this issue that you are having.

Please see the screenshot attached (there is nothing additional under the question mark). The app is not asking for any permissions. I do not see anything in my "security and privacy" preferences. In the previous version I had indeed to give such confirmation, but not with this version.

Is the error possibly due to the app being packaged under a different OS version that I am currently using (macOS Catalina 10.15.4)?

Also, when I test it (using the app itself or just by running main.py), all the files in testfiles/viruses are flagged as viruses. Hopefully this is the case for you as well but let us know if not!

@SebastianZimmeck This is probably not the issue but try right clicking on the app icon and then pressing "open"

But it is important to us that the standalone app works, so it would be great if we could fix this issue that you are having.

OK, I restarted my computer and ran the packaged app again. I received the warning below, allowed access, and it is working now. Not sure what the issue was.

Also, when I test it (using the app itself or just by running main.py), all the files in testfiles/viruses are flagged as viruses. Hopefully this is the case for you as well but let us know if not!

Can you point me to a concrete file that should be flagged as a virus. I tried for example, virus1.txt and eicar_test_file.txt and those were not flagged.

@SebastianZimmeck virus1.txt should not be flagged as a virus because we were using it for old tests but it is not longer included in the virus list. On my computer, all the eicar files are flagged as viruses (as they should be) so I'm not sure why eicar_test_file.txt was not flagged for you

actually virus1.txt is also flagged as a virus for me, we probably just left it in the virus list

But it is important to us that the standalone app works, so it would be great if we could fix this issue that you are having.

OK, I restarted my computer and ran the packaged app again. I received the warning below, allowed access, and it is working now. Not sure what the issue was.

Also, when I test it (using the app itself or just by running main.py), all the files in testfiles/viruses are flagged as viruses. Hopefully this is the case for you as well but let us know if not!

Can you point me to a concrete file that should be flagged as a virus. I tried for example, virus1.txt and eicar_test_file.txt and those were not flagged.

But it is important to us that the standalone app works, so it would be great if we could fix this issue that you are having.

OK, I restarted my computer and ran the packaged app again. I received the warning below, allowed access, and it is working now. Not sure what the issue was.

Also, when I test it (using the app itself or just by running main.py), all the files in testfiles/viruses are flagged as viruses. Hopefully this is the case for you as well but let us know if not!

Can you point me to a concrete file that should be flagged as a virus. I tried for example, virus1.txt and eicar_test_file.txt and those were not flagged.

yeah, like Sam said both of those files are flagged as viruses for me - as well as the two altered eicar files and virus3.txt. do you get the same results from running main.py ?

yeah, like Sam said both of those files are flagged as viruses for me - as well as the two altered eicar files and virus3.txt. do you get the same results from running main.py ?

The result is still the same. I am using python3 main.py instead of python main.py, though. The latter does not seem to work. With the former, I am getting the following.
There are a few "file not found" items, but that is really not the case. I am selecting the file, which exists, in the single file analysis dialog.
Also, I am on a Quad-Core MacBook Pro. Not sure if this is related to the problem.

Process Process-3:2:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Process Process-3:3:
Process Process-3:4:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Process Process-3:5:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Process Process-3:6:
Process Process-3:7:
Process Process-3:8:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Process Process-3:9:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'
Process Process-3:10:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/is_virus.py", line 131, in is_virus_v2
file_hex_str = get_hex_compressed(filefile, compress_bool=False)
File "/Users/szimmeck/Desktop/Code/Open-Source-Anti-Virus/helpers.py", line 44, in get_hex_compressed
file_bin = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'Users/szimmeck/Downloads/eicar_test_file.txt'


Don’t block API response with network call

The current response filtering approach waits for Trebble to respond (or fail) before allowing Spring to respond to the client. A more robust approach would be to save the Trebble request to a local bounded buffer, and upload to Trebble from the buffer asynchronously. This will reduce the latency impact to clients.

We have taken that into account - if you look at the implementation of the maskAndSendPayload in TreblleServiceImpl it is annotated with @Async (and the starter will also enable the required async configuration, see TreblleAutoConfiguration). By default it should use the applicationTaskExecutor (which can also be configured in terms of the number of threads).
I have specifically tested exactly this scenario while working on the SDK, but maybe I missed some edge case - can you give a short example when you see that the SDK is actually waiting for Trebble.


[feature] add test case filter for sg test

Relevant code is here https://github.com/ast-grep/ast-grep/blob/main/crates/cli/src/verify.rs#L114-L134
The idea is to support sg test -f test_case_glob to run only test that matches the name.
-f or --filter is used in mocha/ava to filter test cases using unix glob.

Any help or description?

Relevant code is here https://github.com/ast-grep/ast-grep/blob/main/crates/cli/src/verify.rs#L114-L134
The idea is to support sg test -f test_case_regex to run only test that matches the name.
-f or --filter is used in mocha/ava to filter test cases.

Hello! Just FYI, I saw this was not taken yet so I started working on implementing the feature, I will be pushing a draft PR soon. I hope that's OK with you!

Fine by me, have been a little busy somehow skipped this one completely


InfluxDBClient.get_list_series() was broken by InfluxDB v0.11.0 API change

The InfluxDBClient.get_list_series() method runs the "SHOW SERIES" query, but InfluxDB's response to that query has completely changed in v0.11.0 so the client can't parse the response.

This causes the test_get_list_series_and_delete to fail and, obviously, requires changes to get_list_series.
In particular data is now being returned in line format (cf. https://docs.influxdata.com/influxdb/latest/write_protocols/line/) and needs to be parsed.

Any updates on this issue, guys?

Closing stale issue, please let me know if still required and we can re-open to investigate.


After a user updates their email address, editing an application with the old email address might associate it with a new account

Describe the bug
After updating an email address in the user's profile, a subsequent edit to the application which does not update the email creates a new account.
To Reproduce

Submit an application
Log in as the user associated with the application
Update your email address in the user profile
Now edit the application without changing the email
The application will no longer appear in your account

Expected behavior
The application to remain tied to that account.
Priority

Low priority (annoying, would be nice to not see)

Affected roles

Applicants

Additional context
One possible solution would be to update the email on applications tied to the user when they update an email in their profile (or when an admin updates it for them).
Related to #2758.

If this is so it is clearly a bug, the submission should definitely follow the account.
When I test this I can not replicate it.
The function in question is:
https://github.com/HyphaApp/hypha/blob/6997240cebc84fe539dd3986d363aaad9ed3a77a/hypha/apply/funds/models/submissions.py#L526-L547
The issue we have had is when updating the e-mail address on an account the submission still show the old address until it is resaved. This has confused a very small number of users.

Ah, right. Okay. Not an upstream issue.


feat: publish page

Description
Publish page for multi bot.
Have done

bot status list
publish history
ux align with design

Need to refine or complete

code refactor
unit test

Remain bugs

multi publish targets should have different profile.
Successful multiple bot publishes are unstable.
publish time is too long.

Task Item
closes  #3963
Screenshots
I split into two parts for publish time too long.
publishing

published

Current PR implementation

@sangwoohaan designs

@alanlong9278 this is a snapshot from @sangwoohaan designs. There a few design changes this PR needs to make.

Column widths dont seem adjustable.
Column headers need to be bold. The ascending descending sort arrows are off.
The expanded section styles are not stylistically there yet.

Is there a reason why you are not using https://developer.microsoft.com/en-us/fluentui#/controls/web/detailslist. You are currently using table elements. DetailList is the component closest to matching @sangwoohaan designs.

This is the error message when publish failed. This message is wrong. Maybe "Publishing {botname} to {target} failed."
@sangwoohaan  need some copy help here

Aligning of text inside columns is off.  The error messages appear with the wrong font "is not a valid publishing target".
I have tried creating multiple publish targets. I haven't been able to publish a bot successfully even individually. Multiple Bot publishing simentaneously keeps failing instantly.

Current PR implementation

@sangwoohaan designs

@alanlong9278 this is a snapshot from @sangwoohaan designs. There a few design changes this PR needs to make.

Column widths dont seem adjustable.
Column headers need to be bold. The ascending descending sort arrows are off.
The expanded section styles are not stylistically there yet.

Is there a reason why you are not using https://developer.microsoft.com/en-us/fluentui#/controls/web/detailslist. You are currently using table elements. DetailList is the component closest to matching @sangwoohaan designs.

I dont use the detailsList because detailsList doesn't support circulation list, and for our need. the publishHistory list is a child list depends on the bot status list.

This is the error message when publish failed. This message is wrong. Maybe "Publishing {botname} to {target} failed."
@sangwoohaan  need some copy help here

Have change the failed message @srinaath  @sangwoohaan     Publishing {name} to {publishTarget} failed.

@srinaath are you convinced that DetailsList can't be used?

@hatpick I think we can still leverage the DetailsTree for the PublishPage.  Basically, we can do a nested details tree by overriding the renderRow method.
const onRenderRow = (props, defaultRender) => {
 return (
 <React.Fragment>
 {defaultRender(props)}

 nested element

 </React.Fragment>
 );
};

Me and @alanlong9278  spoke about this. He was going to try and let me know the results

@srinaath  @hatpick  I have refactored list display with <DetailsList>.

@luhan2017
The functionality works good now. Great work @alanlong9278 .

Lots of unit tests and some integration tests are breaking


set plunger to 'bottom' during pick-up-tip to avoid firmware bug

A user discovered a new bug, where given a special combination of coordinates, the AB motors squeal again during pipette._position_for_aspirate() where the plunger is set to bottom position.
This patch sets the plunger to bottom position during pipette.pick_up_tip() to avoid this moment all together. However, the long-term solution will be to switch to a newer build of smoothieware EDGE that fixes this bug.
This issue has not been reported by any other user.

Coverage increased (+0.001%) to 91.212% when pulling 69a2d1d15b1d23aeb97058fe067b3a9ae8163ceb on pick-up-tip-firmware-bug-patch into b26aaf9e9f8b48a03d974e747cfb3408a8b65dfc on master.


[Feature Request]: Add wildcard subdomain support for origin in the configuration

Description
Add wildcard subdomain support for origin in the configuration.

Done with #812


int() argument must be a string, a bytes-like object or a number, not 'Image'

follow "https://augmentor.readthedocs.io/en/master/userguide/extend.html#using-non-pil-image-objects" and face error above.

I have overcome this problem just now. This is because the parameter of function perform_operation "image" is a list().  add a for loop, it worked.

Yes indeed, this is not clear from that part of the documentation and is out of date. The perform_operation() function expects a list of images to be returned, even if only one image is returned.
Thanks for taking the time to clear this up.


[CAS] Don't create the same CAS multiple times from the same Oracle

If the oracle has been used to create a CAS for the path, reuse the CAS if it is asked to create the same CAS again. This not only save the work of creating the CAS, but also avoid a potential problem that the CAS closing synchronization doesn't work across threads in the same process due to its uses of file lock.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test


deploy on multiple vm or from remote vm

hi to all,
from the use of it(I may be wrong), the files must be on the client running the iis and the win_iis driver, is there a way to make the deployment from remote, I mean from any nomad agent machine, and have the file transported to a windows server, just like for docker?

@Vulfox Can't you use the Artifact stanza for this?

Yea, the artifact stanza should still be applicable here. The nomad config needs to be updated to look for where the exe gets installed at and then followed up with a restart of the nomad service to register those changes. template stanza may help here or running a raw_exec script to update and run the commands.


Compute distance sphere-box

We need this to use Nearest with Sphere. Right now we can only use Sphere with Intersect.

Retest this please

The CI fails because PGI times out on ArborX_DistributedTree_Test. I don't see how this can be related on the PR.


Eisenstein series are modular forms

We show Eisenstein series are modular forms.

[x] depends on: #12456

As for the TODO, I already have this, but its waiting on the q-expansion stuff thats coming. Should that still be a todo if its in the process of being PRd?

I would do so - the next PR can remove the TODO. If that other PR doesn't land soon (for whatever reason), the current file is in a more useful state.

bors r+


Component: Media

Support for images is done. Is in master branch - docs are updated.
TODO: support for videos

done, in master branch


Primary change: add dap4 support

Specific changes:

Add dap4 code: libdap4 and dap4_test.
Note that until the d4ts server problem is solved, dap4 is turned off.
Modify various files to support dap4 flags:
configure.ac, Makefile.am, CMakeLists.txt, etc.
Add nc_test/test_common.sh. This centralizes
the handling of the locations of various
things in the build tree: e.g. where is
ncgen.exe located. See nc_test/test_common.sh
for details.
Modify .sh files to use test_common.sh
Obsolete separate oc2 by moving it to be part of
netcdf-c. This means replacing code with netcdf-c
equivalents.
Add --with-testserver to configure.ac to allow
override of the servers to be used for --enable-dap-remote-tests.
There were multiple versions of nctypealignment code. Try to
centralize in libdispatch/doffset.c and include/ncoffsets.h
Add a unit test for the ncuri code because of its complexity.
Move the findserver code out of libdispatch and into
a separate, self contained program in ncdap_test and dap4_test.
Move the dispatch header files (nc{3,4}dispatch.h) to
.../include because they are now shared by modules.
Revamp the handling of TOPSRCDIR and TOPBUILDDIR for shell scripts.
Make use of MREMAP if available
Misc. minor changes e.g.

#include <config.h> -> #include "config.h"
Add some no-install headers to /include
extern -> EXTERNL and vice versa as needed
misc header cleanup
clean up checking for misc. unix vs microsoft functions

Change copyright decls in some files to point to LICENSE file.
Add notes to RELEASENOTES.md

Seeing a failure on OSX; in test_common.sh, uname -o fails.  -o isn't a legal argument to uname on OSX it seems.  I don't have access to a cygwin environment at the moment;  is the -o strictly required?

The -o is supposed to return the operating system name.
Is there an OSX equivalent?
On 3/30/2017 3:02 PM, Ward Fisher wrote:

Seeing a failure on OSX; in |test_common.sh|, |uname -o| fails. |-o|
isn't a legal argument to uname on OSX it seems. I don't have access to
a cygwin environment at the moment; is the |-o| strictly required?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-290544026,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P29u3tMoo1MCZ7mtMFmGtqyBlRmLDks5rrBhugaJpZM4MXiqT.

Ah; the equivalent on OSX is -s, or uname with no arguments.

Wht do you get under OSX if you use this command?
uname | cut -d '_'  -f 1
=Dennis
On 3/30/2017 3:02 PM, Ward Fisher wrote:

Seeing a failure on OSX; in |test_common.sh|, |uname -o| fails. |-o|
isn't a legal argument to uname on OSX it seems. I don't have access to
a cygwin environment at the moment; is the |-o| strictly required?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-290544026,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P29u3tMoo1MCZ7mtMFmGtqyBlRmLDks5rrBhugaJpZM4MXiqT.

:~$ uname | cut -d '_'  -f 1
Darwin
:~$

I'll merge this next ASAP so that we can quit worrying about merge failures.

4 of the tests failed; it wasn't across the board. I've restarted the failed ones to see if it was something on their side.  Travis is currently reporting 2 issues, although I haven't checked to see what they are.  If those failures persist I'll take a look at what those configurations are and see if there's an obvious reason for the tests to fail.

Checks have passed.  Running it through on OSX, ARM and Windows next.

Still seeing a failure due to a call to uname -o in tests_common.sh.  Did the fix for that get properly committed?

I believe it is fixed (modulo typos). The primary problems were as follows.

When running using visual studio under cygwin, it is possible to get visual-studio
code trying to open a cygwin path  (e.g. /cygdrive/...) and this causes an  error.
Fix is to add code tp to examine the incoming path for this situation and do a  conversion.
This was ocurring in libdap4/d4read.c, but may occur elsewhere.
Shell scripts in the git repo are not being  checked-out with the executable mode set. Had
core.filemode set to false.  Was a major hassle to fix.

Seeing some failures on Visual Studio where bash is available on the path; there are some problems with the shell scripts it looks like.  The results are viewable here:

http://cdash.unidata.ucar.edu/viewTest.php?onlyfailed&buildid=885

I will take a look at this and see what is going on.  I know when I was first porting netCDF to Visual Studio/Windows, I encountered a number of issues stemming from how paths are determined in cygwin/linux vs. path substitution under an msys shell.

On OSX with an autotools-based build, I'm seeing the following compilation failure:

libtool: link: gcc -dynamiclib -Wl,-undefined -Wl,dynamic_lookup -o .libs/libnetcdf.11.dylib .libs/libnetcdf_la-nc_initialize.o   -Wl,-force_load,../libdispatch/.libs/libnetcdf2.a -Wl,-force_load,../libdispatch/.libs/libdispatch.a -Wl,-force_load,../libsrc/.libs/libnetcdf3.a -Wl,-force_load,../libdap2/.libs/libdap2.a -Wl,-force_load,../oc2/.libs/liboc.a -Wl,-force_load,../libdap4/.libs/libdap4.a -Wl,-force_load,../libsrc4/.libs/libnetcdf4.a  -L/Users/wfisher/local-serial/lib /Users/wfisher/local-serial/lib/libhdf5_hl.dylib /Users/wfisher/local-serial/lib/libhdf5.dylib -ldl -lm -lz -lcurl    -install_name  /usr/local/lib/libnetcdf.11.dylib -compatibility_version 12 -current_version 12.4 -Wl,-single_module
duplicate symbol _nc_utf8proc_utf8class in:
../libdispatch/.libs/libdispatch.a(libdispatch_la-dutf8.o)
../libdispatch/.libs/libdispatch.a(libdispatch_la-utf8proc.o)
ld: 1 duplicate symbol for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I am not observing this on Linux, or cmake-based builds under OSX.  I tested with both gcc and clang on both platforms, in case it was compiler-specific optimization, but the results were the ame.

I have discovered that there is a problem when
using visual studio in a cygwin environment.
This is because you can sometimes pass /cygdrive/d/...
paths to e.g. open instead of d:/...
Could this be what you are seeing?
=Dennis
On 4/4/2017 3:06 PM, Ward Fisher wrote:

Seeing some failures on Visual Studio where bash is available on the
path; there are some problems with the shell scripts it looks like. The
results are viewable here:

http://cdash.unidata.ucar.edu/viewTest.php?onlyfailed&buildid=885

I will take a look at this and see what is going on. I know when I was
first porting netCDF to Visual Studio/Windows, I encountered a number of
issues stemming from how paths are determined in cygwin/linux vs. path
substitution under an msys shell.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-291631902,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P2zHQk_WLroUJq91OrZAi8aGI9BWaks5rsrDGgaJpZM4MXiqT.

RE OSX and utf8, what happened to the new utf8 code?

RE OSX and utf8, what happened to the new utf8 code?

Nothing that I'm aware of or that was explicitly done.  I assume it's still in there, although now I'm questioning that based on your comment.

Ok the utf8 issue on OSX isn't specific to this pull request it's in the master branch too. Fixing.

Got almost all of the tests working but there is still one failing, dap4_test/test_raw.sh.  Details here:

http://cdash.unidata.ucar.edu/testDetails.php?test=23046&build=950

The fix is not obvious to me and I'm uncertain how to address this. It works on other platforms.

can you give the full commit # for 8c48ac9
I cannot seem to find it.

Ok, the proximate problem for the test_raw case is that the path

[dap4]file:/g/netcdf-c/dap4_test/daptestfiles/test_anon_dim.2.syn
is apparently converted somewhere to this (judging by the error message):
[dap4]file;g:\netcdf-c\dap4_test\daptestfiles\test_anon_dim.2.syn
Note the change of 'file:/g/...' to 'file;g...' ; where is that semicolon coming from?
It must be the path change code I added in libdap4/d4read.c
It appears that the mingw compiler (and runtime?) is being used, correct?
But what is the Operating system: windows? Linux?

It is Windows but in an MSYS environment; I believe the issue is the same one I alluded to at our last meeting, i.e. path expansion by the underlying system. Cygwin does it one way, bash via MSYS does it a different way.  I was able to work around this though, but that was in my first 6 months so it has been a while and I don't recall exactly how.

I should be able to fix d4read.c but I need two
compile or runtime flags to test for:

running under windows os -- presumably __WIN32 will do this
compiled under mingw -- what flag is available for this?

Also, need the path form that mingw provides" It looks like
g:... is provided by mingw as /g/... correct?
=Dennis
On 4/7/2017 2:05 PM, Ward Fisher wrote:

It is Windows but in an MSYS environment; I believe the issue is the
same one I alluded to at our last meeting, i.e. path expansion by the
underlying system. Cygwin does it one way, bash via MSYS does it a
different way. I /was/ able to work around this though, but that was in
my first 6 months so it has been a while and I don't recall exactly how.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-292638591,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P26ORh1dlPsFdRZmrXhYQ-0IwCBcmks5rtpcMgaJpZM4MXiqT.

Dennis, It appears I left my external drive with my windows VM on it at home, so I will confirm my answers later and answer as best I can now.
The path expansion isn't mingw specific, but a side effect of the underlying environment, so the /g/ expansion we are seeing is a function of the MSYS environment.  That said, mingw is a fork of gcc so you could check for that perhaps?
The rules and examples of MSYS path expansion are outlined here:

http://www.mingw.org/wiki/Posix_path_conversion

Running under Windows can be checked with __WIN32, and Visual Studio specifically can be checked with _MSC_VER.

ok, As near as I can tell there is no MSYS equivalent to the CYGWIN
flag, so I guess I will have to assume that MINGW32 => MSYS

ok, the problem is how it treats file:/... urls. If the URL
is of the form file:///..., it should work ok. So initially,
I am going tp make that fix.

The changes you made to test_common.in are incorrect. It breaks the use of visual studio
as the compiler. You clearly have a situation in which VS (e.g. /Debug) should
not be part of the path. If so, then we need to test specifically for that case.
Can you characterize what case you have?

In what way is it broken? It works properly across my various visual studio installs. Appending '/Debug', '/Release',etc and updating cmakelists.txt to use these directories fixes test_common.in but breaks every other test script insofar as they look for '../ncdump/ncdump', not '../ncdump/[build type]/ncdump'.

That is why the macros ${NCDUMP}, ${NCGEN}, etc are defined
in test_common
so they isolate the test scripts. I thought I changed
all the test scripts to use e.g. ${NCDUM} instead of
e.g. ../ncdump/ncdump; did you undo those changes?
=Dennis
On 4/7/2017 5:04 PM, Ward Fisher wrote:

In what way is it broken? It works properly across my various visual
studio installs. Appending '/Debug', '/Release',etc and updating
cmakelists.txt to use these directories fixes test_common.in but breaks
every other test script insofar as they look for '../ncdump/ncdump', not
'../ncdump/[build type]/ncdump'.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-292672066,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P28FreqSNOf7U4etTTYKl8YXlAqpQks5rtsEWgaJpZM4MXiqT.

I did not. I'm still confused why test_common.sh is appending the build type to the path; the build type directory structure format has been suppressed for quite a while; since Russ was here at least. The reason I removed it  is that since the convention is unused in both Visual Studio builds and *nix builds, I wasn't sure what purpose it served.
Sent from my iPhone

On Apr 7, 2017, at 5:07 PM, DennisHeimbigner<EMAIL_ADDRESS>wrote:
That is why the macros ${NCDUMP}, ${NCGEN}, etc are defined
in test_common
so they isolate the test scripts. I thought I changed
all the test scripts to use e.g. ${NCDUM} instead of
e.g. ../ncdump/ncdump; did you undo those changes?
=Dennis
On 4/7/2017 5:04 PM, Ward Fisher wrote:

In what way is it broken? It works properly across my various visual
studio installs. Appending '/Debug', '/Release',etc and updating
cmakelists.txt to use these directories fixes test_common.in but breaks
every other test script insofar as they look for '../ncdump/ncdump', not
'../ncdump/[build type]/ncdump'.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-292672066,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P28FreqSNOf7U4etTTYKl8YXlAqpQks5rtsEWgaJpZM4MXiqT.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

My bad for not explaining.
When I use visual studio under cmake
in a cygwin environment, the e.g. /Debug
or /RelWithDgInfo directory is where compiled
executables end up. I know that it is unused
by cmake when using gcc (and I speculate mingw).
I infer from what you are saying that there is some
way to tell visual studio not to use the mode
as part of the path, correct?
=Dennis
On 4/7/2017 5:15 PM, Ward Fisher wrote:

I did not. I'm still confused why test_common.sh is appending the build
type to the path; the build type directory structure format has been
suppressed for quite a while; since Russ was here at least. The reason I
removed it is that since the convention is unused in both Visual Studio
builds and *nix builds, I wasn't sure what purpose it served.
Sent from my iPhone

On Apr 7, 2017, at 5:07 PM, DennisHeimbigner
<EMAIL_ADDRESS>wrote:
That is why the macros ${NCDUMP}, ${NCGEN}, etc are defined
in test_common
so they isolate the test scripts. I thought I changed
all the test scripts to use e.g. ${NCDUM} instead of
e.g. ../ncdump/ncdump; did you undo those changes?
=Dennis
On 4/7/2017 5:04 PM, Ward Fisher wrote:

In what way is it broken? It works properly across my various visual
studio installs. Appending '/Debug', '/Release',etc and updating
cmakelists.txt to use these directories fixes test_common.in but breaks
every other test script insofar as they look for '../ncdump/ncdump', not
'../ncdump/[build type]/ncdump'.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-292672066,
or mute the thread

https://github.com/notifications/unsubscribe-auth/AA3P28FreqSNOf7U4etTTYKl8YXlAqpQks5rtsEWgaJpZM4MXiqT.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-292673510,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P28NrgTXpDk8HoTNjrIZIshXyqeg6ks5rtsOsgaJpZM4MXiqT.

Ok, I did a general cleanup of test_common including
your changes. Hope this gets things back on track.

I ran with the debug output you incorporated; you can see it here:

http://cdash.unidata.ucar.edu/testDetails.php?test=25536&build=1063

I'll follow up, but does anything leap out at you?

The only thing I can figure is that the /g/... prefix is not
being interpreted as the equivalent of (I assume) g:/...
Does it fail in same way if you run
g:\netcdf-c\build\ncdump\ncdump.exe
'[dap4]file:///g/netcdf-c/dap4_test/daptestfiles/test_anon_dim.2.syn'
directly on the command line?
I wonder if changing the path to
'[dap4]file:///g:/netcdf-c/dap4_test/daptestfiles/test_anon_dim.2.syn'
would work?
=Dennis
On 4/10/2017 1:48 PM, Ward Fisher wrote:

I ran with the debug output you incorporated; you can see it here:

http://cdash.unidata.ucar.edu/testDetails.php?test=25536&build=1063

I'll follow up, but does anything leap out at you?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-293058716,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P22lL8SSlYerOPetLTLjN_3wffvETks5ruod_gaJpZM4MXiqT.

Error in readfile(), line ~168, failure at fd=open(filename,flags);.

filename	0x00000220948817a0 "/g/netcdf-c/dap4_test/daptestfiles/test_anon_dim.2.syn.dap"	char *

fd is -1.

The following works:

ncdump.exe [dap4]file://g:\netcdf-c\dap4_test\daptestfiles\test_anon_dim.2.syn

also,

ncdump.exe "[dap4]file://g:\netcdf-c\dap4_test\daptestfiles\test_anon_dim.2.syn"

works in the MSYS shell as well.
Is there a way to force the windows-style path, in the shell script? I'm having a bit of trouble unravelling the shell scripts at a glance and might not have time to dive into it tomorrow, but I can investigate this as well.

ok that is progress; Will it still work if those backslashes are
converted to forward slashes e.g.
ncdump.exe
'[dap4]file://g:/netcdf-c/dap4_test/daptestfiles/test_anon_dim.2.syn'
On 4/10/2017 9:31 PM, Ward Fisher wrote:

The following works:
ncdump.exe
[dap4]file://g:\netcdf-c\dap4_test\daptestfiles\test_anon_dim.2.syn

also,
ncdump.exe
"[dap4]file://g:\netcdf-c\dap4_test\daptestfiles\test_anon_dim.2.syn"

works in the MSYS shell as well.
Is there a way to force the windows-style path, in the shell script? I'm
having a bit of trouble unravelling the shell scripts at a glance and
might not have time to dive into it tomorrow, but I can investigate this
as well.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-293140013,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P26hv4KkHh8JssUHPT4pY5vC_7Pt8ks5ruvQegaJpZM4MXiqT.

ILooking at this page,
http://stackoverflow.com/questions/12015348/msys-path-conversion-or-cygpath-for-msys
it claims that MSYS2 has a version
of cygpath  that might resolve this problem.

It looks like cygpath is an msys2 thing, which I was not not aware of.  I'm installing now to give this a try.

Does that mean you have been using msys and not msys2?
=Dennis
On 4/13/2017 2:48 PM, Ward Fisher wrote:

It looks like cygpath is an msys2 thing, which I was not not aware of.
I'm installing now to give this a try.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-294017488,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P2_aN2z7NVWd4cQaSnOwe4zTIvtB_ks5rvooPgaJpZM4MXiqT.

Ok. Running from the msys2 shell causes more failures, but running the same tests from the same directory (using the code built with msys2) in msys1 doesn't fail.  I'll try to sort this out soon.  In regards to cygpath, however, it is very useful.  It generates a windows-style path which can then be passed and should work based on what we've already seen (I'll confirm that shortly).

ok. So I've confirmed that the following works in msys1 and I'll assume msys2 for now:

$ ../ncdump/ncdump [dap4]file://$(cygpath -w /g/netcdf-c/build/dap4_test/daptestfiles/test_vlen8.nc)

So how to codify this? Currently, the shell scripts only run if bash is available. If we can determine that the environment msys and cygpath exists (in addition to bash being available) then run the test, otherwise don't.
Running uname -a in msys1 and msys2 gives the following, respectively:

MINGW32_NT-6.2 LIVINGSTON 1.0.19(0.48/3/2) 2016-07-13 17:45 i686 Msys
MINGW64_NT-10.0 livingston 2.7.0(0.306/5/3) 2017-02-14 08:57 x86_64 Msys

So perhaps a conditional on Msys in uname?

ddap4 worked, is it all inclusive of dap4.dmh?

I think so; I can just mergeit into dap4.dmhif you want.
=Dennis
On 4/14/2017 3:12 PM, Ward Fisher wrote:

ddap4 worked, is it all inclusive of dap4.dmh?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-294237700,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P239DkWirCH-RPdNG0VMO1xq8vTddks5rv-EtgaJpZM4MXiqT.

Ok, I merged dap4.dmh into ddap4 so you can
either merge ddap4 back to dap4.dmh or just use ddap4
=Dennis
On 4/14/2017 3:12 PM, Ward Fisher wrote:

ddap4 worked, is it all inclusive of dap4.dmh?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-294237700,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P239DkWirCH-RPdNG0VMO1xq8vTddks5rv-EtgaJpZM4MXiqT.

Merged; congratulations Dennis, this is great to get this into netcdf-c!  I appreciate all your help getting things working.  I should have asked, does netCDF-Java support DAP4 yet?

Yes, the java side should be working. But more important
is that the c code can talk to the thredds server.
=Dennis
On 4/18/2017 5:20 PM, Ward Fisher wrote:

Merged; congratulations Dennis, this is great to get this into netcdf-c!
I appreciate all your help getting things working. I should have asked,
does netCDF-Java support DAP4 yet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/pull/374#issuecomment-295012720,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3P285KsLTcFB2bC_Z2ynU-H-papuSaks5rxUUqgaJpZM4MXiqT.


RDK-35544: HdmiInput thunder plugin changes for ALLM method/event

Reason for change: added code for getSupportedGameFeatures,
getHdmiALLMStatus
and the event HdmiALLMStatusUpdate
Test Procedure: available in Jira
Risks: Low
Signed-off-by: Dhivya Ilangovan<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Closing older PRs. Pls resubmit to latest 2205 sprint.


user-block when you chat-block an impteam 1on1 convo

chat-blocking and user-blocking are confusingly very different things, and we have at least one log of someone who thought they blocked but they only chat-blocked.
so, when a user chat-blocks (or reports) a 1-on-1 implicit conversation, also go ahead and user-block that person. this should only execute if it's two people in a non-team conversation, one of whom is the caller.
i didn't implement unblocking. i figured this was unlikely to be a heavily demanded thing, it adds a bunch of complexity, and you can always manually unblock the person if you go their profile.

@keybase/picnicsquad

@xgess in the ticket PICNIC-367 that I filed, I had blocked the user from their profile.  Does this PR still fix that case?  (From the description it seems like it's fixing something different.)

@strib the description of the ticket is fixed by https://github.com/keybase/keybase/pull/4337. this fixes the logdump that was linked. while not exactly a bug, this does fix something that's pretty unintuitive.

Ah makes sense, I missed that one.  Thanks!


Allow login while proxing a sub-domain

Fixes #2128.

tbh this feels a little jank to me but I'm not sure how else this could be implemented.

Pretty much all of this conflicts with the Express rewrite so I'm going to bundle it into that PR.


fix(grpc): use Numbers to represent long values

Configure gRPC to use Numbers to represent long values instead of Strings. This ensures that we can do accurate handling of numerical values.

Description:
This ensures that numbers are actually numbers and not strings so a check like if (val) { ... } would will give expected results.
For example:
if (0) { echo "something" }
   => undefined

but
if ("0") { echo "something" }
    => "something"

("0" is a truthy value, whilst 0 is not)
Motivation and Context:

Fix #727
Types of changes:

Bux fix
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage remained the same at 15.758% when pulling c12ca878af9bbcc2895792b2ab59e877262444f4 on mrfelton:fix/long-as-number into 2931effec4b07ecb4bbbd115cf28cd2572bc1810 on LN-Zap:master.

utACK 47b95a0


Dynamic Height of Slides ?

I'm having an issue with the height of the slides, I would like the Carousel to be flex: 1 and all the slides height be dynamic to the content. I.e. I have some slides with an image and some with text.
But I have found I cannot get the slides height to be greater than 277 ? but it can vary under 277, but it seems that is the max of the Carousel height.
I've tried putting flex: 1 on all the props I could find related to style lol
 renderCarouselRow = ({ item }) => (
        <View style={{ flex: 1, width: itemWidth, paddingHorizontal: 5 }}>
            <View style={{ flex: 1, backgroundColor: '#fff' }}>
                <ServerImage style={{flex: 1, resizeMode: 'cover'}} path={item.fullpath} />
                <Text>{item.title}</Text>
            </View>
        </View>
    );

<Carousel
                    data={this.state.list}
                    renderItem={this.renderCarouselRow}
                    sliderWidth={sliderWidth}
                    itemWidth={itemWidth}
                    snapOnAndroid={true}
                    showsHorizontalScrollIndicator={false}
                    inactiveSlideScale={1}
                    inactiveSlideOpacity={1}
                    containerCustomStyle={{ flex: 1 }}
                    contentContainerCustomStyle={{ flex: 1 }}
                    slideStyle={{ flex: 1 }}
                    style={{ flex: 1 }}
                />
            );

Hi @sean-macfarlane,
You shouldn't have any issue with dynamic height; I'm almost always using the carousel that way. I see two possibilities as to why it doesn't work for you:

your < Carousel /> is rendered inside a view with a constrained height
setting contentContainerCustomStyle to { flex: 1 } can mess with slides' layout. By the way, your style prop won't have any effect.

Here is a setup I've used for flexible slides:
const styles = StyleSheet.create({
    flexContainer: {
        flex: 1
    }
});

export default class MyComponent extends PureComponent {

    _renderItem ({item, index}, parallaxProps) {
        return (
            <TouchableOpacity
              key={`carousel-entry-${index}`}
              style={styles.flexContainer}
              onPress={() => this._onItemPress(index)}
            >
                <ParallaxImage
                  source={item.source}
                  style={styles.flexContainer}
                  {...parallaxProps}
                />
            </TouchableOpacity>
        );
    }

    render () {
        return (
            <View style={styles.flexContainer}>
                <Carousel
                  ref={(c) => { this._carousel = c; }}
                  data={this.state.entries}
                  renderItem={this._renderItem}
                  sliderWidth={SLIDER_WIDTH}
                  itemWidth={ITEM_WIDTH}
                  containerCustomStyle={styles.flexContainer}
                  slideStyle={styles.slide}
                  apparitionDelay={300}
                  hasParallaxImages={true}
                />
                <Pagination
                  dotsLength={this.state.entries.length}
                  activeDotIndex={this.state.currentSlideIndex}
                  containerStyle={styles.sliderPaginationContainer}
                  dotStyle={styles.sliderPaginationDot}
                  inactiveDotOpacity={0.45}
                  inactiveDotScale={0.55}
                />
            </View>
        );
    }
}

@bd-arc this doesn't seem to work.  While the slide itself becomes full height, the content inside the style is not.  For example, changing your _renderItem to:
_renderItem ({item, index}, parallaxProps) {
        return (
            <View style={{ flex: 1, backgroundColor: "blue" }}>
                <Text>Hello world</Text>
            </View>
        );
    }

The view does not flex the entire height - instead it is only high enough to fit the text.