INITGROUPS fails on GIDs ><PHONE_NUMBER>

User max is member of a group with GID<PHONE_NUMBER>.
Running id max, nsncd (unstable-2022-11-14 from nixpkgs 96f8f4a038a190f7511da79ef7e77bec5e4b811a) returns:
uid=646000001(max) gid=646000001(max) groups=646000001

nscd returns:
uid=646000001(max) gid=646000001(max) groups=646000001(max),2709991565(infra_admins)

Log for nsncd shows this error message:
Jun 11 00:00:39.334 ERRO error handling request, err: out of range integral type conversion attempted, request_type: INITGROUPS, thread: worker_5

Users with GIDs <=<PHONE_NUMBER> get their groups listed correctly, so I'm guessing this is a signed vs. unsigned issue.

This is a fork of https://github.com/twosigma/nsncd (with the intention to upstream), and doesn't have any changes regarding initgroups. Please open the issue there.


Disable matchers that run on PRs and raise warnings about unchanged files

The "Files Changed" tab on every GitHub PR is now showing eslint warnings about files not changed, and this is annoying.
There are quite a lot of TODOs and this this distracts from being able to see only what the PR changed.

Gosh yes- this bit is super infuriating.

I think I might have disabled the eslint matchers by using the trick here:
https://dev.to/sibiraj/disable-annotations-in-github-actions-2ih2


need  whereBetweenJoin()...!!

missing and urgent whereBetweenJoin.. thanks in advance

for now you can combine "where >=" and "where <="


[Enhancement] Reduce some meaningless log in FE

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #SR-14355
Problem Summary(Required) ：

This is first PR to reduce logs.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

@mergify backport branch-2.5


Filtering src recursively when creating a derivation

I'm creating a derivation using items in a big folder, but I only actually want to use a few files to create the derivation. These files are nested in various directories. So I thought I could use sourceByRegex to select those files. But unfortunately sourceByRegex only seems to search through the top level of a folder.
For example:
$ tree
.
├── 1
├── 2
├── 3
├── default.nix
├── folder
│   ├── 1
│   └── 2
└── result -> /nix/store/951h922h1cx55703yk5fqj1l059d7kvg-my-drv

2 directories, 6 files

