[LiveComponent] Add Events listening infos in Profiler and DebugCommand

Q
A

Bug fix?
no

New feature?
no

Issues
No issue

License
MIT

Hello,
First of all, thank you for your work and dedication.
Playing with LiveComponents on my spare time, I have a SPA using several events to communicate between components.
In several occasion, it took me a little time to find out bugs in my application concerning these events.
That's why I propose these small enhancements for debug concerning events in LiveComponent :

In TwigComponentDebugCommand, allow to filter components by the events they are listening to :

bin/console debug:twig-component --listening buyEvent

Still in TwigComponentDebugCommand, when displaying a unique Component, add a line to display the events it listens to, if any :

In Profiler, add a line to display the events it listens to, if any :

What do you think of it ? I will add test in a second time.

I'm really thinking more and more about "splitting" Twig & Live component to be honest...
So this "extensible system" you suggest @kbond is for me a step in the "good" direction whatever we do next 👍

Hey @mbuliard, #1722 should be merged soon. Basically, what's going to be needed here is:

Add the new CLI option to LiveComponentDebugCommand (this will require some extension points to TwigComponentDebugCommand)
Add the new profiler logic to LiveComponentDataCollector (this will require some extension points to TwigComponentDataCollector)
Add new profiler info to live_component.html.twig

I apologize in advance for the added complexity, I can definitely jump in and help if you run into problems.

Hey @mbuliard, #1722 should be merged soon. Basically, what's going to be needed here is:

Add the new CLI option to LiveComponentDebugCommand (this will require some extension points to TwigComponentDebugCommand)
Add the new profiler logic to LiveComponentDataCollector (this will require some extension points to TwigComponentDataCollector)
Add new profiler info to live_component.html.twig

Making changes to Twig Components that are required by Live Components in the same PR is a bit tricky. You need to temporarily add the following to live component's composer.json while you're working on it:
"repositories": [
    {
        "type": "path",
        "url": "../TwigComponent"
    }
],

Then run composer update for live components.
I apologize in advance for the added complexity, I can definitely jump in and help if you run into problems.

Hi @kbond , thanks for the update.
I will rework  my PR in the end of the week if yours is merged, otherwise it may be next week.
I hope to sort it out by myself, but will call for help if needed, thanks !

I hope to sort it out by myself, but will call for help if needed, thanks !

I'm hoping to resolve https://github.com/symfony/ux/pull/1722#issuecomment-2069518347 soon in a way that will make this PR easier.


Use Wants instead of Requires for systemd services

Fixes #276

LGTM

Merging this as failures are flakes.


Comments are often double-posted when network is bad

Recently I've been using CW on a 5G router. When the network becomes bad, I need to wait for a while before comments I post are accepted by the server and reflected on the page, but secretly these comments often become duplicated, and is unnoticeable until I refresh the page.
This can cause other problems, such as beta katas immediately sent to draft (because 2 issues are posted).
Normally this shouldn't happen. Is CW re-sending the request, or is the server somehow processing the same request twice?

Similar behavior has been already noticed and mentioned by B4B, me, and other users, but under different conditions: the reported behavior was that sometimes, when you try to write a post on a translation you just published, hitting the "POST" button appears to do nothing, so you click it again and again, only to find out that the posts counter actually goes up, multiple posts appear on the dashboard, and after a couple of minutes there's a bunch of posts under the translation. I cant remember it to be reported for other types of discourses than translations. I also don't recall it being reported as connected with some network issues, but in my case it didn't seem so (I was able to work with a translation, save it, publish it, and edit it without problems). Refresh did not show the posted comments, but they appeared after a minute or two.
I am not sure if these two are related or have a common root cause, but at least symptoms are very similar.


Add checks

Check some results from our examples.

Done in https://github.com/Rupt/c-explicitly-controlled-testing/commit/974c303d7f58f1111366620dd1a1399205cebd7b


[Question] Run scrcpy from an Android device

Hello!
Would it be possible to create a release for Android devices? Then one could connect from an Android device via adb-over-ip to another Android device, e.g. from a mobile phone. Prerequisite: adb works on that device (it does e.g. on a Samsung S7, from which I can connect to my Amazon Fire TV via adb-over-ip).
If there is already another/better way to do something with another method as equally cool as  scrcpy between 2 Android devices (which I doubt), I would be happy to learn about it.
Thanks for your kind consideration and feedback.

See #100.

@rom1v
Thank you so much for your hint and your scrcpy tool, which is really a great daily helper.
@tailcode (Anwar Mohammad Abdul in GitLab)
Thank you so much for your Android port of scrcpy. This opens so many additional possibilities for scrcpy.


Added Ref type and updated grammer

many to many relationships and updated the grammar.php to parser a many to may ref.
Also made the tokens case insensitive.

You don't need to change generated PHP code.
You need to write unit tests and make changes to the https://github.com/butschster/dbml-parser/blob/master/ebnf.pp2 file.
When you run tests, new grammar.php will be generated.


Support for TTL in Update query

