Consider reexporting crates for which ShaderType are implemented

Could you reexport crates like nalgebra to make sure consumers of encase are using the exact types for which ShaderType is implemented?
Right now, I think the only way to handle this is to manually check the version of nalgebra being used by encase and add that as a direct dependency of the dependent project.
Also, thank you for the amazing work =)

Thank you! :)
Reexporting the crates sounds reasonable, would you like to open a PR?

I think I can give it a shot over the weekend =) I'll have to look around and see what the best practices for this are

Hey, sorry it took me this long to get back to this, but here:
Does this look ok to you?


Video captions don't work

I've been trying to get video captions to work for a while and they've been consistently failing.
Unfortunately, since I don't understand much of the source code that goes into rendering captions, I'm unable to track the issue down further.
However, to help pinpoint the issue, I have created a test page should help in reproducing the bug.
I've written this page that demonstrates the problem.
As seen on that page, on Chrome, the native player renders captions correctly whereas plyr does not.

Sounds like I need to have another look at how the captions are working. The support was mixed at the start of the project and I think things have changed recently. I'll take a look.

Thanks for your fix, can we close this one now?

Not quite.
Chrome still plays captions natively whereas plyr goes down a code path
that thinks it cannot play them natively.
I'm not sure if that's okay or if it makes more sense to have it render
natively.. (Perhaps there are some performance implications?)
I'll leave that decision to you.
On Feb 20, 2016 8:30 PM, "Sam Potts"<EMAIL_ADDRESS>wrote:

Thanks for your fix, can we close this one now?
—
Reply to this email directly or view it on GitHub
https://github.com/Selz/plyr/issues/162#issuecomment-186716490.

The problem seems to be that Chrome decides it doesn't want to fire a cuechange event after doing it twice. Safari and Firefox have some pretty tragic bugs in this area too. It might be better to manually render them for everything which is a shame, I guess it's not important for the browser developers :-1: I'll keep digging.

Found the issue - my demo webVTT file didn't pass validation and the browsers don't seem to communicate these issues, they just fail silently - awesome.
Found a good validator and added it to the docs:
https://quuz.org/webvtt/
Done a quick test in the various browsers, Firefox still doesn't support the cuechange event - a bug has been raised. I found that Safari now supports TextTrack... well, apart from ignoring the type attribute on the track (we set it to hidden to hide default rendering) but that's solved by hiding them with CSS.
Grab v1.5.12 and let me know how you go but I think we're all good?


dotnetfx v<IP_ADDRESS>80712 installation fails

Hi.
The dotnetfx v<IP_ADDRESS>80712 version cannot be installed
The remote file either doesn't exist, is unauthorized, or is forbidden for url 'https://download.microsoft.com/download/6/E/4/6E48E8AB-DC00-419E-9704-06DD46E5F81D/NDP472-KB4054530-x86-x64-AllOS-ENU.exe'. Exception calling "GetResponse" with "0" argument(s): "The remote server returned an error: (404) Not Found."
Could you please tell if this is going to be resolved or some other package should be used for this purpose?
Thank you

For the record, dotnetfx version <IP_ADDRESS>10909 with updated url and checksum was published today.


Add IActionResults factory methods to Page and PageModel

We have support for returning IActionResult from a page or pagemodel, but we haven't yet ported over the API surface that users need.
This item tracks adding boilerplate for things we already have, not inventing new action results or factory methods which will be tracked separately.

I've done the gap analysis here between Controller/ControllerBase and Page/Page model:
TLDR here's what to do....
Add the following to Page
Challenge
Content
Forbid
File
LocalRedirect
NotFound
PhysicalFile
Redirect (missing some of the variations for preservemethod)
RedirectToAction
RedirectToRoute
SignIn
SignOut
StatusCode
Unauthorized
Add the following to PageModel
Challenge
Content
Forbid
File
LocalRedirect
NotFound
PhysicalFile
Redirect (missing some of the variations for preservemethod)
RedirectToAction
RedirectToRoute
SignIn
SignOut
StatusCode
Unauthorized

My notes...
All Methods on Controller/ControllerBase
Accepted
AcceptedAtAction
AcceptedAtRoute
BadRequest
Challenge
Content
Created
CreatedAtAction
CreatedAtRoute
File
Forbid
Json
LocalRedirect
NoContent
NotFound
Ok
PartialView
PhysicalFile
Redirect
RedirectToAction
RedirectToRoute
SignIn
SignOut
StatusCode
Unauthorized
View
ViewComponent
All of the Redirect kinds have 4 Permanent/PreserveMethod variations - I'm not going to repeat that detail everywhere 👍
Methods Pages Should Have
Challenge
Content
Forbid
File
LocalRedirect
NotFound
PhysicalFile
Redirect
RedirectToAction
RedirectToRoute
SignIn
SignOut
StatusCode
Unauthorized
Including the authorization-related stuff here because we will eventually add it if we don't today. The things that aren't added are the HTTP/API related helpers, legacy things like JSON and View-related stuff
Methods Pages Don't Need
Accepted
AcceptedAtAction
AcceptedAtRoute
BadRequest
Created
CreatedAtAction
CreatedAtRoute
Json
NoContent
Ok
PartialView
View
ViewComponent

@ryanbrandenburg adding to your plate.

Turning your lists into checklists for me to keep track with while getting waylaid by build buddy stuff.
Add the following to Page:

[ ] Challenge
[ ] Content
[ ] Forbid
[ ] File
[ ] LocalRedirect
[ ] NotFound
[ ] PhysicalFile
[ ] Redirect (missing some of the variations for preservemethod)
[ ] RedirectToAction
[ ] RedirectToRoute
[ ] SignIn
[ ] SignOut
[ ] StatusCode
[ ] Unauthorized

Add the following to PageModel:

[ ] Challenge
[ ] Content
[ ] Forbid
[ ] File
[ ] LocalRedirect
[ ] NotFound
[ ] PhysicalFile
[ ] Redirect (missing some of the variations for preservemethod)
[ ] RedirectToAction
[ ] RedirectToRoute
[ ] SignIn
[ ] SignOut
[ ] StatusCode
[ ] Unauthorized

Methods Pages Don't Need... View

View already exists on PageModel and Page, I assume you mean that we don't need another copy of it?

The View on Page and PageModel renders the page. It's not the same as View, do not remove it.


[sai] Update SAI for adding port extension support.

Updating SAI commit to current latest master: 7bffa33c7ad27960fe8c6baabf3605a2edd5f2d4.
This is for adding the port extension support, which includes both port attributes and port stats. The port extensions can be used for adding data path counters on port level, so we can have a higher layer of view.
See commit https://github.com/opencomputeproject/SAI/commit/7bffa33c7ad27960fe8c6baabf3605a2edd5f2d4 for more details.

thanks a lot Kamil and Chris!


Huobi USD spot - no USD wallet

Hello, how can I get the USD wallet from Huobi? By default I get CNY, BTC and LTC wallets as well as loan wallets.
I set
specs.setExchangeSpecificParametersItem("market", "usd");
but getWallet returns the same wallets as before.
Thank you!

Huobi does not have a spot USD market directly. You can access that by setting the exchange specific parameter use_bitvc to True, which then enables BitVC.

Consequently, you can't have USD and CNY currency pairs within the same exchange. If you need that you need to instantiate two Exchange instances with the relevant exchange specific parameters.

Thanks, I will try it.

I used
specs.setExchangeSpecificParametersItem("use_bitvc", true);
and provided bitvc api keys  but I still get CNY, BTC and LTC. Still missing something...

@Muffon Actually just realised that this is not supported/tested. BitVC is just the leveraged Huobi platform for CNY and only recently merged in BitYes which had a USD spot market. So getting USD is untested for now and will most likely not work.

Thanks, I am implementing USD spot in BitVC.

Great, let me know how you get on.

@Muffon Updates? Should we close this issue?


Add patch review interface to Achievement Editor

We have a patch acceptance interface in the Article Editor, like this:

The code for that starts here and here.
We need a similar thing for the Achievement Editor, which has no way yet of accepting all its pending i18n patches:

Hi I am new here, i would like to work on this bug, can someone guide me a bit through it

I'm a newbie trying to tackle this issue, making some progress but wondering if someone can push me in the right direction.
I have the patches view added:

However, it is not being populated with patch data correctly.
The articles patch view makes a database call like '/db/article/524c8cb26ee802556d00000e/?status=pending' to populate the patches list.
However, when the same call is made for achievements it looks like this '/db/achievement/undefined/?status=pending'
The call that returns undefined is '@achievement.get('original')' ('@article.get('original')' returns '524c8cb26ee802556d00000e')
This is where I am stuck, I cannot trace the calls further than this. I am finding no 'get' method in article.coffee or achievement.coffee.
Any tips would be appreciated! Also, let me know if this is the right place to be posting this kind of question.

I am also having the same problem, as described by rmarren1, i found the get method in CocoModel.coffee, it returns the attribute from the model (i think so). When i opened my database dump/achievements.metadata.json i was not able to find any key value pair for 'original', whereas there was one such pair in articles.metadata.json. Can this be a database issue ?

Looks like the PatchesCollection only knows how to fetch patches for versioned models (the ones that have the original property). Maybe something like this will work for using _id instead, for non-versioned documents where the original value just would be the same as _id (since the single Achievement document is the original):
    identifier = if forModel.schema().properties.original then 'original' else '_id'
    @url = "#{forModel.urlRoot}/#{forModel.get(identifier)}/patches?status=#{@status}"

there is no object at  http://localhost:3000/db/achievement//patches
instead we get the json at  http://localhost:3000/db/achievement//?status='status'
the json objects at article and that achievement are handled very differently, hence i suppose we need to write a different viewer to do so, can you guide me through it?

Nah, http://localhost:3000/db/achievement/signed-up?status=pending is the right format and the right URL; I just asked about /patches to see if there was anything there. If you have patches from the ?status=pending URL, they should show up in the viewer, no?

I have the menu working, and all get requests are going through

However, I am not actually seeing any patches. Is something wrong, or are there just no patches in the database? I don't want to make a pull request until I'm sure it works.

There are no patches in the local DB unless you make some. You can submit a PR and I can test with the live DB to see if they are showing up, or you can log into a non-admin account and try to submit some patches to the achievement to test it. (For example, if you go to http://localhost:3000/i18n/achievement/signed-up and submit an en-US change from a non-admin account, it will create a patch.)

@sderickson we now have the interface in there, but 1) the patches don't update the model, and 2) the patches don't save when accepted:

What do we normally do to make patches actually update the models, and what can we do to make them save on these non-versioned Achievements? (Also, the patches don't update the models on Articles, so it's probably the same problem.)

Don't know if this helps, but the after I make a patch just access the database, the patches show up.
e.g. http://localhost:3000/db/achievement/5499ec66a88ecca974d501e6 shows

But when I try to access the patches directly at:
http://localhost:3000/db/achievement/5499ec66a88ecca974d501e6/patches
There is just a [ ].


ci: remove irrelevant import

fix ci as there's an unused import

i think that the failing CI pull_request_target happens because the main in the original kaaper have bug in the CI due to previous merge


Please update ionic docs

The documentation says that the maximum allowed version for use with ionic is 5.4.30, but the @ionic-native/appsflyer plugin was updated to support the 6.x.x version.
The warning can be removed so as not to intimidate or mislead
Here is a merged PR for updating the plugin:
https://github.com/ionic-team/ionic-native/pull/3563

Thank you @rediska1114 . will do.


doesn't work properly with Strapi 3.6.x

When ChromePicker handles changes and setState value, it causes some lags because it rerenders all page. How to solve it?

@Hellomik2002 i know it's been awhile, but check out my PR - might put you on a right path, or whomever else that still needs this


feat: Hybrid app with stateful authentication

Adds a Spring-Boot Security layer to protect Flow and Hilla views.
Related-to https://github.com/vaadin/hilla/issues/2227
Fixes https://github.com/vaadin/platform/issues/5159

@Lodin could you please help to convert the explicit routing in this example into FS routing for authentication?

This example project has one potential anomaly that I don't fully understand yet.
When I navigate from public Hilla view to any protected Hilla view, the layout without page content is shown, expected behaviour is to redirect to /login as it happens for Flow view.
@platosha this public view probably should be placed outside of the main layout, as showing the protected views link publicly doesn't look like a good idea and not a real use case. What do you think? Does it make sense to have a separate public view, or even to remove it from example project ?

Well, having public view in the main layout is a valid use case. It depends on the business logic.
I propose to remove it from the example, but file an issue in Flow or Hilla.

UPD: the issue above is eliminated by hiding the protected views, if user is not authenticated.
I'm not even sure at the moment if this is a bug: navigating to a Flow view also doesn't redirect to login page by default.
The issue might be in that the Hilla doesn't show anything, but blank page. It could show access denied text for unification and 404 for production.
@platosha what do you think?

I'm not even sure at the moment if this is a bug: navigating to a Flow view also doesn't redirect to login page by default.

@mshabarov Cannot reproduce: it redirects me to /login if I'm not logged in. It does not redirect when I'm logged in and missing a role required for /admin though, I then see “Access denied” from Flow, which makes sense to me.

@mshabarov sorry, it does reproduce, but somehow depends on the previous navigation. Looking into...

It does not redirect when I'm logged in and missing a role required for /admin though, I then see “Access denied” from Flow, which makes sense to me.

Yeah, this is a default behaviour in Flow and is expected. I believe this is configured in Flow, so you can redirect to arbitrary view when you got AccessDeniedException.

The original issue I referred to with the blank content of Hilla views, when not authenticated, is not visible anymore with user info check:
{state.user ? (
        <NavLink className={navLinkClasses} to="/about">
               Hilla Authenticated
        </NavLink>
) : null}

I you remove it, and leave
       <NavLink className={navLinkClasses} to="/about">
               Hilla Authenticated
        </NavLink>

it would be visible.

Another anomaly that I noticed is that when I do a logout when staying on the Flow view, it doesn't redirect to a login view. It doesn't redirect actually onto "/" as it configured in SecurityConfig:
setLoginView(http, "/login", "/");

When on Hilla view, I'm redirected to login view, which is expected.

I bumped into another issue here, showing blank page view after steps:

Login
Load a genuine 404 by typing a URL, for example /404
Click a link to /flow
Expected: "Access denied", actual: blank view

I bumped into another issue here, showing blank page view after steps:

This is by the way reproducible even without security layer in current v24.4 branch.

Issues:

[ ] https://github.com/vaadin/flow/issues/18995
[ ] https://github.com/vaadin/flow/issues/18993

Could we merge this PR and address these issues separately?

@platosha I've modified the names of the views according to FS router conventions and added a page reload on logout. I hope now we can merge this PR and later remove routes.tsx custom content when we have protectRoutes included by default.


Hi, your status bar is very nice, what software and theme are you using?

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
OS (The operating system you're using):

e.g. Linux/Windows/Mac

**Neovim (Your neovim version)

e.g. 5.0

Additional context
Add any other context about the problem here.

Sup!
The bar is Polybar, you can check it on my dotfiles :+1:


Better readme for UI Automator

After working a few weeks with UI Automator I added some insights to the Readme which would make other people's life much easier.
Now it's my chance to share it with others!

I signed it!

I signed it!

Hey Marcos, thanks for this pull request. We already have an internal version with better phrasing which is similar to yours. This should be released with the next sample push. To keep track of the issue, can you please open up an issue for this here. I will close this pull request and update the issue. Thanks again for your time and contribution to testing samples.


Set CLOUDSDK_PYTHON and clear PYTHONHOME and PYTHONPATH for managed Cloud SDK on Windows

From https://github.com/GoogleCloudPlatform/google-cloud-eclipse/issues/2949#issuecomment-373540253
We need to ensure our auto-install and update does not fail under any circumstances.
I'll also file a bug upstream. I think it is a bug that gcloud is affected when CLOUDSDK_PYTHON is defined.

Internal bug: 75262593
At the moment, we should clear the variables on our side.

Well, actually, this affects not only install/update but just every gcloud invocation. Knowing that, it's probably WAI, especially on Linux where gcloud relies on the pre-installed Python.
However, for our managed SDK on Windows, because we always download the Python-bundled SDK, we should always set CLOUDSDK_PYTHON and clear PYTHONHOME for every gcloud usage, not just for maanged SDK install/update.

i am facing above issue, below is the environment variable that i have saved,
C:\Users\muham\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Python 3.8
can anyone guide on how to install Google cloud sdk?

Facing the same issues here, set up the wrong CLOUDSDK_PYTHON path in my .zshrc. Now basically the entire environment is paralysed with anything related with gcloud commands.

@TiansuYu , @MrFarhan it appears your issues are not related to this repository. This is a project for java appengine users. See the description here. If you would like help with the cloud sdk, please see https://cloud.google.com/sdk/docs/getting-support

I believe this no longer an issue for us (appengine-plugins-core and the build/IDE plugins). Also looks like b/75262593 is resolved.


Separate attention backends

This PR refactors the attention layer. Specifically, it separates the code paths for Ampere or more recent NVIDIA GPUs (which can directly use FlashAttention) and other GPUs, so that the code for the former becomes much simpler. This PR will also bring some performance improvements for ALiBi models, since we now directly call FlashAttention instead of using xformers in the middle.

@zhuohan123 What do you think about this design? Please note that while I used flash_attn for now, but this will be replaced with FlashInfer.

@zhuohan123 PTAL. Please note that I intentionally didn't make changes to other models than Llama.

Very nice! This is mainly a cosmetic suggestion, but have you considered structuring this as a Python submodule? Like have a folder attention in the vllm/model_executor/layers folder with the following files:

__init__.py:

from vllm.model_executor.layers.attention.attention import Attention

__all__ = [
    "Attention",
]

(this is the public interface)

attention.py: Your attention.py file goes here
backend/paged_attn.py: Your paged_attn.py file goes here
backend/flash_attn.py: Your flash_attn.py file goes here
backend/xformers.py: Your xformers.py file goes here

This makes it a little clearer what the interface here is and also follows the structure of the quantization and fused_moe folder :)
It is interesting that the paged_attn.py file is more of a common implementation used by everybody else, I wonder how you think that will evolve in the future :)

This is an awesome change! When is this planned to be merged to OSS btw? soon?

Also, separating out profiling run could further simplify the logic
def forward():
    profiling_run = key_cache is None or value_cache is None

    if profiling_run():
        return XformerBackendImpl.prefill()

    assert key_cache is not None and value_cache is not None
    ...

@pcmoritz Thanks for the suggestion! I've changed the directory structure following your advice.

@Yard1 @rkooo567 Thanks for your inputs! Actually, I intentionally avoided the design you proposed to ensure the flexibility in implementing the attention backends. As you pointed out, an attention backend performs 4 tasks: 1) storing the input KV tensors into the KV cache, 2) compute prefills, 3) compute prefills with prefixes, and 4) compute decodes. Currently, the two attention backends (FlashAttentionBackend and XFormersBackend) have a kernel for each task. However, this may not be necessary true in the future. For example, depending on the kernel implementation, one can compute prefills with and without prefixes (2&3) at the same time. For anther example, an attention kernel in TRT-LLM stores KV cache while computing decodes (1&4). These can be even more complicated if we implement something like Cascade inference. Hence, I believe we shouldn't fix a certain structure for the attention backends.

A reasonable explanation！

Hmm actually I wonder if this is the good level of abstraction. For example, if I'd like to add chunked prefill, I need to reimplement the same thing in both xformer and flash attention backend.

@rkooo567 Actually, this refactoring is indeed for your case. To my knowledge, xformers currently does not support Paged KV cache since it is a pretty recent feature in FlashAttention. If the backends are not separated, we need to add more conditionals inside the unified Attention layer.

It turns out that adding flash_attn to vLLM's dependencies is pretty tricky. Related issue: Dao-AILab/flash-attention#453

The most convenient (and fast) to install flash-attn might be downloading the wheels from flash-attn's release page, rather than pip3 install flash-attn as it builds from source, quite time-consuming and error-prone, and, it will prevent vllm being installed on machines without GPU>=A100 (where flash-attn cannot be built).
Checking its existence and its version at runtime in vLLM would be better.

It turns out that adding flash_attn to vLLM's dependencies is pretty tricky. Related issue: https://github.com/Dao-AILab/flash-attention/issues/453

@WoosukKwon I have also been experimenting a bit with calling flash-attn directly (instead of going xformers), but instead of adding it as an explicit dependency, I found that it was already installed as part of xformers can could be imported like this:
import xformers._C_flashattention as flash

