Collection mobile

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[x] Refactoring

Context

[x] Closes #5076
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[x] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

will require merge resolution with #5146
need to merge #5120  before merging this

@daiagi yup, that is correct, there is no grid button on mobile - mobile nft cards are also bit different

I will have a look 👍

oki, menu icon into a button - like on mobile fixes, stroke + dropshadow

change the nft cards on mobile to the minimized version like on explore

[] count icon on filter menu toggle button
Like sort have

:heavy_check_mark:  oki, menu icon into a button - like on mobile fixes, stroke + dropshadow
:heavy_check_mark: change the nft cards on mobile to the minimized version like on explore
:heavy_check_mark: and with the animation, there is a like a flash in darkmode before the screen moves in in (solid bg with lowered opacity - white), not sure if this was applied to desktop as well but the icons in menu are delayed and then jump over the whole screen
:heavy_check_mark: count icon on filter menu toggle button (mobile only)
:heavy_check_mark: sort button after chain selector on explore (and collection) in mobile

niceee, its looking good

please change the menu so it looks same as on explore

changer are - "filter" to bold, remove dropshadow around reset all button

when is sorting applied and filters not, the menu icon is not aligned with content around

looks like its broken on bigger touch screens as well

animation looks good now 👍
Please remove the divider that was separating the breadcrumbs and content on desktop

And then we may delete buy now by default from collections? Thinking out loud. Wdyt @roiLeo

when is sorting applied and filters not, the menu icon is not aligned with content around

yep, notice it too, fixed already

the item count now is on weird spot due to removing the divider on all touch devices I think.

also on desktop the divider is missing now

the creator name changes to black bold on mobile, it should remain blue normal weight

It's due to cases where the identity pop over is present and cases where it's renderd as simple link
Is the blue color app wide? Can I change the default color of the link to be blue?
Thank you

@daiagi ah, oki
the link isnt blue everywhere, see gallery item detail - creator and owner are bold black

the item count now is on weird spot due to removing the divider on all touch devices I think.

apologies, it slipped my vision. fixed

also on desktop the divider is missing now

sure? i could not reproduce it

Firefox

Brave

the creator name changes to black bold on mobile, it should remain blue normal weight

Fixed

And then we may delete buy now by default from collections? Thinking out loud. Wdyt @roiLeo

Sure, we should show all items on collection item page by default.

yes, that was wrong on my end then :D
last changes:

the whitespace
remove buy now by default from collection

thanks!

yes, that was wrong on my end then :D
last changes:
* the whitespace

* remove buy now by default from collection

thanks!

Done

weird animation on close sidebar filter

exists on beta as well, should solve in separate issue

explore layout spacing issue (compare it with beta)

indeed, apologies. fixed

if i remove the has-text-link class from the internal <a> tag the text is black

Can't reproduce it, we have same stuff going in GalleryItemDescription component so I want to unify how it looks

if i remove the has-text-link class from the internal <a> tag the text is black

Can't reproduce it, we have same stuff going in GalleryItemDescription component so I want to unify how it looks

toggle mobile device in dev tools and make sure it us "mobile" mode aka  touch enabled. refresh page

from IdentityLink.vue

maybe remove this !isMobileDevice  => render identityPopover on touch devices as well ? wtyt @roiLeo

toggle mobile device in dev tools and make sure it is in "mobile" mode aka touch enabled. refresh page

Aight, I can reproduce it with mobile mode.

from IdentityLink.vue

This is where issue come from, since IdentityLink provide a nested <a/> tag, IMO we should replace this component by something else since when you change a tag to div it work like a charm

maybe remove this !isMobileDevice => render identityPopover on touch devices as well ? wtyt @roiLeo

yep easy solution, but I think this has been introduce for a particular reason

Mind approving this PR and I'll open a new issue to deal with this?

What need to be improved:

profile link on mobile (IdentityLink component)
use NeoSidebar instead of Buefy Sidebar (MobileFilter component)
index folder in /components/explore ???
remove linkClass prop
remove class="mb-5" (total in ItemsGrid & Gallery component)
change position: relative with is-relative helper
v-if on ActiveCount usage

All done except:
Leftovers

[ ] profile link on mobile (IdentityLink component)
[ ] remove linkClass prop

@Jarsen136 @preschian  could i have your review please?


