Update DOM from TSJS-lib-generator

Update Navigator and other small updates.
Make ProgressEvent generic.
Make window: Window & typeof globalThis.
Add types:

CSS Overflow
CSS Masking
Web Authentication
WebGL 2

The big change is that window now includes globals in its type via typeof globalThis. This helps some codebases a lot, such as chrome-devtools-frontend.

@typescript-bot user test this

@typescript-bot test this

@typescript-bot run dt

User tests look good; no surprises there. Chrome-devtools-frontend especially benefits.

From RWC:

We lose related spans on members declared as part of Window and declared globally:

window.navigate
       ~~~~~~~~
       'navigate' not found on 'Window & typeof globalThis'. Did you mean 'navigator'?

But it no longer has a related span "navigator is declared here", because navigator now has two declarations.

Casting window as MyMadeupWindowSubtype will not work until MyMadeupWindowSubtype is revised to include properties from globalThis, or the cast is doubled to cast through unknown.

VS Code walks a parent chain of windows, but starts the loop with let w = window. It now needs to be let w: Window = window.

I don't think any of these breaks are bad.

DT looks fine, just a couple of $ExpectType comments need to be updated.


simple inference ?

I want to make simple inference for some images I have and the results are bad

from models.GCoNet import GCoNet
import torch
from pathlib import Path
from torchvision.io import read_image
from torchvision.utils import make_grid, save_image
from torchvision import transforms

dataset_path = Path("/path/to/data")

device = torch.device("cuda")
model = GCoNet()
model = model.to(device)
gconet_dict = torch.load("/path/to/weights")
# gconet_dict = torch.load("/shared_volume/rashad/salient_app/ultimate.pth")
trans = transforms.Compose([
            transforms.Resize((320, 320)),
            # transforms.ToTensor(),
            transforms.Normalize([0.485, 0.456, 0.406], [0.229, 0.224, 0.225]),
        ])

model.to(device)
model.load_state_dict(gconet_dict)

model.eval()
images = [read_image(str(img_path))[None, ...] for img_path in dataset_path.glob("11/*")]
# print(images)
torch_images = torch.concat(images).float()
torch_images = trans(torch_images)
print(torch_images.shape)
with torch.no_grad():
    result = model(torch_images.cuda())[-1]
    result = torch.concat([torch.nn.functional.interpolate(res.unsqueeze(0), size=(res.shape[1], res.shape[2]), mode='bilinear', align_corners=True) for res in result]).sigmoid()
result = (result > result.mean()).float() * 255
print(result.shape)
print(result.max(), result.min())
save_image(make_grid(result), "salient_example.png")

And this is the result,

@HzFu @ZhengPeng7

They are standard JPG images, this is an example

Yeah, but I mean... can you give me some group of images? Since it's co-sod, the number of images in one group does affect the predictions.

Hi Mohamed, I made my inferene code on the colab online with the ultimate.pth, including one group of CoCA (accordion) and the group you gave (kettle). Here are the screenshots of the colab.

I still have some other urgent task nearly finished but not yet. So I will polish up my colab tomorrow for anyone to do the inference by our model online. Welcome to visit it.
By the way, if you want to use our model for general co-sod on images outside the available datasets, we recommend you to use the ultimate_duts_cocoseg (The best one).pth here, since it is trained by two datasets.
Maybe there is something mistaken in your inference codes, you can give it a double check or just wait for my updates on colab :)

I think the problem is in the preprocessing, but I am not sure where.
If you could add the colab notebook I will be thankful

Hi, Mohamed,
Sorry that I forgot this for days...😂
I just polish the processing of my codes in colab.
You can have a try on the colab-demo now.
I've checked it, the predicted results are exactly the same as what I show in README and uploaded to google-drive.
Hope this help you. And any new question, just comment.

@ZhengPeng7 To come late is better than not coming at all.
Thanks for your efforts, I will close the issue now.


Add MODEL_URI to env to download artifacts (e.g. Scaler) at runtime

Add MODEL_URI to env to download artifacts (e.g. Scaler) at runtime

@pille321 can you explain why you need the MODEL_URI as and environment variable? It is passed over the provided mlflow deployment API and piped through into the template as a parameter, anyway.

@sklingel We train our model with scaled values and store the scalers next to the model in our S3-Bucket
mlflow.log_artifact('input_scaler.pkl')
mlflow.log_artifact('output_scaler.pkl')

To get valid predictions from our model after the deplyoment we have to scale the input and output values again. Therefore we need to download the *.pkl-Files from the S3-Bucket.

Thank you @pille321 for answering this, tbh though, I don't understand how an environment variable will help you in that case given the rest of the logic from the deploy_with_auth.yml.
However, though a more cleaner and mlflow integrated approach might be to either use something like https://scikit-learn.org/stable/modules/generated/sklearn.pipeline.Pipeline.html and save everything packed together with a coherent API.
Or given your machine learning package does not support something like this, wrap your input/output scalar in a custom MLflow flavor (https://www.mlflow.org/docs/latest/models.html#id43).
In my experience both approaches work fine and are perhaps easier to handle and future-proof in terms of maintainability and scalability.


Theme support fixes

Visual clarity for disabled buttons
Theme support for Handle statistics dialog (for scrollbar need IAT hook of OpenNcThemeData, because there is no Scroll bar control in this dialog).

Theme support for Handle statistics dialog (for scrollbar need IAT hook of OpenNcThemeData, because there is no Scroll bar control in this dialog).

Enabling the theme for the listview also enables the scrollbar theme?

Theme support for Handle statistics dialog (for scrollbar need IAT hook of OpenNcThemeData, because there is no Scroll bar control in this dialog).

Enabling the theme for the listview also enables the scrollbar theme?

I moved down theme init line (after whole dialog init code), and the scrollbar also started working. I added theme support for Job dialogs. Video preview in PR description.


The time when "slick-active" class is being added is inconsistent across slides

Currently:

If the next slide is a cloned slide, "slick-active" class is being added after animation ends.
If the next slide is not a cloned slide, "slick-active" class is being added before animation starts.

The second behavior is supposed to be the correct one.
This issue can cause visual inconsistency when we need to use "slick-active" class to implement some CSS transitions.

If the next slide is a cloned slide, "slick-active" class is being added after animation ends.

It's not quite right.
The class is also added before the animation begins, just, it's added to the non-cloned slide.
But the problem is live!

$('.sliderella').on('beforeChange', function(event, slick, currentSlide, nextSlide) { 	$('.slick-slide').removeClass('slick-quick'); 	$('.slick-active').next().addClass('slick-quick'); });
Then use the .slick-quick class in your CSS transitions. A quick fix, until this issue gets some attention.

I also noticed this problem today, following this issue for a fix!

In centerMode you can use slick.center for CSS animations, which seems to work better on infinite sliders.


Functions taking &[T] might be more generic with impl IntoIterator<Item = T>

We have various functions that take a slice as argument currently. Most of them would convert the slice into a Vec<*mut _> or similar before passing to C functions. As such it would make sense to directly allow using anything that can be iterated over.
For functions taking some plain type T this is straightforward (e.g. &[&str] would become impl IntoIterator<Item = &str>), for functions taking a &dyn T (e.g. &[&dyn ToValue]) this is a bit problematic because of https://github.com/rust-lang/rust/issues/89222.
Because of that I would suggest to only do it for slices that are not containing trait objects for now, unless someone knows a way around that.
@bilelmoussaoui @GuillaumeGomez Any opinions?

If the compiler errors are acceptable, then why not.


Fix typo in README

Fixing a typo in the check definition example

@apaskulin Great find! Do you think you could add a sign-off to your last commit? See https://github.com/pi-hole/pi-hole/wiki/How-to-signoff-your-commits. for brief instructions.
I should totally add a "Contributing" section to the readme to explain this!

Success! Thanks for the link :)


Develop

Merging in CI fixes and new workloads with the rewrite changes from Brad.

Whoops! Ignore!


V2 - Add CLA Manager - Issues while adding CLA Manager

Issue1 - Appropriate error message is not displayed when trying to add already added user (without LFID) as CLA Manager

**Issue2 - Invalid email format is allowed **

Observed that Issue2 is fixed. However still 400 is returned for Issue1. 202 should be returned. As of now for when duplicate CLA Manager designee is made it returns 202.
Since 400 is returned, Error is displayed in the UI

Issue1 is not valid anymore as we are not allowing to add user without LFID as CLA Manager as per #1882. Other issues are fixed


[INLONG-4292][Sort][Manager][Agent][TubeMQ] Upgrade log4j version configuration

Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-4292][Sort][Manager][Agent][TubeMQ] Upgrade log4j version configuration

Fixes #4292

Motivation
Unify all log output formats，simplifies user configuration
Modifications
Upgrade all log4j properties to log4j2 properties. Syntax upgrade

Duplicate with #4293.


SA1011 incorrectly triggers on list pattern with variable declaration

1.2.0-beta.507
case "Something" when t is [SomeType s]:

With the code above I get a SA1011 on for the end ] in the list pattern.

Does it want a space after the closing square bracket?
Did this start happening when upgrading to the new release? Just curious if it broke in that release, or if it was broken before.

Yes, it wants a space after the closing square bracket.
I tested with beta.435 and it behaves the same so not a new issue.
Simpler case:
    public static void Foo(object[] stuff)
    {
        switch (stuff)
        {
            case [string s]:
                break;
        }
    }

Thanks for the update!


[SPARK-18551] [Web UI] [Core] [WIP] Add functionality to delete event logs from the History Server UI

What changes were proposed in this pull request?
Sometimes a Spark user will only have access to a History Server to interact with their (past) applications. But without access to the server they can only delete applications through use of the FS Cleaner feature, which itself can only clean logs older than a set date.
I've added functionality to delete individual event logs from the history server, added a request handler to call said function and added a button to the UI to call it. The code is based on the current cleanLogs and kill jobs/stages code. I've also set this functionality to off by default.
I've left this as WIP because I still need to add tests and documentation for the new code and have design questions that I need input on (commented inline below)
How was this patch tested?
Manual testing and dev/run-tests

Test build #69041 has started for PR 15984 at commit d243293.

Test build #69041 has finished for PR 15984 at commit d243293.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69041/
Test PASSed.

Merged build finished. Test PASSed.

Like you note, tests will be good here. Don't forget the corner cases: unknown app, duplicate POSTs, known app but unknown attempt. I'm also curious about what the policy would be if there is >1 attempt in the history, but no attempt Id in the delete.
Also: would it ever be the case that a history provider didn't support deletion? I can see where that would arise, but suspect that in such a case, you'd just deploy it with the deletion option disabled

Note: Pausing work on this until JIRA discussion is addressed.

Closing this based on security and complexity issues raised by @vanzin on the JIRA


Reordered logging levels, added alert logger

What's inside:

Added the "alert" logger, it is intended for messages that must be seen, for example, to manually catch an error.
The "debug" level now has a low priority and is not so visually highlighted: it is intended for unimportant messages that are usually not needed, but which would be useful to have in the system if something goes wrong.
In case someone wants to inherit from the "Signale" class, the "scope" method now correctly creates an instance of the current class with which it was created.
Added the ability to set your own log levels, without having to inherit to override the "_logLevels" getter.

Fixes #96

Ok, if someone wanted this PR merged right now for production use I made a fork and published npm package signales with #105 PR merged into it.

I find that move kind of harsh. I agree with you on the content. It seems that this repo is going on the dark path of inactivity, and this is really frustrating since there is some works pending.
But I don't think this is the good solution. If everybody starts to fork and release their own version of the application, it will be a mess. And this is an heavy anti-pattern of Open Source. We need an issue to discuss about the inactivity and the right solutions to deal with it.
On an other point, your PR is not mergeable :

You didn't write any test. How are we supposed to say if it's correct or not ? Try to think of other people that will deal with your code in the future.
I already did a PR about level reordering, if you want to add alert level, this is another pr. If you have some doubts about my work, let's talk about it on the existing PR, I'm totally open to critics. I find your action disrespectful against my work and the community.

I'm sad to see this selfishness in the open source world.

Added two fixes to PR:

logLevel now is debug (so, no one debug output will be lost by default)
GitHub checks are happy now: lock "xo" version to "0.24.0" in order to have reproducible checks


Should 'start' CPO require that argument is a non-const lvalue?

Extracting the remaining issue from #100.
The start() CPO specification (as of P2300R10) says that it is ill-formed to pass an rvalue and otherwise start(op) is equivalent to MANDATE-NOTHROW(op.start()).
However, this permits op to be a const-lvalue if the operation-state object has a const-qualified .start() member-function.
Do we want to allow this?
Or do we want to instead specify that passing anything other than an unqualified lvalue to start() is ill-formed?

what would be the reason to require (or mandate) that .start() cannot be called on a const operation state?

The only reason we can see for restricting calling start() on a const-lvalue would be to reserve the right to assign different semantics for this in future by preventing existing code from using this pattern.
However, there is no clear use-case for supporting const-qualified operation-states that I can imagine. And even if we did come up with some use-case/new-semantic we want to describe here, we can always just add a new method-name instead as a more refined concept.
Closing as "not worth spending more time on at this stage".


Fix: Index canister candid parsing issue

Motivation
There is a bug in the candid definition of the Index canister.
Changes

Change the type of oldest_tx_id from GetTransactions

Tests
Not applicable

@peterpeterparker @mstrasinskis please review
There was an issue with the previous candid definition. https://dfinity.slack.com/archives/C03H6QEPW5D/p1666684478231609
The fix is already in master and I created an issue to update it as soon as we can have the new index canister deployed with the SNSs.
This fix unblocks me and allows me to keep working.


Add new "next location" methods

Currently, players are shown three random locations on a map and asked to navigate to those locations. New methods would allow different experiences. These new methods should include

[x] Map (done)
[ ] Clues / description

The system gives the player a clue or a description of the location, which they must then find

[ ] Free roam

Players are free to scan what they like, in any order. Map is optional

This is largely completed in #10 However, I need to extend the Gameplay Service to actually serve these modes


Added parameters to the ConsoleLogger to change the Passed,Failed, an…

Added parameters to the ConsoleLogger to change the Passed,Failed, and Skipped text
Description
Added ability to override the default text the ConsoleLogger uses for Passed, Failed, Skipped indicator text
passedTestIndicator
failedTestIndicator
skippedTestIndicator

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


Pytorch example internal error

Description
What steps will reproduce the problem?

In the Coursera course on machine learning lab (https://www.coursera.org/learn/machine-learning-duke/ungradedLab/IP4b8/logistic-regression/lab) the following code is given.  When I run it in Spyder I get an internal error:
import numpy as np
#import matplotlib.pyplot as plt
import torch
import torch.nn.functional as F
#from tqdm.notebook import tqdm
from torchvision import datasets, transforms
Load the data
mnist_train = datasets.MNIST(root="./datasets", train=True, transform=transforms.ToTensor(), download=True)
mnist_test = datasets.MNIST(root="./datasets", train=False, transform=transforms.ToTensor(), download=True)
train_loader = torch.utils.data.DataLoader(mnist_train, batch_size=100, shuffle=True)
test_loader = torch.utils.data.DataLoader(mnist_test, batch_size=100, shuffle=False)
Training
Initialize parameters
W = torch.randn(784, 10)/np.sqrt(784)
W.requires_grad_()
b = torch.zeros(10, requires_grad=True)
Optimizer (lr is the learning rate, or step size)
optimizer = torch.optim.SGD([W,b], lr=0.1)
Iterate through the training set minibatches (one epoch)
for images, labels in train_loader:
# Zero out the gradient
optimizer.zero_grad()
# Forward pass
x = images.view(-1, 28*28)
y = torch.matmul(x, W) + b
cross_entropy = F.cross_entropy(y, labels)
# Backward pass
cross_entropy.backward()
optimizer.step()
Testing
correct = 0
total = len(mnist_test)
with torch.no_grad():
# Iterate through the test set minibatches
for images, labels in test_loader:
# Forward pass
x = images.view(-1, 28*28)
y = torch.matmul(x, W) + b
    predictions = torch.argmax(y, dim=1)
    correct += torch.sum((predictions == labels).float())

print('Test accuracy: {}'.format(correct/total))
Traceback
Exception in comms call get_namespace_view:
  File "C:\Users\615237\AppData\Local\Programs\Spyder\pkgs\spyder_kernels\comms\commbase.py", line 315, in _comm_message
    encoding='latin-1')
ModuleNotFoundError: No module named 'torch'

Versions

Spyder version: 5.0.1
Python version: 3.7.9
Qt version: 5.12.10
PyQt5 version: 5.12.3
Operating System: Windows 10

Dependencies

# Mandatory:
atomicwrites >=1.2.0          :  1.4.0 (OK)
chardet >=2.0.0               :  4.0.0 (OK)
cloudpickle >=0.5.0           :  1.6.0 (OK)
cookiecutter >=1.6.0          :  1.7.2 (OK)
diff_match_patch >=20181111   :  20200713 (OK)
intervaltree                  :  None (OK)
IPython >=7.6.0               :  7.22.0 (OK)
jedi =0.17.2                  :  0.17.2 (OK)
jsonschema >=3.2.0            :  3.2.0 (OK)
keyring >=17.0.0              :  23.0.1 (OK)
nbconvert >=4.0               :  6.0.7 (OK)
numpydoc >=0.6.0              :  1.1.0 (OK)
paramiko >=2.4.0              :  2.7.2 (OK)
parso =0.7.0                  :  0.7.0 (OK)
pexpect >=4.4.0               :  4.8.0 (OK)
pickleshare >=0.4             :  0.7.5 (OK)
psutil >=5.3                  :  5.8.0 (OK)
pygments >=2.0                :  2.8.1 (OK)
pylint >=1.0                  :  2.7.4 (OK)
pyls >=0.36.2;<1.0.0          :  0.36.2 (OK)
pyls_black >=0.4.6            :  0.4.6 (OK)
pyls_spyder >=0.3.2           :  0.3.2 (OK)
qdarkstyle =3.0.2             :  3.0.2 (OK)
qstylizer >=0.1.10            :  0.1.10 (OK)
qtawesome >=1.0.2             :  1.0.2 (OK)
qtconsole >=5.0.3             :  5.0.3 (OK)
qtpy >=1.5.0                  :  1.9.0 (OK)
rtree >=0.8.3                 :  0.9.4 (OK)
setuptools >=39.0.0           :  56.0.0 (OK)
sphinx >=0.6.6                :  3.5.4 (OK)
spyder_kernels >=2.0.1;<2.1.0 :  2.0.1 (OK)
textdistance >=4.2.0          :  4.2.1 (OK)
three_merge >=0.1.1           :  0.1.1 (OK)
watchdog                      :  1.0.2 (OK)
zmq >=17                      :  22.0.3 (OK)

# Optional:
cython >=0.21                 :  0.29.23 (OK)
matplotlib >=2.0.0            :  None (NOK)
numpy >=1.7                   :  1.19.3 (OK)
pandas >=1.1.1                :  1.2.4 (OK)
scipy >=0.17.0                :  1.6.2 (OK)
sympy >=0.7.3                 :  1.8 (OK)

Hey @tbress, thanks for reporting. If you want to use to other packages that don't come with our Windows installer (like Pytorch), you need to install Miniconda, create a conda environment after that with the packages you want to use and spyder-kernels, and finally connect Spyder to that environment.
Please try that and let us know if it works for you.

Thanks, Carlos.  I'm trying to follow your advice using this website: Working
with packages and environments in Spyder · spyder-ide/spyder Wiki
(github.com)
https://github.com/spyder-ide/spyder/wiki/Working-with-packages-and-environments-in-Spyder#the-modular-approach.
I installed miniconda and have a base environment activated.  I installed
spyder-kernels in that environment.  When I list the packages in the
environment I see that I have spyder-kernels v1.10.2.  When I point Spyder
at the miniconda directory I get an error message saying "Your Python
environment or installation doesn't have the spyder‑kernels module or the
right version of it installed (>= 2.0.1 and < 2.1.0). Without this module
is not possible for Spyder to create a console for you."
I then tried updating spyder-kernels using "conda install -c conda-forge
spyder-kernels" in my base environment, but when I list the packages I
still see the v1.10.2 and I still can't open a kernel in Spyder.  I'm new
to setting up environments, is there a step I'm missing? Thanks.
On Sun, May 2, 2021 at 7:12 PM Carlos Cordoba @.***>
wrote:

Hey @tbress https://github.com/tbress, thanks for reporting. If you
want to use to other packages that don't come with our Windows installer
(like Pytorch), you need to install Miniconda
https://docs.conda.io/en/latest/miniconda.html, create a conda
environment
https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#creating-an-environment-with-commands
after that with the packages you want to use and spyder-kernels, and
finally connect Spyder
http://docs.spyder-ide.org/current/faq.html#using-existing-environment
to that environment.
Please try that and let us know if it works for you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/15486#issuecomment-830923498,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATT2ZUNCWZZTAA5L3654HW3TLXL6LANCNFSM437ZJGKA
.

I'm trying to follow your advice using this website

Great! That page has a more detailed explanation than the one I provided above.

I'm new to setting up environments, is there a step I'm missing? Thanks

Ok, please then run
conda install -c conda-forge spyder-kernels=2.0.1

to force the installation of that version instead of 1.10.2

