[release] engine release builder only starts after previous build finishes

on dashboard, the empty build block of the not started build, corresponds to the yellow running block on an existing build

And on luci milo dashboard, the start time of the not started build, corresponds to the finish time of the previously existing build

@drewroengoogle from triage: I think this is fixed by Drew?

We have increased the concurrency to 2 for engine release branch candidates, so 2 builds per platform can run at once. This is different from how tasks in the flutter pool are run, so if there are a ton of cherry picks at once, you may still run into blank tiles until they are picked up, but that is a consequence of using the internal scheduling mechanism. I can close this though


create requirement.txt or environment.yml

The current environment/requirement file is too verbose:
https://github.com/neurobooth/neurobooth-os/blob/master/environment_stm.yml
https://github.com/neurobooth/neurobooth-os/blob/master/docs/neurobooth.txt
it mentions dependencies and dependencies of dependencies.
Also it should be in top level

closed by #49


Add filename kwarg to plot_bloch_vector()

Summary
This commit adds a new filename kwarg to the plot_bloch_vector function.
When set this will save the bloch sphere visualization to the specified
file instead of calling show() on it. This enables people to generate
bloch sphere visualizations with non-GUI matplotlib backends.
Details and comments
Fixes #1095

@jaygambetta yep looks that way. I was only looking at plot_bloch_vector(), but you're right all of the plots in the file have the same issue. I'll update them all now


Fixes bug in leafletPathEvents

Fixes #728 and #729

Nice catch, so I guess there was in recursive watch loop there. BTW both watches should really check against unnecessary changes.
leafletScope.$watch('layers.overlays', function(newOverlayLayers, old) {
if(angular.equals(newOverlaysLayer, old)) return;
});

Can you add in the watch checks as well?

Are you sure it should double check the watch? Isn't that the definition of a watch that it only fires when old and new values are different?

I guess we can skip it, but I have seen them fired a lot when nothing changes. They also always run on the first check regardless.

@tombatossals looks like you have another PR that conflicts with this. #734 can you jump in here?

Added the check. We can always remove it if necessary.

@pieterjandesmedt LGTM, but I am waiting on @tombatossals since he has #734

Hey guys, I'm merging this and will solve the conflics in the https://github.com/tombatossals/angular-leaflet-directive/pull/734
Nice work


Unfortunately it's broken

Heyho, I just followed the stream recording to understand this whole thing and everything worked fine until the last bunch of minutes.
I figured that there're two issues in the final version:
auth.js
export const checkSession = callback => {
  const isLoggedIn = localStorage.getItem("isLoggedIn")
  console.log({ isLoggedIn })
  if (isLoggedIn === "false") {
    console.log("Not logged in")
    callback()
    return
  }
  auth.checkSession({}, setSession(callback))
}

Here the addition of the callback() and return statement breaks the app. Whenever the account page is called now it just does login again and redirects back to the callback without logging the user in.
also (a very minor thing):
for the logout process, you set (also in auth.js inside the logout process)
auth.logout({
    returnTo: location.href
  })

which does work for the production version but gatsby develop fails with
error  Unexpected use of 'location'  no-restricted-globals.

Now i tried using the auth0.js  and the gatsby-browser.js from the store but the same behavior occurs.
Something with if (!isAuthenticated()) return callback(); isn't working for me. When I'm commenting that out, I can at least login again. But with that in place the auth0.checkSession({}, setSession(callback)); is never getting called as it seems.

I also encountered this error. My immediate solution was to add a second local storage value for isLoggingIn. I feel like once i have a more holistic understanding I might get a better idea but for now it looks like this:
/**
 * Automatically re-fetch the session from auth0
 * (Used in wrapRootElement)
 */
export const checkSession = cb => {
  if (!isLoggedIn() && !isLoggingIn()) {
    console.log("checkSession - not logged in")
    cb()
    return
  }
  console.log("checkSession...", isLoggedIn())
  auth0.checkSession({}, setSession(cb))
}

/**
 * Begin the login flow through auth0
 */
export const login = () => {
  if (isBrowser) {
    console.log("login...")
    localStorage.setItem("isLoggingIn", "true")

    auth0.authorize()
  }
}

export const logout = () => {
  console.log("logout...")

  user = {}
  tokens.idToken = null
  tokens.accessToken = null

  const { protocol, host } = window.location
  const returnTo = `${protocol}//${host}`

  localStorage.removeItem("isLoggedIn")
  auth0.logout({ returnTo })
}

// Set tokens (in memory)
const setSession: (cb?: () => void) => Auth0Callback<any> = (cb = () => {}) => (
  err,
  authResult
) => {
  console.log("setSession...")
  if (err) {
    if (err.error === "login_required") {
      login()
    } else {
      throw err
    }
  } else if (authResult && authResult.accessToken && authResult.idToken) {
    tokens.idToken = authResult.idToken
    tokens.accessToken = authResult.accessToken
    auth0.client.userInfo(authResult.accessToken, (error, userProfile) => {
      if (error) {
        throw err
      }
      user = userProfile

      localStorage.removeItem("isLoggingIn")
      localStorage.setItem("isLoggedIn", "true")

      cb()
    })
  }
}

Sorry I missed this issue!
@amberleyromo sent me this tutorial, which resolves the issue: https://auth0.com/blog/securing-gatsby-with-auth0/ (repo: https://github.com/auth0-blog/gatsby-auth0)
If anyone has time to PR a fix, I'd greatly appreciate it! If not, I'll put this on my todo list for as soon as I have free time.

@jlengstorf I think https://github.com/jlengstorf/gatsby-auth0-app/pull/4 fixes this, along with some other bug fixes and enhancements. Hopefully it keeps your to-do list a little lighter!


Hive table stats

Collect hive table statistic in mysql as opposed to derby DB

Can you please squash the commits?

Squashed

Looks good to me.


stats api created as get_stats.sh; menu_launcher now calls stats api; closes 43

Current coverage is 90.43% (diff: 100%)

Merging #164 into master will decrease coverage by 2.33%

@@             master       #164   diff @@
==========================================
  Files            26         26
  Lines          2866       2866
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           2659       2592    -67
- Misses          207        274    +67
  Partials          0          0

Powered by Codecov. Last update b78a94c...206b642


Add note explaining how to replace an organization

Why:
Closes: https://github.com/github/docs/issues/28915
What's being changed (if available, include any code snippets, screenshots, or gifs):
Add a note explaining that, in the case where you want to "replace" an organization, you should rename it instead of removing it.

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

Thanks @HectorCastelli! ✨  I'm going to confirm this with the product team, then will get back to you 👀

Nice spot @cmwilson21 ! I've update it now.


Testtoken (Sponsorship) does not work

Describe the bug
The given testoken till 3.Nov. seems to be corrupted. Maybe the issue is, when the testperiod overlaps from previous month to next month? This was now the case. Since I updated to the latest token I always get this output:
starting evcc: 'EVCC_DATABASE_DSN=/data/evcc.db evcc --config /config/evcc.yaml'
[main  ] INFO 2024/10/30 21:25:55 evcc 0.131.4
[main  ] INFO 2024/10/30 21:25:55 using config file: /config/evcc.yaml
[main  ] INFO 2024/10/30 21:25:55 listening at :7070
[main  ] FATAL 2024/10/30 21:25:55 sponsortoken: rpc error: code = Unknown desc = token has invalid claims: token is expired
[main  ] FATAL 2024/10/30 21:25:55 will attempt restart in: 15m0s
Steps to reproduce

Have a config needing a sponsor token
Copy the current test token in the evcc.yaml
Start EVCC
...

Configuration details
I run EVCC in Home Assistant AddOn. Don't think this makes any difference, but who knows...

Log details
starting evcc: 'EVCC_DATABASE_DSN=/data/evcc.db evcc --config /config/evcc.yaml'
[main  ] INFO 2024/10/30 21:25:55 evcc 0.131.4
[main  ] INFO 2024/10/30 21:25:55 using config file: /config/evcc.yaml
[main  ] INFO 2024/10/30 21:25:55 listening at :7070
[main  ] FATAL 2024/10/30 21:25:55 sponsortoken: rpc error: code = Unknown desc = token has invalid claims: token is expired
[main  ] FATAL 2024/10/30 21:25:55 will attempt restart in: 15m0s

What type of operating system are you running?
HomeAssistant Add-on
Nightly build

[X] I have verified that the issue is reproducible with the latest nightly build

Version
0.131.1 and also 0.131.4

#16070

yes, looks like duplicate. Please comment, when the solution in the linked issue does not work for you.

Sure will try now. So I need to delete the EVCC.db. Hope I will find the files. Since I am in trial phase, the data is not relevant for me.

Hmm I am to stupid to find evcc.db. Seems complicated steps with docker to access that file when installed as HA AddOn. I deinstalled EVCC and install it new, in the hope that will recreated the evcc.db... Will report.

Confirmed. Reinstall fixed it. So evcc.db seems to be rebuild after reinstall, was a lot easier for me, even in the docu part everything is explained ;-) Shame on me.
And yes it fixed the broken token issue as well. Thanks for support.


IntelliSense is blocked by Find all References on long operations

Open a file with a token references in many places, so that Find All References will take some time
Invoke Find All References on the Token
Hover over something in the editor, or attempt to get SignatureHelp on something (e.g. DateTime.)

Expected: I get IntelliSense in the editor
Actual: I'm blocked on any IntelliSense until Find All References completes

This happens because Find all refs checks each of involved projects entirely, as a one fat Reactor operation. See what @dsyme suggests to fix this here https://github.com/Microsoft/visualfsharp/pull/2358#issuecomment-277248643

Yes, the way this is currently coded means it is cancellable but not interleaved.

We would need to make the 50ms Eventually chunks interleaved with other reactor queue operations
We should likely use an explicit priority queue for the reactor queue. Currently we rely on a mailbox processor

@cartermp It should not reproduce on master because stale file check results are used if fresh ones are not available in 1 second.

@vasily-kirichenko That's good!

It's not block, but produces no results.

Closing since @vasily-kirichenko says fixed on master (or at least not blocking any more)


Fix servicemonitor relabeling

Trying to deploy keda using this values file:
prometheus:
  operator:
    enabled: true
    port: 8080
    serviceMonitor:
      enabled: true
      relabellings:
      - regex: (rest_client_.*|go_.*|workqueue_.*)
        action: drop

