added applicant evaluation page

Targets #2848

Description

Added evaluation page for applicants to evaluate thier levels with thier feedbacks .
Checklist:

[x] I have performed a self-review of my own code.

I have checked all the functionality everything is working fine please do a big check too

ok i will try to do the changes you suggest

ok i will try to do the changes you suggest


[FEATURE] Investigate Using a Lock Free Queue

I'm curious to see if a lock-free queue significantly improves performance for the thread pool. We should do some tests with industry standard lock free queues in combination with std::move_only_function to see the difference in performance.
Queues/deques to test:

https://github.com/ConorWilliams/ConcurrentDeque
https://github.com/cameron314/concurrentqueue

https://github.com/erez-strauss/lockfree_mpmc_queue

Got an implementation in #54 but having trouble addressing some pesky bugs as I can't pin down the reason for them. Could use some assistance.


Event Study and Staggered Diff in Diff

Working on this issue now. Will aim to have a PR after the major code refactor. Currently just exploring modelling options...

The big refactor (#381) is now complete, so no major blockers now for whenever you wanted to work on this.


Add configurable compiler to global section

Add a section in the global config to be able to specify the compiler. This makes it easy to switch to other compilers that still accept gcc's arguments, like aarch64-linux-gcc and others.

This needs to be done differently - the same wrapper posted on github with its cfg file needs to be processed by different compilers, based on which user installs the wrapper. It either needs to be auto-discovered or passed to nimble/nimgen without hard-coding in the cfg.
I'm thinking something like an environment variable makes sense but it is kludgy. A nimgen global cfg file might also be an idea since everything would run through the same compiler in theory. Perhaps, nimgen should use the same compiler as specified in nim.cfg?

I agree with you completely for the case of actually compiling the code. However, my use case is simply being able to specify the preprocessor for cross compiling to the Nintendo switch.
https://github.com/jyapayne/nim-libnx/blob/38c9efc2ab84e1f41c4a0730fe8e9cadd2d3ae4b/libnxGen.cfg#L3

So are you changing nim.cfg to point to the aarch64-none-elf-gcc compiler?

Nope, but I'm adding the cross compiler as an OS switch in the Nim compiler. https://github.com/nim-lang/Nim/pull/8069

I'm thinking something like an environment variable makes sense but it is kludgy. A nimgen global cfg file might also be an idea since everything would run through the same compiler in theory. Perhaps, nimgen should use the same compiler as specified in nim.cfg?

It looks like I ignored your question, my apologies. I would say that your idea of using the same compiler in the nim.cfg is a good idea! Env vars are kind of kludgy, but I do still use them from time to time.

I did research this but doesn't seem trivial since I'll have to parse nim.cfg myself and get stuff out of extccomp. Unless Nim can tell me itself.
More important though, I will have no idea if the user uses a -os flag that changes the compiler to be used. Hope you have a better understanding of how it works to think of a way to pull this detection off.

Yeah, you're right. It would be quite difficult to do this without leveraging the Nim compiler in some way to give us the parsed data from the nim.cfg.
I'm not really sure what the way forward is on this PR. The code I've submitted works for me because I'm cross compiling using aarch64-none-elf-gcc on Linux only, but it won't work for everyone like you mentioned. The simple way here is to just add an environment variable (or use CC/CXX by convention) and modify nimgen to use environment variables. It's kludgy but simple. What are your thoughts?

So if we simplify this to CC/CXX/CPP, how will it help your use case? Nimgen will pick up the env vars but you will need to set them before installing the nimble package.
Does it really matter whether you use gcc or aarch...gcc for the preprocessing? I guess the #defines might change.

Ok then let's do the env var method. Are you okay with updating this PR?

I think there's some confusion here - when I said env vars, I meant loading gCCompiler from the environment variable CC rather than the n.global section.
I'm cool with the ${} syntax along with ${output} but am wondering about staying consistent with the pipe syntax using $file. Should that be changed to ${file} as well?
It appears the ${} syntax is supported only in [n.include] and [n.exclude] sections.

I think there's some confusion here - when I said env vars, I meant loading gCCompiler from the environment variable CC rather than the n.global section.

But then it's not clear where the env vars are coming from. The way I have it now makes it so that you can set cpp_compiler=${CXX}, which will tell the user where the compiler config is coming from. Sure, we could put it in the docs, but this way it's a little bit more self explanatory.

I'm cool with the ${} syntax along with ${output} but am wondering about staying consistent with the pipe syntax using $file. Should that be changed to ${file} as well?

That syntax is actually identical in terms of Nim's string interpolator from strutils. It supports either $identifier or ${identifier}. The latter is to make it so that the identifier is clear, otherwise $identifierbar will not replace $identifier, unless ${identifier}bar is used. This is useful for paths, for example.

It appears the ${} syntax is supported only in [n.include] and [n.exclude] sections.

Not true, it is actually supported where ever you access a key from any of the configs. See this function

I feel we should change pipe to use ${file} as well then.

It already is this way :) Like I said, either syntax works because of how Nim's strutils.% operator is implemented. I could put a note in the README.md that says you can use either syntax because it isn't clear unless you know how the backend is implemented.

I see what you are saying, just felt it was easier to change the compiler by simply setting an env var. If you have to set it in .cfg as well then every wrapper dev will have to be explicit in setting those values in n.global. If they didn't then you would have to edit the package and couldn't simply nimble install.

With this change, you don't have to specify cpp_compiler or c_compiler, as the default will be gcc/g++. But I think I see what you're saying. What if we did this:
If cpp_compiler or c_compiler is set, use that value and any env vars referenced. If not, use CC/CXX/CPP. If those are not set, default to gcc/g++. Would that work?

I think the env var CC should have higher priority than the value defined in c_compiler. This way, a local overload has priority over a generic wrapper definition. What's your opinion on that?

I think it could be either way. For me, I prefer having the cfg file have highest priority because my libnx wrapper should not allow overrides, however your way makes sense from the perspective of the user of the wrapper because they may want to use a different compiler.
I think we should have a global value in the config called allow_compiler_override with a default value of true that allows the wrapper creator to allow CC/CPP/CXX to override the settings in n.global. Then we can have the priority you just said with the ability to disable it so that my project works as well. Is that good with you?

I think allow_compiler_override will just be overkill. User should install and use packages as documented. If they are using your package, it is unlikely they will want to fiddle with the compiler anyway since it wouldn't work. They would only consider an override if the default was selected and they were wanting to do something custom.
With that logic, I think n.global:c_compiler should override ${CC} which would override default gcc.

Awesome work, appreciate the efforts!

Thank you for saying that! I appreciate you taking the time to review these as well. It helps a lot because you're well versed in making these changes cross platform to work for everyone.


yabar: support cross-compilation, format

