http_conn_manager_prefix stat tag is not matched accordingly

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
All of our http filter stats do not correctly include the http_conn_manager_prefix tag. Here is an example:
/stats

http.InboundPassthroughClusterIpv4.jwt_authn.allowed: 0
http.InboundPassthroughClusterIpv4.jwt_authn.denied: 0
http.InboundPassthroughClusterIpv4.rbac.allowed: 0
http.InboundPassthroughClusterIpv4.rbac.denied: 0
http.InboundPassthroughClusterIpv4.rbac.shadow_allowed: 0
http.InboundPassthroughClusterIpv4.rbac.shadow_denied: 0
http.inbound_<IP_ADDRESS>_8080.jwt_authn.allowed: 11486
http.inbound_<IP_ADDRESS>_8080.jwt_authn.denied: 0
http.inbound_<IP_ADDRESS>_8080.rbac.allowed: 11486
http.inbound_<IP_ADDRESS>_8080.rbac.denied: 0
http.inbound_<IP_ADDRESS>_8080.rbac.shadow_allowed: 11486
http.inbound_<IP_ADDRESS>_8080.rbac.shadow_denied: 0
http.inbound_<IP_ADDRESS>_8443.jwt_authn.allowed: 154936
http.inbound_<IP_ADDRESS>_8443.jwt_authn.denied: 0
http.inbound_<IP_ADDRESS>_8443.rbac.allowed: 154936
http.inbound_<IP_ADDRESS>_8443.rbac.denied: 0
http.inbound_<IP_ADDRESS>_8443.rbac.shadow_allowed: 154936
http.inbound_<IP_ADDRESS>_8443.rbac.shadow_denied: 0

/stats/prometheus

envoy_http_InboundPassthroughClusterIpv4_jwt_authn_allowed{} 0
envoy_http_InboundPassthroughClusterIpv4_jwt_authn_denied{} 0
envoy_http_InboundPassthroughClusterIpv4_rbac{authz_enforce_result="allowed"} 0
envoy_http_InboundPassthroughClusterIpv4_rbac{authz_enforce_result="denied"} 0
envoy_http_InboundPassthroughClusterIpv4_rbac{authz_dry_run_result="allowed"} 0
envoy_http_InboundPassthroughClusterIpv4_rbac{authz_dry_run_result="denied"} 0
envoy_http_inbound_10_38_36_249_8080_jwt_authn_allowed{} 11618
envoy_http_inbound_10_38_36_249_8080_jwt_authn_denied{} 0
envoy_http_inbound_10_38_36_249_8080_rbac{authz_enforce_result="allowed"} 11618
envoy_http_inbound_10_38_36_249_8080_rbac{authz_enforce_result="denied"} 0
envoy_http_inbound_10_38_36_249_8080_rbac{authz_dry_run_result="allowed"} 11618
envoy_http_inbound_10_38_36_249_8080_rbac{authz_dry_run_result="denied"} 0
envoy_http_inbound_10_38_36_249_8443_jwt_authn_allowed{} 156829
envoy_http_inbound_10_38_36_249_8443_jwt_authn_denied{} 0
envoy_http_inbound_10_38_36_249_8443_rbac{authz_enforce_result="allowed"} 156829
envoy_http_inbound_10_38_36_249_8443_rbac{authz_enforce_result="denied"} 0
envoy_http_inbound_10_38_36_249_8443_rbac{authz_dry_run_result="allowed"} 156829
envoy_http_inbound_10_38_36_249_8443_rbac{authz_dry_run_result="denied"} 0

Theoretically InboundPassthroughClusterIpv4 and inbound_<IP_ADDRESS>_8080 should be extracted as a tag http_conn_manager_prefix which is included in istio bootstrap templates
We were using BDS to fix this which was deprecated in #48693 and now are left with only tweaking istio bootstrap templates which we were hoping to avoid.
Now this seems like a bug in Istio because the tag extracion regex is defined but does not match values generated by istio itself. Maybe some time ago Istio changed the format used for generating http connection manager names and never adjusted the tag regex accordingly?
FYI, we're using Sidecar CRD to define ingress ports.
Version
1.21.2

Additional Information
No response

@keithmattix @kyessenov @zirain is there not some way to have envoy do structured stats without performing a regex against a flat string? I think I have asked this before and was told "no", but it just seems kind of hard to accept?

@keithmattix @kyessenov @zirain is there not some way to have envoy do structured stats without performing a regex against a flat string? I think I have asked this before and was told "no", but it just seems kind of hard to accept?

There are some ways to generate structured stats but internally in envoy and I think they're note used for most stuff, only used for some filters.
Another way to look at this is probably do something like we do with Istio custom stats. Istio custom stats generate tag keys and values with a specific pattern which is then easy to match in regexes. Maybe we can do the same for stat_prefixes to ensure we can always extract those tags

Yeah I think stats is only through regex in Envoy

Envoy stats pre-date labels so labels can only be added in post-processing.
Yes, I agree we should default to something sane for regexes. Not sure how it ended up in this state. We should set "stat_prefix" to something that's regex friendly or elide superficial details, like port numbers, to avoid a cardinality explosion.

Any updates on this issue? We're also running into it, and are also hoping to avoid tweaking bootstrap templates.

@keithmattix did you fix this?

I suspect @keithmattix 's change won't fix this specific problem - it looks like that PR addresses a similar issue for the cluster name, but not for the http connection manager stat prefix. I think a similar approach should work here, too, though?
I'm happy to dive into the code a bit and contribute a PR here, but admittedly also hesitant given the backwards compatibility issue with the cluster name stat fix.

No I didn't catch this in my initial PR; I can make a follow-up with the same approach. I'd prefer to get #52031 merged first though

Cheers, Keith. Happy to wait for the other PR to get merged!

@keithmattix, we just rolled out the update. We are seeing a double _ in the metrics:
envoy_http__downstream_rq_total{http_conn_manager_prefix="outbound_<IP_ADDRESS>_8081"} 0
Would this be an easy fix?

Thanks for the report! I'm surprised my tests didn't catch this but yeah hopefully this'll be a simple fix

@keithmattix is this the simple fix you were alluding to? https://github.com/istio/istio/pull/52234

@keithmattix is this the simple fix you were alluding to? #52234

Nope it was #52829 and the accompanying cherry-pick

Nope it was https://github.com/istio/istio/pull/52829 and the accompanying cherry-pick

Oh sweet! When can we expect this patch to be released/available?

During the next 1.23 release; should be ~1 month?


Chart no respect div size

General Questions

Issue Type

[X] I have a question to ask about how to use ECharts to ...；我想提问如何使用 ECharts 实现某功能
[ ] I have a bug to report；我想要报 bug
[ ] I have a feature to request, e.g.: I'd like a new feature that ...；我需要一个新功能
[ ] I have a feature to enhance, e.g.: The current feature should be improved in the way that ...；我希望改进某个功能
[ ] There's something wrong with the documents；文档有些问题
[ ] Others, or I'm not sure which issue types to choose here；其他，或我不知道应该选什么类型

Issue Details

Hi, how set to chart respect a div size?
Expected Behavior

Current Behavior

Online Example

Topics

Anything Else We Need to Know

Environment

ECharts version；ECharts 版本: {DESCRIBE_HERE}

[ ] It happens only on certain browsers or operating systems. 对于特定浏览器或操作系统才会出现的问题，请提供相应环境信息：{BROWSER_VERSION_OR_OS_INFORMATION_HERE}

Hi, could explain what's happening here? What's wrong and what's expected?


fixes #183

I don't know if we are going to show these id's this way but I know MGI:MGI:12345 really bugs people here.

This fixes the gene page to show RGD:123456, so I'm bringing it in, but the bummer is that it's still linking from search with /gene/123456 rather than /gene/RGD:123456
So, more to do, but this is still worth bringing in.

+1 to fixing to double MGI error before it becomes too late
On 23 Feb 2017, at 16:47, Olin Blodgett wrote:

I don't know if we are going to show these id's this way but I know
MGI:MGI:12345 really bugs people here.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/alliance-genome/agr/pull/187#issuecomment-282169648

I deleted that comment, after I realized there is a conditional for a specific organism.

We fixed the MGI issue already before this bug - this fix is then to fix the RGD ids that went prefix-less by the fix for MGI.  Some folks have prefixes built into their IDs and some don't.


fix nfl game

also needs https://github.com/microsoft/pxt-common-packages/pull/1054, as it just crashes immediately otherwise due to starting with the __palette buffer

fix in kickoff was https://github.com/microsoft/pxt-kickoff/commit/ffc0ae261a568f2e8d80a3454b43920d9c1111e5, this is updating those refs~


Move this repo to https://github.com/mozillaweb ?

There's a new mozillaweb org for website repos. Should this repo move there to improve its discoverability?

Checking now (a bit late), it does not seem this org still exists. Closing...


Purging Events Not Working

Line 21 in purgeEvent/go has the wrong collection. Need to revisit this code to make sure it's parsing the database correctly.

I made significant changes to this code but it still does not seem to be working. I also need to change the logging to omit all the event that are excluded and just list the excluded pubkeys or maybe even just a log that says skipping purging for whitelisted pubkeys

this is now working for kwys listed specifically but it seems I've broken keys that should be whitelisted by domain

marking this as fixed but I would like to have more testing moving forward


Add clustering capabilities for Myricom SNF

Clearly describe the problem and solution
Current Myricom SNF plugin causes Moloch to fail to start if multiple processes are assigned to a given capture board even when environment variables and flags are set properly.  This adds support for clustered multi-process listeners gaining parity with the features in pfring and tpacketv3/AF_PACKET.
The design of this PR requests rings for all 3 processes (which is required in SNF) within each process, but then binds each process to only its rings.  This is accomplished by multiplying the required number of rings by the number of cluster members, and then binding each cluster member to its own slice of rings.
Further details are in the README changes.
Relevant issue number(s) if applicable
Addresses: https://github.com/aol/moloch/issues/1118
Did you run npm run lint from the viewer or parliament directory (whichever you are making changes to) and correct any errors
Did you Ensure that all tests still pass by navigating to the tests directory and running ./tests.pl --viewer
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

thanks!  Could you also do a PR for aol/molochweb and add to the settings.html


Cosmos DB property "$type" is skipped when serializing to JSON

My Cosmos DB NoSQL-container contains documents with a string property $type (possibly nested within the object), e.g.
{
    "id": "fb9351ca-6459-4b03-af08-43c488cc748d",
    "foo": {
        "$type": "typeA",
        "bar": "baz"
    }
}

Exporting the container to a JSON file results in documents where $type is skipped:
{
    "id": "fb9351ca-6459-4b03-af08-43c488cc748d",
    "foo": {
        "bar": "baz"
    }
}

Duplicate of #97


Fix : There bug/error occur in profile.html which creates conflicts.