Result is:
Error: YAML parse error on keda/templates/manager/servicemonitor.yaml: error converting YAML to JSON: yaml: line 20: did not find expected key

Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #
Helm chart fixed to use right indents.
helm template -f a.yaml -n keda . | grep ServiceMonitor -A20
kind: ServiceMonitor
metadata:
  name: keda-operator
  annotations:
    {}
  labels:
    app.kubernetes.io/name: keda-operator
    helm.sh/chart: keda-2.10.2
    app.kubernetes.io/component: operator
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/part-of: keda-operator
    app.kubernetes.io/version: 2.10.1
spec:
  endpoints:
  - port: metrics
    path: /metrics
    relabelings:
      - action: drop
        regex: (rest_client_.*|go_.*|workqueue_.*)
  namespaceSelector:

Looking good, could you update the test values yaml to use service monitor on all the resources with a relabeling (to prevent this in the future)?
You can achieve it modifying this section: https://github.com/kedacore/charts/blob/main/.github/workflows/ci-core.yml#L107-L115

Sure. Test data updated. Templating is ok, but helm deploy failing on timeout. Is it common?

Is it common?

No it isn't. Metrics server pod is failing to start, which shouldn't happen. I don't think that it's related with your commit, let me check

The only idea I have is an invalid regex expression. Had no chance to check it in real deployment.

The only idea I have is an invalid regex expression. Had no chance to check it in real deployment.

I don't think that it's enough to break the metrics server startup as it's something related with Prometheus, not with the metrics server itself. I'll test it locally during the morning

Thanks Jorge. One more thing to discuss: here is inconsistency between relabellings and relabelings. Do we want to fix this?

It'd be nice, but we should maintain backward compatibility, so we can't just rename it, we have to maintain both, the old one and the new one (adding a deprecation note on the old one)

It's because there is a PR merged in core but still not merged here: https://github.com/kedacore/charts/pull/432
Let me push it

Could you rebase your branch? The fix has been merged

I'm on top of main:
* 0bd1bc1 - (18 hours ago) Fix servicemonitor relabeling - Alexey Kubrinsky (HEAD -> fix-servicemonitor, origin/fix-servicemonitor)
* c5fad71 - (5 days ago) fix(http-add-on): Fix resources variables for the http-add-on deploym… (#454) - Jocelyn Thode (origin/main, origin/HEAD, main)

Jorge, please check last commit. Made relabelling deprecated. Is it good?

I think so.
Do we need to create an issue for the deprecation @tomkerkhove ?

Yes, we should follow https://github.com/kedacore/governance/blob/main/DEPRECATIONS.md#introducing-new-deprecations.
I'll do this


Collapsibles button on facelessui.com

The collapsibles button on https://facelessui.com/ links to /docs/collapsible, which routes to a 404 error, and should link to /docs/collapsibles, which is the correct route.
Would be great to have the web frontend in a public repo to be able to contribute to that.

Hi @JuliusGudni. Good catch! The repo for the website is in fact public, and can be found here. The reason it is not nested under the Faceless UI org is to avoid Vercel hosting fees. PR's are happily accepted, or otherwise we'll be sure to take care of being that it's front-facing. Thank you!

Hey @JuliusGudni I went ahead and took care of that broken link with this commit. Thanks again for heads up. Keep it coming!

Hey @jacobsfletch. That makes sense, and thank you for the nice and welcoming response. I would have submitted a PR, but somehow I missed the email notification and didn't see your response until now.
I'd love to contribute on this project and I have some ideas. Is there some way I can contact you, everything's fine with me, Slack, Discord, Email... or whatever you prefer ;)

@JuliusGudni absolutely, my email is<EMAIL_ADDRESS>or<EMAIL_ADDRESS>I'm on Slack too.


Closing the user friendly data rendition should not close the sidekick.

Is your feature request related to a problem? Please describe.
Feedback from customer -> closing the user-friendly data rendition for JSON should not close the sidekick.
Describe the solution you'd like
There should be a way to turn on/off the user-friendly rendition of the JSON that is disconnected from the sidekick being visible/not visible.
Describe alternatives you've considered
Currently, if closing the rendition, the sidekick goes away and then has to be brought back with couple of clicks on the sidekick extension Or a refresh of the page and clicking on sidekick extension. Not the best user experience.

@amol-anand clicking one the "close" link doesn't close the sidekick, does it?

@rofe This was happening previously, but now it doesn't.
@Viru21 What happens when you hit this close button? It should not close the sidekick (it does not for me anymore)
Once you close the user-friendly view, There is no button to re-enabled that view from the sidekick outside of a refresh. Not sure if we need that since people can just click refresh instead of clicking on a button. Thoughts?


COBOL: copy file not found - may I missing something?

Development environment used

Z Open Editor version:
Editor Platform

[x] Visual Studio Code
[ ] CodeReady Workspaces
[ ] Eclipse Che
[ ] Standalone Theia

Editor Platform Version: 1.55.2
Operating System: Windows 10 20H2 19042.928
Java Version (when using VS Code or Theia, extecute java -version and paste the details here):
java version "1.8.0_281"
Java(TM) SE Runtime Environment (build 1.8.0_281-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.281-b09, mixed mode)
[ ] Related to RSE API?

RSE API Plugin version:
Zowe CLI version:
Node.js version:

Logs attached (see here how to get them): no
NOTE: No ZOS connection, local files only.

Problem Description
Detailed steps for reproducing the problem:
COPY file in the same folder as the source program is not found with message:
Unable to find copybook xxxxx
Exact sentence:
COPY INQSET1.
Observed behavior
I tried the following:

Change case of copybook's extension (.cpy to .CPY)
Add ".CPY" (uppercase) in "Cobol: include File Extension"
Remove copybook extension

On all cases issuing "Pause the COBOL server language" then reopening the source file, making some modification then save but no luck.
Expected behavior
Copybooks being recognized.

@ssamayoa I do not see you are mentioning your property group. Even if the file is in the same folder you still need to specify a property group with workspace relative or absolute location.
If you have a property group you can use these instructions to check the log files for error messages and warning resolving your copybooks: https://ibm.github.io/zopeneditor-about/Docs/locating_local_client_logs.html
Let me know if that helped.

@phaumer
Resolved!
I should RTFD!
Many thanks!


Unable to find view (After publish)

When I run locally my code everything works as expected, but after I publish it in docker I get this error:
`An Exception has occured when generating PDF Document: Unable to find view '~/Views/CashBookTemplate.cshtml'. The following locations were searched:
~/Views/CashBookTemplate.cshtml
Hint:

Check whether you have added reference to the Razor Class Library that contains the view files.`

Can you help?

Hi @aleksandaryanakiev, thanks for reporting the issue. Is it possible for you to provide a minimal sample that reproduces the issue?

Hi, I've spent around an hour this morning trying to get a simple Unit Test working.
`    [Test]
public async Task SendSmtpEmail()
{
// https://code.soundaranbu.com/send-emails-with-razor-templating-in-asp-net-core/
    WelcomeEmailModel? model = new WelcomeEmailModel
    {
        Name = "Test",
        Email =<EMAIL_ADDRESS>        CompanyName = "Razor.Templating.Core",
        SenderName = "Soundar"
    };

    string? body = await RazorTemplateEngine.RenderAsync("~/Views/WelcomeEmailTemplate.cshtml", model);

    SmtpClient? client = new SmtpClient
    {

`
I'd created a local Views folder in the NUnit Test project, but that wouldn't work. I've tried following your examples here: https://code.soundaranbu.com/send-emails-with-razor-templating-in-asp-net-core/ more closely by creating a separate class library for the EmailTemplates. Just continue to get:
_System.InvalidOperationException : Unable to find view '/Views/WelcomeEmailTemplate.cshtml'. The following locations were searched:

/Views/WelcomeEmailTemplate.cshtml
Hint:

Check whether you have added reference to the Razor Class Library that contains the view files.
Check whether the view file name is correct or exists at the given path.
Refer documentation or file issue here: https://github.com/soundaranbu/RazorTemplating_

Am I missing something here? I just wanted to use Razor rendering for email templates, and your library looked like a simple solution.

Hi @gabs247, it's unfortunate that you are facing these issues.
I would like to know if you have followed all these steps here:
https://soundaranbu.medium.com/render-razor-view-cshtml-to-string-in-net-core-7d125f32c79
You can also clone this repository and navigate to the examples folder or test cases to see if it's working for you. If it works then you can compare it to yours.
If still you are facing the issue, I am happy to take a look at a simple reproducible sample if you can provide one.
Awaiting your response. Thanks.

Thanks @soundaranbu, will try cloning when I have some time and see if I can get the samples to work.
I tried following those steps and the other examples to see if I'd missed something, I'm using .NET 6, but kept getting the error indicating that the View couldn't be found.
cheers

I'm not able to reproduce this issue. Hence closing. Feel free to reopen if you can provide any sample project to reproduce the issue.
Thanks :)

I am getting the same issue. Request to reopen.

Hi @nsbgit, thanks for reporting your issue. As mentioned earlier, this can be reopened with a minimal sample that reproduces the issue.
I look forward to the sample. Thanks

Not sure if this was the issue for the other users, but I was running into the same issue with the latest version (1.9.0). I got the same error after copy/pasting pieces of code from one project (where everything worked fine) to another. In the end I found out that I forgot to add the dependency in Startup.cs as described in the documentation:
In ASP.NET Core, add dependency like below in Startup.cs -> ConfigureServices
...
services.AddTransient<ExampleService>();
//add after registering all the dependencies
services.AddRazorTemplating();

Hi, getting same issue, is it necessary to create RazorLibrary, what if we want to keep razor template in same project as web api project?

Hey @prabhpahul, you don't need to. But you'll have to make your API's .csproj look similar to this  https://github.com/soundaranbu/Razor.Templating.Core?tab=readme-ov-file#razor-views-in-library

But I do have another project which works perfectly fine with out this. Also, how does only complaints in docker works fine if you run on IIS or dotnet run command locally through vs code

Yes, it works as expected. Only when I run on it docker complains.
Also, I'm currently using Microsoft.NET.Sdk.Web changing to Razor Sdk is triggering lot of other changes

I was able to fix it, use was Template folder not mounting in docker after publish, had to add a itemgroup in cs proj,
 <ItemGroup>     <EmbeddedResource Include="PDFTemplates\KaizenPDFTemplate.cshtml">       <CopyToOutputDirectory>Always</CopyToOutputDirectory>     </EmbeddedResource>   </ItemGroup>


Update DispatcherFactory.php

rpc register method filter __construct

