Add support for command HVALS

Add support for the HVALS command in DiceDB similar to the HVALS command in Redis. Please refer to the following commit in Redis to understand the implementation specifics - source.
Write unit and integration tests for the command referring to the tests written in the Redis codebase 7.2.5. For integration tests, you can refer to the tests folder. Note: they have used TCL for the test suite, and we need to port that to our way of writing integration tests using the relevant helper methods. Please refer to our tests directory.
For the command, benchmark the code and measure the time taken and memory allocs using benchmem and try to keep them to the bare minimum.

Hey, I'll work on this!

@rootxrishabh Thanks for taking this up. assigned.


ctlv3: make flags, commands formats consistent

Capitalize first letter
Remove period at the end

(followed the pattern in linux coreutil man page)
diff --git a/a.txt b/a.txt
index 162d3db..8c75d13 100644
--- a/a.txt
+++ b/a.txt
@@ -1,44 +1,43 @@
-COMMANDS:
-	get			Get gets the key or a range of keys.
-	put			Put puts the given key into the store.
-	del			Removes the specified key or range of keys [key, range_end).
-	txn			Txn processes all the requests in one transaction.
-	compaction		Compaction compacts the event history in etcd.
-	alarm disarm		disarm all alarms
-	alarm list		list all alarms
-	defrag			defrag defragments the storage of the etcd members with given endpoints.
-	endpoint health		health checks the healthiness of endpoints specified in `--endpoints` flag
-	endpoint status		status prints out the status of endpoints specified in `--endpoints` flag
-	watch			Watch watches events stream on keys or prefixes.
-	version			Print the version of etcdctl.
-	lease grant		grant is used to create leases.
-	lease revoke		revoke is used to revoke leases.
-	lease keep-alive	keep-alive is used to keep leases alive.
-	member add		add is used to add a member into the cluster
-	member remove		remove is used to remove a member from the cluster
-	member update		update is used to update a member in the cluster
-	member list		list is used to list all members in the cluster
-	snapshot save		save stores an etcd node backend snapshot to a given file.
-	snapshot restore	restore an etcd member snapshot to an etcd directory
-	snapshot status		status gets backend snapshot status of a given file.
-	make-mirror		make-mirror makes a mirror at the destination etcd cluster
-	migrate			migrates keys in a v2 store to a mvcc store
-	lock			lock acquires a named lock
-	elect			elect observes and participates in leader election
-	auth enable		enable authentication
-	auth disable		disable authentication
-	user add		add a new user
-	user delete		delete a user
-	user get		get detailed information of a user
-	user list		list up all users
-	user passwd		change password of user
-	user grant-role		grant a role to a user
-	user revoke-role	revoke a role from a user
-	role add		add a new role
-	role delete		delete a role
-	role get		get detailed information of a role
-	role list		list up all roles
-	role grant-permission	grant a key to a role
-	role revoke-permission	revoke a key from a role
+	get			Gets the key or a range of keys
+	put			Puts the given key into the store
+	del			Removes the specified key or range of keys [key, range_end)
+	txn			Txn processes all the requests in one transaction
+	compaction		Compacts the event history in etcd
+	alarm disarm		Disarms all alarms
+	alarm list		Lists all alarms
+	defrag			Defragments the storage of the etcd members with given endpoints
+	endpoint health		Checks the healthiness of endpoints specified in `--endpoints` flag
+	endpoint status		Prints out the status of endpoints specified in `--endpoints` flag
+	watch			Watches events stream on keys or prefixes
+	version			Prints the version of etcdctl
+	lease grant		Creates leases
+	lease revoke		Revokes leases
+	lease keep-alive	Keeps leases alive (renew)
+	member add		Adds a member into the cluster
+	member remove		Removes a member from the cluster
+	member update		Updates a member in the cluster
+	member list		Lists all members in the cluster
+	snapshot save		Stores an etcd node backend snapshot to a given file
+	snapshot restore	Restores an etcd member snapshot to an etcd directory
+	snapshot status		Gets backend snapshot status of a given file
+	make-mirror		Makes a mirror at the destination etcd cluster
+	migrate			Migrates keys in a v2 store to a mvcc store
+	lock			Acquires a named lock
+	elect			Observes and participates in leader election
+	auth enable		Enables authentication
+	auth disable		Disables authentication
+	user add		Adds a new user
+	user delete		Deletes a user
+	user get		Gets detailed information of a user
+	user list		Lists up all users
+	user passwd		Changes password of user
+	user grant-role		Grants a role to a user
+	user revoke-role	Revokes a role from a user
+	role add		Adds a new role
+	role delete		Deletes a role
+	role get		Gets detailed information of a role
+	role list		Lists up all roles
+	role grant-permission	Grants a key to a role
+	role revoke-permission	Revokes a key from a role
 	help			Help about any command

/cc @heyitsanthony @xiang90

lgtm

Thanks! Merging in greens.
@xiang90 Should we backport this to release-3.0?

sure. let's backport it.

ok will do after merge.


Text document proxy doesn't provide all text before and after the text input

When you ask for the content before and after the text init, the text document proxy only gives you a little text, typically up to the next sentence or line.
I thought this was a privacy limitation, but it seems like there are ways to access more content:

Can we use this gist by Gary to get more information asynchronously?
https://gist.github.com/gtokman/af1ea26a8e43b6e0125d943832b2b947
This is the code
var before = textDocumentProxy.documentContextBeforeInput

var completePriorString = ""
while let beforeInput = before, !beforeInput.isEmpty {
    completePriorString = beforeInput

    let length = beforeInput.lengthOfBytes(using: String.Encoding.utf8)

    textDocumentProxy.adjustTextPosition(byCharacterOffset: -length)
    Thread.sleep(forTimeInterval: 0.01)
    before = textDocumentProxy.documentContextBeforeInput
}

// Move the cursor back to the original position
textDocumentProxy
    .adjustTextPosition(byCharacterOffset: completePriorString.count)
Thread.sleep(forTimeInterval: 0.01)

@danielsaidi, I tried this technique in simulator. It doesn't work. Are you able to make this work? Thanks.

@ctawong Hi! I was planning to look at this after releasing 6.0, but haven't tried it yet.

@danielsaidi - tried this with a real device, and also used some other code found on SO.
I'm dispatching it on global queue and seems that with 0.01 sleep interval it doesn't work. Tried with other values, but only 0.035 seems to be the minimum working and I see the caret jumping a lot forward and backwards.
I've tried Grammarly and their caret seems it's not jumping at all, but can't find how they're doing it.
Did you have time to look into this also?

Hi @DanMotataeanu! Grammarly are doing some amazing wizardry with many things, that I am yet to understand how they pull off 😅
Haven't looked into this yet, but am interested in any experiences you may have to make this happen.

This is available in KeyboardKit Pro 6.4.1.


fix: Shuttle Bus time schedules

Asana task: Shuttle Bus Widget: Our travel time estimates for peak / non-peak are not showing up at the scheduled time
Slight timezone shifting issue. We need to find the time for the service day, so the day we look at is correct. The time we look at should be local time though. Just need to have two separate shifts for date and time.

[ ] Tests added?

Created a ticket! https://app.asana.com/0/1185117109217422/1202933006890213/f


Learn Django in Visual Studio Step 2-2.2 incorrect code generates error

Step 2-2: Run the app from the Django project
Step 2 has a code section that uses django.conf.urls to import url, which looks like it's been deprecated and removed (source on removal here) from the latest version of Django.
Old code:
from django.conf.urls import include, url
import HelloDjangoApp.views

# Django processes URL patterns in the order they appear in the array
urlpatterns = [
    url(r'^$', HelloDjangoApp.views.index, name='index'),
    url(r'^home$', HelloDjangoApp.views.index, name='home'),
]

I stumbled over it, but I found the correct implementation (source) using django.urls to import re_path (below):
from django.urls import include, re_path
import HelloDjangoApp.views

# Django processes URL patterns in the order they appear in the array
urlpatterns = [
    re_path(r'^$', HelloDjangoApp.views.index, name='index'),
    re_path(r'^home$', HelloDjangoApp.views.index, name='home')
]

