[bug] how to add system tray

Describe the bug
I set up the system tray according to the method of v1, and I can report an error. I saw that v2 seems to use the tray-icon feature to implement the system tray, but I don't know how to write a system tray. Can you give me a minimum example that can run?
Reproduction
No response
Expected behavior
No response
Platform and versions
[build-dependencies]
tauri-build = { version = "2.0.0-alpha", features = [] }

[dependencies]
tauri = { version = "2.0.0-alpha", features = ["tray-icon"] }
tauri-plugin-window = "2.0.0-alpha"
tauri-plugin-shell = "2.0.0-alpha"

Stack trace
No response
Additional context
this is my run code:
#[cfg_attr(mobile, tauri::mobile_entry_point)]
pub fn run() {
  // TrayIconBuilder::new()
    tauri::Builder::default()
        .manage(AppState {
            db: Default::default(),
        })
        .setup(setup)
        .plugin(tauri_plugin_window::init())
        .plugin(tauri_plugin_shell::init())
        .invoke_handler(tauri::generate_handler![greet])
        .invoke_handler(tauri::generate_handler![start_hy])
        .run(tauri::generate_context!())
        .expect("error while running tauri application");
}

I don't know how to use tray-icon， please help me, thanks.

You can do your as here
// src/main.rs
use tauri::{CustomMenuItem, SystemTray, SystemTrayMenu};

fn main() {
  let tray_menu = SystemTrayMenu::new(); // insert the menu items here
  let system_tray = SystemTray::new()
    .with_menu(tray_menu);
  tauri::Builder::default()
    .system_tray(system_tray)
    .run(tauri::generate_context!())
    .expect("error while running tauri application");
}

Everything coupled with interesting you topic is avaiable in tauri documentation

.system_tray(system_tray)

I think you misunderstood me. The version I am using is not v1, but v2. When I use tray like v1, I get an error:
error[E0432]: unresolved imports `tauri::CustomMenuItem`, `tauri::SystemTray`, `tauri::SystemTrayMenu`
 --> src/lib.rs:9:13
  |
9 | use tauri::{CustomMenuItem, SystemTray, SystemTrayMenu};
  |             ^^^^^^^^^^^^^^  ^^^^^^^^^^  ^^^^^^^^^^^^^^ no `SystemTrayMenu` in the root
  |             |               |
  |             |               no `SystemTray` in the root
  |             no `CustomMenuItem` in the root

The v2 apis where renamed a bit: https://docs.rs/tauri/2.0.0-alpha.18/tauri/tray/index.html - the migration guide should be up to date i think: https://beta.tauri.app/guides/upgrade-migrate/from-tauri-1/#migrate-to-tray-icon-module

@FabianLars Whenever I add the code to test with the v2 tray code it simply seems to ignore it and it gives me that let tray is unused implying that something is missing in the setup.
https://gist.github.com/jdegger/aeea3dc369359775d426dd636fae9220
I also have a main fn which is not in the migration guide.
Things are probably getting mixed up on my end but so far I have only followed al the migration guides and this is the result. Any clue where I went wrong?

If you don't need the tray handle you can just remove the let tray = part because .build(app) is what actually registers it to the app.
Rust doesn't know what build() does in this context, it's just complaining about the unused let binding.

Thanks for the quick reply, I will keep on debugging then. Whenever I remove the tray+menu icon altogether it does not change anything. The menu is not registered as well.
Thanks again and no worries

Not sure why, but I was not able to get it working without binding it to the main id as configured in my tray config in tauri.conf.json.
My code is as follows now which seems to work:
#[cfg_attr(mobile, tauri::mobile_entry_point)]

use tauri::{
  Manager,
  tray::{ClickType, TrayIconBuilder},
};

fn main() {

  tauri::Builder::default()
      .setup(|app| {
        let _tray = TrayIconBuilder::<_>::with_id("main")
            .on_tray_icon_event(|tray, event| {
                if event.click_type == ClickType::Left {
                    let app = tray.app_handle();

                    #[cfg(not(target_os = "macos"))] {
                      if let Some(webview_window) = app.get_webview_window("main") {
                        let _ = webview_window.show();
                        let _ = webview_window.set_focus();
                      }
                    }

                    #[cfg(target_os = "macos")] {
                      tauri::AppHandle::show(&app.app_handle()).unwrap();
                    }

                }
            })
            .build(app)?;

        Ok(())
    })
    .on_window_event(|window, event| match event {
      tauri::WindowEvent::CloseRequested { api, .. } => {
        #[cfg(not(target_os = "macos"))] {
          window.hide().unwrap();
        }

        #[cfg(target_os = "macos")] {
          tauri::AppHandle::hide(&window.app_handle()).unwrap();
        }
        api.prevent_close();
      }
      _ => {}
    })
    .run(tauri::generate_context!())
    .expect("error while running tauri application");

}

In this example I have removed the menu and menu items as this was not working (also not when I removed the left click event on the tray icon). I do not really care for the menu so in my case it doesn't matter but it still interesting as to why it does not work.
For now I'm good though, this might help somebody else but it should also be taken with a grain of salt because at this point I do not really have a clue why it works. 🤷

