[WIP] Make model builder lambda APIs work with public fields on entities

Please check if the PR fulfills these requirements

[x] The code builds and tests pass (verified by our automated build checks)
[ ] Commit messages follow this format

    Summary of the changes
    - Detail 1
    - Detail 2

    Fixes #bugnumber

[x] Tests for the changes have been added (for bug fixes / features)
[x] Code meets the expectations our engineering guidelines. https://github.com/aspnet/Home/wiki/Engineering-guidelines.

Fixes #11410

Open points:

Figure out what model builder methods should accept expression of either properties or fields
Figure out what's the best place for the added unit test methods

@ajcvickers Did you had a chance to take a look at this one?

@ilmax We will look again next week, since right now we are not putting new features into 2.1, but we haven't branched yet.

Figure out what model builder methods should accept expression of either properties or fields

Will let you know in a day or two after discussing with team.

@smitpatel, I would like to make some progress with this one. What do you suggest to avoid duplicating GetPropertyAccess and also we should figure out all the *builder's methods that needs to work with both property and field based expressions

@ilmax - We should enable field based expression for all the modelbuilder APIs which works with string & property based expressions. That would also help us uniquifying pipeline.
cc: @divega

@smitpatel I'm looking into this and I've updated almost all the methods that takes either a string or an expression. Two places cannot be updated without introducing breaking changes, they are HasOne, HasMany because ReferenceNavigationBuilder and CollectionNavigationBuilder both exposes a PropertyInfo object.
Should I update those types as well?

@smitpatel, @ilmax While those breaking changes should be ok there are more things to consider for the navigations API, see https://github.com/aspnet/EntityFrameworkCore/issues/6674. Therefore it would be better to do in a separate PR

@ilmax - Let's exclude navigation APIs from using Fields and do it for all other APIs

Great! I will work on this one in the coming days!

@smitpatel, @ajcvickers, @AndriySvyryd this needs a bit (read waaaaaaaaaaaayyyyyyyyyy more) test coverage, but I would like to know what to do with the duplicate methods, can we live with that for now? what do you think?

@ilmax If it is difficult to remove duplicated code then we can certainly live with it and do clean up later.
I see you have added some breaking changes. Perhaps that requires review if we want to take breaking changes or we should delay them till 3.0 release.
@AndriySvyryd - Any idea of testing? ModelBuilderTest is the right location but can we do some magic to run all things for fields/properties?

@smitpatel I will try to remove duplication when I'm done with tests, I think that with some generics gymnastics we can probably unify the methods. @AndriySvyryd thank you for the suggestion, I will proceed as you suggested!

Hi @smitpatel, I had a couple of issues recently and had no chance to work on this, yesterday I made some changes to avoid code duplication and I added few more tests.
I also read your comment and realised I should undo some of the changes, so here it is the list of methods that now supports handling expression with either properties or fields:
ReferenceNavigationBuilder<TEntity, TRelatedEntity>.WithMany

CollectionNavigationBuilder<TEntity, TRelatedEntity>.WithOne

EntityEntry<TEntity>.Property
EntityEntry<TEntity>.Reference
EntityEntry<TEntity>.Collection

EntityTypeBuilder<TEntity>.Property
EntityTypeBuilder<TEntity>.Ignore
EntityTypeBuilder<TEntity>.OwnsOne
EntityTypeBuilder<TEntity>.HasKey
EntityTypeBuilder<TEntity>.HasAlternateKey
EntityTypeBuilder<TEntity>.HasIndex

QueryTypeBuilder<TQuery>.Property
QueryTypeBuilder<TQuery>.Ignore

ReferenceNavigationBuilder<TEntity, TRelatedEntity>.WithMany
ReferenceNavigationBuilder<TEntity, TRelatedEntity>.WithOne

ReferenceOwnershipBuilder<TEntity, TRelatedEntity>.Property
ReferenceOwnershipBuilder<TEntity, TRelatedEntity>.Ignore
ReferenceOwnershipBuilder<TEntity, TRelatedEntity>.OwnsOne
ReferenceOwnershipBuilder<TEntity, TRelatedEntity>.HasForeignKey
ReferenceOwnershipBuilder<TEntity, TRelatedEntity>.HasPrincipalKey
ReferenceOwnershipBuilder<TEntity, TRelatedEntity>.HasIndex

ReferenceReferenceBuilder<TEntity, TRelatedEntity>.HasForeignKey
ReferenceReferenceBuilder<TEntity, TRelatedEntity>.HasPrincipalKey

ReferenceCollectionBuilder<TPrincipalEntity, TDependentEntity>.HasForeignKey
ReferenceCollectionBuilder<TPrincipalEntity, TDependentEntity>.HasPrincipalKey

RelationalEntityTypeBuilderExtensions.HasDiscriminator

RelationalQueryTypeBuilderExtensions.HasDiscriminator

I would like to agree on which one to undo, there are some method like HasForeignKey very close to the navigation property api that we might still want to allow accepting lambda with fields because we allow HasKey to be called with a field based expression.
Moreover I'm hitting a couple of issues in the CI around the MonsterFixupChangedChangingSqlServerTest and not sure why, and getting unit test run locally on VS is getting harder and harder, I will investigate what's happening here but any help would be highly appreciated :)

@ilmax - Looking at test failures it seems like perhaps in WithMany API with lambda is not getting correct type inferred. That is causing type difference. You can compare the models with & without change to figure out why the type is different now.

@smitpatel Thank you for pointing that out, I'm having hard times to have unit testing work in visual studio but I'm on it. What do you think about the changes in ExpressionExtensions?

@ilmax - Changes in ExpressionExtensions look good. Perhaps more unification can be done but I may be missing something. I will investigate during merge so you are good on that side.
Let see what @AndriySvyryd says for breaking changes for navigation API & relationship APIs like HasFK/HasPK. If they are fine to change then we would need tests for relationship based too.

@smitpatel Good to know, I'm tracking down the test failure and it's caused by an incorrect collection type instantiated for the nav property, it's a HashSet intead of an ObservableCollection because the CollectionTypeFactory get's an interface instead of a concrete type, so this check
if (typeof(INotifyPropertyChanged).GetTypeInfo().IsAssignableFrom(entityType.GetTypeInfo()))
{
	var observableHashSetOfT = typeof(ObservableHashSet<>).MakeGenericType(elementType);
	if (collectionType.GetTypeInfo().IsAssignableFrom(observableHashSetOfT.GetTypeInfo()))
	{
		return observableHashSetOfT;
	}
}

is failing and thus EF creates an HashSet.
Now I have to track down why it get's an interface instead of a concrete type

@AndriySvyryd any hint on this one? Are we ok with the breaking changes introduced here?

Triage: closing this for now since it has been inactive for a while. We would be happy to re-open as needed.


Nix: remove materialization

In principle this saves us time, but in practice the difference seems to
be not so much.
Time to run nix-shell --run true:

Version
Modification
Time

master
clean
20-25s

master
.cabal file change
22s

master
.hs file change
20-25s

no-materialization
clean
22-25s

no-materialization
.cabal file change
30-40s

no-materialization
.hs file change
22-25s

The times weren't very stable, but overall not using materialization
makes us a few seconds slower in the normal cases, and quite a bit
slower when modifying a .cabal file, but that's a comparatively rare
kind of change.
Overall, I think it's no longer worth it, so let's bin it. Which means
we can also bin updateMaterialized, hooray!

Ah, I found the place we pay for this. It makes Hydra much slower to evaluate. Evals take ~60s on master and ~180s with this PR. I think 180s is still okay-ish, though.


Matching problem of TW-FINCH

It is amazing that this unsupervised clutering method outperforms other paradigms on five challenging action segmentation datasets. However, some details puzzle me a lot, just about how to map the obtainded segments with different action labels (including background) using Hungarian algorithm. It would pretty appreciate if these problems would be explained.

Same question here. I am also very confused about how to use Hungarian algorithm in the action segmentation setting. Could you help give some explanation on it? Thanks


希望出一个隔壁 infra 组 rspack 的按需加载插件

[x] I'm sure this does not appear in the issue list of the repository

Basic Info
What are the similar cases of this feature
none
What problem does this feature solve?
modernjs 都推 v2 了，都加了 rspack 了，内部不支持一波吗 :)

可以参考我的unplugin-react-components项目，按需引入

看已经有了
https://github.com/arco-design/arco-plugins/tree/main/packages/unplugin-react

看已经有了
https://github.com/arco-design/arco-plugins/tree/main/packages/unplugin-react

在modernjs.dev中确实不生效


Update opis/closure

In some cases, opis/closure v2 throws Exception: Serialization of 'Closure' is not allowed.

Coverage remained the same at 90.494% when pulling c1c0da655e9d862e699128eaad9fa66c36354501 on update-opis-closure into eb96ddc908940c139140a86b0605923018d7dad4 on master.


Fixed cart button

Fixed Cart Button in About-Us Page
Issue #529 : The Cart icon is not visible in About Us Page solved solved
Type of PR

[X] Bug fix

Description
This pull request aims to to fix the Cart icon which is not visible in the About-Page but now it fixed and designed in a way the repo owners asked
Screenshots
Before:
The cart icon is not visible in about us page

After:
Now its visible in about us page

Checklist

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

Thank you for reviewing my pull request!

Now I have fixed cart icon in about us page inside the button tag

Please merge the PR @swaraj-das

Thanks @swaraj-das for merging my PR I am very grateful, can you label it as level 1


BUG: Chatbot Design and Response Issues

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The chatbot interface is not optimized for mobile screens. Buttons are too small to interact with, and text is not properly readable. The chatbot often fails to generate responses, leaving users without the necessary interaction or information.  Additionally, the CSS implementation is not good, I want to make it look better.
These issues need to be addressed to ensure a seamless user experience on both desktop and mobile platforms. A thorough review and redesign of the chatbot’s CSS for responsive design, along with investigating the backend response mechanism, are essential steps to resolve these problems.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Its-Aman-Yadav Sir please assign this issue to me under GSSOC 24


Does it only download in PDF?

Would it be possible to add an option to download in .txt format? I can try implementing it if not, never coded in python but it should be a simple change.
edit: looks great btw, thanks for sharing!

For now, it only downloads individual pages. If they also have them in txt, then that should probably be an easy change.
The whole program is only 40 lines, so even without much proficiency in Python you should probably still be able to get the hang.
I suspect that if they have everything in txt as well, then the only thing which needs to be changed is the download url and filename of the outputted document.
https://github.com/Addono/HathiTrust-downloader/blob/6dcdbf989fde2962a92b703f2b1565a15e32a7f1/hathitrustdownloader/cli.py#L18-L21

