EXT: Resource Graph

Description
Changes proposed in this pull request:

Add ResourceGraph handling to Extensibility
Simplify our builtin config of relations
Use per item comparison instead of list $filter in ext dataSources
Fix APIRules graph not showing

TODOs

optimize adding graph styles? Now multiple duplicate classes are added (which may not be a problem).

Related issue(s)
Closes #1736

/retest

/test pre-busola-integration-test-k3s-namespace

/retest

/test pre-busola-integration-test-k3s-namespace

/retest

/test pre-busola-integration-test-k3s-namespace


Change assertions to raising appropriate Exceptions

Change assertions into raising appropriate Exceptions.
Closes #15

Looks good. Thanks for doing this!


I want to maintenance of the project

@paulirish  I think the maintenance of the project, if there is no better choice. I with he also for a long time. While on the function and the underlying implementation not carefully studied, but I really feel he is a good project. Due to technical, I might have to him temporarily unable to do a lot of updates, but I will try my best. Thank you

@loveMrStar great. the best way forward is to start a brand new repo and begin building a v2 of the project there. as it matures, we can start to migrate this project into yours.

Really appreciate it

I wish I could help!!! I'm more of a Ruby developer, but I could help with updating the website for infinte scroll.

There's this other infinitescroll plugin:
https://github.com/pklauzinski/jscroll
In case for any reason you want brand new code: I've written a minimal version of an infinitescroll plugin here:
https://github.com/philippze/infinitescroll

Thank you for your interest and willingness to contribute Infinite Scroll. After discussing this matter with Paul, he's allowed me to take over the project. Read the blog post: Metafizzy taking over Infinite Scroll. I'll be releasing a brand new version in the coming weeks.
Moving forward, your feedback contributions are still appreciated with the new version. 🌈🐻


Rename triggers to include schema for uniqueness in schema based mult…

Addresses  #370 where triggers were being overwritten due to their naming convention.

Change trigger name from {table}_{name} to {schema}_{table}_{name}
test_base.py 33 tests passed successfully after the change.

@toluaina can you help me review? Tks!


Threshold UV index does not work?

for whatever reason the threshold for UV index does not work.
I've set it to 6 but Homebridge shows occupancy with value of 0.
I thought the bug is the typo of tresholdUvIndex because of the missing h after t but after editing the config and changing it to thresholdUvIndex it won't work either.
I'm using weatherunderground as API, with Homebridge 1.3.4 and newest plugin version v3.2.6
thanks.

There were bugs in threshold handling.
Can you paste your config here and I can verify it?

Hi David,
only if it doesn't take another 3 years for you to reply again 🤣

Hi,
Thanks for your feedback. I've only recently got involved in this project as an open source contributor. Using my limited skills I am just trying to help and make the product better. I actually only use the Tempest weather station, but thought while I'm working on the code, I'll take a look at some of the issues.
If you are still running into issues, I could look into to it for you.
David

I believe this is the same issue as #122.
If it is a problem still after #122 has been resolved, please re-open.


Future update not reflecting

Hi,
Thanks for this great plugin. But I am facing one issue in this.
Once you bind the SnapList on page it is not updating when data changed.
I tried this with streambuilder, futurebuilder & async data list.
In FutureBuilder & StreamBuilder, its loading the data & list count is changed but SnapList items are not changing. Also it keep firing positionUpdate callback after list refresh.
In async data list, one the page load I am getting the data from server & when I call setState its showing first item. No gesture is working & positionUpdate callback fires with index as "-1".
I am trying now with this project's branch. Will let you know if I found something.

That is fixed, thank you!


Is there a way of being notified when a product is already purchased?

Greetings!
First of all, thanks for the hard work!
I'm not able to find a way of being notified when a non-consumable product's been purchased before.
For example, if after a re-install I try to buy an already owned product, the message of "You already own this. We'll restore it for free" is shown, but I'm not notified in any way, so I can't restore anything client side.
Is there a way of being notified about this? I'm using version 2.0.2 and I'm testing IOS right now
Thank you in advance!

@Donerkebap13 I think you need to use getAvailablePurchases() method to get purchases previously made

@Donerkebap13 I think you need to use getAvailablePurchases method to get previously made purchases

I see. So, when do you recommend to call and process this? During startup of just before trying to purchase? I'd put it on launch but I don't know if I'm missing something. It's my first time dealing with non-consumable in app purchases.
Thanks in advance!

@jesuslou I would use it during startup if you want to properly display the app state.

@xPutnikx when I call it on startup on iOS, I get a popup that the user must sign in on iTunes (while it already is signed in). can I prevent this? Can I check if a user purchased something without logging in?

@boylenssen afaik that only happens in sandbox mode with sandbox accounts. I've released an app calling getAvailablePurchases on startup and no itunes popup is shown


Put all local storage keys and functions into one file in util

Pulling out all local storage keys/const and local storage-related functions into one file local-storage-keys.tsx under the util directory. This makes the code easier to maintain in the future

Thank you for making the change, I think this is 👍  once CI passes

Fixed conflict


MITK and VTK compiling

If I have already compiled VTK on my computer, will it conflict with my local VTK environment when compiling MITK, and do I need to configure anything in cmake？

MITK uses a super-build approach. Instead of configuring and building MITK only, the MITK super-build also downloads, configures, and builds (nearly) all of its dependencies. Hence, an MITK build can be considered isolated and does not interfere or conflict with any existing builds of its dependencies.
Exceptions prove the rule, though. The two most prominent counter examples are probably Qt v6.6+ and OpenSSL, which are not included in the MITK super-build but searched for during the configure step.
If you want to use your own build of VTK with MITK, you can do so by setting EXTERNAL_VTK_DIR during the configure step. This is considered an advanced feature, though, and we strongly recommend to go with a plain vanilla default standard build first before getting creative with building MITK with more complex configurations.
For more information, see the Build Instructions.


Add cmake support

The new esp-idf build system is based on cmake.
Components require their own CMakeLists.txt with their dependencies.

Thank you @gierdo


ch11 运行loop_closure出错

terminate called after throwing an instance of 'std::runtime_error'
what():  Vocabulary::load Could not open file :./vocabulary.yml.gz for reading
已放弃 (核心已转储)

可能是路径不对,改一下试试?

是不是需要先解压一下？

如果是在build文件夹里面，正确的路径应该是 ../vocabulary.yml.gz


Pull request jobs run with errors after completed all default steps

[ERROR] Could not create local repository at /var/jenkins_home/.m2/repository -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/LocalRepositoryNotAccessibleException

sudo chown -R your_user:your_user ~/.m2
sudo chmod -R a+rws ~/.m2
./stop.sh
docker-compose rm -fv
./start.sh

Can't repro now.


Tweak checksum to fix build

Thanks a lot @togatoga!
I still haven't been able to figure out how to reliably get Gradle to update the gradle.properties file automatically (and for some reason, my machine doesn't complain).


[BUG] TopologyException. Failed to convert polygon for object [...] to Shapely format

I have a dataset in the COCO format with instance masks.
I am using compute_max_ious() with use_masks = True:

It does work, however I am getting lots of errors of the sort:

TopologyException: side location conflict at 0.14662885273972603 0.34934043632673767.
This can occur if the input geometry is invalid.
Failed to convert polygon for object '629f6e19dd2937174e7bb41f' to Shapely format

Does that mean the compute_max_ious() function would skip the IoU computation for every sample with this error?
Why does it fail? All my masks are simple pixel masks.
How can I visualize the masks it fails to convert? Is the object '629f6e19dd2937174e7bb41f' an ID of the sample?
Is there any recommendation of how to circumvent the problem? Maybe to convert all pixel masks to polylines or something else?

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Win 10
FiftyOne installed from (pip or source): pip
FiftyOne version (run fiftyone --version): FiftyOne v0.16.1
Python version: 3.7.13

All Label objects (Detection, Classification, Polyline, etc) have IDs; that's what's being shown in the logs.
You can use select_labels() to retrieve only the objects with the specified IDs. For example:
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset("quickstart")

sample = dataset.first()

ids = [
    sample.ground_truth.detections[0].id,
    sample.ground_truth.detections[-1].id,
]

view = dataset.select_labels(ids=ids)
print(view)

Can you share some example data that has this problem:

The documentation explains the error-handling behavior. By default, any IoUs that can't be computed are treated as zero.
compute_max_ious() currently computes IoUs for instance segmentations (Detection objects whose mask attribute is populated), by first converting to polygons using shapely. But, shapely does not like something about your mask's geometry...
Perhaps the implementation may need to be updated to use masks directly rather than assuming that polygon conversion is possible, but, as discussed in https://github.com/voxel51/fiftyone/issues/1862, this is complicated.


New ThemeIcon color in tree views can pollute other nodes

Setting the ThemeIcon color on a node, can cause other nodes to get that color when the tree is re-rendered.

To verify, install the following extension (remove the .zip extension)
simple-0.0.1.vsix.zip
And make sure the Compare icon stays uncolored (white or black depending on your theme).