Motivation for this change
More cross-compilation support!
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I've also diffed the output for native using diffoscope (before and after this change), the only difference is the order of paths embedded in the executable.
Note: targeting staging due to zstd cross-compilation being broken in master (https://github.com/NixOS/nixpkgs/pull/129873 fixes this). It's impossible to check compilation on master, on staging it's at least possible.

pkgsCross.aarch64-multiplatform.yabar builds for me, yay!


Add some improvements to the Gatsby Open Preview button

We can fetch the exact path e.g. blog/post from the preview instance, so attempt to fetch or fallback to instance parameters.
This requires gatsby 2.4.7 users using prior versions will get a CORS error and the behavior will fallback to the original instance parameter method

Hello!
This is a fairly substantial improvement for some of our Gatsby Preview customers (using Contentful!), so we'd love a review on this so we can get it in their hands.
Looks like @floelhoeffel has been requested--let either @abhiaiyer91 or myself know if we can help with anything!

Sorry for the delay guys. I get an engineer to review before end of week.

@jelz addressed your feedback; thanks!

@jelz ✅ to your feedback; thanks!

@DSchau one last small comment.
To answer your question from the commit message - yes, you still need to commit the built bundle. We're thinking about automation here.

@jelz @abhiaiyer91 will test soon when he gets a chance--but this seems good to go from my end. Thanks!

ok, just ping me when done testing

@jelz everything is running great! We're ready! Thanks everyone!


Add responsiveness on quickstart-json page

Fixes Issue
solves #7918

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Please remove the package-lock file from this PR, it is not needed. Otherwise, testing in Gitpod this behaved as expected. Thank you for contributing

Done.
Happy to be a part of this community.

@amandamartin-dev good spot, yes the side nav needs an issue to change it to a dropdown for mobile, otherwise we can't reach the page - I was meant to raise it and forgot 🤦♂
Great collaboration 💪

@Shubham185y the package-lock.json file should not be deleted, your changes to the file are not needed as part of your PR, please undo your changes to this file.
I made a video on how you can fix the lock file https://www.youtube.com/watch?v=jHkIXYVpMTo

@SubhamB2003 did you mean to close the PR?

@SubhamB2003 did you mean to close the PR?

No sorry.

Looks good! Thanks for removing the lock file changes from the PR. Sorry for confusing you about deleting it versus removing the changes from the PR. THanks for your contributions!

Yeah, It's alright.


Can't run in standard user (without admin right)

I tested v2ray-core client on windows x64 as a standard user (without admin right). It didn't work. It stays on line "A unified platform for anti-censorship."

All right. Did you specify the listening port in Windows global proxy
settings?

From: AliM1988 @.>
To: v2fly/v2ray-core @.>
CC: Subscribed @.*>
*Date: *Nov 24, 2022 13:00:17
*Subject: *[v2fly/v2ray-core] Can't run in standard user (without admin
right) (Issue #2167)

I tested v2ray-core client on windows x64 as a standard user (without
admin right). It didn't work.
—
Reply to this email directly, view it on
GitHub[https://github.com/v2fly/v2ray-core/issues/2167], or
unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYBAVLLD5KKQON6EBPDWJ5RGBANCNFSM6AAAAAASKL75VA].
You are receiving this because you are subscribed to this
thread.[Tracking
image][https://github.com/notifications/beacon/AKGBAYGLEFHOHYHRPDNVIATWJ5RGBA5CNFSM6AAAAAASKL75VCWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHFOOCXVA.gif]Message
ID: @.***>

Yes, I set http=<IP_ADDRESS>:10809 and socks=<IP_ADDRESS>:10808


PipeReader.AdvanceTo(int,int) does not exist

reader.AdvanceTo(buffer.Start, buffer.End); does not exist anymore. It only accepts SequencePosition. Can you please provide an updated version for the code?
I am using     <PackageReference Include="System.IO.Pipelines" Version="5.0.1" />
Thank you!

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

ID: 2f5384a7-5e96-d977-7748-317a2c758f63
Version Independent ID: 06e72212-a052-a351-0e0b-1b002ce214de
Content: I/O pipelines - .NET
Content Source: docs/standard/io/pipelines.md
Product: dotnet-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@JKamsker The docs are talking about PipeWriter.Advance(int), while your code is using PipeReader, which indeed does not have an Advance(int) or AdvanceTo(int) method.

@svick Oh, sorry for the confusion.
But still, in the docs, they appearantly use reader.AdvanceTo(int,int) which does not exist.

@JKamsker I'm not sure it's what you're looking for, but if you have a ReadOnlySequence<byte> buffer, you can do buffer.GetPosition(someInt) to get a SequencePosition.


Why do u use Disruptor

Hi, i am reading your project, i know you are using Disruptor, why do you choose it for middleware ? can u tell me what is Pros and Cons of Disruptor in the real world and your project ? Thank u

Well, thank you for your reading my project. The project is just Disruptor's source code comment because I'm learning Disruptor and I want to know The Disruptor how runs. And my English is not well, sorry, but I will give you some feedback and try my best.
q: Why do you choose it for middleware?
a: I choose it because my company uses Disruptor to handle messages to improve performance.  The Disruptor is a high-performance message queue in memory, compare to BlockingQueue, The Disruptor has no lock and doesn't need blocking, so it's faster.
q: What are the Pros and Cons of Disruptor in the real world and your project?
a: The pros are Disruptor has high performance, it can handle 6 million orders per second when choosing YieldingWaitStrategy as its WaitStrategy. The Cons is Disruptor needs more CPU when choosing YieldingWaitStrategy because the YieldingWaitStrategy is implemented by using spinning. if you set the wrong parameter, your servers may be CPU up to 100% always and your server can't do other jobs.


[Test][RayJob] Transition to Complete if the JobStatus is STOPPED

Why are these changes needed?
This is the test for #1855.
Related issue number
Closes #1866
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

cc @astefanutti @andrewsykim


Mentorship button next to theme toggle is not required

Description
Problem

Landing page already have Join Mentorship button.
The one next to theme toggle is uncalled-for and confuses as both the Mentorship buttons links to different things

Solution

We can remove it to make landing page look clearer.

I can remove it, if its assigned to me.

I would like this issue to be assigned to me.

@barkha-gupta Actually I have updated the button and its Join Discord Community button now. I have raised a pr for the same

@khushijohri2001 I understand. I mentioned the button to be removed all together. I am just waiting for the issue to be assigned by following the guidelines.

hey @malthaf9 please wait for maintainer to assign the issue this will create unnecessary confusion among contributors and duw to this many contributors can end up working on the same issue

the mentorship button text is being changed in #222 so we don't have to remove the button alltogether

Closing this as not planned


STORM-914: AutoHDFS should also update hdfs.config into Configuration

PR for STORM-914
When storm user use a specify hdfs cluster, he will put some config such as namenode.host into hdfs.config when submit storm topology, and when HdfsBolt create HdfsConfiguration, it will use hdfs.config update HDFSConfiguration, then HDFSBolt will visit the right hdfs cluster. But in AutoHDFS's renew and getHadoopCredentials, Configuration not for the HDFS that storm user specify, we should update all the Configuration with hdfs.config.

@revans2 Will you take a look at this, thanks a lot.

@tedxia This PR is rather stale. Is it still being worked or should we close it?


Update README.md

fixes some typo mistakes

@googlebot I signed it!

@googlebot I signed it!

Thanks for the contribution, couple of small nits.

Ty , review It again plz

Hi, you resolved the comments but did not make changes, correct?

Thanks both. I am going to close this PR as the changes are not relevant and impactful enough. Feel free to have a look at the open issues and pick one of those. Happy to review a more substanstial PR.


Current/Open Reservations

The home page should display differently if there are current loans out.
Each loan should have its own bootstrap card and the button to create a new loan should be smaller and in the bottom left.
The current loan card should have some basic details about the loan and clicking on it should open a modal that allows you to progress the loan.
If the loan requires a decision, the card component should have a yellow notification flag on the top right corner of it denoting that something is necessary to progress
If the loan is late or undelivered, then there should be a different red notification flag that shows there's a problem.

duplicate with #22


Create a dummp MorphTargetBuffer if skinning is enabled

Fixes #5085

Nice to see you a fox again!

I'm closing this PR because it is not the right approach, this logic should not be in gltfio.

@pixelflinger Yes it's just W/A. I don't know where is the best place to create dummy resource.


Check that workshop URL is to repo not to site

We should at least warn people when they enter a something.github.io/yyyy-mm-dd-site URL instead of the github.com/something/yyyy-mm-dd-site URL that we want.

On Sat, May 30, 2015 at 12:22:40PM -0700, Greg Wilson wrote:

We should at least warn people when they enter a
something.github.io/yyyy-mm-dd-site URL instead of the
github.com/something/yyyy-mm-dd-site URL that we want.

Hmm.  Why do we care about this distinction?  For building lists of
workshops, links to the rendered website (something.github.io/…) seem
to make more sense.  Are we cloning or performing other Git-based
activity on the event repositories?  I don't think we are, in which
case I don't mind if the event website is versioned in Mercurial (for
example) or not versioned at all.  If we're relying on the presense of
specific files (e.g. the YAML header of index.html), we should be able
to scrape what we need from the meta tags:
$ curl http://swcarpentry.github.io/2015-05-30-hastac-swc/ | grep 'meta name'

Not all headers are returned as <meta> tags. I think Greg's right about this one.

On Fri, Jun 05, 2015 at 09:05:48AM -0700, Piotr Banaszkiewicz wrote:

Not all headers are returned as <meta> tags.

So just add (and document) the ones we need 1?
Alternatively, we could have folks enter the URL to a YAML file with
the required metadata in the first YAML document, so folks using
workshop-template could point us at their 2, and folks using other
websites (or no website at all) could throw up some YAML and point us
there.

I would actually far prefer having a link to the website, not the repo. The repo is basically useless to me as an admin, and the website is fantastically useful to confirm dates, location, instructors, registration etc.
If we don't store the website URL, at least can we generate a link to the website on the "event" display page?

If we don't store the website URL, at least can we generate a link to the website on the "event" display page?

I'll add it tomorrow.

Ooops, forgot to add this to the #373… I'll create a new PR for this.

@amyrbrown would you rather have that link to the website displayed on All Events page, or on event details page?
Right now I have something like this:

so you have both links on the event details page.

On the Event Details page is fine with me -- I don't use the All Events page much. Thank you!


Remove SetupCard

This no longer needs to exist as something distinct from EvalCard since we run the deck forwards from the point of a new card after adding it anyway, which amounts to the same thing as what SetupCard was used for.
There are some special cases that may need restoring that I've commented out for now, but I don't think they were contributing anything meaningful anyway, since all the tests that used to pass still pass without this.

Great, I'm happy to see this go...


How does it work? Any API available?

Hi,
I like the idea of a simple open source AR SDK-ish library.. but reading your README doesn't really give me an idea of how to work with it... I'm looking for a solution where i have my dashboard in which i can add an image (which is uploaded/converted into a Marker, underwater) and a mobile app (Appcelerator Titanium and cordova based for now) that can scan the same image printed on a info sign i.e.  a zoo...

Any leads are appreciated.

yes there is an API avaible. it is an opensource project
here is a link with a tutorial on how to use it https://aframe.io/blog/arjs/

Wow, quick response! nice! So if I understand correctly, it is not possible to create image markers without the big fat black border... which is a shame since Wikitude can. (kind of looking for a cheaper solution with sort of the same features, at least for the looks of the markers... )

You are correct, currently image based marker arent supported by AR.js.

Are there any three.js arjs API documentations?


Update the version of Effeckt on npm

We have an official npm package but it's really outdated. Let's bump it.
@paulirish Mind adding me (or anyone else from @h5bp/effeckt-css) to the npm package so we can bump the version?

@paulirish I was on but seem to have been removed?

to be honest, I didn't know we had a npm package.

Not exactly what you're asking for but i'm going to unpublish the npm package for now.

:+1:


Bump the CocaoPod

@tristanhimmelman could you release the bug fixes in a new cocoa pod please? thanks

Yep, just did it! You should be good to go

:+1:


Update ug pictures

Standardize screenshots

Codecov Report

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

@@            Coverage Diff            @@
##             master     #223   +/-   ##
=========================================
  Coverage     83.20%   83.20%
  Complexity     1077     1077
=========================================
  Files           148      148
  Lines          3167     3167
  Branches        381      381
=========================================
  Hits           2635     2635
  Misses          441      441
  Partials         91       91

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 29087bb...39a8538. Read the comment docs.


style: merge editor style fix

Types

[x] 💄 Style Changes

Background or solution

🤖[deprecated] Generated by Copilot at ffe34b4

Fix scroll bar overlap bug in merge editor by changing padding-bottom to margin-bottom for .merge-editor class (link)

Changelog

style: merge editor bottom action buttons position fix

🤖[deprecated] Generated by Copilot at ffe34b4
Fixed a UI bug in the merge editor that caused the content to overlap with the scroll bar. Changed the CSS property padding-bottom to margin-bottom in merge-editor.module.less.

/next

@AhkunTa 加一下优化前后对比图？


Modify some syntax

Modify some syntax

Upgrade the black version in pre commit: solve known problems https://github.com/psf/black/issues/2964

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

Beautiful!


Can't remove delete line in editor

Informations
Can't remove delete line on content in editor
Browser version:
Firefox 62 (64Bit)
OS:
Windows 10
Screen resolution:
2560x1440 (150% DPI)
How to reproduce the bug?

Use the demo at https://alex-d.github.io/Trumbowyg/
Selete some text in the editor and click Deleted in toolbox
Click Deleted again, but you can't remove the delete line

Thank you.
Best Wish.

Duplicates #293


Form

In this milestone I;

added a method that takes the first user in the DB.
added forms to add new posts, add new comments, and likes.
added a method for simulating the first user creating new posts

@MeqdamAlqudah Thank you for reviewing my project. I will work on the post view asap

@MeqdamAlqudah I see that my project is approved on the dashboard. But I have added the changes. Please let me if I can merge or do more corrections.
Thank you

@MeqdamAlqudah Thank you for the review. I will work on the optional suggestions before merging


Add AlmaLinux OS 8.7

@LKHN Thanks for the changes, looks good
Signed-off-by: Elkhan Mammadli<EMAIL_ADDRESS>

Thank You!


Why do you not use derive?

It will simplify the code a lot.

The non-derive implementations on Tree are to handle the id. The other ones, mostly on NodeId, are because the implementations don't actually depend on T. For example, the code has impl<T> Clone for NodeId<T>, but #[derive(Clone)] would generate impl<T: Clone> Clone for NodeId<T>.

I see. Makes sense.


Methods to find length of .gif and if it is animating

The regular [UIImage isAnimating] method is not working and it would be great if I could get the total length of an animated gif. -1 if it is an infinite animation. Finally, the ability to stop/start an animation at a given frame would be great.

Now in latest code gif is animate for one time


SB-17930

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-2.8.0@d6b4417). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release-2.8.0     #808   +/-   ##
================================================
  Coverage                 ?   40.98%
  Complexity               ?     1020