For now, it only downloads individual pages. If they also have them in txt, then that should probably be an easy change.
The whole program is only 40 lines, so even without much proficiency in Python you should probably still be able to get the hang.
I suspect that if they have everything in txt as well, then the only thing which needs to be changed is the download url and filename of the outputted document.
https://github.com/Addono/HathiTrust-downloader/blob/6dcdbf989fde2962a92b703f2b1565a15e32a7f1/hathitrustdownloader/cli.py#L18-L21

Thanks! Yea it looks simple. Ill get around to it maybe next week.

Thanks! Yea it looks simple. Ill get around to it maybe next week.


See if it's still necessary to spin up a new thread/set priority after RecyclerView is used for the editor

It absolutely will be necessary to spin up a new thread since the highlighter can't save state, and therefore has to run against the entire source text in one go. Setting a low pri for the new thread can't hurt.


kernel declaration differs from its actual calling arguments

From neon/backends/kernels/ptx/sconv_fprop_K128_N128.ptx, we can see there is a parameter called param_CRST, which I think is supposed to represent C*R*S*T. However, in the actual calling code in neon/backends/convolution.py line 130, the argument is K*R*S*T.
Is it a typo or something? I believe the ptx declaration (which is generated from neon/backends/kernel_specs.py) should be param_KRST instead.

You're right, it should be KRST in kernel_specs.  But as it's coded now,
those params are never actually used by name.  They're just there to
generate the signature.  Then the kernels are called with the driver api
and a packed struct.
In bprop, K and C are swapped so it is correct there.  Anyway, I'll change
it for better consistency.
I'll have a brand new set of more flexible direct conv kernels out soon.
On Fri, Mar 25, 2016 at 1:07 AM, Ming Wei<EMAIL_ADDRESS>wrote:

From neon/backends/kernels/ptx/sconv_fprop_K128_N128.ptx, we can see
there is a parameter called param_CRST, which I think is supposed to
represent CRST. However, in the actual calling code in
neon/backends/convolution.py line 130, the argument is KRST.
Is it a typo or something? I believe the ptx declaration (which is
generated from neon/backends/kernel_specs.py) should be param_KRST
instead.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/NervanaSystems/neon/issues/220


Make sure python request timeout is less than apache and nginx timeouts

cc @JCCChris
The python code makes three kinds of requests to the A2P API on Azure:
fetch_citation_data
fetch_case_data
submit_interview
These all make calls to requests.post in __do_request() here.
We should add a timeout param to this post call that is less than the nginx and apache timeouts. See https://2.python-requests.org/en/master/user/quickstart/#timeouts.
If there is a timeout exception, we should bubble it up to the UI and show an error similar to this one.

Added timeouts to the requests in https://github.com/JudicialCouncilOfCalifornia/docassemble.jcc.abilitytopay/commit/a40339dd3d473122d26b751fd949b3f7a3726e6b.
Still need to bubble up timeout errors to the UI. See: https://github.com/JudicialCouncilOfCalifornia/docassemble.jcc.abilitytopay/issues/101.


Design improvements

Motivation
Fixes some issues with mobile menu, adds transparent background before scrolling, implements zigzag pattern for facts (both components and linked items, applied for stack as a whole, not for each fact in it).
Checklist

[ ] Code follows coding conventions held in this repo
[ ] Automated tests have been added
[ ] Tests are passing
[ ] Docs have been updated (if applicable)
[ ] Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test
Check top menu functionality in desktop and mobile version, check zigzag pattern on a page with multiple facts/stacks

made the slightest change to rendering of H1 headings to include some top margin.

suggested improvements split into a standalone issue as they revolve solely around refactoring, without any modifications to the actual experience. public release was agreed upon as a priority, therefore I'm merging the branch. any polishes and refactors shall be dealt with as a part of the low prio epic.


envoy: Implement locality-based prioritized (affinitized) endpoint traffic routing

This issue is for the following sub-issues:

[ ] Build a quick proof of concept of the idea described below #3557
[ ] Create a small design document
[ ] Create sub-issues and link to this issue
[ ] Implement (if feasible) the design

Context
We need to implement traffic routing such that traffic is first routed to the group of endpoints closest to the origin, then to those groups further and further away (next cluster, next zone, next region etc.)
Documentation:

Locality weighted load balancing
config.endpoint.v3.LocalityLbEndpoints

Take a look at the priority field. From the Envoy docs on the priority field:
Under usual circumstances, Envoy will only select endpoints for the highest priority (0).
In the event all endpoints for a particular priority are unavailable/unhealthy, Envoy will
fail over to selecting endpoints for the next highest priority group.

Also seems like the locality should be set more intelligently? https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/core/v3/base.proto#envoy-v3-api-msg-config-core-v3-locality

Also take a look at this: https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/load_balancing/zone_aware.html. I feel it is quite similar to locality based routing

Can I get a early review for this WIP mini design doc?
https://docs.google.com/document/d/1OxDPYkdffEJmraSMZpl83JOt3Kjxjfmoehp4T1ebpfs/edit#

I think we don't need to create sub-tasks for the implementation. Using just this task should be sufficient.
Before the implementation, there are two prerequisites:

OSM Multicluster CRD is ready to be consumed
OSM Multicluster service is completed, so that source service can reach destination service in all clusters.

Then the changes need to be made are:

OSM controller watches MultiCluster resource changes
Updates the endpoint and load_balancing_weight field of the Envoy CDS API response. This should be part of the catalog.

Is this being put behind a feature flag?


[FR] Display the popover without having to press any function key

Hello and thanks a bunch for your amazing plugin!
I love the feature that you could display a preview by hovering over it. But I am often focused with a mouse only and would love the possibility to show the preview just by hovering the heading in the quiet outline sidebar.
I would be glad if you would implement it.

Maybe if a user can set a delay level (milliseconds-seconds), this would be even more interesting.

I did a search here and I think it was implemented a while ago and then changed to have a functional key needed. So I think I can close this one.


Add descriptions to Float & BigFloat rounding methods

These descriptions exist for BigDecimal - adding them for the float methods.

Thanks for taking a look and being willing to accept the change.
Happy to see Crystal still going strong post 1.0!


Updated project description

@"An An': is that your name? I was not sure how to contact you directly. Therefore I send you an updated project description as an issue... ;o)
Your old description reads better when you change it to this:

AAChartKit is an elegant, friendly and easy to use chart library for iOS, based on the open source ‚Highcharts‘ JS libraries. AAChartKit is extremely powerful, easy to configure and a pleasure to use. Currently AAChartKit includes support for the following chart types: column chart, bar chart, area chart, area spline chart, line chart, spline chart, radar chart, polar chart, pie chart, bubble chart, pyramid chart, funnel chart, column range and arrange chart. More chart types are planned to be supported. AAChartKit gives you easy and fast access to a wide range of chart types, hassle free and quickly to integrate into your own project.

Hope that helps. When I find the time I'll need updating the readme and will also commit a few things. Great project - keep up the brilliant work.
Best wishes,
Stefan

The space of description in GitHub is too small to show the whole content ,so I replaced my preface in README with your new description rather than  my old description .^_^

🥇 :o)


Use ruby 2.7

Similar to: https://github.com/cloudfoundry/bosh-azure-cpi-release/pull/634
Also depends on: https://github.com/bosh-packages/ruby-release/pull/20

Also needs a commit update to 2.7.3 rather than 2.7.2

Done


Update fa.yml

updated Persian translation with all the new strings.
fixed a few typos and bad translations

I am not sure why its reporting a conflict? Can you resolve?


About imglists and cityscape_train_gt_roidb.pkl

I want to train mx-maskrcnn with my own data. I found the data infromation was initially obtained from train.lst. Then how could I creat a .lst file of my own dataset. Any help will be appreciated!

You need write a python script which walk through you image folder and get the filename and write to an train or val.lst
you need https://github.com/mcordts/cityscapesScripts to convert you json file to mask images, remember to change label.py to you own label
delete original pkl in model folder and rcnn/dataset/cityscape.py will create new one.

@perp Thanks for your generous help! I will have a try.


Scanning stops at 2897 subdomains...

as above, when I run cloudfail.py the whole process stops on subdomains scans!
I waited over 2 hours for the results but he was there! 'please wait...' he's still there!!! :)

This is not a bug.
A quick look at the subdomains file in the data folder shows there is exactly 2897 subdomains the script scans for.

@Elycin thanks for that answer but i already knew that before ;)
I understand that the script needs so much time to complete the scan?

There is an indication as to when the scan has finished, progress bars are a feature which I am currently working on and will be included in a later version.

This has now been included as of commit 325ccbd4a880f5f4d1c39f07b167aabdbecad4c9 thanks too: @bdoner

hi! Thx for re. and progress bar ;)
If somebody have similar problem with long time scanning, try to upgrade your distribution!
In my case that helped! ;)


Updated use of URL to URI

This PR attempts to clean up some of the language that was copied from OAS that we have since realized was ambiguous.  It also changes the specification to use URI references instead of URLs for the field extends.
The reason for this change is that we want to give the option for tooling to choose whether to treat the URI reference as an identifier or a locator.  With the previous wording a tooling author may interpret the specification as saying that the tooling is required to dereference the extends URI reference to acquire the target document.  That wouldn't be good, for security reasons and it could lead to users to assume that they can pass an overlay with the extends keyword to tooling that expect to receive an OpenAPI document, because it appears that the Overlay is self contained. (This is similar to how Kubernetes Kustomize overlays can be used https://kubectl.docs.kubernetes.io/guides/config_management/introduction/)
This aligns Overlay's use of URI references with the OAS's use of URI references.

Committed @handrews' suggestion


Settings Sync always starts as logged out before realizing it's actually signed in

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.83.0-insider (user setup) 11bfd76a61a299156a9f3138ecfad70937af3527
OS Version: Windows_NT x64 10.0.22631

Steps to Reproduce:

Start VS Code
Note that the Accounts menu is badged and asking for settings sync sign-in
Ignore it and wait a few seconds...
Note that the Accounts menu is now un-badged and settings sync is showing as signed-in

Note: I am signed into Settings Sync using my Microsoft account.
https://github.com/microsoft/vscode/assets/2766036/276ad948-9f86-4309-838c-df2227fdec28
Here's some logs if they're interesting...let me know if I need to turn on trace debugging.

Microsoft Authentication

2023-09-21 16:00:41.970 [info] Getting sessions for all scopes...
2023-09-21 16:00:41.970 [info] Got 13 sessions for all scopes...
2023-09-21 16:00:41.973 [info] [email offline_access openid profile] Getting sessions
2023-09-21 16:00:41.973 [info] [email offline_access openid profile] Got 6 sessions
2023-09-21 16:01:28.905 [info] Getting sessions for all scopes...
2023-09-21 16:01:28.906 [info] Got 13 sessions for all scopes...
2023-09-21 16:01:28.907 [info] [email offline_access openid profile] Getting sessions
2023-09-21 16:01:28.907 [info] [email offline_access openid profile] Got 6 sessions

