V3 Intellisense not working correctly

For what version of Nuxt UI are you asking this question?
v3-alpha
Description
I'm experience problems with the VS Code Tailwind Intellisense plugin. I have used ui v2 without problems so far, but when using v3 it only shows suggestions when a tailwindcss.config.ts is freshly edited, but then after reloading or restarting VS Code, it doesn't work anymore. When it's working, it only shows a few of the classes, but not the most common ones like padding, margin etc..
Has someone experienced the same issue and knows a fix for that?

To add to this, When I deploy my app with Vercel. I have these warnings with Nuxt UI 3 and tailwind 4.

When I build the app locally It works but I'm not sure what the issue might be.
Intellisense not working for me too btw

I found  the v3 version currently doesn“t have the "@nuxtjs/tailwindcss" module(which would generate a tailwind configuration file in the .nuxt folder), so I think we need to manually configure tailwind for now.
It works for me  after I added a css file in the assets folder with the content
@tailwind base;
@tailwind components;
@tailwind utilities;

Although the extension works without needing to add the tailwind configuration file,this doesn't align with my expectations.

based on this blog,I made some small adjustments
assets/global.css
@import "tailwindcss";
@import "@nuxt/ui";

nuxt.config.ts
export default defineNuxtConfig({
  css<EMAIL_ADDRESS>})

@metkm that screenshot is a different issue to this one, if you are still experiencing it you could open up a new issue with a reproduction. Although I'm currently not able to reproduce it.
In regard of this Intellisense issue:
As @lilijh pointed out, and as it is described in UIv3 docs, you have to add both @import "tailwindcss"; and @import "@nuxt/ui"; either in your css files or more simply in your app.vue.
Tailwind v4 now has a css-first approach, so the tailwind.config wont be used.
Also, as stated in the already linked Tailwind v4's announcement, you might have to switch to the pre-release version of their VS Code extension for proper Intellisense (although, once you add the css imports, even the current extension release should work).
Last note: one of the reasons why @nuxtjs/tailwindcss dependency has been removed is to make this module compatible with Vue projects https://github.com/nuxt/ui/pull/2416

I'm currently not able to reproduce it.

I have this issue only on vercel. Did you use vercel too?

I have this issue only on vercel. Did you use vercel too? @sandros94

Ok no, my bad. Double checking my deployments I was in fact NOT deploying on Vercel.
Could you please open up a new issue with a reproduction link?

I did several clean installs. I find this here in the output for the Tailwind CSS Intellisense pre-release plugin:
Locating server…
Booting server...
Checking if /Users/silvio/uitest2/.nuxt/nuxt-fonts-global.css may be Tailwind-related…
Setting up server…
Listening for messages…
Searching for Tailwind CSS projects in the workspace's folders.
[Global] Creating projects: []
[Global] Preparing projects...
[Global] Initializing projects...
[Info  - 14:44:09] [GLOBAL] No matching project for document {
  fsPath: '/Users/silvio/uitest2/app.vue',
  normalPath: '/Users/silvio/uitest2/app.vue'
}
[Global] Initialized 0 projects
[Info  - 14:48:32] [GLOBAL] No matching project for document {
  fsPath: '/Users/silvio/uitest2/app.vue',
  normalPath: '/Users/silvio/uitest2/app.vue'
}

Tailwind itself works. So any classes I use get applied. Only the intellisense is not working.
Anyone an idea what could be wrong?

@silvio-e could you try and disable all VS Code extensions and only reactivate Tailwind's one? I'm starting to suspect there might be a simple conflict between them

I can confirm this is an issue with Nuxt UI Module, I created fresh projects with
1 - Nuxt - Nuxt UI Module - intellisense does not work ❌
2 - Nuxt - Nuxt Tailwind Module - intellisense works ✅
3. NextJS - intellisense works ✅
4. Astro - intellisense works ✅
5. Vue Vite - intellisense works ✅

1 - Nuxt - Nuxt UI Module - intellisense does not work ❌
2 - Nuxt - Nuxt Tailwind Module - intellisense works ✅

Just to make sure I'm on the same page as yours: 1. is referring to Nuxt UI v3 (that uses Tailwind CSS V4) while 2. is referring to<EMAIL_ADDRESS>(that uses Tailwind CSS v3), right? What about others points?
Just in case, I would like to remember everyone that UI v3 uses an alpha release of Tailwind CSS v4. As such I would need your help with as many details as possible (OS, IDE, installed Tailwind CSS Extension, versions), or even better via a link with a full reproduction.
So far I'm struggling to pinpoint the issue (although, it happened myself once too, but haven't been able to reproduce since). I would also like not to open up a new upstream issue until I'm objectively sure about it, since they do already have a number of issues on this topic without a tangible pattern.

Closing this as the issue lies on where you define the @import "tailwindcss";. It won't work well when doing it in app.vue but without issue in a CSS file. Added this to the docs:


P3: Refactor synchronize function

Description: Refactor the synchronize function from the file synchronize.py file in accordance to the previous refactor plan found in the report branch - report/Refactoring-plans/Refactor-plan-function-synchronize.md.
DoD: Code is refactored, CCN complexity is reduced according to the refactor plan and changes are merged to refactor branch
Tests: No tests

Merged to refactor branch


[boost-regex] Build error

I don't know how to handle this problem. Please help

install-x64-windows-dbg-out.log

Duplicated of https://github.com/microsoft/vcpkg/issues/24473, https://github.com/microsoft/vcpkg/issues/26850. Did you install anti-virus software on your machine?

Yes,i did,thank you for your help.What should i do?
原始邮件
发件人："Lily Wang"< @.*** >;
发件时间：2023/11/8 11:02
收件人："microsoft/vcpkg"< @.*** >;
抄送人："dailuo124"< @.*** >;"Author"< @.*** >;
主题：Re: [microsoft/vcpkg] [boost-regex] Build error (Issue #34959)
Duplicated of #24473, #26850. Did you install anti-virus software on your machine?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


Definition conflicts with previous value (iOS)

Hello, I encounter a problem after install LiveKit and running the app.
Xcode Version 12.3 (12C33)
LiveKit main branch

Hi @HeroSony ! I will check it out in a few hours. 😁

@HeroSony I confirmed it compiles with Xcode Version 12.5.1 (12E507), so Swift ver should be 5.4.2.
Maybe this is due to some changes in Swift but I'm not sure yet. Xcode 13 is available now, can you try updating ?
Can you try if the test app compiles https://github.com/livekit/ios-test-apps ?

@hiroshihorie I shall update Xcode to the latest and try compiling again. Letting you know afterward. Thanks for your confirmation.

Hi @hiroshihorie, Is there any updates? have the same issue on Xcode Version 12.4 (12D4e)

@Zharas0904 Hi, may i know which issue exactly, "Definition conflicts with previous...." ?

Hi @hiroshihorie, yeap, exactly the same issue "Definition conflicts with previous value", with value name selectedDimension

Hi @Zharas0904 I will look into this today. I might need to get lower versions of Xcode to reproduce this error.

Hi @hiroshihorie, how you doing? is there any updates? Did you solve this issue?

Hi @Zharas0904 , I'm struggling to reproduce it. But I did release a new v0.9.0. Could you try if that works ?

Hi @hiroshihorie, yeah, sure we are using Xcode Version 12.4 (12D4e)  and Swift 5

Ok thanks for the detailed feedback will dig into it now

Hello can you try v0.9.1

Hi @hiroshihorie, sorry for late reply, compiling on Xcode 12.4 works fine, thanks for the prompt solution of the problem!)
... but we found new issue:
Previously we've started implementing LiveKit delegate methods, all methods called/worked fine on package version v0.8.2 and we received all call related events, but after updating package version to v0.9.2 we are not getting any events (except didConnect)
Could you, please, check your last updates? maybe you missed something or forgot to change dependency versions?

Hi @Zharas0904 , I will take a look but
It sounds like your object is getting deallocated. Can you make sure your object has strong reference somewhere, since Room.add(delegate:) keeps weak reference only.

closing this issue as the original problem has been resolved. @Zharas0904 please find us on slack if there are other issues that you need help with.


Disable inverse foreign keys in maven

Hello,
I'm using querydsl sql with an oracle database (11gR2) and one thing I noticed, that does not happen with other databases, is that the code generation with inverse foreign keys takes a long time (several minutes). This is a known issue with oracle (google: "oracle slow slow getExportedKeys").
As an experiment I commented the code that collects the inverse foreign keys and the time reduced dramatically.
Most of the time I do not use the inverse foreign keys. Is it possible to create a flag for maven plugin to not generate the inverse foreign keys?

exportForeignKeys to skip the foreign key export is already available. What kind of changes to you suggest to the configuration options: http://www.querydsl.com/static/querydsl/4.0.9/reference/html_single/#d0e716

I know about exportForeignKeys. But I don't want to disable all foreign keys.
I was referring to this
https://github.com/querydsl/querydsl/blob/master/querydsl-sql-codegen/src/main/java/com/querydsl/sql/codegen/MetaDataExporter.java#L362

Shall I add exportInverseForeignKeys as a boolean property?

To me that sounds good at least.

Yes please.
Sent from my iPhone

On 24/04/2016, at 10:06, Timo Westkämper<EMAIL_ADDRESS>wrote:
Shall I add exportInverseForeignKeys as a boolean property?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

is it also usable via the maven pluging cause there is only 3.7.4 available? Or is ther a trick somehow to use  a newer version width the plugin


[material-ui][docs] Add TemplateFrame to templates

[x] I have followed (at least) the PR section of the contributing guide.

Follow-up to #43396 based on https://github.com/mui/material-ui/pull/43396#issuecomment-2304549182
Fixes the layout and double scrollbars.
👉  https://deploy-preview-43406--material-ui.netlify.app/material-ui/getting-started/templates/

@aarongarciah what browser are you using? Tested on chrome here and couldn't reproduce it

@zanivan I'm seeing it in at least Chrome, Safari and Arc (which is basically Chrome) in macOS with scroll bars always visible.

https://deploy-preview-43406--material-ui.netlify.app/material-ui/getting-started/templates/marketing-page

I was recording a video to show that here it's not happening and a bugger bug showed up 😅  Do you have any idea on what might be happening?
https://github.com/user-attachments/assets/7b5cd3c6-f2d5-4aea-8331-276e02703ffa

I'll take a look next week 👀 To see the double scroll bar, you need to update this macOS setting to "Always":

@zanivan I need to look deeper but removing the footer fixes the double scroll bar and the issue you shared in the video (is the same issue).