$ cat default.nix
{pkgs ? import <nixpkgs> {}}:
pkgs.stdenv.mkDerivation {
  name = "my-drv";
  src = pkgs.lib.sources.sourceByRegex ./. ["2" "folder\/.*"];
  installPhase = ''
    set -e
    mkdir -p $out/bin
    cp -r $src/* $out/bin/
    '';
}

$ ls /nix/store/951h922h1cx55703yk5fqj1l059d7kvg-my-drv/bin
2

I hoped to see the both the file 2 and also folder in that last output, but only 2 was copied.
Is there another way to achieve this?

Check out https://github.com/NixOS/nixpkgs/pull/56985, which adds a function to make this pretty easy


Proxying with puma seems to be unreliable even with non-chunked encoding

While creating a testcase for bug #6 I ended up finding an unreliable result with proxying non-chunked content. Here's the testcase:
https://github.com/pedrocr/camerasink/blob/b01aac64e8aaef96f8527f6e29768cc1f5f6199b/testcases/puma_unreliable_proxying.ru
Again this is runnable as "rackup puma_unreliable_proxying.ru".
Then asking for the same resource multiple times from the original server works fine:
$ curl http://localhost:9000
Hello!
pedrocr@wintermute:~$ curl http://localhost:9000
Hello!
pedrocr@wintermute:~$ curl http://localhost:9000
Hello!
pedrocr@wintermute:~$ curl http://localhost:9000
Hello!
pedrocr@wintermute:~$ curl http://localhost:9000
Hello!

But doing the same from the proxy server sometimes returns an empty result:
pedrocr@wintermute:~$ curl http://localhost:9292
Hello!
pedrocr@wintermute:~$ curl http://localhost:9292
pedrocr@wintermute:~$ curl http://localhost:9292
Hello!

I was just curious, so I wrote a test for your test.
# puma_unreliable_proxying_test.rb
cnt = 0
1000.times do
  str = `curl http://localhost:9292 --silent`
  cnt += 1 if str == "Hello!\n"
end

puts cnt

When I commented the puma require and WEBrick was used, 1000/1000 requests were proxied each time.
When using puma, between 970 and 980 requests would be proxied, and the remainder returned an empty string.
I did two 10k runs. They proxied 9741 and 9734 requests.
puma version 2.11.3
rack-streaming-proxy 2.0.1


docs: add CONNECT-UDP documentation.

Commit Message: Added a section for CONNECT-UDP in the HTTP upgrades document. Added example configuration files for CONNECT-UDP.
Additional Description:
Risk Level: Low, this is a document-only change.
Testing: N/A
Docs Changes: Added an explanation and example configurations for CONNECT-UDP in the HTTP upgrades document.
Release Notes: N/A
Platform Specific Features: N/A

/assign @DavidSchinazi
Can you take a look at this CONNECT-UDP documentation, David? Thank you.

/retest

Thank you all! I found that there is a small config bug that makes the test fail for the config files.
I wrote a PR (#29612) to fix this. I will re-run the CI once the PR is merged.

/retest


Confusing links on "Priority unstudied genes" page

There are some links in the paragraph at the top of this page that are confusing:
http://preview.pombase.org/status/priority-unstudied-genes
When you click it goes to the query builder and the query is at the top of the history.  Should the link go to a results page instead?
The "Use these genes in query..." link on the term_genes page is confusing too if you don't know what to expect:
http://preview.pombase.org/term_genes/GO:0000070

Should the link go to a results page instead?

Yes, good point!

Should the link go to a results page instead?

I've changed the links in the text on the "Priority unstudied genes" page to go directly to the results.  But it's a bit odd on the results page because the "Back" button doesn't go to the "Priority unstudied genes" page, it goes to the query builder.  What should we do?

The "Use these genes in query..." link on the term_genes page is confusing too if you don't know what to expect

I find it confusing because it's not obvious that you have added a query to the history.  Maybe we should highlight the most recent addition to the history?  Any other ideas?

It is a bit odd but not a show stopper.
let's discuss on call....

Last week I talked with Midori about the general problem of linking to the query builder and to query results pages.  I've made some improvements so that we can link to either from the documentation.
The queries links on the priority unstudied genes page are defined in the "predefined_queries" section in the config file (pombe-embl/website/pombase_v2_config.json in SVN).  I think that's a good way to do it because all of the query definitions are then in one place and can be referred to by name.
There is an HTML like syntax for adding these inline links to predefined queries.  It's like this:
<app-query-link [goToResults]="true" [linkText]="'List of unknown genes'"
    [predefinedQueryName]="'priority-unstudied-genes:unknown_genes'">
</app-query-link>

Sorry it's so verbose.  That can all be on one line if that's tidier.
These links can appear anywhere in the Markdown documentation including in the panels on the front page.
If the goToResults attribute is set to "false" the link will go to the query builder instead.
The linkText attribute is optional.  If it's not specified the result count will be used as the link text.  That's what we do on the priority unstudied genes page.
The predefinedQueryName attribute is required and must match an entry in the "predefined_queries" section of the config file.
Unfortunately value of the linkText and predefinedQueryName attribute need to be doubly-quoted, there have to be two nested sets of matching quotes, one double-quote and one single quote, like:
"'List of unknown genes'"

It's easy to forgot and the page won't load if it's forgotten.  :-/
Annoyingly, the goToResults attribute shouldn't be doubly-quoted, so eg. [goToResults]="false"

I've added the text from my comment to the wiki: https://github.com/pombase/website/wiki/Links-to-the-query-builder

It is a bit odd but not a show stopper.
let's discuss on call....

We should discuss what to do about the "Back" button.

Is it possible that, if you do not come to the query builder results via a direct search, or your history, you see only a link to the “main query builder page”
Need to discuss Thurs


Nikhil-Jimmy-Thomas-Introduction

Notes for Reviewers
This PR fixes #
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 4614617b134b475591adf915d8a5a6fe1b593b03 at: https://66fe9b6144535999b8920360--meshery-docs-preview.netlify.app

Thank you 😎 @NikhilJ2005


Sorted Columns

Summary of problem or feature request
Needed sorted columns
Code snippet of problem
private function compileTableHeaders()
    {
        $th = [];
        foreach ($this->collection->toArray() as $row) {
            $thAttr = $this->html->attributes(array_merge(
                array_only($row, ['class', 'id', 'width', 'style', 'data-class', 'data-hide']),
                $row['attributes']
            ));
            $th[] = '' . $row['title'] . '	';
        }

        return $th;
    }

System details
We could add a sortBy in the foreach in line 587 of Builder file
private function compileTableHeaders()
    {
        $th = [];
        foreach ($this->collection->sortBy('order')->toArray() as $row) {
            $thAttr = $this->html->attributes(array_merge(
                array_only($row, ['class', 'id', 'width', 'style', 'data-class', 'data-hide']),
                $row['attributes']
            ));
            $th[] = '' . $row['title'] . '	';
        }

        return $th;
    }

Sorry for late reply. Please do not hesitate to submit if you still need. Thanks!

Hi, it's only a proposal, it could be interesting sort columns before render.

Sort before render, I think you just need orders option?
->orders([[1, 'desc'], [2, 'asc']])
->orderBy(3, 'desc')

It's a proposal about columns order, not rows or data.


LangVersion support for F#

It is already implemented for C# and VisualBasic, while F# has this feature as well.

@cartermp can you take a look at this feedback?

This is going to just be a part of the set of docs I'll update/add when F# 4.7 ships, since it's not in any released F# today.

Closing this feedback then since the feature hasn't shipped yet. Thanks!

Oh, I understand. You wrote it like it has already landed, should have read the release notes.
Thanks :)


Add option to only open external links in new window

I was thinking it might be nice to add an option like openExternalLinksInNewWindow
and then use a filter something like this (hopefully better, just whipped this up)
    converter.addExtension(function () {
        return [{
            type: 'output',
            regex: /<a\shref.+">/g,
            replace : function (text) {
                var url = text.match(/"(.*?)"/)[1]
                if(url.includes(window.location.hostname)){
                    return text
                }
                return '<a href="' + url + '" target="_blank">'
            }
        }]
    }, 'externalLink')

@oscarmorrison thank you for your suggestion but... There's already an option to open links in external windows 😅 🤣  openLinksInNewWindow
or... do you mean opening ONLY links that point to external URLs in a new window?
If it's the second, unfortunately that cannot be put into showdown since it doesn't make sense in nodejs environment (and the 'window' object is not available in Node).
But as you rightly pointed out, that can easily be achieved with an output extension (with the side bonus of said extension being able to also change tags that the user wrote in HTML in the original document).

@oscarmorrison Did you manage to find a solution for your problem?

I did thankyou @tivie
https://github.com/oscarmorrison/md-page/blob/master/script.js#L33

I found that openLinksInNewWindow: true does not work along with simplifiedAutoLink: true The latter overrides the former, and links are shown in current window itself


Adding Basic Enemy prefab into dev

merge conflicts look like they are just meta files which shouldn't be a problem.

Thanks!


Migration to AndroidX and many more changes

Hallo everyone, first I want to thank all of the contributers for their hard work to this project.
I have updated the whole project to the new level.
261 changed lines in 236 files.
Kore changes

updated to API level 16 to get all these new Google features
migrating Project to AndroidX
compiled with Android-R sdk
Removed Butterknife (deprecated/no longer mainteined) and moved to View Binding recomended by Google
Replaced Picasso by Glide
Updated all libs to the recent versions
Pictures were converted to webp to save space
Remote Icons were converted to Vector drawables
Launcher icon is now a vector drawable available for Api level 16+
German translation is now in sync

The Kore app lost 11 - 7 = 3mb in size and is now state of the art.
I have not tested everything.
But i noticed in the notification above the remote control the Channel name is not displayed anymore.
I did not found out the error.
And the remote buttons are big now due to vector drawable migration.
I couldnt find out how to change the size.
But it could be an improvement for people with vision incapabilities.
Hope I could contribute to the project.

Thank you. This is really nice!
But in my opinion, as this contains many changes that can break a lot of things I really urge you to issue separate PRs for this. Making the PRs smaller allows us to discuss the changes and makes it easier to merge them. Some can probably be merged immediately. My suggestion is to issue separate PRs for each of the following changes:

migrating Project to AndroidX
compiled with Android-R sdk
Removed Butterknife (deprecated/no longer mainteined) and moved to View Binding recommended by Google
Updated drawables (can go in a single PR)

Pictures were converted to webp to save space
Remote Icons were converted to Vector drawables
Launcher icon is now a vector drawable available for Api level 16

German translation is now in sync

I'm not so sure about the following:

updated to API level 16 to get all these new Google features. What new google features are you talking about?
Updated all libs to the recent versions. Why do these need to be updated? Do you use new APIs, are there bugs that need to be fixed?
Replaced Picasso by Glide. Why do you want Glide?

Thanks for your reply.
Unfortunately i havent commited after every step.
The changes after the Androidx (libs+xml files) migration were huge and I thought I should also update everything else.
And I saw that Kore has a Beta Program on Google Play Store because of that I didnt find the urge to push everything separately. Neither would I expect any immediate push to the master branch.
I thought rather of an Beta branch on GitHub.

I needed API level 16 for the 'expandabletextview" lib and the vector drawable support. And some other google libraries whicht I dont remember. You are right it is not quite a feature but a support.
There were no flaws but I just like everything up to date. :)
Glide is better supported than Picasso (Updates and Userbase).
It is also used in Googles Android programming courses which are held by their Developer Advocates.

First of all, thanks for the contribution. It's useful and it contains stuff that needed to be done.
Nevertheless, i share @poisdeux concern's, regarding the scope of this PR. It's quite difficult to review, comment and test it as is, and it introduces a level of risk that could be much better managed if it was done in separate, smaller and more focused bits.
The separation that @poisdeux suggests seems good, but we are open to other that you feel is more appropriate. Taking the migration to AndroidX separately, for instance, would make a world of a difference IMO - that's a relatively straightforward change, relatively easy to review and test, and it constitutes a large portion of the code changes, so subsequent changes should be cleaner. Likewise for changes to assets (icons and such) and code. I might also have issues with using Glide instead of Picasso, that i would like to review and discuss separately, and that is impossible and confusing in this PR current form.
I hope you understand our point. We feel it's a valuable contribution that we want to get in, but it's an unmanageable risk merging it all at once (alpha/beta/whatever programs don't substitute code reviews - i prefer to prevent bugs than to fix them).

Thanks for you your reply.
Yes, I totally understand your point of view. Even though I have to do everything again. :D
I will start again in smaller chuncks like poisdeux said above.
It will now take longer because I now dont have much time to work on side projects.

Guess this can be closed, as it is being done in separate PRs. If not, comment.


Use name ranges instead of decl ranges for node IDs. [phab:D1834]

Created by zarko at 2017-08-04 00:01:42:
Some Clang decls, like FunctionDecl, report unexpected source
ranges. For instance:
bool foo();
may report a macro location on its left-hand side if bool is
defined by stdbool.h. This is a problem if foo is declared
in a header that is included in different preprocessor contexts.
It will commonly result in multiple disjoint foos in the graph.
Instead of using the whole decl range, we'll instead just use the
decl's name range, which avoids this issue (on both sides of the
range).
Objective-C interfaces are excluded because of some internal
Clang strangeness, possibly alluded to by #1486.

salguarnieri wrote at 2017-08-04 01:58:58:
Interesting. I would not have expected objcinterfacedecls to be special here. They don't have a return type before the name, so perhaps that is enough to preclude the problem.

fromberger accepted this revision at 2017-08-04 02:56:33

zarko closed via commit 1c36d3085280e3f864fba40acc9dd3e99505f0c6 at 2017-08-04 16:27:32


Declaration can't reference /Common/serverssl object

Do you already have an issue opened with F5 support?
Github Issues are consistently monitored by F5 staff, but should be considered as best effort only and you should not expect to receive the same level of response as provided by F5 Support. Please open an case with F5 if this is a critical issue.
ISSUE TYPE

Bug Report

AS3 BUILD/ VERSION

12.1.4
SUMMARY

Sample declaration mention "ServerTLS" / Common/serverssl profile, however I am getting response that component does not exist. All other objects in /Common like /Common/clientssl processed fine. Other server ssl profiles are not working as well
STEPS TO REPRODUCE
Run the declaration
{
	"class": "AS3",
	"action": "deploy",
	"persist": true,
	"declaration": {
		"class": "ADC",
		"schemaVersion": "3.9.0",
		"id": "Export_Date:_03-20-2019_09:32:12",
		"target": {
			"hostname": "<IP_ADDRESS>"
		},
		"AS3_Exports": {
			"class": "Tenant",
			"defaultRouteDomain": 0,
			"test1": {
				"class": "Application",
				"template": "https",
				"serviceMain": {
					"class": "Service_HTTPS",
					"remark": "test-1 - none",
					"virtualPort": 443,
					"virtualAddresses": ["<IP_ADDRESS>"],
					"redirect80": false,
					"pool": "test",
					"profileTCP": {
						"egress": "wan",
						"ingress": {
							"bigip": "/Common/tcp"
						}
					},
					"profileHTTP": {
						"bigip": "/Common/http"
					},
					"persistenceMethods": [],
					"clientTLS": {
						"bigip": "/Common/clientssl"
					},
					"serverTLS": {
						"bigip": "/Common/serverssl"
					},
					"snat": "none",
					"iRules": [],
					"policyWAF": {
						"bigip": "/Common/policy"
					},
					"securityLogProfiles": [{
							"bigip": "/Common/none"
						}
					]
				},
				"test": {
					"class": "Pool",
					"monitors": ["icmp"],
					"reselectTries": 0,
					"loadBalancingMode": "round-robin",
					"serviceDownAction": "none",
					"members": [{
							"servicePort": 3,
							"serverAddresses": ["<IP_ADDRESS>"]
						}
					]
				}
			}
		}
	}
}

EXPECTED RESULTS

{
    "code": 422,
    "errors": [
        "/AS3_Exports/test1/serviceMain/serverTLS: requested component /Common/serverssl does not exist"
    ],
    "declarationFullId": "",
    "message": "declaration is invalid"
}

ACTUAL RESULTS
{
    "code": 422,
    "errors": [
        "/AS3_Exports/test1/serviceMain/serverTLS: requested component /Common/serverssl does not exist"
    ],
    "declarationFullId": "",
    "message": "declaration is invalid"
}

Hello bgekhtman,
You are running into a difference between naming conventions in AS3 and BIG-IP configuration that can be a little confusing.  Essentially a "TLS_Server" in AS3 creates a "Client SSL" profile on the BIG-IP system (TLS server parameters are for connections arriving to BIG-IP).  TLS_Client creates a Server SSL profile (TLS client parameters are for connections leaving the BIG-IP).
While this is briefly mentioned in the documentation in the examples section and in the schema reference (ie https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/latest/refguide/schema-reference.html?highlight=clienttls#service-http-clienttls), it is not very clear.  We will update the documentation to better explain these differences.
In the example declaration you posted above, if you swap ClientTLS and ServerTLS, your declaration will succeed.

Documentation update task tracked internally as 958.

Hello @bgekhtman, we added an FAQ entry for this issue: https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/latest/userguide/faq.html#tlsserver to help clarify this.  Going to close this issue, thanks for bringing it to our attention.


not found FAPanelController

Your module is very good. I think this is my fail. Please help me.
Thank you.

@Kitty555
You forgot to import FAPanel in the App Delegate file. Please try to add import line on top and then build the project.


error: SyntaxError<EMAIL_ADDRESS>Unexpected token, expected ")" (120:16)

Getting this compilation error with the latest version.
error: SyntaxError<EMAIL_ADDRESS>Unexpected token, expected ")" (120:16)

Hi @alexkendall, there’s no “)” on line 120 that could cause syntax error.
besides that, it’s bot possible for us to ship bad syntax on npm since we build the code prior publishing.
Can you please make sure you haven’t touched anything on that file? (also check if you have a patch-package file for this module)

@alexkendall closing the issue, if you’re still getting error let us know.


Opaque error message on array argument with non-native byte order

I understand that non-native byte order arrays can or should not be accepted as arguments to functions compiled with numba.jit. However the current error message when doing so is not as informative as it can be in my opinion. See the following interactive session.
>>> import numpy, numba
>>> a = numpy.arange(1, dtype=float)
>>> b = a.newbyteorder()
>>> a.dtype.byteorder
'='
>>> b.dtype.byteorder
'>'
>>> @numba.jit(numba.void(numba.float64[:]))
... def f(v): pass
...
>>> f(a)
>>> f(b)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Conda\lib\site-packages\numba\dispatcher.py", line 399, in _explain_matching_error
    raise TypeError(msg)
TypeError: No matching definition for argument type(s) pyobject

The type pyobject is obviously very generic here and rather confusing, especially since numpy itself is quite cool with mixing byte orders. For example in the repr() output of the arrays nothing about byte order is included.
>>> repr(a)
'array([ 0.])'
>>> repr(b)
'array([ 0.])'

Also note that in the typeof call stack this is actually asserted properly, but the message gets silenced somewhere.
>>> numba.typing.typeof.typeof(b)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Conda\lib\site-packages\numba\typing\typeof.py", line 28, in typeof
    ty = typeof_impl(val, c)
  File "C:\Conda\lib\site-packages\singledispatch.py", line 210, in wrapper
    return dispatch(args[0].__class__)(*args, **kw)
  File "C:\Conda\lib\site-packages\numba\typing\typeof.py", line 195, in _typeo_ndarray
    raise ValueError("Unsupported array dtype: %s" % (val.dtype,))
ValueError: Unsupported array dtype: >f8

Thanks for the report. I believe what you are seeing is valid behaviour. Upfront declaring that the function will have a given signature means it forgoes the ability to compile new signatures on the fly. As a result anything that doesn't match the provided signature will be rejected, I believe these are simply matching errors and not a matter of "support" (although you are right about byte order support). For example:
f(10)

gives:
TypeError: No matching definition for argument type(s) int64

The compiler recognises 10 as an int64 type and so can state that matching is invalid using the type information it has derived in the message. Whereas:
class bar:
    pass
f(bar)

gives:
TypeError: No matching definition for argument type(s) pyobject

as the compiler has no idea what a bar is. This is similar to the case presented above in that the compiler tries to see the type of the variable, and a ValueError is raised as it doesn't know, but the foremost problem is that it can't match it to the given signature. This probably needs a chained exception so that it is noted that the TypeError is caused by the ValueError, however this may be easier said than done as I'm fairly sure at present part of the no python mode -> object mode fallback relies on PyObjects being identified in an exception free manner.


More details on hovering coverage indicators

hover to see List of tests, status, errors
Ability to select a given test to rerun/debug

Fixed with the above changes.


Anything like ONEAR/NEAR in reference to PHP?

I'm new to Vespa and am using the NEAR/ONEAR operators as a search metric similar to the slop parameter in Elasticsearch.
https://docs.vespa.ai/documentation/query-language.html#near-and-ordered-near
https://docs.vespa.ai/documentation/reference/rank-features.html
Is there anything like ONEAR or NEAR in reference to PHP?

What do you mean by  in reference to PHP?

Not sure but if you are asking if Vespa has a search api client written in the programming language PHP then the question is no. See https://docs.vespa.ai/documentation/search-api.html


Add your name to the list of Contributors (Mandatory)

Issue Description

Add your name to the contributor.xml located in the res/values folder.
Follow the guidelines written in the file itself to add your name.
After adding the required details, create a Pull Request.
PS: This issue is open for all, so no need to claim this issue. You can directly create a pull request

Happy Contributing!

claim

@anshgeez31  This is an open for all issue, no need to claim it.


RC.1 Error setting up entry Frigate NVR for frigate Traceback - ValueError: Invalid entity ID

Version of the custom_component
v3.0.0-rc.1
Configuration
Home Assistant Core 2022.6.3
Frigate 0.11-beta2

Describe the bug
Upgraded in HACS to RC 1, HA restart , integration error?
Removed Frigate integration / HA restart / re-add Frigate integration... worked! Once. The new binary occupancy sensors were triggering on object detection in Frigate, but it soon stopped, and now still not working after a HA restart / remove integration / restart / re-added integration / multiple restarts and removes/re-adds. All Frigate entities show unknown.

Debug log
2022-06-07 22:27:59 INFO (MainThread) [custom_components.frigate]
-------------------------------------------------------------------
Frigate
Integration Version: 3.0.0-rc.1
This is a custom integration!
If you have any issues with this you need to open an issue here:
https://github.com/blakeblackshear/frigate-hass-integration/issues
-------------------------------------------------------------------
2022-06-07 22:28:00 DEBUG (MainThread) [custom_components.frigate] Finished fetching frigate data in 0.218 seconds (success: True)
2022-06-07 22:28:00 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Frigate NVR for frigate
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 339, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/frigate/__init__.py", line 273, in async_setup_entry
entity_registry.async_update_entity(
File "/usr/src/homeassistant/homeassistant/helpers/entity_registry.py", line 626, in async_update_entity
return self._async_update_entity(
File "/usr/src/homeassistant/homeassistant/helpers/entity_registry.py", line 561, in _async_update_entity
raise ValueError("Invalid entity ID")
ValueError: Invalid entity ID

Quick check: do you have a camera name with unusual characters in it? (Including hyphen?). If yes, try renaming the camera and see if it works?

No, all camera names are single words, with a capital first letter. i.e Driveway, Balcony, Deck, etc

It's the capital letter. Here is the regexp that HA uses:
VALID_ENTITY_ID = re.compile(r"^(?!.+__)(?!_)[\da-z_]+(?<!_)\.(?!_)[\da-z_]+(?<!_)$")

Clearly this is not something the user should need about -- i.e. this is a bug that I will fix. In the meantime, if you go with just one word lowercase camera names it should work fine.

@jasonpstokes Try the just released v3.0.0-rc.2 and let me know how it goes.

Yes rc.2 working. Thank you for the quick fix!


Update calibration methods, add tests.

MBAR calibration currently not working because it depends on context in the initialization. Will fix this at another time.

MBAR calibration currently not working because it depends on context in the initialization

Can you elaborate?

Sure. The calibration class that uses MBAR currently uses context as an argument in it's __init__ function. However, since we now use a compound integrator, and generate a context using that integrator, we have to define the Titration object first before we have a context.
I may be able to move this so context isn't needed in __init__. I could potentially merge it into the other calibration class.

Here is the test case that shows the issue:
https://github.com/choderalab/openmm-constph/blob/master/constph/tests/test_explicit.py#L132-L137

Ah, so you just need to fix up the API.
Why not just replace context with system, and take an optional platform argument?

For the sake of unifying the APIs, I think with a little bit of work, we can include an mbar option as an alternative to eq9 and eq12, as opposed to fixing the MBAR class with a separate api.
The one thing in which the classes fundamentally differ -- which will be the part that takes some work --is storing all the previous states. Right now MBarCalibrationTitration use this adaptation_tracker dictionary entry to keep track of all past samples. Its __init__ requires the context because I need to extract the potential for the initial state. I would have to incorporate that into the CalibrationTitration code.
It'll take some time, and for now it isn't high priority, since we can already use SAMS on-line. MBAR was just an interesting at-line alternative, and we could just run MBAR off-line if we wanted to. Not fixing this right now shouldn't hold up our development.

Added reminder to #20.

You've probably already done this, but have you compared the free energies you get with SAMS with the free energies you get with MBAR? This is an important validation test for the SAMS code.

@gregoryross Very qualitatively yes, they seemed to match up, but I'd like to set up a test suite/example set that proves this.

Added reminder for that to #20.


Update README.md from branch example

Updates
I updated readme file. Please review and accept.

##Thoughts
I probably shouldn't, but okay. 👍


replace logo

we have old app image (app store / playmarket)
Now: triangle
Should be: omi

https://github.com/BasedHardware/omi/pull/1241


Initial checkin for java debugging.

@jeffreyCline,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

Thanks @jeffreyCline for the good work, really appreciate you taking a look at this.
Removed most of the switch statement code to try to keep the cli code language agnostic. Added both launch configurations as defaults. Passed IConfiguration style arguments to the IConfigurationBuilder, which allows the LanguageWorkerConfigs in the host to consume the configuration.
In order to specify debug, you now need to call:
func host start /worker:java:debug=(true || port#)
func host start /workers:node:inspect=(true || port#)
We could normalize these options (perhaps add debug to node as well as inspect with the same behavior).
It might be nice to provide backwards compatibility with the previous node debug experience.
I'm working on a host update to consume IConfiguration in the language workers, see here:
https://github.com/mamaso/azure-webjobs-sdk-script/commit/ce2936c4b8d82cc1c59f2367decd228beaa5748d

@jeffreyCline, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

closed in #247


Support responsive images

The photos currently hosted on the site are currently rather large (~4MP). Support for responsive images should be added to improve mobile performance.
There is a Jekyll plugin to support responsive images.

An alternative CDN for hosting images will also need to found since the Dropbox public folders are deprecated and no longer seem to be functioning.
Using Google Drive may be a good alternative since this will also generate the thumbnail images.


Support aarch64 sendmmsg, recvmmsg and YIELD

Expected behaviour
Equivalent network performance on aarch64
Actual behaviour
Slower than I was hoping when under load.
We deploy a high network throughput application using Netty w/Epoll on x86. I’ve been testing out arm/aarch64 as a possible replacement as we’ve seen great results with some of our compute intensive applications. Sadly, I’ve been having some performance issues in doing so and slowly been digging into why this might be. About 80% of our CPU time is in Netty land regardless of platform, and workloads are identical.
We're see higher rescheduling interrupts (roughly 8x) and much higher CPU time spent in __softirqentry_text_start, __wake_up_common_lock and arch_local_irq_enable on aarch64 than x86. We also see overall higher p95,p99 latencies once we start getting up to high CPU usage.
This has led me to https://github.com/netty/netty/blob/4.1/transport-native-epoll/src/main/c/netty_epoll_native.c
I noticed that netty_epoll_native.c defines SYS_recvmmsg and SYS_sendmmsg for x86, but not aarch64
https://github.com/netty/netty/blob/4.1/transport-native-epoll/src/main/c/netty_epoll_native.c#L98
It also adds an asm pause - when compiled for x86 in cpu_relax()
https://github.com/netty/netty/blob/4.1/transport-native-epoll/src/main/c/netty_epoll_native.c#L295
I’m barely c and asm literate, but at this surface this seems just like a case of adding a few blocks of code
#elif defined(__aarch64__)
// See https://thog.github.io/syscalls-table-aarch64/latest.html
#define SYS_recvmmsg 365
....
#elif defined(__aarch64__)
// See https://thog.github.io/syscalls-table-aarch64/latest.html
#define SYS_sendmmsg 296

And maybe
#if defined(__aarch64__)
    asm volatile("yield\n": : :"memory");
#endif

I haven't tested this yet or even written anything that compiles. Literally only checked out the Netty code a few hours ago.
What do you think the odds of this helping/solving my issues are? Is anyone already looking at doing this? or experienced similar issues?
Thanks
Netty version
4.1.65.Final
JVM version (e.g. java -version)
openjdk version "11.0.11" 2021-04-20 LTS
OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (build 11.0.11+9-LTS)
OpenJDK 64-Bit Server VM Corretto-<IP_ADDRESS>.1 (build 11.0.11+9-LTS, mixed mode)
OS version (e.g. uname -a)
Linux 5.10.35-31.135.amzn2.aarch64 #1 SMP Tue May 11 22:05:38 UTC 2021 aarch64 aarch64 aarch64 GNU/Linux

@ChristopherWirt this is a good catch... You want to open a PR to add:
#elif defined(__aarch64__)
// See https://thog.github.io/syscalls-table-aarch64/latest.html
#define SYS_recvmmsg 365
....
#elif defined(__aarch64__)
// See https://thog.github.io/syscalls-table-aarch64/latest.html
#define SYS_sendmmsg 296

?
I would be happy to review it.
I would do this first and then look into the volatile as a followup

Will do. Thanks

With the SYS_sendmsg i don't believe the code would work at all if they weren't defined. I'm guessing these existed on x86 to enable building on platforms with very old libc/headers versions that didn't define these constants. Given the age of the system calls and that they've probably been around before 64b Arm was defined these defined aren't needed.
The change in spin behavior might improve performance here, however with a yield it won't help. On most Arm processors a yield is a nop while pause tends to cause a 30-100 cycle delay. We've found isb on Arm ends up producing delays that are closer to pause x86 in other projects (e.g. MySQL), so there is a chance this will improve the rescheduling behavior you see.
There are also similar changes that have just went into Corretto here and here that will be included in the next version of Corretto released in July.

Actually @AGSaidi is right ... @ChristopherWirt so this can't be the reason really

@ChristopherWirt is there a small reproducer you see the problem with you can share?

With the SYS_sendmsg i don't believe the code would work at all if they weren't defined.

@AGSaidi Do you mean that without SYS_sendmsg defined for aarch64, it's fine because it does the right thing anyway? we just spend a little longer in Javacode and call a different Socket method? So unlikely the cause of my troubles. Which I think makes sense.
RE: the explicit pause in cpu_relax() - It seems like it's still worth trying 'isb' still?
I don't have a reproducer at the moment.

With the SYS_sendmsg i don't believe the code would work at all if they weren't defined.

@AGSaidi Do you mean that without SYS_sendmsg explicitly defined for aarch64, it's fine because it does the right thing anyway? we just spend a little longer in Javacode and call a different Socket method? So unlikely the cause of my troubles. Which I think makes sense.

Yes even without this definition it should do the correct thing.

RE: the explicit pause in cpu_relax() - It seems like it's still worth trying 'isb' still?
I don't have a reproducer at the moment.

RE: the explicit pause in cpu_relax() - It seems like it's still worth trying 'isb' still?

Yes, it might make a difference.

That said this is only for "fast poll" which is not used by default so I think that is not what affect you.

Any updates here ?


Ping reducer

Added a simple scoreboard ping reducer.
It's not really good and only reduces the ping by ~20-30ms

Don't merge, will change this later


fixed docstring

fixed docstring for partitioning functions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3b1a807) 56.25% compared to head (ccf893f) 56.24%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3643      +/-   ##
==========================================
- Coverage   56.25%   56.24%   -0.01%
==========================================
  Files         101      101
  Lines       12177    12172       -5
==========================================
- Hits         6850     6846       -4
+ Misses       5327     5326       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


When starting rancher-agent: FATA[0002] Error response from daemon: Cannot destroy container

When you start the rancher-agent container, you see the following error:
FATA[0002] Error response from daemon: Cannot destroy container 446f0351cf1f378a6fc8d18fb220b453cb1e27812637e59bc337268ce4dfec05: Driver aufs failed to remove root filesystem 446f0351cf1f378a6fc8d18fb220b453cb1e27812637e59bc337268ce4dfec05: rename /mnt/sda1/var/lib/docker/aufs/mnt/446f0351cf1f378a6fc8d18fb220b453cb1e27812637e59bc337268ce4dfec05 /mnt/sda1/var/lib/docker/aufs/mnt/446f0351cf1f378a6fc8d18fb220b453cb1e27812637e59bc337268ce4dfec05-removing: device or resource busy

With different directory names, but you always get that error.
The error is misleading because the container ultimately starts fine.
Note that this is different than #26 but I suspect the two may be related.

$ docker info
Containers: 1
Images: 47
Storage Driver: aufs
Root Dir: /mnt/sda1/var/lib/docker/aufs
Dirs: 61
Execution Driver: native-0.2
Kernel Version: 3.16.4-tinycore64
Operating System: Boot2Docker 1.2.0 (TCL 5.4); master : 8a06c1f - Fri Nov 28 17:03:52 UTC 2014
CPUs: 8
Total Memory: 999.6 MiB
Name: boot2docker
ID: 3WJU:FUAI:G4QO:GPOQ:BHBV:NAII:MRO2:NMPG:MNWI:L5B2:SWXX:RVVM
Debug mode (server): true
Debug mode (client): false
Fds: 18
Goroutines: 18
EventsListeners: 0
Init Path: /usr/local/bin/docker
Username: ~redacted~
Registry: [https://index.docker.io/v1/]

$ docker version
Client version: 1.3.1-dev
Client API version: 1.16
Go version (client): go1.3.3
Git commit (client): 831d09f
OS/Arch (client): darwin/amd64
Server version: 1.3.1-dev
Server API version: 1.16
Go version (server): go1.3.3
Git commit (server): 831d09f

docker images rancher/agent
REPOSITORY          TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
rancher/agent       latest              61678f70cabd        2 weeks ago         271.2 MB

I'm having the same problem, with both v0.2.0 and v0.2.1.
$ docker info
Containers: 2
Images: 28
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Dirs: 40
Execution Driver: native-0.2
Kernel Version: 3.13.0-32-generic
Operating System: Ubuntu 14.04.1 LTS
CPUs: 4
Total Memory: 15.64 GiB
Name: machine
ID: M8VM:XHEO:5P5J:WTKX:IOWG:WEQQ:AXHQ:K5VH:OQI4:CDMB:DALR:Y6CO
WARNING: No swap limit support
$ docker version
Client version: 1.4.1
Client API version: 1.16
Go version (client): go1.3.3
Git commit (client): 5bc2ff8
OS/Arch (client): linux/amd64
Server version: 1.4.1
Server API version: 1.16
Go version (server): go1.3.3
Git commit (server): 5bc2ff8

@Rjvs Sorry for delayed response! If you are having other problems, they may be different and this one may be just a red herring.
Note that the container that controls networking (will be using the rancher/agent-instance image) is not started until you start your first user container via the Rancher UI/API that uses the "Managed" networking option.
Does the rancher-agent container stay up and running or does it keep restarting? If it stays up and running, it is likely configured and running correctly.

By using the command in the UI (per #267), this warning no longer shows up.
@Rjvs If you continue to have issues after trying the command from the UI, please feel free to open another issue.


Entra ID support for dotnet-core quickstart

Purpose

...

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Other Information

@yugangw-msft @philon-msft please help review and approve


ICEORYX error! POPO__CHUNK_LOCKING_ERROR

Required information
Operating system:
E.g. Ubuntu 20.04 LTS
Compiler version:
E.g. GCC 7.4.0
Observed result or behaviour:
Iox-roudi publisher application exits saying popo chunk locking error
Expected result or behaviour:
What do you expect to happen?
publisher continues operation without exiting
Conditions where it occurred / Performed steps:
Additional helpful information
ICEORYX error! POPO__CHUNK_LOCKING_ERROR**

Can you give us more infos, like the iceoryx version, OS and your setup?
My guess is that there are two processes which publish to the same topic, right?
I would also guess that one of these two process crashed or was killed with kill -9 ... I'm not sure if ctrl+c could also lead to this scenario if there is no signal handler to shut the application down gracefully.
Are my assumptions correct?

we are using iceoryx-version v2.0.3. OS is custom LMP OS(but consider it close to ubuntu).
SetUp: Roudi daemon is started and then I start the publisher application. It runs and sometimes it throws the below error
navsensord[10963]: Informational: �[0;90m2024-02-11 19:00:04.330 �[0;1;31;103m[ Error ]�[m: ICEORYX error! POPO__CHUNK_LOCKING_ERROR
When this error is thrown my publisher application is exiting out as below
887490 2024/04/22 22:28:42.000000 70.2888 255 LNX SYS JOUR 689 log info verbose 5 2024/02/11 19:00:04.331012 70.217582 navsensord[10963]: Informational: terminate called without an active exception
887492 2024/04/22 22:28:42.000000 70.2888 1 LNX SYS JOUR 689 log warn verbose 5 2024/02/11 19:00:04.333680 70.221799 kernel: Warning: CPU: 2 PID: 10963 Comm: navsensord Tainted: G W OE 5.15.104 https://github.com/eclipse-iceoryx/iceoryx/issues/1
887515 2024/04/22 22:28:42.000000 70.5656 24 LNX SYS JOUR 689 log warn verbose 5 2024/02/11 19:00:04.539121 70.425722 systemd[1]: Warning: navsensord.service: Main process exited, code=dumped, status=6/ABRT
887516 2024/04/22 22:28:42.000000 70.5656 25 LNX SYS JOUR 689 log warn verbose 5 2024/02/11 19:00:04.539226 70.425953 systemd[1]: Warning: navsensord.service: Failed with result 'core-dump'.

We are not closing the publisher application by ourselves this popo chunk error occurs and the publisher app crashes on its own

Do you have more infos?
Which iceoryx version are you running? Which OS? What's the output of RouDi? What's your exact setup, e.g. which appications are connected in which way to the crashing application?

issue closed


Data from sensor.

Hi,
I m looking at your code.
where do you get the raw data from the sensor? I was looking at the code NFCReader.kt but I can't understand where you get the 32 samples and in which variable you are downloading them. Can you tell me this passage?

Sure, the API for the sensor is not so straight forward that I just request the values. What I do is dump the memory from the sensor and then parse it (which took quite some time to develop, by basically looking at a lot of data dumps):
The function readout: https://github.com/exoji2e/Sugartop/blob/master/app/src/main/java/io/exoji2e/sugartop/NFCReader.kt#L43
reads and returns the entire memory of the nfctag. It reads 8 bytes of data from the nfc memory for each 0x20 command, which is in the ISO-standard for nfc-communication (see page 30).
You can find the raw data of the last readings you did in the menu "Show Last Reading Raw".
The actual parsing of the memory into the 32 + 16 data points happens in RawParser.kt:
https://github.com/exoji2e/Sugartop/blob/master/app/src/main/java/io/exoji2e/sugartop/RawParser.kt, which is called from DataContainer.
One thing to note is that the sensor does not measure in a reasonable unit like mmol/L or mg/dL. It reports some values that linearly corresponds to your glucose level. So what I get for each measurement is an int in the interval [500 - 3000], which I then recalculate based on my own measurements. These parameters to transform from the int to your glucose level can be recalibrated by adding manual glucose measurements.

Hi, I tried to read a value log directly by doing
Log.d ("data", resp.contentToString ());
and I get about 41 arrays this way
[126, 113, 64, 19, 3, 0, 0, 0]
[0, 0, 0, 0, 0, 0, 0, 0]
[0, 0, 0, 0, 0, 0, 0, 0]
[11, -127, 10, 16, -20, 3, -56, 68]
[-100, -128, -13, 3, -56, 68, -100, -128]
[-19, 3, -56, 60, -100, -128, -11, 3]
and so on ...
from what you're telling me, these values represent the 8 bytes.
I saw the "Show Last Reading Raw" that has the values from 0 to 359 (but the real value is until 327).
I read u comment about (History and Recent, but I really don't understand what they mean).
I tried to log some in RawParser too, but when I read from NFC never call RawParser.kt functions.
Maybe I should look at the "append" function in DataContainer?
I'm trying to get that data after you've transformed it for use in some analysis

I see, yeah, the call to RawParser is in DataContainer.append
History and Recent are my names of the different measuring buffers the sensor uses.
Each minute the sensor measures and saves it to the circular buffer Recent, which holds measurements for the last 16 minutes.
Each 15th minute a measurement is saved to the History buffer, which holds 32 measurements spanning the last 8 hours.
If you want to extract the data the easiest way is to either export a csv-file of the raw measurements, or a sql3 db/dump of the entire app database. You can do that by pressing the Share icon in the upper right corner :)

I also tried that one, extracting a .csv, but I extract all the db. I need that every time I take a reading I only extract the 8 hour window.
For example I take a reading now (4 pm) and I take the window of the 8 hours before, values from 9am to 4 pm.
If you export your db now, you will get all the values before 4 pm, even those of the past days, for example yesterday.
After an NFC reading, you parse the data? Where do I find this window-parsed?

Here, in Datacontainer

Sorry again for the question. I understand your division between history and recent ones.
I still have a doubt, however, I have analyzed the recent ones (which are the last 16 minutes) but I have values in this format:
SensorChunk (value = 897, status = 200, history = false, rest = 15244672)
Obviously value I think it is the value of glucose (even if it does not correspond to the current one) while what would a timestamp be?
In all the detected values (both history and recent) I did not find the current value, did I still miss something?
Same thing in the story, how do you define the timestamp of those values?
Thank you so much for your support!

Yes, so timestamps are not set by the sensor, but implied from previous measurements, so them I set later in the function (the recent history_prepared and recent_prepared variables have all the newly found values.)
However they are in order, so the last one is the most recent measurement, which for now_recent means that it was taken somewhere between just now and one minute ago.
The default transformation from sensor_values to glucose_value (in mmol/L) is:
glucose_value =  0.00567 * sensor_value - 0.04605
if you want to get mg/dL you just multiply with 18.
This calibration can be changed by adding manual values. The code that applies the transformation can be found in SensorData.kt

This transformation is not perfect, it was just the mean over multiple sensors that I used. It varies quite a lot. Some sensors become really off when you're at a glucose level of 16-20 mmol/L. Usually the error is less than a unit away from the default calibration when the values are between 2-8 mmol/L for me. If you get other results it would be super interesting if you could share :)

You will certainly be happy to share my results with you and I hope to be able to contribute with this very interesting project.
The idea is precisely to analyze the data with recent techniques. Collect a lot of data from many sensors and make an analysis that also concerns a lifestyle. I'm doing this for my thesis that will cover digital therapies.
Is it possible that I lose data between the now_recent and now_history and the prepared data?
Correctly I see in now_recent (16 elements) and 32 in now_history,
but once I read the preparations, in history I see it empty (I don't understand why) and the recent ones are only 3 elements.
So the value must still be multiplied then? For example, working with mg / dLf
glucose_value = (0.00567 * sensor_value - 0.04605) * 18 to the value in the "value" object?

Yes, the history_prepared variable only contains values not yet saved to the database.
I think you options are:

set the timestamps yourself (similar to what is done in the prepare function)
after the values are inserted into the database you can do a query to the database via DataContainer.get. Note that this is a database function, so it is not allowed to be called from the UI-thread.

The reason the values are removed between now_history and history_prepared is because I don't want to save duplicates.

Closing for now, reopen if need more help/have results to share! :)


Missing Ficsmas items

What the tittle says. They aren't there.

Hello, yes seasonal content is filtered out. Only the "regular" items are present in the tool.

Is there any plan to add them?

No, I do not plan to add seasonal content to the tool


Discovery into AWS Amplify

As a developer, I want to look into how we can use Amplify so we can deploy our website using static hosting.

Successfully deployed: https://s2vx.com
See wiki page for notes: https://github.com/maxrchung/S2VX/wiki/Website-Deployment


avoid error when disposing an arc with hosted particles

this fixes bug number 3 on the list.

Scott and I were managing a separate doc with remote-planning work https://docs.google.com/document/d/1LSFhIe8C7CWuGQ1E8W3lMHcl9eG4Kid_LkAJ0nteYug/edit#


Multiple session factories with different options

Hi,
We are currently working on splitting our database into multiple 'shards'. Each shard will have it's own database connection/session factory and the next step is to configure each shard to have a separate redis store.
What I'd actually like to do is to to just use the region name to isolate the caches for each session factory.
Would it be possible to use the NHibernate redis cache in this way? To initialise the cache with different options for each session factory? The fact that RedisCacheProvider.SetOptions is static is giving me some doubts on whether this is achievable.

Hi @paulduran, thanks for your interest in the project!
Can you confirm for me how you'd like to separate each cache? Is the plan still to have a single Redis instance?
If the goal is to still use a single Redis instance (no sharding) and you're sharing the ConnectionMultiplexer for Redis, I believe you should still be able to use the static RedisCacheProviderOptions and then set the RedisCacheConfiguration for each model's specific region. For example if you have two model types for both session factory, you can just add both cache configurations for each.
If each session factory shares the same RedisCacheProviderOptions (cache configurations etc) and the only thing you want to change is the key prefix per-session factory, you should only need to configure the session factory to have a different cache region prefix (it should just work). See the cache.region_prefix property for configuring NHibernate.
PS - The only reason (originally) that the RedisCacheProviderOptions is static is because NHibernate creates the RedisCacheProvider... we can't inject anything into. However, I've had a fix in mind for a while that could remove the need for the RedisCacheProviderOptions to be static. But, I'd like to see if we can solve your problem with what's there today. :smile:

Hey @TheCloudlessSky ,
Thanks heaps for that suggestion! I don't know how I missed it.. But yep, that's all that was needed - to set the cache prefix when configuring the session factory.
I'm using FluentNHibernate so I just updated the setup code to be something like:
            .Cache(cache => cache.ProviderClass<RequestRecoveryRedisCacheProvider>()
                    .UseSecondLevelCache()
                    .RegionPrefix("CommonDb")
                    .UseQueryCache()
                )

thanks for your help!

Great! :smile:


Search field on the records page.

Hi!
When I use the search field:

"SQLSTATE[HY000]: General error: 1 near "group": syntax error (SQL: select count(*) as aggregate from "martin_forms_records" where (((lower(martin_forms_records.id) LIKE %j%) or (lower(martin_forms_records.group) LIKE %j%) or (lower(martin_forms_records.ip) LIKE %j%) or (lower(martin_forms_records.form_data) LIKE %j%))) and "martin_forms_records"."deleted_at" is null)" on line 664 of /vendor/laravel/framework/src/Illuminate/Database/Connection.php

Could you please help me out?
Thanks in advance.

Can confirm the error

October v 458
MagicForm v 1.4.16
Didn't get any errors !


Add Live Analysis

Update analyse.sh to read output from net-test.sh live and update stats as new values come in.

this can now be done with the watch terminal command. Since the analyze command is written in C now it takes less than a second to analyze a weeks worth of data.


collectstatic fails with latest buildpack (v96) but works with v83

Recently pushed a new version of my Django app, and experienced the following error:
remote:      $ python manage.py collectstatic --noinput
remote:        Traceback (most recent call last):
remote:          File "manage.py", line 36, in <module>
remote:            execute_from_command_line(sys.argv)
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
remote:            utility.execute()
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/__init__.py", line 377, in execute
remote:            self.fetch_command(subcommand).run_from_argv(self.argv)
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
remote:            self.execute(*args, **options.__dict__)
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/base.py", line 338, in execute
remote:            output = self.handle(*args, **options)
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/base.py", line 533, in handle
remote:            return self.handle_noargs(**options)
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py", line 168, in handle_noargs
remote:            collected = self.collect()
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py", line 98, in collect
remote:            for path, storage in finder.list(self.ignore_patterns):
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/contrib/staticfiles/finders.py", line 111, in list
remote:            for path in utils.get_files(storage, ignore_patterns):
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/contrib/staticfiles/utils.py", line 27, in get_files
remote:            directories, files = storage.listdir(location)
remote:          File "/app/.heroku/python/lib/python2.7/site-packages/django/core/files/storage.py", line 270, in listdir
remote:            for entry in os.listdir(path):
remote:        OSError: [Errno 2] No such file or directory: '/app/appname/static'

Given I hadn't made any major changes in the app was quite surprised: this worked the last time I pushed it (November 7th). After some editing got it to stop making the error, but suddenly started seeing errors that collectstatic wasn't generating the actual output files (my app's slug size dipped from 100M to 78M). After some additional futzing around I got this fixed by forcing the buildpack version to match the last version I successfully pushed with using
heroku buildpacks:set https://github.com/heroku/heroku-buildpack-python#v83
Looking at https://github.com/heroku/heroku-buildpack-python/compare/v83...v96 it appears that the BUILD_DIR is no longer set to /app (hence the above error). Previously my STATICFILES_DIR setting was this:
STATICFILES_DIRS = (
    os.path.join(APPPATH, 'appname/static'),
)

What should I use to properly comply with the requirements of v96 and later?

The problem goes beyond just collectstatic. Doing the build in /tmp/... and then running the app from /app seems to be the source?
I can confirm that v83 is the last one that was functional for me.
Here's a minimal sample that exhibits the problem: https://gist.github.com/ipmb/4d8f2def75ef23f56944477349ce1ca2
Note the path for __file__ on the script created by setup.py on the current buildpack  vs the actual location:
Running bash on ⬢ ll-test-build... up, run.8309 (Free)
~ $ which server.py
/app/.heroku/python/bin/server.py
~ $ cat `which server.py`
#!/app/.heroku/python/bin/python
# EASY-INSTALL-DEV-SCRIPT: 'build-test==1.0','server.py'
__requires__ = 'build-test==1.0'
__import__('pkg_resources').require('build-test==1.0')
__file__ = '/tmp/build_a5b88f1d3cffa5ad64ba2ef11e4ff8b7/server.py'
exec(compile(open(__file__).read(), __file__, 'exec'))

Correct — you should not be referencing /app in your codebase.

I had same problem except I wasn't even seeing any errors, so I spent lot of time debugging and trying to figure out where /app/static disappeared, previously in settings/production.py I had:
STATIC_ROOT = '/app/static'
MEDIA_ROOT = '/app/media'

Changing to following fixed the problem:
STATIC_ROOT = os.path.join(BASE_DIR, 'static')
MEDIA_ROOT = os.path.join(BASE_DIR, 'media')

And while building output is:
112 static files copied to '/tmp/build/static', 112 post-processed.


I created 2 pull requests

Hi Makk,
I created 2 pull requests. One for the Rscript and one for the snippet. Can you please check? I am only able to see one open pull request (snippet) in the main repo. I may have accidentally closed my first pull request. Please let me know if so.
Thank you,
Fan

So i checked the history and i did close the first pull request by mistake. My apologies. I was not able to re-open it and somehow I was not able to create new pull request either. So I just git committed and git pushed the Rscript to this repo. Please take a look at the Rscript and let me know if there are any changes that need to be made. I created a new folder called "script" where the Rscript is saved into. Thanks.

Hi Makk,
Adding to the previous comment above, I also did data wrangling and it is included in the Rscript.  Please take a look.
Thanks,
Fan


website does not work

https://jameswlane.github.io/status-board/

@DanielRuf somehow the wrong address got added into the website section for the repo. But the real demo site is down also. I will work on getting that fixed and updating the address on the repo.


Plugin not updating

Obsidian (v1.3.4) shows that I have version 0.1.0 installed (see image below). When I click the "Update" button Obsidian appears to install an update and then says the plugin is now up-to-date, however the installed version never changes from 0.1.0. I installed version 1.0.0 through the Obsidian community plugin portal, so I do not know where it is getting v0.1.0; I have never installed it through BRAT. I've tried uninstalling and reinstalling the plugin to no avail.

I am also missing the "Alt Text Format" and "End Inclusive" options in the plugin settings, and links are defaulting to displaying as note > h1..h2:

Hi!
Thanks for letting me know, I'll investigate why it's stuck at an older version.

Ok I just rebuild and updated the plugin. If you could try updating it again you should pull the latest now.
Thanks!

Works as expected now. Thanks for the super quick response!


feat: newsletter support

Summary:
Add support for newsletter
Changes:

Add method getNewsletterInfo()
Add method createNewsLetter()
Add method getSubscribedNewsletters()
Add method toggleMuteNewsletter()
Add method followNewsletter()
Add method unFollowNewsletter()
Add method updateNewsletterName()
Add method updateNewsletterDesc()
Add method updateNewsletterPicture()
Add method updateNewsletterReactionSetting()
Add method removeNewsletterPicture()
Support to received newsletter message
Support to send newsletter message (text, image, video, ptt, sticker, edit, delete)
Add newsletter in MediaGenerationOptions

Note:
To send media to the newsletter, we have to upload the media to the WhatsApp server without encrypting it first.
Newsletter media works mostly the same way as normal media, with a few differences:

Since it's unencrypted, there's no mediaKey or fileEncSha256 fields.
There's a handle that needs to be passed in additionalAttributes.

any for function to send message, link preview, message and other ?
cause i see only to manage channel only in this code

any for function to send message, link preview, message and other ?
cause i see only to manage channel only in this code

Just sendMessage normally with newsletter id
sock.sendMessage("13731837@newsletter", { text: "Hello" })

any for function to send message, link preview, message and other ?
cause i see only to manage channel only in this code

Just sendMessage normally with newsletter id
sock.sendMessage("13731837@newsletter", { text: "Hello" })

for this i was success send as plain text,
but fail to link preview message, media message, and file message.

I didn't see that you also did a PR on this

any for function to send message, link preview, message and other ?
cause i see only to manage channel only in this code

Just sendMessage normally with newsletter id
sock.sendMessage("13731837@newsletter", { text: "Hello" })

for this i was success send as plain text, but fail to link preview message, media message, and file message.

how do you send other the message? does it use the sendMessage function?

any for function to send message, link preview, message and other ?
cause i see only to manage channel only in this code

Just sendMessage normally with newsletter id
sock.sendMessage("13731837@newsletter", { text: "Hello" })

for this i was success send as plain text, but fail to link preview message, media message, and file message.

How to send files into channel?, it's not that whatsapp currently can't send files into channels, it can only send media such as voice, video and image, other than that it's just a poll, is it possible?

error in followNewsletter(id)
error:[{"message":"Bad Request","path":["xwa2_newsletter_join_v2"],"extensions":{"error_code":400,"severity":"CRITICAL","is_retryable":false}}]

Summary:
Add support for newsletter
Changes:

Add method getNewsletterInfo()
Add method createNewsLetter()
Add method getSubscribedNewsletters()
Add method toggleMuteNewsletter()
Add method followNewsletter()
Add method unFollowNewsletter()
Support to received newsletter message
Support to send newsletter message (text, image, video, ptt, sticker, edit, delete)
Add newsletter in MediaGenerationOptions

Note:
To send media to the newsletter, we have to upload the media to the WhatsApp server without encrypting it first. Newsletter media works mostly the same way as normal media, with a few differences:

Since it's unencrypted, there's no mediaKey or fileEncSha256 fields.
There's a handle that needs to be passed in additionalAttributes.

Cannot send media {documents, audio, video, photos, vn) using the sendFile function to the Group
(conn.sendFile = async (
      jid,
      path,
      filename = "",
      caption = "",
      quoted,
      ptt = false,
      options = {},
    ) => {
      let type = await conn.getFile(path, true);
      let { res, data: file, filename: pathFile } = type;
      if ((res && res.status !== 200) || file.length <= 65536) {
        try {
          throw { json: JSON.parse(file.toString()) };
        } catch (e) {
          if (e.json) throw e.json;
        }
      }
      let opt = { filename };
      if (quoted) opt.quoted = quoted;
      if (!type) options.asDocument = true;
      let mtype = "",
        mimetype = type.mime,
        convert;
      if (
        /webp/.test(type.mime) ||
        (/image/.test(type.mime) && options.asSticker)
      )
        mtype = "sticker";
      else if (
        /image/.test(type.mime) ||
        (/webp/.test(type.mime) && options.asImage)
      )
        mtype = "image";
      else if (/video/.test(type.mime)) mtype = "video";
      else if (/audio/.test(type.mime))
        (convert = await (ptt ? toPTT : toAudio)(file, type.ext)),
          (file = convert.data),
          (pathFile = convert.filename),
          (mtype = "audio"),
          (mimetype = "audio/mpeg");
      else mtype = "document";
      if (options.asDocument) mtype = "document";

      let message = {
        ...options,
        caption,
        ptt,
        [mtype]: { url: pathFile },
        mimetype,
      };
      let m;
      try {
        m = await conn.sendMessage(jid, message, { ...opt, ...options });
      } catch (e) {
        console.error(e);
        m = null;
      } finally {
        if (!m)
          m = await conn.sendMessage(
            jid,
            { ...message, [mtype]: file },
            { ...opt, ...options },
          );
        return m;
      }
    });

@zennn08  How to get list of newsletter in which I am admin? similar to how we get list of groups

any for function to send message, link preview, message and other ?
cause i see only to manage channel only in this code

Just sendMessage normally with newsletter id
sock.sendMessage("13731837@newsletter", { text: "Hello" })

I am able to send message but messages can be seen only in web.whatsapp.com but not on mobile WhatsApp
anything am I doing wrong?

How can I list all newsletters? I just created one but I don’t know how I get your metadata.
I tried the next and I get a empty array:
const nn = await getSubscribedNewsletters();
console.log("nn", nn);

on updates on the WhatsApp update, i found this part, is this part of the profile picture of the channel?

how to make it like that

How to send voice message, I tried it normal way but its not playing


Broken algorithm

The algorithm will always wait too much.
import time
from ratelimiter import RateLimiter

rate_limiter = RateLimiter(max_calls=1, period=1)

start = time.time()
for i in range(3):
    with rate_limiter:
        print("%.02f" % (time.time()  - start))
        time.sleep(1)

outputs
0.00
2.00
4.01

I would expect it to output 0 1 2

This simple code works
class RateLimiter:

    def __init__(self, max_calls, period):
        self.calls = collections.deque(maxlen=max_calls)
        self.period = period
        self.max_calls = max_calls

    def wait(self):
        if len(self.calls) == self.max_calls:
            wait = self.period + self.calls[0] - time.time()
            if wait > 0:
                time.sleep(wait)
        self.calls.append(time.time())

Hi Louis! I forked the repository and I changed the code according to your suggestion: https://github.com/holasoftware/ratelimiter
This code works as you expected:
import time
from ratelimiter import RateLimiter

rate_limiter = RateLimiter(max_calls=1, period=1)

start = time.time()
for i in range(3):
    with rate_limiter:
        print("%.02f" % (time.time()  - start))
    time.sleep(1)

Great! I also published my code there


Right way to use debugger to debug test failures

Problem: getting a 404, expecting a 200. Want to use node-inspector to debug the app that tests are running against.
What's the right way to go about this? Current hypothesis (update: it worked, steps revised to reflect what we did):

run node-inspector and connect to debugger in Chrome/whatever using the URL echoed by node-inspector
set launch = false in .testiumrc
Specify constant port number via app.port in .testiumrc OR export PORT=something
add debugger statement in production code
run your app with node --debug-brk app.js
run tests as normal (not under debugger)

Gotchas: don't simultaneously run your app under the debugger and your unit tests under the debugger. Bad things will happen.
About to try this out, but would appreciate any other advice on how to debug our app under test. :beers:

Hi! Sorry for the late reply. I'm assuming 4. and 5. should be switched around but otherwise that sounds like the way to go. You also might want to fix down the app port in .testiumrc in case you didn't already. Would be great to hear how it went/if there were any problems!

Got it working. Updated description above with the steps that worked for us. Closing.

:+1: on sharing your discoveries, the XKCD approved way! :-D


Style messages as "info"

Following the wiki as recommended by @steelbrain
Along with some readme cleanup (new screenshot, remove broken images, update links).

I would suggest you do not rename Info to pep257 as we'll be showing provider names along with messages in the near future, and it'll look awkward [pep275][pep275] This is an info message.

Updated.

IMO, You shouldn't store images in github repos, instead goto create an issue, post an image, copy that link over here. It won't ever expire.

It won't ever expire

Indeed, much better.

LGTM, merge at will.


shorter more descriptive chat titles

Description
this prompt produces titles which are shorter, more descriptive and less likely to be truncated, this reliably produces more descriptive titles on smaller models, the changes are less pronounced on larger models but still prevents things such as starting with a verb in the present participle ("summarizing", "describing", "discussing")  which don't contribute meaning to the title.
Screenshots

this shows the old summaries alongside new ones, I think it is pretty easy to tell which is which.
Testing instructions
tested mainly with llama 3.1 8b and chatgpt 4o

This is nice!


Chest breaking - Put everything back where it was

It would be nice if the player breaks the chest, it puts everything back where it was before the player died.
One of my players suggested this idea. He's tired to put everything back where it was one by one.

You mean back to the right slots in the player's inventory?

You mean back to the right slots in the player's inventory?

Yes, exactly


Cannot be used in version 1.19.4purpur(same sa before)

latest.log

It was because of one class file.
I have fixed it and updated the plugin, this issue will no longer occur.


EnableMoveReplicaFaults does not work when set to $false. Has anyone tested the scenario where we need to disable the EnableMoveReplicaFaults and run chaos? I am getting this error " ##[error]A positional parameter cannot be found that accepts argument 'False'."

EnableMoveReplicaFaults does not work when set to $false. Has anyone tested the scenario where we need to disable the EnableMoveReplicaFaults and run chaos? I am getting this error " ##[error]A positional parameter cannot be found that accepts argument 'False'."
https://docs.microsoft.com/en-us/rest/api/servicefabric/sfclient-model-chaosparameters#enablemovereplicafaults

@BharathRahulDS Hi, thanks for your feedback here, can you please share which document you are referring to so that we can look into it?

Thank you for the response. Below is the link.
https://docs.microsoft.com/en-us/rest/api/servicefabric/sfclient-model-chaosparameters#enablemovereplicafaults

@BharathRahulDS I see there are a few ways to initiate Chaos. How are you invoking this mode?

@BharathRahulDS I will close this for now. When ready let me know and we can reopen and continue.


do u have any paper for this ?

any research paper for this or proper doc

No, This was done as a fun project. So unfortunately there was no research paper. If you want to quote this in your research please give the source to my github code.


remove migration script from ssclj module

The migration script from HSQLDB to ES brings in dependencies that are not needed for the server (notably the database) and requires compilation, which further bloats the size of the jar file.

[ ] Move the migration script to a separate module.
[ ] Package the script such that it does not need to create an uberjar.
[ ] See if dynamic loading of the implementation from the main function is worthwhile.

The migration scripts have been entirely removed.  This ticket is no longer relevant.


On App start with visual studio, got only a error

Hello,
this error i got : Unhandled Exception:
System.Reflection.ReflectionTypeLoadException: Exception of type 'System.Reflection.ReflectionTypeLoadException' was thrown.
The App- Icon is show.
After (tested on single stepp) this (MainApplication.cs):
public void OnActivityResumed(Activity activity)
{
CrossCurrentActivity.Current.Activity = activity;
}
"CrossCurrentActivity.Current.Activity = activity;" works (activity has value:"{md5284d34c42aded56ab4276d9b253081c8.SplashScreen@7e07a0f}")
After "NExt Stepp" on "}" i got the error.
But i tested on Emulator with Android 7, 8 and also on Device with Android 9
All the same issue
Output:
03-24 22:35:47.065 D/Mono (24630): Assembly Ref addref MvvmCross.Binding.Droid[0x72297dbf80] -> MvvmCross.Platform.Droid[0x721d47c680]: 3
03-24 22:35:47.086 D/AwareBitmapCacher(24630): handleInit switch not opened pid=24630
03-24 22:35:47.091 D/Mono (24630): Assembly Ref addref BLE.Client.Droid[0x72297db180] -> AndHUD[0x72297db580]: 2
03-24 22:35:47.092 D/Mono (24630): Assembly Ref addref BLE.Client.Droid[0x72297db180] -> MvvmCross.Binding.Droid[0x72297dbf80]: 3
03-24 22:35:47.093 D/Mono (24630): Assembly Ref addref BLE.Client.Droid[0x72297db180] -> Splat[0x721d47ce00]: 2
03-24 22:35:47.111 D/Mono (24630): Unloading image System.Reflection.dll [0x7203ad4000].
03-24 22:35:47.112 D/Mono (24630): Image addref System.Reflection[0x7203b1b680] -> System.Reflection.dll[0x7204e7f000]: 7
03-24 22:35:47.112 D/Mono (24630): Config attempting to parse: 'System.Reflection.dll.config'.
03-24 22:35:47.112 D/Mono (24630): Config attempting to parse: '/Users/builder/jenkins/workspace/xamarin-android-d15-9/xamarin-android/external/mono/sdks/out/android-arm64-v8a-release/etc/mono/assemblies/System.Reflection/System.Reflection.config'.
03-24 22:35:47.112 D/Mono (24630): Assembly Ref addref MvvmCross.Binding[0x72297dbe80] -> System.Reflection[0x720bdc9580]: 5
03-24 22:35:47.124 D/Mono (24630): Unloading image System.Linq.dll [0x7203ad4000].
03-24 22:35:47.124 D/Mono (24630): Image addref System.Linq[0x7203b2af00] -> System.Linq.dll[0x7204ea0000]: 7
03-24 22:35:47.124 D/Mono (24630): Config attempting to parse: 'System.Linq.dll.config'.
03-24 22:35:47.124 D/Mono (24630): Config attempting to parse: '/Users/builder/jenkins/workspace/xamarin-android-d15-9/xamarin-android/external/mono/sdks/out/android-arm64-v8a-release/etc/mono/assemblies/System.Linq/System.Linq.config'.
03-24 22:35:47.124 D/Mono (24630): Assembly Ref addref MvvmCross.Binding[0x72297dbe80] -> System.Linq[0x7204faa080]: 5
Unhandled Exception:
System.Reflection.ReflectionTypeLoadException: Exception of type 'System.Reflection.ReflectionTypeLoadException' was thrown.

Hi @harrypotter06,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!


Add test email form on admission overview

Adds an option to send a test email from any @samfundet email to any recipient.
Useful for testing that the email system works before rejection emails are sent.

We don't really need this and it is a bit dangerous to expose, closed and not merged


Integration Test Optimizations

resolves #9498
Problem
Integration tests perform a lot of identical rework for every test.
Solution
Cache important static objects between test runs.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[x] This PR includes type annotations for new and modified functions

does this result in test runs caching macros in ways that dbt-core wouldn't do typically? That seems to create a divergence between actual and test behavior of dbt-core

Blocking on validation this doesn't impact adapter expectations around macros/test fixtures: https://github.com/dbt-labs/dbt-snowflake/actions/runs/7733189834/job/21084750889

Looks like it succeeded so it's probably good to go, will check a couple other adapters

@colin-rogers-dbt I considered that, and wouldn't be doing this if it didn't save us a ton of testing time. With that said, the objects we are caching appear to be static after construction, and are computed as a pure function of their construction parameters. So while I can not completely rule out future complications without deeper knowledge of jinja internals, my sense is that complications are unlikely.


statically setting increment_build_number appears to work but pilot / testflight ignores value

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
I'm using increment_build_number in my Fastfile like this:
platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :beta do
    api_key = app_store_connect_api_key(
      key_id: ENV['ASCAPI_KEY_ID'],
      issuer_id: ENV['ASCAPI_ISSUER_ID'],
      key_content: ENV['ASCAPI_KEY_CONTENT']
    )

    increment_build_number(build_number: ENV['VERSION'])
    build_app(scheme: "myapp (iOS)")
    upload_to_testflight
  end

VERSION is set by the CI pipeline, and appears to work:
[14:01:38]: $ cd /usr/local/concourse/work_dir/volumes/live/b0e6478e-9417-4a06-4f2e-4626be00341b/volume/familyfoodlegacy && agvtool new-version -all 0.1.206 && cd -

[14:01:38]: ▸ Setting version of project familyfoodlegacy to:

[14:01:38]: ▸ 0.1.206.

[14:01:38]: ▸ Also setting CFBundleVersion key (assuming it exists)

[14:01:38]: ▸ Updating CFBundleVersion in Info.plist(s)...

[14:01:38]: ▸ Cannot find "familyfoodlegacy.xcodeproj/../YES"

[14:01:38]: ▸ Updated CFBundleVersion in "familyfoodlegacy.xcodeproj/../familyfoodlegacy--iOS--Info.plist" to 0.1.206

However, upon upload_to_testflight, I see that the build version has not been applied, rather an incremental bump has been made to the previous version (presumably retrieved from testflight):
[14:02:57]: Waiting for processing on... app_id:<PHONE_NUMBER>, app_version: 1.0, build_version: 55, platform: IOS

[14:02:57]: Read more information on why this build isn't showing up yet - https://github.com/fastlane/fastlane/issues/14997

[14:02:57]: Waiting for the build to show up in the build list - this may take a few minutes (check your email for processing issues if this continues)

[14:03:27]: Waiting for the build to show up in the build list - this may take a few minutes (check your email for processing issues if this continues)

[14:03:59]: Waiting for App Store Connect to finish processing the new build (1.0 - 55) for IOS

[14:04:29]: Waiting for App Store Connect to finish processing the new build (1.0 - 55) for IOS

I've stepped through the output of the xcode build process, and confirmed that the Info.plist generated in the archive does include the expected values:
		<key>CFBundleShortVersionString</key>
		<string>1.0</string>
		<key>CFBundleVersion</key>
		<string>0.1.206</string>

So it would seem that pilot is ignoring CFBundleVersion, and just incrementing my build number based on the previous existing builds in testflight?

No, we abandoned fastlane in the end, and went to xcode cloud builds :(

Any update on this issue? I am also facing the same problem. I printed the existing and incremented build value, but when I try to upload it, I get an error.
[Application Loader Error Output]: The provided entity includes an attribute with a value that has already been used The bundle version must be higher than the previously uploaded version: ‘1’. (ID: 1bb8181b-599e-4627-90a3-ec74a58562d9) (-19232)

However, when I checked the app store console it has only one build under that version on which I am trying to upload the new build.
Logs:
Using existing authorization token for App Store Connect API
Fetching the latest build number for any version
Latest upload for version 0.10.1 on ios platform is build: 1
------------------------------------
--- Step: increment_build_number ---
------------------------------------
cd /path/to/ios && agvtool new-version -all 2 && cd -
Setting version of project Runner to: 2.
Also setting CFBundleVersion key (assuming it exists)
Updating CFBundleVersion in Info.plist(s)...
Updated CFBundleVersion in "Runner.xcodeproj/../Runner/Info.plist" to 2
Cannot find "Runner.xcodeproj/../YES"

-------------------
--- Step: pilot ---
-------------------
INFO [2024-05-11 16:14:42.42]: Creating authorization token for App Store Connect API
DEBUG [2024-05-11 16:14:42.42]: App identifier (io.abc.app)
INFO [2024-05-11 16:14:43.93]: Ready to upload new build to TestFlight (App: xxxx)...
DEBUG [2024-05-11 16:14:43.96]: App Platform (ios)
INFO [2024-05-11 16:14:44.76]: Wrote XML data to '/var/folders/n_/6_y2gmw17fq8jr38bf4q3jt00000gp/T/d20240511-57139-hqu8nk/6478811022-f5a9a8bb-2994-49b8-a60d-6ce7d837fd46.itmsp'
DEBUG [2024-05-11 16:14:44.76]: Using altool as transporter.
INFO [2024-05-11 16:14:44.86]: Going to upload updated app to App Store Connect
INFO [2024-05-11 16:14:44.86]: This might take a few minutes. Please don't interrupt the script.
DEBUG [2024-05-11 16:14:44.87]: API_PRIVATE_KEYS_DIR=YourTmpP8KeyDir xcrun altool --upload-app --apiKey YourKeyID --apiIssuer YourIssuerID -t ios -f /var/folders/n_/6_y2gmw17fq8jr38bf4q3jt00000gp/T/381b8a50-a3ba-4a50-9ad9-968a723a8782.ipa -k 100000
DEBUG [2024-05-11 16:14:57.99]: [altool]:  {

DEBUG [2024-05-11 16:14:57.99]: [altool]:     NSLocalizedDescription = "The provided entity includes an attribute with a value that has already been used";

DEBUG [2024-05-11 16:14:57.99]: [altool]:     NSLocalizedFailureReason = "The bundle version must be higher than the previously uploaded version: \U20181\U2019. (ID: 1bb8181b-599e-4627-90a3-ec74a58562d9)";

DEBUG [2024-05-11 16:14:57.99]: [altool]:     NSUnderlyingError = "Error Domain=IrisAPI Code=-19241 \"The provided entity includes an attribute with a value that has already been used\" UserInfo={status=409, detail=The bundle version must be higher than the previously uploaded version., source={\n    pointer = \"/data/attributes/cfBundleVersion\";\n}, id=1bb8181b-599e-4627-90a3-ec74a58562d9, code=ENTITY_ERROR.ATTRIBUTE.INVALID.DUPLICATE, title=The provided entity includes an attribute with a value that has already been used, meta={\n    previousBundleVersion = 1;\n}, NSLocalizedDescription=The provided entity includes an attribute with a value that has already been used, NSLocalizedFailureReason=The bundle version must be higher than the previously uploaded version.}";

DEBUG [2024-05-11 16:14:57.99]: [altool]:     "iris-code" = "ENTITY_ERROR.ATTRIBUTE.INVALID.DUPLICATE";

DEBUG [2024-05-11 16:14:57.99]: [altool]:     previousBundleVersion = 1;

DEBUG [2024-05-11 16:14:57.99]: [altool]: }

Versions Details:
Fastlane: fastlane 2.220.0
Flutter Version: 3.19.6

still happening.

I am also getting the same above issue. Any update on this?  @jainrounak6


ENH: Implement functionality to extract index in a bounded region

TransformPhysicalPointToIndex & TransformPhysicalPointToContinuousIndex are 2 sets of functions that convert a physical point to an index with the usage of rounding. Geometrically speaking, if a point is outside the region of an image by at max spacing/2 across every dimension, the point is going the get an index that can exist inside the image.
I developed TransformPhysicalPointToIndexInBoundedRegion & TransformPhysicalPointToContinuousIndexInBoundedRegion that given a point and a region of the image, performs the same operation, but if a point is geometrically speaking outside of the given region, then an out of bounds index is going to be returned.
If the region is the largest possible region, then it can understand if a point is geometrically speaking outside of the image.
So in simple words, it is strict at the boundaries.
For more information see here: ITK discussion

Nice work!
Please look at SimpleITK's fundamental concepts of images to understand the details of the image geometry I'm discussing below: https://simpleitk.readthedocs.io/en/master/fundamentalConcepts.html
Under the ITK's Image indexing concepts the continuous index of -0.49 is considered with the image's region boundary.
The index represents the center of each voxel. So each voxel has an extent of 1, and +/- 0.5 from either side. Therefore for a 1-D image with a region of [0,10], the valid continuous index range is [-0.5, 9.5]. I believe
your implementation needs a correction to account for the correct behavior on the 0.5 boulder.
Additionally, testing is required so we can verify the correctness of the methods and not regression of behavior occur in the future. For this level of functionality, I would suggest looking at the GTest style tests as an example.

If understand correctly your example, let's assume we have a 1D image with origin = {0,0}, spacing = {1} and size = {11} (since it is [0, 10]). In this case, the valid continuous index is [-0.5, 10.5], but with my additionally bounded methods, i restrict to [0, 10] cause anything out of that is geometrically speaking outside of the image.

That is a very good illustration of this behavior or issue.

For example, i have attached an image with origin {0, 0, 0}, spacing = {0.1, 0.6931469, 0.2}, and size = {41, 6 , 21}. If you preview it on ParaView, you will see that the image is not going below {0, 0, 0} geometrically speaking.
data.zip

3D Slicer and ITK Snap correctly use the ITK standard of origin at the center of the voxel and then volume extent going down to the {-0.5} continuous index. These applications are designed for medical image analysis and have focus on very precise geometric consistency of volumes and data.

So in my methods, I allow the interior pixels to extend by spacing[i]/2 to every dimension as in theory, but when it comes to the boundary ones, I don't allow that to happen, because of geometric reasons.
Yeah, I know that this creates conflict with the fundamental definition of the image, but I believe it's useful when you want to represent such bounded/strict domains (grids), as images.

I can understand how this particular implementation could be useful. But the inconsistency with the rest of ITK is going to be confusing and problematic if this method is added to ITK itself.
For example it is not consistent with the bool ImageRegion::IsInside(const ContinuousIndex<...> & ) const method. Introducing another "inside" or "out of bounds" concept would be a bad thing.

ParaView is based on VTK, and VTK's current convention on images is that pixel's origin is in it corner. This is different from medical imaging standard (essentially set by DICOM) where pixel's origin is in its center. ITK, ITK-SNAP and Slicer follow this medical imaging convention. So if you are dealing with medical images, ParaView is not the best choice of viewer.


[Document] Creating a new logic adapter

Error
In Providing extra arguments part, these several lines of code:

class MyLogicAdapter(LogicAdapter):
    def __init__(self, **kwargs):
        super(MyLogicAdapter, self).__init__(kwargs)

        self.api_key = kwargs.get('secret_key')

super(MyLogicAdapter, self).init(kwargs) "kwargs" should be "**kwargs"

enhancement:
In Directory structure part, there is only one file example.
Now I want to put adapter into directory. such as /myadpters/logic/Myadpter.py
Do we must add /myadpters/init.py, add /myadpters/logic/init.py
and add content in /myadpters/logic/init.py

from .MyAdapter import MyAdapter

Hi @RominYue Thank you, I will make adjustments to the documentation to clarify this.

Regarding your last question, from .MyAdapter import MyAdapter is not required to be added to the __init__.py file.
Adding it to __init__.py is just a Python convention that allows you to import the class like this:
from myadapters import MyAdapter

instead of like this:
from myadapters.logic.my_adapter import MyAdapter

yeah...
but when I want to use it in ChatBot,
ChatBot('RominYue',
        logic_adapters=[
            'chatterbot.logic.BestMatch',
            'adapters.logic.MyAdapter'
        ]
    )

it would pop out error when I do not add from myadapters import MyAdapter in init.py

I'm going to close this off as I believe all of the issues have been addresses. Feel free to reopen it if I missed any documentation changes that were needed,


Add traits to a TestNode properties

Based on this discussion: https://github.com/microsoft/testfx/discussions/3991
A few things:

Currently a framework author would have to know about a specific traits key that needs adding to the PropertyBag - Leaking internal testing platform implementation details and could be easily missed or unknown. This could be abstracted away behind a custom property object.
Authors would also need to know how Categories are defined vs Traits. (Seems to be the category is a key with no value, and a trait is a key value pair)
SerializableNamedKeyValuePairsStringProperty which the VSTest bridge uses is internal, so framework authors don't have the option of using that.

Suggestion:
public sealed record TestTraitsProperty(
        IEnumerable<string> Categories,
        IEnumerable<KeyValuePair<string, string>> Traits) : SerializableNamedKeyValuePairsStringProperty("traits",
    [
        ..Categories.Select(category => new KeyValuePair<string, string>(category, string.Empty)),
        ..Traits
    ]);

Even without this, there is still the issue they the underlying property type is internal. I think this is blocking framework authors from setting traits/categories unless there's a workaround?

Marking as bug although I don't have time to test/investigate right now. We should have supprot for traits/properties/metadata as it's a heavily used feature in VS.

It's probably a better idea that exposing this technical type, let me discuss with the team and get back to you.

Hey @thomhurst,
Good news, I was doing some checks and tests about this and found out we already implemented TestMetadataProperty, it's public and it's already plugged into the communication protocol for Test Explorer.
I'll close as done but please if there are any bug please reopen/comment.

So for a category do I just add a metadata property with an empty key?

Null or empty string value, the category is the key. I guess we could provide a different ctor for that!

Thanks seems to be working great. Up to you if you want to leave this open to create a simpler constructor 😄


Update Edit command

In response to issue #108, we now have Edit Command to enable editting of student particulars using StudentId as unique primary key

Codecov Report
Base: 65.32% // Head: 64.16% // Decreases project coverage by -1.16% :warning:

Coverage data is based on head (9e04d8b) compared to base (9e934d0).
Patch coverage: 91.30% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #137      +/-   ##
============================================
- Coverage     65.32%   64.16%   -1.17%
- Complexity      484      539      +55
============================================
  Files            91      102      +11
  Lines          1638     1850     +212
  Branches        181      194      +13
============================================
+ Hits           1070     1187     +117
- Misses          516      598      +82
- Partials         52       65      +13

Impacted Files
Coverage Δ

src/main/java/jeryl/fyp/commons/core/Messages.java
0.00% <ø> (ø)

src/main/java/jeryl/fyp/model/Model.java
100.00% <ø> (ø)

...ain/java/jeryl/fyp/logic/commands/EditCommand.java
93.24% <87.50%> (-2.65%)
:arrow_down:

src/main/java/jeryl/fyp/model/FypManager.java
80.00% <90.90%> (-1.09%)
:arrow_down:

...java/jeryl/fyp/logic/parser/EditCommandParser.java
85.71% <100.00%> (-7.15%)
:arrow_down:

src/main/java/jeryl/fyp/model/ModelManager.java
86.41% <100.00%> (-4.06%)
:arrow_down:

...l/student/exceptions/StudentNotFoundException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...ain/java/jeryl/fyp/logic/commands/HelpCommand.java
77.77% <0.00%> (-22.23%)
:arrow_down:

src/main/java/jeryl/fyp/logic/LogicManager.java
60.00% <0.00%> (-15.00%)
:arrow_down:

...ava/jeryl/fyp/model/student/UniqueStudentList.java
78.68% <0.00%> (-13.47%)
:arrow_down:

... and 52 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Test reliability: Activity order could be swapped

Screenshots

https://webchattest.blob.core.windows.net/travis-result/2020/01/28/c2x4d/__image_snapshots__/chrome-docker/__diff_output__/carousel-js-carousel-with-avatar-initials-4-attachments-and-no-message-1-diff.png
Version
master
Describe the bug
Racing conditions between activities could rarely make the user's message appears below the bot's response.
Steps to reproduce
Sometimes, it fail in this test case, carousel-js-carousel-with-avatar-initials-4-attachments-and-no-message-1-diff.
Expected behavior
Expect test to succeed.
Additional context
We should consider also using reply-to ID for sorting, given the randomness behavior from Direct Line channel.
[Bug]

Fixed in MockBot.


Median of Two Sorted Arrays

Leetcode Question no. - 4
Problem name -  Median of Two Sorted Arrays
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed

Programming Language

[x] Python

Happy Coding

Hi again @pawangeek,
Can I take up this issue? :)


Building error on v1.3.2

Error message
Task :app:compileDebugJavaWithJavac FAILED
path-to-project/android/app/src/main/java/com/**/**/MainApplication.java:67: error: constructor RNAppsFlyerPackage in class RNAppsFlyerPackage cannot be applied to given types;
            new RNAppsFlyerPackage(MainApplication.this),
            ^
  required: no arguments
  found: MainApplication
  reason: actual and formal argument lists differ in length
Note: path-to-project/android/app/src/main/java/com/**/**/MainApplication.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 error

Error version
Upgrade from 1.3.1 to 1.3.2

i have this same issue with AppsFlyer v6.0.30,
unlinking and linking didn't help me either

hi @aposhettigar, do you able to import RNAppsFlyerPackage manually inside ReactApplication class?
RNAppsFlyerPackage constructor has no arguments. https://github.com/AppsFlyerSDK/appsflyer-react-native-plugin/blob/master/android/src/main/java/com/appsflyer/reactnative/RNAppsFlyerPackage.java#L16-L17
From which version you want to upgrade the AppsFlyer plugin?