Separate Load Functions for Server and Browser

Describe the problem
Right now there is not a way to separate them.  We can run the same code in browser-only, server-only, or both, but there is no way to run SEPARATE code in both server-only, browser-only versions.
Most database fetches cannot be run in the browser. This can be fixed by using just +page.server.ts, however, this forces a new database call from the server on each route reload.  What if I want DIFFERENT code depending on if it is a server or browser call? This is currently impossible. You can't use the if (browser) option, as it will load the same code and packages for the server, which you don't need or can't use in the browser.
The only work-around is to completely move your server load function code outside of your load function, and create and endpoint.  That way you can do: if (!browser), fetch data from the endpoint.  This is extremely problematic, as you are forced to create an endpoint for something that is already loading on the server.  This enables access to your data from an endpoint, which you don't want.  Plus, with endpoints, you lose type-safety like you have with PageData or $page.
Describe the proposed solution
The most logical solution which would allow backwards compatibility is to allow BOTH the +page.server.ts file AND the +page.ts files. If it is the initial load, it will run +page.server.ts, and every call to the component afterwards will run the +page.ts file.
Alternatives considered

Another option maybe a +page.browser.ts file.
You could also add a export const separate = true
Allow two load functions in a +page.ts file (loadBrowser, loadServer)

Importance
would make my life easier
Additional Information
I can use SvelteKit without it, but I cannot use it correctly without an extraneous endpoint.
A good example of this is a feed. When I click on the item on the feed, I should not have to refetch that item if it is already in memory. There is no way to do this without an extraneous endpoint if I can only fetch the data on the server.  With Firebase, for example, this could also potentially add extraneous read costs.

The most logical solution which would allow backwards compatibility is to allow BOTH the +page.server.ts file AND the +page.ts files. If it is the initial load, it will run +page.server.ts, and every call to the component afterwards will run the +page.ts file.

This would be breaking. You can already have both load functions; they just both run.

They are both executed. You can use the event.data in your shared load function on the server and execute different code when running in the browser
haven't tested this, writing from my phone
export const load = (event) => {
  if (!browser) return event.data;

  return { custom: 1234 }
}

@tcc-sejohnson - Didn't realize that, good to know. However, this doesn't solve the problem of running them separately instead of consecutively.
@david-plugge - Not sure what you mean. I can check for the browser on the +page.ts, but browser will always return false on the server.  If you can think of any work-arounds, I would love to test them!
Perhaps then the best feature request would be:
export const load_once = true for +page.server.ts.  That way it will always only load once when the app is loaded...
J

I'm a bit confused about what you're trying to accomplish. There's a very high chance it's already possible. Can you describe the actual use case a bit more?

@david-plugge - So it seems I can skip the load function on +page.server.ts if I use this code...
export const load = (async ({ params, request }) => {

  // don't run after loaded once
  if (request.url.includes('__data.json')) {
    return {
        ...
    };
  }

  // otherwise fetch data from database...
  // return fetched data...

}) satisfies PageServerLoad;

However, this won't help as the +page.ts seems to always overwrite the +page.server.ts return data in $page or PageData.

@tcc-sejohnson - Yes, I want to run two different load functions: one from the server on first load, and another from the browser on every other load. This is how the load function works now if you use only +page.ts, but unfortunately it shares the same code base. This often doesn't make sense since you often can't share the same code in the browser as you can in the server (getting data from prisma, mongoose, etc). The work around is to just always load data on the server with +page.server.ts, but this is only one use case and not always efficent.
Here one example with Posts:

The user goes to the home page at https://mysite.com
The routes/+page.server.ts loads some data from a getAllPosts() function
The home page routes/+page.svelte is loaded and displays all post (post-list) keeping them in memory (perhaps a store)
The user clicks on a single posts, which navigates to https://mysite.com/posts/some-page
The page routes/posts/[slug]/+page.server.ts loads the post detail data again, even though it was already in memory
The page routes/posts/[slug]/+page.svelte is displayed with post detail data

As you can see here, the app is over fetching, or basically violating the n+1 rule put in other terms.  For a database that charges on reads for example, this can be a big deal.
Here is how I should be able to do it:

If a user goes directly to https://mysite.com/posts/some-page, it loads the +page.server.ts file as expected. However...
If a user navigates to the same page after clicking on a post-list page where the data is already in memory, it doesn't need to reload the data...

That is my specific problem. There is a workaround with an endpoint, but that creates additional developer overhead and an endpoint I don't need.
However, there should be a way in SvelteKit to have two completely different load functions depending on if the request is from the browser or the server (instead of sharing a load function).
J

Yeah, all of these needs are taken care of one way or another:

