Too long cache filenames

Sorry - pressed enter too fast.
Some combinations of platform / packages might generate too long cache file names. Tested with 0.1.18 on my mint Linux
(reproducible with main airflow`).
[jarek:~/code/airflow]└2 [apache-airflow] main(+4/-4)+ 8s ± uv pip install --upgrade --editable '.[devel-ci]'
   Built file:///home/jarek/code/airflow                                                                                Built 1 editable in 5.20s
error: Failed to download: sqlalchemy==1.4.52
  Caused by: Failed to write to the client cache
  Caused by: Failed to persist temporary file to /home/jarek/.cache/uv/wheels-v0/pypi/sqlalchemy/sqlalchemy-1.4.52-cp38-cp38-manylinux1_x86_64.manylinux2010_x86_64.manylinux_2_12_x86_64.manylinux_2_5_x86_64.manylinux_2_17_x86_64.manylinux2014_x86_64.msgpack:  File name too long (os error 36)

Oh wow. But that actually is the filename as on PyPI, right?

It seems like 255 is the common limit on Linux, but that it can be as low as 144 if you have encryption enabled: https://stackoverflow.com/questions/6114301/git-checkout-index-unable-to-create-file-file-name-too-long/6114588#6114588

I'm curious what pip does if you try to download that wheel :)

Yep. My homedir where I hit it (my linux workstation with Debian Mint). And the reason it is 143 (seems) is explained here https://wiki.archlinux.org/title/ECryptfs#Deficiencies
As of newer version of pip (v23.3+) pip store hashed names in the cache - avoiding the problem altogether:

Yeah, there's no inherent reason that we need to use full-fidelity filenames here. It's just helpful for debugging.

Suggestion: Only use hash if you've hit too long name issue.

@charliermarsh have we addressed this with recent changes to the cache?

No, hasn’t changed yet

Not trivial right now because we read the tags from the wheel name in the built wheel cache

This can be a real pain, on my machine the uv default cache dir is:
C:\Users\DMpassy\AppData\Local\uv\cache\
already 40 chars
If we take, for instance, DjangoRestFramework, a super popular django lib
C:\Users\DMpassy\AppData\Local\uv\cache\wheels-v0\index\d92ad663158e5f0f\djangorestframework\djangorestframework-3.10.3-py3-none-any\rest_framework\templates\rest_framework\vertical\checkbox_multiple.html
206 chars, not that far off from windows 260 upper limit.
I'm currently struggling to use UV on CI with jenkins due to that:
C:\Users\jenkins\AppData\Local\uv\cache\.tmp60o0NX\.venv\lib\site-packages\setuptools\command\build_py.py:207: _Warning: Package 'django_saml2_auth.templates.django_saml2_auth' is absent from
the `packages` configuration.
!!
# bunch of other stuff

error: could not create 'build\bdist.win-amd64\wheel\django_saml2_auth\templates\django_saml2_auth': The filename or extension is too long

I'm not sure how it got that long, I imagine it has to do with uv caching + setuptools building.

For windows, there might be another solution, I just came across this https://github.com/BurntSushi/ripgrep/blob/master/pkg/windows/README.md, which talks about using a manifest to declare that the application works with paths longer than 260 characters (I suppose the OS libraries change their behaviour then).

For windows, there might be another solution, I just came across this https://github.com/BurntSushi/ripgrep/blob/master/pkg/windows/README.md, which talks about using a manifest to declare that the application works with paths longer than 260 characters (I suppose the OS libraries change their behaviour then).

Yeah. Note though that, AIUI, the manifest is only 1 of 2 things that are required. The user also needs to apply a registry edit: https://learn.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation?tabs=registry#enable-long-paths-in-windows-10-version-1607-and-later

@potiuk Did you manage to use sqlalchemy with uv? I just tried uv with our codebase and it failed on the same package :-/

Hello @charliermarsh @zanieb  - would it be possible to prioritise that one a bit now?
To add a bit of context - we are now discussing Airlfow 3 approach of managing our complex dependencies and providers, and one of the options considered is to use uv workspace feature for that (as discussed before - I looked at it recently and I think it has all we need now, and I will make a POC to show to other PMC members and development community how uv workspaces might be useful to simplify and standardise our setup for providers.
Now - this one makes my life really hard ... Either I have to remove cache or move the cache somewhere aside of my home directory to another filesystem (but then hard-links don't work) etc. etc.  Also for me - this is quite a blocker to propose it - with our number of contributors (we just passed 3000!) - surely there will be some like me who have they home dir encrypted - especially that it's an option in multiple distros that you can select (For me - that came out-of-the box with my Linux mint installation and as a security freak I obviously enabled it).
Maybe a good solution will be to check if the filesystem is encrypted or simply react to "too long filename" and use a hash instead - there is no need to change it in bulk - simply handling the exception when it occurs should be good-enough and have very little effect on debuggability.
Looking forward to having this one solved (I'd do it myself - but I have far too many things on my plate  now to learn rust and all the internals of uv.

Yeah we def need to fix this.

Hi everyone, the issue with long cache names also displays some cryptic erro messages such as Caused by: stream did not contain valid UTF-8, but it has no relation to UTF-8 or string encoding.
Resolved 203 packages in 2ms
error: Failed to prepare distributions
  Caused by: Failed to fetch wheel: django-async-messages==0.3.1
  Caused by: Failed to run C:\Users\JoaoBernardo\Documents\Projects\alphatools-main\.uv_cache\builds-v0\.tmp8EzDKf\Scripts\python.exe
  Caused by: stream did not contain valid UTF-8

If you shorten the the cache dir to e.g. c:\cache\ it will work fine.
There is sometimes a second very long error message such as Build backend failed to build wheel through build_wheel() with exit code: 1 and a large stderr after.
These messages are all irrelevant IMO as they just show where in time UV crashed due to a long file name, but the solution will to shorten the path somehow.

Just a small ping on that one.. We already started to move to uv for workspaces, and I got my Linux PC with encryption back from cooling system replacement. Would be nice to start using it with uv workspaces, since my homedir is encrypted :)

FYI. We've switched Airflow (and I also did it) to use uv for recommended workflows and while I have been using uv heavily for syncing and installing things on the regular, encrypted volume I had, I stopped seeing the issue in recent uv version and airflow development environments.
I have also not heard anyone else having similar issue.
Not sure what caused it though. W have not changed our requirements for sqlalchemy, but maybe either the algorithm that decides which packages to install or other "generic" error handling in uv made the error disappear.
From my perspective - this issue can be closed, I think I'd have seen it again with the heavy usage of uv for a number of cases and switching to older versions of airflow for backporting if it was still a problem. It used to occur every time when I reported it.

There were some recent but minor improvements on some file names (according to the changelog on 0.5.x line), but for us, on Windows we still get inexplicable issues that goes away by changing the cache path via pyproject.toml that cause other issues still.
And the issue with Windows docker containers that get an even smaller limit for file names. We were able to partially overcome that on our specific user case, but the whole thing is too buggy for my taste.
Please keep the issue open. You may unsubscribe from it if you like

Thanks @potiuk. I'll leave it open but I don't plan to prioritize any work here until we have clear reproductions and evidence of significant need.

Please keep the issue open. You may unsubscribe from it if you like

I can be still subscribed :) no worries. I have no plans from my side to close it - not my decision, I think it's far more important to see if there is - like @charliermarsh mentioned - more users affected and clear reproduction/evidence that it disrupts more users.
From Airflow at least (and our 700+deps + the uv workflow that we implemented) - we see the problem is generally gone. I just know - being an OSS maintainer myself - how cool it is to see a "fresh" status on some old "low-priority" issues and feedback from those who raised them.
BTW. I'd personally agree with @charliermarsh  - and even more - qualify that issue as a "fix it if you need" kind of thing. Since this seems like a very low/rare impact issue - I'd say to anyone who wants to get it fixec - "Yeah we know it, and we know how loe impact it has. If you really want to have it solved, feel free to provide a complete PR that we can review and merge, that will not disrupt the 9x% of cases. We are ok with loosing the 100-9x%".
But that's just me - with side comment and assessment.
It's just the question of what else (more important) will not be done if that one is prioritized (i.e. "missed opportunities").


Aggregate regions within regionmask

I am opening this issue to keep track of a feature that was discussed in #90: The aggregation of smaller regions into larger ones. One way to implement this for the raster masks is implemented in cmip6_preprocessing (here).
@mathause suggested that this could be implemented on a lower lever, combining the actual polygons:
import regionmask
basins = regionmask.defined_regions.natural_earth.ocean_basins_50
basins_comb = basins.combine(pacific=["p_S", "p_N"])

I think this would be a great feature. Having the flexibility to combine any set of regions would be aweseome. It is also worth thinking about having some 'default aggregations' (like e.g. large ocean basins) available. This could be a cheap way to extend the 'built in' regions of regionmask.

geopandas has a nice feature for that: https://geopandas.org/aggregation_with_dissolve.html?highlight=dissolve

Ah very cool. The only concern I would have is that has been difficult in terms of dependencies, so it might be a risky dependency for regionmask? Could always be optional.

geopandas and cartopy (and thus fiona and gdal) are already required dependencies...
This would kind of require a to_geopandas method (which is anyway a good idea #50). I haven't used dissolve yet - it needs a column with equal values to aggregate over, so this would have to be constructed. But yes might be easier than a dedicated function within regionmask.

I add an example to construct the EUR-28 regions this is done outside of regionmask.

import regionmask

from cartopy.io import shapereader
import geopandas
from shapely.geometry import box

# read natural_earth dataframe
resolution="50m"
category="cultural"
name="admin_0_countries"
shpfilename = shapereader.natural_earth(resolution, category, name)

df = geopandas.read_file(shpfilename, encoding="utf8")

# select countries
# TODO: add all countries
sel = df.NAME.isin(["Germany", "Austria", "France"])
df_EURO = df.loc[sel]

# get rid of overseas territories
# TODO: check box
bbox = geopandas.GeoDataFrame(geometry=[box(-25, 0, 30, 90)])
df_EURO_res = geopandas.overlay(df_EURO, bbox, how="intersection")

# create one MultiPolygon
multipoly = df_EURO_res.unary_union

# create a region
eur28 = regionmask.Regions(
    [df_EURO_res.unary_union],
    names="Euro-28",
    abbrevs="EUR28",
    name="EURO-28 based on naturalearth"
)

eur28.plot()

so you would recommend aggregating inside geopandas and not regionmask

regionmask cannot aggregate Polygons - the only way would be to combine the mask e.g.
da.where(mask == 2 | mask == 3)

thus depends a bit on what you want to achieve


BUG: Normalize matplotlib backend check

Stumbled upon this issue while working on something else. Matplotlib backend MacOSX also works as macosx, but this check is checking case-sensitive parameters.
Modify and compare it against a lowercase string instead.

Thanks @jakereps! 🌮 🌚


Black screen

It might be because Riot update client and Valstore is on black screen now.

experiencing the same

Same problem

So what is the solution ?

Same problem, but @z4roc hasn't posted any updates for the last 2 days

I think he knows that problem, probably he is fixing it
чт, 21 нояб. 2024 г., 09:53 0ember1 @.***>:

Same problem, but @z4roc https://github.com/z4roc hasn't posted any
updates for the last 2 days
image.png (view on web)
https://github.com/user-attachments/assets/f5c75961-d943-487a-87cf-7f241959b104
—
Reply to this email directly, view it on GitHub
https://github.com/z4roc/valstore-mobile/issues/19#issuecomment-2490212312,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BNBS3VHZFYZQ5KKNX53BCZ32BV7P5AVCNFSM6AAAAABSA3GIISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJQGIYTEMZRGI
.
You are receiving this because you commented.Message ID:
@.***>

Yup im onto it riot indeed messed with the api again, this time I unfortunately have to do some more work, also I'm migrating to a newer flutter version and with that different packages, can't tell if im done today

thanks for the updates :) appreciate your work

So as far as now I finally managed to push the new stuff, migrating took ages, since the project was still based of a flutter version of 2 years ago, but with the latest release, everything should work now. https://github.com/z4roc/valstore-mobile/releases/tag/valstore_2_1_0


adding Woltka v0.1.4, pairends + suppress singleton & discordant + rm mux/demux

… mux/demux

just to clarify, we are consulting with @wasade  on one small point before final approval and merging.

@charles-cowart, this is ready for final review & merge.


Adapter contains multiple objects

I'm try to use adapter that contains Object A,B,C and D for example,
But I got this Exception
java.lang.ClassCastException: A$ViewHolder cannot be cast to B$ViewHolder

@hmedat2008 I assume you talk about having an adapter with 4 different Item's right?
Do those items have different types?
https://github.com/mikepenz/FastAdapter/blob/develop/app/src/main/java/com/mikepenz/fastadapter/app/items/SampleItem.java#L105
Do all have their own ViewHolder?
https://github.com/mikepenz/FastAdapter/blob/develop/app/src/main/java/com/mikepenz/fastadapter/app/items/SampleItem.java#L163
Does the generic definition define the correct type?
https://github.com/mikepenz/FastAdapter/blob/develop/app/src/main/java/com/mikepenz/fastadapter/app/items/SampleItem.java#L28

I will close this for now as it seems to be a problem with the items, rather than the library. We can keep up the discussion in here.
Please make sure first that the above things are done correctly for the items.
The AdvancedSample for example contains different items in one list
https://github.com/mikepenz/FastAdapter/blob/develop/app/src/main/java/com/mikepenz/fastadapter/app/AdvancedSampleActivity.java


[BUG] Can not install FVM at Chcolatey in Windows

Before creating a bug report please make check the following

[x] You have read our FAQ
[x] If you have used flutter. Please install correctly, run pub cache repair. Close the terminal and try again.
[x] If you are on Windows. Make sure you are running the terminal as administrator or with developer permissions.
[x] Run fvm doctor if possible and add the output to the issue.

Describe the bug
FVM installation fails using chocolatey in windows.
To Reproduce
Steps to reproduce the behavior:

Go to powershell (run as Admin)..
Run choco install fvm...
Check...
See error

Expected behavior
Install Success
Logs
Please check photo

Desktop (please complete the following information):

OS: Windows
FVM Version 2.4.1
If Windows: Which Powershell are you using? Powershell as Admin

Additional context
Add any other context about the problem here.

With Flutter registered as a system environment variable within Windows, it is confirmed that there is a problem when installing fvm. In my case, the installation proceeded normally when the environment variable of the previously installed Flutter was removed and reinstalled. I don't know the detailed cause, but please check it.

I'm having the same issue at the moment, I can remove Flutter from my PATH, but that will also remove Dart, since they're in the same folder...
If I manually delete the old Flutter binary, I still get the same error.


Add reviews and photos of POIs

As stated in the Telegram group, OMaps could use http://openplacereviews.org/ for both reviewing and adding photos to places mapped on OSM. This would be an amazing and very useful feature.

There's also https://mangrove.reviews/ , which at least seems to be promising more openness/privacy.
Don't know more about them, though.

There's also https://mangrove.reviews/ , which at least seems to be promising more openness/privacy.
Don't know more about them, though.

There's also https://mangrove.reviews/ , which at least seems to be promising more openness/privacy.
Don't know more about them, though.

It would be amazing to view reviews from more than one source. For example, there are apps to search for movies, and for every movie they show both IMDB, TMDB votes and reviews from multiple websites. In my opinion the tough part is to link an OSM point of interest with the relative object on a certain review website.

There's also https://mangrove.reviews/ , which at least seems to be promising more openness/privacy.
Don't know more about them, though.

It would be amazing to view reviews from more than one source. For example, there are apps to search for movies, and for every movie they show both IMDB, TMDB votes and reviews from multiple websites. In my opinion the tough part is to link an OSM point of interest with the relative object on a certain review website.


x/pkgsite: remove autocomplete experiment code

Given the code churn on the homepage and header, autocomplete no longer works when turned on. As far as I know, no one is actively maintaining it. We should remove the code until we are ready to launch it to users.

I can get this working again, when we're ready to launch.
We can wait until the header code stabilizes, but if that has already happened I'd be happy to fix it rather than remove it.

@findleyr sgtm!


Data access logs support

Add data access logs support to folders and projects

We're already supporting this in the organization module, but we're making it hard to find the feature.
What we should do

rename the organization module variable to make it clear it's related to audit logging, and move resources from iam.tf to logging.tf
add the same support to the folder module via the google_folder_iam_audit_config resource


Use sbt-unidoc plugin for generating public API documentation

Rather than running Javadoc/Scaladoc commands separately to build our
public API documentation, we can now just run "sbt unidoc". That
generates both Java and Scala documentation in the root target
directory, including docs combined from the japi, sapi, and udf
subprojects. This also allows for easily adding new public API
documentation for other features, such as planned DSOM walker, by just
creating a new subproject and adding it to the unidocProjectFilter sbt
setting.
DAFFODIL-2368

The daffodil-japi subproject is written in scala, but we want to generate javadocs for, since the intended usage is with Java code. There is a scala compiler plugin called genjavadoc that will generate javadocs from scala source files. You don't actually need to use the unidoc plugin to get this feature. The latest version of this scalac compiler plugin is 0.16.
The sbt-unidoc plugin essentially just takes java/scaladocs from subprojects and combines them into one big java/scaladoc. The 0.4.3 is the version of the sbt-unidoc plugin. Note that sbt-unidoc also acts as a wrapper around genjavadoc, so you gen generate javadocs for a subproject using the scala compiler plugin, and then sbt-unidoc will pull those in when it combines everything.
So in short, one version is for the sbt-unidoc plugin, the other is for the genjavadoc plugin that it wraps.
I originally did a visual inspection and things looked the same. To confirm, I just ran a diff to see what the actual changes were:

Javadoc includes a generation timestamp. I'm thinking it make sense to include the {{-notimestamp}} option when generating docs so we don't get a 1000 line diff of just timestamps.
The order of two rows in a deprecation table are randomly flipped. No idea why this happened, but it doesn't really matter I guess.
I modified the root-doc.txt file, which is apparently included in many scaladoc files as a tooltip. This diff is expected, but the tooltip should be something different instead of the entire contents of that file. I'll see if that's easy to fix.
The UDF API additions, which is also expected.

So all diffs are expected. I think it's worth ammending this PR to disable timestamps and fix the tooltip.

Github isn't updating this PR with new commits to the branch. I'm going to close and create a new PR and see if that works.


Use the arrow in a consistent way - turn the opposite the arrow when close the division and district.


feat(fiatconnect): Check KYC after submitting account

Description
Fixes ACT-384. This PR adds checks for KYC status after a user successfully submits a fiat account on the FiatDetails page, when KYC is required for the quote the user has selected. Depending on the KYC status, we either navigate to the Review screen, or a particular KYC status screen. In the exact case where KYC is required and it has been approved by the time the user has submitted their account, we show a green checkmark to the user to indicate their ID verification success.
Other changes
Describe any minor or "drive-by" changes here.
Tested
Unit tested. Not easily end-to-end testable at the moment.
How others should test
Does this need to be tested by QA in the next release cycle? If so please give a brief explanation of how to test these changes.
Related issues
Fixes ACT-384.
Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

Hey @MuckT please let us know do we need to test this PR ?
CC: @jophish
Thanks.!


when operator install done, but not find sc resource

Output of lsblk:
NAME        MAJ:MIN RM   SIZE RO TYPE MOUNTPOINT
sr0          11:0    1  1024M  0 rom
nvme0n1     259:0    0   300G  0 disk
├─nvme0n1p1 259:1    0     1G  0 part /boot
└─nvme0n1p2 259:2    0   299G  0 part
  ├─cs-root 253:0    0    70G  0 lvm  /
  ├─cs-swap 253:1    0   7.9G  0 lvm
  └─cs-home 253:2    0 221.1G  0 lvm  /home
➜  ~

Output of blkid:
/dev/nvme0n1: PTUUID="135b2a6a" PTTYPE="dos"
/dev/nvme0n1p1: UUID="e93c8edd-2776-4bed-a329-2af316358e01" BLOCK_SIZE="512" TYPE="xfs" PARTUUID="135b2a6a-01"
/dev/nvme0n1p2: UUID="jXakmC-VzYf-0yld-9d0n-WfvS-bsnN-gw2Lyf" TYPE="LVM2_member" PARTUUID="135b2a6a-02"
/dev/mapper/cs-root: UUID="d86da7da-c159-4008-8f53-44c3bd850d45" BLOCK_SIZE="512" TYPE="xfs"
/dev/mapper/cs-swap: UUID="634a6aeb-895f-4b57-9fcf-5f4cbd877fe0" TYPE="swap"
/dev/mapper/cs-home: UUID="81924b71-0993-4359-8be6-9bfd79976914" BLOCK_SIZE="512" TYPE="xfs"

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.0", GitCommit:"b46a3f887ca979b1a5d14fd39cb1af43e7e5d12d", GitTreeState:"clean", BuildDate:"2022-12-08T19:58:30Z", GoVersion:"go1.19.4", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.0", GitCommit:"b46a3f887ca979b1a5d14fd39cb1af43e7e5d12d", GitTreeState:"clean", BuildDate:"2022-12-08T19:51:45Z", GoVersion:"go1.19.4", Compiler:"gc", Platform:"linux/amd64"}

Output of kubectl get lsn -o yaml:
apiVersion: v1
items:
- apiVersion: hwameistor.io/v1alpha1
  kind: LocalStorageNode
  metadata:
    creationTimestamp: "2023-10-21T12:51:23Z"
    generation: 1
    name: master
    resourceVersion: "125017"
    uid: 3a8a2591-2813-4e15-aab0-335a1af3e6be
  spec:
    hostname: master
    storageIP: <IP_ADDRESS>
    topogoly:
      region: default
      zone: default
  status:
    state: Ready
- apiVersion: hwameistor.io/v1alpha1
  kind: LocalStorageNode
  metadata:
    creationTimestamp: "2023-10-21T12:51:32Z"
    generation: 1
    name: node1
    resourceVersion: "137739"
    uid: 28c8aba8-7569-4b89-8ca4-698923d6f20d
  spec:
    hostname: node1
    storageIP: <IP_ADDRESS>
    topogoly:
      region: default
      zone: default
  status:
    state: Ready
- apiVersion: hwameistor.io/v1alpha1
  kind: LocalStorageNode
  metadata:
    creationTimestamp: "2023-10-21T13:55:20Z"
    generation: 1
    name: node2
    resourceVersion: "137575"
    uid: e154618e-38ac-46c1-8266-76f4e02c3fa3
  spec:
    hostname: node2
    storageIP: <IP_ADDRESS>
    topogoly:
      region: default
      zone: default
  status:
    state: Ready
kind: List
metadata:
  resourceVersion: ""

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
# my PC

k get sc

No resources found

can you show the output of 'kubectl get ld -owide' and 'kubectl get ldc'?
normally, the available lds should be bound and get a owner 'local-storage', if claiming successfully done, the ldcs should disappear automatically.

sorry, i just noticed that you only have one disk on your node named nvme0n1,and it's not clean, which means data already written on it. hwameistor will leave the used disks alone,  not add them into hwameistor storage pool.

https://hwameistor.io/docs/quick_start/install/operator
you can have a look at this note in this page

thank you, I think it want clean Disk?

thank you, I think it want clean Disk?

yes, absolutely clean, no data on it, even filesystem metadata or partition info.
if it's clean, you can see a ld with 'available' phase correspond to the disk.

OK


patch of ember data store to enable push events causes error in acceptance tests

When running acceptance tests with the async helpers, the following error is triggered
Uncaught Error: Assertion Failed: You have turned on testing mode, which disabled the run-loop's autorun. You will need to wrap any code with asynchronous side-effects in a run
The source appears to be the ember data store patch to enable push events.
https://github.com/firebase/emberfire/blob/master/addon/initializers/emberfire.js
It needs the run loop that is not available during testing. Wrapping the async helpers does not help.
In the test I visit a route that has a model that does a query.
export default Ember.Route.extend({
  model(params) {
    return Ember.RSVP.hash({
      thing: this.store.query('things', { orderBy: 'name', equalTo: params.thing_name })
        .then(list => list.get('firstObject')),
      otherThing: this.store.query('otherThings', { orderBy: 'name', equalTo:  params.otherThing_name })
        .then(list => list.get('firstObject'))
    });
  }
});

Any thought on preventing the error?

I'm running acceptance tests and not seeing this error. Can you show the context of the failing test?

@gabrielgrant Hi, I have had to abandon the firebase based project due to issues with privacy regulation in the eu. If I get around to the issue again. I'll include more context. Thank you.


Capstone

I have coded my capstone project for "Urban Landscape & Design Expo 2023" to match the design provided.
I have personalized only some of the content of my page.
The project pages look almost identical to the original design with small adjustments like text or image changes.
I have not used any of the frameworks (etc... bootstrap).
I have  build only these 2 pages:
1. The home page.
2. The about page.
Both pages have versions for 2 different screen sizes:
1. Mobile: up to 768px wide.
2. Desktop: 768px or wider.
Links
1. The home page have a link in the menu to the about page.
2. The logo in the header links to the home page.
Mobile menu

When the user clicks (or taps) the hamburger button on the header, the mobile menu appears over the page.
The mobile menu should have a close (X) button that closes the menu.

Dynamic page

The section "Featured speakers" is created dynamically in JavaScript.
I have used a JavaScript variable with the data about the speakers and used it when the page loads to create the HTML for this section dynamically.

I have made the changes in the design as per the reviewer's comments. Kindly check.

I have revised the code under the reviewer's comments to adjust my design as per the sample design.


Single-shot generators

Perhaps not the most accurate title.
Implementation path to address #29 by preventing the feature - generators would no longer append nets / connections, rather the single generator on a block would define all the connections. This may also simplify parts of the compiler logic, since there was logic and state to handle partially defined blocks.
Before the generator runs, these would be defined on the block:

Parameters
Parameter assignments
No internal blocks are defined
No connections are defined

The generator then defines these parts of the block:

Internal blocks
Connections (between internal blocks, and between boundary ports and internal blocks)
... including any parameters

Example issues: buck converter generator

Actual output voltage is dependent on the feedback resistive divider - and dependent on internal blocks
The inductor sizing is dependent on the current draw
There are some components where the current draw is dependent on the voltage (perhaps this is bad practice and can lead to brittle design? perhaps listed voltages and currents must be independent of each other to avoid deadlock?)
Possible solution: move the generator boundary for the buck converter to internal blocks. For example, instead of the buck converter block generating the power stage, the power stage would be a separate generator block contained within a non-generator buck converter block. This allows the feedback resistive divider and power stage to generate independently, but at the cost of more levels of hierarchy / blocks. However, this structure might enable better re-use in the power converter generator through better modularization.

In general, the idea would be to have very minimal generators (instead of more monolithic generators). Downside is that it may mean more individual sub-blocks so we can have those generator boundaries. This structure (top-level block is just a non-generator encapsulating block) also avoids needing the generator to propagate internal block information that the generator needs to run.

Alternatively, a pre-generator block can be more akin to a library block, where it defines nothing at all. This would probably simplify and unify the block elaboration logic.

Additional notes:

The above style of minimalistic blocks (eg, instead of a generator implementing the buck converter interface, there is a buck converter power stage generator that is instantiated by the buck converter itself) may be incompatible with where the block needs to provide pin-mapping. For example, an abstract microcontroller block would need to map connections from the external grab-bag-of-IO interface to the physical pins of the internal chip.
Under this style, the pin mapping functionality would need to be implemented as a dedicated block, which internally would need edge-port-to-edge-port connections (semantics not well specified), and require links (and possible direction flipping to make the model work out) when connecting to that block. This makes much less sense compared to a direct export.
One solution to enable the above include some kind of pass-through connection type - but this seems excessive for what may be a rare / special case.
However, semantics-wise, the proposed single-shot generators would work for pin-mapping blocks, since they have no required parameters and can generate and elaborate immediately (which avoids potential deadlock that requires the above minimalistic generator block style). The downside is that while it's semantically consistent, it is two different styles from the user-facing view.
Related: generators will likely be required for block-side port arrays, and it is likely that they must be implemented in this style (may have no dependent parameters, aside from optionally the array size of ports). Where more advanced generation is required, it can be delegated to an minimalistic internal block.
But the above seems to be the least worst solution so far.

Need some thought on how dependency parameters are to be expressed, especially if a generator block's parameters aren't declared / elaborated until the generator runs.
Options:

Generator blocks contain some parameter declarations. This could just be the constructor argument parameters (so it's more guaranteed to be consistent), or may also include boundary port (which should also be guaranteed to be static, with the exception of parameters like arrays).

For version 1, constructor argument parameters should be done, and all generator parameters should be explicitly passed in through those. Where others are needed (eg, port parameters), this can be handled by mapping those from an wrapping block to inner generator parameters. More ergonomics can be explored in the future, if needed.
Generator parameters would be checked for consistency with parameters declared in the generated block.
Downside: some level of generator and generated block "merging" still needs to happen, but vastly simplified from before.

Generators refer to parameters without those parameters being declared. Works with the current compiler implementation, because it doesn't check for the declarations (just assignments). Not very principled, may not continue to work if declarations are checked.

Like the above, minus parameter declarations.

Generators can only refer to parameters in the containing block. This would be very odd, since the expression would need to refer to parameters in the parent scope (which there currently is not support for).


Upgrades the codebase to C++ 20

Not exactly sure if this requires any special testing. I ran keyboardwalkfake and it didn't crash :shrug:
I also noticed that the standard wasn't explicitly set for modules, so this change explicitly sets them to C++20 as well. Tests too

We should try and update llvm before we go to c++20.

Wrong approach, and premature


Xcode compile failed，Undefined symbols for architecture x86_64:   "_FBLPromiseErrorDomain",

user zip file, not pod.
version: Firebase-6.21.0
I drag this framework：

as you see,  I also  drag PromisesObjC.framework to my project.
and  FBLPromiseErrorDomain is also in file FBLPromiseError.h.
the class FBLPromise is also in PromisesObjC.framework，
but why Xcode build Error：
Undefined symbols for architecture x86_64:
"_FBLPromiseErrorDomain", referenced from:
___60-[FIRSecureStorage getObjectForKey:objectClass:accessGroup:]_block_invoke in FirebaseInstallations(FIRSecureStorage.o)
"OBJC_CLASS$_FBLPromise", referenced from:
objc-class-ref in FirebaseInstallations(FIRInstallationsIDController.o)
objc-class-ref in FirebaseInstallations(FIRInstallationsAPIService.o)
objc-class-ref in FirebaseInstallations(FIRInstallationsIIDStore.o)
objc-class-ref in FirebaseInstallations(FIRInstallationsIIDTokenStore.o)
objc-class-ref in FirebaseInstallations(FIRInstallationsStore.o)
objc-class-ref in FirebaseInstallations(FIRSecureStorage.o)
ld: symbol(s) not found for architecture x86_64
I am already include they in my project all.

It's possible you have a mix of frameworks from different Firebase versions.  I recommend removing all the Firebase frameworks from the app and reinstalling from scratch following the zip README's instructions.

I have a similar issue on Xcode 10.4, installed via Carthage.
ld: warning: Could not find or use auto-linked framework 'FBLPromises' Undefined symbols for architecture x86_64:   "_FBLPromiseErrorDomain", referenced from:       ___60-[FIRSecureStorage getObjectForKey:objectClass:accessGroup:]_block_invoke in FirebaseInstallations(FIRSecureStorage.o)   "_OBJC_CLASS_$_FBLPromise", referenced from:       objc-class-ref in FirebaseInstallations(FIRInstallationsIDController.o)       objc-class-ref in FirebaseInstallations(FIRInstallationsAPIService.o)       objc-class-ref in FirebaseInstallations(FIRInstallationsIIDStore.o)       objc-class-ref in FirebaseInstallations(FIRInstallationsIIDTokenStore.o)       objc-class-ref in FirebaseInstallations(FIRInstallationsStore.o)       objc-class-ref in FirebaseInstallations(FIRSecureStorage.o)   "_OBJC_CLASS_$_FIRRemoteConfig", referenced from:       objc-class-ref in RemoteConfig+extension.o       objc-class-ref in FirebasePerformance(FPRRemoteConfigFlags_71b80df7b60132dc34996f1564188b0a.o) ld: symbol(s) not found for architecture x86_64 clang: error: linker command failed with exit code 1 (use -v to see invocation)

@vburojevic,
I had the same issue with Carthage, but I resolved it.
Please check if the PromisesObjc.framework is added to your project (make sure the correct target is selected). In my case it wasn't because I believe this is a new framework they've added recently. It goes without saying that "Do Not Embed" should be selected under Frameworks, Libraries and Embedded Content".
I hope this helps, let me know :)
Cheers!

thanks all~
it's my problem, my target don't add PromisesObjc.framework correct。
I am already fixed it out.

thanks all~
it's my problem, my target don't add PromisesObjc.framework correct。
I am already fixed it out.

I am facing the same issue. could you please help me out.

@run
just make all the frameworks add to project. if compile wrong, remove all frameworks and add it again.
you need to read the readme file, then you will know which framework need to add to your project.


[Q] Is there is away set general rules for wandb.define_metric

Hi,
Instead of defining every individual metric, is there is away to set more general rule, something like
summary="min" if "loss" in metric_name

Hello! There isn't a way to set a general rule with define rule with define_metric since we would need all the data surrounding each metric in order to full graph it in the UI. If you want to look into the details of define_metric, here is the exact line of code we run in order to make a new metric.

Correct, the define_metric function does need to have the metric defined already. I can certainly add in the second point to the feature request.

WandB Internal User commented:
MoH-assan commented:
Thanks, but this would be a nice feature.
Or allow the rule to be defined when doing wandb.log the first time the metric is logged or something like that.
If I am not wrong the current implementation requires knowing the names of the metrics to be logged and individually defining the rule for them before logging them, right?


Suggestion: Maybe one comment per example to show directly what the example's main message is about

Please provide comments!

The idea behind my issue is:
I would love to see in the code a comment related to the idea that is discussed in the example, for example for the "delete to-do", where in the code is the filter, so that absolute beginners directly see what the example is about.

I will consider adding some explanations.

I have added comments to all the examples. Now it should be easier to understand the examples and the motivations for them.
Thanks for your feedback and see you next time. :)

@doemser thank you, love the comments!


Apply linting and stylechecking to JS test files

This PR doesn't add any functionality to Empress. All it does is include the test JS files in make stylecheck and make jsstyle. This'll be nice going forward, because it'll help keep these tests linted and looking consistent with the rest of the JS codebase.
(Figured it'd be easiest to add this in as its own PR, so that these changes can be reviewed in isolation.)
Most of the changes here are pretty minor stylistic things (e.g. single -> double quotes, spacing), but there are a handful of more substantial things that should probably be looked over. Some of these things I can think of:

The tree data in test-empress.js had duplicate inSample keys (each node had both a false and true version of this key set when initialized). I think that only the trues were being kept (also, if you remove all the falses nothing changes, but if you remove all the trues instead then one of the tests breaks), but another set of eyes on this code might be nice.

There are a few places in the tests where a negative value was used as the right term in a subtraction, e.g. this line which used angle.sin - (-1/2). I ran prettier first, which (IIRC) changed this to angle.sin - -1 / 2, and then jshint complained about that, so I wound up just changing this to angle.sin + 1 / 2. But I can see how the first version of this might be preferable.

@kwcantrell Thanks for the review! Per our discussion at this morning's meeting, I've removed the setNonSampleBranchVisibility() test in test-empress.js, as well as all references in this test file to the .inSample and .visible attributes of nodes in the tree data (since #182 will remove the functionality that uses these attributes).
The Travis build should pass now. Let me know if you see anything else :)

thanks @fedarko!


Updated progress indicator to GIF image.

Docs Build status updates of commit 4a5c13e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/app-patterns-landingpage.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/UX/MRTK_ProgressIndicator.gif
:white_check_mark:Succeeded
View

mixed-reality-docs/images/UX/UX_Hero_Progress.jpg
:white_check_mark:Succeeded

mixed-reality-docs/progress.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


PHP example should URL-encode business IDs used in /v2/business path.  Closes #118.

Calling print get_business('la-fusión-san-francisco-2') before:
PHP Fatal error:  Curl failed with error #400: {"error": {"text": "Signature was invalid", "id": "INVALID_SIGNATURE", "description": "Invalid signature. Expected signature base string: GET\u0026https%3A%2F%2Fapi.yelp.com%2Fv2%2Fbusiness%2Fla-fusi%25C3%25B3n-san-francisco-2\u0026oauth_consumer_key%3DsWc_ig2aihT0s-6Yf-zGIQ%26oauth_nonce%3D34b42ebffd71df54c5fadf22b6368cd3%26oauth_signature_method%3DHMAC-SHA1%26oauth_timestamp%3D1458326155%26oauth_token%3DTu7Aj3WBM2cIA1uurCEyRq7W9ykKbQQy%26oauth_version%3D1.0"}} in /nail/home/kmitton/pg/other/yelp-api/v2/php/sample.php on line 94

After:
{"is_claimed": true, "rating": 4.5, "mobile_url": "http://m.yelp.com/biz/la-fusi%C3%B3n-san-francisco-2...

Seems straightforward, while we are addressing this issue, can you urlencode the term and location in the search function?

@watterso : Those go through http_build_query() already.  business_id is part of a path so it doesn't, which is why we need to explicitly encode it.

Ah duh. In that case, :shipit: :+1: :ship:


[Improvement]: Change Language Navigation to Language Page in Settings

Contact Details
No response
Describe what you do not like or what problem you ran into
The language page before the welcome is not used a lot and sadly only seen when the user first opens Stereum. I would like to change this by using it as a selector in the settings page.
Attach a Screenshot!

Suggestion Box for Improvement
After @ithilion2007 has handed us the necessary adapted animation for the language page (as the animation end entering the page), I would like to change it so the user has the chance to see the language page again accessing it through the settings.

Launcher Version
2.0.3
Desktop OS
Windows
Node Server OS
Ubuntu 22.04
Acknowledgment

[X] I looked through the existing issues of this project

Fixed with the PR #1812


Dockerfile ignores

What is the purpose of the .dockerignore contents? When trying to build this with jenkins I get errors about ca-certificates.pem not existing (even if it is present in the workdir), and an explicit complaint that the .dockerignore file includes Dockerfile.

Hi,
The docker ignore file is used to determine what context is passed to the docker daemon at build time. The contents of the .dockerignore file are:
*
!_bin
!ca-certificates.crt

The first line is a catch all, as the majority of the repos content doesn't need to be send to the daemon. The next two lines are exceptions to this catch all (starting with '!'). So all that should be passed to the Docker daemon is the ca-certificates.crtand the contents of _bin (the key-manager executable).
The reason you might be seeing the error regarding ca-certificates.crt is because it is not actually part of the repo, but is downloaded by the Makefile at build time, and if you're not using the Makefile then it would still be missing.
The other error regarding the Dockerfile being ignored, I've never seen that before. I always assumed that the Dockerfile wouldn't need to be part of the build context. What version of Docker is your Jenkins server using and can you give me anymore details?
Thanks
Alex

Assuming this is resolved and closing.


Why the mono depth loss was multipied by 50？

Hi, please refer to https://github.com/autonomousvision/monosdf/issues/18


Print exceptions as errors

Tagging the issue here: #1192

@ocalles  we still need the suggestion from @AbhitejJohn , the onError call in the try catch block of the wasm runner is missing.
The onError call makes the error from the exception flow as event while the RunResult is a synchronous results that contains all outputs and errors.
Do you have a test that shows that your fix flows into the CSharpProjectWithWasmRunnerIKernel and then captures in the trudtonet.js so that customer integration can display in the output pane.


Please add the ability to track AMD cards as i'm sure they'll be scalped as well. Thanks for all you're doing!

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other context or screenshots about the feature request here.

Hoping to grab a card before the scalpers pick them all off.

Maybe we'll do that but I also wanted to let you know you are supposed to replace the descriptions on the template.


WARNING: The gcloud ml-engine commands have been renamed and will soon be removed. Please use gcloud ai-platform instead.

found this example is legacy but there are no current similar examples using ai-platform

closing this out

related issue GoogleCloudPlatform/cloudml-samples#452 (comment)

Hey, I checked out the related issue but was still unable to find the proper commands for this problem. Would you mind following up with me on your solution?
Thanks!


portico: Add End Point Dev case study page.

Testing:

Manually tested links
Checked that the links on /for/business look fine at different screen widths

Notes and follow-ups:

We should probably feature a quote on /for/business, but that can be a follow-up, as it's going to require a bit of thought, and possibly some rearranging.
#24678 will need to be updated with a link

Screenshots

 Top of new page

 Top nav

 /use-cases

 /for/business

 /case-studies/idrift

Looks great, aside from the CI failure, which I resolved by making a link relative. We should be able to deploy this to zulip.com soon, such that #24678 can be the place where we restore the zulip.com URL.


E0d/pylint

pylint fixes.

@benpatterson @feanil checks passed.

Sorry @e0d I missed this PR and posted a commit directly onto Feanil's branch earlier this a.m.
I did not deal with E402 though, b/c I wasn't sure if that would have an impact on running the files. I just set pep8 to ignore that rule.


Update README.md

First install also requires npm install and aligned the content for more consistency with runnable commands.

Thank you for the updates!


MS Docs Publishing: Front Matter Generated for README Files Fails Validation

Summary
The front matter that is generated by the MS Docs publishing toolchain is failing the automated validation pass.
Details
Validation Issues

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.

Line 11, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'eventhubs'.

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

Generated Front Matter
---
title: Azure Event Hubs Event Processor client library for .NET
keywords: Azure, dotnet, SDK, API, Azure.Messaging.EventHubs.Processor, eventhubs
author: maggiepint
ms.author: magpint
ms.date: 02/09/2021
ms.topic: article
ms.prod: azure
ms.technology: azure
ms.devlang: dotnet
ms.service: eventhubs
---

Previously Generated Front Matter (September, 2020)
---
title: Azure Event Hubs Event Processor client library for .NET
description: Azure Event Hubs Event Processor client library for .NET - Version 5.2.0
keywords: Azure, dotnet, SDK, API, Azure.Messaging.EventHubs.Processor, eventhubs
author: maggiepint
ms.author: magpint
ms.date: 09/08/2020
ms.topic: article
ms.devlang: .net
ms.service: event-hubs
---

Success Criteria

Determine if the validation issues are an indication of a problem, or are expected and benign.

If a problem was determined, front matter generation has been updated to correct the issues.

BONUS POINTS:  Front matter generation has been updated to attribute the author of the most recent commit as the article author.

Related and References

Docs.MS Release Updates for Azure.Messaging.EventHubs.Processor (#1797)
Docs.MS Release Updates for Azure.Messaging.EventHubs (#1796)

The customer impact here is minimal (better reporting data).  We haven't gotten to this in 2+ years.
To get this right, we need to figure out what the fields need to be and then get them into our release plans or have them in a common location and read them out as part of the release.
Please reactivate if you feel strongly about this one.


Formal description of Byzantine Consensus algorithm

The consensus algorithm used in Ripple, is the core of the software. However, there is no formal explanation of how it works, or any sort of unit-testable pseudo-code. We need a formal description along with supporting research, pseudo-code, and sample programs.
There's a more in-depth discussion here:
https://ripple.com/forum/viewtopic.php?f=2&t=4118&p=23393#p23393

There's been a lot of progress on this, including consensus refactoring and simulation tests. This issue is obsolete.


Update reply lib metadata.json to support Bangle JS 1

The reply lib metadata.json file currently does not list Bangle JS 1 support, which then blocks newer versions of the Messages app from installing on Bangle JS 1 devices as the lib can't be found. This adds support for Bangle JS 1, as I've tested in the emulator and there doesn't appear to be anything blocking this running on both devices

Great, thanks!


Add support for new MasterCard BINs (2-series)

MasterCard has just introduced their new Bank Identification Number series, which will start with the number 2. According to their explanation page:

Mastercard® has received an additional range of 2-series numbers to add to the 5-series that we currently provide. The 2-series BINs operate in the same way as the 5-series and we have made changes to our systems to accommodate the new numbers. Readiness for the 2-series across our issuers, merchants and other partners is tracking on target dates listed below. Mastercard customer financial institutions can expect to be issued 2-series BINs starting in 2017.

According to Wikipedia, the 2-series range is comprehended by 222100–272099.

Thanks @emyller! I've merged your path.


Update Solr files with changes made in Krikri

See:

https://github.com/dpla/KriKri/commit/5e23bc4c27fd53f28fac755002f9e37d7d886e1e
https://github.com/dpla/KriKri/commit/b5b8b184fb0ae70507f433ca0a1f875309b1b8a5

These are changes that were made to schema.xml and solrconfig.xml in the
Krikri project in April.

This looks good. I'll let it pass CI, then :merge:


Update NOTICE

Could you check why the tests all fail?

The docker service on macos occasionally has connection refused, which hinders the CI from passing. I have no solution.
Similar issues: https://github.com/abiosoft/colima/issues/962

We could exclude Mac tests for now, and keep eyes on the issue, only add the tests back when it is back to stable again.


Clicking start server on 2 separate clients on the same host machine prevents client B from being able to close their game window

Resolved by 0e29cdf200437ce10698b89f8a8b1bcd073ead0d
Also see 6b6cb5e85a3ff5cc9d27b607ae8b22093d0b37ea


try gulp with a retry

It would be nice to do a syntax/compilation check and then do the retry , but this is def better than what we have , assuming a retry would correct things

FYI I tried this out locally so syntax should be fine

also this is a general retry function so compilation checks etc. might not fit in this funciton

oh, i mean, doing a compilation on the JS code prior to retry and abort if thats the case

LGTM


Handle saturating references with included resource in value[x] fields

When a reference in a resource is part of a value[x] field, the generated code for the value[x] takes the shape of an enum that wraps around the different possible types within. An example of this is found in the DeviceRequest resource. It has a field code, which can either be a Reference or a CodeableConcept (side-note: the FHIR wiki refers to this as a CodeableReference):
pub enum DeviceRequestCode {
    /// Variant accepting the Reference type.
    #[serde(rename = "codeReference")]
    Reference(Reference),
    /// Variant accepting the CodeableConcept type.
    #[serde(rename = "codeCodeableConcept")]
    CodeableConcept(CodeableConcept),
}

This enum is expected to look a little like:
pub enum DeviceRequestCode {
    /// Variant accepting the Reference type.
    #[serde(rename = "codeReference")]
    Reference(DeviceRequestCodeReference),
    /// Variant accepting the CodeableConcept type.
    #[serde(rename = "codeCodeableConcept")]
    CodeableConcept(CodeableConcept),
}

#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
pub struct DeviceRequestCodeReference {
    /// The resource that is being referred to. When doing searches, the client will fill this field if possible.
    #[serde(skip)]
    pub target: Option<Box<Device>>,
    /// The FHIR Reference field
    #[serde(flatten)]
    pub reference: Reference,
}

There seem to be two separate issues:

Generated code does not differentiate/wrap the reference with a custom reference struct that has a target field.
Related to <1>, even if included in the same bundle, the referenced resource is not attached to the reference.

#7 is now merged, and I am closing this in the hope of fixing your issue. :saluting_face:


Add containHeight prop to menus

Description
With the current menus if you have content that taller than the available space (ex: menu with many options) that menu will be cut off by the viewport
Example from docs website:

While this might be able to be cleaned up in user land it requires a whole bunch of spaghetti code and is hard to maintain.
Luckily the underlying popper library floating-ui has a middle ware that supports sizing the popper content based on available space.. This PR add a prop containHeight that enables this behavior. When passed true the size middleware get called without any options. containHeight can also take all the options of size as an object as well.
Result:
https://user-images.githubusercontent.com/1192452/189739968-272c50ac-a5c5-4407-887a-bc67f68cc7e6.mp4
now the menu will be fully on screen 🎉

This looks very similar to my PR #1597 which has also been kinda just sitting there.

Thanks for this @hipstersmoothie , I left a follow up in the linked PR explaining our current position regarding changes to these options.

Superseded by #1942


add pair generator

#293

Codecov Report

Merging #315 into 2.0 will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##            2.0   #315   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files        28     29    +1
  Lines       741    798   +57
===================================
+ Hits        741    798   +57

Impacted Files
Coverage Δ

matchzoo/generators/pair_generator.py
100% <100%> (ø)

matchzoo/generators/__init__.py
100% <100%> (ø)
:arrow_up:

matchzoo/generators/point_generator.py
100% <100%> (ø)
:arrow_up:

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 8f39fa3...4e4502c. Read the comment docs.


allow using onConnect with onMessage

We now allow using onConnect with onMessage. This lets you get access to the context {request}. Warning: You can't use .addEventListener when using onMessage, but it currently just silently fails. We'll make this an error later.

landing to unblock matt


Avoid gcov filenames too long

In PR add support for Ninja #22,  I realized that I had introduced two issues:

The gcov file name may exceed the maximum length limit
When build gcov target, always regenerate the gcov file, because the OUTPUT parameter in the add_custom_command command is not the actual generated file.

Solutions:

Replace -p with -x, and move the gcov file to the path corresponding to the hash(-x) for better readability
OUTPUT is the file moved above

Can you link some resources about the "maximum length limit"? As fas as I know a path may take up to 4096 characters in Linux.


Update SsmsMin to latest from SSMS 19

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.406%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
29238

Relevant Lines:
64159

💛  - Coveralls


Inkwell two finger tap doesn't work

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Use Inkwell or GestureDetector above another widget with onTap events

I didn't test it alone but in this context it doesn't work.
There where similar bugs months ago but they didn't fixed this issue.
Expected results
onSecondaryTap works with two finger click on macOS (windows has probably the same issue).
Im not the latest Flutter version for both operating systems.
Actual results
It does not get triggered.
Code sample
Code sample
return GestureDetector(
      onSecondaryTap: () => _navigate(context),
      child: ListTile(
        enableFeedback: true,
        onTap: () => _navigate(context),
        onLongPress: () => _navigate(context),

return InkWell(
      onSecondaryTap: () => _navigate(context),
      child: ListTile(
        enableFeedback: true,
        onTap: () => _navigate(context),
        onLongPress: () => _navigate(context),

Screenshots or Video

Screenshots / Video demonstration
There is nothing to show

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output for windows
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.10.6, on Microsoft Windows [Version 10.0.19045.3208], locale de-DE)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.2)
[√] Android Studio (version 4.0)
[√] VS Code (version 1.80.2)
[√] Connected device (3 available)
[√] Network resources

Hi @Sesa1988
This seems to be fixed as I cannot seem to reproduce this on the master channel. Please try switching to the master channel and check if it still persists on your end.

I can confirm that I am experiencing this as well, I was using onSecondaryTap in a GestureDetector for a context menu and it previously worked well on all three desktop platforms.
Now on Linux and MacOS it no longer works but continues to work as expected on Windows.
I am on the stable channel currently using:
Flutter 3.10.6 • channel stable • https://github.com/flutter/flutter.git
Framework • revision f468f3366c (3 weeks ago) • 2023-07-12 15:19:05 -0700
Engine • revision cdbeda788a
Tools • Dart 3.0.6 • DevTools 2.23.1


Add support for reading HDF5

HDF5 is a popular format to store complicated datasets (alternative to proto files).
Also has different random seeking functionality and is widely used in scientific community.
This is FR for:

Adding HDF5 reader.
Support of HDF5 in tf.learn.

(originally from https://github.com/tensorflow/skflow/issues/52)

May be the hdf5 reader prefer to read from a single huge hdf5 file, instead of a file-name queue.
The bioinformatics data often stores in one mat/gtcx/hdf5 file.
However, open it with h5py.File as numpy array and feed the tf.placeholder is OK for now.

I'll be happy to help with any questions regarding the best way to utilize HDF5.

I would like to work on this issue. I am new here and looking for help to get started.

This can be closed. See an example in examples/skflow.

Thanks @terrytangyuan!

The skflow is not suitable for out-of-core hdf5 data, some hdf5 or matlabv7.3 data are the same size as ImageNet

@ilblackdragon The skflow is not suitable for out-of-core hdf5 data, some hdf5 or matlabv7.3 data are the same size as ImageNet. Maybe a reader randomly acesss hdf5 data and feed the model with reasonbale memory is still necessary.

@sun9700 Yes, the out-of-core doesn't work yet. Feel free to file a separate Feature Request about it (support streaming from hdf5) - and of cause PR will be highly welcome :)


TODO: add ability to run shell/powershell commands

finished, gotta post it in releases but it's present in commit nr https://github.com/AlexDeFoc/goRunner/commit/9f86b03d6a8afa4bdea6c4baaaca3aac9bccb4f3
but gotta update the DOCS to post it. It's really exiting cuz it brings much more power and freedom.

FINISHED THE DOCUMENTATION and Posted to release!!! https://github.com/AlexDeFoc/goRunner/releases/tag/V1.3.0


after selecting hours , popup getting closed

Fixed now!
See live demo - https://plnkr.co/edit/P38jXpcLPnAzWVcpN5Sv?p=preview


Terminal: Better highlighting support when using line cursor

When using "terminal.integrated.cursorStyle": "line" the last letter doesn't get highlighted.
Here are some comparisons of what it looks like in code and conemu
Code

Conemu

VSCode Version: 1.10.0-insider
OS Version: Windows 10

Steps to Reproduce:

Set "terminal.integrated.cursorStyle": "line"
Open terminal
Type some text and then highlight it.

cc: @Tyriar

Upstream issue: https://github.com/sourcelair/xterm.js/issues/529

I'm not sure there's a way to fix this, closing


Merge completion-activities-alone branch development branch

[x] It was decided to make this pull request to be able to perform a merge with all my activities carried out alone.

Great, good Work


Project Dependencies

Users can express dependencies on other model projects in both the same and other Legend instances.

Updating the thread with @beekemarie's latest response as this an essential feature we are awaiting and hoping this might help others wanting to use and deploy LEGEND

Hi Ajith – thanks for reaching out. The metadata server code that is required to set up project dependencies in Studio is fully open sourced. You can find it in this repo.
What we’re still working on is to start releasing Legend-depot versions along with other Legend components and also work on enabling project dependencies in the shared Legend instance hosted by FINOS.
What this means to you, is that you shouldn’t be blocked setting up project dependencies in your local version of Legend. You would just need to decide where you want to publish the Studio project JARs to (e.g. your own Maven repo).

Thanks,
Beeke
cc: @beekemarie @robmoffat

This issue is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This issue was closed because it has been inactive for 35 days. Please re-open if this issue is still relevant.


Error: "No attachment found for service ___"

"No attachment found for service ___" Error is thrown when VC with multiple credential formats is offered.
The error is thrown when the VC offer is accepted by the holder.  The error is thrown at the Issuer's end.

@ksanjaykumar1 are you using the released version of AFJ 0.4.0? This happened to me once when instantiating CredentialsModule with a V2CredentialProtocol that includes both LegacyIndyCredentialFormatService and AnonCredsCredentialFormatService but I thought we've fixed it some time ago.

I am using "0.4.0-alpha.130".


frost-toggle: UX design spec lists hover color - needs implementation

The UX design spec lists hover color as a visual feature of frost-toggle. This is currently not implemented.

I've opened PR https://github.com/ciena-frost/ember-frost-core/pull/466 that includes the implementation for the hover color in frost-toggle.

PR #466 has has been merged so this issue can be closed.


[MonorepoBuilder] Release command inspiration

https://github.com/leviy/release-tool

https://github.com/liip/RMT/blob/master/README.md


Integrate with pyre incremental and adapt the TypeWriter search strategy

It would be interesting to see how well the TypeWriter algorithm (https://software-lab.org/publications/TypeWriter_arXiv_1912.03768.pdf) for searching type annotation suggestions works against type4py. We might get dramatically better results for two reasons:

type4py's ML model seems to perform quite a bit better
today's pyre incremental is orders of magnitude faster than pyre was when the TypeWriter paper was written, so we may be able to try many more combinations and get correspondingly better results

At one point we'd considered hacking this very quickly as an internal project in my company, but we ran out of time. I think it would be better done open-source anyway because then

it would be easier to try out against external projects
we could publish our results with code if they are interesting enough to be worth a paper
the entire OSS community could benefit

I'm unsure if I can find time to prioritize this in the next 6 months at work but it's a little more likely if I treat it as a side project, which would also open the door to an informal weekend hackathon as a way to kick it off :)
I could do this in a separate repository or inside of type4py. What do you think @mir-am ? And does this sound interesting to you?

@stroxler, thanks for your interest in Type4Py and sharing your ideas.
Yes. It is indeed an interesting idea.
We had a similar idea to yours but couldn't do it due to lack of time and working on other projects.
If you don't mind, I'd like to continue the discussion of this idea via email. I believe that GH issues are for bugs and concrete features, not for discussing research ideas. I can send a mail to the email in your GH profile and we continue our conversation there if you're okay.

@stroxler
Thanks for your interest in Type4Py and sharing your ideas.
Yes, it is indeed an interesting idea. We very welcome contributions.
We actually had a similar idea to yours, i.e., using a type checker (mypy) to validate Type4Py's predictions.
I prefer to implement this idea inside Type4Py's repo on another branch. I am also glad to help you to incorporate the idea into Type4Py.
It may be worth mentioning that I used pyre qurey to infer type annotations for augmenting the dataset of Type4Py (see here). Regarding this, I have a couple of questions about using pyre, which I will ask later in this thread.


Is your feature request related to a problem? Please describe.
Not really, want to be able to track metrics.
Describe the solution you'd like
A stats page that shows various metrics:

Central bar graph shows for a specific day (current day, can change it to different days or week, month), the focus time (minutes) vs. time of day (hour)

i.e. worked for 25 minutes from 9:47 am to 10:12am. There will be a bar for 12 minutes at the 9-10am section and a bar for 13 minutes at the 10-11am section

On the side, single values for total focus time for the day, week, month, all time, longest focus time for the day, week, month, all time, average focus time for the day, week, month, all time, total break time for day, week, month, all time, average break time for day, week, month, all time
On the bottom, heat map similar to GitHub commits showing for each day the amount of focus time

Based on the average and the max average for all days, max average would be opaque, lower average would be more transparent

Additional context
N/A

To start, everything will be persisted in local storage as with tasks. Later on, might introduce auth and user accounts to persist across devices.

Tasks

[ ] Central bar graph

[ ] Select different dates

[ ] Single values

[ ] Total focus time
[ ] Longest focus time
[ ] Average focus time

[ ] Heat map


Updated annotation model for certain numerical fields

Fields like proteinStart, proteinEnd, polyphenScore, siftScore, etc. is modeled as numbers in the external APIs we use. We should also model them the same way.

Codecov Report

Merging #108 into master will decrease coverage by 0.27%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #108      +/-   ##
===========================================
- Coverage      5.97%   5.69%   -0.28%
  Complexity       11      11
===========================================
  Files            45      45
  Lines           636     632       -4
  Branches         72      70       -2
===========================================
- Hits             38      36       -2
  Misses          590     590
+ Partials          8       6       -2

Impacted Files
Coverage Δ
Complexity Δ

...exus/service/mixin/TranscriptConsequenceMixin.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...va/org/cbioportal/genome_nexus/util/Numerical.java
80.64% <100%> (+3.5%)
8 <5> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Setup application structure

Description
This PR updates the application to include an Express + MongoDB server. The new server-side code is located in the /server folder.
Structure
This PR introduces a new structure for the codebase for the server and database related code. Here is a snapshot of the new structure:
/config
/database
/server
    /controllers
    /middleware
    /models
/src

Env
This PR also introduces new environment variables for deploying the server code. Here are the new variables.

PORT - The port on which the server will listen to requests (Default: 3000)
DATABASE_URI - The uri of the database used to store data
SESSION_SECRET - A secret phrase used to hash client-side tokens.

Sorry its taking me forever to get to you on this pull request, couldn't get things up and running on Windows after 1.5 hrs of debugging. Just syntax wise I can't get the backend server functioning. Switching over to a Linux VM and everything works just fine


Transition leadership in clubs that are solely led by seniors

We have a bunch of clubs that are only led by seniors right now. Once they graduate, the clubs will die unless we transition leadership.
This issue is to track that transition.

I emailed all of the leaders last week about this and got a bunch of responses. I've been periodically processing those responses as they've flowed in.
I'd like to do a proper sweet of our leader database to make sure every leader is accounted for at some point this week.

Update: we're set on nearly all of our clubs, I just need to follow-up with two leaders to get everything set up.

Okay, so there are a few remaining follow-ups, but I think we're largely set for next year. I'm expecting ~15% club attrition, which we'll easily be able to offset with applications for next year.


Add attachment points rendering to the functional group window

Background
Functional groups window does not show attachment points.
Solution

Change the functional groups to include attachment point and change rendering.
Will require the #2165 to be implemented first
Attachment point should be rendered for the atom which is currently used in SD file to attach to a molecule
If add structure to the canvas - attachment point should be removed

#184325922

Incorporated by:
https://github.com/epam/ketcher/issues/2467
https://github.com/epam/ketcher/issues/2165


[Feature Request]: Improve VM removal

Description
Research if possible and if so optimize the removal logic of VMs. Commont tests currently take a few minutes to be deployed and almost an hour to be removed. This may be related to the removal of the VM backup.
See, for example, if modifying the resource type order in the removal utilities can improve the solution.

Hey @eriqua, is this still relevant? Just checked the max test for a recently successfull run in AVM. The deployment took about 20-ish minutes and the removal about half an hour. I somehow doubt we'll be able to get it much faster - removing a backup doesn't seem to be something that can be optimized.


form shouldDisplay logic causes crash due to stale captured variables -- updated demo to reveal

Describe the bug
form contains code like this inside field closures:
					w.Styler(func(s *styles.Style) {
						f := fm.structFields[i]

but the value of i depends on what was actually  shown previously, so it can crash with out-of-range errors here. this happens in axon, and I updated the order of fields in demo to show it.
we need a canonical path name, not an index, and to find the field data by that path name.
How to reproduce
open demo, click things in the first inline guy.
Example code
No response
Relevant output
No response
Platform
macOS

@kkoreilly the solution to this bug requires keylist (better ordmap), but that is buried in goal repo -- we should just merge that!

Shouldn't this logic prevent this issue? https://github.com/cogentcore/core/blob/e7d7703/core/form.go#L165

hmm good point, but I think it is the label that is causing the problem it seems. the label widget gets stale and it uses the index for styler and doubleclick closures. We actually do have an existing issue for that: #1098  -- not so rare I guess. The original issue was #1096, which is what the code you're pointing to fixed.
in any case, it is much better to just use the path instead of the index, to get the field data, which obviates both issues. you definitely can't just use f directly -- need to re-get it. but there is actually no need to embed the changable index -- just use the path and lookup in an ordered map of fields by path. will be more efficient too.


Is this the original design idea?Producers can create short accounts by eosio

eos3@eos3-virtual-machine:~/eos-dao-node$ cleos get account eosio
privileged: true
permissions:
owner 1: 1 eosio.prods@active,
active 1: 1 eosio.prods@active,
memory:
quota: unlimited used: 1.113 Mb
net bandwidth:
used: unlimited
available: unlimited
limit: unlimited
cpu bandwidth:
used: unlimited
available: unlimited
limit: unlimited
producers:
The eosio account permissions are assigned to all producers as shown above
eos3@eos3-virtual-machine:~/eos-dao-node$ cleos system newaccount eosio eosio111 EOS55UqxQmM1A7Hdfp9kAe9vgHbjJM9iXp7EpCH5sNQier9sAB5gP --stake-net '100000.0000 SYS' --stake-cpu '100000.0000 SYS' --buy-ram-kbytes 8
1569431ms thread-0 main.cpp:428 create_action ] result: {"binargs":"0000000000ea30550000002104ea305500200000"} arg: {"code":"eosio","action":"buyrambytes","args":{"payer":"eosio","receiver":"eosio111","bytes":8192}}
1569436ms thread-0 main.cpp:428 create_action ] result: {"binargs":"0000000000ea30550000002104ea305500ca9a3b00000000045359530000000000ca9a3b00000000045359530000000000"} arg: {"code":"eosio","action":"delegatebw","args":{"from":"eosio","receiver":"eosio111","stake_net_quantity":"100000.0000 SYS","stake_cpu_quantity":"100000.0000 SYS","transfer":false}}
executed transaction: 956f1c955ce39fca5319e83676d1c72f0db9de5681246ac31ea37b3e7b11a3de 336 bytes 11416 us
eosio <= eosio::newaccount {"creator":"eosio","name":"eosio111","owner":{"threshold":1,"keys":[{"key":"EOS55UqxQmM1A7Hdfp9kAe9v...
eosio <= eosio::buyrambytes {"payer":"eosio","receiver":"eosio111","bytes":8192}
eosio.token <= eosio.token::transfer {"from":"eosio","to":"eosio.ram","quantity":"0.1197 SYS","memo":"buy ram"}
eosio <= eosio.token::transfer {"from":"eosio","to":"eosio.ram","quantity":"0.1197 SYS","memo":"buy ram"}
eosio.ram <= eosio.token::transfer {"from":"eosio","to":"eosio.ram","quantity":"0.1197 SYS","memo":"buy ram"}
eosio.token <= eosio.token::transfer {"from":"eosio","to":"eosio.ramfee","quantity":"0.0006 SYS","memo":"ram fee"}
eosio <= eosio.token::transfer {"from":"eosio","to":"eosio.ramfee","quantity":"0.0006 SYS","memo":"ram fee"}
eosio.ramfee <= eosio.token::transfer {"from":"eosio","to":"eosio.ramfee","quantity":"0.0006 SYS","memo":"ram fee"}
eosio <= eosio::delegatebw {"from":"eosio","receiver":"eosio111","stake_net_quantity":"100000.0000 SYS","stake_cpu_quantity":"1...
eosio.token <= eosio.token::transfer {"from":"eosio","to":"eosio.stake","quantity":"200000.0000 SYS","memo":"stake bandwidth"}
eosio <= eosio.token::transfer {"from":"eosio","to":"eosio.stake","quantity":"200000.0000 SYS","memo":"stake bandwidth"}
eosio.stake <= eosio.token::transfer {"from":"eosio","to":"eosio.stake","quantity":"200000.0000 SYS","memo":"stake bandwidth"}
warning: transaction executed locally, but may not be confirmed by the network yet
eos3@eos3-virtual-machine:~/eos-dao-node$
Although eosio on the main network does not have eos tokens, the hypernode can be transferred to eosio and then created a short account through eosio without bidding？？thanks
@jgiszczak
Contributor
jgiszczak commented 2 hours ago
The eosio account can do anything before it drops it privileges, including create short named accounts without bids. That privilege is ordinarily used to create the initial producer accounts. On mainnet, eosio has dropped its privileges, so no one can bypass name bidding.
@jgiszczak jgiszczak closed this 2 hours ago
@shishi614
shishi614 commented 27 seconds from now
how to drops it privileges ,my test network has
Make eosio.msig a privileged account
We make eosio.msig privileged using the following.
$ push action eosio setpriv '["eosio.msig", 1]' -p eosio@active
and
eosio resigns
Once producers have been elected and the minimum number requirements have been met, the eosio account can resign, leaving the eosio.msig account as the only privileged account.
Resigning is basically involves setting the keys of the eosio.* accounts to null. Use the following command to clear the eosio.* accounts' owner and active keys:
eos3@eos3-virtual-machine:~/eos-dao-node$ cleos get account eosio
privileged: true
permissions:
owner 1: 1 eosio.prods@active,
active 1: 1 eosio.prods@active,
memory:
quota: unlimited used: 1.113 Mb
net bandwidth:
used: unlimited
available: unlimited
limit: unlimited
cpu bandwidth:
used: unlimited
available: unlimited
limit: unlimited
producers:
the eosio of my testnetwork and main network is the same .

eosio drops its privileges by nulling its keys as stated in the quoted documentation.  eosio.msig has a contract loaded which does not allow any unilateral action, so it is not equivalent to eosio.  That contract can itself only be updated via a multi-signature process requiring 2/3+1 of all producers signing the update.

Further questions about the design of EOS should be asked at https://eosio.stackexchange.com/ or the various other public discussion forums.  Github issues are for reporting defects or making specific feature requests in the software.


Use bm_core

Part of https://github.com/bristlemouth/bm_protocol/pull/231
Some of the changes:

FreeRTOS.h -> bm_config.h and bm_os.h
pvPortMalloc -> bm_malloc
printf -> bm_debug
remove asserts
cpp -> c (casts, namespace, constexpr)

I see some .cpp files here still. Can/should those be renamed to .c?

I see some .cpp files here still. Can/should those be renamed to .c?
I vaguely remember discussing this at some point and we decided not to change them to .c. I'll dig around some of our meeting notes/other PR's to see if there is any evidence of this discussion.

I didn't find anything in our meeting notes or other PR's. Unless anyone else has any recollection of a conversation where we talked about leaving them as cpp files, I think we can go ahead and make them .c files 👍


Website not rendering correctly

The website hosted by this GitHub repository duplicates its presentation of some iOS apps several times, each with a different number of GitHub stars. This doesn't appear to be desired behavior.
Here are a few screenshots:

The context of this is in #917.

Unfortunately it's hard to know why the cache is interfering, and I was unable to reproduce it on my netlify build. Plus I don't have access to the production netlify as it's on @dkhamsing's account. Maybe there's a way to exclude some folders from the netlify cache which would fix the issue. If somebody is open to investigating that, there might be a long term fix. I don't know if it's also possible to simply disable netlify caching, but then the builds might take longer, and the account may run out of minutes by the end of the month...

Should be fixed, for now - see #917


Add build_capsule utility to jax.ffi

This allows one to build PyCapsules ready for XLA custom call registration out of function pointers retrieved from shared libraries using ctypes. In particular, this obviates the need to create Python bindings to custom call targets.

Looks good! Please squash the commits so we can import.

@superbobry done, thank you for reviewing.

Let's hold off merging until after @dfm's https://github.com/google/jax/pull/21531 goes in. I'll rebase after that.

Ha I was just about to propose the opposite :D
But that sounds good! Should get it in shortly.

@superbobry I've rebased to update for the new location of ffi.py, please re-review!


App: Unable to drag n drop files

cannot drag n drop images / vrm into the browser window of the app to spawn, tested on brave browser.

see issue #1189 - drag and drop may / may not be in an unsupported state.


Kitchen test fails on default-fedora-23 instance

* service[php-fpm]: Service is not known to chkconfig.
================================================================================
Error executing action `enable` on resource 'service[php-fpm]'
================================================================================

Appears to be the same as with ubuntu-16.04 f7e15e6f699d69e9905eb9a0486c79770f0d3fc9
but unlike with ubuntu it wasn't the only issue, please see #97


Bump google.golang.org/protobuf to v1.33.0

Description
Bump google.golang.org/protobuf to v1.33.0.
Type of change

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

Testing performed?

[X] Unit tests
[ ] Integration tests
[ ] Acceptance tests

Checklist:

[X] This PR is being made against the main branch, or relevant version branch
[ ] I have made corresponding changes to the documentation
[ ] I have added testing for my changes

If you have any questions, or want to get attention for a PR or issue please reach out on the #logging-and-metrics channel in the cloudfoundry slack

Closing this PR because protobuf was bumped in this PR: https://github.com/cloudfoundry/loggregator-release/pull/726


Which Values Can Be Passed on Startup?

I see that you can pass --db in the startup cmd but what else can be passed as it would be nice to avoid creating a toml file if possible.
For example, can I pass --description and --name etc?

Pretty much the only command line arguments are --db for the database, and --config for the configuration file. (plus --help & --version).
Running nostr-rs-relay --help will print out the full list.
In src/cli.rs you can see all the CLI args.


go key map works strange when gt is rebound

When I press go in NERDTree window it usually switch current tab to first or second, I realized it only happens when I use my own remapping for gt (I prefer it be relative when I specify {count} for it, I use ,gt as default gt behavior):
nnoremap gt :<C-u>exec join(repeat(['tabnext'], v:count1), '\|')<CR>
xnoremap gt :<C-u>exec join(repeat(['tabnext'], v:count1), '\|')<CR>
nnoremap ,gt gt
xnoremap ,gt gt

When I remove these lines from my config everything goes well.

Environment: My config is pretty complex: https://github.com/unclechu/neovimrc (https://github.com/unclechu/neovimrc/tree/de98da753cfe6a2667b52fb03b565253a98226e2) but my point it is only connected to these maps for gt.
Anyway, you could reproduce it with this minimal config:
" Cloned NERDTree repo with 97433ed
set rtp+=./nerdtree

nnoremap gt :<C-u>exec join(repeat(['tabnext'], v:count1), '\|')<CR>
xnoremap gt :<C-u>exec join(repeat(['tabnext'], v:count1), '\|')<CR>

Operating System: Fedora Workstation 25
Vim version :version:

NVIM v0.2.0
Build type: Debug
Compilation: /usr/bin/cc -Wconversion -DNVIM_MSGPACK_HAS_FLOAT32 -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/build/config -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/src -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/.deps/usr/include -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/.deps/usr/include -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/.deps/usr/include -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/.deps/usr/include -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/.deps/usr/include -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/.deps/usr/include -I/usr/include -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/build/src/nvim/auto -I/home/unclechu/.compile_n_build/neovim/neovim-0.2.0/build/include
Compiled by unclechu@fedorafarm25

Optional features included (+) or not (-): +acl   +iconv    +jemalloc +tui
For differences from Vim, see :help vim-differences

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/home/unclechu/neovim-v0.2.0/share/nvim"

NERDTree version git rev-parse --short HEAD: 97433ed
NERDTree settings applied in your vimrc, if any:

let NERDTreeIgnore = ['\.swp', '\.swo', '\.pyc', '__pycache__']
let NERDTreeShowHidden = 1 " always show hidden files in NERDTree
let NERDTreeShowLineNumbers = 1
let NERDTreeWinSize = 40
let NERDTreeMapHelp = '<Leader>?' " heals backward search
let NERDTreeMapOpenSplit = 'ss'
let NERDTreeMapPreviewSplit = 'gss'
let NERDTreeMapOpenVSplit = 'sv'
let NERDTreeMapPreviewVSplit = 'gsv'

Process: List the steps that will recreate the issue.

Open few tabs (for example four)
Open some file in for example third tab
Open NERDTree
Press go on some file

Current Result:
I switched to another tab.
Expected Result:
Just open this file (under cursor) in the same tab and cursor supposed to stay in NERDTree window.

Optional
Screenshot(s):
I just recorded a screencast to demonstrate it (also with minimal config):
https://youtu.be/WwCxZ9v_gTQ
Possible Fix:
I haven't discovered the code but maybe it happens because somewhere in NERDTree map for go set by map instead of noremap?

I confirm that now it's working correctly, thanks!


Documentation --> Config

Under "Ethslurp's cache folder and config file"

"in a binary format" --> are you referring to one of many different types of binary formats or is there only one singular binary format. if there is only one binary format then it should read "in binary format"

"we store the program's configuration file in the parent. What does "in the parent" refer to?

first paragraph, beginning with the word "each" --> In the first sentence, ethslurp is doing the action. In the second sentence, "we" is doing the action.  I fixed them all. Nice job.


Status in tracker

Active pokemon's Status is now drawn over the top right corner of the pokemon's icon while out of battle
Added comments on the status logging
Adjusted the Status enum and added Status-to-Status-Code map

Thanks for getting this added. A simple addition that goes a long way.


Remove context argument from cache.Cache.Store method

What this PR does:
The Redis and Memcached implementations of the Cache interface use a job queue to perform writes in the background after the Store method has already returned. Including a context argument creates the possibility of the writes being cancelled after the originating request finishes while the writes are still being run in the background. Removing the context makes it harder to misuse the cache like this.
This change also renames the Store method to StoreAsync to make this behavior more obvious.
Which issue(s) this PR fixes:
See grafana/mimir#4341
See grafana/mimir#4342
Checklist

[X] Tests updated
[X] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

What about Delete? Is that async operation too?

What about Delete? Is that async operation too?

No, it's blocking. The underlying Redis/Memcached implementations use the job queue but block until the task completes.


SABL_HEAD Assertion error when class agnostic is False

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
Reproduction

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

This is because SABL does not support class-agnostic regression for now. You can use the class-specific version.

okay.


Add sem_score metric for generation evaluation

This new metrics can be used at generator and prompt_maker
The details at issue description.
close #154

FYI : I tested at full.yaml file that new strategy input shape (input as list of dictionary) works well.


Cannot create object from UTF-8 mail address at Sisimai::Address

https://github.com/Exim/exim/blob/master/test/mail/4223.यहलोगहिन्दीक्योंनहींबोलसकतेहैं
$VAR1 = {
          'feedbacktype' => '',
          'deliverystatus' => '5.0.0',
          'rhost' => 'the.local.host.name',
          'timestamp' => 920367873,
          'diagnostictype' => 'SMTP',
          'addresser' =><EMAIL_ADDRESS>          'listid' => '',
          'diagnosticcode' => 'host <IP_ADDRESS> [<IP_ADDRESS>]',
          'reason' => '',
          'subject' => 'test',
          'action' => 'failed',
          'lhost' => 'the.local.host.name',
          'alias' => '',
          'timezoneoffset' => '+0000',
          'recipient' =><EMAIL_ADDRESS>          'smtpcommand' => '',
          'softbounce' => 0,
          'messageid' =><EMAIL_ADDRESS>          'smtpagent' => 'Exim'
        };

Sisimai::Address->new cannot create a object from the value of "addresser" above.

Sisimai::Address->parse() could not parsed the UTF-8 address.

For example<EMAIL_ADDRESS>should be encoded<EMAIL_ADDRESS>by Punycode. This issue will be closed soon.

RFC 6532 extends RFC 2045 to use raw UTF-8 for address fields in message header (Punycode should not be used anyway).
Additionally, RFC 6533 defines new "utf-8-addr-xtext" and "utf-8-addr-unitext" encodings to use UTF-8 addresses in delivery reports.

Thanks for the comment. I did not follow these RFCs.  A short while ago, I have added 3 emails which are "Cat" in the local part of the recipient address.


Remove documentation on how to setup test reports.

The test reports feature stopped working after my Google Cloud trial
expired. I never used the test reports so it's easier to remove them
instead of updating the GCP credentials.

Merged this into main.

Can I ask why? The instructions were valid anyway, no?
(I'm asking since I think the feature is pretty cool and I was gonna talk about it in my upcoming talk :D )

We should be able to just use the reports with different credentials, I managed to set it up in Metals, so let's at least leave the docs. We can remove the reports from munit if they are unused.

I’m sorry I rushed too quickly to merge this. I’m ok with adding the page back. My impression was only that this feature is not used much. I generally want to avoid documenting functionality that’s half complete.


big query jobs can be cancelled

I have only included one test, I wasn't sure what else to include. In particular I couldn't figure out how to check the complete URL from a test, which I thought would be a good test to check the interceptor built the right url ('project' instead of 'projects'). If the Big Query REST api is updated, the interceptor could be removed.

Thank you for doing this! I just opened #1035 which adds some more tests and docs. I'll close this one out, but would you mind checking it out and making sure it still looks okay? Thanks!


Add Rust SDK Capabilities Built on Existing Python Functionality

Adds support for Rust to the MEXC SDK by building on the existing Python functionality. The Rust implementation uses the PyO3 library to call the Python SDK directly, so Rust developers can interact with the MEXC API without needing to use Python themselves. This keeps the functionality consistent across both languages while making it more accessible.
The MexcClient struct in Rust is analogous the Python Spot class. These Rust methods internally call their Python equivalents, so the behaviour matches what’s already in place for Python users and doesn't change any functionality under the hood of the existing project.

@MXCDev would  you mind reviewing this PR?
The Rust functionality is built directly on top of the existing Python functionality using PyO3 for function mappings so there's no difference in implementation under the hood.
The readme.md is also updated accordingly to show the analogous set up configuration as with the other supported languages. There are also integration tests included and the added files are zipped at dist/rust which matches the predefined structure of the project.