Users with same display name breaks @-mention

It's possible for people to have the same display name, while having different email addresses. This becomes more likely in larger companies.
For example:

John Doe<EMAIL_ADDRESS>John Doe<EMAIL_ADDRESS>
@-mentioning @John Doe has unpredictable results. This is because display name is used to match the name:
https://github.com/zulip/zulip/blob/e95739961f28916011d36fe58cc4eb45a20e20a3/zerver/lib/bugdown/__init__.py#L794
        user = db_data['full_names'].get(name.lower(), None)
        if user is None:
            user = db_data['short_names'].get(name.lower(), None)

Since display name is not a unique attribute it would be much better to use the email address to identify the user as part of the send_message UI & API.

Agreed!  I've for a long time wanted to change the way the display works for @-mentions to use either the email address or perhaps better the user ID.

I am working on this.

Cool, @ragsagar, which implementation approach are your attempting?

We show the same autocomplete prompt, when user selects Iago, instead of @**Iago** we will fill<EMAIL_ADDRESS>What do you think?

Rough implementation of what I have in mind.

Cool.  Haven't tested that, but is the idea that it would still display the same way, just you'd and/or the autocomplete would type different syntax?  I think that makes sense.
One change that it might make sense to do while we're at it is moving to identifying the user by user ID, rather than email, so that we can better support users changing email addresses.  To support that, some options for this would be:

do some fancier syntax like @<1235415=Tim Abbott> that encodes the user ID but also has the display name handy in it.
Make our editor a rich text editor so we can just have the thing you see look like the pretty bubble you'll get out the other end (e.g. https://github.com/zulip/zulip/issues/595)
Something else

One change that it might make sense to do while we're at it is moving to identifying the user by user ID, rather than email, so that we can better support users changing email addresses.

I didn't consider that scenario. Is there a way for an user to change his email id now?

do some fancier syntax like @<1235415=Tim Abbott> that encodes the user ID but also has the display name handy in it.
Make our editor a rich text editor so we can just have the thing you see look like the pretty bubble you'll get out the other end

I feel we should go with rich text editor, After filling in user should only see @TimAbott and should be able to remove whole @TimAbott we fill in with a single backspace.
@<1235415=Tim Abbott> will look like gibberish to a non technical person. Changing the user also will be not very user friendly.

Is there a good rich text editor library we can use for this?

I went through some examples of jquery TextExt plugin. But doesn't look like it is exclusively for our purpose (tag between texts). Were you able to find any similar examples?

There is another thing to fix as well. Suppose i want to @-mention a person with name 'Vaibhav' and say there exist 6-7 people with name vaibhav but different surnames. Since we display only 5 results, there is possibility of missing out the name of required person from the list. I can also use email of the person for the same but usually people find it easier to remember names over email ids.
To solve this, instead of stopping the search by name at whitespace, we can continue it and start searching on surname after name. We can use special characters (like underscore) to indicate this.
Ex: Vaibhav_Singhal would indicate search for user with name Vaibhav Singhal.
I have also tried to find a rich text editor plugin for our case but couldn't. Maybe we can try other approach, like the fancier syntax. If we find one later, we can modify our code then.

@vabs22, yeah, that problem annoys me as well.   I worry a bit about the whitespace approach, because if for whatever reason you don't want to @-mention someone, it's pretty annoying to have to use escape to get out of autocomplete.
One reasonable option would be to see if there's a way we can make it show more than 5 options if there are several exact matches (or scroll through them, or something).  Another would be to ignore whitespace in full names when matching (so typing e.g. vaibhavSing would let one filter to just you, even if that didn't happen to be part of your email address).
Assuming those sound reasonable, can you open new GitHub issues for those 2 approaches?  I think ideally we'd do both.

Sure! both of these approaches are a better solution.

One new development here is that our mention autocomplete now deals with white space better.