(still only on my phone so won't be able to do more than giving hints)
I think this is just a documentation issue. In v1 you had to configure the icon in tauri.conf.json and then the menu in rust.
In v2 this is also still possible but the tauri.conf.json entry is optional and if you use trayiconbuilder (with an id other than main) it will create a new separate tray icon which will not have the icon configured in tauri.conf.json, you will have to use the builder's icon setter instead.
If you prefer setting the icon in tauri.conf then you can get a handle to the tray via app.tray().get_by_id("main") which has methods to set the menu (iirc). I would expect that using the builder on the same Id as configured in the conf would cause an error similar to duplicate window labels so I'm surprised your code works.

Took me a while to figure this one out but I think I got there in the end. The app.tray() method does not seem to be exposed or I might be missing a feature flag?
This is what I ended up with:

I removed the tauri.conf.json entry for the tray icon altogether
I enabled the feature image-png in tauri.conf.json in the tauri dependency

This is my code:

@FabianLars As an additional pointer for the docs, this is what I found difficult to understand as a newbie in rust and tauri:

had no clue I had to enable the feature flag to allow from_path and couldn't figure out why this method was not available. I found in the Rust docs that it needed the flag and that worked. I now also see this in the code: #[cfg(any(feature = "image-ico", feature = "image-png"))] which makes it quite obvious once you know
The migration docs for the tray do not show a main fn which is confusing
The migration docs to not add an icon, which was the hard part to figure out

Thanks for the feedback!

If someone struct with from_path error. You can update Cargo.toml to this:
[dependencies]
tauri = { version = "2.0.0-beta", features = [
    'tray-icon',
    'image-ico',
    'image-png',
] }

@jdegger and @chenka thanks for the help! This should really be in the docs... Not adding the icon in the example is really confusing since it doesn't show up in the system tray unless the image is there.

The v2 apis where renamed a bit: https://docs.rs/tauri/2.0.0-alpha.18/tauri/tray/index.html - the migration guide should be up to date i think: https://beta.tauri.app/guides/upgrade-migrate/from-tauri-1/#migrate-to-tray-icon-module

For future searchers, the updated link is: https://beta.tauri.app/start/migrate/from-tauri-1/#migrate-to-tray-icon-module


Host should be optional (media_player/onkyo)

Description:
Per the docs Host is supposed to be an optional parameter.  I'm guessing this was just an accidental change when moving to voluptuous as changing it to optional and leaving it off works just fine in my testing.
Related issue (if applicable):
Pull request in home-assistant.github.io with documentation (if applicable):
Example entry for configuration.yaml (if applicable):
Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

There is discovery. Looks like that I didn't pay attention while migrating to voluptuous.

Sam thing happends also for webos discovery, ip also there required but thays another bug I guess


HotFix - vrouter is not starting

@Attil could you comment how this issue was fixed, for posterity?

The problem was somew@m-kostrzewa The problem I've discovered here is not directly related. vRouter is initialized inside SxExtCreateSwitch routine, while device object used for ksync/shared memory and other stuff like this is created in SxExtInitialize, which means if you send a request between the driver initializing and enabling the extension, it'll crash.
I should've requested this change to the original pull request (#42), but I've already fixed it locally. It will be applied in JW-296 commits.


feat: Added to see other user profile and their posts successfully Issue 186

Pull Request Resolves [#186 ]
Title:  Added to see other user profile and their posts successfully.
Description

I created a route and controller in the backend to fetch user profile with their ID's.
In frontend the post page is navigating to profile section with params i am fetching their profile.
Also implemented toolkit to understand user that tey can see the other users.
Alos implemented on the leaderboard also.
Responsive design.

Related Issues
Fixes #186
Changes Made
New features: Added to see other user profile and their posts.
Screenshots
Other user profile:

Your own profile:

LeaderBoard:

I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.
Thank You for this contribution.

Since there are so many changes, I may request some more changes after reviewing completely

Since there are so many changes, I may request some more changes after reviewing completely

Sure, and see the changes you requested

@VaibhavArora314 it should level 3 if you see i had work on both front + backend that's why else you can evaluate it 😄


Multithreading capture images from few cameras

Hi, I would like to capture images from few cameras in the same time. But I would like the errors or delays of downloading images from one camera to not affect all. At the moment the problem is that when one camera has the exception "timeout getting image" the rest is waiting for it to be downloaded or until the above mentioned exception pops up. I would like to make it independent. I'm using a modified example of : https://github.com/sarxos/webcam-capture/tree/master/webcam-capture-examples/webcam-capture-live-streaming

@adelanteDev You should be able to achieve that, at least partially, by opening webcams in a non-blocking mode. By default when you call webcam.open() it's being open in a blocking mode, which means that every call to webcam.getImage() will wait for the new image being available. However, when you open webcam with webcam.open(true) the webcam will be open in a non-blocking asynchronous mode, which means that webcam.getImage() will never wait for new image to be available but return currently available image every time when you call it. Please note that there are some downsides of a non-blocking mode which has to be taken into account! If you invoke getImage() few times and very fast in a row you will get the same image (this is because webcam will return current image, not the new one). This mode may require some changes in your program architecture (you may need to switch from blocking and synchronous design to non-blocking asynchronous design).

Okay, thank you for your help. So I will do that :)

Ok, in this case I'm closing the ticket. Good luck!


docs: make it unixy

Updates description to be a bit more clear in what sheetify brings to the table

merging, can always be changed I reckon (':


Added CI-demo badge

Pull Request Description
Summary
Update README.md to have to Django ci-badge
Changes Made

Update README.md with Django ci-badge

Good work


[Bug]: datasource form error message gets garbled on smaller screen/larger zoom level

Is there an existing issue for this?

[X] I have searched the existing issues

Description
when I either increase zoom level or reduce the window size, the datasource form's field validation errors get garbled. Video:
https://user-images.githubusercontent.com/94514895/219954065-af0ba1f4-f992-46c2-bce6-7063dd10604c.mov
Screenshot

Steps To Reproduce

increase the zoom level to 125%
reduce the window width a bit
add a new datasource, e.g. elasticsearch
enter a wrong url
--> the error message that is shown overlaps the field and is not clear

Public Sample App
No response
Issue video log
No response
Version
Cloud

I Think -This issue was"I think the issue has been solved.
https://www.loom.com/share/479a1e7a7371473b84fdff6a7ad9290f

thanks @raza001 you're right. This may have gotten fixed with some other issue. I am closing this one.


helm: apply Role and RoleBinding for envoy gateway in namespaced mode

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #1619

thanks for working on this @shawnh2, sharing some thoughts, @Xunzhuo would also love your thoughts on this

maybe its time to stop auto generating the ClusterRole https://github.com/envoyproxy/gateway/blob/2edcafe430a2ac1e61ccb4d1a0f5a5ce00ad5a08/tools/make/kube.mk#L34 and just manage it ourselves. This allows us to use your current logic to decide if we need to create a ClusterRole or a Role
I think we need ClusterRole for GatewayClass even for the namespaced mode, so we can split up roles into 2 resources, one for the gateway class cluster role and the other for everything else (cluster role for regular mode and role for namespaced mode)
Can we use {{ include "eg.fullname" $ }}-envoy-gateway-role instead of {{ include "eg.fullname" $ }}-{{ $ns }}-envoy-gateway-role since they are created in different namespaces so the ns separation shouldnt matter ?
Would be good to test this out with the steps highlighted in https://gateway.envoyproxy.io/v0.4.0/user/deployment-mode.html#multi-tenancy and see that it works

sure we can mange ClusterRole and Role ourselves
+1, we should create ClusterRole and Role resources no matter what, but create RoleBinding if we run in namespaced mode, otherwise create ClusterRoleBinding
thanks for pointing it out, missing that oops
thanks for heads up, i will test it out

LGTM thanks @shawnh2, since this PR makes an API change (and affects the docs dir), lets merge this after v0.5 releases. In the interim would be great if you can share the rbac o/p for default case as well as namespaces case in this PR, thanks !

sure, i added some usage in https://gateway.envoyproxy.io/latest/user/deployment-mode.html

@arkodg ready to merge？


[Bug]: v1 features are not correctly migrated to v2 features

A customer reached out to support, as he noticed that their branding suddenly was lost and had changed from light only to auto mode.
While investigating we noticed that there was no entry for their instance in the instance_features projection. Counting all feature-set events and comparing with the instance_features projection showed, that there are ~300 instances missing.
We then noticed that the bug is caused because the v1 features had a generated aggregate id, while the v2 features rely on the id of their level, e.g. instance features use the instance id as aggregate id.
This creates a mismatch. WriteModel will not get any existing events (is probably negligible) and the instance_features projection will compute a wrong state. What's unclear is why ~300 instances are not projected at all.
Additional Info:
https://github.com/zitadel/zitadel/blob/main/internal/repository/feature/feature.go#L20-L27
https://github.com/zitadel/zitadel/blob/main/internal/query/projection/instance_features.go#L101-L109
We also noticed that some v1 feature events marshalled the baseEvent into the payload, but looks like it's not relevant for the projection itself:
{"ID": "", "Agg": {}, "Pos": 0, "Seq": 0, "Value": {"Boolean": true}, "Creation": "0001-01-01T00:00:00Z"}
vs
{"Value": {"Boolean": true}}

We also noticed that some v1 feature events marshalled the baseEvent into the payload, but looks like it's not relevant for the projection itself:
{"ID": "", "Agg": {}, "Pos": 0, "Seq": 0, "Value": {"Boolean": true}, "Creation": "0001-01-01T00:00:00Z"}

As we looked in the DB, these are probably old events. That was fixed in https://github.com/zitadel/zitadel/pull/6858

What's unclear is why ~300 instances are not projected at all.

Conclusion of further investigation was that those events are from removed instances.


Add local-storage-blob-store to README

Adds https://github.com/xicombd/local-storage-blob-store to the list.
Thanks!

awesome thanks


Appendix A.2

Great work. I am slightly confused with the sentence, "When a matrix with identical rows multiplies another matrix, the resultant matrix also exhibits identical rows". Can you provide more examples/illustrations on this?
Thanks.

Thanks for your interest.
Let's consider two matrices:
Matrix A:
| a b |
| a b |
Matrix B:
| c d |
| d c |
Multiplying A by B:
| ac + bd  ad + bc |
| ac + bd  ad + bc |
As we can see, both rows of the resultant matrix are identical, which aligns with the property mentioned.

Thanks for your interest.
Let's consider two matrices:
Matrix A: | a b | | a b |
Matrix B: | c d | | d c |
Multiplying A by B:
| ac + bd ad + bc | | ac + bd ad + bc |
As we can see, both rows of the resultant matrix are identical, which aligns with the property mentioned.

Yes I know that. I think that this operation actually reduces the impact of cross-attention to a large extent. If this, why don't you directly mute the whole cross-attention (e.g., constant attention map)

Yes, you can do in that way. I believe their performance would be similar, but your way would be much more efficient than mine. I didn’t come up with this smart way at that time. Thank you for your good question.

Yes, you can do in that way. I believe their performance would be similar, but your way would be much more efficient than mine. I didn’t come up with this smart way at that time. Thank you for your good question.
Get Outlook for iOShttps://aka.ms/o0ukef

From: YuhaoLiu @.>
Sent: Friday, April 26, 2024 11:09:02 PM
To: Shilin-LU/TF-ICON @.>
Cc: #LU SHILIN# @.>; Comment @.>
Subject: Re: [Shilin-LU/TF-ICON] Appendix A.2 (Issue #22)
Thanks for your interest.
Let's consider two matrices:
Matrix A: | a b | | a b |
Matrix B: | c d | | d c |
Multiplying A by B:
| ac + bd ad + bc | | ac + bd ad + bc |
As we can see, both rows of the resultant matrix are identical, which aligns with the property mentioned.
Yes I know that. I think that this operation actually reduces the impact of cross-attention to a large extent. If this, why don't you directly mute the whole cross-attention (e.g., constant attention map)
—
Reply to this email directly, view it on GitHubhttps://github.com/Shilin-LU/TF-ICON/issues/22#issuecomment-2079579993, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A43KI7VRSARSMN2EEQKIG43Y7JUY5AVCNFSM6AAAAABG25EZ5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZZGU3TSOJZGM.
You are receiving this because you commented.Message ID: @.***>

Yes, you can do in that way. I believe their performance would be similar, but your way would be much more efficient than mine. I didn’t come up with this smart way at that time. Thank you for your good question. Get Outlook for iOShttps://aka.ms/o0ukef
…
________________________________ From: YuhaoLiu @.> Sent: Friday, April 26, 2024 11:09:02 PM To: Shilin-LU/TF-ICON @.> Cc: #LU SHILIN# @.>; Comment @.> Subject: Re: [Shilin-LU/TF-ICON] Appendix A.2 (Issue #22) Thanks for your interest. Let's consider two matrices: Matrix A: | a b | | a b | Matrix B: | c d | | d c | Multiplying A by B: | ac + bd ad + bc | | ac + bd ad + bc | As we can see, both rows of the resultant matrix are identical, which aligns with the property mentioned. Yes I know that. I think that this operation actually reduces the impact of cross-attention to a large extent. If this, why don't you directly mute the whole cross-attention (e.g., constant attention map) — Reply to this email directly, view it on GitHub<#22 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A43KI7VRSARSMN2EEQKIG43Y7JUY5AVCNFSM6AAAAABG25EZ5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZZGU3TSOJZGM. You are receiving this because you commented.Message ID: @.***>

Actually, even if this can maintain a nearly perfect reconstruction, the original editing capabilities will be lost. This is why your method must rely on the composition process. It would be more interesting to have a direct balance of reconstruction and editing all at once. In any case, your method is still a good solution.


Bootstrap 4: Button Dropdown Block - Centered Arrow Placement

Whenever I use a button dropdown with the btn-block class, the arrow indicator for the dropdown is weirdly centered with the text in the button.
Is there a way to have the arrow flushed to the right like in Bootstrap 3?

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via CodePen or JS Bin and report back with your link, Bootstrap version, and specific browser and OS details.

Custom CSS or use a segmented button in a button group.


Kotlin Multiplatform support

Is your feature request related to a problem? Please describe.
Is there plans for enabling Kotlin for multiplatform support? One current pain point is standardizing data flow coming from client apps. Looking at the source it looks well equipped for a Multiplatform future (pure Kotlin, coroutines, kotlinx serialization, etc)
Describe the solution you'd like
Id like to share higher level architecture for data schemas / middlewares between clients.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Android, Native, JS targets

I would also like to have this, but mainly for:
Android & Native(iOS)

I would also like to have this, but mainly for:
Android & Native(iOS)

@scottkruse - I have created a KMP wrapper which should work for you.
https://github.com/Reedyuk/Segmenkt

If this could support Android and Native (iOS) also, so mainly the KMM approach, that would be fantastic!

+1 for this.

+1 here too, we'd be interested in this.

Joining in on this - Android and iOS would be great. Actually got here while looking for such library for segment

@yosi199 - I am using the library I posted above, it's in production.

@yosi199 - I am using the library I posted above, it's in production.

Cant get it to compile. getting this:
Could not determine the dependencies of task ':analytica:transformAppleMainCInteropDependenciesMetadataForIde'.

Could not resolve all dependencies for configuration ':analytica:appleMainResolvableDependenciesMetadata'.
The project declares repositories, effectively ignoring the repositories you have declared in the settings.
You can figure out how project repositories are declared by configuring your build to fail on project repositories.
See https://docs.gradle.org/8.0/userguide/declaring_repositories.html#sub:fail_build_on_project_repositories for details.
Could not find com.myunidays:segmenkt:0.0.9.
Required by:
project :analytica

Possible solution:

Declare repository providing the artifact, see the documentation at https://docs.gradle.org/current/userguide/declaring_repositories.html

@yosi199 - I am using the library I posted above, it's in production.

Cant get it to compile. getting this:
Could not determine the dependencies of task ':analytica:transformAppleMainCInteropDependenciesMetadataForIde'.

Could not resolve all dependencies for configuration ':analytica:appleMainResolvableDependenciesMetadata'.

The project declares repositories, effectively ignoring the repositories you have declared in the settings.
You can figure out how project repositories are declared by configuring your build to fail on project repositories.
See https://docs.gradle.org/8.0/userguide/declaring_repositories.html#sub:fail_build_on_project_repositories for details.

Could not find com.myunidays:segmenkt:0.0.9.

 Required by:

     project :analytica

Possible solution:

Declare repository providing the artifact, see the documentation at https://docs.gradle.org/current/userguide/declaring_repositories.html

Can you post this on the project and not here as it's an unofficial library

We have created a custom wrapper similar to https://github.com/MyUNiDAYS/Segmenkt and we integrate segment on the android and iOS side only

I'd like this as well


Error generating tuples in distance QC

Versions
PopPUNK v2.4.4
PopPUNK-sketch v1.7.0
Command used and output returned
python /home/github_packages/PopPUNK/poppunk-runner.py --create-db \
 --r-files acba_poppunk_narrow.txt --output acba_cpu --overwrite --min-k 15 \
 --max-k 30 --k-step 3 --qc-filter prune --threads 32 \
 --type-isolate GCA_009759685 --length-sigma 2 \
 --max-pi-dist 0.03 --max-a-dist 0.8

The error message:
Traceback (most recent call last):
  File "/home/github_packages/PopPUNK/poppunk-runner.py", line 10, in <module>
    main()
  File "/home/github_packages/PopPUNK/PopPUNK/__main__.py", line 333, in main
    seq_names_passing, distMat = qcDistMat(distMat,
  File "/home/github_packages/PopPUNK/PopPUNK/utils.py", line 294, in qcDistMat
    long_edges = poppunk_refine.generateTuples(long_distance_rows,
TypeError: generateTuples(): incompatible function arguments. The following argument types are supported:
    1. (assignments: List[int], within_label: int) -> List[Tuple[int, int]]

Describe the bug
When creating the distance sketches, poppunk gets past the initial prop N QC then appears to fail at the distance QC step.

Thanks - unfortunately, I'm struggling to reproduce this - can you do a local reinstall of popunk_refine by removing the existing compiled code, which can be located through:
import poppunk_refine
poppunk_refine.__file__

Then install through conda, or remake the local install with:
rm -rf build
python setup.py build
python setup.py install

Can you let me know if that changes anything?

Thanks Nick, that's sorted it out now, previously I was using poppunk_refine v2.2.0, updating appears to have fixed it


Does this really support Azure Container Instances?

The introduction implies that Azure Monitor for Containers supports both ACI and AKS, but everything after that refers to Kubernetes only. I have played around with ACI and there doesn't seem to be an option to surface ACIs into Azure Monitor for Containers.
Am I missing something?
Thanks

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

ID: ac3c529f-fe57-558f-bd90-02162ff531e8
Version Independent ID: 8300d958-120e-16c1-09cc-29740e6fae00
Content: Overview of Azure Monitor for containers
Content Source: articles/azure-monitor/insights/container-insights-overview.md
Service: azure-monitor
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@fruffin Thanks for the question! We are investigating and will update you shortly.

To monitor Azure Container Instances via Azure Monitor, you need to enable Container Monitoring Solution.

@fruffin - To correct what @SnehaAgrawal-MSFT responded with, the Container Monitoring Solution does not support monitoring ACI.  Azure Monitor for Containers supports ACI+AKS hosted on Azure Kubernetes.  Today your option to monitor ACI is documented here - https://docs.microsoft.com/en-us/azure/container-instances/container-instances-monitor.
Hope that helps.  #please-close.


Decide what to do with multiple predicate rules not in the 'Kitchen Sink'

We should discuss how to handle the following rules with multiple predicate rules:

check_bindings
check_future_1
check_uk_mdattr
check_uk_mdrps
check_uk_rands
check_ukfedlabel

Questions:

do we need to test them?
if so, do we include them in the default validator or do we need another?

I can't remember what the definition of "kitchen sink" was such that these are not included. With the possible exception of check_future_1, they are all in use in production and will all need to be fixed before we're able to transition away from Xalan. For example, check_uk_mdrps is called from uk_registeredEntities which is the start of every production run. This might not be clear from where you're sitting, and we've thought for some time that this kind of "what policy is applied where" issue isn't well structured in the codebase as it stands.
Maybe we need a bigger kitchen sink.
check_future_1 may be easy to take out of the equation, though. The "future" tests are normally things we're considering adding to the battery but are evaluating the implications of. That specific one has been there for six years now and maybe @alexstuart can tell us if it's still under consideration. If not, we should just clear it out. If it's now regarded as applicable we can move it into the standard set (and add tests, and fix the predicate issue in it).
If check_future_1 needs to stay in its current limbo form, I'm inclined to say that this is what additional profiles / validator pipelines might be useful for: adding one for "future" tests would make sense.

check_future_1 may be easy to take out of the equation, though. The "future" tests are normally things we're considering adding to the battery but are evaluating the implications of. That specific one has been there for six years now and maybe @alexstuart can tell us if it's still under consideration. If not, we should just clear it out.

check_future_1 is still under consideration, in the sense that there's an open issue in ukf-meta. However, the issue's deferred and there's been no movement for some time, and the tests in check_future_1 would be easy enough to re-add. I'll clear it out.

Great, @philsmart I think that means you can just copy across the trimmed version of that file from ukf-meta and we have one less thing to worry about.

I will move than over in a separate PR.

Given check_future_1 has no rules in it, I will not move this over for now.


no search results for specific series

What TMM version are you using?
tmm_3.0-SNAPSHOT_bf170ff_win
release, pre-release, nightly, or directly from GitHub/branch?
NIGHTLY
What is the actual behaviour?
Can not find series with ";" e.g. "ChäoS;Child" or "Steins;Gate"
I need to remove all ; in the search textbox to get a result.
What is the expected behaviour?
A search result because it's in the database https://www.thetvdb.com/series/chaos-child
Steps to reproduce:
Rightclick scrap a serie, enter in search "ChäoS;Child".
No Result found.
Additional
Have you attached the Logfile from the day it happened?
Your Operating system? (win/mac/linux? version?)
Windows 10 x64

Well, yes, but there's nothing we can do about :| (at least, if we don't need to)
We pass the title/searchbox 1:1 to the API.... and they find something or not.
Usually, we do not want to compensate their lack of searchability. (other sites do not have such problems)
Ok, removing a ; from searchrequest ist not hard (done for next build), but where does it end?
What other special handlings does the TVDB have?
Anyway.. thx for reporting :)


Docker

Codecov Report

Merging #116 into master will decrease coverage by 7.65%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
- Coverage   51.58%   43.92%   -7.66%
==========================================
  Files           5        3       -2
  Lines        1642     1416     -226
==========================================
- Hits          847      622     -225
+ Misses        795      794       -1

Impacted Files
Coverage Δ

test_server.py

promotion/promotion.py

Continue to review full report at Codecov.

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

Code coverage check fails. Will retry and then issue a pull request.


[Bugfix] Cannot find global function mlc.llm_chat_create

Should fix https://github.com/mlc-ai/llm-perf-bench/issues/32


Clear stat cache in FileCache::set()

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
https://github.com/yiisoft/yii2/pull/20304

@vjik good ideas. Thanks. Applied.


[AIRFLOW-YYY] Add GKE example with XCOM

Make sure you have checked all steps below.
Jira

[ ] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.
In case you are proposing a fundamental code change, you need to create an Airflow Improvement Proposal (AIP).
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does
If you implement backwards incompatible changes, please leave a note in the Updating.md so we can assign it to a appropriate release

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@cfd6022). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #298   +/-   ##
=========================================
  Coverage          ?   80.02%
=========================================
  Files             ?      594
  Lines             ?    34753
  Branches          ?        0
=========================================
  Hits              ?    27810
  Misses            ?     6943
  Partials          ?        0

Impacted Files
Coverage Δ

...flow/gcp/example_dags/example_kubernetes_engine.py
0% <0%> (ø)

Continue to review full report at Codecov.

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


Please no full path by default

My problem is described in this issue: https://github.com/vuejs/vueify/issues/22
There was given a solution: one should use NODE_ENV=production
However I think there should be no abosulte paths included by default. In case someone is not careful enough.
For example I use laravel-elixir-vueify and by running gulp --production, I'm still getting the full path. One could expect that --production does this job. But it doesn't...

+1

gulp --production and NODE_ENV=production are not the same.
Full paths or not, as long as you are not using NODE_ENV=production you will still be including unnecessary code. And the argument of "people not being careful" is just silly; docs are there for a reason.

Well, I'm still getting full paths while using NODE_ENV=production. It leads to insert-css lib
2:[function(e,t,s){var o=e("/Users/xxx/project/node_modules/vueify/lib/insert-css.js");
It's only there when including stylesheets in .vue files. Any solution to prevent this?

@automagically it should no longer include this in 8.5.2.

@yyx990803 Yep confirmed, full paths are no longer included. Thanks for the quick response!
For people using gulp --production attribute you could include these lines in your gulpfile.js:
var gutils = require('gulp-util');
var inProduction = gutils.env.production || process.env.NODE_ENV === 'production';

if(inProduction) {
    process.env.NODE_ENV = 'production';
}

the argument of "people not being careful" is just silly; docs are there for a reason

No, it should be the other way round. The most secure, or at least the most reasonably secure config should be the default.  And if a user specially wants a less secure functionally, ok, (s)he's free to go, but in order to do this, it's inevitable to read the docs. Everyone is safe and everyone is happy. And this is for general.
Example
Firefox, by default, checks https certificates whether they are valid or not. However if it bothers you, you can turn it of. Here's how: https://support.mozilla.org/en-US/kb/advanced-panel-settings-in-firefox#w_certificates-tab
Now according to you, it would perfectly fine if this setting in Firefox was default to false. And then I, as an average user  who knows nothing about web security but is intended to use a web browser, according to you, I should read cross all Firefox documentation, find this setting, and set it to true. But of course only after reading some articles about ssl, private and public keys and signatures.
You can't possibly expect this from an average user, can you? Luckily, at Mozilla, they were wise enough to ship their product with secure default settings.
You might say, here we are developers, not average users, we are a separate cast that stands above all others, right? So it's perfectly required from us to be much more circumspect regarding default behaviour in a given codebase that, God forbid, we'd like to use. And of course compared to anyone else, we have infinite amount of time for reading every documentation, in all cases, up until the smallest details.
I hope you understand irony.
So, @yyx990803  I think my reasoning completely stands.  I just don't understand why are you being so stubborn in this question. I noticed this let's call it phenomenon, only by accident. I presume there are plenty of other developers out there who just don't check the generated output of this program. As it's pointed out here even Jeffrey Way, missed it once. Although he's made several video series about vue.
So I'm begging you, please fix this. I don't see why would it be that hard, it wouldn't need much effort. I think you'd need to change only one line of code Is it that much? Why is another id not good enough there? If you think so, I can even make a PR fixing this, if I'm assured it's not going to be rejected.
But of course if you have a valid argument, why it would be necessary to keep this as it is, I'm persuadable. I just don't agree with you that my argument would be silly nor that including other unnecessary code would be a reason to keeping full paths as id. This is not simply unnecessary code. It's a security risk. Not a horrible one though, but still it is.

@balping your original post had not a single word mentioning that you are opening it as a "security risk", maybe you could've saved the wall of the text if you made your intention clear in the first place. And maybe, just maybe, instead of pointing out the line of code to change, you could've opened a PR and let others benefit from your work.

OK, yes not pointing out that I consider this was a fault. But as I said, I'm happy to make a PR, as I will do so.

There you go https://github.com/vuejs/vueify/pull/92


ci: run tsan in RBE

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>Description:
add shard_count for ssl_socket_test to avoid OOM in TSAN, as now we run more tests (including IPv6) in RBE.
Risk Level: Low
Testing:
Docs Changes:
Release Notes:

this needs repo setting too. @mattklein123


batch size confusion

Hi. I am a bit confused: What happen when I have an torch.utils.Dataloader with batch size = 7 BUT in the deepspeed config i have set>
"gradient_accumulation_steps": 1,
"train_micro_batch_size_per_gpu": 20,

P.S. I am performing training on single GPU P100

In this case, DeepSpeed will report a larger samples per second statistics (assuming micro batch size of 20 instead of 7). But it should not have any impact on training correctness.
What are you observing?

In this case, DeepSpeed will report a larger samples per second statistics (assuming micro batch size of 20 instead of 7). But it should not have any impact on training correctness.
What are you observing?

Just asked out of curiosity. Thank you for your response and by the way, your framework is very cool!
Also, if you have some time to explain:
I have set up parameter offloading and optimizer offloading for stage 3 in json for my model (6M params), single gpu P100, but it doesn't help increasing batch size: my original max batch size is 7 and even with offloading i get OOM for batch size bigger than 7. Could this be because of some specific architecture/dataset properties or parameter+optimizer offloading alone do not give big memory savings? P.S. here is the config i use:
{
    "gradient_accumulation_steps": 1,
    "train_micro_batch_size_per_gpu": 10,
    "gradient_clipping":1.0,
    "zero_optimization": {
        "stage": 3,
        "reduce_bucket_size": 2e6,
        "allgather_bucket_size": 2e6,
        "stage3_max_live_parameters": 1e7,
        "stage3_max_reuse_distance": 1e7,
        "stage3_prefetch_bucket_size": 5e6,
        "stage3_param_persistence_threshold": 1e5,
        "contiguous_gradients": true,
        "overlap_comm": false,
        "offload_param": {
            "device": "cpu",
            "pin_memory": true
        },
        "offload_optimizer": {
            "device": "cpu",
            "pin_memory": true
        }
    }
}

And this is the initialization part:
...
with torch.no_grad():
    self.model = RandomModel()

if torch.cuda.is_available() and torch.cuda.device_count() > 0:
    self.model.cuda()

self.optimizer = deepspeed.ops.adam.DeepSpeedCPUAdam(self.model.parameters(), lr=0.01, bias_correction=True, betas=(0.9, 0.999), eps=1e-08, weight_decay=0.0001, amsgrad=False, adamw_mode=True)
self.scheduler = deepspeed.runtime.lr_schedules.WarmupDecayLR(optimizer= self.optimizer, total_num_steps = 300, warmup_min_lr = 0.0, warmup_max_lr = 0.01, warmup_num_steps = 2, last_batch_iteration = -1)
self.model, _, _, self.scheduler = deepspeed.initialize(model=self.model, optimizer=self.optimizer, lr_scheduler=self.scheduler, model_parameters=self.model.parameters(), config='./modules/ds_config.json')
...

And this is training part
...
output = model(in_vol)
loss_m = someloss(output)
model.backward(loss_m)
model.step()
...

A 6M model should easily fit into 12GB of a single P100. This suggests OOM is due to something other than model size. ZeRO is most useful when model size is the bottleneck.  A few things to try?

Share a stack trace of the OOM
Use fp16 mixed-precision training
Take a look at #467

@Moldoteck, is this still an issue?

@Moldoteck, is this still an issue?

Thanks for asking. Main improvement where from model checkpoint and CPU activations offload. Other things didn't help too much. The fp16 training produced exploding gradients. Anyway, that's not part of the ticket, so it can be closed

Thanks for the confirmation.


Fix in portraitToPortrait condition

Thank you Sir. Pushed to Cocoapods

My pleasure. Expect a PR from Mr. Zeyad, too, as we suspect BANGing as a source
for common crashing in our beloved product. D.
On Wed, Feb 3, 2016 at 2:36 PM, Jan Gorman <<EMAIL_ADDRESS>> wrote:
Thank you Sir. Pushed to Cocoapods
—
Reply to this email directly or view it on GitHub .
-- Daniel Bauke Pracownia programistyczna Sol Niger
+48-505-607-343 http://solniger.com


Make it easy to pass keys to keyboard event triggers

The current keyboard event triggers require you to pass the explicit key as a string you want, e.g. cut.Find("input").KeyPress("a"). That works ok for normal characters, but for keys like "backspace" it can be confusing to the user.
To make it easier, we should investigate having a helper class, type, or enum, that provides the values, so we avoid the "magic strings" in the test code. E.g.:
cut.Find("input").KeyPress(Key.Enter);
cut.Find("input").KeyPress(Key.Backspace);
cut.Find("input").KeyPress(Key.A);
cut.Find("input").KeyPress(Key.a); // difference between lower case and upper case letters?

This would be great for people new to the bUnit library. It would definitely make it more accessible and other test libraries, like Selenium have a Keys class for doing this.
It is difficult to find the documentation of the special keys (without asking someone), and they are useful in testing input controls. Backspace is a relatively easy one to figure out, but whether it's Up, UpArrow, or ArrowUp is more complicated.

Cool. Great tip with Selenium, could definitely draw some inspiration from them.
Btw., I accept pull requests if you want to give this a shot @DarthPedro, and I repay with nitpicky code reviews :-)

I may be able to find some time to help build that into bUnit. It probably won't be for a couple of weeks though. And, I'm really good at ignoring comments. ;)
I didn't see a Contributing.md file so not sure if there are any particular steps to follow.

Ahh yes, I need to add one of those Contributing.md files. I actually have an issue for that #97 ...
Anyway, the basic guidelines is to use C# 8, follow the general asp.net core framework coding guidelines and create a pull request that targets the DEV branch.
But it would also be super helpful if you could help brainstorm what the API should look like.

I can handle those guidelines. :) I'll make the change.
I think the API should only be for special characters. It should be a static class with static properties for each special key, like:
    public static class Keys
    {
        public static string Backspace { get; }
        ...

Then in a test you would use it as follows:
    element.KeyPress("z");               // for text keys
    element.KeyPress(Keys.Backspace);    // special keys

I think the API should only be for special characters...

I think makes sense, since there are a lot of characters in many languages. I do wonder if the text keys should be changed to the char type instead of string? I think that it is only the special keys like backspace, up, etc., that consist of more than one character?
Have you considered the alternative to a static class, e.g. an enum? Then there could be a dedicated overload to the KeyPress method that takes the enum instead of a string. That might make the SpecialKey  enum more discoverable by users?

@egil Your questions made me remember something. .NET already has an enum definition for Keys. However it's in the System.Windows.Forms namespace and assembly (https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.keys?view=netcore-3.1). I'm not sure if you want to add that dependency to bUnit, but we can certainly use the design.

It looks as if that enum contains more options than we should need for this purpose, so I think a custom for bUnit is preferred.

Yes, that makes sense. I will create a custom enum with only special keys for bUnit. Then update the KeyPress, KeyUp, KeyDown methods with overloads which take that enum. Does that sound like a plan?

Sounds able right. I think it needs to be an overlead that replaces thekey argument in this method: https://bunit.egilhansen.com/api/Bunit.KeyboardEventDispatchExtensions.html#Bunit_KeyboardEventDispatchExtensions_KeyDown_IElement_System_String_System_String_System_Single_System_Boolean_System_Boolean_System_Boolean_System_Boolean_System_Boolean_System_String_

Right. The Keys enum would overload the string key parameter in those extension methods.

Hey @DarthPedro, should I officially assign this to you?

Yes, I was looking at what I could work on next. This looks like a good one since we already talked through the design. Assign it to me.
Do you have any changes to the design discussion above?

Enum of special keys.
Overload methods in KeyboardEventDispatchExtensions methods to support special keys.
Tests for these methods.
Update docs on sending keys.

Sounds good. Can't think of anything else.

Hey @DarthPedro, just going through all issues and you were interested in this, so I took the liberty and assigned it to you. If you want to do it just let me know.

Hey @DarthPedro, just an FYI, @duracellko beat you to it.


Missing CommonJS import Intellisense after changing file extension from .js to .ts

🔎 Search Terms
CommonJS import, require Intellisense, migrating from js to ts
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about require calls in ts files

⏯ Playground Link
https://github.com/kamil-pogorzelski-allegro/missing-intellisense-common-js
💻 Code
// a.js file
module.exports.exportedValue = 'something';

// b.js file
const { exportedValue } = require('./a');
//          ^ type: 'something'    ^ can follow this path to a.js file

//b.js renamed to b.ts
const { exportedValue } = require('./a');
//          ^ type: any            ^ cannot follow this path to a.js file

// tsconfig.json
{
  "compilerOptions": {
    "allowJs": true,
    "types": ["node"],
    "noEmit": true,
  }
}

🙁 Actual behavior
After renaming b.js to b.ts all type information about imported value vanishes, also path passed to require is no longer treated as an filepath
🙂 Expected behavior
No changes in type information / Intellisense for imported value.
Additional information about the issue
I'm maintaining CommonJS package with Node scripts written in pure JS. I've been trying to improve type safety and ergonomics on top of allowJs option by migrating to Typescript and run the scripts with tsx. I was surprised that changing file extension to ts actually reduces the type information as const ... = require(...) imports stop being treated as imports.
I'd like to avoid having to change the import syntax - since type information is available in js files then why it's not available in ts files. On top of that I'd want to avoid struggling with transpilation of import syntax and keep TS as strictly type linter.
I've checked if this is editor or TS issue by adding "typescript.tsserver.log": "verbose" option in VSCode settings and turns out that server returns empty suggestions list when invoking autocomplete action in editor.

TypeScript files only recognize import id = require("./a"); and unfortunately can't use destructuring in that form. Only JS inference presupposes that require means the CJS require function.

Only JS inference presupposes that require means the CJS require function.

Huh, TIL; for some reason I always thought bare require worked.  So does TS still require you to write import foo = require(...) under verbatimModuleSyntax for CommonJS targets?

@RyanCavanaugh Could you explain why is that? Couldn't inference check if e.g. type: "commonjs" is set in package.json or certain module and moduleResolution options for node are present?


'~' prefix is not allowed in k8s ingress path

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
kong 3.0 requiring that any regular expression begin with a ~ prefix ,but '~' prefix is not allowed in k8s ingress path, when I set ~  prefix to ingress path ,
  - host: example.com
    http:
      paths:
      - backend:
          service:
            name: example-service
            port:
              number: 80
        path: ~/api/v1/example/(.*)

it has following errors
 # ingresses.networking.k8s.io "in" was not valid:
 # * spec.rules[5].http.paths[0].path: Invalid value: "~/api/v1/example/(.*)": must be an absolute path

Expected Behavior
No response
Steps To Reproduce
No response
Kong Ingress Controller version
2.6.0

Kubernetes version
v1.20.6

Anything else?
No response

Having the same issue here, i can't update my path to match the v<IP_ADDRESS> new format
Kubernetes path must start with / https://gitlab.com/kubernetes/kubernetes/blob/60f4fbf4f25764dbd94b7f8146d927ddc684514d/pkg/apis/extensions/validation/validation.go#L417-418
it seem nginx solved it with an annotation: nginx.ingress.kubernetes.io/use-regex: "true"
https://kubernetes.github.io/ingress-nginx/user-guide/ingress-path-matching/

Yeah, that validation is pretty straightforward. For the time being, although it's labeled a temporary solution, you'll want to enable the controller-side legacy heuristic and write regular expression paths without the leading ~.
We'll need to review with the gateway team to determine a long-term strategy. I don't think we'd use an annotation because it's not possible to apply those per path. We'd have to insert the prefix into all paths on the Ingress, and that would require splitting or rewriting existing Ingresses in many cases.

As we have discussed internally, the solution is make the prefix controllable by an annotation. For example given the manifest:
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: httpbin-ingress
  annotations:
    konghq.com/regex-prefix: "/~"
    kubernetes.io/ingress.class: "kong"
spec:
  rules:
  - http:
      paths:
      - path: /~/example/[a-z0-9]*
        pathType: ImplementationSpecific
        backend:
          service:
            name: httpbin-deployment
            port:
              number: 80

Then the path matches path regular expression /example/[a-z0-9]*.  For routes that does not start with /~, the regular expression matching would not be used. The users are free to define the value of konghq.com/regex-prefix that works for them, but we assume the /~ will be used most often.
For the simplicity, the regular expression prefix could default to /~ if the annotation is not specified. The konghq.com/regex-prefix annotation would be able to override the default.


IRGen, benchmarks: add an option -align-module-to-page-size for benchmarking

The option aligns all modules to the page size. This help giving more consistent results when doing performance testing with the swift benchmark suite.
It solves the problem that benchmarks which compile down to identical code give different runtime data because of different alignment of the code within a page.
Also compile the benchmarks with the new option, if supported by the compiler

@swift-ci smoke benchmark

@swift-ci smoke test

@eeckstein I have also seen weird alignment things with environment variables. I think it is b/c the os puts the env on the stack so all subsequent stack allocations can get their alignment changed if the env variables change... but my memory might be wrong You may want to look at that as well if you are working in this area.

This shouldn't be a problem (and I didn't see this) as long as both benchmark runs have the same environment. And that's usually the case. But thanks, it's a good point.

I am saying that it would be easy to write either a wrapper script that detects bad behavior or some sort of simulation script that standardizes the environment no matter where it runs. Then we can at least have a way of identifying (or eliminating) the cause of such a regression without needing to do anything.
That being said, I don't know how important it is in practice. I think I have seen it a couple of times before in other test suites. Seems simple enough that it may be worth defining away.


Stacks 2.1 troubleshooting guide

@lgalabru Please add 2.1 epoch changes troubleshooting notes here. Thanks!

Stacks 2.1 became the default chain a few weeks ago. Hence, this issue is no longer required.


[RELEASE} First release

Description

Remaining Tasks

Related PRs in other projects (PASQAL developers only)

Additional merge criteria

Breaking changes

Checklist

[x] The title of the PR follows the right format: [{Label}] {Short Message}. Label examples: IMPROVEMENT, FIX, REFACTORING... Short message is about what your PR changes.

Versioning (PASQAL developers only)

[x] Update the version of pasqal-cloud in _version.py following the changes in your PR and by using semantic versioning.

Documentation

[x] Update CHANGELOG.md with a description explaining briefly the changes to the users.

Tests

[ ] Unit tests have been added or adjusted.
[ ] Tests were run locally.

Internal tests pipeline (PASQAL developers only)

[ ] Update and run the internal tests while targeting the branch of this PR.
If your PR hasn't changed any functionality, it still needs to be validated against internal tests.

After updating the version (PASQAL developers only)

[ ] Open a PR on the internal tests that updates the version used for the pasqal-cloud backward compatibility tests.

I think we should disable automatic squash when mergin to main. Otherwise we'll have to handle lot of conflicts in the hotfix flow. I don't know if you have control over this @Augustinio

Unfortunately it's not possible to disable squash merge for a specific branch. I have enabled rebase merge on this repository for this specific use case. Can we make sure to document in the process that one should select that type of merge when creating a new release? Thanks


Window cut on opening

Describe the bug
I am on Linux Pop!_OS 22.04 LTS (x11, with 2 external monitors).
When I open Lens, the window is outside the screen. I have to stretch the window on another screen to see the title bar.

To Reproduce
Steps to reproduce the behavior:

Open Lens

Expected behavior
The entire window is visible on the screen
Screenshots

Environment (please complete the following information):

Lens Version: 6.4.15
OS: Pop!_OS 22.04 LTS
Installation method (e.g. snap or AppImage in Linux): deb package

Does this happen every time or just the first time that you launched Lens?

Yes, every time.
By doing additional tests, it is related to the following context:
3 screens :

laptop
external screen
external screen (defined as main screen)

Lens opens all the time on my laptop screen, but the top of the window aligns with the top of the main screen, i.e. the 3.

if I line up screens 1 and 3 like this:

Lens opens correctly.

It should be aligned with the screen it opens on, right ?

yes it should thanks for the report.


Cross-Cloud Migration Doc Update

Re: Cross-Cloud Migration
Issue Description
In the text:
Install the CockroachDB version of YCSB: go get github.com/cockroachdb/loadgen/ycsb
This doesn't seem to work as a link.
Suggested Resolution
Use the current YCSB, which might need a different method considering that it was put into the workload app.

We switched to using the built-in workload in the meantime.


Custom default inline content items resolver registered through builder is not used

Brief bug description
When I register a custom default inline content items resolver and a resolver for UnretrievedContentItems through the DeliveryClientBuilder, the default implementation of the resolvers is used in the InlineContentItemsProcessor instead of the custom ones.
Expected behavior
When I register a custom default inline content items resolver or the resolver for unretrieved content items DeliveryClientBuilder, they are used in the InlineContentItemsProcessor.

will be released in https://github.com/Kentico/delivery-sdk-net/releases/tag/9.0.0


State persists even making a new compilation

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (7398b65 2023-03-30T00:09:40.119345Z)
What command(s) is the bug in?
forge script
Operating System
macOS (Intel)
Describe the bug
After changing the code inside my contracts, when running the command forge script scripts/InitialDep.s.sol:MyInitialScript --fork-url localhost --broadcast it makes the compilation but the script is run as if it is using the previous code.
E.g
Changed `console.log("previousCode");` to `console.log("currentCode");`

forge script ...
forge script scripts/InitialDep.s.sol:MyInitialScript --fork-url localhost --broadcast
zsh: correct 'script' to 'scripts' [nyae]? n
[⠢] Compiling...
[⠆] Compiling 1 files with 0.8.19
previousCode

Its compiling changes so I have no idea why it keeps using the previous codebase.
I thought of deleting cache files in case it was getting a previous state mistakenly but now I have tried deleting all the autogenerated files: ./out ./broadcast ./forge-cache etc. and nothing seems to make it work.

Hello @mattsse .
Here is a repro of my code:
//SPDX-License-Identifier: MIT
pragma solidity >=0.8.0 <0.9.0;

import "forge-std/Script.sol";
import "forge-std/console.sol";
import "../contracts/LomEventV3.sol";
import "../contracts/Minter.sol";
import "../contracts/MarketPlace.sol";
import "../contracts/Organizer.sol";
import "../contracts/Staff.sol";
import "../contracts/Registry.sol";
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
import "../test/utils/TestingSamples.sol";

contract MyInitialScript is Script, TestingSamples, EIP712Upgradeable {
    using ECDSAUpgradeable for *;
    function run() external {
        uint256 deployerPrivateKey = vm.envUint("ANVIL_PRIVATE_KEY");
        vm.startBroadcast(deployerPrivateKey);
        Registry registryTemplate = new Registry();
        console.log("Registry template deployed to: ", address(registryTemplate));
        Organizer organizerTemplate = new Organizer();
        console.log("Organizer template deployed to: ", address(organizerTemplate));
        Minter minterTemplate = new Minter();
        console.log("Minter template deployed to: ", address(minterTemplate));
        MarketPlace marketPlaceTemplate = new MarketPlace();
        console.log("MarketPlace template deployed to: ", address(marketPlaceTemplate));
        LomEventV3 lomEventV3Template = new LomEventV3();
        console.log("LomEventV3 template deployed to: ", address(lomEventV3Template));
        Staff staffTemplate = new Staff();
        console.log("Staff template deployed to: ", address(staffTemplate));

        ERC1967Proxy registryProxy = new ERC1967Proxy(address(registryTemplate), abi.encodeWithSelector(
            Registry(address(0)).initialize.selector,
            address(organizerTemplate),
            address(lomEventV3Template),
            address(marketPlaceTemplate),
            address(minterTemplate),
            address(staffTemplate)
            )
        );
        console.log("Registry Proxy deployed to: ", address(registryProxy));
        RegistryLibrary.TempOrgProxy memory organizerStruct = RegistryLibrary.TempOrgProxy({organizationName:TestingSamples.stringOrganizerName,
           creator:msg.sender});

        bytes32 beforeDigest = keccak256(abi.encode(RegistryLibrary.ORGPROXY_TYPEHASH,keccak256(abi.encodePacked(organizerStruct.organizationName)),organizerStruct.creator));

        bytes32 messageHash = ECDSAUpgradeable.toTypedDataHash(domainSeparatorRegistry, beforeDigest);
        (uint8 v, bytes32 r, bytes32 s) = vm.sign(TestingSamples.eoaPrivateKey, messageHash);
        bytes memory bytesSignature = abi.encodePacked(r, s, v);
        address preComputedOrganizer = Registry(address(registryProxy)).preComputeOrgProxy(TestingSamples.organizerName);
        address organizerProxy = Registry(address(registryProxy)).createOrganizerProxy(organizerStruct, bytesSignature);
        console.log(
            "Organizer Proxy deployed to:",
            organizerProxy,
            " and same as pre-computed: ",
            preComputedOrganizer
        );

        runEventProxy(address(organizerProxy));

        vm.stopBroadcast();
        }

        function runEventProxy(address _organizerProxy) internal {

        string memory creatorString =  Organizer(address(_organizerProxy)).getOrganizationName();
        OrganizerLibrary.TempEventProxy memory eventStruct = OrganizerLibrary.TempEventProxy({seriesName:TestingSamples.stringEventSeriesName,
           creator:creatorString,
           numberOfSubEvents:TestingSamples.numberOfSubEvents
           });

        bytes32 beforeDigestEvent = keccak256(abi.encode(OrganizerLibrary.EVENTPROXY_TYPEHASH,keccak256(abi.encodePacked(eventStruct.seriesName)),
        keccak256(abi.encodePacked(eventStruct.creator)),eventStruct.numberOfSubEvents));

        bytes32 messageEventHash = ECDSAUpgradeable.toTypedDataHash(domainSeparatorOrganizer, beforeDigestEvent);

        (uint8 v2, bytes32 r2, bytes32 s2) = vm.sign(TestingSamples.eoaPrivateKey, messageEventHash);
        bytes memory bytesSignatureEvent = abi.encodePacked(r2, s2, v2);
        address preComputedEvent = Organizer(_organizerProxy).preComputeEvProxy(TestingSamples.eventSeriesName, TestingSamples.numberOfSubEvents);

        address eventProxy = Organizer(_organizerProxy).createEventProxy(eventStruct, bytesSignatureEvent);
        console.log(
            "LomEventV3 Proxy deployed to:",
            eventProxy,
            " and same as pre-computed: ",
            preComputedEvent
        );
        console.log(
            "Current code");
    }

    // Write addresses to the fs
    function writeAddress(address _address, string memory nameOfContract) public {
        string memory path = "./deployments/initialDeployments.ts";

        string memory line = string.concat('export const ',nameOfContract,' = \"',StringsUpgradeable.toHexString(_address),'\";');
        vm.writeLine(path, line);

    }
}

This is my  foundry.toml file:
[profile.default]
src = 'contracts'
out = 'out'
libs = ['node_modules', 'lib']
test = 'test/foundry'
cache_path  = 'forge-cache'
fs_permissions = [{ access = "read-write", path = "./"}]

[rpc_endpoints]
localhost = 'http://<IP_ADDRESS>:8545/'

# See more config options https://book.getfoundry.sh/reference/config.html

Its using a contract where I have some constants:
//SPDX-License-Identifier: MIT
pragma solidity >=0.8.0 <0.9.0;

abstract contract TestingSamples {

  address eaoWallet = 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266;

  uint256 eoaPrivateKey = <ANVIL'S_PRIVATE_KEY>;
  uint256 eoaPrivateKey2 =
  <ANVIL'S_PRIVATE_KEY2>;

  bytes32 public constant organizerName =
  0x6c6162656c370000000000000000000000000000000000000000000000000000;
  // bytes32(abi.encodePacked("label7"));

  bytes32 public constant domainSeparatorRegistry = 0x4e2e440f5e0403969dfa878041e7ac69f84814e809e725f5aaa532d03f947f58;

  bytes32 public constant domainSeparatorOrganizer = 0x457963150845d99cf035ef92b880bf26a43dfcb64dd0a106d2c2b10be7ca09a1;
  ...

Which other code would you need?

@mattsse I can confirm that whats causing the bug is using the --broadcast flag. So here is a breakdown of the process to fail:

I run the script with --broadcast flag for the first time ==> Code runs smothly
I run the script with --broadcast flag for the second time (either having changes or not) ==> Code uses past codebase
I kill anvil and restart it. Run again the script with the --broadcast flag ==> Code runs smoothly

It this expected behaviour. Maybe Im not understanding what is the proper way to use broadcast...

Hello @mattsse . Foundry has been working perfectly and all of a sudden its taking me again to a previous state. I tried checking out a commit I flagged in the past as "passing tests" and its failing with a previous state. How can I debug this?

Hey,
sorry about that.

and all of a sudden its taking me again to a previous state

I'm not exactly sure what this means. Do you mean it doesn't recompile?
I appreciate the example contract, but it would help me tremendously if you could prepare a repository and instructions that I can just run and debug.

Closing, stale.


Android: Create an UI to manage device subscriptions per user

A user can see which devices he / she has registered for Push Notifications and can unregister them as well.

Project stopped


Using custom scncamera issues while recording video deformation occurs

Hi,
Let me explain  you clearly the issue.
Create a custom camera and set it to a node.
Let cameraNode = SCNNode()
cameraNode.camera = SCNCamera()
cameraNode.camera.useOrthographic =true
zfar= 30 / zNear = 1
For the orthographicScale =3.8
//add the cameraNode to the scene root Node
then make the position it to x0,y0,z17
sceneView pointOfView = cameraNode
When you record the video it cut when ur Animoji is in front your face.
Swift 4.2 /
If you need more precision let me know thanks
SceneView.pointOfView = cameraNode
When you using ARFaceTracking as config, set the background to any color you place a character with blendshapes to mimic.

Which iOS device are you using?

issue is coming on iPhone 7 but working good on iPhone X


<EMAIL_ADDRESS>

A4 Initial Commit Submission.

Hi all - some brief comments on my A4 submission (specifically, on what I've implemented for the A4 requirements) before our code review.

I've added a CareKit implementation to track adherence for three tasks (two professional development exercises, one survey task). The survey task, though, feels disjointed - I have a checklist (to help users track when in the day they should be filling out the survey), and a survey task (to provide easy access to start the survey), and it seems as though there should be an easy way to blend the two of these interfaces into one unified view. I'm hoping we can discuss this more during our code review.
I've added two CareKit charts: one for tracking survey completion, the other for tracking step count throughout the day. (Though the step count one has pretty limited testing, as everything has been done in simulator so far).
My new Firestore rules are as follows (and they have been added/deployed to the Firestore backend):

rules_version = '2';
service cloud.firestore {
  match /databases/{database}/documents {
    // Make sure the uid of the requesting user matches name of the user
    // document. The wildcard expression {userId} makes the userId variable
    // available in rules.
    match /users/{userId} {
      allow read, update, delete: if request.auth != null && request.auth.uid == userId;
      allow create: if request.auth != null;
    }
  }
}

EHRs are not a required feature of this application - I would like to verify with you that this is not a required portion of my A4 submission.
I've updated the "Your Team" screen to include contact card information for member of the research team, replacing the design I had previously used which was my A1 submission.


pip install pkgcheck fails

I tried to start using this in https://github.com/flammie/flammie-overlay but it fails in pip installing pkgcheck https://github.com/flammie/flammie-overlay/runs/7538103278?check_suite_focus=true , I tried in a venv with same results:
$ python -m venv pkgcheck
$ cd pkgcheck/
$ . bin/activate
$  pip install pkgcheck
Collecting pkgcheck
  Downloading pkgcheck-0.10.11.tar.gz (474 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 474.1/474.1 KB 2.8 MB/s eta 0:00:00
  Installing build dependencies ... error
  error: subprocess-exited-with-error

  × pip subprocess to install build dependencies did not run successfully.
  │ exit code: 1
  ╰─> [66 lines of output]
      Collecting wheel
        Downloading wheel-0.37.1-py2.py3-none-any.whl (35 kB)
      Collecting setuptools
        Downloading setuptools-63.2.0-py3-none-any.whl (1.2 MB)
           ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.2/1.2 MB 6.2 MB/s eta 0:00:00
      Collecting chardet
        Downloading chardet-5.0.0-py3-none-any.whl (193 kB)
           ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 193.6/193.6 KB 6.7 MB/s eta 0:00:00
      Collecting lazy-object-proxy
        Downloading lazy_object_proxy-1.7.1-cp310-cp310-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_17_x86_64.manylinux2014_x86_64.whl (62 kB)
           ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 62.3/62.3 KB 7.1 MB/s eta 0:00:00
      Collecting lxml
        Downloading lxml-4.9.1-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl (6.9 MB)
           ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 6.9/6.9 MB 4.2 MB/s eta 0:00:00
      Collecting pathspec
        Downloading pathspec-0.9.0-py2.py3-none-any.whl (31 kB)
      Collecting tree-sitter>=0.19.0
        Downloading tree_sitter-0.20.0.tar.gz (116 kB)
           ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 116.2/116.2 KB 5.7 MB/s eta 0:00:00
        Installing build dependencies: started
        Installing build dependencies: finished with status 'done'
        Getting requirements to build wheel: started
        Getting requirements to build wheel: finished with status 'done'
        Preparing metadata (pyproject.toml): started
        Preparing metadata (pyproject.toml): finished with status 'done'
      Collecting snakeoil~=0.9.10
        Downloading snakeoil-0.9.10.tar.gz (264 kB)
           ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 264.4/264.4 KB 3.8 MB/s eta 0:00:00
        Installing build dependencies: started
        Installing build dependencies: finished with status 'done'
        Getting requirements to build wheel: started
        Getting requirements to build wheel: finished with status 'error'
        error: subprocess-exited-with-error

        × Getting requirements to build wheel did not run successfully.
        │ exit code: 1
        ╰─> [17 lines of output]
            Traceback (most recent call last):
              File "/tmp/tmp__nukgk7_in_process.py", line 363, in <module>
                main()
              File "/tmp/tmp__nukgk7_in_process.py", line 345, in main
                json_out['return_val'] = hook(**hook_input['kwargs'])
              File "/tmp/tmp__nukgk7_in_process.py", line 130, in get_requires_for_build_wheel
                return hook(config_settings)
              File "/tmp/pip-build-env-o21ixm7o/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 177, in get_requires_for_build_wheel
                return self._get_build_requires(
              File "/tmp/pip-build-env-o21ixm7o/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 159, in _get_build_requires
                self.run_setup()
              File "/tmp/pip-build-env-o21ixm7o/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 174, in run_setup
                exec(code, locals())
              File "<string>", line 10, in <module>
              File "/tmp/pip-install-brbg8pcx/snakeoil_d720d790c0bc4e699cddb37b29729708/src/snakeoil/dist/distutils_extensions.py", line 54, in <module>
                raise ImportError('this module is only meant to be imported in setup.py scripts')
            ImportError: this module is only meant to be imported in setup.py scripts
            [end of output]

        note: This error originates from a subprocess, and is likely not a problem with pip.
      error: subprocess-exited-with-error

      × Getting requirements to build wheel did not run successfully.
      │ exit code: 1
      ╰─> See above for output.

      note: This error originates from a subprocess, and is likely not a problem with pip.
      WARNING: You are using pip version 22.0.4; however, version 22.2 is available.
      You should consider upgrading via the '/home/flammie/tmp/pkgcheck/bin/python -m pip install --upgrade pip' command.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× pip subprocess to install build dependencies did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.
WARNING: You are using pip version 22.0.4; however, version 22.2 is available.
You should consider upgrading via the '/home/flammie/tmp/pkgcheck/bin/python -m pip install --upgrade pip' command.

I am a new user so it is possible I did something very basic wrong.

Can you please retry, as we had big problems with the whole stack installing through pip, which were all fixed hopefully.

I think it has worked, I get an actual error report in the ci run.


Cannot establish connection via Model due to invalid argument

So I've set up Laravel 5.4 and Mongo 3.4.1 database and using Jenssegers extension. I'm trying to test if the application can connect to database using the model by typing php artisan tinker and then querying App\Patient::all(). However, the system returns the following error, and I'm close to desperation because it refuses to work with MongoDB :
TypeError: Argument 1 passed to Jenssegers\Mongodb\Query\Builder::__construct() must be an instance of Jenssegers\Mongodb\Connection, instance of Illuminate\Database\MySqlConnection given, called in /Users/janisozolins/www/laravel/patients/vendor/jenssegers/mongodb/src/Jenssegers/Mongodb/Eloquent/Model.php on line 503
When I checked the Model.php file on line 503, there is a function:
protected function newBaseQueryBuilder()
    {
        $connection = $this->getConnection();

        return new QueryBuilder($connection, $connection->getPostProcessor());
    }

My model file looks like this:
<?php

namespace App;

use Jenssegers\Mongodb\Eloquent\Model as Model;

class Patient extends Model {

    protected $connection = 'mongodb';
    protected $table = 'patient';

}

My connection to the database:
'mongodb' => [
            'driver' => 'mongodb',
            'host' => env('DB_HOST', '<IP_ADDRESS>'),
            'port' => env('DB_PORT', 27017),
            'database' => env('DB_DATABASE', 'testdb'),
            'username' => env('DB_USERNAME', 'testdb'),
            'password' => env('DB_PASSWORD', 'testdb'),
            'options' => array(
                'db' => 'testdb' // sets the authentication database required by mongo 3
            )
        ],

@JanisOzolins can you tell me what you found wrong? how you fixed this?

@acidjazz I had several issues I was dealing with while trying to establish the connection between Laravel and my MongoDB. But the two main faults were:

wrong authentication database in your config/database.php file
incorrectly extended User class

Currently, this is how my model class looks like:
<?php

namespace App;

use Jenssegers\Mongodb\Eloquent\Model as Eloquent;

class User extends Eloquent {
    protected $collection = "users"; // specify the name of your collection within your Mongo database
}


Blossom bougon: add startOpen prop to popover

Some popovers in the community site need the ability to be open on page load, so this adds that functionality

:rocket: PR was released in v0.19.0 :rocket:


Fix typo

Option name was typo'd.

@GoodForOneFare, I'm just a random with no merge access, so you'll have to merge it or ping someone else to. 😄

@GoodForOneFare, I'm just a random with no merge access, so you'll have to merge it or ping someone else to. 😄


Find / use alternative mail service

Google doesn't want us to use our own mailserver ...
All outgoing traffic to port 25 (SMTP) is blocked.
Most outgoing traffic to port 465 or 587 (SMTP over SSL) is blocked, except for known Google IP addresses.
Traffic that uses a protocol other than TCP, UDP, and ICMP is blocked, unless explicitly allowed through Protocol Forwarding.

Sendgrid Emailing works. Text template needs to be adjusted, registration text vs confirmation text


Torch installs without itorch

For some reason it seems that itorch isn't installed together with torch (I believe that it was installed in earlier versions):
max@max-UX31A:~/$ sudo docker run --rm -it -p 8888:8888 kaixhin/cuda-torch
Unable to find image 'kaixhin/cuda-torch:latest' locally
latest: Pulling from kaixhin/cuda-torch
bbe1c4256df3: Pull complete
911d09728ffd: Pull complete
615765bc0d9f: Pull complete
a3ed95caeb02: Pull complete
f6c40ea017da: Pull complete
a53854637f3f: Pull complete
1cd0c8506d8b: Pull complete
687b23b1ba76: Pull complete
73a547b0c44e: Pull complete
964cc0d8070b: Pull complete
c146c215733f: Pull complete
d0ba2846eec7: Pull complete
Digest: sha256:2e22615195b4ebb19bd633a9afca997e083625b70e457f2ad1a847f96aec7ad7
Status: Downloaded newer image for kaixhin/cuda-torch:latest
root@e80bdeb5c974:~/torch# ls
CMakeLists.txt  README.md  clean.sh  exe    install       install.sh  test.sh
LICENSE.md      build      cmake     extra  install-deps  pkg         update.sh
root@e80bdeb5c974:~/torch# itorch
bash: itorch: command not found

Since the jupyter notebook is one of the most natural ways for interacting with a docker container I believe that it would be beneficial to add itorch to the build or have separate builds that includes the itorch

iTorch did use to be installed (as I remember someone asking me to fix a bug with it earlier). Not entirely sure what's changed, and unfortunately I don't use Jupyter notebooks so I'll need some help on this one. Do you have a solution? If so, then depending on how much extra is installed (i.e. the size of the Docker images), I'll either add it to kaixhin/torch or create kaixhin/itorch.

Here's an alternative to the torch\Dockerfile that seems to work. I've simply added pip and installed through it the jupyter notebook. It seems to get the right foundations for iTorch to be installed.
# Start with Ubuntu base image
FROM ubuntu:14.04
MAINTAINER Kai Arulkumaran<EMAIL_ADDRESS>
# Install git, apt-add-repository and dependencies for iTorch
RUN apt-get update && apt-get install -y \
  git \
  software-properties-common \
  ipython3 \
  libssl-dev \
  libzmq3-dev \
  python-zmq \
  python-pip

RUN pip install notebook

# Run Torch7 installation scripts
RUN git clone https://github.com/torch/distro.git /root/torch --recursive && cd /root/torch && \
  bash install-deps > /dev/null && \
  ./install.sh

# Set ~/torch as working directory
WORKDIR /root/torch

# Export environment variables manually
ENV LUA_PATH='/root/.luarocks/share/lua/5.1/?.lua;/root/.luarocks/share/lua/5.1/?/init.lua;/root/torch/install/share/lua/5.1/?.lua;/root/torch/install/share/lua/5.1/?/init.lua;./?.lua;/root/torch/install/share/luajit-2.1.0-beta1/?.lua;/usr/local/share/lua/5.1/?.lua;/usr/local/share/lua/5.1/?/init.lua'
ENV LUA_CPATH='/root/.luarocks/lib/lua/5.1/?.so;/root/torch/install/lib/lua/5.1/?.so;./?.so;/usr/local/lib/lua/5.1/?.so;/usr/local/lib/lua/5.1/loadall.so'
ENV PATH=/root/torch/install/bin:$PATH
ENV LD_LIBRARY_PATH=/root/torch/install/lib:$LD_LIBRARY_PATH
ENV DYLD_LIBRARY_PATH=/root/torch/install/lib:$DYLD_LIBRARY_PATH
ENV LUA_CPATH='/root/torch/install/lib/?.so;'$LUA_CPATH

Also, in the docker hub you may want to hint that the image should be initiated with a port opened and the notebook with --ip='*', e.g.
> sudo docker run --name=tag -p 8888:8888 -it --rm kaixhin/iTorch
> jupyter notebook --ip="*" --no-browser

This will give you a notebook available at localhost:8888. As I've just started to explore Torch I'm not that familiar with all its features, looking at this there may be a few more additions worth considering although the iTorch docs indicate that this should suffice.

Looks good to me. Updated the images and readmes - could you please check the latest build of kaixhin/torch that I just set up?

Works fine. I just realized that if you specify the IP to <IP_ADDRESS> the following warning disappears:

[W 12:58:14.645 NotebookApp] WARNING: The notebook server is listening on all IP addresses and not using encryption. This is not recommended.
[W 12:58:14.646 NotebookApp] WARNING: The notebook server is listening on all IP addresses and not using authentication. This is highly insecure and not recommended.

I therefore suggest changing the instructions to:
"To use Jupyter/iTorch open up the appropriate port. For example, use docker run -it -p 8888:8888 kaixhin/torch. Then run jupyter notebook --ip="<IP_ADDRESS>" --no-browser to open a notebook on localhost:8888."
Another thing that may be worth considering is adding ipywidgets as this causes an unnecessary warning at start:

[W 12:56:40.753 NotebookApp] ipywidgets package not installed.  Widgets are unavailable.

The addition necessary is:
RUN  pip install ipywidgets
Thanks for the terrific work with these docker-files.

Thanks for the update - think we can close this issue now :)


[Exam] Copy/ Import existing exam

Is your feature request related to a problem?
Currently, exams always have to be created from scratch, although many parameters (such as exercises/ types, registered students, ...) may stay the same. This makes testing exam mode slower, as everything has to be set up over again, even though there may be little changes.
Describe the solution you'd like
A button to re-use an existing exam and select some new parameters (mostly times/ dates) for import. The registered students and exercise groups with the exercises should stay the same. Then only student exams have to be generated to start testing.
Describe alternatives you've considered
Manually creating the exam and importing everything else (exercises) + registering students again. This is the current workflow while testing different exam features.

Duplicate of https://github.com/ls1intum/Artemis/issues/3848


InfluxDB V2 HTTPS cloud access

Making code to work over HTTPS connection.
Server's fingerprint is required to be specified manually.
Fixed a bug where subsequent writes() gets failed due to missing 'Token' header

This will not work for ESP32.  There is not #include for WifiClientSecure.h and WifiClientSecure for ESP32 doesn't have ability to validate certificate with fingerprint.

This will not work for ESP32. There is not #include for WifiClientSecure.h and WifiClientSecure for ESP32 doesn't have ability to validate certificate with fingerprint.

Hmm.. Then, this should be under #ifdef I believe?

conditionally compiling out ESP32 for this feature


scrollable recents

It would be nice if the recents were scrollable so you could have a longer list than  the width/height of the screen.

This is a highly requested feature that is on my to-do list for a future version.

This has been implemented and will be released as part of the 2.1 release.


linking error with avtMfem

Describe the bug
[  0%] Built target tess2
[  0%] Built target cognomen
[  0%] Built target bow
[  0%] Built target nek5000_interp
Scanning dependencies of target visitcommon
[  0%] Built target MRUCache_test
[  0%] Building CXX object common/CMakeFiles/visitcommon.dir/misc/ConfigureInfo.C.o
[  1%] Built target vtkqt
[  1%] Linking CXX shared library ../../lib/libavtmfem.dylib
Undefined symbols for architecture x86_64:
  "DebugStream::Level1()", referenced from:
      avtMFEMDataAdaptor::LowOrderMeshToVTK(mfem::Mesh*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineGridFunctionToVTK(mfem::Mesh*, mfem::GridFunction*, int, bool, bool) in avtMFEMDataAdaptor.C.o
  "DebugStream::Level5()", referenced from:
      avtMFEMDataAdaptor::LowOrderMeshToVTK(mfem::Mesh*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineMeshToVTK(mfem::Mesh*, int, int, bool) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::LowOrderGridFunctionToVTK(mfem::GridFunction*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineGridFunctionToVTK(mfem::Mesh*, mfem::GridFunction*, int, bool, bool) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineElementColoringToVTK(mfem::Mesh*, int, int) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineElementAttributeToVTK(mfem::Mesh*, int) in avtMFEMDataAdaptor.C.o
  "DebugStream::Stream1(char const*, int)", referenced from:
      avtMFEMDataAdaptor::LowOrderMeshToVTK(mfem::Mesh*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineGridFunctionToVTK(mfem::Mesh*, mfem::GridFunction*, int, bool, bool) in avtMFEMDataAdaptor.C.o
  "DebugStream::Stream5()", referenced from:
      avtMFEMDataAdaptor::LowOrderMeshToVTK(mfem::Mesh*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineMeshToVTK(mfem::Mesh*, int, int, bool) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::LowOrderGridFunctionToVTK(mfem::GridFunction*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineGridFunctionToVTK(mfem::Mesh*, mfem::GridFunction*, int, bool, bool) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineElementColoringToVTK(mfem::Mesh*, int, int) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineElementAttributeToVTK(mfem::Mesh*, int) in avtMFEMDataAdaptor.C.o
  "VisItException::SetThrowLocation(int, char const*)", referenced from:
      avtMFEMDataAdaptor::LowOrderMeshToVTK(mfem::Mesh*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineGridFunctionToVTK(mfem::Mesh*, mfem::GridFunction*, int, bool, bool) in avtMFEMDataAdaptor.C.o
  "VisItException::Log()", referenced from:
      avtMFEMDataAdaptor::LowOrderMeshToVTK(mfem::Mesh*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineGridFunctionToVTK(mfem::Mesh*, mfem::GridFunction*, int, bool, bool) in avtMFEMDataAdaptor.C.o
  "InvalidVariableException::InvalidVariableException(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >)", referenced from:
      avtMFEMDataAdaptor::LowOrderMeshToVTK(mfem::Mesh*) in avtMFEMDataAdaptor.C.o
      avtMFEMDataAdaptor::RefineGridFunctionToVTK(mfem::Mesh*, mfem::GridFunction*, int, bool, bool) in avtMFEMDataAdaptor.C.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [lib/libavtmfem.dylib] Error 1
make[1]: *** [avt/MFEM/CMakeFiles/avtmfem.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[  1%] Linking CXX shared library ../lib/libvisitcommon.dylib
[  3%] Built target visitcommon
make: *** [all] Error 2

@JustinPrivitera FYI: These are the kind of things we don't see on Linux, but will see on macOS and Windows. (they are things I missed)
@biagas are these the issues you hit on Windows?
I also hit an include issue. I will put up a PR for that into the RC.
To Reproduce
Desktop

OS and version: macOS 15
VisIt Version: 3.3RC (working towards 3.3.1)
Server info (if applicable):

This should have been fixed with my merge last night. #18062

But I didn't put my changes on RC yet. (The CMakeLists.txt changes have VTK9 support that shouldn't go on RC).

avtblueprint needs to link with mfem, and both avtmfem and avtblueprint need to link with visitcommon.

Thanks! I can get them on RC, sound good?

Yes, thanks!

Fixed by #18086


build: canonicalise source path before use (NFC)

If the source tree is checked out into a patch behind a symlink, the path can
get expanded and fail tests.  Be more resilient by canonicalise the path prior
to use.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @gottesmm @Rostepher

@swift-ci please smoke test


A fully onchain Counter / Controller

Recursion is a milestone update of the ordinal theory, which has brought a lot of interesting applications.
However, in terms of the ecological construction of ordinal theory, there is currently a lack of a fully onchain counter/controller, such as an onchain counter which can record the times of the inscriptions that others using your inscriptions by recursion technology; Also if there is a fully-chained controller or switch that allows the owner of the inscription to choose to be recursively enable/unable; these will greatly enhance the rights of the owner of the inscription, who not only has the full ownership of an inscription ，but also fully has the right to use and control the inscription.
Do you like the idea?

Technically seems it's difficult to develop such product or standard. Maybe it's necessary to add such function at protocol level ?

The current design for parent-child collections, recursion and reinscriptions do not consider multi-party participant controls.
There are use cases for this such as art collectives and git-like repo orgs/teams as approved contributors.
There are ways to send onchain signals that effectively create allow-lists of addresses that can formally be part of "collections" and also to be removed from collections.
I am not sure if this has been discussed anywhere but happy to continue the discussion here.

I'm not enthusiastic about implementing DRM for inscriptions.


add gitstatus command

Also amended 'main' in package.json.

Very cool! Nothing to add here, merging!

Published to NPM as v1.0.6. Thanks a lot!

You're welcome, thanks for merging and publishing to npm, makes life easier.


Redirects

Implementation PR: https://github.com/withastro/astro/pull/7067/
Stage 3 proposal: https://github.com/withastro/roadmap/pull/587
Stage 2 proposal: https://github.com/withastro/roadmap/issues/466

Needs

API reference docs
? - This feature is most useful in SSG, so maybe it should go there? This feature does work in SSR as well, but you can already do redirects in SSR (manually) so it's not adding a new capability, just making one easier.

How it works
You can add a top-level redirects config with keys that are from routes and the values are the to route. For example:
import { defineConfig } from 'astro/config';

export default defineConfig({
  redirects: {
    '/api': '/docs/api'
  }
})

API reference is being reviewed in the current PR. Is there an additional place we should document this feature?

Just making a quick note that in addition to docs for redirects explaining how to use them, we'll need to update:

api-reference for Astro.redirect() which currently describes it as an SSR-only feature
the SSR page which has an example of Astro.redirect() as an SSR feature (e.g. redirecting after logging in) - this is not incorrect usage, but it's listed here as an SSR feature which is now not correct.

Also, I think the best place for this content for now at least is on the Routing page. Since this is mostly an SSG feature (and we don't have an SSG page) but also can be used in SSR, and since it relates to (re)routing, that's my suggestion for where this content can go!

Yes, if it's no longer exclusively an SSR feature, which the RFC seemed to say. I think this can go with the new redirects content on the routing page.


No metadata added to Accessible Folders in New-AcAampConfiguration

No metadata added to Accessible Folders in New-AcAampConfiguration

Moved CreateInstanceFromClassName objects into Begin section.
$AccessibleFile = $Configuration.CreateInstanceFromClassName("AM.File")
$AccessibleFolder = $Configuration.CreateInstanceFromClassName("AM.Folder")
$DigitalCertificate = $Configuration.CreateInstanceFromClassName("AM.DigitalCertificate")


regressiontest Regression_test_similar_query_boolean

Proposed changes
Issue Number: close #17705
Problem summary
Add a regression test about issue#17705
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
This issue existed in version 1.2.2, but it seems to have been resolved in master branch.

run buildall

so the issue is not a problem for master?

Yes, the results just tested show that there is no problem on the master branch.

run buildall

run buildall

run buildall

Pls rebase and run build all again

run buildall


Release with the memory leak fixed?

Overdrive: 0.2.2
Package manager (SPM, Carthage, CocoaPods, Manual): CocoaPods
Xcode: 8.3.1
Platform: iOS
Hi @arikis,
The project I'm working on requires Overdrive to work, and while I know I'm not supposed to be asking for ETAs around here, I wanted to know if we can just release a minor bumped version of Overdrive (maybe 0.2.3) with mainly the task observer memory leak fixed. I would be happy to help in any way I can.

Hi @biocross, I wanted to release a 0.3 for some time now, but I've been putting in of due to some new features I want to introduce. But, I decided to put those new features in some next version.
Version 0.3 will be released today then :)

Really great to hear! Thanks a lot :D

Hey @arikis ,
Just a reminder, looks like you didn't push 0.3 to the master cocoapods specs repo.

Pushing it at the moment, updating pods always take some time :)

:)

Should be up now.

Got it. Thanks.


MAID-2527 refactor/snippet: changes MData entries forEach to listEntries

r? @krishnaIndia
(maidsafe_highfive has picked a reviewer for you, use r? to override)

formatting fixed

Let's merge this only once we have a version of the browser which actually supports the changed API.

PR is obsolete because it's for deprecated API. Closing.


CI: fix missing /nsls2/ dirs for continuous integration testing

This update only affects the continuous integration setup for Azure Pipelines, it does not affect the production code.

@AbbyGi, @ambarb, this is to fix the CI after the RE metadata location update via bc98e559a0e4232a60ac0b6cda93313a949c4e36.

I went ahead and merged it to have the tests passed.


Specify steps needed when updating the kit from version .beta-9 to .beta-10

Addresses: #549
Trello ticket: https://trello.com/c/6pfdWXK5/1221-1-document-changes-between-beta-9-and-beta-10-in-prototype-kit-changelog

no not broken. inline for $(document).ready(function () { and block for the actual line.
alternative is to rewrite the sentence to be

modify app/assets/javascripts/application.js file to be:

$(document).ready(function () {
   window.GOVUKFrontend.initAll()
})

do i rewrite the last list item as suggested above?

updated


GitHub Actions: use MacOS runner

macOS hosted runners have more cores and more ram and allow graphics HW acceleration.
See: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources

Despite the more cores and ram the build takes far longer on the macOS runner. Closing this.