Just found the culprit: the hidden label in the email input in the footer, which is absolute positioned, is causing the overflow.
https://github.com/user-attachments/assets/0f72efe8-89e5-40bb-bf5f-bf1e96db4388


A better thread-safe profiler constructor and dump profile method

A better thread-safe singleton Get function
Move DumpProfile out of destructor of engine
Fix operator name bug in bulk execution at graph executor

profiler is not in test suites. Did you test it manually? Could you add a test case?

yep, I've tested it manually. The problem is that I need to change the compile flag to compile with profiler for test case just like MKL. Also, the test case will fail in other build versions without profiler enabled. Do you have any suggestion?

we can change all test scripts to build with profiler on

um, should I add -j\$(nproc) in amalgamation build?


Implement caching layer

Design doc: https://docs.google.com/document/d/1P6boQoVya126qm0JI7oqyJ24bLdUw1cD7-BrGlxPuUs/edit
Changes:

[ ] Generate and store RequestHash field in Dynamo RequestTable
[ ] Implement cache hit path in query_runner.py
[ ] Update output matrix S3 prefix to include data_version and request_hash
[ ] Update GET endpoint to retrieve cached entries

DoD:

[ ] Requesting the same matrix multiple times (defined by resulting from the same cellkey set) does not generate a new matrix and returns a copy of an existing matrix
[ ] No regression in v0

This was postponed to Milestone 2. We need to buffer a bit more time for emerging issues such as https://humancellatlas.slack.com/archives/CC9AR11T7/p1563988186044800


[Tech]: Add a timer to reduce sms amount when user changes phone number

https://github.com/sora-xor/sora-card-android/assets/17094492/5544dac6-5153-4c9a-8a33-d3dce91e2438
Problem statement
User can spam with sms
Description
User can change phone number and send new sms
Definition of Done
User has some limitation per time period
Requirements
No response

#58


fix initial pickling on windows

from #84
The problem is that for Task I did not define getstate method. Because sometimes I want to really pass the full Task including the attributes that might contain functions, and sometimes I know I can leave these attributes out.
So the fix would be to define a Task.getstate that removes unsafe attributes by default, and figure out a way to create a pickle from an object ignoring its getstate! I am not sure how to do that. Maybe using cloudpickle will just solve the problem in a easier way.
I will take some time to experiment on that... meanwhile you can remove this change in UptodateCalculator (and let the test fail). This change will actually break the code if you have a getargs attribute in a DelayedTask (the tests doesnt cover that). So I will merge the improvement on not using wheels and fix this issue later. Of course you are welcome to try to fix it yourself :)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25432633-fix-initial-pickling-on-windows?utm_campaign=plugin&utm_content=tracker%2F1885485&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1885485&utm_medium=issues&utm_source=github).


Ready for prime time, or unmaintained?

Considering how this implementation is mentioned at the original simple-graph repo, is it correct that SQLiteGraph kinda depends on it?
Is the slow or halted development of this repo related to that?
Or do we simply have not the resources at the time, to update it?

is it correct that SQLiteGraph kinda depends on it?

No.  I simply borrowed/stole some ideas from there and wanted to give credit.

Is the slow or halted development of this repo related to that?

I implemented everything I had a use case for and then stopped.

Ready for prime time, or unmaintained?

I'd say neither but closer to unmaintained (I would certainly review any PRs however).  It follows semantic versioning (https://semver.org/#spec-item-4), so as a 0.X release nothing is stable.  I may have a reason to pick this up again soon, but that reason also may not happen.