Also reproes on a different machine using a one-week old Insiders version,
Version: 1.83.0-insider (user setup)
Commit: 03671d332bf6a34782d5adc64086bc8e083a1590
Date: 2023-09-12T15:07:34.330Z
Electron: 25.8.0
ElectronBuildId: 23503258
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.25958


Add button to innate spell list

Add button to innate spell list to add spell duration to turn order.

Moved to #11


Train model in R and problem of using it in C

I am having problem of using a model trained in R to do prediction in C. When tracing in debug mode all the way to the end of the prediction (see C code below), the 'out_len' variable seemed to get garbage value and the 'f' variable was unreadable .
So I am not sure if the model I saved in R is in a compatible/readable format in C. and why the two variables were not assigned the correct values.
Environment info
Operating System:
Windows 8.1
Compiler:
Package used (python/R/jvm/C++):
R3.3.3
xgboost version used:
0.6-4
If installing from source, please provide

The commit hash (git rev-parse HEAD)
Logs will be helpful (If logs are large, please upload as attachment).

If you are using python package, please provide

The python version and distribution
The command to install xgboost if you are not installing from source

If you are using R package, please provide

The R sessionInfo()

sessionInfo()
R version 3.3.3 (2017-03-06)
Platform: i386-w64-mingw32/i386 (32-bit)
Running under: Windows 8.1 x64 (build 9600)

locale:
[1] LC_COLLATE=Chinese (Simplified)_China.936
[2] LC_CTYPE=Chinese (Simplified)_China.936
[3] LC_MONETARY=Chinese (Simplified)_China.936
[4] LC_NUMERIC=C
[5] LC_TIME=Chinese (Simplified)_China.936
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] pROC_1.9.1       DiagrammeR_0.9.0 xgboost_0.6-4
loaded via a namespace (and not attached):
[1] igraph_1.0.1       Rcpp_0.12.10       rstudioapi_0.6     magrittr_1.5
[5] munsell_0.4.3      colorspace_1.3-2   viridisLite_0.2.0  lattice_0.20-34
[9] R6_2.2.0           brew_1.0-6         stringr_1.2.0      plyr_1.8.4
[13] dplyr_0.5.0        visNetwork_1.0.3   Rook_1.1-1         tools_3.3.3
[17] grid_3.3.3         data.table_1.10.4  gtable_0.2.0       DBI_0.6-1
[21] influenceR_0.1.0   htmltools_0.3.5    yaml_2.1.14        lazyeval_0.2.0
[25] digest_0.6.12      assertthat_0.2.0   tibble_1.3.0       Matrix_1.2-8
[29] gridExtra_2.2.1    RColorBrewer_1.1-2 ggplot2_2.2.1      htmlwidgets_0.8
[33] viridis_0.4.0      rgexf_0.15.3       stringi_1.1.5      scales_0.4.1
[37] XML_3.98-1.6       jsonlite_1.4

The command to install xgboost if you are not installing from source

Steps to reproduce
library(xgboost)
library(DiagrammeR)
library(pROC)
d1=read.table("training.txt", sep="\t")
d2=read.table("testing.txt", sep="\t")
bst <- xgboost(data = as(as.matrix(d1[,-1]),"sparseMatrix"), label = d1[,1],
max_depth = 2, eta = 1, nthread = 2, nrounds = 2,
objective = "binary:logistic")
pred <- predict(bst,  as(as.matrix(d2[,-1]),"sparseMatrix"))
auc <- roc(d2[,1],pred)
print(paste('AUC score:', auc$auc))
#xgb.plot.tree(model = bst)
importance_matrix <- xgb.importance(model = bst)
print(importance_matrix)
xgb.plot.importance(importance_matrix = importance_matrix)
xgb.dump(model=bst, fname="xgb.model.txt", dump_format ="text")
What have you tried?
The new data (named 'test')used for prediction in C is pre-processed into a float array of length=rows x cols. Here's the C code:
            DMatrixHandle h_test;
	myDll.XGDMatrixCreateFromMat((float *)test, rows, cols, -1, &h_test);

	BoosterHandle h_booster;
	int ret = myDll.XGBoosterCreate(NULL, 0, &h_booster);
	ret = myDll.XGBoosterLoadModel(h_booster, "xgb.model.txt");

	bst_ulong out_len;
	const float *f;
	ret = myDll.XGBoosterPredict(h_booster, h_test, 0, 0, &out_len, &f);

	for (unsigned int i = 0; i < out_len; i++)
		print("Predicted val %d: %.2f\n", i, f[i]);

The binary model needs to be saved using xgb.save, not xgb.dump.

Hi, I changed to xgb.save to save the model in R and call the XGBoosterLoadModel in C the same way as before. The problem is still there: the out_len still returns garbage and F is still unreadable.
On Wednesday, April 19, 2017 11:12 PM, Vadim Khotilovich<EMAIL_ADDRESS>wrote:

The binary model needs to be saved using xgb.save, not xgb.dump.—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Another not so good thing is that you were training your model using sparse data, but then running predictions from dense data while using -1 as a missing value indicator. In such situation, 0 needs to be used as a missing value indicator.
If you still would be having issues, please provide a fully reproducible example using some dummy data.


implement UTXO tracking for on_disk service

Let's wait on this pending https://github.com/ZcashFoundation/zebra/pull/857#issuecomment-671197639 (hopefully tomorrow).


rdp inside rdp session freezing with smartcard redirection enabled

To Reproduce
Steps to reproduce the behavior:

connect with xfreerdp to windows server 2012r2 on linux with smartcard redirection enabled
inside rdp session connect to second server with smartcard redirection enabled
connection to second server wiil be frozen after credential input

Expected behavior
another rdp sesion inside rdp session should work
Application details

FreeRDP version (xfreerdp /version) 2.10
command:  xfreerdp  /floatbar /f /v:myserver  /cert-ignore /d:mydomain /smartcard /a:printer /bpp:32
(myserver and mydomain has changed for security reason)

OS version connecting to windows server 2012 r2

If available the log output from a run with /log-level:trace 2>&1 | tee log.txt

Environment (please complete the following information):

OS: linux 5.15.0-70-generic
Version/Distribution: Astra Linux 1.7 x86-64
Architecture:amd64

Additional context
If i swith off smardcard redirection to second server inside running rdp session then connection to second server will work fine without freezing.

Output of xfreerdp /buildconfig:
xfreerdp /buildconfig
This is FreeRDP version 2.10.0 (2.10.0)
Build configuration: BUILD_TESTING=OFF BUILTIN_CHANNELS=ON HAVE_AIO_H=1 HAVE_EXECINFO_BACKTRACE=1 HAVE_EXECINFO_BACKTRACE_SYMBOLS=1 HAVE_EXECINFO_BACKTRACE_SYMBOLS_FD=1 HAVE_EXECINFO_H=ON HAVE_EXECINFO_HEADER=1 HAVE_FCNTL_H=1 HAVE_GETLOGIN_R=1 HAVE_GETPWUID_R=1 HAVE_INTTYPES_H=1 HAVE_JOURNALD_H=TRUE HAVE_MATH_C99_LONG_DOUBLE=1 HAVE_PIXMAN_REGION=OFF HAVE_POLL_H=1 HAVE_PTHREAD_MUTEX_TIMEDLOCK=ON HAVE_PTHREAD_MUTEX_TIMEDLOCK_LIB=1 HAVE_PTHREAD_MUTEX_TIMEDLOCK_SYMBOL= HAVE_SYSLOG_H=1 HAVE_SYS_EVENTFD_H=1 HAVE_SYS_FILIO_H= HAVE_SYS_MODEM_H= HAVE_SYS_SELECT_H=1 HAVE_SYS_SOCKIO_H= HAVE_SYS_STRTIO_H= HAVE_SYS_TIMERFD_H=1 HAVE_TM_GMTOFF=1 HAVE_UNISTD_H=1 HAVE_XI_TOUCH_CLASS=1 WITH_ALSA=ON WITH_CAIRO=OFF WITH_CCACHE=ON WITH_CHANNELS=ON WITH_CLANG_FORMAT=ON WITH_CLIENT=ON WITH_CLIENT_AVAILABLE=1 WITH_CLIENT_CHANNELS=ON WITH_CLIENT_CHANNELS_AVAILABLE=1 WITH_CLIENT_COMMON=ON WITH_CLIENT_INTERFACE=OFF WITH_CUPS=ON WITH_DEBUG_ALL=ON WITH_DEBUG_CAPABILITIES=ON WITH_DEBUG_CERTIFICATE=ON WITH_DEBUG_CHANNELS=ON WITH_DEBUG_CLIPRDR=ON WITH_DEBUG_DVC=ON WITH_DEBUG_KBD=OFF WITH_DEBUG_LICENSE=OFF WITH_DEBUG_MUTEX=ON WITH_DEBUG_NEGO=OFF WITH_DEBUG_NLA=OFF WITH_DEBUG_NTLM=OFF WITH_DEBUG_RAIL=ON WITH_DEBUG_RDP=ON WITH_DEBUG_RDPDR=ON WITH_DEBUG_RDPEI=ON WITH_DEBUG_RDPGFX=ON WITH_DEBUG_REDIR=ON WITH_DEBUG_RFX=ON WITH_DEBUG_RINGBUFFER=ON WITH_DEBUG_SCARD=ON WITH_DEBUG_SND=ON WITH_DEBUG_SVC=ON WITH_DEBUG_SYMBOLS=OFF WITH_DEBUG_THREADS=ON WITH_DEBUG_TIMEZONE=ON WITH_DEBUG_TRANSPORT=ON WITH_DEBUG_TSG=ON WITH_DEBUG_TSMF=OFF WITH_DEBUG_TSMF=OFF WITH_DEBUG_TSMF_AVAILABLE=0 WITH_DEBUG_URBDRC=OFF WITH_DEBUG_WND=ON WITH_DEBUG_X11=ON WITH_DEBUG_X11_CLIPRDR=ON WITH_DEBUG_X11_LOCAL_MOVESIZE=ON WITH_DEBUG_XV=ON WITH_DSP_EXPERIMENTAL=OFF WITH_DSP_FFMPEG=ON WITH_EVENTFD_READ_WRITE=1 WITH_FAAC=OFF WITH_FAAD2=OFF WITH_FFMPEG=TRUE WITH_FFMPEG=TRUE WITH_GFX_H264=ON WITH_GPROF=OFF WITH_GSM=ON WITH_GSSAPI=OFF WITH_ICU=ON WITH_INTERNAL_MD4=OFF WITH_INTERNAL_MD5=OFF WITH_IPP=OFF WITH_JPEG=ON WITH_LAME=OFF WITH_LIBRARY_VERSIONING=ON WITH_LIBSYSTEMD=ON WITH_MACAUDIO=OFF WITH_MACAUDIO=OFF WITH_MACAUDIO_AVAILABLE=0 WITH_MANPAGES=ON WITH_MBEDTLS=OFF WITH_OPENCL=OFF WITH_OPENH264=OFF WITH_OPENSLES=OFF WITH_OPENSSL=ON WITH_OSS=ON WITH_PAM=ON WITH_PCSC=ON WITH_PROFILER=OFF WITH_PROXY=OFF WITH_PULSE=ON WITH_SAMPLE=OFF WITH_SANITIZE_ADDRESS=OFF WITH_SANITIZE_ADDRESS_AVAILABLE=1 WITH_SANITIZE_MEMORY=OFF WITH_SANITIZE_MEMORY_AVAILABLE=1 WITH_SANITIZE_THREAD=OFF WITH_SANITIZE_THREAD_AVAILABLE=1 WITH_SERVER=ON WITH_SERVER_CHANNELS=ON WITH_SERVER_INTERFACE=ON WITH_SHADOW=ON WITH_SMARTCARD_INSPECT=OFF WITH_SOXR=OFF WITH_SSE2=ON WITH_SWSCALE=ON WITH_THIRD_PARTY=OFF WITH_VAAPI=OFF WITH_VALGRIND_MEMCHECK=OFF WITH_VALGRIND_MEMCHECK_AVAILABLE=1 WITH_VERBOSE_WINPR_ASSERT=ON WITH_WAYLAND=ON WITH_WINPR_TOOLS=ON WITH_X11=ON WITH_XCURSOR=ON WITH_XDAMAGE=ON WITH_XEXT=ON WITH_XFIXES=ON WITH_XI=ON WITH_XINERAMA=ON WITH_XKBFILE=ON WITH_XRANDR=ON WITH_XRENDER=ON WITH_XSHM=ON WITH_XTEST=ON WITH_XV=ON WITH_ZLIB=ON
Build type:          Debug
CFLAGS:              -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -Wall -Wno-unused-result -Wno-unused-but-set-variable -Wno-deprecated-declarations -fvisibility=hidden -Wimplicit-function-declaration -Wredundant-decls -g -fno-omit-frame-pointer -DWINPR_DLL
Compiler:            GNU, 8.3.0
Target architecture: x64