close https://github.com/hyperf/hyperf/pull/2730


Compilation with Clang leads to constexpr errors

Compiling with Clang 8 on Mac:
/Users/gus/universal-llvm/./include/universal/posit/posit.hpp:1732:27: error: constexpr variable 'nbits' must be initialized by a constant expression
        constexpr size_t nbits = number.nbits;
                                 ^~~~~~~~~~~~
/Users/gus/universal-llvm/numerical/thin_triangle.cpp:223:35: note: in instantiation of function template specialization
      'sw::unum::to_binary<sw::unum::posit<32, 2> >' requested here
        ostr << "    a  = " << sw::unum::to_binary(a) << " " << sw::unum::to_base2_scientific(a) << " : " << std::showpos << a << std::noshowpos << std::endl;
                                         ^
/Users/gus/universal-llvm/numerical/thin_triangle.cpp:289:3: note: in instantiation of function template specialization
      'printTriangleConfiguration<sw::unum::posit<32, 2> >' requested here
                printTriangleConfiguration<Posit>(cout, a, b, c);
                ^

There are many instances of this same error. Any ideas?

It looks like we need to rewrite that code as it is a const variable but for some reason clang doesn't recognizes it as such.
I'll work on this fix now.

Let me know if you figure this out -- I'm sure it's a clang vs GCC
difference. Curious whether GCC is "smarter" in this case, and is able to
detect that it's constexpr.
Might it be related to the use of templates? That was my initial guess when
I looked at the code.
On Fri, Oct 4, 2019 at 11:12 AM Theodore Omtzigt<EMAIL_ADDRESS>wrote:

It looks like we need to rewrite that code as it is a const variable but
for some reason clang doesn't recognizes it as such.
I'll work on this fix now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stillwater-sc/universal/issues/104?email_source=notifications&email_token=AAJFZALODCDACCEO55DPRULQM6BR5A5CNFSM4I5RKLHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAMPEGA#issuecomment-538505752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJFZAP54SAQW6HAO26JYKTQM6BR5ANCNFSM4I5RKLHA
.

It is the problem of how you pass template arguments down into different processing layers. It may simply be a matter of making the const expr explicit, but I need to do some experimentation with that compiler. I first need to set up that environment before I can attempt the experiments, :-(

still haven't been able to set up a clang8 environment, but I have removed the constexpr ambiguity. Can you give it a whirl?

Thanks Theo! Only just got around to testing it, but you're right, it's fixed.


The addon seems to break built-in CSS hot reloading

As you might know, ember-cli has a built-in styles hot reloading feature. So when you change your SCSS/CSS files, only styles will be reloaded and your app will keep the same state in the browser.
But if I add ember-cli-svgstore to the package.json, the hot reloading feature doesn't work at all, so any changes in style files will force the whole browser tab to reload.

I'm unable to reproduce this just by dropping ember-cli-svgstore into a vanilla Ember app. Can you provide more information about your setup?
CLI version, any extra configuration for this addon, etc. would be super helpful in trying to track this down.

I use Ember CLI v2.5.0
Here you can see my app, including package.json: https://github.com/ivanvotti/time-tracker

I've just made a new ember app and could reproduce the problem.

I added ember-cli-svgstore to package.json
I added the code below to the ember-cli-build.js file:

  var app = new EmberApp(defaults, {
    svgstore: {
      files: {
        sourceDirs: 'icons',
        outputFile: '/assets/icons.svg'
      }
    }
  });

Aha, I see. This looks like an issue upstream in broccoli-svgstore – it doesn't cache its output, so the entire app tree gets invalidated each time there's a rebuild.
I'm not likely to have a chance to get to this in the near future (we're not actually using this addon internally at the moment), but fixing this should just be a matter of swapping in broccoli-caching-writer as the base class for that plugin.

Added an issue (above), and a PR to fix the issue (https://github.com/svgstore/broccoli-svgstore/pull/12) to the broccoli-svgstore repo.

broccoli-svgstore 0.2.0 has just been released -- giving it a much-needed update after almost 2 years 😄. It builds on top of broccoli-caching-writer, so I'm thinking it should resolve this issue.

Thanks for your effort. BTW, there's already a much better solution: broccoli-symbolizer.
It is actively maintained, supports essential configuration options, such as custom ID generator for symbols and more. It relies on broccoli-persistent-filter, so it should build even faster than a broccoli-caching-writer solution.
Also if you use Ember there's ember-svg-jar that does even more for you.
I created both projects after @dfreeman wrote here: I'm not likely to have a chance to get to this in the near future (we're not actually using this addon internally at the moment).
So give it a shot;)


libdbusmenu-qt: 0.9.3+14 -> 0.9.3+16

The upgraded version has memory leak fixes and test case fixes for qt5.
This also swaps to git due to difficulties in fetching src with bzr without
'not a branch' errors.
Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build heaptrack kcachegrind

CC @ttuegel


Display additional units

ex: https://beta.opendota.com/matches/2743155083

We should see if we can do this before public release.


Is there anyway to use this on multiplayer servers?

When I go to my private server, it says its not compatible. And when I playing on it, the mod doesn't work. It only happens with this mod.

The mod unfortunately isn't client only, it has to be on the server as well.
If you put it on the server, the clients will need the mod too.

I was trying to do that, unfortunately they dont have this mod on the list mods for the server.

Sorry to bring the old issue up, but would it be possible to write Bukkit plugin that would calculate and send attackedAtYaw via messaging channel?

In the Spigot mappings this value is already calculated and has the name ap (A public field of the EntityLiving class, at least in v1_16_R2, this name is probably unstable). It can be sent to the player every PlayerVelocityEvent via the messaging channel, but I suspect issues similar to #5 will happen, so this needs some testing.

I would really like to see a server-side only version of this for Bukkit/Spigot!
Also, is there some sort of history thing for Forge methods? I'm on 1.6.4 and I'd like to know if this could work.

Hello! I have made a plugin for Paper 1.16.5+ out of experiment. You're welcome to test if it works.
https://github.com/Brawaru/DamageTiltHelper


api: gpio: add gpio direction query functions and driver api

Some services require the ability to query whether a GPIO pin is configured as an input, output, both, or neither prior to performing any operations at the service level. This is done in order to reduce state tracking within the service.
To that end, this change adds

gpio_port_get_direction()
gpio_pin_is_input(), and
gpio_pin_is_output()

Additionally, we provide tests under gpio_basic_api and implementations for a handful of drivers.

Implementations forthcoming for gpio_emul and gpio_cc13xx_cc26xx..

@mmahadevan108 - reworked. It's a bit cleaner now. Used IS_ENABLED() rather than #ifdefs everywhere. CONFIG_GPIO_GET_DIRECTION is now user-configurable, and there is a runtime check within the syscall for support (so that multiple drivers on the same system can co-exist with support / lack of support - e.g. in the case of local GPIO and a GPIO expander).

fix typos in gpio_handlers.c

@mnkp - can you please review? @mmahadevan108 - I think I've addressed your concerns, so please take another look.

@henrikbrixandersen - yeah, i feel that if the function isn't supported in all drivers, it can't technically be a part of the basic api. Can always move in at a later date if it becomes more widely adopted.

added [EXPERIMENTAL] and select EXPERIMENTAL to Kconfig

Seems we have two very related PRs for extending the GPIO API with query functions. This one and #43225.

assert port is not NULL in tests
remove _raw from port_get_direction

The way  gpio_pin_is_output/input() loop over the entire port only to throw away all pins but one seems really inefficient.  Since is goes through a function pointer the compiler won't be able to optimize away any of it.
Why not have port_get_direction() take a mask of which pins to query?  This makes gpio_pin_is_in/output() much more efficient.
/// @param pins Set of pins to query the direction of.  The value of input and output for bits corresponding to those NOT set in pins is undefined.
__syscall int gpio_port_get_pins_direction(const struct device *port, gpio_port_pins_t pins,
                                           gpio_port_pins_t *inputs, gpio_port_pins_t *outputs);

static inline int gpio_pin_is_input(const struct device *port, gpio_pin_t pin)
{
	…
	rv = gpio_port_get_pins_direction(port, BIT(pin), &pins, NULL);

	return !!((gpio_port_pins_t)BIT(pin) & pins);
}

If you want to get all the pins, gpio_port_get_direction() is a trivial wrapper:
static inline int gpio_port_get_direction(const struct device *port, gpio_port_pins_t *inputs, gpio_port_pins_t *outputs)
{
        return gpio_port_get_pins_direction(port, (gpio_port_pins_t)-1, inputs, output);
}

The implementations that already have the direction in a bitfield don't need to change.  Those that need loops only get slightly more complex.  Example from gpio_emul.c:
pins &= BIT_MASK(config->num_pins);
for (int i = find_lsb_set(pins) - 1; pins; pins &= pins - 1, i = find_lsb_set(pins) - 1) {
        ip |= !!(drv_data->flags[i] & GPIO_INPUT) * BIT(i);
}
*input = ip;

Example of compiled code, https://godbolt.org/z/foq7rcsMx

The way gpio_pin_is_output/input() loop over the entire port only to throw away all pins but one seems really inefficient. Since is goes through a function pointer the compiler won't be able to optimize away any of it.

True - I thought about this before as well. Eventually, I settled on simplicity because platforms vary in the way they access this information (e.g. some only require 1 read-register instruction, some HALs require looping, and ffs can be expensive on some platforms).

Why not have port_get_direction() take a mask of which pins to query? This makes gpio_pin_is_in/output() much more efficient.

I think that's fine at the API level, and yes, the implementation is fairly trivial.

The implementations that already have the direction in a bitfield don't need to change. Those that need loops only get slightly more complex. Example from gpio_emul.c:
pins &= BIT_MASK(config->num_pins);
for (int i = find_lsb_set(pins) - 1; pins; pins &= pins - 1, i = find_lsb_set(pins) - 1) {
        ip |= !!(drv_data->flags[i] & GPIO_INPUT) * BIT(i);
}

Sure, that works.

fixed typo

@mnkp, @henrikbrixandersen - does this look acceptable now? Was discussed in last week's dev-review, and the consensus was that there should be room for both this and #43225

Compliance check seems to be a false positive


Backbone for classes

Remember to add this application under INSTALLED_APPS on settings.py too. Otherwise, the project will not have the application inside.

The rest looks good to me. Feel free to merge after considering the above

Yes good lets go

Sorry lets go


Release Google.Cloud.PubSub.V1 version 3.0.0