First time submitting an issue - hope I did it right! :)
Edited to update to using re_path instead of url. This applies for the other sections later in the guide where the other pages are added (About, etc).

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 09d5f2d3-bebb-5564-87df-8ab0eedb410c
Version Independent ID: c1dac521-a8f2-1b81-af35-bd9da71fb6e8
Content: Learn Django tutorial in Visual Studio step 2, views and page template
Content Source: docs/python/learn-django-in-visual-studio-step-02-create-an-app.md
Product: visual-studio-windows
Technology: vs-python
GitHub Login: @cwebster-99
Microsoft Alias: cowebster

@MarissaB -- Marissa, thank you for your feedback and solution.  You're welcome to submit these edits yourself via a Pull Request (PR).  This will allow authors to directly add your text to the documentation. This also displays your GitHub image on the page of authors, and lists you as a contributor to the documentation.

Navigate to the page of interest
Select the "Edit" button at the top right of the page
Select the "Pencil" icon on the right
Modify the text within the text box in markdown format
Name your file change and copy the link to your Git Issue into the "extended description" box
Select "Propose file change"
Select "Create pull request" on the 'Comparing changes' page
Select "Create pull request" on the 'Open a pull request' page
Wait for an author to review and approve the change request - this can take a bit depending on the complexity

If you'd prefer to put in our queue, use the #reassign automation command to assign it to @cwebster-99:  #reassign: @cwebster-99

Okay awesome, I'll make a pull request in the next few days. :)

@MarissaB -- Marissa, thank you for your two PRs!  Your contribution is appreciated.


Roll The Dice Getting Started for C++

To complete #2623 we need an updated version of the C++ Getting Started with a "Roll The Dice" app. To simplify things one could use oatpp or drogon or any other C++-based framework.

Hi @svrnm, I would like to work on this! I just need to figure out a few things

@Akhaled19 that's great to hear, thank you!
To provide you with some help, here are a few pointers:

Take a look at the Rust & Swift getting started, they represent good examples of what we are aiming for as a starting point: provide code for a roll the dice webserver, instrument it, provide pointers to next steps.
My own C++ skills are rusty, but I have an oatpp app as a starting point for you, if you like to take that as an example (not a requirement, just a suggestion, use whatever framework suits you):

#include "oatpp/web/server/HttpConnectionHandler.hpp"
#include "oatpp/network/Server.hpp"
#include "oatpp/network/tcp/server/ConnectionProvider.hpp"

class Handler : public oatpp::web::server::HttpRequestHandler {
public:
  std::shared_ptr<OutgoingResponse> handle(const std::shared_ptr<IncomingRequest>& request) override {
    std::string response = std::to_string(1 + (std::rand() % 6));
    return ResponseFactory::createResponse(Status::CODE_200, response);
  }
};

void run() {
  auto router = oatpp::web::server::HttpRouter::createShared();
  router->route("GET", "/rolldice", std::make_shared<Handler>());
  auto connectionHandler = oatpp::web::server::HttpConnectionHandler::createShared(router);
  auto connectionProvider = oatpp::network::tcp::server::ConnectionProvider::createShared({"localhost", 8000, oatpp::network::Address::IP_4});
  oatpp::network::Server server(connectionProvider, connectionHandler);
  OATPP_LOGI("MyApp", "Server running on port %s", connectionProvider->getProperty("port").getData());
  server.run();
}

int main() {
  oatpp::base::Environment::init();
  run();
  oatpp::base::Environment::destroy();
  return 0;
}

@svrnm thank you for the helpful pointers!

@Akhaled19 sure, let us know if you need any more help and don't hesitate to raise your PR early to get feedback!

Hi @svrnm I managed to get the app up and running but I'm having a hard time linking OpenTelemetry with my oatpp Application. I'm new to cmake. Here is what my CMakeLists.txt file looks like

@Akhaled19, if you share the work you have done so far in textual form (snippets here in the comments or even a github repo!), making it easier for us to take a look into what you have tried to accomplish so far.
@open-telemetry/cpp-approvers

Note that C++11 support has been dropped: https://github.com/open-telemetry/opentelemetry-cpp/pull/2342

Got it. I will work on the pr

@svrnm Please close, was fixed by PR #3423


Adding a minimal inference example

Addressing #5 and some other comments:
inference-example.py is a minimal example for running inference with Unlimiformer. The default model in this example is our GovReport Unlimiformer model, but this can be swapped out for any other pretrained encoder-decoder (e.g. BART, PRIMERA) by simply changing the modelname and tokenizer used.

Addressing #5 and some other comments:
inference-example.py is a minimal example for running inference with Unlimiformer. The default model in this example is our GovReport Unlimiformer model, but this can be swapped out for any other pretrained encoder-decoder (e.g. BART, PRIMERA) by simply changing the modelname and tokenizer used.

The code works fine in default setting. I can confirm that I could generate for T5 models. However, I cannot get it to work with the use_datastore set to true. I always get the error shown below. Can you confirm this parameter has to be set to true to actually enable the knn swap out of attention?
Error
Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/home/oppenheimer/longcontexts/unlimiformer/src/unlimiformer.py", line 474, in pre_generate_hook     self.reset_memory(input_ids, kwargs['attention_mask'])   File "/home/oppenheimer/longcontexts/unlimiformer/src/unlimiformer.py", line 394, in reset_memory     self.datastore.train_index()   File "/home/oppenheimer/longcontexts/unlimiformer/src/index_building.py", line 28, in train_index     index.train_index()   File "/home/oppenheimer/longcontexts/unlimiformer/src/index_building.py", line 83, in train_index     self.move_to_gpu()   File "/home/oppenheimer/longcontexts/unlimiformer/src/index_building.py", line 71, in move_to_gpu     self.index = faiss.index_cpu_to_gpu(faiss.StandardGpuResources(), 0, self.index, co)   File "/home/oppenheimer/.local/lib/python3.10/site-packages/faiss/swigfaiss.py", line 10275, in index_cpu_to_gpu     return _swigfaiss.index_cpu_to_gpu(provider, device, index, options) RuntimeError: Error in void faiss::gpu::GpuIndexIVF::init_() at /project/faiss/faiss/gpu/GpuIndexIVF.cu:41: Error: 'nlist > 0' failed: nlist must be > 0

Hi @KurtFeynmanGodel -- I'm not seeing this on my end, could you let me know which parameters you're using?
If you have ```test_unlimiformer`` set to true-- which I've just changed to the default here-- it will be using Unlimiformer to augment the context regardless of the datastore parameters. The datastore parameters are useful if your context is very long-- otherwise you may get CUDA out-of-memory issues during inference.

Hi @KurtFeynmanGodel -- I'm not seeing this on my end, could you let me know which parameters you're using?
If you have ```test_unlimiformer`` set to true-- which I've just changed to the default here-- it will be using Unlimiformer to augment the context regardless of the datastore parameters. The datastore parameters are useful if your context is very long-- otherwise you may get CUDA out-of-memory issues during inference.

