Copy-paste bug found by static code analysis tool

A static code analysis tool found a copy-paste bug that I want to share with you.
I don't really want to explain it, I made a quick fix instead, feel free to cherry-pick:
https://github.com/npapp/adapter/commit/dc9420708405723ba11712a5b3777f3d0439d345

Sorry, I was on holiday.
I was too lazy to read the CLA for this small change, that's why I wanted you to cherry-pick. :)
Oh the release folder is generated, should have noticed, my bad.
I ran a static code analyzer tool on a library which is dependant on webrtc-adapter npm package and browserified. It found this problem and I saw that it is not in the library's code but in the webrtc-adapter's so I decided to share it with you, that's how. :)


Can you do something similar to Calamity mod?

I would like to know if it is possible to create a mod based on this code, a mod similar to the calamity, but I do not have enough knowledge to know if it is possible or not.

Someone probably doing it I know.

It seems very difficult because coding mods in TL Pro only uses JavaScript 💀


ref(Jenkinsfile): disable win agent

Codecov Report

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

@@           Coverage Diff           @@
##           master     #300   +/-   ##
=======================================
  Coverage   72.71%   72.71%
=======================================
  Files          59       59
  Lines        4109     4109
=======================================
  Hits         2988     2988
  Misses        793      793
  Partials      328      328

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 90df5f4...008615a. Read the comment docs.


Add analysis provider for Datadog

What this PR does / why we need it:
This PR adds basic implementation for the Datadog provider using Datadog's official client package.
While I've tested the behavior is what we expected using my Datadog account, I'm in the middle of learning the query on Datadog. So I settled on putting the evaluate stuff off.
Which issue(s) this PR fixes:
Ref https://github.com/pipe-cd/pipe/issues/1143
Does this PR introduce a user-facing change?:

NONE

@nghialv Fix to check all data points as you suggested. Let me address the remaining tasks as other PRs:

Ensure to configure the queriedTimePeriod from RunQuery function via a QueryParams or QueryRange
Handle multiple data points from Prometheus
Think about the case of no values

Fixed done. PTAL 🙏

Nice. 👍
/lgtm

🏃
/approve


coc-kotlin fails to start & hangs up at creating client

Result from CocInfo
## versions

vim version: NVIM v0.9.1
node version: v16.15.0
coc.nvim version: 0.0.82-ae4ecfa5 2023-09-03 14:14:50 +0800
coc.nvim directory: /Users/rgupta2/.vim/plugged/coc.nvim
term: tmux
platform: darwin

## Log of coc.nvim

2023-10-29T10:36:48.541 INFO (pid:4311) [plugin] - coc.nvim initialized with node: v16.15.0 after 89
2023-10-29T10:36:52.834 INFO (pid:4311) [core-watchman] - watchman watching project: /Users/rgupta2/Downloads/workspace/marketplace-demand-service
2023-10-29T10:36:53.138 INFO (pid:4311) [language-client-index] - Language server "kotlin" started with 4405
2023-10-29T10:36:57.718 ERROR (pid:4311) [timing] - activate coc-kotlin timeout after 5000ms
2023-10-29T10:37:06.221 INFO (pid:4311) [attach] - Request action: fillDiagnostics [ 3 ]
2023-10-29T10:37:53.609 INFO (pid:4311) [attach] - receive notification: runCommand [ 'workspace.showOutput' ]
2023-10-29T10:38:34.315 INFO (pid:4311) [attach] - receive notification: showInfo []

Description
coc-kotlin fails to start & I don't see any auto completions even after waiting for 10-15 mins.
Extension version

Latest version of coc-kotlin as I just reinstalled it

Additional context
Here is what :CocCommand workspace.showOutput says
[10/29/2023, 10:36:52 AM 730] [debug]: Looking for Java in JAVA_HOME (environment variable): /Users/rgupta2/.jenv/versions/11.0
[10/29/2023, 10:36:52 AM 739] [info]: Activating Kotlin Language Server...
[10/29/2023, 10:36:52 AM 743] [info]: Querying GitHub API for new Kotlin Language Server version...
[10/29/2023, 10:36:53 AM 120] [info]: Connecting via Stdio.
[10/29/2023, 10:36:53 AM 129] [info]: Creating client at /Users/rgupta2/.config/coc/extensions/coc-kotlin-data/langServerInstall/server/bin/kotlin-language-server

Try using kotlin-language-server directly works
I tried removing coc-kotlin & since it uses kotlin-language-server it works for me!
I cloned & built kotlin-language-server from it repository & added the following in my :CocConfig
    "languageserver": {
        "kotlin": {
            "command": "~/Downloads/workspace/dotfiles/kotlin-language-server/server/build/install/server/bin/kotlin-language-server",
            "filetypes": ["kotlin"],
            "trace.server": "verbose"
        }
    }

The same worked, but I had to wait for 3-5 mins before it starts & have to keep an eye on my workspace.showOutput to know when it's ready to use, so not very convenient. I think coc-kotlin uses this language server under the hood & provide helpful info like the state of plugin & building it etc.
Would you be able to help me debug what the issue might be?

I tried removing the ~/.config/coc directory & installed coc-kotlin again & got the following error with :CocInstall coc-kotlin
[coc.nvim] Error: Launching server "kotlin" using command /Users/rgupta2/.config/coc/extensions/coc-kotlin-data/langServerInstall/server/bin/kotlin-la
[coc.nvim]: UnhandledRejection: Launching server "kotlin" using command /Users/rgupta2/.config/coc/extensions/coc-kotlin-data/langServerInstall/server
/bin/kotlin-language-server failed.
Error: Launching server "kotlin" using command /Users/rgupta2/.config/coc/extensions/coc-kotlin-data/langServerInstall/server/bin/kotlin-language-serv
er failed.
    at /Users/rgupta2/.vim/plugged/coc.nvim/build/index.js:244:6399
    at async Md.createConnection (/Users/rgupta2/.vim/plugged/coc.nvim/build/index.js:240:15149)
    at async Md._start (/Users/rgupta2/.vim/plugged/coc.nvim/build/index.js:240:7683)

I tried running this command directly:
/Users/rgupta2/.config/coc/extensions/coc-kotlin-data/langServerInstall/server/bin/kotlin-language-server

& this seem to be working without errors..

On opening a kotlin file now, it worked 🎉
For future folks who might be facing this:

Remove coc config data:

rm -rf ~/.config/coc

Reinstall coc & coc-kotlin

@PezCoder Oh my god, bless you so much. For future readers, during the reinstallation, when downloading the kotlin language server again, it might fail again, just ignore, let it finish its thing, restart gracefully and give it some time.
At the restart I didn't get the error anymore and after several seconds I finally have Kotlin support. Awesome.


Assertion Error in vgg16.ipynb

Hi again,
The 4th cell in vgg16.ipynb throws an empty assertion error, apparently due to a shape mismatch"
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-13-38a95cb2fa54> in <module>
      1 # Show the keyed image
----> 2 keyed = sensor.fromimage(im_source).encrypt()
      3 keyed.toimage().mindim(512, interp='nearest').show()

~/projects/privacy/keynet/keynet/system.py in fromimage(self, im)
    202
    203     def fromimage(self, im):
--> 204         assert (1, im.channels(), im.height(), im.width()) == self._inshape
    205         self._im = im
    206         self._tensor = im.float().torch().contiguous()

AssertionError:

Thanks,
Niko