Changes in this release:
No API surface changes since 3.0.0-beta01. (See 3.0.0-beta01 release notes for details.)

Created release for Google.Cloud.PubSub.V1-3.0.0


reuse_server: kill/restart it after timeout

After a bunch (15-20) tests the suite from https://github.com/Vimjas/vim-python-pep8-indent stalls.
Adding some puts the last --remote-expr is vim --servername VIMRUNNER0.687345666041246 --remote-expr feedkeys("iclass Foo(\<CR>")
In this case vim --servername VIMRUNNER0.687345666041246 --remote-expr 1 hangs already.
It would be nice if this could be detected and e.g. after a few seconds the server getting restarted.
A good idea might be to send a ping always --remote-expr 1 and have a short timeout for this.
It looks like a bug in Vim after all - I cannot pinpoin it to a single test, and I've not looked at strace too closely, but vim is taking ~60% CPU.

Yeah, I've noticed this issue in the past. For larger test suites, Vim hangs. I'm not sure if it's a bug with Vim, or some problem in the way I spawn processes, but I have no idea how to debug it, either :/.
I'll consider your ideas about auto-restarting. For the moment, a quick fix is to use gvim (or mvim on a Mac). With Vimrunner, it's Vimrunner.start_gvim. That one doesn't seem to hang. If you start xvfb-run in your build, it won't be a problem, even in travis.

I've tried using xvfb-run explicitly, and it did not seem to help?!
Also in Docker with ENTRYPOINT ["sh", "-c", "xvfb-run rspec spec -f doc $@", "ignore"].

it also happens with IO.popen instead of PTY
it seems to make sense to include --noplugin -i NONE as Vim options (from looking at what -V10/tmp/vim.log provides).

I've tried using xvfb-run explicitly, and it did not seem to help?!

But did you try it with the graphical version of Vim? What I'm saying is that, in my experience, the gui version doesn't hang.

it seems to make sense to include --noplugin -i NONE as Vim options (from looking at what -V10/tmp/vim.log provides).

That might make sense. I'll see about making that change and check if it breaks anything. In general, I'm not sure what exactly --noplugin disables, so I'll need to experiment with it a bit.

Ah, misunderstood it. gvim works for me, too.


How to use the transformer model on this Robust ReID task?

The codebase is a very nice work. I just find the script file of using the resnet to train and would like to know that if you have the plan to release the script of transformer model.
Look forward for your reply.
Best wishes.

Thank you for your interest in our work.
The script files and the corresponding config files of ViTs have been provided.


Change 3D Tiles refine to inherit from parent

This is another 3D Tiles breaking change.
Spec update: #526.
Part of #3241.

Looks good here too. Merged.

@lilleyse I deleted the 3d-tiles-refine branch.  Please delete branches after merging a pull request.


OpenCL Support when Convert Weight [Feature Request]

🚀 Feature

MLC_CHAT ADD OPENCL SUPPORT
Motivation

When I used
python3 -m mlc_llm.build ...

on my opencl mobile device, i got the error as #1541 .
So i used the comdline
mlc_chat convert_weight...

to convert my custom llm.
But I got the error as follow:
ValueError: Auto detection for device opencl is not supported. Currently only supports: dict_keys(['cpu', 'cuda', 'metal', 'vulkan', 'rocm'])
As so far, I can't covert and quantization my custom llm model and run on my orangepi 5 plus( I follow the #821 , and it seems not work.)
Alternatives

Additional context

@Hzfengsy Yeah definitely! Would you like to enhance these lines: https://github.com/apache/tvm/blob/main/python/tvm/target/detect_target.py#L77

@Hzfengsy Yeah definitely! Would you like to enhance these lines: https://github.com/apache/tvm/blob/main/python/tvm/target/detect_target.py#L77

oh! Thanks for your reply and I'll try it later

@Hzfengsy Yeah definitely! Would you like to enhance these lines: https://github.com/apache/tvm/blob/main/python/tvm/target/detect_target.py#L77

Oh I see you are already on top of this (you asked me to open a ticket for this issue but I see BeiningWu has brought it up already)

have the function been enhanced yet?

Hi @BeiningWu, I fixed it in #16854, Can you try it again with latest mlc llm and relax. Thanks.


Assets image not loading in new react native version 0.76

Description
I have tried following code to show images from assets folder but it is not showing:
<View style={[styles.mainContainer,{backgroundColor: bgColorMap[item?.color]}]}>
<Image src={require('../../assets/zincfolder.png')} style={{height: 50, width: 50}} resizeMode='contain' />
<Text style={{fontSize:14, color: textColorMap[item?.color]}}>{item?.title}
<Text style={{fontSize:12, color: textColorMap[item?.color]}}>{item?.count}

Steps to reproduce
You can install the latest react native version 0.76
try to show image from assets folder locally no image shows
in console image path log some number
React Native Version
0.76
Affected Platforms
Runtime - Android, Runtime - iOS
Areas
Fabric - The New Renderer, TurboModule - The New Native Module System, Bridgeless - The New Initialization Flow
Output of npx react-native info
(base) mac@Ahsans-Mac Findey % npx react-native info
info Fetching system and libraries information...
System:
  OS: macOS 13.7
  CPU: (8) x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
  Memory: 24.05 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.12.1
    path: /usr/local/bin/node
  Yarn:
    version: 3.6.4
    path: /usr/local/bin/yarn
  npm:
    version: 8.19.2
    path: /usr/local/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /usr/local/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK:
    API Levels:
      - "21"
      - "23"
      - "28"
      - "29"
      - "30"
      - "31"
      - "32"
      - "33"
      - "34"
      - "35"
    Build Tools:
      - 28.0.3
      - 29.0.2
      - 30.0.3
      - 31.0.0
      - 33.0.0
      - 33.0.1
      - 34.0.0
      - 35.0.0
    System Images:
      - android-29 | Google APIs Intel x86 Atom
      - android-30 | Google APIs Intel x86 Atom
      - android-31 | Google APIs Intel x86_64 Atom
      - android-32 | Google APIs Intel x86_64 Atom
      - android-33 | Google APIs Intel x86_64 Atom
      - android-34 | Google APIs Intel x86_64 Atom
      - android-UpsideDownCakePrivacySandbox | Google Play Intel x86_64 Atom
    Android NDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.18034.62.2411.12169540
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.8
    path: /usr/bin/javac
  Ruby:
    version: 3.3.5
    path: /usr/local/opt/ruby/bin/ruby
npmPackages:
  "@react-native-community/cli":
    installed: 15.0.0-alpha.2
    wanted: 15.0.0-alpha.2
  react:
    installed: 18.3.1
    wanted: 18.3.1
  react-native:
    installed: 0.76.0
    wanted: 0.76.0
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: true
iOS:
  hermesEnabled: true
  newArchEnabled: true

(node:6698) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
info React Native v0.76.1 is now available (your project is running on v0.76.0).
info Changelog: https://github.com/facebook/react-native/releases/tag/v0.76.1
info Diff: https://react-native-community.github.io/upgrade-helper/?from=0.76.0&to=0.76.1
info For more info, check out "https://reactnative.dev/docs/upgrading?os=macos".

Stacktrace or Logs
no logs show in the console

Reproducer
I have no git repo
Screenshots and Videos
No response

I have already added the react native 0.76 version in my issue version section and also mentioned that you can create new project and try to show image in it from assets folder.

what was the issue, if this was resolved? Im facing a similar thing may b I messed up something while upgrading.

what issue you are facing can you tell me?
maybe some code or some details


CI for documentation deployment

Fixes https://github.com/neuropoly/ivado-medical-imaging/issues/248

@jcohenadad Can I open GH pages for this repo ? for the moment is off.

@jcohenadad Can I turn on GH pages for this repo ?

@jcohenadad Can I turn on GH pages for this repo ?

why would you like to do that?

Isn't the documentation of the project made using GH pages ?

Isn't the documentation of the project made using GH pages ?

https://ivado-medical-imaging.readthedocs.io/

Sorry, my mistake, I thought it was similar with shimming-toolbox.org
We don't need this PR, because this one is for GH pages. I'll do the change in namecheap.
This PR can be closed without merging.

@jcohenadad The change in DNS is made. We need 30min to propagate it. Sorry for the misunderstanding.

changes out of scope


Add the Atmel pin names to the mapping table

Since the ATtiny 48/88 come in a variety of packages with different pin numberings, add the Atmel pin designators for ease of layout

:+1: Thanks!


join not only by time but additionally also by column

How can I join not only by time but also by a column?
Currently, I get: Found duplicate columns, but I would like to perform the time series join per group.
val left = Seq((1,1L, 0.1), (1, 2L,0.2), (3,1L,0.3), (3, 2L,0.4)).toDF("group", "time", "valueA")
  val right = Seq((1,1L, 11), (1, 2L,12), (3,1L,13), (3, 2L,14)).toDF("group", "time", "valueB")
  val leftTs = TimeSeriesRDD.fromDF(dataFrame = left)(isSorted = false, timeUnit = MILLISECONDS)
  val rightTS        = TimeSeriesRDD.fromDF(dataFrame = right)(isSorted = false, timeUnit = MILLISECONDS)

  val mergedPerGroup = leftTs.leftJoin(rightTS, tolerance = "1s")

fails due to duplicate columns.
When renaming the columns:
val left = Seq((1,1L, 0.1), (1, 2L,0.2), (3,1L,0.3), (3, 2L,0.4)).toDF("groupA", "time", "valueA")
  val right = Seq((1,1L, 11), (1, 2L,12), (3,1L,13), (3, 2L,14)).toDF("groupB", "time", "valueB")
  val leftTs = TimeSeriesRDD.fromDF(dataFrame = left)(isSorted = false, timeUnit = MILLISECONDS)
  val rightTS        = TimeSeriesRDD.fromDF(dataFrame = right)(isSorted = false, timeUnit = MILLISECONDS)

  val mergedPerGroup = leftTs.leftJoin(rightTS, tolerance = "1s")
  mergedPerGroup.toDF.printSchema
  mergedPerGroup.toDF.show
+-------+------+------+------+------+
|   time|groupA|valueA|groupB|valueB|
+-------+------+------+------+------+
|1000000|     1|   0.1|     3|    13|
|1000000|     3|   0.3|     3|    13|
|2000000|     1|   0.2|     3|    14|
|2000000|     3|   0.4|     3|    14|
+-------+------+------+------+------+