Removed entity is blocking creating another entity with the same id

Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
I have two entity types with a shared interface, Split and User. When I go to create a new Split entity, I check to see if a User entity exists with the same id, and if yes I remove it first. I then create the split entity with that id. However, I am getting the following error:
Subgraph writer failed, error: tried to set entity of type 'Split' with ID "0xf8843981e7846945960f53243ca2fd42a579f719" but an entity of type 'User', which has an interface in common with 'Split', exists with the same ID

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
The error is occurring on this subgraph: https://thegraph.com/hosted-service/subgraph/0xsplits/splits-subgraph-kovan?version=pending
You can view the code here: https://github.com/0xSplits/splits-subgraph/blob/master/src/ethereum.mapping.ts#L99
A condensed version of it looks like:
export function handleCreateSplitCall(call: CreateSplitCall): void {
  let splitId = call.outputs.split.toHexString();

  let splitUserId = User.load(splitId);
  if (splitUserId) store.remove("User", splitId)

  let split = new Split(splitId);
  split.save();

What is the expected behavior?
I would expect the split to be successfully created since the user entity with the same id was removed before the split was saved.

It's a limitation in how we deal with ids for interfaces: we insist that once an id has been used, it's only ever used for the type it was initially used with (like User) in the example. I am not entirely sure about the background of that decision (@leoyvens might remember)
We could change it so that at any given block height the id must be unique across implementers of an interface, so that at block 10 the id 0x1 gives you a User, and at block 11 it gives you a Split. The example in this issue shows that there'll be some ordering concerns that must be taken into account so that the removal of the User happens before we check for id uniqueness.

@mihoward21 how important is it that you have a User and a Split with the same id? You could work around this issue by using different ids, and having an additional realID attribute; is that's part of the interface, you can query for that and get the right kind of entity back. Not super efficient, but would work.

@lutter yes I think that change could work for us. We will need to be a bit careful rolling out a change like that since it'll break current usage of our subgraph, might need to just create a new one.
If you all do think this an update you'll make at some point in the future to handle this case, that would be great. This issue isn't pressing for us. It's just an edge case caught in testing (it technically could happen at any time on mainnet, but it is a pretty rare situation for our data to get into, and as you pointed out we have a workaround if necessary).

Still relevant.


Co-rotational 2D Beam Element

@isaar/nl
Addition of co-rotational (non linear) beam element based on Crisfield's formulation
References:

Non-linear Finite Element Analysis of Solids and Structureg - M. A. Crisfield, R. De Borst, J. Remmers
2D Corotational Beam Formulation - L. L. Yaw
Co-rotational beam elements in instability problems - J.M. Battini

Please attach a PDF containing the appropriate theory, if possible.

What about 3D formulation?

3D corotational element


towards showing that extrinsic and intrinsic progress agree

I am also refactoring both versions of progress to make them easier to reason about

Not sure why I got a hydra failure related to playgrounds in one the above commits...

I am hoping to fix this along the way https://github.com/input-output-hk/plutus/issues/700 by making VTel a predicate over Tel.

Functions that use the with rule are hard to reason about. It's hard to abstract all the right things to make things compute. It encourages you to make large monolithic functions which require very large monolithic proofs to reason about them.


Theme Check plugin errors/warnings/notices

There is the WordPress Theme Check plugin which reports errors and warnings,
though some of them don't apply like "extraneous directories like .git or .svn".
REQUIRED: This theme doesn't seem to display tags. Modify it to display tags in appropriate locations.
REQUIRED: .sticky css class is needed in your theme css.
REQUIRED: .gallery-caption css class is needed in your theme css.
REQUIRED: .bypostauthor css class is needed in your theme css.
WARNING: Your theme appears to be in the wrong directory for the theme name. The directory name must match the slug of the theme. This theme's correct slug and text-domain is app.
(If this is a child theme, you can ignore this error.)
WARNING: Found PHP short tags in file templates/header.php.
Line 3: <a class='brand' href='<?= esc_url(home_url('/')); ?>'><?php bloginfo('name'); ?>
WARNING: Found PHP short tags in file templates/entry-meta.php.
Line 1: <time class='updated' datetime='<?= get_post_time('c', true); ?>'><?= get_the_date(); ?></time>
Line 2:
<?= __('By', 'sage'); ?> <a href='<?= get_author_posts_url(get_the_author_m
WARNING: file_get_contents was found in the file lib/assets.php File operations should use the WP_Filesystem methods instead of direct PHP filesystem calls.
Line 13: $this->manifest = json_decode(file_get_contents($manifest_path), true);
WARNING: .bowerrc .editorconfig .git .github .gitignore .jscsrc .jshintrc .travis.yml .gitkeep .bower.json .eslintrc.json .scss-lint.yml Hidden Files or Folders found.
REQUIRED: Please remove any extraneous directories like .git or .svn from the ZIP file before uploading it.
REQUIRED: No content width has been defined. Example:
if ( ! isset( $content_width ) ) $content_width = 900;
REQUIRED: Could not find add_theme_support( 'automatic-feed-links' ). See: add_theme_support
 <?php add_theme_support( $feature ); ?>
RECOMMENDED: Screenshot size should be 1200x900, to account for HiDPI displays. Any 4:3 image size is acceptable, but 1200x900 is preferred.
RECOMMENDED: No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.
RECOMMENDED: No reference to add_theme_support( "custom-header", $args ) was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.
RECOMMENDED: No reference to add_theme_support( "custom-background", $args ) was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.
RECOMMENDED: Could not find the file readme.txt in the theme. Please see Theme_Documentation for more information.
RECOMMENDED: Tags: is either empty or missing in style.css header.
INFO: Only one text-domain is being used in this theme. Make sure it matches the theme's slug correctly so that the theme will be compatible with WordPress.org language packs.
The domain found is sage

Wouldn't it make sense to use a dist/ directory that contains purely the theme files sans
any extra files, like dist/ for styles/css, but also for the theme PHP files?

sage will never pass the theme check plugin or wordpress coding standards 😊
ref #567, #545
ps. please don't close out the issue template when creating new issues. feel free to drop by https://discourse.roots.io/ if you'd like to discuss it further.


Root3 App Catalog integration

As an end user I would like to be able to install a pending app update, for example when it requires user interaction and closing the application. The Support App would allow me to quickly update the app at a more convenient time instead of the automatic daily update schedule.

This was implemented and can be closed


Feature: Convert file (*.heic, *.tiff, *.jpg, *.pdf) to images (*.png)

btw, kudos on the team collaboration.
I see that you are effectively following trunk-based development for small team for this short-lived feature branch.


Actions.refs not Working on V.4.0.6

Version
Tell us which versions you are using:
"react-native-router-flux": "^4.0.6",
"react-redux": "^7.1.3",
"redux": "^4.0.5",
"react": "16.9.0",
"react-native": "0.61.5",
Expected behaviour
Actions.refs.ScreenKey.functionName  should trigger 'functionName' in ScreenKey
Actual behaviour
I used this refs operation without redux its working when i use redux in my application, not working
im using like this ;
<Scene key="Welcome" component={Welcome}  onEnter={()=> Actions.refs.Welcome.myOnEnterFunc } />

I am using  "react-native-router-flux": "4.2.0" and it seems it is not working either.


⚠ Bofrost.de has degraded performance

In 27071f1, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1159 ms

Resolved: Bofrost.de performance has improved in 6c570eb after 10 minutes.


feat!: Change variable type for secrets

This PR makes below changes,

Change type of variable secrets from list(map(string)) to list(object({})
Create secret-version based on a newly introduced flag called create_version (true by default)

https://github.com/GoogleCloudPlatform/terraform-google-secret-manager/issues/88

/gcbrun

/gcbrun

/gcbrun

/gcbrun

Thanks @q2w! Few comments

Can we create a small upgrade guide showing how user can go from 0.3 to 0.4 showcase how to use the new create_version field rather than leaving secret data empty?  https://github.com/terraform-google-modules/terraform-google-project-factory/blob/master/docs/upgrading_to_project_factory_v13.0.md#add-dimensions-field-to-consumer_quota-object is an example
Let's also update to >= 1.3 due to optional usage https://github.com/GoogleCloudPlatform/terraform-google-secret-manager/blob/329bb3c6b7acafb9453fa82d9152db2994f744ee/versions.tf#L18

Done!


Changed message on not accepting default shell

Just clarified the message 😄

Thanks for catching this! Merged.


CIS Check Ensure 'MUST_CHANGE' Option is set to 'ON' for All SQL Authenticated Logins

Ensure 'MUST_CHANGE' Option is set to 'ON' for All SQL Authenticated Logins

Close #743


Fix code breaking Gradle 4.3

Fixes #66
Correctly use withPropertyName() - this method was moved from TaskPropertyBuilder to TaskFilePropertyBuilder in Gradle 4.3
Test against Gradle 4.3, replace 4.2 with 4.2.1
Add Idea files to .gitignore

Oh, I hope you didn't spend too much time on this. I had it fixed locally hours ago already (started working on it soon after you reported the issue; should have acknowledged it), but had a meeting so didn't push. Thanks a lot anyway, I really appreciate the pull request. Will go with my code here, but will probably cherry-pick a couple of your latest commits.
Fwiw, I went with using TaskInputFilePropertyBuilder and TaskOutputFilePropertyBuilder, as both types exist since Gradle 3.0 and override (covariant return type) the withPropertyName method, so there's actually no need to add a code path for Gradle 4.3.


Add more cider functions to the clojure keymap

There are quite a few useful cider functions that are not mapped in doom's clojure module. The keys chosen for running tests match the keys used in the default cider keybindings.

Looks good to me. Thanks for your help!


RuntimeError: Class values must be smaller than num_classes.

Hi, when I try to run the code with single GPU (I deleted the distributed-related code), I got the following error, it's very strange:
Epoch 1, lr = 0.009699: 100%|█| 154/154 [02:29<00:00,  1.03it/s, loss=0.41
13 INFO:0: Epoch 1, Class Loss=0.4461328089237213, Reg Loss=0.0
14 INFO:0: End of Epoch 1/30, Average Loss=0.4461328089237213, Class Loss=0.4461328089237213, Reg Loss=0.0
15 INFO:0: End of Validation 1/30
16 INFO:0: Epoch 2, lr = 0.009398
17 Epoch 2, lr = 0.009398: 100%|█| 154/154 [02:31<00:00,  1.02it/s, loss=0.33
18 INFO:0: Epoch 2, Class Loss=0.3124752342700958, Reg Loss=0.0
19 INFO:0: End of Epoch 2/30, Average Loss=0.3124752342700958, Class Loss=0.3124752342700958, Reg Loss=0.0
20 INFO:0: End of Validation 2/30
21 INFO:0: Epoch 3, lr = 0.009095
22 Epoch 3, lr = 0.009095:  31%|▎| 48/154 [00:47<01:46,  1.01s/it, loss=0.233Traceback (most recent call last):
23   File "run.py", line 227, in <module>
24     main(opts)
25   File "run.py", line 123, in main
26     epoch_loss = trainer.train(cur_epoch=cur_epoch, train_loader=train_loader)
27   File "/home/lttm/Desktop/chang/WILSON-main/train.py", line 194, in train
28     loss = criterion(outputs, labels)  # B x H x W
29   File "/home/lttm/miniconda3/envs/pytorch1.8/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
30     result = self.forward(*input, **kwargs)
31   File "/home/lttm/Desktop/chang/WILSON-main/utils/loss.py", line 73, in forward
32     targets = F.one_hot(labels_new, inputs.shape[1] + 1).float().permute(0, 3, 1, 2)
33 RuntimeError: Class values must be smaller than num_classes.

And I can't wake up my PC. Does anyone meet this strange problem?

Hey @adaxidedakaonang.
The error states that your labels_new has a class index that is greater than inputs.shape[1] + 1.
Please, check if the number of classes is properly set and if the labels contain some mistaken value.

Grazie, seems it's the problem of the GPU instead of the code. Decreasing num_workers (from 4 to 2) and decreasing batch size solve the problem......


Bump pydantic

Thank you for submitting this pull request! We appreciate you spending the time to work on these changes.
What is the motivation?
Modern tools are using pydantic 2.x.  In particular, it is required to use pydantic-settings.  The pinning of surrealdb 0.3.x to pydantic 1.x causes a version requirement conflict.
Type of Change

[x] 📚 Examples / docs / tutorials / dependencies update
[ ] 🔧 Bug fix (non-breaking change which fixes an issue)
[ ] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

What does this change do?

Bump pydantic dependency to ^2
Poetry update
Fix a breaking change from pydantic 1.x -> 2.x regarding validators

What is your testing strategy?
Any requests to the websocket client will test validate params with the updated decorator.
Is this related to any issues?
(https://github.com/surrealdb/surrealdb.py/issues/71)
Have you read the Contributing Guidelines?

[x] I have read the Contributing Guidelines

Closing this as #66 has been merged


Improved Dynamic Typing for Azure Scalesets in Azure Cloud Provider

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Steps to reproduce (least amount of steps as possible):
When creating a custom cluster and defining an Azure Cloud Provider block it would seem you can only choose between using VMs or Scalesets when setting vm_type
Example code:
https://github.com/luthermonson/rke-tools/blob/master/cloud-provider.sh#L48
It would be preferable for the type to be dynamic based on the node itself by querying the Microsoft API directly
For example, it is not optimal to use ScaleSets for Control Plane and ETCD Nodes but it is preferred for Worker Nodes
Result:
A more mature and appropriate cloud provider integration by dynamically assigning type based on the Microsoft API
gz#12273
gz#12393

I think that it'd be most beneficial to apply the VM type to the node pools so that you could manage your different node types that way as well.

@leflambeur Hi,
When you say that ScaleSets for Control Plane and ETCD Nodes is not optimal, I guess you refer to the dynamic nature (scalability) of them ? If I am not mistaken, Azure ScaleSet - even if not scaled - can have different Azure Zone, while Availability Set don't...
Thanks


Changing an IP to login into web UI

Hello, there am new here to install firezone.
I have successfully installed firezone on my fedora server 35. Now at the end of the installation, it shows, Use these credentials to sign in to the web UI at https://*.*.*.*..
Unfortunately am just trying to set up a VPN behind my firewall on some regular PC that is on some VLAN in my network. Currently firezone picked my public IP which doesn't work for me currently because I didn't have done port forwarding on my firewall, I will do that for the production but for now, am using zerotier to expose my local network to the public by creating some private network.
Before I installed firezone, I created some sub-domain and configured a DNS to point it to an IP from zerotier, which is my web app hosting PC IP.
Now what am trying to achieve is, accessing firezone web UI from the web through this IP Address instead of my public IP address which is used for accessing the internet. The main goal here is to access my web app through firezone only. Thank you.

Hey @elhananjair -- this should be possible if I'm understanding correctly. You'll want to do the following:

Set default['firezone']['external_url'] to the URL you plan on using for the web portal
Make sure the Endpoint is entered correctly on the /settings/site page. This is used to populate generated WireGuard configs.

Hey @elhananjair -- this should be possible if I'm understanding correctly. You'll want to do the following:

1. Set `default['firezone']['external_url']` to the URL you plan on using for the web portal

Thank you so much that's exactly what I wanted now I can log in to web UI.

2. Make sure the Endpoint is entered correctly on the `/settings/site` page. This is used to populate generated WireGuard configs.

Yes, I changed the IP in settings>defaults>endpoint.
Can I ask how I let firezone communicate with my webserver now? Currently, I am running nextcloud on that pc using apache web server, so how can I access that through firezone? Thank you.

Can I ask how I let firezone communicate with my webserver now? Currently, I am running nextcloud on that pc using apache web server, so how can I access that through firezone? Thank you.

Sorry, I'm a bit unclear on what you're asking. You should just be able to access your webserver after connecting to VPN using its local LAN ip. By default Firezone allows all traffic outbound so you shouldn't need to do anything special other than ensure the Firezone instance is on the same network as your webserver.

Thanks, @jamilbk for replying.
In my case, my webserver (apache) is listening to port 80 and 443, and Firezone listen to that port too, so am getting a problem running the apache server.
Therefore,

How does firezone forward such traffics to the web server in order to access nextcloud after connecting to firezone?
Secondly, I am trying to use certificates from my domain so that I can import it locally to secure the communication to the webserver, and I read somewhere that firezone needs certificates too, my question here is where should I use the private and public certificates in order to secure the communication? in firezone or in nextcloud config? Thanks again.

Can I ask how I let firezone communicate with my webserver now? Currently, I am running nextcloud on that pc using apache web server, so how can I access that through firezone? Thank you.

Sorry, I'm a bit unclear on what you're asking. You should just be able to access your webserver after connecting to VPN using its local LAN ip. By default Firezone allows all traffic outbound so you shouldn't need to do anything special other than ensure the Firezone instance is on the same network as your webserver.

Thanks, @jamilbk for replying. In my case, my webserver (apache) is listening to port 80 and 443, and Firezone listen to that port too, so am getting a problem running the apache server. Therefore,
* How does firezone forward such traffics to the web server in order to access nextcloud after connecting to firezone?

* Secondly, I am trying to use certificates from my domain so that I can import it locally to secure the communication to the webserver, and I read somewhere that firezone needs certificates too, my question here is where should I use the private and public certificates in order to secure the communication? in firezone or in nextcloud config? Thanks again.

@jamilbk please help me

@elhananjair Two services can’t both listen on the same port so you’ll either need to change Apache’s listen port or change Firezone’s.
Details for changing this and specifying an SSL cert should be explained in your config file in the Nginx section, or you can reference the setup guide or config file reference in our docs: https://docs.firezone.dev/
Once your device is connected to the tunnel, you should be able to access the nextcloud service on the internal tunnel IP for the Firezone instance which is <IP_ADDRESS> by default.

@jamilbk thank you so much for replying.
I have changed firezone's listening port in Nginx section (default['firezone']['nginx']['non_ssl_port'] and default['firezone']['nginx']['ssl_port'])

Now my apache web server is running well.
After I did that I tried to browse my domain without connecting to the tunnel from my laptop, and unfortunately, I could access nextcloud. But that's not what am looking for as you said I want to access nextcloud only when I am connected to the tunnel.
Do I miss something? Sorry taking your time @jamilbk

Hmm. Did you reload the configuration with firezone-ctl reconfigure? Is your firewall blocking inbound to those ports?
To close off nextcloud from the outside you could either try changing its bind address to <IP_ADDRESS> and/or adding a firewall rule to block access from non-tunnel IPs

@jamilbk thank you so much again 😊.

Hmm. Did you reload the configuration with firezone-ctl reconfigure? Is your firewall blocking inbound to those ports?

Absolutely, I have opened that port now and it's working, the only thing I should do is add a certificate lets encrypt generated into firezone configs to remove this error on browser The certificate is not trusted because it is self-signed.

To close off nextcloud from the outside you could either try changing its bind address to <IP_ADDRESS> and/or adding a firewall rule to block access from non-tunnel IPs

I thought by default everything will be blocked if you are not connected to the tunnel, my bad I am new to VPN configurations and am really sorry.
what do you mean by this exactly try changing its bind address to <IP_ADDRESS> I didn't fully understand and adding a firewall rule to block access from non-tunnel IPs - does firezone gives an IP to internal services? Or how do I know non-tunnel IPs?


Some stubbed Vuetify components can't be found with find wrapper method

Version
1.0.0-beta.29
Reproduction link
https://github.com/anthonygore/vuetify-test
Steps to reproduce
npm i
npm run test:unit
What is expected?
All tests should pass
What is actually happening?
The second test fails as the VTextField component can't be found.

I don't know if this is a Vuetify issue or a Vue Test Utils issue, but some stubbed child components can't be found with the find wrapper method when the parent is shallow mounted.
It's not just VTextField, either, VSelect also has this problem, while others like VIcon work just fine.
import { shallowMount } from "@vue/test-utils";
import Vuetify from "vuetify";
import { VBtn, VTextField } from "vuetify/lib";

Vue.use(Vuetify);

const component = {
  template: `

      <v-btn />
      <v-text-field />

  `
};

// Passes
it('should find v-btn', () => {
  const wrapper = shallowMount(component);
  expect(wrapper.find(VBtn).exists()).toBe(true);
});

// Fails
it('should find v-text-field', () => {
  const wrapper = shallowMount(component);
  expect(wrapper.find(VTextField).exists()).toBe(true);
});

Update
I think the issue is something to do with how some global components are registered because this works:
Vue.use(Vuetify);

const component = {
  template: `

      <v-btn />
      <v-text-field />

  `,
  components: {
    VTextField
  }
};

Note that I don't need to locally register VBtn, just VTextField...

Hey @anthonygore , This is likely something to do with the fact VTextField extends from VInput, as shown here in the source code for Vuetify.
It's been quite some time since I've worked on a Vue project, or vue-test-utils ( 😢  ), but iirc if you pass a component to find it will match against the prototype, and in this case VTextField.prototype !== VInput.prototype it returns false.
A work around would be to pass an object with name to find. So you could try:
it('should find v-text-field', () => {
  const wrapper = shallowMount(component);
  expect(wrapper.find({ name: 'v-text-field' }).exists()).toBe(true);
});

That may work.

Hey @lmiller1990, I tried your solution and it doesn't work. But, I think you're onto something with the extends thing. It seems that all the Vuetify components that don't work with find do extend another component. That gives me a lead to keep researching. Thanks!

Interesting. I recently got asked to help out with a Vuetify project that has vue-test-utils in it, so I'll have a hack this weekend and see if I can figure this out (likely I will run into this soon as well).

Champion, thank you.

Happens to v-list-tile-title too: https://github.com/vuetifyjs/vuetify/blob/6f32c6439a4c789ac992794f5d514db6fec4a5fc/packages/vuetify/src/components/VList/index.ts#L12
There is currently no way to find a functional component - I doubt this will change.

Are you sure? v-icon is functional and I can find that one.
it('should find v-icon', () => {
  const wrapper = shallowMount(Test);
  expect(wrapper.find(VIcon).exists()).toBe(true); // Passes
});

Hm, I guess that isn't what prevents me finding v-list-tile-title.
I wonder what the common thing that prevents find from working...

I'm having this issue as well in my own library (not using or related to Vuetify).
In my jest setup I use Vue.component('FooBar', require('.../path/to/FooBar.vue').default)
In my test I import 'FooBar' from '.../path/to/FooBar.vue'
I provide stubs: {FooBar: true} in my options when mount-ing (not shallow)
Then wrapper.find(FooBar) fails, even though I'm using <foo-bar .../> in the component.
I've noticed that wrapper.find('foobar-stub') works, but the props are empty so testing it is moot.

I find it very interesting too that the stub I found is named foobar (without the dash!)

I still don't understand why wrapper.find(VBtn) works above, but your update makes sense if you look at the code
When stubbing, vue-test-utils uses the component's components options. So without providing any components (relying on the ones already given to Vue) then it will fail to use the component to stub and make a new stub component instead.
In your update, since you pass the components option, it'll find it correctly.
Seems to me that line 56 could be changed to Object.assign({}, _Vue.options.components, componentOptions.components) to also reference the globally registered components.
Testing this out locally did the trick for me.

As a workaround before I declare my tests I do:
FooBar.components = Object.assign(Object.create(Vue.options.components), FooBar.components || {});

And then the stubs work like a charm

Are you sure? v-icon is functional and I can find that one.
import { VIcon } from "vuetify/lib";

it('should find v-icon', () => {
  const wrapper = shallowMount(component);
  expect(wrapper.find(VIcon).exists()).toBe(true); // Passes
});

For me, I cant use wrapper.find with a VBtn, for example. The console returns:
 TypeScript diagnostics (customize using `[jest-config].globals.ts-jest.diagnostics` option):
    tests/unit/infosets/components/InfosetsHeader.spec.ts:17:25 - error TS2345: Argument of type 'Component<DefaultData<never>, DefaultMethods<never>, DefaultComputed, Record<string, any>>' is not assignable to parameter of type 'NameSelector'.
      Type 'FunctionalComponentOptions<Record<string, any>, PropsDefinition<Record<string, any>>>' is not assignable to type 'NameSelector'.
        Types of property 'name' are incompatible.
          Type 'string | undefined' is not assignable to type 'string'.
            Type 'undefined' is not assignable to type 'string'.

    17     expect(wrapper.find(VBtn).exists()).toBe(true);

@aislanmaia I noticed I could find some functional components too - I can't see any common pattern that makes a component unfindable at this point.

@lmiller1990 I've some progress here, but in the same point 0 again.
Take a look:
 InfosetsHeader.component
    ✓ renders Minhas Listas at top when rendered (3ms)
    ✕ renders a Button component to create new lists (13ms)

  ● InfosetsHeader.component › renders a Button component to create new lists

    expect(received).toBe(expected) // Object.is equality

    Expected: true
    Received: false

      32 |     const btn = wrapper.find(".v-btn");
      33 |     // console.log("btn", btn);
    > 34 |     expect(btn.exists()).toBe(true);
         |                          ^
      35 |
      36 |     /* mock event */
      37 |     const event = jest.fn();

      at Object.<anonymous> (tests/unit/infosets/components/InfosetsHeader.spec.ts:34:26)

The v-btn and any vuetify components are unfindable.
The current state is that Vuetifiy is just untestable at all, at least with Jest.
Things like that, just increase the Javascript fatigue for me, because IMHO, things like this (testing, etc), should work out-of-box really.

It feels like Vuetify is responsible for a lot of problems around vue-test-utils, I've had an equally frustrating experience (with attempting to test Vuetify based apps).
One work-around you can try is adding <v-btn data-test="button"> ..... </v-btn> and trying to do wrapper.find('[data-test="button"]') and see if you can find it that way.

Don't work either. It seems that the element isn't there yet, or maybe it never wasn't there in the component at the moment the test is running.

What if you do console.log(wrapper.html({ pretty: true })) (not sure if the pretty: true API is correct or not, been a while). Can you see the markup you expect?

What if you do console.log(wrapper.html({ pretty: true })) (not sure if the pretty: true API is correct or not, been a while). Can you see the markup you expect?
If you are using post vue-test-utils beta 28, the way rendering updates are applied changed a bit. You might need to do await wrapper.vm.$nextTick() to ensure Vue has had time to update the DOM.

It doesn't accept a parameter like pretty. Just calling wrapper.html() is enough to get returned the html, however in unformatted way.
The markup returned show up some Vuetify components marked with the "-stub" in the names.
It happens that even reference this names like wrapper.find('.v-btn-stub') the "find" cannot return my Vuetify components.

I might take (another) look at this one this week. Whoever solves this will be a hero; this issue has plagued us for a while. I have noticed this only happens sometimes, especially in larger projects. Have you had a similar experience?

@lmiller1990 no similar, because our project is in early stages and not considered big right now. Saying that, this is our first unit tests with Vuetify, and it is disappointing that it not working at all with jest.

@aislanmaia  I just found that removing the . actually solves this. When running the wrapper.html() I saw that the stubs aren't actually divs or similar with classes applied but actually a v-btn-stub element.

What if you do console.log(wrapper.html({ pretty: true })) (not sure if the pretty: true API is correct or not, been a while). Can you see the markup you expect?
If you are using post vue-test-utils beta 28, the way rendering updates are applied changed a bit. You might need to do await wrapper.vm.$nextTick() to ensure Vue has had time to update the DOM.

It doesn't accept a parameter like pretty. Just calling wrapper.html() is enough to get returned the html, however in unformatted way.
The markup returned show up some Vuetify components marked with the "-stub" in the names.
It happens that even referencing this names like wrapper.find('.v-btn-stub') the "find" cannot return my Vuetify components.

Regarding Vuetify, what I am currently doing is just using mount and doing things on the raw <input /> or <button /> that VInput and VBtn render. I'm starting to think shallowMount should be avoided wherever possible - it's not ideal to be testing against something different (eg stubs) to your actual component.
We are exploring better ways to test third party UI components for the Vue 3 integration.


Refactor Marlin implementation

Motivation
This PR refactors the Marlin implementation to simplify the prover and verifier algorithms, with the aim of making future changes (batch proving + lookup tables) easier to implement.
Test Plan

(Write your test plan here)
Related PRs

(Link your related PRs here)

This PR is ready for review. The proof size has gone down, which is good, and as a result the tests aren't passing any more (which makes sense). However I can't get tests to pass locally even after resampling parameters, so any insight would be valuable.

Ah sorry, should have closed this; the PC refactor contains these commits. Can we revert this?


MM-20453: Migrate 'components/integrations/installed_outgoing_webhooks' module and associated tests to TypeScript

Summary
This PR migrates 'components/integrations/installed_outgoing_webhooks' module and associated tests (also refactor to simplify default props) to TypeScript
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/16121
JIRA: https://mattermost.atlassian.net/browse/MM-20453
Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

Please rerun CI checks, it failed on the first part (i.e. install)

/update-branch

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/218662


propagate back assignments to block scoped binding from the loop body

fixes #6877.
NOTE: it will be cleaner to emit loop body as
let loop = function(...) {
    try { loop body }
    finally { copy local values to out parameters }
}

instead of writing copy snippet at every point where control flow exits loop. However currently V8 does not optimize protected regions so between prettiness and perf of generated code I'd prefer to favor the second one

:+1: with the removal of the extra assignment.


WEBSITE - Updated What's New page for 4.1

patch by Michael Semb Wever, Erick Ramirez; reviewed by Ekaterina Dimitrova, Berenguer Blasi, Erick Ramirez for CASSANDRA-17976
Co-authored by: Mick Semb Wever<EMAIL_ADDRESS>Co-authored by: Erick Ramirez<EMAIL_ADDRESS>

Committed as a4b90fb.


feat: bring backup status of snapshots to parity with barman

Closes #2813

/test limit=local

/test limit=local

/test limit=local

/test

/test

/ok-to-merge E2E tests green on EDB fork https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/6313760209/job/17142807170


[Toggle Audio Input (microphone)] not restoring last audio input LEVEL

Extension
https://www.raycast.com/Quentin23Soleil/mute-microphone
Raycast Version
1.75.1
macOS Version
14.5
Description
The extension in not able to restore the previous input volume level. For example if is set to 50% and and toggle twice the extension, I should have the volume input level still to 50% and not 100%.
Steps To Reproduce

Press Option+F10 to open the sound settings in macos.
Set the input volume level.
Toggle twice the extension (with hotkey or with raycast)

Current Behaviour
The extension restore the input volume level to 100%.
Expected Behaviour
The extension should restore the input volume level to the last one saved by the user. So if I the volume level before muting the microphone with the extension was set to 50% before toggling the extension, when the users toggles again the extension to unmute the audio input, the input volume level should be restore to 50%.

Noticed very same issue on my machine today attempting to use this plugin. Also running same versions listed.

Hi there! @NotFiliberto @1activegeek https://github.com/raycast/extensions/pull/12614


getting cross-origin issues when running web ui locally.

Type of issue

[x] Bug
[ ] New feature
[ ] Enhancement

Description
I'm trying to Prepare a local development environment for customize UI according to the article on https://docs.microsoft.com/en-gb/azure/iot-accelerators/iot-accelerators-remote-monitoring-customize . I have completed all the steps but when i  run web ui locally i'm getting cross origin issues.
Steps to reproduce
1.Prepare a local development environment for the UI according to the article https://docs.microsoft.com/en-gb/azure/iot-accelerators/iot-accelerators-remote-monitoring-customize
Expected behavior
...
Current behavior <!-- The actual behavior observed --
...
Known workarounds
...
Possible solution
...
Context and Environment

Operating System: Window 10
GitHub branch/tag: ...

Screenshot

This is likely a similar issue that is currently being fixed. https://github.com/Azure/pcs-cli/pull/513
If you want to try out the fix you basically need to edit your /app/auth.sh script that is in your VM to be the same as what you see in that pull request.
Essentially the issue was that some parameters that need to be edited to allow local development when using --unsafe flag were not being set.

Thank you so much Elvin Morales it worked.


ES6 Arrow functions?

var test = {
  stuff: () => { // gives 'Unexpected token ) at line 4 col 12
    console.log('it works')
  },
  moreStuff: () => {
    console.log('it works still')
  }
}

test.stuff()

Babel seems to successfully compile this to
var test = {
  stuff: function stuff() {
    console.log('it works');
  },
  moreStuff: function moreStuff() {
    console.log('it works still');
  }
};

test.stuff();

I should add that standard only complains about the first arrow function stuff, the moreStuff function has no error

Looks like an eslint bug to me.
@feross confirm?

Sorry guys, I apparently had 2 conflicting linters. closing...

Thanks @internalfx,  I hadn't had time to check this yet.


Reduce size of docker image

This pull request reduces the image to 700mb by using node:slim

@dtjohnson are you still maintaining this project?

@dtjohnson  much appreciated :) it is a great tool


Doesn't work on Firefox

Hello, I just download Sidr and test it with simplest.sample.sidr.html
...and it doesn't work with Firefox!!
it's OK with Chrome.

Preparing a new version with a fix for this https://github.com/artberri/sidr/tree/reborn


How to use with a RecyclerView

If I have a RecyclerView whose children are layouts that only contain a PhotoView, the PhotoView doesn't respond to touch events.

Hello, did you find the solution ? I'm attaching photoview into recyclerview but unable to zoom the images.


Fix using length as a property name

Fixes https://github.com/CartoDB/carto-vl/issues/1247

It seems that render tests are broken because the docker image is not updated.

Yes, you're right, we're trying to fix the image 👍


fix: DEV-4087: Fix multiple labels in Outliner

Labels assigned from multiple Labels tags lead to duplicated region's text in Details panel. Cleaned up the code related to labels and text displaying.
Improved Details panel appearance.
Also cleaned up connected places:

unused WithStates mixin (getOneColor was not used also)
commented out typo in displayMode param usage

Codecov Report
Base: 9.40% // Head: 9.41% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (f1d70a6) compared to base (c3ed720).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #1142      +/-   ##
=========================================
+ Coverage    9.40%   9.41%   +0.01%
=========================================
  Files         109     108       -1
  Lines        7785    7776       -9
  Branches     1953    1948       -5
=========================================
  Hits          732     732
+ Misses       5908    5903       -5
+ Partials     1145    1141       -4

Impacted Files
Coverage Δ

src/src/regions/RectRegion.js
3.46% <0.00%> (ø)

src/src/regions/BrushRegion.js
3.32% <0.00%> (ø)

src/src/regions/EllipseRegion.js
4.72% <0.00%> (ø)

src/src/regions/PolygonRegion.js
2.95% <0.00%> (ø)

src/src/regions/KeyPointRegion.js
6.81% <0.00%> (ø)

src/src/mixins/WithStates.js

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fMRIprep 21.0.0 doesn't ignore fieldmaps intended for dwi

What happened?
fMRIprep doesn't work if in the fmap folder there are files intended for the dwi processing (unless the --ignore=fieldmaps is specified)
What command did you use?
singularity run -C -B /EBC:/EBC,$2:/opt/templateflow /EBC/home/mblesa/fmriprep_21.0.0.simg $2/BIDs_data $2/derivatives participant --participant-label ${NAME} -w $2/${NAME}_working_directory --nthreads 16 --omp-nthreads 16 --fs-license-file /EBC/local/infantFS/freesurfer/license.txt --output-space MNI152NLin2009cAsym:res-2 --stop-on-first-crash --skip_bids_validation

What version of fMRIPrep are you running?
21.0.0
How are you running fMRIPrep?
Singularity
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
/opt/conda/lib/python3.8/site-packages/bids/config.py:39: FutureWarning: Setting 'extension_initial_dot' will be removed in pybids 0.16.
  warnings.warn("Setting 'extension_initial_dot' will be removed in pybids 0.16.",
220121-18:20:33,737 nipype.workflow IMPORTANT:
         Running fMRIPrep version 21.0.0

         License NOTICE ##################################################
         fMRIPrep 21.0.0
         Copyright 2021 The NiPreps Developers.

         This product includes software developed by
         the NiPreps Community (https://nipreps.org/).

         Portions of this software were developed at the Department of
         Psychology at Stanford University, Stanford, CA, US.

         This software redistributes the versioneer Python package, which is
         Public domain source code.

         This software is also distributed as a Docker container image.
         The bootstraping file for the image ("Dockerfile") is licensed
         under the MIT License.

         This software may be distributed through an add-on package called
         "Docker Wrapper" that is under the BSD 3-clause License.
         #################################################################
220121-18:20:33,846 nipype.workflow IMPORTANT:
         Building fMRIPrep's workflow:
           * BIDS dataset path: /EBC/home/mblesa/study_5y/PRUEBA/CONVERSION/BIDs_data.
           * Participant list: ['007'].
           * Run identifier: 20220121-182022_55f8045e-4b1f-4b24-98f2-bbfcf8947ec0.
           * Output spaces: MNI152NLin2009cAsym:res-2.
           * Pre-run FreeSurfer's SUBJECTS_DIR: /EBC/home/mblesa/study_5y/PRUEBA/CONVERSION/derivatives/sourcedata/freesurfer.
Process Process-2:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/opt/conda/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/opt/conda/lib/python3.8/site-packages/fmriprep/cli/workflow.py", line 118, in build_workflow
    retval["workflow"] = init_fmriprep_wf()
  File "/opt/conda/lib/python3.8/site-packages/fmriprep/workflows/base.py", line 85, in init_fmriprep_wf
    single_subject_wf = init_single_subject_wf(subject_id)
  File "/opt/conda/lib/python3.8/site-packages/fmriprep/workflows/base.py", line 324, in init_single_subject_wf
    fmap_estimators = find_estimators(
  File "/opt/conda/lib/python3.8/site-packages/sdcflows/utils/wrangler.py", line 324, in find_estimators
    fm.FieldmapFile(fmap.path, metadata=fmap.get_metadata())
AttributeError: 'NoneType' object has no attribute 'path'
(pam_mount.c:133): clean system authtok=0x559a3239ace0<PHONE_NUMBER>)

Additional information / screenshots
No response

I think this case should be fixed in the next release, planned for next week. I'll leave this open in case not.

21.0.1 is released. Please let us know if this is not resolved.

dear all, I'm having a similar issue in 21.0.1. the fmap scans intended for the dwi scan are used in fmriprep for the fieldmap estimation and the susceptibility induced distortion correction but actually make the distortion worse than they were before the 'correction'. Removing the dwi fmap scans resolves the issue and ensures that only the fmap for the functional scan are used (--ignore=fieldmaps is therefore not an option)

Well, this got buried. @chrisvriend have you tried in 22.0.2? If this is not resolved, could you open a new issue, filling out the whole template?


Parameterized Generators and Pipelines

Users can now generate their own pipelines using this script from their own repositories to where they want to generate them.
Implement repository checkout in downstream generators
Implement file path params in downstream generators

Closes: #2116
Signed-off-by: Morgan Davies<EMAIL_ADDRESS>

https://ci.adoptopenjdk.net/job/build-scripts/job/utils/job/alpha-build-pipeline-generator/49/console proves this works with custom values and repository (the test jobs were created in wip: https://ci.adoptopenjdk.net/view/work-in-progress/job/build-scripts-testing/). I'll revert the file changes and continue testing

run tests

@AdamBrousseau @johnoliver Can I get reviews please?

@M-Davies - script approvals done

run tests

run tests

run tests

Tester Failed due to unrelated sign build failure on windows (e.g. https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk8u/job/jdk8u-windows-x64-hotspot/304/console).

@sxa I intend on addressing that as part of https://github.com/AdoptOpenJDK/openjdk-build/issues/2129. Get all of the documentation out of the way in one go 🙂

@M-Davies - Needs a rebase

@AdamBrousseau Can you take a look at the defaults.json file and let me know if that's the sort of thing you're looking for? I'll be replacing any empty or hard coded strings with a call to this JSON, leaving any overrides being handled by altering of the defaults file OR by Jenkins parameter.

run tests

Current TODO

[ ] Get script approval for above failure
[ ] Verify everything works, inducing  different and null values and as many use cases as can be thought of
[ ] Write a testcase to cover the new defaults.json

run tests

run tests

run tests

@M-Davies test compilation failed
TestCompilation > compile_pr_test_pipelineTest() FAILED
    groovy.lang.MissingMethodException: No signature of method: java.lang.String.entrySet() is applicable for argument types: () values: []
    Possible solutions: toSet(), getBytes(), every()
        at org.codehaus.groovy.runtime.ScriptBytecodeAdapter.unwrap(ScriptBytecodeAdapter.java:70)
        at java_lang_Throwable$printStackTrace.call(Unknown Source)
        at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:47)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:125)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:130)
        at TestCompilation.doCompile(TestCompilation.groovy:62)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:566)
        at org.codehaus.groovy.runtime.callsite.PlainObjectMetaMethodSite.doInvoke(PlainObjectMetaMethodSite.java:43)
        at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite$PogoCachedMethodSiteNoUnwrapNoCoerce.invoke(PogoMetaMethodSite.java:193)
        at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.callCurrent(PogoMetaMethodSite.java:61)
        at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallCurrent(CallSiteArray.java:51)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:171)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:194)
        at TestCompilation.compile_pr_test_pipelineTest(TestCompilation.groovy:79)

@M-Davies Still not happy with the compile...

run tests

run tests

run tests

run tests

run tests

run tests

run tests

run tests

run tests

run tests

run tests

run tests

run tests

run tests

Aside from today's issues on Windows, the pr tester is passing with these changes (I've updated my checklist comment). I'll do some testing with the alpha generators and https://ci.adoptopenjdk.net/view/work-in-progress/job/build-scripts-testing/ to ensure all the possible params that can go in have been covered