log.txt
Attached log

@kazan417 your build is broken (you did enable stuff that is meant for development for your compile), retry with a normal build


Create Documentation for Popover component

### Tasks
- [ ] https://github.com/appsmithorg/appsmith/issues/26285

@tanvibhakta Completed the documentation. Can you review the same?
https://www.notion.so/appsmith/Popover-Documentation-11017cc81fde45bd847f897d86af6e2b?pvs=4

@tanvibhakta has to create document on storybook

Documentation on usage of size variants of popover has to be created by @vasanthappsmith

Size variants and Scroll should be added as well I believe?


feat: define Filter.absorbing

Also prove that a set is von Neumann bounded
iff Filter.absorbing 𝕜 S ≤ 𝓝 0.
These lemmas make it easy to simplify  IsVonNBounded 𝕜 S,
provided that we have 𝓝 0 = ⨅ (a1 a2 ... ak), 𝓟 (f a1 a2 ... ak),
even if the family of sets in the RHS is not directed,
so Filter.HasBasis is false.

bors merge


python311Packages.ufmt: init at 2.1.0

Description of changes
Safe, atomic formatting with black and usort
https://github.com/omnilib/ufmt
Fixes #239794

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 239825 run on x86_64-linux 1

24 packages built:

python310Packages.moreorless
python310Packages.moreorless.dist
python310Packages.stdlibs
python310Packages.stdlibs.dist
python310Packages.trailrunner
python310Packages.trailrunner.dist
ufmt (python310Packages.ufmt)
ufmt.dist (python310Packages.ufmt.dist)
usort (python310Packages.usort)
usort.dist (python310Packages.usort.dist)
python310Packages.volatile
python310Packages.volatile.dist
python311Packages.moreorless
python311Packages.moreorless.dist
python311Packages.stdlibs
python311Packages.stdlibs.dist
python311Packages.trailrunner
python311Packages.trailrunner.dist
python311Packages.ufmt
python311Packages.ufmt.dist
python311Packages.usort
python311Packages.usort.dist
python311Packages.volatile
python311Packages.volatile.dist


switch base64 to use OpenZeppelin

LGTM
+1 from Security. Justification: Core assembly has been audited as part of OZ's v5.0 release.


Final Preparation for the SFML 2.6 Release

[x] Update SFML version

[x] In CMake
[x] In the config header
[x] In the find_package example
[x] In the info.plist

[x] Update the changelog

[x] In the repository

[ ] Build & Package

[x] VS 2017 (32bits)
[x] VS 2017 (32bits)
[x] VS 2019 (32bits)
[x] VS 2019 (64bits)
[x] VS 2022 (32bits)
[x] VS 2022 (64bits)
[x] MinGW GCC 13.1.0 (32bits)
[x] MinGW GCC 13.1.0 (64bits)
[ ] Linux GCC 64bits
[ ] macOS Clang 64bits (OS X 10.7?)
[ ] Source Code
[ ] HTML Documentation

Complete checklist: https://github.com/SFML/SFML/wiki/SD%3A-Releases
Website Updates: https://github.com/SFML/SFML-Website/pull/161

While for the website, not everything is fully ready yet. It makes it easier to build stuff from source, if we've merged and tagged the code. The builds look so far so good (after the fix from yesterday).
I'll wait for CI to pass before tagging the commit. 😉


WordBazaar - voting hide initiate vote per default

closed with https://github.com/aeternity/superhero-ui/pull/974

closed with https://github.com/aeternity/superhero-ui/pull/974


Add an API for installing a handler

Currently, handlers are specified declaratively in the web manifest. This is great for indexing, but it means the browser needs to decide when to prompt the user to install a handler. (On first page load? After some time using the site? When the site is added to home screen?)
Instead, we should add an API that lets the web app programmatically request to prompt the user to install the handler. Note: The handler would still need to be declaratively specified; the API would just be a trigger to install. Then the website can provide a contextual button where the user requests to install the handler.
Downsides: We may actually want to leave the decision to browser policy (e.g., in Chrome, when the user clicks "Add to home screen" or "Add to desktop", and tie handling to app installation). This may be giving too much control to the website.

I think to start with we would definitely want to leave the decision to the browser.

I asked the question of the Web Manifest editors but I wonder why you'd not want to allow a website to at least request that a manifest is installed? [1] This says nothing about how the website handles user consent.
There are use cases for installing an app that go beyond simply adding the app to your home screen or desktop (such as registering a handler).
We're particularly interested in this for the Web Payments WG work and also possibly having "pay" as an action.
So, rather than having a way to register a handler wouldn't it be better to have a way to explicitly request that browser install a manifest?
[1] https://github.com/w3c/manifest/issues/446


Log about how rerunner is configured

This null pointer problem occurred because I worked with the latest code of ladybug and the FF!. This code was under construction by that time. Niels and me should communicate better about what code to test.


Don't use responseType = blob | safari fails

Ideally i think this should be reported to Safari but you could fix it much faster
I found out about this bug on a page that is running https

the page has an iframe with a sandbox attribute
the sandbox attribute only has "allow-script"
the iframe is executing fetch's polyfills that boils down to this code:

xhr = new XMLHttpRequest()
xhr.open('GET', 'https://httpbin.org/get')
xhr.responseType = 'blob'
xhr.send()
xhr.onload = () => {
  var fr = new FileReader()
  fr.onerror = () => console.error
  fr.onload = () => console.log(fr.result)
  fr.readAsText(xhr.response)
}

The result i'm getting is

[blocked] The page at https://xxxxx/ was not allowed to display insecure content from blob:null/bd7df00e-cc00-41d7-a81d-64e9962417bf.