In our fork I have been experimenting with calling flash.varlen_fwd for prefills, which allows one to remove all padding from the batch. This has some performance advantages in certain scenarios and removes the need to tune --max-paddings. Given this new refactoring, could this also be something worth supporting in your view? Is there a reason that vLLM needs to use rectangular padding other than supporting older GPUs where flash isn't supported?

@tdoublep actually varlen will be more useful after we merge https://github.com/vllm-project/vllm/pull/3236.


License?

This project doesn't seem to have a license. Can you please add a license so that other people are legally allowed to use it?

Thanks for catching that. Added MIT license. Best.


Derivation platform checking compares against wrong machine

Issue description
The logic in pkgs/stdenv/generic/check-meta.nix for checking the meta.platforms attribute currently compares meta.platforms against the stdenv.system attribute. In my cross-compilation environment stdenv.system is the double of my build platform, x86_64-linux.
It seems to me like this should rather be compared against stdenv.hostPlatform.system. Unfortunately, this attribute has slightly different semantics than stdenv.system. For instance, in the case of armv7l-hf-multiplatform it is arm-linux whereas the value expected by uboot is something like armv7l-linux.
How much detail should meta.system capture?
Pinging @Ericson2314.

It's a Nix rule that the system field of each derivation is the build platform, as a Nix-style 2-component string. No changing that just yet.