@M-Davies great progress - LMK when it's ready for review and we can go through it

@M-Davies great progress - LMK when it's ready for review and we can go through it

Will do! I'm going through brute testing it now, inducing weird and wonderful params to these new pipelines. So far so good, assuming it all turns out ok, the next step and final test will be creating a full set of working pipelines and generators from a completely different repository. @AdamBrousseau Is this more like what you envisioned for this PR? I've tried to keep it as close the openj9 solution as possible while conforming to our standards and circumstances

When running the downstreams pipelines, there is a line in the template that generates them that complicates things:
https://github.com/AdoptOpenJDK/openjdk-build/blob/7509ff6cc3e0452f1f3d189899f73856ebc6d26c/pipelines/build/common/create_job_from_template.groovy#L39
The refspec searches for a master branch and my test repository did not contain one at first, causing the downstream jobs to fail:
Started by upstream project "build-scripts-testing/openjdk8-pipeline" build number 30
originally caused by:
 Started by user Morgan Davies
 Started by upstream project "build-scripts-testing/openjdk8-pipeline" build number 29
 originally caused by:
  Started by user Morgan Davies
  Started by upstream project "build-scripts-testing/openjdk8-pipeline" build number 28
  originally caused by:
   Started by user Morgan Davies
Checking out git https://github.com/M-Davies/openjdk-build-fork.git into /home/jenkins/.jenkins/workspace/build-scripts-testing/jobs/jdk8u/jdk8u-mac-x64-openj9@script to read pipelines/TestLocation/scripts/others/kick_off_build.groovy
The recommended git tool is: NONE
No credentials specified
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url https://github.com/M-Davies/openjdk-build-fork.git # timeout=10
Pruning obsolete local branches
Fetching upstream changes from https://github.com/M-Davies/openjdk-build-fork.git
 > git --version # timeout=10
 > git --version # 'git version 2.29.0'
 > git fetch --tags --force --progress --prune -- https://github.com/M-Davies/openjdk-build-fork.git  +refs/pull/*:refs/remotes/origin/pr/* +refs/heads/master:refs/remotes/origin/master +refs/heads/*:refs/remotes/origin/* # timeout=10