In cassandra cql documentation it's said that you can specify ttl in update query:
https://cassandra.apache.org/doc/cql/CQL.html#update
And there is no support for it in DSL (https://github.com/websudos/phantom/wiki/Querying)
Are you planning to add such support? Is there any graceful workaround for now?

Hi @tyth ,
This looks like an oversight, this will be resolved very quickly and should be out in the next incremental release. I will try and release today itself to fix this and keep you posted.
Regards.

Hi @tyth ,
This should now be available in Phantom 1.13.0.
Regards.


creating a data structure.

A simple data structure for simple data processing, suitable for DE.  Added to unit test.

Why not NumPy's genfromtxt1 for data import?
# Code in the edgePy.io.Importer class could be replaced by
array = numpy.genfromtxt(filename, delimiter='\t', names=True)
header = array.dtype.names

This method will read the data into a contiguous block of typed memory. Supports gzip and bzip2 archives on-the-fly as input too.2

https://docs.scipy.org/doc/numpy-1.13.0/reference/generated/numpy.genfromtxt.html#numpy.genfromtxt
https://docs.scipy.org/doc/numpy-1.13.0/user/basics.io.genfromtxt.html


Add version tag

Occasionally there is confusion around the opaque :dev tag.
This PR adds a version tag so we could tell the workload version without deploying first.

As mentioned in a meeting yesterday, we should first look into the possibility of publishing to GHCR through OCI Factory

As @lucabello pointed out, oci-factory already publishes to ghcr: https://github.com/canonical/oci-factory/pkgs/container/oci-factory%2Fgrafana
Closing.


[Feature Request] Add Filter Builder

It would be helpful to have a class-based filter in MeiliSearchIndex.search instead of just raw string, e.g.:
QueryOperator in MongoDB https://github.com/kfiross/flutter_mongodb_realm/blob/master/lib/database/query_operator.dart
I can create a PR for this if there is a consensus on the design

Hi @ahmednfwela, that's a good idea! I saw the implementation, it looks simpler, but I would like to visualize how the user will use it. Can you give me some pseudo code as an example?

I added this proof of concept: https://github.com/meilisearch/meilisearch-dart/pull/252


Podcasting Website Added

Description
Project Description
This is a podcasting website Front-end where users can explore various podcasts, listen to their favorite episodes, and stay updated with the latest content from different creators. The website features an intuitive and responsive design, ensuring a seamless experience across devices. In addition to podcasts, the website also includes a section for reading blogs and articles on various topics, enhancing the overall user experience.
Tech Stack
HTML5: Structure the web pages.
CSS3: Style the website for responsive layouts and visually appealing design.
JavaScript: Add interactivity, dynamic elements, and user interaction.

Type of change

[x] New Project
[ ] Issue fix

Checklist:

[x] I have included a README.md file for my project clearly explaining its purpose, functionality, and usage instructions (applicable for new projects, not bug fixes).
[x] My code adheres to the style guidelines outlined by this project.
[x] I have thoroughly reviewed my code to identify and address any potential errors or inconsistencies.
[x] I have added clear and concise comments to my code, especially in sections that might be challenging to understand at first glance.
[x] I have reviewed and followed the project's Contributing Guidelines to ensure my contribution aligns with the project's requirements.

Good work @SoumikGhosh9


Embedded root.json is expired

Description

The default root.json for the public good instance expired last week: https://github.com/sigstore/sigstore-go/blob/ce459a84121500bf0a658aba8503a5836fd55ad6/pkg/tuf/repository/root.json#L6
Version

$ git rev-parse --short=8 HEAD
ce459a84

That's fine, this metadata is the root of trust for the TUF metadata. As per the TUF spec, a TUF client will download N+1 versions until it receives an error, verify N-1 signed N, and then check the timestamp on the most recent root metadata.
We should still update the root as an optimization though so that clients don't have to download the latest root metadata each time.


Extension needed for abort_unless

We need another type specifying extension. This time for abort_unless
It should be the almost the same as #330
This can be a good first issue to start contributing to larastan! :+1:

And I guess same thing can apply to throw_if and throw_unless

No. I think they should have seperate extensions.

Okay. I thought they need the very same action.

You can register one extension multiple times with different constructor arguments. abort_if and throw_if seem like they do the same thing with the first argument.


os-igmp-proxy Error: Sender VIF was down

os-igmp-proxy Error: Sender VIF was down
Before adding this new report, I have checked it:

[*] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[*] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[*] The title contains the plugin to which this issue belongs

Describe the bug
There is an issue on the plug-in: os-igmp-proxy 1.5_2 (Opnsense 22.7.11).
Plugin Error in Opnsense Log: Sender VIF was down.; Errno(50): Network is down
Shortly after I started the plugin, the error occurs.
To Reproduce
Steps to reproduce the behavior:

Download the plugin os-igmp-proxy
Configure the up- and downstream to LAN
Start os-igmp-proxy plugin
Connect Device to LAN and start Device
View Logs
Look for the error in the logs

The used Device is a brand new Telekom Reveicer (MagentaTV)
Expected behavior
I expect the plugin to run normally and without errors.
Screenshots
A screenshots is not necessary, but can be submitted later if required.
Relevant log files
Plugin Error in Opnsense Log: Sender VIF was down.; Errno(50): Network is down
Additional context
Unfortunately, I am trying in vain to get a Telekom Reveicer (MagentaTV) to run.
Briefly about the setup:
Vigor Modem ---(PPPOE)----> Opnsense ----> Reveiver
VLAN7 is handled by the Opnsene.
If more information is needed please let me know.
Environment
Software version of os-igmp-proxy is 1.5_2
Opnsense 22.7.11

An error was found that is not due to the plugin.


Enhance the docker-compose/makefile setup for running/reverting migrations

Overview
This PR contains the following changes

Added discrete setup/migrate/rollback commands to makefile which trigger corresponding docker-compose services.
Updated root migration to include a rollback step that drops everthing

make db-setup - Run the database migrations + seeding
make db-migrate - Run the database migrations
make db-rollback - Rollback the latest migration(s)
Note:
make web - still runs db-setup as part of its workflow, so if you're just building the app for regular use then you don't need to worry about any of the above commands.
Note:
If the root migration gets rolled-back, then the migrations AND seeding will have been reverted.  So getting the app back to its previous state would need a make db-setup, not just a make db-migrate.
This PR contains the following types of changes

[ ] New feature (change which adds functionality)
[ ] Enhancement (improvements to existing functionality)
[ ] Bug fix (change which fixes an issue)
[ ] Misc cleanup / Refactoring / Documentation
[ ] Version change

Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
General

[ ] I have performed a self-review of my own code

Code

[ ] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[ ] I have avoided duplicating code when possible, moving re-usable pieces into functions
[ ] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[ ] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[ ] My code changes account for null/undefined values and handle errors appropriately
[ ] My code uses types/interfaces to help describe values/parameters/etc, help ensure type safety, and improve readability

Style

[ ] My code follows the established style conventions
[ ] My code uses native material-ui components/icons/conventions when possible

Documentation

[ ] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation, as needed

Tests

[ ] I have added/updated unit tests for any code I've added/updated
[ ] I have added/updated the Postman requests/tests to account for any API endpoints I've added/updated

Linting/Formatting

[ ] I have run the linter and fixed any issues, as needed
See the lint commands in package.json
[ ] I have run the formatter and fixed any issues, as needed
See the format commands in package.json

SonarCloud

[ ] I have addressed all SonarCloud Bugs, Vulnerabilities, Security Hotspots, and Code Smells

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots, if applicable.

Looks good.  I tested it locally and it works great!


Bump kafka dependency to 3.2.3

Fixes: #81

In fact, does anything speak against going to 3.3 (3.3.1, precisely) right away?


Fix Sidekiq monitoring typo related to Manuals Publisher

Remvoe _api from the hieradata. This was causing the redis envvar
configuration to use the default values, e.g. host <IP_ADDRESS> that
don't work.
I think this was a copy/paste error when I added the configuration.

Awesome, thanks for taking a look @1pretz1


Need to display torrent information

Webtorrent exposes information about the torrents it is tracking.
At a minimum, we should expose the following information:

[x] Global upload
[x] Global download
[ ] Global peers connected
[x] Global websites shared

Landed in https://github.com/retrohacker/peerweb/pull/13


Minor accelerator fixes

closes: https://github.com/opendatahub-io/odh-dashboard/issues/1765
Description

Minor UX and bug fixes found by testers

making naming in the error states of the dropdown dynamic so they can use image or servingruntime
improved backend logging to be more verbose and not miss any exit points
remove double array usage in accelerator state hook
remove usage of unknown when we know that there is not an accelerator for description fields and logging - replaced with None
prevent count from going to 0 -> same logic as other number inputs. The default and limit are now 1.
removed italics from dropdown None option

How Has This Been Tested?

Migration should always give a logging on error, success, or skipping
test that errors display correctly in migration (try removing crd for acceleratorprofiles so it throws a 404 error on migration)
serving runtime dropdown errors should not reference an image
count should always default to 1 and should not be able to go to 0

Test Impact

no tests are made for this feature yet. These will be created once the feature has gone through development
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

There are UI changes, but they are not changes in the UX, just small bugs. I will still tag UX, however there is not much action needed.
@yannnz
After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@lucferbux I migrated everything that was using || to use ??


nginx-php-fpm container should not change permissions/ownership of mounted files

What happened (or feature request):
The /start.sh that we ship with nginx-php-fpm7 container does this:
chgrp -R nginx /var/www/html
This changes the file permissions of the mounted files, which is pretty much a no-no.
It also takes forever, and makes our ddev startup take 30+ seconds when it could probably take just 5-6 seconds (to be seen).
What you expected to happen:
File permissions on the mounted filesystem should be left as-is.
How to reproduce it (as minimally and precisely as possible):
Inspect the start.sh
Anything else do we need to know:
Related source links or issues (like source JIRA issue):
Related issue, #96 - which deals with the problems that result from changing owner and perms.

@rfay is this still valid/actionable?

I think we solved this long ago.

However, we'll certainly look closely in the security change and make sure it's not doing it, so it's valid to leave it open for review. Bottom line is we don't want to change owner or group (ever) on host-mounted files.

Alrighty. We'll leave it in v0.7 to confirm. Thanks!

The offending chgrp is definitely not in there any more. Closing.


install zip for 3.23 release

There's no install zip for the latest release, 3.23, just source code.  Should there be one?
Thanks.

Ah it was just not there, woops.  Thanks for the heads up - fixed.


Fix TypeVariable to getOptionalModule

Codecov Report
Attention: Patch coverage is 33.33333% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 80.22%. Comparing base (c97cc9b) to head (86490cf).

Files
Patch %
Lines

...c/main/scala/com/ing/baker/types/TypeAdapter.scala
0.00%
1 Missing :warning:

...s/src/main/scala/com/ing/baker/types/package.scala
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1673      +/-   ##
==========================================
- Coverage   80.36%   80.22%   -0.14%
==========================================
  Files         143      143
  Lines        4792     4794       +2
  Branches      168      165       -3
==========================================
- Hits         3851     3846       -5
- Misses        941      948       +7

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


Userscripts implementation checklist

[x] @require meta data must be stored as a list of { "url": "", "content": "" } objects
[x] Make sure that @connect works: https://tampermonkey.net/documentation.php#_connect
[x] Make sure that @grant permissions are enforced on the server-side, not just by the client-side: https://tampermonkey.net/documentation.php#_grant
[ ] Make sure that @noframes is properly implemented by the client-side: https://tampermonkey.net/documentation.php#_noframes

todo list for @connect:
. domains like tampermonkey.net (this will also allow all sub-domains)
. sub-domains i.e. safari.tampermonkey.net
. self to whitelist the domain the script is currently running at
. localhost to access the localhost
. <IP_ADDRESS> to connect to an IP address

core/pull-requests/591

Make sure that @noframes is properly implemented by the client-side: https://tampermonkey.net/documentation.php#_noframes

Seems that there is a bug. Currently this check condition (isAllowedToExecute) is calculated but is not used in run() function.

Summoning @maximtop here


Convert paragraph to aside note

What kind of changes does this PR include?

[x] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[ ] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description

Convert paragraph to aside

Before

After

Thanks Kevin! That note is actually about to vanish in a puff of smoke as part of #759, so I hope you don’t mind too much if I close this in favour of that 💜

It's ok!


feat(tutorial): complete step 1

Closes #
First pull request for carbon tutorial

Someone could help me?
My build is not finishing correctly... I don't have any idea about the error.
Is running correctly with no errors:


LOSSST (#1368)

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #1368

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸
Demo Vedio:
https://github.com/kunjgit/GameZone/assets/98798977/306e3186-b7bc-40ba-b73e-c1342c1957fa

Thank you soo much for contributing to our repository 💗

Thank you  @Durgesh4993 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Undocumented use of ^ in formulae

What is the syntax for using ^ in a formula? I can get all the other specials to work, but I can't seem to make ^ work. Can anyone elaborate on the correct syntax?

I think it's not actually used for anything anymore. Formulas have changed quite a bit since they were first implemented. At one point there was a way to quote expressions that would lazily evaluated against a data frame during model matrix construction, but that feature was removed in general from DataFrames itself for performance reasons.

That seems to answer my question. Thanks.


Fix import path in App.jsx

Fix Import to fix component shadowing App.jsx

The thing is, I'm baffled that this is needed. We use a special webpack resolver plugin that "rewrites" all the imports, so this sort of thing shouldn't be needed.
@jimbiscuit Can I have a confirmation that you've tried to shadow those files and they didn't work? Maybe, if you have time, you can try poking around https://github.com/plone/volto/blob/master/webpack-relative-resolver.js to see if you can get some info on what could be wrong?

Or maybe it's just a problem of "if I shadow App.jsx, then I'll have to rewrite those imports in the shadowed file" ?

I'm +1 on using absolute imports at least in the component level in all volto-core components. @tiberiuichim This can also go into one of https://github.com/plone/volto/issues/2810 :)

@nileshgulia1 I'm conflicted on this one. Personally, as a developer writing Volto code I like being able to reference components and modules with relative paths. It's quicker to write the imports (they're usually shorter), avoids having to add them to the big components/index.js index, etc.
Now, one year ago, Volto code that was written with this style of imports was impossible to customize. For example, if App.jsx imports '../SomeComponent', the SomeComponent.jsx file would be impossible to shadow without also shadowing App.jsx. So I wrote the webpack-relative-resolver plugin that fixes the problem, so now we are free to use whatever the code style we want, as we're guaranteed that shadowing will work.
But there's still a problem, though it's an easy fix: when I copy a Volto file in my project, to shadow, if the file contains code with relative imports, I'll have to tweak my copy to fix those imports. Indeed, not the best.


Question about the training of TI

Hi author, I would like to ask one question, did you use all the masks in the test set when training the TI or only the first 1/3 of the masks？
Looking forward to your answer, thanks!

Sorry I do not get what 'TI' represents. But for all the training process, we always use the first 1/3 of the data.

Sorry I do not get what 'TI' represents. But for all the training process, we always use the first 1/3 of the data.

It refers to the Textual Inversion method that you used to generate different masks.

Sorry I do not get what 'TI' represents. But for all the training process, we always use the first 1/3 of the data.

It refers to the Textual Inversion method that you used to generate different masks.

If so, we only use the first 1/3 of the masks.

Thanks, I have one more question, the image of the training sample given in the paper is the 7th corresponding to the wood_color class (there are 8 samples in this class) and does not belong to the first 1/3 of the data mentioned

This experiment is not the final version, where we train the models by randomly selecting 1/3 of the data. Since it is only for qualitative comparison, we keep it to show the comparison results. Sorry for bringing such confusion to you.

Dear Hu Teng,
I observed there is red color on the generated images in MetalNuts, However there is not red color in the training set. Could you please clarify this issue?

Dear Hu Teng,
I observed there is red color on the generated images in MetalNuts, However there is not red color in the training set. Could you please clarify this issue?

For this extremely biased dataset, we use crop-paste to crop the red color from 012.png and paste it on 000.png with the mask of 000.png manually. Since this is not a common case, we do not mention it in the paper and code. And actually, the generated red color does not improve the anomaly detection accuracy obviously.

Dear Hu Teng,
For How many classes you did this? Could you please clarify?

Hi, thank you very much for your answer. I have two more questions:

for object class defects, how to make sure that the randomly generated mask is in the corresponding position of the object, not in an unreasonable area or background area.
for the logic class defects (cable swap, metal_nut flip and transistor misplaced), is there any special treatment done and how is the reasoning process implemented.

Hi, thank you very much for your answer. I have three more questions:

for object class defects, how to make sure that the randomly generated mask is in the corresponding position of the object, not in an unreasonable area or background area. I have read about mask filtering operation in other issues, and I would like to ask how to filter it, because the object area is different in each image.
for the logic class defects (cable swap, metal_nut flip and transistor misplaced), is there any special treatment done and how is the reasoning process implemented.
we downloaded the provided segmentation model for unet weights and found that unet handles the segmentation of logic class defects very well, would like to ask how this is achieved?

There is a problem where sometimes the anomaly mask is outside the object. But we do not filter out them in our experiments since some of the anomaly types have anomalies outside the object while some do not. But you can try using object segmentation model to filter out some of the unreasonable masks.
We do not treat the logic class defects specially.
I think that for metal_nut flip, it can also be regarded as a type of different texture. And for transistor misplaced, it mainly relies on the mask generation method, since it is strongly related to the position. For cable swap, I find out that our generated data has some wrongly generated results. Therefore, even though there is not a special design for logic anomaly, for most of the situations(metal_nut flip and transistor misplaced), our model still performs well. But for some cases like cable swap, It do not performs so well.

Dear Hu Teng,
Thank you so much for your kindness. For How many classes you did this? Could you please clarify? I observed similar things in carpet and wood.

I need to recount it carefully. But I'm quite so busy before ACM MM 2024, I will rely you once I have enough free time or after ACM MM. Thanks for your patience.

Thank you @sjtuplayer, for your kind response.


Sponsor Workshops should be part of the program export

Currently the program export functionality doesn't include Sponsor Workshops/Sponsor Tutorials. It would be great if they could be exported as well.
If it's not a trivial/quick fix, that's fine for this year; there are few enough of them that I can easily copy them to the mobile guide manually. Of course, if it is a quick fix, that would be fantastic :)

You're in luck! It was a straightforward change, and has been deployed to production. Files for sponsor tutorials are at program_export/presentations/csv/sponsor_tutorials.csv and program_export/presentations/rtf/sponsor_tutorials.rtf.
Fixed in f07a1a3.

Thanks @felixc & @rebecca-caktus!!

Thank you! It's all there and looks great.


[Feature]: Initial LLM token

🚀 The feature, motivation and pitch
Not sure that this has been implemented but could add in initial tokens (i.e. text) to the beginning of the generation process be possible. So basically having the first few tokens be "Sure Thing!", for example, then the model continues generating tokens from that point on.
Alternatives
No response
Additional context
This is in effort to have more control of the model output for returning certain formats and to reduce randomness in the responses.

I think I understand this use case but just to confirm is there a reason that this cannot be part of the prompt?

There is a difference between instructing the model with a prompt like Start all your responses with "Sure Thing!" and the model having Sure Thing! as the initial tokens that follow the begin-response token. The former is an instruction given to the model each time, while the latter means the model is pre-configured to always start responses with 'Sure Thing!'
Here is an example for my use case:
You can also think of it this way Start all your responses with "[" and create a json with the following information that I provide . I am telling the prompt to start as a Json it is not a guarantee but all too often it will start off saying Here is your Json: [{. But if you have the initial token following the begin response token as [ then it will basically guarantee returning a Json.

Why not just using the system prompt for this one?
Also, where have you seen this before? I have never encountered such feature

@Etelis What I described was for the system prompt.

I think anthropic's api has similar feature.
https://docs.anthropic.com/en/docs/build-with-claude/prompt-engineering/prefill-claudes-response
I'm not familiar but may it achievable with outlines?

A potential workaround may be found here: https://github.com/vllm-project/vllm/issues/6772#issuecomment-2251961924

You might find the continue_final_message extra call argument helpful - docs
Example:
curl -X POST "http://<vllm-address>/v1/chat/completions" -H "Content-Type: application/json" -d '
{
  "model": "<model name>",
  "messages": [
    {"role": "user", "content": "Hello there!"},
    {"role": "assistant", "content": "Hi! My name is"}
  ],
  "add_generation_prompt": false,
  "continue_final_message": true
}'

Make sure you're running the latest vllm server version


Optimise get_random_peer_for_block_sync

    True.

And we might need to revisit this at a later date.
Originally posted by @appetrosyan in https://github.com/hyperledger/iroha/pull/2762#discussion_r979675774

This issue has already been done in previous refactors.


🐛 Add regex validation for signer

Summary
Related issue(s)
Fixes https://github.com/open-cluster-management-io/ocm/issues/382

/hold
will add integration test.
/assign @zhujian7

/unhold

/lgtm


Run multiple containers in same time

I have issue when try to run more than one container.
Only 1 MT5 terminal can open, another tab is can not show MT5 terminal. I'm try to restart this docker container, It will show but the first one is gone :(
I'm using config like this to run another port:

version: '3'
services:
mt5-2:
image: gmag11/metatrader5_vnc:1.0
container_name: mt5-2
volumes:
- ./config:/config
ports:
- 3001:3000
- 8002:8001
environment:
- CUSTOM_USER=
- PASSWORD=

Can you give me advise?

Hi, may you paste your complete docker compose file with multiple instances?

version: '3'
services:
mt5-1:
image: gmag11/metatrader5_vnc:1.0
container_name: mt5-1
volumes:
- ./config1:/config
ports:
- 3000:3000
- 8001:8001
environment:
- CUSTOM_USER=
- PASSWORD=

mt5-2:
image: gmag11/metatrader5_vnc:1.0
container_name: mt5-2
volumes:
- ./config2:/config
ports:
- 3001:3000
- 8002:8001
environment:
- CUSTOM_USER=
- PASSWORD=
Hi, this is my compose file.
I already solve the issue!
No problem with multiple instances. The reason is my ram too small.
It's require at least 1gb ram per instance.
Thank @gmag11  your image can help me to run multiple MT5 terminals with lower cost.


Warn on cookie length?

I've just spent a little while debugging an issue with the cookie not being saved as expected. Upon inspection I realised that the API I was working with started returning more information, thus exceeding the cookie byte limit.
Although that issue is on me, if someone else was to run into it, having a warning on cookie size (even dev only) would have reduce the debugging time.
Great work so far!

@seanislegend that's the kind of great feedback I was expecting from people using this module. Do you happen to know if it was the browser that was refusing the cookie or any other piece of code inside next-iron-session? I will have a try at replicating this but if you have more information, let me know :)

Great to hear. I'm afraid that at the moment I realised, I didn't have enough time to take a deeper dive into the root of the issue. Happy to throw together a reproducible repo for you in the mean time, if that helps.

@seanislegend Yes that would help me a lot indeed, if you have some time

I have duplicated the Next.js example folder and included a reproduction here:
https://github.com/seanislegend/next-iron-session-byte-exceeded-bug/blob/master/pages/api/login.js#L11-L254
The change I made is storing additional data in the session. Let me know if there's anything else I can do to help.

:tada: This issue has been resolved in version 4.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Thanks for reproducible code @seanislegend!


[quilt_rails] Expose header passing API from reverse_proxy

Problem Space
It's relatively common for apps to need some kind of initial (meta)data before they can do too much. With the current setup suggested by quilt_rails + react-server we push folks to use GraphQL for everything, which has a few unfortunate side-effects:

to get simple trivial initial data (eg. custom locale preferences or other user specific metadata) we are forced to introduce waterfalling GraphQL requests which is a big performance no-no
until we have our SSR GraphQL architecture sorted out it practically means that nothing can be server rendered (since the initial data request would not be able to be run on the server and nothing could be rendered before it is done)

In complex applications this gets worse, as we have seen in our largest React+GraphQL app where we actually have two different top level queries we need to run in series to figure out what sections we can render (and one we have to run before we even start rendering the app!)
Beyond the above, it's also a bit of DX bummer for extremely simple applications to need to setup graphQL before even the most trivial of data can be sent to their React app.
Proposal
We already have a hook that we recommend devs use to access request headers inside the app
import {useRequestHeader} from '@shopify/react-network';

// ...

const someData = useRequestHeader('some-header');

On top of this, the gem we use internally to proxy requests to react-server exposes a headers option.
reverse_proxy(headers: { foo: 'bar' })

If we were to just expose a similar option on render_react we could pretty easily support simple cases (specifically strings) being passed to the react app directly from Rails
# ui_controller.rb

def index
  render_react(headers: {'x-custom-locale': current_user.custom_locale})
end

// App.tsx
import {useRequestHeader} from '@shopify/react-network';

// ...
export function App() {
  const locale = useRequestHeader('x-custom-locale');

This setup would be a bit less nice for complex data, which is both a good thing (pushes users towards GraphQL for complex data needs), and a bad thing (might just mean they use it anyway and their code is worse). I think it's a sane API that will make 80% easy and 20% possible for our devs, and if we see a consistent need for data that it is a poor fit for we can do more then.
Type

[x] New feature

@jordanAtShopify  heard you might be interested in helping out with this

@TheMallen Thanks for the awesome write-up!


potential fix for issue 11988

SUMMARY
Cast one side to float, which seems to fix the issue for me but it may have ramifications I'm unaware of; as requested in the Issue chat
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[x] Has associated issue: #11988
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

closing due to inactivity


add CassandraKVStore

Adds:

stores/CassandraKVStore
utils/CassandraTable
Support for IN operator

Breaks:

Interfaces {Column, Index, Filter, WhereClause} now exported from utils/cassandra rather than vectorstores/cassandra

@jacoblee93 I'm not sure what is the docs failure - the doc preview in the Dev Container doesn't work for me (Windows fwiw, but should be isolated from that with the container right?). The process for building/previewing docs isn't terribly clear to me from CONTRIBUTING.md :/

@jacoblee93 figured out the docs problem (yarn docs worked great, and allowed me to preview the docs too which is nice! I was trying previously to use the MDX preview, but that's not worked for me)...

Thank you! I think looks broadly reasonable, see comments.

Thank you!


GitHub actions ( build checks )

New feature

GitHub actions can be very useful to check build process on every commit or pull request .
It would be a script that will be present in .GitHub/workflows folder
I would like to work on this issue .

@majordwarf shall i raise a pr for the same ?

What things do you plan on testing via GH Actions?

for now it would be simply build pass ( like it github actions will show if the current commit done to the repo is breaking the build process  or not ) it is highly useful to see which commit id breakingh the build process of website .
More advanced checks would be using unit testing on github servers too( in future if unit test are to be written for react componet and server apis )

done


Filter Vector layers before rendering

Is there any way to filter features before rendering? My use case is filters that can be dynamically added/changed to a layer. I know I could use the style function for add the filters, but it kinda breaks down if the user resets the style function after filters are applied.

There is currently no built-in way, but you can control yourself what features you put into the vector source or which features you remove from it, or am I missing something?

I can, I'm just worried about performance, having to process large amounts of features and having to keep the unfiltered and filtered arrays synchronized and also take into account features being added/removed at arbitrary times. One of the filters I have, for example, is hiding all features outside of a time range T (current time) minus X and T minus Y. Even if you don't change X or Y, T is constantly changing. I can envision a way to do this somewhat performantely, was just wondering if there was a good way to do it already.
On 21 Nov 2016 12:06, Bart van den Eijnden<EMAIL_ADDRESS>wrote:
There is currently no built-in way, but you can control yourself what features you put into the vector source or which features you remove from it, or am I missing something?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/openlayers/ol3/issues/6122#issuecomment-261919610, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AO7ifj-M66Y4kH5J9YL2wpm2sL5gBWArks5rAYkWgaJpZM4Kw0dp.

i  also try to find the way to  Filter Vector layers  as you says
mark.

Closing due to lack of activity.


Extension attachment: one file type is missing in alert message

Describe the bug in current situation
In extension, the error message for uploading a invalid file will list allowed file types. "eml" is missing in the file types.
Chance of Occurring (high/medium/low/very low)
low priority
Steps to Reproduce
Steps to reproduce the behavior:
Click on the ellipsis of the top extension
Select "edit"
Drag a invalid file (other than pdf, docx, eml, xlsx) into box to trigger the error message

Confirmed that the error message pops up and notes the 5 appropriate file types. Nice work @aparna-aot


Auto Backup is not working on OnePlus 9

Before you report your issue, please consider perusing the https://github.com/jberkel/sms-backup-plus#faq and searching the issues page to see if it has already been reported.
Expected behaviour
Auto backup would backup sms and calls one minute after they happen.
Actual behaviour
Backup only occurs when I manually run it
Steps to reproduce the behaviour
Set your auto backup settings (mine is set to 30min/1min)
Please specify the following:

Android version 11
Phone model / brand OnePlus 9
SMS Backup+ version installed 16.0-BETA2
Messaging app Google Messenges (Beta)

Thanks for the report.
Duplicate of #758


Clarify the usage of LICENSE template in docs

Closes: https://github.com/ansible/ansible-creator/issues/245 and https://github.com/ansible/ansible-creator/issues/244

Closing it, as this will need more work as per: https://github.com/ansible/ansible-creator/issues/260


Check for SPDX licenses

It checks via https://github.com/enarx/spdx action if a SPDX file exists.

I do not know if this is the best solution. First I leveraged enarx/spdx@master but they would look through every file and it would take a long time (after 2 hours of running the action I stopped it). So I went with another approach to check with git diff which files were modified and only check them. Therefore I will match to https://spdx.org/licenses/Apache-2.0.html. But currently not working it seems. I will investigate further.

Once you have verified my changes I will remove the two test files.

Concerning the "fork" discussion, you could possibly compare with https://github.com/COVESA/vss-tools/blob/master/.github/workflows/pre-commit.yml where fetch-depth: 0 seems to do the trick to make sure that everything between "from-version" and "new-version" is present. Have however not tested if that is the solution here as well

Will close this PR because of #541 which solves it better


Refactor BlockChainAppService

Solve Refactor BlockChainAppService problems in the community
issues 1789
Update the origin  BlockChainAppService.BlockChainAppService
split the business logic in to the AppServices:

Add:folder AElf.WebApp.Application.Chain.AppServices
Add:Class Library AppBlockService.cs
Add:Class Library AppTaskQueueService.cs
Add:Class Library AppSendTransactionService.cs
Add:Class Library AppTransactionService.cs
Add:Class Library AppTransactionGetResultService.cs
-Add: Class Library AppTransactionResultService.cs

:x: Build AElf <IP_ADDRESS>96 failed (commit https://github.com/AElfProject/AElf/commit/761c648d99 by @tomatopunk)

:x: Build AElf <IP_ADDRESS>05 failed (commit https://github.com/AElfProject/AElf/commit/4e1fbccad4 by @tomatopunk)

ci failed, maybe should config the router

ci failed, maybe should config the router

hi:
I modified the code,there should be no unit test errors : )

:white_check_mark: Build AElf <IP_ADDRESS>12 completed (commit https://github.com/AElfProject/AElf/commit/08e75782df by @tomatopunk)

Return value should be DTO
Only api class can implement IApplicationService
Api class naming should be  like XXAppService
Reconsider the name of AppTransactionGetResultService

:white_check_mark: Build AElf <IP_ADDRESS>44 completed (commit https://github.com/AElfProject/AElf/commit/866662a526 by @tomatopunk)

maybe it will help you, https://github.com/abpframework/abp/blob/master/docs/en/Application-Services.md

AppService should not inject other AppService.
this issue is required to split one AppService into some small AppServices, but small AppService should not reference other,
For example, you should not inject IBlockAppService, you should inject IBlockService in AElf.Kernel
so do not change the implements of origin class, just split one class into small classes

AppService should not inject other AppService.
this issue is required to split one AppService into some small AppServices, but small AppService should not reference other,
For example, you should not inject IBlockAppService, you should inject IBlockService in AElf.Kernel
so do not change the implements of origin class, just split one class into small classes

Roger that,I need to pay attention to the cross-reference problem between AppService!

sorry for your preview code reviewer, he maybe not tell you the point.
In this issue, field A,B,C with method D, and field A,F,E with method G
so we should make DAppService, including  field A,B,C with method D
and GAppSerivce, including  field A,F,E with method G

对不起您的预览代码审核者，他可能不会告诉您这一点。
在本期中，字段A，B，C使用方法D，字段A，F，E使用方法G.
所以我们应该用方法D制作DAppService，包括字段A，B，C
和GAppSerivce，包括方法G的字段A，F，E

hi:Roger that,i will modify it as required

sorry for your preview code reviewer, he maybe not tell you the point.
In this issue, field A,B,C with method D, and field A,F,E with method G
so we should make DAppService, including field A,B,C with method D
and GAppSerivce, including field A,F,E with method G

hi:Roger that,i will modify it as required

hello everyone ~
I re-edited the code according to the following principles:

now. AppService does not reference each other~
The refactored AppService inherits the ApplicationService each one~
Ensure that the private level before refactoring does not become public
No new namespace is created, all AppServices are placed under AElf.WebApp.Application.Chain

ps:In the commit, a new class TransactionResultProxyService has been added...
what`s he?
It is a proxy service class that injects two fields....A little bit like a decorator
If you feel that my name is not appropriate, please help me to modify it..
thank you very much :)

:white_check_mark: Build AElf <IP_ADDRESS>54 completed (commit https://github.com/AElfProject/AElf/commit/0d8a5fd46e by @tomatopunk)

:x: Build AElf <IP_ADDRESS>10 failed (commit https://github.com/AElfProject/AElf/commit/973e2ffffb by @tomatopunk)

:white_check_mark: Build AElf <IP_ADDRESS>38 completed (commit https://github.com/AElfProject/AElf/commit/affcd4ec58 by @tomatopunk)


add optional keyword for VNI

for unset values you can read https://google.aip.dev/149
instead of using 0 value which is legal...

see api changes https://github.com/opiproject/opi-api/pull/325


Local image rendering

Local image rendering may only be possible when run in production build.
Production build:

Development build:

Do not render local image


fix(#minor); uniswap forks; Set rewards related fields to not populate unless staking is enabled.

Context
rewardTokenEmissionAmount, rewardTokenEmissionsUSD, and stakedOutputTokenAmount are optional fields in the DEX schema. Currently, these fields are being returned with values, regardless of whether or not the LP tokens minted from these pools can be used to earn rewards.
Problem
We want these fields to be equal to null when these LP tokens cannot be used to earn rewards.
Proposal
Removed the instantiation of these fields at the time of creation of the liquidity pool entity. Then, these fields will only be populated if there is an analogue MasterChef pool for the LP token.

Looks good to me.
Curious about the purpose of the change? :P

The purpose of the change was to make sure that the fields related to earning rewards from LP staking are null when the LP tokens for that pool are not and never have been stakeable for rewards. These fields are rewardTokenEmissionsAmount, rewardTokenEmissionsUSD, stakedOutputTokenAmount, and rewardToken. So unless there is (or has been) an analogue masterchef pool for the liquidity pool, these fields will be null.

Looks good to me.
Curious about the purpose of the change? :P

@steegecs as more people start to contribute, and the code gets more complex, it'll be valuable to have a more detailed description on the purpose of each PR. @jaimehgb sets a really good example here that you can follow: https://github.com/messari/subgraphs/pull/722

Related issue: https://github.com/messari/subgraphs/issues/740

Great work!


Login using credentials from IVLE

It would be good to have our services authenticated by IVLE.
One way to go about doing this is to have the frontend call a URL from IVLE to get a session token. This token can then be stored in the state and used to verify whether a user is logged in.

Worked on this (79997d9). Current progress:

Login Component has a button that calls IVLE with a redirect URL

To get it to work go into /mocks/IVLE_details.ts and add in your IVLE API Key obtained from  https://ivle.nus.edu.sg/LAPI/default.aspx
The callback URL is also in the same file /mocks/IVLE_details

Once there is a session token, an action will be dispatched and it will be stored in the state.

Some changes I have done.

I have changed tslint to allow me to do console logging.
I moved history out to a seperate file so that I can do history.push('/device') for example

Some things to think about:

The state gets erased when u refresh or go back to the same URL. The token does not persist?
Should the session token be stored in the frontend?
Currently there is a saga configured that does the history push. The saga can also call the backend to authenticate the user (whether he is taking CS1101S and his details). Question is should the backend handle all the IVLE calling or should it be done straight from the client-side?

@remo5000 @ningyuansg @evansb for your kind review and advice.

I have changed tslint to allow me to do console logging.

We prefer to disable console.logs in tslint. We use the react/redux dev tools for chrome, @remo5000 uses his IDE, and I work with alert instead (they pause execution, and are conveniently allowed in tslint). No issues with you using whatever you need to develop, of course, but preferably let's try not to modify tslint for that purpose. Would recommend you use something like git's assume-unchanged for that.

I moved history out to a seperate file so that I can do history.push('/device') for example

Are there issues with importing into a ts file from a tsx file? I'm actually not very sure, but would it be sufficient to export the history object and import it where you need it? Nonetheless, I think history.ts (with the 'File' suffix) is a sufficiently descriptive name.

The state gets erased when u refresh or go back to the same URL. The token does not persist?

Probably need cookies for that. They're not yet implemented, but they're definitely on the list. Basically, we haven't had a recent to use cookies so far (but persisting the auth token is a good one).

Should the session token be stored in the frontend?

I don't think there's any way to avoid storing some form of token/key to identify the user, on the client. If you mean our API token to IVLE, I recall a meeting where @evansb said it's okay. If you are talking about the user token, do see #53. Obfuscation of the session token would be a secondary feature IMO, since if a user shares their session token or tries to retrieve someone else's token from their browser, it's really on them (morally speaking)---kind of like sharing or trying to steal a password
Perhaps you could elaborate on what you mean?

Currently there is a saga configured that does the history push. The saga can also call the backend to authenticate the user (whether he is taking CS1101S and his details). Question is should the backend handle all the IVLE calling or should it be done straight from the client-side?

Admittedly, I'm not very care on the details either. here's what I think happens.

User clicks login on frontend.
Frontend redirects user to IVLE login, together with our API token. Redirect is within the same tab.
Because IVLE recognises our API token, they render an appropriate login page with a callback url pointing to our frontend. User logins via this IVLE login page. IVLE redirects them via said callback url.
Frontend receives the callback url with some kind of IVLE identification token attached. Frontend sends the token to Backend.
Backend receives token. Fetches the relevant information (missions, status, etc.), gives it back to the Frontend.
Frontend receives this information, stores it in the redux store, renders the relevant information on the displayed component.

Again, not very certain on the details. Tagging @indocomsoft and @sreycodes  for confirmation.

@rrtheonlyone I checked out the branch, and was able to alter the state using a url param, great to see that in place 👍
A few queries:

Is actions/login a placeholder?
Why is pushing to the history necessary, if IVLE is going to cause a redirect anyways (causing an application refresh)?

@ningyuansg and @remo5000  thanks for the comments.
At @ningyuansg :

I have actually relooked the history part and realised I could just export it out straight from index.ts itself. So there is no need for the separate file!
I have reversed the changes to tslint.
I dont think there is a difference between session token and user token. IVLE provides a session token once the user authenticates himself. Unless, there is another token? Yup I agree with your point about obfuscation and storing on client side.

At @remo5000:

Actions/Login was something I was experimenting with earlier. Removed it as well.
The push to history is necessary because once IVLE redirects, it will append the session token to the url. So it will be for example, localhost:3000/device?token=1241515... I want to remove the token and restore the url back to localhost:3000/device.

I have updated my branch. Do let me know if there is anything else.
I will look into improving this and integrating with the backend as well.

@rrtheonlyone I checked out the latest, looking good! Could you make this a PR?
Also, the IVLE key you used is a test key right?


Compile.py script not working on Windows

When running the compile.py script on Windows the following error is thrown:
Detected missing NPM dependencies as ./node_modules is missing, installing them...
 .. npm install
Traceback (most recent call last):
  File "compile.py", line 704, in <module>
    install_dependencies(prefix=" .. ")
  File "compile.py", line 580, in install_dependencies
    assert execute_command("npm", "install", prefix=prefix)
  File "compile.py", line 56, in execute_command
    return execute_piped_commands(command, **kwargs)
  File "compile.py", line 83, in execute_piped_commands
    last_process = subprocess.Popen(command, stdout=pipe_out)
  File "C:\Users\Buchanan\AppData\Local\Programs\Python\Python37-32\lib\subprocess.py", line 775, in __init__
    restore_signals, start_new_session)
  File "C:\Users\Buchanan\AppData\Local\Programs\Python\Python37-32\lib\subprocess.py", line 1178, in _execute_child
    startupinfo)
FileNotFoundError: [WinError 2] The system cannot find the file specified

This has been added as of 8602969, thanks @Phyrik!

This has been added as of 8602969, thanks @Phyrik!


Modify min and max at once for Bounds

By adding an [] accessor, we should be able to modify Bounds like so:
x_bounds = BoundsOption([bound_min]*n_q, [bound_max]*n_q)
x_bounds[:n_q, 0] = new_equality_constraint

instead of
x_bounds = BoundsOption([bound_min]*n_q, [bound_max]*n_q)
x_bounds.min[:n_q, 0] = new_equality_constraint
x_bounds.max[:n_q, 0] = new_equality_constraint

Working on it.

This PR #190 solves this

Done! GG


Middleware should be configurable

// config file

var serviceHost = require('service-host');

module.exports = {
  middleware: [
    serviceHost.prototype.requestLogMiddleware,
    [someMiddleware],
    serviceHost.prototype.serviceMiddleware,
    [someOtherMiddleware]
  ];
};

TODO

Roll all the service middleware into a single one
middleware should be configurable to only serve from particular endpoints, eg: ['/some-endpoint', someMiddleware, someOtherMiddleware]

Might consider this at some point later. The original need has passed, and I'd rather not hack something together without a yardstick.


Support for render elements (diffuse, reflection, SSS2, material select etc)

Right now the shader doesn't support any render elements; it should support the standard ones expected from V-Ray materials.

This is more or less complete now.


Can't get initial whatsNewCollection to show again

What happened?
I have changed the version store to use user defaults and used that in the .environment call ... but the initial what's new that is created in the App class, never reappears.
.environment(\.whatsNew, .init(versionStore: userDefaultsWhatsNewVersionStore, whatsNewCollection: self))
Also tried to subclass the WhatsNewEnvironment ... but not clear how to pass in a new WhatsNew object, OR, at least then while testing, how to revert back to contents of an existing version to get the content to be correct before publishing.
Thanks !
What are the steps to reproduce?
Outlined above.
What is the expected behavior?
Should have the initial what's new content to reappear

#64 ?

Hi @pkasson,
The default behavior of the WhatsNewEnvironment in combination with a WhatsNewVersionStore is that a version is presented only once to a user.
Additionally, the default WhatsNewEnvironment includes the following fallback mechanism as described in the README:

[...] the WhatsNewEnvironment includes a fallback for patch versions. For example when a user installs version 1.0.1 and you only have declared a WhatsNew for version 1.0.0 the environment will automatically fallback to version 1.0.0 and present the WhatsNewView to the user if needed.

If you like to change this behavior please feel free to subclass the WhatsNewEnvironment or provide your own implementation of the WhatsNewVersionStore.
Alternatively, you can make use of the remove(presentedVersion:) function on an instance of UserDefaultsWhatsNewVersionStore as highlighted by @boehs


pip3 install azure

fixes #8

Azure made considerable changes in their SDK (they went from v2.1.0 to v12.0.0). We are tracking this issue with VECLOUD-799.
"pip install azure" installs all azure packages.  We are looking for a more surgical solution.  While v12.0.0 stabilizes, we are working on a fix to update the setup script to lock to the stable SDK version.

Resolved in v1.2.  The setup script now installs azure-storage-blob v2.1.0.


All Resource Event Sources can handle multiple seconday resources for a primary resources

@metacosm will review some additional parts, and javadoc. But conceptually can be reviewed.


Feature request: grouping messages by various criteria (sender, originating server, etc)

The use case is this: our development team shares a single instance of mailpit, but we each test different apps that are sending messages in, and when one developer goes in to view their messages, they'd rather not see messages from the other developer's testing efforts.  Sometimes, if only one developer is needing to test at a given time, they can check with the others and then delete all the older messages, giving themselves a clean slate, but if 2 people are testing at the same time this is not possible.
The idea would be to have a filter on the left side, that would dynamically group messages by different criteria, like sender email address, originating server, subject substring or other things that might be available in the headers of each mail, which would allow dev teams to partition their view of the inbox.  This would also be part of the url so one could bookmark their unique view to remove the other developer's messages.
Thanks!

Hi @wesyah234 . This sounds to me like something that can already be handled by existing tagging. You would either need to provide an X-Tags email header to auto-tag, or alternatively start your Mailpit instance with word/phrase matches defined.
Alternatively, you could just search for something specific (ie: the hostname) and bookmark the search URL.
Does this do what you need?

Wow, that's a cool feature, and I think it will definitely help us.  I'll play with it a bit.  I'll have to get my sysadmin to allow me to start and stop the instance to test things out with that --tag parameter but I think it might be useful for us.
Just trying out the basic search and then bookmark option. I had a few issues:

Can I do "OR" matching?
per this, Can I search ANY mail header? like (Received:) or just the ones listed (to,from,subject, etc)

Reason being: certain developers might have email coming from a certain server, identified by the Received: header, and this would be handy (along with OR capability) to be able to craft a search string specific for the messages that developer is interested in.

Mailpit uses SQLite for storage, and whilst SQLite is perfect for this use (small, embedded and relatively fast), it's not advanced like some Gmail/Google database backend, so there are some limits.

There is no OR functionality (that becomes very complicated to parse and database-heavy to search) - there is only include ("search this") and exclude (!"exclude this" or -"exclude this"). You can however specify tags multiple times in the Mailpit startup flags (--tag 'host1="<IP_ADDRESS>" host1="<IP_ADDRESS>"') to handle your "OR" situation as you would then just filter by that tag (this is the solution you are probably after).
The search does not include all mail headers (there are just too many which would clutter the database and pollute search results) ~ those prefixes you see there are the only ones stored & indexed. The fulltext search (ie: not prefixed by to: etc) also includes Reply-To and Return-Path, but not hostnames etc

The "word/phrase matches" used in --tag is a 1:1 match on the entire message including headers, so provided you know the hostname then you can tag it as it arrives, and I believe this is the solution you are after.
There is one "catch" you should be aware of though - whilst Mailpit does live load new messages (websockets) as they arrive, this does not apply to filtered views, only the inbox itself. Searches are (in comparison to the inbox) very heavy and slow, so if the server and the browser had to calculate whether each arriving message matches a current search it would cause all sorts of issues. The search needs to be re-run to load new matching messages, or the tag clicked again (tags are effectively just searches, but are much faster to look up than a regular search). I may at some point add live loading to tag views, however I can foresee all kinds of complications and performance issues, so I don't think this will happen any time soon, it at all.
I hope this helps....

Thank you very much for the detailed explanation of how this all works!
The live loading of filtered views would be really nice but we can deal with hitting refresh on the browser.
Being able to search on any header would be nice as well but we can look at using --tag for this.
OR searching would also be nice in the UI but we can also look at using --tag for this.
Thanks again for the help.

One more idea, maybe when you perform a search, it shows up on the left side like the tags do when you apply a tag. This would avoid the need to bookmark a search and be more consistent with the tag behavior.

I appreciate the ideas, but I believe this would be counterintuitive and not logical to the end user. Filtering by a tag is actually just the same as searching, except the tag is highlighted (active) in the side nav. So if I was to search for a tag and a particular word at the same time, with your logic both a tag would be highlighted as well as something else (eg "search results") which would only appear when searching.
I chose to follow the same UI logic as Gmail, and try to keep the UI as simple as possible.
To achieve what you are after (effectively a refresh of a search), you can just submit the search again as it is already filled in with what you are searching for.

I'm just thinking of something like below.  Just like every tag you add becomes a clickable quick action on the left, it would be helpful for every search you perform (maybe not "tag" based searches) also becomes a clickable quick action on the left.

Thank for the suggestion, but I do not think this is a feasible/good idea. The moment you start doing custom searches the search values can get very long, and there will be potentially a lot of them. This, together with the fact that one ultimately does a lot of searching without realising it, sometimes in combination with tags, I think it will clutter the UI for very little benefit.
What problem exactly are you trying to solve with your suggestion?


[BugFix] Abnormally exit in JAVA UDF call_function_in_pthread shows all-zeros of query_id and fragment_instance_id

Some methods' invocations of Java UADF  is off-loaded to other pthreads from bthread, in these off-load threads, query_id and fragment_instance_id is missed to set for the thread_local tls_thread_status, so when crashes happen in these off-load threads,  Abnormally exit print all-zeros query_id and fragment_instance_id, for an example:
query_id:00000000-0000-0000-0000-000000000000, fragment_instance:00000000-0000-0000-0000-000000000000

What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.0
[x] 2.5
[ ] 2.4
[ ] 2.3

prefer SCOPED_SET_TRACE_INFO

@mergifyio rebase

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


ffmpeg[all-gpl]:x64-windows Build of dependency "TensorFlow" fails

C:\Users\user\Code\vcpkg>vcpkg.exe install ffmpeg[all-gpl]:x64-windows
Computing installation plan...
The following packages will be built and installed:
    ffmpeg[all,all-gpl,ass,avcodec,avdevice,avfilter,avformat,avisynthplus,avresample,bzip2,core,dav1d,fontconfig,freetype,fribidi,gpl,iconv,ilbc,lzma,modplug,mp3lame,nvcodec,opencl,opengl,openh264,openjpeg,opus,postproc,sdl2,snappy,soxr,speex,ssh,swresample,swscale,tensorflow,tesseract,theora,vorbis,vpx,webp,x264,x265,xml2,zlib]:x64-windows -> 4.4
  * giflib[core]:x64-windows -> 5.1.4-6
  * leptonica[core]:x64-windows -> 1.80.0#2
  * libarchive[bzip2,core,libxml2,lz4,lzma,openssl,zstd]:x64-windows -> 3.4.3#6
  * libjpeg-turbo[core]:x64-windows -> 2.0.6
  * lz4[core]:x64-windows -> 1.9.3
  * openssl[core]:x64-windows -> 1.1.1k#1
  * tensorflow[core]:x64-windows -> 2.4.1
  * tesseract[core]:x64-windows -> 4.1.1#8
  * tiff[core]:x64-windows -> 4.1.0#2
  * x264[core]:x64-windows -> 157-303c484ec828ed0#15
  * x265[core]:x64-windows -> 3.4#4
  * xxhash[core]:x64-windows -> 0.8.0#1
  * zstd[core]:x64-windows -> 1.4.9
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...

-- Using msys root at C:/Users/user/Code/vcpkg/downloads/tools/msys2/b98a6bfb7dc2001a
-- Using cached C:/Users/user/Code/vcpkg/downloads/tensorflow-tensorflow-v2.4.1.tar.gz
-- Extracting source C:/Users/user/Code/vcpkg/downloads/tensorflow-tensorflow-v2.4.1.tar.gz
-- Applying patch C:/Users/user/Code/vcpkg/installed/x64-windows/share/tensorflow-common/fix-build-error.patch
-- Applying patch C:/Users/user/Code/vcpkg/installed/x64-windows/share/tensorflow-common/fix-windows-build.patch
-- Using source at C:/Users/user/Code/vcpkg/buildtrees/tensorflow/src/v2.4.1-81b5bb6ede.clean
-- Configuring TensorFlow (dbg)
-- Warning: Building TensorFlow can take an hour or more.
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:146 (message):
    Command failed: C:/Users/user/Code/vcpkg/downloads/tools/msys2/b98a6bfb7dc2001a/usr/bin/bash.exe --noprofile --norc -c "'C:/Users/user/Code/vcpkg/downloads/tools/bazel/3.7.0-windows/bazel.exe' build --verbose_failures --compilation_mode=dbg --features=fastbuild    --python_path='C:/Users/user/Code/vcpkg/downloads/tools/msys2/b98a6bfb7dc2001a/mingw64/bin/python3.exe' --define=no_tensorflow_py_deps=true ///tensorflow:tensorflow.dll ///tensorflow:install_headers"
    Working Directory: C:/Users/user/Code/vcpkg/buildtrees/tensorflow/x64-windows-dbg
    See logs for more information:
      C:\Users\user\Code\vcpkg\buildtrees\tensorflow\build-x64-windows-dbg-out.log
      C:\Users\user\Code\vcpkg\buildtrees\tensorflow\build-x64-windows-dbg-err.log

Call Stack (most recent call first):
  installed/x64-windows/share/tensorflow-common/tensorflow-common.cmake:222 (vcpkg_execute_build_process)
  ports/tensorflow/portfile.cmake:7 (include)
  scripts/ports.cmake:141 (include)

Error: Building package tensorflow:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: tensorflow:x64-windows
  Vcpkg version: 2021-05-05-9f849c4c43e50d1b16186ae76681c27b0c1be9d9

Additionally, attach any relevant sections from the log files above.

Please provide the following logs:
C:\Users\user\Code\vcpkg\buildtrees\tensorflow\build-x64-windows-dbg-out.log
C:\Users\user\Code\vcpkg\buildtrees\tensorflow\build-x64-windows-dbg-err.log
Thanks.

@NiQ1, Thanks for posting this issue, Please reopen this issue if it still be a problem for you.


Some pictures not uploading

I always get some 0 bytes files, that were wrongly uploaded. A simple troubleshoot at the early stage of development was adding a await asyncio.sleep(1) but looks that isent fixed yet

Removing await asyncio.sleep(1) seems to make it work better, as noted on that line, discord.py handles its own rate limits

Still getting weird 0-byte files, but they now should be more rare

They are now really rare, should I should close the issue and reopen it when it appears usually again

Looks like its a device/connectivity related issue, nothing I can do


feat: register db.sql metrics

Fixes https://github.com/TBD54566975/ftl/issues/2285
Register the standard otel database metrics using: https://github.com/XSAM/otelsql
Found call sites by grepping for sql.Open and skipping all test code.
Sample:
ScopeMetrics #2
ScopeMetrics SchemaURL:
InstrumentationScope github.com/XSAM/otelsql 0.32.0

Metric #0
Descriptor:
     -> Name: db.sql.latency
     -> Description: The latency of calls in milliseconds
     -> Unit: ms
     -> DataType: Histogram
     -> AggregationTemporality: Cumulative

HistogramDataPoints #0
Data point attributes:
     -> method: Str(sql.conn.begin_tx)
     -> status: Str(ok)
StartTimestamp: 2024-08-08 23:18:01.011762 +0000 UTC
Timestamp: 2024-08-08 23:18:11.012177 +0000 UTC
Count: 23
Sum: 17.082795
Min: 0.137584
Max: 3.233917

Looks like this is breaking tests right now because modulecontext's Build() calls internal/reflect's DeepCopy, which doesn't expect any values of kind=string that can't be cast to a string. The databases now include a bunch of metric.descOpt{...}s wrapping the string descriptions of each metric, so that's confusing the cloning. I'll pick this back up in the morning

The alias in question: https://github.com/open-telemetry/opentelemetry-go/blob/c609b12d9815bbad0810d67ee0bfcba0591138ce/metric/instrument.go#L39

New changes: 2 minor additions to internal/reflect/reflect.go:

Added support for string aliases. Before, this would cast the src string as long as its Kind was string, but that would cause a panic when the true type was an alias. In this case, it was metric.descOpt.
Added support for the list.List type, which is used by the global.meter type inside otelsql.otConnector. Before, DeepCopy would panic whenever a list.List was given as input with reflect.Set: value of type *list.List is not assignable to type *list.Element. This adds a special case for that type and performs a standard linked-list deep copy, which is easier than managing consistency across all the backpointers and parent pointers in that structure.


Plugin: plugin_name & camelCase

This might be a refactoring nuance, but is there any reason why plugin architecture uses plugin_name instead of a camelCased variable like: pluginName ?
Just seeing it as an issue in CI plugin testing shown [here] (https://codeclimate.com/github/absalomedia/grav-plugin-shoppingcart-pin/issues)

It's really just a coding standard we use. We follow PSR-1 and PSR-2 coding standards but variable names are not really covered.
We use camel case for methods and classes, but lower+underscored for variables.  This is really to more clearly differentiate them.
Andy

On Feb 28, 2016, at 8:03 PM, Lawrence Meckan<EMAIL_ADDRESS>wrote:
This might be a refactoring nuance, but is there any reason why plugin architecture uses plugin_name instead of a camelCased variable like: pluginName ?
Just seeing at as an issue in CI plugin testing shown here https://codeclimate.com/github/absalomedia/grav-plugin-shoppingcart-pin/issues
—
Reply to this email directly or view it on GitHub https://github.com/getgrav/grav/issues/707.


Update model-contract.md

Removed language duplicating that found in section 2.4.

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


Project Id should be taken from credentials.json

Is your feature request related to a problem? Please describe.
Yes. If just using
const credentials = require('./cred.json'); // cred from IAM service account key
new PubSub({credentials})

the following error will be thrown:
UnhandledPromiseRejectionWarning: Error: Unable to detect a Project Id in the current environment.
To learn more about authentication and Google APIs, visit:
https://cloud.google.com/docs/authentication/getting-started
    at /home/hb/Dev/Contador-de-Clientes-Firebase/admin/node_modules/google-auth-library/build/src/auth/googleauth.js:87:31
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:216511) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:216511) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Describe the solution you'd like
It should use the project_id from the service account credentials.json
Describe alternatives you've considered
I am, as workaround, using credentials.project_id

@SrBrahma Thanks for the request! I think the generally recommended way of doing this is to feed in the path to the credentials file by way of an environment variable (GOOGLE_APPLICATION_CREDENTIALS). I'm curious to hear your use case for loading it by way of require() though.

Writing a secret to disk (even if it's ephemeral) feels like a bad move compared to having it sit in an environment variable along with all your other secrets, no?

I wonder if there has been a change in recommendation since this issue first opened. Last I heard, using a service key file was the right way to do it, but that may not be needed in e.g. GKE. I know with a recent test project I was able to use gcloud auth login and then kubectl apply to get a container running with access to my project.
I don't think many people are around today but I can post the question to see if anyone knows more.

This is probably most up to date: https://cloud.google.com/docs/authentication
I guess the answer is that it depends on where your services are hosted and what sort of access they need to other services.

There hasn't been much more traffic on this one (and I think the answers above are the right path), so I'm going to close it out.


WIG template should not open IntelliFactory.WebSharper

This brings in the ? operator that accesses JavaScript object fields, overriding the ? from WIG that defines a name for a method argument.

Bitbucket: https://bitbucket.org/intellifactory/websharper/issue/303
Originally reported by: Loic Denuziere
Originally created at: 2014-11-28T20:36:15.780

This is not problematic anymore.


add sqlite functionality

add a functionality to store the data next to json also in a sqlite database.

started with renaming keys in dicts() to single strings, but after this the import of all dicts into the sqllite database did not work.


vscode always keep a useless virtual env and can NOT remove it!

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

VS Code Version:
OS Version:

Steps to Reproduce:

1.the virtual env "paddle" was useless and I had removed it ,but it just keep stay in there
2.

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Add basedpyright

https://pypi.org/project/basedpyright/
Basedpyright is a static type checker for Python that is built on top of the work done by the pyright project
https://github.com/neovim/nvim-lspconfig/pull/3078

i missed that there's already a PR: https://github.com/mason-org/mason-registry/pull/4989


missing some fields for railway preset

https://wiki.openstreetmap.org/wiki/Tag:railway%3Drail
usage=#
frequency=# (with electrified=contact_line/rail)
voltage=# (with electrified=contact_line/rail)

Plus voltage when selecting electrified=contact_line/rail in the dropdown and also frequency for electrified=contact_line

Also frequency and voltage fields should appear when selecting electrified=contact_line/rail in the dropdown

Thanks for more information

Also frequency and voltage fields should appear when selecting electrified=contact_line/rail in the dropdown

We don't currently support having fields appear and disappear depending on values entered
(Except for certain cases like the structure bridge=*/tunnel=*/etc field, because those are popular and worth adding the custom code to do it).

Is there a lot of work involved in adding the custom code for electrification?

Is there a lot of work involved in adding the custom code for electrification?

It would not be a lot of work to have the fields show up all the time and I think it's ok to expect the user to know whether to use them.


Updating root readme to refer to github.io docs rather than relative readme location

If you navigate to the .net doc site the links out to the readme for each specific package are broken. Check the "readme" links for each July 2019 preview package. The urls are rendering incorrectly because the actual readme.md  has relative links vs  github.io paths that will actually work.
FYI
@chidozieononiwu
@weshaggard
@ramya-rao-a
@Azure/azure-sdk-eng

@scbedd what triggered this update? Shouldn't we be linking to things in the repo? Or is it explicitly because this is trying to reference the July release? We also have the option to directly link to the release tags for the given packages.

It's specific for the July release. @ramya-rao-a pinged me with a "are you aware that the links on github.io are broken?"
Each link is horked on the github.io because it attempts to use those relative links. The code path to the relative readme in the repo is is NOT the same as the path on github.io.

I see, we are blindly using the contents of this readme.md on the github.io pages. There are other links that are broken in that context as well and having links to the github.io package seems a little odd when viewing it from the repo itself. We may have to think a little more about the purpose of this file to see what makes the most sense.

We could also consider turning the relative links into fully resolved links as part of the publishing process.

Gotta remember that we don't show the other readmes on the site anymore, so I'm not certain I completely agree when we talk about other broken links in the repo.
That being said, I agree about the generic discussion you're asking for. I'll swing by to talk with you and Dozie to chat and get some issues out of it.

Ha! Just on that one readme we still have broken links.


Weekly reviews every week in the DevPool

Discussed in https://github.com/ubiquity/bounty-bot/discussions/1

Originally posted by pavlovcik September 10, 2022
Communication

[ ] Weekly reviews every week in the DevPool

what-the-diff supports slack but if we can make it work with telegram that would be far more crypto native.
Posting this weekly update in the devpool along with the new bounties would be a huge luxury.

We closed seven pull requests last week! 🚀
This week, we added more tests for the ERC20Ubiquity contract, fixed a bug that was causing incorrect balances to be displayed in the footer, added spell checking for dapp and contract scripts, added a custom 404 page, implemented the ERC-2535 Diamond standard, fixed a bunch of Slither issues, adapted the contract to simplify UBQ farming, and fixed a bunch of warnings.

How to solve

[ ] database scheme and adapter needs to be completed prior to any analytics features
[ ] Telegram bot adapter should be completed prior to this issue.
[ ] Total closed issues, merged pull requests, the number of new bounties released, total payouts, etc. These information needs to be collected once a week and create a proper description for weekly updates with collected information

I'm not sure how easy it is to roll our GPT model to handle this vs trying to build off of what-the-diff. A bit of research is needed here.

Depends on #47 #54
Made it a week to figure out how to get chatgpt or gpt3 to summarize info from the GitHub in the previous week

I'm not sure how easy it is to roll our GPT model to handle this vs trying to build off of what-the-diff. A bit of research is needed here.

Stumbled upon this https://github.com/ubiquity/ubiquity-dollar/discussions/519


Mismatch between the Pad documentation and actual implementation

Hello,
When I read the documentation for the Pad transformation it clearly states that the pad argument can be a sequence. After trying all kind of types(numpy array, python list, python tuples) and constantly receiving assertion errors I decided to look at the code. It seems that the only allowed type is numbers and not sequences. Is the documentation is wrong or did I missed something ?
Thank you

Hi @GuillaumeLeclerc
Can you post a minimum working example showing this error, the following works for me (from master):
from PIL import Image
from torchvision.transforms import Pad
im = Image.open('tmp.jpg')
p = Pad((2,3, 4, 5))
out = p(im)

Granted, the docs say it accepts sequences, but it only accepts tuples for now. I will send a PR to fix this and make the docs a bit clearer.

@fmassa this can be closed now as #239 is merged

I guess the issue was that I was not running the master branch (got it from pip). On my local version I had:
assert isinstance(padding, numbers.Number)

and nothing else. So I was getting and assertion error. I will try to be more careful next time

@GuillaumeLeclerc we will be cutting a new release soon so that installing from pip won't have this problem. Thanks for opening the issue!


ION Spec

Which ION spec does this conform to? Would be helpful if the README mentions this.

You should use RION Ops instead. This repo is dead.
https://github.com/nanosai/rion-ops-java


event marshal JSON

moved escapeString into helpers.go and changed the MarshalJSON method for event to use the correct string escaping

this should end the saga of #28

So you're doing an entirely custom JSON marshaler? Can you remove fastjson then?

fastjson is still used for the fast unmarshaling. The problem with fastjson is that it uses strconv.Quote when escaping strings which does not seem to follow the rules of RFC8259, and was leading to signature verification problems with certain characters. Fastjson does seem to have a RFC8259 compliant string "unescaping" function.
I tried to write my own JSON unmarshaler, but it was much slower than the current one using fastjson...

hooray!


Adding team process around operating hubs

This PR begins adding process around how we onboard new hubs, keep track of running hubs, and designate tasks associated with operating them over time.
To Do

[x] Does the "one issue per hub" approach seem reasonable?

Yes, going to use one issue per hub, along with issues for tasks and goals

[x] How should we utilize project boards? (if at all? I have one proposal in this PR)

Not going to use project boards for now.

[x] How should we use labels to keep track of hub issues?

We will use goal to keep track of longer-term goals. These will have multiple tasks / links to other issues inside them.

[x] We need to define a process of triaging and assigning issues to team members in a way that is efficient and fair!

For now we will use the team sync to keep track of who is working on what, and coordinate ourselves via conversations in the issues. If this doesn't seem to be working we will work out something more structured.

cc @yuvipanda and @GeorgianaElena , would love to have your feedback on this (@consideRatio as well if he is interested!)
closes #220
Related issue: https://github.com/2i2c-org/team-compass/pull/23

This is a great start, @choldgraf! Let's do this and see how it goes?

Hey @yuvipanda - thanks for these comments! I made a few modifications here and I'm curious what you think about them. Here are the main updates:

I've added two new issue templates: one for tasks, and one for goals.
Tasks are actionable and concrete, goals are as you describe in the HackMD
I suggested that we have one issue per task and goal
We self-assign to tasks or goals that we plan to work on
During our bi-weekly team syncs, we provide major updates about completed tasks/goals etc, and note which ones we'd like to focus on next

This also removes any use of project boards etc.
I suggest we move much of this language over here: https://2i2c.org/team-compass/team/tech/coordination/ and keep the pilot-hubs repository focused more around the mechanics of operating the hubs (e.g. "open an issue, do this, do that"), rather than these high-level descriptions of team process (about the team syncs and such).
What do you think?

I LOVE THE GOALS TEMPLATE!
To begin with, how about we just use comments in the goal issue for tasks that don't belong in any other particular issue / PR? I think most of the work to accomplish a goal will be in other specific issues, so we can just link to them than have a (possibly duplicate) task. We can reconsider this at a defined checkpoint?

Alrighty @yuvipanda I removed tasks, and opened up https://github.com/2i2c-org/team-compass/pull/33 to track the "team process" stuff

I like this as is, @choldgraf! Gonna merge, we can iterate as we go.

@GeorgianaElena thanks for the suggestions! I've updated the comments to use HTML comments instead, as the <> was introducing inconsistent behavior (I think that it was interpreting the <> as HTML tags). I also consolidated the last two sections per your suggestion.

\o/ This looks great. One question - the 'new hub issue template' is meant to be used by 2i2c engineers, right? Since we can't expect end users to know what clusters we have, for example.

yes 100% re: engineers, this is just so that we can standardize our own processes around keep track of the hubs

@GeorgianaElena wanna merge when you are happy?


[BUG] Better error message when Coin doesn't exist in the pair vault

When user trying to send out fund from a vault , if the sending coin doesn't exist in the pair device's vault,  currently it show "Wrong Vault or Pair Device."
We need to update it to say "Please add coin "xxx" into your vault" , or something like that
Previously , we put all coin's information in the Qr code , which means the pair vault don't need to add the coin , but recently we update it to using protobuf , then we need to match coin based on the information in proto, thus it is required to have the coin added to vault already

fixed #898


MQTT - Sleep - clean session, subscribing to retained sleep message when waking

I thought I might be able to send a retained message to the command topic: sleep 3600, and the Nat would sleep, wake hourly and then subscribe to the retained message and sleep again.
I cant get that to work, I tried setting QOS to 1 as well.
Im wondering if the NAT MQTT client is connecting in clean session mode? and therefor not subscribing to the retained messages?
Is there a way to change this, did I miss something?
So now I want the Nat Repeaters to sleep all night, so I have to keen sending sleep 3600 very hour though the night?
P

The client connect with clean session. You can change that in the sources in user/user_main.c#L4511

The client connect with clean session. You can change that in the sources in user/user_main.c#L4511

Thank-you Martin for the help with this information.
Since I am not setup for making builds, I made a workaround automation in Home Assistant, whereby at night HA sensor listens to the LWT topic and when it goes online, HA publishes sleep 3900 to the command topic.
3900 ( not 2600) is like an hour on my nodeMCU for some reason.
Worth noting that the MQTT timeout is 3 mins, so it take 3 minutes to show as offline in HA.
All good, its working very well!
P


Set-up continuous integration and nuget deployment

This task is to track the set-up of AppVeyor to:

Build all branches and pull requests on push/change
Run all tests
Block pull requests from being merged unless all tests pass (webhook provided by GitHub+Appveyor)
Manually publish the nuget package (issue #17 and pull request #22) to nuget.org from AppVeyor with a single click.
Add a build status badge to the GitHub page (readme)

The reason for manual nuget package push is to be able to push a version when it's deemed ready. for nuget.
@JonathanWilkins  Are you happy with this scope? Would it be possible to temporarily make me an admin of the GitHub repository, so I can set this up? Or we can pair or something? (I've done this before for some of my repo's in GitHub)

All happy with that.

I have added you so if you want to take a look.

I have mostly finished this now - you can see it in action in here: #25

https://github.com/machine-visualstudio/machine.vstestadapter/wiki/Continuous-Integration-and-Release-Process


BUG: Cannot find module '@ngx-tonysamperi/dom'

Describe the bug
When using this library, by npm install @popperjs/core ngx-popperjs --save, @ngx-tonysamperi/dom is not installed as it is not marked as a peer dependency.
Therefore, I get an error when consuming NgxPopperjsModule:
Cannot find module '@ngx-tonysamperi/dom' from '../../../../node_modules/ngx-popperjs/fesm2020/ngx-popperjs.mjs'

Workaround:
Can be solved by running npm i @ngx-tonysamperi/dom --save
To Reproduce

Run npm install @popperjs/core ngx-popperjs --save
Either:

Check node_modules: No @ngx-tonysamperi/dom available
Consume NgxPopperjsModule anywhere in your app

Expected behavior
ngx-tonysamperi/dom should be installed as dependency as well when running npm install @popperjs/core ngx-popperjs --save
Versions

ngx-popperjs [14.0.0]
@angular/core version [14.0.3]
@angular/material version [14.0.3]
typescript version [4.7.4]

Additional context
Seems like I'm the only one with this issue, maybe there is something that I'm missing?

Before dom there was smp-core-utils and it's never been marked as dependency, but peer.

@habtim
I moved @ngx-tonysamperi/dom and @popperjs/core to dependencies, so that installing ngx-popperjs will automatically get everything!
v 13.3.0


App crashed when moving window during sync

while the app was syncing, I moved the window on the screen and it became unresponsive (Defi Wallet not responding) then it closed. After reopening it the database became corrupt :cry:
I can attach the log if it's needed.

Hi @nippur72 kindly check if you turned off your firewall / anti-virus. Windows tend to have Anti Virus installed, the Anti Virus hooks into file system natively for real time detection which is a common cause of file related corruption issues. If your issue continues to persist, we can further discuss on the issue here Unable to Start Blockchain. We would like to keep same issues in one thread. Thank you.


[Bug 550] Optimized loadedLibraries reducer

Prevented unnecessary updating of context during AdvancedMarkers rendering

Awesome, thanks a lot for spotting and fixing this. I wasn't even aware that there was a significant difference in the code-path taken between the render and createRoot approaches.


Shadowing command with alias as a default configuration

Describe the problem you are trying to solve
I often run the same commands over and over with the same set of flags (--all-features, --all-targets, --deny warnings). Then I found out about the .cargo/config and the [alias] section. So I thought about doing a recursive alias (like I would do with aliases in a shell).
[alias]
clippy = "clippy --workspace --all-features --all-targets -- --deny warnings"

But this is not possible as it is shadowing the existing clippy command. Actually, it might even be a bug because this is what I get when I'm trying to do that.
$> cargo clippy

thread 'main' has overflowed its stack
fatal runtime error: stack overflow
[1]    2198266 abort (core dumped)  cargo clippy

Describe the solution you'd like
Like aliases in a shell, I'd expect the alias defined above to just work when typing cargo clippy.
One nice advantage would be that all the project's contributors would follow the same configuration for the project (including the CI).
Notes
I'm throwing the idea but I'm also aware of the weirdness in a recursive definition. It might not be desirable.
Another possible drawback: it might be harder to use the original clippy command. For example, in shell, if we had an alias ls="ls -l", we could use the original command (and therefore deactivate the -l option) by doing /usr/bin/ls. Not sure how to reproduce a similar behavior with cargo.
Related issues
https://github.com/rust-lang/cargo/issues/2901

@rustbot claim

Closing this as resolved by #9791.


Create onboarding documentation for test automation for plugins

Add technical documentation for repositories to help onboard to test automation.

[x] #224

[ ] Test Orchestration Layer
[ ] Performance Tests
[ ] Integration Tests

[x] #223

@setiah Can you update the status of this issue? Are you working on creating the needed documentation?

@setiah Are you actively working on this?

[triage] Looks like tasks from the issue description are complete. closing the issue. Feel free to reopen with more information.
Thanks!