Server load functions have an isDataRequest parameter that you can use to detect if the request is the result of a client-side nav (don't check the URL like you currently are)
If you have both a server load and a client load, you need to merge the responses from them:

// +page.server.ts
export function load() {
  return { foo: 'foo' }
}

// +page.ts
export async function load({ parent }) {
  const { foo } = await parent()
  return { bar: 'bar', foo }
}

If a user goes directly to https://mysite.com/posts/some-page, it loads the +page.server.ts file as expected. However...
If a user navigates to the same page after clicking on a post-list page where the data is already in memory, it doesn't need to reload the data...

The above should get you as far as you need, but architecturally, you should probably put these loads in a +layout.server.ts and a +layout.ts file that wraps the whole system.

Ok, so you definitely got me where I needed to be after changing your code:
+page.server.ts
export const load = (async ({ isDataRequest }) => {

  if (isDataRequest) {
    return;
  }

  // fetch data
  ...
  return {
    post: data
  };
}) satisfies PageServerLoad;

+page.ts
export const load = (async ({ data }) => {

  if (browser && !data.post) {
    return {
      post: get_data_from_store()
  };
  return data;

}) satisfies PageLoad;

So, for future references, this particular problem has the above work-around solution. That being said, I still think it would be better if they were separate functions entirely.
That being said, I digress now that I found a new issue... what if the post is not in memory because the user did not come from a post-list page, but a link from somewhere else on the site? No way to condition the +page.server.ts file to run, or send data to it? But this is a different issue, so I will post elsewhere for that.

The above should get you as far as you need, but architecturally, you should probably put these loads in a +layout.server.ts and a +layout.ts file that wraps the whole system.

Still same issue, just layout files, but I get your point.
J

Still the same issue, just layout files

No, actually! The layout load shouldn't rerun unless a dependency changes. So if you put your [slug] route's load in a +page.ts but put the posts load in a layout, as long as you don't access the url from the layout load, it won't rerun on navigations between pages "under" it.

Ah, yes very true. I don't think that helps me in this particular situation, as I need to not get the slug on route change sometimes, but good to know, thanks!

For clarity, https://github.com/sveltejs/kit/issues/10004#issuecomment-1556293080 isn't quite right — if you have +page.server.ts and +page.ts together they should be combined like so:
// +page.server.ts
export function load() {
  return { foo: 'foo' }
}

// +page.ts
-export async function load({ parent }) {
-  const { foo } = await parent()
+export async function load({ data }) {
+  const { foo } = data
  return { bar: 'bar', foo }
}

The canonical solution here is to use an API route. On the server, event.fetch('/my-api-route') doesn't result in a network request; SvelteKit is smart enough to treat it as a regular function call, so the only overhead is converting to and from a Response. Doing it this way means you can make requests for data outside load functions, which is useful for e.g. infinite pagination — see e.g. this server load function that requests data from this API route, which also serves requests inside this event handler.
So it seems to me that the two concerns are

you don't want to explicitly create a new route (stress 'explicitly' — an endpoint is going to be created whatever happens)
you don't want to use type safety, but fetch is inherently un-type-safe

There's an idea here for typed API routes, but it sounds like you really want something like tRPC (via e.g. trpc-sveltekit) or telefunc. So to me the question is whether it's enough that the problem is solved in userland (this seems pretty nice, honestly) or whether the framework needs to introduce new primitives.

SvelteKit is smart enough to treat it as a regular function call, so the only overhead is converting to and from a Response

Very very interesting and good to know! In the end, I needed an endpoint to cover my needs. I don't want the extra overhead or configuration of TRPC when SvelteKit works just fine. But you're right, endpoint is what I needed.
Thanks!
J

Ok, so you definitely got me where I needed to be after changing your code:
+page.server.ts
export const load = (async ({ isDataRequest }) => {

  if (isDataRequest) {
    return;
  }

  // fetch data
  ...
  return {
    post: data
  };
}) satisfies PageServerLoad;

+page.ts
export const load = (async ({ data }) => {

  if (browser && !data.post) {
    return {
      post: get_data_from_store()
  };
  return data;

}) satisfies PageLoad;

So, for future references, this particular problem has the above work-around solution. That being said, I still think it would be better if they were separate functions entirely.
I also digress now that I found a new issue... what if the post is not in memory because the user did not come from a post-list page, but a link from somewhere else on the site? No way to condition the +page.server.ts file to run, or send data to it? But this is a different issue, so I will post elsewhere for that.

The above should get you as far as you need, but architecturally, you should probably put these loads in a +layout.server.ts and a +layout.ts file that wraps the whole system.

Still same issue, just layout files, but I get your point.
J

This way of coalescing data from server load and client load is nowhere to be found in the SK documentation. If found this issue after hours of googling... Thanks a lot!


Save inputs on change (not just blur)

Description
Until now, changes in InputComponent, TextAreaComponent and AddressComponent were only saved server-side when the user defocuses/blurs the input field. This also meant that validation runs at the point when the user finishes filling out a certain field in the form. This change adds a hook to automatically save data (and validate) ~400ms after the last of (usually multiple) change events, effectively saving and validating after the user has stopped typing for a short while.
As the 'required'/empty field validation can get quite annoying if it pops up while you're focusing on a field and thinking of what to type, this specific validation has been postponed to run at the point when full page validation runs. Other validations will run as normal when the user stops typing.
Related Issue(s)

#98
#155

Reading material regarding debounce timeout:

https://ux.stackexchange.com/questions/95336/how-long-should-the-debounce-timeout-be
https://lawsofux.com/doherty-threshold/

Verification

[x] Your code builds clean without any errors or warnings
[x] Manual testing done (required)
[x] Relevant automated test added (if you find this hard, leave it and we'll help out)
[x] All tests run green
[x] I have searched for other relevant issues regarding onBlur and closed/marked as duplicate/mentioned this change

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Presented a demo of this on friday, and the most important item of feedback received was that it should be possible to disable this altogether (or possibly adjust the debounce timeout) per-component. Will look into that before opening for review.

@nkylstad Thanks for the review! Could you have a look at the cdn and docs PRs as well? They should be merged when this is released, at least.


[bug] Context not showing up in results with full path formatted mentions

For some of my notes, just the header, and not the lines containing the link, show up on the list.
By way of example: this is the file "Clocks Notes MOC":

And this is the beginning of "2022-10-24", which you'll note is the first backlink in the influx list:

All of my daily notes are formatted basically like that, and none of them show content. As far as I can tell, the only notes that actually show content are ones wherein I don't use any bullet points at all.
So: is this a bug? Or is this, as I suspect, something to do with the way my daily notes are formatted? If the latter, I recommend describing what that is in the README, because I cannot see any significant difference between what I'm doing and what's in your screenshot.
(Edit: Github automatically added the bug tag based on my original title, sorry about that)

That's curious. I'm not sure what causes this bug, maybe it could be something with the formatting of mentions. Could you post the raw markdown?

here's today's daily note, which exhibits the same problem. I suspect everything after the timeline header is irrelevant but i include it just in case:
---
tag: #daily #log
---

# Tue 2022-10-25
---
## Notes
### timeline

- 13:18
	- I am mentioning [[../02 OUTPUT/CLOCKS split/Notes/Clocks Notes MOC]] so I can send some test markdown for debugging influx.
## Navigation  %% fold %%

- [[00. Homepage]] - [[000 Index]] - [[005 Active MOC]] - [[TASKS searches]] - [[Dashboard]]
- [[stats Kanban]] - [[../../03 daily/Week Kanban|Week Kanban]] -  [[Subs Kanban]] - [[Writing Tracker]]

[[2022_Calendar |2022]] » [[2022-10 | October]]  » [[2022-W44 | Week 44]]
[[2022-10-24 | ⇦ Previous Day]] | [[2022-10-26 | Next Day ⇨]]

%%
clocks::
fiction::
writing::
phys::
%%

## Modified %% fold %%
---
```dataview

TABLE file.cday as "Created On", file.mday as "Last Modified"
FROM -"02 OUTPUT/CLOCKS Scrivener sync"
WHERE file.mday = date(2022-10-25)

SORT file.name ASC

[[2022-10-24]]

Based on your changed title, I did a brief test, and, yes, I think you're right that the fully formed path is the problem. If I remove the path from a mention it seems to show up.


Update SDXL.py

added style and refiner params

Some conflicts. Made you the admin. You can resolve and merge PR.


Rendering loop goes occasionally stale

Seen once on Windows, rendering loop went stale at app startup: canvas was updating only when kicked by gestures. Not sure if this bug will remain in the next release as the rendering loop for SkiaSharp v3 is already different.
Anyway will leave this as a note to track and watch..

With the implementation of a display-synched constantly ticking loop this issue is now gone.


Build and deploy docker image to docker hub

Build an image and deploy it to docker hub.

Available here : ghcr.io/devmachine-fr/cert-manager-alidns-webhook/cert-manager-alidns-webhook


Add support for dynamic scoring on challenges

see this for reference: https://github.com/sdslabs/backdoor/pull/76/files

Resolved


build(deps): summary of dependabot updates

CVE is in base-image distroless, we'll have to wait for an update on that.
https://github.com/GoogleContainerTools/distroless


v1.7.0 error  recipe for target 'EosioClang' failed

Encounter error when make j8
Error copying file "/root/eosio.cdt/build/eosio_llvm/bin/clang-7" to "/root/eosio.cdt/build/bin/".
CMakeFiles/EosioClang.dir/build.make:130: recipe for target 'EosioClang' failed
make[2]: *** [EosioClang] Error 1
CMakeFiles/Makefile2:296: recipe for target 'CMakeFiles/EosioClang.dir/all' failed
make[1]: *** [CMakeFiles/EosioClang.dir/all] Error 2
Makefile:140: recipe for target 'all' failed
make: *** [all] Error 2

I'm also seeing this.

Can you please post the complete steps you carried out that produced this error (including clone [commit hash or tag], etc.)?

I gave up and installed with Ubuntu. I’m on macOS latest trying to compile
from source.
On Wed, Aug 18, 2021 at 1:22 PM sanaraufx @.***> wrote:

Can you please post the complete steps you carried out that produced this
error (including clone [commit hash or tag], etc.)?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/EOSIO/eosio.cdt/issues/1188#issuecomment-901291974,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIGVTNAGESWSON72JAPQHDDT5PT45ANCNFSM5BZZAJOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
Natanaël Prudhomme

Did it work when you tried with macOS?

I suggest first check if eosio.cdt/build/eosio_llvm/bin/clang-7 exists or not, esp. if there are no building errors before "Error copying file". Sometimes it's because git submodules (e.g. eosio-llvm) was not updated correctly, and a different version of clang (e.g. clang-9) was built.
Below are the steps I just tried and it builds v1.7.0 correctly
git checkout tags/v1.7.0 -b v1.7.0
git submodule update --init --recursive
mkdir build && cd build && cmake .. && make -j10


[Repo Rearchitecture] Use DevEx service to generate ID and write back to quickstart config

Description
New quickstarts are added by pinging the GitHub API in an interval and storing the results in memory. Because we no longer want to use the open install library as the data source, we need to ensure new/updated quickstarts are written to the catalog service on release.
NOTE: This will not replace the current process yet, but will rather work in parallel with the new system so new workflows should be created for this work
Job-generate-release

Jobs/Scripts we will need to change to use NerdGraph

Generate-uuid (BLOCKED by DevEx WIP - separate ticket to be created) - The new service will generate the ID if this is a new quickstart
This will then need to be written back to the quickstarts config.yml

A/C

[ ] Write the newly generated quickstart ID from DevEx service back to the config.yml

Helpful Links
Schema - https://docs.google.com/spreadsheets/d/1EmAvq4kJUUPlpvJZ-s23uKDk_KCws9VgHzG1bJtwE4Y/edit#gid=0
NerdGraph - https://api.newrelic.com/graphiql?#query=mutation {
  nr1CatalogSubmitMetadata
}

DevEx Service - https://source.datanerd.us/nr1-dev-experience/catalog-service-elixir

This is currently blocked by Virtuoso's planned work to add a dryRun argument to queries in order to validate quickstarts

I've added info for our 3 different endpoints so this may need to be repointed

This is no longer required as we are using a single endpoint submitQuickstart for both creating and updating now


Support for 3rd party skins

It would be nice if the endpoints accepted direct urls to a skin file beside a player name/uuid.
I guess it wouldn't consume more server resource because the raw skin has to be downloaded anyway, from a mojang server or from the url parameter.
Example:
https://crafatar.com/renders/body/https://i.imgur.com/E9B0424.png

Hi Chris, unfortunately, we don't directly support 3rd party skins, but you can use the default parameter. See here for more info on that. I see that you're using renders in this case, and currently, we only render default steve or alex images.

I think the amount people who would use this is very low, and I thus we won't be implementing this.
If you really need it, just fork it and mess around with the textures url.


Calculate links to from register

This PR adds functionality to calculate links between all registers, exposed through a new RegisterLinkService and makes this data available to HomePageView. Links between registers will be recalculated when the main register application starts, in addition to on demand through the /delete-register-data endpoint when register and field configurations are re-loaded.

I've checked out this branch and i'm having a look.

@johnollier In the same way that we access the RegistersConfiguration and FieldsConfiguration in the application, I thought it best to follow a similar here. We could pass in RegisterLinks to the constructor of PostgresRegister if you think this would be a better way of doing things?

@karlbaker02  Does the linked from list include curie links from other registers?

@robmckinnon No, curies are excluded for both the 'Links To' and 'Links From' sections.


Fix APIError and pyinquirer python3.10 error

Fixes #31
Fixes #24

Updated dependencies to fix

Traceback (most recent call last):
  File "/home/mandy/.local/bin/pydeezer", line 5, in <module>
    from pydeezer.cli import cli
  File "/home/mandy/.local/lib/python3.10/site-packages/pydeezer/cli.py", line 5, in <module>
    from PyInquirer import prompt
  File "/home/mandy/.local/lib/python3.10/site-packages/PyInquirer/__init__.py", line 6, in <module>
    from prompt_toolkit.token import Token
  File "/home/mandy/.local/lib/python3.10/site-packages/prompt_toolkit/__init__.py", line 16, in <module>
    from .interface import CommandLineInterface
  File "/home/mandy/.local/lib/python3.10/site-packages/prompt_toolkit/interface.py", line 19, in <module>
    from .application import Application, AbortAction
  File "/home/mandy/.local/lib/python3.10/site-packages/prompt_toolkit/application.py", line 8, in <module>
    from .key_binding.bindings.basic import load_basic_bindings
  File "/home/mandy/.local/lib/python3.10/site-packages/prompt_toolkit/key_binding/bindings/basic.py", line 9, in <module>
    from prompt_toolkit.renderer import HeightIsUnknownError
  File "/home/mandy/.local/lib/python3.10/site-packages/prompt_toolkit/renderer.py", line 11, in <module>
    from prompt_toolkit.styles import Style
  File "/home/mandy/.local/lib/python3.10/site-packages/prompt_toolkit/styles/__init__.py", line 8, in <module>
    from .from_dict import *
  File "/home/mandy/.local/lib/python3.10/site-packages/prompt_toolkit/styles/from_dict.py", line 9, in <module>
    from collections import Mapping
ImportError: cannot import name 'Mapping' from 'collections' (/usr/lib/python3.10/collections/__init__.py)

Fixes pydeezer error

Traceback (most recent call last):
  File "/home/codespace/.python/current/bin/pydeezer", line 5, in <module>
    from pydeezer.cli import cli
  File "/usr/local/python/3.10.4/lib/python3.10/site-packages/pydeezer/__init__.py", line 5, in <module>
    from .Deezer import Deezer
  File "/usr/local/python/3.10.4/lib/python3.10/site-packages/pydeezer/Deezer.py", line 6, in <module>
    from deezer.gw import APIError as GWAPIError
ImportError: cannot import name 'APIError' from 'deezer.gw' (/home/codespace/.local/lib/python3.10/site-packages/deezer/gw.py)

Added simple test to see if the generated whl can be run without errors

Traceback (most recent call last):
  File "/home/entrider/.local/bin/pydeezer", line 5, in <module>
    from pydeezer.cli import cli
  File "/home/entrider/.local/pipx/venvs/py-deezer/lib/python3.11/site-packages/pydeezer/cli.py", line 5, in <module>
    from PyInquirer import prompt
  File "/home/entrider/.local/pipx/venvs/py-deezer/lib/python3.11/site-packages/PyInquirer/__init__.py", line 6, in <module>
    from prompt_toolkit.token import Token
ImportError: cannot import name 'Token' from 'prompt_toolkit.token' (/home/entrider/.local/pipx/venvs/py-deezer/lib/python3.11/site-packages/prompt_toolkit/token.py)


Release Date

Hello Packtpub team and @carlomilanesi,
When is the book going to be released?
The website says April 17h, but the book isn’t available yet.
What is the plan for the release?
Thanks and have a good day!

@nathanqueija i will give the useful comment of "i am in the same boat waiting on release ("

@1init they just updated the website saying it will be released on September 11th 😢

Not sure that is a wise date to choose to release anything. wonder if 11/9 will the final date.

Hi. The books has been released, It is already available as an e-book.

The book is available also printed on paperback.

Oh yes! Gonna start reading it now. Thanks @carlomilanesi


Show message when server is offline

Detects a timeout after forwarding pings from client to server unsuccessfully, marking the server offline, logging a warning, and showing an entry in the LAN server list as "server offline".
Keeps forwarding the pings to the server even when it's offline so that when it comes back online, phantom detects it and shows the correct server in the list -- it's instantly updated.

Thanks, very cool 👍 tought too about deleting client, but i left it out.


I get error of this kind: https://hydra.nixos.org/build/215463210
The following installed extensions are outdated, because `atom' requires later versions:

  sxml-serializer (0.0.0 -> 0.2)

Have you pointer how to solve this ? Many package have the same kind of error.

Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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

[ ] Fits CONTRIBUTING.md.

cc @nagy

@ofborg build chickenPackages_5.chickenEggs.allegro

The first commit recude number of error so it can help another user to fix this package

Have you pointer how to solve this ? Many package have the same kind of error.

I have seen this before and I think it is because the package is pinning a specific version of an egg and we are always using the latest version. You could add -force to the chicken-install call but I have seen other packages break because of this. If you find some packages that build with that flag we could enable it optionally.
Can we rebase this on #225775 once that is merged because there are a lot of darwin mark-as-broken in there.

The first commit reduce number of error so it can help another user to fix this package

We don't have iup packaged in nixpkgs, anyway... Maybe the easiest way is to autoPatchElf on binaries in https://aur.archlinux.org/packages/iup-bin?

The first commit reduce number of error so it can help another user to fix this package

We don't have iup packaged in nixpkgs, anyway... Maybe the easiest way is to autoPatchElf on binaries in https://aur.archlinux.org/packages/iup-bin?

Source code is here https://www.tecgraf.puc-rio.br/iup/

Can we rebase this on https://github.com/NixOS/nixpkgs/pull/225775 once that is merged because there are a lot of darwin mark-as-broken in there.

Maybe remove allegro as broken in this other PR so we can merge without order.

Maybe remove allegro as broken in this other PR so we can merge without order.

I guess there'll still be a conflict, since broken = ... is just 2 lines away from allegro...

My commit seems cherry-picked in the other PR. Thanks for rebasing.


The ODM should the common proxy interface

This would make it compatible with libraries handling this proxy (for instance JMSSerializerBundle has some logic to handle doctrine proxies but won't catch the CouchDB ones currently) and will allow you to use Doctrine\Common\Debug::dump()

:+1:


Integration

This pull request adapts PAGE syntax to run with an in-progress new release of Mimi.  It will become relevant when this new version of Mimi is released, but should not be merged until then.

This PR also adapts PAGE's Monte Carlo simulation functionality to the new mcs framework.

NOTE:  the .yml file currently calls  - julia -e 'Pkg.checkout("Mimi", "release-0.5.1")' ... we can probably remove his line now that we have tagged this branch

Actually, before we merge this, we should try to get the CI tests to pass again.

Ok I'll take a look at those.

I think it should be enough to change the julia version to 1.0 in the travis file, and then change lines like - julia -e 'Pkg.add("Mimi")' to - julia -e 'using Pkg; Pkg.add("Mimi")'.

@davidanthoff that made Travis very angry ERROR: Unsatisfiable requirements detected for package Mimi [e4e893b0]: ...

:) it's the smoke.  Great, will do.

@davidanthoff build success, ready to merge!


Card#all and #find, VdcOperation and Refund (nested in Card) Bump version to 0.2.3

@myhalard

:shipit:


Error generating json from cli

I am getting this error no matter how I try to run the command:

Does not work with either single quotes or double quotes, or even if a remove the quotes.
I cloned the project and ran npm install.
Node Version: 10.16.0.
Thanks.

Hi @ccarrero301 , I suspect your Node version is too old (to my best knowledge, it is already out of support) and includes an outtdated fs module, with no support for fs.promises.
Can you please try with recent Node, e.g. v14.17 ?

@scale-tone Thanks! That was it. Thank you for the quick help!


Lock icon is slightly overlapping C in Commitment of About screen  - ios App v1.0.1

Version

Type of phone: iPhone
OS Version: ios 13.3
Model: iPhone XR

Steps to reproduce

open App
Tap on main menu option
open "About "
Screen displays content
on screen - Lock icon is slightly overlapping C in Commitment

Expected Behavior
Lock icon is does not overlap C in Commitment

Issue exists in Android 9 as well.

Assign this to me . i will take a look into this .

Assign this to me . i will take a look into this .
Great Thanks! Please coordinate with UI/UX people on Slack with your proposed solution to make sure it's not overlapping other work.

Here's the design reference file: [https://www.figma.com/file/aOz2hCg6agulId3mGCQIv8/Safe-Paths-Version-1.0.1?node-id=1243%3A3605]
For both titles on that screen:
line-height: 32px;
padding: 11px

Here's the design reference file:
https://www.figma.com/file/aOz2hCg6agulId3mGCQIv8/Safe-Paths-Version-1.0.1?node-id=1243%3A3605
For both titles on that screen:
line-height: 32px;
padding: 11px

FYI @driedgrape  - That file/link does not appear to have global view permissions


update generator-jhipster v8.0.0

The error is fixed:
Exception in thread "main" java.lang.NoSuchMethodError: 'org.springframework.aot.hint.support.FilePatternResourceHintsRegistrar$Builder org.springframework.aot.hint.support.FilePatternResourceHintsRegistrar.forClassPathLocations(java.util.List)'

jwt applications are failing now:
[0] Caused by: java.lang.ClassCastException: org.springframework.aop.framework.CglibAopProxy$SerializableNoOp cannot be cast to org.springframework.cglib.proxy.Dispatcher
[0] 	at com.mycompany.myapp.service.MailService$$SpringCGLIB$$0.setCallbacks(<generated>)
[0] 	at org.springframework.aop.framework.ObjenesisCglibAopProxy.createProxyClassAndInstance(ObjenesisCglibAopProxy.java:91)
[0] 	at org.springframework.aop.framework.CglibAopProxy.buildProxy(CglibAopProxy.java:213)

Merging since main is not passing anyway.

@mshima Thank you for the swift update to v8. I will also look into the error causes on my end.

[0] Caused by: java.lang.ClassCastException: org.springframework.aop.framework.CglibAopProxy$SerializableNoOp cannot be cast to org.springframework.cglib.proxy.Dispatcher
[0] 	at com.mycompany.myapp.service.MailService$$SpringCGLIB$$0.setCallbacks(<generated>)
[0] 	at org.springframework.aop.framework.ObjenesisCglibAopProxy.createProxyClassAndInstance(ObjenesisCglibAopProxy.java:91)
[0] 	at org.springframework.aop.framework.CglibAopProxy.buildProxy(CglibAopProxy.java:213)

@mshima
I came across a similar issue here:
https://github.com/spring-projects/spring-framework/issues/30985
I attempted to analyze it myself, but it has been challenging and time-consuming to resolve issues such as the unintended creation of two CGLIB classes( MailService$$SpringCGLIB$$0 and MailService$$SpringCGLIB$$1, with the latter being the correct one), where the behavior remains unclear.
I plan to modify the code to avoid using @Lazy in MailService.


docs: fixing syntax

line 151: withdraw - withdrawal
line 162: build -> built
line 162: dAPP -> dApps
line 166: unkonwn -> unknown - on Godwoken Bridge page -> on the Godwoken Bridge page -  this is because that your account is not registered on Godwoken network - this may be because your account is not registered on the Godwoken network. This can be fixed by depositing some CKB to the Godwoken network, then the issue should be resolved.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@aff6c5b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #163   +/-   ##
==========================================
  Coverage           ?   48.17%
==========================================
  Files              ?       29
  Lines              ?     1343
  Branches           ?      195
==========================================
  Hits               ?      647
  Misses             ?      515
  Partials           ?      181

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 aff6c5b...d86c664. Read the comment docs.


How to turn off zoom/movement inertia?

At now, when user zoom or drag move panorama image, inertia zoom or inertia move are working.
How to turn off this feature?

Ok I found. set friction to 1 in config object

Ok I found. set friction to 1 in config object


liquids and buckets dupe

Hello, here is video.
https://youtu.be/T7bECqkE9ag
[GP] Running GriefPrevention 1.12.2-<IP_ADDRESS>6
[GP] Running SpongeForge 1.12.2-2705-7.1.0-BETA-3374
[GP] Running luckperms 4.2.51

Deny interact-item-secondary and it shouldn't dupe. If it does, log a proper ticket in Sponge.


Sequence of questions

Did current framework, support user to ask sequence of questions?
like some kind of state management?
Or can I trigger another intent or story by api or set in UI?
Did current framework support some kind of logical handle?
Because we will have several questions with similar answer, eg. Yes or No. How can the bot determine, which answer is related which question?

As of now, there is no jumping between stories. But we do have support for maintaining states across stories through global context.
This project is just a basic platform with Natural language understanding and basic actions. We'd like to have logical branching kind of features. Please do contribute.

As of now, there is no jumping between stories. But we do have support for maintaining states across stories through global context.
This project is just a basic platform with Natural language understanding and basic actions. We'd like to have logical branching kind of features. Please do contribute.

and sequence of questions can be given as parameter prompts under a story.

How do the existing platforms perform maintaining states across stories through global context? Can you give me some examples?

Global context is possible if multiple stories/intents maintain uniform global entity names. You can have separate flags for identifying which entities goes into global context and which of them are to be updated over the time.
For example if username is already input by one intent, it can be stored in global context and can be accessed from other stories when ever required. It can be replaced at some other time also. Similarly favorite color, location etc can be stored in global context once input.

Hi Alfred,
How would one go about storing input in global context to make it accessible by other stories?
Is there a better way to ask multiple questions and have an api call for each of those questions?


deprecate kube-rbac-proxy from kueue repo

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
kube-rbac-proxy is being deprecated. Rather than rely on an image, we can remove.
Which issue(s) this PR fixes:

Fixes #3737
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/hold
Let me confirm that this works via a manual test and then we can merge.

oh @mbobrovskyi just opened this up also..
https://github.com/kubernetes-sigs/kueue/pull/3760
/close


P0476R2 Bit-casting object representations

https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2018/p0476r2.html

実装済み


one time jobs

Is it possible to add jobs that needs to be executed only once?
For example i need one job to be executed after 1 month.
Im using this library with persistance and it works well for cron like jobs, but i also have few commands that just needs to be executed once. One way would be to remove job once it is executed.
I know this isnt entirely scope of this library, but alternative would be to introduce some queue or something and it msot likely would mean some redis or rabbitmq, and it seems like unnceseary dependency. Would love to keep it in beam.
Maybe you could point me in right direction with alternative library with said functionality? Thanks.

I'm not sure if it's the quantum's purpose. Maybe you could try something like Oban for that.


update api sha

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: quanjielin
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@quanjielin: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/assign @geeknoid for owner approval

@quanjielin: GitHub didn't allow me to assign the following users: approval, for, owner.
Note that only istio members and repo collaborators can be assigned.
For more information please see the contributor guide

In response to this:

/assign @geeknoid for owner approval

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: quanjielin, wattli
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, quanjielin, wattli
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest

/cc @utka

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-trustdomain

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-trustdomain

prow/istio-integ-k8s-tests.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-trustdomain

prow/istio-integ-k8s-tests.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-mixer-no_auth

prow/e2e-bookInfoTests-trustdomain.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test e2e-bookInfoTests-trustdomain

prow/istio-integ-k8s-tests.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
00ae72de8cc7a47209ca9430ec729bf599a79ac8
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
819b44c4c5601a1428684b3888de6d29e467d081
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Adding Extra Question Regarding Machine Learning and Deep Learning

Supervised Learning:
Unsupervised Learning:
Deep Learning:
Evaluation and Metrics:
I have Some of the question under these topic ...
So Assign this issue to me under Hacktoberfest'23

Hi @klaus78! I have few questions and would like to contribute. Please assign this issue under Hacktoberfest'23. Thanks

Hi @jahnavigundakaram you are welcome to contribute

Hi @klaus78! I have a few questions and would like to make a contribution. Please label this issue for Hacktoberfest'23. Thank you!

@klaus78   kindly check it sir #18 and merge it

@klaus78 ?

Hi @Kapileswar-Moharana going to check it very soon thanks

Hi @Kapileswar-Moharana going to check it very soon thanks

@klaus78 ok sir check it if any changes you want then let me know I will fix it


examples: Add missing defer call

What do you think about making the TraceCollection.Close() close all the tracers? If you agree, I will create a simple issue for it.


feat: add field variants to receive data about video(s) when media type is video

An example can be obtained from this tweet ID:<PHONE_NUMBER>718926338;
We are using this struct when do recent search API;

Hi @g8rswimmer. Can you review, please?


Understanding the difference between CLS features vs PATCH features.

Hi, first of all, thanks for the great work with DinoV2.
Imagine that I want to find dogs (and their positions) among several images in my dataset.

I have DINOv2 CLS features obtained from an image of a dog
I have several DINOv2 patch features for each image in my dataset.

I can confirm that I'm able to find images with dogs in my dataset by calculating a similarity score (e.g. dot product) between CLS feature of the dog image and patch features for each image in dataset. It did work.
What I'm trying to find out is if this result is just a coincidence or if it is intentional for DinoV2. I've skimmed the paper and couldn't find the answer.
Thank you.

Coincidence I would say.

I've seen this results happening while using different classes of images (animals, objects, plants). Maybe this result has emerged unintentionally?

That's very much a possiblity; at no point we expect the CLS and patch tokens to align though !

Hi @barbolo
i'm trying to understand the difference between cls token and patch features. Can you please point me to some materials? I know that cls tokens are used as embedding for classification for example, but patch features I don't know what can be used for?
Thanks

Hi @barbolo,
ok thanks. I understand now.


Unable to install 0.5 alongside commerceguys/addressing 0.5

This package requires the dev-master version of commerceguys/addressing - I therefore cannot install version 0.5 of that package:

Can the composer.json requirement be changed to ~0.5 (for stable releases) or * (any version)?
See also: https://github.com/commerceguys/pricing/issues/10

Looks like the commerceguys/zone dependency might have the same issue too

Thanks.


Icons not rendering properly in tmux with alacritty

Instead of drawing the proper symbols, this is happening:

Here is my tmux.conf: https://github.com/brunobmello25/dotfiles/blob/main/tmux/.tmux.conf
My alacritty.yml: https://github.com/brunobmello25/dotfiles/blob/main/alacritty/.config/alacritty/alacritty.yml
And I have the Fira Code font installed on my system:

I also have already tried with another font like jetbrains mono.
 Turns out there is a firacode nerd font version hehe
Thanks guys! I'll close this issue now. Sorry


Event sourcing/storing with SAGA

(please correct me if i am wrong or something is missing)
Event sourcing helps in storing all the changes (events) to the system, rather than just its current/latest  state, means we can persist different version(change history) for for any domain object.
So we can see the audit trail and or back track(roll back) to an early stage.
Questions:
(a)
So in future do you have any plan to use SAGA to demonstrate the roll back among multiple micro-services?
(b)
Kindly also explain the realtime usage of IsReplaying property
    public class WorkshopPlanning
    {
        /// <summary>
        /// Indication whether the aggregate is replaying events (true) or not (false).
        /// </summary>
        private bool IsReplaying { get; set; } = false;

and of course thanks again for your wonderful help and support - I am highly obliged and grateful to you!
Originally posted by @findkashif in https://github.com/EdwinVW/pitstop/issues/42#issuecomment-538516625

(a)
I'm not sure I get your saga question. I know sagas as a way to orchestrate a process that spans multiple microservices. I would not use the eventstore to do rollbacks. Only "roll-forwards". If you need to fix anything in an event-sourced service, you add events to correct things (like an accountants ledger). Such a correction could be part of a saga (as a compensating action).
(b)
The IsReplaying indication is set when we are replaying events from the event-store to rebuild the state of the aggregate (using the set of events received from the database and passed to the constructor). This offers us the opportunity to change behavior in the event-handlers when replaying. I have not used this in Pitstop, but this is standard pattern I always use. I have used it once in a different situation to disable logging of event-handling during replay.
Again, I hope I've answered your questions  :)

Thanks again for detailed answer
The Saga Pattern is as microservices architectural pattern to implement a transaction that spans multiple services.
https://dzone.com/articles/microservices-using-saga-pattern
So basically this pitstop project really great place where you have implemented most of the aspect of DDD with a very pragmatic approach.
So i was looking for, whether you have any future plan to demonstrate enabling Transaction among multiple microservices.

What kind of transactions would you like to see? I guess (and hope) you're not referring to ACID transactions over services (this is usually a very bad (expensive) idea).
I always use the eventual consistency model over services using events. I have no compensating actions in Pitstop, but this is perfectly doable with the current setup. A service could emit an event that would trigger a compensating action in another service.
But I always strive to keep transactions within the boundaries of a service. If this is not possible and you need multi-service transactions, you should look at your service boundaries. Perhaps the services shouldn't have been split then (DDD bounded contexts).

Yes I was  not talking for ACID transaction and i was referring something like
https://dzone.com/articles/distributed-sagas-for-microservices
Here it has given very common and famous example of flight, car and hotel booking example.
and mentioned clearly:
"With the microservices approach, you can’t just book a flight, a car, and a hotel in a single ACID transaction. To do it consistently, you would be required to create a distributed transaction."
....and yes I also see/agree it is doable with current architecture setup with Pitstop, just wanted to check with you whether do you have any future plan to find-out & implement any such example for demonstration purpose.

Yes, those are the canonical examples. But no, I have no plans to add something like that to Pitstop.


Refine browser to be a UiPart

We can get rid of BrowserManager and make Browser a UiPart.

@yl-coder can you review this? It seems to be working, but the layout is messed up. Can you fix?

Looks good. fixed

Thanks @yl-coder 👍


Declaration of Brightspot\Nova\Tools\DetachedActions\DetachedAction::jsonSerialize() must be compatible with Laravel\Nova\Actions\Action::jsonSerialize(): array

I have received the following error after installing this through Composer in my Laravel Nova app and attaching it to a Contact resource model as per instructions:
Declaration of Brightspot\Nova\Tools\DetachedActions\DetachedAction::jsonSerialize() must be compatible with Laravel\Nova\Actions\Action::jsonSerialize(): array
Please provide any input to fix this. Thank you!

I can confirm modifying Ln 115 in /vendor/gobrightspot/nova-detached-action/src/DetachedAction.php
from
public function jsonSerialize()
to
public function jsonSerialize() : array
has fixed this issue for me and I suggest an update to the source code to refelct this as I suspect this is incompatible with the latest version of Laravel Nova as it stands now.
Thank you!


api: Hooks for - Item Hover On/Off

Set up hooks for hovering on and off items. Pass an args object with at least { item: Item5e, event: Event }.
Stretch
Consider also dogfooding these hooks with the Item Info Card feature, rather than relying directly on mouse enter/leave within Tidy.
Reference
thatlonelybugbear — Today at 8:49 AM
Feature idea/request
async hooks call when one hovers on and off Items, passing at least the Item5e and event (maybe).
A relevant example of what could be done with such hooks https://github.com/illandril/FoundryVTT-npc-quick-actions/issues/17#issue-2023890200 (sorry for the external link)

Copy homework from the PR https://github.com/illandril/FoundryVTT-npc-quick-actions/commit/1bdababf8b36f5c4257c545b78c24473a5db0d04

Hooks added in next release:
Hooks.on('tidy5e-sheet.itemHoverOn', (event, item) => {
  console.log('item hover ON for ' + item.name, { event, item });
});
Hooks.on('tidy5e-sheet.itemHoverOff', (event, item) => {
  console.log('item hover OFF for ' + item.name, { event, item });
});

Note: the event is not always mouseenter / mouseleave. The hover OFF event also occurs when a user starts dragging, because mouseleave doesn't always trigger when a user is dragging from an item table row.
This behavior of triggering hover-off on drag is also required for Tidy 5e Sheets to dismiss the item info card during drag/drop scenarios.

Closed as completed.


Caret not showing on MRT Keyboard Input Field

Overview
The 'KeyboardTest' scene inside any Examples package has everything already set up. Otherwise, create a new scene with MixedRealityCameraParent prefab, DefaultCursor, Input Manager prefab, MixedRealityToolkit Keyboard prefab, a main canvas with a child InputField which script has been replaced by a KeyboardInputField component and which 'text' property has been filled with some short single line text.
Expected Behavior
On Hololens, when I click/tap on the GameObject containing the KeyboardInputField, the MixedRealityToolkit keyboard appears and the caret on the inputfield of the keyboard should be visible and blanking.
Actual Behavior
The MixedRealityToolkit keyboard appears but the caret of the kayboard input field does not (on Hololens).
Steps to reproduce
Uuse this repo
Otherwise, create a new project, switch to Universal Windows Platform, create a new scene, import an MixedRealityToolkit >= 2<IP_ADDRESS>, import the relative 'Example' unity package (found on the releases page), open the KeyboardTest scene, search for the email input field and fill the 'text' property of its 'KeyboardInputField' with some short text (e.g., an email address),  from MixedRealityToolkit menu, select Configure -> Apply MR Project Settings -> Apply, add the scene to the built scenes, build and deploy. On Hololens, tap the GameObject containing the KeyboardInputField.
Unity Editor Version
2017.2.1p2
Mixed Reality Toolkit Release Version
2<IP_ADDRESS> Release Candidate 2 but the problem is in every MRT version starting from MixedRealityToolkit for Unity 2017.2.1f1 (included). MixedRealityToolkit for Unity 2017.2.0p1 MRTP4 and previous are not affected by this problem.

Re-enabling the caret causes issue #1940 to repro. I am investigating a fix/workaround to that issue that will accompany the restoration of the caret.

This was released in the 2<IP_ADDRESS> release candidate


suggestion: context object for layerlist and viewer to be moved to app model

currently, the layerlist and the viewer have a _ctx ... but I think it might be better suited on the app itself, which will listen for change events of those objects to update it's context?

welp i just accidentally clicked somewhere on the page and it made this sooooo

For context, this was created from the TODO list for app menu migration. Supposedly it should be tracked as a draft issue to help planning but was mistakenly created. If not too misleading or distracting, I will leave this open and get to this while we march on the app model migration

Currently app-model does not have a registry for context, so this is not possible but it is on to-do the list for app-model: https://github.com/pyapp-kit/app-model/pull/142#issuecomment-1792405334
Just a note that this would be a longer term task.


Remove patch files

forgot to remove the patch files

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks


Add test/validation framework

Test bed of "medium-sized" Terraform codebases

Work in progress related to this in https://github.com/pulumi/pulumi-converter-terraform/compare/main...justin/tests.


Feature Request: Horizontal Scrolling

How hard would this be too do?
Or is there anything that would make it difficult for me to do?
Oh it was really easy. I'm probably going to for it and make it work with Swift 3.  Would you be interested in me making a PR with it?

@aasatt Hello! Swift 3 branch is already on master. Check it out!


Check for unexpected cargo/rustc before install

Hi!
This PR fixes #681, stopping the installation if rustc or cargo are found in any directory listed in $PATH if rustup wasn't invoked with -y.
Besides ignoring rustc and cargo if they exist in $HOME/.cargo/bin, as described in #681, this PR also ignores them if they're found in $HOME/.multirust, a situation that popped up during testing. What do you think?
I was also unsure about the error message, so I ended up copying a large portion from a similar situation in do_pre_install_sanity_checks (if rustc_exists, to be specific). Better alternatives would be very appreciated.
Finally, thank you for your work on rustup, it's a very nice tool. :)

Hm, I didn't account for $HOME/.multirust staying in $PATH even when the homedir member of a test configuration is changed. This makes the tests fail when run with cargo test but not when run manually (i.e., target/debug/…).
I worked around this issue in 3897c2a by filtering any path entry that contains ".multirust", but it feels hacky.

Looking at Travis' build logs more closely, it seems I need to exclude a couple more possible entries in $PATH. I think extracting the closure used in filter and then using conditional compilation is a reasonable solution.
I'm not sure what to make of the test failures in Windows. I guess I'll boot up a virtual machine and debug it there.

Thanks at @meqif, and thanks for writing tests!
From the various build failure here it's clear this needs to be considered carefully. Some observations:
The windows failure might be because the you are filtering out paths ending with ".cargo/bin" and windows uses backslashes canonically; though I'd be a bit surprised if the Path type didn't just handle that. You might consider just filtering out any path containing ".cargo" for simplicity.
For the error message I think we should print out the path to the binary that we found so that users can decide whether what rustup is thinking makes sense. Maybe it can say "of Rust at:", and the next line can print the path. I suspect this will look a bit ugly but it's a rare case so maybe it doesn't have to be beautiful (and rustup already has a lot of ui issues...).
By setting env::set_var here it's going to override PATH for all subsequent tests, creating an inconsistent environment depending on the order tests are run. Instead I'd suggest running your commands using the clitools::run method, which lets you set the environment. By using that method you won't get all the benefits of the expect_err helper though. It's a tradeoff. As a compromise I'd say you don't need to verify the entire output string, just that the process returns the correct value and includes some part of the error message.
The linux failures are interesting. The builders use a Rust not installed by rustup, so the test cases are finding those toolchains that are being used to test rustup and failing! Messy situation. The best idea I have to fix this is to use an environment variable just for the test suite to tell rustup-init to skip this sanity check (see the existing "RUSTUP_INIT_SKIP_SUDO_CHECK"), and set it for the entire test suite it clitools. The only wrinkle in here is that you somehow have to disable that environment variable for the three tests you do want that check running. For simplicity I'd say you can accomplish this by setting the "skip" environment variable in clitools::setup with env::set_var (instead of in clitools::env where the others are set), and then removing it again in your individual test cases. Setting the environment globally here is different than in the PATH case I mentioned before since PATH doesn't have any pre-existing content (also all these tests hold a mutex so they are run sequentially and there is no risk of them clobbering each others environment).
OK, I know that was a lot to digest, and it's kinda messy, but that's sorta the territory of this tool. Let me know if it makes sense.

Oh, also I think it's fine to just filter out all paths containing ".multirust".

Thank you so much for the thoughtful feedback, @brson. 👍

The windows failure might be because the you are filtering out paths ending with ".cargo/bin" and windows uses backslashes canonically; though I'd be a bit surprised if the Path type didn't just handle that. You might consider just filtering out any path containing ".cargo" for simplicity.

That makes sense, you're probably right. I suppose it would have been fine if I used something like Path::new(".cargo").join("bin"), as Path would have used the correct separator in that case.

For the error message I think we should print out the path to the binary that we found so that users can decide whether what rustup is thinking makes sense. Maybe it can say "of Rust at:", and the next line can print the path. I suspect this will look a bit ugly but it's a rare case so maybe it doesn't have to be beautiful (and rustup already has a lot of ui issues...).

Good idea. I was doing that during the development of the PR, but removed it because it did look a little ugly. Oh well, now it's back.

By setting env::set_var here it's going to override PATH for all subsequent tests, creating an inconsistent environment depending on the order tests are run. Instead I'd suggest running your commands using the clitools::run method, which lets you set the environment. By using that method you won't get all the benefits of the expect_err helper though. It's a tradeoff. As a compromise I'd say you don't need to verify the entire output string, just that the process returns the correct value and includes some part of the error message.

Ah, thank you. I wasn't aware there was such a significant difference.
I ended up doing something similar to what you suggested. I checked where RUSTUP_INIT_SKIP_SUDO_CHECK was being set (clitools::env) and added RUSTUP_INIT_SKIP_PATH_CHECK there, then overrode the default value in the tests, like for PATH. I tried to keep this change as small as possible. Let's see if I didn't miss anything else. :)

There seems to be a (spurious?) issue with fetching one of ring's dependencies. What an annoying coincidence.

I tracked down that build issue and opened another pull request that fixes it: #711.

Patch looks good to me! I think now that your rustls fixes are merged you'll need to rebase onto master to get travis to test this pr correctly.

Great! Let's hope nothing else goes wrong this time around. 💃

The tests pass in every Travis target, but not in Windows:
---- install_stops_if_cargo_exists stdout ----
	stderr: error: unable to read from stdin for confirmation
thread 'install_stops_if_cargo_exists' panicked at 'assertion failed: out.stderr.contains("it looks like you have an existing installation of Rust at:")', tests\cli-misc.rs:416
note: Run with `RUST_BACKTRACE=1` for a backtrace.
---- install_stops_if_rustc_exists stdout ----
	stderr: error: unable to read from stdin for confirmation
thread 'install_stops_if_rustc_exists' panicked at 'assertion failed: out.stderr.contains("it looks like you have an existing installation of Rust at:")', tests\cli-misc.rs:396

I assume I'm missing some special setup, but I'm not sure what it is. Any advice?

Since only the interactive tests I wrote fail, but not the non-interactive one, I've been looking at the tests in cli-inst-interactive. PATH changes asides, the major difference I noticed is that those tests are run with fake user input, and that seems to be enough for that error to be absent in Windows.
So, I added a helper similar to cli-inst-interactive::run_input, which adds "\n\n" as input to stdin. The tests pass in my machine (OSX), so I'm going to push a commit with those modifications and hope it's enough to make Windows happy.

Oh bother, they still fail on Windows — the installation isn't aborted as it should. Well, at least I learned something.

I finally noticed that the file checks didn't consider EXE_SUFFIX. After fixing that, the tests pass on all targets except one Windows target, TARGET=x86_64-pc-windows-gnu, MSYS_BITS=64. More digging will follow. 🔍

:umbrella: The latest upstream changes (presumably #850) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #842) made this pull request unmergeable. Please resolve the merge conflicts.

Thanks for the rebase @meqif.
The x86_64-pc-windows-gnu error looks legit. I haven't been able to reproduce it though because I don't have a working mingw environment :-/

:umbrella: The latest upstream changes (presumably #1012) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #1005) made this pull request unmergeable. Please resolve the merge conflicts.

Hi @meqif, are you still looking into this? Would you like any help debugging the test failures?

Hi @Diggsey! I've been trying to reproduce the test failure on TARGET=x86_64-pc-windows-gnu, MSYS_BITS=64 whenever I get the chance, without much success. I plan on trying again this weekend again.
I'd love if someone tried their hand at finding and fixing that elusive issue, though.

@Diggsey @brson Looks like some other commit in master fixed the issue I was having in this branch. I'd say it's good to go, unless you have any objections. 😄

Looks good to me, but it's slightly worrying that this code will effectively be untested, since we have to disable the path check on CI, and if it goes wrong it will prevent rustup from being installed.
@brson thoughts?

@meqif oh wow, thanks for sticking it out this long.
@Diggsey I think I'm fine with it since there are a few tests where the path check is enabled. Whatever, let's just go for it!

It was a long fight, but it was worth it. 💪 Thanks for merging!


Customizing a regEx error message removes default error messages of all other regEx

If you customize a regEx error message it removes the default error messages of all the other regEx.
SimpleSchema.messages({
  regEx: [
    {
      exp: SimpleSchema.RegEx.Phone,
      msg: '[label] must be a valid phone number'
    }
  ]
});

The way regEx messages are defined in v2 is different, and I don't think this would be an issue anymore.


Fix border on input group elements and custom form inputs

c3e4cbd changed the border of input elements to be translucent. This makes input group elements and custom form inputs follow the same pattern.
Fixes #21098

This might be perfect! I'm trying to prioritize only the essentials for our first beta though, so going to wait just a bit on merging this <3.

Shouldn't we change the color of the border when the input is focused as well?


"Open editors" should be able to sort in order of most recently viewed

Currently VS Code (while editing a Dart project with Dart-Code) shows the open tabs in a list at top right, called "OPEN EDITORS". This is sorted in the same order as the open tabs.
It would be really awesome if there were an option button at the top left of this list that allowed the user to sort in different orders, specifically adding the ability for the user to view the tabs in order of most recently viewed, so that the most recently viewed editor is at the top of the list.
This would help a lot when a lot of tabs are open, and you can't remember which file you made an edit in a few tab switches ago.

Alphabetical (lexicographical) sort order would also be useful for quickly visually finding files in the open editors list.

Alphabetical (lexicographical) sort order would also be useful for quickly visually finding files in the open editors list.

Did you look at the explorer.openEditors.sortOrder setting?

I didn't (I didn't know that setting existed, but then again, I have no idea how anyone is supposed to know what all the settings are, out of the hundreds that exist). But that's not what I'm asking for, I'm asking for a filter button at the top of the list (that sets that setting). UIs should be discoverable and contextual. Putting a button on the list itself would expose that setting, and make it easy to set in context.


Formatting objects differently depending on their inheritance

Is your feature request related to a problem? Please describe.
For models and schemas it would be nice to display these in table format, rather than the current list of headers. I realise this makes sense for normal classes, but it is very verbose for a model class like pydantic, dataclasses, and marshmallow. I would like to show these in a more compact format.
Describe the solution you'd like
Markdown table would be good
Additional context
If you point me in the right direction i could look at adding a per class type jinja template or something?
Related to https://github.com/pawamoy/pytkdocs/pull/51

Already possible! Instead of using individual docstrings, use an Attributes section in the class docstring:
class MyModel(BaseModel):
    """
    My model.

    Attributes:
        v1: Hello.
    """

    v1: str

Please reopen if it doesn't work as you would like.

With marshmallow i dont use docstrings at all, and i really dont want to add them as they add significant duplication. It seems to be fairly common place to describe the field as so:
class MySchema(Schema):
    v1 = fields.Str(description="Hello")

This keeps the code pretty succinct, and these are easily transformed into jsonschema where description can be reused on the front end. For the marshmallow implementation i added i take description from the field metadata and pass that as the docstring.
Therefore i think for these specific types of classes i would like to override the renderer template. I could do something as simple as allowing (and maybe autodetecting model class types?)
renderer:
    template: table

I see, you make valid points :slightly_smiling_face:
Then yes, you can use custom templates
to achieve what you want.
You could override the class.html template to check if the class has a marshmallow-model property and render its attributes in a table before the other children. To render the attributes, you would define a new template, something like marshmallow_attributes.html, that would look like the attributes.html one.
Something like this:
     ... previous contents

      {% if "marshmallow-model' in class.properties %}
        {% with attributes = class.attributes %}
          {% include "marshmallow_attributes.html" with context %}
        {% endwith %}
      {% endif %}

      {% with obj = class %}
        {% set root = False %}
        {% set heading_level = heading_level + 1 %}
        {% include "children.html" with context %}
      {% endwith %}

    ... next contents

Also I like the
rendering:
  template:

This is tracked in https://github.com/pawamoy/mkdocstrings/issues/105

I opened this feature request to support Marshmallow: https://github.com/mkdocstrings/griffe/issues/339. Closing this issue since there's nothing actionable here I believe.


add sponsor

testing: using valid ETH address for sponsor

use Go validator instead multiple conditions

github.com/go-playground/validator/v10


Created a javaScript Code for Pascal triangle

@lionelsamrat10 I'm closing the PR as it is not fulfilling the requirement...you can send a PR again after maintaining all the steps and eligibility criteria for a valid PR...


Initia.js is not supported in the browser environment

Expected behavior
I'd expect to be able to use LCDClient in the browser.
import { LCDClient } from<EMAIL_ADDRESS>

Actual behavior
When doing:
import { LCDClient } from<EMAIL_ADDRESS>
It fails with:
ReferenceError: Buffer is not defined
    at<EMAIL_ADDRESS>
Given that Buffer is not part of the Web API

To reproduce
Import LCDClient in the browser.
System information
<EMAIL_ADDRESS>Tested in latest Chrome

This library uses Buffer, so we are aware that it currently can't be used in web browser-based projects without a polyfill. While resolving this issue is on our roadmap, it's not our highest priority at the moment. In the meantime, please choose one of the following options:
1. Polyfill Buffer in your frontend project.
How to do this:
If you're using Vite, configure vite.config.js as follows. Alternatively, there are many examples available online that you can refer to.
import { nodePolyfills } from "vite-plugin-node-polyfills"

export default defineConfig({
  plugins: [react(), nodePolyfills()],
})

If you're using Create React App, you can find plenty of examples online that suggest using react-app-rewired instead of react-scripts to resolve this issue.
2. Instead of using this library, use @initia/initia.proto directly and use cosmjs, etc.
For example, you can change the following code
import { MsgExecute } from<EMAIL_ADDRESS>
const msgs = [
  new MsgExecute(
    address,
    "0x1",
    "dex",
    "swap_script",
    [],
    [
      bcs.address().serialize("0x").toBytes(),
      bcs.address().serialize("0x").toBytes(),
      bcs.u64().serialize("1000000").toBytes(),
      bcs.option(bcs.u64()).serialize(null).toBytes(),
    ]
  ),
]

to this code.
import { MsgExecute } from<EMAIL_ADDRESS>
const messages = [
  {
    typeUrl: "/initia.move.v1.MsgExecute",
    value: MsgExecute.fromPartial({
      sender: address,
      moduleAddress: "0x1",
      moduleName: "dex",
      functionName: "swap_script",
      typeArgs: [],
      args: [
        bcs.address().serialize("0x").toBytes(),
        bcs.address().serialize("0x").toBytes(),
        bcs.u64().serialize("1000000").toBytes(),
        bcs.option(bcs.u64()).serialize(null).toBytes(),
      ],
    }),
  },
]

Let me know if you have any further questions.

I ended up using the polyfill to square away the issue for now. Thank you!

closing this: please feel free to raise another issue if you have another question!


feat: add logs policy

This PR adds a new supabase RLS policy that allows all users to read from the logs table. This is required for the logging app. If policy is not set then the logger app can't read from the logs table using supabase's public anon key.

We will need to run this again for the new database from the refactor.

We will need to run this again for the new database from the refactor.

No, we need to run this migration only once. So it's enough to merge this PR and then merge the development branch into main to create the policy introduced in the current PR. When https://github.com/ubiquity/ubiquibot/pull/644 is merged there will be no need to run this migration again because RLS policies are kind of decorators over tables and persist across schema updates.


Remove organisations and parent from HTML publications links.json

These are provided by base_links, this is duplication – if one of these was changed (either base or the one in links) it might have unexpected results.

Rebased with master.


pat: correct numeric matching

addresses #27
This is now consistent with current Ruler behavior except for the bug reported in https://github.com/aws/event-ruler/issues/163, which does not occur in Quamina.

Drat. Pulled in math/rand/v2, not realizing it's a go v1.22+ thing.  I wish golangci-lint would catch this on my desktop.  I guess the correct thing to do is roll back to math/rand which will cause CI complaints but those can be ignored. Fortunately Quamina doesn't need secure random numbers, they're just for test data.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 93.93939% with 6 lines in your changes missing coverage. Please review.

Project coverage is 96.23%. Comparing base (b1c32f5) to head (49e80c7).

Files
Patch %
Lines

value_matcher.go
84.61%
5 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #329      +/-   ##
==========================================
- Coverage   96.48%   96.23%   -0.26%
==========================================
  Files          18       18
  Lines        1764     1804      +40
==========================================
+ Hits         1702     1736      +34
- Misses         35       40       +5
- Partials       27       28       +1

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

Hm, those benchmarks took me by surprise.  Will need to study some more.  Converting to draft PR.


S3DIS Semantic Segmentation Training from Scratch

In both PointContrast and ContrastiveSceneContexts papers, semantic segmentation results on S3DIS are stated as 68.2 mIoU. But in MinkowskiNet's GitHub repository(https://github.com/chrischoy/SpatioTemporalSegmentation), they achieve 66.3 mIoU using Mink16UNet34. You are also using Res16UNet34C with 5cm voxel size. When I train the model using my own repository with Res16UNet34C I also get around 66.4 mIoU. Is there anything I am missing? Can you explain how you get +2 mIoU compared to the original Minkowski model? Is it about data augmentation, optimizer etc. ?

Hi, there,
Could be the training recipe: have you tried to train on 8 GPUs with a total batch size of 48?

No I did not as I do not have access to 8 GPUs. In the PointContrast paper it is mentioned that replacing the backbone with SR-UNet is the reason why you get improvement from 65.4 to 68.2. SR-UNet is Res16UNet34C right?

But in the Minkowski repository model zoo they also use Res16UNet34C and achieve 66.4 mIoU. So, I thought it is not only the change of backbone that makes you achieve 68.2 mIoU. I will try larger batch size. If you have any suggestions I would appreciate it.

Hi, I am not exactly sure whether it is because of the architecture or batch size; as I trained the S3DIS semantic segmentation from scratch once with the PointContrast code base last year and at that time I used 8 GPUs, and since you mentioned that the architecture is the same, I assume it is probably due to the training recipe.

Hi, thank you for the explanation. I am using ME 0.5.4.

I c, please notice the number reported in the PointContrast paper actually uses ME0.3; but I tried semantic segmentation in S3DIS in ME0.4 last year. Have not tried ME05 though.


Notepad 7.8.7 crashes after clicking Show JSON Viewer on specific input

Steps to reproduce

Install JSON Viewer 1.34
Open a new file
Type in "true"
Click Show JSON Viewer in the plugins menu

Expected outcome: Notepad++ would show me the JSON viewer (or let me know it cannot display it for this input)
Actual outcome: Notepad++ closes, I guess as the result of a crash
Format JSON works fine, no errors displayed.

Download 1.4 from releases section. It's fixed there.

This fixes it, thanks


Seems like osc/osc_global.h is missing

osc/osc_global.h is referred to in the .pro file, but not referenced by any of the source files.
Is this a missing file, or an extra line in the .pro file?
thanks!

Hello,
Sorry, this repo is a mirror of our internal repository and has not been updated since a while...
In fact the osc_global.h file is missing.
Originally it was used for symbol export. It has been replaced by OSC_API.h
I just update the repository to mirror our internal changes.

thanks!
This library will really help me, and totally willing to contribute back... my other posted issue describes where I thought my starting place would be.  We can chat privately if you want.

If you updated the repo... should I re-pull?

Yes, you can update your repo !
You can also send email to me directly using the<EMAIL_ADDRESS>email address.


[Feature] Need integration with Office365 sheets like Google sheets.

Request by community on discord:
Hello like to gsheets "integration" could be done the something similar with office 365 sheets ?

Handy to get the google sheets id
how to get google sheet id


docker installation in c++

Hi guys can anyone guide me to install tensorrt client in docker . I ran $ docker run -it --rm --net=host tensorrtserver_client this code i entered into workspace docker what i have to do next. I an new to docker and c++ . I run basic code with tensorrt import

please find attachments for more details.

Unfortunately you don't show how you actually compile your code.
I suppose you don't use cmake or make but call the compiler directly somehow?
If you use g++ directly you probably need to add include dirs and link against the request libraries by using the -I (include) and -l (link) options (see this post as a quick overview).
Consider using cmake for this though. You can find how this is done under src/clients/c++/CMakeLists.txt. The easiest way would of course be if you just change any of the exemplary clients and rebuild everything or just duplicate on of the client examples in the cmake and under src/clients/c++/.
If you don't feel comfortable using c++ for this you might also be better off using python where you don't have to care about compiling stuff. There are many examples on how to do this in python in the client docker file.

There is example client code in the src/clients directory (both Python and C++). A simple C++ example is here: https://github.com/NVIDIA/tensorrt-inference-server/blob/master/src/clients/c%2B%2B/simple_client.cc
Also, the documentation has a section on building an running the clients and examples.


CB-11530 Upload freeipa-client dependencies to nexus

Previously, in CDPSDX-2788, the build scripts were updated to upload
freeipa-client as a jar into nexus. However, freeipa-client has two
dependencies: common, and cluster-proxy. We need to upload these
also to nexus so that thunderhead can use the freeipa-client.
Verified that common and cluster-proxy do not have any further
dependencies.
See detailed description in the commit message.

I would like to see if we can remove the freeipa client dependency on the cluster proxy. I would think it would be best if we could inject an interface that handles this. That way other consumers that don't know anything about cluster proxy don't need to have the dependency.

I think both cluster-proxy and common module has spring-boot dependency so you should use the uploadBootArchives task to upload them.
Also I don't know how much is used from these modules. If only a few class maybe it worth duplicating them instead of shipping those with freeipa-client

I think both cluster-proxy and common module has spring-boot dependency so you should use the uploadBootArchives task to upload them.
Also I don't know how much is used from these modules. If only a few class maybe it worth duplicating them instead of shipping those with freeipa-client

Good point. I'll address both yours and Jamison's comment by trying to remove the dependencies by duplicating them or making them a runtime dependency.

@jamisonbennett I removed the cluster proxy dependency by making the exception classes as interfaces and abstract. This is probably ok assuming that thunderhead has its own ClusterProxyException and ClusterProxyError classes.
@lacikaaa I could not remove the common dependency because it pulls in a bunch of other things (like Tracer, common-logging framework etc). So I think its best to just get the common module as a dependency as well. I modified to use uploadBootArchives

have you managed to test:

the upload works for common? (eg with using local nexus)
ClusterProxyException handling in the client still works as before?


Expand the ' broadcast' function to combine variables with 1+ dimensions

Hi, @awais307 and I often defined new parameters for new technologies using the current  broadcast  function, which basically expands an existing dataframe for an additional dimension.
However, we often use it to expand the year_vtg and year_act dimensions, which are also linked to a mapping that takes lifetime into account (see issue #571 in message_ix )
if we use broadcast for the sets  year_vtg  and  year_act  separately, we define also a number of vtg-act years that are not needed
import pandas as pd
from message_ix import make_df
from message_ix_models.util import broadcast, private_data_path

inp0 = make_df(
        "input",
            technology="irrigation_oilcrops",
            value=1,
            unit="-",
            level="water_supply",
            commodity="freshwater",
            mode="M1",
            time="year",
            time_origin="year",
            node_origin=['AFR','NAM'],
    )
inp1 = inp0.pipe(broadcast, year_vtg=[2010,2020],
           year_act=[2010,2020])

in this case, inp1 is wrong
what we would like to achieve is basically the combination of the dataframe with the missing variables inp0 and the mapping dataframe between year_vgt and  year_act
map_v_a = pd.DataFrame({
    'year_vtg' : [2010,2010,2020],
    'year_act' : [2010,2020,2020] }).astype(object)

inp2 = broadcast(inp0,toadd = map_v_a)

of course it would not work with the current version of broadcast, but I edited it so that it could work (see below).
I wonder if this is the way to go for you, if we should make a dedicated function, or if anybody have suggestions to make the code 'cleaner'.
Notes:

I explored the merge option vs concat and merge seems simpler with multiple variables
the print() are just for testing it

def broadcast(df,toadd = None, **kwargs):
    """Fill missing data in `df` by broadcasting.

    Arguments
    ---------
    kwargs
        Keys are dimensions. Values are labels along that dimension to fill.
    """
    print('KWARGS: ',kwargs)
    if toadd is not None:
        print('dataframe')

        for cc in toadd.columns:
            if cc in df.columns:
                assert df[cc].isna().all(), f"Dimension {dim} was not empty\n\n{df.head()}"
                df.drop(cc, axis=1,inplace = True)

        #concat_data = pd.concat([df,to_add['year_vtg']],axis = 1)
        df['tmp'] = 1
        toadd['tmp'] = 1
        df = pd.merge(df,toadd).drop(columns ={'tmp'})

    else:
        print('single parameter')

        for dim, levels in kwargs.items():
            # Checks
            assert df[dim].isna().all(), f"Dimension {dim} was not empty\n\n{df.head()}"
            if len(levels) == 0:
                log.debug(
                    f"Don't broadcast over {repr(dim)}; labels {levels} have length 0"
                )
                continue

            # - Duplicate the data
            # - Drop the existing column named 'dim'
            # - Re-add the column from the constructed MultiIndex
            # - Reindex for sequential row numbers
            df = (
                pd.concat([df] * len(levels), keys=levels, names=[dim])
                .drop(dim, axis=1)
                .reset_index(dim)
                .reset_index(drop=True)
            )
    return df

This is a good idea, thanks!

I think we can take advantage of the way .pipe() works: https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.pipe.html?highlight=pipe#pandas.DataFrame.pipe —note that it accepts either keyword arguments or positional arguments to be passed to the called func.
Currently broadcast() uses the keyword arguments:
https://github.com/iiasa/message-ix-models/blob/6929fd6df95388dc3635a56642688f9f00155d1b/message_ix_models/util/__init__.py#L164
….and understands that the argument name == the dimension/column name to be filled in the df.
I think the implementation could be very similar to what you suggest:

Instead of a keyword argument toadd, make this an optional, single, positional argument, maybe labels: pd.DataFrame (labels for multiple dimensions).
Instead of an if: block, allow both to be used simultaneously: first labels is handled, and then the existing code for any 1 or more dimensions inkwargs.

@adrivinca can we consider this closed by #74?

yes, thanks for pointing it out


how can I call cgal in my dll

Thanks for your dockerfile, But I still have two questions:
1.I build it and got four images, which one should I use?

I used the first image , but how can I load cgal header files in my dll? should I make some configuration ？

I am looking forward to hearing from you soon. Thanks.

Hey ;)

Just tag your image during the building so you can distinguish between them (https://docs.docker.com/develop/develop-images/multistage-build/)
The packages are output in /opt/conda, so you need to tel the linker to look for those there. Additionally you probably will need to set LD_PRELOAD: /opt/conda/lib/libstdc++.so because AM 2 has to old gcc++ installed by default.


In the snakemake rule, the path to load the conda file is incorrect！

Snakemake version
8.0.1

Describe the bug

the path to the conda.yaml defaults is in  rules folder, instead of the path written under the conda:
Minimal example

I write fastp yaml paths in the workflow/envs/fastp yaml, but snakemake search path in workflow/rules/envs/fastp yaml, snakemkae working directory is the current directory ./. yaml files must be placed under rules to be used.

Additional context

my code is follows:

my The file structure is as follows

The path to the conda environment is relative to the snakefile.
Youyou should refer your aml as ../envs/fastp.yaml in workflow/rules/*.smk or envs/fastp.yaml in workflow/Snakefile

Thank you for your response. The conda environment path is currently specified relative to the Snakefile, in contrast to the recommended structure where the results directory is placed in the workflow sibling relative to the workflow folder. Additionally, the working directory is displayed as the parent directory of the workflow folder during the parsing process, e.g., results/.
In Snakemake, file paths are referenced relative to the working path when the workflow is executed, providing an intuitive programming experience. However, the current setup for the conda environment path being relative to the Snakefile feels a bit awkward.
Could you assist in making the conda environment path specification align with the intuitive file path referencing used in the rest of the workflow?

No. the refer manner of env.yaml is consistant with rule.smk as components of wokflow design but not workflow application.
That is, you can import a snakefile with include "any/rule.smk", and won't worry if "env.yaml" is in pwd

Thank you，I now comprehend the rationale behind referencing the env.yaml as a component of the workflow. when I consider it as an  intergal part of the workflow,I no longer feel any discomfort; instead, I appreciate this usage pattern. I'm truly impressed by the design of Snakemake, and I want to express my gratitude to the developers for providing such an excellent tool!


Adds sql::Param to use the resulting string for prepared queries

I had to make a dedicated class because std::strings are automatically enclosed in quotes.
As a side effect parameters must be declared before use.

In the commit called "Fixed some errors" the errors fixed were mine

Thanks @JackLilhammers


Improve the way cookies are handled when sent from the backend

Currently the only way of setting cookies from Django and have them accessible by Next.js is to either do a ajax-request or redirect the user to a pure Django view. Regular SSR calls do not set cookies since they communicate server to server. Let's make it possible to retrieve cookies from SSR api calls and let Next.js set them.

Fixed in https://github.com/Frojd/Wagtail-Pipit/commit/1c772b2f917de8fccb9db37a9636f8ba9edfe843


Buell 2004: USGS ‘S3C’ (Site-specific soil C) database

[x]  Evaluate dataset
[x]  merge study data
[ ]  clean up profile data
[ ]  clean up layer data
[ ]  make graphics
[ ]  Write up summary report narrative
[ ]  Good commenting
[ ]  External review by someone else

see d6d4e6a8821e9ae402e413f0c2b1b065dc7a1f29

duplicate, #72


Versions

Nice little library. Could you possibly create a stable version branch rather than just "master".

Thanks! I'm using tags instead of branches for stable versions - the latest stable version is 2.0.2 which can be installed via composer with composer require "darsyn/ip:^2.0".