Update request: pstoedit 3.78 -> 4.01

Package name: pstoedit
Latest released version: 4.01

Current version on the unstable channel: 3.78
Current version on the stable/release channel: 3.78

[x] Checked the nixpkgs pull requests

Notify maintainers
@MarcWeber

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

ping @Artturin, due to last to commit
maintainer does not seem to be active


[request] Consider adding an option to ignore files / includes property of tsconfig

The Problem
swc-node respects the files and includes properties of tsconfig, which is normally sensible but interferes with monorepo workflows that depend on directly importing typescript files from sibling packages.
The Motivation
I've been using swc-node alongside a small HMR package I built for node as a lightning-fast alternative to ts-node-dev for the last couple months. It's made for a great development workflow, with the small exception that setup is a bit clunkier than it needs to be. Because @swc-node/register respects the files and includes properties of tsconfig, it refuses to transform TS files outside of the project directory when it encounters them. I found experimentally that removing includes and setting files to [] causes swc-node to default to a compile-everything state, which is perfect for my monorepo development flow.
Unfortunately, all of my other tooling requires those properties to be set in order to associate the tsconfig file to them.
Desired Behavior
A flag or alternate entrypoint (e.g. @swc-node/register-wildcard) to explicitly opt into the "compile every TS file I come across" behavior that automatically applies when files is set to an empty array.
Existing Workaround
Create a tsconfig.swc.json which overwrites files and invoke node via SWC_NODE_PROJECT='./tsconfig.swc.json' node -r @swc-node/register ./src/index.ts:
// tsconfig.swc.json
{
  "$schema": "https://json.schemastore.org/tsconfig",
  "extends": "./tsconfig.json",
  "files": []
}

If you believe this feature makes sense as part of @swc-node, I'd be happy to get a PR together for you.

This would be lovely. @joshuafcole did you find a decent alternative? Or fork this?

Just SWC_NODE_PROJECT= node arg1 arg2 (i.e. empty string)
SWC will ignore default tsconfig.json and run everything just fine.


Month name is in Latin

I can't seem to be able to change the month name from Farvardin to فروردین.

It shows me completely in Gregorian calendar!!
How could you config it for jalali?

please provide steps to reproduce this case (code/snippet).

please check this
https://github.com/Omid1285/react-native-calendars-persian

hi
how to rtl week?

hello
i have added a rtl (right to left) prop.
please check this
https://github.com/ahmadi-akbar/react-native-calendars-persian

this solution is not best practice but work fine
add this line in file node_modules/react-native-calendars-persian/src/persian/dateutils.js
moment.loadPersian({dialect: 'persian-modern'})

Thank you  barbodco it work's properly!!!

Welcome!

Thank you @barbodco it work's properly!!!

Welcome!

Thank you very much @barbodco


edgedb.errors.ClientConnectionFailedError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:1129)

EdgeDB Version: image: edgedb/edgedb
EdgeDB CLI Version:
OS Version: windows 10 docker

using:
docker compose file
basic.py
Steps to Reproduce:
1.docker compose up -d
2.python -m basic
docker compose file:

edgedb docker container logs before connecting to edgedb

snippet in basic.py

edgedb docker container logs after connecting to edgedb

after connect to edgedb using basic.py

related issue:
https://github.com/edgedb/edgedb-python/issues/223
just can't figure out the problem related to which,  edgedb, container, python client, setting or code

Thanks for your reply. after further dig into the doc, I think I need more time to grasp all stuffs of edgedb. I found some doc contents and code examples need to be  updated. after some days, I will give you feedback.


feat: ring_exp tactic, subtraction in ring