Thanks, that works.  I appreciate your help, I've used Spyder for
engineering and science work in the past and now I going to take a data
science class and want to use Spyder as my IDE.  I now have miniconda
installed, and pytorch installed in the miniconda environment as well as
the correct version of spyder-kernels.  I can start Spyder and connect to a
kernel.  But when I try running the code I sent you before I still have an
issue.   The code completes successfully and gives me the right answer, but
I still get the Spyder internal problem pop-up window.  When I show the
details of the internal error pop-up I get:
Exception in comms call get_namespace_view:
File
"C:\Users\615237\AppData\Local\Programs\Spyder\pkgs\spyder_kernels\comms\commbase.py",
line 315, in _comm_message
encoding='latin-1')

ModuleNotFoundError: No module named 'torch'
On Mon, May 3, 2021 at 12:17 PM Carlos Cordoba @.***>
wrote:

I'm trying to follow your advice using this website
Great! That page has a more detailed explanation than the one I provided
above.
I'm new to setting up environments, is there a step I'm missing? Thanks
Ok, please then run
conda install -c conda-forge spyder-kernels=2.0.1
to force the installation of that version instead of 1.10.2
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/15486#issuecomment-831370915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATT2ZUI4VCTMOCLIVIWGTQLTL3EDLANCNFSM437ZJGKA
.

I think you're missing the last step I mentioned above:

and finally connect Spyder to that environment.

Please the instructions in the link to connect your Miniconda environment to Spyder.

Thanks, Carlos.  I took your advice again and came across something
strange.  First I rebooted my computer, then started Spyder.  I had
previously set my Python preference for the miniconda environment, so it
started in that environment.  I went through the steps of reselecting the
miniconda Python environment again, there was no issue with that.  But then
I tried to restart the kernel as you suggested in your link.  I got the
following error:
Traceback (most recent call last):
File "C:\Users\615237\Miniconda3\lib\runpy.py", line 197, in
_run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\615237\Miniconda3\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File
"C:\Users\615237\Miniconda3\lib\site‑packages\spyder_kernels\console_main.py",
line 23, in
start.main()
File
"C:\Users\615237\Miniconda3\lib\site‑packages\spyder_kernels\console\start.py",
line 284, in main
kernel.initialize()
File
"C:\Users\615237\Miniconda3\lib\site‑packages\traitlets\config\application.py",
line 87, in inner
return method(app, *args, **kwargs)
File "C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py",
line 567, in initialize
self.init_sockets()
File "C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py",
line 271, in init_sockets
self.shell_port = self._bind_socket(self.shell_socket, self.shell_port)
File "C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py",
line 218, in _bind_socket
return self._try_bind_socket(s, port)
File "C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py",
line 194, in _try_bind_socket
s.bind("tcp://%s:%i" % (self.ip, port))
File "C:\Users\615237\Miniconda3\lib\site‑packages\zmq\sugar\socket.py",
line 173, in bind
super().bind(addr)
File "zmq/backend/cython/socket.pyx", line 542, in
zmq.backend.cython.socket.Socket.bind
File "zmq/backend/cython/checkrc.pxd", line 28, in
zmq.backend.cython.checkrc._check_rc
zmq.error.ZMQError: Address in use
On Mon, May 3, 2021 at 8:09 PM Carlos Cordoba @.***>
wrote:

I think you're missing the last step I mentioned above:
and finally connect Spyder
http://docs.spyder-ide.org/current/faq.html#using-existing-environment
to that environment.
Please follow the instructions in the link to connect your Miniconda
environment to Spyder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/15486#issuecomment-831610484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATT2ZUNCJCIBPWBJCI3K2YTTL43LBANCNFSM437ZJGKA
.

Hello, Carlos.  I believe I've solved the problem.  Originally I had
installed Spyder before having installed miniconda.  I then installed
miniconda later and installed the other packages.  I uninstalled Spyder and
reinstalled using conda and everything works now.  Thanks again for your
help.
On Tue, May 4, 2021 at 12:02 PM Thomas Bress @.***> wrote:

Thanks, Carlos.  I took your advice again and came across something
strange.  First I rebooted my computer, then started Spyder.  I had
previously set my Python preference for the miniconda environment, so it
started in that environment.  I went through the steps of reselecting the
miniconda Python environment again, there was no issue with that.  But then
I tried to restart the kernel as you suggested in your link.  I got the
following error:
Traceback (most recent call last):
File "C:\Users\615237\Miniconda3\lib\runpy.py", line 197, in
_run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\615237\Miniconda3\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File
"C:\Users\615237\Miniconda3\lib\site‑packages\spyder_kernels\console_main.py",
line 23, in
start.main()
File
"C:\Users\615237\Miniconda3\lib\site‑packages\spyder_kernels\console\start.py",
line 284, in main
kernel.initialize()
File
"C:\Users\615237\Miniconda3\lib\site‑packages\traitlets\config\application.py",
line 87, in inner
return method(app, *args, **kwargs)
File
"C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py", line
567, in initialize
self.init_sockets()
File
"C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py", line
271, in init_sockets
self.shell_port = self._bind_socket(self.shell_socket, self.shell_port)
File
"C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py", line
218, in _bind_socket
return self._try_bind_socket(s, port)
File
"C:\Users\615237\Miniconda3\lib\site‑packages\ipykernel\kernelapp.py", line
194, in _try_bind_socket
s.bind("tcp://%s:%i" % (self.ip, port))
File "C:\Users\615237\Miniconda3\lib\site‑packages\zmq\sugar\socket.py",
line 173, in bind
super().bind(addr)
File "zmq/backend/cython/socket.pyx", line 542, in
zmq.backend.cython.socket.Socket.bind
File "zmq/backend/cython/checkrc.pxd", line 28, in
zmq.backend.cython.checkrc._check_rc
zmq.error.ZMQError: Address in use
On Mon, May 3, 2021 at 8:09 PM Carlos Cordoba @.***>
wrote:

I think you're missing the last step I mentioned above:
and finally connect Spyder
http://docs.spyder-ide.org/current/faq.html#using-existing-environment
to that environment.
Please follow the instructions in the link to connect your Miniconda
environment to Spyder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/15486#issuecomment-831610484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATT2ZUNCJCIBPWBJCI3K2YTTL43LBANCNFSM437ZJGKA
.

Great!
And just so you know, we're planning to improve installing environments and packages for the future.


Text input, e.g. for web search

Hi there, love BitBar. I've just started using the href option, but what's missing (for me at least), is a combination of href and a text input field, so you can search on certain user-defined websites or webpages, e.g. add a "search-item" in the BitBar menu—possible format would be | search=URL—, click on it, a window pops up, or (better) a text input field opens within the BitBar menu, and you can enter a term like "bit bar", and it will be added as "bit+bar" to a search URL that the user has predefined for that menu item (see example below), and the full URL is then sent to the default browser. I guess you could concoct a workaround, e.g. call a shell script instead that runs an osascript which opens a window where you can input your search term, but that kinda defeats the purpose of BitBar. ;)
Another possibility would also be to listen for click&ALT… if you click on a menu item, then it's just a standard href, if you click on it and press ALT, you get the search function.
Full format e.g. (for click and click&ALT, with in this case @@@ as string for the search term):
$ echo "GitHub | href=https://github.com search=https://github.com/search?utf8=✓&q=@@@ color=whatever"
PS: If possible, an additional feature could be that if you press ALT the text changes, in this example from "GitHub" to "Search GitHub"… don't know how you could implement it, though.

Would #219 work for you?

Updated the above post: CMD > ALT, and added @@@ for the search term string.

its possible to do this now with an Applescript call in the same plug-in script based on a param and passing the text result entered to a search, like Google, etc. Example:
#!/bin/sh

function getSearchText ()
{

searchString=$(/usr/bin/osascript << EOF
set theString to text returned of (display dialog "What are you searching for?" default answer "" buttons {"Search","Cancel"} default button 1)
EOF)

if [ ! -z "$searchString" ]; then
	textToSearch=$(echo "$searchString" | perl -MURI::Escape -wlne 'print uri_escape $_')
else
	exit
fi

}

if [ "$1" == "search" ]; then
	getSearchText
	open https://www.google.com/#q=$textToSearch
fi

echo "Web Search
---
Go to Google| href=https://www.google.com
Search Google| alternate=true terminal=false bash=$0 param1=search"

I know it would be better somehow built in, but the above is pretty self contained since its not trying to call an external script to do the text input.

@mm2270 Nice! I could really use this for my downloads plugin.

Is there any chance that the above script could be translated to python?
Thank you!

You can run AppleScript within a python script, so that shouldn't really be a problem:
https://stackoverflow.com/questions/2940916/how-do-i-embed-an-applescript-in-in-a-python-script

Unfortunately, it is posing quite a problem. When run my script from the terminal with the right parameter, it works as expected and I get the text input dialog box:

However, when I run it from within the plugin, the dialog box is wrong:

The examples in the link didn't make any sense so I am using this form:
import subprocess

def run_script(script):
    return subprocess.Popen([script], stdout=subprocess.PIPE, shell=True).communicate()[0].strip()

cmd = "osascript -e 'set T to text returned of (display dialog \"Message to Send?\" buttons {\"Cancel\", \"OK\"} default button \"OK\" default answer \"\")'

print run_script(cmd)

I found out what was going on. Apparently, the bash handler/parser can't handle ANSI codes which was causing the problem. Instead of terminal=false, I was getting terminal is: false\^[[0m
class bcolors:
   GREEN = '\033[32m'
   ENDC = '\033[0m'

print "---- " + bcolors.GREEN + "<Custom Message> | image="+str(imsgicon).encode("utf-8") + " bash=%s param1=-t param2=%s terminal=false" % (sys.argv[0], d['Mobile']) + bcolors.ENDC

I had to add a + "  " + before the bcolors.ENDC before it would finally work.

On this same topic, and using the sample code above to get user input, I always get an error when a user types an apostrophe in the message being inputed. So Typing I'm always causes this error when I print the return value of run_script(cmd):
I'm on my way
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file

It looks like a shellcheck thing but I don't know how to resolve it in python. Is there a standard way to sanitize user input that is retrieved using a dialog box?

I was able to work around this issue by replacing the apostrophe ' with the angled apostrophe ’ at the end of the function like this:
def run_script(script):
    return (subprocess.Popen([script], stdout=subprocess.PIPE, shell=True).communicate()[0].strip()).replace("'", "’")


Merge gz-launch7 ➡ main

➡ Forward port
Port gz-rendering7 to main
Branch comparison: https://github.com/gazebosim/gz-launch/compare/main...gz-launch7
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

This looks like it's targeting gz-launch7 instead of main.

This looks like it's targeting gz-launch7 instead of main.

Oops. Just corrected it

@Crola1702  Were you intending to merge the complete branch? In that case, I won't feel confident enough to give you the green checkmark.

@azeey I see CI is passing (windows build shows just a warning). Should we merge it?


Need instructions on how to run on Firefox OS

Hi @janthiemen, I've come across your app in Firefox Marketplace. I would like to contribute some styling and UX tweaks, but not sure how to run the code in github on Firefox OS. Could you help? Thanks.