Thanks for tracking this down. Ideally, we shouldn't be setting xhr.responseType = 'blob' for every request, mostly because this is sort of wasteful in scenarios where we're only interested in response.text() (the most common ajax scenario in today's applications), which causes the responseText -> Blob -> string conversion.
However, since we legitimately need to support non-text response types, such as when you use response.blob() or response.arrayBuffer(), our only option is to unconditionally set xhr.responseType = 'blob' in advance, because at request time we can't know which type will the user need to consume later.
So for now, I don't see a way how we can avoid opting into blob mode when just reading text. If anyone has ideas, I would like to hear them.

I'm running a forked version of your fetch in order to support streams but i don't think you would want it, adds to much dependencies.

I'm building it in es6 and always making sure ReadableStream, URLSearchParam, blob, typed arrays exist as polyfills, i'm also including core.js to support everything web-streams-polyfill needs

But i'm not here to talk about that. The way i'm able to stream is by not setting any responseType so it's just plain text. That way I can use the progress event and get part of the response and then be able to stream it. if i would have used any other kind (arraybuffer, blob) then i wouldn't be able to slice it and stream it.
So you maybe want to consider using just plain text if you ever want to support streams
you can always transform the text to an arraybuffer, json or blob later.
one thing you should be worrying about then is encoding if you want to turn a binary text into a blob.
The solution is simple really, you just have to add this:
xhr.overrideMimeType('text/plain; charset=x-user-defined')

if it's to any interested in my fork then i can share it with you

So you maybe want to consider using just plain text if you ever want to support streams
you can always transform the text to an arraybuffer, json or blob later.

Interesting. I'd like to see a PR that makes this change, if you have the time. I'd prefer to see it separate from the streaming implementation, for clarity.

My version is quite different cuz everything is transformed into stream no mather what you put in as argument into the Response constructor (the initbody fn). Even FormData gets transformed into a buffer/stream since i'm able to read it with the newest spec. I don't have _bodyText, _bodyBlob, _bodyFormData, _bodyArrayBuffer any longer and treating everything as a stream since you need to be able to stream it anyway and truly knowing the state of bodyUsed and locked to a consumer,
So you could say that mine is built all around buffers/streams, where as you still keep the reference to string, Blob, FormData, typedArray etc. So it's not some easy cherry picking from my code exactly your response methods are very conditional... Mine is closer to the spec but ways a lot more (due to dependencies)
But it maybe dosen't have to be so complicated as it may seems
how about just switching
      if ('responseType' in xhr && support.blob) {
        xhr.responseType = 'blob'
      }

to
      xhr.overrideMimeType('text/plain; charset=x-user-defined')

This is just exactly the reason why i went with my fork instead of your polyfill

if you use overrideMimeType then you don't get the original content-type :(
setting it to responseType = arraybuffer worked much better

I am also having this issue in opera 25 running on a tv: xhr.onerror is called right after xhr.send. If I remove xhr.responseType = 'blob', the request succeeds.
One option may be to conditionally set responseType based on the response headers in the onreadystatechange callback.
      xhr.onreadystatechange = function() {
        if (
          xhr.readyState === XMLHttpRequest.HEADERS_RECEIVED &&
          xhr.status === 200 &&
          'responseType' in xhr &&
          support.blob &&
          xhr.getResponseHeader('Content-Type') !== 'application/json'
        ) {
            xhr.responseType = 'blob'
        }
      };

This let's the tests pass, but I'm not sure about any other content types we need to pass on.

We don't have to conditionally set the response type to blob. We need to change the response type to arraybuffer

If anyone has ideas, I would like to hear them.

@mislav
We are experiencing a performance hit with a STB box called Humax - GTN-1008S with one of our users - Apparently it runs out of memory.
We did this change and it fixed it https://github.com/github/fetch/pull/752/files
Now, I would like to discuss the best solution to see if can get this fixed here too.

If anyone has ideas, I would like to hear them.

@mislav
We are experiencing a performance hit with a lower powered device with one of our users - Apparently it runs out of memory.
We did this change and it fixed it https://github.com/github/fetch/pull/752/files
Now, I would like to discuss the best solution to see if can get this fixed here too.

I believe this is fixed by https://github.com/github/fetch/pull/752 -- If I'm wrong and this is still an issue, please comment and I will reopen the issue and investigate some more :-)


Cleanup and version bump

promotes Bart to maintainer
cleanup of change logs
version bump to a consistent 0.3

https://github.com/micro-ROS/micro_ros_diagnostics/issues/34
Signed-off-by: Nordmann Arne (CR/ADT3<EMAIL_ADDRESS>

Codecov Report

Merging #35 (51a94ca) into master (6cbf0fd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #35   +/-   ##
=======================================
  Coverage   38.52%   38.52%
=======================================
  Files           2        2
  Lines         122      122
  Branches       19       19
=======================================
  Hits           47       47
  Misses         61       61
  Partials       14       14

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 6cbf0fd...51a94ca. Read the comment docs.


[Bug] skywalking-client-js reporting error unnormal on SPA

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
NodeJS Client Side Agent (apache/skywalking-client-js)
What happened
in browser panal, the error information skywalking-client-js collects always come from the single page,even I already set enableSPA=true.
first enter page A, then enter page B and triggering errors in page B, DevToolds network shown that it send three request, two is perfData,one is errorlog.
the perfData's pagePath is correct path,and I believe its come from setPerformance.
the errorLog's pagePath is wrong,it's shown the error created in pageA,but the error acutally created in pageB.
先打开A页面，在进入B页面，并在B页面触发一个错误。网络里显示发送了三条请求，两条是perfData,一条是errorLog.
perfData的pagePath是正常的，一条是进入A页面的记录，一条是进入B页面的记录，我认识是router.beforeEach中的setPerformance方法产生的。
而errorLog产生的记录错误，它显示错误来自A页面，而错误实际是在B页面触发的，我认为是register方法的pagePath没有更新。
What you expected to happen
I want all metrics in browser panal display normally. the error's pagepath is pageB.
How to reproduce
Environment: Vue 2.6.10  Skywalking9.0.0 Skywalking-client-js0.8
in permission.js, I have this code:
import Vue from 'vue'
import ClientMonitor from 'skywalking-client-js';
router.beforeEach(async(to, from, next) => {
  ClientMonitor.setPerformance({
    collector: `${process.env.VUE_APP_BASE_API}/skywalkingHandler`,
    service: 'owind-web',
    pagePath: to.path,
    enableSPA: true,
    vue: Vue,
    useFmp: true,
    serviceVersion: 'v1.0.0'
  });
  versionControl.isNewVersion()
  next();
})
----------------
in main.js, I have this code:
import ClientMonitor from 'skywalking-client-js';
ClientMonitor.register({
  collector: `${process.env.VUE_APP_BASE_API}/skywalkingHandler`,
  service: 'owind-web',
  pagePath: location.href.substring(location.href.lastIndexOf('#') + 1),
  autoTracePerf: true,
  // enableSPA: true,
  vue: Vue,
  useFmp: true,
  serviceVersion: 'v1.0.0'
});

Anything else
I also want know is my usage correct？on SPA,if I use setPerformance,do I still need register?
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@SiuChau Are you going to fix this? You checked Yes I am willing to submit a PR!.

sorry,I still dont know why is it happend,and how to fix it. if anyone could show me how to use it correct,Im willing to discuss and submit a pr. 我是愿意提交修复的，不过我是后端开发，没怎么写过前端代码，现在也不确定问题出在哪里，是不是我用法错了。
我看了一下client-js的源码，register方法会调用segments  handleerrors  perfData。但是我不太理解前端是怎么监控错误产生的，以及为什么传入handleerror的pagePath不会更新。

Open-source community can't afford to explain the codes, I am afraid.
It is better you could read codes and debug, or provide a demo app to reproduce the error. Otherwise, we could do guessing only

那我可能无法提供pr，但我仍认为这是一个BUG，复现的环境和代码我贴在了How to reproduce，希望能得到帮助

Please update all your comments into English to dual languages, otherwise, ppl in other countries can't follow what happens.

@Fine0830 if u need more infomation please let me know


I cannot see my SocialIcon on github.io

Describe the bug
Hello, I can see my SocialIcon on the localhost
But, when I upload to the github.io, I cannot see my SocialIcon
To Reproduce
Steps to reproduce the behavior:
This is my blog - https://github.com/JacksonK9/blog
Can you check my data?
Screenshots
this is my localhost

this is my github.io

Desktop (please complete the following information):

Windows 11 OS
Whale, Microsoft Edge Browser
hugo v0.95.0-9F2E76AF windows/amd64 BuildDate=2022-03-16T14:20:19Z VendorInfo=gohugoio

**SmartPhone

Iphone 12 Pro max

Additional context
I already check this issue https://github.com/lxndrblz/anatole/issues/31

Hi there,
the reason why they do not appear correctly is because of an integrity error.

Which version of the theme are you currently using?
Additionally, you could check out this issue:
https://github.com/lxndrblz/anatole/issues/114
Hope that helps.

After check your comment, I updated to the lastest version.
And I checked the line endings.
It works!
Thank you
I love your theme 👍


CCN HMI

ESA submited the Sepcification. CS will respond. Deadline 16/02/2021(?)

This is done. Thanks.


YAML folds not working

Neovim version (nvim -v | head -n1)
0.7.2
Operating system/version
Who cares
How to reproduce the issue
Open any YAML file
Expected behavior
Expect folds to work
Actual behavior

Tried every configuration example I can find on this site including #21 but nothing works.

-- Option 2: nvim lsp as LSP client
-- Tell the server the capability of foldingRange,
-- Neovim hasn't added foldingRange to default capabilities, users must add it manually
local capabilities = vim.lsp.protocol.make_client_capabilities()
capabilities.textDocument.foldingRange = {
    dynamicRegistration = false,
    lineFoldingOnly = true
}

That does not work. Have you opened a yaml file yourself with that setup (with yamlls installed and attached to the buffer)?

Work for me. Debug yourself to make sure capabilities.textDocument contains foldingRange.

Acttually, I had exactly same error with exactly same yaml langserver.
Problem was exactly in the fact that I missed to pass capabilities to exactly that langserver.
FYI, @windowsrefund

I have added this lines of code to include the capabilities in the available language servers (this file is in my .config/lvim/after/plugin/nvim-ufo.lua)
vim.o.foldcolumn = '1' -- '0' is not bad
vim.o.foldlevel = 99 -- Using ufo provider need a large value, feel free to decrease the value
vim.o.foldlevelstart = 99
vim.o.foldenable = true
vim.o.fillchars = [[eob: ,fold: ,foldopen:,foldsep: ,foldclose:]]

-- Tell the server the capability of foldingRange,
-- Neovim hasn't added foldingRange to default capabilities, users must add it manually
local capabilities = vim.lsp.protocol.make_client_capabilities()
capabilities.textDocument.foldingRange = {
    dynamicRegistration = false,
    lineFoldingOnly = true
}
local language_servers = require("lspconfig").util.available_servers() -- or list servers manually like {'gopls', 'clangd'}
for _, ls in ipairs(language_servers) do
    require('lspconfig')[ls].setup({
        capabilities = capabilities
        -- you can add other fields for setting up lsp server in this table
    })
end

local builtin = require("statuscol.builtin")
require("statuscol").setup({
  relculright = true,
  segments = {
    { text = { builtin.foldfunc }, click = "v:lua.ScFa" },
    { text = { "%s" }, click = "v:lua.ScSa" },
    { text = { builtin.lnumfunc, " " }, click = "v:lua.ScLa" },
  },
})

require('ufo').setup()

But for some reason I keep getting this "Request textDocument/foldingRange failed with message: Cannot read properties of undefined (reading 'lineFoldingOnly')" error message every time I open a yaml file.
When I do check if the ymlls server is available, it in fact is!

How can I check if the capability has been added to my ls on runtime (i.e. after lunarvim has gone through the whole configuration setup) ?

@kevinhwang91 this is not fixed, passing capabilities doesn't fix the issue, reopen please.

I've also experienced this issue. For me, it was caused by initializing ufo before my LSPs.
What I'm now still confused about is why this was only happening with yamlls and not with other LSPs like gopls or lua_ls. For the other LSPs, :UfoInspect always said Selected provider: lsp and folding was working as expected.
Would be awesome if someone knows what's the reason behind that 🙂


[Feature] New API's for poll management

Is your feature request related to a problem? Please describe.
Frequent polling of api's can be problematic
Describe the solution you'd like
Two new api's would be very handy to help reduce frequent api polling of data heavy endpoints

An api to indicate whether there is new data by returning a timestamp of the last time data was updated
An api to indicate whether there is new data per api by returning a list of api's with a timestamp of the last time data was updated for that api

Describe alternatives you've considered
I could not come up with any solutions simpler than this
Additional context
These new api's should be intended to be polled very frequently

initial work done with #208

Hi @Vutlhari,
How would you like to handle automatically updating the timestamps?
ie read file last modified when the api server starts up?
Would it be possible to setup a github hook to redeploy the api server when master is updated so that the lastest changes reflect on the api?

I think we can setup a git hook to write the value somewhere, maybe a simple key-value store.
Then the API can read the value from the same place. We could ride the value in Redis for quick access.

Cool just let me know what I can do to help

An api to indicate whether there is new data per api by returning a list of api's with a timestamp of the last time data was updated for that api

HTTP has standard headers to manage this sort of thing: Last-Modified on responses to indicate when the content was last updated, and If-Modified-Since on requests to indicate that the body should only be returned if it has been updated after a particular time. Browsers will (I think) automatically use this (together with cache-control headers) to cache content, without needing a custom API.
It might be worth checking if raw.githubusercontent.com provides Last-Modified headers that reflect the date of the last commit. If so, then the API code can just forward those headers from whichever CSV file is used. If that works then I suspect it should be possible to write some middleware that would handle this mostly automatically for all handlers.
If load on either the API itself or on Github is a concern (the latter because requests all come from one place, possibly making Github treat it as a DoS), then I'd suggest putting something like Varnish in front of the API to cache responses.

@bmerry would you like to look into this?

@bmerry would you like to look into this?

I'll take a look when I get around to it; but it might only be a few weeks, so if someone has time to work on it sooner I don't want to stand in the way.

@bmerry would you like to look into this?

I'll take a look when I get around to it; but it might only be a few weeks, so if someone has time to work on it sooner I don't want to stand in the way.

I'm looking at an issue now, so maybe I will pick it up after that.

@Vutlhari
Thanks @bmerry, had a look into it, would just require adding header to the existing responses
ctx.Response().Header().Set("Last-Modified", "Wed, 21 Oct 2015 07:28:00 GMT")
Last-Modified: <day-name>, <day> <month> <year> <hour>:<minute>:<second> GMT
We will still need to implement a way to ascertain and inject the date ie the Redis suggestion.
Will also need to add to the readme advising people to use the "If-Modified-Since" header in their requests.

@Vutlhari are the api's fetching the data failes from github then serving them rather than from its local file system?
I had a look at how to get the timestamps and came up with one way that stores the timestamp of the last commit to modify said file when the api server starts up, then uses that in the header "Last-Modified" when sending the response. Does this sound like a reasonable solution?

@Remwrath I'm trying to understand your use case a little better:

Do you want to know for sure the last time the data changed, or do you just want an indication that it might be worth checking again?
Are you worried about frequent queries putting a load on the server, or on the client (or both)?
Is the client a web browser (using Javascript to pull the data from the API) or something else?

@bmerry

Mainly looking for an indication that its worth checking again, could also be useful to to know the last time the data changed.
Worried about frequent queries to the server, I realise may be premature optimisation but currently the api is doing a file lookup, processing the entire file and sending all the data irregardless of how much of that data you need.
I am imagining a system using the data wanting the latest data and so polling very frequently(say every 5sec to 5min) to make sure its has the up to date data.

@Vutlhari does Heroku provide any statistics on the number of queries or amount of traffic?
I think most of these concerns can be addressed by a combination of providing Cache-control headers in the application (should be pretty easy to do with a middleware) and putting Varnish in front of it. Varnish will then cache the data, which will limit the rate at which we query Github regardless of how fast queries are coming in.
We would need to decide how long to allow data to be cached. Given that each data set is typically only updated once a day, 2-5 minutes seems quite reasonable.
I've never actually used Varnish so I'm not totally sure how this will all work, but I'm hoping it will also insert suitable headers to allow web browsers (and other clients that wish to) to validate that their copy is up to date (relative to the copy in the cache) without fetching the content.

I am imagining a system using the data wanting the latest data and so polling very frequently(say every 5sec to 5min) to make sure its has the up to date data.

It sounds a bit crazy to poll every 5 seconds for data that only updates once per day. If an app is doing that, I'd suggest it should first reduce the polling rate before trying anything clever to reduce bandwidth.

@bmerry

Yes, Heroku does provide metrics, BUT I am using the free-tier (with dynos that sleep after 30 minutes of not being queried).
I have a lot of AWS credits so I can consider hosting the API there and setup proper logging & rate limits.
I’ve seen requests for adding a DB to make querying of data a bit more efficient.
3.1. I initially chose to write the API in Golang out of interest for the language( but I don’t have lots of experience building APIs in it).
3.2. I could host this API in AWS if I can get more helping fixing the structure of the app OR I could migrate the API to .NET core where I have more experience.

If the decision is to move to API I can breakup the application into two:

A simple cron job that pulls the CSV data & writes to a DB.
 This would allow us to use API keys whomever is interested in using the API & rate limit their requests to avoid situations like ‘calling the API every 5 seconds’.

Lastly, I might be overthinking this whole thing 😅

Yes, Heroku does provide metrics, BUT I am using the free-tier (with dynos that sleep after 30 minutes of not being queried).

You might be able to get a very rough idea from heroku logs --tail, since it logs every HTTP request.

@Vutlhari please close if you are happy with this.

@vukosim i still don’t have the invite to officially join the repo.

@Vutlhari  go to https://github.com/dsfsi/covid19za/invitations


Get event time

This patch add support for getting the event time of key-press, key-release, button-press, button-release and cursor motion. This is useful especially for cases of frame rate variation, like frame drops or disabled v-sync.
The test event.c was updated to print the event time, showing the event time between parenthesis, as printf("...(event time: %0.3f)...").
I tested the Wayland, X11 and Windows implementations. The Mac/Cocoa and Mir implementations are untested.
How it is done. The global structure of each platform contains a member variable lastEventTime to hold the time of the last event handled. When an event is handled, the time the event was added in the queue is stored in lastEventTime and later can be retrieved by the function glfwGetEventTime(), which will make the proper conversion to return the value in seconds.

How do you handle wrapping?

Since the function is meant to be called from within callbacks and is limited to the main thread, there is no need for the window parameter. The time could be stored in the global struct.

Style-wise it is excellent.

How do you handle wrapping?

I didn't understand what you mean.

Since the function is meant to be called from within callbacks and is limited to the main thread, there is no need for the window parameter. The time could be stored in the global struct.

Fixed that. Now it doesn't take the window as argument and the event time is stored in the global struct.

I have a question. When I first read the PR description, and the use cases, I expected the implementation to bundle the event time with all events. So that if a key is pressed at time t0 and a mouse button is pressed at t1, you can tell exactly when those events happened.
However, looking at the implementation, I see it stores a single global time of the "latest" event.
Is it okay that each latest event overwrites the time of previous events? How does everything fit together?

Is it okay that each latest event overwrites the time of previous events? How does everything fit together?
As far as I can tell, it's ok., because internall, GLFW will immediately call the respective callback of the event, and they will be processed in order of their arrival:

Receive the event
Store the event time
Do necessary work with the event properties
Call the callback
Receive next event...

Between 1 and 2, theoretically, shouldn't have any event, since they are collected in order from the event queue (part of the system in the case of Windows and MacOS, or the window sytem in the case of X11, Wayland or Mir).

Thanks for clarifying, that makes sense if that's how it works.

@ferreiradaselva I mean that milliseconds stored in a uint32 will overflow every 49 days or so, counted from system boot and not the start of the process. That applies to the Windows, X11 and Wayland implementations of this function. The Windows value is signed as well and will wrap to -2³¹. The code should instead probably accumulate the difference between timestamps into a uint64.

Got it. I will work on a fix for that.

Won't the user need to also know the current time, according to whatever time source the message queue uses, for this feature to be fully useful? If so, what do we return on X11?

@ferreiradaselva, @elmindreda, I would argue that returning time in double format is not the best/practical/useful solution. If this was retuning time in nanoseconds, it would be more useful, because you would be able to reliably compare those values.

@elmindreda As far as I know, the point is being able to compare the time that the event was pushed in the queue with the time returned by glfwGetTime()
@ghuser404 glfwGetTime() returns double that's why this function is returning double, too.

@ferreiradaselva, makes sense. Although I think glfwGetTime() would be useful as a reference time if we could get current time of event handling clock, and set it via glfwSetTime().
Too bad it's all using imprecise floating point values.

@ghuser404 I completely understand your instinct to use integral numbers to represent time, but the fact is that your program has to be running for a really long time before floating-point precision starts to cause any problems.

@tombsar, see http://floating-point-gui.de/errors/comparison/
float a = 0.15 + 0.15
float b = 0.1 + 0.2
if(a == b) // can be false!
if(a >= b) // can also be false!

This is enough for me to not use floating point values for such problems.

@ghuser404 We are well aware of the limitations of floating-point representations. I have yet to see a single real-world bug caused by storing time as a double, and I can't think of any that could not be fixed in the user's code (e.g. by not comparing for exact equality with a constant that is known to not have an exact representation). If you concerned, you can always use uint64_t millis = glfwGetTime() * 1000.0 or similar as your basis for time comparison.

@ferreiradaselva I wish we could use glfwGetTime as a reference, but it's on at least some platforms backed by a different time source and it uses an offset that the user can change at any time with glfwSetTime.
I've looked into making this line up, but there's still the problem of reference time on X11 where we may not even be running on the same machine as the event time source.

User side timestamps are imo are not a solution here. This would also not be making the exposed glfw api multithreaded but the backend platform implementations instead.


feat: pack and push to local repo

In my workflow I usually need to push packages locally to test them before pushing to the remote repo.
For example, given my problem:

I have one repo with the database
I have other many repos which connect to that database
I want to update the database schema, and this is not an application, just a library

My solution:

change the database schema locally, then change the Entities related to that schema
To tests those entities I need to add the new database version to an application
Packing and pushing locally helps me to add the library to my local application

Indeed, I already have code for this, I was just wondering if this is a good improvement for the plugin
with overseer:
local tmpl = {
  name = "Pack Nugets",
  builder = function(params)
    return {
      name = "Pack Nugets " .. LazyVim.root.git():match("([^/]+)$"),
      cmd = "dotnet pack",
      cwd = LazyVim.root.git(),
    }
  end,
}
return tmpl

local tmpl = {
  name = "Push All Nuget Packages Locally",
  builder = function(params)
    local root_git = LazyVim.root.git()
    local command = "find " .. root_git .. " -type d -name 'Release' -exec find {} -name '*.nupkg' \\;"
    local result = vim.fn.systemlist(command)
    bash_cmd = ""
    for i, res in ipairs(result) do
      if i > 1 then
        bash_cmd = bash_cmd .. " && "
      end
      bash_cmd = bash_cmd .. "dotnet nuget push " .. res .. " --source /root/.nuget/local_source/ --api-key ''"
    end

    return {
      name = "Push nugets locally",
      cmd = bash_cmd,
      cwd = "/",
      components = { { "dependencies", task_names = { "Pack Nugets" } }, "on_exit_set_status", "default" },
    }
  end,
}
return tmpl

I could choose which packages to pack/push, but I think this is not needed (at least in my case, where I don't too many of them)

Yeah, good suggestion. Ill take a closer look once I get my head above water on the testrunner 😅


Debugger cannot find the local file

Hi,
I am having issues remote debugging a service under Docker, I can add breakpoints and I can see on the logs that VS is receiving the local variables and all information, In fact, I can see the local variables on the left menu. The issue though is VS is failing to display the current line in the file.
I am getting the following error:
Unable to open 'SearcherService.cs': Unable to read file '/app/Services/SearcherService.cs' (Error: Unable to resolve non-existing file '/app/Services/SearcherService.cs').

I am not sure where this path is coming from /app/Services/ but it is definitely now where my local files are.
OSX host VS
Service running on docker using Ubuntu 16.04

You must be compiling code inside of the container (or downloading the .dll+.pdb) from somewhere else.
VS Code doesn't support browsing to find source files. So you need to configure the debugger up front to map source paths that in in your .pdb files to local file paths. You do this with sourceFileMap in launch.json (instructions).


[💡 Feature]: Add due date to todo

Is your feature request related to a problem?
Some todos have a specific due date/time and a user doesn't want to miss this deadline.
Describe the solution you'd like.
Add a due date/time input for todos as an optional configuration.
Describe alternatives you've considered.
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is this what you had in mind?

@kaydenvanrijn this looks lovely, would you like to take a stab at it?

@christian-bromann Yup, I'll see what I can accomplish on my spare time

Sweet. Just fyi: we have a large refactor branch almost ready to get merged. I would suggest to work on top of cb-state-management-cleanup to avoid any conflicts. This one is the last major rewrite we have planned for now.

@kaydenvanrijn just wanted to check in with you how things are going. Have you made progress on this?


Fixes #6640

This Fixes #6640.
This is the structure that is causing the infinite loop:

The cause of the loop is the dependency between the potential stereochemistries of atom 8 and  the imide double bond at the opposite side of the ring. Given that the imide bond does not have any "real" neighbors (just an implicit H), it can never be resolved to cis or trans, and would stay undersolved after https://github.com/rdkit/rdkit/pull/6473 and https://github.com/rdkit/rdkit/pull/6527. The reason we didn't see this after any of these patches is that none of the structures we looked at there did have a dependency on the newly added (and fixed) imine bonds.
This fix solves the infinite loop problem by catching the neighborless imine bonds in updateBonds() and definitively marking them as "Unknown", since the lack of neighbors on one side prevents us from ever resolving the bond into cis or trans. Once the imine bond is no longer indetermined, the dependency does no longer trigger an infinite loop.

@ricrogz Thanks a lot for the quick fix!

No problem!


Safetensors

Description
Closes #1249
This PR implements support for downloading and converting model checkpoints from huggingface which use the safetensors format rather than the pth binary (pickle) format.
Changes

Allow the tensor map file to be found under different *.index.json names
Allow loading model files with safetensors.torch.load when needed
Allow downloading safetensor files if they exist without pth files or the model explicitly prefers them

Testing

I have tested that the download and load for llama3.1 are un-changed with these changes (no safetensors are downloaded)
I have verified (on my WIP branch for Granite Code) that for models with only safetensors, they are not ignored and can be cleanly converted

NOTE: The CLA is in process since I'm contributing through the IBM corporate CLA.

Thanks @gabe-l-hart
I appreciate you contributing. Code looks good, please rebase when you're ready and i'll approve and merge.
Also since it looks like you're already adding another model to the list and have tested it feel free to bring in the definition so others can benefit by downloading via the cli

Thanks @byjlw! I'll get the rebase done shortly. As you noted, this is part of my work to add Granite Code support. I have a single branch pointer for all that work, but I have also tried to organize the commits so that they can be merged as bite-sized features rather than a big overhaul. Right now, the addition of the configs for Granite Code are at the end of the chain since the model won't work at all until the other features are present (it also currently only works in the python runtime). I'm totally open to whatever convention the torchchat team prefers in terms of contribution chunking, so just let me know what you prefer!

chunks are definitely better. Would love to learn more about the overall goals for the Granite code support

Would love to learn more about the overall goals for the Granite code support

Good point, I didn't ever spell this out! Let me open a top-level issue about that support that I can use to track it all.

Top-level Granite Code support issue: https://github.com/pytorch/torchchat/issues/1262

Actually though, when i tested it by changing the the model.json to use safetensors for 11b base download errored out. It looks like this code works as long as it's not using definitions that use the torchtune format. Do you mind testing this case and resolving the issue?
"meta-llama/Llama-3.2-11B-Vision": {
        "aliases": ["llama3.2-11B-base", "Llama-3.2-11B-Vision-base"],
        "distribution_channel": "HuggingFaceSnapshot",
        "distribution_path": "meta-llama/Llama-3.2-11B-Vision",
        "prefer_safetensors": true
    },

Fetching 19 files: 100%|█████████████████████████████████████████████████████████████████████| 19/19 [1:06:30<00:00, 210.04s/it]
Converting meta-llama/Llama-3.2-11B-Vision to torchtune format...
Traceback (most recent call last):100%|████████████████████████████████████████████████████| 4.99G/4.99G [30:11<00:00, 3.52MB/s]
  File "/Users/byjlw/Documents/source/working/torchchat/torchchat.py", line 85, in <module>| 4.92G/4.92G [12:17<00:00, 3.34MB/s]
    check_args(args, "generate")
  File "/Users/byjlw/Documents/source/working/torchchat/torchchat/cli/cli.py", line 52, in check_args
    download_and_convert(args.model, args.model_directory, args.hf_token)
  File "/Users/byjlw/Documents/source/working/torchchat/torchchat/cli/download.py", line 123, in download_and_convert
    _download_hf_snapshot(model_config, temp_dir, hf_token)
  File "/Users/byjlw/Documents/source/working/torchchat/torchchat/cli/download.py", line 82, in _download_hf_snapshot
    convert_hf_checkpoint_to_tune( model_dir=artifact_dir, model_name=model_config.name)
  File "/Users/byjlw/Documents/source/working/torchchat/.venv/lib/python3.11/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/Users/byjlw/Documents/source/working/torchchat/torchchat/cli/convert_hf_checkpoint.py", line 184, in convert_hf_checkpoint_to_tune
    raise RuntimeError(f"Could not find {consolidated_pth}")
RuntimeError: Could not find /Users/byjlw/.torchchat/model-cache/downloads/meta-llama/Llama-3.2-11B-Vision/original/consolidated.pth

Ah, good catch! I guess N == 2 is a small sample size. I'll dig into your error and see how far I can get.

@ebsmothers can also help

Great. I'll report progress or blockers as they come up. @ebsmothers let me know if you dig in and get anywhere!

@byjlw so I'm not super familiar with how torchchat is handling checkpoint conversion but if you're switching from the .pth format to .safetensors format you will no longer be able to just do the simple move that's happening here. The safetensors format for Llama 3.2 11B Vision distributes the model weights across multiple files (see here) so they will need to be loaded and merged into a single state dict as we do in torchtune here.

That makes sense. A similar approach is being taken in convert_hf_checkpoint to load the sharded weights and convert them into the .pth format needed by torchchat. If I'm understanding the logic in convert_hf_checkpoint_to_tune correctly, it looks like it's really just a simpler version of the logic in convert_hf_checkpoint that doesn't require the tensor renaming or permuting. If that's the case, I think the fix should be to hoist out the shard-loading logic into a helper and then only do the post-processing in convert_hf_checkpoint before resaving as model.pth.
I finally have the safetensors downloaded locally, so I'll see how far I can get with this approach.

I have the mechanics of this working for the safetensors weights with Llama-3.2-11B-Vision-Instruct, but it appears that there is a naming difference between the tensors in original/consolidated.pth and the safetensors weights. This is likely similar to the name remapping needed in the standard conversion logic.
Given that, and given the fact that these models do already have checkpoints that match the target naming scheme, I think it might make sense to leave the PR as-is for now and not switch prefer_safetensors to true for these models. I could also see an argument for using safetensors all-the-way-through to avoid the known pickle vulnerabilities with .pth, but this PR doesn't address that anyway since the models are converted to .pth during the conversion process.


Implement Swipe Navigation for Memory Review in iOS App

Current Behavior
In the current iOS app, when a user is viewing memories:

User selects a memory from the Memories screen
To view another memory, user must return to the main Memories screen
This process repeats for each memory review

Proposed Feature
Implement swipe gesture functionality to navigate between memories within the memory view.
Technical Requirements

Add swipe gesture recognition to the memory view controller
Implement logic to load the next/previous memory based on swipe direction

Swipe left: Load next memory
Swipe right: Load previous memory

Ensure smooth transition animation between memories
Handle edge cases (e.g., first and last memories in the list)

Expected Behavior

Users can swipe right or left to move to the next or previous memory
Navigation between memories occurs without returning to the main Memories screen
Transition between memories should be smooth and intuitive

Benefits

Improves user experience by providing a more fluid navigation
Reduces the number of taps required to review multiple memories
Aligns with common iOS gesture patterns, enhancing app's native feel

Potential Challenges

Ensuring performance remains smooth when loading memories on-the-fly
Maintaining consistency with other parts of the app's navigation structure

Additional Considerations

May need to update the UI to indicate the availability of swipe navigation
Consider adding a setting to enable/disable this feature for user preference

Sounds good, but will not likely be implemented in the near future, closing for now.

Thanks for the note. I was hoping this would be an easy one. We'll wait for future updates.

@mdmohsin7 @thinhx can we try this again?


[CLOSED] C11 requires definiting atomic types with _Atomic specifier

 Issue by hvolos
Wednesday Aug 31, 2016 at 16:39 GMT
Originally opened as https://github.com/hkimura/libnvwal/issues/2

Currently, we use regular type-specifiers for defining objects that are modified through C11 atomic functions. However, according to the C11 reference we also need to define such objects as atomic with the _Atomic type specifier:
http://en.cppreference.com/w/c/language/atomic

 Comment by hkimura
Wednesday Aug 31, 2016 at 18:28 GMT

`
void nvwal_atomic_test(void* a) {
atomic_int* aa = (atomic_int*) a;
atomic_fetch_add(aa, 1);
}
0000000000003491 <nvwal_atomic_test>:
3491:   55                      push   %rbp
3492:   48 89 e5                mov    %rsp,%rbp
3495:   48 89 7d e8             mov    %rdi,-0x18(%rbp)
3499:   48 8b 45 e8             mov    -0x18(%rbp),%rax
349d:   48 89 45 f8             mov    %rax,-0x8(%rbp)
34a1:   48 8b 45 f8             mov    -0x8(%rbp),%rax
34a5:   f0 83 00 01             lock addl $0x1,(%rax)
34a9:   90                      nop
34aa:   5d                      pop    %rbp
34ab:   c3                      retq
`
vs.
`void nvwal_atomic_test(void* a) {
int* aa = (int*) a;
atomic_fetch_add(aa, 1);
}
0000000000003491 <nvwal_atomic_test>:
3491:   55                      push   %rbp
3492:   48 89 e5                mov    %rsp,%rbp
3495:   48 89 7d e8             mov    %rdi,-0x18(%rbp)
3499:   48 8b 45 e8             mov    -0x18(%rbp),%rax
349d:   48 89 45 f8             mov    %rax,-0x8(%rbp)
34a1:   48 8b 45 f8             mov    -0x8(%rbp),%rax
34a5:   f0 83 00 01             lock addl $0x1,(%rax)
34a9:   90                      nop
34aa:   5d                      pop    %rbp
34ab:   c3                      retq
`

 Comment by hkimura
Wednesday Aug 31, 2016 at 18:27 GMT

As we chatted, the language standard wants it, but most likely the compiler wouldn't do anything different as far as we explicitly call atomic accessors like atomic_fetch_add().
I did check the compiled code w/ and w/o, but there were no difference. See the next post.

 Comment by hkimura
Wednesday Aug 31, 2016 at 18:32 GMT

And, the reason I'm hesitant to have the keyword in _types.h is that it definitely requires C11 to compile it.
At least in public headers, I don't want to have _Atomic keyword.

 Comment by hvolos
Wednesday Aug 31, 2016 at 18:36 GMT

Sounds good.


Add leading zero to Slides' name

Add leading zeros to files in python-for-beginners/Slides to arrange well.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ChaeyeonSeo sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Ryan dev4 - 2023-12-21

Update leftside bar CSS and function
Add 'My projects' page
Add 'NewProject' widget

Where did you find these icons? I'm looking for an arrow icon. No other problems.

@kesesek

I download them in Todoist.
Click F12 in your browser, you can inspect the element in the page. Click the button at 1, you could choose the element in the page. Step 2, click where you want to know, the browser will locate the element in the right widow as shown in the picture with number 3. Then you could copy them.


Update node.js.yml

package-lock.json doesn't exist so i replaced npm ci with npm install

Hmm that's not a bad change but I already added the package-lock on the main branch. How about disabling the v15.x checks that fail everytime for now?


broken when viewed from https server

Because of browser restrictions the CDN content doesn't get loaded and you get an unstyled page when viewing generated files from a https server.
I've fixed in a branch 6d49c9337f72dd37f65e79c94d245ae713fe0612 but I've just realised that using protocol relative URLs means it fails when you open it as a file locally in your browser (i.e. not from a server).
I'm not sure if there's another fix that works for both file:// http:// and https://

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/4235161-broken-when-viewed-from-https-server?utm_campaign=plugin&utm_content=tracker%2F332585&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F332585&utm_medium=issues&utm_source=github).

In theory, this should be fixed in 2.1.0. Closing issue for now.


Test-AzTemplate:  Unescaping brackets in nested templates (fixes #476)

@bmoore-msft:  We may have some extra work to do here, in order to make this scenario testable.
We need a test to exercise this scenario, however, since this part of expansion is actually done inside of Test-AzTemplate, the only way we currently have to hit it is to write a test that uses some of the data.
Unfortunately, -TestCase isn't working the way I'd expect it to (and thus this isn't as straightforward as I'd like )

#476 is non-repro without this fix - so close this out for now.


Build fail with "parent snapshot does not exist"

Description
Build will fail with
FATA[0000] apply layer error for "docker.io/library/test:latest": failed to prepare extraction snapshot "extract-810343375-eUiw sha256:2f8da9dda8e00a45de9a98ebd2dbc4394149baf8686dbda33d20899c797c9f5f": parent snapshot sha256:e644ff0c302d7de4c3e03dc718bc4b77ebeb324f7c8f5483b463fa230027b02c does not exist: not found

This is random and may take some time.
To repro:
while true; do nerdctl build --debug-full --no-cache -t test .; done

Steps to reproduce the issue
na
Describe the results you received and expected
Loaded image: docker.io/library/test:latest
DEBU[0000] stateDir: /run/user/501/containerd-rootless
DEBU[0000] RootlessKit detach-netns mode: true
DEBU[0000] rootless parent main: executing "/usr/bin/nsenter" with [-r/ -w/tmp --preserve-credentials -m -U -t 972422 -F nerdctl build --debug-full --no-cache -t test .]
DEBU[0000] Choosing the buildkit host "unix:///run/user/501/buildkit-default/buildkitd.sock", candidates=[unix:///run/user/501/buildkit-default/buildkitd.sock unix:///run/user/501/buildkit/buildkitd.sock]
DEBU[0000] Choosing the buildkit host "unix:///run/user/501/buildkit/buildkitd.sock", candidates=[unix:///run/user/501/buildkit-default/buildkitd.sock unix:///run/user/501/buildkit/buildkitd.sock]
DEBU[0000] Chosen buildkit host "unix:///run/user/501/buildkit/buildkitd.sock"
DEBU[0000] worker labels: map[org.mobyproject.buildkit.worker.executor:oci org.mobyproject.buildkit.worker.hostname:lima-v2 org.mobyproject.buildkit.worker.network:cni org.mobyproject.buildkit.worker.oci.process-mode:sandbox org.mobyproject.buildkit.worker.selinux.enabled:false org.mobyproject.buildkit.worker.snapshotter:overlayfs]
DEBU[0000] running /usr/local/bin/buildctl [--addr=unix:///run/user/501/buildkit/buildkitd.sock build --progress=auto --frontend=dockerfile.v0 --local=context=. --output=type=docker,name=docker.io/library/test:latest --local=dockerfile=/tmp --opt=filename=Dockerfile --no-cache]
[+] Building 0.5s (6/6)
[+] Building 0.5s (6/6) FINISHED
 => [internal] load build definition from Dockerfile                                                                     0.0s
 => => transferring dockerfile: 124B                                                                                0.0s
 => [internal] load metadata for docker.io/library/nginx:latest                                                          0.1s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => CACHED [1/2] FROM docker.io/library/nginx:latest@sha256:04ba374043ccd2fc5c593885c0eacddebabd5ca375f9323666f28dfd5a9  0.0s
 => => resolve docker.io/library/nginx:latest@sha256:04ba374043ccd2fc5c593885c0eacddebabd5ca375f9323666f28dfd5a9710e3    0.0s
 => [2/2] RUN echo 'Hello Nerdctl From Containerd' > /usr/share/nginx/html/index.html                                    0.0s
 => exporting to docker image format                                                                                0.3s
 => => exporting layers                                                                                0.0s
 => => exporting manifest sha256:2cab31a5770a6e4b1b883e12003f7db13dbec31f1453ffd9c5cf40f595786007                        0.0s
 => => exporting config sha256:d17d7567c3198103f72ee65d06acb0b63a5bc524a9afcef847fd647fb6eeec83                          0.0s
 => => sending tarball                                                                                0.3s
FATA[0000] apply layer error for "docker.io/library/test:latest": failed to prepare extraction snapshot "extract-153144429-zNRp sha256:29099085e37cb571e3c6e6f499df323f4965d1bd7cd8139415fc03fedb121077": parent snapshot sha256:3b487b874707d138c99a5dc4c6ffdb9a19b45937998141fa43aefab26d607347 does not exist: not found
fail

What version of nerdctl are you using?
Client:
 Version:	v2.0.0-rc.1-132-g814418aa.m
 OS/Arch:	linux/arm64
 Git commit:	814418aa3becd4231ca39e400832bc86dc5481d1.m
 buildctl:
  Version:	v0.15.2
  GitCommit:	9e14164a1099d3e41b58fc879cbdd6f2b2edb04e

Server:
 containerd:
  Version:	v2.0.0-rc.3-211-g7b948faa0.m
  GitCommit:	7b948faa084506e72967d88b5d35423211f13b07.m
 runc:
  Version:	1.2.0-rc.3+dev
  GitCommit:	v1.2.0-rc.3-1-g961b8031

Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

@AkihiroSuda suggesting we make this a blocker for 2.0.
This might be yet another variant of #827 #3415 and many others, which apparently would stem from:
https://github.com/containerd/containerd/issues/8973

Is this a recent regression?
Probably this should be rather reported to https://github.com/moby/buildkit/issues ?

Is this a recent regression?
Probably this should be rather reported to https://github.com/moby/buildkit/issues ?

I will test with older versions soon to check that.
My hunch is that this is the same root issue in containerd that is causing the other issues with commit and push, but this is unverified yet.
i am not sure where the fix should go yet.

The containerd bug is stale and the PR to fix got closed.
we might have to work with this inside nerdctl.


Please add Jython support

Some of the system using scripting still uses jython a lot. It is requested to add support for jython in vs code as well

Duplicate of #4762.


Http/keycloak session timeout causes changes in UI to be discarded.

Describe the bug
Occassionally when adding many changes in the UI the http session or Keycloak token timeout or  is exceeded. Eg adding 30 Labels with an Extractor on a schema.
To Reproduce
Create a Schema and click save.
Add x30 Labels by hand with an Extractor or 2. Then add a combination function.
By this time something will have timed out. A popup window might indicate the user has to login again.
Clicking Save again will occur. No popup to confirm it has been saved.
Navigate away from the Schema. Come back to the Schema and the 30 Labels are missing.
Version
0.7.11

Java
17

Can you verify this against master ?


added support for icon color @ tabbar

First draft for adding custom colors to the tabbar icons. Based on the changes one would use it in the following way (note the use of 'renderAsOriginal').
The example uses the Icon "users" from the Entypo iconset in two different colors.
import Icon from 'react-native-vector-icons/Entypo'

<Icon.TabBarItemIOS
    iconName='users'
    selectedIconName='users'
    iconColor='#ffffff'
    selectedIconColor='#000000'
    renderAsOriginal={true} />

Reason for the fix is due to the default grey template color being used otherwise. First time making a pull request so comments and/or help would be greatly appreciated!

Thanks for your PR! I've added some comments, would be great to mention this usage in the README :-)

Thanks a lot!


Relationship loader api

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
[ ] Pre-Commit Checks were ran and passed
[ ] Tests were ran and passed

Description

Adds a relationship loader API to the repository layer, which allows fine grained loader control at the query level, e.g. selectinload(Author.books).load_only(Book.id)

Closing this as it's now merged into main.


Maven Function plugin cannot install function extensions in the right place

Expected behavior
When running mvn clean package. The extension dlls should be installed at the function staging directory (under 'target/azure-functions/your-function-name/bin/').
Actually
But now the bin folder is generated under the project base path.
Root cause
This is a regression issue in the Azure-Functions-Core-Tools: 2.0.1-beta.29: https://github.com/Azure/azure-functions-core-tools/issues/531.
Workaround
When you want to run function locally, or deploy to the cloud side. Please copy the bin/ folder to target/azure-functions/<function-name>/bin/

Fixed in https://github.com/Azure/azure-functions-core-tools/issues/531


✨ Sketch polars plugins

#1  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sjquant and the rest of your teammates on  Graphite


[test] Change availability guarding test in StringGraphemeBreaking.swift

The change to fix this behavior never actually landed in Swift 5.8, so bump the availability of the test to 5.9.
Resolves: rdar://125041033

@swift-ci please test

Shouldn't we cherry-pick this to the 6.0 branch?


Update SQL Server 2019 extension for 0.8.0 release

@kevcunnane I'll merge to get extension updates staged pending docs publishing.


List of buffers on top like with airline?

Hey guys,
first of all thanks for the great plugin! I was wondering if I can config a second bar on the top for all my open buffers like the tabline in airline?

Sorry but I will not support that feature in lightline.