ERROR: Error fetching remote repo 'origin'
hudson.plugins.git.GitException: Failed to fetch from https://github.com/M-Davies/openjdk-build-fork.git
	at hudson.plugins.git.GitSCM.fetchFrom(GitSCM.java:996)
	at hudson.plugins.git.GitSCM.retrieveChanges(GitSCM.java:1237)
	at hudson.plugins.git.GitSCM.checkout(GitSCM.java:1297)
	at org.jenkinsci.plugins.workflow.steps.scm.SCMStep.checkout(SCMStep.java:125)
	at org.jenkinsci.plugins.workflow.cps.CpsScmFlowDefinition.create(CpsScmFlowDefinition.java:155)
	at org.jenkinsci.plugins.workflow.cps.CpsScmFlowDefinition.create(CpsScmFlowDefinition.java:69)
	at org.jenkinsci.plugins.workflow.job.WorkflowRun.run(WorkflowRun.java:309)
	at hudson.model.ResourceController.execute(ResourceController.java:97)
	at hudson.model.Executor.run(Executor.java:429)
Caused by: hudson.plugins.git.GitException: Command "git fetch --tags --force --progress --prune -- https://github.com/M-Davies/openjdk-build-fork.git  +refs/pull/*:refs/remotes/origin/pr/* +refs/heads/master:refs/remotes/origin/master +refs/heads/*:refs/remotes/origin/*" returned status code 128:
stdout:
stderr: fatal: couldn't find remote ref refs/heads/master

	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:2450)
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandWithCredentials(CliGitAPIImpl.java:2051)
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.access$500(CliGitAPIImpl.java:84)
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl$1.execute(CliGitAPIImpl.java:573)
	at hudson.plugins.git.GitSCM.fetchFrom(GitSCM.java:994)
	... 8 more