Hello,
This is the WebOS version of NISI, I forgot to upload the Firefoxos version, sorry for that. You can find it here: https://github.com/janthiemen/nisi_fxos. It can be installed just like every other FirefoxOS app. Again my apologies for the confusion.
Jan Thiemen

Thank you! I love the fxos version.
Jan Thiemen<EMAIL_ADDRESS>於 2015年7月13日 週一 下午11:44寫道：

Hello,
This is the WebOS version of NISI, I forgot to upload the Firefoxos
version, sorry for that. You can find it here:
https://github.com/janthiemen/nisi_fxos. It can be installed just like
every other FirefoxOS app. Again my apologies for the confusion.
Jan Thiemen
—
Reply to this email directly or view it on GitHub
https://github.com/janthiemen/nisi/issues/1#issuecomment-120973452.


[docs] Add import modules to the sorter and thickness calculator docs

In map2loop-3-notebooks examples 4 and 5 the child classes of Sorter and ThicknessCalculator require extra modules, add this to the docs

Formatting issue with import lines. Will fix

Merged with fixing-all-the-ci-problems branch


Why is the signing output two lines? Do I need to merge them?

 When I'm putting them into my appcast.xml file - do I need to separate them somehow or just merge them together? The example here also shows two lines, but not how/if to merge them. If they need to be merged: what reason is there for putting them in separate lines?

Thanks in advance!

Yes please merge it on one line


After 1.19 is released, create a redirect from /docs/reference/scheduling/profiles to /docs/reference/scheduling/config

In the 1.19 release, KubeSchedulerConfiguration graduated to beta. The /docs/reference/scheduling/config page was created for the Scheduler Configuration which is a rework of /docs/reference/scheduling/profiles. Create a redirect from /docs/reference/scheduling/profiles to /docs/reference/scheduling/config
/cc @celestehorgan
/language en

Looks like this redirect is in place.
An attempt to go to https://kubernetes.io/docs/reference/scheduling/profiles redirects to https://kubernetes.io/docs/reference/scheduling/config/#profiles
@celestehorgan is this sufficient to close this issue?

Yup!
/close


how do i get previous number in types?

here is a pattern that gets the next number in types domain
type Nums = [1, 2, 3, 4, 5, 6, 7, 8, 9];
type Num = Nums[number];
type Next<N extends Num> = Nums[N] extends number ? Nums[N] : never;
type N2 = Next<1>; // 2

is there a way to get a previous number?

Not sure why you'd want this, but isn't it obvious?
type Nums = [-1, 0, 1, 2, 3, 4, 5, 6, 7];
type Num = Nums[number];
type Prev<N extends Num> = Nums[N] extends number ? Nums[N] : never;
type N2 = Prev<1>; // 0

can you use the original sequence?

Shouldn't this be on Stack Overflow?
type Prev<N extends Num> = { [K in Num]: N extends Next<K> ? K : never }[Num];
// Prev<1> is never, just like Next<9> is never. 🤷♂


Change from hooks to config.xml

Hi there, hooks directory is deprecated in the latest version of Cordova, according to its documentation. Now the hook is defined in config.xml.
You might want to take that into account, that is, add your uglify JS script to config.xml.

It looks like this issue and #52 are related -- the newest cordova-lib doesn't appear to be exporting the process.env.<whatever> variables, even if a <hook> node is added to the config.xml file. The whole uglify-config needs to be redone with the module.exports stuff.

Thank you both for the info. I am no longer maintaining this project and have not been doing development using cordova for awhile now. PRs are welcome.

Will do -- this is a current pain point. :-) I've got a reworked uglify.js working on iOS locally, but Android is throwing fits ATM. If/when I can get something working, I'll send it back your way. Thanks!

This should be resolved in the latest version published to npm (0.3.5).


Verification when delete of Queue

Hi!
It is a very good tool, but i have a problem that it is very easy to delete a queue with no verification question. Is it possible for me to disable the delete options on queue level? or is it possible to get an verification, do you really want to delete queue as it is when you delete messages?
#feature
/Jennie

Verification (confirmation) is already in place

Oh, in wich version did this come?
I run the 4.1.112, is this in an later version?

Been there a while...

@jemob, unfortunately older versions of SBE always report their version as the latest released version since they get the version number using a REST call to Github. If your SBE states its version in the title of the main window it is an old version. The title should only say Service Bus Explorer, like the figure below:

However I think the default button should be No and you should have to type the name of the namespace when deleting all queues.

@Jemob, unfortunately older versions of SBE always report their version as the latest released version since they get the version number using a REST call to Github. If your SBE states its version in the title of the main window it is an old version. The title should only say Service Bus Explorer, like the figure below:

Thank you so much for the reply. I will try to download a new version then. :)
It is like you wrote an version number in the title.


Resolve name clash on npm package name std

We have "name": "std" in skip/skiplang/prelude/ts/package.json while there is already a package named npm https://www.npmjs.com/package/std.

Yes, this is a temporary situation, and we will not publish that package.

Closing this, as the package is now published under @skip-wasm/std.


Add support for async signature computation

In WebCrypto API, all the operations, including signature computation, are async and return promises. Trying to implement RSA-SHA1 signature method, I found out that it was impossible to do because the library always uses getSignature() in a synchronous manner.
The idea is to add an async method that is able to work with async getSignature() (and getBodyHash()) methods.
Please note that I did not add anything to the TS file. Could you please give me a hint: what changes are to be added there?

pls check the ci test fail

@ddo done, the tests have passed

Bump?


Question: How about the constraints and index supportability in babelfish?

Hi @staticlibs ,
How about the constraints and index supportability in babelfish like Forign key, CLUSTERED INDEX, etc

@psrinivasa2 , constraints, foreign keys and indices are supported in Babelfish. There is no specific handling for clustered indices.

What about primary key? how about the index performance if a column is clustered in SQL and not in babelfish?

@psrinivasa2 , primary keys are supported. About the performance effect of indices that were created as CLUSTERED in MSSQL - this needs to be looked at on particular data.

how about non clustered index Alex?

@psrinivasa2 , all indices created in Babelfish can be seen/understood as non-clustered indices in MSSQL.

To understand it clearly, when I execute below statement in babelfish, will it a non-clustered index created for the column?
CREATE TYPE [dbo].[Ids] AS TABLE(
[Id] [int] NOT NULL,
PRIMARY KEY CLUSTERED
(
[Id] ASC
)WITH (IGNORE_DUP_KEY = OFF)
)

@psrinivasa2 , in Postgres there is no direct corresponding feature like CLUSTERED index in MSSQL (there is a CLUSTER command, but it is not the same thing). So you can think about all indices created in Babelfish like they are non-clustered ones.


Save File mode as instance variable

In some cases you may want to know what mode a file was opened in, but currently that doesn't seem to be possible since the mode is just passed to the FileDescriptor and forgotten about. It would be nice if we could get a #mode getter with a corresponding instance variable.

This is possible without storing them in an instance variable using fcntl(F_GETFL)

Good idea. It would just be nice to have it. A to_unsafe or similar method that returns a C compatible FILE would be nice too. I do a lot of C lib wrapping and this isn't the first time I've had to implement it myself.

See #3151 for FILE*.

Yeah I saw that issue. Not sure why we don't want it in LibC at least though. It would be nice to have LibC be as complete as possible, no?

No, we'll probably move LibC to Crystal::LibC eventually to make it clear it's not public. It's there for the stdlib only really, i think it's a better pattern to bind what your shard uses from libc in your own shard.


Sending message on callback

@GreatAndPowerfulKing I thought that it would be better to represent a message in chat only in case we can send it to server. Otherwise showing user a specific error.

@GYFK I suppose it's terrible solution to leave message in text field while sending. It provides unexpected behavior for user. Also it has no sense without blocking message sending ui.

@GreatAndPowerfulKing more unexpected behavior is provided when you display a message in chat, whereas it never reaches the interlocutor. Yet if you are to give a better solution with resending messages and properly displaying those that for some reason haven't reached the other end, we should close this PR in it's favor.


Check for parameters shadowing class members

Fixes #63092
By the way, should shadowing be disallowed for members in their setters?

Thanks!

By the way, should shadowing be disallowed for members in their setters?

Yeah, I guess if you are shadowing the variable that you are writing the setter for, you are essentially making the setter do nothing, so we can forbid this specific case to avoid this problem.


apache urlrewrite not working

Hi,
First of all, thank you very much for this docker image. It gets me on codeigniter4 on macbook in only a few hours.
I have 1 suggestion for improvement. Currently the mod rewrite is not enable the .htaccess does not work (at least in my installation). I needed to ssh into the docker and run "a2enmod rewrite" and restart the apache to get the rewrite working.
Maybe can add a line in Dockerfile:
RUN a2enmod rewrite
Just in case you are interested, I have also map apache.conf and startScript.sh to local directory.
Again, thank you for this project. Great job!

Hi.
Thanks a lot for your report.
As required, I added the "RUN a2enmod rewrite" line in Dockerfile
Enjoy


Renaming all azure-spring-boot-xxx to spring-cloud-azure-xxx

Parent issue: https://github.com/Azure/azure-sdk-for-java/issues/23402

Should the new artifact-id be like azure-spring-cloud-xxx or spring-cloud-azure-xxx? Given this issue https://github.com/Azure/azure-sdk-for-java/issues/23931 seems it's spring-cloud-azure-xxx

@yiliuTo Updated the title.

Closing this one, because it's duplicated with this one: https://github.com/azure/azure-sdk-for-java/issues/23931


Enabling static type hint checks

Ticket:
https://dluhcdigital.atlassian.net/browse/SMD-815

[ ] Unit tests and other appropriate tests added or updated
[ ] README and other documentation has been updated / added (if needed)
[ ] Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

How to test
If manual testing is needed, give suggested testing steps
Screenshots of UI changes (if applicable)

Duplicate PR, closed.


Update/geos 11.5.2

Update the dsl/develop branch to the geos v11.5.2 state, a.k.a v2.5.2 for this comp.
WARNING: This will make the OpenACC port out-of-sync. Last sync'ed version is document in G002.1 of the SMT24-26 plan.

@mathomp4 I don't know anything about dsl/develop. Maybe you can fill me in next time we chat ...

@mathomp4 I don't know anything about dsl/develop. Maybe you can fill me in next time we chat ...

Allow me to reply, @sanAkel. dsl/develop is/will be the working branch for the Python-based DSL port of the physics parametrization my group will be attacking later this year. I'll be giving a talk to the GMAO at large sometime in April to go more in detail.
Functionally, the Fortran code will be lightly touched, with a zero-diff, and most of the work will be moved to a new set of files stored in a python sub-directory.

Thanks @FlorianDeconinck.