Description

Fix:  There bug/error occur in profile.html which creates conflicts.
This pull request addresses a minor usability issue. A bug was identified in profile.html that caused conflicts when rendering user profiles. This issue led to unexpected behavior, such as incorrect data being displayed or elements not appearing as intended.

Changes Made :

Correct the indentations which causes conflicts.

Rationale :

The primary goal of these changes is to enhance the stability and usability of the profile page. By addressing the bug directly, users will have a smoother experience without the disruptions caused by the previous errors.
The modifications also ensure that the codebase remains maintainable and adheres to best practices, which will benefit future development efforts.

Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@anuragverma108 ,  @abhi03ruchi  assign me this pls.. with labels of Gssoc extd , hacktoberfest and level 2 i need points so, it would be great!✨

@abhi03ruchi hey can u assigned me? why closed this..

what have u done ? nothing .
work on some real issue .

Ok @abhi03ruchi i'll do it


tests/eas/acceptance: Increase required busy time for idle pull test

90% is too forgiving for EXPECTED_BIG_BUSY_TIME_PCT, it allows traces to
pass the "Bug cpus are fully loaded" test even when 100% tasks have run
on LITTLE CPUs while big CPUs are idle for significant periods.
Since traces contain a perdiod of inactivity at the beinning, to require
99% busy time it's necessary to adjust the trace window over which
assertions apply. Instead of starting at t=0 we use
SchedMultiAssert.getStartTime to find the time at which the first task
started, and use that as the start of our window.

@JaviMerino to me this makes sense. If you agree I can merge it.

Thanks!

@bjackman next time please assign me or mention me in the comments in order for me to get a notification.


Repo Indexer Type Elastic Search does not work with new migrated repositories

Gitea version (or commit ref): Gitea version 1.15.0 built with GNU Make 4.3, go1.16.7 : bindata, timetzdata, sqlite, sqlite_unlock_notify
Git version:git version 2.30.2
Operating system: docker

Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No (the problem is about code indexer so I can't do it there)

Log gist:

I set logo level to INFO in app.ini
Description

I'm trying to build a gitea with Code Search feature. And I hope it will immediately refresh index after any updates.
I choose elastic search as the engine and in most cases it works well. But when I migrated a repository from Github and tried using Code Search feature on the UI to find a line from that repository, it did not return any output. And in log it seems like that gitea didn't report to elastic search after migrating.
I have to manually commit something new to the new repo to trigger report to elastic search, or delete the old elastic search topic and create a new index.
I've found a related issue: #16152, but my search engine is different so I still post a new issue.
Because the original log does not provide enough info for me, I add a few log.Info codes with "zyx test" as prefix, and tactically it won't affect the original progress.
the log

In MigrateRepositoryGitData() , gitea has already set repo.IsEmpty to false(means that the repo in Github is not empty). But when doing NotifyMigrateRepository(), gitea create a new repo, so it still thinks the repo is empty and won't report to elastic search.
So my question is, is there a special reason for gitea to consider new migrated repo as empty? If so, it seems to me that NotifyMigrateRepository() shouldn't check repo.IsEmpty field when reporting to code search engine.
relevant code NotifyMigrateRepository(with my log codes)

Thanks for reading my question, look forward to your kind reply~
Screenshots

@Joannazyx Could you help to confirm that #16991 fix this?

@lunny thanks for your reply and timely bugfix.
I rebuild gitea from the lastest main branch, and #16991 fix this.
Now after migrating a new repo, gitea will immediately report to elastic search engine.


Use object type over nested type with nested structures?

Hi,
Similar to what I believe is implemented here: https://github.com/elastic-rs/elastic/issues/72. Is there a way to tell elastic-rs that I want it to auto derive the mapping yet override the type to object instead of nested?
#[derive(ElasticType)]
pub struct B {
    pub b: String,
}

#[derive(ElasticType)]
pub struct A {
    pub a: String,
    #[elastic(ty="object")]  <- Prefer object over nested?!
    pub b: Option<B>
}

Apologies if this already exists and I have just totally glossed over it. Currently when I look in the elastic mapping my nested structures are always type nested which I do not want, I do not need the nested overhead and hence need these to be type object but do not really want to create these mappings by hand. Otherwise awesome library :D

Hi @AlexKornitzer! Yeh, this is a bit of a deficiency at the moment. The DocumentMapping type isn't a perfect fit for how documents are actually stored in Elasticsearch. The way things work now, if you want B to be mapped as an object in A then we need to configure it on B's mapping. So right now the best we can do is tell B to always be mapped as object when it appears in another type by deriving DocumentMapping explicitly on B, and setting its data_type to OBJECT_DATATYPE.
#232 should make it more ergonomic to use a different type for documents that appear as fields in other documents, so it'll look closer to the example you posted.

Ah that first link looks like what I need, still beats writing the mappings by hand :) Let me try that out, I must have glanced over it when reading the docs.

I think the mapping is all a bit buried in the docs at the moment, because the structure doesn't really make it easy to find something if you don't already know what you're looking for.
Some clearer examples at the top level would probably be helpful.

No worries, the link you posted above has done what I needed. Thanks for the help, feel free to close this issue.

Thanks @AlexKornitzer! Feel free to open new issues for any other questions you have.


Remove unnecessary null pointer checks

An extra null pointer check is not needed in functions like the following.

Anvil::GLSLShaderToSPIRVGenerator::bake_spirv_blob_by_calling_glslang
Anvil::ObjectTracker::destroy
GenericPool

Nice observation, thanks. Any chance you could consider contributing a merge request which would remove all the extraneous checks? Thanks!

Fixed internally.


Downtime tab visible from Crash's 5e downtime tracking module

When Crash's 5e Downtime Tracking module is activated, the Downtime tab is visible alongside Inventory and Biography. It's very weird for a chest to have downtime! Unless it's a mimic...

You can uncheck the Show Downtime Tab (NPCs) in the Crash's 5e Downtime Tracking module.


calcparam_desoto fails with cryptic error when weather.temp_air has dtype=object

Hi,
Describe the bug
I had the following error and it took a while for me to find out the cause:

File "", line 1, in
mc.run_model(times=time_mc, weather=weather)
File "/home/johanneskaufmann/.virtualenvs/pfp-gUXDy98y/lib/python3.6/site-packages/pvlib/modelchain.py", line 848, in run_model
self.dc_model()
File "/home/johanneskaufmann/.virtualenvs/pfp-gUXDy98y/lib/python3.6/site-packages/pvlib/modelchain.py", line 409, in singlediode
self.temps['temp_cell']))
File "/home/johanneskaufmann/.virtualenvs/pfp-gUXDy98y/lib/python3.6/site-packages/pvlib/pvsystem.py", line 307, in calcparams_desoto
self.module_parameters['dEgdT'], kwargs)
File "/home/johanneskaufmann/.virtualenvs/pfp-gUXDy98y/lib/python3.6/site-packages/pvlib/pvsystem.py", line 1066, in calcparams_desoto
(np.exp(EgRef / (k(Tref_K)) - (E_g / (k(Tcell_K))))))
AttributeError: 'float' object has no attribute 'exp'

I used temperature data which I input from some csv files using pd.read_csv, looking like this:

weather.temp_air.head()
Out[18]:
2017-01-03 00:00:00+09:00    10.292
2017-01-03 00:30:00+09:00    10.124
2017-01-03 01:00:00+09:00     9.987
2017-01-03 01:30:00+09:00     9.856
2017-01-03 02:00:00+09:00     9.748
Name: temp_air, dtype: object

After some searching, I found out that the error occurs, because the Series has the attribute dtype: object instead of float. Consequentially, the error disappears when I performed
weather = weather.astype(float)
To Reproduce
Provide a weather-DataFrame with dtype: object for modelchain.run_model with dc_model='desoto'
Expected behavior
No error in output calculation.
Versions:

pvlib.__version__: 0.6.0
pandas.__version__:  0.23.4
python: 3.6

I am a bit unsure, if this constitutes a bug. I feel that to a non-technical user, this behaviour comes unexpected and is a bit confusing, because the weather data looks and feels valid.
What do you think? Should the method perform an explicit type conversion and/or provide a more detailed/customized error message?

Perhaps the same problem as https://stackoverflow.com/questions/54484478/attributeerror-in-sapm-celltemp-function-when-using-15-minute-weather-data-res#comment95852619_54484478
I would not call this a pvlib bug. Did pandas or numpy change something recently or has this always been the behavior? It's hit multiple users so I'm open to a ModelChain.run_model warning if weather contains object dtypes.
I don't think we should push type checking down to the low-level functions, nor do I think we should convert types.

Thanks for the input! I'll see if I get around creating a PR to warn users.

(np.exp(EgRef / (k(Tref_K)) - (E_g / (k(Tcell_K))))))
AttributeError: 'float' object has no attribute 'exp'

I puzzled over this bit when the stackoverflow question first came up.  If the error message is accurate, it suggests the the name np has been reassigned inadvertently somewhere..

If the error message is accurate

The error message is misleading - the exception occurs within the line rather than at the line (if that makes any sense). Related issues have been reported on the numpy issue tracker and stackoverflow for many years.

@jkfm any objections to closing this as a pvlib issue?

No objections! :)


Limit KubernetesApplications to KubernetesCluster in their namespace

Description of your changes

Fixes #1000
Currently KubernetesApplications are scheduled to a KubernetesCluster in any namespace, which is surprising behaviour for two namespaced resources.
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant documentation, examples, or release notes.
[ ] Updated the RBAC permissions in clusterrole.yaml to include any new types.

I've tested this as follows:
# The default has a KubernetesCluster, so the KubernetesApplication is scheduled and deployed
$ kubectl -n default get kubernetescluster
NAME                     STATUS   CLASS-KIND        CLASS-NAME         RESOURCE-KIND   RESOURCE-NAME                          AGE
wordpress-demo-cluster   Bound    GKEClusterClass   standard-cluster   GKECluster      default-wordpress-demo-cluster-n8bb8   50m
$ kubectl -n default get kubernetesapplication
NAME             CLUSTER                  STATUS      DESIRED   SUBMITTED
wordpress-demo   wordpress-demo-cluster   Submitted   3         3
$ kubectl -n default get kubernetesapplicationresource
NAME                        TEMPLATE-KIND   TEMPLATE-NAME   CLUSTER                  STATUS
wordpress-demo-deployment   Deployment      wordpress       wordpress-demo-cluster   Submitted
wordpress-demo-namespace    Namespace       wordpress       wordpress-demo-cluster   Submitted
wordpress-demo-service      Service         wordpress       wordpress-demo-cluster   Submitted

# The nope namespace has no KubernetesCluster, so the KubernetesApplication remains pending
$ kubectl -n nope get kubernetescluster
No resources found in nope namespace.
$ kubectl -n nope get kubernetesapplication
NAME             CLUSTER   STATUS    DESIRED   SUBMITTED
wordpress-demo             Pending