So I modified your inference code such that I did not need the test_unlimiformer. I am probably making some silly error somewhere, but here is my code in its entirety. If you can spot the issue, I would appreciate it.
`from unlimiformer import Unlimiformer
from random_training_unlimiformer import RandomTrainingUnlimiformer
from usage import UnlimiformerArguments, training_addin
from transformers import BartForConditionalGeneration, AutoTokenizer, T5ForConditionalGeneration
from datasets import load_dataset
import torch
device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
example using T5
modelname = "google/flan-t5-large"
tokenizer = AutoTokenizer.from_pretrained(modelname)
model = T5ForConditionalGeneration.from_pretrained(modelname)
example_input = "Q: Can Geoffrey Hinton have a conversation with George Washington? Give the rationale before answering."
example = tokenizer(example_input, truncation=False, return_tensors="pt")
example.to(device)
print(f"INPUT LENGTH (tokens): {example['input_ids'].shape[-1]}")
defaults = UnlimiformerArguments()
unlimiformer_kwargs = {
'layer_begin': defaults.layer_begin,
'layer_end': defaults.layer_end,
'unlimiformer_head_num': defaults.unlimiformer_head_num,
'exclude_attention': defaults.unlimiformer_exclude,
'chunk_overlap': defaults.unlimiformer_chunk_overlap,
'model_encoder_max_len': defaults.unlimiformer_chunk_size,
'verbose': defaults.unlimiformer_verbose, 'tokenizer': tokenizer,
'unlimiformer_training': defaults.unlimiformer_training,
'use_datastore': True,
'flat_index': defaults.flat_index,
'test_datastore': defaults.test_datastore,
'reconstruct_embeddings': defaults.reconstruct_embeddings,
'gpu_datastore': defaults.gpu_datastore,
'gpu_index':  defaults.gpu_index
}
model = Unlimiformer.convert_model(model, **unlimiformer_kwargs)
model.eval()
model.to(device)
the output of the model /with/ unlimiformer
unlimiformer_out = tokenizer.batch_decode(model.generate(**example, max_length=512), ignore_special_tokens=True)[0]
print("From unlimiformer with datastore")
print(unlimiformer_out)`

I'm able to use my script with google/flan-t5-large (changing only the import, the model name, and the tokenizer), so I think this may be an issue with your changes-- I do get the error you report when using your version. Possibly this is an issue with your tokenization of the new text you use?
I'll merge the PR since this seems to be slightly tangential, but @KurtFeynmanGodel  please feel free to open an issue if you're having problems using this script!

What import did you use?


[Question]: How do i connect into SSH and VNC?

Is your question not already answered in the FAQ?

[X] I made sure the question is not listed in the FAQ.

Is this a general question and not a technical issue?

[X] I am sure my question is not about a technical issue.

Question
How do I connect to the SSH?, and how can I connect via VNC too?
services:
  qemu:
    image: qemux/qemu-docker
    environment:
      DISPLAY: "vnc"
      RAM_SIZE: "512M"
      DISK_SIZE: "1G"
      BOOT: "https://dl-cdn.alpinelinux.org/alpine/v3.9/releases/x86_64/alpine-virt-3.9.6-x86_64.iso"
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 2204:22
      - 5900:5900
    stop_grace_period: 2m

if I remember that Vnc is on 5900 right?, I've read some of the issues created by other too https://github.com/qemus/qemu-docker/issues/573#issuecomment-2240634671
but when I access it it showed nothing on the web

also how do I connect into the ssh?, since I got this

Okay i managed to connect into ssh and qemu, using ubuntu iso, i think the problem was from the iso (idk so much about this)

But what's the password of the ssh?

You seem to be using a web browser to connect to port 5900. Instead, you need to use an actual VNC client, like Remmina. You could instead disable VNC and connect to port 8006 with this docker-compose file
services:
  qemu:
    image: qemux/qemu-docker
    environment:
      RAM_SIZE: "512M"
      DISK_SIZE: "1G"
      BOOT: "https://dl-cdn.alpinelinux.org/alpine/v3.9/releases/x86_64/alpine-virt-3.9.6-x86_64.iso"
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 2204:22
      - 8006:8006
    stop_grace_period: 2m

Note that the DISPLAY environment variable has been removed and we're mapping port 8006 instead of 5900. Then, in your web browser, instead of connecting to http://localhost:5900, connect to http://localhost:8006


Possible bug in to_table() method

Dear all,
I am still playing with astroplan for scheduling observing run.
However, I recently came across a problem, that I cannot easily reproduce, so I decided to share a pickled version of my schedule object:
schedule.txt
Here is the code that allows to generate the error
import pickle
f = open('./schedule.txt','rb')
o = pickle.load(f)
o.to_table()

The error message is the following:

UnitConversionError                       Traceback (most recent call last)
~/anaconda3/lib/python3.6/site-packages/astropy/units/quantity.py in float(self)
1200         try:
-> 1201             return float(self.to_value(dimensionless_unscaled))
1202         except (UnitsError, TypeError):
~/anaconda3/lib/python3.6/site-packages/astropy/units/quantity.py in to_value(self, unit, equivalencies)
877             if unit != self.unit:
--> 878                 value = self._to_value(unit, equivalencies)
879         return value if self.shape else value.item()
~/anaconda3/lib/python3.6/site-packages/astropy/units/quantity.py in _to_value(self, unit, equivalencies)
815         return self.unit.to(unit, self.view(np.ndarray),
--> 816                             equivalencies=equivalencies)
817
~/anaconda3/lib/python3.6/site-packages/astropy/units/core.py in to(self, other, value, equivalencies)
978         """
--> 979         return self._get_converter(other, equivalencies=equivalencies)(value)
980
~/anaconda3/lib/python3.6/site-packages/astropy/units/core.py in _get_converter(self, other, equivalencies)
912
--> 913             raise exc
914
~/anaconda3/lib/python3.6/site-packages/astropy/units/core.py in _get_converter(self, other, equivalencies)
898             return self._apply_equivalencies(
--> 899                 self, other, self._normalize_equivalencies(equivalencies))
900         except UnitsError as exc:
~/anaconda3/lib/python3.6/site-packages/astropy/units/core.py in _apply_equivalencies(self, unit, other, equivalencies)
882             "{0} and {1} are not convertible".format(
--> 883                 unit_str, other_str))
884
UnitConversionError: 'deg' (angle) and '' (dimensionless) are not convertible
During handling of the above exception, another exception occurred:
TypeError                                 Traceback (most recent call last)
 in ()
----> 1 o.to_table()
~/anaconda3/lib/python3.6/site-packages/astroplan/scheduling.py in to_table(self, show_transitions, show_unused)
302         return Table([target_names, start_times, end_times, durations, ra, dec, config],
303                      names=('target', 'start time (UTC)', 'end time (UTC)',
--> 304                             'duration (minutes)', 'ra', 'dec', 'configuration'))
305
306     def new_slots(self, slot_index, start_time, end_time):
~/anaconda3/lib/python3.6/site-packages/astropy/table/table.py in init(self, data, masked, names, dtype, meta, copy, rows, copy_indices, **kwargs)
409
410         # Finally do the real initialization
--> 411         init_func(data, names, dtype, n_cols, copy)
412
413         # Whatever happens above, the masked property should be set to a boolean
~/anaconda3/lib/python3.6/site-packages/astropy/table/table.py in _init_from_list(self, data, names, dtype, n_cols, copy)
675             elif isinstance(col, np.ndarray) or isiterable(col):
676                 col = self.ColumnClass(name=(name or def_name), data=col, dtype=dtype,
--> 677                                        copy=copy, copy_indices=self._init_indices)
678             else:
679                 raise ValueError('Elements in list initialization must be '
~/anaconda3/lib/python3.6/site-packages/astropy/table/column.py in new(cls, data, name, dtype, shape, length, description, unit, format, meta, copy, copy_indices)
805                                           shape=shape, length=length, description=description,
806                                           unit=unit, format=format, meta=meta,
--> 807                                           copy=copy, copy_indices=copy_indices)
808         return self
809
~/anaconda3/lib/python3.6/site-packages/astropy/table/column.py in new(cls, data, name, dtype, shape, length, description, unit, format, meta, copy, copy_indices)
228             if not six.PY2 and np.dtype(dtype).char == 'S':
229                 data = cls._encode_str(data)
--> 230             self_data = np.array(data, dtype=dtype, copy=copy)
231
232         self = self_data.view(cls)
~/anaconda3/lib/python3.6/site-packages/astropy/units/quantity.py in float(self)
1201             return float(self.to_value(dimensionless_unscaled))
1202         except (UnitsError, TypeError):
-> 1203             raise TypeError('only dimensionless scalar quantities can be '
1204                             'converted to Python scalars')
Do you have an idea of a workaround for this ?
Thank you in advance for your help !

@gnthibault - Sharing pickes are usually not the safest way around, they are best when just passing between pipeline elements in the same environment.
Have you tried to convert the schedule to a Table before picking, and save it to disk then?

I pickled the schedule only for sharing it, because I was not able to provide a simple script that exhibit the proble (MWE). Otherwise I never use Pickle inside my pipeline.
It looks like this bug is target dependant.

OK, then I rephrase the question.
 If yes, could you provide a small example to reproduce?

I just wanted to note, that I encountered this as well. I tracked the issue to the automatic dtype inference by the Table initiation when there are no TransitionBlock. I fixed it by specifying all dtypes as strings.

    return Table([target_names, start_times, end_times, durations, ra, dec, config],
                 names=('target', 'start time (UTC)', 'end time (UTC)',
                        'duration (minutes)', 'ra', 'dec', 'configuration'),
                 dtype=[str, str, str, str, str, str, str])

"""
I realise this is maybe a bit hacky solution, so let's explore alternative fixes.