================================================
  Files                    ?      149
  Lines                    ?    10132
  Branches                 ?     1123
================================================
  Hits                     ?     4153
  Misses                   ?     5560
  Partials                 ?      419

Impacted Files
Coverage Δ
Complexity Δ

...sunbird/services/sso/impl/KeyCloakServiceImpl.java
0% <ø> (ø)
0 <0> (?)

...rd/common/util/KeycloakRequiredActionLinkUtil.java
0% <ø> (ø)
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 d6b4417...6d0e3f2. Read the comment docs.


Ported Spec samples to Pharo 9 (old Spec 1 yet)

Add preferences button and menu items

Coverage decreased (-0.2%) to 52.255% when pulling d66136d7405257684ab0e7fdc8e89516a6abcfca on iss_port_pharo9 into cf4f4fbdc8c33ed82f8eb47ade776301be4e5ef2 on master.


Flutter instal error using flavors

I try install my app in any IOS device (Simulator and real device) get's me this error. I'm using flavors in this project, but when i build and install for android everything works fine.
Using ios debug and flutter run works fine as well
Flutter install error:
Install failed

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      InstallCommand.runCommand (package:flutter_tools/src/commands/install.dart:44:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:478:18)
<asynchronous suspension>
#3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:383:33)
<asynchronous suspension>
#4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#5      _rootRun (dart:async/zone.dart:1124:13)
#6      _CustomZone.run (dart:async/zone.dart:1021:19)
#7      _runZoned (dart:async/zone.dart:1516:10)
#8      runZoned (dart:async/zone.dart:1463:12)
#9      AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#10     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:375:20)
#11     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:396:21)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#14     _rootRun (dart:async/zone.dart:1124:13)
#15     _CustomZone.run (dart:async/zone.dart:1021:19)
#16     _runZoned (dart:async/zone.dart:1516:10)
#17     runZoned (dart:async/zone.dart:1463:12)
#18     AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#19     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#20     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#21     new Future.sync (dart:async/future.dart:224:31)
#22     CommandRunner.run (package:args/command_runner.dart:112:14)
#23     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#24     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:22)
<asynchronous suspension>
#25     _rootRun (dart:async/zone.dart:1124:13)
#26     _CustomZone.run (dart:async/zone.dart:1021:19)
#27     _runZoned (dart:async/zone.dart:1516:10)
#28     runZoned (dart:async/zone.dart:1500:12)
#29     run.<anonymous closure> (package:flutter_tools/runner.dart:60:18)
<asynchronous suspension>
#30     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#31     _rootRun (dart:async/zone.dart:1124:13)
#32     _CustomZone.run (dart:async/zone.dart:1021:19)
#33     _runZoned (dart:async/zone.dart:1516:10)
#34     runZoned (dart:async/zone.dart:1463:12)
#35     AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#36     runInContext (package:flutter_tools/src/context_runner.dart:56:24)
<asynchronous suspension>
#37     run (package:flutter_tools/runner.dart:51:10)
#38     main (package:flutter_tools/executable.dart:62:9)
<asynchronous suspension>
#39     main (file:///Users/pedro_myeong/Desktop/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#40     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#41     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

Flutter doctor -v:
[✓] Flutter (Channel stable, v1.7.8+hotfix.3, on Mac OS X 10.14.6 18G103, locale en-US)
    • Flutter version 1.7.8+hotfix.3 at /Users/pedro_myeong/Desktop/flutter
    • Framework revision b712a172f9 (5 months ago), 2019-07-09 13:14:38 -0700
    • Engine revision 54ad777fd2
    • Dart version 2.4.0

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/pedro_myeong/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Library/Java/JavaVirtualMachines/jdk-10.0.1.jdk/Contents/Home/bin/java
    • Java version Java(TM) SE Runtime Environment 18.3 (build 10.0.1+10)
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.

[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.3, Build version 10G8
    • CocoaPods version 1.7.5

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/setup/#android-setup for detailed instructions).

[✓] Connected device (1 available)
    • iPhone de Pedro • f6de5020e89defb2cb58089bb965423ada8d2054 • ios • iOS 13.1.3

! Doctor found issues in 2 categories.

Hi @PedroTheRealOne
does this solution  works for you?
thank you

Hi @PedroTheRealOne
does this solution  works for you?
thank you

Doesn't work for me

In flutter install :
[   +1 ms] $(PRODUCT_BUNDLE_IDENTIFIER)
[   +7 ms] Installing br.com.nexusoficial.lojista to iPhone 7...
[   +1 ms] executing: /usr/bin/xcrun simctl get_app_container 52D0E992-73E0-4E1E-A6A4-87D6710C6E5A br.com.nexusoficial.lojista
[ +347 ms] executing: /usr/bin/xcrun simctl install 52D0E992-73E0-4E1E-A6A4-87D6710C6E5A build/ios/iphonesimulator/Runner.app
[ +242 ms] An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=2):
                    lstat of /Users/pedro_myeong/Documents/Quantum/Projects/nexus/nexus-lojista/build/ios/iphonesimulator/Runner.app failed: No
                    such file or directory
                    No such file or directory

Not finding the correct bundle. In the flavors i change the app bundles to .dev, .hmg. I suppose he is not changing my app bundle

I build and install in a real device and works fine.
This problems only occurs when i try to install in a simulator.
My build doesn't generate this path: build/ios/iphonesimulator/Runner.app, generate this path: build/ios/iphoneos/Runner.app
There is any config that change this?

You can't install an application to a phone with flutter install, and build iOS does not build for a simulator. We should provide a better error message here


how do i build this with cmake (im a bit retarted)