Optionally build worldspawn brushes as separate objects

I have a use-case that I think would require building each worldspawn brush as a separate mesh/collider.
I've been implementing a way to determine footsteps/noise level in my game, and I was testing it on a scene made with CSG objects. When I tried using it on a TB map I realized TBLoader builds the entire map as a single mesh/collider, and I'm not sure I can do it that way.
The process involves raycasting at the floor to get the material from the mesh, and then check it's metadata for the surface type. But if the map is a single mesh/collider, then it has several surfaces and materials, and I don't think I can get any information from the collision that could help me know which mesh surface is the one I'm touching.

An alternative would be to mark each floor brush in some way that makes TBLoader separate them (maybe I can even do that now, if I turn the floors into some kind of custom entity?), but I think that would be more laborious and error prone, and personally, I think it would reduce my willingness to iterate over and reshape my maps. Having worldspawn brushes separated might be just a simple checkbox, and I don't think the physics engine would care.

Actually, I think I was a bit off in the wrong direction. I think I'd need all brushes to be separated by material, regardless of them being worldspawn or not.

If you are using different textures for each surface, there will actually be multiple meshes generated as long as you are using "Build Meshes" and not the old CSG mesh system. Then, you can specify a custom material for each texture by having a .tscn or .material file in place where the texture would be (see the readme for more info on how this works).

I am using "Build Meshes", but it's still generating a single mesh, though.

Oh that's right, we recently merged a PR that turned it into singular meshes. I forgot about that, my bad 😅
Regardless, each surface is a different material. If you know the specific surface that was hit, you could get the material that way. I haven't looked too much into what info you have available though.

Ah I see, you would indeed only get the collider, the actual rendered mesh is separate from that. I wonder if this is something Qodot has solved? Otherwise, we could potentially just make a setting to create multiple objects 🤔

I wonder if this is something Qodot has solved?

It's not.
But they discuss it in their docs under Tips for Veterans # Footstep Sounds, maybe that's useful.

Qodot also creates a single mesh, but it creates a collision shape per brush. I don't think it really solves the issue, tbh. Kinda seems to me you end up in the same spot. You can get the collision point, and that's it...
Seems like their suggested solution is about the same as my alternative suggestion: create a solid class, make each floor a solid of that class. That separates the floor into entities on build, each with its own mesh and collider. It actually works perfectly, except its a bit tedious and prone to human errors. In my test map I found myself neglecting to turn some ramps and stairs into said entities. It's very easy to forget small parts as well.

I found a video where someone explains how to determine the mesh surface from the collision point information, but... he points out in the comments that this system ended up not being performant in practice when applied to many actors. And also as I suspected, it's a whole can of worms to delve into (using the MeshDataTool).
https://www.youtube.com/watch?v=WhMfocT9l-o


Save SVG block doesn't save image.

In Palette of media objects, I use "show" with any image of the Journal, and then I use the "save SVG" block and after running, it doesn't save the image. I can see the file at the journal, but when I open it, the file shows nothing.
Using Fedora 18, Sugar 0.110.
XO-1, XO-1.5, XO-1.75, XO-4
There are no error logs at the activity log.

Reproduced using TurtleConfusion-195 from activities.sugarlabs.org.  The journal object is empty; it contains no written data.  Are you using TurtleConfusion-195 or 2619b5235caec28d34da55a211e651e9e5eeac00?
@jammily, you must be looking in the wrong place for logs?  After reproducing a problem, start the Log activity, or use SSH and BASH, and in /home/olpc/.sugar/default/logs you will find a file org.laptop.TurtleConfusionActivity-1.log which contains important evidence of the problem;
Traceback (most recent call last):
  File "/home/olpc/Activities/TurtleConfusion.activity/TurtleArt/talogo.py", line 560, in doevalstep
    self.step.next()
  File "/home/olpc/Activities/TurtleConfusion.activity/TurtleArt/talogo.py", line 530, in _evalsym
    result = self.cfun.fcn(self, *self.arglist)
  File "/home/olpc/Activities/TurtleConfusion.activity/TurtleArt/taprimitive.py", line 314, in __call__
    return_value = new_prim.func(*new_args, **new_kwargs)
  File "/home/olpc/Activities/TurtleConfusion.activity/TurtleArt/tawindow.py", line 4709, in save_as_image
    datastore.write(dsobject)
  File "/usr/lib/python2.7/site-packages/sugar/datastore/datastore.py", line 385, in write
    transfer_ownership)
  File "/usr/lib/python2.7/site-packages/sugar/datastore/datastore.py", line 336, in _create_ds_entry
    transfer_ownership)
  File "/usr/lib/python2.7/site-packages/dbus/proxies.py", line 145, in __call__
    **keywords)
  File "/usr/lib/python2.7/site-packages/dbus/connection.py", line 651, in call_blocking
    message, timeout)