Allow me to reply, @sanAkel. dsl/develop is/will be the working branch for the Python-based DSL port of the physics parametrization my group will be attacking later this year. I'll be giving a talk to the GMAO at large sometime in April to go more in detail. Functionally, the Fortran code will be lightly touched, with a zero-diff, and most of the work will be moved to a new set of files stored in a python sub-directory.

Looking forward to the GMAO at large sometime in April meanwhile, would you be available/open to a private conversation about it all sometime next week? If yes, would you please email me a few convenient times. 🙏


Temporarily disable performance test that is 'failing' by running too…

… fast.
I am reworking these to be based on the scope counter rather than a
timer.
rdar://problem/42744631

@swift-ci Please smoke test and merge

@eeckstein This should clear up the failure for the moment.


Jumpy zooming with touchpad on Chrome

Zooming is jumpy with CTRL+MOUSE_WHEEL and ALT+MOUSE_WHEEL when using touchpad on Chrome.
I'll look into this and find out what's the current implementation and try to fix this.

Pull request: https://github.com/jgraph/mxgraph/pull/95
Fix must be also copied to draw.io from Graph Editor after approval.


Bugfix: Possibility to get non published posts

Thank you again!
I have given you access to https://github.com/bootscore/bs-isotope if you want to have a look to the shortcodes there as well.
Cool, we'll take a look sometime next week, it's been a bit hectic with the holidays coming.

BTW, you did a lot of useful contributions, do you want to join the team? Then you have access to everything. Talked already to the other member and he is fine with that.
Let me know what you think.

Thanks, we'd love to, our schedule can get a bit hectic though.


LOAM Mapping Task 3 - Integrate featureExtraction from LIO-SAM

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
This task is the subtask of Point cloud feature extraction for LOAM based localization.

https://github.com/autowarefoundation/autoware.universe/issues/6741

For using them in the LOAM Based Localization, we need to export point cloud feature maps separately. Related issue:

https://github.com/autowarefoundation/autoware/issues/4592

To implement feature extraction from a projected point cloud, the below task needs to be done:

https://github.com/autowarefoundation/autoware.universe/issues/6837

Purpose
Extracting the features (planar, edge...) from the generated point clouds.
Possible approaches

Use instantaneous point clouds for detecting the features.
Use the full point cloud map for detecting the features.

Definition of done

[ ] Undistort point clouds.
[x] Visualize the extracting process of the features in RViz as point clouds.
[x] Export the point cloud feature maps to pcd files.

The tool can export corner feature point cloud map and surface feature point cloud map. Before exporting, it is visualizing both in the rviz. Right now, point cloud undistortion is not implemented. We wanted to see we can see the feature point clouds.
RViz Visualization:
Corner Cloud

Surface Cloud

Exported .pcd:
Corner Cloud

Surface Cloud

Thanks for your effort @ataparlar . Could you share .pcd files too to examine

I noticed that the projected image is not correct for some reason. The edge points are not extracted from the corners. They seems like just picking some random points from the point cloud.
I debugged the projected point cloud's point angles with HSV color space for loam_mapper, original LIO-SAM and Velodyne VLP16 ROS2 driver. The videos of them can found below:
Velodyne VLP16 ROS2 driver video:
https://youtu.be/NdwLPB2fWHc
Original LIO-SAM video:
https://youtu.be/sQA-Ilz-8BY
loam_mapper video:
https://youtu.be/tULkP0HXCQs
According to the comparison between those videos, in loam_mapper, the pixels that must be empty are very noisy. I am trying to solve that right now.

The image_projection seems good right now. But the same error still exist. I am observing the cloudInfo message which is crucial for feature extraction. It keeps the point cloumn info, point range info, point start ring index and point end ring index for each point for each cloud. This thing is most probably filled wrong in the program. I am working on it. Here is the video of the last situation:

https://youtu.be/9cJQO9Oyed4
The turquois points are the edge feature points extracted. They are so wrong obviously. I will keep here posted.

Hey everyone. I solved the problem and I added point deskewing for feature extraction. Here is the demonstration video and an image of it:

https://youtu.be/MbFCKkah0y4

I made the changes in imu_deskew branch and merged this branch into the master after it is done.
This subtask is done with the commit:

https://github.com/ataparlar/loam-mapper/tree/ec3697dcf67b0d4db0c21e648646685cf88df0f9


Paren highlighting is broken next to strings

The following cases highlight correctly:
f( ( 1 ) , 1 )
f( ( 1 ) , 1)
f( ( 1 ) , " " )

But things break if you remove the final space from the third case. When the cursor is to the left of the final paren, there's no paren highlighting:

And when it's to the right, the wrong paren is highlighted:

cc @bigdogmat

This is due to the fact GetTokenAtPosition returns the token to the left side of the cursor. I didn't even think to test this due to the fact self.Rows[pos[1]]:sub(pos[2], pos[2]) returns the character right of the cursor.
Just need to add an offset.


i03 testing: Round 5

Particularly need to test:

https://github.com/DiamondLightSource/python-artemis/issues/59
Other fixes in https://github.com/DiamondLightSource/python-artemis/issues/58
New ispyb details for https://github.com/DiamondLightSource/python-artemis/issues/84
Look into https://github.com/DiamondLightSource/python-artemis/issues/61
Initial testing of #30

Documentation of testing:

We should change the default IP in GDA to be localhost
GDA refactor was missing the scattergaurd
New ispyb user doesn't have permission to retrieve the visit ID
https://github.com/DiamondLightSource/python-artemis/issues/61 is due to:

inter function in det_distance_converter
GDA sending snapshots as a list

Need to do module load zocalo - put in a startup script
Setting the z step size and number is required
GDA needs to more gracefully handle when no json results file exists - currently locks out the user
GDA needs to understand that we have written 2 nexus files (with incremented run numbers) but only one data file - there is also an error around this in artemis
Hitting the 1000 image limit in nexgen - need to take this into account as when files are written out the eiger they have a 1000 image cap


Llama CPU Guide

Situation
@scottyeager and I did a demo session on how to deploy Llama chatbot on the grid with GUI.
Todo

Write a guide on how to deploy such chatbot

Fixed when #578 is merged.


Extend the annotations picked up in documentation generation

Can we add more annotation properties to be picked up when generating a module's documentation? For instance I would like to also store the CQs solved by the module in an annotation of the module itself.

The easiest will probably be to fork the pyLODE repo and modify the following file(s):

rdf_elements.py: Add corresponding property to the ONT_PROPS
...
Then update CEON to use this fork instead.

I'll check this out in more detail and get back with some more details.


Going overboard on all caps

Should tweak file about instruction writing a bit.

Appears to have been an isolated incident. Still doesn't take that methodical of an approach to what's important, but defer this polishing to later.


Migrate content from cf-infra-docs to conda-forge.org (maybe iteratively)

cf-infra-docs: https://github.com/Quansight-Labs/cf-infra-docs
conda-forge's infrastructure: https://conda-forge.org/docs/maintainer/infrastructure/

Merge and mix

This is an overview of all the files to track handling.
docs

[x] getting-started.md (placeholder)
[ ] index.md
[x] miniforge.md (placeholder)

fundamentals

[ ] feedstocks.md
[x] index.mdx (technical)
[ ] life-cycle.md
[x] publication.md (placeholder)
[ ] staged-recipes.md

howto

[ ] index.md

reference

[ ] bot-commands.md
[ ] feedstock-settings.md
[ ] glossary.md
[x] index.mdx (technical)
[ ] pinnings.md
[ ] runners.md

infrastructure

[ ] _events-triggers.md
[ ] automated-maintenance.md
[ ] diagrams.md
[ ] feedstocks.md
[x] index.mdx (technical)
[ ] services.md
[ ] staged-recipes.md
[ ] tooling-data.md
[ ] website.md

advanced

[ ] automation.md
[x] index.mdx (technical)
[x] security.md (placeholder)

tutorials

[x] index.mdx (technical)
[x] submit-a-recipe.md (placeholder)

What are the checkboxes here for? 🤔

What are the checkboxes here for? 🤔

They get ticked when the file has been dealt with, meaning either determined to not need migration in its current form (e.g. purely technical or boilerplate contents), assigned to a different issue for tracking, or fully integrated into the migration branch at https://github.com/zklaus/conda-forge.github.io/tree/improve-infrastructure-documentation.

Ok, let's keep working on that branch until next week. Then we can start opening PRs, @goanpeca can help with the review(s) when time comes. To keep things scoped, let's open one PR per source (origin) page (e.g. "infrastructure docs revamp" discusses what happens with maintainer/infrastructure.md).

We should also add https://github.com/regro/conda-forge-feedstock-check-solvable to the documented infrastructure.

Infra docs have migrated. Next in line: Diataxification of the docs via https://github.com/conda-forge/conda-forge.github.io/issues/2164


HOTFIX: Increase kafkatest startup wait time on ConnectDistributed service

The fact that the wait time for the same service in standalone is 30 seconds and for distributed is 15 seconds indicates that 15 might be small wait time, that could result in occasional failures in tests using the distributed flavor of the service.
/cc @sriramsub @ewencp @ijuma

LGTM


(#390) - Simplify grammar, remove anonymous

Per #390

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 9min)


🌱 Add unit tests for remediation controller

Some unit tests for the reconcileNormal function on the remediation controller to try and describe what the expected behavior is.
What this PR does / why we need it: Ensures that the logic in the remediation controller is adequately covered by tests.

/cc @slintes

/test-v1b1-integration

/test-v1b1-integration

/cc @namnx228


Create public API for set canvas size and reset without margin.

🚀 Feature

It will be nice to have a public API to set the canvas size. Even in public documentation, we show an option to use napari from a notebook to make screenshots. But not always data ratio fit to canvas ratio. Also, screenshot resolution depends on the size of the canvas.
Because of this, it will be nice to have option to adjust canvas size to data. I think that API for controlling canvas size is better than API for set window size as the second one require knowledge about all docked widget size for a proper calculation.
During the implementation of this, it may be nice to allow adjust the scale factor for reset view
https://github.com/napari/napari/blob/9ea0159ad2b690556fe56ce480886d8f0b79ffae/napari/components/viewer_model.py#L311
That should allow adjusting margins for screenshots to reduce postprocessing steep.
Motivation

Here discussion with @alisterburt https://forum.image.sc/t/making-napari-canvas-match-data-extents/70720
Pitch
Add set_canvas_size to the viewer model that will resize the main window to get the proper canvas size.
Move 0.95 factor of reset_view to function argument to allow better adjust reset margin from python code.

Alternatives

Additional context

https://forum.image.sc/t/making-napari-canvas-match-data-extents/70720

This sounds great! Somewhat related to https://github.com/napari/napari/pull/4201
where the goal was enabling data only screenshots at full res, with no canvas background.
Being able to set the canvas size to the data and then screenshot would be awesome.