Yeah; that helps a lot with ensuring folks pick out the user they want.  Now we just need to figure out how to handle exact name collisions; either by preventing them (you can't change your name to a name someone already has) or changing how we encode @-mentions (which would probably be reasonable as well).

@timabbott  Let's take best practices from Vkontakte social netowrk (the biggest in Russia).

Preventing name change isn't reasonable, as we have first name + last name, not nickname like on github.
Show only 5 results can cause problems in large streams. We can make thin scroll or write on the top of suggestion that you can navigate up and down with tab button.
About @-mentions encoding, imho, best way is: @id12 (Tim Abbot), but when user see the same names in suggestions we need to show emails, as ids doesn't make any sense for user. Actually having nickname solves the collisions easier, but we haven't. Therefore we should apply this memory-consuming solution (get id and email).

Let's take best practices from Vkontakte social network ... not nickname like on github.

err, Vk has nicknames like GitHub ... https://vk.com/jayvdb
(It is under settings)

On using email addresses, I see they appear a lot in the user interface so that would at least be consistent, and noticed https://github.com/zulip/zulip/issues/6902 which is a similar UI issue.


WithArgs xml doc mentions only ContainerResource is supported

This is more a documentation enhancement than a bug.
WithArgs does work with ProjectResource but xml docs states:
Adds the arguments to be passed to a container resource when the container is started.

Can you send a PR @g7ed6e ?


No net40 target after 0.15.2

Is there a reason why net40 target was removed? I see that there is no specific code for net45 so I guess net40 can also be included in nuget package.

Microsoft doesn’t support net40 anymore.  I put net45 on purely to make the net standard compatibility easier to manage.  They still support net35 so that’s why that is there.

Thanks for the reply. Yes I know but I think net40 is still widely used, most essential libraries still have it in their nuget package but I understand it's hard to manage all this different compatibility levels. I could also use net35 of course, but I prefer the more performant Directory.EnumerateFiles over Directory.GetDirectories (net35).
Nevermind the issue, I can still build for net40 when I add it to TargetFrameworks in csproj.

It doesn’t bother me if you want to add net40 in a PR.
Having net35 is already a pain because of mono deprecating it.


Support Mac installation

It would be great to be able to install pytriton on Macs for ease-of-development. Even with the lack of CUDA support for Macs, being able to develop using only the CPU would be a real time saver.
As a far reaching stretch goal, being able to use mps on Macs would also be great.
As of now pip install nvidia-pytriton throws an error being unable to install cuda-python on my machine.
My Machine:

Apple M2 Max
32GB Memory
Year - 2023
Python - 3.11.6

As far as I understand, that update should come from triton-inference-server, but since it's supposed to run on an NDVIDIA GPU I don't think it'll happen.
What you can do is run on a docker emulated image running docker image with flag --platform linux/amd64. It will run on CPU and slower but you'll be able to debug.
Hope that helps

Thanks for the quick fix, that's what I'll start with.
Is it possible to decouple pytriton from the cuda-python library?

The cuda-python is required by tritonclient package which is PyTriton dependency. As for now we are not able to remove it, but if you would run installation inside the Docker container with Ubuntu 22.04 there should be no issue despite the CUDA is not available in the system.

Thanks all!

@jkosek It seems cuda-python is optional for tritonclient, is it possible to make it optional in pytriton?

@martin-liu you are correct. We are going to fix dependencies on our side to avoid cuda dependency at the moment. Thanks!

@jkosek, fantastic! I'm truly appreciative of your efforts to resolve this issue. This adjustment will significantly enhance local development experience on Mac. Thank you!

@martin-liu the change has been applied on the main branch and will be available with next wheel release.

@jkosek Great! Thank you very much!

@jkosek may I know what's the ETA for next release?

@jkosek Thank you for the update, I'll check out!

@jkosek Seems there are no wheels available that support macOS. Are there any specific issues or blockers preventing the creation of macOS wheels?

Hey @martin-liu . At the moment we are not able to provide a macOS wheel. The PyTriton contains a Triton binaries which are bounded to the Linux OS and specific version of GLibc. In such case the only option we can suggest is to use Docker image, ex. Ubuntu, on macOS.

@jkosek got it, thanks


[hdom] ideas/plans about typing hdom?

It will be nice to have hdom typed, something like:
type HElement = (
  ["div", ({ style? : string } | HElement)?, ... HElement[]] |
  ["canvas", ({ width? : number, heigth? : number } | HElement)?, ... HElement[]]
)

For this, emmet style tags like "tag#id.class1.class2" needs to be sacrificed, but I think is a good tradeoff. Do you have any plans about this?

Hi @nahuel - that's an issue which is partially been discussed in the past and I'm of several minds about it. First some refs to some prev context:

https://github.com/thi-ng/umbrella/issues/140#issuecomment-544494226
https://github.com/thi-ng/umbrella/issues/140#issuecomment-544555688 (& onwards)

To add: If hiccup/hdom trees would only consist of simple elements, then your above approach could of course work (by now, pre-TS3.7 we couldn't even do recursive tuple types...), but it's also very common for component trees to include:

ES6 iterables (not just arrays) - which should then be typed/have generics as well
Component objects with hdom's ILifecycle interface
Types which implement the IDeref interface - either as element children or as attrib values
Lazy evaluated component functions in this form: [func, ...arbitraryArgs]

Also, currently any other value is allowed in the tree, but will be cast to string...
Finally, there's currently a lot of work underway on a new version/alt approach under the working title hdom2020 and you can follow along with progress on this feature branch and/or on the #umbrella channel in our Discord (link in main readme header). Be warned though, there're no docs yet and only a single example published so far... FWIW the approach in that new version will be more typed, but is also somewhat different (not entirely) to the current hdom...
I think if you (and others) really struggle with the untyped approach for your components, why don't use the "Call method B" described in that first comment linked above (which will give you ful type safety) or create some typed helper functions to at least type check parts of your tree in your above mentioned manner. IMHO either of these two (or a combination) would be much more pragmatic and useful than attempting to define a single type definition which covers all of the above supported cases (which actually are used in the wild) and then also still plays nicely with all sorts of custom component types one might define.

Thanks for the detailed response, I will go with the typed helper functions approach!

Cool. Also, am really not meaning to close the door on a more typed approach, just wanting to emphasize that:

an alternative is slowly emerging (with yet not fully understood consequences)
it's essential to fully support the above mentioned value types which can be or be included in a hypothetical HElement
the loss of Emmet-style tags would be potentially problematic for various people (incl. me), but possibly livable if there're some real tangible other benefits (which I personally don't really see, yet :)

I think it would be reasonable to just allow any value for unknown strings and maybe (just maybe) offer something like this:
[tag("something").i("idk").c("idk2")(), ...other stuff]

or something... Idk if it would be worth it but it could represent some potential alternative solution for people who want to have a little of both words. Arbitrary function calling seems the most problematic to me...

Hi @nahuel, @Mateiadrielrafael - i've started working on this new package @thi.ng/hiccup-html which addresses and helps with some of these aspects, at least in the realm of HTML elements (but I still think a completely generic / general purpose approach is not ever gonna be feasible, too many edge cases in reality, not gonna fight it). The approach of that new package though, gives you full type checking and auto-complete for attributes (incl. arg types for event listeners), which should be a great help...

just released version 0.1.0 of @thi.ng/hiccup-html


Switch from Scala String + comma separated values syntax to whitespace delimited list

Is your feature request related to a problem? Please describe.
To improve the ability to copy-paste snippets, such as compiler options.
Describe the solution you'd like
implement changes in the directives parser so that
//> using options "-Xasync", "-Xfatal-warnings"

can be written as
//> using options -Xasync -Xfatal-warnings

Quotes can still be used for when a value needs to include whitespace, with the same escape characters for scala strings.
Here is a larger example:
//> using scala 3.2.2
//> using toolkit latest
//> using options -Xfatal-warnings -Yexplicit-nulls -Ysafe-init -source:future
//> using dep com.lihaoyi::cask:0.9.0
//> using some.key "I have spaces"

We do not anticipate issues in adapting scala/vscode-scala-syntax to support the new syntax. Additionally, values are still of string type.
Describe alternatives you've considered
no alternative considered.
Additional context
Since directives will not become scala syntax, there is no need to reuse the same rules as the scala parser.
Discussed at the Scala Tooling Summit March 2023

Bringing some additional associated spec that we discussed:

For any value that does not immediately start with a ":

They cannot contain any " until the next whitespace
All other characters, including backslash, are taking verbatim.

For values that start with a ":

They follow Scala syntax for single-line single-quoted strings
Including backslash-based escapes

This is a very nice simplification for newcomers!


Update to egui 0.29

Tracking the latest master of egui, in preparation of egui 0.29's release

egui 0.29 has been released!
https://github.com/emilk/egui/releases/tag/0.29.0


Stucking at "Connecting to Docker Hub" when create a container from images in kitematic UI

Setup
OS: macOS 10.15
Tool: Docker Toolbox, VirtualBox 6.0.14, Kitematic
Expected behavior
A container created after selecting an image
Actual behavior
Kitematic keeps showing Connecting to Docker Hub, no new container created.
Information about the Issue
Everything is fine with Docker CLI. Containers can be created and managed using Docker CLI.

Steps to reproduce the behavior

Select an image from the image list in the UI, a new screen appear with Connecting to Docker Hub
Nothing happend next, no image is downloaded, no new container created

Same here on Windows 10

try this pls https://github.com/Trackhe/kitematic/releases

@Trackhe - this build is not signed

@Miirek okey srry but i cant sign it becouse i dont give 100$ for a developer acc every jear.

@h3d-haivq found a solution?

@h3d-haivq found a solution?

I've stopped using docker toolbox and kitematic since the new Docker for Mac is quite good.


Scaffolding for model validation (leaderboard style)

Requires:

Metric implementation
Model abstraction
Weights and Biases integration

(to be filled)

I've assigned everyone who has input into this — do not worry about any mystery responsibility you didn't know about!

@phinate could we close this as the first version is complete?


[buildmgr] support translate ctrls options with AC5/AC6

Contributed by STMicroelectronics
Signed-off-by: Tarek BOCHKATI<EMAIL_ADDRESS>

Tested OK with my projects with AC6 and:
optimize: size
debug: on

But failed with

debug: off
warnings: off
:-(

But failed with

debug: off
warnings: off
:-(

@jeromecoutant thanks a lot for this test, this revealed that copying a list to another looses the empty values, I will commit a proper fix ASAP

@tarek-bochkati Thanks for this PR, please proceed with the test cases.


Chrome and FF extension not visible

I installed the chrome extension (it's a great idea, if my assumption about what it does is correct) but there is no icon showing.  I verified that it is enabled and I'm not in incognito mode so I assume there should be an icon to click which will automatically redirect from the gist to bl.ocks.org?
I also tried in FF but same result.  I'm using Windows 7.
I checked the source code and document.querySelector(".gist-sidebar")  returns null...
PR submitted to align with the structure I see in Gist.

I think a recent gist site change broke it. The link I on disappeared a couple of weeks ago when GitHub changed.

Yes, it's easy enough to fix (once you figure out how). I submitted a PR including commits for two options: one with the hex logo added to the button and one without.  I did it for chrome and firefox only because I'm not confident testing safari. Hopefully it's ok.

Just confirming that, yes, the extension isn't displaying the "view block" button in either Safari (foreground) or Chrome (background):

Fixed in 1.3.5, which is waiting approval from the respective stores.

Thanks, @mbostock!


Lost lights switches and duplication of existing switches with 1.5.2 update

The latest 1.5.2 update nuked a lot of my z-wave lights and added switches with duplicate names.

Hmm.  The missing lights were actually removed from the Home View and when I choose the light from the room view and select "Show as Separate Tiles" the redundant switch disappears and the device is once again back on the Home View.

The update supports multi components whereas before it would only load capabilities from the "main" component.  That and support for buttons is the only change made.
I assume you are good now?  Can we close this?

Yes.  Once I separated the multifunction from the light it’s behaving again.  Thanks.


Move the creation of immediate switch blocks to the initializer reactor

This PR moves the code creating and storing immediate switch blocks to ChainspecLoader, which does its work in the initializer reactor.
Closes #3212

bors merge


Clarify what needs admin rights and how to handle events in the docs

This clarifies when to handle events, which confused me until I got help (#265), and that you need to handle events promptly and request updates, and grab pixels regularly or your display may freeze.
As far as I can tell, it is possible to write a program that uses evdi and doesn't require administrator permissions without too much difficulty (either have a helper with admin rights add devices, or ensure devices are added on module load). I document that as well.
Closes #268

Love it. 😇


withParam with empty strings - wrong output url

Using -
resource("/api").withParam("x","").withParam("y","")
expected output - "/api?x=&y="
current output - "/api?x"
some web apis need all their params to find the correct route.
this is how i found out about this issue.
please advice.

Ah, that's a bug. Mysterious!
(The correct result should be /api?x&y, not /api?x=&y=. Siesta drops the = for empty string param values.)


[libwebp] do not include all feature by default

Currently vcpkg install libwebp will try to install these packages:
  * freeglut[core]:x64-windows
  * giflib[core]:x64-windows
  * libjpeg-turbo[core]:x64-windows
  * liblzma[core]:x64-windows
  * libpng[core]:x64-windows
    libwebp[all,core]:x64-windows
  * opengl[core]:x64-windows
  * sdl1[core]:x64-windows
  * tiff[core]:x64-windows
  * zlib[core]:x64-windows

I dont think that a reasonable default.  If a user really wants everything, he should install libwebp[all].

There were many problems with the “bare” port, that’s why I made “all” the default feature. At the same time, having the option to reduce building was very interesting to expose.

Hi @kiwixz, Please also update the CONTROL version. Thanks. I notice that it is 1.0.2-1 now and you can modify it as 1.0.2-2.

@NancyLi1013 thanks, it is done now

@kiwixz @Rastaban The freeimage library fails to build after this update. Seems that webp/mux.h is missing.
Any ideas?

https://github.com/Microsoft/vcpkg/pull/5886#issuecomment-478424587
I cannot fully remember the details. But again, I tried saying “please check, together with @vicroms we were not crazy in that PR one month ago...”

@theoractice It looks like libwebp's CMakeLists.txt does not install this header when not building gif2webp/img2webp (and consequently libwebpmux), see https://github.com/webmproject/libwebp/blob/1.0.2/CMakeLists.txt#L396.
If it's that sufficient to fix subsequent problems, it would probably be better to just force every headers to be installed in my opinion.

@theoractice freeimage was fixed in https://github.com/Microsoft/vcpkg/pull/5969

Got it, thank you all.


[Event Request] Table 123 - Procedure CalcQty

Reason:
Need the event to support extensibility of CalcQty function for the Purchase Invoice Line Table an upgrade project in BC Cloud Tenant.
Sample Code:
local procedure CalcQty(QtyBase: Decimal): Decimal
begin
  //REQUEST BEGIN
  OnBeforeCalcQty(Rec, QtyBase, IsHandled);
  if IsHandled then
    exit(QtyBase);
  //REQUEST END

  if "Qty. per Unit of Measure" = 0 then
    exit(QtyBase);
  exit(Round(QtyBase / "Qty. per Unit of Measure", UOMMgt.QtyRndPrecision));
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeCalcQty(PurhcInvLine: Record "Purch. Inv. Line"; var QtyBase: Decimal; var IsHandled: Boolean)
begin

end;

Do we have an idea about timelines and how to expedite this request because it is holding up a client upgrade, we need to inform the stake holders.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47832.


fix: crashes when in background and try to show downing notification

Ticket: https://dicetech.atlassian.net/browse/DEV-5632

fix: crashes when in the background and tries to show downing notification
feat: support refresh downloading notification

Reproduce Crash Step:

CAUTION
Disconnect the adb connection between the Android Devices and Your Computer

Open Demo App
Expand the Normal Test and find the BBB
Click the download button and wait for the dialog showing
Turn off the internet (disconnect from WIFI and Celler)
Click the Dialog OK button
Put the App into the background
Wait 10 seconds then Turn on the internet
Crashes

https://github.com/user-attachments/assets/e0166b17-e7fb-4b7a-bfb8-8f1225ee918b
Exception:
 E  FATAL EXCEPTION: main
    Process: androidx.media3.demo.main, PID: 25621
    android.app.ForegroundServiceStartNotAllowedException: Service.startForeground() not allowed due to mAllowStartForeground false: service androidx.media3.demo.main/.DemoDownloadService
     at android.app.ForegroundServiceStartNotAllowedException$1.createFromParcel(ForegroundServiceStartNotAllowedException.java:54)
     at android.app.ForegroundServiceStartNotAllowedException$1.createFromParcel(ForegroundServiceStartNotAllowedException.java:50)
     at android.os.Parcel.readParcelableInternal(Parcel.java:5015)
     at android.os.Parcel.readParcelable(Parcel.java:4997)
     at android.os.Parcel.createExceptionOrNull(Parcel.java:3177)
     at android.os.Parcel.createException(Parcel.java:3166)
     at android.os.Parcel.readException(Parcel.java:3149)
     at android.os.Parcel.readException(Parcel.java:3091)
     at android.app.IActivityManager$Stub$Proxy.setServiceForeground(IActivityManager.java:6861)
     at android.app.Service.startForeground(Service.java:862)
     at androidx.media3.common.util.Util$Api29.startForeground(Util.java:3770)
     at androidx.media3.common.util.Util.setForegroundServiceNotification(Util.java:330)
     at androidx.media3.exoplayer.offline.DownloadService$ForegroundNotificationUpdater.update(DownloadService.java:923)
     at androidx.media3.exoplayer.offline.DownloadService$ForegroundNotificationUpdater.startPeriodicUpdates(DownloadService.java:895)
     at androidx.media3.exoplayer.offline.DownloadService.notifyDownloadChanged(DownloadService.java:812)
     at androidx.media3.exoplayer.offline.DownloadService.access$400(DownloadService.java:56)
     at androidx.media3.exoplayer.offline.DownloadService$DownloadManagerHelper.onDownloadChanged(DownloadService.java:1043)
     at androidx.media3.exoplayer.offline.DownloadManager.onDownloadUpdate(DownloadManager.java:637)
     at androidx.media3.exoplayer.offline.DownloadManager.handleMainMessage(DownloadManager.java:602)
     at androidx.media3.exoplayer.offline.DownloadManager.$r8$lambda$BgKJ2Nse_hhWOJ980vXMskcdoes(Unknown Source:0)
     at androidx.media3.exoplayer.offline.DownloadManager$$ExternalSyntheticLambda0.handleMessage(Unknown Source:2)
     at android.os.Handler.dispatchMessage(Handler.java:103)
     at android.os.Looper.loopOnce(Looper.java:232)
     at android.os.Looper.loop(Looper.java:317)
     at android.app.ActivityThread.main(ActivityThread.java:8501)
     at java.lang.reflect.Method.invoke(Native Method)
     at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)
     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:878)

@NormanWangEndeavor have you found a way to reproduce it before the fix?

@NormanWangEndeavor have you found a way to reproduce it before the fix?

Yes, I have added the Step in the description, you can try it.

@NormanWangEndeavor sorry I completely missed that :D I will test this today, thank you!

@NormanWangEndeavor
I couldn't reproduce the issue based on your steps on my Pixel 4A (Android 13), tried on the main branch.
However, in dce-mobile the d2g feature is handled by dice-shield-android and that module is using google's androidx/media, see L80-L83.
This means we would need to migrate our androidx-media in dice-shield-android to actually make this fix work. In that case, we would need to call  DemoDownloadService.sendRefreshForeground(activity, DemoDownloadService.class) from dce-mobile?
Btw, have you checked the changelog between our forked androidx-media version and google's latest one (1.4.1)? Could be that they addressed this already?

@NormanWangEndeavor
I couldn't reproduce the issue based on your steps on my Pixel 4A (Android 13), tried on the main branch.
However, in dce-mobile the d2g feature is handled by dice-shield-android and that module is using google's androidx/media, see L80-L83.
This means we would need to migrate to our androidx-media in dice-shield-android to actually make this fix work. In that case, we would need to call  DemoDownloadService.sendRefreshForeground(activity, DemoDownloadService.class) from dce-mobile?
Btw, have you checked the changelog between our forked androidx-media version and google's latest one (1.4.1)? Could be that they addressed this already?

Hi @szaboa
I am using Pixel 6 with Android 14 and this video may help:
https://github.com/user-attachments/assets/e0166b17-e7fb-4b7a-bfb8-8f1225ee918b
And the dce-mobile excludes the dice-shield-android's Google androidx-media dependencies, so the dce-mobile is using our's  androidx-media actually
Then I checked the change log and the source code of androidx-media from 1.2.1 to 1.4.1 and find none of fix about this crash.

@NormanWangEndeavor i see, it's a bit confusing but it's fine, eventually the d2g feature will be implemented in vesper-sdk so i am happy to go with this solution. I'll try to reproduce/test again based on the video you attached. However the question remains the same, do we need to call sendRefreshForeground from dce-mobile, right?

@szaboa
Yes, we need to call sendRefreshForeground  from dce-mobile like the demo do. Maybe we should add this call into dice-shield-android?

Maybe we should add this call into dice-shield-android?

on dice-shield-android it  dependence on google media3, instead of our forked media3 which have sendRefreshForeground , so I think we can not add sendRefreshForeground call on dice-shield-android

Maybe we should add this call into dice-shield-android?

on dice-shield-android it dependence on google media3, instead of our forked media3 which have sendRefreshForeground , so I think we can not add sendRefreshForeground call on dice-shield-android
Before merge this PR in, I think we need to test sendRefreshForeground call on dce-mobile.

I test sendRefreshForeground on dce-mobile and it works fine.


Failure/Error: require 'active_support/all', while loading spec_helper.rb

Steps to reproduce

 Run RSpec tests with the command bundle exec rspec.

Expected behavior
RSpec tests should run successfully without any errors.
Actual behavior
An error occurs while loading spec_helper.rb with the following message:
An error occurred while loading spec_helper.
Failure/Error: require 'active_support/all'

NameError:
  uninitialized constant TZInfo::DateTime
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/tzinfo-2.0.6/lib/tzinfo/datetime_with_offset.rb:19:in `<module:TZInfo>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/tzinfo-2.0.6/lib/tzinfo/datetime_with_offset.rb:6:in `<top (required)>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/tzinfo-2.0.6/lib/tzinfo.rb:30:in `require_relative'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/tzinfo-2.0.6/lib/tzinfo.rb:30:in `<top (required)>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/activesupport-<IP_ADDRESS>/lib/active_support/values/time_zone.rb:3:in `<top (required)>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/time/conversions.rb:5:in `<top (required)>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/time/calculations.rb:4:in `<top (required)>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/time.rb:4:in `<top (required)>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/activesupport-<IP_ADDRESS>/lib/active_support/time.rb:12:in `<top (required)>'
# /usr/share/rvm/gems/ruby-3.2.2@engineer.ai/gems/activesupport-<IP_ADDRESS>/lib/active_support/all.rb:4:in `<top (required)>'
# ./spec/spec_helper.rb:7:in `<top (required)>'
No examples found.

Finished in 0.00004 seconds (files took 0.47417 seconds to load)
0 examples, 0 failures, 1 error occurred outside of examples

System configuration
Rails version: <IP_ADDRESS>
Ruby version: 3.2.2
Contents of spec/spec_helper.rb:
# frozen_string_literal: true

require 'dotenv'
Dotenv.load('.env.test')

require 'rest-client'
require 'active_support/all'
require 'active_record_extended'
require 'capybara/rspec'
require 'database_cleaner/active_record'
require 'selenium/webdriver'
require 'webdrivers/chromedriver'

I've tried to reproduce this in the philr/tzinfo-issue158 repository, but everything works as expected for me.
The line that is failing is referencing the DateTime class from the Ruby standard library (from the date default gem). 'date' has already been required at this point (on line 4), so the class should be available.
It would seem that something has removed or redefined the default date gem in your Ruby installation. What does gem list date output? On Ruby 3.2.3 I see:
*** LOCAL GEMS ***

date (3.3.4, default: 3.3.3)

If date is missing, you could try installing it. Either add gem 'date' to your Gemfile or run gem install date.
What does running ruby -r date -e 'p defined? DateTime; p DateTime; p DateTime.now' output? I see:
"constant"
DateTime
#<DateTime: 2024-04-13T17:34:46+01:00 ((2460414j,59686s,375544210n),+3600s,2299161j)>

I note from the stack trace that your Ruby installation is in a directory named<EMAIL_ADDRESS>It is possible that is something non-standard.
If you can get this to reproduce using a clean, standard Ruby installation then I can look at fixing it. Otherwise I don't think there's anything that can be done?

Hi @philr, thank you very much for your response.
I've found the problem. It looks like we made our own Date class in the lib folder to add some new methods. When ActiveSupport loads and tries to load tzinfo, it hits a line in tzinfo/datetime_with_offset.rb:4
require 'date'
This makes our special date thingy load, and that's when everything starts failing.
After deleting the date.rb file from the lib directory. it's working fine now.


Email with base64-encoded UTF8 header Python2 bug

A bug in future.backports.email.base64mime.header_encode raises an exception when serializing an email message object containing a base64-encoded UTF8 header.
I have observed this problem on Python 2 only, with future library versions 0.17.1 and 0.18.2.
>>> import future.backports.email as email
>>> message = email.message_from_string(u"TO: Laȝamon<EMAIL_ADDRESS>>>> message.as_string()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/message.py", line 155, in as_string
    g.flatten(self, unixfrom=unixfrom)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/generator.py", line 118, in flatten
    self._write(msg)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/generator.py", line 190, in _write
    self._write_headers(msg)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/generator.py", line 217, in _write_headers
    self.write(self.policy.fold(h, v))
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/_policybase.py", line 321, in fold
    return self._fold(name, value, sanitize=True)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/_policybase.py", line 360, in _fold
    maxlinelen=self.max_line_length))
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/header.py", line 375, in encode
    formatter.feed('', lines[0], charset)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/header.py", line 462, in feed
    encoded_lines = charset.header_encode_lines(string, self._maxlengths())
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/charset.py", line 362, in header_encode_lines
    lines.append(encoder(header_bytes))
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/backports/email/base64mime.py", line 73, in header_encode
    header_bytes = header_bytes.encode(charset)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/future/types/newbytes.py", line 406, in __getattribute__
    raise AttributeError("encode method has been disabled in newbytes")
AttributeError: encode method has been disabled in newbytes

Here's a suggested fix future.backports.email.base64mime.header_encode.  One line is changed, marked New.
def header_encode(header_bytes, charset='iso-8859-1'):
    """Encode a single header line with Base64 encoding in a given charset.

    charset names the character set to use to encode the header.  It defaults
    to iso-8859-1.  Base64 encoding is defined in RFC 2045.
    """
    if not header_bytes:
        return ""
    if isinstance(header_bytes, str):
        # header_bytes = header_bytes.encode(charset)  # Old
        header_bytes = bytes(header_bytes, charset)    # New
    encoded = b64encode(header_bytes).decode("ascii")
    return '=?%s?b?%s?=' % (charset, encoded)

When I monkey patch the library with the suggested fix, it works.
>>> import future.backports.email as email
>>> from base64 import b64encode
>>> from future.builtins import bytes
>>> def header_encode(header_bytes, charset='iso-8859-1'):
...     """Encode a single header line with Base64 encoding in a given charset.
...
...     charset names the character set to use to encode the header.  It defaults
...     to iso-8859-1.  Base64 encoding is defined in RFC 2045.
...     """
...     if not header_bytes:
...         return ""
...     if isinstance(header_bytes, str):
...         # header_bytes = header_bytes.encode(charset)  # Old
...         header_bytes = bytes(header_bytes, charset)    # New
...     encoded = b64encode(header_bytes).decode("ascii")
...     return '=?%s?b?%s?=' % (charset, encoded)
...
>>> email.base64mime.header_encode = header_encode
>>> message = email.message_from_string(u"TO: Laȝamon<EMAIL_ADDRESS>>>> message.as_string()
u'TO: =?utf-8?b?TGHInWFtb24gPHRvQHRlc3QuY29tPg==?=\n\n'

While I was working on fixing this, I found that it was already fixed in #534 .


Show instructors on multiple lines

If a course has multiple instructors, they all show in an array-like object.  Create a new line for each instructor instead for a more user-friendly view

Instead of creating a new line for each instructor, they all show on the same line!


iam: iam user access key resource v1alpha1

Description of your changes
Fixes #394
I have:

[X] Read and followed Crossplane's contribution process.
[X] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Manual testing w/ an IAM User and an Access Key (both under example/identity)
Unit testing for the IAM Access Key resource

Something I wanted to add to this PR - you cannot delete an IAM User without removing all the Access Keys. So currently, the user would need to manually remove their access keys prior to deleting an IAMUser CR. I think this behavior is okay, and the correct approach for now. However, to make this more clear, it might make sense to add a finalizer.

cc: @muvaf

If both the IAM User and the Access Keys are represented in Crossplane, then a user should see an error when deleting an IAM User that indicates keys exist, then they could check for IAMAccessKeys present in the cluster right? When you mention adding a finalizer, how would you imagine that working? We don't want to block deletion of an access key on an IAM user, and the IAM user will automatically block because it fails to delete right?

Yes you are correct, you are correct in that it isn't necessary for the IAMUser (and I would prefer not having it), but I just wanted to bring this up. I was thinking about it from a clarity perspective, having a finalizer would make it clear there are resources connected to an IAMUser.

Fixed @muvaf, should be good to merge :)

@krishchow looks like you need a make generate here.

@hasheddan I ran make generate locally, and there weren't any changes in any of the CRDs, I am not sure why check-diff failed (it's succeeding local). Is there a way to run the CI action again?

go mod tidy could help

Should be all fixed now, @hasheddan @muvaf, but I think pushing while the action was running led to this push skipping the CI step


[MM-50955]: Fix for "Download our apps" link in final onboarding step

Summary
When an admin clicks on the "Download our apps" link in final onboarding step it only goes to the page but not the anchored section part-way down the page as it should.
This PR fixes that link.
Ticket Link
MM-50955
Screenshots

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard


WFLY-13354  Add Galleon layers for the EJB subsystem

JIRA: https://issues.redhat.com/browse/WFLY-13354
Related JIRA: https://issues.redhat.com/browse/EAP7-1524
Analysis doc: https://github.com/wildfly/wildfly-proposals/pull/317
This pull request implements Galleon layers for the EJB subsystems. It includes 2 main ejb3 layers: ejb-lite and ejb, and 2 ancillary layers: ejb-local-cache and ejb-dist-cache. This PR also includes basic testing configuration and community doc for this feature. The other 2 layers included in the original plan, ejb-iiop and iiop-openjdk will be implemented in the future.
MDB-related configuration and tests is not included in this PR; instead it will be implemented and delivered as a separate follow-up feature (WFLY-13797 Add a Galleon layer for messaging integration with a remote ActiveMQ broker)
This request currently includes multiple commits over time. I'll squash them if so desired.

18 clustering tests failed in Galleon Linux JDK11 (not related to ejb). They seem to be intermittent and probably due to CI not stable. Error messages like: Timed out waiting for responses for request 3 from node-2. Will re-run it.
org.jboss.as.test.clustering.cluster.web.DistributableTestCase.testSerialized
org.jboss.as.test.clustering.cluster.web.DistributableTestCase.testSessionReplication
org.jboss.as.test.clustering.cluster.web.DistributableTestCase.testGracefulServeOnUndeploy
org.jboss.as.test.clustering.cluster.web.DistributableTestCase.testGracefulServeOnShutdown
org.jboss.as.test.clustering.cluster.web.ExternalizerTestCase.test
org.jboss.as.test.clustering.cluster.web.FineNonTransactionalSessionActivationTestCase.test
org.jboss.as.test.clustering.cluster.web.FineTransactionalSessionActivationTestCase.test
org.jboss.as.test.clustering.cluster.web.NonDistributableTestCase.test
org.jboss.as.test.clustering.cluster.web.async.AsyncServletTestCase.test
org.jboss.as.test.clustering.cluster.web.expiration.CoarseSessionExpirationTestCase.test
org.jboss.as.test.clustering.cluster.web.expiration.FineSessionExpirationTestCase.test
org.jboss.as.test.clustering.cluster.web.passivation.CoarseSessionPassivationTestCase.test
org.jboss.as.test.clustering.cluster.web.passivation.FineSessionPassivationTestCase.test
org.jboss.as.test.clustering.cluster.web.shared.LegacySharedSessionTestCase.test
org.jboss.as.test.clustering.cluster.web.shared.SharedSessionTestCase.test
org.jboss.as.test.clustering.cluster.jpa.ClusteredJPA2LCTestCase.testEntityCacheReplication
org.jboss.as.test.clustering.cluster.jpa.custom.ClusteredJPA2LCCustomRegionTestCase.testEntityInCustomCache
org.jboss.as.test.clustering.cluster.jpa.custom.ClusteredJPA2LCCustomRegionTestCase.testPropsInCustomCache

@chengfang Nice :)
I'll look through this again more thoroughly but I've gone through it and it looks good. @fabiobrz @tommaso-borgato fyi
2 requests, both about testsuite/layers:

Please add executions that provision the 'ejb-local-cache' and 'ejb-dist-cache' layers. We want one for all layers as part of a basic smoke check that all layers produce bootable servers. Such a server may not be useful but it should be bootable; that proves the layer doesn't have an unstated dependency that might be a source of bugs.

Can you take one of your executions and duplicate it, but add ejb-dist-cache and exclude ejb-local-cache? If you look in testsuite/layers/pom.xml for 'excluded-layers' elements you'll see examples of that kind of thing.

Will update the PR to address the 2 areas.

@chengfang I created #13532 to combine this with the JSF layer PR and add some testing that was part of the JSF RFE's test plan.  Once I had the branch I decided to use it to see if EJB testing in testsuite/integration/clustering would help with coverage of jpa-distributed ot help wrap up the JPA RFE.

I saw there was no EJB layer execution in testsuite/integration/clustering so I added a commit to #13532 for that. That would be better squashed in here eventually though.

Running those tests revealed a problem in the layers here, with the default-sfsb-cache attribute. Basically that should not be directly set by ejb-lite, it should be set by ejb-local-cache and ejb-dist-cache. This results in the usual standalone-full-ha value of 'distributable' when ejb-dist-cache is used. The 2nd to last comint in #13532 does that. I think that would be better squashed in here too though.