Also, this suggestion does not seem to be fool-proof

@jselsing - could you provide a full example? That would help enormously to reproduce it and come up with a permanent fix for it.
Alternatively if you're interested contributing and have a fix please open a PR.
(Casting everything as a string is not a preferred way, I suppose we need something along the line of adding a conditional before the return)

Yea, I figured.
My astroplan usage is part of a larger piece of code, and not straightforward to produce a MWE. It only fails in small subset of calls - maybe one where there is no TransitionBlock. I will try and take look at it, Monday!

Closing as stale. Please reopen if necessary!


Functional swap test: Use yaml where possible

Codecov Report

Merging #297 (6bbcb7c) into main (ee61c40) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #297   +/-   ##
=====================================
  Coverage   15.3%   15.3%
=====================================
  Files         27      27
  Lines       6574    6574
=====================================
  Hits        1004    1004
  Misses      5570    5570

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee61c40...6bbcb7c. Read the comment docs.


There are some generated class which does not generate automatically. please add the instructions for generating those class.

Please mention the solution for the problem. i can say that on every class where this issue indicate has same format
***_model.g.dart

@kapilb145  :  can you please share the flutter doctor command report for your system ?

@kapilb145  :
A normal solution to what you are experiencing will be executing the build command of the uild_runner package.
flutter pub run build_runner build
or
dart run build_runner build
depending on your  flutter version.
Recommending using the flag --delete-conflicting-outputs
So your final command in the terminal for solving the issue will be
flutter pub run build_runner build --delete-conflicting-outputs
or
dart run build_runner build --delete-conflicting-outputs


Synchronize API with the schema changes

Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

This PR depends on https://github.com/nmstate/nmstate/pull/631


improve error handling read fn

Overview Of PR
Improve error handling. Currently the projects have a lot of unwrap() that would cause panics with no specific errors. this PR fixes that
More refer to Eliminate unwrap() - #3

Hey @LCrossman, I opened a WIP pull request. I will break down the areas that need changes into related commits. Please feel free to take a look and let me know if I am on track.

This all looks good, thanks! Actions to build and run have passed


⚠ Bofrost.lu has degraded performance

In 262ecb3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1208 ms

Resolved: Bofrost.lu performance has improved in 21bb20b after 6 minutes.


Ipsec interface description is not stored when saving.

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at

Describe the bug
When i configure an ipsec connection and establish a connection i used to be able to set a interface description by going to:
Interfaces > IpsecX > Description field
This field was then used as the interface name throughout the opnsense environment.
Now it seems the description field does not get stored when the settings are saved, instead after pressing save the screen is reloaded and the description disappears.
I've tried this with two opnsense installs, my main router and a vm spun up for this purpose. (both running OPNsense 21.7.7-amd64)
I've also tried two browsers, chrome and firefox.
Can you validate this finding?
Thanks,
Peter

You can change the name in the tunnel itself for VTI interfaces, if I remember correctly, that has been the case all along. The automatically added interfaces retrieve their (permanent) description from
https://github.com/opnsense/core/blob/26df508f075aeb3ab119abb3aa72fc792c676fd5/src/etc/inc/plugins.inc.d/ipsec.inc#L206

I don't fully understand what you mean, I've attached example screenshots to iilustrate.
interfaces - assignments.PNG = showing existing named ipsec interfaces and my new ipsec3 interface
interface with description.PNG = showing the interface definition of one of the named interfaces
interface without description.PNG = showing the interface definition of the unnamed ipsec3 interface
The named ipsec interfaces have the description altered and saved a long time ago, may be years ago.
When i try to change the description of the new ipsec3 interface it does not persist a save.
The ipsec tunnels also have a description but (at least for me) this does not match the interface naming in the environment.

You need to change the IPsec descriptions in VPN->IPsec->Phase 1 (or phase 2 if the definition leads to more VTI devices)


Diesel doesn't generate belonging_to trait for structs with derive associations

Setup
Versions

Rust: rustc 1.61.0-nightly (6a7055661 2022-02-27)
Diesel: diesel 1.4.1
Database: Postgres 14.0
Operating System: Debian 11.14-0+deb10u1

Feature Flags

diesel: features = ["chrono", "extras", "postgres", "r2d2"]

Problem Description
Just like in #1203 , diesel does not generate the belonging_to trait for table associations using the belongs_to derive macro.
What are you trying to accomplish?
I'm trying to fetch all classes attended by registered students
#[derive(Identifiable, PartialEq, Queryable, Insertable)]
#[table_name = "applicant"]
pub struct User {
    pub id: i32,
    pub email: Vec<u8>,
    ...
}

#[derive(Identifiable, Queryable, Associations, Insertable)]
#[belongs_to(User, foreign_key = "email")]
#[table_name = "classes"]
pub struct Class {
    pub id: i32,
    pub email: Vec<u8>,
    pub school: String,
    ...
}
...

let students = users::table.load::<User>(conn).unwrap();
let classes = Class::belonging_to(&users).load::<Token>(conn).grouped_by(&users).unwrap();
let students_classes = users.into_iter().zip(tokens).collect::<Vec<_>>();

generates the error:
 Class::belonging_to(&users).load::<Token>(conn).grouped_by(&users).unwrap()
    |   ^^^^^^^^^^^^ function or associated item not found in `Class`
    |
    = help: items from traits can only be used if the trait is implemented and in scope
    = note: the following trait defines an item `belonging_to`, perhaps you need to implement it:
            candidate #1: `BelongingToDsl`

What is the expected output?
No error
What is the actual output?
 Class::belonging_to(&users).load::<Token>(conn).grouped_by(&users).unwrap()
    |   ^^^^^^^^^^^^ function or associated item not found in `Class`
    |
    = help: items from traits can only be used if the trait is implemented and in scope
    = note: the following trait defines an item `belonging_to`, perhaps you need to implement it:
            candidate #1: `BelongingToDsl`

Are you seeing any additional errors?
No
Steps to reproduce
gh repo clone ch3ck/dieserr && cd dieserr && cargo build

Checklist

[x] #1203
[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)
[x] This issue can be reproduced without requiring a third party crate

Issue fixed, no longer necessary

What is the fix for this?

@rcasurion Thats impossible to answer in the general case. Please open a new discussion thread with your question instead of commenting on old closed issues. Include at least some information about your setup, the relevant code and the exact error message as emitted by the compiler.


Possible to cancel modal closing by aborting a transition?

I have a route modal (modal that opens automatically when entering a route) that transitions to a different route on close. The route that the modal is on also has a willTransition hook that warns if the user is trying to leave the page with a dirty changeset. This works great with normal routes (non-modal), however, on modal routes, the transition is aborted correctly but the modal still closes.
Is there a way to somehow cancel closing the modal if the exit transition is aborted?

The Transition has an isAborted property, so we're just returning !transition.isAborted from the modal cancel action

Glad to hear it worked out for you. Have assumed that this property is set async. Feel free to reopen if you face additional issues.


Add initial implementation

@bastjan would you also like a look at this one?

@bastjan would you also like a look at this one?

Sure 😊 Will take a look this afternoon 👍

I had an idea:
It would be cool to have architecture and process diagrams describing the process in high-level.
Like we have at https://k8up.io/k8up/1.1.0/explanations/architecture.html. You can easily create SVG graphics with drawio, e.g https://marketplace.visualstudio.com/items?itemName=hediet.vscode-drawio