Would the original use case become obsolete with a rendering API suggested in https://github.com/napari/napari/issues/6334?
(I'm also interested in this.)
Otherwise, it can happen that set_canvas_size makes the canvas mismatch the Napari window. Should the gap be black (or overflow hidden)? Or should the window size be adjusted? (Which can cause undesired side effects, GUI elements off-screen, window jumping to front.)


Standardize format of streamer lists

The list of streamers to track is currently distributed between several files, and the format is inconsistent between repositories. A standard list of streamers should be created to allow each project to pick which streamers to track from a master list. The list should include English names, YouTube channel IDs, names, and handles, Twitch IDs and names, and an organization identifier. Files like birthdays.json could also be merged into this master list.

[ ] Import current streamer lists into a new repository
[ ] Update lists with all information mentioned above
[ ] #7

A master repository is currently in progress, though it will not be directly referenced once the change is made to keep streamer information in a database. The list will simply serve as a reference for what information is contained in the database. The benefit of storing channel names and handles is unclear given that they are subject to change and contain redundant information, so there is currently no plan to track these in the new repository.


VRP - Minimizing Makespan - the total duration vehicles are on the road

minimizing makespan minimizes the time the last vehicle will arrive to end node. Basically optimizing getting the full problem done as fast as possible.
Simple example highlighting what's getting minimized:

Currently: 9 vehicles on the road for 2 hours, 1 vehicle on the road for 8 hours (26 hours -> min)
Minimizing Makespan: 10 vehicles on the road for 2.6 hours (2.6 -> min) (26 hours)

@AlirezaH320 any progress on this issue?

For an example on how to use the GlobalSpan()
https://developers.google.com/optimization/routing/vrp#example

@Mizux
I dont think using the global span is the same as minimizing the longest route as it leaves vehicles unused, even for very high values. I have opened an issue with a minimal example about it here:
https://stackoverflow.com/questions/51207808/minimize-max-distance-among-vehicles
and here:
https://github.com/mapbox/node-or-tools/issues/12

For the record
from six.moves import xrange
import ortools.constraint_solver.pywrapcp
from ortools.constraint_solver import routing_enums_pb2

stops = 20
vehicles = 5
depot = 0

routing = ortools.constraint_solver.pywrapcp.RoutingModel(stops+1, vehicles, depot)

def cost_function(x,y):
    return 1

routing.SetArcCostEvaluatorOfAllVehicles(cost_function)

evaluator = cost_function
slack_max = 20
capacity = 20
fix_start_cumul_to_zero = True
name = 'time'
routing.AddDimension(evaluator, slack_max, capacity, fix_start_cumul_to_zero, name)

time_dimension = routing.GetDimensionOrDie(name)
time_dimension.SetGlobalSpanCostCoefficient(1000)

search_params = ortools.constraint_solver.pywrapcp.RoutingModel.DefaultSearchParameters()
search_params.first_solution_strategy = (routing_enums_pb2.FirstSolutionStrategy.LOCAL_CHEAPEST_INSERTION)
#search_params.first_solution_strategy = (routing_enums_pb2.FirstSolutionStrategy.PARALLEL_CHEAPEST_INSERTION)
#search_params.first_solution_strategy = (routing_enums_pb2.FirstSolutionStrategy.PATH_CHEAPEST_ARC)
assignment = routing.SolveWithParameters(search_params)

def print_solution(routing, assignment):
    """Prints assignment on console"""
    print('Objective: {}'.format(assignment.ObjectiveValue()))
    total_distance = 0
    for vehicle_id in xrange(vehicles):
        index = routing.Start(vehicle_id)
        plan_output = 'Route for vehicle {}:\n'.format(vehicle_id)
        distance = 0
        while not routing.IsEnd(index):
            plan_output += ' {} -> '.format(routing.IndexToNode(index))
            previous_index = index
            index = assignment.Value(routing.NextVar(index))
            distance += routing.GetArcCostForVehicle(previous_index, index, vehicle_id)
        plan_output += ' {}\n'.format(routing.IndexToNode(index))
        plan_output += 'Time of the route: {}h\n'.format(distance)
        print(plan_output)
        total_distance += distance
    print('Total Time of all routes: {}h'.format(total_distance))

print_solution(routing, assignment)

is that working code?

Same sample but using the v7.x API...
#!/usr/bin/env python3
from six.moves import xrange
from ortools.constraint_solver import pywrapcp
from ortools.constraint_solver import routing_enums_pb2

stops = 20
vehicles = 5
depot = 0

# Create the routing index manager.
manager = pywrapcp.RoutingIndexManager(stops+1, vehicles, depot)

# Create Routing Model.
routing = pywrapcp.RoutingModel(manager)

def distance_callback(from_index, to_index):
    del from_index, to_index
    return 1

transit_callback_index = routing.RegisterTransitCallback(distance_callback)
routing.SetArcCostEvaluatorOfAllVehicles(transit_callback_index)

slack_max = 20
capacity = 20
fix_start_cumul_to_zero = True
name = 'time'
routing.AddDimension(
        transit_callback_index,
        slack_max,
        capacity,
        fix_start_cumul_to_zero,
        name)

time_dimension = routing.GetDimensionOrDie(name)
time_dimension.SetGlobalSpanCostCoefficient(1000)

search_params = pywrapcp.DefaultRoutingSearchParameters()
#search_params.first_solution_strategy = (routing_enums_pb2.FirstSolutionStrategy.LOCAL_CHEAPEST_INSERTION)
search_params.first_solution_strategy = (routing_enums_pb2.FirstSolutionStrategy.PARALLEL_CHEAPEST_INSERTION)
#search_params.first_solution_strategy = (routing_enums_pb2.FirstSolutionStrategy.PATH_CHEAPEST_ARC)
assignment = routing.SolveWithParameters(search_params)

def print_solution(manager, routing, assignment):
    """Prints assignment on console"""
    print('Objective: {}'.format(assignment.ObjectiveValue()))
    total_distance = 0
    for vehicle_id in xrange(vehicles):
        index = routing.Start(vehicle_id)
        plan_output = 'Route for vehicle {}:\n'.format(vehicle_id)
        distance = 0
        while not routing.IsEnd(index):
            plan_output += ' {} -> '.format(manager.IndexToNode(index))
            previous_index = index
            index = assignment.Value(routing.NextVar(index))
            distance += routing.GetArcCostForVehicle(previous_index, index, vehicle_id)
        plan_output += ' {}\n'.format(manager.IndexToNode(index))
        plan_output += 'Time of the route: {}h\n'.format(distance)
        print(plan_output)
        total_distance += distance
    print('Total Time of all routes: {}h'.format(total_distance))

print_solution(manager, routing, assignment)

the issue is most First Strategy only consider the arc cost and not the global function cost.
if you replace in the previous sample
search_params.log_search = True
search_params.time_limit.seconds = 30
search_params.local_search_metaheuristic = (
        routing_enums_pb2.LocalSearchMetaheuristic.GUIDED_LOCAL_SEARCH)

assignment = routing.SolveWithParameters(search_params)

you'll get
./plop.py
...
# ~30s later
I0417 10:45:27.368408 84388 search.cc:251] Solution #4505 (7024, objective maximum = 20022, time = 22554 ms, branches = 67535, failures = 16355, depth = 33, Cross, neighbors = 4716071, filtered neighbors = 4505, accepted neighbors = 4505, memory used = 31.79 MB, limit = 75%)
I0417 10:45:27.368937 84388 search.cc:251] Solution #4506 (6025, objective maximum = 20022, time = 22554 ms, branches = 67543, failures = 16357, depth = 33, Cross, neighbors = 4716091, filtered neighbors = 4506, accepted neighbors = 4506, memory used = 31.79 MB, limit = 75%)
...
I0417 10:46:27.504631 84534 search.cc:251] Solution #6341 (6024, objective maximum = 20022, time = 29998 ms, branches = 90476, failures = 22733, depth = 33, Exchange, neighbors = 6571135, filtered neighbors = 6341, accepted neighbors = 6341, memory used = 31.79 MB, limit = 99%)
I0417 10:46:27.505337 84534 search.cc:251] Finished search tree (time = 29999 ms, branches = 90481, failures = 22762, neighbors = 6571238, filtered neighbors = 6341, accepted neigbors = 6341, memory used = 31.79 MB)
I0417 10:46:27.505391 84534 search.cc:251] End search (time = 29999 ms, branches = 90481, failures = 22762, memory used = 31.79 MB, speed = 3016 branches/s)
Objective: 6024
Route for vehicle 0:
 0 ->  0
Time of the route: 0h

Route for vehicle 1:
 0 ->  8 ->  16 ->  12 ->  4 ->  19 ->  0
Time of the route: 6h

Route for vehicle 2:
 0 ->  18 ->  11 ->  13 ->  20 ->  2 ->  0
Time of the route: 6h

Route for vehicle 3:
 0 ->  9 ->  17 ->  15 ->  14 ->  10 ->  0
Time of the route: 6h

Route for vehicle 4:
 0 ->  3 ->  6 ->  7 ->  5 ->  1 ->  0
Time of the route: 6h

Total Time of all routes: 24h

note: it's always around the solution #4500 that the solver find the optimal solution (i.e. 5 nodes in each route)

@Mizux,
how I can show this: I0417 10:45:27.368408 84388 search.cc:251] Solution #4505 (7024, objective maximum = 20022, time = 22554 ms, branches = 67535, failures = 16355, depth = 33, Cross, neighbors = 4716071, filtered neighbors = 4505, accepted neighbors = 4505, memory used = 31.79 MB, limit = 75%) when I code with python? where is it from?

It is from the C++ layer, enable by the log_search or trace_search parameter.


Make placeholder accessible

Placeholder won't read to screen readers, it technically can be read but it is not in the expected position in the flow. Instead, it should use ~placeholder~ aria-placeholder on the contenteditable.
Placeholder is now part of the ContentEditable (backward compatible for now), for two reasons:

It makes sense to couple the contenteditable with the placeholder. That's how native placeholder and input work.
This is a common pitfall. By coupling them we can ensure that either these properties are provided or none (when there's no placeholder).

Before, doesn't read, interferes with the flow:
https://github.com/facebook/lexical/assets/193447/f56858f7-95d6-4d19-b5ff-baa954e1bdea
After:
https://github.com/facebook/lexical/assets/193447/3ebd395e-aca1-4c34-9e8e-b14b0ecaf6df

I can't repro that integration test failure locally (likely has something to do with the github node cache behavior) but perhaps removing the --no-save --no-package-lock from the suite might fix it
https://github.com/facebook/lexical/blob/main/scripts/__tests__/integration/utils.js#L92


Install yum-utils

Install yum-utils if it's not exists yet.

This change is

@svetak-quali  - Please review.


chore: modify simapp to fulfill the sdk runtime.AppI interface

Description

closes: #3152
Modify simapp to fulfill the sdk runtime.AppI interface
Commit Message / Changelog Entry
chore: modify simapp to fulfill the sdk runtime.AppI interface

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards and Go style guide.
[x] Wrote unit and integration tests.
[x] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[x] Added relevant godoc comments.
[x] Provide a commit message to be used for the changelog entry in the PR description for review.
[x] Re-reviewed Files changed in the Github PR explorer.
[x] Review Codecov Report in the comment section below once CI passes.

Codecov Report

Merging #3216 (f808a46) into main (62e09ed) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3216   +/-   ##
=======================================
  Coverage   78.54%   78.54%
=======================================
  Files         181      181
  Lines       12580    12580
=======================================
  Hits         9881     9881
  Misses       2262     2262
  Partials      437      437

Impacted Files
Coverage Δ

modules/core/02-client/migrations/v7/store.go
68.85% <100.00%> (ø)

modules/core/02-client/simulation/decoder.go
100.00% <100.00%> (ø)

...dules/light-clients/06-solomachine/client_state.go
88.28% <100.00%> (ø)


How does yii2 set the master db to write only and read - only from the slave

I want to set the master db for wrte only,  and set the slave db to read only.
but when the slave all is shutdown , it try to connect master,
I want it not to connect master, how does?

See $fallbackToMaster param
http://www.yiiframework.com/doc-2.0/yii-db-connection.html#getSlave()-detail


Make Processing Servers available

The webapi currently only offers ocrd-dummy: http://<IP_ADDRESS>:5050/processor/. My goal is to make at least ocrd-minimum processors available as processing servers.

we decided to change the implementation, we want to implement the processing broker. So this is obsolete now


Update is not working on Windows

Version: 1.0.112
Platform: win32 10.0.17134
Plugins: none

Version 113 also fails to complete an upgrade. It shows the present icon in cyan in the toolbar, and when clicked that shows the dialog that says "Installing the update will close all tabs and restart Terminus" with an "Update" button, but when pressed the dialog goes away and nothing happens. Downloading a more recent installer and running it does allow for an upgrade.
My copy is installed only for a single user.

I started to notice this too, and then I ran Terminus with elevated privileges. Clicked "Update" on the dialog and nothing happened, but then around 2-3 minutes later the setup window appeared.


Issue with Specifying Fine-Tuned Model Endpoints in VertexAIModelGarden

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.

Example Code
def determine_subcategory(main_category, keyword):
    try:
        with open('subcategory_mapping.json') as file:
            json_data = json.load(file)
        category_info = json_data.get(main_category)
        if not category_info:
            return "Category info not found"

        # Retrieve the model ID from the category_info
        model_id = category_info.get("model")
        if not model_id:
            return "Model ID not found in category info"

        # Initialize the fine-tuned model specified in the subcategory mapping
        parameters = {
            "candidate_count": 1,
            "max_output_tokens": 1024,
            "temperature": 0.9,
            "top_p": 1
        }
        model = TextGenerationModel.from_pretrained("text-bison@002")
        model = model.get_tuned_model(model_id)

        # Construct the prompt including the keyword
        prompt_context = category_info["prompt_context"]
        prompt = f"{prompt_context}\n\nKeywords: {keyword}"

        # Invoke the fine-tuned model with the constructed prompt
        response = model.predict(prompt, **parameters)

        # Extract 'text' from the response
        subcategory = response.text.strip() if response.text else "Response format error"
        return subcategory
    except Exception as e:
        logger.error(f"Subcategory determination error: {e}")
        return "Subcategory determination failed"

Error Message and Stack Trace (if applicable)
[00:01:03] ERROR    Subcategory determination error: 404 Endpoint projects/1055022903754/locations/europe-west4/endpoints/4453747770167132160 not found.
Description
When using the VertexAIModelGarden class to send requests to fine-tuned models on Vertex AI, the class is designed to target endpoints rather than directly to a model. However, for my use case, I need to send requests directly to a fine-tuned model URL. The current implementation seems to only allow sending requests to an endpoint, which does not fit the requirement.
Steps to Reproduce

Instantiate the VertexAIModelGarden class with the project ID and endpoint ID.
Use the predict method to send a prompt to the model.
The request is sent to an endpoint URL rather than the fine-tuned model URL.

Expected Behavior
I expect to be able to specify a fine-tuned model URL directly, similar to how it's done using the TextGenerationModel class from the vertexai package:
import vertexai
from vertexai.language_models import TextGenerationModel

vertexai.init(project="my-project-id", location="my-location")
model = TextGenerationModel.from_pretrained("base-model-name")
model = model.get_tuned_model("projects/my-project-id/locations/my-location/models/my-model-id")
response = model.predict(prompt, **parameters)

### System Info

MacOS

Currently, requests for all models, including the finetuned LLMs, such as the one tailored for bison002, are being made to this endpoint:
projects/1055022903754/locations/europe-west4/endpoints/4453747770167132160
However, as per Google's documentation, the finetuned model requests ought to be directed to the following model URI:
projects/1055022903754/locations/europe-west4/models/4453747770167132160
For accurate routing and to align with the documentation, please use the model URI provided for finetuned models or creating a new function for fine-tuned Vertex AI models would be great!

Here's an example of how to modify the endpoint_path property to return a model path for contributor:
class VertexAIModelGarden(_VertexAIBase, BaseLLM):
    # ... existing code ...

    @property
    def model_path(self) -> str:
        # Construct the model path instead of the endpoint path
        return self.client.model_path(
            project=self.project,
            location=self.location,
            model=self.endpoint_id,  # Assuming endpoint_id actually contains the model ID
        )

    # ... existing code ...

    def _generate(
        self,
        prompts: List[str],
        stop: Optional[List[str]] = None,
        run_manager: Optional[CallbackManagerForLLMRun] = None,
        **kwargs: Any,
    ) -> LLMResult:
        # ... existing code ...
        # Use the model_path instead of endpoint_path when making the predict call
        response = self.client.predict(model=self.model_path, instances=instances)
        return self._parse_response(response)

    # ... existing code ...


TypeError: 'LabelSetView' object is not callable

I get this error when running the code against a Neo4j 3.4.0 db and V4.X of Py2Neo Library.
It has to be a version issue, any suggestions?

It seems to be, there is a pull request #6  that might solve it, i'll check it ASAP.
Thanks for the report @profversaggi

I'm getting the same issue

Getting the same error when running the following notebook on local computer.
https://nicolewhite.github.io/neo4j-jupyter/hello-world.html

I'm getting the same issue.

@Niko-La Please note that the neo4j-jupyter project that you link above is different from this one. Please find its sources at https://github.com/nicolewhite/neo4j-jupyter

still gettin the error in V5


Fix missing error check of bufio.Scanner (#29882)

Backport #29882 by @forsaken628
maybe more

Don't backport since it is not an important "bug" fix, and no test covers it.


[Bug report]: TdsHeaderItem ignores slot="end" when *ngIf directive value changes

Requirements before reporting

[X] No duplicated issue reported.
[X] I have checked the latest version if the bug exist there. See all available packages at npmJS.com
[X] I have followed the installation guide.

Package versions
@scania/tegel-angular: 1.6.1
Browser
Firefox
Framework
Angular
Version
Angular 16.2.0
Reproduction steps

Write a TdsHeader and one or more TdsHeaderItem elements in your html template
Set an *ngIf directive on the TdsHeaderItem together with slot="end"
Change the value *ngIf directive

Code example
HTML template:
<tds-header *ngIf="vm$ | async as vm">

  <tds-header-item slot="end" *ngIf="vm.user" title="Orders">
    Orders
  </tds-header-item>

</tds-header>

TS component:
export class NavbarComponent {
  private readonly document: Document = inject(DOCUMENT);

  // KeyPress event to toggle the ngIf directive
  private readonly user$ = fromEvent<KeyboardEvent>(
    this.document,
    'keydown'
  ).pipe(
    filter((e: KeyboardEvent) => e.key === 'x'),
    startWith(undefined),
    scan((acc: boolean) => !acc, false)
  );

  // View model for the html template
  readonly vm$ = combineLatest({
    user: this.user$,
    // Other exposed variables
  }).pipe(
    //concatMap((value) => of(null, value).pipe(delay(0)))
  );
}

Please note that I’ve constructed a basic example to illustrate this issue. You can replicate the behavior by copying and pasting the provided code into your environment. Once you’ve done that, press X in your browser to toggle the value of the *ngIf directive. Initially, you’ll observe that it renders on the right side. However, upon disabling and re-enabling it, the rendering shifts to the left instead of maintaining its position on the right.
I’ve implemented a temporary workaround for this issue by triggering a ‘re-render’ of the entire TdsHeader component whenever the view model vm$ emits. However, this is merely a quick fix as it causes all child components to re-render as well. To illustrate this, please uncomment the concatMap function operator in the pipe. Here’s what happens: when vm$ emits, it first emits a null value. This effectively unrenders the entire TdsHeader component due to its *ngIf directive: <tds-header *ngIf="vm$ | async as vm">. Following this, it emits the actual value, which triggers a re-render of the entire TdsHeader component. As a result, the TdsHeaderItems are correctly positioned.
Screenshots

Expected behaviour
The TdsHeaderItem element should consistently stay anchored to the right side, regardless of whether the element has been re-rendered or not.
Console errors
No response
Contact information
<EMAIL_ADDRESS>

This issue is still valid in 1.19.
Example use case is having a little help button in a TdsHeadItem with slot="end", but you only want to show this help if a user is authenticated. If the user loads the website and then authenticates the TdsHeaderItem appears to the right, ignoring slot="end". Reloading the page while authenticated fixes it as then the header is created with the help button already visible.

Hello @luistabotelho & @Patrick2607 ,
We have looked into this issue and concluded that we cannot solve this at the moment because the proposed solution would introduce a breaking change. We will look at this in the future when we are planning for a release of Tegel with breaking changes. Plan and timing for this is to be defined.

Hi @luistabotelho & @ckrook . It has been a while since I opened this issue, so I wanted to share the workaround I have implemented. Currently, I use the CSS Flexbox property order and created my own version of the slot="end". The SCSS is as follows:
.tds-header-component-list {
  li.tds-header-middle-spacer {
    order: 0;
  }
  @for $i from 1 through 9 {
    & > *[cslot='end']:nth-child(#{$i}) {
      order: $i;
    }
  }

  tds-header-brand-symbol {
    order: 10;
  }
}

This solution manually reorders the elements within the header list that have the attribute cslot="end". Note the c prefix I added to slot — cslot can be any name, as long as it is not a reserved word or a Tegel attribute. Elements without the cslot="end" (i.e., those on the left side) remain unaffected. I chose * here as selector, because I have a custom component that needs to be rendered as well. You can replace * for tds-header-item if desired.
I have set the li.tds-header-middle-spacer's order value to 0 so it always stays on the left of the items aligned to the right. For the tds-header-brand-symbol, I set the order value to 10 to ensure it remains on the far-right.
You can test this with the following HTML:
<tds-header>
  <tds-header-title routerLink="/">{{ title }}</tds-header-title>

  <tds-header-item title="Item 1 Left">
    <button id="item1l">Item 1 L</button>
  </tds-header-item>
  @if (toggleZ()) {
    <tds-header-item title="Item 2 Left">
      <button id="item2l">Item 2 L</button>
    </tds-header-item>
  }
  <tds-header-item title="Item 3 Left">
    <button id="item3l">Item 3 L</button>
  </tds-header-item>

  <tds-header-item title="Item 1 Right" cslot="end">
    <button id="item1r">Item 1 R</button>
  </tds-header-item>
  @if (toggleX()) {
    <tds-header-item title="Item 2 Right" cslot="end">
      <button id="item2r">Item 2 R</button>
    </tds-header-item>
  }
  <tds-header-item title="Item 3 Right" cslot="end">
    <button id="item3r">Item 3 R</button>
  </tds-header-item>

  <tds-header-brand-symbol cslot="end">

  </tds-header-brand-symbol>
</tds-header>

And this TypeScript snippet for your component:
export class AppComponent {
  readonly title = "Tegel Issue 528";

  readonly toggleZ = toSignal(
    fromEvent<KeyboardEvent>(document, 'keydown').pipe(
      filter((val) => val.key === 'z'),
      scan((acc, _) => !acc, false)
    ),
    { initialValue: false }
  );

  readonly toggleX = toSignal(
    fromEvent<KeyboardEvent>(document, 'keydown').pipe(
      filter((val) => val.key === 'x'),
      scan((acc, _) => !acc, false)
    ),
    { initialValue: true }
  );
}

Once you load the page, press z to toggle Item 2 L, and x to toggle Item 2 R. The desired behavior should now work as expected. :)


Table csv links in viewer do not allow changing of directory name

The links aren't relative enough -- they include in the path the results_dir, which means I can't change the name of results_dir post-run without breaking all of the links to the tables.
Example: link is to "acme_diags_results-2018-01-31_13-51-53/viewer/table-data/ANN_metrics_table.html", whereas the other links are more relative -- one of the pngs links to "[results_dir, doesn't matter what it's called or if it changes]/viewer/zonal/debugging-test/prect-global/ann.html" and the others use similar relative paths.

@kaabre Thanks for letting us know. We'll get to this asap.

Done in 587e5af0521493c4264fc0babb927f150d60d980.


WIP Do not merge

Signed-off-by: Nikolaos Moraitis<EMAIL_ADDRESS>/cc @ghost
/hold

/payload 4.10 nightly informing


Modals get no backdrop when first constructed with isOpen={true}

I'm writing a desktop app using Electron. If the app is not configured when it is first opened, I show a modal with a dialog for providing the configuration. The app therefore first opens with this modal open.
Unfortunately, it seems that in order to get the backdrop for a modal, a page must first render without the modal open. My workaround it to set a timer for opening the modal after a second.
It's easy to recreate this issue by hardcoding a modal with isOpen={true}.
UPDATE #1: Including the following code in my component corrects the problem:
  <Portal>
    <ModalBackdrop {isOpen} {fade} />
  </Portal>

UPDATE #2: The above workaround causes two modal backdrops to appear on modals that aren't initially shown (making the backdrop extra dark), so I had to solve the problem as follows:
  let isOpen = false;

  onMount(async () => {
    await tick();
    isOpen = true;
  });

Oh!  Thanks for the heads up @jtlapp .  Will take a look at best fix, that should just work.

Oh!  Thanks for the heads up @jtlapp .  Will take a look at best fix, that should just work.

I ended up having additional problems related to the modal tracking whether it is open, which I resolved by implementing modals myself and having the visibility of the modal be a function of its inclusion in the container, rather than a function of the value of a variable. I don't remember the additional conflicts at this point, but having a component track whether it is visible seems to be at odds with how Svelte is designed to work, where components render upon conditional inclusion.

I ended up having additional problems related to the modal tracking whether it is open, which I resolved by implementing modals myself and having the visibility of the modal be a function of its inclusion in the container, rather than a function of the value of a variable. I don't remember the additional conflicts at this point, but having a component track whether it is visible seems to be at odds with how Svelte is designed to work, where components render upon conditional inclusion.

Released in v5.7.0 thanks for the heads up 👍

Released in v5.7.0 thanks for the heads up 👍


Create an Admin Group to administer the Transition Analysis project

Here is the list of github IDs to have admin privileges for the group, they can then add access for people directly.
sostrades-matthieu-meaux
sostrades-mariemorere
sostrades-pjbarjhoux
sostrades-valentinjoncquieres v
sostrades-dpeltre
sostrades-thierry-chevalier

@ryanaslett and @redmikhail can you please set this up for @sostrades-matthieu-meaux

@HeatherAck (or @sostrades-matthieu-meaux) What resources do they need to have permissions to? I see six repos (5 public, 1 private) that begin with "sostrades". Is it just those repos?

Also related to the exact scope of the group would be what to name it. "sostrades-admin"? "transition-analysis-admin"? "capgemini-admin" (based on the email that references this issue)?

Hi,
I am at the origin of this demand, so I can provide you some inputs :
We would need a group (sostrades-transition-risk would be fine - it may be a subgroup of the existing transition-risk team) where the above mentionned members have admin rights, (so that we can add new members to this group, for example), in order to manage all these repositories :
sostrades-value-assessment
sostrades-core
sostrades-webgui
sostrades-webapi
sostrades-authapi
witness-core
witness-energy
thanks & Regards
Pierre-Jean

I've created the group and granted admin permissions to the listed repos.

Thank you @MightyNerdEric @HeatherAck

@sostrades-pjbarjhoux Whoops, sorry about that oversight. I have changed you all to team maintainers.


CLI install example is powershell while the subsequent usage examples are bash

On this page in the CLI section, it starts by providing an example of how to install the Azure CLI.  This example uses powershell and is windows specific (it calls an msi installer).
However, the subsequent examples in this section use bash;  my assumption is that the examples are meant to run under linxux (or WSL), so the installer example above doesn't really help for someone trying to walk through the examples for the first time.
My suggestion is to make the examples consistent; either all windows/powershell or all linux/bash.

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

ID: 7d8793d6-bb58-2beb-f959-4654d7bdad69
Version Independent ID: a42a02a9-36b0-70dc-0737-816904f0e1f1
Content: Incremental snapshots (preview) for managed disks
Content Source: articles/virtual-machines/windows/disks-incremental-snapshots.md
Service: virtual-machines-windows
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

@larryclaman assigning this to author to review as we discussed :)