dbus.exceptions.DBusException: org.freedesktop.DBus.Python.ValueError: Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/dbus/service.py", line 707, in _message_cb
    retval = candidate_method(self, *args, **keywords)
  File "/usr/lib/python2.7/site-packages/carquinyol/datastore.py", line 312, in create
    lambda * args: self._create_completion_cb(async_cb,
  File "/usr/lib/python2.7/site-packages/carquinyol/filestore.py", line 47, in store
    raise ValueError('No file at %r' % file_path)
ValueError: No file at dbus.String(u'/tmp/turtle_output.svg')

The error prevents the data being written.
TurtleConfusion was based on TurtleArt and last major update was in 2013.  Looking at the TurtleArt directory in TurtleConfusion, and comparing to the TurtleArt directory in TurtleArt https://github.com/sugarlabs/turtleart-activity there are plenty of changes that TurtleArt has undergone that TurtleConfusion has not.
@walterbender, what do you recommend?

This is most likely fixed now with the latest commits which bring Turtle Confusion up to date with Turtle Art.


Precompilation warning

With Julia 1.10.2 with updated modules as of today (2024-3-29), the following warning is shown for CTProblems, CTFLows, CTDirect, OptimalControl and CTBase:
┌ CTBase
│  WARNING: Method definition apply_recipe(Base.AbstractDict{Symbol, Any}, CTBase.OptimalControlSolution,
|  Union{Symbol, Tuple{Symbol, Integer}}, Union{Symbol, Tuple{Symbol, Integer}}) in module CTBase at
|  /Users/LOGIN/.julia/packages/RecipesBase/BRe07/src/RecipesBase.jl:296 overwritten on the same line (check
|  for duplicate calls to `include`).
│  ERROR: Method overwriting is not permitted during Module precompilation. Use `__precompile__(false)` to
|  opt-out of precompilation.
└

This should be ok now! Thanks @horasio!

thank you @ocots testing on my side on linux / x86 🤞🏾

I can confirm CTBase precompiles just fine now with CTDirect. Thanks !


SuperForms doesn't quite support input names including . (setError(), splitPath())

[x] Before posting an issue, read the FAQ and search the previous issues.

Description
Bug: input (path) names with . always treated as nested objects; input names cannot include .
Expected: input (path) names with . should match even if not nested; input names should be able to include .
If applicable, a MRE
https://stackblitz.com/edit/superforms-2-testing-tzpke3?file=src%2Froutes%2F%2Bpage.server.ts
In the screenshot below, Error from setError is not rendered next to the input with name user.email because setError() assumes paths with . are nested objects.

However dataType is not 'json'
If all instances of user.email are changed to email, setError() error is rendered as expected.

Why . is needed in input names:

Non-nested, non-object forms:

work without JS
slightly easier to work with

However the input names reflect a nested structure that has been flattened:

For simple unflattening on the server, etc.
https://www.npmjs.com/package/flat#unflattenoriginal-options
Lodash set/get/etc
Convention used by Felte

Workaround:

Use another character like - instead of . in input names
Convert input names to paths with . before passing to other API's like Lodash get()

This cannot be implemented, as the whole nested data functionality (FormPathLeaves, proxies, the validate method, get/set in onChange, etc) depends on dot path access, as in normal JS syntax. You have to use another character, or add the errors manually and keep the form without proxies and non-nested data.


Create a design system

Our current designs and implementation are pretty well-organized, however we lack a proper design system with guidelines etc. We should aim to create the following:

[ ] Shared Figma library - This will include palettes, shadows, font styles and any components that will help us prototype fast
[ ] Design guidelines wiki - Any guidelines we can gather, design language etc.
[x] Copy guidelines wiki - Guidelines on how to create copy and present a consistent image.
[ ] Design examples wiki - A shared document of how everything fits together
[x] Mockups and reusable resources - A first step towards a brand book
[x] Logo files - All versions of our logo and splash imagery for quick access

Note that some of the above can either live in a wiki here or we can create a dedicated repository for all of the design/branding/media work and add everything there.

Closing, as we're tracking this internally at this time.


[Codespaces] There is a confused dialog after opening the cloned repository in a new window

Environment: Remote Codespaces
Build Version: 20210609.3
Repro Steps:

Right click a static web app -> Select "Clone Repo..." -> Click "Open in New Window" when asking to open the cloned repository -> Open a new window in browser.

Return to the previous window.
Check whether there is a confused dialog.

Expect:
There is no confused dialog.
Actual:
There is a confused dialog.

More Info:
Steps details:

This is by-design.  Since Codespaces is running in the browser, VS Code requires user permission to open a new tab.  If you click "Open", it should grant access and the window should open.
You can see that it is planned behavior in this issue: https://github.com/microsoft/vscode/issues/122354
Also, this is somewhat external since the dialogue and command are both coming from the Git extension.  We don't actually control that behavior, we are just calling its git.clone command.


add proxy to proxy these urls to these sites

do this for security reasons

If I make it act as a proxy then I am essentially serving content off of my servers, which is bad. By only redirecting to direct links, it keeps Filmic from actually acting as any sort of mirror or anything.


Added an HTML demo page.

I made this simple little HTML demo that would with the G2P server script from the project.
I had to change the "access-control-allow-origin" header in the original script to allow it to work from within the browser context. Apart from that, no functionality was modified. If you accept this change, it would be worthwhile to mention somewhere that this header should be customized in case anyone decides to use it in production.

Hey cool thanks!  Definitely open to this sort of thing.  It would be especially useful to be able to perform some kind of collaborative editing and retraining on a given lexicon/model, but the scope of such an app is well outside anything I currently have free time for.  I'll merge it this weekend when I have a chance to run through it once.

This was obviously a very quick and simple little page. It seemed like using curl was not giving the demo enough justice.
Doing what you're saying isn't impossible and I will think about it, but it involves a few extra requirements, like a database. I mean, it would be possible to do this using files only, but that is usually both inelegant and inefficient. How many people do you think should be editing the lexicon at the same time? Anyway, I'd be interested in helping out, since it is something I thought of doing myself, as well.


[5.5] Bugfix in php artisan preset vue

Renames ExampleComponent.vue to Example.vue
laravel/framework/src/Illuminate/Foundation/Console/Presets/Vue.php tries to copy  __DIR__.'/vue-stubs/Example.vue',. Fixes error on php artisan preset vue:
 [ErrorException]
  copy(/Users/developers/Sites/newsophi/vendor/laravel/framework/src/Illuminate/Foundation/Console/Presets/vue-stubs/Example.vue): failed to open stream: No such file or directory

This seems to be fixed, but it's not in a tagged release, yet.
But i noticed something that could potentially be fixed, on this line, shouldn't that be ExampleComponent.vue?

@brunogaspar You're correct, this has been fixed! Great.
as for this line, i think it's meant to delete the example React component if one exists, so it is probably correct as-is.


[Bug] Public resumes don't look good on Mobile

Is there an existing issue for this?

[X] Yes, I have searched the existing issues and none of them match my problem.

Product Variant
Cloud (http://rxresu.me)
Current Behavior
On mobile, going to a link for a public resume doesn't appear very well on Galaxy S23. Despite the viewport metadata in the , the resume appears half cut off (only showing the left half of the resume).
Also you can't see the "download pdf" button unless you pan down to the bottom right. Relatedly, when the button overlaps the resume it can be very hard to read. I think it needs a background.
Expected Behavior
No response
Steps To Reproduce
No response
What browsers are you seeing the problem on?
No response
What template are you using?
None
Anything else?
No response

public resumes don't load on mobile
Details:

browser: duckduckgo
phone: iPhone 14 Pro
ios: 17.4.1

Similar issue on iOS - resumes don't load,  but desktop view does not work for me either (safari and firefox)


Upgrade dialog shown on Android 10,11 but not shown on Android 12

I did check multiple devices,
That update dialog is shown on android 10 and android 11 devices
But same app is not working update on android 12 devices.
Please suggest what I missed

same here


Hindi Translations for the Country Input in the Demographic form

Overview
We managed to source JSON files full of translations for countries for German, Spanish, Portuguese and French. But we didn't manage to find a file that would give us all the translations for Hindi. Google translate didn't seem quite adequate when we checked. Please note, this ticket is preferably completed by a Hindi speaker but not a requirement.
Action Items

[ ] Take a look at constants/countries.ts
[ ] For each country in the array, see that the label is keyed by locale. However, there is no locale for hindi.
[ ] We need a locale for hindi and to translate each country into hindi.
[ ] You can decide whether to look for a premade hindi JSON file like those found here. Or whether to update the JSON file manually.
[ ] Once the hindi file has been updated, please test the sign up flow by following the next steps:
[ ] Ensure the front and backend code is running with the database seeded.
[ ] Go to localhost:3000/auth/register in Hindi
[ ] Create an account by clicking on the Hindi "Get started" button.
[ ] Fill out the sign up survey forms on /account/about-you
[ ] Ensure that the country options served up are in Hindi rather than english
[ ] Run all cypress tests to ensure that everything is working smoothly

Resources/Instructions

See bloom backend for instructions on the backend setup
See #908 for PR updating country input options
See Cypress.md for instructions on cypress

Hi @kyleecodes , Can I work on this ?

@kyleecodes @eleanorreem Hi, I would like to work on this issue for two main reasons. First, being an Indian, I can contribute by providing accurate Hindi translations. Second, I have previously contributed to this project, giving me a solid understanding of the codebase and its structure.


icpsr ids - fix issue #601

Fixes #601
Current urls for getting Vote View data are being depreciated. This PR updates the urls in icpsr_ids.py script to adhere to new format. Script also had to be updated to handle the new data being returned by Vote View. Ran the script and updated icpsr ids for legislators-current.yaml. Now, the icpsr_ids.py script correctly queries Vote View and sets the icpsr id for current and historical legislators (based off congress number passed to script).

Thank you!!


FileNotFoundError: [Errno 2] No such file or directory: 'data/pretrained_model/resnet101_caffe.pth'

hi, when I run the script of "trainval_net.py", it reports that No such file or directory: 'data/pretrained_model/resnet101_caffe.pth'

Hi
This repo modified from https://github.com/jwyang/faster-rcnn.pytorch . I used the pre-trained weights provided by that repo.


The onSuccess callback has confusing behavior when specified on useMutation vs. on mutate

When I use the useMutation hook and then call the mutate function multiple times, the onSuccess callback has a confusing behavior for which I can't see a good reason.
When I specify onSuccess globally, as useMutation option, and then do 4 mutations, the callback is called 4 times:
function MutateGlobal() {
  const { mutate } = useMutation(
    async ({ id }) => {
      await new Promise((r) => setTimeout(r, 1000));
    },
    {
      onSuccess: (_, { id }) => {
        console.log("global onSuccess:", id);
      },
    }
  );

  React.useEffect(() => {
    for (let i = 0; i < 4; i++) {
      mutate({ id: i });
    }
  }, []);

  return null;
}

This logs to the console:
global onSuccess: 0
global onSuccess: 1
global onSuccess: 2
global onSuccess: 3

However, when I specify a local onSuccess on each mutate call:
function MutateLocal() {
  const { mutate } = useMutation(async ({ id }) => {
    await new Promise((r) => setTimeout(r, 1000));
  });

  React.useEffect(() => {
    for (let i = 0; i < 4; i++) {
      mutate(
        { id: i },
        {
          onSuccess: (_, { id }) => {
            console.log("local onSuccess:", id);
          },
        }
      );
    }
  }, []);

  return null;
}

only the last mutation is logged:
local onSuccess: 3

I know this is documented and probably intentional, but still I don't understand how it can possibly make sense? A global callback is diligently called for all mutations, but local, supposedly independent callbacks influence and cancel each other? If anything, it should be the other way around. But the ideal behavior I'd expect is to call onSuccess on each mutation, no matter how it's defined.
onError and onSettled has the same behavior.
Tested on React Query 3.34.0.

as you said, its intentional. I see that we've only documented it in the api reference, but it likely also should be mentioned here under additional callbacks.
I think the reason behind it is separation of concerns. useMutation callbacks will always fire because they are used for things required for the mutation to work properly, like optimistic updates and query invalidation.
the callbacks on onSuccess are mostly for component-specific side effects. They will not be called at all if the component unmounts before the mutation finishes, for example. They are useful for triggering redirects or showing error notifications.
You can also get more fine-grained control with mutateAsync, but don't forget error handling, as mutateAsync returns the Promise:
  React.useEffect(() => {
    for (let i = 0; i < 4; i++) {
      mutateAsync({ id: i }).then(() => console.log("local onSuccess:", i));
    }
  }, [mutateAsync]);

the callbacks on onSuccess are mostly for component-specific side effects.

That explains why it's convenient that the local onSuccess is cancelled on unmount: if the intended use case is to update the component UI with some setState, it saves me from doing the is-mounted check myself.
But calling it only for the last mutate call? I don't see a good rationale for that. Imagine I'm rendering a todo list where each item has a trash icon, and clicking on that icon triggers a mutate({ todoId }) call, and I use local onSuccess to do a local UI effect on successful deletion, like showing some notice or "undo" button. Then, as I user, suppose I delete two items quickly after each other, before the first request finishes. Then I get only one onSuccess call. No UI update for the first deletion.
At this moment, my takeaway would be to avoid mutate({ onSuccess }), because its behavior can be erratic, and always do mutateAsync().then(onSuccess) or useMutation({ onSuccess }) instead.

But calling it only for the last mutate call? I don't see a good rationale for that. Imagine I'm rendering a todo list where each item has a trash icon, and clicking on that icon triggers a mutate({ todoId }) call, and I use local onSuccess to do a local UI effect on successful deletion, like showing some notice or "undo" button. Then, as I user, suppose I delete two items quickly after each other, before the first request finishes. Then I get only one onSuccess call. No UI update for the first deletion.

For this specific use-case, I would move the mutation down to each item and give every item their own mutation. I haven't really fired the same mutation multiple times to be honest :)
const Item = ({ id }) => {
  const { mutate } = useMyDeleteMutation(id)
  return <Button onClick={() => mutate({ onSuccess: () => ... )}) />

I haven't really fired the same mutation multiple times to be honest

Our specific use case, the one where we discovered this issue during review, was "deleting selected items":
const { mutate } = useDeleteMutation();
const [selectedIds, setSelectedIds] = useState([]);
const deleteSelected = () => selectedIds.forEach((id) => mutate({ id }, { onSuccess }));
<button onClick={ deleteSelected }>Delete selected</button>

I guess it's always possible to refactor away from the "multiple mutate calls with a local onSuccess callback" pattern, but ... even if a bug has a convenient workaround, it still continues to be a bug doesn't it?
I know, the mutate docs document this:

If you make multiple requests, onSuccess will fire only after the latest call you've made.

but it still feels to me like a documented bug rather than a reasonable behavior.

I see how that can happen with the above code. Looking through the code, all of it is intentional though. We only have one observer, running mutate again will unsubscribe the current observer and create a new one, which is equivalent of unmounting the component. So without a large change, we can't really have the one case we want (only call callbacks if the component is still mounted) without having the second case (only call it for the latest one)

Thanks @TkDodo for your time looking at this issue. I think I now understand the pros and cons of the onSuccess callback, and that calling mutate multiple times can have some weird results. After all, the isSuccess field has a similar trait -- after:
const { mutate, isSuccess } = useMutation( ... );

and calling mutate multiple times, the isSuccess value will come from the observer for the last mutation, and the previous ones are kind of forgotten.


Gsoc2021 double float chris modify and test

Hi @sinandredemption this PR merges your recent developments and adds a couple of little changes that have been made to get CI going green.
Please feel free to update your branch from develop of the fork any time. I have tried to incorporate your changes into develop via merging your active branch into mine, then merge mine to develop of the fork.

There are still isolated, spurious test failures for extreme values of exponent. I think we might need to look into some tolerance(s) at some point later.

There are still isolated, spurious test failures for extreme values of exponent.

@ckormanyos what specific tests are your referring to that are failing?

what specific tests are you referring to that are failing

Hi @sinandredemption I failed to make a snapshot of the failed test and simple re-ran the CI job.
I will attempt to reproduce this claim of failed test case(s) and when (or if) I have an actual failing test case (or if one can be found), i will post it.

attempt to reproduce this claim of failed test case(s)

There is at least one failure in, I believe, multiplication of double_float<float> here. It seems like it might be in multiplication.
There is another failure here that I am having difficulty interpreting.

Thank you, @ckormanyos. I will try to reproduce and fix these issues and report back.


[3px] Display a form for adding reservations

Display Add Classes Form:

Create a form component to enable the user to add cars
Create input fields for relevant details
Create a button to save the entered data to the database

Add Integration Tests and Apply Proper Styling:

Add integration tests to ensure the proper functioning of the component
Apply proper styling to make the form visually appealing and user-friendly

https://github.com/AhmadiGu/Book-an-Appointment_front/pull/4


Panel Version
1.11.7
Wings Version
1.11.13
Service
Portal Knights
Modified
No, I did not modify the egg
Expected Behavior
Install the game, then start up.
Actual Behavior
Game installs but crashes at startup.
@sSteamCmdForcePlatformType" = "windows"
Success! App '1007' already up to date.
ERROR! Failed to install app '374040' (No subscription)
First launch will throw some errors. Ignore them
:/home/container$ cd /home/container/dedicated_server; xvfb-run --auto-servernum wine pk_dedicated_server.exe -config server_config.json -log server.log
_XSERVTransmkdir: ERROR: euid != 0,directory /tmp/.X11-unix will not be created.
Application could not be started, or no application associated with the specifie
d file.
ShellExecuteEx failed: Environment variable not found.
X connection to :99 broken (explicit kill or server shutdown).
X connection to :99 broken (explicit kill or server shutdown).
X connection to :99 broken (explicit kill or server shutdown).
container@pterodactyl~ Server marked as offline...
[Pterodactyl Daemon]: ---------- Detected server process in a crashed state! ----------
[Pterodactyl Daemon]: Exit code: 1
[Pterodactyl Daemon]: Out of memory: false
[Pterodactyl Daemon]: Aborting automatic restart, last crash occurred less than 60 seconds ago.
Steps To Reproduce
Step 1. Install the game from the Egg
Step 2. start the game.
Install logs
https://ptero.co/pofiwereje

As mentioned in the readme you need an account that owns the game with steamguard off.


not working in Yosemite

It crashes with:
01-10-14 21:47:14,395 ClipMenu[670]: CoreData: error: fatal: This NSManagedObjectContext was created on the main thread and illegally passed to a background thread.

Hi, everybody.
Should we wait for complete working release of ClipMenu?
I'm missing other features, beside just history :)
2014-11-04 3:20 GMT+01:00 Barry Allard<EMAIL_ADDRESS>
Not to "works for me" disregard a possible issue but one box w/ clean
Yosemite install and 0.4.3 appear to work:

it tracks clipboard changes.
clicking drop down status menu icon copies to the clipboard
correctly.

I hope it works out.
—
Reply to this email directly or view it on GitHub
https://github.com/naotaka/ClipMenu/issues/8#issuecomment-61584221.

Looks like the project is dead :(
Does anybody know the analog of ClipMenu?

@adenisovgit there are a lot of clipboard managers in the App Store these days, a few of which seem to match ClipMenu pretty well in terms of features (history, snippets, etc.). I see http://www.apprywhere.com/copy-em-paste.html looks pretty capable, for example.


chore: fix zsh prompt

Signed-off-by: Keming<EMAIL_ADDRESS>
fix #617

ref to:

https://gist.github.com/romkatv/2a107ef9314f0d5f76563725b42f7cab
https://superuser.com/a/974942
https://zsh.sourceforge.io/Doc/Release/User-Contributions.html#Prompt-Themes

To try this new zsh theme, you may need to run envd prune && rm -rf $HOME/.cache/envd

I thought it was a feature before....

We want to show (envd) instead of (envd)(envd) in zsh. Now we always activate conda, thus (envd)(envd) is shown.

Should we open it since we already have #681 ?

Should we open it since we already have #681 ?

Yes.

close this since we have a better solution in #681


Add test file for EazyDiner module

Note

Test all the methods of the eazydiner module and add a test file for the same.
Add test file in directory src/test.
Check the codechef_test.py file as a reference to get an idea about structuring the code.
Use unittest only.

Hi, please assign me this issue.

Go ahead @Master-Keshav


Make the tests run

Update MANIFEST.in and make the existing tests (haha) run.

closed in favor of https://github.com/inveniosoftware/invenio-users-resources/pull/9


Snap crashes when opening 'Help' > 'About' in Ubuntu 21.04

Describe the bug
Installed insomina 2021.3.0 using snap in Ubuntu 21.04.  When trying to open 'Help' > 'About', the app crashes.
To Reproduce

Go to 'Help'
Click on 'About.'
The app crashes

Expected behavior
The 'About' window should be opened.
*Screenshots
https://user-images.githubusercontent.com/8375708/123530327-4a6a8000-d6cf-11eb-8cd6-9e8ed2bc6a7c.mp4
Desktop:

OS: Ubuntu 21.04
Installation Method: snap
App Version 2021.3.0

Additional context
This is the info shown in the terminal debug:
22:20:30.630 › [ga] Page /debug

(insomnia:217360): Gtk-WARNING **: 22:20:39.242: Could not load a pixbuf from icon theme.
This may indicate that pixbuf loaders or the mime database could not be found.
**
Gtk:ERROR:../../../../gtk/gtkiconhelper.c:494:ensure_surface_for_gicon: assertion failed (error == NULL): Failed to load /snap/insomnia/132/data-dir/icons/Yaru/48x48/status/image-missing.png: Unable to load image-loading module: /snap/insomnia/55/gnome-platform/usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/loaders/libpixbufloader-png.so: /snap/insomnia/55/gnome-platform/usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/loaders/libpixbufloader-png.so: cannot open shared object file: No such file or directory (gdk-pixbuf-error-quark, 5)
Aborted (core dumped)

@santiagofn are you still able to reproduce this on latest 2021.7.2?
Here's my attempt on that version, using Kubuntu 21.10 and installing/running with snap:

Hi @filfreire! I've gave it shot with my current Ubuntu (21.10) and the bug isn't reproducible any more:

Thanks!


adding support for linux-ppc64le

Hi there,

made change to support linux-ppc64le
required for using this action on self-hosted ppc64le github runner

please review and merge.

Hello @adilhusain-s. Sorry for the late response. Could you please sync with the main branch?

Hello @adilhusain-s. For now I'm going to close the pull request because we can't change ppc64 to ppc64le. The architecture ppc64 is used for aix os.


Compatibility with React Router v7

Hi 👋 React Router v7 was released today. Will you update the library to be compatible with the new version?

Hi! Yes, at first glance it looks easy. However, it'll likely be a part of the 2.0 release.

I updated the library for React Router v7 and released it just now.

Yeah, thanks a lot 🙏


Add version: Coder.Coder version 2.9.4

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

🤖 cc: @deansheather @matifali

Hi @cdrci,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 877.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-04-22 13:40:05.750 [FAIL] Installer failed security check. Url: https://github.com/coder/coder/releases/download/v2.9.4/coder_2.9.4_windows_amd64_installer.exe Result: 0x80004005

(Automated response - build 877.)

@stephengillie It is undoubtedly a false positive, as #150533 has passed the validation.

My scripted automation will retry this about every 18 hours until it passes.


extra interval

chrome 59.0.3071.109

trello-linux-0.1.9

ubuntu 17.04 x64 unity

this is related do #12 , I'm making a release to normalize the behaviour in all operating systems


Add update containers role

This work is inspired from @marios Dataplane-adoption update openstack version work[1][2].
The main goal is this role is to generate cr file so that users can apply it to update all openstack services, ansiblee, EDPM OS_Image containers by using oc apply.
It will replace set_openstack_containers role in future.
Links:
[1]. https://github.com/openstack-k8s-operators/data-plane-adoption/pull/415 [2]. https://github.com/openstack-k8s-operators/data-plane-adoption/pull/420
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

In general - I am fine with merging this as it's a new role.
Only concern is that https://github.com/openstack-k8s-operators/ci-framework/pull/1574/files#diff-6c85baae2bd6f5668af7dc29b3c8ff9d02f0c664e7e220dc293545a9c3273907 seems like a lot to maintain. If any operator is added/removed etc.

In general - I am fine with merging this as it's a new role. Only concern is that https://github.com/openstack-k8s-operators/ci-framework/pull/1574/files#diff-6c85baae2bd6f5668af7dc29b3c8ff9d02f0c664e7e220dc293545a9c3273907 seems like a lot to maintain. If any operator is added/removed etc.

+1 May be consider getting the images from installed CRD itself to keep the list up to date https://github.com/openstack-k8s-operators/openstack-operator/blob/d2703d3a321c979dacaca95b5d4a634bf116e0db/apis/bases/core.openstack.org_openstackversions.yaml#L46

In general - I am fine with merging this as it's a new role. Only concern is that https://github.com/openstack-k8s-operators/ci-framework/pull/1574/files#diff-6c85baae2bd6f5668af7dc29b3c8ff9d02f0c664e7e220dc293545a9c3273907 seems like a lot to maintain. If any operator is added/removed etc.

+1 May be consider getting the images from installed CRD itself to keep the list up to date https://github.com/openstack-k8s-operators/openstack-operator/blob/d2703d3a321c979dacaca95b5d4a634bf116e0db/apis/bases/core.openstack.org_openstackversions.yaml#L46

Thank you @rlandy @karelyatin for the review. The idea looks good . If we get the list of installed CRD, then we need to do  find and replace operations to update the containers (Which is what we were doing in set_openstack_containers role, We want to avoid that).
May be we can add a validation in the role which will compare with the list of images from installed CRD and images list of the update_containers role. It will allow us to keep the role up to date and avoid breakage. What do you say?

In general - I am fine with merging this as it's a new role. Only concern is that https://github.com/openstack-k8s-operators/ci-framework/pull/1574/files#diff-6c85baae2bd6f5668af7dc29b3c8ff9d02f0c664e7e220dc293545a9c3273907 seems like a lot to maintain. If any operator is added/removed etc.

+1 May be consider getting the images from installed CRD itself to keep the list up to date https://github.com/openstack-k8s-operators/openstack-operator/blob/d2703d3a321c979dacaca95b5d4a634bf116e0db/apis/bases/core.openstack.org_openstackversions.yaml#L46

Thank you @rlandy @karelyatin for the review. The idea looks good . If we get the list of installed CRD, then we need to do find and replace operations to update the containers (Which is what we were doing in set_openstack_containers role, We want to avoid that).

IIRC in set_openstack_containers find/replace was done in CSV. But here AFAIU idea should be get the list of images from installed CRD and generate/apply OpenStackVersion CR dynamically based on passed registry specific parameters

May be we can add a validation in the role which will compare with the list of images from installed CRD and images list of the update_containers role. It will allow us to keep the role up to date and avoid breakage. What do you say?

In general - I am fine with merging this as it's a new role. Only concern is that https://github.com/openstack-k8s-operators/ci-framework/pull/1574/files#diff-6c85baae2bd6f5668af7dc29b3c8ff9d02f0c664e7e220dc293545a9c3273907 seems like a lot to maintain. If any operator is added/removed etc.

+1 May be consider getting the images from installed CRD itself to keep the list up to date https://github.com/openstack-k8s-operators/openstack-operator/blob/d2703d3a321c979dacaca95b5d4a634bf116e0db/apis/bases/core.openstack.org_openstackversions.yaml#L46

Thank you @rlandy @karelyatin for the review. The idea looks good . If we get the list of installed CRD, then we need to do find and replace operations to update the containers (Which is what we were doing in set_openstack_containers role, We want to avoid that).

IIRC in set_openstack_containers find/replace was done in CSV. But here AFAIU idea should be get the list of images from installed CRD and generate/apply OpenStackVersion CR dynamically based on passed registry specific parameters

May be we can add a validation in the role which will compare with the list of images from installed CRD and images list of the update_containers role. It will allow us to keep the role up to date and avoid breakage. What do you say?

@karelyatin Here is the cr  from openstackversion interface. As you can see there some of the images follow different url pattern.
apacheImage, ceilometerSgcoreImage,  edpmNodeExporterImage, ManilaShare, CinderVolumes, Ansibleee.
We also update few images in edpm-image-builder and edpm-ansible pr.
If we update the registry specific params dynamically then we need to maintain another set of hacks to maintain the exceptions(Which I donot want to do that.)
Do let us know if there is a easier way to get this addressed. I am happy to modify the role.
For right now, I want to get this role merged and do a follow to add a validation check so that any image getting removed from openstackversion interface get catched with in the role.

/lgtm

/lgtm

/lgtm

In general - I am fine with merging this as it's a new role. Only concern is that https://github.com/openstack-k8s-operators/ci-framework/pull/1574/files#diff-6c85baae2bd6f5668af7dc29b3c8ff9d02f0c664e7e220dc293545a9c3273907 seems like a lot to maintain. If any operator is added/removed etc.

+1 May be consider getting the images from installed CRD itself to keep the list up to date https://github.com/openstack-k8s-operators/openstack-operator/blob/d2703d3a321c979dacaca95b5d4a634bf116e0db/apis/bases/core.openstack.org_openstackversions.yaml#L46

Thank you @rlandy @karelyatin for the review. The idea looks good . If we get the list of installed CRD, then we need to do find and replace operations to update the containers (Which is what we were doing in set_openstack_containers role, We want to avoid that).

IIRC in set_openstack_containers find/replace was done in CSV. But here AFAIU idea should be get the list of images from installed CRD and generate/apply OpenStackVersion CR dynamically based on passed registry specific parameters

May be we can add a validation in the role which will compare with the list of images from installed CRD and images list of the update_containers role. It will allow us to keep the role up to date and avoid breakage. What do you say?

@karelyatin Here is the cr  from openstackversion interface. As you can see there some of the images follow different url pattern. apacheImage, ceilometerSgcoreImage, edpmNodeExporterImage, ManilaShare, CinderVolumes, Ansibleee.
We also update few images in edpm-image-builder and edpm-ansible pr. If we update the registry specific params dynamically then we need to maintain another set of hacks to maintain the exceptions(Which I donot want to do that.)
Do let us know if there is a easier way to get this addressed. I am happy to modify the role. For right now, I want to get this role merged and do a follow to add a validation check so that any image getting removed from openstackversion interface get catched with in the role.

Re looking i see CRD don't have defaults injected, so likely need more work to have the list available, ok for me to look for that in followup.

/approve
We can refine once this role is actually put to use.


Receive "Can't bind to 'matTableFilter' since it isn't a known property of 'table'" after upgrading to version 1.2.1

I have just tried upgrading the mat-table-filter package to version 1.2.1 and am receiving the following error: "Can't bind to 'matTableFilter' since it isn't a known property of 'table'".
I am running Angular Materials 8.2.3.
Cheers

Thanks for clarifying. This is expected behavior. matTableFilter input is removed in b76b448. That was a breaking change.

Thanks for clarifying @HalitTalha


HowToAcknowledge not explicit in data type.

Text for the field in dataset_description.json in the current spec:
"OPTIONAL. Instructions how researchers using this dataset should acknowledge the original authors. This field can also be used to define a publication that should be cited in publications that use the dataset."
bids-standard/bids-validator#772 We had settled on the validator interpreting this field as a string. This breaks an example we have in bids-examples:
https://github.com/bids-standard/bids-examples/blob/master/ds000248/dataset_description.json#L10
Should a data type be added for this field, and should it be string, an array of strings, either a string or a an array of strings, or some other type.
From discussion with @effigies.

@bids-standard/raw

the example you link to was prepared by me: I use an array of strings ... each entry in the array is one source to acknowledge.
I see that this is not very machine readable and that a simple string would be more straight forward. And a string would also be more in line with what is written in the specification.

Well, I don't hear a great clamoring for preserving lists, and if it's just the one example, clarifying that it should be a string makes sense to me.
At least three OpenNeuro datasets will be affected:
$ grep -rI '"HowToAcknowledge":\[' ds*/dataset_description.json
ds000222/dataset_description.json:{"Name":"Sequential Inference VBM","BIDSVersion":"1.0.0rc1","License":"PDDL","Authors":["Thomas H B FitzGerald","Dorothea Haemmerer","Karl J Friston","Shu-Chen Li","Raymond J Dolan"],"HowToAcknowledge":["Citation of FitzGerald et al. Sequential inference as a mode of cognition and its correlates in fronto-parietal and hippocampal brain regions. PLoS Computational Biology (2017)","This data was obtained from the OpenfMRI database. Its accession number is ds000222"]}
ds000235/dataset_description.json:{"Name":"Whole-brain background-suppressed pCASL MRI with 1D-accelerated 3D RARE Stack-Of-Spirals Readout- Dataset 2","BIDSVersion":"1.0.0rc1","License":"PD","Authors":["Marta Vidorreta","Ze Wang","Yulin V. Chang","Maria A. Fernandez-Seara","John A. Detre"],"Acknowledgements":"Tiejun Zhao","HowToAcknowledge":["We  are  grateful  to  Tiejun  Zhao  from  Siemens  Healthcare  for  his  valuable  help  in  the  implementation of the online reconstruction algorithm of the sequence for the Siemens platform.","This data was obtained from the OpenfMRI database. Its accession number is ds000235."],"Funding":"NIH grants no. P41EB015893 and MH080729, and NSFC grant no. 81471644, and Hangzhou Innovation Seed Fund"}
ds000236/dataset_description.json:{"Name":"Whole-brain background-suppressed pCASL MRI with 1D-accelerated 3D RARE Stack-Of-Spirals Readout- Dataset 3","BIDSVersion":"1.0.0rc1","License":"PD","Authors":["Marta Vidorreta","Ze Wang","Yulin V. Chang","Maria A. Fernandez-Seara","John A. Detre"],"Acknowledgements":"Tiejun Zhao","HowToAcknowledge":["We  are  grateful  to  Tiejun  Zhao  from  Siemens  Healthcare  for  his  valuable  help  in  the  implementation of the online reconstruction algorithm of the sequence for the Siemens platform.","This data was obtained from the OpenfMRI database. Its accession number is ds000236."],"Funding":"NIH grants no. P41EB015893 and MH080729, and NSFC grant no. 81471644, and Hangzhou Innovation Seed Fund"}

Each of these was OpenFMRI, so we can go in and manually fix them up.

Well, here's a case of us hitting this in the wild: https://neurostars.org/t/problem-uploading-dataset-to-openneuro-not-a-valid-bids-dataset-error/5532

My 2c would be to enforce HowToAcknowledge as a string. It's intended to be added to human-centered documentation (manuscripts, etc), where having more than one sentence (as in all the linked cases) shouldn't be a hindrance, I'd think !

I agree with @emdupre that we should enforce it as a string

Sounds like consensus. I'll update those datasets on OpenNeuro. Would somebody like to propose some clarifying language for the spec?

Would somebody like to propose some clarifying language for the spec?

- OPTIONAL. Instructions how researchers using this  dataset should
- acknowledge the original authors. This field can also be used to define
- a publication that should be cited in publications that use the dataset.
+ OPTIONAL. A string of text containing
+ instructions how researchers using this  dataset should
+ acknowledge the original authors. This field can also be used to define
+ a publication that should be cited in publications that use the dataset.

I was looking for other examples, and at least in common derivatives, we state a type with:
REQUIRED. Boolean.

Would it work for people to specify types with:
OPTIONAL. String.

Or maybe:
OPTIONAL. `string`.

This would tie into @yarikoptic's suggestion in #350.

I would be +1 to specify the expected variable type like I commented before:

it'd be good to consistently advertise the expected data type in the spec and validate it.

On a related note:  has there been any consideration of an optional field
in the description for IRB information or is this something that goes into
the How-to-Acknowledge?   Most journals require authors to provide IRB
information for any data that they use in a publication.
On Thu, Nov 21, 2019 at 12:41 PM Ross Blair<EMAIL_ADDRESS>wrote:

Text for the field in dataset_description.json in the current spec:
"OPTIONAL. Instructions how researchers using this dataset should
acknowledge the original authors. This field can also be used to define a
publication that should be cited in publications that use the dataset."
bids-standard/bids-validator#772
https://github.com/bids-standard/bids-validator/issues/772 We had
settled on the validator interpreting this field as a string. This breaks
an example we have in bids-examples:
https://github.com/bids-standard/bids-examples/blob/master/ds000248/dataset_description.json#L10
Should a data type be added for this field, and should it be string, an
array of strings, either a string or a an array of strings, or some other
type.
From discussion with @effigies https://github.com/effigies.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bids-standard/bids-specification/issues/372?email_source=notifications&email_token=AAJCJOUAV4D4J7FILUNKNPTQU3I7FA5CNFSM4JQGRLEKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H3F5Q6A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJCJOXZVIW57DAYUXMB4WTQU3I7FANCNFSM4JQGRLEA
.

We are now quickly running from one issue into three issues with this thread. Let's divide and conquer.

@VisLab I have opened a new issue for your question: #379
regarding specification of datatypes, I suggest we continue discussion in #350

as for this thread: It was about whether we want to accept STRING only for the HowToAcknowledge field ... and make this clear in both validator and spec.
So far, everybody has agreed that STRING would be good. I suggest to go ahead with this, make the adjustments, and close this issue (continuing the open points in their own issues as linked above).

Sounds good to me.


fix(ci): Temporarily disable panic on subtree validation failure

Motivation
This PR disabled subtree validation panics until we fix subtree bug #7532.
Solution
Replace a panic with an error
Review
This is an urgent fix that is blocking unrelated PRs from merging.
Reviewer Checklist

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

This test is failing because the latest image is invalid in another way - it finished at 2 million blocks, rather than 2.2 million.
This is either a bug in the self-hosted runner CI workflow (which might have been fixed since), or an unrelated bug in Zebra. The simplest way to fix it is to delete the image. (We don't need it any more, because we can reproduce the bug.)
Here's how I searched for the latest image:
https://console.cloud.google.com/compute/images?hl=en&project=zfnd-dev-zebra&tab=images&pageState=("images":("f":"%255B%257B_22k_22_3A_22Created%2520by_22_2C_22t_22_3A10_2C_22v_22_3A_22_5C_22zfnd_5C_22_22_2C_22i_22_3A_22creator_22%257D%255D","s":[("i":"creationTimestamp","s":"1"),("i":"type","s":"0"),("i":"name","s":"0")]))


Fixed Tutorial Bugs and Reduced Total Run Cost

Description: Some of the tutorial graphs had some bugs in them. Also, some graphs ran really costly due to using GPT4 by default.
Bug Fixes: Resolved issues in the tutorial that were causing errors.
Cost Optimization: Converted all chat nodes to the cheapest model (GPT3.5 Turbo) to reduce the overall run cost of the application.
Testing:
All tutorial steps were tested and verified.
Monitored cost metrics to confirm reductions.
Passed all unit tests and built successfully.

@all-contributors please add @altaiiiir for maintenance


Implement QuotaResourceMutexFactory to synchronize based on QuotaResource value.

Codecov Report

Merging #2048 (4e110d9) into master (cd1cf0a) will decrease coverage by 58.54%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #2048       +/-   ##
=============================================
- Coverage     73.60%   15.06%   -58.55%
+ Complexity    10060     2281     -7779
=============================================
  Files           723      724        +1
  Lines         54941    54948        +7
  Branches       6797     6798        +1
=============================================
- Hits          40442     8279    -32163
- Misses        12341    46186    +33845
+ Partials       2158      483     -1675

Impacted Files
Coverage Δ

.../github/ambry/quota/QuotaResourceMutexFactory.java
0.00% <0.00%> (ø)

.../src/main/java/com/github/ambry/router/Router.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/com/github/ambry/store/Message.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/named/PutResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaMode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/KMSConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/SSLConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/router/ByteRange.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 569 more

Continue to review full report at Codecov.

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


TensorFlow Lite Model Maker has version conflict during install on RPi02W

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
https://www.tensorflow.org/lite/guide/model_maker
Please provide a link to the documentation entry, for example:
Model Maker Object Detection
Description of issue (what needs changing):
Cannot install tflite_model_maker and tflite_support on Raspberry Pi Zero 2 W
uname -a Linux raspbari17 5.10.63-v7+ #1488 SMP Thu Nov 18 16:14:44 GMT 2021 armv7l GNU/Linux
Clear description
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/user_guide/#fixing-conflicting-dependencies
Correct links
Is the link to the source code correct? n/a
Parameters defined
Are all parameters defined and formatted correctly? Yes
Returns defined
Are return values defined? n/a
Raises listed and defined
ERROR: Cannot install tflite-model-maker==0.1.0, tflite-model-maker==0.1.1, tflite-model-maker==0.1.2, tflite-model-maker==0.2.0, tflite-model-maker==0.2.1, tflite-model-maker==0.2.2, tflite-model-maker==0.2.3, tflite-model-maker==0.2.4, tflite-model-maker==0.2.5, tflite-model-maker==0.3.0, tflite-model-maker==0.3.1, tflite-model-maker==0.3.2, tflite-model-maker==0.3.3 and tflite-model-maker==0.3.4 because these package versions have conflicting dependencies.
The conflict is caused by:
tflite-model-maker 0.3.4 depends on tensorflow-addons>=0.11.2
tflite-model-maker 0.3.3 depends on tensorflow-addons>=0.11.2
tflite-model-maker 0.3.2 depends on tensorflow-addons>=0.11.2
tflite-model-maker 0.3.1 depends on tensorflow-addons>=0.11.2
tflite-model-maker 0.3.0 depends on tensorflow-addons>=0.11.2
tflite-model-maker 0.2.5 depends on tflite-support==0.1.0rc4
tflite-model-maker 0.2.4 depends on tflite-support==0.1.0rc4
tflite-model-maker 0.2.3 depends on tflite-support==0.1.0rc3.dev2
tflite-model-maker 0.2.2 depends on tflite-support==0.1.0rc3.dev2
tflite-model-maker 0.2.1 depends on tf-nightly==2.4.0.dev20200811
tflite-model-maker 0.2.0 depends on tf-nightly==2.4.0.dev20200810
tflite-model-maker 0.1.2 depends on tf-nightly
tflite-model-maker 0.1.1 depends on tf-nightly
tflite-model-maker 0.1.0 depends on tf-nightly
Usage example
Is there a usage example?
sudo python3 -m pip install tflite-model-maker
Request visuals, if applicable
Are there currently visuals? No but terminal output entered above.
Submit a pull request?
Are you planning to also submit a pull request to fix the issue? No, beyond my job grade. Sorry.

I've been getting the same error trying to install on an M1 Mac for a few months. I've tried to install:

In a Miniforge3 environment;
In a Python Docker image;
In a Miniconda3 x86 environment, using a version of Terminal running through Rosetta.

I was, however, able to install it using a Miniconda environment or a Python Docker image on Windows via WSL2.

@baqwas,
Could you let us know what is the version of tflite-model-maker and tensorflow installed?
Please refer requirements.txt for dependent libraries and let us know if it helps you? Thanks!

Hello @chunduriv,
Sorry, I couldn't locate any applicable requirements.txt file (related to TFL) except for Pete Warden's famous Magic Wand example. My installation steps are:
git clone https://github.com/tensorflow/tensorflow.git tensorflow
cd tensorflow && ./tensorflow/lite/tools/make/download_dependencies.sh
./tensorflow/lite/tools/make/build_rpi_lib.sh

I don't have the knowledge to loosen the version requirements. I tried the -Wno-error=class-memaccess option but that did not eliminate that specific error. The Flatbuffers team claim to have fixed the issue at their end several versions ago. Here is the advice from Rients at Q-Engineering (who success in leveraging RPi0 I was trying to emulate with RPi02W):
They repaired the issue on May 29, 2020. The next fixed version will be v2.0.0 released on May 10, 2021.
Looking at the download script of TF 2.6 and 2.7 you see they use v1.12.0. That is why it errors. (BTW, only with gcc 10.2.1 found in Bullseye).
TF has deprecate the script building of lite, leaving with the hint to use Bazel.
Tip: try replacing the v.1.12.0 with the v2.0.0.tar.gz in the download_dependencies.sh (tensorflow/lite/tools/make/)

I apologize for not being able to reply more precisely to your request for information (so that you can provide further guidance).
The hardware platform information is:

cat /proc/cpuinfo
...
processor	: 3
model name	: ARMv7 Processor rev 4 (v7l)
BogoMIPS	: 38.40
Features	: half thumb fastmult vfp edsp neon vfpv3 tls vfpv4 idiva idivt vfpd32 lpae evtstrm crc32
CPU implementer	: 0x41
CPU architecture: 7
CPU variant	: 0x0
CPU part	: 0xd03
CPU revision	: 4

Hardware	: BCM2835
Revision	: 902120
Model		: Raspberry Pi Zero 2 Rev 1.0

cat /proc/version
Linux version 5.10.63-v7+ (dom@buildbot) (arm-linux-gnueabihf-gcc-8 (Ubuntu/Linaro 8.4.0-3ubuntu1) 8.4.0, GNU ld (GNU Binutils for Ubuntu) 2.34) #1488 SMP Thu Nov 18 16:14:44 GMT 2021

cat /home/pi/projects/tensorflow/tensorflow/lite/micro/examples/magic_wand/train/requirements.txt
numpy==1.16.2
tensorflow==2.5.0

We're all doing TFL modeling on our desktops and downloading to our TinyML devices but being to do one stop shopping using RPi02W would be more productive, versatile and ubiquitous. Thanks for your help and understanding.
Kind regards.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.

Understood. Thanks for all the previous help.


Revert "Update publish.js"

Reverts mobxjs/mobx-react#779

Coverage remained the same at 95.27% when pulling bb5cb40824fe740134d6868e868f444e0e4740b7 on revert-779-patch-1 into b7b2f23402177bad92cd9d123d0167d335fc9ab1 on master.


PWA: login feature is now working.

This change brings a fully working 'login' system; once the user
succesfully logs in, his JWT is stored in his local storage.

LGTM. But please add some comment at the end of the functions for readability. (at least the main parts)

Thanks @tizianasellitto! I've addressed your comment and I've better documented the Promise in emotly.js.

Thanks. LGTM.


Accessing UserState/ConversationState properties

Hi all,
I'm trying to set up a little bot architecture, where botframework composer will act as the leading part (easy deployment + set up). Im coding my own directline chat at the moment and trying to pass data objects around via webviews that will also be used by the facebook chat.
I came up with a little "middleware" concept, where I'm using azure functions for creating endpoints (such as for sending pro active messages to certain users) together with botbuilder js in order to use "adapter.continueConversation" for communicating with my deployed bot.
The communication works without any problem, I can for instance send pro active messages upon clicking a button in a webview. Now I want to pass more complex data into my conversation flows from the mentioned "middle ware".
I'm using Cosmos DB as a storage and created two dataAccessor objects, as I knew it from other tutorials in the past
const conversationState = new ConversationState(storage);
const userState = new UserState(storage);
let conversationDataAccessor = conversationState.createProperty("conversationData");
let userProfileAccessor = userState.createProperty("userProfile");

Now I'm struggling with accessing the "UserState" + "ConversationState" in botframework composer, what is the best way to achieve it? Does the approach via ConversationState and UserState even makes sense in this context or is there an easier way to achieve this shared data scenario?
Sorry for my bad English, I hope that I wrote it clearly and understandable, if there are open questions feel free to ask.
Best regards

@Rouvo when you author the bot in Composer, I presume you are using the user. and conversation.. When you load information from CosmosDB, are you not seeing the runtime behave as expected with these memory scopes?

@Rouvo when you author the bot in Composer, I presume you are using the user. and conversation.. When you load information from CosmosDB, are you not seeing the runtime behave as expected with these memory scopes?

Thanks for your quick reply!
Yes, so far I tried to set data on the user scope in the continueConversation adapter method via
const userProfile = await userProfileAccessor.get(turnContext);
userProfile.name = "Max Mustermann";

Upon retriggering the azure function, I can access the previously set name for one user id accessing userProfile.name, but trying to access it in bot composer via user.name (and some other variations i have tried) fails and ends up in a undefined value.
Besides this I've also added a simple "Ask a question" in the composer to also set a property called user.name,  I can access this property by simply calling ${user.name}
After taking a look into Cosmos DB in Azure I saw 2 different entries created after my approach.
The one created by the composer:
{
    "id": "directline*2fusers*2f23071337",
    "realId": "directline/users/23071337",
    "document": {
        "$type": "System.Collections.Generic.Dictionary`2[[System.String, System.Private.CoreLib],[System.Object, System.Private.CoreLib]], System.Private.CoreLib",
        "name": "rouven"
    },
    "_etag": "\"1300bfe9-0000-0700-0000-5f19c0840000\"",
    "PartitionKey": "directline*2fusers*2f23071337",
    "_rid": "ihw7AIzkPJNXAwAAAAAAAA==",
    "_self": "dbs/ihw7AA==/colls/ihw7AIzkPJM=/docs/ihw7AIzkPJNXAwAAAAAAAA==/",
    "_attachments": "attachments/",
    "_ts":<PHONE_NUMBER>
}

The one created by my "middleware":
{
    "id": "directline*2fusers*2f23071337*2f",
    "realId": "directline/users/23071337/",
    "document": {
        "userProfile": {
            "test": "Max Mustermann"
        }
    },
    "_rid": "ihw7AIzkPJNaAwAAAAAAAA==",
    "_self": "dbs/ihw7AA==/colls/ihw7AIzkPJM=/docs/ihw7AIzkPJNaAwAAAAAAAA==/",
    "_etag": "\"13008bed-0000-0700-0000-5f19c2090000\"",
    "_attachments": "attachments/",
    "_ts":<PHONE_NUMBER>
}

I have noticed the different ids and nested objects in document

@vishwacsena I believe @Rouvo is running into an issue sharing sdk v4 defined memory with adaptive defined memory. Is there something we can share with @Rouvo to help unblock him?


Symmetric Encryption support

[x] Whole DB Encryption
[ ] Collection Level Encryption
[ ] take parameters in defineCollection() method

There is no actual demand for document level encryption since it will make searching unnecessarily slow.

Solved https://github.com/iShafayet/atomicdb/commit/9d69fb1747ef1fb7f59b3205897bd82a60ebef59


Unnecessary URLs in "Helpful Links"

Describe the bug
URLs present rather than links in the Helpful Links section.
Not a bug, an enhancement.
To Reproduce
Steps to reproduce the behavior:

Go to 'covid19india.org'
Click on 'Helpful Links'
See error

Expected behavior
Links should replace the URLs
Screenshots
If applicable, add screenshots to help explain your problem.

vs

Desktop (please complete the following information):
For all devices
Smartphone (please complete the following information):
For all devices

@pravinagrawal145 I already created a pull request. This issue has not been reviewed yet.

Hey thanks for opening this issue, unfortunately this is a stylistic choice to show the contents of the url instead of having them hidden :)


Translate host suffixes in no_proxy to JVM syntax

This ended up being a bigger change than intended. I decided to write property tests for the no_proxy parsing to make sure all the edge cases got covered. But to make it easier to write the tests, I first separated the initialization code in HttpProxySupport into two parts:

parse the values of the system properties (without side effects)
set the values in the global system properties

@cruhland It seems to me that the other PRs are passing with some frequency. How many time have you rerun this?

@larioj probably about ten times by now...

It's not consistently the same test that fails, but it's definitely the same error. Seems really unlikely that it could be related to my PR.

Ninja-merging this PR, as the tests did pass once (see status for commit ab58211).


What's the dashed line in signcolumn?

Sometimes, navigator will draw a dashed line in the sign column for no discernible reason. I use set signcolumn=number to merge the sign column with the line number indicator so that the window contents don't jump around so much, and the dashed line then obscures line numbers completely:

I've not been able to figure out why this line is drawn, or why it only appears in some files, even varying in files of the same type. Since it's mostly only in my way, is there a config option to turn it off?

Navigator does not change the style of signcolumn. You saw those is very likely something is defined in DiagnosticSign[Warn|Error|Info|Hint] in your colorscheme.
Also one of the LSP is complaining about a range of lines having diagnostic warning/info/hint

You know what, it's a git plugin.  Sorry for bothering you.


Update README.md

https://github.com/tc39-transfer/proposal-array-find-from-last/issues/14

Thanks for the review!


The app runs but just blank

I get error when i tried to run
npm run dev-server

Here is the error:
WARNING in ./~/to-function/index.js
Module not found: Error: Cannot resolve module 'props' in /Users/myuser/Gokotta/node_modules/to-function
 @ ./~/to-function/index.js 11:9-25

Console of electron output:
Uncaught TypeError: Cannot set property 'WaveSurfer' of undefined

see #3


N^2 RDS memory for SNI

Title: N^2 RDS memory for SNI
Description:
Every filter_chain_match in SNI has its own HTTP connection manager which gets its own copy of the full RDS config. For every filter chain match there's going to be a route entry (N^2).
I was looking at how to share the ConfigImpl between HTTP connection managers but what stumped me was the Server::Configuration::FactoryContext& factory_context that's passed down to ultimately be used by PerFilterConfig

What's the best way to address this short term? (i.e. a hack to reduce our memory footprint of 30GiB)
What's the real solution? (i.e. something I could contribute)
How does this intersect with @AndresGuedez's work?

/cc @lrouquette @bryanlatten

This is envoy_inuse_space for heap dumps taken until we hit the 30GiB steady state

@atrifan ☝

cc @htuch @fredlas @AndresGuedez @stevenzzzz
I think the real solution here is to allow the RDS watches to point to a single subscription and single shared route config. I think this is probably not that hard given the work that @fredlas just did and the work that @stevenzzzz is doing right now but I will let them weigh in.

I think the real solution here is to allow the RDS watches to point to a single subscription and single shared route config.

Agreed. This is one of the primary goals of the Config Provider framework that was introduced as part of the SRDS work.
Once the SRDS implementation is complete, we are planning to transition RDS to use the framework as well.

Sounds good I will assign this over to @stevenzzzz for now.

I think the real solution here is to allow the RDS watches to point to a single subscription and single shared route config.

@mattklein123 It does point to a single subscription but different route config due to https://github.com/envoyproxy/envoy/pull/3960 to fix https://github.com/envoyproxy/envoy/pull/3953. I believe this is already in the consideration of @AndresGuedez 's Config Provider framework.
#7870 could fix this from the other side by not having multiple HCM for different SNI.

I think that what @AndresGuedez is the only viable solution and there is no place for a “quick fix” or a “work-around”. Reloads, hot reloads, gracefull reloads and other in-place modifications need to happen without affecting other ongoing clients - this is the main responsability of a proxy so right now keeping a copy of the RDS for each filter_chain (which you can look at as a middleware of that request) is by design a safety mechanism so you don’t break other request handlings that are ongoing. Also keeping a copy makes sure that you update each filter chain when you are ready, that beeing later then another filter_chain or earlier. In general - other proxies that support gracefull updates (you can check out nginx as well) have a big problem with the memory footprint - in different phases though or diferrent situations.

@lizan yeah I think we should be able to get back to a shared route config for a shared subscription. I think this is also related to https://github.com/envoyproxy/envoy/issues/7902 to which @stevenzzzz is working on.
@atrifan for identical route configs which point to the same named RDS subscription we can definitely make it shared. That is planned.

@stevenzzzz and I were discussed moving RDS to the config provider framework today. This might need to happen prior to SRDS landing due to some complexities around the lifetime management that are trickier than expected in https://github.com/envoyproxy/envoy/pull/7451. @stevenzzzz is analyzing complexity of different approaches here last we spoke.
We definitely need to go sharing for memory use reasons, but at the same time, we need to be very careful how we structure this. There is a lot of lifetime and ownership complexity in RDS/SRDS that we need to make sure is maintainable and grokkable by developers.

yeah, migrate RDS to the config-provider-framework is on my radar.
Will do that after the SRDS impl.

I believe this is fixed by https://github.com/envoyproxy/envoy/pull/9209


Don't look for babelrc file during test

Fixes #949

Thanks for the pull request @nhajidin! Can you describe the steps you followed to check that this works correctly?

I did the following:

Run create-react-app issue-949
Add a .babelrc file with { "presets": ["es2015"] }
Run cd issue-949
Run npm test; the test will fail
Edit file node_modules\react-scripts\config\jest\transform.js to include babelrc: false
Run npm test; the test will now pass

Thanks!


chore(plugin-cloud): refresh session on ExpiredToken error code

Issue: https://github.com/payloadcms/payload/issues/8404
This code will refresh the session token upon receiving an ExpiredToken error when storageClient.getObject() receives that error.

The Error Code detected is determined by the error reported in the issue, and is an actual  code from AWS documentation for Error Responses.

If the request fails again, the error is thrown.

The code is looking good. I've just triggered a canary release 3.0.0-canary.ab70b75. Please run this on a deployed cloud project and verify base functionality is still working.

Verified payloadPluginCloud() works with cognito credentials in a local env and in Payload Cloud.


Eliminate tap lag

The current tap gesture recognizer adds a lag to the keyboard buttons, which is notable when you try to type fast. Eliminate this to make the keyboard as responsive as the system keyboard.

This was solved by adding .allowUserInteraction to the standard tap animation.


[build] Upgrade used Maven plugins

Is your feature request related to a problem? Please describe.
The build uses outdated versions of Maven plugins.

This report was created with
./mvnw versions:plugin-updates-report -Dmaven.version.ignore=".*-beta.*,4\.0\.0-.*"

Describe the solution you'd like
Manage plugin versions in the pluginManagement section of the root POM.

Done via https://github.com/operaton/operaton/pull/71


Differences between this plugin and Daily Notes

Hey,
Thanks a lot for your work on Obsidian plugins!
I wonder what the differences are between the Daily notes and this plugin.  Is it just an extended version?
I'm also curious whether lynchjames/obsidian-day-planner#20 will work with this plugin (instead of Daily notes).

This plugin is mainly just an extended version of the Daily Notes plugin (since it adds weekly and monthly notes). In the future, I might break compatibility with the daily notes plugin and add additional features, such as new template variables (#16) or better note navigation.
Unfortunately, the day planner author has been MIA for a while. I have considered forking the library myself to improve compatibility.

I think it would be great if this plugin preserved some sort of interoperability with the core Daily Notes plugin (I mean replace it, but provide the same interface).
It would make integrations with other plugins much more easier. The Review and the Slated plugins come to mind (in addition to the Day Planner).

Totally agree! That's why this plugin is built on top of the daily-notes-interface which is another package I wrote to provide the plugin interoperability. Slated and Review already use that package which is why they support this plugin OR the daily notes plugin.

That's great news! I guess the Day Planner just need to use that interface then.
Thanks again for the plugin.


remake documentation

closes #160

Currently the documentation looks like this. Sphinx does not create sections for each class, so navigating the API reference is not great. We can either leave it like this (rely on the search function, and ctrl+f), or we could manually write .. autoclass:: pronotepy.Class for all the classes, and add sections in between. This would also require more maintenance...

I've decided to write out all the classes. Documentation is just as important as the code itself, so I think the little maintenance it adds is worth it.