a cross join is performed between each group and time series.
that needs to be manually reduced.
mergedPerGroup.toDF.filter(col("groupA") === col("groupB")).show
+-------+------+------+------+------+
|   time|groupA|valueA|groupB|valueB|
+-------+------+------+------+------+
|1000000|     3|   0.3|     3|    13|
|2000000|     3|   0.4|     3|    14|

Is there any functionality to perform this type of join more efficiently / built in?

leftJoin takes a "key" argument that allows you to specify the secondary join key (equality only)

Thanks.
one fighter question: when using an interval which is rather large (i.e. multiple values from the right fall into the interval from the left the join will only join the first record. Which means the distinct I previously used is not required.
leftTs.leftJoin(rightTS, tolerance = "1s", key = Seq("group")).toDF.show
+-------+-----+------+------+
|   time|group|valueA|valueB|
+-------+-----+------+------+
|1000000|    1|   0.1|    11|
|1000000|    3|   0.3|    13|
|2000000|    1|   0.2|    12|
|2000000|    3|   0.4|    14|
+-------+-----+------+------+

is looking better
leftTs.leftJoin(rightTS, tolerance = "1hour", key = Seq("group")).toDF.show
+-------+-----+------+------+
|   time|group|valueA|valueB|
+-------+-----+------+------+
|1000000|    1|   0.1|    11|
|1000000|    3|   0.3|    13|
|2000000|    1|   0.2|    12|
|2000000|    3|   0.4|    14|
+-------+-----+------+------+

I believe this is also stated in the documentation:

leftJoin A function performs the temporal left-join to the right TimeSeriesRDD, i.e. left-join using
inexact timestamp matches. For each row in the left, append the most recent row from the right at or before the same time. An example to join two TimeSeriesRDDs is as follows.


How to invoke the constructor interface of Executor"

How to invoke the constructor interface of Executor"

You can refer to the gptManagerBenchmark implementation to learn how to use the executor API: https://github.com/NVIDIA/TensorRT-LLM/blob/main/benchmarks/cpp/gptManagerBenchmark.cpp


ineffassign reporting is incorrect

Ineffassign's negative check reported as positive. At the refreshed repo, goreportcard detects an ineffassign positive:

While ineffassign itself does not detect any errors:

This looks suspiciously similar to the behavior mentioned in #202, I hope it has the same root cause.

I also have this problem.

I can confirm that this problem is still there https://goreportcard.com/report/github.com/perun-network/go-perun#ineffassign
It is the same pattern: goreportcard's ineffassign claims that there is an unused err variable (which is evidently used in the next line though). Running ineffassign locally doesn't show any errors.


Tom/vault minimum secure collateral

@bvotteler some automatic formatting was applied, presumably as a result of the prettier/eslint changes. I committed these first, so the important changes are in the second commit: https://github.com/interlay/interbtc-api/pull/432/commits/10ef8b40353b22f6c6cb2b3a711bc7702dbd9db9

@bvotteler some automatic formatting was applied, presumably as a result of the prettier/eslint changes. I committed these first, so the important changes are in the second commit: 10ef8b4

That's to be expected. Thanks for the heads up.


Add Termux install documentation

I know it might be insignificant, but it happened to me because I somehow forgot to bring my laptop during my business trip.
I suddenly got a crazy idea since I heard about Termux years ago but never use it before. "Use my android phone to code, and start learning neovim !!".
Alas, it didn't go as planned, Turso CLI crashed
SIGSYS: bad system call
PC=0x12f80 m=8 sigcode=1

goroutine 8 [syscall]:
syscall.Syscall6(0x4000515920?, 0x27?, 0x27?, 0x4000515950?, 0x0?, 0x4f?, 0xffffffffffffff9c?)
        syscall/syscall_linux.go:91 +0x2c fp=0x4000463460 sp=0x40004633d0 pc=0x8f3ec
syscall.faccessat2(0x4000515920?, {0x4000515920?, 0xddc67a200998?}, 0x1, 0xc0?)
        syscall/zsyscall_linux_arm64.go:33 +0x84 fp=0x40004634d0 sp=0x4000463460 pc=0x8b674
syscall.Faccessat(0x4000515920?, {0x4000515920, 0x27}, 0x1, 0x200)
        syscall/syscall_linux.go:167 +0x3c fp=0x40004635c0 sp=0x40004634d0 pc=0x88f8c
internal/syscall/unix.Eaccess(...)
        internal/syscall/unix/eaccess_linux.go:10
os/exec.findExecutable({0x4000515920, 0x27})
        os/exec/lp_unix.go:31 +0x58 fp=0x4000463610 sp=0x40004635c0 pc=0x11f0c8
os/exec.LookPath({0x199a40b, 0x3})
.....

Apparently it's a go issue and from the looks of it, There's no hope so far.
However, there's a comment here that made me wonder. As desperate as I am now, I decided to install proot and use ubuntu as my termux distro.
pkg install proot
pkg install proot-distro
proot-distro install ubuntu
proot-distro login ubuntu

After that I tried to install Turso-cli again, guess what ? It works !!
I haven't tried the rest of the distros offered by termux yet since it works with my first choice, but I think you should add this to documentation for the sake of helping people that forgot their laptops while going somewhere far.

All turso cli does can be done through HTTP API so as an alternative you could just use curl :).

I know, but turso cli is really helpful


API change proposal

@mikecann Thanks for creating this library. It makes handling of async flows much more type-safe.
I propose a change in API that I believe will address following limitations in the current API:

Number of steps are limited to 5 (and overloads need to be added for every step).
Type-inference gets messed up in some cases (I think this is a TS bug).
Error handling can not be interleaved and there can only be one terminal error handler step.

The proposed API looks like this:
flowPipe((input: number) => input + 1)
        .then((num) => (num > 100 ? Promise.reject("naa") : num + 100))
        .catch((err) => "something bad happened")
        .end()

The API is intentionally similar to the Promise API that I expect most users to be already familiar with.
Please let me know your thoughts.

This is awesome work @lorefnon, I have updated the readme, bumped the version to 2.0 and added one other test. Ill add you as a contributor too, phenominal work :)

Thank you :)
I missed on updating the README, sorry about that.


Consider removal of output-dir build task parameter