@larryclaman Thanks for reaching out, there seems to be some confusion here. The article is broken up by language. The bash code blocks are relegated the CLI, the PowerShell segments are, well, in the PowerShell section. Each section has their own installation method and the PowerShell section doesn't have bash code blocks.
Please let us know if you've any further questions, otherwise I'll close this out in two days.

Hmm, I still think the CLI section in the document needs some edits for clarity.  As I noted, the CLI section:

starts off saying you need to use the latest version of the CLI
provides instructions on how to use powershell to install the latest version of the CLI on Windows (but doesn't explicitly mention windows)
then provides an example of how to create a snapshot.  This example is written in Bash (eg, the way it sets variables, and the way it continues a multiline command).

I think the context switch between bullet 2 and bullet 3 is too much (going from a windows env to a bash/linux env without mentioning the switch.)
I'll take a shot at a rewrite for clarity and will submit a PR.  Hope you like it.  :)

@larryclaman Ah, now I see what you mean. Looks like I was the confused one here. ;)
I'll adjust the section to direct users here, since there's multiple ways to install the CLI per distro. https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest

Hi @roygara I just created PR #43266; please take a look.

I'll go ahead and close this out since your PR has been merged. Again, thanks for your help improving our docs. :)
#please-close


Add "sample" term to glossary

Addresses issue #750

Btw., this is also documented in https://prometheus.io/docs/concepts/data_model/#samples, but it's a good idea to have it in the glossary as well.

@juliusv @brian-brazil I've modified the definition

Looks good besides the nit about ending files with newlines.

@juliusv Yes, and I can add an editorconfig file that enforces that.


Adding the selenium basics with the python for beginners.

All other changes are looking good. Just change that line and lmk.

Thanks, @mohitpeshwani for contributing to this repository. I do not have any Active Flutter or Kotlin projects but I do have some react projects. If you would Like to see, please visit this link : https://logan1x.github.io/projects.html

Cool, thanks!
On Wed, Feb 1, 2023 at 4:53 PM Khushal Sharma @.***>
wrote:

Thanks, @mohitpeshwani https://github.com/mohitpeshwani for
contributing to this repository. I do not have any Active Flutter or Kotlin
projects but I do have some react projects. If you would Like to see,
please visit this link : https://logan1x.github.io/projects.html
—
Reply to this email directly, view it on GitHub
https://github.com/Logan1x/Python-Scripts/pull/107#issuecomment-1411901160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQA3NLOJOCNTG3VWF2XS3LLWVJBU3ANCNFSM6AAAAAAUGSLUXQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


[FLINK-34152] Add an option to scale memory when downscaling

This adds an option to increase heap and managed memory when removing TaskManagers. The scaling is applied after adjusting the memory pools (heap, metaspace, network, managed) and only affects heap memory.
The reason for adding this functionality is that the likelihood of running into memory constrained scenarios when downscaling is increased after applying memory tuning. As a precaution, we temporarily increase the total cluster
memory proportionally to the removed TaskManagers up to the maximum allowed TaskManager memory.

@1996fanrui I'm merging but I'll take any comments from your side that you might have reviewing this later on.

@1996fanrui I'm merging but I'll take any comments from your side that you might have reviewing this later on.

Thanks @mxm for the improvement and ping! Sorry for the late response here. I just finished my vacation, I will take a look these 2 days.


added functions for editing only one message at a time

It fixes #8344 .

@SatyendraBanjare can you clean up your commits and resubmit if you'd like to work on this?  We can't review a change like this that doesn't explain why it's correct.
Check out the Zulip commit and commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/contributing/version-control.html


is ttyd the right tool for my remote managment scenario, when SSH is block on my machine

Hello,
it took me some time to find apps to enable me to share a shell in a browser.
Me and friend managing some simple VPSs that shall be managed from a central management server to simplify workflow.
There are cases that we still need a shell and allowing us to share this session would be extremely helpful.
In addition, when I'm at work, SSH is blocked so it quite challenging to manage them.
So is ttyd the tool to get around this?
The architecture is close to this one:

thank you for answering my humble question which answer might be obvious but due to my endless searching I cannot see the woods for the trees.
Stefan

I don't know, it's up to your requirements and usage. BTW, I don't use ttyd on production environment that open to the internet (for security concerns).


set number of initial nodes to overcome the Ingress quota limit of 100

This change does introduce a new variable, but intentionally not added to CONFIG-VARS.md.
It's a complex issue. If there is a need for documentation, a might want to have a separate page.

LGTM. Merging.