ERROR: Error fetching remote repo 'origin'
ERROR: Maximum checkout retry attempts reached, aborting
Finished: FAILURE

Luckily and usually, most repositories have a master branch but I would like to gather some opinions for those usecases where we may not have one or it's called something different (i.e. by default, GitHub sets default branchnames to main instead of master which would also fail these jobs).
Should we try to cover these eventualities?

Just to give an update, passing in custom locations by parameter is functional. Passing in custom values by the defaults.json is still in testing, as is pulling in default files from a custom repository

To give a little update on this, I understand it's been brewing for a while so I'm pleased to announce the end is in sight

https://ci.adoptopenjdk.net/job/build-scripts-testing/job/jobs/job/jdk8u/job/jdk8u-linux-x64-hotspot/74/consoleFull shows that this works using my repository, I can build openjdk-scripts from custom locations using custom scripts.
The final stage of testing will be trying to run the scripts when everything is in a different location (relative to jenkins). I am doing this via https://github.com/M-Davies/openjdk-build-fork/tree/testing_params/pipelines/TestLocation
After that, I will need to address the merge conflicts and address the remaining agenda in my checklist comment

To give a little update on this, I understand it's been brewing for a while so I'm pleased to announce the end is in sight

https://ci.adoptopenjdk.net/job/build-scripts-testing/job/jobs/job/jdk8u/job/jdk8u-linux-x64-hotspot/74/consoleFull shows that this works using my repository, I can build openjdk-scripts from custom locations using custom scripts.
The final stage of testing will be trying to run the scripts when everything is in a different location (relative to jenkins). I am doing this via https://github.com/M-Davies/openjdk-build-fork/tree/testing_params/pipelines/TestLocation
After that, I will need to address the merge conflicts and address the remaining agenda in my checklist comment

Test cases have been written. I will do some final builds on all platforms to ensure everything came out ok and then start cleaning up some of my TODO's and placeholder code

Test cases have been written. I will do some final builds on all platforms to ensure everything came out ok and then start cleaning up some of my TODO's and placeholder code

NOTE: I am expecting RepoHandlerTest to fail in Compile Groovy as I have changed the Default File URL to what it will be after merge. I am also expecting the PR tester to fail initially as the job hasn't been pointed at the kick_off_tester.groovy file yet (as per my checklist, I'll update it with the other jobs after the merge).
@karianna @sxa @AdamBrousseau @smlambert This is ready for reviews

NOTE: I am expecting RepoHandlerTest to fail in Compile Groovy as I have changed the Default File URL to what it will be after merge. I am also expecting the PR tester to fail initially as the job hasn't been pointed at the kick_off_tester.groovy file yet (as per my checklist, I'll update it with the other jobs after the merge).
@karianna @sxa @AdamBrousseau @smlambert This is ready for reviews