Install the necessary tools and invoke the build script (https://github.com/RealEthanPlayzDev/LuauCeption/blob/main/utils/Build.sh)


WIP: 'existing' keyword

Contributing a Pull Request
If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Contributing a feature

[ ] I have opened a new issue for the proposal, or commented on an existing one, and ensured that the bicep maintainers are good with the design of the feature being implemented
[ ] I have included "Fixes #{issue_number}" in the PR description, so GitHub can link to the issue and close it when the PR is merged
[ ] I have appropriate test coverage of my new feature

Fixes #258

I think more "natural" syntax would be existing resource res 'Provider/Type@version = { } or even just existing res 'Provider/Type@version = { } (without resource)

I think more "natural" syntax would be existing resource res 'Provider/Type@version = { } or even just existing res 'Provider/Type@version = { } (without resource)

I think more "natural" syntax would be existing resource res 'Provider/Type@version' = { } or even just existing res 'Provider/Type@version' = { } (without resource keyword)

Please see #258 for syntax discussion - this PR is just to implement the design we closed on. I do think there's scope for changing this and definitely share some of your thoughts here, but it'll have to come in as a later revision.

I think more "natural" syntax would be existing resource res 'Provider/Type@version' = { } or even just existing res 'Provider/Type@version' = { } (without resource keyword)

Please see #258 for syntax discussion - this PR is just to implement the design we closed on. I do think there's scope for changing this and definitely share some of your thoughts here, but it'll have to come in as a later revision.

Codecov Report

Merging #1339 (5d1c1d5) into main (984f559) will increase coverage by 0.37%.
The diff coverage is 93.04%.

@@            Coverage Diff             @@
##             main    #1339      +/-   ##
==========================================
+ Coverage   94.83%   95.21%   +0.37%
==========================================
  Files         343      340       -3
  Lines       17450    17439      -11
  Branches       14        0      -14
==========================================
+ Hits        16549    16604      +55
+ Misses        901      835      -66

Flag
Coverage Δ

dotnet
95.21% <93.04%> (-0.16%)
:arrow_down:

typescript
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...Core.IntegrationTests/Emit/TemplateEmitterTests.cs
100.00% <ø> (ø)

.../Bicep.Core/TypeSystem/Az/AzResourceTypeFactory.cs
90.74% <ø> (-1.26%)
:arrow_down:

...cep.Core/TypeSystem/Az/ManagementGroupScopeType.cs
80.00% <0.00%> (-20.00%)
:arrow_down:

...Bicep.Core/TypeSystem/Az/ResourceGroupScopeType.cs
80.00% <0.00%> (-20.00%)
:arrow_down:

.../Bicep.Core/TypeSystem/Az/SubscriptionScopeType.cs
80.00% <0.00%> (-20.00%)
:arrow_down:

src/Bicep.Core/TypeSystem/Az/TenantScopeType.cs
60.00% <0.00%> (-20.00%)
:arrow_down:

src/Bicep.Core/TypeSystem/ModuleType.cs
83.33% <0.00%> (-16.67%)
:arrow_down:

src/Bicep.Core/TypeSystem/ResourceType.cs
87.50% <0.00%> (-12.50%)
:arrow_down:

src/Bicep.LangServer/SemanticTokenVisitor.cs
0.00% <0.00%> (ø)

src/Bicep.Core/Emit/EmitLimitationCalculator.cs
83.07% <66.66%> (-6.78%)
:arrow_down:

... and 47 more

Codecov Report

Merging #1339 (5d1c1d5) into main (984f559) will increase coverage by 0.37%.
The diff coverage is 93.04%.

@@            Coverage Diff             @@
##             main    #1339      +/-   ##
==========================================
+ Coverage   94.83%   95.21%   +0.37%
==========================================
  Files         343      340       -3
  Lines       17450    17439      -11
  Branches       14        0      -14
==========================================
+ Hits        16549    16604      +55
+ Misses        901      835      -66

Flag
Coverage Δ

dotnet
95.21% <93.04%> (-0.16%)
:arrow_down:

typescript
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...Core.IntegrationTests/Emit/TemplateEmitterTests.cs
100.00% <ø> (ø)

.../Bicep.Core/TypeSystem/Az/AzResourceTypeFactory.cs
90.74% <ø> (-1.26%)
:arrow_down:

...cep.Core/TypeSystem/Az/ManagementGroupScopeType.cs
80.00% <0.00%> (-20.00%)
:arrow_down:

...Bicep.Core/TypeSystem/Az/ResourceGroupScopeType.cs
80.00% <0.00%> (-20.00%)
:arrow_down:

.../Bicep.Core/TypeSystem/Az/SubscriptionScopeType.cs
80.00% <0.00%> (-20.00%)
:arrow_down:

src/Bicep.Core/TypeSystem/Az/TenantScopeType.cs
60.00% <0.00%> (-20.00%)
:arrow_down:

src/Bicep.Core/TypeSystem/ModuleType.cs
83.33% <0.00%> (-16.67%)
:arrow_down:

src/Bicep.Core/TypeSystem/ResourceType.cs
87.50% <0.00%> (-12.50%)
:arrow_down:

src/Bicep.LangServer/SemanticTokenVisitor.cs
0.00% <0.00%> (ø)

src/Bicep.Core/Emit/EmitLimitationCalculator.cs
83.07% <66.66%> (-6.78%)
:arrow_down:

... and 47 more


Ressource not found when setting up Author App in Teams

Following the guide and installing the app(s) went fine.
Adding the Cosumer part app even works fine in Teams.
However when trying to add the Authors App and trying to set ip up to a Team, where the UPN user allowed to author is owner of the Team fails with the following error:
The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.
What could have gone wrong here?

Unfortunately we did not find a solution up to day. Did you manage to solve it in the meantime?

Just redeployed Version 4.1.0 from scratch. Now it's working. Unfortunately I cannot tell what I did differently to the last deployment except Naming the app not like the default in the first place.


The event details popup does not appear if the attendee does not have a name.

Description:
If an event has attendees, and one of them doesn't have a name field, then the event details popup won't appear after clicking on the event cell.
Environment:

Operating System: macOS Sonoma
App Version: 0.15.1 (2262)
Xcode Version: 15 (15A240d)

Steps to reproduce:

Add an event with a list of attendees and ensure that one of them doesn't have a name field (mine is from Microsoft Teams calendar, and the attendee has only an email).
Click on the event cell to show the details popup.

Expected behaviour:
The event details popup appears.
Actual behavior:
Nothing happens, but an assertion failure occurs in AgendaViewController.m:1184.
*** Assertion failure in +[NSTextField labelWithString:], NSTextField.m:1 751
Invalid parameter not satisfying: stringValue != nil
	0   CoreFoundation                      0x00007ff80220e0b6 __exceptionPreprocess + 242
	1   libobjc.A.dylib                     0x00007ff801d03231 objc_exception_throw + 48
	2   Foundation                          0x00007ff8031f3766 -[NSCalendarDate initWithCoder:] + 0
	3   AppKit                              0x00007ff80596b0e0 +[NSTextField(NSTextFieldConvenience) labelWithString:] + 302
	4   Itsycal                             0x0000000101f8a453 -[AgendaPopoverVC populateWithEventInfo:] + 11027
	5   Itsycal                             0x0000000101f7f9e7 -[AgendaViewController showPopover:] + 663

Possible solution:
It turns out that the name property in EKParticipant is nullable. Therefore, I believe it would be better to check if it's nil and, if so, use an alternative (for example, an email).

Thanks for the report.
I'm kind of flying blind when it comes to attendees since I don't work with any kind of collaboration software and can't really generate good test cases.
Indeed, the code was simply using EKParticipant name. I've changed it to check for nil and fall back to EKParticipant URL.resourceSpecifier (email) or just a dummy string if that too is nil.
Can you check if this works?...
https://itsycal.s3.amazonaws.com/Itsycal-0.15.2-issue-239-1.zip

Thanks for the quick response!
I have checked your fix and now everything works as expected.


Txn version not supported message when trying to send KSM

Hi I am getting a Txn version not supported message when I am trying to send out my ksm using my ledger nano S and the polkadot.js.org website. I tried turning off the keep alive checks thinking it might help but it still shows this message. So currently I can't send my KSM out! I even tried changing my node from Parity to Onfinality.
It worked before on the 20th of this month (sept) which was the last time I sent KSM but now it doesn't? Why?
Thanks in advance.
Current Specs:
Ledger Nano S Firmware 2.1.0 latest
Ledger Kusama app Version 12.9250.0 latest (I checked on Ledger Live)
https://polkadot.js.org/ website
Windows 10
Chrome latest version

:link: zboto Link

Hi!
Ledger is currently reviewing our upgrade for the new version and should be available in a few days in Ledger Live!
Sorry for the inconvenience!

In the meantime, you could try using our release but as Carlo said, the official Ledger release will be available soon 🙂

In the meantime, you could try using our release but as Carlo said, the official Ledger release will be available soon 🙂

Thank you to both of you!
If I can't wait, how do I install the latest release you linked onto my Nano S? Is there a guide?

I'm afraid this installers are only available for macOS and Linux, which OS are you currently using?

I'm afraid this installers are only available for macOS and Linux, which OS are you currently using?

I am on Windows 10

But I have access to a Macbook

Just downloading and executing the script would be enough (you probably need to install ledgerblue using pip install ledgerblue)

Ledger released the new version in Ledger Live, we can close this!
Sorry for the inconvenience, and have a nice day! :)


Thanks for contributing!

          Thanks for contributing!

Someone will soon review your request.
Here is the JSON result for the reviewer:
{
  "name": "Infectious ( NoCopyrightSounds Mix )",
  "artist": "Tobu, 7obu, NoCopyrightSounds",
  "source": "youtube",
  "startOffset": 0,
  "yt-id": "MCxjJV9nQjw",
  "songs": [
    593702
  ]
}

To accept the request, a reviewer needs to write "accept".
Originally posted by @auto-nong[bot] in https://github.com/FlafyDev/auto-nong-indexes/issues/1170#issuecomment-2219708617

reject
This can not be accepted because it's missing everything.  Please make an new post with the right info.


Debloater not removing some apps on Windows 10 1909

Hi
Some applications are still present after passing the GUI debloater. Skype, all Microsoft Office applications, Onenote app, linkedin, Microsoft Solitaire Collection, Windows Feedback Hub, weather...
I'm on Windows 10 v1909
thx

Just to check but are  you running this on your own account? As far as I'm aware, it will only clean up new or unused user accounts. So basically the default user profile. Then every new user that is created will get a nice clean, debloated experience. It doesn't work on existing accounts though.  (in the past that was case, unless somethings changed)

Check your execution-policy with Get-ExecutionPolicy -List.
Check if your currentUser is set to unrestricted.
If not set unrestricted with Set-ExecutionPolicy Unrestricted -Force -Scope CurrentUser

When I run the scripts post installation, the Execution-Policy is -Scope Process Bypass I have also run it -Scope Process Unrestricted, both seem to have no positive effect when running these scripts.


Symbolic execution support

Adding symbolic execution support (meaning some memory spaces don't have bytes, they just have a symbol which can contain arbitrary data depending on the problem being solved (e.g. they can contain the tree arg0 + arg1 if these args are unknown))
This is useful for:

testing functions for memory bugs without having known input arguments
implementing const generic unification (so knowing that C + C + 1 is the same as 2 * C + 2 - 1 without knowing C)
evaluating associated constants of traits with generic parameters

I recall having seen a fork of miri doing this some time ago, but I can't recall its name.

@Bobo1239 thanks, that's it. :+1:

Woah... how do you think that will be possible without making everything painfully abstract or painfully slow?^^
I mean, I agree with the goal, I just think it is impossible. Not that this has ever stopped anyone in Rust land ;)

We already have the Machine abstraction. It would of course now have to infect Allocation and Scalar and bubble up from there. I just opened the issue to not forget about the use cases.

Okay. Currently, Allocation I think goes through some lengths to NOT depend on the Machine.
Also, a lot of code assumes it can turn a scalar into an integer and work with that.