It seems changing the shape when calling (sensor, model) = Keynet(inshape=(3,224,224) to (3,512,512) fixes the issue. Probably resizing the image would also fix it.

Thanks for pointing this out.  This has been resolved.


"vintage" cursor shape + blinking with an escape sequence?

In Windows Terminal, can I get the "vintage" cursor (~30% blinking horizontal bar) with an escape sequence?
Both of these (below) give a non-blinking cursor of the desired size. Other similar pairs suggest that 7 should be blinking?
ESC[7 q
ESC[8 q

Vince

@DHowett had a horrible idea in chat

if we did that, we could also add a subparameter to it
\e[7:25 q sets vintage at 25%
and therefore communicate the cursor state from conpty to terminal . . .

which is actually an amazing idea IMO. We don't currently support subparams in sequences, but we're not too far away from that.
I'm gonna summon @j4james to sanity check that using these sequences like this don't have an existing conflict. We might need to pick something silly like \e[9001:25 q to avoid any existing conflicts, but I really like this idea.

I would generally not encourage the use of sub parameters, because they're non-standard, and tend to break parsers in unpredictable ways. If you were an app developer that wanted to use this option, you'd then have to detect the terminal type to determine whether it was safe to send the sequence or not.
So my preference would probably be to invent a completely new sequence for the size, if you really think you need it - something that would generally be treated as a no-op by terminals that don't support it (you can't always guarantee that, but the idea is to minimize the damage you're doing).
I should also add that I thought someone already had a sequence for this. I had a brief look yesterday when the issue was first raised and didn't find anything, but it's probably worth doing a bit more research before reinventing something that may already exist.

I just wanted to get back to my default (blinking "vintage").  In '\e[N q', N = 1/2 gives blink/non-blink block cursor, N = 3/4 the same for underscore, N = 5/6 the same for vertical bar.  Since n = 8 gives the "vintage" size and shape, non-blinking, it seems reasonable that 7 give blinking "vintage" (it doesn't).
I don't know what your restrictions are.  Returning to the default should be straightforward (maybe even '\e[0 q' which now duplicates '\e[1 q').
Where can I find the handling of control sequences in the code?

@vefatica you may be interested in #7379

Since n = 8 gives the "vintage" size and shape, non-blinking, it seems reasonable that 7 give blinking "vintage" (it doesn't).

Just to be clear, we don't technically support parameter 8 as "vintage". Every number that isn't a recognised parameter just ends up as that by default. Personally I think that's a bug - if it's a not a supported parameter we should be ignoring it. If we want to use 7/8 as blinking and non-blinking vintage then those should be explicitly supported parameters.

Yes, I figured parameters greater than 6 weren't supported.  If unknown parameters are not ignored, and are going to lead to the default, it should respect whether the default is blinking or not (or at least preserve the current blink state).
As for supported parameters, "vintage" (blinking/non-blinking) really should be among them.

We didn't invent this sequence - it's just a standard that we are a following. Parameters 0 to 4 were originally defined by DEC, and 5 and 6 were later added by XTerm, and have since become a de facto standard. "Vintage" isn't a style that anyone else supports, so that would require a proprietary extension, which is not something that we want to rush into.

You have (I guess) respected the "de facto standard".  What's the problem with augmenting it?  [I'm serious.]  I wonder if XTerm agonized over adding 5 and 6?

We have a specific resistance to extending any existing protocols like this. We'd much rather entirely invent our own sequence to use, that isn't even sorta used by anyone else, rather than extend an existing one.
I agree with @j4james's assessment - using \e[8 q to restore to vintage is a bug, that we should probably fix, as well as introduce a vintage cursor shape sequence.

FWIW, I found the escape sequence I was thinking of. It's a private CSI that was used by SCO Unixware:
ESC[= s ; e C
    Sets cursor parameters (where s is the starting and e is the ending scanlines of the cursor).

Not exactly what we want, because I think we can only set the height rather than the scanline range (at least in conhost). Also there's the question of how many scanlines there are in a character (you assumedly wouldn't want it to depend on font size).
That said, it's worth noting that Kermit implemented this (when in "ANSI" mode) by calculating the requested height (i.e. e - s), and then mapping certain height ranges to cursor sizes.

"height" <= 1 is an underline cursor
"height" < 10 is a half block cursor
"height" >= 10 is a full block cursor

Assumedly this sequence would also function as a proxy for selecting the "vintage" cursor. And choosing whether the cursor should be blinking or not is already covered by the existing cursor blink mode (private mode 12).
I'm not sure about this, but it's at least something to consider.

As I mentioned at https://unix.stackexchange.com/a/597558/5132 , I've extended this to support 4 more shapes, myself.  They are an outline box, a star, underline+overline, and a reversed "L" shape, inspired by some literature on real terminals.  So if you do ever extend DECSCUSR, I'd appreciate it if we were compatible.

Thanks for the info - that's useful to know. We do have an "Empty Box" cursor style, which sounds the same as your "outline box", so that's something I think we could easily support if we were going to extend DECSCUSR. Am I right in assuming that 7 and 8 are the blinking and non-blinking versions of the outline box in your implementation?

Yes.  https://github.com/jdebp/nosh/blob/79b1c0aab9834a09a59e15d47710f355c5c0417a/source/SoftTerm.cpp#L1201

What about these?  (from https://docs.microsoft.com/en-us/windows/console/console-virtual-terminal-sequences)  They work in the Windows console.

And, after settings are reloaded, the cursor shape is restored but the blink state is not restored.  This might be related to #7571

Maybe there should be a "cursorBlink" (true/false) profile setting?

From #16785:

The blocking issue for this is https://github.com/microsoft/terminal/issues/7382. We need to pass on the request for the overwrite-mode cursor via conpty, which requires an equivalent escape sequence for the half-height block cursor, but there isn't a standard way to achieve that.
It's also worth noting that this particular cursor change is achieved with an internal SetCursorDBMode call, which toggles the "double cursor" mode, independent of the actual cursor size - it's not using the public SetConsoleCursorInfo API. So that's something we need to bear in mind if we want to get this working in conpty.

I've been meaning to comment here that there's another escape sequence we could potentially use for this: CSI ? <size> c, which is used by the Linux kernel console to set the height of the cursor. It's very similar to the way the Windows vintage cursor works, and it's also supported by Mintty, which is another advantage.
The only negative is that it's not really an appropriate escape sequence, because CSI c is technically reserved by ANSI for device attribute queries, and on some terminals a CSI ? c sequence will trigger a DA report.
But it's at least something to consider if nobody has any better ideas.

Hello!
There are a lot of different issues I found online about vim in PowerShell 7 used in the Windows Terminal not displaying the correct cursor shape for insert mode, command mode, visual mode, replace mode et cetera. From what I could find, this thread is supposed to be the original one, I hope I am posting in the right place.
It's working by adding set termguicolors before the escape sequences in the C:\Program Files (x86)\Vim _vimrc file.
set termguicolors
let &t_SI = "\e[4 q"
let &t_SR = "\e[6 q"
let &t_EI = "\e[2 q"
let &t_ti .= "\e[1 q"
let &t_te .= "\e[0 q"
Without set termguicolors or set t_Co=256 it doesn't work, but with one of them it does. Hopefully the people in this thread that still has problems can solve them with this.
found it here: https://github.com/vim/vim/pull/11028

I am also using the following:
&t_SI = "\e[6 q"
&t_EI = "\e[2 q"

and it works reasonably well, but:

The cursor won't pick the color of the colorscheme.  That is, if in WT you have a bar and in vim you have a block, when exiting Vim the cursor stays in block shape,
When entering Vim the cursor shape change don't happen immediately, but you have to go to insert mode and then back to normal mode to see the change.

The issues are present only in WT.  In Windows bundled powershell/wsl terminals the issue is not present. I have the same setting for iTerm2 on MacOs and the described issues are not present.


[Power Rename] use check boxes

Description of the new feature / enhancement
Apply to

[x] folders (first level)
[x] sub(folder) items
[x] file name
[ ] file extension

Scenario when this would be used?
Makes sense to me. Don't need ambiguous icons.

@niels9001 PowerRename is not listed in the Projects in Visual Studio, so can't debug it. When I build a local version, PowerRename falls back to the GA version. When I uninstall the GA version and run the local build, Power Rename does now show up in the Explorer menu. How can we debug the UI?


prompt is too long: 201335 tokens > 200000 maximum

Describe the bug
prompt is too long: 201335 tokens > 200000 maximum
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-2k4hfj
Steps to reproduce

Start
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum

Expected behavior
To work
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 95852930,
  "usedJSHeapSize": 90601890,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = smartguyio
Chat ID = 05529ff094a5

Additional context
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum
prompt is too long: 201335 tokens > 200000 maximum

Appreciate the feedback! This is a known issue with the AI model’s context window size and is being tracked here: #1322.


error when installing

running php artisan la:install results in the first question and endless [error] aborted in command prompt

Hi, this is a nice package I think that the developers have stopped working on it. Maybe you can try this package: https://github.com/Muhammadinaam/speed-admin. It supports Laravel 8+.


Customizable Embedder and Logit Mapper

Hi @lucidrains, I use x-transformers extensively and often run into the need to use custom embedder and logit mapper classes together with the AutoregressiveWrapper class, for e.g., to implement pointer networks (https://arxiv.org/abs/1506.03134). Currently this is not possible, so I work with the lower level Decoder class and implement the wrapper myself.
In this PR, I tried to enable support for this by slightly reorganizing the TokenEmbedding class, and adding a class called LogitMapper to be used as the TransformerWrapper.to_logits layer. The idea is that we can implement custom classes conforming to these interfaces and everything should work seamlessly. These classes accept kwargs in their forward method that allows us to pass additional tensors to them.
The unit tests all pass and I added some additional ones. I hope I didn't break anything! Let me know if you're ok to merge this change, or if you have better ways of achieving this behavior. Thank you.

@pradeep-pyro this is cool, thank you!
do you think it is possible to just keep the default as LinearNoBias, without the LogitMapper indirection?
also, perhaps we can just have token_emb_kwargs: dict and to_logits_kwargs: dict in the forward method instead

@lucidrains I modified the code like you suggested. It is certainly more cleaner this way. Please take another look when you get the chance. Thanks!

@pradeep-pyro looks great Pradeep! graciously accept! 🙏


[Bug] React 17 peer dependency

Describe the bug
Cannot install storybook/addon-jsx in project using React 17.
Steps to reproduce the behavior

Create a project with "react": "^17.0.1" and the newest Storybook
Issue command:

npm i --save-dev storybook-addon-jsx

Expected behavior
JSX addon is installed correctly.
Screenshots

Environment

OS: Ubuntu 20.10
Node.js version: v15.6.0
NPM version: 7.4.0
Browser (if applicable): N/A
Browser version (if applicable): N/A
Device (if applicable): N/A

Additional context
N/A

Are there any plans to fix this? This repo seems fairly inactive. Happy to create a PR if that will get this through sooner as it's a blocker for us right now.

Make a PR and I'll merge it :D

https://github.com/storybookjs/addon-jsx/pull/142

Can't tell what the PR is failing on as it's running in Circle. Seems like all PR's are having the issue though.


feat: temporary tablespaces

This patch allows the user to set a tablespace as temporary, by changing
the value of the temp_tablespaces PostgreSQL setting.
Closes: #11

E2e: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/7103071346

E2e: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/7113543781


Add a utility script for list and comparing lemma over the RU limit

A script I wrote to look at the lemmas over the resource unit threshold. Might be useful to keep with the repository.

I think you can do this (and other checks) using dafny-reportgenerator. I've been looking into this a little because I'd like to add something like this to the CI. Will keep you posted.

I think you can do this (and other checks) using dafny-reportgenerator. I've been looking into this a little because I'd like to add something like this to the CI. Will keep you posted.

Thank you for sharing it. Should we merge this PR first and you can create another PR for dafny-reportgenerator?

How about, rather than merging this PR, we merge #52. It adds some basic instructions on measuring proof stability using dafny-reportgenerator to the README.

Made an issue to track the CI update (#53)

@john-h-kastner-aws are you OK with closing this PR and merge Kesha's instead?


Update wiki documentation with these FAQ

Hi team - as requested by Adrian and Kiran, myself and our team have been together a comprehensive FAQ that we'd like to please add to the FAQ wiki so others can leverage. All the updated FAQ is in the PDF here: https://aka.ms/TeamsBroadcastHubPublicFAQ. I have the original (editable) file, happy to share if you would like.

hi @jacemoreno  can you send me the editable file , i'll create a pull request for you. and format it in markdown so that it appears on Github.

Awesome thanks! [docx icon]  The Microsoft Teams Broadcast Hub 🔊 FAQs.docxhttps://microsoft.sharepoint.com/:w:/t/TheMicrosoftTeamsBroadcastHub/Ea4CINdNx6dHvEz_nkLFEVkBYgLTk3i-N9l8sQGMYuTm0w?e=pxXdyS
If you would, please save a separate copy and we’re happy to help make this easier on you so please let us know if we can. 🤜🤛
Jace Moreno | Microsoft Teams Platform Lead, Customer Success |
P: +1 -425-722-9592 | E<EMAIL_ADDRESS>@.***D73CD9.D203C070]
Polite disclaimer: if you haven’t heard back from me, it’s probably because I don’t check this email thing very often. Try me on Teams 🙌
From: Jason Baxter @.>
Sent: Thursday, April 29, 2021 12:45 AM
To: OfficeDev/microsoft-teams-apps-company-communicator @.>
Cc: Jace Moreno @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-apps-company-communicator] Update wiki documentation with these FAQ (#374)
hi @jacemorenohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjacemoreno&data=04|01|jace.moreno%40microsoft.com|ca29c384c8b6466fb4f708d90ae2b6cd|72f988bf86f141af91ab2d7cd011db47|1|0|637552791112833623|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=1W3dMYoP7CNHrivbTa%2FItdtoykM7OgwuXH0oRQ32dHk%3D&reserved=0 can you send me the editable file , i'll create a pull request for you. and format it in markdown so that it appears on Github.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2Fmicrosoft-teams-apps-company-communicator%2Fissues%2F374%23issuecomment-829013412&data=04|01|jace.moreno%40microsoft.com|ca29c384c8b6466fb4f708d90ae2b6cd|72f988bf86f141af91ab2d7cd011db47|1|0|637552791112843566|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=aw67PcfehWrq8rX4WQwGXjgVfzI4tfXsAjeXB4AS7WY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATZSJYG7A6XV7ARV4TYU443TLEFALANCNFSM43NQRTDA&data=04|01|jace.moreno%40microsoft.com|ca29c384c8b6466fb4f708d90ae2b6cd|72f988bf86f141af91ab2d7cd011db47|1|0|637552791112843566|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7cDq8XSfEDCt%2BI6XZNAie6tWeJeX6%2B%2F6VnZ4HcOpSoA%3D&reserved=0.

Hi team – I don’t see the updated FAQ in the wiki. Did we move it somewhere else? Thanks!
Jace Moreno | Microsoft Teams Platform Lead, Customer Success |
P: +1 -425-722-9592 | E<EMAIL_ADDRESS>@.***D756BF.9097F8E0]
Polite disclaimer: if you haven’t heard back from me, it’s probably because I don’t check this email thing very often. Try me on Teams 🙌
From: v-smahaj @.>
Sent: Monday, May 31, 2021 2:33 AM
To: OfficeDev/microsoft-teams-apps-company-communicator @.>
Cc: Jace Moreno @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-apps-company-communicator] Update wiki documentation with these FAQ (#374)
Closed #374https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2Fmicrosoft-teams-apps-company-communicator%2Fissues%2F374&data=04|01|jace.moreno%40microsoft.com|7f6206ee59e04fb40d1208d924170807|72f988bf86f141af91ab2d7cd011db47|1|0|637580503611205687|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=VYNWKOoy3lx5DkE1fla%2BNvGuIW1nS9eYrrw4nRv3v8s%3D&reserved=0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2Fmicrosoft-teams-apps-company-communicator%2Fissues%2F374%23event-4819650740&data=04|01|jace.moreno%40microsoft.com|7f6206ee59e04fb40d1208d924170807|72f988bf86f141af91ab2d7cd011db47|1|0|637580503611215644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=VdXMEkq9tKdOyC1K7XTmvTTVoWuh1tAaB8X1Aif7lkI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATZSJYH3PQSKJ5O5RQ6VBDTTQNJTNANCNFSM43NQRTDA&data=04|01|jace.moreno%40microsoft.com|7f6206ee59e04fb40d1208d924170807|72f988bf86f141af91ab2d7cd011db47|1|0|637580503611215644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=nTkjE2veuxowSqhvHp6Qu8A4tsxZMjlELpMx9WYANvI%3D&reserved=0.

Hi Jason – did this get uploaded? I got confirmation that it was closed but don’t see it up on the repo. Thank you.
Jace Moreno | Microsoft Teams Platform Lead, Customer Success |
P: +1 -425-722-9592 | E<EMAIL_ADDRESS>@.***D757A7.CF677DA0]
Polite disclaimer: if you haven’t heard back from me, it’s probably because I don’t check this email thing very often. Try me on Teams 🙌
From: Jason Baxter @.>
Sent: Thursday, April 29, 2021 12:45 AM
To: OfficeDev/microsoft-teams-apps-company-communicator @.>
Cc: Jace Moreno @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-apps-company-communicator] Update wiki documentation with these FAQ (#374)
hi @jacemorenohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjacemoreno&data=04|01|jace.moreno%40microsoft.com|ca29c384c8b6466fb4f708d90ae2b6cd|72f988bf86f141af91ab2d7cd011db47|1|0|637552791112833623|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=1W3dMYoP7CNHrivbTa%2FItdtoykM7OgwuXH0oRQ32dHk%3D&reserved=0 can you send me the editable file , i'll create a pull request for you. and format it in markdown so that it appears on Github.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2Fmicrosoft-teams-apps-company-communicator%2Fissues%2F374%23issuecomment-829013412&data=04|01|jace.moreno%40microsoft.com|ca29c384c8b6466fb4f708d90ae2b6cd|72f988bf86f141af91ab2d7cd011db47|1|0|637552791112843566|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=aw67PcfehWrq8rX4WQwGXjgVfzI4tfXsAjeXB4AS7WY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATZSJYG7A6XV7ARV4TYU443TLEFALANCNFSM43NQRTDA&data=04|01|jace.moreno%40microsoft.com|ca29c384c8b6466fb4f708d90ae2b6cd|72f988bf86f141af91ab2d7cd011db47|1|0|637552791112843566|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7cDq8XSfEDCt%2BI6XZNAie6tWeJeX6%2B%2F6VnZ4HcOpSoA%3D&reserved=0.


List of missing images for dark mode

There are some images which are required for dark mode.

[ ] #5267 Large logo

This is R.image.mascot_happy() in the art assets.

How do I close this issue? @hboon

Since there's nothing else, just close it.

Large logo still required for dark mode.

[ ] Buzz #6468 when done


Update JS dependencies

Update all JS deps
Regenerate SVGs
Add new eslint rules, fix issues
Tested Mermaid, Swagger, Vue, Webpack, Citation

… Your PR on khroma was denied… :smiling_face_with_tear:

Stubbornly, yes.

It's a well motivated reason (https://github.com/fabiospampinato/khroma/pull/33#issuecomment-1321250887), those tools are unreliable almost by definition. But sure, I can understand the frustration.

Of note is that license-checker-webpack-plugin is primarly there to extract the licenses, the checking functionality is more of a side effect. If in the future we find a way to only extract licenses without checking, I'm happy to switch over to that. Might be necessary anyways once we migrate off webpack.

If in the future we find a way to only extract licenses without checking, I'm happy to switch over to that. Might be necessary anyways once we migrate off webpack.

fwiw I'd like to eventually build a standalone little library that given the current package extracts all of its NPM dependencies, detects their repositories, their readmes, their license files, and their license identifier (like just "MIT" rather than the whole string). If you might consider using this and there's some other feature that you'd need just let me know 👍

make L-G-T-M work


Fix example

Description & motivation

To-do before merge

Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[ ] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects

Thanks @runleonarun & @mirnawong1 :pray:
I guess we're waiting on @jtcohen6's :+1:
Is that why you've blocked merging with the "requested changes" review @runleonarun ?

many apologies, @b-luu ! with our recent reorganization of the docs, it caused a lot of PRs to have merge conflicts -- this unfortunately was one of them. we have been working our way through the list to resolve them.
looks like PR #2435 has merged a fix for this same issue. i will go ahead and close this one. please don't hesitate to reopen if there's still something remaining. again, apologies for the delay!


Enhance Coffee Cup Image with Parallax Effect

Fixes #52
It  introduces enhancements to the Landing component by applying a parallax effect to the coffee cup image. The key changes include:

Parallax Effect: The image moves slightly based on the mouse position, creating an engaging user experience. The translation effect is calculated to ensure smooth motion, enhancing the interactive feel of the landing page.

Changes Made:
Updated the parallaxEffect to combine both translation and rotation in the transform property of the motion.img element.
Ensured that the rotation does not interfere with the parallax effect, maintaining the visual consistency of the design.
Testing:

[x] Verified the responsiveness and visual effect across different screen sizes and browsers.
[x] Ensured no negative impact on existing functionalities or layouts.

Summary by CodeRabbit

New Features

Enhanced the landing page with interactive mouse tracking and animations.
Introduced a parallax effect for the coffee cup image that responds to mouse movements.

Bug Fixes

Minor structural adjustments to improve the layout and remove unnecessary code.

@anshi05 can you some 3d type effect on it
or 1st the coffee cup will come whilw scrolling the texts will come

Ok let me try if I can do that . Though I've never done that before.

@anshi05 sure

I have added the fading animation to the text "PLAYCAFE". So now the coffee mug is visibele and then the text comes on top of it  after few seconds enhancing overall UI.


Priority Checklist for Last Day.

@CherylCrawford
Priority Checklist for Last Day.
These things should be completed in the order given.

[x] Complete the Word Document which contains the information about the site as defined in #1 .

I used use a single-column layout for mobile, and then transition to a Holy Grail Layout with a header, footer, and center as a multi-column layout with two columns for tablets, three columns for laptops, four columns for desktop, and four columns limited to 1200px centered for large desktops. The layout is implemented by using flex boxes, percent-based element widths, use of relative units of measure (em, rem, vw, vh), and media break points at 600, 768, 992, and 1200 px.

[x] Appropriate use of graphics (at least one per page) as required in #1 .

[x] Testimonials, suggest adding a face img above each h3. Select your faces from uifaces.com.

[x] Privacy, suggest using privacy-shield.png at the top of the article, position it absolute to the right of the h3 h4 under the h2. You may have to change to width as well.

[x] Terms, google for icon or use Legal Notice Icon.

[ ] Complete the layout for mobile, tablet, and laptop using the media break points as required in #1

[ ] padding of text should be in rem instead of px for responsive design.

[ ] margin should be in % instead of px for responsive design.

[ ] #15 Add photo and comment to home page.
[ ] For mobile, Chrome dev tools will be used to emulate and test the site on an iPhone 6 as required in #1 .
[ ] Validate CSS and HTML, have brackets add webkit.
[ ] Create a .zip of the HTML, CSS, and image files which make up your website.
[ ] Submit your zip file and doc file for your project.

[ ] Change center from 2 column to one column.
For phone and tablet you will need:

#center {
  flex-direction: column;
}

How to js dropdown men on click

Project completed and turned in.  Final grade 97%


Fix bug where Tensor.randn returns inf

Tensor.randn can return ±inf in the rare case of our RNG returning a 0. The cause is the log function in the Box-Muller transform we use for randn.
PyTorch also uses Box-Muller, but avoid this issue by mapping the RNG value from [0, 1) -> (0, 1]. This PR implements that fix. [src]

Since zeros values are extremely rare, it's a bit of a pain to test. I mined a seed that would reproduce the bug on the current implementation, but I didn't add it because it seems to specific. Let me know if you want me to add it, or if you have a better test in mind.
def test_randn_isnt_inf(self):
  Tensor.manual_seed(195866)
  a = Tensor.randn(128)
  assert not np.any(np.isinf(a.numpy()))

Needs a test that was failing before change and now passes

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/tensor.py                         1               1
------------------------------------------------------------
lines added in the tinygrad folder: 0

This test is kind of ugly, but less likely to break than a mined seed.
Let me know if you prefer this or the one from the description above.

Alternatively we could make a test doing a fixed number of loops checking for infs and mark it as SLOW.
Generated (4096, 4096) matrices had a very high chance of containing infs, so ~20 iterations (less than 10s), would be pretty accurate.

I think that's fine, cute. Thanks, will merge when tests pass!


Shell Launcher Script Issue

Hello,
I am attempting to use the powershell WMI bridge provider and the provided xml script.
I keep getting error:
Set-CimInstance : A general error occurred that is not covered by a more specific error code.
At C:\Users\SomashekarChandrappa\Downloads\acm-sw\acm-sw-installation-script-sayantan_acm-automation-scripts\automation
-scripts\shell_laucnher.ps1:47 char:9
+         Set-CimInstance -CimInstance $obj
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (MDM_AssignedAcc..../Vendor/MSFT"):CimInstance) [Set-CimInstance], CimExce
   ption
    + FullyQualifiedErrorId : MI RESULT 1,Microsoft.Management.Infrastructure.CimCmdlets.SetCimInstanceCommand

My code:
if (-not ([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole([Security.Principal.WindowsBuiltInRole]::Administrator)) {
    Write-Host "This script requires administrator privileges. Please run as administrator." -ForegroundColor Red
    exit $LASTEXITCODE
}

$assignedAccessConfiguration = @"
<?xml version="1.0" encoding="utf-8"?>
<ShellLauncherConfiguration xmlns="http://schemas.microsoft.com/embedded/2011/08/shelllauncher"
xmlns:cmd="http://schemas.microsoft.com/embedded/2013/10/shelllauncher">
  <Profiles>
    <Profile Name="ACM Kiosk Profile" Description="ACM Software Kiosk Profile">
      <LaunchAction Type="DefaultApp">
        <App User="AutoLogonUser" Path="%SystemRoot%\system32\WindowsPowerShell\v1.0\powershell.exe"
             Args="-ExecutionPolicy Bypass -File %APPDATA%\Microsoft\Windows\Start Menu\Programs\Startup\acm_start.lnk" />
        <App User="AutoLogonUser" Path="%ProgramFiles(x86)%\Microsoft\Edge\Application\msedge.exe"
             Args="--kiosk http://localhost:3000/ --edge-kiosk-type=fullscreen" />
        <App User="AutoLogonUser" Path="cmd.exe" Args="/c echo Breakout key sequence: Ctrl+A" />
      </LaunchAction>
    </Profile>
  </Profiles>
  <AutoLogonAccount DisplayName="ACM_SW" />
  <DefaultProfile Name="ACM Kiosk Profile" />
</ShellLauncherConfiguration>
"@

# Set the Assigned Access Configuration
$namespaceName = "root\cimv2\mdm\dmmap"
$className = "MDM_AssignedAccess"

try {
    Write-Host "Getting the CIM instance."
    # Get the CIM instance
    $obj = Get-CimInstance -Namespace $namespaceName -ClassName $className

    if ($obj -ne $null) {
        Write-Host "CIM instance retrieved successfully."

        # Debug: Output the current configuration
        Write-Host "Current Configuration: $($obj.Configuration)"

        # Update the configuration
        $obj.Configuration = [System.Net.WebUtility]::HtmlEncode($assignedAccessConfiguration)

        # Apply the changes using Set-CimInstance
        Set-CimInstance -CimInstance $obj
        Write-Output "CIM instance updated successfully."

    } else {
        Write-Host "No instance of class $className found."
    }
} catch {
    Write-Host "Error setting CIM instance: $_"
}

Windows specification:
Edition	Windows 11 Enterprise
Version	23H2
Installed on	29-06-2024
OS build	22631.3737
Experience	Windows Feature Experience Pack 1000.22700.1009.0
Can someone point me what is going wrong?

Hello Sbasu2512, I am also straggling with this error, I've noticed that when you trying to implement "Shell Launcher" you should use $shellLauncherConfiguration = @" not $assignedAccessConfiguration = @". The same in # Update the configuration section: for ShellLauncher should use $obj.ShellLauncher = [System.Net.WebUtility]::HtmlEncode($shellLauncherConfiguration) not $obj.Configuration = [System.Net.WebUtility]::HtmlEncode($assignedAccessConfiguration). BTW - I did these two examples and in the end nothing works for me.
When I implemented:
<Configs>
    <Config>
        <AutoLogonAccount/>
        <Profile Id="{e7b80de1-ccdd-4478-876a-5fe8011cdbc7}"/>
    </Config>
</Configs>

It works, but only with local an account, I need domain an account... but when I change this part of code connected with Autologon then errors appear.
Do you have any suggestions?


Update arbitrary to prevent stack overflow.

The newest release fixes the stack overflow that
may occur for recursive types which derive Arbitrary.
The type BodyStructure was affected by this.
Fixes #61

Thanks!


Maybe something wrong in ‘/imagenet_pytorch/utils.py’？

I noticed that:
in file ‘/imagenet_pytorch/utils.py’
line 152
block_feature_II = torch.cat((block_feature_II, self.blob_dict_list[1][str(layer_id)]), 1)
maybe the self.blob_dict_list[1] should be replaced with self.blob_dict_list[self.loop_num]
to get the final loop result instead of the first loop result?

Hi,
You are right.  When we debuged our code, we always set loop_num = 1. It does not matter. We have rewrite it. Thanks!

Thank you for your notice!
I checked the code again,
but it might be self.loop_num not self.layer_num?

sorry, error has been fixed


Recent kerberos changes do not work on Darwin

In afeef240dce28f1de4c6d5a29ce5c23e562c53a4, the way that kerberos is referenced changed, and also several packages were changed to now build with kerberos support.  These changes break several deviations on Darwin, because neither heimdal nor krb5 builds there (at least, this is true on Yosemite).

There is at least one package that depends on kerberos being a string, and not null.

There is at least one package that depends on kerberos being a string, and not null.

Or rather, it does: GSSAPI="${kerberos}", which fails if it is just null.

Or rather, it does: GSSAPI="${kerberos}", which fails if it is just null.

I'll see if I can work on making it so that we can null out the kerberos dependency then.

I'll see if I can work on making it so that we can null out the kerberos dependency then.

Hopefully cd833ea6ddae0a0d57077b56d0ddeef30587a9ef fixes it for now.

Hopefully cd833ea6ddae0a0d57077b56d0ddeef30587a9ef fixes it for now.

No, I still get:
while evaluating the attribute ‘buildPhase’ of the derivation ‘serf-1.3.7’ at "/Users/johnw/src/nixpkgs/pkgs/development/libraries/serf/default.nix":4:3:
cannot coerce null to a string, at "/Users/johnw/src/nixpkgs/pkgs/development/libraries/serf/default.nix":19:16

No, I still get:
while evaluating the attribute ‘buildPhase’ of the derivation ‘serf-1.3.7’ at "/Users/johnw/src/nixpkgs/pkgs/development/libraries/serf/default.nix":4:3:
cannot coerce null to a string, at "/Users/johnw/src/nixpkgs/pkgs/development/libraries/serf/default.nix":19:16

@wkennington How does this change look?
diff --git a/pkgs/development/libraries/serf/default.nix b/pkgs/development/libraries/serf/default.nix
index fb01f17..df87496 100644
--- a/pkgs/development/libraries/serf/default.nix
+++ b/pkgs/development/libraries/serf/default.nix
@@ -18,9 +18,11 @@ stdenv.mkDerivation rec {

   buildPhase = ''
     scons PREFIX="$out" OPENSSL="${openssl}" ZLIB="${zlib}" APR="$(echo "${apr}"/bin/*-config)" \
-        APU="$(echo "${aprutil}"/bin/*-config)" GSSAPI="${kerberos}" CC="${
+        APU="$(echo "${aprutil}"/bin/*-config)" CC="${
           if stdenv.isDarwin then "clang" else "${stdenv.cc}/bin/gcc"
-        }"
+        }" ${
+          if stdenv.isDarwin then "" else "GSSAPI=\"${kerberos}\""
+        }
   '';

   NIX_CFLAGS_COMPILE = stdenv.lib.optionalString stdenv.isDarwin "-L/usr/lib";

@wkennington How does this change look?
diff --git a/pkgs/development/libraries/serf/default.nix b/pkgs/development/libraries/serf/default.nix
index fb01f17..df87496 100644
--- a/pkgs/development/libraries/serf/default.nix
+++ b/pkgs/development/libraries/serf/default.nix
@@ -18,9 +18,11 @@ stdenv.mkDerivation rec {

   buildPhase = ''
     scons PREFIX="$out" OPENSSL="${openssl}" ZLIB="${zlib}" APR="$(echo "${apr}"/bin/*-config)" \
-        APU="$(echo "${aprutil}"/bin/*-config)" GSSAPI="${kerberos}" CC="${
+        APU="$(echo "${aprutil}"/bin/*-config)" CC="${
           if stdenv.isDarwin then "clang" else "${stdenv.cc}/bin/gcc"
-        }"
+        }" ${
+          if stdenv.isDarwin then "" else "GSSAPI=\"${kerberos}\""
+        }
   '';

   NIX_CFLAGS_COMPILE = stdenv.lib.optionalString stdenv.isDarwin "-L/usr/lib";

Yeah that looks fine, I thought I changed serf to incorporate that
change.....
On Jan 1, 2015 12:35 PM, "John Wiegley"<EMAIL_ADDRESS>wrote:

@wkennington https://github.com/wkennington How does this change look?
diff --git a/pkgs/development/libraries/serf/default.nix b/pkgs/development/libraries/serf/default.nix
index fb01f17..df87496 100644--- a/pkgs/development/libraries/serf/default.nix+++ b/pkgs/development/libraries/serf/default.nix@@ -18,9 +18,11 @@ stdenv.mkDerivation rec {
buildPhase = ''
scons PREFIX="$out" OPENSSL="${openssl}" ZLIB="${zlib}" APR="$(echo "${apr}"/bin/-config)" -        APU="$(echo "${aprutil}"/bin/-config)" GSSAPI="${kerberos}" CC="${+        APU="$(echo "${aprutil}"/bin/*-config)" CC="${
if stdenv.isDarwin then "clang" else "${stdenv.cc}/bin/gcc"-        }"+        }" ${+          if stdenv.isDarwin then "" else "GSSAPI="${kerberos}""+        }
'';
NIX_CFLAGS_COMPILE = stdenv.lib.optionalString stdenv.isDarwin "-L/usr/lib";
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/5512#issuecomment-68492540.

Yeah that looks fine, I thought I changed serf to incorporate that
change.....
On Jan 1, 2015 12:35 PM, "John Wiegley"<EMAIL_ADDRESS>wrote:

@wkennington https://github.com/wkennington How does this change look?
diff --git a/pkgs/development/libraries/serf/default.nix b/pkgs/development/libraries/serf/default.nix
index fb01f17..df87496 100644--- a/pkgs/development/libraries/serf/default.nix+++ b/pkgs/development/libraries/serf/default.nix@@ -18,9 +18,11 @@ stdenv.mkDerivation rec {
buildPhase = ''
scons PREFIX="$out" OPENSSL="${openssl}" ZLIB="${zlib}" APR="$(echo "${apr}"/bin/-config)" -        APU="$(echo "${aprutil}"/bin/-config)" GSSAPI="${kerberos}" CC="${+        APU="$(echo "${aprutil}"/bin/*-config)" CC="${
if stdenv.isDarwin then "clang" else "${stdenv.cc}/bin/gcc"-        }"+        }" ${+          if stdenv.isDarwin then "" else "GSSAPI="${kerberos}""+        }
'';
NIX_CFLAGS_COMPILE = stdenv.lib.optionalString stdenv.isDarwin "-L/usr/lib";
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/5512#issuecomment-68492540.

@jwiegley is this still a problem for you on master?

still an issue?

darwin builds for serf appear to be working on current master


pkg/fileutil: add filelock support for solaris

Suggested fix for Solaris. Changed commit message as requested.

The fix should be updated with

--- a/pkg/fileutil/lock_solaris.go
+++ b/pkg/fileutil/lock_solaris.go
@@ -86,7 +86,7 @@ func (l *lock) Destroy() error {
 }

 func NewLock(file string) (Lock, error) {
-       f, err := os.Open(file)
+       f, err := os.OpenFile(file, os.O_WRONLY, 0600)
        if err != nil {
                return nil, err
        }

I pushed the update to the original clone branch and verified that it works.

@akolb1 Can you please squash the commits? This looks good to me.

Closed in favor of https://github.com/coreos/etcd/pull/2751


Dont stop on parse error.

Travis is failing.

I have no idea why it fails it a really small change and I fail to see why this pr should make it fail

Replace both os.path and os.walk imports by a simpler import os

Coverage decreased (-0.1%) to 91.702% when pulling 9e6096d13f8b3ca0185c966fe83ef36cc4bcf7d8 on Hellowlol:dontstop into 32578ee626225567c106c17963acb96e67cdab9a on Diaoul:develop.

Coverage decreased (-0.1%) to 91.702% when pulling 9e6096d13f8b3ca0185c966fe83ef36cc4bcf7d8 on Hellowlol:dontstop into 32578ee626225567c106c17963acb96e67cdab9a on Diaoul:develop.

Coverage decreased (-0.1%) to 91.702% when pulling 9e6096d13f8b3ca0185c966fe83ef36cc4bcf7d8 on Hellowlol:dontstop into 32578ee626225567c106c17963acb96e67cdab9a on Diaoul:develop.

Coverage decreased (-0.1%) to 91.702% when pulling 9e6096d13f8b3ca0185c966fe83ef36cc4bcf7d8 on Hellowlol:dontstop into 32578ee626225567c106c17963acb96e67cdab9a on Diaoul:develop.

Coverage decreased (-0.1%) to 91.702% when pulling 9e6096d13f8b3ca0185c966fe83ef36cc4bcf7d8 on Hellowlol:dontstop into 32578ee626225567c106c17963acb96e67cdab9a on Diaoul:develop.

What can i do to get this merged?

ValueError is quite broad, I think it's best to restrict the scope of the exception handling to the only portion of the code that is problematic.

Valueerror was the exception i got. Ill try to run it again and see if can make it less broad

@Hellowlol any news on this one? I'm helping Diaoul to maintain v2 so as soon you fix the review we can merge

@Diaoul This is the exception from datetime:
ValueError: year is out of range
I don't think we can make less broad. What do you think?

Then do something like this:
# skip old files
try:
    file_age = datetime.utcfromtimestamp(os.path.getmtime(filepath))
except ValueError:
    logger.warning('Could not get age of file %r in %r', filename, dirpath)
else:
    if age and datetime.utcnow() - file_age > age:
        logger.debug('Skipping old file %r in %r', filename, dirpath)
        continue

Closing this as it was cherry-picked to another PR


Update link to usage scopes

I noticed that the usage scopes link leads to a 404. I updated that link with the current location

Thanks!


UI issues.

- Tested on iPhone SE

On Android pressing the button twice (when setting up the passcode), the app hangs.

Verified this issue on staging app v2.0.5(382).Need the confirmation from user.


Incorrect warning "unnecessary grouping operator" from singleGroups when using Object Destructuring Assignment without declaration (ES6 feature).

Citation from MDN:

Destructuring assignment can be made without a declaration in the assignment statement.
var a, b;

({a, b} = {a:1, b:2});

The ( .. ) around the assignment statement is required syntax when using object literal destructuring assignment without a declaration.

JSHint should not warn in that case.

You are correct! Complete test case:
// jshint esnext: true, singleGroups: true
var x;
({ x } = {});

Thank you @nicolo-ribaudo! However, note that the parens are necessary only for object destructuring assignment, not for array destructuring assignment. The following code does work:
var a, b;

[a, b] = [1, 2];

See MDN

@myrdd Yup, the patch over at gh-2862 takes that into account.

Oh, you're right! I should have checked what addError() actually does. :] Great work!


how wait for async load to complete

I am not sure if this is the correct place to open this issue but I am trying to load a local wasm node module in my app, It works fine in the dev build but when I do a prod build imports from wasm are undefined. as a workaround I had to add
await import('game-of-life');

before I call any of my wasm functions. I also have a top-level import for the types
import { Universe, Cell } from 'game-of-life';
import { memory } from 'game-of-life/game_of_life_bg.wasm';

I understand async loading of wasm is a bit new, so I am unsure if I am doing this correctly. It just doesn't feel correct 2 have import statements to the same module
complete source code can be found here

Let me check

I found that the direct reason that make it undefined is the assignment to wasm variable is missing in the generated bundle. And it works with Rollup's tree-shaking disabled.
diff --git a/www/apps/app-game-of-life/src/app/app.element.ts b/www/apps/app-game-of-life/src/app/app.element.ts
index 1fa599a..34a8235 100644
--- a/www/apps/app-game-of-life/src/app/app.element.ts
+++ b/www/apps/app-game-of-life/src/app/app.element.ts
@@ -18,7 +18,7 @@ export class AppElement extends HTMLElement {
   private height = 128;

   async connectedCallback() {
-    await import('game-of-life'); // wait for wasm module to load, should be better way
+    // await import('game-of-life'); // wait for wasm module to load, should be better way

     this.innerHTML = `<canvas id="${this.ID}"></canvas>`;

diff --git a/www/apps/app-game-of-life/vite.config.ts b/www/apps/app-game-of-life/vite.config.ts
index 5624b7a..6a5cf10 100644
--- a/www/apps/app-game-of-life/vite.config.ts
+++ b/www/apps/app-game-of-life/vite.config.ts
@@ -7,7 +7,10 @@ export default defineConfig({
   cacheDir: '../../node_modules/.vite/app-game-of-life',

   build: {
-    target: 'esnext'
+    target: 'esnext',
+    rollupOptions: {
+      treeshake: false
+    }
   },

   optimizeDeps: {

It doesn't seem to be an issue with this plugin but seems to be Vite/Rollup's. I'll try to produce it and submit an issue to Vite or Rollup. If so, let's keep this issue open for reference before it's fixed.

No, I think it's an issue with wasm-pack. It generates the pacakge.json with sideEffects: false and use a __wbg_set_wasm statement to set the wasm module object. I think it's the exptected behavior for it to be tree-shaked...

I think the bug is introduced in https://github.com/rustwasm/wasm-bindgen/commit/721c86c3467750bf2a46bf85941e6c211ae9e87a. Let me submit a bug to rustwasm.

https://github.com/rustwasm/wasm-pack/issues/1235

Thanks for looking into it and providing the explanation, it makes sense I'll track the issue on waan-bindgen maybe try a fix

yeah adding to side-effects fixes it


[Refactor] Refactor Starrocks LOG to reduce the log file size(part3)

Why I'm doing:
Refactor Starrocks LOG to reduce the log file size.
What I'm doing:
Refactor Starrocks FE LOG to reduce the log file size.
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@mergify backport branch-3.2

@mergify backport branch-3.3


Update imgui

Updated to latest ImGui.NET package.
This should also work with code first now... that is still super specific though so whatever. Worst case it will work as it did before.
Need to validate that the key maps were not broken.

I may have found a better ImGui .NET implementation managed by the Hexa game engine. I already converted this library to that one and I think I will switch to that instead of this update if you're ok with that @Eideren
It also works out of the box with some useful addon libraries which is a huge bonus.

I decided to go with the full overhaul using Hexa.ImGui.
This PR now includes fixes for window resizing and the ability to scale text through the ImGuiSystem and gives the users access through the BaseWindow class.

@Eideren speaking of the Keys, did you want me to go ahead and just add all the available ones to that dictionary?

The reason for using Hexa

Thanks a bunch, looks really good ! Can you quickly go over why you've decided to swap to Hexa's bindings ? I'm guessing that it could be because of a lack of support for the latest imgui features ... just want to get your thoughts.

It was the latest features that became the issue like you guessed. Also since Hexa maintains the nuget packages for ImNodes, ImPlot and ImGuizmo I did not have to worry about the ImGui version like I was facing with the ImGui.NET package seen here.
My only initial concern is that ImGui.NET is maintained by a community whereas Hexa seems to only be one or two so there is some risk there but they did publish the repo that generates the bindings which made me feel more comfortable.

@Doprez afaik those keys are only there for common UI operation shortcuts, let's leave it for now and investigate again if something doesn't work as expected.
Thanks again !


Decompiled code with errors can't be edited through right clicking, but can be properly edited using the search menu

Describe the bug

A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

Open a class file with decompilation errors
Try to right click and bytecode edit (the menu shouldn't appear)
Open the search and find the value or piece of text to edit
Right click to bytecode edit
The editor should appear and you can edit and properly save the code.
Look at the decompiled code, the code should look the exact same as before, as if the edits weren't even made.
Export it, then look at it again and it should update

Screenshots
https://github.com/Col-E/Recaf/assets/146917292/a89ed86d-a402-44ce-9c7b-19d3212e5d5b

2X: Change the class display mode - https://www.coley.software/Recaf-documentation/use-modes.html
3X: Use the side panel to interact with members when the code cannot be parsed into an AST

Changing the title as this is normal behavior outlined in the documentation


feat: Rudimentary belongs to many support

In addition to the other limitations, btm doesn't support limit

Hmm, gogogo circle!

https://github.com/sequelize/sequelize/pull/6560

1.1.0


Distinguish heading in methodology microsite by different sizes

Different headings in methodology on microsite are more distinguished by size. Made the changes in the markdown file as per the acceptance criteria of the issue.
fixes #325
cc - @ericksod
Checklist

[x] PR description included and stakeholders cc'd
[x] tests are changed or added
[x] yarn test passes
[x] yarn lint has been run
[x] git pre-commit hook is successfully executed.
(Please refrain from using --no-verify)

Notes
Additional information relevant to the changes
© 2021 Thoughtworks, Inc.

Than you for the contribution!


Reset locale on login

During login reset the user's locale to ensure that it matches their
preferred locale.
Fix #15612
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@013fb73). Click here to learn what that means.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##             main   #17734   +/-   ##
=======================================
  Coverage        ?   45.51%
=======================================
  Files           ?      807
  Lines           ?    89797
  Branches        ?        0
=======================================
  Hits            ?    40873
  Misses          ?    42386
  Partials        ?     6538

Impacted Files
Coverage Δ

routers/web/user/auth.go
12.06% <0.00%> (ø)

modules/context/api.go
74.41% <100.00%> (ø)

modules/context/context.go
61.09% <100.00%> (ø)

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 013fb73...386bb78. Read the comment docs.

Similar unrelated CI errors occur again and again 😭


Requirements

We already have a lot of requirements for the different use-cases in different Fachschaften, I'll try to summarise things here.
Things should be added to this list in-place, with a note below if the changes were significant. The different requirements can then be spun out to tickets for implementation.
Common Features (non-exhaustive)

[ ] pre- and post-paid, should probably be a global setting, with some user settings in the post-paid case. When post-paid is globally set, identifiable users should be allowed to set themselves to "pre-paid" as a means of cost control, pseudonymous/non-identifiable users should probably be pre-paid only.

Kasse des Vertrauens/KIF
Required

[ ] self-service
[ ] 1-to-1 relationship between identifiers and products
[ ] 1-to-1 relationship between identifiers and users
[ ] users and products share the same identifier space
[ ] 1-to-1 relationship between products and prices
[ ] post-paid, i.e. negative balance possible

Nice-to-haves

[ ] paper trail/transaction log
[ ] pseudonymous mode
[ ] automatic reminders for negative balances (obviously not in combination with pseudonymous users)
[ ] password protection for user accounts possible, i.e. no sale without the password if a password is set

Full-fledged Kiosk/Dortmund
Required

[ ] full-service (a.k.a mit Verkäufer)
[ ] many-to-1 relationship between identifiers and products
[ ] many-to-1 relationship between identifiers and users
[ ] 1-to-many relationship between products and prices
[ ] pre-paid, i.e. no negative balance possible
[ ] password protection for user accounts possible, i.e. no sale without the password if a password is set
[ ] inventory management

Nice-to-haves

[ ] an administrator role besides buyers and sellers that can manage inventory and prices
[ ] notes on prices (e.g. "this is the default price" "this is an action price for the next 20 bottles sold", …)
[ ] choose the order of prices, with the default being "cheapest first"
[ ] paper trail/transaction log
[ ] categories/tags/some sort of taxonomy

Open Kiosk/Tübingen
Required

[ ] password protection for user accounts
[ ] 1-to-1 relationship between identifiers and products
[ ] 1-to-1 relationship between identifiers and users
[ ] 1-to-1 relationship between products and prices
[ ] post-paid, i.e. negative balance possible
[ ] tracking returnables (Pfand), i.e. a user cannot return more returnables than he bought
[ ] paper trail/transaction log

Nice-to-haves

Ping @infusion, @fabian-rump: do you see anything missing, respectively for Tübingen and KIF?

I have nothing to add for KIF, except for putting "password protection" on the nice-to-have list. Yeah I know, KIF is a trust-based system anyways, but it never hurts to have some extra security ;-)
Furthermore I suggest to allow setting pre-paid/post-paid on a per-user basis. I think about deploying the system in Bonn where this could be handy (e.g. post-paid for student association members only). This could also be a first solution to pseudonymous mode (e.g. have some extra user identifiers which are not linked to a specific person, but require the users to put money into their accounts first).

@fabian-rump good points. I have added the password-protection to the KIF list, and I have started a "Common Features" section for things like pre-/post-paid discussions.
Though I have added a few more notes to that point, discussions to specific points should happen in separate issues though.

Hey guys, nice work :3  !
Maybe you already did this and this ist just repetition:
I tried to merge the requirements for the tickets, because a lot does't look opposing

self-service vs full-service (a.k.a mit Verkäufer)
no problem with a self-service account system, full-service can have a ''vendor" account
many-to-1 relationship between identifiers and products
so 1-to-1 is no problem
many-to-1 relationship between identifiers and users
so 1-to-1 is no problem
users and products share the same identifier space
seems other have no problem with that
many-to-1 relationship between products and prices
so 1-to-1 is no problem
optional post-paid, i.e. negative balance possible
admin has to allow negative balance
optional password
users can set one, if they want to
inventory management
tracking returnables (Pfand)
"Pfand" just as another product with negative cost?
paper trail/transaction log
when working with financial data you should track every flowing cent and transaction

To add on the Tübingen nth:

notify 'someone' on low product stock

regards David-Elias (Tübingen)

@lasergans Thanks for chiming in, I really appreciate it!
I had not yet really started summarising the requirements together, but a lot of things are overlapping, and the ones that don't can be made configurable/optional without too much hassle indeed. There's also a few things already implemented, but I'll mark them later.
I have also added the nice-to-have point for Tübingen, though I suspect that would be nice-to-have for everyone :smile_cat:

Tansactions: ✔ Werden nur nicht angezeigt oder nix.


Bug with non generic ResolveWith on the ObjectFieldDescriptor class

Looking at the generic method ResolveWith<> you can see that an IObjectFieldDescriptor but this is not the case for the non generic implementation.
I have amended the code so that it returns the and object in the same place as the generic version.


How to verify the response content in async tests?

Looking at this example, I wonder what is the point of specifying the body in .willRespondWith()?
Looks like the body is verified in the success function, and if this line is removed, the test passes (so the body that is specified in .willRespondWith() is basically ignored).
In other words, looks like I need to duplicate the expected response in 2 places:

body in .willRespondWith()
expect in success()

Is there a way to avoid the duplication?

The .willRespondWith() is required to be set to stub out the response, and to allow us verify (on the provider side) that the real response matches the actual response. If you took that out, the test would fail, because the client code would have no body to parse.
The expectation in success asserts that your code has correctly handled the response.

Thanks, makes sense now


POST https://domain/webauthn/register 422 at webauthn.js:159

Please check these requirements

[X] This feature helps everyone using this package
[X] It's feasible and maintainable
[X] It's non breaking
[ ] I issued a PR with the implementation (optional)

Description
Please i am trying to use webauthn in my app i had to upgrade to laravel 9 to user Laragear/WebAuthn because i was unable to set up Larapass and i can't create any issue there.
Wel Successfull Set WebAuthn but having Issue registering Device...
I get 422 Error Code
And when I tried publishing the config file.
php artisan vendor:publish --provider="Laragear\WebAuthn\WebAuthnServiceProvider" --tag="config"
It doesn't work.
Please I need guidelines on how to successfully set it up
Code sample
public function register(AttestedRequest $request): Response
    {
        $request->save();

        return response()->noContent();
    }

Hi.
This is not a support forum. You may create a thread on other communities.
Since it's a 422 code, it means there was a validation error, telling what failed. You can also debug if you think is an error with the library.


Hello. Press and hold the master text at the left bottom corner. You can

Hello. Press and hold the master text at the left bottom corner. You can
then select a branch to check out to.
It is still not letting me change it to the master branch. How do I reset my source control?
On Wed, 19 Jan 2022 at 10:42, a-delacerda1 @.***> wrote:

Hello! I was wondering if there is a toggle so that I can change my branch
from master to main in this app. Thank you so much!
—
Reply to this email directly, view it on GitHub
https://github.com/thebaselab/codeapp/issues/335, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJE6T247F2RWEJAB7IRR6N3UWYQILANCNFSM5MIXTYXQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Originally posted by @bummoblizard in https://github.com/thebaselab/codeapp/issues/335#issuecomment-1016025865

Can you send me a screen capture of the issue?


Update cloud-services-guestos-msrc-releases.md

February MSRC automatic updates

@v-piwijs : Thanks for your contribution! The author, @raiye, has been notified to review your proposed change.

#sign-off

@v-piwijs @raiye - I've done my best to verify that this content submission will render well as published, but we'd much prefer to use the private repos for large additions and changes to content. The private repo lets us use staging and validation tools to ensure content layout and links are properly working. Can you make a note to submit PRs to the private repo in the future? Thanks.

Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.

@Jak-MS - Jak, thanks for noticing that, I have wondered what changed, or what I was doing wrong.
Using azure-docs-pr in future, rather than azure-docs.


Text of dialog boxes overlap

If two dialog boxes (hero say or NPC) are present the text in them overlaps.

Do the chat boxes have a border layout?

Do the chat boxes have a border layout?


hardcoded: Warning about point-free style

@alfonsogarciacaro I am having the warning about point-free style on the hardcoded decoder. But I think, we should allow this function style no ?
I also, don't know how to rewrite it to mimic the >> operator and so remove the point-free style. Can you please help me to removed the warning one way or another ?
Error:

fable: Compiled src/Thot.Json/Encode.fs
(!) fable plugin: /home/mangelmaxime/Workspaces/Github/MangelMaxime/Fable.Json/src/Thot.Json/Decode.fs(432,4): (432,13) warning FABLE: Looks like Thot.Json.Decode.hardcoded uses point-free style, this may create problems in runtime, please declare all arguments explicitly.

In latest Fable version the warning is removed again, sorry! 😅
You could use something like this (with better parameter names):
let hardcoded<'a, 'b, 'c> (x: 'a) (y: Decoder<('a -> 'b)>) (z: 'c) (u: Result<'b,DecoderError>) v =
    succeed v |> custom

But even like that dynamic CurriedLambdas are still being generated so I guess it doesn't really matter:
// let hardcoded<'a, 'b, 'c> : 'a -> Decoder<('a -> 'b)> -> 'c -> Result<'b,DecoderError> = succeed >> custom
function hardcoded() {
    return CurriedLambda($var3 => CurriedLambda(function (d1, d2) {
        return custom(d1, d2);
    })(($var1 => $var2 => function (output, arg10_) {
        return succeed(output, arg10_);
    }($var1, $var2))($var3)));
}

// let hardcoded<'a, 'b, 'c> (x: 'a) (y: Decoder<('a -> 'b)>) (z: 'c) (u: Result<'b,DecoderError>) v =
//    succeed v |> custom
function hardcoded(x, y, z, u, v) {
    return CurriedLambda(CurriedLambda(function (d1, d2) {
        return custom(d1, d2);
    })(function (arg10_) {
        return succeed(v, arg10_);
    }));
}

Thanks you :) will try it and update dependencies to the latest version too


Stats page branch

-Adds charts on stats page
-Adds class that reads CSV synthetic data
-Stores synthetic data in data store

I can't see statistics.html, did you commit it?

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.


NO-ISSUE: Improve EnrollmentRequests list loading/empty states for st…

…andalone/embeded version

@coderabbitai review


Add sgort to owner organization MinBZK

Owner right is required for @sgort to claim the github organization MinBZK.

Added sgort to owners.


Update README.md

add NoiseTotal for false positive intelligence

add a link to popular NoiseTotal - the threat intelligence for false positives & security noises in all kinds of security detections, tools, etc.


Error: java.lang.ArrayIndexOutOfBoundsException: length=1; index=1

java.lang.ArrayIndexOutOfBoundsException: length=1; index=1
How can I do?

Having the same issue and can't figure out how to solve this. Any ideas?

Take a look at the README, the solution is there. Override the ViewGroup the PhotoView is in with the solution provided in the README.


Fix console output in production

@agherzan @floion
Signed-off-by: Theodor Gherzan<EMAIL_ADDRESS>

Addresses #30

LGTM

PR rebased (removed) from master.


Redundant reset methods?

This is not a big deal I've noticed that your Cpu has a reset method. I did something similar when I implemented a GameBoy emulator but I got rid of it after a while. The main problem is that I was worried about adding a new state variable to the struct and forget to reset it properly in the method. Moreover you end up having a lot of redundant code between your constructor and reset function.
For this reason I prefer to have a constructor that builds an instance of the module "at reset". When I need to reset something I simply build a new instance and replace the old. Since Rust will raise an error if you don't initialize something in your constructor you're sure you won't forget to set the reset value correctly.
Good luck with your project!

I guess you could also be cheeky and have a reset fn like this:
pub fn reset(&mut self) {
    *self = Cpu::new();
}

Then you still get the same semantics with that fn, and the niceties you get with initializing everything properly in the ctor :)

Something "feels" a bit wrong with doing away with the function entirely; perhaps I'm thinking I really want a soft_reset fn that will maintain certain bits of state, and hard reset would just be recreating the whole graph.

Yeah it's a matter of "style" really, I can see wanting a reset method to stay closer to the actual chip or something.
On the PlayStation I know that the RAM, VRAM and some registers are not reset when you press the "reset" button but I don't actually bother keeping track of that when a reset is requested by the user. I just put random "garbage" data in them by default. My main concern with a "partial reset" would be to end up with a strange bug that would only happen in certain situations after a partial reset and would be a pain to reproduce and track down. If you always reset to the the same "clean" state you run should be more deterministic, hopefully...

Considering power_on_reset simply initializes the state of the Cpu and all dependencies, doesn't it make sense to have new simply do that? Is there any circumstance in which you'd call new without calling power_on_reset? Is there any circumstance in which you'd call power_on_reset but you couldn't simply call new?
Obviously doing so would require you to add a new function for some types like Cp0 and RegConfig, but I don't see why the separation of concerns. new is for initialization of a struct and that's exactly what power_on_reset is doing. It just seems like you're initializing each struct half in new/Default and half in power_on_reset.

@iherbig: For power_on_reset it is indeed a bit strange to have the separation, but I suppose I was thinking you might want to only instantiate the graph once, and call power_on_reset later to simulate a hard reset. It could be argued that you could just re-instantiate the graph (as @simias is also suggesting) and I'll probably switch to that, but I'll likely also have a soft_reset method as well that simulates the reset button on the system (or the reset pin in the case of the subcomponents).

I'll go ahead and fix this on one of the next streams probably. I'd like to request nobody does a PR for it, because I want to talk about why we want to do this on stream :)

I definitely think a separate soft_reset is called for, especially if you're going for accuracy of emulation. I'm not sure if there's any examples on the N64, but I know that the NES has at least one case (Final Fantasy) where the soft reset mechanism can actually be used to influence RNG. (Speed runners of the game will often power down the console and let the RAM expire to reset the system back to a cold-reset state.)
Look forward to your next stream, Jake.


local: add statsd-prometheus exporter for MW

Bug: T364107

No point in merging this until we have a new enough version of mediawiki to use it (probably 1.41+)


Counter App is not adhering to flutter lint "use_key_in_widget_constructors", "prefer_const_constructors_in_immutables"

Is your feature request related to a problem? Please describe.
It doesn't affect where the counter example app describes the behavior for Riverpods.
However, I would like the example app to adhere to lint.
"use_key_in_widget_constructors",
"prefer_const_constructors_in_immutables"
Describe the solution you'd like
Widget build(BuildContext context) {
    return MaterialApp(home: Home());
  }
}

/// Providers are declared globally and specify how to create a state
final counterProvider = StateProvider((ref) => 0);

class Home extends ConsumerWidget {

to
  @override
  Widget build(BuildContext context) {
    return const MaterialApp(home: Home());
  }
}

/// Providers are declared globally and specify how to create a state
final counterProvider = StateProvider((ref) => 0);

class Home extends ConsumerWidget {
  const Home({super.key});

Describe alternatives you've considered
Additional context

Closing, as discussed in the PR


Error in chapter 8 KL divergence formula

Information
The problem arises in chapter:

[ ] Introduction
[ ] Text Classification
[ ] Transformer Anatomy
[ ] Multilingual Named Entity Recognition
[ ] Text Generation
[ ] Summarization
[ ] Question Answering
[THis one] Making Transformers Efficient in Production
[ ] Dealing with Few to No Labels
[ ] Training Transformers from Scratch
[ ] Future Directions

Describe the bug

The formula used to calculate KL divergence is wrong
To Reproduce
Steps to reproduce the behavior:

loss_fct = nn.KLDivLoss(reduction="batchmean")
loss_kd = self.args.temperature ** 2 * loss_fct(F.log_softmax(logits_stu / self.args.temperature, dim=-1),F.softmax(logits_tea / self.args.temperature, dim=-1))

Expected behavior

loss_fct = nn.KLDivLoss(reduction="batchmean")
loss_kd = self.args.temperature ** 2 * loss_fct(F.log_softmax(logits_stu / self.args.temperature, dim=-1),F.log_softmax(logits_tea / self.args.temperature, dim=-1))

Reference :https://pytorch.org/docs/1.13/generated/torch.nn.KLDivLoss.html#torch.nn.KLDivLoss

Hi, you might find this answer helpful https://discuss.pytorch.org/t/kl-loss-and-log-softmax/69136/2


Any way to disable beeline feature?

Is there any easy way to disable beeline feature similarly to draw function (https://github.com/nrenner/leaflet-routing/blob/dev/src/L.Routing.js#L786)? I know there is a toggleBeelineDrawing function but even if it's disabled it still responds to a click event on an existing segment (https://github.com/nrenner/leaflet-routing/blob/dev/src/L.Routing.Edit.js#L360) which is undesirable from my perspective. Actually the line 366 from the mentioned function is what I'd like to modify as it always set beeline to true:
prev._routing.beeline = !prev._routing.beeline;

I believe it would be more correct if the "Toggle straight line" button turned off the beeline feature completely including clicking on a segment. It should also depend on the drawing mode if it's active or not. I tried to modify the code as described but failed unfortunately.
Background: I'm maintaining my fork of brouter-web adding Google Street View integration and this beeline feature really interferes with it.

draw only refers to appending new segments, not editing.
You should be able to disable editing like this (haven't tried):
routing._edit.disable()

This is perfect, exactly what I needed. Here is the patch https://github.com/thof/brouter-web/commit/2aede019ad0f6dc6ee32fe342efdc4411d9df893 in case anyone has a similar request.
Thanks for your prompt help, I appreciate it.


cmd/compile: constant indexing to private array should be folded

Lookup tables are quite common, as in https://golang.org/src/math/bits/bits_tables.go. However none of functions that rely on those tables can be constant folded.
Simplest example is https://go.godbolt.org/z/MhMTqb which compiles to:
movblzx math/bits.rev8tab+12(SB), AX
movb    AL, "".~r0+8(SP)

The rev8tab only contains indexing to that table, so it should be possible to prove that the array is not being modified. Of course that only holds, when assignment, slicing or addressing operator is not used.
If such table is replaced with a string, then the constant folding does work. However, that seems like an ugly hack to have.
Related Issues:

https://github.com/golang/go/issues/10848
https://github.com/golang/go/issues/21872

If anyone wants to work on this, one way to approach it would be to mark the array symbol as readonly in the pre-SSA phase of the compiler (if you can prove it is in fact readonly). Then SSA optimizations for loading from readonly memory should kick in.

@mdempsky mentioned that two other concerns are assembly code and //go:linkname.

I also realized there's a way to declare an array or slice, such that it's guaranteed to be read-only:
func lut() [256]byte { return [256]byte{1,2,3,4} }


Changed abandoned dependency

Changed abandoned depenency from "liip/doctrine-cache-bundle" to "doctrine/doctrine-cache-bundle"
No side effects found
Fixing https://github.com/elcodi/bamboo/issues/199
Ping @elcodi/owners

Are we even using this one?

Well, we're using this in some of the Elcodi bundles like some other requirements, maybe we could clean the list.

This dependency has been removed. Closing


Implement crypto_aead_chacha20poly1305

Hey @Skycoder42 i want to work on https://github.com/Skycoder42/libsodium_dart_bindings/issues/46, but i need some help with the Crypto api.
I'm not sure about this first implementation, but i didn't find another solution to not cause a breaking change.
I will work on the docs and tests after we decide the new api.

The API looks fine to me. Simply reusing the Aead interface for this was my intention anyways. Have a look at my review comments for a few minor ideas on the implementation.

Ok, thank's for your time.

Having a "breaking" change by adding new methods to the interfaces is fine, as they are not intended to be implemented outside of this package, so from a logical perspective it's not a breaking change, as consumers of the API will not have to change any code

I was thinking about changing the aead getter in the Cryto api to make more clear that the default implementation is xchacha20poly1305_ietf, maybe renaming the getter which would cause a breaking change.

I will continue this implementation for now, but i think there's a better way to represent the default aead method.
* sorry for my English, i'm still learning

I finished the adjustment in the documentation!
Thanks again for the time to review.


Fix DeprecationWarning in unescaped literal and SyntaxWarning over literal is comparison.

Fix DeprecationWarning and SyntaxWarning in CI
Description
This PR fixes DeprecationWarning and SyntaxWarning raised in Python 3.8 in CI. Reference : https://travis-ci.org/apache/libcloud/jobs/619055704#L587
Status
Replace this: describe the PR status. Examples:

done, ready for review

Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[x] Tests
[ ] ICLA (required for bigger changes)

I am not sure CLA is required for this minor set of fixes. Let me know if you need one. Thanks.

Codecov Report

Merging #1378 into trunk will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##            trunk    #1378      +/-   ##
==========================================
+ Coverage   86.27%   86.27%   +<.01%
==========================================
  Files         372      372
  Lines       76319    76319
  Branches     6979     6979
==========================================
+ Hits        65844    65846       +2
+ Misses       7679     7678       -1
+ Partials     2796     2795       -1

Impacted Files
Coverage Δ

libcloud/compute/drivers/vcloud.py
82.1% <ø> (ø)
:arrow_up:

libcloud/test/compute/test_cloudsigma_v1_0.py
100% <100%> (ø)
:arrow_up:

libcloud/test/compute/test_dimensiondata_v2_4.py
88.02% <100%> (ø)
:arrow_up:

libcloud/test/compute/test_dimensiondata_v2_3.py
88.01% <100%> (ø)
:arrow_up:

libcloud/test/compute/test_upcloud.py
91.39% <0%> (+1.32%)
:arrow_up:

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 ad404dc...6799001. Read the comment docs.

@tirkarthi Thanks for the contribution.
Since it's a small change, I don't think ICLA is really needed, but if you plan any future contributions it would still be good to sign it :)

Thanks @Kami :)


Create Listview for prospects

I'll be working on this.

I created the #170 PR which resolves this issue.
An API endpoint was created which interacts with ZuriCore to read the "prospects" collection stored then returns an array of prospect objects.
Slack username: Mikaleson
Test Link: https://sales.zuri.chat/prospects/

Key Error

Someone changed the URLs but its been fixed.
Here's the new test link https://sales.zuri.chat/prospects/


Add debug lines for colliders

While Rapier has a debug system for colliders. It is difficult to use effectively and requires commenting of major sections of code to remove sprites. Let's instead use the bevy_debug_lines plugin to draw debug lines where colliders are instead just like in Amethyst Theta Wave.
https://github.com/Toqozz/bevy_debug_lines
Debug lines should only appear in debug builds: cargo run
They should not appear in release builds: cargo run --release

There is a system ordering issue between the bevy_prototype_debug_lines plugin and bevy_rapier2d causing the drawn hitbox on moving entities to "lag" behind the actual position. This has been confirmed through talking with the developer.


KYLIN-4491 Support presto pushdown

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[x] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[x] Compiling and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[x] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Test Prepare

Configuration

Test Pushdown

Presto PushDown
No matter what data source you use, you can specify the corresponding data source to push down the query
kylin.query.pushdown.runner-class-name=org.apache.kylin.query.pushdown.PushDownRunnerOtherImpl
kylin.source.jdbc.dialect=presto
kylin.source.jdbc.adaptor=org.apache.kylin.sdk.datasource.adaptor.PrestoAdaptor
kylin.query.pushdown.jdbc.url=jdbc:presto://<IP_ADDRESS>:8083/hive/default
kylin.query.pushdown.jdbc.driver=com.facebook.presto.jdbc.PrestoDriver
kylin.query.pushdown.jdbc.username=presto
kylin.query.pushdown.jdbc.password=

kylin.query.pushdown.runner-class-name=org.apache.kylin.query.pushdown.PushDownRunnerOtherImpl :  whether to enable query pushdown
kylin.source.jdbc.dialect=presto : specifies JDBC dialect. The default value is default
kylin.source.jdbc.adaptor=org.apache.kylin.sdk.datasource.adaptor.PrestoAdaptor : specifies JDBC adaptor

LGTM

Codecov Report

Merging #1209 into master will decrease coverage by 0.03%.
The diff coverage is 0.00%.

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

Codecov Report

Merging #1209 into master will increase coverage by 0.86%.
The diff coverage is 0.00%.

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

Pull Request Test Coverage Report for Build 5962

4 of 113 (3.54%)  changed or added relevant lines in 5 files are covered.
4366 unchanged lines in 84 files lost coverage.
Overall coverage increased (+0.9%) to 28.268%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

datasource-sdk/src/main/java/org/apache/kylin/sdk/datasource/framework/SourceConnectorFactory.java
4
5
80.0%

query/src/main/java/org/apache/kylin/query/util/PushDownUtil.java
0
2
0.0%

server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java
0
2
0.0%

query/src/main/java/org/apache/kylin/query/util/PushDownExecutor.java
0
4
0.0%

datasource-sdk/src/main/java/org/apache/kylin/sdk/datasource/adaptor/PrestoAdaptor.java
0
100
0.0%

Files with Coverage Reduction
New Missed Lines
%

cache/src/main/java/org/apache/kylin/cache/cachemanager/RemoteLocalFailOverCacheManager.java
1
85.71%

engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/CalculateStatsFromBaseCuboidMapper.java
1
95.35%

query/src/main/java/org/apache/kylin/query/util/PushDownExecutor.java
1
0%

query/src/main/java/org/apache/kylin/query/util/PushDownUtil.java
1
85.37%

core-cube/src/main/java/org/apache/kylin/cube/cuboid/TreeCuboidScheduler.java
2
68.46%

core-metadata/src/main/java/org/apache/kylin/metadata/expression/ExpressionCountDistributor.java
2
94.44%

core-cube/src/main/java/org/apache/kylin/cube/CubeDescManager.java
3
38.57%

source-hive/src/main/java/org/apache/kylin/source/hive/HiveTableMeta.java
3
0%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
5
77.81%

core-metadata/src/main/java/org/apache/kylin/measure/bitmap/BitmapMeasureType.java
5
18.52%

Totals

Change from base Build 5905:
0.9%

Covered Lines:
25739

Relevant Lines:
91053

💛  - Coveralls