No errors or messages appear when none of the recipients' domain MX records can be resolved

If none of the MX records of the recipients;' domains can be resolved, an error should appear.

Resolved issue


Add new Android project wizard

API Level info dialog

Wizard first step

I think this is more for the Create Activity Template, but it's done.


Bug/Question:  arrow'sFixedSizeList is not roundtrippable

Bug
What happened:
Currently if you try to write an arrow FixedSizeList using delta, the original type is converted to a list. I'm wondering if this is a bug or intentional. Are there any suggested workarounds for working with arrow FixedSizeList without losing the fixed size?
What you expected to happen:
I'd expect being able to roundtrip arrow's FixedSizeList to/from delta without losing the original type.
How to reproduce it:
More details:

Intentional, the delta primitive types doesn't have a fixedsizedlist/array, only list


chore(github actions): add action which check if all tests pass

Describe your changes
I added a Github Action which check if all the tests pass
Issue ticket number and link
Issue #93
Checklist before requesting a review

[x] Set Up image where the tests will run
[x] Install dependecies
[x] Run tests

Areas to improve
At the moment I set up when there is a PR on "main" and "v.0.13.0" the github action runs, but that could change

Hi @devrrior, I am hesitant to add this just yet due to I believe the tests need an instance running to test against. We may need to find a way around that.

Ok, I don't get it completely. But I'd appreciate it if you could give me some details (here or on discord) about it to try to find a solution. Thanks :D

Hey @devrrior let's catch up on discord to talk through this. 👍


add option to enable experimental swarm features

Like described here https://github.com/docker/swarm/tree/master/experimental
As far as I understand the -experimental flag has to be passed before the actual manage or join command - I couldn't figure out a way to do so with currently available options.
hacking the source now... :)

That's what I'd need https://github.com/schmunk42/machine/commit/b741f603c70c306b068cbf1591b5436f5718e588
But I am unsure about the best way how to implement the option.

Some more feedback...
I tried to use it via docker-compose
appcli:
  environment:
    - 'reschedule:on-node-failure'

But got the error:
ERROR: invalid reschedule policy: on-node-failure=

I also tried
appcli:
  labels:
    - 'com.docker.swarm.reschedule-policy=["on-node-failure"]'

What basically works is:
appcli:
  labels:
    com.docker.swarm.reschedule-policy: "on-node-failure"

I couldn't see rescheduling yet, because I ran into
time="2016-01-18T22:53:30Z" level=error msg="HTTP error: Engine no longer exists" status=500

I just stopped and started the engine, don't know why this happens (yet)?!

The main workaround is to install a standard machine ie withtout the --swarm flags and start a swarm container within it atm.
I'm not sure we would want to add such a global flag for this, but this is open for discussion. cc @docker/machine-maintainers


Fix unnecessary hyphenation

Solve https://github.com/typst/typst/issues/5360#issue-2631092481
By manually set slight difference as 0.
The root reason is in grow in typst-layout/src/pad.rs, (s + p.abs) / (1.0 - p.rel.get()) is (31.592pt + 10.0pt)/(1.0-0.0). It should be 41.592pt but the given result is 41.59199999999999pt.
https://github.com/typst/typst/issues/5360#issuecomment-2465876365 but not sure how to solve this kind of calculation error.

done

I didn't catch this before, but I find the version in your first commit (with mutable let) a fair bit more readable actually.

Yes. But it will import a mutable delta.

The function isn't that long, I don't think it matters much.

Changed to the first version.

Thanks!


⚠ PrivateHD has degraded performance

In 4ba805b, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1594 ms

Resolved: PrivateHD performance has improved in ee149c9.


HAWQ-749. segment registration should consider same ip across nodes

+1

＋1 except remove -Subproject commit ca59dfa6f727fe3bf3a01904ec30e87f7fa5a67e


[spark][BUILD] Upgrade protobuf-java to 3.25.4

What changes were proposed in this pull request?

Why are the changes needed?

https://github.com/protocolbuffers/protobuf/compare/v3.25.3...v3.25.4
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Pass GA.
Was this patch authored or co-authored using generative AI tooling?

No.

Can you file a JIRA? Thank you @dev-lpq

Can you file a JIRA? Thank you @dev-lpq

