Demangling of Microsoft-specific symbols doesn't work

Unless I'm missing some setting, the demangling (un-decoration) of the latest Microsoft-specific symbols doesn't seem to work.
I'm testing it with Ghidra v.9.0.2
Example: While de-compiling a function from cmctrl32.dll (x86, v.6.10.17134.286 on my Windows 10) almost all Microsoft symbols are not demangled:

I'm not sure if the current implementation uses UnDecorateSymbolName function or not. If so, it is old and doesn't work well with most modern mangling schemes that Microsoft employs in Visual Studio compiler.
I did this research for my own tool that seems to demangle such symbols without a problem. So let me share it. Here's a screenshot of the same symbol that Ghidra failed to demangle:

So it would be nice if that is corrected as copying and pasting those names back and forth gets quite tedious.

My solution
Use undocumented but exportable function from vcruntime140.dll called __unDNameEx. (It beats me why Microsoft doesn't want to document it.)
I am not good at Java, so instead let me show the C++ implementation (so hopefully someone can translate it into Java).
To make it safer (while dealing with "undocumented" function) I would statically link the module that uses it to VC RT v.14. After that you can declare that function as a static import as such:
//Import this undocumented function from the vcruntime140.dll (if we're statically linking to it)
extern "C" PSTR __cdecl __unDNameEx(
	PSTR buffer,
	PCSTR mangled,
	DWORD cb,
	void* (__cdecl* memget)(DWORD),
	void (__cdecl* memfree)(void*),
	PSTR (__cdecl* GetParameter)(long i),
	DWORD flags
);

Otherwise, if static linking is not possible, you will have to load it at runtime:
typedef PSTR __cdecl UNDEC_NAME_EX(
	PSTR buffer,
	PCSTR mangled,
	DWORD cb,
	void* (__cdecl* memget)(DWORD),
	void (__cdecl* memfree)(void*),
	PSTR (__cdecl* GetParameter)(long i),
	DWORD flags
);

HMODULE hModVcRT140 = ::LoadLibrary(L"vcruntime140.dll");
UNDEC_NAME_EX *__unDNameEx = (UNDEC_NAME_EX*)::GetProcAddress(hModVcRT140, "__unDNameEx");

if(__unDNameEx)
{
	//Do work here ....

}

Obviously remember to free the module when done:
::FreeLibrary(hModVcRT140);

Lastly use that function to demangle Microsoft-specific symbols, as such:
//IMPORTANT: This function is not multi-threading safe. If calling it from
//           multiple threads, do so from under a critical section.

const static char p_strNameToUndecorate[] = "?_SBGetParms@@YGHPAUHWND__@@HPBUtagSBDATA@@PAUtagSCROLLINFO@@@Z";

//Flags for conversion (see documented UnDecorateSymbolName() function for details)
//eg: UNDNAME_COMPLETE or UNDNAME_NO_MS_KEYWORDS, etc.
DWORD dwFlags = UNDNAME_COMPLETE;

//Undecorate now (let it reserve its own memory by passing NULL as a first parameter)
char* pUndecorated = __unDNameEx(NULL, p_strNameToUndecorate, 0, _funcAlloc, _funcFree, _funcGetParameter, dwFlags);
if(pUndecorated)
{
	//All good
	printf("Result: %s\n", pUndecorated);

	//Free mem
	delete[] pUndecorated;
	pUndecorated = NULL;
}
else
{
	//Failed
	printf("FAILED\n");
}

This function doesn't come in the wide-string (or Unicode-16) flavor so you'll have to convert such strings to ANSI first.
Lastly, use these callbacks for memory management inside that function:
static void* __cdecl _funcAlloc(ULONG cb)
{
	//Reserve mem for 'cb' chars
	return new (std::nothrow) char[cb];
}

static void __cdecl _funcFree(void* p)
{
	//Free memory in 'p'
	if(p)
		delete[] p;
}

static PSTR __cdecl _funcGetParameter(long i)
{
	static char dummy[] = "";
	return dummy;
}

If you delete the function, then place your cursor on the mangled label, and run the script DemangleSymbolScript, what happens?

@emteere in that case it seems to demangle it correctly for one function I tried it with. Then the issue is how do you incorporate it into the static analysis to do it automatically on all symbols? (Again, am I missing the setting?)

You can use the DemangleAllScript, or you can re-analyze the program.
There is already an automated Demangler analyzer, however I suspect your symbols were loaded from a .PDB file after the fact.  If the symbols were applied by loadding the .PDB, the demangler may not have been kicked off again, although I think it should have been kicked off.  I'd need to know more about exactly how you loaded the binary and when you applied the .PDB symbols.
If you were to re-import the binary with the .PDB already downloaded and ready to apply, then I believe it would have demangled automatically.

The msdia.dll is used to load the PDB.  Ghidra has it's own internal demangler.  We try not to rely on software that is only available on one platform.

@emteere yeah, I admit I wasn't sure how to load the symbols. It's somewhat counter-intuitive the way it does it. I think I went to File -> Download PDB file and downloaded it from the Microsoft symbol server. After that it offered to apply it to my project, which I did.
So instead of me guessing, what is the correct sequence of applying PDB symbols from an online symbol server?
And also how can I apply them to a file I've been already working on? I'd hate to lose several days of RE work on it.
Thanks.
PS. As for having its own internal demangler, it may be both good and bad. As you probably know Microsoft doesn't really stick to standards. In my own project that I listed above I had to literally RE how they do it in their debuggers because their documented demangling API wasn't doing a good job. So after I find a way to load all the symbols automatically in Ghidra I can report back to you about the results. So maybe your own internal demangler does a good job. At this point I don't know.

http://msdl.microsoft.com/download/symbols/ucrtbased.dll/C09C8E211C2000/ucrtbased.dll contains quite a few symbols that don't demangle correctly. Conveniently, it also contains the correct logic for demangling them.

@Zhentar possibly. What I provided in my original post is the source code to do it in general. (Obviously for a Windows platform.) If you want to test it in action here's the link to the tool in my original screenshot that implements that __unDNameEx call from the VC run time library. Go to View -> Undecorate C++ Symbols. That API is totally undocumented, but it seems to demangle most of the latest Microsoft symbols.


WFLY-7115 KeyAffinityService blocks Infinispan's topology change thread

https://issues.redhat.com/browse/WFLY-7115
Replaces Infinispan's default KeyAffinityService with a custom implementation without the problem described in WFLY-7715.
Also encapsulates all key distribution logic into a single object to ensure agreement between disparate parties.

@rhusar @rachmatowicz  Please review.


Draft: Author Profiles for review by designers

fix: editors can specify author by URL or by name
fix: horizontal and vertical author profiles supported
fix: lists of 2 and 3 enforce a grid/row of 2 (or 3, respectively)
fix: vertical and list-of-1 show using vertical orientation
Fix #148
Test URLs:

Before: https://main--hlx-test--urfuwo.hlx.page/draft/hupe/author-profiles-validation
After: https://148-l2-author-profile-card--hlx-test--urfuwo.hlx.page/draft/hupe/author-profiles-validation

Discussion: https://adobe-dx-support.slack.com/archives/C06H8UNQ3RS/p1709641879813599

[ ] New Blocks introduced in this PR

Block Name
Documentation
Library Link

author-profiles
doc-link
Library Link

I think the current approch is fine in general. Listing the authors like done on the sample page is fine. Metadata authors should be reserved for the authors of the actual page. In this case I would keep the author links in the block, but with a few changes:

rename the block - to me these are just people profiles, could be authors could be other persons and IMHO it should not be the same block as we use on the L3 pages.
simplify the block structure, we have links & names at the moment? Why? Could be as follows: (if you queryselect the a tag it works with both plain links or list with links.

+-------------------------------------------------------------------+
| Person Profiles                                                   |
+-------------------------------------------------------------------+
| * <https://main--hlx-test--urfuwo.hlx.page/author/thomassaueressig> |
| * <https://main--hlx-test--urfuwo.hlx.page/author/aarongreen>       |
| * <https://main--hlx-test--urfuwo.hlx.page/author/jacquelineprause> |
+-------------------------------------------------------------------+

or
+-------------------------------------------------------------------+
| Person Profiles                                                   |
+-------------------------------------------------------------------+
| <https://main--hlx-test--urfuwo.hlx.page/author/thomassaueressig> |
| <https://main--hlx-test--urfuwo.hlx.page/author/aarongreen>       |
| <https://main--hlx-test--urfuwo.hlx.page/author/jacquelineprause> |
+-------------------------------------------------------------------+

@mhaack

I don't think we'll have non-author persons under /author
I don't think we can change that URL
I think "Author Profile" is what it's called in Figma + Frontify, so I think it's not a bad name

FYI @pahupe

Agreed on 1. & 3. 2. I don't understand, I'm not suggesting to change any url?

Author Profile is fine for me. But it is the same block as we have on the L3 pages. This will give one block two somewhat different functions and behaviors. I'm not a fan off overloading block with different use cases.

On links vs. name. @pahupe IMHO in the doc we should prefer the links and support links only. If all authors have an author page link is not an issue, it is consitent and less complex code.

@mhaack , @benpeter : Ok, I'll change the logic to support links only

Update:

grid is out (again) @mhaack
description text color has been changed to Grey/7
breakpoint M: changed 4 elements per row to 2 elements
2 elements only: show elements using the design 22A
Ref: https://adobe-dx-support.slack.com/archives/C06H8UNQ3RS/p1709652867817059?thread_ts=1709641879.813599&cid=C06H8UNQ3RS

Please review @mhaack , @saurabh-khare , @benpeter , @rootpea , @karlpauls
All further changes should be done in a separate GH ticket


Fix 4796: Prepend global:: to references of generated C# types

This should fix #4796 by prepending global:: to references of generated C# types.
@deinok: Can you confirm if using a kiota build of this branch to generate your clients resolves the issue? I did some tests with samples that I had at hand and it seems to work, but having confirmation from a real-world example would be nice.
cc @baywet

@deinok this is because of #4823 Just update your abstractions reference

Hmm, I see.
At first sight it looks like it could be unrelated. Could you try using a build that's based on the current main branch and see if the problem occurs as well?
Also could you share the OpenAPI documents you're using, or are those internal ones?
I can have a look tomorrow morning if the issue was introduced by the PR - for today we probably are both watching Spain-Italy 😄

Hmm, I see. At first sight it looks like it could be unrelated. Could you try using a build that's based on the current main branch and see if the problem occurs as well?
Also could you share the OpenAPI documents you're using, or are those internal ones? I can have a look tomorrow morning if the issue was introduced by the PR - for today we probably are both watching Spain-Italy 😄
EDIT: Alright, seems like this is unrelated as per @baywet s comment

Yup, unrelated, its the Remove LINQ change.
Regarding my OpenAPI file, its not the issue as it is. Probably its because i have a layout like this:

So they clash a bit internaly, thats the reason i need the global.
Regarding Spain-Italy, try not to drink too much and have fun 😄

@baywet I added a changelog entry and a dedicated test for the global:: prefix to make sure this does not see any regression.
@deinok Well, Spain would have deserved more goals. 😄
I don't really have more time to create a unit test for that scenario. As far as I can tell, it would become quite complex, as the test statement would boil down to "does it compile?" after the generation run(s).


Design Flaw Enhancement

How hard would it be to add functionality to this honeypot similar to what is talked about here:
https://www.bleepingcomputer.com/news/security/mysql-design-flaw-allows-malicious-servers-to-steal-files-from-clients/
Discuss?

I am not sure if it is possible here… As far as I understand, a malicious MySQL server asks a client to transfer it a local file. This honeypot aims at catching clients brute forcing passwords.
In order to detect a malicious server, you probably need a scanner, not a honeypot.


Arctan2 instead of arctan for lon and stable h conditions

Hopefully could solve #1290. Thanks!
From the Bowring paper:

I realized this still doesn't solve the problem for latitude = pi/2. Would work on it!

Windows failure is again unrelated.


Better communicate the need to stay in the WalletConnect Safe app

What is this feature about? (1 sentence)
This will add more info text for the user to understand that they need to be in the WalletConnect app in order for dapp txs to show up.
Why is it needed? What is the value? For whom do we build it?

This is connected to our problem area "Inconvenient way to interact with standalone Dapps" on our roadmap.
Currently, the WalletConnect app needs to be open in order for WalletConnect dapp txs to show up.
We mention this in the wording, but it's not super apparent.

High-level overview of the feature
details to tbd
Option 1:

Add in bold to point 2: "You need to have this WalletConnect Safe app open for transactions to pop up. You will not receive transaction requests when you don't have it open."

Option 2:

Have a popup that the user needs to click away for the first time with the info above.

@posthnikova could you please make suggestion for this when you have time?

@posthnikova could you please make suggestion for this when you have time?

I was thinking about a banner like 'Don't leave the page' which we have on Safe creation.

Or maybe some inline banner or even a warning:

Of course we can just change the text but it's not so prominent.

Another solution could be a warning like on websites when you have unsaved changes and try to leave the page.

I was thinking about a banner like 'Don't leave the page' which we have on Safe creation.

Or maybe some inline banner or even a warning:

Of course we can just change the text but it's not so prominent.

Another solution could be a warning like on websites when you have unsaved changes and try to leave the page.

I think the last proposal will be hard to imement (https://stackoverflow.com/questions/49445671/how-to-trigger-beforeunload-event-from-within-an-iframe)

I think the last proposal will be hard to imement (https://stackoverflow.com/questions/49445671/how-to-trigger-beforeunload-event-from-within-an-iframe)

Thanks for the alternatives!
My personal opinion is to go with the first one since the message there is super prominent (which it should be). And it's more prominent than the other ones.
Interesting thought with that last one. I feel it's a bit too intrusive for user who know what they are doing.

Thanks for the alternatives!
My personal opinion is to go with the first one since the message there is super prominent (which it should be). And it's more prominent than the other ones.
Interesting thought with that last one. I feel it's a bit too intrusive for user who know what they are doing.

I like the first option too.
Zeplin updated: https://zpl.io/V1XlkwJ

I like the first option too.
Zeplin updated: https://zpl.io/V1XlkwJ

How would the first one look with the mendatory padding that we put in for safe apps?

How would the first one look with the mendatory padding that we put in for safe apps?

@rmeissner This green rectangle will be inside the padding.

@rmeissner This green rectangle will be inside the padding.


Request for a detailed repo structure

The suggestion is to have a more detailed folder structure or attributes for the maps here.
Suggested attributes:
Admin level, name of largest admin level, year of data, boundary code.

Admin level is 0, 1, 2, 3, 4, 5, 6...(0 country, 1 state, 2, district, 3 taluka, 4 village, 5 ward, 6+ assembly/parliamentary constituencies etc)
Name of largest admin level: If it is just a state or  district map, then we would need to know the name of that
year of data- self explanatory
boundary code- which boundary does it follow  (explanation below)

Reasons:

There are tons of requests for boundaries of different admin levels all the time in Datameet google group, and the data gets unorganized
-Ward Boundaries may get updated during non census years. Ahmedabad itself has had 2 updates since last census- from 64 wards to 57 to 48 wards.
District boundaries get updated during non census years
https://blog.socialcops.com/intelligence/data-stories/boundary-changes-districts-in-india/
According to this, 44 new districts have been created since the last census (2011) for 12 different states.
-Boundaries are not the same in all maps

Example:
Recently, I needed a district level map of Gujarat for a publication. Gujarat's districts had been updated in 2013, but the ones here showed 2011. I found some district maps from other sources, but they were wrong. Finally, someone had nearly accurate maps on the group, and I used them for this.
This is why I believe it is important to have state and district level maps organised by district/state as well as year.
Similarly, if we know which boundary a state or district map follows (GADM, OSM, Natural earth etc)  then comparing data between two geographical entities would be easier. This is more for the visualization purpose
For example, if I want to compare 3 states on district wise population density and show that on a larger map of India, I need to make sure that all 3 district maps follow the same boundary/map source.

Some ideas:

Having a circa year (like: this is the boundary as of..) in the filename or nearby may help. For Pune wards I'm keeping years (pre-2012, 2012-17, 2017-onwards) in the filenames.
I think the folders, files are organised enough right now. I wouldn't want to put just 0,1,2 etc and require people to look up to know what's district, what's state etc. And we don't have some major programming requirements either to put a 0 / 1 / 2 etc in the main names.
Village boundaries might be better off in their own repo's.
Sub-dividing into multiple repo's along admin levels or regions may be one expansion strategy as this grows, as typically someone wanting, say, districts may not want to download another few 100 mb's of other boundaries.

Annnnd:

I think if we want it badly enough then let's just fork, re-organise, pull-request or just post back here, help do instead of asking/demanding to do.


Sweet alert fix - NO by default, outside click, or esc

About this change - What it does
In every dialog for YES/NO, clicking outside the dialog or ESC will be considered as NO
Ex : when approving a topic request, a dialog is popped up with yes/no options, and no is considered when esc or clicked outside
Resolves: #xxxxx
Why this way

I have tested some of these already and they look good, I am just working on making my way through all of the different views

The only slight issue I found is that when you try to switch teams and click outside of the box that it doesnt switch as expected but the name of the new team is still in the box making it look like you have, so if someone selected outside of the box accidentally they might think that it was actually working.

The only slight issue I found is that when you try to switch teams and click outside of the box that it doesnt switch as expected but the name of the new team is still in the box making it look like you have, so if someone selected outside of the box accidentally they might think that it was actually working.

Fixed. Now after selecting a different team, and clicking NO or outside, the team is reverted.


EH pylint next

should be used jointly with typing to make sure docstring types are correct -- some instances of wrong doc descriptions too
This is only a first pass-over of documentation, we will need to update docstrings again and add more in as we get pyamqp closer to GA

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-eventhub

Overall looking good. There are some things like incoming_header that will be fixed up during mypy


Return gracefully when a node hasn't been found.

When initializing my d3 graphics, including the d3-tip function, (getSVGNode) cannot find a proper node yet, and el.tagName generates an exception. Therefore I've added a simple check to see if the svg node isn't null.
Hope you will accept it...

:+1:  on this


stable/sonarqube crash loop

Describe the bug
A clear and concise description of what the bug is.
-After performing the helm install the Sonarqube container enters a CrashLoopBackOff state in a minikube cluster.
Version of Helm and Kubernetes:
-helm 2.13.1
-kubectl 1.14.2
-minikube v1.1.0
Which chart:
-stable/sonarqube
What happened:
-Install completes without error, Sonarqube container enters a CrashLoopBackOff state.
-kubectl logs  returning:
./bin/run.sh: line 24: exec: java: not found
What you expected to happen:
-Sonarqube container to enter a "running state".
How to reproduce it (as minimally and precisely as possible):
-Not doing anything special here, just helm install stable/sonarqube
Anything else we need to know:
-I am assuming the issue is with helm, since I can pull the Sonarqube docker image and it doesn't appear to have the same issue when ran without the chart:
sonarqube$ echo $JAVA_HOME
/usr/local/openjdk-8

Hi,
I have the same problem here, stack:
Helm: v2.13.1
Kubernetes (GKE): 1.11.8-gke.6
Kubectl: v1.14.1
kubectl logs sonarqube-sonarqube-5cf99d8c5-tbjwn -p
./bin/run.sh: line 24: exec: java: not found

The last time I displayed the sonarqube chart and it worked was on May 22nd. If it was helpful.

Got here with same bug.  Seems that in the underlying image, the environment var PATH is prepended with /usr/local/openjdk-8/bin..   (which looks like a workaround to compensate for an incomplete java install that didn't set up 'alternatives' properly..).  Yet In the k8s deployment, /usr/local is re-mounted for no other reason than to home the 'copy_plugins.sh' script which is delivered via a configMap.  Hence the java install has been effectively hidden from view.   How does this chart get classed as 'stable'?

I was able to get a working setup using the 7.6-community image in the values file. More recent images don't appear to work with the current helm chart still.

Experiencing the same issue.

Same problem for both 7.7 and 7.8 ... but we had been running 7.7-community for a month and when the cluster restarted the pod, it failed.  I'm assuming that indicates a problem with the underlying docker image (which did change underneath the Helm chart).

Tested today against 7.8 and 7.9: same issue

Here's some further activity ... this was still a problem last time we checked.

Still a problem for us

I'll check our most recent installation

WHat is the result of kubectl describe on the sonarqube pod? I think I might be seeing the same issue and it looks like the healthcheck is wrong. Happy to investigate more if folks are seeing similar issues:
 Warning  Unhealthy  9m7s  kubelet, docker-desktop  Liveness probe failed: Get http://<IP_ADDRESS>:9000/sessions/new: net/http: request canceled (Client.Timeout exceeded while awaiting headers)

it's 2021, the same problem exists.


[hotfix][ConnectorV2][Neo4j] fix check config logic

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

@getChan @EricJoy2048 PTAL

@FWLamb I think it is better to repair this at #3565. It is unnecessary to resubmit a PR

@FWLamb I think it is better to repair this at #3565. It is unnecessary to resubmit a PR

all right


Geospatial Commands Queries

circle
distance
fill
geojson
to_geojson
get_intersecting
get_nearest
includes
intersects
line
point
polygon
polygon_sub
These are all brand new. Pseudotypes will be necessary.
Usual docs/tests/specs.

Done!


mpma memo support

Need to update MPMA sending to create array of memos and pass memo and memo_is_hex lists
https://github.com/CounterpartyXCP/counterparty-lib/issues/1065

Fixed with commit e5dc6998ba48b39ca41691e49cff416652cfc4e8


set style as a global attribute

Not sure if there was a reason this wasn't a global attribute. Handy to have!

there are a few others too… I'll make a PR on @rgbkrk's branch rather than as a separate PR on this.

Thanks.


Ability to add key:value pairs to existing and new hashes

There appears to be support for viewing/editing key:value pairs inside an existing hash but you can not create new key:value pairs (either for existing hashes or new hashes) w/o going down to the console and using hmset.

Is this feature going to be considered? It is crucial for us.

The feature requested is available with latest version.
If there is something missing at the implementation feel free to reopen issue with an updated description.


Question: How to Activate Quick Tabs from Keyboard

My apologies if this is addressed in the docs, I didn't see it.
Is there a way to activate QuickTabs from the keyboard so I don't have to use the mouse to click the Chrome toolbar shortcut?
Thanks,

Niz

Hey @nizdobs, I don't know which version of Chrome implemented this but if you have latest version of Chrome then it should give you options to create your own keyboboard shortcuts under "Extension" tab (at the bottom). This extension used to have default keyboard shortcut but got removed because of that option provided by Chrome leaving up to the user to create their own shortcut instead of clashing with others (atleast that's what I think was the intention).

This worked perfectly, thank you.

Thanks @sinkingshriek =)


hidden-* class removed?

I recently downloaded the bootstrap 4 compiled css files, and noticed that all classes for "hidden" (hidden-sm-down, hidden-sm-up, etc...) are not included within. Was there a decision to remove the class, or was their inclusion simply neglected? I double checked to see if they were there, so I'm hoping the fault isn't on me, the user.
link used to download files (top-most download link):
https://getbootstrap.com/docs/4.0/getting-started/download/

see the Migration

Closed thanks to @fchiumeo 👍

To replace
hidden-sm-down
you need to use
d-none d-sm-block
I find this new method very regrettable because to replace hidden-*-down element, what is nevertheless the most common, it requires to know the default display of the element which is much less practical than before ...
Moreover the name is confusing because d-*-none has the impression that it hides an element only for a single breakpoint and not the upper also breakpoints.


⚠ Bofrost.de has degraded performance

In 915034e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1631 ms

Resolved: Bofrost.de performance has improved in 6ec6f86.


It looks like you are using a string compare for the Homebridge version. This fails on higher version custom Homebridge spins.

Issue
I am getting the Homebridge version not satisfied error.
We are using a custom spin of Homebridge. With a version 3.0.2. This is higher than 0.4.50.
If you compare using strings "0.4.50" is greater than "3.0.2", which is false

I am getting the Homebridge version not satisfied error.

That error is issued by homebridge itself, not by homebridge-hue.  Homebridge-hue does semver checking and issues a warning in case of mismatch.

We are using a custom spin of Homebridge.

Not something that I would recommend or support.


Add post 'Get started with JSON Schema in Node.js'

I haven't rewritten the introduction as I think it's still appropriate in this context.

It would be great if you could mention, at least at the end, which version of AJV you're using here.

No problem.

I'd also like to just see a touch on what AJVs safe mode is, and how and why you would want to disable it.
Not to say that AJVs safe mode doesn't provide some good defaults, such as https://ajv.js.org/strict-mode.html#unconstrained-tuples, but it can easily prevent valid schemas from being allowed to run. It's a "gotcha" we are sometimes seeing.

I'm happy to add a paragraph or two on Ajv's strict mode. I haven't run into any issues myself though, so haven't had the need to adjust the strict defaults — could you give me an example of a valid schema that it would prevent from running?

I'd also like to change the header image. I really like https://unsplash.com/photos/a8YV2C3yBMk I think it's good symbolism of getting started, not too difficult, but lots of potential. (Happy to make that change if you've enabled edit by maintainers on your branch)

Go for it! (edits by maintainers are enabled on this branch)


Issue with hot code reload: clearing db on mirror-server refresh

I've been having issues lately with the same scenario reloading while making edits, and the results of the tests being inconsistently different.
All tests in a scenario will pass, then I'll make an edit to a test further down in the bottom of the scenario, and when the server refreshes itself, a different test up the chain in the beginning of the scenario will start to fail.
Or there will be a step that doesn't pass:
this.When(/^I click on "([^"]*)"$/, function (selector, callback) {
      this.browser
        .waitForExist(selector, 3000)
        .isExisting(selector, function (err, isExisting) {
          assert(isExisting);
        })
        .waitForVisible(selector, 3000)
        .isVisible(selector, function (err, isVisible) {
          assert(isVisible);
        })
        .click(selector)
        .call(callback);
    });

If more time is added to the .waitFor tests and I save the step file, it will refresh the server and this will pass.
Then I'll do a full restart of the mirror-server, running CUCUMBER_TAGS='@randomscenario' meteor again with the added .waitFor time and it will fail at the .isExisting assertion.

hmm, I can't quite track what's happening there. Could you recreate the error in n isolated repo? That exercise will either show an error in the implementation or a bug in the framework. Both ways will lead to a fix!

Okay here's the repo: https://github.com/jwjames/cucumber-issue
Most of the time it works perfectly.
Sometimes I'll get this on startup:
@dev
  Scenario: Creating new todo items                       # features/sample.feature:12
    Given I have landed on the website for the first time # features/sample.feature:4
    And I navigate to "/join"                             # features/sample.feature:5
    And I type<EMAIL_ADDRESS>in the email input         # features/sample.feature:6
      AssertionError: false == true
        at WebdriverIO.<anonymous> (/Users/jeffjames/Projects/todos/tests/cucumber/features/step_definitions/sample_steps.js:70:11)

Or when I change something minor in the file like:
@dev
  Scenario: Creating new todo items
    When I see ".btns-group-vertical"
    When I see ".js-user-menu"

to
@dev
  Scenario: Creating new todo items
    When I see ".btns-group-vertical"
    And I see ".js-user-menu"

or vice versa, I'll get the error above. Sometimes it shows up when I restart the server.

I shortened the waitFor times from 3000ms to 1000ms and it's happening a lot more frequently now. Pushed to the repo.

Perfect. Thanks for this, I'll investigate and get back to you

the issue looks looks like it was due to multiple chimp servers starting.
Can you try again with v0.11.0 - I scenarios to work at all locally


Getting disconnected from a server doesn't close the entity spawn menu. (And I assume other menus)

Yeah. This didn't close any of the menus I had open. (instrument, spawn menu, etc) Also kept hearing my midi track play flawlessly.

Very closely related to #616 and #399, i recommend closing this in favor of those


auth/simple_token.go token not removed when etcdctl session closes

The Token generated for a user during Authentication is being cached in auth/authStore
All tokens for a user are being removed when a user is deleted, or the user changes password, but not removed when the user session closes, or the same user gets authenticated again.
ideally the token should be removed when the authenticated session is over. at least it should be removed when the same user gets a new Token.
currently, if etcdctl is used for many operations with auth enabled, the map size keeps on growing.
46 func (as *authStore) assignSimpleTokenToUser(username, token string) {
47     as.simpleTokensMu.Lock()
48
49     _, ok := as.simpleTokens[token]
50     if ok {
51         plog.Panicf("token %s is alredy used", token)
52     }
53
54     as.simpleTokens[token] = username
55     plog.Errorf("added user %s , total = %d", username, len(as.simpleTokens))
56     as.simpleTokensMu.Unlock()
57 }

The above modified code prints the following log.
17:35:39 etcd1 | 2016-09-29 17:35:39.365732 E | auth: added user root , total = 1
17:35:39 etcd2 | 2016-09-29 17:35:39.377584 E | auth: added user root , total = 1
17:35:39 etcd3 | 2016-09-29 17:35:39.379829 E | auth: added user root , total = 1
17:35:52 etcd1 | 2016-09-29 17:35:52.018620 E | auth: added user root , total = 2
17:35:52 etcd3 | 2016-09-29 17:35:52.023928 E | auth: added user root , total = 2
17:35:52 etcd2 | 2016-09-29 17:35:52.037939 E | auth: added user root , total = 2
17:35:53 etcd3 | 2016-09-29 17:35:53.858867 E | auth: added user root , total = 3
17:35:53 etcd1 | 2016-09-29 17:35:53.871008 E | auth: added user root , total = 3
17:35:53 etcd2 | 2016-09-29 17:35:53.880090 E | auth: added user root , total = 3
17:35:55 etcd1 | 2016-09-29 17:35:55.794764 E | auth: added user root , total = 4
17:35:55 etcd2 | 2016-09-29 17:35:55.799397 E | auth: added user root , total = 4
17:35:55 etcd3 | 2016-09-29 17:35:55.812799 E | auth: added user root , total = 4
17:35:57 etcd3 | 2016-09-29 17:35:57.196612 E | auth: added user root , total = 5
17:35:57 etcd2 | 2016-09-29 17:35:57.221568 E | auth: added user root , total = 5
17:35:57 etcd1 | 2016-09-29 17:35:57.229336 E | auth: added user root , total = 5
17:35:58 etcd2 | 2016-09-29 17:35:58.512297 E | auth: added user root , total = 6
17:35:58 etcd3 | 2016-09-29 17:35:58.526757 E | auth: added user root , total = 6
17:35:58 etcd1 | 2016-09-29 17:35:58.528447 E | auth: added user root , total = 6
17:35:59 etcd2 | 2016-09-29 17:35:59.872109 E | auth: added user root , total = 7
17:35:59 etcd1 | 2016-09-29 17:35:59.877785 E | auth: added user root , total = 7
17:35:59 etcd3 | 2016-09-29 17:35:59.885849 E | auth: added user root , total = 7

i think this is a problem even when using embedded v3 client.

@vimalk78 yes, current simple token has a problem of not having timeout mechanism. I'm planning to add the mechanism after finishing jwt token support: https://github.com/coreos/etcd/pull/6082

@mitake Can we fix this for simple token first? I will add jwt in 3.2 not 3.1

@xiang90 , @mitake  what is your idea about invalidating the tokens, remove after session is closed, or have a keep alive time and remove after timeout

@xiang90 ok, then I'll fix the simple token first.
@vimalk78 removing after a timeout would be good. Do you have any ideas?

@mitake , removing the token as soon as the session closes will be best. next best thing is to remove earlier token for the same user when user gets authenticated again. this way we wont need a timer based approach. with timer based approach we will also need mechanism for clients to reset the timer, if they choose to continue the session, else forcing to authenticate again.

@xiang90 , @mitake may i work on this and send a PR ?

@vimalk78 of course, it will be great.

@xiang90 should we finish it before releasing 3.1.0? Then I should work on it instead of @vimalk78 for finishing soon. If backporting is ok, I can cooperate with @vimalk78

my solution direction is to remove the simpleToken when the client closes connection. so was trying to find the hook in grpc for the same.
https://groups.google.com/forum/#!topic/grpc-io/EIcQlLqlNQg
for now i can finish by checking for any existing token for username in the auth.assignSimpleTokenToUser and removing it before adding the new Token

Fixed by https://github.com/coreos/etcd/pull/6574


Call iterators' finalizers in Evaluator's finalizer

Squashed version of https://github.com/chainer/chainer/pull/4137 .

Fix typo, and next time I'll use my own forked repo to avoid further typoes remain in commit logs.

LGTM!

Thank you~


Placeholders instead of labels

Currently, we have to create our own templates or specify the placeholder in html to get rid of labels and use placeholders. It would be awesome to specify this in schema - to use labels or placeholders.
As 5.0 is coming, maybe it can be added in release version.

It's already possible to put any component attribute in the autoform object of the schema:
{
   name: {
      type: String,
      autoform: {
         label: false,
         placeholder: "schemaLabel"
      }
   }
}

@aldeed thank you a lot! didn't find it in the docs..

For posterity: https://github.com/aldeed/meteor-autoform#putting-field-attribute-defaults-in-the-schema
Will create a better docs site someday.

@aldeed huge thank you! I tried to read the docs for your 3 awesome packages (simple form, collection 2, autoform) from top to bottom. They are really big and impressive. Didn't pay big attention to that small section.
Maybe you can add this example with placeholders in the demo page - it is a great place to find the correct usage and possibilities for newcomers like me. Thank you!


Make it obvious when test repo is in use

Describe the bug

The test repo backend is indistinguishable from other backends in the UI. If a user doesn't know they're using it, the whole experience is confusing.
Expected behavior

It should be explained in the UI that a test backend is in use, probably with a link to the docs for more info.

Can I take a crack at this?  I'm fairly new to development, but would enjoy a chance to contribute.

@sgarcia30 thanks for that! Looks like somebody beat you to the punch: https://github.com/netlify/netlify-cms/pull/2678
Feel free to find another ticket you'd like to try though, our priorities board has a bunch:  https://github.com/netlify/netlify-cms/projects/7


Deploy APIM CampusEvents V2

Time Tracking: MA-101
This is done when:

[ ] APIM CampusEvents V2 deployed to Prod
[ ] APIM CampusEvents V2 verified in Prod

not needed - v1 QA now in sync with v1 PROD.
v2 now deprecated in QA.
A separate issue will be added to deprecate v2 in PROD when migration away from v2 has been completed (v2/student/events --> v1/student/events)


Where is "convert_habitat_pos_list_to_cropped_map_pos_list"?

convert_habitat_pos_list_to_cropped_map_pos_list is called in robot/habitat_lang_robot.py but I could not find the function definition.

sorry for the confusion. During the refactoring, we forgot to remove this method load_gt_region_map. It is calling some functions' names before the refactoring. That's why you couldn't find the convert_habitat_pos_list_to_cropped_map_pos_list. To achieve the same function now, you can do:
cropped_map_pos_list = []
for habitat_pos in habitat_pos_list:
    habitat_tf = np.eye(4)
    habitat_tf[:3, 3] = np.array(habitat_pos).flatten()
    self.vlmaps_dataloader.from_habitat_tf(habitat_tf)
    row, col, theta_deg = self.vlmaps_dataloader.to_cropped_map_pose()
    cropped_map_pos_list.append([row, col])


install script requires both python-jsbeautify and js-beautifier which are in conflict with eachother

i can't use the install script without having both those packages installed, but i cant install both of them at once
distro: arch linux
package manager: pacman
aur helper: yay

OldGlory should only need jsbeautifier not sure what python-jsbeautify is

either way, this is what i get running
python old_glory.py

Since 5.12, you'll also need to install Rust.
I'll try settting up a Linux Virtual Machine so I can inform you of further steps you need to take.

I have added some details to the Linux page in the Wiki: https://github.com/Jonius7/SteamUI-OldGlory/wiki/Linux
I have successfully got it running in Linux Mint, though with the following error: https://imgur.com/a/3eZsXVb
This means CSS tweaks are working, but JS tweaks aren't.

Thanks,
although building this on arch is a pain, you might want somebody to look into that

I have successfully got it running in Linux Mint, though with the following error: https://imgur.com/a/3eZsXVb
This means CSS tweaks are working, but JS tweaks aren't.

Turns out fixes.txt had CRLF line endings (from Windows), converting the file to use LF line endings fixed the error.


Update to Electron 29 (Electron 23 No Longer Supported)

The version of Electron that Electron.NET is using (23) is no longer supported as of August last year.  The project needs to update to the latest version of Electron.

I looked at updating this myself in the package.json files, but it's a six version jump and I'm not sure I would be able to test the code base well enough to support the community.


[RND-701] put descriptor load under authorization

Closing & reopening to trigger the build process.


register event not getting called

i have the following code in onDeviceReady methd
var push = PushNotification.init({ "android": {"senderID": "12345679"},
"ios": {"alert": "true", "badge": "true", "sound": "true"}, "windows": {} } );
push.on('registration', function(data) {
    alert(data.registrationId);
});

push.on('notification', function(data) {

});

But registration event is not getting called. We are testing on ios 9 device

@teledna did you setup your certificates correctly with Apple?

Thanks for your reply.Its working

Hey.
I'm facing the same problem. Can you please explain what you did that made it work? If it was indeed related to apple certification, can you let me know what fixed that for you?

We have upgraded our cordova version to 5.0
From: Yonatan Wolowelsky<EMAIL_ADDRESS>Sent: Tuesday, December 08, 2015 5:47 PM
To: phonegap/phonegap-plugin-push
Cc: Vijayalakshmi Devi A M
Subject: Re: [phonegap-plugin-push] register event not getting called (#370)
Hey.
I'm facing the same problem. Can you please explain what you did that made it work? If it was indeed related to apple certification, can you let me know what fixed that for you?
—
Reply to this email directly or view it on GitHubhttps://github.com/phonegap/phonegap-plugin-push/issues/370#issuecomment-162864099.


remove unused directories

Directories fldc22/ and d9simple/ are not needed.
This PR removes these directories.

There's nothing wrong with this, but it will happen over and over again because of the submodule addition that runs on startup...

We do have to solve the underlying problem. Thanks for working on this!

Seems like the original problem of these directories being included in the repo, was the LACK of .gitmodules file.
Now that .gitmodules is generated - we shouldn't see this issue happen again.


#37 Renaming things for packaging sake.

Codecov Report

Merging #38 (42fb226) into main (90144e4) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #38   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          203       203
  Branches        41        41
=========================================
  Hits           203       203

Flag
Coverage Δ

metadata-jsonschema
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

metadata-jsonschema/src/utils.ts
100.00% <100.00%> (ø)


rename pool-coordinator to yurt-coordinator for other

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage

/kind feature
What this PR does / why we need it:
Unify the naming rules of each component in openyurtio/openyurt repo, it is more understandable for end users.
Which issue(s) this PR fixes:

Fixes #1482
Special notes for your reviewer:

/assign @rambohe-ch
Does this PR introduce a user-facing change?

other Note

/lgtm


Django Diary date picker bug

Add a new customer function of diary test app located at
http://<IP_ADDRESS>:8000/diary/customer_add/
date picker functions correctly as user is selecting year/month/day, but resulting date is not valid. Cannot be manually edited by user

I'll give this one a shot

The date picker sets the field in the format DD/MM/YYYY, but something is expecting MM/DD/YYYY. This means validating succeeds when you select a date with a day <= 12! Problem solved, as long as you weren't born in the later 2/3rds of any month!
Since this page is from django-diary, I checked that out by itself to see what the behavior was. Surprisingly it seems like it's fine? I'm looking around in both projects to find where freegeek was specifying a different date format but searching is turning up nothing and I'm not familiar enough with django to know precisely where to look. It looks like the date format ends up the same in the raw sqlite database so if it's generated in the same format and stored in the same format... where does it fail in between?
Still looking...

Thanks for the update Nick!

this might be useful (search for "date_format", etc...)
https://docs.djangoproject.com/en/1.11/ref/settings/
my guess is something needs to be changed in settings.py.

"Fixed" in #95


Add Prism.languages.shell to bash file as an alias

Fixes #1320

Great! Thanks for contributing!

@Golmote thank you :).
Im trying to switch from highlight.js to prism in gerrit https://gerrit-review.googlesource.com/c/gerrit/+/163090 :)


[Bug] Protected routes

Currently you don't need to be logged in to be able to see other routes like /about etc one can just type in /about or some other routable URI after localhost:3000..
Protect the app by not allowing users to manually go to any other route without first asking the user to log in and authenticate and authorize themselves, perhaps even register.
This will improve security

i'm interesting in working on this issue if no one is working on it currently

Ok @Amrani-Farouk-Hossam-Eddine
I assigned you.


Change checks for present command line arguments

This changes the command line argument parsing behaviour to return all flags that were explicitly provided in the arguments by the user.
This PR fixes #83310
https://github.com/cdr/code-server/issues/1109

Thanks @featherbear


adjust navbar according of adjustable length of browsers and desktops

The navbar is just covering half of the screen in one desktop/browser or going out of bounds in another.Adjust it.

Yes this is critical!

Can I use bootstrap?? Or should I only use media queries?

You can use bootstrap if you want 😉


feat: create @jest/config-utils package

Summary
Not too happy about the name. Main idea is to colocate both types and utils, since jest-config depends on the world to be able to resolve default dependencies in a pnp context.
This makes it so only jest-circus, jest-jasmine, jest-each and jest-environment* depends on @jest/types.
Fixes #8570

Test plan
Green CI

Doesn't it make sense to create @jest/config-types with config types only then?


Update community meeting time

Community meetings are happening every other Thursday now, updating kcp.io accordingly.

/lgtm
/approve

/hold


ncp: sys/types.h not available in some embedded toolchains

The IAR toolchain libc doesn't have sys/types.h included by spinel, and thus doesn't compile at this time.  It would be good if the spinel client code didn't have this dependency so it can easily be retargeted for a wide variety of MCUs and toolchains.

I can't remember what I'm using sys/types.h for. I'm curious what breaks if you comment it out.

Hi RQ, yes, deleting that line from spinel.h seems to build fine.

Correction, these symbols seem to be missing without it:
EOVERFLOW
EINVAL
ssize_t

Fixed in #193.

EOVERFLOW and EINVAL should come from errno.h, included in spinel.c. Maybe IAR doesn't define these errors?
ssize_t should get pulled in by stdio.h, which needs it for the return value of printf(). Er, nevermind, printf() returns int. >_<
I'll just get rid of the use of ssize_t. Will figure out what to do about EOVERFLOW...

Reopening, based on @turon's comments.

Fixed in #203.


TextField Clear Causes UI Shift: App Moves Down by Keyboard Height on Text Deletion

Steps to reproduce

Environment Setup: Ensure you are running iOS 17 or above. This issue does not occur on iOS 16.7.10.
Browser: Open Safari on your iOS device.
Run the Flutter App: Use the following command in your terminal to start the Flutter app on a web server:
flutter run --release -d web-server --web-port 8000 --web-hostname <IP_ADDRESS> I used this because I deploy the webapp using docker.
flutter build web can also be used.

Expected results
The UI should behave normally..
Actual results
The UI acts weirdly and sometimes makes the app totally inaccessible until we put the browser in background, and resume the browser app(safari) again as this causes rebuild of the UI.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: Text('TextField Issue Demo'),
        ),
        body: Center(
          child: Padding(
            padding: const EdgeInsets.all(16.0),
            child: TextField(
              decoration: InputDecoration(
                labelText: 'Enter text',
                border: OutlineInputBorder(),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/user-attachments/assets/07590a6c-b990-4042-a3be-8fa5dedeebcb

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.22.2, on macOS 14.4 23E214 darwin-arm64, locale en-IN)
    • Flutter version 3.22.2 on channel stable at /Users/hrithikmishra/sdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 761747bfc5 (5 months ago), 2024-06-05 22:15:13 +0200
    • Engine revision edd8546116
    • Dart version 3.4.3
    • DevTools version 2.34.3

[!] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at /Users/hrithikmishra/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)

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

[✓] Connected device (4 available)
    • iPhone 15 (mobile)              • C0295D96-D6BA-49B3-95D0-2F6AAD795E89 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-17-5 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS
      14.4 23E214 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS
      14.4 23E214 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google
      Chrome 130.0.6723.60

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

PS: This issue is not simulator specific, I'm facing this exact issue on real devices too.

Hi @hrithikmtorana
I can't reproduce this issue on iPhone 16 Pro, iOS 18.0 emulator. Your demo video shows your project app. Can you confirm this also happens on the corresponding simple sample code as provided? (to make sure there is no unexpected configuration from project that was not mentioned in the report)

Hi @huycozy, sorry for the inconvenience. This issue was caused by some messed up splash logic in the index.html file of my project app. Fixing the index.html file solved the issue for me.
Thanks for the response. Closing this issue.


Fix blueprint packages getting added to history

Removed the updateBlueprint function since it provided no functionality not being handled by redux and was adding/removing components for a blueprint's entire history.

The updates relative to changes stored in localPendingChanges seems to behave as expected. But I'm noticing some issues with workspacePendingChanges.
Try these steps, and you'll see a couple of weird behaviors:

Create a new blueprint
Add a few packages
Log out, then log back in.

The packages are still listed in the UI as expected, but they are also listed in the state for the past version of the blueprint, which is not expected.

Hit Discard Changes

The packages are still listed in the UI, but shouldn't be there.
Additionally, the package description now displays with the name. This is the only time I'm seeing a description display in the list, but I think it should always be there. But if this is unrelated to this PR, we can create a separate issue for it.

I am seeing this behavior in master as well so I don't think it is related to this PR. I am going to look into that issue now if you don't have any other issues with this PR.

Sure, that sounds good to me.


Document mlf_core functionality

Follow up to #220 and #232

@Emiller88 Do you think that you could bring this one in today/tomorrow?

@Emiller88 thanks!


istio stats of traffic  from outside passing through ingress gateway updated only when client close the connection(case: websocket)

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
istio stats reflect the real traffic
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ X ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context
In my case it is websocket connection , traffic from outside through ingressgateway ,vs,dstrule ;finally tcp to pod with istio sidecar.
istio_requests_total  and istio_tcp_sent_bytes_total(in my case) stats only get updated when client(remote) close the connection
I notice istio_tcp_sent_bytes_total is a tcp metric. When two pods both with sidecars, communication in the mesh with tcp ,istio_tcp_sent_bytes_total get updated in the life of connection.
In env a client from outside without sidecar request to server in istio , the server responses to client, can we update istio_tcp_sent_bytes_total at regular intervals until the client close the connection, not update after client close the connection.

Websocket is special, it is to build a connection using http handshake. And the following data is transported via the tcp connection. There is no HTTP request/response concept. So it behaves like tcp here

Websocket is special, it is to build a connection using http handshake. And the following data is transported via the tcp connection. There is no HTTP request/response concept. So it behaves like tcp here

istio_tcp_sent_bytes_total should update at regular intervals，not update after client close the connection

Websocket is special, it is to build a connection using http handshake. And the following data is transported via the tcp connection. There is no HTTP request/response concept. So it behaves like tcp here

istio_tcp_sent_bytes_total should update at regular intervals，not update after client close the connection

I recall it update every 5s by default, @kyessenov anything sepcial to websocket?

good to know, can you point to it?

Periodic reporting is only done for gRPC traffic, not every HTTP request. This is a known limitation.

Periodic reporting is only done for gRPC traffic, not every HTTP request. This is a known limitation.

I notice TCP telemetry collection in mesh with client and server envoy.
What about tcp client outside of mesh through gateway, We can get metrics in the tcp server side (istio-proxy)


Async slower than sync for linkchecker

This is just an option but the async await is much slower the .then().catch
but you can turn your await fetch to  fetch().then.catch()
It will be a fire and forget it method and is much quicker, I have forked your repo and shown a potential option to quicken checking links
https://github.com/zg3d/URLpilgrim/blob/master/urlFunctions/urlFuncs.js#L17-L40
The downfall is that you Ora spinner won't work anymore but we can find a potential fix to it.
The above method quicken the tool by a great margin and might be a great addition.

@zg3d
Sorry but I just checked and tested it today.
Yes, the performance was increased.
Thanks.


How can I implement this server in a glitch project?

Hello, I just wanted to know how to implement this server on a glitch project that includes the client from the demo. If anyone can help, thanks.

new commit has the code and instruction to do it


Passing further info to the hide/hidden events to see which object triggered the event

I was in the situation of having a modal with an 'ok' and 'cancel' button and needed to figure out which caused the modal to close.
I needed the event info that was passed to the Modal.hide() function to be passed to the hide and hidden events as well. So now I can do this:
$('#modal-from-dom').bind('hidden', function(e, f) {
  if ($.inArray("cancel",f.srcElement.classList) < 0) {
    // some stuff
  }
});

Given these elements:
Ok
Cancel

Not a fan of non standard event callbacks like this. I would be open to merging the srcElement/target etc into the actual event object though if you want to reopen a pull request against 2.0-wip


Adds client

Use Case:
As a fhir-aggregator user, I need a way to connect to FHIR servers, determine their contents and run queries.
This PR:

Adds a client with plugin authorization modules
Adds convenience methods to interrogate servers' CapabilityStatement and ResearchStudy objects
Harvests test fixtures to be used for unit tests see here
Simplifies and aggregates ResearchStudy into a tsv

Decision log:

Base client on existing smart on fhir client

Installation:
pip install fhir-aggregator==0.2rc1
To install development dependencies
pip install -r requirements-dev.txt
Testing:
Unit tests:

run pytest tests/unit

Integration tests:

Please see tests/fixtures/README.md about getting a dbgap token
run pytest tests/integration

Unit test passes after installing inflection module 👍 LGTM
For integration test - not sure about the syntax of dbgap-task-specific-token.json or where the last sentence "Consult the API documentation for how to include the token in your request to the API" is referring to.

For integration test - not sure about the syntax of dbgap-task-specific-token.json or where the last sentence "Consult the API documentation for how to include the token in your request to the API" is referring to.

I've updated instructions. See https://github.com/FHIR-Aggregator/client/pull/3/commits/94f69313ed6e0a655c47be57e4a786b34d4235b1


Question: Can I backup to Github repo?

Hello,
Is there any way to backup to github ripo?
Best regards

I mean, you could backup locally into a directory with a git repository, commit the files, and push them but I don’t see a great way to do that without a botch like that.

You can't and you shouldn't

Git isn't a very good tool to "backup" binary files. You should use tools for what they ment to do.
If you need to backup you're code or text notes -- you're already doing it by git pushing
Using GitHub as a "backup storage" is extremely bad technical decision. This usecase is rare, insecure, with bad performance (because restic and git would keep history of files changes, so backup size would grow at least 2x as fast).

@rawtaz I suggest to close this issue, because of the points written above

As @brualan has explained, GitHub would make for a very bad "backup storage". In addition, GitHub repositories are intended to store code or some projects, not your personal backup.


Work on background

Feature Request
Motivation Behind Feature
Sometimes getting the position when app is on background is crucial.
Feature Description
Get the position on background.
Alternatives or Workarounds
Tried to make the app available on background using a plugin but this is not working and not a good behaviour at all.
The GPS should be working on background as well.

where you able to make it work on background?

2019 request and still this isn't available? Wow... well I'm sure going to abandon using this one.
Just to confirm, this plugin will not retrieve geolocation if you're running your app in background mode.

2019 request and still this isn't available? Wow... well I'm sure going to abandon using this one.
Just to confirm, this plugin will not retrieve geolocation if you're running your app in background mode.

To be clear, background processing is very limited on hybrid apps as the webview JS engine will be completely paused on iOS, and could be completely paused on some Android devices while the application is in the background.
This means at most, we may be able to collect GPS events that occurred in the background, but can only be passed to your application once the application is brought back into the foreground. Obviously in most cases, you'd want to be able to react to background location events immediately to produce a notification to the user, or to do something with it so receiving a stream of location events that has happened sometime in the past isn't exactly ideal.
Depending on the use case / requirements, the only workaround is to build a cordova plugin that contains your application business logic that responds to background location events.
With that being said, there are existing third-party plugins that do implement background location.

Thank you for responding and pointing me in the right direction @breautek ! Really appreciate it

Until W3C spec actually declares background usage + the limited execution ability of the WKWebView/js engines on iOS when app is suspended, I don't foresee background support being implemented.
This plugin aims to maintain the W3C standard so that it can cease to exist when the webview itself is capable of handling geolocation.
If background geolocation support is necessary, as mentioned in a previous post there are other third-party plugins that does (or claims to) provide background location updates.


Looking for permission to modify and publish a revision of your data science life cycle diagram, in a new article I would like to publish in Medium

Hello, I saw your data science life cycle picture in an article one of your staff published in Toward Data Science.  Here is that picture:

I have revised the picture to add a focus on social determinants of health and health equity, as follows.  I would like to publish this one in an article I will submit to Medium.    Assuming it would be better to have your permission to revise and use the new picture, will you grant that permission in writing?   Medium requires written permission for borrowed images to be used.

Please let me know if you cannot see either image; if you cannot I'd be happy to email both directly, if you send me an email address.
Ron Ozminkowski, PhD
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 925cd64a-03a0-dc29-5b52-e51483782d3f
Version Independent ID: f4ccbd9c-1bc2-c069-9ca8-a34bc61409e2
Content: The Team Data Science Process lifecycle - Azure Architecture Center
Content Source: docs/data-science-process/lifecycle.md
Service: machine-learning
Sub-service: team-data-science-process
GitHub Login: @marktab
Microsoft Alias: tdsp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 495213

@rozmink - Looks good. Thanks for checking!
Please credit and link to the original article.

Will do, thank you
On Thu, Sep 30, 2021 at 1:01 PM Ed Price - MSFT @.***>
wrote:

Closed #3084
https://github.com/MicrosoftDocs/architecture-center/issues/3084.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/architecture-center/issues/3084#event-5389160880,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMKRF5WBGOSXOZCLOEBPGQ3UESJWVANCNFSM5FATGD5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Issue 544: Case Histories - SEG

This PR addresses issue #544 . Citations for SEG materials have been added to the case history pages. The bibtex file containing citations has also been updated.

@lheagy I have tried to find where this is failing by locally running the python tests in the /tests directory, however they all pass. Unfortunately Travis fails. I am not entirely sure what the differences are between the tests I run locally and the tests being run on Travis. From what I understand, we specify the 'test_dir' in the travis.yml file which means the same tests are being run locally and on Tavis, correct?.


[9.x] Add prompt to optimize command in non-production environments

Various places in the Laravel docs say that the caching of routes etc should only be done as part of a deployment process. Yet various developers get tripped up after inadvertently running any of the cache:* or optimize commands and then wonder why things aren‘t working when they’re changing source files.
This PR adds a prompt when trying to run php artisan optimize in non-production environments. This means that any one running php artisan optimize during a deployment to their production environments shouldn’t be impacted (so long as they have set their APP_ENV value to production); but any one trying to run it in other environments (say local) will be prompted to actually confirm they want to run the command.

Couldn't this break applications deploying to staging environments where they actually do want optimize to run?
IMO this prompt should only display if the environment is actually local.

IMO this prompt should only display if the environment is actually local.

Can certainly change the prompt callback to that case if you prefer as yeah, the prompt would be displayed if the environment was set to something like staging, qa, etc.

Honestly I think I would just maybe add a comment to the output if the environment is not production - no prompt.

I rebuild the cache a lot from scripting in my local environment. I know I have to do that when I change routes and when I introduce a new class. This additional confirmation message would only be annoying to me. At least it should be suppressable by a commandline switch.
So I agree with @taylorotwell that a message would be better.

Yeah - I think just a comment would be sufficient.

@taylorotwell Can you re-open this and I’ll change it to just an informational message instead?


Network Proximity Checker No Longer Excludes ForceHidden Objects

Describe the bug
Player objects with a NetworkProximityChecker component with forceHidden set to TRUE are not hidden from clients who have already connected.
Repro project
https://www.dropbox.com/s/mbgkj8u6nl42gs1/Network Proximity Checker Bug Repro Project.zip?dl=0
To Reproduce
Steps to reproduce the behavior:

Build and run the attached project.  In this project, the Player prefab has a NetworkProximityChecker with forceHidden = true set.
Duplicate the built app (optional, but it helps highlight the issue).
Start LAN Host in one of the built apps.
Run Unity Editor and connect as a client.
Observe that the client cannot see the LAN Host player object (the NetworkProximityChecker hides it as expected).  However, observe that the LAN Host CAN see the client player object (the NetworkProximityChecker is NOT hiding it as expected).
Run the second copy of the Built app as a client
Once again, observe that the two players who were already connected can see the new player (bug), and the new player cannot see the two players who were already connected (expected behavior).

Expected behavior
Player prefabs with a NetworkProximityChecker component with forceHidden set to TRUE should not appear (e.g. not be present in the hierarchy and not receive network traffic) on other clients.
Desktop (please complete the following information):

OS:  MacOS
Build target: Standalone
Unity version: 2018.3.11f1
Mirror branch: master

Additional context
I had to roll Mirror back to release 3.2.0 in order to restore the expected functionality.

Please see if this is fixed in 3.6.6

I'm afraid the bug persists in 3.6.6.  :(

5\. However, observe that the LAN Host **_CAN_** see the client player object (the NetworkProximityChecker is **_NOT_** hiding it as expected).

LAN Host is the server and can see everything.  Is that the crux of your issue?

No, sorry if I wasn't clear.  The clients can see each other, when they shouldn't be able to.

Created PR #893 to resolve

:tada: This issue has been resolved in version 3.7.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Introspection

This is working for some cases. There is still more I need to implement (like fields/values etc.) but wanted to open up a PR for comments if this looked like the right direction.
This felt like a good balance between leveraging the existing codegen by defining the wrapper types in Introspection.hack while giving us more control over how objects expose their values.
This does introduce a lot of new classes/interfaces though I'm not sure if there is a way we can simplify that.
Getting the SDL via introspection will be easier if we have fragments: https://github.com/graphql-python/graphql-core/blob/03c2b06e4636ed8a89c7a4c7e56d244fd9d00bde/src/graphql/utilities/get_introspection_query.py#L24 but we can write the fragments out until we support them.

Once https://github.com/mwildehahn/hack-graphql/pull/67 is merged I'll put that in and clean this up a bit. This doesn't implement all the features of introspection but it's enough and this is already really large 😄 .
The remaining ones are returning interfaces a class implements, classes that implement an interface etc.

I'm going to close out this PR. I have a v2 which is much simpler. The part I was missing was this: https://github.com/mwildehahn/hack-graphql/pull/66#discussion_r632842802. Generating interfaces for the introspection and having the generated types implement them worked much better!


[BEAM-11772] Fix auto-sharding parameter for BigQuery sink with FILE LOADS

Using user triggering frequency to limit GroupIntoBatches could introduce performance penalties as the triggering frequency could be set to a large value and we have to buffer all the data in the states (as GroupIntoBatches uses state/timer for batching).

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @pabloem

CC @kennknowles

LGTM.


Determine how to handle before_request for gRPC

I discovered that I did something totally wrong when I implemented our grpc transport.
All of our HTTP transports call credentials.before_request to get the authorization header populated (here). However, the gRPC transport instead calls credentials.refresh directly and constructs the header itself (here).
I think the gRPC transport needs to do the same as the others, but that's where it gets tricky. before_request takes 4 arguments request, method, url, request_headers. gRPC provides our auth plugin with a context that looks like AuthMetadataContext(service_url='https://pubsub.googleapis.com/google.pubsub.v1.Publisher', method_name='ListTopics'). Does it make sense to call before_request(http_request, context.method_name, context.service_url, headers) even though context.method_name is not an HTTP method? It it okay to overload the term method for different transports?
@dhermes @lukesneeringer thoughts?

I never understood why we cared that there was a gRPC "transport" since it can't do much/any of the handshakes over the wire. So in that light, why do we care about giving it a novel interface?

@dhermes I'm not sure what you mean. What's the alternative?

Just a module with gRPC helpers (which is essentially what it is already).

What would you remove from the current module? Or do you just not like that it's in google.auth.transports.
At any rate, gRPC needs to support all of our existing credentials and needs to interact with them properly, so the gRPC auth metadata plugin needs to call before_request just like everything else.

This might be easier to explain in code https://github.com/GoogleCloudPlatform/google-auth-library-python/pull/116
The key thing I'm stuck on here is look at the updated docstring for before_request.


Reintroduce bake enhancements

from 3.x

Codecov Report

Merging #24 into master will decrease coverage by 1.92%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #24      +/-   ##
============================================
- Coverage     52.47%   50.55%   -1.93%
- Complexity      497      527      +30
============================================
  Files            23       24       +1
  Lines          1574     1634      +60
============================================
  Hits            826      826
- Misses          748      808      +60

Impacted Files
Coverage Δ
Complexity Δ

src/View/Helper/SetupBakeHelper.php
0.00% <0.00%> (ø)
30.00 <30.00> (?)

Continue to review full report at Codecov.

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


[feature] clarify the tile parameter

Hello,
This is more a feature request than a bug. Your tool is really great !
A function is missing from my point of view: "list the available tiles". In fact, I can identify the middle of the needed map, but I am unable to identify the best values for tiles. Furthermore, a function to be able to stitch the flat images after import could be great. I try to import 2000x2000 tiles and my computer freezes.
Finaly, there is a mistake in your readme n the paragraph Then list all maps avaialble for this world :
dynmap-timemachine.py --list-worlds http://map.majncraft.cz/ world

The parameter should be --list-maps.
Best regards.

It would be cool to have a parameter to just download everything. Or, how do i know, how big one tile is. Is it 16 if Scale for the Dynmap Perspective is 16?


Use jsaddle instead of stubbing

Oh I had a go at doing this a while back to check what (if anything was missing in JSaddle).  But I did not have time to check if the code actually worked. I'll send a pull request as it might still be a good starting point.

I have send a pull request that seems to build ok for me, but I have still not tested that the JSaddle code actually works (although it should work at least as well as the stubs did).

@hamishmack Awesome! I never dreamed the work would already be started (and maybe mostly finished)!


Label issue under New License Order (Agent Panel)

Description of the bug
Instead of License Renewal there should be "License Renewal Type".
Steps to reproduce the behavior:

Go to 'https://www.pulseptt.com/agent.html'
Login with valid credential:- 1' User name; john 2. Password; 12345678
Navigate to License Management.
See error

Expected behavior
Proper Labelling.
Screenshots

Desktop
OS: Windows 10
Browser :Version 100.0.4896.60 (Official Build) (64-bit)
Version [0.1.1]

What do you mean by proper labelling?

Instead of "License Renewal" there should be "License Renewal Type". As we see, when we click on Drop-down button then we get "Select License Renewal Type".

Okay, I'll fix it,
Thanks for explaining!


Load Google fonts in single request

To request multiple font families, separate the names with a pipe character (|).
- Google Fonts - Specifying font families and styles in a stylesheet URL

:+1:


query{_vec} use IndexSummary

This builds on the work from https://github.com/rust-lang/cargo/pull/12749 and the discussion of the overall approach is at https://rust-lang.zulipchat.com/#narrow/stream/246057-t-cargo/topic/better.20error.20messages.20for.20filtered.20versions.2E
What does this PR try to resolve?
Changing the two traits Registry and Source to use the new `IndexSummary' involves a lot of changes all throughout the code base. This would be hard to review if it also included any logic changes. So this PR is just adding the type to the trait and immediately unwrapping every place it is used.
The hope is that reviewing changes to logic/ergonomics will be easier to review once the mechanical changes have been merged.
How should we test and review this PR?
This is an internal re-factoring and all the tests still pass.

@bors r+
I guess this is like my work on #12801.  Too big to do in a single PR but unsure of the results.  So only one way to find out.

:pushpin: Commit 6835fa386720c19cd0bf56b078b8f7d7dfa7dfc0 has been approved by epage
It is now in the queue for this repository.

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 43abf906193c8a810361a39c5cf2bb3bb104637a to master...


[add-resource] New Plugin

I would like to add my new plugin/theme to the Grav Repository.
Here are the project details: JulianSchoenbaechler/grav-plugin-zooming

Added to GPM


Add support for export keyword

Description of the issue
It's common for developers to have a .env file which is a shell script, so they can easily source them manually if the need arises.
Often they will use export for a variable for that purpose as well.
Take the following file as example:
#!/usr/bin/env bash
export KEY=value

It is a valid and loadable env file for many tools such as direnv, dotenv (NPM), etc, and also supports direct source
However, it does not play well with dotenvx. Trying to migrate and encrypt the values does not result in the desired output.
❯ dotenvx encrypt
✔ encrypted (.env)
✔ key added to .env.keys (DOTENV_PRIVATE_KEY)
ℹ add .env.keys to .gitignore: [echo ".env.keys" >> .gitignore]
ℹ run [DOTENV_PRIVATE_KEY='05baf46b1de94218e746b410765be902ed29921ca62114cedb65fa3b8cc31990' dotenvx run -- yourcommand] to test decryption locally

❯ cat .env
#/-------------------[DOTENV_PUBLIC_KEY]--------------------/
#/            public-key encryption for .env files          /
#/       [how it works](https://dotenvx.com/encryption)     /
#/----------------------------------------------------------/
DOTENV_PUBLIC_KEY="0243547f7fa8eb4dc04382166c1ecc255d90d6e336bde9a667b86ef175b7f62ebf"

# .env
#!/usr/bin/env bash
export KEY=value

❯ cat .env.keys
#/------------------!DOTENV_PRIVATE_KEYS!-------------------/
#/ private decryption keys. DO NOT commit to source control /
#/     [how it works](https://dotenvx.com/encryption)       /
#/----------------------------------------------------------/

# .env
DOTENV_PRIVATE_KEY="05baf46b1de94218e746b410765be902ed29921ca62114cedb65fa3b8cc31990"

The keys were properly created and added
The value was not encrypted at all

Using dotenvx set also doesn't resolve this:
❯ dotenvx set KEY value
✔ set KEY with encryption (.env)

❯ cat .env
#/-------------------[DOTENV_PUBLIC_KEY]--------------------/
#/            public-key encryption for .env files          /
#/       [how it works](https://dotenvx.com/encryption)     /
#/----------------------------------------------------------/
DOTENV_PUBLIC_KEY="0243547f7fa8eb4dc04382166c1ecc255d90d6e336bde9a667b86ef175b7f62ebf"

# .env
#!/usr/bin/env bash
export KEY=value

Expected behavior
Since export is probably not relevant anymore as sourcing directly won't work anyway, the export keyword doesn't have to remain in the final output.
However, encrypting a file or setting a value should still work out of the box, to reduce friction by a lot. (Took me a solid 20 minutes to realize this was the source of the issue, I suspect not everyone will be able to in a reasonable time)
I expect the output to probably be:
❯ cat .env
#/-------------------[DOTENV_PUBLIC_KEY]--------------------/
#/            public-key encryption for .env files          /
#/       [how it works](https://dotenvx.com/encryption)     /
#/----------------------------------------------------------/
DOTENV_PUBLIC_KEY="0243547f7fa8eb4dc04382166c1ecc255d90d6e336bde9a667b86ef175b7f62ebf"

# .env
KEY="encrypted:........."

Also note that the shebang also moves to an incorrect place, it should either be removed or remain at the top.
Proposal
This requires 2 things to be mainly done:

Find the correct lines even if they are prefixed with "export "
Ignore the export keyword when parsing the key name before encrypting

As for the shebang, I don't think it's as high of a priority, but could also be handled better.
What is the stance on this from the author(s)?
I might be able to make a PR myself but before I work on it I would like to know if it's something you guys want to resolve.
If it's not to be resolved, maybe the docs could be updated to reflect this caveat. The simplest solution on the user's end is to simply remove all export  prefixes before encrypting, but it should be mentioned somewhere (if it was, I apologize - but I couldn't find it)
Thanks for a truly great package either way :)

Just noticed these related issue:

https://github.com/motdotla/dotenv/issues/213
https://github.com/motdotla/dotenv/issues/521

Is there any progress done in the subject, or anywhere to track progress? Would love to see this implemented.

dotenv supports export these days: https://github.com/motdotla/dotenv/blob/master/lib/main.js#L9
the problem here is the encryption logic in dotenvx is not smart enough to recognize the export KEY. i'll patch up.

Thanks for a truly great package either way :)

It's been my pleasure. 🎩

fix going out now as lucky number 11 11: v1.11.1 https://github.com/dotenvx/dotenvx/pull/357
it also preserves shebangs now.

Wonderful! Thank you :)


Spek 1.1.1 is not working when using IDEA Spek 0.3.4 plugin

When using Spek 1.1.1 dependencies and the latest Spec IDEA plugin (0.3.4), fails to run test case with below error, but gradle test works.

dependencies {
testCompile "org.jetbrains.spek:spek-api:1.1.1"
testRuntime "org.jetbrains.spek:spek-junit-platform-engine:1.1.1"
}

@innobead Please add org.junit.platform:junit-platform-launcher:1.0.0-M4 to your test classpath for now.

testRuntime "org.junit.platform:junit-platform-launcher:1.0.0-M4" works, perfect. Thanks for the tip.

keep it for future fix in next release.

Is there a Maven config example for fixing it?

@digizeph https://github.com/spekframework/spek-maven-sample


Add support for defining test data

Have a way to define complex test data that can be used as a reference in specs. Table based data works for flat, repeated data.
Consider having a YAML/JSON file, that Gauge would parse and would either pass to the runner as a test data object or have an api to look up against.
The YAML/JSON definition should contain a key, and Gauge should be able to de-serialize this data.
Helps writing succinct Steps, without writing verbose setup steps/code.

This is really important for a couple of reasons.

Separation of concerns .. test data should not be mixed in with the test case itself.
Allowing tests to be "data driven" with large, datasets.

I think that a good implementation would be to allow an abstracted "data source" to be utilised. What I am thinking from a practical point of view is how can I use a parwise or random generator to generate the test data, as well as a set of "known good" test data.
Data sources I am thinking of are: CSV, JSON, YML and a SQLite database.

Support for JSON files at a minimum would be a huge step forward.

I'm wondering whether this is now doable with https://github.com/getgauge/gauge-java/issues/140
given the following json file (or any other format you like)
{
    "userName" : "John Doe"
}

and a custom parser like
public class PersonGaugeParser extends CustomParameterParser<Person> {
    @Override
    public boolean canParse(Class<?> aClass, Spec.Parameter parameter) {
        return aClass.equals(Person.class);
    }

    @Override
    protected Person customParse(Class<?> aClass, Spec.Parameter parameter) {
        return new Gson().fromJson(parameter.getValue(), (Class<? extends  Person>) aClass);
    }
}

one can now use as step implementation
@Step("Create from file <file:person.json>")
public void fromFile(Person person) {
      Assert.assertEquals(person.getUserName(), "John Doe");
}

is this what the issue is about ?

@jackperalta It does address the issue partially. It is still upto the user to define these parsers. It would be helpful to have a few of these parsers bundled with Gauge.

Observation
Gauge tests can be written and run in multiple languages. Java, C#, Ruby, Javascript and Python are the officially supported language runners.
@jackperalta - The issue getgauge/gauge-java#140 deals with parsing Custom types for Java.
Suggestion
The following suggests one approach for this. This implementation will have 2 steps
In Gauge-core

Have a JSONObject in Gauge. JSON parameters are converted to set of map values here.
The language runners
Handling this format in languages Javascript, Ruby, Python is straightforward. Java & C# language runners will have to handle converting it to the corresponding Custom type.

In the approach mentioned above, Gauge is responsible for recognizing convert from format i.e String or JSON. The parsers in the language runners will find the convert to format and perform the parsing.
In the approach given below, the individual language runners will be responsible for recognizing convert from and to formats and the parse it.
Another approach
Today when <file:user.json> parameter is used, Gauge passes the contents of the file as the parameter value. The language runners parse the parameter and do the required conversion.

Currently Javascript, Python and Ruby language runners do not have custom parsing parameters.

Custom Parameter parsing is available in Java  - getgauge/gauge-java
Today, the c# runner attempts to convert the parameter to any type that is IConvertible

This will be partially solved when implemented using code blocks https://github.com/getgauge/gauge/issues/1240


Add on apps (Pantry Party and Grocy Mobile (IOS app)) both fail with same error when attempting to add product to database when using bar code scanning.

See https://github.com/grocy/grocy/issues/2240
There appears to be an incompatibility between Pantry Party and Grocy v3.3.2 and likely also later releases with the removal of the qu_factor_purchase_to_stock field.

I'm sorry to hear this. Unfortunately I'm not using grocy anymore and I let
my apple developer account lapse.
All this means I'm likely not going to be able to release a fix.
On Sun, May 21, 2023, 5:03 PM kingswindsor @.***> wrote:

See grocy/grocy#2240 https://github.com/grocy/grocy/issues/2240
There appears to be an incompatibility between Pantry Party and Grocy
v3.3.2 and likely also later releases with the removal of the
qu_factor_purchase_to_stock field.
—
Reply to this email directly, view it on GitHub
https://github.com/PantryParty/pantry_party/issues/146, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABQ6MXHPJE7XBXPJRQEEADXHJ7KDANCNFSM6AAAAAAYJT5GUU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

There appears to be an incompatibility between Pantry Party and Grocy v3.3.2 [...] with the removal of the qu_factor_purchase_to_stock field

Just to clarify that also here:
v3.3.2 has not (!) removed this field, so this can't be the problem when running the current Grocy release v3.3.2.
@kingswindsor is most likely running the current development version (see https://github.com/grocy/grocy/issues/2240#issuecomment-1556293526) and also uses an unsupported PHP runtime version, so naturally anything can go wrong without real surprise I guess.


river-extra is not compatible with river version

Hello again,
I have a problem when I import some metrics from river_extra. I think that this version of river_extra is not compatible with the last version of river 0.11.1 . For example when I import purity metric, this is the error that gives me:

This is an other one when importing cluster:

Mmm so sorry about this, I'll look into it.
There haven't been many users of this repo until now, so apologies for you acting as the guinea pig :)

Ok try again, it works for me:
from river_extra import metrics

metrics.Purity()
metrics.cluster.BIC()

Thank you, I tested almost all metrics, there are just 2 metrics that don't work and need to be updated with the new version of river: CalinskiHarabasz (ssq_based.py) and BIC.
This is the error that I got:

Good catch, fixed. Keep letting me know if there are remaining issues.


Jonas add GitHub field

Codecov Report

Merging #53 (019b439) into master (92b235e) will decrease coverage by 1.73%.
The diff coverage is 31.57%.

@@             Coverage Diff              @@
##             master      #53      +/-   ##
============================================
- Coverage     62.14%   60.41%   -1.74%
- Complexity      464      472       +8
============================================
  Files            87       90       +3
  Lines          1749     1839      +90
  Branches        208      222      +14
============================================
+ Hits           1087     1111      +24
- Misses          566      623      +57
- Partials         96      105       +9

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/GithubCommand.java
0.00% <0.00%> (ø)

...java/seedu/address/logic/commands/ProfCommand.java
66.66% <ø> (ø)

...a/seedu/address/logic/commands/StudentCommand.java
100.00% <ø> (ø)

...n/java/seedu/address/logic/commands/TaCommand.java
66.66% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
68.18% <0.00%> (-3.25%)
:arrow_down:

...eedu/address/logic/parser/GithubCommandParser.java
0.00% <0.00%> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
75.40% <0.00%> (-6.74%)
:arrow_down:

.../seedu/address/logic/parser/ProfCommandParser.java
70.58% <0.00%> (-4.42%)
:arrow_down:

...edu/address/logic/parser/StudentCommandParser.java
6.25% <0.00%> (-0.42%)
:arrow_down:

...va/seedu/address/logic/parser/TaCommandParser.java
5.88% <0.00%> (-0.37%)
:arrow_down:

... and 15 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


chain size through rpc

Is it possible to get chain size on disk through rpc?
The equivalent of bitcoind size_on_disk property from the getblockchaininfo rpc method.

Not currently. This was added by bitcoin/bitcoin#11367 which has not been backported at this point.

this was backported in https://github.com/dashpay/dash/pull/3284
Will be released in <IP_ADDRESS>


docs.akri.sh is down?

Describe the bug
I can't seem to connect to https://docs.akri.sh. DNS resolve failed with SERVFAIL:

@johnliu55-msft, thank you for bringing this up. We need to update our CNAME record with the CNCF and are actively working to make the docs available. In the mean time, Markdown docs can be found here https://github.com/project-akri/akri-docs/tree/main/docs. @adithyaj can you post any updates here?

Thanks @johnliu55-msft!
We filled a ticket with the CNCF to update the CNAME record for our docs. Once I hear back I'll post updates here (and we can use this issue to track the status).

Updating here - got confirmation that they got the request and the webhost maintainer on their side is on vacation. We will work with them to get this back up once they get back shortly.

I worked with them yesterday and today so this should be resolved, once the changes propagate and I see the domain back up I'll close this issue.

@adithyaj looks like the site is pointing to a spam site.

The old URL for gitbook seemed to have lapsed and pointed to a spam site so when it was restored it pointed to it. We immediately took the domain down again and are slowly starting to spin back up with the new URL.
Sorry for the inconvenience!

The website is back up again! 🎉


feat: 🎸 modify portfolio name

JIRA Link
https://polymesh.atlassian.net/browse/DA-76
Changelog / Description

adds endpoint to modify portfolio name

Checklist -

[x] New Feature ?
[x] Updated swagger annotation (if API structure is changed) ?
[x] Unit Test (if possible) ?
[x] Updated the Readme.md (if required) ?

:tada: This PR is included in version 2.5.0-alpha.3.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


tutorials: Athena Iceberg

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

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 0941f96b1df5334a45eee05c444f9465cb0ac72a
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


lhs2tex: Distribute again

In https://github.com/NixOS/nixpkgs/commit/ad6a0e8a63e1 I added lhs2tex (a useful tool of its own) to the toplevel  and the release-haskell.nix file, so that users will find it on the cache.
Turns out that this didn’t quite work, because it was removed from things to be distributed in pkgs/development/haskell-modules/configuration-common.nix, since https://github.com/NixOS/nixpkgs/commit/b6b13e45f8a9 7 years ago. (HT to @vcunat for helping me figure that out.)
I assume (but couldn’t quite reproduce) that at some point in the past, the lhs2tex package built the users guide during the build, which pulled in TeXLive during the build, which was not welcome on hydra. But at least today, it seems that this is no longer the case (the build script says “Documentation cannot be rebuilt without pdflatex” and the final package does not contain the users guide).
So I guess we can easily build lhs2tex again.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/which-packages-make-it-to-the-hydra-jobset/27465/5

I'm not aware of any problem with using texlive on Hydra.  The old commit is shortly before texlive got split from huge derivations to tiny ones (https://github.com/NixOS/nixpkgs/commit/0fdb93864ee81056238b6bdd368b5db23e8ef2cf) so maybe before then it was being avoided on Hydra?  I don't recall.

Cool. In that case, I'll maybe have a look to actually build and ship the documentation again.

Please make sure your commit message complies with CONTRIBUTING.md. Otherwise no issues.

Which part of the message needs refining? (pkgs.lhs2tex already exists right now, just not on hydra, so it’s not an init at … commit and borg’s “Package: new” is misleading, if that’s what you mean.)

Perhapslhs2tex: distribute again would suffice?

Well, let me solve this by the squash-merge button.

I love the squash merge button :-D


No default role for new design

A new design has no default role making it so it cannot be reopened since fails best practice.  Needs a default role of Engineered Region.

Added this default role to new designs.

Did you commit this fix to master?

Sorry not committed yet.

On Nov 26, 2016, at 11:52 PM, Michael Zhang<EMAIL_ADDRESS>wrote:
Did you commit this fix to master?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/SynBioDex/SBOLDesigner/issues/100#issuecomment-263093093, or mute the thread https://github.com/notifications/unsubscribe-auth/ADWD954uO1yxyJLEhmNeoNZ9paLR9BF4ks5rCMY6gaJpZM4K9ArM.


Follow nested import statements

That would be neat but I bet, not easy to implement.

Yeah, this would be really nice, but I don't have time to implement right now. Happy to accept a PR