@bstansberry that's great to have ejb layer clustering tests, and have the cache setting fixed. I'll cherrypick the 2 commits and update this PR.

@chengfang Great. When you pull those in can you squash this down? I don't care about any commits from me getting squashed into other changes.


Action does not run on arm64 runners

When attempting to use the hashicorp-contrib/setup-packer@v1 action with arm64 runners, jobs fail with an HTTP error and the message "Error: Unsupported architecture: arm64".
Here's a full error message from the most recent run:
Run hashicorp-contrib/setup-packer@v1
  with:
    github-token: ***
    packer-version: latest

Error: Unsupported architecture: arm64
/home/runner/actions-runner/_work/_actions/hashicorp-contrib/setup-packer/v1/dist/index.js:6899
        const err = new HTTPError(response.message.statusCode);
                    ^

HTTPError: Unexpected HTTP response: 404
    at /home/runner/actions-runner/_work/_actions/hashicorp-contrib/setup-packer/v1/dist/index.js:6899:[2](https://github.com/***/***/actions/runs/***/job/***#step:4:2)1
    at Generator.next (<anonymous>)
    at fulfilled (/home/runner/actions-runner/_work/_actions/hashicorp-contrib/setup-packer/v1/dist/index.js:6800:26)
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  httpStatusCode: [4](https://github.com/***/***/actions/runs/***/job/***#step:4:4)04
}

Here's the YAML for the workflow:
on: workflow_dispatch
name: Build Image
jobs:
  build:
    name: Build
    runs-on: buildjet-2vcpu-ubuntu-2204-arm
    defaults:
      run:
        working-directory: .
        shell: bash
    steps:
      - name: Checkout
        uses: actions/checkout@v3

      - name: Setup Packer
        uses: hashicorp-contrib/setup-packer@v1
      - name: Setup Node
        uses: buildjet/setup-node@v3
        with:
          node-version: 18
          cache: npm

      - name: Install Node Dependencies
        run: npm ci

      - name: Install Build Dependencies
        run: xargs sudo apt-get install <depends.txt

      - name: Make Images
        run: mkdir deploy && make images

Is there any reason not to try v2?

Not that I'm immediately aware of. I will try v2 and let you know if that resolves it

That fixed it, and no apparent issues migrating to v2. Thanks!
Run hashicorp-contrib/setup-packer@v2
  with:
    github-token: ***
    packer-version: latest

/usr/bin/unzip -o -q /home/runner/actions-runner/_work/_temp/9c6[2](https://github.com/***/***/actions/runs/***/job/***#step:4:2)6789-50ab-420f-b112-0d[3](https://github.com/***/***/actions/runs/***/job/***#step:4:3)588fb7635
Version: 1.9.2
Variant: linux_arm6[4](https://github.com/***/***/actions/runs/***/job/***#step:4:4)
Cache location: /opt/hostedtoolcache/packer/1.9.2/linux_arm[6](https://github.com/***/***/actions/runs/***/job/***#step:4:7)4/packer

I'm glad to hear that!


feat(polyfill): add module (fixes #107)

this PR adds module to a list of std/node polyfills
closes #107

can you please add some testings for #107 and #163 ?

or i will add later

https://github.com/alephjs/esm.sh/tree/master/test/deno/common


Clone Project is NOT cloning ALL the fields from the Cloned item

Attempting to create a new Job with the option Copy existing Item -> Clone from, the only field cloned is the Git Project Repository...
All of the other fields are empty... I had most of the fields filled out here in the original project.

thanks
Marcello

Bumped into this as well.

@nschloe Yeah, a bummer, but this plugin rocks!!!

The missing part of config is stored under ${WORKSPACE}/template/config.xml. The following groovy script (yeah it is messy/could be written better, has no error catching, etc) is a quick workaround to copy an existing Multi-branch job to a new one and load with the extra config without restarting/reloading all of Jenkins.
import java.io.InputStream;
import java.io.FileInputStream;
import java.io.File;
import java.nio.file.*
import javax.xml.transform.stream.StreamSource;

//source_job and destination_job are defined as scriptler/groovy parameters
def hudson = hudson.model.Hudson.instance;
def job = hudson.model.Hudson.instance.getItem(source_job);

hudson.copy(job, destination_job);

def job_copied = hudson.model.Hudson.instance.getItem(destination_job);
def target_dir = job_copied.getTemplateDir();
def source_dir = job.getTemplateDir();

Path source = Paths.get("$source_dir/config.xml");
Path target = Paths.get("$target_dir");
Files.copy(source, target.resolve(source.getFileName()), StandardCopyOption.REPLACE_EXISTING);

def configXMLFile = job_copied.getConfigFile();
def file = configXMLFile.getFile();
InputStream is = new FileInputStream(file);

job_copied.updateByXml(new StreamSource(is));
job_copied.save();

@a-wheeler Can you please describe the step-by-step of your hack? I'm quite not familiar with it...

This requires Java 7+.

Get the scriptler plugin. (Or use a system groovy build job). https://wiki.jenkins-ci.org/display/JENKINS/Scriptler+Plugin
Add a new scriptler script.
Add two parameters 'source_job' and 'destination_job'.
Paste in the above groovy script.
Run script on your master specifying the job to copy and the name of the new job.

@a-wheeler Thank you... I will try it ASAP...

Closing in favor of #44


Update UserGuide.md

closes #119
closes  #126
closes #138

Codecov Report

Merging #169 (bd2b923) into master (2b602e5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #169   +/-   ##
=========================================
  Coverage     70.98%   70.98%
  Complexity      488      488
=========================================
  Files            79       79
  Lines          1606     1606
  Branches        171      171
=========================================
  Hits           1140     1140
  Misses          412      412
  Partials         54       54

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Dial Graph Temperature

Hej,
how is the dial graph working with temperatures? Does it take the max temp which is provided from OpenHardwareMonitor?

Oh and same thing for other resources. I'm using currently the Dial to display memory load. It shows about 13%, which is correct, but the dial looks like it's at 90%.

It does use the min/max which was returned by Open Hardware Monitor data.
Perhaps it should have an option for a manual range override?


Add markus-haack.com

Here are the details of my personal site:
Domain: https://markus-haack.com/
Size: 118KB (compressed size)
GTmetrix report: https://www.debugbear.com/test/website-speed/ejc3kPpm/overview
Thank you for maintaining 250kb.club.

Welcome to the 250kb club. Your page entry will be https://250kb.club/markus-haack-com/


[KNI][release-4.16] snyk: update ignore list

The linter is now complaining about an old dep,
for the time being (and likely for the mid term)
we will ignore the warning. cobra is an indirect dep anyway.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>(cherry picked from commit 68523f6f46381645d937296b4abf644ed7bfbddf)

/lgtm


recipe/py27_noffmpeg.patch: really disable libav for win and py27

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Refactor engine to support hardware accelerators

Describe your changes
make engine register the Pass config instead of Pass instance to support hardware accelerators
Checklist before requesting a review

[x] Add unit tests for this change.
[x] Make sure all tests can pass.
[x] Update documents if necessary.
[x] Format your code by running pre-commit run --all-files

(Optional) Issue link

Can you update the paths to the footprints json and model json according in these examples?
https://github.com/microsoft/Olive/blob/main/examples/directml/stable_diffusion/stable_diffusion.py#L77
https://github.com/microsoft/Olive/blob/main/examples/whisper/test_transcription.py#L47


[BUG] azure-spring-boot-starter-keyvault-certificates can not use certificate EGS1cert

The following contents are copied from Email:
I have a customer who is running a Spring boot app in Azure VM which uses Managed Identity to connect to Azure KV.
This customer wants to use the certificate EGS1cert (private key marked non-exportable) stored in Azure KV for
establishing a Mutual TLS with the server.
The issue here is that this is causing an SSL handshake error. Is it possible to do this with a non-exportable key
 or is this exactly the issue?

Specifically what they want to do (enable mutual SSL on client side): Azure Key Vault Certificates Spring Boot
starter client library for Java | Microsoft Docs (they’re following this technical documentation)

I already asked them to test in the meanwhile with the certificate configured as exportable just to see
what's the behavior. I have my doubts regarding if it is the key and its settings (namely the part of
it being non-exportable) that is causing this.  Even the customer is having doubts if the technical
documentation works both for exportable and non-exportable certificates.

I attached to this e-mail the SSL handshake error that the customer is having.

SSL_Handshake_error.log

How do you create EGS1cert in Azure Portal?
I can not choose this kind of cert in Azure Portal:

Closing this issue, because no response from customer for a long time.


Need to insert a procedure to fix a problem

Hi Owen,
Dev team has suggestion on changing this document. There is also another problem about switching between service account. The fix
Also documenting the step to fix this error
Server Gen2 cryptokey is not present, but server assembly object System is set to use server gen2 cryptokey. Terminating server

I need to make one more line of instruction
cryptkeygen2.bin needs to be deleted as well as those asm.xml file. The next SSAS start will regenerate those assemblies and perform an upgrade

@Minewiskan

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

NOTE: images must live in the repo so the new image will need to be either removed from the content or added into the appropriate location in the repo.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I need to create a new pull request for some correction . I will file new one later


Added support for messasge type

Added support for messasge type whether it's marketing or transactional.

Hi @chetanmeh  @sawhneyh  @cmathis
Can you please have a look?

@anilkbunkar -  Are you making enum changes as suggested by @sawhneyh or this PR is ready to me merged . Please Advise

Hi @prabhum2
We have a doubt, can we add more values in hard-enum later if required?
If yes, please merge the PR.

Hi @prabhum2
Can you please merge the PR?
I confirmed with @akshitjain that we can add more values to hard-enum if required later.
Thanks.


Fixed parameter ordering

Fixes #5586

I wonder now if the "start" is really meant to be "type", as "start" it is out of place as well in the constructor parameters since it's supposed to come after stride and not between normalized and stride (which is the "type" place)


Add 'id' setter to ProxyAnimation to fix broken TypeIt animations

WHAT
Added a setter to "id" attribute for ProxyAnimation class.
WHY
Fix https://github.com/flackr/scroll-timeline/issues/108
When polyfill is loaded, it breaks TypeIt animations on the page
HOW
TypIt animations attempt to set animation ids. Missing setter was causing an exception.
TEST
I built it locally and tested with my own TypeIt setup. It solved the issue and everything is working fine for me.
@kevers-google @flackr: I see there is a plan to rewrite the ProxyAnimation class to be a "full-fledged proxy" so the issues like this one won't happen but this task seems ambitious and there are not enough hours in a day. But I need a fix in a very short time frame.
https://github.com/flackr/scroll-timeline/blob/29b7b6e1be4bb40192f987d7c001f6e2910e0912/src/proxy-animation.js#L769-L771
So, please could we have it merged meanwhile?

Hi @kevers-google @flackr may I please ask for someone to review this PR?
This is a pretty simple and small fix that solves the problem I raised in https://github.com/flackr/scroll-timeline/issues/108

@dream-elite
Sorry for the slow response.
Can you please run "npm run dev" to generate the binaries. These are updated alongside the source and should be part of the pull request.

@kevers-google  Done


Add option to install using winget on Windows

Title.

Any updates on this? I'd like to see more tools getting on winget don't want to pollute my laptop with more third party package managers.

#1782, #1673, and #1465 are all basically asking the same thing (TLDR: it isn't going to be done by the maintainer but if you want to do it yourself you can).

Installation via winget is up to whoever packages neofetch for it.
Listing of installed packages is something we can add to neofetch.

whoever packages neofetch for it.

Currently it's nobody.

Installation via winget is up to whoever packages neofetch for it.

winget requires either an EXE installer with a silent install option or an MSI. If anyone can provide me with an official link to either of those two, I'll happily maintain the package for it.

We're working on:

microsoft/winget-cli/issues/182
microsoft/winget-cli/issues/140

Those should make it even easier in the Windows Package Manager 1.3 release.

can't wait for this

I'm also excited for this!

Winget 1.4 supports zip file installation and It's GA now. They are accepting it on their repositoy.

As an FYI, there are examples of how to automate with GitHub actions:

https://github.com/microsoft/PowerToys/blob/main/.github/workflows/package-submissions.yml
https://github.com/JanDeDobbeleer/oh-my-posh/blob/main/.github/workflows/winget.yml


Crash in PickColor.cs

connect at least one Vector4 to PickColor
enter something which is  not a number in the index Parameter
-> crash

System.ArgumentOutOfRangeException
  HResult=0x80131502
  Message=Index was out of range. Must be non-negative and less than the size of the collection. Arg_ParamName_Name
  Source=System.Private.CoreLib
  StackTrace:
   at System.ThrowHelper.ThrowArgumentOutOfRange_IndexException()
   at System.Collections.Generic.List`1.get_Item(Int32 index)
   at T3.Operators.Types.Id_3539c3a0_73f1_412c_aad2_517810126ac6.PickColor.Update(EvaluationContext context) in C:\dev\repos\t3\Operators\Types\lib\math\vec4\PickColor.cs:line 30
   at T3.Core.Operator.Slots.Slot`1.Update(EvaluationContext context) in C:\dev\repos\t3\Core\Operator\Slots\Slot.cs:line 73
   at T3.Gui.OutputUi.OutputUi`1.Recompute(ISlot slot, EvaluationContext context) in C:\dev\repos\t3\t3\Gui\OutputUi\OutputUi.cs:line 45
   at T3.Gui.OutputUi.OutputUi`1.DrawValue(ISlot slot, EvaluationContext context, Boolean recompute) in C:\dev\repos\t3\t3\Gui\OutputUi\OutputUi.cs:line 31
   at T3.Gui.Windows.Output.OutputWindow.DrawOutput(Instance instanceForOutput, Instance instanceForEvaluation) in C:\dev\repos\t3\t3\Gui\Windows\Output\OutputWindow.cs:line 237
   at T3.Gui.Windows.Output.OutputWindow.DrawContent() in C:\dev\repos\t3\t3\Gui\Windows\Output\OutputWindow.cs:line 101
   at T3.Gui.Windows.Window.DrawOneInstance() in C:\dev\repos\t3\t3\Gui\Windows\Window.cs:line 93
   at T3.Gui.Windows.Output.OutputWindow.DrawAllInstances() in C:\dev\repos\t3\t3\Gui\Windows\Output\OutputWindow.cs:line 68
   at T3.Gui.Windows.Window.Draw() in C:\dev\repos\t3\t3\Gui\Windows\Window.cs:line 49
   at T3.Gui.Windows.WindowManager.Draw() in C:\dev\repos\t3\t3\Gui\Windows\WindowManager.cs:line 96
   at T3.Gui.T3Ui.Draw() in C:\dev\repos\t3\t3\Gui\T3UI.cs:line 70
   at T3.Program.<>c__DisplayClass13_0.<Main>g__RenderCallback|1() in C:\dev\repos\t3\t3\Program.cs:line 225
   at SharpDX.Windows.RenderLoop.Run(Control form, RenderCallback renderCallback, Boolean useApplicationDoEvents)
   at T3.Program.Main() in C:\dev\repos\t3\t3\Program.cs:line 241

Anaysis
This is an interesting one, because from the look of the code, the negative values are prevented. However, input of a invalid number causes the value to be -2147483648, and the negation of -2147483648 cannot be represented in an 4 byte int (MAX_INT is<PHONE_NUMBER>). See evaluation window in screen shot

Potentially this is an issue in other operators as well. Related to #123

wow interesting find! super edge case here, maybe we can handle this in a simple way at a low level

Fixed by #172, thanks @domportera

Wow... This is just crazy!
What a great example of open source. A perfect bug report and a valid fix a shortly thereafter. Thanks ♥

no problem ♥ a bit of a lucky break on the timing but for real nobody beats @hollyhook's bug reports


dotnet subscriber unable to receive published message from controller post method

Enviornment:
Windows 10 vm
dapr AspNetCore 0.11.0-preview 2
[app_root_folder]\Components\subscription.yaml:
apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
name: pubsub
spec:
topic: newOrder
route: /newOrderCreated
pubsubname: pubsub
Controller code:
[HttpPost]
[Route("newOrderCreated")]
public ActionResult AddPayment(
[FromBody] Payment paymentItem)
{
Console.WriteLine("New order arrived for {0} {1} {2} {3}",
paymentItem.Item,
paymentItem.Count,
paymentItem.Total);
        payments.Add(paymentItem);
        return this.Ok(paymentItem);
    }

Powershell command:
dapr run --app-id paymentService --components-path .\Components\  --log-level debug --app-port 5050  --dapr-http-port 5055 dotnet run
Expected:
controller method with Route('/newOrderCreated is called when message published
Actual:
controller method is NOT called when message is published
Log:
Starting Dapr with id paymentService. HTTP Port: 5055. gRPC Port: 62616
←[93;1m== DAPR == time="2020-11-23T18:50:23.08916Z" level=info msg="starting Dapr Runtime -- version 0.11.3 -- commit a1a8e11" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3
←[0m
←[93;1m== DAPR == time="2020-11-23T18:50:23.08916Z" level=info msg="log level set to: debug" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3
←[0m
←[93;1m== DAPR == time="2020-11-23T18:50:23.090154Z" level=info msg="metrics server started on :62617/" app_id=paymentService instance=zhang-vm scope=dapr.metrics type=log ver=0.11.3
←[0m
←[93;1m== DAPR == time="2020-11-23T18:50:23.0961647Z" level=debug msg="No Access control policy specified" app_id=paymentService instance=zhang-vm scope=dapr.configuration type=log ver=0.11.3
←[0m
←[93;1m== DAPR == time="2020-11-23T18:50:23.0961647Z" level=info msg="standalone mode configured" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3
←[0m
←[93;1m== DAPR == time="2020-11-23T18:50:23.0961647Z" level=info msg="app id: paymentService" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3
←[0m
←[93;1m== DAPR == time="2020-11-23T18:50:23.0961647Z" level=info msg="mTLS is disabled. Skipping certificate request and tls validation" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3
←[0m
←[93;1m== DAPR == time="2020-11-23T18:50:23.0961647Z" level=info msg="application protocol: http. waiting on port 5050.  This will block until the app is listening on that port."   Status: Running. Init Elapsed 2867.3437000000004ms" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3
←[0m
Updating metadata for app command: dotnet run
You're up and running! Both Dapr and your app logs will appear here.

Transferred issue to Dotnet SDK repo. @rynowak @vinayada1 can you assist?

@htzhang2 - Could you add the Topic attribute as in the sample: https://github.com/dapr/dotnet-sdk/blob/47b59b4a97f73a386d74fa900effd06527735224/samples/AspNetCore/ControllerSample/Controllers/SampleController.cs#L67

I did add, still not receiving the message:
    [Topic("pubsub", "newOrder")]
    [HttpPost]
    [Route("/newOrderCreated")]
    public ActionResult<Payment> AddPayment(
        [FromBody] Payment paymentItem)
    {

@htzhang2 - Could you please share your code in Startup.cs. Also, could you try the publish via the CLI and see if it is working as mentioned in the sample:
Withdraw Money (pubsub)
Publish events using Dapr cli:
On Linux, MacOS:
dapr publish --pubsub pubsub -t withdraw -d '{"id": "17", "amount": 15 }'
On Windows:
dapr publish --pubsub pubsub -t withdraw -d "{"id": "17", "amount": 15 }"

public class Startup
{
    public Startup(IConfiguration configuration)
    {
        Configuration = configuration;
    }

    public IConfiguration Configuration { get; }

    // This method gets called by the runtime. Use this method to add services to the container.
    public void ConfigureServices(IServiceCollection services)
    {
        services.AddCors(o => o.AddPolicy("MyPolicy", builder =>
        {
            builder.AllowAnyOrigin()
                   .AllowAnyMethod()
                   .AllowAnyHeader();
        }));

        services.AddControllers().AddDapr();

    }

    // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
    public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
    {
        if (env.IsDevelopment())
        {
            app.UseDeveloperExceptionPage();
        }

        app.UseHttpsRedirection();

        app.UseRouting();

        app.UseCloudEvents();

        app.UseCors("MyPolicy");

        app.UseAuthorization();

        app.UseEndpoints(endpoints =>
        {
            endpoints.MapSubscribeHandler();
            endpoints.MapControllers();
        });
    }
}

$ dapr  publish --pubsub pubsub -t newOrder -d "{"Key":"volkswagen-golf-2021","Item":"volkswagen-golf-2021","Count":"1","Total":20000}"
Event published successfully

The Startup.cs code looks okay. Also, was your subscriber able to see the event published using dapr cli command? If yes, then the issue seems to be with the publisher. You could try publishing to the Dapr endpoint: http://localhost:/v1.0/publish// directly using a rest client such as Postman. Let's see if that works.

No. Subscriber can't see event published with dapr cli command

@htzhang2 Hi, do you receive any logs from your .NET application when you publish the event? Can you try and set log level to default information from all sources, publish event and share logs from API, if any show up that is.

Publisher log looks fine:
←[94;1m== APP == info: System.Net.Http.HttpClient.IPublishService.LogicalHandler[100]
←[0m
←[94;1m== APP ==       Start processing HTTP request POST http://localhost:5025/v1.0/publish/pubsub/newOrder
←[0m
←[94;1m== APP == info: System.Net.Http.HttpClient.IPublishService.ClientHandler[100]
←[0m
←[94;1m== APP ==       Sending HTTP request POST http://localhost:5025/v1.0/publish/pubsub/newOrder
←[0m
←[94;1m== APP == info: System.Net.Http.HttpClient.IPublishService.ClientHandler[101]
←[0m
←[94;1m== APP ==       Received HTTP response after 2080.015ms - OK
←[0m
←[94;1m== APP == info: System.Net.Http.HttpClient.IPublishService.LogicalHandler[101]
←[0m
←[94;1m== APP ==       End processing HTTP request after 2091.8117ms - OK
←[0m
←[94;1m== APP == Publish new order successfully

node subscriber can receive message from same publisher.
Something with dotnet subscriber.

If you add the [Topic...] attribute to your subscriber code, then you should see logs similar to these at startup:-
== DAPR == time="2020-12-03T14:22:24.162915-08:00" level=info msg="app is subscribed to the following topics: [deposit withdraw] through pubsub=pubsub" app_id=controller instance=Pacha.redmond.corp.microsoft.com scope=dapr.runtime type=log ver=1.0.0-rc.1
I don't see this in the logs that you have pasted. Could you try comparing your code with the Controller Sample and see what is different.

Right, there is no such log in dotnet subscriber. But nodejs subscriber has similar log.
I didn't see anything different from sample controller.
Could it be something wrong in actor state store?
←[93;1m== DAPR == time="2020-12-03T23:18:32.3358265Z" level=warning msg="failed to init actors: state store does not support transactions which actors require to save state - please see https://github.com/dapr/docs" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3

If copy both pubsub.yaml and subscription.yaml under same folder (.\Components), then dotnet subscription log shows up.
However, an error occurred when a message arrives:
←[93;1m== DAPR == time="2020-12-03T23:46:53.7467606Z" level=warning msg="retriable error returned from app while processing pub/sub event 7cc24661-a016-4932-9bc4-244a61ff6bfd: . status code returned: 307" app_id=paymentService instance=zhang-vm scope=dapr.runtime type=log ver=0.11.3

From the error, the issue seems to be in the application code. Could you add logs/breakpoint to your AddPayment method? Make sure the code is entering this method when a message is published and then debug further.

@htzhang2 - Is this still an issue?

@htzhang2 - Is this still an issue?

Yes. Still an issue

Yes. Still an issue

@htzhang2 - Could you refer to https://github.com/dapr/dotnet-sdk/blob/master/docs/troubleshooting-pub-sub.md and try to see if you can find the issue?
Potentially...app.UseHttpsRedirection(); might be the culprit.

@htzhang2 - Could you refer to https://github.com/dapr/dotnet-sdk/blob/master/docs/troubleshooting-pub-sub.md and try to see if you can find the issue?
Potentially...app.UseHttpsRedirection(); might be the culprit.

@htzhang2 - Were you able to debug the issue?

Closing due to lack of response

@vinayada1 @msfussell - I can confirm that app.UseHttpsRedirection() is the culprit when using the TopicAttribute, which gives the following error:
time="2021-04-27T16:08:14.8816947-04:00" level=error msg="app returned http status code 500 from subscription endpoint" app_id=myapi instance=DESKTOP scope=dapr.runtime type=log ver=1.1.2
time="2021-04-27T16:08:14.882223-04:00" level=debug msg="app responded with subscriptions []" app_id=myapi instance=DESKTOP scope=dapr.runtime type=log ver=1.1.2

Commenting that line and using the HTTP application port fixes the issue. I hope that the declarative topic subscription will allow me to bring back the app.UseHttpsRedirection().
Environment:
Windows 10
Dapr 1.1.2 self-hosted (Docker)
ASP.NET core 3.1
Pub/Sub component: Azure Service Bus Topics

@htzhang2 - Could you refer to https://github.com/dapr/dotnet-sdk/blob/master/docs/troubleshooting-pub-sub.md and try to see if you can find the issue?
Potentially...app.UseHttpsRedirection(); might be the culprit.

That definitely did the trick. Hours with this the same error... thank god I reached here
Thanks

I was having the same issue using two WebApi services running as Docker containers, and their sidecars running as containers as well using Visual Studio's Docker Compose support.
The log contained " app returned http status code 307 from subscription endpoint".
I disabled app.UseHttpsRedirection(); in the startup then everything started working.
Thank you for the trick!


Exclude setup files from coverage

Summary
Fixes #7761
shouldInstrument function will return false if the file is one of the setupFiles or setupFilesAfterEnv.

Test plan
Added unit tests

This is great, thanks! Can you also add globalSetup and globalTeardown?
This also broke a couple of old tests. I'm not sure what the behavior should be if the files are explicitly asked coverage for. @thymikee thoughts?

@SimenB not sure about that either, but I lean towards ignoring Jest-related files anyway, like with coverageIgnorePatterns. What's the point of having coverage for setup file? I can imagine someone doing that, but have never encounter anything like that.

@SimenB checks for globalSetup and globalTeardown added

Yes, please :)

I'm fine with still ignoring setup files even if they are included in the coverage pattern - the coverageReport e2e test will need updating though (not just snapshot updates), otherwise "collects coverage only from specified file" will collect from 0 files and "collects coverage only from multiple specified files" will collect from 1 file. Should use a different collectCoverageFrom pattern than setup.js then.