Yes, by symbolic execution you may prove/reject  "const generic unification (so knowing that C + C + 1 is the same as 2 * C + 2 - 1 without knowing C."  (Problems may arise due to panicing behavior, which may pose a "cut", which leads to partial correctness/equivalence, etc. so the question is what we mean by "same as".
The problem with Seer (and symbolic execution in general) is that it involves an (SMT) solver for the path constraints (under partially symbolic assignments). This may lead to vast number of queries, each one complex to solve, thus may be computationally expensive. However, if applied only to cases  of const generics, it would not affect the overall compiler performance (besides const generics I mean), so it could be price we are willing to pay.
For finding memory bugs (array indexing etc.), it could be done either for a specific application or on library functions. In the former case, the application implies assumptions on the path conditions, and can therefore prove the absence of memory bugs for specific use cases, see e.g., No Panic! https://ieeexplore.ieee.org/abstract/document/8471992, where we apply KLEE for symbolic execution of rust code at LLVM-IR level.
To the latter case (library code), one can prove absence of memory bugs as well, but that would likely require contracts, stating the assumptions. A primitive way is to use "klee-assume", but that does not give us quantification, thus we cannot generalize (klee assumptions are on the form of expressions.) With Seer and integration to the Rust compiler/language we may do better (adding path constraints on the form of first order logic, which would allow for quantification).
In any case bringing in symbolic execution would add extra dependencies to the rust compiler (e.g., Z3 in the case of Seer).
Regarding const function evaluation, I assume there is still a lot of untapped potential with MIRI, I'm not sure the status on that (what is in the pipeline for the compiler team).

On another note regarding symbolic execution, where it really hurts is when introducing lots of data dependencies causing path explosion, e.g. bubble sort. But if we are interested only in finding memory bugs, not the actual results of execution, each path needs to be visited just once (under the least restrictive constraint possible), so perhaps that is doable. KLEE at least has some notion of "visit once". If the purpose is just bug finding, one could think of symbolic execution as a superpowered lint pass in the compiler (an opt in pass as a complement to ordinary regression testing).

Problems may arise due to panicing behavior, which may pose a "cut", which leads to partial correctness/equivalence, etc. so the question is what we mean by "same as".

It wouldn't be enough to assume that two functions are equal if they panic for the same inputs?

Yes, intuitively, but I'm not sure if there might be something lurking. But I think its worth further exploring. Another thing that came to mind. There are some limitations to the Seer implementation, if I recall correctly it does not cover all legal Rust programs.

I think Kani and other MIR based projects have this well covered and we don't need to try to add it to miri.


Support python <3.6

I use 3.6 at home, but at work I use some machines that are still only running 3.5. It would be nice to be able to use this without having to maintain user-local python installs for each one.
Would it be possible to revert fe8244d48c86d66d827408b069e631bdf6f605f8?

Can you describe your use case more clearly?
1.0.0 supports Python 3.5.  That release is available on Github and PyPI.

Use case is I want to use this package without needing Python 3.6 installed. 1.0.0 works; thanks.


Support versioned buckets/objects in AWS plugin

AWS lets you version S3 objects. We should have some way of exposing this information in the AWS plugin. An initial layout could be something like
s3Object
  version1
  version2
  version3
  ...

where cat'ing the object yields the latest version's content. However, that would make s3Object both a file and directory, which would make it more difficult to work with FUSE.

GCP may also have versioned objects/buckets.
In a more general sense, versioned entries could be possible. For VMs, these are snapshots. For S3 objects, these are previous object versions. But that is beyond the scope of this work, and a little too early.


[JA] Translations

Translated by Alconost

Lint warning are License/Copyright (inherited from English counterparts)

@pcoet Can this be merged? There doesn't seem to be anything out of the ordinary here?


Open ERROR: Can not open the file as [FAT] archive

Title is what is says when i try to use this tool

I also get this error, in both the Heleper and the file manager for 7zip.
It does mount properly using OSFMount 2.0.1001 so it might be 7zip not supporting this specific FAT image.

It seems I had 7zip version 18.1 and updated to 19.0. After running the Helper again it works properly. You are probably using an outdated 7zip version and should update to latest 7zip.

Problem Solved, Thanks Kodtiz3D

Added 7-Zip version requirements to readme. Closing...


added ability to specify goalposes in requestAcquirePosition

Types of PR

[ ] New Features
[x] Upgrade of existing features
[ ] Bugfix

Link to the issue
Description
Write the goal poses specified in requestAcquirePosition on the blackboard.
How to review this PR.
Others

Please also add setter/getter function for plugin base class.
https://github.com/tier4/scenario_simulator_v2/blob/5d7c674944785baab6a31e88a65aa544b99fb4b8/simulation/traffic_simulator/include/traffic_simulator/behavior/behavior_plugin_base.hpp#L57-L76

can you also add goal_poses to vehicle entity for future work?


fix linkedList.Peek for stack

Codecov Report
Base: 96.60% // Head: 96.61% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (0ecec6a) compared to base (cbb7d06).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #26   +/-   ##
=======================================
  Coverage   96.60%   96.61%
=======================================
  Files          35       35
  Lines        2064     2068    +4
=======================================
+ Hits         1994     1998    +4
  Misses         64       64
  Partials        6        6

Impacted Files
Coverage Δ

collections/linked_list.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Azure DevEx Architecture Board: Review of IC3 ACS Calling

New SDK Review meeting has been requested.
 In a multi-party call API events can identify which meeting participants are currently speaking. Active speakers identify which participants are being heard in each received audio frame. Dominant speakers identify which participants are currently most active (or 'dominant') in group conversation, even though their voice may not be heard in every audio frame. The set of dominant speakers can change as different participants take turns speaking.”
Detailed meeting information and documents provided can be accessed here

APIViews:
Android: https://apiview.dev/Assemblies/Review/4dd7657657474562bbb20883dea70253/
iOS: https://apiview.dev/Assemblies/Review/4042eb5aea8f4a16a480a598633c6b01/4da2132c783d490eb11db9172b82b042
Windows: https://apiview.dev/Assemblies/Review/aa5678020922463b9d257d10fd4af7f3
Hero Scenarios:
Android:
First need to obtain the Dominant Speakers feature API object:
DominantSpeakersFeature dominantSpeakersFeature = call.feature(Features.DominantSpeakers);

The Dominant Speakers feature object have the following API structure:

OnDominantSpeakersChanged: Event for listening for changes in the dominant speakers list.
getDominantSpeakersInfo(): Gets the DominantSpeakersInfo object. This object has:
getSpeakers(): A list of participant identifiers representing the dominant speakers list.
getTimestamp(): The date when the dominant speakers list was updated.

To subscribe to changes in the Dominant Speakers list:
// Obtain the extended feature object from the call object.
DominantSpeakersFeature dominantSpeakersFeature = call.feature(Features.DominantSpeakers);
// Subscribe to the OnDominantSpeakersChanged event.
dominantSpeakersFeature.addOnDominantSpeakersChangedListener(handleDominantSpeakersChangedlistener);

private void handleCallOnDominantSpeakersChanged(PropertyChangedEvent args) {
    // When the list changes, get the timestamp of the last change and the current list of Dominant Speakers
    DominantSpeakersInfo dominantSpeakersInfo = dominantSpeakersFeature.getDominantSpeakersInfo();
    Date timestamp = dominantSpeakersInfo.getTimestamp();
    List<CommunicationIdentifier> dominantSpeakers = dominantSpeakersInfo.getSpeakers();
}

iOS:
First need to obtain the Dominant Speakers feature API object:
let dominantSpeakersFeature = call.feature(Features.dominantSpeakers)

The Dominant Speakers feature object have the following API structure:

didChangeDominantSpeakers: Event for listening for changes in the dominant speakers list.
dominantSpeakersInfo: Which gets the DominantSpeakersInfo object. This object has:
speakers: A list of participant identifiers representing the dominant speakers list.
timestamp: The date when the dominant speakers list was updated.
To subscribe to changes in the dominant speakers list:

// Obtain the extended feature object from the call object.
let dominantSpeakersFeature = call.feature(Features.dominantSpeakers)
// Set the delegate object to obtain the event callback.
dominantSpeakersFeature.delegate = DominantSpeakersDelegate()

public class DominantSpeakersDelegate : DominantSpeakersCallFeatureDelegate
{
    public func dominantSpeakersCallFeature(_ dominantSpeakersCallFeature: DominantSpeakersCallFeature, didChangeDominantSpeakers args: PropertyChangedEventArgs) {
        // When the list changes, get the timestamp of the last change and the current list of Dominant Speakers
        let dominantSpeakersInfo = dominantSpeakersCallFeature.dominantSpeakersInfo
        let timestamp = dominantSpeakersInfo.timestamp
        let dominantSpeakersList = dominantSpeakersInfo.speakers
    }
}

Windows:
First need to obtain the Dominant Speakers feature API object:
DominantSpeakersCallFeature dominantSpeakersFeature = (DominantSpeakersCallFeature)call.GetCallFeatureExtension(HandleType.DominantSpeakersCallFeature);

The Dominant Speakers feature object have the following API structure:

OnDominantSpeakersChanged: Event for listening for changes in the dominant speakers list.
DominantSpeakersInfo: Which gets the DominantSpeakersInfo object. This object has:
Speakers: A list of participant identifiers representing the dominant speakers list.
Timestamp: The date when the dominant speakers list was updated.

To subscribe to changes in the dominant speakers list:
// Obtain the extended feature object from the call object.
DominantSpeakersCallFeature dominantSpeakersFeature = (DominantSpeakersCallFeature)call.GetCallFeatureExtension(HandleType.DominantSpeakersCallFeature);
// Set the delegate object to obtain the event callback.
dominantSpeakersFeature.OnDominantSpeakersChanged += DominantSpeakersFeature__OnDominantSpeakersChanged;

private void DominantSpeakersFeature__OnDominantSpeakersChanged(object sender, PropertyChangedEventArgs args)
{
    DateTimeOffset date = dominantSpeakersFeature.DominantSpeakersInfo.Timestamp;
    IReadOnlyList<ICommunicationIdentifier> speakers = dominantSpeakersFeature.DominantSpeakersInfo.Speakers;
}

@garchiro7 the .NET API view is not showing a diff. Could you try again or reach out to our Eng Sys team: https://teams.microsoft.com/l/channel/19%3A3adeba4aa1164f1c889e148b1b3e3ddd%40thread.skype/APIView?groupId=3e17dcb0-4257-4a30-b843-77f47f1d4121&tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47

Hi @ronniegeraghty Thanks for pointing that out. You are right, for the Windows API there's no diff because the feature has already been published into BETA drops (but not reviewed). We want a review of this feature for BETA and GA purposes, but the Windows SDK has not GA'ed yet, so revisions are based on BETA API surfaces right now.
I've put comments on the classes / symbols needed for the review, is that good enough? Or do you think we should do something different?

No, It's never gone well when folks try to hack around a diff with just comments.  Types or members always end up getting missed when only using comments.  Reviewers miss things when looking at your API.  It slows down the review meeting trying to scroll around to find it all.  The "Approve" button ApiView was actually added for this - we kept having unreviewed APIs slipping into releases when diffs were handwaved.
Why can't we start with the GA baseline as the first revision and then add the Beta  as the second revision?  We need the GA API ready for future Calling reviews anyway.

Hi @tg-msft I created a brand new Windows apiview with the following revisions:

Rev 0 : Current state of our GA master branch.
Rev 1: Latest BETA release on nuget 1.0.0-beta.35.
Rev 2: GA master branch with the Dominant Speakers feature

https://apiview.dev/Assemblies/Review/090107fe99d34fc18c06478d0120b722

Recording (MS INTERNAL ONLY)


How to make IS4 add 'aud' claim with /resources value?

IS3 added aud : Is3BaseAddr/resources claim to access token for the requested scopes openid offline_access
But IS4 does not add aud claim at all unless client requests particular scope.
The main difference between IS3 and IS4 configurations is that in IS3 no resource is defined, so I made an experiment and removed resources, but still there is no audience among the claims.
How to make IS4 to add this value to access token?

I do not know where it is a best option, but I did it this way:
public sealed class TokenCreationWithDefaultAudienceService : DefaultTokenCreationService
    {
        private readonly Settings _settings;

        public TokenCreationWithDefaultAudienceService(
            Settings settings,
            ISystemClock clock, IKeyMaterialService keys, IdentityServerOptions options, ILogger<TokenCreationWithDefaultAudienceService> logger)
            : base(clock, keys, options, logger)
        {
            Guard.NotNull(settings, nameof(settings));

            _settings = settings;
        }

        public override Task<string> CreateTokenAsync(Token token)
        {
            if (token.Audiences.IsEmpty() && !_settings.DefaultAudience.IsNullOrEmpty())
            {
                token.Audiences.Add(_settings.DefaultAudience);
            }

            return base.CreateTokenAsync(token);
        }

        public sealed class Settings
        {
            public string DefaultAudience { get; }

            public Settings(string defaultAudience)
            {
                DefaultAudience = defaultAudience.NullIfWhiteSpace();
            }
        }
    }

why?

@leastprivilege, because IS3 Access Token audience validation requires resources.
We have APIs which use IS3 UseIdentityServerBearerTokenAuthentication

OK - well you above workaround looks fine to me.

Great work voroninp!  This one saved my bacon since we are making the jump from Idserver2 to 4, and running non core.

@deniedOne  We hit so many issues migrating from IS3 to IS4 that it took us about 2 months instead of initially estimated two weeks. But most of them were not caused by new version of IS per se, rather by other services each validating tokens in their own way. That was a mess.

I added a flag in the latest release to bring the /resources audience back for backwards compat.

https://github.com/IdentityServer/IdentityServer4/releases/tag/3.0.2

One more step for backwards compatibility

using IdentityModel;
using IdentityServer4.Services;
using IdentityServer4.Stores;
using IdentityServer4.Stores.Serialization;
using Microsoft.Extensions.Logging;
using System.Security.Cryptography;
using System.Text;

namespace Luscii.Identity.Core.Services
{
    public sealed class RefreshTokenStoreWithIS3KeyHashing : DefaultRefreshTokenStore
    {
        private static readonly HashAlgorithm Hash = HashAlgorithm.Create("SHA256");

        public RefreshTokenStoreWithIS3KeyHashing(
            IPersistedGrantStore store, IPersistentGrantSerializer serializer,
            IHandleGenerationService handleGenerationService, ILogger<RefreshTokenStoreWithIS3KeyHashing> logger)
            : base(store, serializer, handleGenerationService, logger)
        {
        }

        protected override string GetHashedKey(string value)
        {
            var bytes = Encoding.UTF8.GetBytes(value);
            var hashedBytes = Hash.ComputeHash(bytes);
            var hashedString = Base64Url.Encode(hashedBytes);
            return hashedString;
        }
    }
}

@leastprivilege Thanks Dom and not for just this issue, but for the product support.  I'm new to all of this so it's a lot to take in, but a lot of resources, examples, etc will surely get me going.


[ic-select]: Return an object as the value instead of string

Summary
At the moment, the value returned from the ic-select is a string. This ticket is to investigate if it is possible for the value returned to be an object if provided as part of the options.
💬 Description
At the moment, the value of the selected option is returned as a string. There is a requirement for the returned value to be an object. The object would include additional information.
{
   label: 'Foo',
   value: {
      type: 'bar',
   }
}

At the moment, the value of the option is stored in a hidden input that can be retrieved when submitting a form so this may need updating.
In addition to the basic select functionality, this will need to work with ic-select searchable. At the moment (for both default and searchable), the value from the selected menu option is used to match the value in the options object and then display the label getLabelFromValue. This may still be required for this enhancement but in addition, check for an new property (something like 'details') which can be emitted as part of icChange.
{
   label: 'Foo',
   value: 'foo',
   details: {
      type: 'bar'
   }
}

Another option is to allow:
{
   label: 'Foo',
   value: {
      type: 'bar',
   }
}

..and use getValueFromLabel to output value object without the need to match any value property and the hidden input will have the value as the option 'label'.
It is possible to emit additional information from the custom event (i.e. icChange).
💰 Use value
Enhances the use case of ic-select and is a requirement from the community.

A branch has been pushed up for continuous work. This work has been paused now.


[yeoman] generator- botbuilder needs to have a supporting build and release pipeline

We need a build pipeline and a release pipeline for generator-botbuilder.  this should be modeled in the way we do our JS modules (version information in package.json, is updated by the build process.  builds are published to botbuilder.myget until we move off of that service.  and lastly we need a release pipeline that will publish to npmjs.org

I can do this for you. Shall we meet and go over the details? I.e. source repo and build commands.
Thanks.
--Bruce
From: Scott Gellock<EMAIL_ADDRESS>Sent: Monday, July 22, 2019 10:29 AM
To: microsoft/BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
We need a build pipeline and a release pipeline for generator-botbuilder. this should be modeled in the way we do our JS modules (version information in package.json, is updated by the build process. builds are published to botbuilder.myget until we move off of that service. and lastly we need a release pipeline that will publish to npmjs.org
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FBotBuilder-Samples%2Fissues%2F1667%3Femail_source%3Dnotifications%26email_token%3DABAY6UT6PNT7VHU46KGWOWTQAXU4HA5CNFSM4IF2YJH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAWEC2A&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=vHpj7K7ifGjqQS3VEDEzrNJEt9CHNiFmyvjQYbBql5E%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABAY6UUP2A5KDG2LYJKNOZ3QAXU4HANCNFSM4IF2YJHQ&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=QeA3qsfuh7K3EcmElHhkx6nPJuZXmdV9JbxkCWKZppI%3D&reserved=0.

Let me know how I can help

I will get started on a build for https://github.com/microsoft/BotBuilder-Samples/tree/master/generators/generator-botbuilder
--BH
From: Scott Gellock<EMAIL_ADDRESS>Sent: Monday, July 22, 2019 10:29 AM
To: microsoft/BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
We need a build pipeline and a release pipeline for generator-botbuilder. this should be modeled in the way we do our JS modules (version information in package.json, is updated by the build process. builds are published to botbuilder.myget until we move off of that service. and lastly we need a release pipeline that will publish to npmjs.org
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FBotBuilder-Samples%2Fissues%2F1667%3Femail_source%3Dnotifications%26email_token%3DABAY6UT6PNT7VHU46KGWOWTQAXU4HA5CNFSM4IF2YJH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAWEC2A&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=vHpj7K7ifGjqQS3VEDEzrNJEt9CHNiFmyvjQYbBql5E%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABAY6UUP2A5KDG2LYJKNOZ3QAXU4HANCNFSM4IF2YJHQ&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=QeA3qsfuh7K3EcmElHhkx6nPJuZXmdV9JbxkCWKZppI%3D&reserved=0.

The build: BotBuilder-Generator-JS-daily https://fuselabs.visualstudio.com/SDK_v4/_build?definitionId=591&_a=summary
The release: Push-BotBuilder-Generator-JS-to-MyGet-daily https://fuselabs.visualstudio.com/SDK_v4/_release?definitionId=63&view=mine&_a=releases
The released package on MyGet: aitemplates - generator-botbuilder 4.6.0-preview.72039 https://botbuilder.myget.org/feed/aitemplates/package/npm/generator-botbuilder
The release definition for npmjs is coming soon.
Thanks.
-Bruce
From: Bruce Haley (Harvey Nash)
Sent: Monday, July 22, 2019 11:36 AM
To: microsoft/BotBuilder-Samples<EMAIL_ADDRESS>microsoft/BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: RE: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
I will get started on a build for https://github.com/microsoft/BotBuilder-Samples/tree/master/generators/generator-botbuilder
--BH
From: Scott Gellock <notifications<EMAIL_ADDRESS>Sent: Monday, July 22, 2019 10:29 AM
To: microsoft/BotBuilder-Samples <BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed <subscribed<EMAIL_ADDRESS>Subject: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
We need a build pipeline and a release pipeline for generator-botbuilder. this should be modeled in the way we do our JS modules (version information in package.json, is updated by the build process. builds are published to botbuilder.myget until we move off of that service. and lastly we need a release pipeline that will publish to npmjs.org
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FBotBuilder-Samples%2Fissues%2F1667%3Femail_source%3Dnotifications%26email_token%3DABAY6UT6PNT7VHU46KGWOWTQAXU4HA5CNFSM4IF2YJH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAWEC2A&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=vHpj7K7ifGjqQS3VEDEzrNJEt9CHNiFmyvjQYbBql5E%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABAY6UUP2A5KDG2LYJKNOZ3QAXU4HANCNFSM4IF2YJHQ&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=QeA3qsfuh7K3EcmElHhkx6nPJuZXmdV9JbxkCWKZppI%3D&reserved=0.

Hi Scott,
Completed:
The build: BotBuilder-Generator-JS-daily https://fuselabs.visualstudio.com/SDK_v4/_build?definitionId=591&_a=summary
The release: Push-BotBuilder-Generator-JS-to-MyGet-daily https://fuselabs.visualstudio.com/SDK_v4/_release?definitionId=63&view=mine&_a=releases
The released package on MyGet: aitemplates - generator-botbuilder 4.6.0-preview.72039 https://botbuilder.myget.org/feed/aitemplates/package/npm/generator-botbuilder
The prod release definition: [[PROD]]Push-BotBuilder-Generator-JS-to-npmjs https://fuselabs.visualstudio.com/SDK_v4/_apps/hub/ms.vss-ciworkflow.build-ci-hub?_a=edit-build-definition&id=591&view=Tab_Tasks
Thanks.
-Bruce
From: Bruce Haley (Harvey Nash)
Sent: Monday, July 22, 2019 11:36 AM
To: microsoft/BotBuilder-Samples <reply<EMAIL_ADDRESS>microsoft/BotBuilder-Samples <BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed <subscribed<EMAIL_ADDRESS>Subject: RE: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
I will get started on a build for https://github.com/microsoft/BotBuilder-Samples/tree/master/generators/generator-botbuilder
--BH
From: Scott Gellock <notifications<EMAIL_ADDRESS>Sent: Monday, July 22, 2019 10:29 AM
To: microsoft/BotBuilder-Samples <BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed <subscribed<EMAIL_ADDRESS>Subject: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
We need a build pipeline and a release pipeline for generator-botbuilder. this should be modeled in the way we do our JS modules (version information in package.json, is updated by the build process. builds are published to botbuilder.myget until we move off of that service. and lastly we need a release pipeline that will publish to npmjs.org
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FBotBuilder-Samples%2Fissues%2F1667%3Femail_source%3Dnotifications%26email_token%3DABAY6UT6PNT7VHU46KGWOWTQAXU4HA5CNFSM4IF2YJH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAWEC2A&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=vHpj7K7ifGjqQS3VEDEzrNJEt9CHNiFmyvjQYbBql5E%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABAY6UUP2A5KDG2LYJKNOZ3QAXU4HANCNFSM4IF2YJHQ&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=QeA3qsfuh7K3EcmElHhkx6nPJuZXmdV9JbxkCWKZppI%3D&reserved=0.

Oops, the prod release definition link below was incorrect. Now corrected.
From: Bruce Haley (Harvey Nash)
Sent: Monday, July 22, 2019 2:41 PM
To: microsoft/BotBuilder-Samples<EMAIL_ADDRESS>microsoft/BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: RE: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
Hi Scott,
Completed:
The build: BotBuilder-Generator-JS-daily https://fuselabs.visualstudio.com/SDK_v4/_build?definitionId=591&_a=summary
The release: Push-BotBuilder-Generator-JS-to-MyGet-daily https://fuselabs.visualstudio.com/SDK_v4/_release?definitionId=63&view=mine&_a=releases
The released package on MyGet: aitemplates - generator-botbuilder 4.6.0-preview.72039 https://botbuilder.myget.org/feed/aitemplates/package/npm/generator-botbuilder
The prod release definition: [[PROD]]Push-BotBuilder-Generator-JS-to-npmjs https://fuselabs.visualstudio.com/SDK_v4/_releaseDefinition?definitionId=64&_a=environments-editor-preview
Thanks.
-Bruce
From: Bruce Haley (Harvey Nash)
Sent: Monday, July 22, 2019 11:36 AM
To: microsoft/BotBuilder-Samples <reply<EMAIL_ADDRESS>microsoft/BotBuilder-Samples <BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed <subscribed<EMAIL_ADDRESS>Subject: RE: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
I will get started on a build for https://github.com/microsoft/BotBuilder-Samples/tree/master/generators/generator-botbuilder
--BH
From: Scott Gellock <notifications<EMAIL_ADDRESS>Sent: Monday, July 22, 2019 10:29 AM
To: microsoft/BotBuilder-Samples <BotBuilder-Samples<EMAIL_ADDRESS>Cc: Subscribed <subscribed<EMAIL_ADDRESS>Subject: [microsoft/BotBuilder-Samples] [yeoman] generator- botbuilder needs to have a supporting build and release pipeline (#1667)
We need a build pipeline and a release pipeline for generator-botbuilder. this should be modeled in the way we do our JS modules (version information in package.json, is updated by the build process. builds are published to botbuilder.myget until we move off of that service. and lastly we need a release pipeline that will publish to npmjs.org
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FBotBuilder-Samples%2Fissues%2F1667%3Femail_source%3Dnotifications%26email_token%3DABAY6UT6PNT7VHU46KGWOWTQAXU4HA5CNFSM4IF2YJH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAWEC2A&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=vHpj7K7ifGjqQS3VEDEzrNJEt9CHNiFmyvjQYbBql5E%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABAY6UUP2A5KDG2LYJKNOZ3QAXU4HANCNFSM4IF2YJHQ&data=02|01|v-bruhal%40microsoft.com|82956229140a4234c8cd08d70eca06c3|72f988bf86f141af91ab2d7cd011db47|1|0|636994133168459131&sdata=QeA3qsfuh7K3EcmElHhkx6nPJuZXmdV9JbxkCWKZppI%3D&reserved=0.

Thanks @BruceHaley you're the build and release pipeline ninja


Doing setup and then manually creating the application doesn't push any image to ECR

Following the steps in the README to set up and then deploy the application (without setting up the pipeline), our Fargate service ends up in a thrashing state. The README indicates that we might be getting 503s for a little bit while the application starts up, but it appears as though we'll get 503s indefinitely.
In ECS, we observe that Fargate tries to provision a new task, which then fails due to
STOPPED (CannotPullContainerError: Error response from daem)

So, it gets in a state where 0 tasks are running, but 1 is desired.
It looks like manually deploying the application doesn't push any image to ECR. We should note this.

Closed by #29.


AddednumberCheckingfiles

Have added the folder with python files containing Python scripts that implement functions to check various properties of numbers, including Armstrong, Automorphic, Neon, Palindrome, Perfect, and Strong numbers. Each script provides a simple interface for users to determine the mathematical characteristics of a given number.

@Richajaishwal0 have you raised the issue first?

Yes it's about adding palindrome, Armstrong checker...
On Thu, Oct 10, 2024, 3:14 PM UTSAV SINGHAL @.***>
wrote:

@Richajaishwal0 https://github.com/Richajaishwal0 have you raised the
issue first?
—
Reply to this email directly, view it on GitHub
https://github.com/UTSAVS26/PySnippets/pull/93#issuecomment-2404608862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFKXBHJP7R5XCRJFIJ6JXPLZ2ZD7PAVCNFSM6AAAAABPTUE5ESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBUGYYDQOBWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Richajaishwal0 add the test cases too in test folder.

I have added the test cases too, please check


Build optimized FFT functions for Xtensa hifimini  architecture.

BUG=http://b/264262664

@Mergifyio refresh


defaultRules

Would it be possible to use defaultRules instead of rules?

There's still no update from webpack regarding this issue/doc request so I guess we'll need to wait a little bit longer for this one.

closing, as there are no changes in sight.


parser: Return statements must always have a value

Early returns in functions which do not return a value are current impossible, as the parser will always attempt to parse an expression after a return keyword.

I think EOL (end of line) is supposed to terminate a statement. It would be intersting to discuss wether the void return should be handled as return with an expression of implicit type void or as a special case.
As a side note,  this example would also be ambiguous without that rule of EOL  termination ( return (1 + 3 - 1) or return 1 ?).
function f() -> i32 {
    return 1
    +3 - 1
    let x = 10
}

I just checked, EOL is indeed terminating a statement. I got confused because of the example I gave. It was parsing the following line as an expression for return, which made me believe Eol didn't matter. But seeing the code for parsing expression it makes sense why that happened. Thanks. I'll try to make a pr for this issue

Seems to be fixed by https://github.com/SerenityOS/jakt/commit/c213b581ce544d2882d7b13a0e48ae40ad00aeb8


Added MVC viewengine and layout code

As mentioned in #71.

I've updated the README to explain how to use the ViewEngine, and also install from Nuget (despite the package not existing yet :)

The Travis build isn't keen on the MVC project, I think...

I'm doing a little rework. I'll submit again in a couple of days when I've got the code stable. Sorry for any confusion.


fixed: deadlock in flush (#82)

Fixes the vortex lock.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "manipulate",
    "pr-id": "83",
    "commit-sha": "c5de49b1bf1e420142d75cc4e9320799ded11148"
  },
  {
    "project": "",
    "component": "backend",
    "pr-id": "404",
    "commit-sha": "8d72dcda18efbbed7b2dda24f9e5ac5ad52ffbb3"
  }
]


Default to focus on monaco loading

fixes https://github.com/microsoft/pxt-minecraft/issues/1761
Not sure if this will affect accessibility or anything?

I think it's fine accessibility wise when you navigate to the page from within the app, but if you were given a direct link (e.g. if someone shared a link to https://minecraft.makecode.com/beta?ipc=1&inGame=1#tutorial:ts:/tutorials/chicken-rain) it might not be ideal. When we load in for the first time we need the accessibility toolbar to be focusable easily / as the first few stops when outside modals, but if we focus to the monaco editor to start it's hard to get there - the menu isn't available when you tab, and you also need to know to use shift+m or ctrl+shift+m to switch tab from inserting a tab to navigating the page.

ooo good point. I'll make the changes

I just tested it for accessibility, and there's currently no way to get to the tutorial steps anyways, even with tab navigation. I'll file a bug in arcade, since Minecraft isn't accessible


Add digital climate strike script

This PR adds a script that

before September 20 will add a banner to our site
on September 20 will add an overlay

as described at https://digital.globalclimatestrike.net/
See also https://github.com/giantswarm/giantswarmio-webapp/pull/593

💚


WIP: Adding support for path parameters for api gateway

@mdeuser @dubeejw Can one of you review this as part of the changes for path parameter support? Need these changes before i can commit the changes in the incubator-openwhisk-cli repo

@mdeuser Any other comments?

@dubeejw Can you look at this so it can get merged? Its blocking my changes in the incubator-openwhisk-cli repo...

or maybe @csantanapr


added windows 11

after reading this article, i confirmed windows 11 is supported

Learn Build status updates of commit a8d9846:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows/security/threat-protection/security-policy-settings/user-account-control-only-elevate-uiaccess-applications-that-are-installed-in-secure-locations.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Fix engine path resolution

Closes issue #56
BREAKING CHANGE: The returned path for mounted engines now includes the path prefix.
An error was raised when testing a route that is mounted after an engine that shares the same route prefix.
The path returned for a mounted engine didn't include the route prefix (only the engine sub path was returned).
On the engine test case, the expected path was changed from /eng_route to /my_engine/eng_route, to match the mounted path.

@pcoliveira is there anything I can do to help this PR along? This has been nagging me for a long while and finally has become more of an issue

My organization recently ran into the issue fixed by this PR, so I would love to see this released ❤

@pcoliveira Hi. Sorry for late response.
Could you check your PR?
I left a question.

@pcoliveira Hi. Sorry for late response. Could you check your PR? I left a question.

sorry, I missed your question.

@exoego How may I help? is there anything I need to do to move the PR forward?


[SOLR-10059] Handle appends params in distributed requests

Description
In distributed requests, the appends section of a RequestHandler is evaluated not only on the non-distributed coordinator requests but also during the distributed requests. This might lead to parameters (especially harmful for fq params) being attached multiple times.
<requestHandler name="/myHandler" class="solr.SearchHandler">
    <lst name="appends">
        <str name="fq">{!collapse field=routingKey hint=top_fc}</str>
    </lst>
</requestHandler>

The issue exists for some time in the bug tracker: SOLR-10059
Solution
The fix ignores the appends section of the handler definition during distributed requests.
Tests
I added a simple unit test, no cloud setup test though
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

Did you see the precommit failure on missing license header @tboeghk ?

Whoopsie, my bad, ran the gradlew check with -Pvalidation.git.failOnModified=false. Added license header in the last commit

I came across an edge case that I want to consider: In the distributed environment we should only ignore the appends section if we do not change the handler during the distributed requests. I'll add that as well and write some tests.

Hey @epugh any chance to merge this? Or has anybody else directions for improvement?

Yes (I'm 95% sure): The macros don't get expanded on the shard. With the fix above the configuration is not picked up on the shard and the macros should stay expanded. Do we need another test for that?

Does anyone ever say "no, we don't need another test" ;-)   I'd love one, partly because this whole space is kind of new for me, so it might help me feel confident of your fix!!!!  unit tests are super helpful in understanding the logic of the fix!

@cpoerschke I wanted to get this sorted, any chance you could be another set of eyes????

@cpoerschke I wanted to get this sorted, any chance you could be another set of eyes????

From looking at the changes I'm wondering if RequestUtil might be an alternative place for the logic since the "don't expand macros on the shard" logic already happens there.
From reading about the context of the changes on https://issues.apache.org/jira/browse/SOLR-10059 on 2017-03-01 there's a mention of multi-collection requests, I don't yet fully understand how that fits into the bigger picture.
From running the new tests locally, hmm, they don't pass here at the moment.

... no cloud setup test though ...

#426 is about adding a SearchHandlerAppendsCloudTest test.

@cpoerschke @tboeghk want to pick this one up again....?   (Though I know @tboeghk is out on vacation till September!)....

I reviewed the fix recently and came up with a smaller fix that isolates the issue better: https://github.com/otto-de/solr/commit/a82b500d3c633621b0062698f540c2974a920fbb
The fix contains a test that breaks without the fix applied (macros in fq parameter)

I reviewed the fix recently and came up with a smaller fix that isolates the issue better: otto-de@a82b500
The fix contains a test that breaks without the fix applied (macros in fq parameter)

Ok, translate for me what that means. Can this PR or the JIRA be closed? Will there be a new PR?

I'll file a new PR 👍

I created a new PR https://github.com/apache/solr/pull/2303 that superseeds this PR.


!!! cert-manager cannot access GoogleCloud API

Hello,
I am installing cert-manager using this script:
#!/bin/sh

CURRENT_PROJECT_ID=$(gcloud config get-value core/project)
read -p "Provide GCP Project ID for cluster [$CURRENT_PROJECT_ID]: "  PROJECT_ID
if [ -z "$PROJECT_ID" ]; then
    PROJECT_ID="$CURRENT_PROJECT_ID"
fi

kubectl create namespace cert-manager
kubectl label namespace cert-manager certmanager.k8s.io/disable-validation=true --overwrite=true
kubectl label namespace cert-manager istio-injection=disabled --overwrite=true

kubectl create clusterrolebinding cluster-admin-binding \
  --clusterrole=cluster-admin \
  --user=$(gcloud config get-value core/account)

CLOUD_DNS_SA=cert-manager-cloud-dns-admin
gcloud --project $PROJECT_ID iam service-accounts delete<EMAIL_ADDRESS>gcloud --project $PROJECT_ID iam service-accounts create $CLOUD_DNS_SA \
    --display-name "Service Account to support ACME DNS-01 challenge."

<EMAIL_ADDRESS>gcloud projects add-iam-policy-binding $PROJECT_ID \
     --member serviceAccount:$CLOUD_DNS_SA \
     --role roles/dns.admin

KEY_DIRECTORY=`mktemp -d`
gcloud iam service-accounts keys create $KEY_DIRECTORY/cloud-dns-key.json \
     --iam-account=$CLOUD_DNS_SA
kubectl delete secret --namespace cert-manager cloud-dns-key
kubectl create secret --namespace cert-manager generic cloud-dns-key \
     --from-file=key.json=$KEY_DIRECTORY/cloud-dns-key.json
rm -rf $KEY_DIRECTORY

CERT_MANAGER_VERSION=0.6.1
DOWNLOAD_URL=https://github.com/jetstack/cert-manager/archive/v${CERT_MANAGER_VERSION}.tar.gz

wget $DOWNLOAD_URL
tar xzf v${CERT_MANAGER_VERSION}.tar.gz

cd cert-manager-${CERT_MANAGER_VERSION}

kubectl apply -f deploy/manifests/00-crds.yaml
kubectl apply -f deploy/manifests/cert-manager.yaml

cd ../
rm -rf cert-manager-${CERT_MANAGER_VERSION}
rm v${CERT_MANAGER_VERSION}.tar.gz

However, when it tries to validate DNS-01 challenge, it keeps failing with:
challenges controller: Re-queuing item "istio-system/nebula-certificate-1671088635-1"
due to error processing: GoogleCloud API call failed: googleapi:
Error 403: Forbidden, forbidden

I have spent almost 2 days trying various different settings / combinations / versions / re-installs .. but have not been able to resolve this issue.
Would really appreciate any help I can get here.

After cloning https://github.com/jetstack/cert-manager and tracing through the problem manually, I have figured it out.
Turns out Google Cloud does not like re-use of the sa email.
In my script above, I am re-using the same sa email every time I setup the cluster by deleting the old one and creating the new one.
CLOUD_DNS_SA=cert-manager-cloud-dns-admin
gcloud --project $PROJECT_ID iam service-accounts delete<EMAIL_ADDRESS>gcloud --project $PROJECT_ID iam service-accounts create $CLOUD_DNS_SA \
    --display-name "Service Account to support ACME DNS-01 challenge."

<EMAIL_ADDRESS>gcloud projects add-iam-policy-binding $PROJECT_ID \
     --member serviceAccount:$CLOUD_DNS_SA \
     --role roles/dns.admin

Instead of that, I am doing this now to ensure a new unique sa email is used everytime. Timestamp helps me figure out the which one is the older one that should be removed manually.
CLOUD_DNS_SA=certmgr-cdns-admin-$(date +%s)
gcloud --project $PROJECT_ID iam service-accounts create $CLOUD_DNS_SA \
    --display-name "Service Account to support ACME DNS-01 challenge."

<EMAIL_ADDRESS>gcloud projects add-iam-policy-binding $PROJECT_ID \
     --member serviceAccount:$CLOUD_DNS_SA \
     --role roles/dns.admin


do not check for homebrew is packages present

if cocoapods, ideviceinstaller and ios-deploy are installed and working fine, we do not need homebrew at all
resolves #16458

Looks like CI is failing because you're missing a semicolon on line 128.

I'll try to write the tests, but I have no idea to be honest. I just wrote my first line of Dart yesterday :)

Sure, check out ios_workflow_test.dart. You can copy paste one of the existing test. The mocking should be somewhat self explanatory. You can mock out the new conditions that matches your ifs but have hasHomebrew false and see that there are no error messages.

@championswimmer  I'm going to close this PR since you're not working on it right now, but if you have time to come back to it please don't hesitate to reopen or open a new one or whatever, and if you need anything please don't hesitate to cc me. Thanks!

@jslavitz Can we close this?


Create LRO partial class for gRPC automatically

I suspect files like this could now be autogenerated by toolkit:
https://github.com/GoogleCloudPlatform/google-cloud-dotnet/blob/master/apis/Google.Cloud.Speech.V1/Google.Cloud.Speech.V1/CloudSpeechGrpcPartial.cs

@jskeet Can you add a priority to this?

This is now complete (very recently).


integrate my own fine-tuned model

Use Case
I am currently developing a sports app and need to integrate a large model specific to a vertical domain.
Problem
I found that there is no code supporting the integration of our own LLM.
Solution
Develop a module similar to integrating OpenAI.
Alternatives considered
There is no solution available at the moment.
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct and Contributing Guidelines

@Bourne-alt Thank you for looking into Spezi LLM!
If you want to execute a model locally, I would advise you to take a look at the SpeziLLMLocal. It allows you to execute models locally on your device and we demonstrate it as part of the UI Testing application in this repo using some open-weight models.
If the model size is quite large (which it is for all the models we are using), I would suggest looking into SpeziLLMLocalDownload as an additional context and potential to download it at start time to your app.
I can encourage you to look through the README and documentation to explore these two targets in SpeziLLM, they might be helpful. I addition, we more than welcome PRs to this package and other work we are doing as part of our open-source work to extend the functionality of the frameworks.
In addition to that CoreML and other Apple-provided tools might be helpful.


Cropped curved text ( padding included )

Hello,
I was trying to create curved/bold/italic texts but some of them are cropped ( padding is already included ). Please any idea how to fix it?
Expected Behavior
The whole text visible
Current Behavior

Steps to Reproduce
https://www.pixiplayground.com/#/edit/AkpqkMJtxVGFmkPw5hL9Z
Environment

pixi.js version: 4.8.8
Browser & Version: Chrome 76.0.3809
OS & Version: Mac OS
Running Example: https://www.pixiplayground.com/#/edit/AkpqkMJtxVGFmkPw5hL9Z

It's works propertely.
You try use texture from Text on rope, but there is difference between sprite ( Text is Sprite-like) and Mesh.
Paddings generate  trimed texture, but trim works only as vertex shift on Sprites.
Instead of text use canvas and draw text directly, or extend PIXI.Rope and override _refresh method and fix uvs generation.
http://pixijs.download/v4.8.8/docs/mesh_Rope.js.html#line120


Grid element concat fails if elements contain uneven len of views

Raise an error not all grid element's num of rows are the same

Resolved by #240.


Get-CSStartMenuEntry: don't exclude desktop.lnk or desktop.exe

Makes sense. Thanks for the catch! Do you mind doing a PR to the dev branch and I'll merge.


[FEATURE] A better way to access string value of ClassLabels

🚨🚨 Feature Request

[x] A new implementation (Improvement, Extension)

Is your feature request related to a problem?
Accessing the string value of ClassLabels is inconvenient and complicated as illustrated below:-
import numpy as np
from hub import schema, Dataset
my_schema = {"label": schema.ClassLabel(names=["red", "green", "blue"])}

ds = Dataset("./data/classlabel_ds", shape=(5,), schema=my_schema)
ds["label", 0] = 1
ds["label", 1] = 2
ds["label", 2] = 0

# this works well
assert ds["label", 0].compute() == 1
assert (ds["label", 0:3].compute() == np.array([1, 2, 0])).all()

# getting the actual string label is a little more complicated
cl_label = my_schema["label"]
assert cl_label.int2str(ds["label", 0].compute()) == "green"
assert [cl_label.int2str(ds["label", i].compute()) for i in range(3)] == ["green", "blue", "red"]

If your feature will improve HUB
Yes, it will make accessing the string value of ClassLabels much easier
Description of the possible solution
Add a parameter called get_label to .compute() of TensorView that is False by default. If it is set to True:-

if the dtype of TensorView is not ClassLabel, do nothing
if the dtype of TensorView is ClassLabel, use the dtype to convert the result into string(s) using int2str

Teachability, Documentation
# ideal implementation, continued from the code above
assert ds["label", 0].compute(get_label=True) == "green"
assert ds["label", 0:3].compute(get_label=True) == ["green", "blue", "red"]

love this feature request!

I would like to work on this if no one else currently is.

I would like to work on this if no one else currently is.