Fix rounded rectangle of GraphicsContext is a little off at the bottom side (fix #3795)

This correction improves the symmetrical appearance of the corners.
However, some issues remains even with this fix:

The size of input rectangle doesn't match with the output rounded rectangle bounds
In some cases the rounded rectangle has non symmetrical corners.
In some cases, some pixels are missing in the stroke of the corners.

Fix https://github.com/aseprite/aseprite/issues/3795

Merged in f89c72e5bfdc6d8923fe0b3fbef15ec38333630a


vim_customizable: Include manpages in the output

Motivation for this change
Fixes #24691.
Rebased fix from #26111, resolved conflicts, addressed review comments from that pull request.
Things done
Tested the following configurations:

vim_configurable.customize { name = "vimx"; }
vim_configurable.customize { wrapGui = true; wrapManual = false; name = "vim"; vimExecutableName = "vimb"; gvimExecutableName = "gvimc"; }

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Related https://github.com/NixOS/nixpkgs/pull/164994


there are some errors when i run the demo,i don't know how to fix it! please help me!  thanks

SecretBox.zip

the error is          -[_SwiftValue itemImage]: unrecognized selector sent to instance     . And this occur when i set the item image for tabbaritem.If i remove these codes which set the image,and there will be nothing wrong.

Hi @qzslz! Thanks for the demo project. Please, update to 1.1 version. There was a bug related to nullability directives in objective-c


refresh button is not at the top

Describe the bug
The refresh button in the home page is displayed below the joined room. Is this the intended behavior?
To Reproduce
Steps to reproduce the behavior:

Join a room
Go to the home page

Expected behavior
The expected behavior would be that the button remains at the top.
Screenshots

What device are you on?
Ubuntu 18.04
Same thing happens on my Android device

Can I work on this?

Although, after going through the code, I am pretty sure this is the intended behavior and not a bug.

Although, after going through the code, I am pretty sure this is the intended behavior and not a bug.

yea, I know. But doesn't this look weird?

Although, after going through the code, I am pretty sure this is the intended behavior and not a bug.

yea, I know. But doesn't this look weird? Maybe they'll change it in the new design

Well if you think about it, refresh action is tied to list of available rooms. When you are in room already it doesn't make sense to refresh it eg. put it in same group below button


how to predict vertical or 90 degree rotated image

hi, how to improve prediction for this case :

I tried to run the model on this image but the result was not so good (it predicted as "i"). I would like to know if there is a certain mechanism to force the model in recognizing specific character only such as -.,0123456789
thank you

In practice, we firstly rotate this image by 90° or -90°. And then select the result with a higher recognition score.


trace_tcpconnect: Fix missing src and dst ip and port on error

When an error happens in the connect call the src ip addr + port and the dst port is cleared in the kernel before returning. Therefore we need to save all the information at a point when it is available.
Edit: Actual source line is: https://elixir.bootlin.com/linux/v6.11/source/net/ipv4/af_inet.c#L738
See https://elixir.bootlin.com/linux/v6.11/source/net/ipv4/tcp_ipv4.c#L350
Before
RUNTIME.CONTAINERNAME        SRC                                   DST                                   COMM                  PID        TID        UID        GID ERROR
quizzical_cannon             <IP_ADDRESS>:34286                      <IP_ADDRESS>:80                    wget                41798      41798          0          0
quizzical_cannon             <IP_ADDRESS>:35018                      <IP_ADDRESS>:80                     wget                41798      41798          0          0
quizzical_cannon             <IP_ADDRESS>:0                             <IP_ADDRESS>:0                         wget                41882      41882          0          0 ECONNREFUS
quizzical_cannon             <IP_ADDRESS>:0                             <IP_ADDRESS>:0                        wget                42026      42026          0          0 EHOSTUNREA

After
RUNTIME.CONTAINERNAME        SRC                                   DST                                   COMM                  PID        TID        UID        GID ERROR
quizzical_cannon             <IP_ADDRESS>:53208                      <IP_ADDRESS>:80                     wget                40138      40138          0          0
quizzical_cannon             <IP_ADDRESS>:56386                      <IP_ADDRESS>:80                      wget                40138      40138          0          0
quizzical_cannon             <IP_ADDRESS>:47944                      <IP_ADDRESS>:80                        wget                40195      40195          0          0 ECONNREFUS
quizzical_cannon             <IP_ADDRESS>:32952                      <IP_ADDRESS>:80                       wget                40736      40736          0          0 EHOSTUNREA

I think tracepoint/sched/sched_process_exit should clean up src_dst_per_process and sockets_per_process in case kretprobe/inet_stream_connect does not get called due to maxactive being too low (example of kretprobe-maxactive issue).


Modbus TCP "Length" field was calculated wrongly.

"Length" field in MBAP header didn't calculate "UnitId" field

I have published new version to the pub. Thank you.


Prefix icon doesn't align properly with Input hint or text

The prefixIcon property doesn't align the widget with the hintText or the input entered whereas prefix does.
It needs rectification since the prefix widget is shown only when the TextField has the focus.
Steps to Reproduce

Execute the attached code on DartPad
You will see the country flag/code misaligned to hintText.
Change prefixIcon property name to prefix.
Run the code again and the flag/code will get aligned to hintText or any input when you would click the input.

Expected results: prefixIcon widget aligned with hintText.
Actual results: prefixIcon widget pushed a bit downward (or upward in other implementations) as compared to hintText.

Proposal Solution: Either the prefixIcon position should be fixed or a property alwaysShowPrefix (false by default) could be added to InputDecoration and the prefix widget should be shown always when the property is set to true.
Additional: Similar issue https://github.com/flutter/flutter/issues/91174
The icon property displays the same behavior as prefixIcon.

Code sample
import 'package:flutter/material.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark().copyWith(
        scaffoldBackgroundColor: darkBlue,
      ),
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: Center(
          child: Padding(
            padding: const EdgeInsets.all(8),
            child: TextFormField(
              decoration: InputDecoration(
                hintText: 'Your Phone number',
                prefixIcon: TextButton.icon(
                        onPressed: () {},
                        icon: ConstrainedBox(
                          constraints: const BoxConstraints(
                            maxWidth: 24.0,
                          ),
                          child: const Text('🇮🇳'), // Flag network image
                        ),
                        label: const Text('+91'),
                      ),
              )
            ),
          ),
        ),
      ),
    );
  }
}

Logs

Closing as setting textAlignVertical: TextAlignVertical.center for the TextField fixes the concern.

I'm reopening the issue since setting textAlignVertical: TextAlignVertical.center seems to resolve the issue on DartPad only.
In a real device, prefixIcon and hintText/input are not perfectly aligned like prefix property does.

Hi @AmitSonkhiya, as you mentioned above, I see #91174 addressing the case you described.
https://github.com/flutter/flutter/issues/91174#issue-1014212306

Currently the prefix property of InputDecoration correctly aligns the prefix with the text while the prefixIcon property does not.

Closing as a duplication, so please follow up on it for further updates.


Problem while training: zero mAp.

Hello ,@victordibia, and thank you for your work! I have almost the same issue with training my custom hand detector as in Issue #5. I use a pre-trained model on EgoHands dataset you have provided and I do transfer learning on EgoFinger dataset. This dataset includes hand images taken from egocentric viewpoint. Image size is 640x480, format is png. For testing purposes I train on a little subset. But after every epoch I get zero mAp.
Also, I tried to learn by taking vanilla SSD and train on my dataset, but result is the same.
I am pretty sure that I have generated right csv files and tfrecords. On tensorboard "Image" tab I can view visualization of ground truth boxes and they are correct.
Are there any thoughts for what can be done? Thank you in advance!

The Ego finger dataset looks interesting!
Zero mAP looks like an issue with your pipeline config or issues with incorrect labels.
Are you able to follow the TF Object detection API examples and successfully train a model?

Hi @Yemen-Romanian ,
If you have any updates to share, please feel free to post here...
Closing for now


Release monix task is failing master build

See here.
@NeQuissimus Do you think you could disable this until we get it fixed? I'd like to do a release.

I commented out interop-monix for now. I genuinely do not know why it is failing the way it is. But it is also late here and I am tired :)

@NeQuissimus Thanks for the quick fix...er bandaid. 😄 🙏

Hello there, any chance to have Monix interop back on its feets?
Thanks in advance! :)


Roll Flutter Engine from 3bb936567e60 to 9e63c1ae1480 (1 revision)