stdenv.system can't go away so long is stdenv is a derivation, but I'd like stdenv to be a plain attribute set. [See https://github.com/NixOS/nixpkgs/pull/30549#issuecomment-342247583 for other reasons why.]

https://github.com/NixOS/nixpkgs/blob/master/pkgs/stdenv/generic/make-derivation.nix#L171-L173 we can at least make it a bit saner on cross my doing inherit (stdenv.hostPlatform) system; there instead. This will break some cross packages, so I'd like to do after we merge your https://github.com/NixOS/nixpkgs/pull/30882 and all the cross tests are fixed (if you fixed cross-built gcc then they will be, yay!).

arm-linux vs rmv7l-linux has to do with me changing everything https://github.com/nixos/nixpkgs/tree/master/lib/systems/ code. armv7l-linux is nicer than arm-linux, but if Nix expects that latter for all 32-bit arm linux native builds (Nix itself won't care for cross), we have a problem and that cross build will have to be adjusted accordingly.

I don't like how meta.platforms works by exhaustive enumeration, when platforms can be tweaked in such fine-grained ways. I suppose the huge space of cross platforms quotiented to just the system field isn't so big, but I still rather do arbitrary predicates or something, as if the space of platforms really was infinite so that was the only way.


Adds download button display logic. Simple event handler.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.276%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
538

Relevant Lines:
878

💛  - Coveralls


Test Structure

Make sure database structure makes sense.

Done: https://github.com/PHCNetworks/phc-scriptcdn/commit/33ab26344892377c1f219218845e8d49a32e9426


Make dependency on maven plugin optional.

The dependency on the maven plugin is pretty small and if you aren't building Maven (or even Java) projects on a Jenkins instance, this helps to remove clutter in the configuration screen.

@evandy0 @olivergondza @jglick Not really sure who own this, so mentioning you as the most recent contributors. Can this PR be merged?

Should I rebase to resolve the conflict?


To provide Application/environment isolation via weave subnets.

As we are using weave proxy and not overriding the docker bridge now we should be able to provide a solution with granular subnet isolation per application/environment.
http://docs.weave.works/weave/latest_release/features.html#application-isolation

Do we have a suggested approach on this or we need some investigation here ?


Lexer is too eager to treat - as unary negation

fn main() {
    log 3-2;
}

b.rs:2:9: 2:11 error: expected ';' or '}' after expression but found -2

Oops, didn't notice that f26e770 included a test.

marijn: do you want to leave this open until it's solved better (as indicated in your comments)?

Yeah, my hack is not a proper solution (doesn't work for, for example x-0xFFFFFFFFu, unlikely as that is to occur). Someone will implement constant folding for our constants. Once we have that, negative literals can be removed from the lexer again, and the code that parses literal patterns can just have a special case for unary -.

When I suggested to pcwalton that we do constant folding instead of parsing negative numbers his counter example was that that wouldn't allow for negative numbers in patterns (I think).

and the code that parses literal patterns can just have a special case for unary -.

.. and then call the constant folder on the result. It's less involved than the current hack.

Constant folding is in


Browser subscribed me to pewdiepie while executing nocode

Hi
While struggling with executing nocode I found myself subscribed to pewdiepie, rather than the T-Series youtube channel. Please help! What can I do ? I have compiled nocode for days and nights while suddenly experiencing fits of laughter. My family think I am going insane! LOL. Oops.

Did you per chance build a neural network? It only takes a couple of milliseconds of training for properly constructed neural networks to realize that PewDiePie is superior in every way to T-series. Since nocode requires either administrator privileges (windows) or to be run as root (Linux/OSX), It opened up your browser and subscribed to PewDiePie for you.


Update benchmarks and test them with PkgJogger

Closes #168.

Benchmark failure on master is expected, merging.


How to remap gc[count]{motion} and gb[count]{motion}

Hi, I'm new to neovim and inverted my motions, for me K is down and J is up, so I wanted to remap gc[count]{motion} and gb[count]{motion} to respect that. How can I do it?
Thanks!

Please use Discussions to ask your question.


Latest updates from master of opensim-models repo

Fixes issue #0
Brief summary of changes
Bring fixes from opensim-models repo to opensim-gui. In particular leg39 model, update StrengthOfSimulaion tutorial files for motionType
Testing I've completed
CHANGELOG.md (choose one)

no need to update because internal
updated...

Cool. All these changes have been reviewed/merged in the models repo? Not sure if I need to review anything here.

I think we can merge this once the tests pass.

Tests passed, this can be merged or we can test the artifact from the PR build to make sure the files are properly installed so we can update before merging.

Tested locally on Mac and all looks good. Merging.


Documentation section numbering is incorrect

Throughout the documentation the numbering in the table of contents does not match the section numbering of the rest of the document. It looks like the subsection numbering resets to 1 for every new section in the ToC (correctly), but continues to count up across sections in the body of the document (incorrectly).
e.g. https://google.github.io/filament/Materials.html#overview/coreconcepts is the first subsection of section 2 (so appears as 2.1 in the ToC), but is labelled as 2.2.
It's possible this is a browser bug, as I don't see anything wrong with the CSS. I'm using Firefox 85.0.2 (64-bit) on Windows 10.

It's properly labeled 2.1 in Chrome and Safari.


tf.contrib.distributions: undocumented behaviour in Multinomial and Categorical

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes?
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS Sierra 10.12.3
TensorFlow installed from (source or binary): binary, via pip
TensorFlow version (use command below): 1.1.0 (v1.1.0-rc0-61-g1ec6ed5)
Bazel version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce:

import tensorflow as tf
categorical = tf.contrib.distributions.Categorical(probs = [0.25, 0.5, 0.25])
multinomial = tf.contrib.distributions.Multinomial(total_count = 1., probs = [0.25, 0.5, 0.25])
mvn = tf.contrib.distributions.MultivariateNormalDiag(loc = [0., 0., 0.], scale_diag= [1., 1., 1.])

# expected values (points 1 and 2)

# The docs for Categorical say value should be float or double, but it expects an int
categorical.log_prob([0, 0, 1])
# <tf.Tensor 'Categorical_2/log_prob/Neg:0' shape=(3,) dtype=float32>
categorical.log_prob([0., 0., 1.])
# TypeError: Value passed to parameter 'labels' has DataType float32 not in list of allowed values: int32, int64

# The docs for Categorical say value should be float or double, which is how it behaves (though this is unlike categorical)
multinomial.log_prob([0, 0, 1])
# ValueError: Tensor conversion requested dtype int32 for Tensor with dtype float32: 'Tensor("Multinomial_2/log_prob/Log:0", shape=(3,), dtype=float32)'
multinomial.log_prob([0., 0., 1.])
# <tf.Tensor 'Multinomial_1/log_prob/sub:0' shape=() dtype=float32>

# output shape (points 3 and 4)

# The docs for both say that the output should be:
# "a Tensor of 'shape sample_shape(x) + self.batch_shape' with values of type self.dtype"
# though sample_shape doesn't seem to be relevant here, it's an argument to param_shapes() and sample()

# for Categorical (with int value), the result is a vector, matching the shape of value
categorical.log_prob([0, 0, 1])
# <tf.Tensor 'Categorical_2/log_prob/Neg:0' shape=(3,) dtype=float32>

# for Multinomial (with float value), the result is a scalar
multinomial.log_prob([0., 0., 1.])
# <tf.Tensor 'Multinomial_1/log_prob/sub:0' shape=() dtype=float32>

# for Multivariate Normal the result is a scalar
mvn.log_prob([0.1, 0.2, 0.3])
# <tf.Tensor 'MultivariateNormalDiag_2/log_prob/add:0' shape=() dtype=float32>

Describe the problem
There are four related issues:

The expected type of value for the log_prob() method in tf.contrib.distributions.Categorical is inconsistent with the documentation.

The expected values for tf.contrib.distributions.Categorical and tf.contrib.distributions.Multinomial are inconsistent with one another, which is odd as the categorical distribution is a special case of the multinomial, with total_count  = 1

The output dimensions for tf.contrib.distributions.Categorical and tf.contrib.distributions.Multinomial are inconsistent with the documentation

The output dimensions for tf.contrib.distributions.Categorical are a vector, which doesn't really make sense for a multivariate distribution, and is inconsistent with tf.contrib.distributions.Multinomial and tf.contrib.distributions.MultivariateNormal*

Details are in the code snippet above

Passing documentation issue off to contrib/distributions czar @langmore.

Punting this issue to @jvdillon. PTAL.

Thanks for reporting this issue! I'll put together a fix right now and report back.

Sorry for the delay! Small update: we implemented a fix and it's currently under review.

Fix submitted. Should be in master in a few days.


Support email addresses with +

Hi there,
I just tried the sample application you have (thanks for that) and I signed up using an email address that contains a plus. The format being<EMAIL_ADDRESS>I recieved the verification email, clicked the link, but the verification failed. I am guessing it is the plus character because it was not URL encoded in the verification link.

The error message also did not contain the plus character
User me<EMAIL_ADDRESS>has not been Verified, sorry.

I think you're right, I just put a quick fix in the master branch, using encodeURIComponent for the email in the verification link. Do you have time to check if that works through the verify.html page?

Testing here: http://lambdauth.danilop.net/
Still fails

Yes - I didn't redeploy there yet - I'll let you know, sorry for the
misunderstanding.
On Sat, Aug 29, 2015 at 12:46 AM, Peter Sankauskas<EMAIL_ADDRESS>
wrote:

Testing here: http://lambdauth.danilop.net/
Still fails
—
Reply to this email directly or view it on GitHub
https://github.com/danilop/LambdAuth/issues/12#issuecomment-135905222.


[bug] sfpegActionBarCmp - Default value when navigating to standard creation page not supporting special characters

When initialising a new record Name from fields beginning with some special characters such as '/' and '+', these characters are skipped in the resulting value.

Fixed in commit 8c9a1fa
Added a new UTF() escape logic in the sfpegMergeUtl component.


[SPARK-28325][SQL] Support ANSI SQL: SIMILAR TO ... ESCAPE syntax

What changes were proposed in this pull request?
[ NOT ] SIMILAR TO <similar pattern> [ ESCAPE <escape character> ]
This is a ANSI SQL.
For example:
SELECT 'abc' SIMILAR TO 'abc' AS `true`;
SELECT 'abc' SIMILAR TO 'a' AS `false`;
SELECT 'abc' SIMILAR TO '%(b|d)%' AS `true`;
SELECT 'abc' SIMILAR TO '(b|c)%' AS `false`;

There are some mainstream database support the syntax.
PostgreSQL:
https://www.postgresql.org/docs/current/functions-matching.html#FUNCTIONS-SIMILARTO-REGEXP
Redshift:
https://docs.aws.amazon.com/redshift/latest/dg/pattern-matching-conditions-similar-to.html
Sybase
http://infocenter.sybase.com/help/index.jsp?topic=/com.sybase.help.sqlanywhere.12.0.0/dbreference/like-regexp-similarto.html
Firebird
http://firebirdsql.org/file/documentation/html/en/refdocs/fblangref25/firebird-25-language-reference.html#fblangref25-commons-predsiimilarto
SIMILAR TO is similar to RLIKE, but with the following differences:
1. The SIMILAR TO operator returns true only if its pattern matches the entire string,
unlike RLIKE behavior, where the pattern can match any portion of the string.
2. The regex string allow use _ and % as wildcard characters denoting any single character
and any string, respectively (these are comparable to . and .* in POSIX regular
expressions).
3. The regex string allow use escape character like LIKE behavior.
4. '.', '^' and '$' is not a meta character for SIMILAR TO.
Why are the changes needed?
Support ANSI SQL: SIMILAR TO ... ESCAPE syntax
SIMILAR TO ... ESCAPE  has a lot of convenience compared to RLIKE.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Exists and new UT

Test build #117729 has started for PR 27432 at commit 865bcc6.

Merged build finished. Test PASSed.

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

Test build #117729 has finished for PR 27432 at commit 865bcc6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class EscapeRegexExpression(str: Expression, pattern: Expression, escape: Expression)
case class Like(str: Expression, pattern: Expression, escape: Expression)
case class SimilarTo(str: Expression, pattern: Expression, escape: Expression)

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

Test build #117735 has started for PR 27432 at commit b15e5c7.

Test build #117735 has finished for PR 27432 at commit b15e5c7.

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

Merged build finished. Test PASSed.

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

Test build #119439 has started for PR 27432 at commit ce13f8e.

Merged build finished. Test PASSed.

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

Test build #119439 has finished for PR 27432 at commit ce13f8e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #119456 has started for PR 27432 at commit dca86e5.

Merged build finished. Test PASSed.

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

Test build #119456 has finished for PR 27432 at commit dca86e5.

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

Merged build finished. Test PASSed.

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

Test build #119470 has started for PR 27432 at commit 5cf6f9a.

Merged build finished. Test PASSed.

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

Test build #119470 has finished for PR 27432 at commit 5cf6f9a.

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

Merged build finished. Test PASSed.

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

Test build #123717 has started for PR 27432 at commit 4329287.

Merged build finished. Test PASSed.

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

Test build #123717 has finished for PR 27432 at commit 4329287.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please.

Test build #123755 has started for PR 27432 at commit 4329287.

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #126824 has started for PR 27432 at commit e4805a8.

Test build #126824 has finished for PR 27432 at commit e4805a8.

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

Merged build finished. Test PASSed.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34220/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34220/

Merged build finished. Test PASSed.

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

ping @cloud-fan

can you fix the conflicts? thanks!

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46542/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46542/

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


ADMX configuration does not work through Microsoft Enpoint

Description of the new feature / enhancement
ADMX Ingestion [1] of the provided ADMX file fails due to constrains of Endpoint ADMX ingestion [2]
Registry keys within Software\Policies\Microsoft\Windows\AppInstaller are not writeable. Therefore, the ADMX ingestion does not work.
As we're going AADJ only and the MS Store does not really work with business accounts and is still enabled by default, we need to implement a cumbersome workaround.
[1] https://docs.microsoft.com/en-us/archive/blogs/ukplatforms/google-chrome-gpo-via-intune
[2] https://docs.microsoft.com/en-us/windows/client-management/mdm/win32-and-centennial-app-policy-configuration
Proposed technical implementation details
Direct Endpoint integration would be the best option i guess.
Add a new exception of a writeable registry key location to ADMX ingestion would be the second best option, but still requires the involvement of Microsoft outside the winget team.
Watching a second registry key location would be another (quite unclean) option.
 Configuration Source ID: (2FD40D97-DEA2-463C-918D-EE910BADD8E3), Enrollment Name: (MDMDeviceWithAAD), Provider Name: (Policy), Command Type: (Add: from Replace or Add), CSP URI: (./Device/Vendor/MSFT/Policy/ConfigOperations/ADMXInstall/Microsoft/Policy/DesktopAppInstaller), Result: (Access is denied.).
 Result:(0x80070005) Access is denied..

Event Xml:
<Event xmlns="http://schemas.microsoft.com/win/2004/08/events/event">
  <System>
    <Provider Name="Microsoft-Windows-DeviceManagement-Enterprise-Diagnostics-Provider" Guid="{3da494e4-0fe2-415c-b895-fb5265c5c83b}" />
    <EventID>850</EventID>
    <Version>0</Version>
    <Level>2</Level>
    <Task>0</Task>
    <Opcode>0</Opcode>
    <Keywords>0x8000000000000000</Keywords>
    <TimeCreated SystemTime="2022-02-01T22:01:33.2198261Z" />
    <EventRecordID>4508</EventRecordID>
    <Correlation />
    <Execution ProcessID="15872" ThreadID="11512" />
    <Channel>Microsoft-Windows-DeviceManagement-Enterprise-Diagnostics-Provider/Admin</Channel>
    <Computer>CLFPF21B7DS</Computer>
    <Security UserID="S-1-5-18" />
  </System>
  <EventData>
    <Data Name="Message1">Software\Policies\Microsoft\Windows\AppInstaller</Data>
    <Data Name="Message2">policy</Data>
  </EventData>
</Event>

the only current way seems to be OMA-URI:
https://learn.microsoft.com/en-us/windows/client-management/mdm/policy-csp-desktopappinstaller

Good news. Winget ADMX backed policies made into Intune settings catalog.
Can confirm that using OMA-URI or manual upload of the winget ADMX might fail. this issue is also covered here:
https://andrewstaylor.com/2022/10/26/managing-winget-using-intune-and-admx-import/


ImageCollector Controller - Scanner part

Describe the solution you'd like
[A clear and concise description of what you want to happen.]
Once shared image collector CR is created, scanner should begin and update the Status field of the CR with images that were found vulnerable.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Eraser version:
Kubernetes version: (use kubectl version):

This was completed by #173


Horrible performance on real Android device

Hi,
I've run the app in Android & iOS emulators & real devices. Everything is fine except for the Android phone. The app is slow, sometimes unresponsive in every screen. The sample apps in Play Store run well (not as smoothly as in iOS though).
Here's the device info:

Model: Samsung Galaxy J Docomo SC-02F
RAM: 3GB
Chipset: Qualcomm MSM8974 Snapdragon 800
OS version: 4.4.2

I can't get any other Android device so I'm not sure if this problem is caused by the device or the code.
I wonder if there's any special configuration and/or command required to build the app? (I simply run tns run android)

Hi @fleuverouge,
Could you provide more info about the examples that are not responsive or you have found problem. This is a known issue with slow navigation in Android, while usuing tns-core-modules 2.3.0 . However we upgrade the app to use @next modules, where this issue has been fixed.
Regards,
@tsonevn


Fix warning in cordebuginfo.h

On Alpine, 9 warnings were produced in the end when we build ObjWriter:
/llvm/tools/ObjWriter/cordebuginfo.h:209:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:219:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:228:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:237:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:250:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:264:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:273:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:284:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
/llvm/tools/ObjWriter/cordebuginfo.h:291:13: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
            struct
            ^
9 warnings generated.
[100%] Linking CXX shared library ../../lib/libobjwriter.so
[100%] Built target objwriter

This delta fixes those warnings.

Could you please submit the same change to https://github.com/dotnet/coreclr/blob/master/src/inc/cordebuginfo.h as well?

Thanks!


Oracle 4 Core region with 24GB ram

Are there any regions where it is easier to get 4 Core instances with 24GB of ram?

You can’t change Home Region by yourself after registration, but in general it’s true. I would say usually it’s not a problem for US-based or Frankfurt (Europe)


Method to set seed to random generator

Adding the method set_seed() to the BN::random class
This allows setting the current position of the internal _x value which allows to use of the deterministic random generator in a somehow less deterministic way.
It asserts to a nonzero value to ensure not breaking the shifter.

How can you be sure that changing the seed will not reduce the quality of the output values?
A similar pull request was discussed some weeks ago: https://github.com/GValiente/butano/pull/42

Given how the algorithm, the equivalent of using a seed for my part is the following code, at the start of the game:
bn::random rand = bn::random();
int seed = 14;
for(int i = 0 ; i < seed ; i++){
   rand.update();
}

That's way slower but yeah, it works.

Yes, I was using that approach, but my goal is to use the random generator for a procedural algorithm and the loop can block the gba for quite a while.
Would you be more open to a get_state/set_state + struct approach in which the internal three vars are exposed?

The advantage of the get/set approach is not to change it's values but to store them in the save file, so your generator can continue from it's state after loading the game.

As far as I understand from the Wikipedia article setting the seed like in this approach can change the quality of the generation (that's a risk the user can consider if explained) and as long as the internal values are not all zeroes it will produce a randomized chain of some sort varying it's potential length before repetition.

Check the new commit and let me know if you like the approach

The advantage of the get/set approach is not to change it's values but to store them in the save file, so your generator can continue from it's state after loading the game.

Since bn::random is a plain struct, you can save and load it with bn::sram::read_span and bn::sram::write_span.

Check the new commit and let me know if you like the approach

No, same issues are still present.
As I said in the other pull request, if what you're doing works for you, you always can use your own random generator instead of using bn::random.

How do you alter the internal values if you want to alter the state without using the sram approach?

If you alter the internal values, you can't be sure that you aren't reducing the quality of the output values, so you probably shouldn't.

But you are giving no option to alter the starting point of the chain of randomization.
What I would like to do is to run the update() several number of times (a seed) on a device, obtain it's values and then use them as starting point on my code. that allows to have more than one randomizer in the same runtime producing the same chain on different offsets without having to run the loop when starting and avoidf to slow the program.
If that's not a feature you would like on your framework, feel free to reject the PR.

Would you be ok to at least keep the get_state() method to help on debugging?

Again, if what you're doing works for you, you can always use your own random number generator instead of using bn::random.

After https://github.com/GValiente/butano/commit/e87331fd5d44e3f46b7d9709af9dabf0075d3747 commit, bn::random internal state is protected instead of private, so you can subclass bn::random and do what you want with its internal state :)

thanks


Add update_user_password method to UserManagement module

Description

Adds update_user_password to method to module and returns a user object back up the updated user.

Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

Codecov Report

Merging #220 (d183ff5) into main (2842942) will increase coverage by 0.01%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #220      +/-   ##
==========================================
+ Coverage   97.08%   97.09%   +0.01%
==========================================
  Files          52       52
  Lines        1166     1171       +5
==========================================
+ Hits         1132     1137       +5
  Misses         34       34

Files Changed
Coverage Δ

lib/workos/user_management.rb
100.00% <100.00%> (ø)

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


How to use TypeScript and MyModel.bindKnex(Knex);

Given:

class User exists and extends Model in my code.
I have two database connections defined as separate Knex objects:
A. a mysql connection
B. a SQL Server connection.
(So using Model.knex(k) is not an option)
I'm trying to use TypeScript to bind those connections to each model using

const m:User = User.bindKnex(knex).
When I try to run this code, I get:
[tsserver 2740] [E] Type 'typeof User' is missing the following properties from type 'User': UserId, Email, Password, $modelClass, and 35 more.
The following does not produce the error, but I loose intellisense on const m.
const m:any = User.bindKnex(knex);
I've tried doing:
const m:any = User.bindKnex(knex);
const n:User = m;

This resolves the error on that line, but then when I try to use n.query()... I get:
[tsserver 2576] [E] Property 'query' does not exist on type 'User'. Did you mean to access the static member 'User.query' instead?
Obviously I must be doing something wrong. I can't find any examples online for how to do this properly in TypeScript. How can I get this to work?
I'm using TypeScript 4.3.2.

Closing this in the absence of more information.


enhance node generator

introduce a config file to customize:

which prop msg.payload is mapped to
specify an icon for the node
add additional help text

Closing in favor of a complete redo.


Database diagram

Hi there, 👋
In this branch, My coding partner @Mekuseo and I made the following changes ⬇

We added the necessary table to store different data based on the diagram
We added necessary indexes to improve our database search query execution time.
Note: We added all the tables and indexes based on the above database diagram

Hi @ichala, Thanks a lot for reviewing our project 👍


[goalc] improve reliability of debugger tests

the runtime will only flush its output buffer on the first kernel run after the first incoming message from the compiler (ListenerStatus).  I didn't realize this and assumed we'd get the reset message immediately on connection.  This PR waits for the response before continuing with the test.  I think this will fix the recent debugger test failures we've been seeing.

looks like that's not the only issue:
[2021-10-15 22:57:01:237] [debug] [DECI2] Waiting for listener...
[2021-10-15 22:57:01:288] [debug] [DECI2] Connected!
[Listener] Failed to connect
[DECI2] Got message: 24 0 0xe042 H -> E
[Listener] Socket connected established! (took 0 tries). Waiting for version...
Got version 0.8 OK!
[2021-10-15 22:57:01:374] [info] connected: true

[2021-10-15 22:57:01:374] [info] poking...

[Debugger] Context: valid = true, s7 = 0x147d24, base = 0x7efca45a2000, tid = 5810

[2021-10-15 22:57:01:375] [info] break...

goalc-test: /home/runner/work/jak-project/jak-project/goalc/debugger/Debugger.cpp:439: void Debugger::read_symbol_table(): Assertion `is_valid() && is_attached() && is_halted()' failed.

First issue: failing an assert could cause the test to exit without flushing stdout, so I missed print statements.  I don't really know how this is possible because I thought stdout would be flushed with every newline.
Second issue: The "watcher" thread runs in the background and checks to see if the process segfaults, disappears, etc, then prints a message to user if it has.  I was starting this too early, and it was racing ptrace(PTRACE_SEIZE.  If the watcher got to the waitpid before the ptrace finished, it would look like the process died and the debugger state would be reset.  Then, the debugger initialization would go to read the symbol table and the context would be invalid.


Outside hybridMode, hover might show result of _VLS_ctx instead of particular property (off-by-one offset bug)

Vue - Official extension or vue-tsc version
v2.0.26
VSCode version
1.91.0
Vue version
^3.4.18
TypeScript version
~5.0.4
System Info
No response
Steps to reproduce

hybridMode: false
Consider example file below
Hover over the 1st character of a JS reference inside template, l is the easiest to try.

<template>

{{local}}

</template>

What is expected?

What is actually happening?

Link to minimal reproduction
No response
Any additional comments?
This is affecting WebStorm sadly, as we're not using hybridMode. I've checked and we sent the same offsets as VSCode does, but in hybridMode, Vue LS returns
[Trace - 16:57:08] Sending request 'textDocument/hover - (20)'.
Params: {
    "textDocument": {
        "uri": "file:///Users/tomblachut/Work/Playgrounds/vue3-vite-ts5/src/components/injections/NullChecksTeamCity.vue"
    },
    "position": {
        "line": 11,
        "character": 15
    }
}

[Trace - 16:57:08] Received response 'textDocument/hover - (20)' in 8ms.
No result returned.

Not sure if this is an upstream issue


Tomcat creates backup up traffic_router.service

When uninstalling/upgrading tomcat, the traffic_router.service file is preserved under traffic_router.rpmsave. This creates a problem when installing a new version of traffic router with updated .service. file. The old settings are kept.
# ls -l /usr/lib/systemd/system | grep traffic_router
-rwxr-xr-x 1 root root 1225 Nov 12 13:14 traffic_router.service
-rwxr-xr-x 1 root root 1224 Nov 12 12:54 traffic_router.service.rpmsave

On yum remove, the following occurs:
Transaction test succeeded
Running transaction
  Erasing    : traffic_router-3.0.0-9579.21888fd1.el7.x86_64                                                         1/1
warning: /opt/traffic_router/conf/traffic_ops.properties saved as /opt/traffic_router/conf/traffic_ops.properties.rpmsave
warning: /opt/traffic_router/conf/traffic_monitor.properties saved as /opt/traffic_router/conf/traffic_monitor.properties.rpmsave
warning: /opt/traffic_router/conf/log4j.properties saved as /opt/traffic_router/conf/log4j.properties.rpmsave
warning: /opt/traffic_router/conf/http.properties saved as /opt/traffic_router/conf/http.properties.rpmsave
warning: /opt/traffic_router/conf/dns.properties saved as /opt/traffic_router/conf/dns.properties.rpmsave
warning: /lib/systemd/system/traffic_router.service saved as /lib/systemd/system/traffic_router.service.rpmsave
  Verifying  : traffic_router-3.0.0-9579.21888fd1.el7.x86_64

Also it leaves behind this link:
# ls -l /etc/systemd/system/multi-user.target.wants/ | grep traff
lrwxrwxrwx 1 root root 46 Nov 12 13:29 traffic_router.service -> /usr/lib/systemd/system/traffic_router.service


Update lazy-observables.md

duplicated https://github.com/mobxjs/zh.mobx.js.org/pull/69, but also thanks for your pull request, add you to our repo README.md via https://github.com/mobxjs/zh.mobx.js.org/pull/87


[WEB-968] [WEB-968] [turbopack] next/font not working in a dockerized environment

Verify canary release

[X] I verified that the issue exists in the latest Next.js canary release

Provide environment information
next/font utility is causing issues in docker environments:
warning -
  [fetch]
    [next]/internal/font/google/plus_jakarta_sans_3fefeabb.module.css
      Error while requesting resource
      There was an issue establishing a connection while requesting<EMAIL_ADDRESS>
and in reality this is been stored in .next folder under .next/server/pages/fallback_not_found folder
Which area(s) of Next.js are affected? (leave empty if unsure)
App directory (appDir: true), Turbopack (--turbo)
Link to the code that reproduces this issue
no response
To Reproduce
no response
Describe the Bug
no response
Expected Behavior
no response
Which browser are you using? (if relevant)
No response
How are you deploying your application? (if relevant)
No response
From SyncLinear.com | WEB-968

linear sync duplicated


Phase 2 prompts and responses

RESPONSES (always the same):
TELL TRUTH
WAFFLE
CHANGE TOPIC

QUESTIONS (chosen at random):
Last night location?,Followed lockdown?,Social distancing?,No parties?,Telling the truth?,Working from home?,Quiz happened?,No birthday celebration?,Same rules as us?,Out drinking?

Didn't we want something about Peppa Pig here?

Last night location?,Followed lockdown?,Social distancing?,No parties?,Telling the truth?,Working from home?,Quiz happened?,No birthday celebration?,Same rules as us?,Out drinking?,Been to Peppa Pig world?


omnisharp possibly not detecting some errors and warnings

Hi.
I've been using omnisharp with Helix (https://github.com/helix-editor/helix) and for the most part it works great. However there are some errors and warnings that seem to be ignored by it. For instance, when I declare string name = 14;, I don't receive the "Cannot implicitly convert type 'int' to 'string'" error. Similarly, when using string name = Console.ReadLine();, there's no warning about "Converting null literal or possible null value to non-nullable type."
I'm unsure if this issue is related to Omnisharp or Helix, so I apologize if this is misplaced.

The issue turned out to be that when you open a file using the file picker in Helix, it won't change directory to the location of that file. The solution was to cd to the specific folder instead of opening the file from the home directory through Helix. The weird thing is that omnisharp works if you do the wrong way, it just won't show some types of errors and warnings.


"hardware info parse to json failed, error: %+vinvalid character '\x01' looking for beginning of value"

Computing-Provider version:0.4.7
Node type:ECP
Description: When the ECP node is runnning,I found the ubi list have many fail jobs.Then I check the cp.log,found this message"time="2024-05-20 02:38:31.414" level=error msg="hardware info parse to json failed, error: %+vinvalid character '\x01' looking for beginning of value" func=GetCpResource file="ubi.go:755"

And the log of container resource-exporter shows "
{"gpu":{"driver_version":"","cuda_version":"","attached_gpus":0,"details":null},"machine_id":"d057a478-09c5-430f-9759-9c6b000fe56e","cpu_name":"AMD","cpu":{"total":"16","used":"10","free":"6"},"vcpu":{"total":"16","used":"10","free":"6"},"memory":{"total":"125 GiB","used":"12 GiB","free":"111 GiB"},"storage":{"total":"915 GiB","used":"707 GiB","free":"161 GiB"}}
The node not found nvm libnvidia, if the node does not have a GPU, this error can be ignored.
{"gpu":{"driver_version":"","cuda_version":"","attached_gpus":0,"details":null},"machine_id":"d057a478-09c5-430f-9759-9c6b000fe56e","cpu_name":"AMD","cpu":{"total":"16","used":"10","free":"6"},"vcpu":{"total":"16","used":"10","free":"6"},"memory":{"total":"125 GiB","used":"12 GiB","free":"111 GiB"},"storage":{"total":"915 GiB","used":"707 GiB","free":"161 GiB"}}
The node not found nvm libnvidia, if the node does not have a GPU, this error can be ignored.

we have fixed the bug, please use this branch to rebuild the binary
https://github.com/swanchain/go-computing-provider/tree/releases

Already update to 0.4.8,but this error still comes out.
Here is the cp.log.I have two servers,they both have the same problem.
cp.log

After this error occurs, cp will automatically restart the resource-exporter component to work normally.

Please upgrade to v0.5.1


Fixes #2 (failing duration description test)

This makes the testDurationDescription() pass but how can you reason that the "5m 3w 1d 7:04:00"  is
giving you an extra hour (7h) if we are only add "+ 6.hours + 4.minutes"? Is that because of daylight savings?

Thanks @getaaron & @masters3d !

I'll be checking the rest of the issues during the weekend. Thanks a lot!


Resize direct lvm

Improves device_tool to be able to reload pool && adds docs about how to extend devicemapper thin-pool

/cc @rhvgoyal ptal
Thanks @chenchun

@chenchun  Why are you introducing this new reload command. How does it help?
You seem to be changing block size. Can you really change block size of a running thin pool? I thought block size gets fixed at pool creation time and after that can't be changed?

@rhvgoyal I'm not changing block size of the pool. I'm increasing pool capacity.
The added docs shows how to increase the capacity if the data lv is full and vg-docker volume-group has no more capacity.
#### direct-lvm mode

1. Extend volume group (VG) `vg-docker`

        $ vgextend vg-docker /dev/sdh
        Volume group "vg-docker" successfully extended

2. Extend the `data` logical volume(LV) `vg-docker/data`

        $ lvextend  -l+100%FREE -n vg-docker/data
        Extending logical volume data to 200 GiB
        Logical volume data successfully resized

3. Reload devicemapper thin-pool with device_tool

        $ ./device_tool reload /dev/vg-docker/data /dev/vg-docker/metadata

Why are you introducing this new reload command. How does it help?

If I only do the above 1 & 2 steps, the num_sectors of devicemapper thin pool still stays the original size and I have to reload the thin pool to make it refresh.
I'm following the steps mentioned in https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/6/html/Storage_Administration_Guide/ch37s03.html#procedure-multipathd-afterresize-pre_RHEL5u3 to fresh num_sectors. We can refresh it by using dmsetup command as the document states, but I think it's better to improve device_tool to make it simple.

@chenchun Why don't you use lvm to create lvm thin pool and pass it to docker using dm.thinpooldev. you can look at man lvmthin to figure out how to create lvm thin pool or use docker-storage-setup to set one for you.
https://github.com/projectatomic/docker-storage-setup
Point being, device-tool is not a proper tool to manage thin pool. My understanding is that this is more a debugging tool. lvm can already do whatever you are trying to do with device_tool. What's the point of building lvm capabilities in this device_tool. Using lvm to create lvm thin pool and using lvm to manage that pool is easier option then keep on adding more stuff to device_tool.

@rhvgoyal https://github.com/projectatomic/docker-storage-setup is really professional. But I did not notice that before and I was following docker official docs https://docs.docker.com/engine/userguide/storagedriver/device-mapper-driver/ to create direct-lvm for production. My problem is that devicemapper pool is full on serveral nodes now. I want to extend the pool on the fly because I'm running some important containers on those nodes and it is painful to stop those containers. And it seems there is not any docs talking about how to extend the pool on the fly.

@chenchun So you need a new docker to grow your thin pool. That means you will have to restart docker. And thin pool is reloaded anyway during restart so it will automatically grow. So I am not sure how this tool will help you.
Anyway, why does user need to pass "datadevice" and "metadatadevice" for reload operation. Graphdriver already knows about data device and metadata device and same can be retrieved internally and used?

Why can't you use dmsetup to reload your thin pool. Seriously, I am really not in favor of converting device_tool into another tool which will try to do what dmsetup and lvm already do.

So you need a new docker to grow your thin pool. That means you will have to restart docker.

I'm not sure this is true. I can extend pool without restarting docker with the above steps.

Anyway, why does user need to pass "datadevice" and "metadatadevice" for reload operation.

This is kind of possible. I'll have a try.

Why can't you use dmsetup to reload your thin pool. Seriously, I am really not in favor of converting device_tool into another tool which will try to do what dmsetup and lvm already do.

Because it's a little complicated to use dmsetup directly. You have to do it manually.

dmsetup table thin_pool_device
dmsetup suspend thin_pool_device
Change the second number of table info (i.e. the disk end sector) to reflect the new number of 512 byte sectors in the disk.
dmsetup reload thin_pool_device table_name
dmsetup resume thin_pool_device

Besides we can already use device_tool to resize loop device backend thin pool. So I think it's better to make the above dmsetup steps implemented in device_tool.
Anyway, if device_tool is not the recommended way, I can change the docs to use dmsetup directly.

@rhvgoyal @thaJeztah
I removed the code of reloading pool with device_tool and updated docs to only use dmsetup to reload direct-lvm thin pool.
I think it's helpful to add these docs because the official docs encourages people to use direct-lvm mode with docker created pool in production and I believe not only me have the demand of extending thin pool on the fly. And someone may accidentally using loop device on production, so I also add docs about how to extend loop device backend thin pool.

In general I like the idea of documenting that how to extend thin pool once it gets full. Please refresh your patch and then I will review it further.

/cc @moxiegirl, because I think you also were working in changes in this area

ping @moxiegirl could you have a look at this one?

thanks! LGTM
ping @moxiegirl ptal

Apart from minor comments, this LGTM. Good documentation. We need this to grow thin pool.
Again I am wondering how to integrate docker-storage-setup better. There we setup and use lvm thin pool which automatically grows as thin pool gets filled based on certain thresholds. If volume group is about to be full, all one has to do to add storage to volume group and it will extend automatically.

@rhvgoyal would it be an idea to contribute it to this repository (for inclusion in the /contrib directory)? Just thinking out loud

@chenchun Thanks for the incorporation. LGTM for docs.  @thaJeztah I tested mbentley's add yesterday.  After this merges, I have an update there.

Just updated the PR to address @rhvgoyal 's comments.

LGTM
ping @moxiegirl for the merge

@chenchun Thanks for the quick turnaround!


Add Response::text() to async Client

Closes #550

I don't think the appveyor build failure is related to this PR.

Do this in #552


Upgrade Audited

Updates audited from 4.8.0 to 4.9.0 (Changelog)
Applies old audited migration
Pins Audited at minor version and adds a note to hopefully prevent this recurring

New thing looks good!


[Client] Add support for STREAMING_TRANSFER payment type under conditionalTransfer client endpoint

Is your feature request related to a problem? Please describe.
Allow for "set it and forget it" streaming payments - this is much better UX for platforms like dTok/Pitch Investors Live and is an important payment primitive.
Describe the solution you'd like
A quick guide on how to do this:
Step 1: Add relevant types here: https://github.com/ConnextProject/indra/blob/master/modules/types/src/index.ts

[ ] Add STREAMING_TRANSFER type to TransferConditions ENUM
[ ] Add StreamingTransferParameters and StreamingTransferResponse types (params should be something like: amount, assetId, recipient, rate, startBlock)
[ ] Add ResolveStreamingTransferParameters and ResolveStreamingTransferResponse types (param should probably just be sender unless you want to allow multiple payment streams per sender/receiver combo)
[ ] Add above to ConditionalTransfer union types

Step 2: Write StreamingTransfer app

Standardized app interface/guide: https://specs.counterfactual.com/en/latest/app-definition.html
Example app (above linked transfer app): https://github.com/ConnextProject/indra/blob/master/modules/contracts/contracts/UnidirectionalLinkedTransferApp.sol

General strategy here: you probably don't need to write an applyAction or getTurnTaker fn. Just have the app state include startingBlock and balance, which get set on install. For computeOutcome, compute the ending balance as current block number minus startingBlock times the rate. This would let either party call resolveCondition in the client which would computeOutcome and uninstall the app.
Step 3: Add the new app to the conditionalTransfer controller in client: https://github.com/ConnextProject/indra/blob/master/modules/client/src/controllers/ConditionalTransferController.ts
You should be able to generally copy the existing implementation for linked transfers. Some gotchas:

Use installVirtual not install - see the transfer controller for how to call this
resolveCondition should just call uninstallVirtual instead of takeAction since the app doesn't use an applyAction fn.

I believe that should be it. @LayneHaber anything I'm missing here? What's the best way to test this?

@ArjunBhuptani we should point at this as the example, since it already follows the "general strategy" you outlined: https://github.com/ConnextProject/indra/blob/master/modules/contracts/contracts/SimpleLinkedTransferApp.sol

Right, the only issue with the above is that it's using two (ledger) app (i.e. one between the sender and hub and one between hub and receiver), which is different from using a virtual app (between sender and receiver OVER hub)

Closing this because it's stale and we're moving away from virtual apps for now.


Enhancement: Adding Plot for individual string power generated per day

It would be really cool to see the integrated power that is generated by the individual strings.  I really like to see the string power curves, but also seeing integrated kWh numbers per string per day would be good to determine how each array is performing from month to month based on panels and position.

Hi @saultenuts - thanks for opening this suggestion!  You can edit your panel and add a query for each one of your strings to get something like this (the 1 month graph):

They queries would look something like this:
SELECT integral("A_Power")/1000/3600 AS A FROM "strings"."http" GROUP BY time(1d)

Make sure to select stack option:


[Question] Working with existing Django project

Can I use FASTAPI within an existing Django project? If doing so will it improve some of the latency of the Django endpoint queries? and since it already has /swagger endpoint, /docs from FASTAPI should not interfere with it?
Much appreciated

Can I use FASTAPI within an existing Django project?

Yes
https://fastapi.tiangolo.com/advanced/sub-applications/
https://stackoverflow.com/questions/63726203/is-it-possible-to-use-fastapi-with-django

If doing so will it improve some of the latency of the Django endpoint queries?

Unlikely

and since it already has /swagger endpoint, /docs from FASTAPI should not interfere with it?

It should not, if it does u can change where the endpoint points: https://fastapi.tiangolo.com/tutorial/metadata/#openapi-url

But Django does not support async so FastAPI will prevent timeout as current Django sees this
Sent from my iPhone

On 14 May 2021, at 10:42, ArcLight_Slavik @.***> wrote:

Can I use FASTAPI within an existing Django project?
Yes
https://fastapi.tiangolo.com/advanced/sub-applications/
https://stackoverflow.com/questions/63726203/is-it-possible-to-use-fastapi-with-django
If doing so will it improve some of the latency of the Django endpoint queries?
Unlikely
and since it already has /swagger endpoint, /docs from FASTAPI should not interfere with it?
It should not, if it does u can change where the endpoint points: https://fastapi.tiangolo.com/tutorial/metadata/#openapi-url
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

since 3.1 yes -> https://docs.djangoproject.com/en/3.2/topics/async/
this place is for fastAPI issues , go to https://github.com/tiangolo/fastapi/discussions or https://gitter.im/tiangolo/fastapi
could you close your issue , thanks

But it is related to see if using FastAPI can alleviate CSV timeout in Django, if not then FASTAPI won’t help then?
Sent from my iPhone

On 14 May 2021, at 12:25, raphaelauv @.***> wrote:

since 3.1 yes -> https://docs.djangoproject.com/en/3.2/topics/async/
this place is for fastAPI issues , go to https://github.com/tiangolo/fastapi/discussions or https://gitter.im/tiangolo/fastapi
could you close your issue , thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@raphaelauv Probably shouldn't send gitter anymore, we switched to discord a while back

Thanks for the help here @raphaelauv and @ArcLightSlavik ! 👏  🙇
Sorry @scheung38 , I can't help you if you don't follow the issue template and provide more information.
If you still have problems please create a new issue and follow the template step by step.
Given that, I'm gonna close this one.

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


latest version prophet issue

Project Robyn
hi!
I'm getting an error when facilitating holidays data after latest updating to Robyn version.
I'm working with daily data (1years)
I'm loading the data using the following command (as in the demo):
Describe issue
InputCollect <- robyn_inputs(InputCollect = InputCollect, hyperparameters = hyperparameters)

Warning message:
In prophet_decomp(dt_transform, dt_holidays = InputCollect$dt_holidays,  :
  Currently, there's a known issue with prophet that may crash this use case.
 Read more here: https://github.com/facebookexperimental/Robyn/issues/472

i read this issue articel but i dont know how to solve my problem..
Provide reproducible example
InputCollect <- robyn_inputs(
  dt_input = as_tibble(dt_simulated_daily),
  dt_holidays = dt_prophet_holidays,
  date_var = "date", # date format must be "2020-01-01"
  dep_var = "conversion", # there should be only one dependent variable
  dep_var_type = "conversion", # "revenue" (ROI) or "conversion" (CPA)
  prophet_vars = c("trend", "season", "weekday", "holiday"), # "trend","season", "weekday" & "holiday"
  prophet_country = "KR", # input one country. dt_prophet_holidays includes 59 countries by default
  context_vars = NULL, # e.g. competitors, discount, unemployment etc
  paid_media_spends = c("pc_naver_spend", "mo_naver_spend",
                        "pc_kakao_spend", "mo_kakao_spend",
                        "pc_google_spend", "mo_google_spend"), # mandatory input
  paid_media_vars = c("pc_naver_clicks", "mo_naver_clicks",
                      "pc_kakao_clicks", "mo_kakao_clicks",
                      "pc_google_clicks", "mo_google_clicks"), # mandatory.
  # paid_media_vars must have same order as paid_media_spends. Use media exposure metrics like
  # impressions, GRP etc. If not applicable, use spend instead.
  organic_vars = NULL, # marketing activity without media spend
  # factor_vars = c("events"), # force variables in context_vars or organic_vars to be categorical
  window_start = "2022-01-03",
  window_end = "2022-12-31",
  adstock = "geometric" # geometric, weibull_cdf or weibull_pdf.
)

hyperparameters <- list(
  pc_naver_spend_alphas = c(0.5, 3),
  pc_naver_spend_gammas = c(0.3, 1),
  pc_naver_spend_thetas = c(0.1, 0.3), # geometric

  mo_naver_spend_alphas = c(0.5, 3),
  mo_naver_spend_gammas = c(0.3, 1),
  mo_naver_spend_thetas = c(0.1, 0.3),

  pc_kakao_spend_alphas = c(0.5, 3),
  pc_kakao_spend_gammas = c(0.3, 1),
  pc_kakao_spend_thetas = c(0.1, 0.3),

  mo_kakao_spend_alphas = c(0.5, 3),
  mo_kakao_spend_gammas = c(0.3, 1),
  mo_kakao_spend_thetas = c(0.1, 0.3),

  pc_google_spend_alphas = c(0.5, 3),
  pc_google_spend_gammas = c(0.3, 1),
  pc_google_spend_thetas = c(0.1, 0.3),
  mo_google_spend_alphas = c(0.5, 3),
  mo_google_spend_gammas = c(0.3, 1),
  mo_google_spend_thetas = c(0.1, 0.3),

  train_size = c(0.5, 0.8)
)

Environment & Robyn version
Robyn version: ‘3.9.0’
my R version:
R version 4.2.2 (2022-10-31 ucrt)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19044)

Hi @jupi-analyst What version of prophet are you using? Can you try updating to latest dev version as suggested here?

Hi @jupi-analyst What version of prophet are you using? Can you try updating to latest dev version as suggested here?

packageVersion("prophet")
‘1.1.2’

Please, don't replicate messages in different tickets. I do read them all. Let's continue in the original thread.


Support content scope as a user script

Task/Issue URL: https://app.asana.com/0/1163321984198618/1201071474741984/f
Tech Design URL:
CC:
Description:
Loads the code that is currently in the extension as a user script that's bundled and accounts for the privacy config.
https://github.com/duckduckgo/content-scope-scripts is the bundled code.
This change doesn't implement anything in isolation so would be safe to merge, the scripts are controlled by remote config and require integration into iOS etc.
Steps to test this PR:
1.
1.

OS Testing:

[ ] iOS 13
[ ] iOS 14
[ ] iOS 15
[ ] macOS 10.15
[ ] macOS 11

Internal references:
Software Engineering Expectations
Technical Design Template

Closing this in favour of: https://github.com/duckduckgo/BrowserServicesKit/pull/45 as it doesn't rely on the build script PR.


[WIP] permissions list function

@bakura10 what do you think about this ?
Need more test etc ... It's just a POC
Some typo are incorrect getPermissions return string[] in phpdoc but in fact it's Permission[]

What's the use case?

For retrieve the list of permissions. We can only get role but no permissions. When we use the view helper we don't have to call this method but when you have an API, you need te retrieve permissions on client side and for this an endpoint will be useful.

@Orkin mhmm, but this is out of the scope of this library. I think it fits your own application service layer, not ZfcRbac.

I have implemented in my application. It's just sharing in case it had it up here. If this is not the case no problem;).

@Orkin thanks for your contribution, but I really think it does not fit in this library =/
@bakura10 what do you think?

You're right, I'd prefer not to complexicate this even more by introducing a new service. While "RoleInterface" does not have a "getPermissions", the concrete implementation has one (https://github.com/zf-fr/rbac/blob/master/src/Rbac/Role/Role.php#L49).
You could therefore add the method "getPermissions" right into your Identity object, as you can have access to all roles and, in turn, all permissions. I think the logic better suit there and avoid introducing a new service.
Thanks anyway ;).


Optimize rebuffering

This PR optimizes RepairSetup::rebufferBottomUp with three types of changes:

Vectors are preallocated wherever their size can be predicted
Slack values, used in sorting, are precalculated.
Option pruning is modified from pessimistically quadratic to linear complexity.

I gathered averaged outcomes from 10 runs on chosen designs (cts target):

Version \ Design
Ariane
BlackParrot
TinyRocket
Ibex

Unoptimized
63m40s
3m39s
0m27s
1m24s

Optimized
12m32s
2m57s
0m21s
0m28s

I see these failures
array_ins_delay (tcl) *ERROR* segmentation violation
insertion_delay (tcl) *ERROR* segmentation violation

they were broken in master a while ago and then fixed.  I suspect you need to merge master into your branch.

Wow! Amazing work

Started a secure CI

There was some issue in the secure CI with the SQL server going down while this was running but as far as I can see everything passed.

This ready to go @maliberty?


Feature/toast notifcation

Anytime something is refreshing, there will be the bootstrap alert notification at the top.  When refresh succeeds or fails, a toast notification will appear briefly (1 second) to indicate it succeeded or failed.  If it failed, the bootstrap alert will show the error details.
This fixes the issue with the refresh being too fast that people can't tell that it succeeded through the bootstrap alert alone.  The refresh bootstrap alert is still needed when it's so slow (refresh all workflows/tools).

Codecov Report

Merging #101 into develop will increase coverage by 0.26%.
The diff coverage is 72.91%.

@@             Coverage Diff             @@
##           develop     #101      +/-   ##
===========================================
+ Coverage    65.12%   65.38%   +0.26%
===========================================
  Files           91       93       +2
  Lines         2440     2467      +27
  Branches       308      310       +2
===========================================
+ Hits          1589     1613      +24
- Misses         685      687       +2
- Partials       166      167       +1

Impacted Files
Coverage Δ

src/app/workflow/tool-tab/tool-tab.component.ts
42.85% <0%> (-4.52%)
:arrow_down:

src/app/shared/toaster/toaster.module.ts
100% <100%> (ø)

src/app/shared/toaster/toaster.component.ts
100% <100%> (ø)

src/app/shared/refresh.service.ts
77.96% <69.23%> (+3.45%)
:arrow_up:

src/app/workflow/versions/versions.component.ts
88.57% <72.72%> (-0.32%)
:arrow_down:

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 e0561e7...c9788d5. Read the comment docs.


Display the current EFO release in the Search UI

A user has requested to be able to see the EFO release/version that is in use for the current GWAS Catalog release.
This could be added to the end of this list of information in the Search UI results page:

EFO releases and their dates can be looked up at https://github.com/EBISPOT/efo/releases

confirmed after release, can be closed


UI enhancement

When click "show hexdump" command, the current document should be "switched" to hex view, instead of opening a new, separated hexdump document. Click the command again, the hex view should switch back to the original text window.
The first row that displays column offset should be fixed and not scroll with other rows
The hex inspector when hover over the data is distracting and useless. Please remove it.
The auto highlight of all character that are identical to the one under the current cursor is distracting as well. This is especially true when it is a common char e.g. 0x20 blank space. Please disable it
Edit value under cursor should happen in place inside the document under the cursor, instead of in a separate input box. Also I should be able to edit multiple consequent bytes by just typing along.
After editing value, the hex dump file should be marked as "modified" visually
Ctrl-S should save to the original file name without prompt

You have some valid points here. Unfortunately, most of them are currently not feasible in a vscode extension.
Although, here's what in my control:

Optional Hex Inspector

I can add a setting to enable or not the Inspector when hovering content

Marked as "modified" visually

I cannot display the 'modified' dot in the document tab, but I can add (modified} in the Hex status bar

Save without prompt

I can remap Ctrl-S to save without prompt, and Ctrl-Shift+S to Save As...

Please update to version 1.1.4

v1.1.4 - Marked when (modified) in the status bar (#13)
If it is impossible to display the 'modified' dot on the tab list / open editor, then just do not do it. Adding (modified) to the status bar is inconsistent with the overall VSCode UI, and I'm afraid few people will be able to recognize it easily.
Beside, the status is not working as expected:

Open file. View hex. Modify it. Close the file --> Expect some warning "File is modified, save it first?", but that is not happening
Close both original file and the hex one. Open a different file. View hex. --> The status bar still displays (modified), which is not expected

Also can you change the popup menu "export to binary file" to simply "save as"?


Automatic posts in detailed aid package post in two different orders

[ ] Automatic posts in detailed aid package post happens 1) when the package is in draft mode AND 2) when the package becomes published, AND 3) medicines are in two different orders in both automatic posts.

To reproduce:

Create aid package in draft state - see auto post (the order of medicines in the automatic post is incorrect)
Move the aid package status to "published" by checking the box in aid package details page (the order of the medicines in the automatic post is correct)

Given the dead line its better to first fix the issue in frontend if that is lower effort.

Above should temp fix it form the UI. Auto post is create only if it's created in published state not in draft state. I hope this the expected outcome.

But I noticed another thing, if I jump past published directly form draft, I don't see the auto post. Is that an issue? @nshamit
If so let's create a separate issue.

If above fix is correct, let's create a separate issue to move the auto posting to backend, and close this.

All posts need to go through the published state. Even if they don't get saved as draft first they must go through the published state. So the post needs to be there at the time the aid package moves into the published state.


Add Conjur OSS information to README

Connected to cyberark/conjur-oss-suite-release#45

Closing; it doesn't make sense to add this to this project


Add Dice Loss

Hi, have added an implementation for dice loss as mentioned in #6435
Will add tests once the correctness of the function is confirmed.

A Quick example of how this loss works currently.
import torch
import torch.nn.functional as F

N = 5
input2 = torch.randn(1, N, 3, 5, requires_grad=True)
target = torch.empty(1, 3, 5, dtype=torch.long).random_(N)
target_one_hot: torch.Tensor = F.one_hot(target, num_classes=input2.shape[1])
target_one_hot = target_one_hot.permute((0, 3, 1, 2))
dl = dice_loss(input2, target_one_hot)

2. Few libraries like PaddlePaddle use (B, Num_classes) to compute
See PaddlePaddle/Paddle#10717
https://github.com/PaddlePaddle/Paddle/blob/develop/python/paddle/nn/functional/loss.py#L41

    # compute softmax over the classes axis
    p = F.softmax(inputs, dim=-1)
    p = p.reshape(p.shape[0], -1)
    targets = F.one_hot(targets) #Or we could directly have one-hot tensor as input
    targets = targets.reshape(p.shape[0], -1)

    intersection = torch.sum(p * targets, dim=-1)
    cardinality = torch.sum(p + targets, dim=-1)

    dice_score = 2.0 * intersection / (cardinality + eps)

    loss = 1.0 - dice_score

This will support shape (B, any_shape_here, num_classes) for input and (B, any_shape_here)

Yeah this is what I had in my mind.
If we compute softmax over all the axis.
Very similar to implementation I had in my repo.
Notice now that the tensor we return also depends on the input.

Firstly, apologies for the delay, I had my end-term exams going on so had to put the PR on hold.
I think this implementation should align with the requirements.
I believe I should move on to adding tests, but I believe I will require a little help there.

Sure, can you let me know what help you need in writing tests?

Hi @oke-aditya, I've added the basic tests, could you check once?
Also, I have trouble understanding the failing checks, could you maybe point out where I've gone wrong?

Hi @pri1311 Thanks for continuing the work!
I had a quick review, although I have not fully validated the tests yet.
Note that the test failure is unrelated to this change, you can do that by viewing the logs of the test failure logs (No need to Login to Circle CI)
https://app.circleci.com/pipelines/github/pytorch/vision/22420/workflows/cfac5971-ed1c-4ef6-9ef2-aa8a94a848af/jobs/1801044
But the lint failure seems to be related to this change. You need to lint the code as mentioned in the contributing.md file. (either use pre-commit hooks or lint manually)
https://app.circleci.com/pipelines/github/pytorch/vision/22420/workflows/0908373e-af80-48ec-b8be-fb55ce07ce10/jobs/1801049

I will review this, validate and get back to you soon!

I'm sorry for the delay in review of this PR. I will be reviewing this over weekend definitely. It's been really hard for me this year but I will be back to this.

@oke-aditya I haven't yet given it a detailed look, but I think one thing of concern while comparing the two libraries is the input dimensions, for torchvision, we follow (B, N1, N2, ..., C) and kornia follows (B, C, H, W).

My concern is that results should have matched. At least for cases where we have all zeros. As such dice loss is well defined and adjusting for flexible shapes. Shouldn't affect the values.

Results don't match because in this piece of code here we are specifying the one hot values as per the torchvision input format (B, N1, N2, .., C) (also followed by PaddleSeg)

input_zeros[:, :, :, 0] = 1.0
input_zeros[:, :, :, 1] = 0.0

But for Kornia we need these values on dimension 1, not the last dimension.
if you transpose the matrix, it should work fine. I can make changes so that torch-vision accepts (B, C, N1, N2....).

B C N H W is something which we assume in all the models as well as the codebase. So it would be much better and to use I think

Thanks a lot for working on this @pri1311 I will do another round of validation of results. And we would be good.
Although we have to wait from approval from Nicolas (as he is the maintainer :smile: )

@NicolasHug looks like this missed 0.15.  I tried this loss function and it seems to work well for segmentation problems with unbalanced classes.  Its inclusion would be great, unless there's other functionality I'm overlooking that implements another dice loss function such that this PR is no longer needed.


Frame - Missing client directive in React

Description
I noticed that the React Frame component is missing the "use client" directive, while all other components have it

Link to Reproduction (or Detailed Explanation)
See above description
Steps to Reproduce
Render the Frame component in a React Server Component. It should show the following error:

ModuleDependencyError: export 'useState' (imported as 'useState') was not found in 'react' (possible exports: Children, Fragment, Profiler, StrictMode, Suspense, __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED, __SECRET_SERVER_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED, cache, cloneElement, createElement, createRef, forwardRef, isValidElement, lazy, memo, startTransition, use, useCallback, useDebugValue, useId, useMemo, version)

Ark UI Version
4.1.1
Framework

[X] React
[ ] Solid
[ ] Vue

Browser
No response
Additional Information
No response

@carloitaben
Thanks for your time creating the issue. A fix will be available in the next release


Bump prism version

Prism 0.25 fixes an issue with its rbi files (Sorbet type definitions). Without this fix, ruby-lsp is unusable in projects using Sorbet.
ruby-lsp currently depends on prism (>= 0.22.0, < 0.25. Please change to  < 0.26 or similar.

Thanks for the report!

ruby-lsp is unusable in projects using Sorbet

Not sure how this is the case. We use Ruby LSP with Sorbet in all our projects. Is this more related to not using Tapioca?
Also, do you have ruby-lsp added to your Gemfile? It shouldn't be necessary to add it to the Gemfile since version v0.8.0.
Regardless, we always keep up to date with Prism updates, so we'll be updating soon.


Desktop wallet is not usable on MacOsX Mojave

Here are a couple of issues with running the wallet on an older version of OS-X (Mojave). Look like CSS or rendering issues:

Button heights are not calculated correctly, but they are still clickable, user just doesn't know what they do as the text is hidden:

Clicking on the networks drop down list either crashes something or renders a blank screen - only way out of this is to close and restart the app:

I am not seeing this bug with the latest release

Hey @cdm
This seems to work - please can you re-test now and close if you no longer see the issue?

We need to do this https://github.com/vegaprotocol/vegawallet-desktop/issues/365 first

We need to verify if that's still the case