AQA meeting recording of me discussing these changes (start at 3:05) -> https://youtu.be/Fql05uEF9ZA?t=185

I'm afraid a rebase will also be required now sorry!

Thanks for taking this on @M-Davies (and for walking us through it, re: https://github.com/AdoptOpenJDK/openjdk-build/pull/2132#issuecomment-772457999).

Update thread for addressing issues and the checklist comment above -> https://adoptopenjdk.slack.com/archives/C09NW3L2J/p1612435163113400


Added request to install flows by switch

Closes #202
Summary
This is a draft, implementation may change

Implemented new methods to install flows by specific switches. These methods are slightly modified to fit with the content of the request.
There is room for improvement in the controller side

Local Tests
Tried with these request:
{
    "00:00:00:00:00:00:00:03": {
        "flows": [{"match": {"in_port": 3, "dl_vlan": 1}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "set_vlan", "vlan_id": 1}, {"action_type": "output", "port": 2}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}, {"match": {"in_port": 2, "dl_vlan": 1}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "set_vlan", "vlan_id": 1}, {"action_type": "output", "port": 3}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}],
        "force": true
    },
    "00:00:00:00:00:00:00:02": {
        "flows": [{"match": {"in_port": 3, "dl_vlan": 1}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "set_vlan", "vlan_id": 1}, {"action_type": "output", "port": 2}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}, {"match": {"in_port": 2, "dl_vlan": 1}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "set_vlan", "vlan_id": 1}, {"action_type": "output", "port": 3}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}],
        "force": true
    },
    "00:00:00:00:00:00:00:04": {
        "flows": [{"match": {"in_port": 1, "dl_vlan": 123}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "push_vlan", "tag_type": "s"}, {"action_type": "set_vlan", "vlan_id": 1}, {"action_type": "output", "port": 2}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}, {"match": {"in_port": 2, "dl_vlan": 1}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "pop_vlan"}, {"action_type": "output", "port": 1}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}],
        "force": true
    },
    "00:00:00:00:00:00:00:01": {
        "flows": [{"match": {"in_port": 1, "dl_vlan": 123}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "push_vlan", "tag_type": "s"}, {"action_type": "set_vlan", "vlan_id": 1}, {"action_type": "output", "port": 3}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}, {"match": {"in_port": 3, "dl_vlan": 1}, "cookie":<PHONE_NUMBER>2473034410, "actions": [{"action_type": "pop_vlan"}, {"action_type": "output", "port": 1}], "owner": "mef_eline", "table_group": "evpl", "table_id": 0, "priority": 20000}],
        "force": true
    }
}

End-to-End Tests
TBA

Creating another PR covering this.


Added endpoint extraction from JS file module

This module extracts all the urls and endpoints in a JS file Using a regex.
Checklist

[x] I have followed the Contributor Guidelines.
[x] The code has been thoroughly tested in my local development environment with flake8 and pylint.
[x] The code is both Python 2 and Python 3 compatible.
[x] The code follows the PEP8 styling guidelines with 4 spaces indentation.
[ ] This Pull Request relates to only one issue or only one feature
[ ] I have referenced the coresspoinding issue number in my commit message
[ ] I have added the relevant documentation.
[x] My branch is up-to-date with the Upstream master branch.

Your development environment

OS: Linux
OS Version: Ubuntu 18.04
Python Version: 3.6.9

this can't be supported anymore. modules are yaml #440


the "Get a Hint"  info is no use, in   "registration-of-new-users" challenges

Affected page
https://www.freecodecamp.org/learn/quality-assurance/advanced-node-and-express/registration-of-new-users
click "Get a Hint" button   will to url
https://forum.freecodecamp.org/t/freecodecamp-challenge-guide-registration-of-new-users/301561
but hint info is no use,   I change it ,but I can not pass challenges
at the end  I write  is to pass challenges
  app.route('/register')
    .post((req, res, next) => {
    myDataBase.findOne({ username: req.body.username }, function(err, user) {
        const  hash =  bcrypt.hashSync(req.body.password, 12);
        if (err) {
          next(err);
        } else if (user) {
          res.redirect('/');
        } else {
          myDataBase.insertOne({
            username: req.body.username,
            password: hash
          },
            (err, doc) => {
              if (err) {
                res.redirect('/');
              } else {
                // The inserted document is held within
                // the ops property of the doc
                next(null, doc.ops[0]);
              }
            }
          )
        }
      })
    },
      passport.authenticate('local', { failureRedirect: '/' }),
      (req, res, next) => {
        res.redirect('/profile');
      }
    );

Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Get Help" button on the challenge on freeCodeCamp and choose the "Ask for help" option, which will help you create a question in the right part of the forum. Volunteers on the forum usually respond to questions within a few hours and can help determine if there is an issue with your code or the challenge's tests.
If the forum members determine there is nothing wrong with your code, you can request this issue to be reopened.
Thank you and happy coding.

ok

@luojiyin1987 passed this challenge, and he meant the hint was not helpful. Anyone wants to check the hint? Thanks!

@miyaliu666 the hints are a forum post, to suggest changes to the guild you can open a thread in the forum

To elaborate what @ieahleen mentioned:

Challenge hints are a helping tool. They are normally posted on the forum and updated when needed with help of the community moderators and contributors. These are not a bug per se, in the curriculum or the codebase.
We are keeping this tracker to keep the discussions & threads for issues/bugs in codebase only. I think it's best to post on the forum first when stuck, or if you find something unuseful. Someone from the mod team can confirm the course of action whether it's a bug and needs reporting here or can be addressed in the discussions on the forum.

Thanks again for your reports and feedback.

@raisedadead @ieahleen Thanks for the details. I've told @luojiyin1987 to report on the English forum.


pipelineX fix distinct streaming agg

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


all: add support for new state expiration ledger entries

What
Add support for the new state expiration ledger entries,
This PR also changes getLedgerEntries method so that doesn't filter out expired ledger entries. It simply returns whatever is in the database.
Why
Fixes #873
Known limitations
I left a few TODOs in the CLI, namely:

We should test that some invariants hold in meta matches for the bump/restore commands.
I haven't been able to update get_contract_spec_from_storage() with respect to expirations since I don't think the storage book-keeps expiration ledger entries.

@willemneal could you please take a look at the items above? Also, could you do some testing on the CLI once this is merged? Unfortunately, the XDR is out of sync with futurenet so you will have to use a local network.
We should probably start improving CLI testing considerably (particularly, integration tests against soroban-rpc). I wasn't comfortable fixing (2) since there were no tests covering it (AFICT).
Also, we should probably just get rid of the sandbox mode once and for all.

Also - we could use some clarification from core whether the access to the expiration ledger entry is counted as an additional entry access or not. ( it might be a special case ).

I doubt it since the host doesn't track them and AFAIU otherwise integration tests wouldn't had worked. But this is a question for @sisuresh

I believe that one thing that is missing is the deletion of EvictedTemporaryLedgerKeys. These need to be deleted as soon as they become stale, since temporary entries could not be restored.

Yes, that's why I created https://github.com/stellar/soroban-tools/issues/914

I think that it shouldn't be a testnet blocker here, but we should add some type of validation when inserting the ledger entries into the database to ensure that we always have "pairs" of entry + expiration.
The alternative to that is to add this during the simulation & getledgerenttries, but that won't fix the situation.

I think that it shouldn't be a testnet blocker here, but we should add some type of validation when inserting the ledger entries into the database to ensure that we always have "pairs" of entry + expiration.

This is Core's responsibility, I don't think we should be testing the metadata. Regardless, please create a separate ticket if you think this is important and let's get this merged.

Also - we could use some clarification from core whether the access to the expiration ledger entry is counted as an additional entry access or not. ( it might be a special case ).

I doubt it since the host doesn't track them and AFAIU otherwise integration tests wouldn't had worked. But this is a question for @sisuresh

Accessing the ExpirationEntry does not count as an additional entry access wrt the number of entries accessed, but we do charge for the bytes written in compute_rent_fee (which I believe you already use).

As you've already identified yourself, querying both the ledgerEntry and it's corresponding ledgerExpirationEntry would be better done on the go side; I think that a good starting point for that would be adding an optional expiration ledger sequence to the LedgerKeyAndEntry data structure. Also - we could use some clarification from core whether the access to the expiration ledger entry is counted as an additional entry access or not. ( it might be a special case ).

@2opremio I've created https://github.com/stellar/soroban-tools/pull/916, which implements all the go-side code that would require to support retrieving the LedgerExpirationSeq while getting the ledger entries. I think that it would be best to fully merge your pr first before applying the other.

I don’t think this makes sense unless we also hide the expiration ledger entries from the end user. Otherwise the expiration field is redundant.

I don’t think this makes sense unless we also hide the expiration ledger entries from the end user. Otherwise the expiration field is redundant.

I wouldn't "hide" them explicitly, rather than allowing the user to explore remaining lifetime.
In particular, I'd like to save the js code of doing hash calculations and unnecessary parsing to get this value.

I understand the motivation. What’s the point of allowing to query expiration entries if you are providing the expiration as part of the result? I would go one way or the other, otherwise it’s confusing.

Also, you don’t need to query the endpoint twice, nothing prevents you from querying the expiration ledger entry at the same time as the corresponding entry.

Intentionally merged with a couple of CLI TODOs:

we should test bump and restore commands, for which I added ledger entry pattern match which I am not 100% will work
there is still some code pending clarification https://github.com/stellar/soroban-tools/pull/913#discussion_r1313241027 (waiting for your reply @willemneal  :) )


async-io dependency breaks build

Quite easy to reproduce - just build master and you will see many errors involving libc.

[patch.crates-io]
async-io = { git = "https://github.com/smol-rs/async-io.git", rev = "d9c9ed8" }
socket2 = { git = "https://github.com/rust-lang/socket2", rev = "77e3bb9" }

seems to fix my issue - I suppose we need to wait for async-io and socket2 to release new versions to fix this.

Upstream support for ESP IDF in the smol ecosystem is in an odd shape right now:
We need new releases for the following crates, as the ESP IDF changes are upstreamed in there, but not released yet:

socket2
polling
rust-errno

Crates where the ESP IDF changes are upstreamed and also released:

rustix

Crates where ESP IDF changes are upstreamed, yet their master branch is not usable yet:

async-io, because of this

In the meantime, what you can do is:

pin async-io to an old version as done here and here
Patch polling and socket2 (and possibly, the smol aggregation crate itself) as done here to earlier not upstreamed forks. also done here

@FrozenDroid Small progress update:

polling 3.0 was released recently, with all ESP-IDF-related changes upstreamed
The relevant ESP-IDF changes to rustix,socket2 and errno are also upstreamed and released.

Waiting for the release of async-io 2.0 now, which I hope is imminent, and which would probably be released together with smol 2.0.
That would complete the upstreaming story of ESP-IDF in the smol-rs echosystem.

https://github.com/ivmarkov/edge-net/commit/d96b37b724ee45556df092287015b5cf7ee09735


Some insights about spt-threshold behavior

Hello Joachim,
We had some more issues when testing some configurations with spt-threshold enabled. We tried to understand the mechanisms used to do the switch to SPT and realised a few things. I have a branch with a few modifications:
https://github.com/stormshield-rlibaert/pimd/commit/b8c5442bf4175e26e8f49360ce7b1eb11918c852
To sum up:

There is a bugfix when parsing configuration
I removed a call to try_switch_to_spt which was done regardless of the provided configuration
I moved around the init_route initialization function

The last one is the most risky to me as I don't have a wide understanding of pimd, so maybe it brings breaking changes. But the reason for this is that the spt_threshold timer was set to the default value (defined by SPT_THRESHOLD_DEFAULT_INTERVAL), as the configuration of this is done through init_vifs.

Cool, I say we try it out and see what breaks! :sunglasses:
Please submit a pull request and I'll try to find some time to get back to my backlog (of yet unpushed fixes) in pimd.   It's easier for me to test as well.
(I still haven't gotten around to debugging the BSR + flood-to-RP-when-RP-changes issue, but this might trigger me to get back to it)

There, tested and merged! Sorry it took me a while to get around to, been busy getting up to speed at a new job.


DCOS-61006 - Add gRPC support to Admin Router

High-level description
This PR adds support for gRPC protocol proxying. Please check the design doc for details [1]. The justification for adding a separate listen socket can be found in the design doc as well. The port number was taken arbitrarily and will be made configurable in [2].
The support for gRPC required two additional changes which are included in this PR as separate commits:

the init_by_lua section had to be refactored in order to differentiate initialization of ee's auth.lua basing on the flavor of the configuration (agent/master)
the nginx subrequests ngx.location.capture are not supported in HTTP/2 context. Hence we needed to switch to using lua-resty-http also for policy query requests.

[1] https://docs.google.com/document/d/1XXaTSg43g6G5izQhuLVU5kKasc-gsq57nT30iIkOD8s/edit
[2] https://jira.mesosphere.com/browse/DCOS-61007
Corresponding DC/OS tickets (required)

https://jira.mesosphere.com/browse/DCOS-61006 Admin Router modifications required for authn&authz

Related tickets (optional)

https://jira.mesosphere.com/browse/DCOS-61007 Make the gRPC socket port configurable via install config param

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot request-review @mainred @ichernetsky @jongiddy
@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot request-review

@mesosphere-mergebot request-review @mainred

@mesosphere-mergebot request-review @ichernetsky

@mesosphere-mergebot request-review @jongiddy

@mesosphere-mergebot request-review @mainred

@mainred I can't add you as a reviewer, dunno why. Could you please have a look none-the-less?

@mesosphere-mergebot sync

@mesosphere-mergebot bump-ee

@mesosphere-mergebot sync

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mainred Could you please have another look and approve if applicable? Thank you!


Document fields in the JSON report

Missing functionality

The JSON report has lots of fields but unfortunately, they are very confusing and hard to figure out their meanings. For example, I thought is_unique tells whether a column has unique value or not, but it doesn't seem to be the case. Also, I have no idea what n_unique means. Based on the name, I'd guess it is then number of unique values, but it doesn't seem to be the case either.
Proposed feature

Document fields in the JSON report.
Alternatives considered

Additional context

Agreed, it would be a great addition to have a JSON schema. Contributions welcome.
To answer your question: n_unique is the number of unique values (i.e. occur exactly once). You might be looking for n_distinct, which is the number of distinct values.

Thank you, @sbrugman! I confused myself on unique vs distinct.


Better initial window sizes

The Dialog code has special logic to precompute the size of a window based on contents, and then use that for initial window size -- use that more widely for other windows -- have been lazily relying on window size memory but would be better for it get a reasonable size first.
Failing that, could have better initial guesses that are used instead of the same default for everything.

much improved -- most cases are prefect-sized, but sliceviewbase (and tableview) cannot as they size the view based on avail size -- so they only allocate post render.

Hi, I found that setting the width and height of the main form does not take effect. Even if I have never compiled and run the program before, that is to say, even the width and height parameters set during the first rendering of the main window do not take effect. It seems that it always renders and loads the default window size according to the same width and height value.How can I make it render according to the value I set? Thank you. Examples are as follows:

Yeah it now uses auto-sizing for all windows.  it will also remember the last size automatically.  You can call a SetSize (or something like that) method directly on the Window to resize.

should be a SetPos method.  again it remember last pos after first time.


Disable sponsor skipping and only skip ads?

Is it possible to configure the app to only skip ads and not sponsored segments?
Right now I added a non existing category to get this behaviour, but this does throw a lot of errors in the logs and tries to contact the api, while this would be unnecessary when only skipping ads.
As a side node, could the categories be better documenten? I could not find anywhere what the possible categories actually mean.

Sure! I'll add an environment variable for that in the next update.
I also just came across the SponsorBlock category breakdown, which seems a lot more helpful. If you agree, I'd happily add a link to that in the readme.

That would be amazing!

Ha wait a second, I've already added a flag for this! I just didn't add it to the main readme, since I assumed it would be less commonly used. Try --skip-sponsors=false or CSS_SKIP_SPONSORS=false!


EPIC: Support HA

The below items should be broken out into separate issues, once we get closer to working on this feature
Phase 1

[ ] Support etcd as a backend through manual configuration by user

[ ] Need to expose configurations params needed for etcd. Implement as generic framework that supports all paras in the form of  --<component>-<component flag>

[ ] Support peer discover that Rancher 2 has. Can reuse the framework from nomran, but need to implement the component that does the discovery
[ ] Add logic in agents to allow for auto-discovery of all api nodes. Then, use client-side logic to round robin between them if leader goes does. Effectively

Phase 2

[ ] Embed etcd in k3s image and simply SSL setup for "easy" HA configuration

Could https://github.com/rqlite/rqlite be a alternative to etcd?

AFAIK @Sh4d1 k3s has already dropped etcd for sqlite 😏

Can K3S currently support HA when using sqlite3? If it is not currently supported, is there any plan support in the future?
Does K3S still need to use ETCD now and in the future?

k3s doesnt support HA at all yet. When it does, it will (at least initially) do so with etcd as a the datastore. But, we have plans for making this seemless and painless. Basically, it should be pretty transparent to the operator.

Package the etcd in the single k3s binary, just like the containerd/Flannel?

can't etcd run as a docker image inside k8s? actually kubeadm starts etcd within a kubelet manifest file, wouldn't that be possible aswell? so k3s just needs to write a manifest file every boot and put the etcd image into the airgap tar. so the only thing k3s still needs to handle is how to get all the certificates distributed.

I would like to second the suggestion for rqlite for the HA system.  For those not familiar with it, rqlite is a raft'd SQLite 3.  It uses raft to provide for a pretty high performance clustered SQLite DB.
More details can be found at;
https://github.com/rqlite/rqlite

There is also Consul from Hashicorp that is very similar to etcd but far more easy to manage. Where I work, we managed automatically etcd clusters and Consul clusters, and Consul is really nice to operate from an automation point of view. It can automatically upgrade from a blank state to a clustered state. To the contrary, etcd needs to be started in the right configuration directly, and this requires synchronization beforehand that's difficult to have since etcd is not there. In the end, we managed to automatically operate etcd using Consul as preexisting layer.

would HA in this design require 3 nodes? 2 nodes would be rather nice for many of our applications.

I notice the v0.6 release added support for postgresql/mysql. The other part of this for 0.7 is support for multiple k3s masters?

I've been working on a project that uses a gossip network and an embedded etcd binary to make working with HA etcd clusters easier.  I wanted to throw it out there in case there was interest in using it for something like this:
https://github.com/criticalstack/e2d

Suggestion for the HA system : Consul (Hashicorp)

Hope this would be releasing on Milestone 1.0, we can't able to push this stack into our production iot systems , because of HA unavailability.

I want to link to another feature request - the first step should be to investigate eliminating sqlite entirely . etcd can be embedded into the k3s binary just like sqlite in non-HA mode.
That will make the code easier to switch to an external HA etcd.
https://github.com/rancher/k3s/issues/845

v1.0.0 will be released shortly. Please reference the updated k3s docs here: https://rancher.com/docs/k3s/latest/en/
We describe the various configuration options to set up HA if some down time for the control plane is not permitted: https://rancher.com/docs/k3s/latest/en/installation/ha/
The docs have also been updated to address other install options.
Thank you everyone for all of your support!


Poor Results with ZED Depth Camera

Thank you for sharing your work @j96w.
I'm able to visualize results for YCB objects using my script here.

However, I have issues when I try to visualize results with a ZED camera using my script here. I wonder if it's due to the camera intrinsics baked into SegNet which is highlighted in #150.
I've cropped the 720 x 1280 aspect ratio of the ZED camera to 480 x 640. But SegNet returns these classes labels 'Objects: ', array([ 0, 15, 16, 19, 20, 21]) which the soup can has an ID of 4 as per classes.txt.
depth = depth[50:530, 100:740]
rgb = rgb[50:530, 100:740, :] # height x width

rgb1 = np.transpose(rgb, (2, 0, 1))
rgb1 = norm(torch.from_numpy(rgb1.astype(np.float32))).cuda()
seg_data = self.model(rgb1.unsqueeze(0)) # SEGNET MODEL
seg_data2 = torch.transpose(seg_data[0], 0, 2)
seg_data2 = torch.transpose(seg_data2, 0, 1)
seg_image = torch.argmax(seg_data2, dim=-1)
obj_list = torch.unique(seg_image).detach().cpu().numpy()
label_img = seg_image.detach().cpu().numpy()
print("Objects: ", obj_list)

Could you share your visualization code?I failed to open the link.


Commands - Subfolders

Hello. Below is my suggestion of what could be added to Spark to improve organisation:

Adding subfolders to commands. Basically, within the commands folder, the easy ability to create subfolders inside it to keep commands neatly in different sections for example:

Bot
Commands
Moderation
Utility
Example 3 etc...

AJFire14#1004

Added in v0.1