https://github.com/flutter/engine/compare/3bb936567e60...9e63c1ae1480
2022-12-28<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from c1-ICa-ToxzhYLG7F... to jV7nfgH1Tb3Lw0w_S... (flutter/engine#38532)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from c1-ICa-Toxzh to jV7nfgH1Tb3L
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Corrupted Backup - No Restore Rancher Management Cluster / Rancher Server

Forward-port issue of https://github.com/rancher/rancher/issues/34554

Closing, duplicate of https://github.com/rancher/rancher/issues/34696


Video calls in pandemic crisis

Dear Telegram developers,
The corona virus pandemic is forcing many countries to forbid all sorts of meetings. Worldwide, schools are closed, and people are urged or obliged to work from home. Video calling and conferencing got all of a sudden a central topic in many areas.
Probably, the telegram team plans to deliver video calls when fully ready. Yet, these coming days might prove to be decisive in the adoption or development of one or another communication platform. Aside from the fact that Telegram users would particularly love to see video calls implemented now, it might be a good strategy, so that more new users could find a solution to their present needs in Telegram, instead of moving away to other video-call ready networks.
Pardon me if I'm not writing this post in the right place and in the usual style. Yet I'm sure that you will make your own wise decisions.
Best regards

There is no way this can be implemented in TDLib, unless it is implemented server-side first. You can send general feature suggestions through in-app support or by menthioning @telegram on Twitter.


Migration from 1.3.0 to 2.2.0

I've change the version of me.tatarka.bindingcollectionadapter:bindingcollectionadapter:1.3.0 to me.tatarka.bindingcollectionadapter2:bindingcollectionadapter:2.2.0.
But actually I got a errors like:
:app:compileDebugJavaWithJavacC:\Projects\xxx\adapter\CustomBindingEvenRowAdapter.java:5: error: cannot find symbol
import me.tatarka.bindingcollectionadapter2.BR;
                                           ^
  symbol:   class BR
  location: package me.tatarka.bindingcollectionadapter2

Could someone help me with migartion?

I've got a error too:
Execution failed for task ':app:compileDebugJavaWithJavac'.
> java.lang.NoSuchMethodError: android.databinding.BindingBuildInfo.enableDebugLogs()Z

I use classpath 'com.android.tools.build:gradle:2.2.3'

@evant


Allow for Streams to end

This introduces the notion of ending a stream transmission.
The rationale is that this semantics of "potentially infinite, but may end" data sources is very useful. Examples:

Reading a Socket that closes at some point,
Lazy I/O (e.g. processing a huge file without reading the whole thing into memory).

This is a nonbreaking change, as the semantics of any of the functions remains unchanged.
One concern is "ok, so now Streams are just Lists?". And yes, they are isomorphic, but I think the distinction is necessary for the different use cases (the semantics of Stream and List methods differs, with Stream still being "usually infinite" and produced by IO actions).

I believe supporting streams as lists would be an extremely valuable trait for lists to have.

@piotrMocz has a point, I'm closing this without merge, maybe revisit the concept with the new compiler.


Refactor mrseql

Reference Issues/PRs
See #257
What does this implement/fix? Explain your changes.
Refactor Mr-SEQL to pass all the tests for estimators.
Add multivariate time series support. MrSEQLClassifier can classify multivariate time series now.
Update notebook example.

Hi @lnthach, thanks for the fixes, looks good to me. Let me know when you're all done so that I can merge it 🙂

Thanks @mloning . Sorry for the late response. Yes it's good to go from my side.

No worries, thanks for the fixes! 🙂

Hi @lnthach,
I've noticed that the test_mrseql_on_gunpoint now takes considerably more time. Any idea why?
Before your changes, it ran in roughly 30 secs, not it needs 240 secs.

Hi @mloning ,
Many thanks for pointing that out. It appeared that the way I sliced the dataframe wasn't optimal. I have implemented the fix for this issue. Can I make my pull request on dev ?

Yes please! Thanks @lnthach


[pulsar-broker] Support Disable Replicated Subscriptions

Support disabling "Replicated Subscriptions".
Motivation
https://github.com/apache/pulsar/issues/8030
"Replicated Subscriptions" is always enabled because enableReplicatedSubscriptions is not used.
Modifications

When enableReplicatedSubscriptions is false,

Brokers don't create ReplicatedSubscriptionsController.
Brokers don't check marker messages.
Brokers send NotAllowedException to consumers that use replicateSubscriptionState(true)

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


configure genesis

allocations to

treasury,
founders
multisig
total cap =10M TEER,

ED = 10 mTEER,
fees: start with nominally equal to statemine for a balance::transfer. adjustment later

done, but ED is 1 mTEER now and we start with only a few hundred TEER and create the token after genesis


Solution - Challenge 3 - It is rocket science!

Providing solution to guarantee software stability on any changes.
Cucumber to guarantee software stability separately and Contracts to guarantee working together.

@Hosnidokht The Cargo and Booster services are supposed to fulfill their responsibilities.
The Cargo App or service has a Readme telling you what it should do and creating a Booster Tank is not one of them.
The Booster App or service has a Readme  as well telling you that it should be able to fill tanks. Your model shows the tank having only a title instead a tank should have fuel parts and each fuel part has a priority and a deadline etc... you can check the model in the go app to model your solution better.
We do appreciate what you showcased on the testing side, but we'd like you to improve the application design and use a persistence layer for one of the service and see how we can test with all those constraints.
Simplifying the application too much leads to simple testing, the challenge is how to still fullfill the requirements and be able to cover and test.

@Hosnidokht The Cargo and Booster services are supposed to fulfill their responsibilities.
The Cargo App or service has a Readme telling you what it should do and creating a Booster Tank is not one of them.
The Booster App or service has a Readme  as well telling you that it should be able to fill tanks. Your model shows the tank having only a title instead a tank should have fuel parts and each fuel part has a priority and a deadline etc... you can check the model in the go app to model your solution better.
We do appreciate what you showcased on the testing side, but we'd like you to improve the application design and use a persistence layer for one of the service and see how we can test with all those constraints.
Simplifying the application too much leads to simple testing, the challenge is how to still fullfill the requirements and be able to cover and test.
Additionally some infrastructure to run the services and their persistence layers if used would be much appreciated. (docker-compose would be enough)

@albertaouad  My main objective in the given approach was giving a solution of how we are able to test our services separately and also together, as mentioned in Challenge-3 . I thought proposing a solution is the most important in this challenge than implementing the exact model (as it is just adding more fields in the request responses).  That's why I just follow one of the routers in routers .
I had the same idea for the infrastructure phase also. Indeed, using a YAML file to configure services is a necessity as a part of whole solution but in my opinion it was not the main concern in testing phase in this challenge. I was not sure if it is really needed in this challenge or not.
Anyway, I do agree with your comments and will apply them.

@albertaouad  Done. In response to your first comment I also update my Solution file which is more clear in the given approaches. In short, for testing services with each other I used Contracts instead of Cucumber, That's why you could see a booster contract in cargo service and vice versa.


Add more russian models

Checklist

[X] Run tests locally to make sure nothing is broken using make test.
[X] Run the formatter to format the code using make lint.

Adding datasets checklist

Reason for dataset addition: ...

[X] I have run the following models on the task (adding the results to the pr). These can be run using the mteb -m {model_name} -t {task_name} command.

deepvk/USER-base
deepvk/deberta-v1-base
DeepPavlov/rubert-base-cased
DeepPavlov/distilrubert-small-cased-conversational
DeepPavlov/rubert-base-cased-sentence
cointegrated/LaBSE-en-ru
sergeyzh/rubert-tiny-turbo
sergeyzh/LaBSE-ru-turbo

[X] Run tests locally to make sure nothing is broken using make test.
[X] Run the formatter to format the code using make lint.

Adding a model checklist

[x] I have filled out the ModelMeta object to the extent possible
[x] I have ensured that my model can be loaded using

[x] mteb.get_model(model_name, revision_id) and
[x] mteb.get_model_meta(model_name, revision_id)

[x] I have tested the implementation works on a representative set of tasks.

I think right now it is still https://huggingface.co/datasets/mteb/results? cc @orionw

I think right now it is still https://huggingface.co/datasets/mteb/results? cc @orionw

I haven't move the folder yet, so there is great for now.  I'll try to get to it soon!

Perfect - I will merge this in for the model implementations - but the results will probably be moved.


Webversion: GIT-stuff

Issue
The following is shown in the console, but it is never mentioend int he web GUI: On the contrary it reports all changes were done there.
**
Committing config.py before applying edits.
Cmd('git') failed due to: exit code(128)
cmdline: git add /Users/S/config.py
stderr: 'fatal: Unable to create '/Users/S/.git/index.lock': File exists.
Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.'
Version and model info
aider 0.47.1, OSX
git version 2.39.3 (Apple Git-145)

Thanks for trying aider and filing this issue.
It looks like your git repo may be corrupted. Or maybe you were using aider to edit your repo at the same time as some other git tool?

You're right - it was VSCode :)  What I'd wish for however is for the message that it happened would appear in the webversion also and not only in the console window. (Or is that text actually output by GIT and not aider?)

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


java update automation config

Choose a PR Template
Java changed automation script folder structure:
https://github.com/Azure/azure-sdk-for-java/pull/41087
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

/azp run

/azp run


SVG previews are just white background, no actual image is shown

Microsoft PowerToys version
0.74.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
File Explorer: Preview Pane, File Explorer: Thumbnail preview
Steps to reproduce
SVG previews in both preview and thumbnails are all white, no image is shown, just white. This starts to happen after updating from 0.72.0 to 0.74.0. Tested on two different machines with different hardware, still the same - all white preview. If I open the file in Chrome or Edge it looks fine, but in previews it's just white background.
✔ Expected Behavior
Previews show actual image.
❌ Actual Behavior
Previews show only white background.
Other Software
Windows version - Windows 11 Pro, 22H2, 22621.2361.
If Chrome version is something important - using latest: Version 117.0.5938.92 (Official Build) (64-bit)
Right before the update of PowerToys I did Windows update on both machines.

Can you please share the files where this is happening for you? /needinfo

This has been fixed in 0.74.1. Please update PowerToys to the latest: https://github.com/microsoft/PowerToys/releases

Yes, now it is working as expected. Thanks.


Documentation i18n

Leave a comment if you're interested in translating. Please be prepared to stick around and translate changes in future releases, too.
TL guidelines:

A 1:1 translation is not necessary, feel free to be creative in your translation. It's better to communicate clearly than to attempt a 1:1 translation of the English docs.
It may be more effective to use English terminology for technical terms. Do so if it makes things more clear.
If you're not sure how to translate something clearly, leave it in English.
In the README please indicate whether or not your translation is a WIP and link to this issue so that readers know how to help improve the TL
In the README, clarify the situation for speakers of that language who wish to get support. If you can volunteer to hang out in IRC and triage GitHub issues, mention your username and timezone.
In the README, clarify that I should be emailed in English for distro packaging help, or drop it alltogether.
It's okay to give the FAQ a regional spin. It doesn't have to match the English FAQ. For example, it might be appropriate to give tips on IME configuration in your language.

When translating the README, please add this subsection, translated to your language:

[Language] help
[Username] provides support in [language] on IRC and GitHub in [timezone].
[Information regarding the compelteness of the translation and a link to this issue]

We are not going to localize any messages in Sway itself - error messages and logging will remain in English so that English-speaking developers are able to interpret them when asked for assistance.

[ ] ja_JP: @SirCmpwn

[x] README (#1317)
[ ] Man pages

[ ] sway(1)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security

[ ] FAQ

[ ] es_ES: @jdiez17

[ ] README
[ ] Man pages

[ ] sway(1)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security

[ ] FAQ

[ ] de_DE: @paradoxspiral

[x] README
[ ] Man pages

[ ] sway(1)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security

[ ] FAQ

fr_FR

Thanks, added it to the list.

I wrote up some TL guidelines in the OP, please review them.

[ ] uk_UA:

[x] README (#1325) @Hummer12007 (not up for anything else)
[ ] Man pages:

[ ] sway(1)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security

[ ] FAQ

[ ] el_GR: @kon14

[x] README (#1317)
[ ] Man pages

[ ] sway(1)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security

[ ] FAQ

I plan on doing the rest soon. Usually afking in IRC (kon14), GMT+2

Standard "this language is not English so here's how to get support" README subsection:

[Language] help
[Username] provides support in [language] on IRC and GitHub in [timezone].

In Japanese I'm going to write:

日本語助けの
SirCmpwnはIRCとGitHubでUTC-4に日本語助けがある。

Please include similar information in your translations.

Sorry for frequent updates to these guidelines, i18n is a WIP.

 I think I'm just going to write a new man page authoring tool this weekend.

I think it would be good to discuss where man page translations should be placed and agree on a naming format before we start submitting pull requests for those.
Is a manpage directory to be created in each project the best approach? Should translation filenames be formatted as Name.Chapter.Lang.txt similarly to how README.mds are treated?

Take a look at the branch.

I noticed the man directories, I'm asking more in regards to the filename policy.
Since there's still no translated manoage to make an example out of I'm not sure of what's expected of us. Perhaps I'm missing something?
It would also be nice if we could get some sort of documentation versioning or at least some way for users to tell if a translation is up to date. However that would needlessly complicate things and perhaps checking the last modification date of a translation and comparing that to the original file on this end would be all that's needed to keep things up to date.

There is one: https://github.com/SirCmpwn/sway/blob/localized-manpages/sway/man/sway.1.ja.txt
And I'll require changes to docs to sync them across all man pages when merging PRs in the future, even if they just put in the english docs.

(and I will cc the relevant language maintainer on that PR)

Will take on the Italian translation

For IRC, to redirect Italian support to GitHub:
Attenzione! Il supporto Italiano su IRC al momento non è attivo. Per favore, utilizza il tracker su GitHub

[ ] fr_FR: @abdelq, @nlgranger

[X] README (#1319)
[ ] Man pages

[X] sway(1) (#1333)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security
[ ] FAQ

Is pt_BR intelligible to pt_PT? We should aim to kill two birds with one stone.

There are small differences in the way some words are written (e.g., in some words, pt_PT exchanges the circumflex accent in favor of the acute accent), but the great difference resides in the slang. So I'll try my best to keep the translation neutral.

I've started with Czech translation.

[ ] cs_CZ: @gjask

[ ] README (#1337)
[ ] Man pages

[ ] sway(1)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security

[ ] FAQ

I start the translation into Russian.

[ ] ru_RU: @DarkReef

[ ] README (#1342)
[ ] Man pages

[ ] sway(1)
[ ] sway(5)
[ ] sway-bar
[ ] sway-input
[ ] swaymsg
[ ] swaygrab
[ ] swaylock
[ ] sway-security

[ ] FAQ

I have written a man page generator which I intend to switch sway to soon. It's still missing a couple of important features (tables, mainly), but it's looking pretty good.
https://git.sr.ht/~sircmpwn/scdoc/
Once it's done we should be able to translate man pages.

As my former identity @nuskorpios went AFK, I'm assuming the mantainership of portuguese translation through this account.

I can take place, from the pt_BR, since the last user was deleted :)

Great. I'll be looking into doing i18n for the man pages within a month or two.

I can take place, from the pt_BR, since the last user has been deleted :)

Nope, I'm right here, only a different username.

I can take place, from the pt_BR, since the last user has been deleted :)

Nope, I'm right here, under a different username. This week, I was updating the README to the lastest version, and I'm waiting for the man pages to settle in order to translate them too.
Edit: fix grammar, add context.

We both can do it :)
Two is better than one

We both can do it :)

Sure :))


Introductory fix to issue #2384

This is a draft intended to bootstrap the resolution of issue #2384
Fix to reflect changes of the Instagram API (hinted here https://github.com/yt-dlp/yt-dlp/commit/cf85cba5d9496bd2689e1070005b4d1b4cd3dc6d)
I know neither python nor the codebase so I didn't take the time to refactor anything (graphql_query now takes a wrongly named parameter query_hash) and just surrounded the API call with an if/else.
In the current state I only tested it to work with shortcodes (ex. instaloader -- -DAOdehwP6fC now works as expected) and nothing else.

Thanks for addressing this! This seems a valid approach to fix #2384. I rebased this branch to master.
With the changes of the typenames it raises many warnings like
Warning: <Post ...> has unknown typename: GraphImage

Just ping us when this PR is ready to be merged.

Obsolete now due to 9a74d9e1092e37d4e1f8b6c6431b18a4793781d7.


Historic point latency exactly of 17 minutes

If you decompile the executable you can find out that they are defining HISTORIC_POINT_LATENCY = 17.
By scraping the historic data every minute, you can easily verify that the historic points aren’t updated until 17 minutes later, in fact.

The sensor a key corresponds to its activation timestamp. Most historic points have a lifeCount perfectly divisible by 5 (I am simply dividing by 60 their difference from the activation timestamp) so you can guess correctly when the sensor sends an updated historic value.
Instead of sampling randomly the real-time values every five minutes like the xDrip+ WebFollower probably is doing, you have to consider this 17-minute delay and therefore the timestamps 17, 22, 27, 32, …
Like for the Libre 2 historic values sent via BLE, you can verify that that the saved historic values don’t always correspond to the realtime values at the same timestamp. Probably the new microntroller firmware now does the calibration computations by itself and it takes time to compute wighted averages... Perhaps 8 + 1 + 8 values?


RuntimeError: Given groups=1, weight of size [256, 1024, 3, 3], expected input[1, 512, 28, 28] to have 1024 channels, but got 512 channels instead

During the test, this error occurred. Why is this? How to solve？

Have you found a solution? This looks like a similar problem I had recently when certain image dimensions had the wrong order (channel was not before x and y). What were the input dimensions of your image?

Have you found a solution?


Adds pagination

Adds search results pagination, fixes #94 and fixes #14
I've also added some missing tests and cleaned up some unnecessary code.

Cool, this works! The designs show the pagination component both above and below the search results. Are you thinking to try it above first and see how it goes?
Also, I'm not sure where this was introduced, but the auto-filled dates in the facets modal are funky. Eg. after searching "ford" the date inputs look like:

Finally, this is a little late, but what do you think about giving the tile__hit-count a hover style similar to btn--orange? I know the orange colors are slightly different...
&:hover {
    background: $med-brown;
    border: 1px solid $med-brown;
    color: $off-white;

Thanks for the catch on the pagination - had not noticed it was at the bottom too! I've added it there as well and goshdarn flexbox is awesome.
I also added the background change on hover for the search tips - good suggestion.
The weird dates are coming from indexing. I need to dig into that a bit more and figure out what's going on. There's some Elasticsearch magic that happens around dates that I need to understand better.


C++ syntax highlighter breaks with certain macro definitions

VSCode Version: 1.35.1
OS Version: Linux (I guess also Windows)

Steps to Reproduce:

Open any existing C++ file in Visual Studio Code (no extension required).
Insert the following at some point in the document:

#define EXTERN_C extern "C"

Observe how, from that line on, the syntax highlighting goes bad.

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

Thank you @matter123 for fixing this so quickly! I have pulled in the latest grammar and added a macro to our test file.

This was quick.


fix(comp:textarea): close icon input content is overwritten(#1005)

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added/updated or not needed
[ ] Docs and demo have been added/updated or not needed

What is the current behavior?

close icon overwrites the text
What is the new behavior?
will not overwrite the text
Other information

发现好几个组件都存在这个问题，这个我来修复吧。


Error: http2 feature is not enabled

Issue description
When trying to use the IOTA SDK (version<EMAIL_ADDRESS>in a TypeScript/Node.js project, an error occurs indicating that the http2 feature is not enabled. This issue prevents the successful execution of the code that relies on the IOTA SDK.
Version
Version:
IOTA SDK<EMAIL_ADDRESS>Node.js: v18.16.0
Operating System: Ubuntu 20.04

Expected behaviour
The IOTA SDK should be able to make HTTP requests and communicate with the IOTA network without encountering the "http2 feature is not enabled" error. The code should execute successfully, allowing interaction with the IOTA network.
Actual behaviour
The error indicates that the http2 feature is not enabled in the IOTA SDK, preventing the code from executing successfully.
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
Yes, the issue can be reliably reproduced by running the code that uses the IOTA SDK in a TypeScript/Node.js project.
Steps to reproduce the issue:
Set up a TypeScript/Node.js project.
Install the IOTA SDK using the command: yarn add<EMAIL_ADDRESS>Write code that uses the IOTA SDK to interact with the IOTA network, such as retrieving node information or creating transactions.
Run the code using Node.js.
Errors
thread 'tokio-runtime-worker' panicked at /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/hyper-util-0.1.3/src/client/legacy/client.rs:554:33:
http2 feature is not enabled
stack backtrace:
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Found issue resolved in fork.


'README.md', One line is missing from Related Projects.

Very small document error.
'README.md'.
One line is missing from Related Projects.

@SimbaPing Pull requests are always welcome. What line is missing?


Unable to download VS Code

I am trying to download VS Code from the following link https://code.visualstudio.com/docs/?dv=win.
I'm using Microsoft Edge browser (Microsoft Edge 38.14393.0.0).
On clicking the Download button, I was taken to the download page https://code.visualstudio.com/Download  where I selected Windows as my OS.
The new page will keep loading on hours and no download has started.
I even tried the direct download link which has also got the same problem

No this is a different issue

Is this still a problem? Which country are you downloading from?

I have seen similar download issues with Edge. These download issue were not limited to VS Code downloads. Most of time these issue are resolve via restarting Edge and/or Windows.

@sgtoj Thanks for the info!
@pmkarthikeyan Are  you able to download in other browsers, such as Chrome/Firefox?

I'm having this issue as well. I've tried recycling processes, changing browsers, even changing DNS servers. Here's my summary:
For reasons I can't yet fathom, I am unable to download VSCode.
I go to https://code.visualstudio.com/download and click on 'Windows'
This takes me to https://go.microsoft.com/fwlink/?Linkid=852157
That page attempts to auto-download it via this URL:
https://vscode-update.azurewebsites.net/latest/win32-x64/stable
That URL ultimately returns a "Hmmm... can't reach this page" error, details:

There was a temporary DNS error. Try refreshing the page.
Error Code: INET_E_RESOURCE_NOT_FOUND

I tried switching browsers (Edge, Opera, Firefox) and switching DNS providers (<IP_ADDRESS> and <IP_ADDRESS> both in IPv4 and IPv6 variants) to resolve this, but the error remains.

@aviolentgrace Where are you based at?

Redmond, Washington, USA

@aviolentgrace Are you behind Microsoft VPN?

Ah, yes. I think I have a couple of msft auto-VPN solutions installed. Lemme try disabling them...


Error: Conversation roles must alternate user/assistant/user/assistant/..

Issue description
Running same base prompt with dynamic text in it to analyze it, getting the error at 100
Expected Behavior
I should received output in the format: "Answer: <<0 or 1>>. Text: <<text_being_analyzed>>" as I am receiving other outputs like;
...
Answer: 0. Text: Stars swim in the ocean of infinite poss...
Answer: 0. Text: Moonlit owls hoot the symphony of the en...
Answer: 0. Text: Cotton clouds float on the breeze of whi...
...
Actual Behavior
Error: Conversation roles must alternate user/assistant/user/assistant/
node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1555
throw new Error(args);
^
Error: Conversation roles must alternate user/assistant/user/assistant/...
at ~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1555:13
at FunctionValue.value (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1529:24)
at Interpreter.evaluateCallExpression (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1328:15)
at Interpreter.evaluate (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1493:21)
at Interpreter.evaluateBlock (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1300:34)
at Interpreter.evaluateIf (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1411:17)
at Interpreter.evaluate (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1466:21)
at Interpreter.evaluateBlock (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1300:34)
at Interpreter.evaluateFor (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1452:30)
at Interpreter.evaluate (~/node-llama-cpp-project/node_modules/@huggingface/jinja/dist/index.js:1468:21)
Steps to reproduce
initializeLlama.ts
import {fileURLToPath} from "url";
import path from "path";
import {getLlama, LlamaChatSession, LlamaJsonSchemaGrammar, LlamaLogLevel} from "node-llama-cpp";

let session: LlamaChatSession;
let llamaGrammar: LlamaJsonSchemaGrammar<any>;

const initLlama = async () => {
  const __dirname = path.dirname(fileURLToPath(import.meta.url));
  const modelsFolderDirectory = path.join(__dirname, "../../", "models");

  const llama = await getLlama({
    // gpu: false,
    gpu: 'cuda',
    // logLevel: LlamaLogLevel.debug
  });

  const model = await llama.loadModel({
    // modelPath: path.join(modelsFolderDirectory, "llama-2-7b-chat.Q4_K_M.gguf"),
    modelPath: path.join(modelsFolderDirectory, "Mistral-7B-Instruct-v0.3.Q5_K_M.gguf"),
    gpuLayers: 'max'
  });
  console.log("Model Loaded");

  const context = await model.createContext();
  console.log("Context Created");

  session = new LlamaChatSession({
    contextSequence: context.getSequence()
  });
  console.log("Session Initialized");

}

export {
  session,
  llamaGrammar
}

export default initLlama;

analyzeAndRate.ts
import { texts } from "./dummy/dummyText"; // link is just below the code
import { llamaGrammar, session } from "./llama/initializeLlama";

export const runAnaylyzerAndRate = async (text: string) => {
    const query = `Answer 1 if the given text is semantically correct and sounds like a job experience else answer 0, do not write any text in the answer except for 0 and 1. Text: ${text}`

    const startTime = Date.now();

    const response = await session.prompt(query);

    const endTime = Date.now();
    const duration = (endTime - startTime)/1000;

    console.log(`Answer: ${response}. Text: ${text.substring(0, 40)}...`);
}

export const run = async () => {
    console.log('Analyzing meaning full text!')
    for(let i=0; i<texts.length; i++){
        console.log(`Index: ${i+1}`)
        await runAnaylyzerAndRate(texts[i].text)
    }
}

import text from here -> dummyText.ts pastebin file only valid for 1 year from today
My Environment

Dependency
Version

Operating System
Windows 11

CPU
AMD Ryzen 7 7735HS with Radeon Graphics           3.20 GHz

Node.js version
20.11.0

Typescript version
5.4.5

node-llama-cpp version
3.0.0-beta.36

Relevant Features Used

[ ] Metal support
[X] CUDA support
[ ] Grammar

Are you willing to resolve this issue by submitting a Pull Request?
No, I don’t have the time and I’m okay to wait for the community / maintainers to resolve this issue.

This issue appears to be related to the chat template provided by the model, where it expects to receive a message array with non-standard role names.
To resolve this issue, you can print the chat template provided by the model and investigate it to find the role names that the model expects:
import {fileURLToPath} from "url";
import path from "path";
import {getLlama, LlamaChatSession, JinjaTemplateChatWrapper} from "node-llama-cpp";

const __dirname = path.dirname(fileURLToPath(import.meta.url));

const llama = await getLlama();
const model = await llama.loadModel({
    modelPath: path.join(__dirname, "models", "dolphin-2.1-mistral-7b.Q4_K_M.gguf")
});

console.log(model.fileInfo.metadata.tokenizer.chat_template);

You can also run this command to print all the GGUF file metadata, which includes the chat template:
npx --no node-llama-cpp inspect gguf <path to model>

After you find the roles that the model expects, you can customize the JinjaTemplateChatWrapper used by the chat session like this:
import {fileURLToPath} from "url";
import path from "path";
import {getLlama, LlamaChatSession, JinjaTemplateChatWrapper} from "node-llama-cpp";

const __dirname = path.dirname(fileURLToPath(import.meta.url));

const llama = await getLlama();
const model = await llama.loadModel({
    modelPath: path.join(__dirname, "models", "dolphin-2.1-mistral-7b.Q4_K_M.gguf")
});
const context = await model.createContext();
const session = new LlamaChatSession({
    contextSequence: context.getSequence(),
    chatWrapper: new JinjaTemplateChatWrapper({
        template: model.fileInfo.metadata.tokenizer.chat_template!,
        modelRoleName: "assistant", // change this to what you find the chat template
        userRoleName: "user" // change this to what you find the chat template
    })
});

If you can provide me with a link to the model file you used it'll help me investigate the issue further.

Still receiving the error after your suggested changes, details provided below:
updated my LlamaChatSession according to guide as follows:
const chatWrapper = new JinjaTemplateChatWrapper({
  template: model.fileInfo.metadata.tokenizer.chat_template!,
  modelRoleName: 'assistant',
  userRoleName: 'user',
})

session = new LlamaChatSession({
  contextSequence: context.getSequence(),
  chatWrapper
});

this is what console.log(model.fileInfo.metadata.tokenizer.chat_template!) logged into the console;
{{ bos_token }}{% for message in messages %}{% if (message['role'] == 'user') != (loop.index0 % 2 == 0) %}{{ raise_exception('Conversation roles must alternate user/assistant/user/assistant/...') }}{% endif %}{% if message['role'] == 'user' %}{{ '[INST] ' + message['content'] + ' [/INST]' }}{% elif message['role'] == 'assistant' %}{{ message['content'] + eos_token}}{% else %}{{ raise_exception('Only user and assistant roles are supported!') }}{% endif %}{% endfor %}
after running command: npx --no node-llama-cpp inspect gguf .\models\Mistral-7B-Instruct-v0.3.Q5_K_M.gguf >> Mistral-7B-Instruct-v0.3_Q5_K_M.txt
Mistral-7B-Instruct-v0.3_Q5_K_M.txt
File: ~\node-llama-cpp-project\models\Mistral-7B-Instruct-v0.3.Q5_K_M.gguf
GGUF version: 3
Tensor count: 291
Metadata size: 723.27KB
Tensor info size: 16.88KB
File type: MOSTLY_Q5_K_M (17)
Metadata: {
    general: {
        architecture: "llama",
        name: "models--mistralai--Mistral-7B-Instruct-v0.3",
        file_type: 17,
        quantization_version: 2
    },
    llama: {
        block_count: 32,
        context_length: 32_768,
        embedding_length: 4_096,
        feed_forward_length: 14_336,
        attention: {
            head_count: 32,
            head_count_kv: 8,
            layer_norm_rms_epsilon: 0
        },
        rope: {
            freq_base: 1_000_000,
            dimension_count: 128
        },
        vocab_size: 32_768
    },
    tokenizer: {
        ggml: {
            model: "llama",
            pre: "default",
            tokens: ["<unk>", "<s>", "</s>", "[INST]", "[/INST]", "[TOOL_CALLS]", "[AVAILABLE_TOOLS]", "[/AVAILABLE_TOOLS]", "[TOOL_RESULTS]", "[/TOOL_RESULTS]", ...32758 more items],
            scores: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, ...32758 more items],
            token_type: [2, 3, 3, 3, 3, 3, 3, 3, 3, 3, ...32758 more items],
            bos_token_id: 1,
            eos_token_id: 2,
            unknown_token_id: 0,
            add_bos_token: true,
            add_eos_token: false
        },
        chat_template: "{{ bos_token }}{% for message in messages %}{% if (message['role'] == 'user') != (loop.index0 % 2 == 0) %}{{ raise_exception('Conversation roles must alternate user/assistant/user/assistant/...') }}{% endif %}{% if message['role'] == 'user' %}{{ '[INST] ' + message['content'] + ' [/INST]' }}{% elif message['role'] == 'assistant' %}{{ message['content'] + eos_token}}{% else %}{{ raise_exception('Only user and assistant roles are supported!') }}{% endif %}{% endfor %}"
    },
    quantize: {
        imatrix: {
            file: "./imatrix.dat",
            dataset: "group_40.txt",
            entries_count: 224,
            chunks_count: 74
        }
    }
}
Tensor info: [{name: "token_embd.weight", dimensions: [4_096, 32_768], ggmlType: 13, offset: 0}, {name: "blk.0.attn_norm.weight", dimensions: [4_096], ggmlType: 0, offset: 92_274_688}, {name: "blk.0.ffn_down.weight", dimensions: [14_336, 4_096], ggmlType: 14, offset: 92_291_072}, {name: "blk.0.ffn_gate.weight", dimensions: [4_096, 14_336], ggmlType: 13, offset: 140_460_032}, ...287 more items]

NOTE: You asked for the model link, well I searched it again and the model link is no more valid as the owner seems to have removed the model file. The model name was "Mistral-7B-Instruct-v0.3"(file Mistral-7B-Instruct-v0.3.Q5_K_M.gguf) by "TheBloke" at HuggingFace. I am right now downloading Mistral-7B-Instruct-v0.2.Q5_K_M.gguf to try, I will comment again if the error persist on this model and will share with you the link as well.

Getting same error on the v0.2 of the same model, just click the link to start downloading the Mistral-7B-Instruct-v0.2.Q5_K_M.gguf

@giladgd can you please tell me what should I do to resolve the issue? I have been assigned some work that is due in 3 4 days, I have written a lot of code for other tasks in nodejs including the above one otherwise I would have switched to python to run this model. Please help me resolving it if you can, I can understand you might be really busy as a sole contributor of this whole project (node-llama-cpp).

I've just run your code with the model you linked and it run correctly.
I've used a Windows 10 machine with Nvidia A6000 together with your code.
I've put the output here

Try to scaffold a new project from a template and run it without any modification to see whether it works for you, so we can figure out whether the issue is related to your code, machine, OS or configuration:
npm create --yes node-llama-cpp@beta

If you are asking me to run the index.ts with the default code that it comes with when we create a new project with command;
npm create --yes node-llama-cpp@beta
well then it works fine, but there are just 5 prompts in the index.ts.
What's surprising for me is that it worked on your system totally fine! lol, great anyways. It could be my GPU (less powerful or incompatible) or OS problem.

By the way, can you guess or give me some tip to debug the problem? @giladgd

The issue that you encountered is related to the Jinja template of the model.
node-llama-cpp runs a sanity check with a given Jinja template before using it to ensure that all the messages of the chat history actually appear in the result text, so I think the error you get may be related to failing that sanity check.
It may be possible that there's some feature that is used by the Jinja template that's not available on nodejs 20.
Try updating to nodejs 22 and let me know whether it fixed the issue for you.
As a last resort, you can try to use GeneralChatWrapper instead of the Jinja template of the model by passing chatWrapper: new GeneralChatWrapper() when creating a chat session. It may not perform as good as using the model's template, but it may still be good enough.
Also, try running this command to inspect your GPU:
npx --no node-llama-cpp inspect GPU

Your machine may not have enough RAM of VRAM to run everything correctly, and this is the easiest way to see that.
Unless you specifically know you need it, it's best not to specify a specific gpuLayers (or gpuLayers: 'max' in your case) as node-llama-cpp measures your system and attempts to optimize all parameters for best performance, and that may mean not offloading all the layers to the GPU so there will be enough VRAM left for large enough context for example.

Though I pasted the code with gpuLayers set to max but I did test it with the default settings too and I have been testing it without setting gpuLayers since then.
Also I ran the gpu command before starting the program and 3 4 times in between the program before it exited in error and all of the time it printed the exact same amount (so perhaps it ain't giving me real time stats of the gpu either);
[Updated node to 22, as per suggestion]
`OS: Windows 10.0.22631 (x64)
Node: 22.4.0 (x64)
TypeScript: 5.5.3
node-llama-cpp: 3.0.0-beta.36
CUDA: available
Vulkan: available
CUDA device: NVIDIA GeForce RTX 4060 Laptop GPU
CUDA used VRAM: 13.29% (1.06GB/8GB)
CUDA free VRAM: 86.7% (6.93GB/8GB)
Vulkan devices: NVIDIA GeForce RTX 4060 Laptop GPU, AMD Radeon(TM) Graphics
Vulkan used VRAM: 1.09% (90MB/8.02GB)
Vulkan free VRAM: 98.9% (7.93GB/8.02GB)
CPU model: AMD Ryzen 7 7735HS with Radeon Graphics
Used RAM: 61.1% (9.28GB/15.19GB)
Free RAM: 38.89% (5.91GB/15.19GB)`
However the last resort chatWrapper: new GeneralChatWrapper() saved my life, thank you so much. love you bro :)
Q: Could it be because of low system RAM?

Glad you got it working :)
A chat session holds all the previous messages, and next prompts will be answered in the context of the chat history in that chat session.
If you'd like to prompt with a clean history, you can reset the chat session like this:
const session = new LlamaChatSession({
    contextSequence: context.getSequence()
});
const initialChatHistory = session.getChatHistory();

const a1 = session.prompt(q1);
session.setChatHistory(initialChatHistory); // reset the chat session

const a2 = session.prompt(q2); // the response here will not be aware of the previous prompt

I couldn't reproduce the issue you encountered, so since you have a solution by now I'm closing this issue.
If you find the cause or have a solution for how you fixed it (so I can investigate the cause), then let me know, and I'll reopen the issue.

I tried setting of history to initialChatHistory (which is basically empty) after every prompt and after that it worked with default chat template too. I mean I commented out the code chatWrapper: new GeneralChatWrapper() with resetting history after every prompt and I am not receiving the error "Error: Conversation roles must alternate user/assistant/user/assistant/...".
Though my use case here forces me to reset chat history but it ain't a valid solution to the error. Just updating you cause I though it might help you debugging to the root cause. @giladgd


layer results?

will the output describe the layer in which the software was first introduced?

Indeed it does!
Short answer: take a look at the .location field on each package in the syft-json output.
A little more detail; take an image based on the following dockerfile:
# tag: localhost/example:latest
FROM ubuntu:latest
RUN apt update -y && apt install -y python3-pip
RUN pip install click

And output from:
$ docker sbom localhost/example --format syft-json > sbom.json

The SBOM has some information about the image that was cataloged:
$ cat sbom.json | jq '.source.target.layers'
[
  {
    "mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
    "digest": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7",
    "size": 72756862
  },
  {
    "mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
    "digest": "sha256:c64d7db663506b5aeaa667ef426c5a2416a653796ce3a035c9facc27be445b3c",
    "size": 356087239
  },
  {
    "mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
    "digest": "sha256:2ab470a20aeb5ed65801c513633e7df8d88ec5c44a429b70ccf2cf9c117ebf96",
    "size": 746744
  }
]

These layers are in the same order they appear in the Dockerfile:

layer 1 / from...: sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7
layer 2 /run apt...: sha256:c64d7db663506b5aeaa667ef426c5a2416a653796ce3a035c9facc27be445b3c
layer 3 run pip...: sha256:2ab470a20aeb5ed65801c513633e7df8d88ec5c44a429b70ccf2cf9c117ebf96

The default analysis is to look at the image from the squashed representation of the image, which is the same if you were to create a container from the image and look at the mounted filesystem. Taking a closer look at the packages found in the image, all of the locations that are evident of the packages existence are indicated on the .locations field of the package.
So, back to our example... let's find the click package in the SBOM:
$ cat sbom.json | jq '.artifacts[] | select(.name == "click")'
{
  "id": "22eab18eb19f9d64",
  "name": "click",
  "version": "8.1.2",
  "type": "python",
  "foundBy": "python-package-cataloger",
  "locations": [
    {
      "path": "/usr/local/lib/python3.8/dist-packages/click-8.1.2.dist-info/METADATA",
      "layerID": "sha256:2ab470a20aeb5ed65801c513633e7df8d88ec5c44a429b70ccf2cf9c117ebf96"
    },
    {
      "path": "/usr/local/lib/python3.8/dist-packages/click-8.1.2.dist-info/RECORD",
      "layerID": "sha256:2ab470a20aeb5ed65801c513633e7df8d88ec5c44a429b70ccf2cf9c117ebf96"
    },
    {
      "path": "/usr/local/lib/python3.8/dist-packages/click-8.1.2.dist-info/top_level.txt",
      "layerID": "sha256:2ab470a20aeb5ed65801c513633e7df8d88ec5c44a429b70ccf2cf9c117ebf96"
    }
  ],
  "licenses": [
    "BSD-3-Clause"
  ],
... (truncated)...

Note that the layer information for all location paths are stored, and the earliest layer was in sha256:2ab470a20aeb5ed65801c513633e7df8d88ec5c44a429b70ccf2cf9c117ebf96 (layer 3).
Let's look at another package, bash, which should be in layer 1 (the base image):
$ cat sbom.json | jq '.artifacts[] | select(.name == "bash") | .locations'
 [
    {
      "path": "/usr/share/doc/bash/copyright",
      "layerID": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7"
    },
    {
      "path": "/var/lib/dpkg/info/bash.conffiles",
      "layerID": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7"
    },
    {
      "path": "/var/lib/dpkg/info/bash.md5sums",
      "layerID": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7"
    },
    {
      "path": "/var/lib/dpkg/status",
      "layerID": "sha256:c64d7db663506b5aeaa667ef426c5a2416a653796ce3a035c9facc27be445b3c"
    }
],

It looks like most filed indicate layer 1 but /var/lib/dpkg/status was found in layer 2. This is because we're looking at the squashed representation of the image, and in layer 2 we updated the DPKG database via the apt command.
What about the first indication for /var/lib/dpkg/status?
We can tell docker sbom to consider all layers and not just the squashed representation by generating the SBOM in the same way, but with an additional flag --layers all:
$ docker sbom --layers all localhost/example --format syft-json | jq '.artifacts[] | select(.name == "bash") | .locations'
Syft v0.43.0
 ✔ Loaded image
 ✔ Parsed image
 ✔ Cataloged packages      [301 packages]

[
  {
    "path": "/usr/share/doc/bash/copyright",
    "layerID": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7"
  },
  {
    "path": "/var/lib/dpkg/info/bash.conffiles",
    "layerID": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7"
  },
  {
    "path": "/var/lib/dpkg/info/bash.md5sums",
    "layerID": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7"
  },
  {
    "path": "/var/lib/dpkg/status",
    "layerID": "sha256:867d0767a47c392f80acb51572851923d6d3e55289828b0cd84a96ba342660c7"
  },
  {
    "path": "/var/lib/dpkg/status",
    "layerID": "sha256:c64d7db663506b5aeaa667ef426c5a2416a653796ce3a035c9facc27be445b3c"
  }
]

**Now we get the "full picture" where first evidence of the /var/lib/dpkg/status file having the bash package was found in layer 1 conclusively. **
Why is it also found in layer 2? Since image builders and containers use copy-on-write semantics, and DPKG puts all package entries into a single file, when python-pip was installed the full DB file was copied (which contained the bash entry). Syft, the underlying tool that drives docker sbom is smart enough to detect this and deduplicate the packages, but still leave the extra location evidence on the de-duplicated package.
This was a little verbose, but @rmccarth let me know if you have anymore questions about this 👍


Inspector does not set 'float' or 'int' values until you click outside of the Field

Tested versions
Reproducible in: 4.3.beta1.official [a4f2ea9]
System information
macOS Sonoma 14.1.1 - OpenGL API 4.1 Metal - 88 - Compatibility - Using Device: Apple - Apple M1 Pro
Issue description
Given a custom Resource like this:
class_name MyCustomResource
extends Resource

@export var my_float: float = 100.1
@export var my_int: int = 100
@export var my_string: String = "bye"

And a scene with a script like this:
class_name MyNode
extends Node

@export var my_custom_resource: MyCustomResource

func _ready() -> void:
	print("my float = ", my_custom_resource.my_float)
	print("my int = ", my_custom_resource.my_int)
	print("my string = ", my_custom_resource.my_string)

When running the scene, the output console will show the values in the resource.
Changing the string value and then immediately running the scene works as expected, the new value is shown.
If you change a float and int value in the Inspector and run the scene again immediately, you see the wrong values:

The float and int fields only commit the values to the .tres file when you click outside of the Field (into other fields, the code editor, or the scene tree)
Steps to reproduce

create an empty project
create a script extending Resource, give it class_name MyCustomResource
make it @export a float, an int and a string
create a my_custom_resource.tres resource with the script MyCustomResource
double click my_custom_resource.tres to edit its exported values on the Inspector
change the string value, observe how the the output console immediately says Set my_string as you type in characters
change the float or int value, observe how the the output console only says Set my_float or Set my_int if you click outside of the Field

Minimal reproduction project (MRP)
N/A

I believe this is intentional, so that entering certain values will not make the editor slow down to a crawl (e.g. if real-time processing is required by the node).

Oh I see, that makes sense.
I guess what surprised me the most was that hitting play wouldn't commit what I had as a "draft" either.

Pressing play or saving the scene should save the value, IMO.

And Ctrl + S is also ignored until I click outside the Field.

That's an existing bug report:

https://github.com/godotengine/godot/issues/74557

closing in favor of #74557
which is more comprehensive


fix a typo in extensible-admission-controllers

Fix a copy-and-paste typo in ExternalAdmissionHookConfiguration section.

This change is

dup of #4405

I'm closing this since it is a dup.


:art: Improving UI - added Header Title for the application.

delete the repo and fork again to do your changes to resolve conflicts also add screenshot of the ui change


NotImplementedError: Module [ModuleList] is missing the required "forward" function

I am facing the following error while trying to run the provided notebooks. What could be the possible reasons for this?
NotImplementedError: Module [ModuleList] is missing the required "forward" function
Detailed traceback:
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
Cell In[7], line 5
      3 prompts = [prompt]
      4 controller = AttentionStore()
----> 5 image = run_and_display(prompts=prompts,
      6                         controller=controller,
      7                         indices_to_alter=token_indices,
      8                         generator=g,
      9                         run_standard_sd=True,
     10                         display_output=True)
     11 vis_utils.show_cross_attention(attention_store=controller,
     12                                prompt=prompt,
     13                                tokenizer=tokenizer,
   (...)
     16                                indices_to_alter=token_indices,
     17                                orig_image=image)

Cell In[4], line 19, in run_and_display(prompts, controller, indices_to_alter, generator, run_standard_sd, scale_factor, thresholds, max_iter_to_alter, display_output)
      5 def run_and_display(prompts: List[str],
      6                     controller: AttentionStore,
      7                     indices_to_alter: List[int],
   (...)
     12                     max_iter_to_alter: int = 25,
     13                     display_output: bool = False):
     14     config = RunConfig(prompt=prompts[0],
     15                        run_standard_sd=run_standard_sd,
     16                        scale_factor=scale_factor,
     17                        thresholds=thresholds,
     18                        max_iter_to_alter=max_iter_to_alter)
---> 19     image = run_on_prompt(model=stable,
     20                           prompt=prompts,
     21                           controller=controller,
     22                           token_indices=indices_to_alter,
     23                           seed=generator,
     24                           config=config)
     25     if display_output:
     26         display(image)

File ~/ptp_sd_exps/Attend-and-Excite/notebooks/../run.py:45, in run_on_prompt(prompt, model, controller, token_indices, seed, config)
     42     ptp_utils.register_attention_control(model, controller)
     44 print("Inside run_on_prompt function")
---> 45 outputs = model(prompt=prompt,
     46                 attention_store=controller,
     47                 indices_to_alter=token_indices,
     48                 attention_res=config.attention_res,
     49                 guidance_scale=config.guidance_scale,
     50                 generator=seed,
     51                 num_inference_steps=config.n_inference_steps,
     52                 max_iter_to_alter=config.max_iter_to_alter,
     53                 run_standard_sd=config.run_standard_sd,
     54                 thresholds=config.thresholds,
     55                 scale_factor=config.scale_factor,
     56                 scale_range=config.scale_range,
     57                 smooth_attentions=config.smooth_attentions,
     58                 sigma=config.sigma,
     59                 kernel_size=config.kernel_size)
     60 image = outputs.images[0]
     61 return image

File /opt/conda/lib/python3.8/site-packages/torch/autograd/grad_mode.py:27, in _DecoratorContextManager.__call__.<locals>.decorate_context(*args, **kwargs)
     24 @functools.wraps(func)
     25 def decorate_context(*args, **kwargs):
     26     with self.clone():
---> 27         return func(*args, **kwargs)

File ~/ptp_sd_exps/Attend-and-Excite/notebooks/../pipeline_attend_and_excite.py:207, in AttendAndExcitePipeline.__call__(self, prompt, attention_store, indices_to_alter, attention_res, height, width, num_inference_steps, guidance_scale, eta, generator, latents, output_type, return_dict, max_iter_to_alter, run_standard_sd, thresholds, scale_factor, scale_range, smooth_attentions, sigma, kernel_size, **kwargs)
    205 # Forward pass of denoising with text conditioning
    206 print("calling unet inside __call__")
--> 207 noise_pred_text = self.unet(latents, t, encoder_hidden_states=text_embeddings[1].unsqueeze(0)).sample
    208 print("exiting unet")
    209 self.unet.zero_grad()

File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File /opt/conda/lib/python3.8/site-packages/diffusers/models/unet_2d_condition.py:381, in UNet2DConditionModel.forward(self, sample, timestep, encoder_hidden_states, class_labels, return_dict)
    379 for downsample_block in self.down_blocks:
    380     if hasattr(downsample_block, "has_cross_attention") and downsample_block.has_cross_attention:
--> 381         sample, res_samples = downsample_block(
    382             hidden_states=sample,
    383             temb=emb,
    384             encoder_hidden_states=encoder_hidden_states,
    385         )
    386     else:
    387         sample, res_samples = downsample_block(hidden_states=sample, temb=emb)

File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File /opt/conda/lib/python3.8/site-packages/diffusers/models/unet_2d_blocks.py:612, in CrossAttnDownBlock2D.forward(self, hidden_states, temb, encoder_hidden_states)
    610     else:
    611         hidden_states = resnet(hidden_states, temb)
--> 612         hidden_states = attn(hidden_states, encoder_hidden_states=encoder_hidden_states).sample
    614     output_states += (hidden_states,)
    616 if self.downsamplers is not None:

File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File /opt/conda/lib/python3.8/site-packages/diffusers/models/attention.py:217, in Transformer2DModel.forward(self, hidden_states, encoder_hidden_states, timestep, return_dict)
    215 # 2. Blocks
    216 for block in self.transformer_blocks:
--> 217     hidden_states = block(hidden_states, context=encoder_hidden_states, timestep=timestep)
    219 # 3. Output
    220 if self.is_input_continuous:

File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File /opt/conda/lib/python3.8/site-packages/diffusers/models/attention.py:495, in BasicTransformerBlock.forward(self, hidden_states, context, timestep)
    493     hidden_states = self.attn1(norm_hidden_states, context) + hidden_states
    494 else:
--> 495     hidden_states = self.attn1(norm_hidden_states) + hidden_states
    497 if self.attn2 is not None:
    498     # 2. Cross-Attention
    499     norm_hidden_states = (
    500         self.norm2(hidden_states, timestep) if self.use_ada_layer_norm else self.norm2(hidden_states)
    501     )

File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File ~/ptp_sd_exps/Attend-and-Excite/notebooks/../utils/ptp_utils.py:85, in register_attention_control.<locals>.ca_forward.<locals>.forward(x, context, mask)
     83 out = torch.einsum("b i j, b j d -> b i d", attn, v)
     84 out = self.reshape_batch_dim_to_heads(out)
---> 85 return self.to_out(out)

File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File /opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py:246, in _forward_unimplemented(self, *input)
    235 def _forward_unimplemented(self, *input: Any) -> None:
    236     r"""Defines the computation performed at every call.
    237
    238     Should be overridden by all subclasses.
   (...)
    244         registered hooks while the latter silently ignores them.
    245     """
--> 246     raise NotImplementedError(f"Module [{type(self).__name__}] is missing the required \"forward\" function")

NotImplementedError: Module [ModuleList] is missing the required "forward" function

Hi @aisagarw, thanks for your interest! Are you sure you are using the version of diffusers specified in our requirements file?

It was indeed a version mismatch issue.. Thanks for your quick response!


Changes to match new derpibooru API

Derpibooru is going to deprecate the old API soon:
https://derpibooru.org/forums/meta/topics/old-api-deprecation-notice
This updates the library to match the new API.

I've tried cloning your fork and running it, but I don't seem to be getting image results:
from derpibooru import Search
foo = Search()
for post in foo:
    print(post.tags)

I still need to spend time learning the new API to understand what exactly needs to be fixed before these changes can be merged.

My mistake -- I didn't test very hard on this, and it looks like I completely left out the case where 'filter_id' is left empty. The old API didn't care about that, but the new API returns "Bad Request" when that happens. This commit should fix the issue. No other issues should happen with the request parameters.

This appears to work now, and should be a good start for refactoring the library under the hood.


Frontend: Receipt render bug

Describe the bug
In admin receipt viewing page, if there are multiple sessions with one or more sessions without any data, the page would not render other sessions that contains data.
To Reproduce
Steps to reproduce the behavior:

Create new sessions
Go to '/receipt'
Inspect element
See error

Expected behavior
Sessions with data should render without any error.
Additional context
Add any other context about the problem here.

Bug is patched by moving it to the homepage #29


Marshmallow 3 DeprecationWarning: Field.fail is deprecated. Use raise self.make_error("by_name", ...) instead

Simply changing super(EnumField, self).fail(key, **kwargs) to raise super(EnumField, self).make_error(key, **kwargs) seems to work for Marshmallow 3 on Python 3 but fails for Python 2, at least when running tox locally.
https://marshmallow.readthedocs.io/en/stable/upgrading.html#field-fail-is-deprecated-in-favor-of-field-make-error

Hi!
@justanr are you planning to fix this warning? Or would you consider a PR maybe?
cheers!


fix: RootRelationalClassMapping compilation to include mapping filter

Q
A

Fixed issues?
Fixes #175

Tests added + passed?
Yes

Changeset added?
Yes

Documentation PR link

Any dependency changes?
No

Also Im not too sure what happened to your CI, can you try to do a force push to retrigger it?


Custom Sections or Plugins Support

One of the things (afaik) lacking with fleetctl is the ability to do more than launch units in a consistent way. For example, you need to launch a unit for your web app, reconfigure your process monitor to do log checks and prepare everything to ship logs somewhere. All those steps feel "disconnected" and things like stopping a unit without reconfiguring your health checks will cause failure alerts.
What I believe would be nice is the ability to extend fleetctl/systemd units with more meta data and to be able to take actions on those with an agnostic approach. For example:
[Unit]
Description=MyApp
[Service]
ExecStart=/usr/bin/docker run myapp
[X-Logs]
LogFile=/var/log/myapp.log
LogPattern=/var/log/myapp/*.log
[X-Health-Check]
Run=docker exec myapp /healthcheck.sh
Run=docker exec myapp /healthcheck2.sh
Frequency=1m

The X-Logs and X-Health-Check sections although not understood by systemd or fleetctl could trigger plugins/processes on hosts that registered and processed that meta-data. For example, on running myapp the host that runs the unit could also launch<EMAIL_ADDRESS>that would parse or receive and process that meta data.
All this seems almost possible but there is no way of getting the unit's meta-data out the service files...

@alexmipego, Seems like you want a feature of dependency.
Currently, fleet support "MachineOf" fleet-specific Options. It means it depend on another unit. Is this the feature what you want ?
@mischief , what do you think ?

Not exactly, the purpose would be to be to define properties that configure other abstract units. For example, to use something like monit or nagios you would need to configure them with something like a health check script or ping url, right now this means reconfiguring them manually. With this, a generic solution could be in place that would handle any implementation-specific details.
Another way of thinking about this would be that you provide fleetctl as a service for your company but you dont want people messing around with "utility" services like monitors or log shippers directly.
As I said, with more or less effort this should be possible with systemd dependencies, machineOf, etc... however, you still need to create a specific unit that knows the details of your new unit before the "abstract" units can be useful (e.g. the log paths).


nixos/restic: make it possible to use the existing backup cache for prune/check

Description of changes
Configures the --cache-dir parameter for the prune and check commands run after backing up. For check, also adds a checkOpts flag to enable using the cache, since that is disabled by default.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Couldn't you replace the binary of restic so it prints the command line?

@otavio added a test!

Who would have the permissions to merge this @otavio?

@ofborg test restic


Xcode 9.1 deprecation warnings

I just updated Xcode to the newest 9.1 version, and have 4 deprecation warnings (I use ObjectMapper v3.0.0)  :
(3) 'characters' is deprecated: Please use String or Substring directly.
(1) 'CharacterView' is deprecated: Please use String or Substring directly.

As you probably know already, Swift 4 works without characters, directly with the String. For example, hex.characters.count can be replaced by hex.count.
Here are the concerned line of code :

characters appears in HexColorTransform.swift line 38 and 92.
characters appears in ToJson.swift line 37
CharacterView appears in ToJson.swift line 44

+1 :)

+1:)

+1 :)

I've just released a version with the 9.1 fixes in it


ci: build and deploy with GitHub Actions

related to INFRA-1268
specialties:

use a service to spawn a keycloak container as required by the tests
in order to not duplicate the service and build configuration, we make the publish workflow wait for build to complete successfully.

Example runs:

build: https://github.com/camunda/camunda-bpm-identity-keycloak/runs/738798157?check_suite_focus=true
publish: https://github.com/camunda/camunda-bpm-identity-keycloak/runs/738798184?check_suite_focus=true

@VonDerBeck (tagging you as main contributor): As you can see, we're moving from building the community projects on GitHub actions instead of "our" Jenkins.
We made this move to improve transparency as well as provide better feedback for contributors.
I'm going to leave this note in all affected repositories once we're done making the changes.
If you experience issues related to our changes to GitHub Actions, please let us know!

@lwille:

As you can see, we're moving from building the community projects on GitHub actions instead of "our" Jenkins.
We made this move to improve transparency as well as provide better feedback for contributors.

Absolut great idea. Thank you!

@Iwille:
The current move to using GitHub actions - what does this mean when releasing a Community project?
Previously that has been intransparent for me and was carried out by Camunda staff (besides that it looked like the maven release plugin seems to be involved). How does this work now? Simply use standard maven release? Will the "Deploy artifacts" action automatically publish a release e.g. to Maven Central? Is there anything to be aware of?
Is it even intended that I, as a contributor, now do the release, or is it still intended that I post a JIRA ticket for it?
I'm not planning a release of this extension right now - I'm just curious and want to be prepared.
If you are not the one who can answer this question just let me know as well.
Thanks a lot!
Gunnar


Fix map conflict and update documentation

These changes fix conflict between :TernDef and :TernDoc mappings and add documentation on default mappings.

I do apologize for letting such an excellent pull request sit unhandled for so long. Merged as 0c74dbb9d8985...633b4c8d3