Currently the build tasks have an output-dir parameter which defaults to docker (relative to working-dir which is another task parameter). This can be used to populate a Docker context.
In many build scripts the build output first populates a build dir and is then copied to the output-dir.
A docker context can also be limited using a .dockerignore. In addition newer (non-daemon) builders will likely not copy unneeded files anyhow (see https://docs.docker.com/build/building/context/).
By avoiding copying build output to a docker context, build tasks do not need parameters anymore to control what is being copied and thus should be simpler.
The proposal is to remove the output-dir parameter.
When build caching is enabled (cache-build parameter), copying is still needed. However for this one would not need a build script parameter anymore. Instead the build task would know where the natural build output is for a specific technology.
This change might also impact the ods-package-image task for example to have docker-dir default to ..

This will (negatively) affect compatibility with ODS quickstarters, unfortunately
I am in favour of the change as I agree that output copying is brittle and contributes to confusion

Instead the build task would know where the natural build output is for a specific technology.

I just had a closer look into how that aspects work and worry it isn't that easy. What is the "natural build output"? In theory it can be anywhere so we need a knob to tell the build cache what to retain, no?

Instead the build task would know where the natural build output is for a specific technology.

I just had a closer look into how that aspects work and worry it isn't that easy. What is the "natural build output"? In theory it can be anywhere so we need a knob to tell the build cache what to retain, no?

It could indeed be that this still needs to be declared for continued support of build caching. I was coming from the npm build where there is already a build-dir.
However a problem is with multi module builds for example with gradle it might produce stuff in multiple build dirs. Perhaps it would make sense to delegate the copying to/from cache to the build script itself. The following sections would have to be replaced with such invocations.
https://github.com/opendevstack/ods-pipeline/blob/ca46bcb944f2a9c108324b15148f156112caf2b0/build/package/scripts/cache-build.sh#L76-L87
and
https://github.com/opendevstack/ods-pipeline/blob/ca46bcb944f2a9c108324b15148f156112caf2b0/build/package/scripts/copy-build-if-cached.sh#L86-L98
The mentioned sections would be replaced by invocation of scripts, perhaps
cp-build-to-cache  [-verbose] <cache-dir>
cp-cache-to-build [-verbose]  <cache-dir>
Which are added to the build tools docker image. For each language these would invoke the build tool with the appropriate target.

Now that you mention the npm task, I think there is some duplication between the build and output dir. Maybe one could live with just one dir to declare, and that would be the dir to cache.
About multiple dirs: maybe it should be Gradle's responsibility to unify them back into a single dir.


Contribute to the API List 🎉

Contribute to the API List 🎉
We are building a comprehensive list of APIs across different categories, and you can help! Each contributor can add 1-5 APIs from the table below to the list.
How to Contribute

Choose up to 5 APIs from the table below that aren't yet added.
Copy the JSON format and add the API(s) in the correct position, ensuring the id follows the sequence.
Submit a Pull Request with your changes.

Example Format
{
  "id": [next sequential number],
  "name": "[API Name]",
  "link": "[API Documentation Link]",
  "ctg": "[Category]"
}

New APIs to Choose From

ID
API Name
Link
Category

16
GitHub API
Link
Development

17
Unsplash API
Link
Images

18
NASA API
Link
Science

19
Mailchimp API
Link
Email

20
Stripe API
Link
Payments

21
PayPal API
Link
Payments

22
SendGrid API
Link
Email

23
Twilio API
Link
Communication

24
OpenAI API
Link
AI

25
Google Cloud Vision API
Link
AI

26
Firebase API
Link
Development

27
Slack API
Link
Communication

28
Zoom API
Link
Communication

29
Dropbox API
Link
Storage

30
Imgur API
Link
Images

31
Clarifai API
Link
AI

32
NewsAPI
Link
News

33
Eventbrite API
Link
Events

34
Last.fm API
Link
Music

35
TMDb API
Link
Video

36
Marvel API
Link
Entertainment

37
OpenCage Geocoding API
Link
Mapping

38
OpenLibrary API
Link
Books

39
Jikan API
Link
Anime

40
WeatherAPI
Link
Weather

41
IPinfo API
Link
IP/Location

42
IEX Cloud API
Link
Financial

43
Alpha Vantage API
Link
Financial

44
Bored API
Link
Fun

45
Agify API
Link
Demographics

46
Genderize API
Link
Demographics

47
Nationalize API
Link
Demographics

48
NumbersAPI
Link
Trivia

How to Submit

Fork this repository.
Edit this issue by adding your new API(s) to the list.
Create a Pull Request with your contribution.

Let’s collaborate 😄

Hey @Bashamega , I've added 5 more apis to the api.json file . When i try to push the commit , It says everything up-to-date.

Can you Please guide me how to push the changes?

These steps are correct. Are you sure that you have saved the file?

@Bashamega , Yes I've saved it.

@Bashamega , Yes I've saved it.

This forum can help you: https://stackoverflow.com/questions/1298499/git-push-not-send-changes-to-remote-git-repository

@Bashamega , When i try to commit using VScode, Its showing this error

@Bashamega , When i try to commit using VScode, Its showing this error

Try running first:
npm run format:write

@Bashamega , It worked! Thank you so much. I'll raise a PR.

@Bashamega , It worked! Thank you so much. I'll raise a PR.

Your welcome

@Bashamega , Can you please add a label named hacktoberfest-accepted in the pull request so that it will be added to my contribution for hacktoberfest?

@Bashamega , Can you please add a label named hacktoberfest-accepted in the pull request so that it will be added to my contribution for hacktoberfest?

sure

I want to contribute too, kindly assign me this.

I want to contribute too, kindly assign me this.

You can create a pr. For 5 apis


Correct unsupported parentheses.

There are unsupported parentheses in the manual setup sample script that cause a script run-time error.
So I suggest to correct it.
Which I reported in Issue #72639.
wrong
              console.log('Good to go!');
               // add an event listener
              this.addEventListener('ended', function() {
                console.log('Finished!');
            } //this code segment cause run-time error.

correct
              console.log('Good to go!');
               // add an event listener
              this.addEventListener('ended', function() {
                console.log('Finished!');
            }); //correct unsupported partheses.

@tokawa-ms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

It looks good to me. but a question for @IngridAtMicrosoft - should the code fencing in line 37 be javascript?

It looks good to me, but a question for @IngridAtMicrosoft Ingrid Henkel FTE - should the code fencing in line 37 be javascript?

javascript is used for a similar code block in https://github.com/MicrosoftDocs/azure-docs/pull/72643. I'll make that change here and then merge the PR.


Add sources for Noto Sans Cham, Khmer, Lao

Danh Hong has kindly released the source code for the “phase 2” version of Noto Sans Cham, Khmer, and Lao. We should integrate them into the noto-source tree so that all sources for Noto are in a single place.
https://github.com/danhhong/NotoSansCham/
https://github.com/danhhong/NotoSansKhmer/
https://github.com/danhhong/NotoSansLao/

For Khmer, we’ve already received the revised design sources for Noto “phase 3”, so just adding Cham and Lao.


#1158 permits to disable offset autocommitment

If you want to be sure that your consumers doesn't take a same message in parallel at rebalance, you need to block the until the process of the message is done.
For that, configure a rebalance timeout  : conf.Config.Consumer.Group.Rebalance.Timeout = 5 * time.Minute
When you receive the message, mark offset : consumer.MarkOffset(msg, "")
When you finish your process for this message, commit offsets : consumer.CommitOffsets()
To avoid loosing message if your application reboot when your message is processing or this there is error in process, we don't want to commit before the process is done.
This workflow can work only if the autocommit is disabled. So with this new conf :
conf.Consumer.Offsets.CommitInterval = 0

It seems this workflow doesn't work. The MarkOffset doesn't block rebalancing.


manager - detect peripheral packets after disconnect

Has anyone else had this problem? I'm trying to disconnect from a peripheral and pick it up while scanning again.
For whatever reason, the packets aren't being detected. Any idea what's going on?
React-Native: 0.44
Node: 8.1.3

Hm, I added a delay between stopping the scan, and starting up a new scan, and it's working again.
So for whatever reason, if you start the scan too fast from stopping it, it will not detect packets from a device that's been connected to.
Will try to whip up something reproducible for you guys.


Cancel git/scm operations

Sometimes a git operation such as push or commit takes longer than expected or apparently hangs either due to git hooks or some other issue. It would be helpful to have the option to cancel any running git/scm operation (not just sync) without having to reload the window.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


[ADAM-1579] Add unit test coverage for BED12 format.

Resolves #1579. Found small bugs that would lead to last 6 fields not getting written if the thickStart field wasn't written. Also, identified a bug where the strand would be set to unknown if an independent strand was provided.

Coverage decreased (-0.3%) to 82.744% when pulling ee331a25a03db73819002e9a95dd26e294d9644c on fnothaft:issues/1579-bed12-unittest into 0306717cb952511d48e514475b70ae995a468b5a on bigdatagenomics:master.

Coverage decreased (-0.3%) to 82.744% when pulling ee331a25a03db73819002e9a95dd26e294d9644c on fnothaft:issues/1579-bed12-unittest into 0306717cb952511d48e514475b70ae995a468b5a on bigdatagenomics:master.

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

Thank you, @fnothaft


Sku dev

Added html-support for the diagrams via iframe.
Changed typo in Analytics that saved a wrong named diagram.
Changed html version.

Because of merge conflicts which can not be properly resolved the PR gets refused


Hook compiler-rt up to the swift build system

What's in this pull request?
An updated version of the patch to build compiler-rt along with swift.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

@jrose-apple I addressed your review comments from the original PR:

I've appended '_runtime' to the feature names to prevent conflicts (e.g 'asan' -> 'asan_runtime').
lit.cfg checks for compiler-rt libraries in lib/swift/clang/lib/{platform}/

I'm having some trouble testing the Xcode build, but I'll try and sort that out.

I'm trying to reproduce the test failure on my machine.
W.r.t the Xcode build, it looks like it places the compiler-rt libraries in ./llvm-macosx-x86_64/**RelWithDebInfo**/lib/clang/3.9.0/lib/darwin/**RelWithDebInfo**. Should we change the driver to look for Xcode build directories, or should we add a CMake target that copies over the contents of these directories to the expected locations?

The test failure is a driver bug. It should be looking up libclang_rt.profile_iossim.a, not libclang_rt.profile_ios.a.

Actually, looking at ToolChains.cpp, it has to be that neither one of libclang_rt.profile_iossim.a or libclang_rt.profile_ios.a exist on the builder.

You definitely shouldn't be getting two levels of "RelWithDebInfo" there, but that's probably a CMake bug in compiler-rt because no one's bothered to build it with Xcode. I don't think copying is the right answer; take a look at SwiftApplyXcodeSubstitutions.cmake in swift/cmake/modules/. (There's similar logic in LLVM and Clang; just not sure if it's factored out.)
If it's not easy, I guess it's okay if Xcode support doesn't work yet.

@swift-ci Please test

Hm, looks like passing in the host triple to compiler-rt isn't enough to get it to generate libclang_rt.profile_iossim.a.

@swift-ci Please test

@swift-ci Please smoke test

The latest build failure looks unrelated to this PR ("swift/lib/IDE/Utils.cpp:338:35: error: no member named 'ImplementationOfModule' in 'clang::LangOptions'").

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please test

Ok, watchos and tvos support in compiler-rt is marked as experimental, so I'll disable them in this patch and file a separate bug for it.

@swift-ci Please test

@swift-ci Please test

Ok, it looks like coverage_smoke.swift is failing on iphonesimulator-i386 because the raw profile can't be found. I think the rest of the patch is good, so I'll take the coverage test out and file a separate PR for that.

Instead of removing it, can you just limit it to OS X for now? That way at least the simple case works.

Ok, I re-added the test with 'REQUIRES: OS=macosx'.

Cool. Did it pass CI last time? If so, I think it's fine to commit!

It didn't get all the way through because it failed on the simulators so I'll check it again.

@swift-ci Please test

To git@github.com:apple/swift.git
   f543c33..53f48f8  master -> master

Please just merge pull requests in the future rather than committing out of band. That makes it easier for people following the PR to see what makes it in and what doesn't.


Duplicated email address leads to 500 error

I think 2+ users can share same e-mail address.
However when 2+ users with same address are inactive, djoser will fail so there is no possibility to activate such users.
(And this is danger, because the user can try add next usernames to make the account with such email working.)
As you can see, only Django, Rest_framework & Djoser are in the traceback, so I think Djoser should be fixed.
The problem is in the serializers.py, class UserFunctionsMixin, def get_user() where the orm call User._default_manager.get() is handled for User.DoesNotExist but not for User.MultipleObjectsReturned.
I think instead of .get() we could use .filter().first() here (with appropriate removal of try/except).
This would make the user activating possible, the 1st one first, then the next..
Of course the users identification by email and not by username is not good here.
However I think such solution could give some improvement still.
The other question, which I am not able to answer now, is:
Is it possible to fix it in this way for all scenarios where UserFunctionsMixin is used?
Internal Server Error: /api/v1/users/resend_activation/
Traceback (most recent call last):
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/django/core/handlers/exception.py", line 55, in inner
    response = get_response(request)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/django/core/handlers/base.py", line 197, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/sentry_sdk/integrations/django/views.py", line 85, in sentry_wrapped_callback
    return callback(request, *args, **kwargs)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/rest_framework/viewsets.py", line 125, in view
    return self.dispatch(request, *args, **kwargs)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/rest_framework/views.py", line 509, in dispatch
    response = self.handle_exception(exc)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/rest_framework/views.py", line 469, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/rest_framework/views.py", line 480, in raise_uncaught_exception
    raise exc
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/djoser/views.py", line 202, in resend_activation
    user = serializer.get_user(is_active=False)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/djoser/serializers.py", line 132, in get_user
    user = User._default_manager.get(
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/django/db/models/manager.py", line 85, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/home/mirek/dj/authtemplate/authtemplate/.venv/lib/python3.10/site-packages/django/db/models/query.py", line 653, in get
    raise self.model.MultipleObjectsReturned(
apps.core.models.User.MultipleObjectsReturned: get() returned more than one User -- it returned 4!
[09/Feb/2023 16:28:55] "POST /api/v1/users/resend_activation/ HTTP/1.1" 500 117478

I have not tested it but I think if we will allow duplicate emails, not only Activation (with possible solution above), but more actions will have problem.
So probably a better approach would be to disable duplicate emails. Soemthing as is described here.
https://stackoverflow.com/questions/55969952/how-can-i-avoid-a-user-from-registering-an-already-used-email-in-django.
Maybe this can be applied without Djoser's change, if we would inherit UserViewSet and add email validation for User Create.
.. if so, this can be closed; but I let it open, because I am curious for meaning of others.

Thanks for creating the ticket, you're right. PRs welcome!

I have found some time to have a look into that. This integrity check should be implemented in the DB with a unique constraint.
If such index existed, the create serializer will prevent from creating a duplicate user.
https://github.com/sunscrapers/djoser/blob/0cef72f2e62f1d9e821d49814cd857c098371b7b/djoser/serializers.py#L37-L44
But I agree, this could be a useful in the Djoser as well.

I closed the PR above as we can't justify an additional query to the DB to meet the criteria of a few. I will leave this ticket open. If someone else has a better idea, let me know.


Add authentication support

Companion PR for https://github.com/trenz-gmbh/trenz-docs-api/pull/23


[FEATURE] Shared vision based on token disposition

Is your feature request related to a problem? Please describe.
My players have None permissions for every actor but their own, but I'd still like them to be able to share vision. Enabling Actor permission: None in the module settings, however, also shares the vision of NPC tokens (including those that are hostile!).
Describe the solution you'd like
It'd be cool if there were a way to additionally filter shared vision based on actors' dispositions — perhaps checkbox settings like Actor disposition: Friendly, Actor disposition: Neutral and Actor disposition: Hostile?
Describe alternatives you've considered
This is definitely a nonessential feature request! I can easily circumvent the problem by either A) enabling Global Shared Vision for friendly actors, or B) enabling player actors to share with every other player actor. Adding this feature, however, would make it a global thing that removes the need to change individual actor settings.

This has been added in the latest version


fix(rustup): rustc 1.0.0-nightly (be9bd7c93 2015-04-05)

Copy doesn't imply the Clone trait anymore,
which now has to be specified explicitly.
... it seems

This is working for me.

I just pushed up a very similar fix for this. Thanks though!


DRILL-6492: Ensure schema / workspace case insensitivity in Drill

StoragePluginsRegistryImpl was updated:
a. for backward compatibility at init to convert all existing storage plugins names to lower case, in case of duplicates, to log warning and skip the duplicate.
b. to wrap persistent plugins registry into case insensitive store wrapper (CaseInsensitivePersistentStore) to ensure all given keys are converted into lower case when performing insert, update, delete, search operations.
c. to load system storage plugins dynamically by @SystemStorage annotation.
StoragePlugins class was updated to stored storage plugins configs by name in case insensitive map.
SchemaUtilities.searchSchemaTree method was updated to convert all schema names into lower case to ensure that are they are matched case insensitively (all schemas are stored in Drill in lower case).
FileSystemConfig was updated to store workspaces by name in case insensitive hash map.
All plugins schema factories are now extend AbstractSchemaFactory to ensure that given schema name is converted to lower case.
New method areTableNamesAreCaseInsensitive was added to AbstractSchema to indicate if schema tables names are case insensitive. By default, false. Schema implementation is responsible for table names case insensitive search in case it supports one. Currently, information_schema, sys and hive do so.
System storage plugins (information_schema, sys) were refactored to ensure their schema, table names are case insensitive, also the annotation @SystemPlugin and additional constructor were added to allow dynamically load system plugins at storage plugin registry during init phase.
MetadataProvider was updated to concert all schema filter conditions into lower case to ensure schema would be matched case insensitively.
ShowSchemasHandler, ShowTablesHandler, DescribeTableHandler were updated to ensure schema / tables names (this depends if schema supports case insensitive table names) would be found case insensitively.

Other details in DRILL-6492.

@vvysotskyi please review.


Using Modal - Illegal Argument Exception view=com.android.internal.policy.impl.phonewindow$decorview

Description
Modal is open -> on pressing 'ok', I want to restart the app (it gets restarted). After that, in some random cases it throws Exception
I use RNN but modal is use is react-native's

Solution
Dialog needs to be dismissed after check if its open
ReactModalHostView.java I think
Additional Information

React Native version: 0.34
Platform:  Android 6.0
Operating System:  MacOS Sierra

Can you reproduce this without using react-native-navigation? Maybe it's better to report the error there.

Thank you for the response. The modal is react-native's that was the reason I created it here.
FYI - I removed that modal ( now hiding and unhiding the view as a workaround) and now there's no crash.

I got the same problem but basically it happened because I forgot to dismiss (close) the modal before I restarted the app. Once I fixed it the error didn't happen again.


Looking For A Developer

See below messages

I am not currently interested in long-term job, but if it is just a tool or two... maybe? Depends on what you mean by "tools". Could you elaborate?

Hey, thanks for the reply.
The tools I need are for grabbing live media content (usually in DASH .mpd), decrypt*, combine and re-stream in HLS format.
I do have a couple of base tools that I can provide you that around 80% could be re-used for this project.
If you are interested in discussing further contact me on<EMAIL_ADDRESS>and include your Discord, Telegram or?
Let me know here that its you though please?
*I have a method to get the decryption keys
Thanks

Sorry for asking, but isn't most of what you described, if not all, done by ffmpeg? It can download, decrypt, mux, and I think create HLS stream as well... Pretty sure it can be adapted to the purpose? I might be wrong, I guess. (also, my email is around my profile somewhere)

@Satsuoni no, your e-mail is not around your profile somewhere. I can't be that of a Github noob to not see it.

Sorry for asking, but isn't most of what you described, if not all, done by ffmpeg? It can download, decrypt, mux, and I think create HLS stream as well... Pretty sure it can be adapted to the purpose? I might be wrong, I guess. (also, my email is around my profile somewhere)

I'm sure it can do most of those things, but, its the automation part I need as well.
The tool i have that you could use as a reference flows like this:
URLLIB --> MP4DECRYPT --> FFMPEG
Same as @EnthusiastAnon I can't find your email on your profile, if you dont want to make yours public please email on the address posted above.
Thanks

Hmm, I know ffmpeg can download mpd files, so urllib should be unnecessary, but it seems that decryption_key in ffmpeg is not compatible with cenc, etc, so, that would not work... If it were up to me, I would have either slapped together a shell file in Linux (using temporary file) or mashed together sources of mp4decrypt and ffmpeg, though that would have taken a while. Hmm, not sure I want to bother with setup though, see: lazy.
@EnthusiastAnon : Unlike you people, I am kind of a n00b when it comes to github, so I am not sure where people keep finding my email to write to me directly, but they manage. Maybe there is direct message function? Dunno shrug

@Satsuoni They are probably getting it from somewhere else.
I spent 30 minutes on googling Satsuoni and couldn't find an e-mail unfortunately.
Would really like to know what am I missing :D

@EnthusiastAnon See here.
https://www.nymeria.io/blog/how-to-manually-find-email-addresses-for-github-users
https://github.com/Satsuoni/widevine-l3-guesser/commit/5cbf7693b6909a6e4badc1921af5fbe1e5ef0b95.patch

@Essam3152 marvelous. Thank you. Now I forgot what I wanted to write to him >_>

Right, my git settings. I thought they used my articles. Ah well.

@coin4sports  if you are looking for live stream restream i got plenty of tools @alienmaster9 telegram find it

Please refer to my email, etc...


WIP: Fix for #195: Suppress schema query parameter optimization for providers that are not compatible

Abstracts the change we took to avoid creating duplicate parameters that contain the same value into a ParameterCollectionBuilder class, so that schema discovery queries can be generated with or without the optimization depending on whether the provider supports it.
This approach avoid as much as possible duplicating the code.
Added a couple of test variants that explicitly test the parameter optimization.
TO-DO: still working on setting up a machine in which everything builds, but wanted to get this out there to get feedback.

@ajcvickers @lajones This was also supposed to be in for our release in VS.
The only reason I didn't merge it is because I had some difficulties setting up my environment for testing, but now that @lajones has it, it should be easier.
Can you please take care of running the tests and merge it?

@lajones I have rebased this with current state in the parent branch. Let me know if I can do anything else to help.

Don't worry I'm on this. But I'm working on the Accessibility fixes first.

@divega See #505.

@lajones so this PR can now be closed, correct?

Closing in favor of #505


Feature: add prettier rule for quotes

Is your feature request related to a problem? Please describe.
Currently there are no rules for quotes in the code, it would be good to add single quotes to the prettier rules in order to have an standard in the project.
Describe the solution you'd like
Add single quotes to prettier rules

Hi @jesus4497, thanks for reaching out.
We use double quotes as the standard for our project and we rely on the default prettier rules, in order to enforce it.


Standard library docs are below the fold

The Learn Rust page doesn't show a link to the standard library docs when initially viewed by the user. Instead, the user must scroll down to find the (much less prominently displayed) button labeled 'The Standard Library'
I would argue that the standard library docs are a core part of 'Get(ting) started with Rust'. When I was first learning Rust, I constantly referred back to the standard library documentation to see examples for important types like Vec, Option, Cell, and RefCell.

I accidentally opened this on the wrong repository. Closing in favor of https://github.com/rust-lang/www.rust-lang.org/issues/912


Integrate Postgres with the Play app using EBean.

Connect the app to Postgres database in conf/application.conf.
Add EBean models under app/models/ and repository to use them under app/repository.
Add a PostgresController to allow a simple way to interact with Postgres from HTTP.
Add database evolution script for sample data. Also include tests.
Switch sbt version back to 1.3.13 due to issues with the new version (1.4.3).

It's probably worth taking a look at our ORM docs for Postgres for this. We generally do not want to be using the Database, PostgresDatabase, or DatabaseContext classes directly.

It's probably worth taking a look at our ORM docs for Postgres for this. We generally do not want to be using the Database, PostgresDatabase, or DatabaseContext classes directly.

@bion will do. sorry the PR is not ready for review yet. I will clean it up.

@bion will do. sorry the PR is not ready for review yet. I will clean it up.

I think the PR is now review-ready.

I think the PR is now review-ready.

Wonderful, looks great to me, test looks good.
What happens if you try to launch it without the db?  Is that still workable, or do you now need the DB to be able to use any page?

Wonderful, looks great to me, test looks good.
What happens if you try to launch it without the db?  Is that still workable, or do you now need the DB to be able to use any page?

What happens if you try to launch it without the db? Is that still workable, or do you now need the DB to be able to use any page?

The server will start but once you connect to it, it will in turn try to connect to db and time out. So you won't be able to access any page without db. Not sure if that's okay or should we have a way to let app respond without a db connection.

What happens if you try to launch it without the db? Is that still workable, or do you now need the DB to be able to use any page?

The server will start but once you connect to it, it will in turn try to connect to db and time out. So you won't be able to access any page without db. Not sure if that's okay or should we have a way to let app respond without a db connection.

What happens if you try to launch it without the db? Is that still workable, or do you now need the DB to be able to use any page?

The server will start but once you connect to it, it will in turn try to connect to db and time out. So you won't be able to access any page without db. Not sure if that's okay or should we have a way to let app respond without a db connection.

Just to clarify: Will the DB be there now, after this merge? And do I need to do anything else when locally developing to connect to the DB?

What happens if you try to launch it without the db? Is that still workable, or do you now need the DB to be able to use any page?

The server will start but once you connect to it, it will in turn try to connect to db and time out. So you won't be able to access any page without db. Not sure if that's okay or should we have a way to let app respond without a db connection.

Just to clarify: Will the DB be there now, after this merge? And do I need to do anything else when locally developing to connect to the DB?

Just to clarify: Will the DB be there now, after this merge? And do I need to do anything else when locally developing to connect to the DB?

Yes, the DB will launch along side with app when bin/run-dev is run.

Just to clarify: Will the DB be there now, after this merge? And do I need to do anything else when locally developing to connect to the DB?

Yes, the DB will launch along side with app when bin/run-dev is run.

Just to clarify: Will the DB be there now, after this merge? And do I need to do anything else when locally developing to connect to the DB?

Yes, the DB will launch along side with app when bin/run-dev is run. If you wish to add schema to db, please see files under models/ and repository/ as example. Also you will likely need to update evolution scripts under conf/evolutions/default.

OK, thanks!

Just to clarify: Will the DB be there now, after this merge? And do I need to do anything else when locally developing to connect to the DB?

Yes, the DB will launch along side with app when bin/run-dev is run. If you wish to add schema to db, please see files under models/ and repository/ as example. Also you will likely need to update evolution scripts under conf/evolutions/default.

OK, thanks!

Great work!
It'd be good to get an example of handling a request with synchronous code in (probably not in this PR). The code should be quite a bit simpler e.g.
Person person = new Person();
person.setName(name);
person.save!

Great work!
It'd be good to get an example of handling a request with synchronous code in (probably not in this PR). The code should be quite a bit simpler e.g.
Person person = new Person();
person.setName(name);
person.save!

It'd be good to get an example of handling a request with synchronous code in (not necessarily in this PR). The code should be quite a bit simpler e.g.

Good suggestion. I can take a look at that.

It'd be good to get an example of handling a request with synchronous code in (not necessarily in this PR). The code should be quite a bit simpler e.g.

Good suggestion. I can take a look at that.


Bug Fixer Here

So the End Game screen was inverted so I fixed it, also I fixed bug #36 with this pull request, the bah noise was broken!
Please pull my changes lmao

Thank you!

I think you got confused with the exit game menu. The selected one is the one that's lighter, just like with how it is in the menus.
https://github.com/Jorge-SunSpirit/Doki-Doki-Takeover/assets/15317421/65bea8f5-8bae-43a5-9a8c-893a4af61abe
The change in this PR ended up inverting it, causing it to be the opposite with everything else. I'll have to revert that change, my apologies.


#91: allow to specify custom generator for node projects

Signed-off-by: Kangrui Ye<EMAIL_ADDRESS>

also, please create a test which makes use of this new property with any other value besides the convention


Mitigate errors in URL decoding

There's a concern that the handling of URL path parameter and endpoint configuration for the backend configuration is vulnerable when special characters are used in the product codes. For example non-ascii, chines or polish characters.
We should add some tests to make sure that sanitisation is done in the right way.
An example is give with a question mark in the product code:

the browser will encode the question mark
the application will decode the question mark
the application code doesn't test on the validation of the generated URL (the question mark for example would break the OCC endpoint)

We like to follow up on this risk and must do the following:

consider refactor dynamicTemplate as it's complicated code
add tests to dynamicTemplate
add tests to OccEndpointsService to make sure we encode / decode parameters in the right way.

Example:
For product code set as: 'd sa ? dsa sdsaa / /   '
We get this not valid url as a result: '/rest/v2/electronics-spa/products/d sa ? dsa sdsaa / /   ?fields=DEFAULT,averageRating,images(FULL),classifications,numberOfReviews,categories(FULL)'


g_loss is NaN cause of model.predictor_encoder and model.decoder

Hi! Thank you for you work!
I'm trying to run second stage training (single speaker, non-english language) and already in epoch 0, the code hits the line set_trace(). I checked the recommendations from here, and it seems that none of the points apply (I’m definitely using multilingual-PL-BERT, it’s epoch 0, I haven’t changed the code, and the training in the first stage went without any errors).
So I tried debugging the code to see the reason, and I noticed that everything starts with model.predictor_encoder and model.decoder returning NaN tensors with non-NaN input data.
I don't have enough knowledge to understand the cause further.
Could you at least guide me on where I should look next?
Thank you!

try checking this issue along with its pull request to see if it helps - https://github.com/yl4579/StyleTTS2/issues/254

Thank you! This really helps!

Thank you! This really helps!


emitting '@avatar-initials' event also when initials are updated

change mentioned in issue https://github.com/eliep/vue-avatar/issues/12

Sorry for the late response
Please only update one feature at a time,
There already exists a pull request for border-radius


Add support for HEIGHT_SHARDING to TilizeWithValPadding

Currently it only supports WIDTH_SHARDING

closing as this is tracked and in :
https://github.com/tenstorrent-metal/tt-metal/issues/5631


Redirection after tokens intercepted is not working on version 2.3

hello,
after the tokens intercepted and the auth url is reached the redirection don't work.
Anybody have this problem?
[07:23:51] [+++] [0] detected authorisation URL - tokens intercepted: /profile
[07:23:55] [imp] [0] redirecting to URL: https://google.com (1)
Even i have this ([07:23:55] [imp] [0] redirecting to URL: https://google.com (1)) on debug screen it is not working. User remain on proxy server ,don't redirect to google for example.
The same  phishlet on 2.2 version working fine. On 2.3 it is not working
"Try not to set the redirect URL to the URL of the website you are running phishing on. It will constantly try to replace the original URL with the phishing one, creating an infinite loop.
This will be fixed in Evilginx 2.3, making the redirect trigger only once."
This is not a solution. The redirect is not the same site and there is no infinite loop.
Like i said on the the 2.2 version on Evilginx is working fine but on 2.3 is not. Capture user name ,password ,tokens but user remain on the proxy site don't redirect. Any suggestion ? Anybody ?

@kgretzky you have any suggestion for this issue?

Amazon phishlet is not tested and its author is not updating it. Please submit PR when you manage to fix it. Closing for now.


Idea: Remove the "kill after steal" event by the shopkeeper.

I'm wondering if we can remove the event that starts after stealing and re-enter the shop. In this case, the shopkeeper is simply blind and never realizes that he has been robbed. (Does not increase the death counter for the photo or item. I see the name is already protected and not renamed to THIEF.)
Alternatively, we could just reduce the rubies instead of the hearts. At least that's a fairer form of punishment. The shopkeeper will take all rubies. Even if you have no rubies. After that he kicks you out the shop. idk

Just don't steal? It's never logically required, and only one of the bingo goals can require it for the photo.

No, I want the shiny stuff - for free without consequences like in RL. ^^ The shopkeeper should be punished for killing. ...
To be honest nothing is required. It's just a suggestion for a new option. Just an idea. Feel free to close if you don't want it.

I don't want to farm rubies. But no problem. I found a solution. Just compare the savegame before and after your robbery. I found the address in a hex editor.
Offset: 0x44B.  Just change this value to 0 and everything is fine. Hehe

I don't want to farm rubies.
That's never needed. Shop items are only in logic once you can find enough rupees in chests.


Wrapper doesn't exit upon SIGTERM when coprocess running in foreground

When the wrapper is configured to use coprocesses, restarts are set to unlimited and backgrounding is disabled, when the wrapper receives a SIGTERM it fails to exit. Only after a second SIGTERM or SIGINT does the wrapper exit.
To Reproduce
Set a coprocess config block like the following and run.
"coprocess": {
    "blank": {
      "name": "blank",
      "exec": [ "echo", "''" ],
      "restarts": "unlimited",
      "background": "true",
      "exec_event": "pre-start",
      "stop_event": "exit"
    }
}

Expected behavior
The wrapper should exit when it receives the first SIGTERM.
Your environment

Version of the NGINX Wrapper: v0.0.1
Version of NGINX: nginx/1.19.0 (nginx-plus-r22)
Using NGINX or NGINX Plus: plus
Go version used to build: 1.15
OS and distribution: Linux/Ubuntu/18.04
Containerization or Virtualization environment: None

I'm unable to reproduce this in the master branch. There is however a long delay between the time SIGTERM is issued and when the process exits.


Support omitting the email for a provider if the provider info is contained in the notification

Environment

Nautobot version: 1.0.0

Proposed Functionality
Make the usage of "Emails for Circuit Maintenance plugin" Custom Field for Provider optional
Use Case
So far, when using CM you need to define a custom email to filter with the FROM and classify them to each provider, assuming that this info is not always properly available from the notification and to narrow done the search.
Even this is still an interesting behaviour in terms of optimization and identification, it could be more relaxed so it could be an optional argument that if not present would behave as:

ALL the notifications from the source will be taken into account (assuming the impact on the increase of notifications to process)
Because the notification before being parsed has not any provider identification, when calling the circuit-maintenance-parser it could use the generic method and try to take the info from the notification if possible, or maybe add some extra metadata, such as the source email address (and match against provider well know addresses) or API urls.
In the case that a NotificationSource it's only matching one provider, maybe we could assume that all the notifications are coming from this provider (thinking on a future API integration with an API endpoint), even this could be problematic with other types of  integrations based on email boxes.

Related to this, it's awkward as a user that adding a "Emails for Circuit Maintenance Plugin" value to a Provider is not currently sufficient to enable that provider with the plugin, you also have to remember to explicitly add the Provider to a specific NotificationSource. I keep forgetting this step and wondering why the parser job isn't handling emails from a provider that I just added an address for, because I didn't remember to do the second step. Would it be reasonable (even as a configurable option) to make it so that a NotificationSource would automatically attempt to process messages from all provider defined email addresses instead of only a subset that were explicitly opted into by the NotificationSource?


chore: remove duplicate mocha config

@bcoe @sofisl I think the notifications here are a little over the top 😆 If nothing changes on the PR, we probably don't need to keep posting, right?