Another complete rewrite. This implements one ring to rule them all: it incorporates both ring_exp and ring behaviors, now under the name ring. (ring_exp had some small (1.4x) performance issues that prevented it from being used by default; I'm hoping that those issues are fixed now, and we can revisit later if it becomes an issue.)

What about goals like
example {α} [CommSemiring α] (x y : α) : (x + y)^2 = x^2 + 2 • x * y + y^2 := by ring

which should also count as "in the language of commutative rings"?
Presumably it is easier to first simp away all the •s, and then proceed as is, rather than adding an evalSmul branch, etc.

bors d+

bors merge


Metrics API Improvent

I would like to get the average metric value of past 5 minutes.  However vespa does not support it.
Could you add SUM values to metrics api?
I am planning to calculate metric value of past 5 minutes from the SUM value by using prometheus.
Thanks,

you don't need any special vespa support for this, as long as you sample the vespa metrics every minute and insert it into prometheus you can do the averaging there (just use the standard prometheus sum/count expression from the prometheus documentation).
note that this is exactly how yahoo's internal metrics monitoring / graphing system does it also, so this is the use case that current vespa setup is intended for, and should work very well.

I think it works fine when LAST metrics is supported.However, some vespa metrics do not have LAST but only AVG metrics. For example search_connections does not support it.
Can we add SUM metrics to they?

can't you do the sum of averages?  from the prometheus docs it looks like you want to use their 'summaries' concept to do the summing for you:
https://prometheus.io/docs/practices/histograms/
you shouldn't do summing or averaging of the LAST metric, that will sometimes yield incorrect results, since it will repeat the last value seen when there's no new measurements in a period.

rate() can calculates the per-second average rate of increase of the time series:
https://prometheus.io/docs/prometheus/latest/querying/functions/#rate()

It seems I misunderstood the specification of metrics API of Vespa. Sorry for confusion. Now I understood the correct specification, and I will try to calculate what I need using "rate" and "average" metrics currently available.
Thank you for your explanation.


UI problem

Hi
I have a problem with your project. Each time i play a  music and scroll up and down other progress bar will moving, and also UI will not response after click play button and just before music start to play.

Hey, @erfanegtfi thank you for reporting this. What is the context? Is it a recyclerview or something else?

it's recyclerview, actually I test it in your sample project

Same problem, any solution? also, whats about seekbar is too slow. any way to move more faster?

Same problem, any solution?


Rewrite the repository README.md

Is your feature request related to a problem? Please describe.
Now that the project is really close to release, we should should update the readme.
Describe the solution you'd like
The README.md should makes people easily:

understand the value of Tektasker
get a grasp of the user journey (maybe a short introduction video / GIF would help?)
install and use tektasker
list features
see limitations
go deeper

Solved by #32


[New Function] Add Reset into kvstore, enable set_params() to reset weights in kvstore

I think this is a good enhancement to set_params() when using kvstore.
for details, please see #6005.
@piiswrong @tqchen

I have a new more general idea to solve the part of big weight update problem, that's using compress algorithm to compress the transfer data in the slicer_ of worker's Send.

compression is not a good idea, but I just got a better idea to do partial weight update. It will change a lot code of kvstore related codes. 😄

oh ...I have been finished the code in the ps-lite and kvstore. But there is some bugs. How can I debug easily a distributed system....
@mli @piiswrong

haha, I fixed the bugs. Now my distribution training system (based on mxnet0.8.0) can support partial weight update now!
@mli @piiswrong

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


This is (propably) no longer maintained.

You will probably love using Five Server.
To install, open up VS Code (or Codium), hit CTRL+SHIFT+P and run https://marketplace.visualstudio.com/items?itemName=yandeu.five-server
I think that this is not maintained because the last commit was on 2019, and no response to issues or PRs.

Disclaimer: I did not receive any compensation for creating this issue. I just thought Some of y'all might need this.

Thank you!


Create items per page selector component

Create an "items per page" select component to permit users to define the "limit" or max number of items to retrieve for a single page on the grid.
This setting should be persisted on the client such that when the user visits other tags, the limit is consistent.

Work

Create the component
Place the component above the grid.
When selection changes, update this in the URL with, e.g., limit=20 in query string, and then refetch the data, staying on the current page (keeping the same after cursor).
Make sure this limit is respected by server side rendering, too.

@rymorgan what options should we have (20, 40, 50?) for the number of products to show?

The options should be 20, 60, 100.
If possible, in scenarios where the grid flexes to be 3 products wide(vertical tablet) instead of 4, it should change to numbers divisible by 3 so, 21, 60, and 102.

@cassytaylor will follow up with dropdown styles for the product count selector.

Here's the style for the open dropdown button component - http://zpl.io/aRql70p
The default width of the button is based on the width of the default option, + 20px of padding on each side. In some cases, an option in the dropdown might exceed the length of the button; in that case, the button would expand when clicked to the width of the longest text string (+ padding). Here's an example - http://zpl.io/adMlGw5