[SPARK-49497](https://issues.apache.org/jira/browse/SPARK-49497)

Merged to master.
Thank you @dev-lpq


install crystal by conda ？

Submitting questions
It is possible to install crystal by conda(https://anaconda.org/anaconda/conda） ?
because I have no root permission to install the dependence of crystal.
Submitting bugs
not bug~

had fixed this by conda install -c conda-forge libevent pcre -n crystal

I'm glad you've found a workaround.
We don't use conda.
Depending on the platform you can download .tar.gz from GitHub releases and use it directly.
snapcraft.io might also be an option, but I'm not 100% sure.
Please use https://forum.crystal-lang.org to keep this thread if needed.


Pluggable backend

With the changes proposed in the sql repo branch and from conversations with @bwalsh I would like to begin a conversation about how best to use multiple backends alongside each other. Brian is implementing an ES backend for some parts of the API and it would be valuable to agree on an approach on how to handle SQL backed, file-backed, and other DB backed protocols, and how to use them in concert. This is also looking towards analyzing requirements for supporting the Genome Browser's databases.
The primary requirement is that each container can set its own backend. For example, I can add a variant set backed by VCF, or a variant set backed by SQL, and the request will be routed to the proper type. The current backend.py is tied to the file-backed approach. For sequence annotations, @macieksmuga chose to handle functions that handle paging from within the datamodel. This same pattern is being followed by the RNA work.
To use SA as an example, it might make sense to move the class Gff3DbBackend to a sqliteBackend file so the datamodel classes concern themselves with coercing to and from database responses to protocol elements. When adding a feature set to the repo one must determine the backing it will use, since there are only SQL backings for SA the example is contrived, but hopefully you understand the direction. We should approach the problem so that we don't end up with multiple SQLite databases so that the repo manager and the backend act on the same data structure.

Good to start this discussion.
Lets start with functional requirements.  Each container having
it's own back end is an implementation requirements.  To know if
this is right, we need articulate what we want to accomplish in
terms of functionality.
[I do think this is a right implementation requirement, but that
doesn't mean anything unless the functional requirements are
clear.]
Also, it might make sense to do away with paging before
implementing any big changes, so don't assume that is a
requirement.
David Steinberg<EMAIL_ADDRESS>writes:

With the changes proposed in the sql repo branch and from conversations with
@bwalsh I would like to begin a conversation about how best to use multiple
backends alongside each other. Brian is implementing an ES backend for some
parts of the API and it would be valuable to agree on an approach on how to
handle SQL backed, file-backed, and other DB backed protocols, and how to use
them in concert. This is also looking towards analyzing requirements for
supporting the Genome Browser's databases.
The primary requirement is that each container can set its own backend. For
example, I can add a variant set backed by VCF, or a variant set backed by SQL,
and the request will be routed to the proper type. The current backend.py is
tied to the file-backed approach. For sequence annotations, @macieksmuga chose
to handle functions that handle paging from within the datamodel. This same
pattern is being followed by the RNA work.
To use SA as an example, it might make sense to move the class Gff3DbBackend to
a sqliteBackend file so the datamodel classes concern themselves with coercing
to and from database responses to protocol elements. When adding a feature set
to the repo one must determine the backing it will use, since there are only
SQL backings for SA the example is contrived, but hopefully you understand the
direction. We should approach the problem so that we don't end up with multiple
SQLite databases so that the repo manager and the backend act on the same data
structure.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub*

Thanks @david4096 for starting this discussion!
One suggestion: The specific datamodel-centric code, instead of opening and managing its own database, could instead obtain a connection to a shared database from more generic code in the ga4gh.sqliteBackend module, that maintains a server-wide database with individual tables carved out for the different types of data it serves out. I'm not sure that you could entirely get rid of all intermediate code currently in classes like Gff3DbBackend, at least not in a first approximation refactoring.

This is an excellent time to talk about this @david4096; thanks for getting the discussion going. Some thoughts:

I find the terminology 'backend' a bit confusing here, as we have a Backend object within the server, which does something pretty specific. Perhaps Container would be more suitable? Each Container can be an instance of PysamContainer, SqlContainer, JsonServiceContainer, SparQLContainer, etc, etc. The idea is that one VariantSet, ReadGroupSet, etc is a Container, and the Container is then responsible for fetching the elements within that Container. Each Container is independent, so that each one is responsible for instantiating the correct type. Paging is a painful detail which breaks this nice layering; we'll need to figure out a way of passing page token information between the layers, so that a Container can pick up iteration where it left off. (Without paging it's beautifully simple: the Container just returns an iterator over the ProtocolElements in question.

@macieksmuga, this is a good idea. However, we shouldn't take this too far and merge the Container DBs with the Repo DB. There are advantages in keeping the Container and Repo infrastructure separate.

Perhaps a UML diagram or two would be useful here?

How about `datasource'; although in the generic sense, these are
databases (even BAM).  Container's doesn't implement indexing
and queries.
`database adapter' is very clear to me.
Jerome Kelleher<EMAIL_ADDRESS>writes:

This is an excellent time to talk about this @david4096; thanks for getting the
discussion going. Some thoughts:

I find the terminology 'backend' a bit confusing here, as we have a Backend
object within the server, which does something pretty specific. Perhaps
Container would be more suitable? Each Container can be an instance of
PysamContainer, SqlContainer, JsonServiceContainer, SparQLContainer, etc,
etc. The idea is that one VariantSet, ReadGroupSet, etc is a Container, and
the Container is then responsible for fetching the elements within that
Container. Each Container is independent, so that each one is responsible
for instantiating the correct type. Paging is a painful detail which breaks
this nice layering; we'll need to figure out a way of passing page token
information between the layers, so that a Container can pick up iteration
where it left off. (Without paging it's beautifully simple: the Container
just returns an iterator over the ProtocolElements in question.

@macieksmuga, this is a good idea. However, we shouldn't take this too far
and merge the Container DBs with the Repo DB. There are advantages in
keeping the Container and Repo infrastructure separate.

Perhaps a UML diagram or two would be useful here?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub*

Hmm, DataSource is good. Not hugely keen on DatabaseAdapter --- sounds too much like it's tied to SQL to me...

datasource does have the clarity that it's not a database we
control.
Even with the NoSQL stuff everywhere, I still have to convince
people database != SQL ; we can't escape our engineering
childhood..
Jerome Kelleher<EMAIL_ADDRESS>writes:

Hmm, DataSource is good. Not hugely keen on DatabaseAdapter --- sounds too much
like it's tied to SQL to me...
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub*

Ideally, we should be able to add a new Connector "adding some files that implement a specific, documented convention or interface & configuring via yml".  A key test would be: "As an implementor, in order to integrate a new data source for a portion of the api,  I should not need to fork the server and maintain it as others change core parts of the server."
Currently, all things being equal, it seems like the path to re-use would be to write a new class say datamodel/es-biodata.py,  inherit from biodata.py and add the  elastic-search aware getIndividualByIndex,getBioSampleByIndex methods to it. This would mean patching the code in backend.py, probably bioSamplesGeneratorES, individualsGeneratorES - that doesn’t feel too clean.
In order to add a new backend/container, I needed to make these changes before writing any specific code.  Gist of the my setup

The second thing we run into, at least in the biodata area, is that the keys for individuals, samples etc are  wrapped up with data sets,etc.

Good requirements @bwalsh, I like your thinking.
@diekhans, DataAdapter? It's pretty generic, but we're capturing the idea that we're adapting the data from some source from our needs. I just can't bring myself to call a VCF file a database...

What do we need to do to move this design forward? I see this as being key to wider adoption of the ref server. I think there will be some users of the server who will want higher speed access to indexed data than we can provide with our current file-backed approach. There will also be those who do not want to do an ETL on their data (due to space limitations) and are fine with the current performance. The best way to make both happy is to have a pluggable architecture to facilitate either data layout.

This is part of the ongoing SQL usage refactor @kozbo, I'm working on it...

Step #1: documented goals, requirements, and use cases.
Kevin Osborn<EMAIL_ADDRESS>writes:

What do we need to do to move this design forward? I see this as being key to
wider adoption of the ref server. I think there will be some users of the
server who will want higher speed access to indexed data than we can provide
with our current file-backed approach. There will also be those who do not want
to do an ETL on their data (due to space limitations) and are fine with the
current performance. The best way to make both happy is to have a pluggable
architecture to facilitate either data layout.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.*

To clarify, I'm working on the 'plugability' part of this, not the actual implementations. A key goal of the refactor is to enable simple polymorphism among data sources like VariantSet, so that we can have different types of VariantSet coexisting within the same repository.

Thanks for the clarification @jeromekelleher and sorry that I missed the connection with the sql refactor.
Per @diekhans I have started a design document for this effort here

Being worked on in #1259


Feat: Sentry Performance for HTTP client

:scroll: Description
Feat: Sentry Performance for HTTP client
:bulb: Motivation and Context
Closes https://github.com/getsentry/sentry-dart/issues/595
:green_heart: How did you test it?
running and unit tests
:pencil: Checklist

[X] I reviewed submitted code
[X] I added tests to verify changes
[ ] I updated the docs if needed
[X] All tests passing
[X] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #603 (c6e6164) into main (d07e63a) will increase coverage by 1.71%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #603      +/-   ##
==========================================
+ Coverage   90.38%   92.09%   +1.71%
==========================================
  Files          89        7      -82
  Lines        2953      253    -2700
==========================================
- Hits         2669      233    -2436
+ Misses        284       20     -264

Impacted Files
Coverage Δ

dart/lib/src/protocol/sentry_culture.dart

dart/lib/src/protocol/debug_meta.dart

dart/lib/src/protocol/span_status.dart

dart/lib/src/protocol/sentry_trace_context.dart

dart/lib/src/sentry_traces_sampler.dart

dart/lib/src/protocol/dsn.dart

dart/lib/src/scope.dart

dart/lib/src/version.dart

...art/lib/src/http_client/failed_request_client.dart

dart/lib/src/default_integrations.dart

... and 65 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d07e63a...c6e6164. Read the comment docs.

Isn't there a SentryOption which decides wether tracing is enabled or not?

Isn't there a SentryOption which decides wether tracing is enabled or not?

yes, but SentryHttpClient nor Hub have access to the SentryOptions, Java exposed Hub#getOptions as workaround, we can do that maybe later, I'd not block this PR right now tho


add more sarra documentation

Add more documentation related to sarracenia:

how to see a list of external brokers
how to select a broker
how to filter on 1..n topics
how to enable/disable connection to brokers

website now available for sr3 docs in development/review
uses Sphinx
docs (alot of them!) need review

new web page for documentation for version in wis2box: https://metpx.github.io/sarracenia/


Update README.md

Docs Build status updates of commit 61f87b4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

README.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


feat(data): Add Model#eachAttribute

Adds a memoized iterator if you need to loop over the attribute definitions for a given model.

Coverage decreased (-1.3%) to 46.388% when pulling 8b38753dac2c14bff28969f412ef6896b96d2a83 on each-attribute into 21e7192a9e9467b7219497e013382722ed7e6d49 on master.


feat: reactoring cli & add html5 core

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (9c22c3a) to head (b7d244d).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           89        89
=========================================
  Hits            89        89

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


bug: AUTH opcode should unset and push0 if authority account has code

From EIP 3074:

authority is the address of the account which generated the signature. If EXTCODESIZE of authority is not zero, consider the operation unsuccessful and unset authorized.

What I understand from this:

After loading the account, check the code size

If not 0, unset authorized, push 0, stop executing the logic of AUTH and continue the execution of the current frame.

Which means that here we should add something like (pseudocode)
if acc.code.len() != 0 {
   push_stack(0);
   ctx.remove(AUTHORIZED_VAR_NAME);
   return;
}

https://github.com/paradigmxyz/alphanet/blob/008524c84f044c0463ca692b0eca96dc15f64c78/crates/instructions/src/eip3074.rs#L170-L176
I'd be glad to PR this if approved :)

If EXTCODESIZE of authority is not zero, consider the operation unsuccessful and unset authorized.

I see, so we're lacking a check for this @fgimenez ?
should be easy to add this by checking the account's codehash against KECCAK_EMPTY

yeah, we need the check, although I'm not sure whether we should push 0 to the stack, or just set instruction_result to Stop

It should push 0 as per @lightclients ' comment https://twitter.com/lightclients/status/1779900806990270773

@enitrat thanks for the report! should be fixed here https://github.com/paradigmxyz/alphanet/pull/59 PTAL


Suggestions

Would be possible make some suggestions? I can imageine something like phpDoc notaion
{** @var \Entity\User $test *}
// comment above would suggest public methods and properties from \Entity\User entity
{$test->}

Is this possible and valid for this plugin?

Yes, feel free to send PR.

@JanTvrdik @matej21  I believe this feature is one of most important. If we do some campaign and find some people who are willing to put some money into it (I definitivelly am and believe many more people would also pay for this) would you implement this? Sadly my knowledge of Java and PHPStorm framework is non existant but this is something I would really like to see. It's something that's really missing!

It is possible from 1.0.0

@mesour Is there any reason for choosing macro-way ({varType}) over annotation ({** @var *})? Could not find any mention about it. Thanks.

@dakur because comments in Latte starts with {* and ends with *}. It would be unnecessary to refactor comment handling in Latte and also in parser in the plugin. Therefore, a new tag varType has been created.

Plus I believe with latte 2.7 this will be completelly supported and probably with using strict template will also give exception, maybe there's even phpstan extension planned, so this is not just on this plugin level, but on latte level.

@mesour But isn't this misuse of what macros meant to be for just because "it would need more work"? Isn't it similar if someone started to use functions in PHP to type variables (type() int $abcd) just because /**/ would need more work to distinguish comment from annotation? I personally understand macros as kind of functions which do something, whereas typing is just describing stuff, not doing something.

@dakur thanks.
I think it is not the same as in PHP. Because in Latte are used macros (now named as "tags"). And only in these tags you can use PHP. I think of a tag as a placeholder for certain code in PHP and it can also be an @var annotation. I originally wanted {** ... *} but then someone told me it would be better via tag. David did not object to varType when I asked about the implementation of Latte. It also makes sense to me with other tags for types like templateType.


Standing Bug

Dets at https://www.reddit.com/r/RimWorld/comments/mo70zz/mod_issue_standing_crafters/
I've seen some references to standing bugs in your mod discord history
Thanks!

Update: It is your other mod EM Tools that are the items of conflict. I'll try removing them from storage and try a stockpile

That sounds like quite a complex bug to replicate. Might even have to rewrite most of the mod. You probably already saw the other announcents, but hiatus until September

Happens with a stockpile too.
Yeah I saw. Good luck!

INTERESTING. So I dropped EM Tools and noticed it happened again. I think I know what needs to happen now is a pawn carry weight check is missing.

Good discovery. Indeed I did not add a check for that and I should be able to replicate that. What logic to implement on the other hand... who knows


Update add-users-administrator.md with new resend invitation experience

The screenshot should be updated to the new experience and the documentation should indicate as well that inviting permissions are now required for resend invitations as this was recently changed.

@nubartol : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@msmimart Could you review the proposed changes? If they are approved, we'll need to move the commit to the private repo and create an image file to add to the repo from the link provided. Thanks.

Hi @msmimart thank you for the suggestions, those are perfectly in line with my expectations when I made the update suggestion.
Deleted the screenshot and updated the "7." accordingly. Please let me know how to proceed as this is the first time I've suggested an update to the documentation.

Thank you for making these changes, I'll go ahead and sign off. Then I'll follow up with a pull request for a new screenshot.
#sign-off


[undo-plugin] UndoManager is destroyed on view.destroy but created in state.init

Checklist

[X] Are you reporting a bug? Use github issues for bug reports and feature requests. For general questions, please use https://discuss.yjs.dev/
[X] Try to report your issue in the correct repository. Yjs consists of many modules. When in doubt, report it to https://github.com/yjs/yjs/issues/

Describe the bug
When Prosemirror view calls EditorView.prototype.updatePluginViews, it will destroy existing plugin views and recreate them if a new plugin is added.
This is an issue if the yUndoPlugin has already been registered because it creates the instance of YJS UndoManager in the state lifecycle but destroys it with the view lifecycle.
To Reproduce
Steps to reproduce the behavior:

Load a prosemirror editor with the y-sync and y-undo plugins enabled
Register a new prosemirror plugin
Observe the UndoManager get destroyed
The UndoManager remains destroyed

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment Information

Chrome Version 99.0.4844.84 (Official Build) (arm64)

➜  client git:(main) ✗ yarn info y-prosemirror
└─ y-prosemirror@npm:1.0.17
├─ Instances: 1
├─ Version: 1.0.17
│
└─ Dependencies
└─ lib0@npm:^0.2.42 → npm:0.2.48
➜  client git:(main) ✗ yarn info yjs
└─ yjs@npm:13.5.33
├─ Version: 13.5.33
│
└─ Dependencies
└─ lib0@npm:^0.2.48 → npm:0.2.48
Additional context
This is a new problem with the introduction of the .destroy method in https://github.com/yjs/yjs/commit/e9a0dc4ed2228bf9f2354495d9aeb19c2f66e858

The introduction of destroy on the undo manager was necessary because these undo managers were never cleanup up after the editor was destroyed. Hence, there was still one Y.UndoManager active for each binding ever created, tracking operations that would never be undone. We even had multiple undo managers for the same document.
You are right though, the current implementation is also not ideal as it deletes the undo manager when the view is destroyed.
Is there some kind of method to recognize when the editor has been destroyed, or when the Y.UndoManager state was unregistered?

Is there some kind of method to recognize when the editor has been destroyed, or when the Y.UndoManager state was unregistered?

This is a great question. I'm not sure off the top of my head and after looking through the code. I'd have to ask @philippkuehn or @marijnh about it.
The only idea that comes to mind here is to do lazy instantiation of the UndoManager instance in the .apply method. I.e. each time apply runs, check if the current instance has been destroyed and if so, create a new one as part of the apply lifecycle. It might be more proper to even remove the UndoManager from the plugin state in view.destroy via a transaction.
In most applications, I think plugin registration only happens on load, so this shouldn't really fire too often.

Is there some kind of method to recognize when the editor has been destroyed

Plugin views' destroy method should work for that.

Thank you for jumping in @marijnh!  However, according to @jamesopti the destroy method is also fired when the plugins are reconfigured. The problem is that we don't want to destroy the stateful UndoManager in this case because we would lose all the captured undo-history.
The stateful UndoManager currently lives on a ProseMirror state object. I realize that this is not ideal, but I chose this approach because I need to access the UndoManager from other plugins. y-codemirror.next stores the UndoManager on an Annotation instead, which seems to be much better suited for this use case.

@jamesopti I think there is no ideal solution for this problem. I suggest that we keep the current approach to destroy the undo manager in destroy. However, we can recreate the undo manager when the undo manager is still used after the view has been destroyed.
Furthermore, we can enforce that the undo manager is not going to be destroyed when it is submitted by you manually. This gives you the control to reconfigure plugins without destroying the undo manager. However, this means that you are responsible for destroying the undo manager when you don't need it anymore.

Is this feature still needed or did you find another solution @jamesopti ?
I'm closing for now, but feel free to reopen.

https://github.com/ueberdosis/tiptap/issues/2761

@jamesopti @dmonad could we get this reopened? At the moment registering a Prosemirror plugin after the y-undo plugin breaks all undo functionality. This is because registering a plugin destroys previous plugin views which destroys the undo manager which causes it to stop listening to the afterTransaction event. From my understanding it never starts listening to this event again when the view is recreated.
If we can get it to start listening again when the view is created, that could be a quick fix for now. Thoughts?


Oracle Database Auditor: Workbooks with not results

I'm sorry but I can't reopen the case. My user doesn't have this feature allow it.
Describe the bug
Oracle Database Auditor doesn't bring information about Oracle Database Table Queried and User' Privileges. It only shows the query returned no results
To Reproduce
Steps to reproduce the behavior:
Active Oracle Database Audit.
Enable Syslog in Linux Server.
Wait until Sentinel get the data.
Expected behavior
To be fair all the information are shows but those two. It's possible to modify the query to receive these information or why is in the workbook's sample these information?
If i try to execute use it the TableName object doesn't work either.
i create this issue previously here:
https://github.com/Azure/Azure-Sentinel/issues/9602
it was closed because v-muuppugund closed the issue (https://github.com/Azure/Azure-Sentinel/issues/9987) as per your standard operating procedures
And, i try to execute the sample shows by  v-muuppugund but i don't get nothing new:

Is it possible that the change is not available, not yet?
The case was seeing by the user:
v-muuppugund (thanks for the help). i'm very grateful

Hey @gmarmolejos, we have raised the PR with the corrections. Once PR get merge the changes will be reflected in upcoming solution version.
PR link #10273
Thanks!

I'm sorry but I can't reopen the case. My user doesn't have this feature allow it.
Describe the bug Oracle Database Auditor doesn't bring information about Oracle Database Table Queried and User' Privileges. It only shows the query returned no results
To Reproduce Steps to reproduce the behavior:
Active Oracle Database Audit. Enable Syslog in Linux Server. Wait until Sentinel get the data. Expected behavior To be fair all the information are shows but those two. It's possible to modify the query to receive these information or why is in the workbook's sample these information? If i try to execute use it the TableName object doesn't work either.
i create this issue previously here: #9602
it was closed because v-muuppugund closed the issue (#9987) as per your standard operating procedures
And, i try to execute the sample shows by v-muuppugund but i don't get nothing new:

Is it possible that the change is not available, not yet?
The case was seeing by the user:
v-muuppugund (thanks for the help). i'm very grateful
Mistake due to data during that time duration ,added the updated screen shot

Hi @gmarmolejos, waiting for your response on above comment. Thanks!

Hi @gmarmolejos, Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond to it in the next 2 days. If we don't receive a response by 27-05-2024 date, we will be closing this issue.
Thanks!

Hi @gmarmolejos, since we have not received a response in the last 5 days, we are closing your issue as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


Unable to move file, 'FileSystemDock::_move_operation_confirm' expects 2 arguments from dir_selected signal but called with 1

Godot version:
3.1.x
OS/device including version:
ubuntu, nvidia
Issue description:
Attempting to move a tscn file into a new directory via what I think is called the filesystem dock (the thumbnails of files on the lhs of the editor) does nothing and in the terminal the following error is thrown:
ERROR: emit_signal: Error calling method from signal 'dir_selected': 'FileSystemDock::_move_operation_confirm': Method expected 2 arguments, but called with 1

Just to confirm, this is referring to selecting "Move To..." from the right-click/context menu and then picking a directory in the dialog that pops up?
No problems with drag and drop?


Latest Percepio Tracealyzer Recorder v4.3.11

Latest Percepio Tracealyzer Recorder, v4.3.11
Description
The recorder in the repository is greatly outdated and doesn't compile with later FreeRTOS versions.
As part of Percepio's new release procedure we will be creating pull requests to keep the trace recorder in FreeRTOS-Plus up-to-date.
Test Steps
N/A
Related Issue
N/A
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. Note that the terms of usage stated by Percepio is provided in the source files.

This is the trace recorder you get when installing Tracealyzer v4.3.11.

/bot run checks

@eriktamlin The branch is out of sync due to the uncrustify PRs, and I do not seem to have permission to push to your branch for rebasing purpose. Would you prefer us replacing this PR with a new one on your behalf, or rebasing your fork branch by yourself? Let us know. And thank you for the contribution.

I rebased the PR to upstream/master.
It says a lot of files have changed...?

I believe branch merge caused some fraction. (Please see #169 the attached branch history snapshots, that may better explain.) I'll put down "approved" for now for this PR, since I believe the contents are the same.
(Note for other reviewers -- We'll need to do a squash merge for this PR, if accept as-is.)

/bot run checks

Should I close the this one and we just go with [https://github.com/FreeRTOS/FreeRTOS/pull/169](your redo) instead?

@eriktamlin  No worries, we'll take care of it. This is in queue, we should be able to pick one and close the other.

The changes were accepted in this PR: https://github.com/FreeRTOS/FreeRTOS/pull/201
Also fixed the Tracealyzer demo build error.

Closing this PR. Thank you @mingyue86010.
If any pending discussion, let's move to #201.


Spelling fixes

Low priority PR/changes.  Just looking around to code to become familiar with it.  Feel free to discard if not interested.

In my opinion, fixing most of the spelling errors makes zlib-ng look more credible as a project. I don't see any incorrect fixes in the pull request. @Dead2 is the only one who can merge pull requests...


[WIP] Upgrade guide BDB v0.10 -> v1.0

WIP, @r-marques and me will add notes on all breaking changes.

Codecov Report

Merging #1547 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1547   +/-   ##
=======================================
  Coverage   98.39%   98.39%
=======================================
  Files          58       58
  Lines        2812     2812
=======================================
  Hits         2767     2767
  Misses         45       45

@sbellem, there is a section on CC V2 here. You might want to check it out.

Added a note about the change in the transaction version.

Note: Update the Inputs section if #1596 goes through

@TimDaub I think we can merge this. All the breaking changes are in the document.


Customize formule style

I am using katex on my blog via cdn
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/KaTeX/0.5.1/katex.min.css">
<script src="https://cdnjs.cloudflare.com/ajax/libs/KaTeX/0.5.1/katex.min.js"></script>

The default css is not fit to the theme of my blog.
Is there a way to change the formule css style on client code ?
I am using auto-render. A formule is like the following
$$
\begin{array}{clc}
(3, 4) &\rightarrow f(x) = (3x + 4) \% 12 \\
x = 0, 1, 2 .. 11 &\rightarrow f(x) = 4, 7, 10, 1, 4, 7, 10, 1, 4, 7, 10, 1
\end{array}
$$

I can not get the class of the formule html element. A workaround is make the formule block surrounded by  ... . It is tedious.
In MathJax, I can easily add class to all of the formule block:
<script type="text/x-mathjax-config">
  MathJax.Hub.Queue(function() {
    // Fix <code> tags after MathJax finishes running. This is a
    // hack to overcome a shortcoming of Markdown. Discussion at
    // https://github.com/mojombo/jekyll/issues/199
    var all = MathJax.Hub.getAllJax(), i;
    for(i = 0; i < all.length; i += 1) {
        all[i].SourceElement().parentNode.className += ' has-jax';
    }
});
</script>

And edit the css code of class has-jax
I am not sure Katex has this kind of feature.
Any idea ?

Can you not apply styles to the span.katex elements? I'm not sure what the .has-jax class does.

Thank you. It works. I find the class name in the document.
I should have been more careful.

@invkrh has the issue been resolved?

Yes, it is resolved.
I close the issue


[cxx-interop] Add flag to assume C++ types are resilient

Currently, C++ types cannot appear in resilient interfaces. There are some cases where this is overly restrictive. We plan to improve the logic to detect what types should not appear on resilient moduel boundaries. In the meantime, this PR introduces a flag to disable these errors. Users relying on this flag are on their own, this should only be a temporary workaround until we land further improvements to this diagnostic.
rdar://137457118

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


Workshop dry run prep

Temporarily disabling Common GPU image since gromacs on develop requires CUDA 12, and our base image with conda and cuda only goes til 11.8


Incorrect version show when installing an extension

UI always shows the latest version - if you select another version, while installing, the latest version is always shown instead of the version that you selected.

I am able to install an older version of a UI extension and the UI displays the correct version of the extension that was installed. moving to done.
https://zube.io/files/rancher/c45a8cd49e9c12d3758e5e344dc00392-retest-7282.mp4